Closed
Bug 663596
Opened 14 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)
Tracking
(Not tracked)
RESOLVED
WONTFIX
People
(Reporter: philor, Unassigned)
References
Details
(Keywords: regression)
Attachments
(1 file)
1.17 KB,
patch
|
mfinkle
:
review+
philor
:
checkin+
|
Details | Diff | Splinter Review |
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.
Reporter | ||
Comment 1•14 years ago
|
||
Makes us green on try.
Green.
I've never actually seen the letters "b-c" in green before.
Attachment #538702 -
Flags: review?(mark.finkle)
Updated•14 years ago
|
Attachment #538702 -
Flags: review?(mark.finkle) → review+
Reporter | ||
Comment 2•14 years ago
|
||
Comment on attachment 538702 [details] [diff] [review]
todo it
http://hg.mozilla.org/mozilla-central/rev/1a34672f4337
Attachment #538702 -
Flags: checkin+
Reporter | ||
Updated•11 years ago
|
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.
Description
•