Last Comment Bug 792105 - Permanent orange: TEST-UNEXPECTED-FAIL | test-content-tab.js | test-content-tab.js::test_content_tab_open | test-content-tab.js::test_content_tab_default_favicon
: Permanent orange: TEST-UNEXPECTED-FAIL | test-content-tab.js | test-content-t...
: intermittent-failure
Product: Thunderbird
Classification: Client Software
Component: Testing Infrastructure (show other bugs)
: Trunk
: x86 Linux
: -- normal (vote)
: Thunderbird 18.0
Assigned To: Mike Conley (:mconley) - (Needinfo me!)
Depends on:
Blocks: 741059
  Show dependency treegraph
Reported: 2012-09-18 10:32 PDT by Mike Conley (:mconley) - (Needinfo me!)
Modified: 2012-11-29 02:09 PST (History)
1 user (show)
See Also:
Crash Signature:
QA Whiteboard:
Iteration: ---
Points: ---

Patch v1 (1.05 KB, patch)
2012-09-18 11:55 PDT, Mike Conley (:mconley) - (Needinfo me!)
ehsan: review+
Details | Diff | Splinter Review

Description Mike Conley (:mconley) - (Needinfo me!) 2012-09-18 10:32:20 PDT
These just showed up on our Linux builders. Not sure if they affect Windows / OSX right now due to our tree being in flames on those platforms.

Test Failure: The tab [undefined undefined] should have a favicon with URL http://localhost:43336/whatsnew.png but instead has null
TEST-UNEXPECTED-FAIL | /home/cltbld/talos-slave/test/build/mozmill/content-tabs/test-content-tab.js | test-content-tab.js::test_content_tab_open

Test Failure: The tab [undefined undefined] should have a favicon with URL http://localhost:43336/favicon.ico but instead has null
TEST-UNEXPECTED-FAIL | /home/cltbld/talos-slave/test/build/mozmill/content-tabs/test-content-tab.js | test-content-tab.js::test_content_tab_default_favicon
Comment 1 Mike Conley (:mconley) - (Needinfo me!) 2012-09-18 11:55:50 PDT
Created attachment 662257 [details] [diff] [review]
Patch v1

Looks like bug 741059 changed an interface on us. No big deal.
Comment 2 Mike Conley (:mconley) - (Needinfo me!) 2012-09-18 11:58:30 PDT
Comment on attachment 662257 [details] [diff] [review]
Patch v1

rs=Standard8 via IRC.
Comment 3 :Ehsan Akhgari 2012-09-18 12:40:39 PDT
Comment on attachment 662257 [details] [diff] [review]
Patch v1

Review of attachment 662257 [details] [diff] [review]:

r=me if you need it!
Comment 4 Mark Banner (:standard8) (afk until 26th July) 2012-11-29 02:09:52 PST
This landed ages ago:

Note You need to log in before you can comment on or make changes to this bug.