Closed
Bug 1606375
Opened 5 years ago
Closed 5 years ago
Components loaded multiple times
Categories
(Calendar :: Internal Components, defect)
Calendar
Internal Components
Tracking
(Not tracked)
RESOLVED
FIXED
68.5.0
People
(Reporter: darktrojan, Assigned: darktrojan)
Details
Attachments
(1 file)
|
1.89 KB,
patch
|
Fallen
:
review+
Fallen
:
approval-calendar-esr+
|
Details | Diff | Splinter Review |
I made a mistake in bug 1590665, and discovered that all the files listed in calItemModule.js are read over and over, despite the attempt to prevent this happening by replacing NSGetFactory with its result. It seems the original function is used every time.
|
Assignee | |
Comment 1•5 years ago
|
||
Attachment #9118109 -
Flags: review?(philipp)
Attachment #9118109 -
Flags: approval-calendar-esr?(philipp)
|
||
Updated•5 years ago
|
Attachment #9118109 -
Flags: review?(philipp)
Attachment #9118109 -
Flags: review+
Attachment #9118109 -
Flags: approval-calendar-esr?(philipp)
Attachment #9118109 -
Flags: approval-calendar-esr+
|
|
Assignee | |
Updated•5 years ago
|
Keywords: checkin-needed-tb
Pushed by geoff@darktrojan.net:
https://hg.mozilla.org/comm-central/rev/6ff2528049d8
Prevent calendar components from being loaded multiple times; r=Fallen
|
|
Assignee | |
Updated•5 years ago
|
Target Milestone: --- → 74
|
|
Assignee | |
Comment 3•5 years ago
|
||
Comment on attachment 9118109 [details] [diff] [review]
1606375-components-once-1.diff
Clearly I did this at a time when it was expected on beta automatically.
Attachment #9118109 -
Flags: approval-calendar-beta?(philipp)
|
|
Assignee | |
Updated•5 years ago
|
Attachment #9118109 -
Flags: approval-calendar-beta?(philipp)
|
||
Comment 4•5 years ago
|
||
| bugherder uplift | ||
Thunderbird 68.5.0:
https://hg.mozilla.org/releases/comm-esr68/rev/a15573265e91
|
||
Updated•5 years ago
|
Target Milestone: 74 → 68.5.0
You need to log in
before you can comment on or make changes to this bug.
Description
•