Closed
Bug 906662
Opened 9 years ago
Closed 9 years ago
Fix openAboutHomeTab to work with the changes to the tablet about:home UI
Categories
(Firefox for Android Graveyard :: General, defect, P1)
Tracking
(Not tracked)
RESOLVED
FIXED
Firefox 26
People
(Reporter: AdrianT, Assigned: AdrianT)
References
Details
(Whiteboard: [fixed-fig])
Attachments
(1 file, 1 obsolete file)
5.25 KB,
patch
|
Details | Diff | Splinter Review |
It seems that waitForEnabledText does not work anymore for this test since in the Tablet UI the tabs titles are now always enabled. I am adding a waitForAboutHomeTab method to wait for the tab to be changed after dragging left or right. This should fix the fails of testWebContentContextMenu.
Attachment #792189 -
Flags: review?(lucasr.at.mozilla)
Comment 1•9 years ago
|
||
Comment on attachment 792189 [details] [diff] [review] openAboutHomeTabFix.patch Review of attachment 792189 [details] [diff] [review]: ----------------------------------------------------------------- Good. ::: mobile/android/base/tests/AboutHomeTest.java.in @@ +307,5 @@ > } > } > > + // A wait in order for the about:home tab to be rendered after drag/tab selection > + private void waitForAboutHomeTab(final int tabNumber) { Rename to tabIndex. @@ +312,5 @@ > + boolean correctTab = waitForTest(new BooleanTest() { > + @Override > + public boolean test() { > + ViewPager pager = (ViewPager)mSolo.getView(ViewPager.class, 0); > + if (pager.getCurrentItem() == tabNumber) { Change that to simply: return (pager.getCurrentItem() == tabNumber) @@ +334,5 @@ > ViewPager pager = (ViewPager)mSolo.getView(ViewPager.class, 0); > switch (tab) { > case BOOKMARKS : { > + mSolo.clickOnText(BOOKMARKS_LABEL); > + waitForAboutHomeTab(aboutHomeTabs.indexOf(tab.toString())); tab.toString() -> BOOKMARKS_LABEL ?
Attachment #792189 -
Flags: review?(lucasr.at.mozilla) → review+
Assignee | ||
Comment 2•9 years ago
|
||
Made the requested changes
Attachment #792189 -
Attachment is obsolete: true
Comment 3•9 years ago
|
||
Pushed to try: https://tbpl.mozilla.org/?tree=Try&rev=115b5b1dd8d5
Updated•9 years ago
|
Blocks: new-about-home
Priority: -- → P1
Comment 4•9 years ago
|
||
The orangeness in the try build is caused by an intermittent failure in testWebContentContextMenu which has been backed out now. Pushed: https://hg.mozilla.org/projects/fig/rev/1d639fdd1f26
Whiteboard: [fixed-fig]
Comment 5•9 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/1d639fdd1f26
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 26
Updated•2 years ago
|
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•