Closed Bug 1147905 Opened 10 years ago Closed 7 years ago

[e10s] page-mod-debugger-post/main.js causes unsafe CPOW usage warnings

Categories

(Add-on SDK Graveyard :: General, defect, P4)

defect

Tracking

(e10s+)

RESOLVED INCOMPLETE
Tracking Status
e10s + ---

People

(Reporter: mconley, Unassigned)

References

(Blocks 1 open bug)

Details

(Whiteboard: [unsafe-cpow-usage] triaged)

Mined from test logs In addon-sdk/source/test/addons/page-mod-debugger-post/main.js: https://hg.mozilla.org/projects/holly/file/50e197279ad5/addon-sdk/source/test/addons/page-mod-debugger-post/main.js#l95 https://hg.mozilla.org/projects/holly/file/50e197279ad5/addon-sdk/source/test/addons/page-mod-debugger-post/main.js#l96 mod = PageMod({ include: TAB_URL, attachTo: ['existing', 'top', 'frame'], contentScriptFile: data.url('script.js'), onAttach: function(mod) { ok(true, 'the page-mod was attached to ' + mod.tab.url); require('sdk/timers').setTimeout(function() { let debuggee = getMostRecentBrowserWindow().gBrowser.selectedBrowser.contentWindow.wrappedJSObject; <-- causes unsafe CPOW usage warning debuggee.runDebuggerStatement(); <-- causes unsafe CPOW usage warning ok(true, 'called runDebuggerStatement'); }, 500) } }); ok(true, 'PageMod was created');
tracking-e10s: m8+ → ---
Whiteboard: [unsafe-cpow-usage]
This is test only.
tracking-e10s: --- → +
we are going to check which tests pass - and then disable all that do not to detect breakages from this point on. correcting the test suite issue across the board is a months long effort - that doesn't align with 2017 plans.
Priority: P1 → P4
Whiteboard: [unsafe-cpow-usage] → [unsafe-cpow-usage] triaged
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → INCOMPLETE
You need to log in before you can comment on or make changes to this bug.