651522 - permaorange: browser_bookmarks_star.js | All tags are added. - Got 0, expected 4 on Android

Status: RESOLVED WONTFIX

(Firefox for Android Graveyard :: General, defect)

Description

[Phil Ringnalda (:philor)]

TEST-START | chrome://mochitests/content/browser/mobile/chrome/browser_bookmarks_star.js
TEST-INFO | chrome://mochitests/content/browser/mobile/chrome/browser_bookmarks_star.js | Test appearance and behavior of the bookmark popup
TEST-PASS | chrome://mochitests/content/browser/mobile/chrome/browser_bookmarks_star.js | Bookmark popup should be hidden by clicking star
TEST-PASS | chrome://mochitests/content/browser/mobile/chrome/browser_bookmarks_star.js | Bookmark popup should be hidden by clicking in content
TEST-INFO | chrome://mochitests/content/browser/mobile/chrome/browser_bookmarks_star.js | Test adding tags via star icon
failed to validate file when downloading /mnt/sdcard/tests/mochitest.log!
TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/mobile/chrome/browser_bookmarks_star.js | All tags are added. - Got 0, expected 4


TEST-INFO | chrome://mochitests/content/browser/mobile/chrome/browser_bookmarks_star.js | Test editing uri via star icon
TEST-INFO | chrome://mochitests/content/browser/mobile/chrome/browser_bookmarks_star.js | Console message: [JavaScript Error: "scrollbox.firstChild is null" {file: "chrome://browser/content/bindings/dialog.xml" line: 49}]

No idea whether or not the start of the scrollbox.firstChild console messages in this test is significant: several later failures, they smell related, while in this one starts them, after the failure and without having a failure of its own after they start.

Comment 1

[Ed Morley [:emorley]]

http://hg.mozilla.org/try/rev/ed18eff9dd9a

s: tegra-075
TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/mobile/chrome/browser_bookmarks_star.js | All tags are added. - Got 0, expected 4
TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/mobile/chrome/browser_bookmarks_tags.js | All tags are associated with specified bookmark - Got 0, expected 4
TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/mobile/chrome/browser_bookmarks_tags.js | Test timed out
TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/mobile/chrome/browser_click_content.js | Test timed out
TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/mobile/chrome/browser_navigation.js | Test timed out
TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/mobile/chrome/browser_sidebars.js | Test timed out
TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/mobile/chrome/browser_tapping.js | Fired a good double tap
TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/mobile/chrome/browser_viewport.js | Test timed out

http://tinderbox.mozilla.org/showlog.cgi?log=Try/1304543717.1304544576.28490.gz

Comment 2

[Legacy TBPL/Treeherder Robot]

edmorley
http://tinderbox.mozilla.org/showlog.cgi?log=Try/1304644576.1304645824.28965.gz
Android Tegra 250 try opt test browser-chrome on 2011/05/05 18:16:16

s: tegra-068
TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/mobile/chrome/browser_bookmarks_star.js | All tags are added. - Got 0, expected 4
TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/mobile/chrome/browser_bookmarks_tags.js | All tags are associated with specified bookmark - Got 0, expected 4
TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/mobile/chrome/browser_bookmarks_tags.js | Test timed out
TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/mobile/chrome/browser_click_content.js | Test timed out
TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/mobile/chrome/browser_navigation.js | Test timed out
TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/mobile/chrome/browser_scrollbar.js | The vertical scrollbar should be visible - Got false, expected true
TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/mobile/chrome/browser_sidebars.js | Test timed out
TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/mobile/chrome/browser_tapping.js | Fired a good double tap
TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/mobile/chrome/browser_viewport.js | Test timed out

Comment 3

[Legacy TBPL/Treeherder Robot]

ewong
http://tinderbox.mozilla.org/showlog.cgi?log=Try/1304659576.1304660484.26917.gz
Android Tegra 250 try opt test browser-chrome on 2011/05/05 22:26:16

s: tegra-088
TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/mobile/chrome/browser_bookmarks_star.js | All tags are added. - Got 0, expected 4
TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/mobile/chrome/browser_bookmarks_tags.js | All tags are associated with specified bookmark - Got 0, expected 4
TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/mobile/chrome/browser_bookmarks_tags.js | Test timed out
TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/mobile/chrome/browser_click_content.js | Test timed out
TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/mobile/chrome/browser_navigation.js | Test timed out
TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/mobile/chrome/browser_scrollbar.js | The vertical scrollbar should be visible - Got false, expected true
TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/mobile/chrome/browser_sidebars.js | Test timed out
TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/mobile/chrome/browser_tapping.js | Fired a good double tap
TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/mobile/chrome/browser_viewport.js | Test timed out

Comment 4

[Legacy TBPL/Treeherder Robot]

mounir
http://tinderbox.mozilla.org/showlog.cgi?log=Try/1304612476.1304613605.26165.gz
Android Tegra 250 try opt test browser-chrome on 2011/05/05 09:21:16

s: tegra-068
TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/mobile/chrome/browser_bookmarks_star.js | All tags are added. - Got 0, expected 4
TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/mobile/chrome/browser_bookmarks_tags.js | All tags are associated with specified bookmark - Got 0, expected 4
TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/mobile/chrome/browser_bookmarks_tags.js | Test timed out
TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/mobile/chrome/browser_click_content.js | Test timed out
TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/mobile/chrome/browser_navigation.js | Test timed out
TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/mobile/chrome/browser_sidebars.js | Test timed out
TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/mobile/chrome/browser_tapping.js | Fired a good double tap
TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/mobile/chrome/browser_viewport.js | Test timed out

Comment 5

[Legacy TBPL/Treeherder Robot]

Callek
http://tinderbox.mozilla.org/showlog.cgi?log=Try/1304745076.1304745928.4484.gz
Android Tegra 250 try opt test browser-chrome on 2011/05/06 22:11:16

s: tegra-051
TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/mobile/chrome/tests/browser_bookmarks_star.js | All tags are added. - Got 0, expected 4
TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/mobile/chrome/tests/browser_bookmarks_tags.js | All tags are associated with specified bookmark - Got 0, expected 4
TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/mobile/chrome/tests/browser_bookmarks_tags.js | Test timed out
TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/mobile/chrome/tests/browser_click_content.js | Test timed out
TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/mobile/chrome/tests/browser_navigation.js | Test timed out
TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/mobile/chrome/tests/browser_scrollbar.js | The vertical scrollbar should be visible - Got false, expected true
TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/mobile/chrome/tests/browser_sidebars.js | Test timed out
TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/mobile/chrome/tests/browser_tapping.js | Fired a good double tap
TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/mobile/chrome/tests/browser_viewport.js | Test timed out

Comment 6

[Phil Ringnalda (:philor)]

It's permaorange, and only one of multiple permaoranges. We don't really need people misstarring all their try permaoranges as just this one.

Comment 7

[Benjamin Stover (:stechz)]

Attachment: permaorange: Cleanup and add some temporary logging

Comment 8

[Benjamin Stover (:stechz)]

http://hg.mozilla.org/mozilla-central/rev/718df02869b9

Comment 9

[Benjamin Stover (:stechz)]

Ack, logging is asynchronous because mochitest uses a console listener to report things. This part of the test is indeed synchronous so I am removing the waitFor check now.

Using try from here on out: http://tbpl.mozilla.org/?tree=Try&rev=454bc176d693

Comment 10

[Benjamin Stover (:stechz)]

So far everything looks correct. Right URI, right tags. It seems to me like the API is failing us:

http://tinderbox.mozilla.org/showlog.cgi?log=Try/1305106590.1305111239.12340.gz&fulltext=1#err0

Time to dig deeper!

Comment 11

[Benjamin Stover (:stechz)]

More logging added: http://tbpl.mozilla.org/?tree=Try&rev=c0bc667481ef

There is an observer involved but it seems to be called synchronously when the bookmark is inserted. I've added a lot of logging to nsTaggingService.js which should hopefully illuminate the problem.

Comment 12

[Phil Ringnalda (:philor)]

Disabled on Android in http://hg.mozilla.org/mozilla-central/rev/d0583d604249