Only set precache.last_time when precache ran.

chrome/android/java/src/org/chromium/chrome/browser/precache/PrecacheLauncher.java

Index: chrome/android/java/src/org/chromium/chrome/browser/precache/PrecacheLauncher.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/precache/PrecacheLauncher.java b/chrome/android/java/src/org/chromium/chrome/browser/precache/PrecacheLauncher.java
index 7292c4c95fc95612b84038ab8e6e243fae26a7ca..e4cb45718739e3459e44336ab892977ce32ec1a2 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/precache/PrecacheLauncher.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/precache/PrecacheLauncher.java
@@ -47,15 +47,15 @@ public abstract class PrecacheLauncher {
     }
 
     /** Called when a precache cycle completes. */
-    protected abstract void onPrecacheCompleted();
+    protected abstract void onPrecacheCompleted(boolean precacheStarted);

[bengr, 2015/08/05 23:25:35]

Describe the parameter in javadoc.

[twifkak, 2015/08/06 00:09:46]

Done.

 
     /**
      * Called by native code when the precache cycle completes. This method exists because an
      * abstract method cannot be directly called from native.
      */
     @CalledByNative
-    private void onPrecacheCompletedCallback() {
-        onPrecacheCompleted();
+    private void onPrecacheCompletedCallback(boolean precacheStarted) {

[bengr, 2015/08/05 23:25:35]

Why is this precacheStarted and not precacheCompleted? Add javadoc.

[twifkak, 2015/08/06 00:09:46]

Done.

+        onPrecacheCompleted(precacheStarted);
     }
 
     /**
@@ -80,8 +80,8 @@ public abstract class PrecacheLauncher {
 
         // privacyPreferencesManager.shouldPrerender() and nativeShouldRun() can only be executed on
         // the UI thread.
-        PrecacheServiceLauncher.setIsPrecachingEnabled(
-                context, privacyPreferencesManager.shouldPrerender() && nativeShouldRun());
+        PrecacheServiceLauncher.setIsPrecachingEnabled(context.getApplicationContext(),

[bengr, 2015/08/05 23:25:35]

Please update comment to explain why getApplicationContext() is needed.

[twifkak, 2015/08/06 00:09:46]

Added comment to PrecacheServiceLauncher.

+                privacyPreferencesManager.shouldPrerender() && nativeShouldRun());
         Log.v(TAG, "updateEnabledSync complete");
     }
 
@@ -130,7 +130,7 @@ public abstract class PrecacheLauncher {
 
     private static final PrecacheLauncher sInstance = new PrecacheLauncher() {
         @Override
-        protected void onPrecacheCompleted() {}
+        protected void onPrecacheCompleted(boolean precacheStarted) {}
     };
 
     // Initialized by updateEnabled to call updateEnabledSync when the sync