Closed Bug 1149162 Opened 10 years ago Closed 8 years ago

[e10s] test-tab.js causes unsafe CPOW usage warnings

Categories

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

defect

Tracking

(e10s+, firefox50 fixed, firefox51 fixed, firefox52 fixed)

RESOLVED FIXED
mozilla52
Tracking Status
e10s + ---
firefox50 --- fixed
firefox51 --- fixed
firefox52 --- fixed

People

(Reporter: mconley, Assigned: rpl)

References

(Blocks 1 open bug)

Details

(Whiteboard: [unsafe-cpow-usage])

Attachments

(1 file)

Mined from test logs In addon-sdk/source/test/tabs/test-tab.js: https://hg.mozilla.org/projects/holly/file/50e197279ad5/addon-sdk/source/test/test-tab.js#l66 https://hg.mozilla.org/projects/holly/file/50e197279ad5/addon-sdk/source/test/test-tab.js#l71 https://hg.mozilla.org/projects/holly/file/50e197279ad5/addon-sdk/source/test/test-tab.js#l72 https://hg.mozilla.org/projects/holly/file/50e197279ad5/addon-sdk/source/test/test-tab.js#l77 https://hg.mozilla.org/projects/holly/file/50e197279ad5/addon-sdk/source/test/test-tab.js#l78 https://hg.mozilla.org/projects/holly/file/50e197279ad5/addon-sdk/source/test/test-tab.js#l83 function step2(url, assert) { tabs.open({ url: url, onReady: function(tab) { primaryTab = tab; let window = windowUtils.activeBrowserWindow.content; let matchedTab = getTabForWindow(window); assert.equal(matchedTab, tab, "We are able to find the tab with his content window object"); let timer = require("sdk/timers"); function waitForFrames() { let iframe = window.document.getElementById("iframe"); <-- causes unsafe CPOW usage warning if (!iframe) { timer.setTimeout(waitForFrames, 100); return; } iframeWin = iframe.contentWindow; <-- causes unsafe CPOW usage warning let subIframe = iframeWin.document.getElementById("sub-iframe"); <-- causes unsafe CPOW usage warning if (!subIframe) { timer.setTimeout(waitForFrames, 100); return; } let subIframeWin = subIframe.contentWindow; <-- causes unsafe CPOW usage warning let subSubIframe = subIframeWin.document.getElementById("sub-sub-iframe"); <-- causes unsafe CPOW usage warning if (!subSubIframe) { timer.setTimeout(waitForFrames, 100); return; } let subSubIframeWin = subSubIframe.contentWindow; <-- causes unsafe CPOW usage warning matchedTab = getTabForWindow(iframeWin); assert.equal(matchedTab, tab, "We are able to find the tab with an iframe window object"); matchedTab = getTabForWindow(subIframeWin); assert.equal(matchedTab, tab, "We are able to find the tab with a sub-iframe window object"); matchedTab = getTabForWindow(subSubIframeWin); assert.equal(matchedTab, tab, "We are able to find the tab with a sub-sub-iframe window object"); // Put our primary tab in the background and test again. // The onActivate listener will take us to step3. auxTab.activate(); } waitForFrames(); } }); }
tracking-e10s: m8+ → ---
Whiteboard: [unsafe-cpow-usage]
This is test only.
tracking-e10s: --- → +
Luca has fix believes
Flags: needinfo?(lgreco)
Attached patch bug1149162-test-tab-cpow.patch (deleted) — Splinter Review
The cpows usage described in Comment 0 has been fixed by Bug 1171515 (which has been merge into mozilla-central as part of Bug 1179013). Nevertheless, in the meantime this test file has got a new single cpow usage which has been added by Bug 1179013, which means that I added the only cpow currently used in this test file :-( This cpow usage can be removed by the tests by using the message manager to retrieve the current value of the document.readyState, without using the CPOW. I'm attaching the patch that remove the cpow usage from this test, using the strategy described above.
Flags: needinfo?(lgreco)
Attachment #8802166 - Flags: review?(dtownsend)
Attachment #8802166 - Flags: review?(dtownsend) → review+
Assignee: nobody → lgreco
Status: NEW → ASSIGNED
Pushed by ryanvm@gmail.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/3b4965bb11c0 Remove CPOW shim usage in SDK test-tab test related to tab.readyState. r=Mossop
Keywords: checkin-needed
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla52
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: