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

RESOLVED FIXED

Status

RESOLVED FIXED
8 years ago
3 years ago

People

(Reporter: cjones, Assigned: ttaubert)

Tracking

Details

(Whiteboard: [qa-])

Attachments

(1 attachment, 2 obsolete attachments)

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)

Updated

8 years ago
Assignee: nobody → tim.taubert
Status: NEW → ASSIGNED
(Assignee)

Comment 1

8 years ago
Created attachment 507547 [details] [diff] [review]
patch v1

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
(Assignee)

Comment 3

8 years ago
Created attachment 507968 [details] [diff] [review]
patch for checkin

(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
(Assignee)

Comment 4

8 years ago
Created attachment 507969 [details] [diff] [review]
patch for checkin

Forgot to add r=/a= info.
Attachment #507968 - Attachment is obsolete: true
(Assignee)

Updated

8 years ago
Keywords: checkin-needed

Comment 5

8 years ago
http://hg.mozilla.org/mozilla-central/rev/cb0f7b37208d
Status: ASSIGNED → RESOLVED
Last Resolved: 8 years ago
Keywords: checkin-needed
Resolution: --- → FIXED

Updated

8 years ago
Whiteboard: [qa-]
Product: Firefox → Firefox Graveyard
You need to log in before you can comment on or make changes to this bug.