Bug 1359137 - fix bookmarks menu context test to not rely on AREA_PANEL, r?mikedeboer draft
authorGijs Kruitbosch <gijskruitbosch@gmail.com>
Wed, 31 May 2017 14:21:27 +0100
changeset 588985 15ce2c7a579bde900080d40ebc560a2c90355fa7
parent 588984 b125d74b48669372c32c7046c6ac9f8ac889e8c0
child 631729 ba535391b9224049f25b3fac2c75d6a986cc4c86
push id62208
push usergijskruitbosch@gmail.com
push dateMon, 05 Jun 2017 10:41:48 +0000
reviewersmikedeboer
bugs1359137
milestone55.0a1
Bug 1359137 - fix bookmarks menu context test to not rely on AREA_PANEL, r?mikedeboer MozReview-Commit-ID: Dy0IzQKVazu
browser/components/places/tests/browser/browser_toolbarbutton_menu_context.js
--- a/browser/components/places/tests/browser/browser_toolbarbutton_menu_context.js
+++ b/browser/components/places/tests/browser/browser_toolbarbutton_menu_context.js
@@ -4,17 +4,19 @@ var BMB_showAllBookmarks = document.getE
 var contextMenu = document.getElementById("placesContext");
 var newBookmarkItem = document.getElementById("placesContext_new:bookmark");
 
 waitForExplicitFinish();
 add_task(async function testPopup() {
   info("Checking popup context menu before moving the bookmarks button");
   await checkPopupContextMenu();
   let pos = CustomizableUI.getPlacementOfWidget("bookmarks-menu-button").position;
-  CustomizableUI.addWidgetToArea("bookmarks-menu-button", CustomizableUI.AREA_PANEL);
+  let target = gPhotonStructure ? CustomizableUI.AREA_FIXED_OVERFLOW_PANEL
+                                : CustomizableUI.AREA_PANEL;
+  CustomizableUI.addWidgetToArea("bookmarks-menu-button", target);
   CustomizableUI.addWidgetToArea("bookmarks-menu-button", CustomizableUI.AREA_NAVBAR, pos);
   info("Checking popup context menu after moving the bookmarks button");
   await checkPopupContextMenu();
 });
 
 async function checkPopupContextMenu() {
   let dropmarker = document.getAnonymousElementByAttribute(bookmarksMenuButton, "anonid", "dropmarker");
   BMB_menuPopup.setAttribute("style", "transition: none;");