Note: There are a few cases of duplicates in user autocompletion which are being worked on.

Intermittent TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/browser/base/content/test/tabview/browser_tabview_bug630102.js | application timed out

VERIFIED FIXED in Firefox 7

Status

Firefox Graveyard
Panorama
VERIFIED FIXED
6 years ago
a year ago

People

(Reporter: glandium, Assigned: ttaubert)

Tracking

Trunk
Firefox 7
All
Windows XP
Bug Flags:
in-testsuite +

Details

Attachments

(1 attachment, 1 obsolete attachment)

(Reporter)

Description

6 years ago
TEST-START | chrome://mochitests/content/browser/browser/base/content/test/tabview/browser_tabview_bug630102.js
TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/browser/base/content/test/tabview/browser_tabview_bug630102.js | application timed out after 330 seconds with no output
http://tinderbox.mozilla.org/showlog.cgi?log=Firefox/1304420117.1304421438.20710.gz

Updated

6 years ago
Component: Tabbed Browser → Panorama
QA Contact: tabbed.browser → panorama
(Assignee)

Updated

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

Comment 1

6 years ago
Created attachment 529800 [details] [diff] [review]
patch v1
Attachment #529800 - Flags: review?(ehsan)
So, what is the fix here?  I can't figure out which problem you're trying to fix here, and how...
(Assignee)

Comment 3

6 years ago
(In reply to comment #2)
> So, what is the fix here?  I can't figure out which problem you're trying to
> fix here, and how...

Sorry, I should have explained that a bit:

>  window.addEventListener("tabviewshown", partOne, false);
>  TabView.toggle();

That is the part that should start the test but doesn't if tabview is still or already shown. So I replaced all of these occurrences with the head.js functions showTabView() and hideTabView() that are more robust and just continue if the tabview is already shown/hidden.

The rest is just cleanup according to our current style of writing tests.
Attachment #529800 - Flags: review?(ehsan) → review+
(Assignee)

Comment 4

6 years ago
Comment on attachment 529800 [details] [diff] [review]
patch v1

Passed try:

http://tbpl.mozilla.org/?tree=Try&pusher=tim.taubert@gmx.de&rev=a69993664cb1
(Assignee)

Comment 5

6 years ago
Created attachment 529857 [details] [diff] [review]
patch for checkin
Attachment #529800 - Attachment is obsolete: true
(Assignee)

Updated

6 years ago
Keywords: checkin-needed
Keywords: checkin-needed
Whiteboard: [fixed in cedar]
Version: unspecified → Trunk

Comment 6

6 years ago
http://hg.mozilla.org/mozilla-central/rev/de21696c8853
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Flags: in-testsuite?
Resolution: --- → FIXED
Whiteboard: [fixed in cedar]
Target Milestone: --- → Firefox 6

Comment 7

6 years ago
Backed out from m-c on suspicion of causing browser/base/content/test/tabview/browser_tabview_bug597248.js test timeout permaorange.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
(Assignee)

Comment 8

6 years ago
http://hg.mozilla.org/integration/mozilla-inbound/rev/f5a456d99df9
Whiteboard: [inbound]
Target Milestone: Firefox 6 → ---

Comment 9

6 years ago
Backed out due to mochitest-other orange.
Whiteboard: [inbound]
(Assignee)

Comment 10

6 years ago
http://hg.mozilla.org/integration/mozilla-inbound/rev/0c47cb07f689
Whiteboard: [inbound]
http://hg.mozilla.org/mozilla-central/rev/0c47cb07f689
Status: REOPENED → RESOLVED
Last Resolved: 6 years ago6 years ago
Flags: in-testsuite? → in-testsuite+
Resolution: --- → FIXED
Whiteboard: [inbound]
Target Milestone: --- → Firefox 7
Verified fixed based on results in:
http://tinderbox.mozilla.org/showlog.cgi?log=Firefox/1314762663.1314767018.2727.gz&fulltext=1
Status: RESOLVED → VERIFIED
Product: Firefox → Firefox Graveyard
You need to log in before you can comment on or make changes to this bug.