Remove RuntimeAPIDelegate::GetPreviousVersion.

extensions/browser/extension_registry.cc

Index: extensions/browser/extension_registry.cc
diff --git a/extensions/browser/extension_registry.cc b/extensions/browser/extension_registry.cc
index 9c2bf16f53ec80f74192f883ae8753bbd51d8638..5624b28320a054f1be0a8f447efdbc8cebf9bc61 100644
--- a/extensions/browser/extension_registry.cc
+++ b/extensions/browser/extension_registry.cc
@@ -5,6 +5,7 @@
 #include "extensions/browser/extension_registry.h"
 
 #include "base/strings/string_util.h"
+#include "base/version.h"
 #include "extensions/browser/extension_registry_factory.h"
 #include "extensions/browser/extension_registry_observer.h"
 
@@ -40,6 +41,14 @@ std::unique_ptr<ExtensionSet> ExtensionRegistry::GenerateInstalledExtensionsSet(
   return installed_extensions;
 }
 
+base::Version ExtensionRegistry::GetVersion(const Extension* extension) const {
+  int include_mask = ExtensionRegistry::ENABLED | ExtensionRegistry::DISABLED |
+                     ExtensionRegistry::BLACKLISTED |

[Devlin, 2017/05/19 01:07:09]

Why not terminated?

[lazyboy, 2017/05/19 01:38:48]

I don't know, this is copied from ExtensionService::GetExtensionById, with
include_disabled = true.
In order to avoid churn, I've added a todo and will add that separately. I don't
really want to introduce any functionality change in this CL.

+                     ExtensionRegistry::BLOCKED;
+  const Extension* old = GetExtensionById(extension->id(), include_mask);

[Devlin, 2017/05/19 01:07:09]

|old| is no longer accurate

[lazyboy, 2017/05/19 01:38:48]

Renamed.

+  return old ? *old->version() : base::Version();
+}
+
 void ExtensionRegistry::AddObserver(ExtensionRegistryObserver* observer) {
   observers_.AddObserver(observer);
 }