Our browser-plugins.js code tries to make plugin names less awkward by, among other things, stripping version numbers from the end. We need to avoid that for the openh264 case though, because we get "...h264" → "...h". We could just mark GMP crashes in the custom event code and then skip makeNicePluginName() for them in browser-plugins.
Moving this out of bug 1043531 because it doesn't block QA for that bug and hence shouldn't block it from landing ASAP.
Hi Georg, would you like this added to Iteration 34.1?
Yes, looking into it today.
Created attachment 8463912 [details] [diff] [review] Mark up GMP crash events
Attachment #8463912 - Flags: review?(rjesup)
Created attachment 8463913 [details] [diff] [review] Skip makeNicePluginname for GMP plugins
Attachment #8463913 - Flags: review?(ttaubert)
Attachment #8463913 - Flags: review?(ttaubert) → review+
Status: ASSIGNED → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 34
Hi Liz, can a QA contact be assigned for verification of this bug.
Assigning to Anthony since this blocks bug 1043531.
QA Contact: anthony.s.hughes
Comment on attachment 8463912 [details] [diff] [review] Mark up GMP crash events Approval Request Comment [Feature/regressing bug #]: OpenH264 crash reporting, frontend. [User impact if declined]: Cut-off plugin name (e.g. "gmpopenh" instead of "gmpopenh264"). [Describe test coverage new/current, TBPL]: Manually tested, automated tested pending other bugs. [Risks and why]: Low-risk, 2 small changes: one that just marks an event as GMP specific and another that skips processing the name if that marker is set. [String/UUID change made/needed]: None.
Attachment #8463912 - Flags: approval-mozilla-aurora?
Comment on attachment 8463913 [details] [diff] [review] Skip makeNicePluginname for GMP plugins Approval Request Comment See above.
Attachment #8463913 - Flags: approval-mozilla-aurora?
status-firefox33: --- → affected
status-firefox34: --- → fixed
Attachment #8463912 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Attachment #8463913 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
status-firefox33: affected → fixed
Hi Georg, Florin has asked for details on how to verify this bug. Can you provide him with that.
Steps will be up on bug 1043531 in a bit, this bug only fixes the name being cut off there.
Verified as fixed with Aurora 33.0a2 (Build ID: 20140731004002) and Nightly 34.0a1 (Build ID: 20140731030206) builds on Windows 7 x64, Mac OS X 10.9.4 and Ubuntu 13.04 64bit - the plugin name is no longer cut off in crash reporter notification.
Status: RESOLVED → VERIFIED
QA Whiteboard: [qa+] → [qa!]
status-firefox33: fixed → verified
status-firefox34: fixed → verified
You need to log in before you can comment on or make changes to this bug.