Open
Bug 847971
Opened 10 years ago
Updated 4 months ago
contentAreaUtils's openURL should restrict its recent window search to windows of the same privacy status
Categories
(Toolkit :: General, defect)
Tracking
()
NEW
People
(Reporter: jdm, Unassigned)
References
(Depends on 1 open bug, Blocks 1 open bug)
Details
Attachments
(1 file)
8.61 KB,
patch
|
Dolske
:
review+
|
Details | Diff | Splinter Review |
No description provided.
Reporter | ||
Comment 1•10 years ago
|
||
Attachment #725122 -
Flags: review?(dolske)
Updated•10 years ago
|
Attachment #725122 -
Flags: review?(dolske) → review+
Reporter | ||
Updated•10 years ago
|
Assignee: nobody → josh
Reporter | ||
Comment 2•10 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/f85e650c1df7
Reporter | ||
Comment 3•10 years ago
|
||
Backout: https://hg.mozilla.org/integration/mozilla-inbound/rev/6c9fcfd9f4ce Bad interaction with other patches I pushed.
Reporter | ||
Comment 4•10 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/f029edccac09
Comment 5•10 years ago
|
||
(In reply to Josh Matthews [:jdm] from comment #4) > https://hg.mozilla.org/integration/mozilla-inbound/rev/f029edccac09 I backed this out; this patch makes openURL call RecentWindow.getMostRecentWindow, which doesn't exist. https://hg.mozilla.org/integration/mozilla-inbound/rev/6678ea192a6c https://hg.mozilla.org/integration/mozilla-inbound/rev/867475abb894
Comment 6•10 years ago
|
||
Which causes these timeouts: https://tbpl.mozilla.org/php/getParsedLog.php?id=20781373&tree=Mozilla-Inbound https://tbpl.mozilla.org/php/getParsedLog.php?id=20781208&tree=Mozilla-Inbound etc
Comment 7•10 years ago
|
||
Also assertions in the form of: https://tbpl.mozilla.org/php/getParsedLog.php?id=20780447&tree=Mozilla-Inbound https://tbpl.mozilla.org/php/getParsedLog.php?id=20781393&tree=Mozilla-Inbound
Comment 8•10 years ago
|
||
(In reply to Dão Gottwald [:dao] from comment #5) > I backed this out; this patch makes openURL call > RecentWindow.getMostRecentWindow, which doesn't exist. Was this patch simply never tested, or was this a snafu with renaming the function in another patch?
Comment 9•10 years ago
|
||
Earlier patches for bug 847952 added getMostRecentWindow, I assume this patch was built on top of that one.
Reporter | ||
Updated•4 years ago
|
Assignee: josh → nobody
Updated•4 months ago
|
Severity: normal → S3
You need to log in
before you can comment on or make changes to this bug.
Description
•