Closed
Bug 834961
Opened 12 years ago
Closed 12 years ago
pwpb update window util modules to ignore private windows
Categories
(Add-on SDK Graveyard :: General, defect, P1)
Add-on SDK Graveyard
General
Tracking
(Not tracked)
RESOLVED
FIXED
1.14
People
(Reporter: evold, Assigned: evold)
References
Details
Attachments
(1 file)
No description provided.
Assignee | ||
Updated•12 years ago
|
Target Milestone: --- → 1.14
Assignee | ||
Updated•12 years ago
|
Assignee: nobody → evold
Assignee | ||
Updated•12 years ago
|
Assignee | ||
Comment 1•12 years ago
|
||
Pointer to Github pull-request
Assignee | ||
Updated•12 years ago
|
Attachment #711202 -
Flags: review?(rFobic)
Priority: -- → P1
Updated•12 years ago
|
Attachment #711202 -
Flags: review?(rFobic) → review-
Updated•12 years ago
|
Blocks: sdk-pwpb-fx21
Assignee | ||
Comment 2•12 years ago
|
||
Comment on attachment 711202 [details]
Pointer to Github pull request: https://github.com/mozilla/addon-sdk/pull/769
ready for another round
Attachment #711202 -
Flags: review- → review?(rFobic)
Comment 3•12 years ago
|
||
Comment on attachment 711202 [details]
Pointer to Github pull request: https://github.com/mozilla/addon-sdk/pull/769
Please make sure to address following comments before landing:
https://github.com/mozilla/addon-sdk/pull/769/files#r3198016
https://github.com/mozilla/addon-sdk/pull/769/files#r3198004
I made few others but I'd be ok fixing them in followup changes.
Attachment #711202 -
Flags: review?(rFobic) → review+
Comment 4•12 years ago
|
||
Commits pushed to master at https://github.com/mozilla/addon-sdk
https://github.com/mozilla/addon-sdk/commit/a76b7064dd6da8e79e8d2984f2f18c4b682d1d73
Bug 834961 - pwpb update window modules to ignore private windows
https://github.com/mozilla/addon-sdk/commit/bb69f6051e88b24bb8993c2a5016b91f278efcda
Merge pull request #825 from erikvold/834961-2
Fix Bug 834961 - pwpb update window modules to ignore private windows r=@gozala
Updated•12 years ago
|
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Comment 5•12 years ago
|
||
Commits pushed to master at https://github.com/mozilla/addon-sdk
https://github.com/mozilla/addon-sdk/commit/f7200c0f7d14599b5f8ad510ae333663d9d21e94
Bug 834961: fixes for the private-browsing test addon
https://github.com/mozilla/addon-sdk/commit/3c87ada3cb75c13063f126fdc1ad4afc257b91e6
Merge pull request #827 from erikvold/834961-fix
Bug 834961: fixes for the private-browsing test add-on a=@Gozala
Looks like a bunch of private browsing tests are intermittently failing:
TEST-UNEXPECTED-FAIL | tests/test-windows.testWindowIteratorPrivateDefault | false != true
TEST-UNEXPECTED-FAIL | tests/test-window-utils-private-browsing.testSettingActiveWindowIgnoresPrivateWindow | Correct active window [1]
error: private-browsing-supported: fail: Correct active window [1]
Etc.
Timing issues, I guess?
Comment 7•12 years ago
|
||
Commits pushed to stabilization at https://github.com/mozilla/addon-sdk
https://github.com/mozilla/addon-sdk/commit/980baf5c64fbc2dd91e192efaf5305e3e44bc2ba
Merge pull request #825 from erikvold/834961-2
Fix Bug 834961 - pwpb update window modules to ignore private windows r=@gozala
(cherry picked from commit bb69f6051e88b24bb8993c2a5016b91f278efcda)
Conflicts:
lib/sdk/tabs/utils.js
test/private-browsing/windows.js
https://github.com/mozilla/addon-sdk/commit/960c4f425018ccfb84dfe787e1e9e2346d08fd02
Merge pull request #827 from erikvold/834961-fix
Bug 834961: fixes for the private-browsing test add-on a=@Gozala(cherry picked from commit 3c87ada3cb75c13063f126fdc1ad4afc257b91e6)
You need to log in
before you can comment on or make changes to this bug.
Description
•