Hook up Reader Mode InfoBar

chrome/android/java/src/org/chromium/chrome/browser/dom_distiller/ReaderModeTabInfo.java

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 package org.chromium.chrome.browser.dom_distiller;
 
 import org.chromium.content_public.browser.WebContentsObserver;
 
 /**
  * This class tracks the per-tab state of reader mode.
@@ skipping 12 matching lines @@
     // back from distiller and the user has already loaded a new URL.
     private String mCurrentUrl;
 
     // The distillability heuristics now use a callback to notify the manager that a page can
     // be distilled. This flag is used to detect if the callback is set for this tab.
     private boolean mIsCallbackSet;
 
     // Used to flag the the panel was shown and recorded by UMA.
     private boolean mShowPanelRecorded;
 
+    // The time that the user started viewing Reader Mode content.
+    private long mViewStartTimeMs;
+
+    // Whether or not the current tab is a Reader Mode page.
+    private boolean mIsViewingReaderModePage;
+
     /**
      * @param observer The WebContentsObserver for the tab this object represents.
      */
     public void setWebContentsObserver(WebContentsObserver observer) {
         mWebContentsObserver = observer;
     }
 
     /**
      * @return The WebContentsObserver for the tab this object represents.
      */
     public WebContentsObserver getWebContentsObserver() {
         return mWebContentsObserver;
     }
 
     /**
+     * A notification that the user started viewing Reader Mode.
+     */
+    public void onStartedReaderMode() {
+        mIsViewingReaderModePage = true;
+        mViewStartTimeMs = System.currentTimeMillis();

[wychen, 2017/05/17 07:24:12]

Monotonic time like elapsedRealtime() should be better.

[mdjones, 2017/05/17 16:52:14]

Done.

+    }
+
+    /**
+     * A notification that the user is no longer viewing Reader Mode. This could be because of a
+     * navigation away from the page, switching tabs, or closing the browser.
+     * @return The amount of time in ms that the user spent viewing Reader Mode.
+     */
+    public long onExitReaderMode() {
+        mIsViewingReaderModePage = false;
+        return System.currentTimeMillis() - mViewStartTimeMs;
+    }
+
+    /**
+     * @return Whether or not the user is on a Reader Mode page.
+     */
+    public boolean isViewingReaderModePage() {
+        return mIsViewingReaderModePage;
+    }
+
+    /**
      * @param status The status of reader mode for this object's tab.
      */
     public void setStatus(int status) {
         mStatus = status;
     }
 
     /**
      * @return The reader mode status for this object's tab.
      */
     public int getStatus() {
@@ skipping 51 matching lines @@
 
     /**
      * @param isRecorded True if the action has been recorded.
      */
     public void setIsPanelShowRecorded(boolean isRecorded) {
         mShowPanelRecorded = isRecorded;
     }
 
 }