1484796 - Add console logging to site patches via Content Scripts

Status: RESOLVED FIXED

(Web Compatibility :: Interventions, enhancement, P1)

Description

[Dennis Schubert [:denschub]]

We should add console logging to site patches via content scripts for two reasons:

1. We log something into the console when we alter the User Agent, so that people are aware we did something. However, for content scripts, there is no such thing. We should log something, so web developers are aware that we altered their site in a way.
2. In bug 1484324, we broke Fenenc Beta builds because two site patches share exactly the same code. While we could use the same JS file in both cases, we decided to have one file per webcompat bug, so that managing those is easier. When adding a console logging statement, those files no longer match, so we unbreak the build with that.

Comment 1

[Dennis Schubert [:denschub]]

Attachment: Bug 1484796 - Add console logging statements to site patches. r=kmag,rhelmer

Comment 2

[Kris Maglione [:kmag]]

Comment on attachment 9002561 [details]
Bug 1484796 - Add console logging statements to site patches. r=kmag,rhelmer

Kris Maglione [:kmag] has approved the revision.

Comment 3

[Robert Helmer [:rhelmer]]

Comment on attachment 9002561 [details]
Bug 1484796 - Add console logging statements to site patches. r=kmag,rhelmer

Robert Helmer [:rhelmer] has approved the revision.

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/9bf437ef1af2
Add console logging statements to site patches. r=kmag,rhelmer

Comment 5

[Andreea Pavel [:apavel]]

https://hg.mozilla.org/mozilla-central/rev/9bf437ef1af2