Bug 1300532 - Ensure old and new URL are ordered correctly to show prompt when entering ReaderView r?sebastian draft
authorAndrzej Hunt <ahunt@mozilla.com>
Tue, 06 Sep 2016 15:52:09 -0700
changeset 410818 275f9c28b09f0bfc82f8b207aa94fc946f7882e2
parent 410812 c2f0b1cad434c50698c70dac3a184cadb726be1e
child 410819 43429e506eb979be6d88939640f57b5175a0a251
push id28765
push userahunt@mozilla.com
push dateTue, 06 Sep 2016 22:56:40 +0000
reviewerssebastian
bugs1300532
milestone51.0a1
Bug 1300532 - Ensure old and new URL are ordered correctly to show prompt when entering ReaderView r?sebastian MozReview-Commit-ID: 4Yql0m3JV6M
mobile/android/base/java/org/mozilla/gecko/promotion/ReaderViewBookmarkPromotion.java
--- a/mobile/android/base/java/org/mozilla/gecko/promotion/ReaderViewBookmarkPromotion.java
+++ b/mobile/android/base/java/org/mozilla/gecko/promotion/ReaderViewBookmarkPromotion.java
@@ -36,17 +36,17 @@ public class ReaderViewBookmarkPromotion
     }
 
     @Override
     public void onTabChanged(Tab tab, Tabs.TabEvents msg, String data) {
         switch (msg) {
             case LOCATION_CHANGE:
                 // old url: data
                 // new url: tab.getURL()
-                final boolean enteringReaderMode = ReaderModeUtils.isEnteringReaderMode(tab.getURL(), data);
+                final boolean enteringReaderMode = ReaderModeUtils.isEnteringReaderMode(data, tab.getURL());
 
                 if (!hasEnteredReaderMode && enteringReaderMode) {
                     hasEnteredReaderMode = true;
                     promoteBookmarking();
                 }
 
                 break;
         }