Service Worker: Change the criteria for bumping the last update check time

content/browser/service_worker/embedded_worker_test_helper.h

Index: content/browser/service_worker/embedded_worker_test_helper.h
diff --git a/content/browser/service_worker/embedded_worker_test_helper.h b/content/browser/service_worker/embedded_worker_test_helper.h
index d7139055f4e9aaa77916c673d2b98e57735484f2..5360f0ef0b83ee23a3b6c7f0de25e16d562af42b 100644
--- a/content/browser/service_worker/embedded_worker_test_helper.h
+++ b/content/browser/service_worker/embedded_worker_test_helper.h
@@ -91,12 +91,18 @@ class EmbeddedWorkerTestHelper : public IPC::Sender,
   // is sent to the embedded worker. Override if necessary. By default
   // they verify given parameters and:
   // - OnStartWorker calls SimulateWorkerStarted
+  // - OnStartWorkerFailed calls SimulateWorkerScriptEvaluated with |success|
+  // false
   // - OnStopWorker calls SimulateWorkerStoped
   // - OnSendMessageToWorker calls the message's respective On*Event handler
   virtual void OnStartWorker(int embedded_worker_id,
                              int64 service_worker_version_id,
                              const GURL& scope,
                              const GURL& script_url);
+  virtual void OnStartWorkerFailed(int embedded_worker_id,

[falken, 2015/12/17 13:23:43]

Sorry to nitpick a bit late in the game... this function is quite different than
the rest in this class, which just do the success thing. Tests that want custom
implementation are supposed to subclass this class.

The name is a bit off too, these are designed to be called in response to a
message from the browser, the browser wouldn't send a "OnStartWorkerFailed"
message. The failure hasn't occurred yet until the function runs.

So I think this custom implementation should be done in a subclass of
EmbeddedWorkerTestHelper.

[jungkees, 2015/12/23 03:08:58]

I'd tried subclassing EmbeddedWorkerTestHelper to override OnStartWorker for the
fail-to-start-worker case. But that needed OnStartWorker to extend the parameter
list (|success| to be passed to SimulateWorkerScriptEvaluated) and involved some
changes in other call sites as such.

So, I instead gave a try with subclassing UpdateJobTestHelper and override
OnStartWorker in that class which simulate the fail-to-start-worker depending on
the value its |force_start_worker_failure_|.

+                                   int64 service_worker_version_id,
+                                   const GURL& scope,
+                                   const GURL& script_url);
   virtual void OnStopWorker(int embedded_worker_id);
   virtual bool OnMessageToWorker(int thread_id,
                                  int embedded_worker_id,
@@ -121,7 +127,7 @@ class EmbeddedWorkerTestHelper : public IPC::Sender,
   void SimulateWorkerScriptCached(int embedded_worker_id);
   void SimulateWorkerScriptLoaded(int embedded_worker_id);
   void SimulateWorkerThreadStarted(int thread_id, int embedded_worker_id);
-  void SimulateWorkerScriptEvaluated(int embedded_worker_id);
+  void SimulateWorkerScriptEvaluated(int embedded_worker_id, bool success);
   void SimulateWorkerStarted(int embedded_worker_id);
   void SimulateWorkerStopped(int embedded_worker_id);
   void SimulateSend(IPC::Message* message);