Move the parsing of "chrome_url_overrides" out of Extension.

chrome/browser/extensions/api/url_overrides/url_overrides_api.h

+// Copyright (c) 2012 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.
+
+#ifndef CHROME_BROWSER_EXTENSIONS_API_URL_OVERRIDES_URL_OVERRIDES_API_H_
+#define CHROME_BROWSER_EXTENSIONS_API_URL_OVERRIDES_URL_OVERRIDES_API_H_
+
+#include "base/basictypes.h"
+#include "chrome/browser/profiles/profile_keyed_service.h"
+#include "content/public/browser/notification_observer.h"
+#include "content/public/browser/notification_registrar.h"
+
+class Profile;
+
+namespace extensions {
+
+class URLOverridesAPI : public ProfileKeyedService,
+                        public content::NotificationObserver {

[Joe Thomas, 2012/12/22 01:25:57]

There are no URLOverrides APIs in extensions, but probably we need a similar
class to pull the url_overrides stuff out of ExtensionService and place it in
c/b/e/ instead of c/b/e/api/.

@Yoyo: what do you think?

[Yoyo Zhou, 2012/12/28 03:59:45]

Seems reasonable to have this be separated out, but I would probably leave it in
chrome/browser/extensions as it's something that belongs to the browser.

[Yoyo Zhou, 2013/01/03 22:36:03]

My vote is still to leave this in chrome/browser/extensions.

[Joe Thomas, 2013/01/04 23:36:11]

I made it part of ManifestURLParser in chrome/browser/extensions/. 

+ public:
+  explicit URLOverridesAPI(Profile* profile);
+  virtual ~URLOverridesAPI();
+
+  // content::NotificationObserver implementation.
+  virtual void Observe(int type,
+                       const content::NotificationSource& source,
+                       const content::NotificationDetails& details) OVERRIDE;
+
+ private:
+  Profile* const profile_;
+  content::NotificationRegistrar registrar_;
+
+  DISALLOW_COPY_AND_ASSIGN(URLOverridesAPI);
+};
+
+}  // namespace extensions
+
+#endif  // CHROME_BROWSER_EXTENSIONS_API_URL_OVERRIDES_URL_OVERRIDES_API_H_