Closed Bug 651522 Opened 11 years ago Closed 9 years ago

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

Categories

(Firefox for Android Graveyard :: General, defect)

ARM
Android
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED WONTFIX

People

(Reporter: philor, Unassigned)

References

Details

(Keywords: testcase-wanted, Whiteboard: [test disabled on Android])

Attachments

(1 file)

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.
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
Whiteboard: [orange]
It's permaorange, and only one of multiple permaoranges. We don't really need people misstarring all their try permaoranges as just this one.
Whiteboard: [orange]
Assignee: nobody → ben
Attachment #531387 - Flags: review?(mark.finkle) → review+
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
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!
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.
Disabled on Android in http://hg.mozilla.org/mozilla-central/rev/d0583d604249
Whiteboard: [test disabled on Android]
Assignee: ben → nobody
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.