Invoke callback from WebstoreInstaller when installation is fully complete

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 a0175c25882f0211f5fd308becefbeed43e0bebe..bfd1fddf19c9d0aed6fd50ddfbfdd551af37dd9c 100644
--- a/extensions/browser/extension_registry.cc
+++ b/extensions/browser/extension_registry.cc
@@ -67,6 +67,13 @@ void ExtensionRegistry::TriggerOnWillBeInstalled(const Extension* extension,
           browser_context_, extension, is_update, from_ephemeral, old_name));
 }
 
+void ExtensionRegistry::TriggerOnInstalled(const Extension* extension) {
+  DCHECK(GenerateInstalledExtensionsSet()->Contains(extension->id()));

[asargent_no_longer_on_chrome, 2014/06/11 18:27:54]

I forget, do the bodies of DCHECK's get completely elided in the build process
if DCHECK's are turned off? If not, you might want to wrap this in:

#if DCHECK_IS_ON

#endif 

Since GenerateInstalledExtensionsSet does a little bit of work. 

[tmdiep, 2014/06/11 22:56:18]

The code doesn't get compiled out completely, but the compiler short circuits
the evaluation so the condition won't execute.

+  FOR_EACH_OBSERVER(ExtensionRegistryObserver,
+                    observers_,
+                    OnExtensionInstalled(browser_context_, extension));
+}
+
 void ExtensionRegistry::TriggerOnUninstalled(const Extension* extension) {
   DCHECK(!GenerateInstalledExtensionsSet()->Contains(extension->id()));
   FOR_EACH_OBSERVER(ExtensionRegistryObserver,