Add a test to cover service worker update codepath when extension is updated

extensions/browser/service_worker_manager.cc

 // 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.
 
 #include "extensions/browser/service_worker_manager.h"
 
 #include "base/bind.h"
 #include "content/public/browser/browser_context.h"
 #include "content/public/browser/service_worker_context.h"
 #include "content/public/browser/storage_partition.h"
@@ skipping 30 matching lines @@
   // TODO(devlin): Technically, this can fail. We should ideally:
   // a) Keep track of extensions with registered service workers.
   // b) Add a callback to the (Un)SuspendServiceWorkersOnOrigin() method.
   // c) Check for any orphaned workers.
   content::BrowserContext::GetStoragePartitionForSite(browser_context_,
                                                       extension->url())
       ->GetServiceWorkerContext()
       ->DeleteForOrigin(extension->url(), base::Bind(&EmptySuccessCallback));
 }
 
+void ServiceWorkerManager::OnExtensionWillBeInstalled(
+    content::BrowserContext* browser_context,
+    const Extension* extension,
+    bool is_update,
+    bool from_ephemeral,
+    const std::string& old_name) {
+  if (is_update) {

[Devlin, 2015/11/11 17:10:29]

nit: A brief comment saying why we do this would probably be worthwhile.

[lazyboy, 2015/11/11 22:34:03]

Done.

+    content::BrowserContext::GetStoragePartitionForSite(browser_context_,
+                                                        extension->url())
+        ->GetServiceWorkerContext()
+        ->SetForceUpdateOnPageLoadForOrigin(extension->url());
+  }
+}
+
 }  // namespace extensions