Enable browser_webconsole_cspro.js in the new console frontend

RESOLVED FIXED in Firefox 59

Status

enhancement
P1
normal
RESOLVED FIXED
2 years ago
11 months ago

People

(Reporter: nchevobbe, Assigned: nchevobbe)

Tracking

(Blocks 1 bug)

unspecified
Firefox 59
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [newconsole-mvp])

Attachments

(1 attachment)

No description provided.
Assignee

Updated

2 years ago
Blocks: 1400847
Priority: P3 → P2
Whiteboard: [newconsole-mvp]
Assignee

Updated

2 years ago
Assignee: nobody → nchevobbe
Status: NEW → ASSIGNED
Priority: P2 → P1
Comment on attachment 8935444 [details]
Bug 1408932 - Rename and enable browser_webconsole_cspro.js in new frontend; .

https://reviewboard.mozilla.org/r/206306/#review212330

Looks reasonable and is passing locally on my machine, just one nit comment.
R+
Honza

::: devtools/client/webconsole/new-console-output/test/mochitest/browser_webconsole_cspro.js:26
(Diff revision 1)
> -const TEST_VIOLATION = "http://example.com/browser/devtools/client/" +
> -                       "webconsole/test/test_bug_1010953_cspro.html";
> -const CSP_VIOLATION_MSG = "Content Security Policy: The page\u2019s settings " +
> -                          "blocked the loading of a resource at " +
> -                          "http://some.example.com/test.png " +
> -                          "(\u201cimg-src http://example.com\u201d).";
> +                       "new-console-output/test/mochitest/test-cspro.html";
> +const CSP_VIOLATION_MSG =
> +  "Content Security Policy: The page\u2019s settings blocked the loading of a resource " +
> +  "at http://some.example.com/cspro.png (\u201cimg-src http://example.com\u201d).";
> +const CSP_REPORT_MSG =
> +  "Content Security Policy: The page\u2019s settings observed the loading of a resource " +

nit: the line exceeds 90 chars limit
Attachment #8935444 - Flags: review?(odvarko) → review+
Comment hidden (mozreview-request)

Comment 4

2 years ago
Pushed by nchevobbe@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/150e5c84fc79
Rename and enable browser_webconsole_cspro.js in new frontend; r=Honza.

Comment 5

2 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/150e5c84fc79
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 59
Assignee

Updated

2 years ago
Blocks: 1428006

Updated

Last year
Product: Firefox → DevTools
Assignee

Updated

11 months ago
Duplicate of this bug: 1243967
You need to log in before you can comment on or make changes to this bug.