Closed Bug 663596 Opened 13 years ago Closed 11 years ago

Permaorange on Android: browser_bookmarks.js | Opened the right bookmark - Got about:home, expected chrome://mochitests/content/browser/mobile/chrome/tests/browser_blank_01.html

Categories

(Firefox for Android Graveyard :: Bookmarks, defect)

ARM
Android
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED WONTFIX

People

(Reporter: philor, Unassigned)

References

Details

(Keywords: regression)

Attachments

(1 file)

Apparently a baffling regression from bug 662785 removing a default bookmark, unless I'm confused and it's actually even more bafflingly from bug 653141.

http://tinderbox.mozilla.org/showlog.cgi?log=Mozilla-Inbound/1307764853.1307765487.9617.gz

TEST-PASS | chrome://mochitests/content/browser/mobile/chrome/tests/browser_bookmarks.js | Found the bookmark
TEST-PASS | chrome://mochitests/content/browser/mobile/chrome/tests/browser_bookmarks.js | Bookmark has the right URL via attribute
TEST-PASS | chrome://mochitests/content/browser/mobile/chrome/tests/browser_bookmarks.js | Bookmark has the right URL via property
TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/mobile/chrome/tests/browser_bookmarks.js | Opened the right bookmark - Got about:home, expected chrome://mochitests/content/browser/mobile/chrome/tests/browser_blank_01.html

There's probably something subtle I can't see going on, because it sure looks like we open browser_blank_02.html, then open the bookmarks, find one with the right URL, make sure it's the right URL, make sure it's visible, click halfway down and halfway across it, and as a result get a PageShow for about:home instead of the URL we checked twice.
Attached patch todo itSplinter Review
Makes us green on try.

Green.

I've never actually seen the letters "b-c" in green before.
Attachment #538702 - Flags: review?(mark.finkle)
Attachment #538702 - Flags: review?(mark.finkle) → review+
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: