Bug 1383365 - Fix APZ focus target log parameter order. r=botond
☠☠ backed out by 12f55e239e9d ☠ ☠
authorRyan Hunt <rhunt@eqrion.net>
Sun, 23 Jul 2017 13:58:19 -0400
changeset 370964 14d0e2a451e769dcd23f1a47f954b31231519dae
parent 370963 0989b2e5e0b9f091c8c68a321061508dbf49d094
child 370965 1db44c2e79032184d3fb96d34e84052aa73b603a
push id92988
push userrhunt@eqrion.net
push dateWed, 26 Jul 2017 07:58:08 +0000
treeherdermozilla-inbound@1db44c2e7903 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersbotond
bugs1383365
milestone56.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 1383365 - Fix APZ focus target log parameter order. r=botond MozReview-Commit-ID: DJtXaylyseg
gfx/layers/apz/src/FocusTarget.cpp
--- a/gfx/layers/apz/src/FocusTarget.cpp
+++ b/gfx/layers/apz/src/FocusTarget.cpp
@@ -161,18 +161,18 @@ FocusTarget::FocusTarget(nsIPresShell* a
 
   // The content to scroll is either the focused element or the focus node of
   // the selection. It's difficult to determine if an element is an interactive
   // element requiring async keyboard scrolling to be disabled. So we only
   // allow async key scrolling based on the selection, which doesn't have
   // this problem and is more common.
   if (focusedContent) {
     FT_LOG("Creating nil target with seq=%" PRIu64 ", kl=%d (disabling for focusing an element)\n",
-           mFocusHasKeyEventListeners,
-           aFocusSequenceNumber);
+           aFocusSequenceNumber,
+           mFocusHasKeyEventListeners);
 
     mType = FocusTarget::eNone;
     return;
   }
 
   nsCOMPtr<nsIContent> selectedContent = presShell->GetSelectedContentForScrolling();
 
   // Gather the scrollable frames that would be scrolled in each direction