Closed
Bug 727864
Opened 13 years ago
Closed 13 years ago
Enable DLL preloading when MozillaMaintenance service disabled prefetch
Categories
(Core :: Widget: Win32, defect)
Tracking
()
VERIFIED
FIXED
mozilla15
People
(Reporter: bbondy, Assigned: bbondy)
References
Details
(Whiteboard: [qa?])
Attachments
(1 file, 3 obsolete files)
6.49 KB,
patch
|
bbondy
:
review+
|
Details | Diff | Splinter Review |
We currently only do DLL preloading when we think that the Windows prefetch isn't used. We should also do DLL preloading when the maintenance service has performed a successful clear prefetch operation.
Assignee | ||
Comment 1•13 years ago
|
||
Attachment #597866 -
Flags: review?(jmathies)
Assignee | ||
Comment 2•13 years ago
|
||
Alfred Kayser 2012-02-16 23:59:42 PST
I would move the LOG statement of the failing WritePrefetchClearedReg function inside that function, just as the other LOG statement is also there.
+ if (retCode != ERROR_SUCCESS) {
+ LOG(("Could not open key. (%d)\n", retCode));
+ }
And, add a 'return false' here after the LOG.
Assignee | ||
Comment 3•13 years ago
|
||
Implemented review comments and rebased to m-c tip.
Attachment #597866 -
Attachment is obsolete: true
Attachment #598233 -
Flags: review?(jmathies)
Attachment #597866 -
Flags: review?(jmathies)
Comment 4•13 years ago
|
||
Comment on attachment 598233 [details] [diff] [review]
Patch v2.
Review of attachment 598233 [details] [diff] [review]:
-----------------------------------------------------------------
::: browser/app/nsBrowserApp.cpp
@@ +251,5 @@
> // in the program. Luckily 1 coincides with when prefetch is
> // enabled. If Windows prefetch didn't happen we can do our own
> // faster dll preloading.
> + // The MozillaMaintenance service issues a command to disable the
> + // pretch by replacing all found .pf files with 0 byte read only
nit - 'pretch'
::: toolkit/components/maintenanceservice/prefetch.cpp
@@ +54,5 @@
> + LONG retCode = RegOpenKeyExW(HKEY_LOCAL_MACHINE,
> + BASE_SERVICE_REG_KEY, 0,
> + KEY_WRITE | KEY_WOW64_64KEY, &baseKeyRaw);
> + if (retCode != ERROR_SUCCESS) {
> + LOG(("Could not open key. (%d)\n", retCode));
does this LOG macro do file name / line number logging? If not you might want to make that msg more specific.
Attachment #598233 -
Flags: review?(jmathies) → review+
Assignee | ||
Comment 5•13 years ago
|
||
Implemented nits, carrying forward r+.
Attachment #598233 -
Attachment is obsolete: true
Attachment #599602 -
Flags: superreview?(robert.bugzilla)
Attachment #599602 -
Flags: review+
Assignee | ||
Comment 6•13 years ago
|
||
Comment on attachment 599602 [details] [diff] [review]
Patch v3.
Actually I don't think this one needs a super-review so clearing that flag.
Attachment #599602 -
Flags: superreview?(robert.bugzilla)
Assignee | ||
Comment 7•13 years ago
|
||
Rebased to tip
Attachment #599602 -
Attachment is obsolete: true
Attachment #623704 -
Flags: review+
Assignee | ||
Comment 8•13 years ago
|
||
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Target Milestone: mozilla13 → mozilla15
Updated•13 years ago
|
Whiteboard: [qa+]
Comment 9•13 years ago
|
||
Is there anything to verify here from an end-user perspective?
Whiteboard: [qa+] → [qa?]
Assignee | ||
Comment 10•13 years ago
|
||
No this was verified by seeing telemetry data.
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•