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 422383 14d0e2a451e769dcd23f1a47f954b31231519dae
parent 422382 0989b2e5e0b9f091c8c68a321061508dbf49d094
child 422384 1db44c2e79032184d3fb96d34e84052aa73b603a
push id1517
push userjlorenzo@mozilla.com
push dateThu, 14 Sep 2017 16:50:54 +0000
treeherdermozilla-release@3b41fd564418 [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