1493253 - Embed crash count table to bug pages

Status: RESOLVED FIXED

(bugzilla.mozilla.org :: Extensions, enhancement)

Description

[Kohei Yoshino [:kohei]]

This is a request from the Release Management team.

The goal is to embed a crash count data table, currently offered as a Firefox WebExtension [1], to bug pages where applicable, just like the Socorro crash graph, so no extension will be needed to see the numbers. It could be a big table, like Bug 1463048.

Given that some of the team members are still using the legacy UI, it has to be embedded into both modal and non-modal bug pages.

[1] https://addons.mozilla.org/firefox/addon/bugzilla-crash-stop/

Details of the extension:

https://github.com/mozilla/crash-stop-addon
https://crash-stop-addon.herokuapp.com/

Comment 1

[Kohei Yoshino [:kohei]]

I think all I have to do here is

* Include https://github.com/mozilla/crash-stop-addon/blob/master/webextension/content.js in the BMO extension
* Rewrite the script to encapsulate the functions in a class
* Add a CSP to allow relevant URLs including https://crash-stop-addon.herokuapp.com
* No template modifications

Comment 2

[Kohei Yoshino [:kohei]]

Attachment: GitHub Pull Request

Comment 3

[Calixte Denizet (:calixte)]

:kohei, is there a way to test it ?

Comment 4

[Kohei Yoshino [:kohei]]

So far you have to set up a local Vagrant VM as explained in the README, clone my PR, and file a test bug with crash signature(s). 
I just re-tested this on my own VM; can see the table with no CSP errors but the height of the <iframe> is wrong (only 25px) perhaps because my test bug isn't fulfilled with repo links and tracking flags. Will check with someone tomorrow if this could be tested on the public dev server.

https://github.com/mozilla-bteam/bmo/blob/master/README.rst

Comment 5

[Kohei Yoshino [:kohei]]

The change will be deployed to https://bugzilla-dev.allizom.org/ tomorrow for testing.

Comment 6

[Kohei Yoshino [:kohei]]

I have checked crash bugs on -dev randomly and confirmed crash tables are displayed properly.

e.g. https://bugzilla-dev.allizom.org/show_bug.cgi?id=1377387

Comment 7

[Kohei Yoshino [:kohei]]

My PR is ready to get merged. Let me know when the buildhub issue is solved.

Comment 8

[Calixte Denizet (:calixte)]

:kohei, unfortunately, buildhub is still lagging... so need to wait. So I'll ping you as soon as buildhub is fixed.

Comment 9

[Kohei Yoshino [:kohei]]

Merged to master as the buildhub issue has been solved.

Comment 10

[Kohei Yoshino [:kohei]]

The frame-src CSP is missing :(

Comment 11

[Kohei Yoshino [:kohei]]

Attachment: GitHub Pull Request - follow up

Comment 12

[Kohei Yoshino [:kohei]]

One more thing requested by calixte: bump the VERSION to 0.4.0 so the server can block queries from the Firefox extension.

Comment 13

[Kohei Yoshino [:kohei]]

Attachment: GitHub Pull Request - bump the version