Closed
Bug 666566
Opened 13 years ago
Closed 13 years ago
newWindowWithTabView() should wait for browser-delayed-startup-finished
Categories
(Firefox Graveyard :: Panorama, defect)
Firefox Graveyard
Panorama
Tracking
(Not tracked)
RESOLVED
FIXED
Firefox 8
People
(Reporter: ttaubert, Assigned: ttaubert)
Details
Attachments
(1 file, 2 obsolete files)
947 bytes,
patch
|
dietrich
:
review+
|
Details | Diff | Splinter Review |
newWindowWithTabView() from head.js should wait for browser-delayed-startup-finished because that is fired after TabView.init() is called.
Assignee | ||
Comment 1•13 years ago
|
||
Attachment #541361 -
Flags: feedback?(raymond)
Comment 2•13 years ago
|
||
Comment on attachment 541361 [details] [diff] [review] patch v1 Looks good!
Attachment #541361 -
Flags: feedback?(raymond) → feedback+
Assignee | ||
Updated•13 years ago
|
Attachment #541361 -
Flags: review?(dietrich)
Comment 3•13 years ago
|
||
Comment on attachment 541361 [details] [diff] [review] patch v1 Review of attachment 541361 [details] [diff] [review]: -----------------------------------------------------------------
Attachment #541361 -
Flags: review?(dietrich) → review+
Assignee | ||
Comment 4•13 years ago
|
||
http://hg.mozilla.org/integration/mozilla-inbound/rev/5abf332b2c45
Whiteboard: [inbound]
Comment 5•13 years ago
|
||
backed out from mozilla-inbound because part of a push that increased number of random failures in Panorama browser-chrome tests. Please reland in smaller chunks when ready.
Whiteboard: [inbound]
Assignee | ||
Comment 6•13 years ago
|
||
Patch v1 bounced because we need to add the listener for browser-delayed-startup-finished even before the browser's onLoad event is dispatched because that's sometimes too late (so that the delayed startup was already run and we're waiting for that to occur).
Attachment #541361 -
Attachment is obsolete: true
Attachment #542443 -
Flags: review?(dietrich)
Assignee | ||
Comment 7•13 years ago
|
||
Unrotted.
Attachment #542443 -
Attachment is obsolete: true
Attachment #544427 -
Flags: review?(dietrich)
Attachment #542443 -
Flags: review?(dietrich)
Assignee | ||
Comment 8•13 years ago
|
||
Comment on attachment 544427 [details] [diff] [review] patch v3 Passed try: http://tbpl.mozilla.org/?tree=Try&pusher=tim.taubert@gmx.de&rev=65f1bd3eb710
Comment 9•13 years ago
|
||
Comment on attachment 544427 [details] [diff] [review] patch v3 Review of attachment 544427 [details] [diff] [review]: -----------------------------------------------------------------
Attachment #544427 -
Flags: review?(dietrich) → review+
Assignee | ||
Comment 10•13 years ago
|
||
http://hg.mozilla.org/integration/fx-team/rev/4ea2cae8e2dc
Whiteboard: [fixed-in-fx-team]
Assignee | ||
Comment 11•13 years ago
|
||
http://hg.mozilla.org/mozilla-central/rev/4ea2cae8e2dc
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Whiteboard: [fixed-in-fx-team]
Assignee | ||
Updated•13 years ago
|
Target Milestone: --- → Firefox 8
Updated•8 years ago
|
Product: Firefox → Firefox Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•