Bug 1212658 - Remove needless IsCallerChrome check in nsMenuPopupFrame. r=bz
authorBobby Holley <bobbyholley@gmail.com>
Wed, 07 Oct 2015 16:24:01 -0700
changeset 266965 df9bd3a65292788d5b679ff5d8b1e092e538cabd
parent 266964 ecabb878492d482b3caac0b125df41e8a9278a39
child 266966 da767cf4ce869798fc9cf78b8809c9f709248a50
push id29504
push usercbook@mozilla.com
push dateFri, 09 Oct 2015 09:43:23 +0000
treeherdermozilla-central@d01dd42e654b [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersbz
bugs1212658
milestone44.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 1212658 - Remove needless IsCallerChrome check in nsMenuPopupFrame. r=bz
layout/xul/nsMenuPopupFrame.cpp
--- a/layout/xul/nsMenuPopupFrame.cpp
+++ b/layout/xul/nsMenuPopupFrame.cpp
@@ -1840,18 +1840,17 @@ nsMenuPopupFrame::ChangeMenuItem(nsMenuF
     if (aFromKey && IsOpen()) {
       nsIFrame* parentMenu = GetParent();
       if (parentMenu) {
         nsCOMPtr<nsIDOMXULMenuListElement> menulist = do_QueryInterface(parentMenu->GetContent());
         if (menulist) {
           // Fire a command event as the new item, but we don't want to close
           // the menu, blink it, or update any other state of the menuitem. The
           // command event will cause the item to be selected.
-          nsContentUtils::DispatchXULCommand(aMenuItem->GetContent(),
-                                             nsContentUtils::IsCallerChrome(),
+          nsContentUtils::DispatchXULCommand(aMenuItem->GetContent(), /* aTrusted = */ true,
                                              nullptr, PresContext()->PresShell(),
                                              false, false, false, false);
         }
       }
     }
 #endif
   }