Closed Bug 1246725 Opened 8 years ago Closed 8 years ago

Remove SEC_NORMAL from addon-sdk

Categories

(Core :: DOM: Security, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla49
Tracking Status
firefox49 --- fixed

People

(Reporter: ckerschb, Assigned: ckerschb)

References

Details

(Whiteboard: [domsecurity-active])

Attachments

(1 file)

      No description provided.
Assignee: nobody → mozilla
Blocks: 1193558
Status: NEW → ASSIGNED
(In reply to Carsten Book [:Tomcat] from comment #3)
> backed this out in
> https://treeherder.mozilla.org/#/jobs?repo=mozilla-
> inbound&revision=d00d189cd28b since test failures like
> https://treeherder.mozilla.org/logviewer.html#?job_id=22016049&repo=mozilla-
> inbound when this changes landed

Thanks Carsten, I'll have a look (not that important to land that conversion right now anyway). Potentially we should just wait will we have converted the URILoader to use asyncOpen2().
Flags: needinfo?(mozilla)
Blocks: 1182535
No longer blocks: 1193558
Whiteboard: [domsecurity-active]
Backed out for failing devtools test browser_dbg_addon-modules-unpacked.js

Backout: https://hg.mozilla.org/integration/mozilla-inbound/rev/93c99ed44d94
Push with failures: https://treeherder.mozilla.org/#/jobs?repo=mozilla-inbound&revision=96d28a77926955f1c986703223d49d7bfe4b3ed4
Failure log: https://treeherder.mozilla.org/logviewer.html#?job_id=27505489&repo=mozilla-inbound
08:25:53     INFO -  53 INFO TEST-PASS | devtools/client/debugger/test/mochitest/browser_dbg_addon-modules-unpacked.js | Enough 'Debugger:EditorSourceShown' panel events have been fired. -
08:25:53     INFO -  54 INFO Source shown: file:///tmp/tmpaSVETE.mozrunner/extensions/browser_dbg_addon5@tests.mozilla.org/bootstrap.js
08:25:53     INFO -  55 INFO TEST-PASS | devtools/client/debugger/test/mochitest/browser_dbg_addon-modules-unpacked.js | The correct source has been shown. -
08:25:53     INFO -  56 INFO TEST-PASS | devtools/client/debugger/test/mochitest/browser_dbg_addon-modules-unpacked.js | Saw the right toolbox title. -
08:25:53     INFO -  57 INFO Getting sources.
08:25:53     INFO -  58 INFO TEST-PASS | devtools/client/debugger/test/mochitest/browser_dbg_addon-modules-unpacked.js | retrieved sources -
08:25:53     INFO -  59 INFO TEST-UNEXPECTED-FAIL | devtools/client/debugger/test/mochitest/browser_dbg_addon-modules-unpacked.js | Add-on code should be the first group - Got chrome://browser_dbg_addon5, expected browser_dbg_addon5@tests.mozilla.org
08:25:53     INFO -  Stack trace:
08:25:53     INFO -      chrome://mochikit/content/browser-test.js:test_is:967
08:25:53     INFO -      chrome://mochitests/content/browser/devtools/client/debugger/test/mochitest/browser_dbg_addon-modules-unpacked.js:test/<:22
08:25:53     INFO -      test@chrome://mochitests/content/browser/devtools/client/debugger/test/mochitest/browser_dbg_addon-modules-unpacked.js:11:3
08:25:53     INFO -      Tester_execTest@chrome://mochikit/content/browser-test.js:810:9
08:25:53     INFO -      Tester.prototype.nextTest</<@chrome://mochikit/content/browser-test.js:711:7
08:25:53     INFO -      SimpleTest.waitForFocus/waitForFocusInner/focusedOrLoaded/<@chrome://mochikit/content/tests/SimpleTest/SimpleTest.js:741:59
Flags: needinfo?(ckerschb)
Thanks :aryx for backing this one out. I'll have a look what's going on.
Flags: needinfo?(ckerschb)
Those devtool tests are failing:
* devtools/client/debugger/test/mochitest/browser_dbg_addon-modules-unpacked.js
* devtools/client/debugger/test/mochitest/browser_dbg_addon-modules.js
* devtools/client/debugger/test/mochitest/browser_dbg_addon-sources.js
* devtools/client/debugger/test/mochitest/browser_dbg_addon-workers-dbg-enabled.js
Interestingly I get a different test result when using |ios.newChannelFromURI2| vs. NetUtil.newChannel() but using the same arguments within addon-sdk/source/lib/sdk/url.js. E.g. I would have to update browser_dbg_addon-modules-unpacked.js when using NetUtil.newChannel():

+++ b/devtools/client/debugger/test/mochitest/browser_dbg_addon-modules-unpacked.js
     // Check the inital list of sources is correct
     let groups = yield addonDebugger.getSourceGroups();
-    is(groups[0].name, "browser_dbg_addon5@tests.mozilla.org", "Add-on code should be the first group");
+    is(groups[0].name, "chrome://browser_dbg_addon5", "Add-on code should be the first group");
     is(groups[1].name, "chrome://global", "XUL code should be the second group");
-    is(groups.length, 2, "Should be only two groups.");
+    is(groups.length, 4, "Should be only four groups.");

I suppose those tests are just flaky, let's see if TRY would accept just replacing SEC_NORMAL with SEC_ALLOW_CROSS_ORIGIN_DATA_IS_NULL.
(In reply to Christoph Kerschbaumer [:ckerschb] from comment #11)
> https://treeherder.mozilla.org/#/jobs?repo=try&revision=ed75d318b7f3

Looking at (dt9) - all the failing tests from Comment 9 are passing.
https://hg.mozilla.org/mozilla-central/rev/ece69108baef
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla49
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: