Closed
Bug 1403899
Opened 7 years ago
Closed 7 years ago
Migrate browser_webconsole_block_mixedcontent_securityerrors.js to new frontend
Categories
(DevTools :: Console, enhancement, P3)
DevTools
Console
Tracking
(firefox57 wontfix, firefox58 wontfix, firefox59 fixed)
RESOLVED
FIXED
Firefox 59
People
(Reporter: nchevobbe, Assigned: nchevobbe)
References
Details
Attachments
(1 file)
No description provided.
Assignee | ||
Updated•7 years ago
|
status-firefox57:
--- → wontfix
Priority: -- → P3
Assignee | ||
Updated•7 years ago
|
Assignee: nobody → nchevobbe
Status: NEW → ASSIGNED
Comment hidden (mozreview-request) |
Comment 2•7 years ago
|
||
mozreview-review |
Comment on attachment 8927761 [details] Bug 1403899 - Enable browser_webconsole_block_mixedcontent_securityerrors.js in new console frontend; . https://reviewboard.mozilla.org/r/199038/#review204156 ::: devtools/client/webconsole/new-console-output/test/mochitest/browser_webconsole_block_mixedcontent_securityerrors.js:22 (Diff revision 1) > "use strict"; > > const TEST_URI = "https://example.com/browser/devtools/client/webconsole/" + > - "test/test-mixedcontent-securityerrors.html"; > -const LEARN_MORE_URI = "https://developer.mozilla.org/docs/Web/Security/" + > - "Mixed_content" + DOCS_GA_PARAMS; > + "new-console-output/test/mochitest/test-mixedcontent-securityerrors.html"; > +const LEARN_MORE_URI = > + "https://developer.mozilla.org/docs/Web/Security/Mixed_content" + DOCS_GA_PARAMS; Where is `DOCS_GA_PARAMS` defined?
Attachment #8927761 -
Flags: review?(odvarko) → review-
Assignee | ||
Comment 3•7 years ago
|
||
It comes from head.js. It is added in https://hg.mozilla.org/integration/autoland/diff/3dac33fd437c/devtools/client/webconsole/new-console-output/test/mochitest/head.js (on autoland at the moment, not landed yet on m-c)
Assignee | ||
Updated•7 years ago
|
Attachment #8927761 -
Flags: review- → review?(odvarko)
Comment 4•7 years ago
|
||
mozreview-review |
Comment on attachment 8927761 [details] Bug 1403899 - Enable browser_webconsole_block_mixedcontent_securityerrors.js in new console frontend; . https://reviewboard.mozilla.org/r/199038/#review204166 I see, r+ assuming try is green. Thanks Nicolas! Honza
Attachment #8927761 -
Flags: review?(odvarko) → review+
Pushed by nchevobbe@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/d720dddd8e4e Enable browser_webconsole_block_mixedcontent_securityerrors.js in new console frontend; r=Honza.
Comment 6•7 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/d720dddd8e4e
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
status-firefox59:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 59
Updated•6 years ago
|
Updated•6 years ago
|
Product: Firefox → DevTools
You need to log in
before you can comment on or make changes to this bug.
Description
•