Closed Bug 781124 Opened 10 years ago Closed 9 years ago

Report CPU usage for hung Flash processes

Categories

(Core :: Plug-ins, defect)

17 Branch
All
Windows 7
defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla17

People

(Reporter: gfritzsche, Assigned: gfritzsche)

References

Details

Attachments

(1 file, 5 obsolete files)

Followup to bug 764660. Collect & report CPU usage for the Flash processes too when Flash hung.
Attachment #650152 - Flags: review?(benjamin)
Accidentally uploaded the wrong patch.
Attachment #650152 - Attachment is obsolete: true
Attachment #650152 - Flags: review?(benjamin)
Attachment #650156 - Flags: review?(benjamin)
Attachment #650156 - Flags: review?(benjamin) → review+
Added missing commit message.
Attachment #650156 - Attachment is obsolete: true
Note: this patch is based on bug 781133.
Keywords: checkin-needed
This is bitrotted. Please rebase.
Keywords: checkin-needed
(In reply to Ryan VanderMeulen from comment #5)
> This is bitrotted. Please rebase.

Nevermind me.
This was backed out due to causing Win64 build failures.
https://hg.mozilla.org/integration/mozilla-inbound/rev/40e59c86c2b3

https://tbpl.mozilla.org/php/getParsedLog.php?id=14237753&tree=Mozilla-Inbound

e:/builds/moz2_slave/m-in-w64/build/dom/plugins/ipc/PluginModuleParent.cpp(317) : error C2065: 'mFlashProcess1' : undeclared identifier

e:/builds/moz2_slave/m-in-w64/build/dom/plugins/ipc/PluginModuleParent.cpp(317) : error C2065: 'mFlashProcess1' : undeclared identifier

e:/builds/moz2_slave/m-in-w64/build/dom/plugins/ipc/PluginModuleParent.cpp(320) : error C2065: 'mFlashProcess2' : undeclared identifier

e:/builds/moz2_slave/m-in-w64/build/dom/plugins/ipc/PluginModuleParent.cpp(320) : error C2065: 'mFlashProcess2' : undeclared identifier
Fix missing "#ifdef MOZ_CRASHREPORTER_INJECTOR".

Pushed to try: https://tbpl.mozilla.org/?tree=Try&rev=86efe3cba06a
* Rebased on patch 781133, attachment 650475 [details] [diff] [review]
* Fixed remaining issues
* Try: https://tbpl.mozilla.org/?tree=Try&rev=16ff9fd62753

I wonder how i even got this through my local tests yesterday.
Attachment #650198 - Attachment is obsolete: true
Attachment #650447 - Attachment is obsolete: true
Comment on attachment 650496 [details] [diff] [review]
Add Flash process CPU usage reporting

Try build results are fine. Benjamin, can you do a follow review?
Attachment #650496 - Flags: review?(benjamin)
Attachment #650496 - Flags: review?(benjamin) → review+
Rebased on patch for bug 781133
Attachment #650496 - Attachment is obsolete: true
Checkin note: This is based on patch 653426 on bug 781133.
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/4c81cb568cb4
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla17
You need to log in before you can comment on or make changes to this bug.