Add extensions code to use common updater in components/update_client/

chrome/browser/extensions/chrome_extensions_browser_client.h

Index: chrome/browser/extensions/chrome_extensions_browser_client.h
diff --git a/chrome/browser/extensions/chrome_extensions_browser_client.h b/chrome/browser/extensions/chrome_extensions_browser_client.h
index 852a27c5d416261d6c88f43d3f7f73030017fc3e..2fba1a524909eecb9b9ae5a2b9983b05a6625162 100644
--- a/chrome/browser/extensions/chrome_extensions_browser_client.h
+++ b/chrome/browser/extensions/chrome_extensions_browser_client.h
@@ -6,6 +6,8 @@
 #define CHROME_BROWSER_EXTENSIONS_CHROME_EXTENSIONS_BROWSER_CLIENT_H_
 
 #include <map>
+#include <string>

[Ken Rockot(use gerrit already), 2015/09/25 16:19:19]

I would argue that these are unnecessary since the only references to
std::string and std::vector are for method signatures overridden from
ExtensionsBrowserClient, and it should be safe for a derived class to assume the
base has already provided necessary definitions (and extensions_browser_client.h
does in fact include these headers).

On a related note, it looks like <map> actually isn't used at all now, so that
could be deleted too!

[asargent_no_longer_on_chrome, 2015/09/29 22:59:21]

Removed <map>. As for the <string> and <vector> includes, I just added it to get
a clean run of 'git cl lint', where the IncludeWhatYouUse rule likely doesn't
know about inheritance. 

+#include <vector>
 
 #include "base/basictypes.h"
 #include "base/compiler_specific.h"
@@ -110,6 +112,8 @@ class ChromeExtensionsBrowserClient : public ExtensionsBrowserClient {
                       int view_instance_id) override;
   void AttachExtensionTaskManagerTag(content::WebContents* web_contents,
                                      ViewType view_type) override;
+  scoped_refptr<update_client::UpdateClient> CreateUpdateClient(
+      content::BrowserContext* context) override;
 
  private:
   friend struct base::DefaultLazyInstanceTraits<ChromeExtensionsBrowserClient>;