Allow autoupdate to update disabled extensions.

chrome/browser/extensions/updater/extension_updater.h

Index: chrome/browser/extensions/updater/extension_updater.h
diff --git a/chrome/browser/extensions/updater/extension_updater.h b/chrome/browser/extensions/updater/extension_updater.h
index 53d471909376d7dd46b5e13917400227e63ebfc1..adff32944703afe98b088e6a164a2b10678fa805 100644
--- a/chrome/browser/extensions/updater/extension_updater.h
+++ b/chrome/browser/extensions/updater/extension_updater.h
@@ -19,6 +19,7 @@
 #include "base/timer.h"
 #include "chrome/browser/extensions/updater/extension_downloader_delegate.h"
 #include "chrome/browser/extensions/updater/manifest_fetch_data.h"
+#include "chrome/common/extensions/extension_set.h"

[asargent_no_longer_on_chrome, 2012/03/20 00:24:01]

Can you just forward declare ExtensionSet and skip the include here?

http://dev.chromium.org/developers/coding-style/cpp-dos-and-donts#TOC-Forward...

[Matt Tytel, 2012/03/20 20:34:28]

Done.

 #include "content/public/browser/notification_observer.h"
 #include "content/public/browser/notification_registrar.h"
 #include "googleurl/src/gurl.h"
@@ -68,6 +69,12 @@ class ExtensionUpdater : public ExtensionDownloaderDelegate,
   // already a pending task that has not yet run.
   void CheckSoon();
 
+  // Add fetch records for extensions that are installed and have an
+  // update URL to the downloader, ignoring |pending_ids| so the extension isn't
+  // fetched again.
+  void AddToDownloader(const ExtensionSet* extensions,
+                       const std::set<std::string>& pending_ids);

[asargent_no_longer_on_chrome, 2012/03/20 00:24:01]

Any reason not to have this be private? (Typically helper methods that aren't
called outside a class should be)

[Matt Tytel, 2012/03/20 20:34:28]

Done.

+
   // Starts an update check right now, instead of waiting for the next
   // regularly scheduled check or a pending check from CheckSoon().
   void CheckNow();