move 'web_accessible_resources' parsing out of Extension class.

chrome/browser/extensions/manifest_web_parser.h

+// Copyright (c) 2013 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_MANIFEST_WEB_PARSER_H_
+#define CHROME_BROWSER_EXTENSIONS_MANIFEST_WEB_PARSER_H_
+
+#include "base/basictypes.h"
+#include "chrome/browser/extensions/api/profile_keyed_api_factory.h"
+
+class Profile;
+
+namespace extensions {
+
+class ManifestWebParser : public ProfileKeyedAPI {

[Yoyo Zhou, 2013/01/10 00:37:58]

This name doesn't make sense. "web" by itself doesn't mean anything. (Web
intents also have nothing to do with web-accessible resources.)
WebAccessibleResourcesParser would make more sense.

[Joe Thomas, 2013/01/10 20:56:18]

Done.

+ public:
+  explicit ManifestWebParser(Profile* profile);
+  virtual ~ManifestWebParser();
+
+ private:
+  friend class ProfileKeyedAPIFactory<ManifestWebParser>;
+
+  // ProfileKeyedAPI implementation.
+  static const char* service_name() {
+    return "ManifestWebParser";
+  }
+  static const bool kServiceIsNULLWhileTesting = true;
+
+  DISALLOW_COPY_AND_ASSIGN(ManifestWebParser);
+};
+
+template <>
+ProfileKeyedAPIFactory<ManifestWebParser>*
+ProfileKeyedAPIFactory<ManifestWebParser>::GetInstance();
+
+}  // namespace extensions
+
+#endif  // CHROME_BROWSER_EXTENSIONS_MANIFEST_WEB_PARSER_H_