Closed Bug 629233 Opened 12 years ago Closed 12 years ago

Intermittent "TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/browser/base/content/test/tabview/browser_tabview_bug608037.js | Test timed out" during OSX 10.6 opt Moth

Categories

(Firefox Graveyard :: Panorama, defect)

x86_64
macOS
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: cjones, Assigned: ttaubert)

References

Details

(Whiteboard: [qa-])

Attachments

(1 file, 2 obsolete files)

http://tinderbox.mozilla.org/showlog.cgi?log=Firefox/1296082252.1296083652.15832.gz
Rev3 MacOSX Snow Leopard 10.6.2 mozilla-central opt test mochitest-other on 2011/01/26 14:50:52 

s: talos-r3-snow-048
TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/browser/base/content/test/tabview/browser_tabview_bug608037.js | Test timed out
TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/browser/base/content/test/tabview/browser_tabview_bug608037.js | Found a tab after previous test timed out: http://mochi.test:8888/browser/browser/base/content/test/tabview/dummy_page.html
TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/browser/base/content/test/tabview/browser_tabview_bug608037.js | Found a tab after previous test timed out: http://mochi.test:8888/
TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/browser/base/content/test/tabview/browser_tabview_bug608158.js | Test timed out
TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/browser/base/content/test/tabview/browser_tabview_bug624265.js | Test timed out
TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/browser/base/content/test/tabview/browser_tabview_bug624265.js | Found a tab after previous test timed out: http://mochi.test:8888/
TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/browser/base/content/test/tabview/browser_tabview_bug624265.js | Found a tab after previous test timed out: http://mochi.test:8888/
Assignee: nobody → tim.taubert
Status: NEW → ASSIGNED
Attached patch patch v1 (obsolete) — Splinter Review
Removed duplicate functions that are now provided in head.js. Made the test more robust. Pushed to try.
Attachment #507547 - Flags: review?(ian)
Comment on attachment 507547 [details] [diff] [review]
patch v1

Looks good, except: 

>+function whenTabIsReconnected(tab, callback) {
>+  let tabItem = tab._tabViewTabItem;
>+
>+  if (tabItem._reconnected) {
>+    callback();
>+    return;
>+  }
>+
>+  tabItem.addSubscriber(tabTwo, "reconnected", function () {
>+    tabItem.removeSubscriber(tabTwo, "reconnected");
>+    callback();
>+  });
>+}

... shouldn't be using tabTwo here... tabItem should be fine. R+ with that fixed. A=test
Attachment #507547 - Flags: review?(ian) → review+
Blocks: 585689
Attached patch patch for checkin (obsolete) — Splinter Review
(In reply to comment #2)
> ... shouldn't be using tabTwo here... tabItem should be fine. R+ with that
> fixed. A=test

Oops :)
Attachment #507547 - Attachment is obsolete: true
Forgot to add r=/a= info.
Attachment #507968 - Attachment is obsolete: true
Keywords: checkin-needed
http://hg.mozilla.org/mozilla-central/rev/cb0f7b37208d
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Whiteboard: [qa-]
Product: Firefox → Firefox Graveyard
You need to log in before you can comment on or make changes to this bug.