bugzilla.mozilla.org has resumed normal operation. Attachments prior to 2014 will be unavailable for a few days. This is tracked in Bug 1475801.
Please report any other irregularities here.

Intermittent TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/browser/components/downloads/test/browser/browser_first_download_panel.js | Test timed out

NEW
Unassigned

Status

()

Firefox
Downloads Panel
P3
normal
5 years ago
2 years ago

People

(Reporter: Tomcat, Unassigned)

Tracking

({intermittent-failure})

Trunk
x86
Linux
intermittent-failure
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [test disabled on Linux][leave open], URL)

Attachments

(1 obsolete attachment)

(Reporter)

Description

5 years ago
Rev3 Fedora 12 fx-team debug test mochitest-browser-chrome on 2013-12-12 01:26:17 PST for push 53f6414f956a

slave: talos-r3-fed-030


https://tbpl.mozilla.org/php/getParsedLog.php?id=31864359&tree=Fx-Team

TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/browser/components/downloads/test/browser/browser_first_download_panel.js | Test timed out
TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/browser/components/downloads/test/browser/browser_overflow_anchor.js | Panel should be anchored to the chevron. - Got null, expected [object XULElement]
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Paolo, can you please take a look at what might have caused this very recent regression?
Flags: needinfo?(paolo.mozmail)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)

Comment 28

5 years ago
I don't think I'll be able to look into this before I have made more progress on the work I'm doing for the Login Manager. Maybe Mike or Gijs can help?
Flags: needinfo?(paolo.mozmail)
Flags: needinfo?(mdeboer)
Flags: needinfo?(gijskruitbosch+bugs)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)

Comment 31

5 years ago
Looking...
Assignee: nobody → gijskruitbosch+bugs
Flags: needinfo?(gijskruitbosch+bugs)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)

Comment 34

5 years ago
Can't reproduce locally on my VM. I'll try to make time to add some logging to the CustomizableUI bit that's erroring to get a full stacktrace. This isn't the only bug where that error appears and it seems like it's just generally causing problems. Not 100% sure why/how yet.

Comment 35

5 years ago
Created attachment 8348204 [details] [diff] [review]
Australis' overflowable toolbar might have race condition causing test failures,

Actually, as I was adding stuff here, the only thing that I can think of is that the overflowable toolbar's overriding getter somehow returns null. For that to happen, _list or _target should be null. For that to happen, the problem would have to be that they're not initialized. Which could theoretically (and apparently practically?) happen if you were to call inbetween load() and _delayedBrowserStartup. Hmm. Jared, does this change make sense to you? Just fetching elements earlier shouldn't really make any other differences here, but it might fix this issue. What do you think?
Attachment #8348204 - Flags: review?(jaws)

Updated

5 years ago
See Also: → bug 949357
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Flags: needinfo?(mdeboer)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment on attachment 8348204 [details] [diff] [review]
Australis' overflowable toolbar might have race condition causing test failures,

Review of attachment 8348204 [details] [diff] [review]:
-----------------------------------------------------------------

Worth a try, as long as using the customizationTarget getter doesn't force the binding to be constructed earlier than before.
Attachment #8348204 - Flags: review?(jaws) → review+
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)

Comment 87

5 years ago
(In reply to Jared Wein [:jaws] (Away 20 Dec to 2 Jan) from comment #40)
> Comment on attachment 8348204 [details] [diff] [review]
> Australis' overflowable toolbar might have race condition causing test
> failures,
> 
> Review of attachment 8348204 [details] [diff] [review]:
> -----------------------------------------------------------------
> 
> Worth a try, as long as using the customizationTarget getter doesn't force
> the binding to be constructed earlier than before.

The constructor is invoked from registerToolbarNode, which is invoked from the binding, so that shouldn't happen.

remote:   https://hg.mozilla.org/integration/fx-team/rev/b26925dd9232

(Leaving open to see if this fixes the orange)

Updated

5 years ago
Whiteboard: [leave open]
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)

Updated

5 years ago
Whiteboard: [leave open] → [leave open][fixed-in-fx-team]
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
https://hg.mozilla.org/mozilla-central/rev/b26925dd9232

FWIW, there have definitely been instances of this on fx-team after this landed there.
Whiteboard: [leave open][fixed-in-fx-team] → [leave open]
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
(In reply to :Gijs Kruitbosch (Unavailable Dec 19 - Jan 2) from comment #87)
> (Leaving open to see if this fixes the orange)

This is currently #3 on Orange Factor. Jared, can you please take a look since it appears that Gijs isn't around?
Flags: needinfo?(jaws)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Test disabled at jaws' request.
https://hg.mozilla.org/integration/fx-team/rev/2898275c1c43
Flags: needinfo?(jaws)
Whiteboard: [leave open] → [test disabled on Linux][leave open]
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)

Comment 144

5 years ago
(In reply to Ryan VanderMeulen [:RyanVM UTC-5] from comment #111)
> https://hg.mozilla.org/mozilla-central/rev/b26925dd9232
> 
> FWIW, there have definitely been instances of this on fx-team after this
> landed there.

Yeah, the patch fixed the error that was originally reported and might have fixed other random test failures, but seems unrelated to this actual test failure. It'd be helpful if I could actually reproduce the test failure itself locally, though. :-(
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)

Comment 161

5 years ago
Unassigning because if the error is unrelated I don't actually know what's causing this test to timeout, and as I can't reproduce locally...
Assignee: gijskruitbosch+bugs → nobody

Updated

5 years ago
Attachment #8348204 - Attachment is obsolete: true
Attachment #8348204 - Flags: checkin+

Comment 162

2 years ago
Bulk assigning P3 to all open intermittent bugs without a priority set in Firefox components per bug 1298978.
Priority: -- → P3
You need to log in before you can comment on or make changes to this bug.