680974 - Default "report this crash" bugzilla link for fennec crash reporting shows Fennec instead of Core

Status: VERIFIED FIXED

(Socorro :: Webapp, task)

Description

[Naoki Hirata :nhirata (please use needinfo instead of cc)]

1. go to a crash signature such as https://crash-stats.mozilla.com/report/index/12c72ee6-052e-47f7-8452-b75c52110818
2. click on report this crash

Expected: Most crashes occur in Core, I would expect Core or other something more drilled down than a default of Fennec

Actual: The default is Fennec for Fennec crashes

Comment 1

[Laura Thomson :laura]

We'd need some different rules about what crashes should go where in bugzilla by default.  How would we know which Fennec crashes were not Core?

Comment 2

[Robert Kaiser]

80% or more of our crashes are not specific to what's in Firefox or Fennec, but belong mostly in Core, Toolkit, or Plugins products on Bugzilla.

I wonder if we could get this to some way where we can select one of those few and report the bug in that one.


Also, while someone is looking at this, can you change the default summary from "crash <short-signature>" to "crash in <short-signature>"?

Comment 3

[Ludovic Hirlimann [:Usul]]

A good guess would be to parse the file where the crash occurs and have a directory -> component mapping. if it crashes in /mozilla/netwerk/ then it should go to necko. 
I'm sure that for some components (xpconnect xpcom) that wouldn't work and then we could fall back to a manual move from the person filing the bug.

Comment 4

[(not currently active) Ted Mielczarek]

That's really the only way to get even close to selecting the right component. We could probably build a big mapping from this list:
https://wiki.mozilla.org/Modules/All

but I don't know if that's worth the effort or not.

Comment 5

[Robert Kaiser]

As those who file bugs from that page are usually experience people who right now often mangle the new bug URL to get into the right component, I think we don't need to build up large logic for auto-selection of a "right" product or even component. A product selection dropdown should be enough, IMHO.

Comment 6

[Schalk Neethling [:espressive]]

Attachment: link to most common bugzilla products

link to most common bugzilla products

Comment 7

[Robert Kaiser]

(In reply to Schalk Neethling from comment #6)
> Created attachment 602012 [details]
> link to most common bugzilla products

Basically looks good, but the current one that has the product should be in there as well and be the first of the links.

Comment 8

[Schalk Neethling [:espressive]]

Attachment: added default for current product

Comment 9

[Robert Kaiser]

Comment on attachment 602051 [details]
added default for current product

Yes, I like that!

(though I guess the link could in this case be just on "Firefox", and the extra space after "Firefox" isn't needed *g*)

Comment 10

[Schalk Neethling [:espressive]]

Sent pull request :: https://github.com/mozilla/socorro/pull/405

TEST CASE
----------

Head over to report/index
The report this link should now have a link to report the bug for the current product, in core, plugins or toolkit

Comment 11

[Schalk Neethling [:espressive]]

Updates pull request :: https://github.com/mozilla/socorro/pull/415

Comment 12

[[github robot]]

Commit pushed to master at https://github.com/mozilla/socorro

https://github.com/mozilla/socorro/commit/74ef4a24ce2330b45a8e2b16f09f5059f3510b9d
Merge pull request #415 from ossreleasefeed/bug680974-improve-report-this-link

provide three additional options to submit crashes to core, plugins and toolkit fixes bug 680974

Comment 13

[Matt Brandt [:mbrandt]]

Attachment: qa - reopened

Two very minor nits- 

1. Suggestion, move the word "in" out of the <a href>. The only text that should be linked is "Firefox", "Core", "Plug-Ins", and "Toolkit"

2. Add a comma before "or" - 
Change: 
"Bugzilla - Report this bug in Firefox, Core, Plug-Ins or Toolkit"
To:
"Bugzilla - Report this bug in Firefox, Core, Plug-Ins, or Toolkit

Comment 14

[Schalk Neethling [:espressive]]

Sent pull request :: https://github.com/mozilla/socorro/pull/438 adds suggestions in https://bugzilla.mozilla.org/show_bug.cgi?id=680974#c13

Comment 15

[[github robot]]

Commit pushed to master at https://github.com/mozilla/socorro

https://github.com/mozilla/socorro/commit/76a149820427cc46ace1a6cf5e637521150fa7fd
Merge pull request #438 from ossreleasefeed/qa-suggest-on-bug-680974

made changes as suggested by qa fixes bug 680974

Comment 16

[[github robot]]

Commit pushed to stage at https://github.com/mozilla/socorro

https://github.com/mozilla/socorro/commit/da5658f7c1526b45368881a041ed217c3297f11f
made changes as suggested by qa fixes bug 680974

Comment 17

[Matt Brandt [:mbrandt]]

QA verified on dev and staging. Thx schalk