Closed
Bug 395128
Opened 17 years ago
Closed 16 years ago
reduce nsBrowserGlue.js startup impact
Categories
(Firefox :: General, defect)
Firefox
General
Tracking
()
RESOLVED
FIXED
People
(Reporter: sayrer, Assigned: taras.mozilla)
References
Details
(Keywords: mobile, perf, Whiteboard: startup)
Attachments
(1 obsolete file)
three parts to this one:
1.) Use the preprocessor to strip comments
2.) Use XPCOMUtils to do the boilerplate, since the next thing to happen is FUEL, and that imports XPCOMUtils anyway.
3.) Stop using the subscript loader to load the sanitizer
|
Reporter | |
Comment 1•17 years ago
|
||
Attachment #279844 -
Flags: review?(gavin.sharp)
|
||
Comment 2•17 years ago
|
||
Comment on attachment 279844 [details] [diff] [review]
cover #1 and #2 to reduce source size
>Index: nsBrowserGlue.js
>- getService(Ci.nsIPlacesImportExportService);
>+ getService(Ci.nsIPlacesImportExpor tService);
r=me without this change! :)
Attachment #279844 -
Flags: review?(gavin.sharp) → review+
|
|
Reporter | |
Updated•17 years ago
|
|
|
Reporter | |
Comment 3•17 years ago
|
||
Comment on attachment 279844 [details] [diff] [review]
cover #1 and #2 to reduce source size
This patch landed, just need to do #3 now.
Attachment #279844 -
Attachment is obsolete: true
|
||
Updated•17 years ago
|
OS: SunOS → All
Hardware: PC → All
Version: unspecified → Trunk
|
||
Updated•16 years ago
|
Assignee: nobody → tglek
|
|
||
Comment 4•16 years ago
|
||
Looks like #3 is covered by bug 397719. Anything else left to do here?
|
|
Reporter | |
Comment 5•16 years ago
|
||
Guess not.
Status: NEW → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•