Move the parsing of app.launch related keys out of Extension class.

chrome/browser/extensions/app_launch_manifest_parser.cc

Index: chrome/browser/extensions/app_launch_manifest_parser.cc
diff --git a/chrome/browser/extensions/app_launch_manifest_parser.cc b/chrome/browser/extensions/app_launch_manifest_parser.cc
new file mode 100644
index 0000000000000000000000000000000000000000..724e709139e6a1d9a6ebc1166d772e038d3ad6c9
--- /dev/null
+++ b/chrome/browser/extensions/app_launch_manifest_parser.cc
@@ -0,0 +1,29 @@
+// 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.
+
+#include "chrome/browser/extensions/app_launch_manifest_parser.h"
+
+#include "base/lazy_instance.h"
+#include "chrome/browser/profiles/profile.h"

[Devlin, 2013/03/08 17:44:21]

nit: Don't need this one either, I think.

[Joe Thomas, 2013/03/09 01:59:52]

Done.

+#include "chrome/common/extensions/app_launcher_info.h"
+
+namespace extensions {
+
+AppLaunchManifestParser::AppLaunchManifestParser(Profile* profile) {
+  (new AppLaunchManifestHandler)->Register();
+}
+
+AppLaunchManifestParser::~AppLaunchManifestParser() {
+}
+
+static base::LazyInstance<ProfileKeyedAPIFactory<AppLaunchManifestParser> >
+    g_factory = LAZY_INSTANCE_INITIALIZER;
+
+// static
+ProfileKeyedAPIFactory<AppLaunchManifestParser>*
+    AppLaunchManifestParser::GetFactoryInstance() {

[Devlin, 2013/03/08 17:44:21]

nit: Not sure why, but style for wrapping functions where we break after the
return type is to put them at the same indentation; i.e.

ProfileKeyedAPIFactory<AppLaunchManifestParser>*
AppLaunchManifestParser::GetFactoryInstance() {

[Joe Thomas, 2013/03/09 01:59:52]

Done.

+  return &g_factory.Get();
+}
+
+}  // namespace extensions