Closed
Bug 1058573
Opened 10 years ago
Closed 10 years ago
Add automated test for showing the toolbars dropdown in customization mode (even if the only toolbar available is the bookmarks toolbar)
Categories
(Firefox :: Toolbars and Customization, defect)
Firefox
Toolbars and Customization
Tracking
()
RESOLVED
FIXED
Firefox 35
People
(Reporter: mihaelav, Assigned: mihaelav)
References
Details
Attachments
(1 file, 1 obsolete file)
2.27 KB,
patch
|
jaws
:
review+
|
Details | Diff | Splinter Review |
Automated test for bug 1000377: Add automated test for to check that the toggleable toolbars dropdown is shown in customization mode even if the only toolbar available is the bookmarks toolbar
Comment 2•10 years ago
|
||
Comment on attachment 8480386 [details] [diff] [review] v1 Review of attachment 8480386 [details] [diff] [review]: ----------------------------------------------------------------- ::: browser/components/customizableui/test/browser_1058573_showToolbarsDropdown.js @@ +14,5 @@ > + yield removeCustomToolbars(); > + > + tab = gBrowser.selectedTab; > + yield promiseTabLoadEvent(tab, "about:customizing"); > + info("about:customizing was opened"); You can replace these three lines with: yield startCustomizing(); @@ +25,5 @@ > + > +add_task(function asyncCleanup() { > + // restore the initial location > + gBrowser.addTab(initialLocation); > + gBrowser.removeTab(tab); These can be replaced with: yield endCusotmizing(); which will make the `let tab = null; let initialLocation = gBrowser.currentURI.spec;` at the top of this file unneeded.
Attachment #8480386 -
Flags: review?(jaws) → feedback+
Assignee | ||
Comment 3•10 years ago
|
||
Updated based on review
Attachment #8480386 -
Attachment is obsolete: true
Attachment #8480430 -
Flags: review?(jaws)
Updated•10 years ago
|
Attachment #8480430 -
Flags: review?(jaws) → review+
Comment 5•10 years ago
|
||
https://hg.mozilla.org/integration/fx-team/rev/7c28c12de7aa
Keywords: checkin-needed
Whiteboard: [fixed-in-fx-team]
Comment 6•10 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/7c28c12de7aa
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Whiteboard: [fixed-in-fx-team]
Target Milestone: --- → Firefox 35
Updated•10 years ago
|
Flags: qe-verify-
You need to log in
before you can comment on or make changes to this bug.
Description
•