Closed
Bug 1199246
Opened 10 years ago
Closed 10 years ago
WARNING: content window passed to PrivateBrowsingUtils.isWindowPrivate when running wpt tests
Categories
(Testing :: web-platform-tests, defect)
Testing
web-platform-tests
Tracking
(firefox43 affected)
RESOLVED
DUPLICATE
of bug 1193527
| Tracking | Status | |
|---|---|---|
| firefox43 | --- | affected |
People
(Reporter: ehsan.akhgari, Assigned: ehsan.akhgari)
Details
Attachments
(1 file)
We get this warning once for every page we load:
0:18.59 PROCESS_OUTPUT: ProcessReader (pid:37373) "WARNING: content window passed to PrivateBrowsingUtils.isWindowPrivate. Use isContentWindowPrivate instead (but only for frame scripts)."
0:18.59 PROCESS_OUTPUT: ProcessReader (pid:37373) "pbu_isWindowPrivate@resource://gre/modules/PrivateBrowsingUtils.jsm:25:14"
0:18.59 PROCESS_OUTPUT: ProcessReader (pid:37373) "nsBrowserAccess.prototype.openURI@chrome://browser/content/browser.js:15664:21"
0:18.59 PROCESS_OUTPUT: ProcessReader (pid:37373) "__marionetteFunc@dummy file:27:30"
0:18.59 PROCESS_OUTPUT: ProcessReader (pid:37373) "@dummy file:36:3"
0:18.59 PROCESS_OUTPUT: ProcessReader (pid:37373) "executeWithCallback@chrome://marionette/content/listener.js:791:5"
0:18.59 PROCESS_OUTPUT: ProcessReader (pid:37373) "executeAsyncScript@chrome://marionette/content/listener.js:665:3"
| Assignee | ||
Comment 1•10 years ago
|
||
Attachment #8653522 -
Flags: review?(wmccloskey)
Sorry, we both fixed this at roughly the same time.
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → DUPLICATE
Comment 3•10 years ago
|
||
Comment on attachment 8653522 [details] [diff] [review]
Fix a warning when passing a content window as the opener argument to nsBrowserAccess.openURI()
Review of attachment 8653522 [details] [diff] [review]:
-----------------------------------------------------------------
so do we not need this patch anymore?
| Assignee | ||
Comment 4•10 years ago
|
||
Comment on attachment 8653522 [details] [diff] [review]
Fix a warning when passing a content window as the opener argument to nsBrowserAccess.openURI()
(In reply to Brad Lassey [:blassey] (use needinfo?) from comment #3)
> so do we not need this patch anymore?
No.
Attachment #8653522 -
Flags: review?(wmccloskey)
You need to log in
before you can comment on or make changes to this bug.
Description
•