Move the InputComponents out of extensions entirely.

chrome/browser/extensions/api/input_ime/input_ime_api.cc

 // 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.
 
 #include "chrome/browser/extensions/api/input_ime/input_ime_api.h"
 
 #include "base/json/json_writer.h"
+#include "base/lazy_instance.h"
 #include "base/stl_util.h"
 #include "base/string_number_conversions.h"
 #include "base/values.h"
 #include "chrome/browser/chromeos/input_method/input_method_engine.h"
 #include "chrome/browser/extensions/event_router.h"
+#include "chrome/browser/extensions/extension_function_registry.h"
 #include "chrome/browser/extensions/extension_input_module_constants.h"
 #include "chrome/browser/extensions/extension_system.h"
 #include "chrome/browser/profiles/profile.h"
+#include "chrome/common/extensions/extension_manifest_constants.h"
 
 namespace keys = extension_input_module_constants;
 
 namespace {
 
 const char kStyleNone[] = "none";
 const char kStyleCheck[] = "check";
 const char kStyleRadio[] = "radio";
 const char kStyleSeparator[] = "separator";
 
@@ skipping 286 matching lines @@
 
 InputImeEventRouter*
 InputImeEventRouter::GetInstance() {
   return Singleton<InputImeEventRouter>::get();
 }
 
 #if defined(OS_CHROMEOS)
 bool InputImeEventRouter::RegisterIme(
     Profile* profile,
     const std::string& extension_id,
-    const extensions::Extension::InputComponentInfo& component) {
+    const extensions::InputComponentInfo& component) {
   VLOG(1) << "RegisterIme: " << extension_id << " id: " << component.id;
 
   std::map<std::string, chromeos::InputMethodEngine*>& engine_map =
       engines_[extension_id];
 
   std::map<std::string, chromeos::InputMethodEngine*>::iterator engine_ix =
       engine_map.find(component.id);
   if (engine_ix != engine_map.end()) {
     return false;
   }
@@ skipping 497 matching lines @@
 
   InputImeEventRouter::GetInstance()->OnKeyEventHandled(
       extension_id(), request_id_str, handled);
 
   return true;
 }
 #endif
 
 InputImeAPI::InputImeAPI(Profile* profile)
     : profile_(profile) {
+  ManifestHandler::Register(extension_manifest_keys::kInputComponents,
+                            new InputComponentsHandler);
+
   registrar_.Add(this, chrome::NOTIFICATION_EXTENSION_LOADED,
                  content::Source<Profile>(profile));
   registrar_.Add(this, chrome::NOTIFICATION_EXTENSION_UNLOADED,
                  content::Source<Profile>(profile));
+
+  ExtensionFunctionRegistry* registry =
+      ExtensionFunctionRegistry::GetInstance();
+  registry->RegisterFunction<SetCompositionFunction>();
+  registry->RegisterFunction<ClearCompositionFunction>();
+  registry->RegisterFunction<CommitTextFunction>();
+  registry->RegisterFunction<SetCandidateWindowPropertiesFunction>();
+  registry->RegisterFunction<SetCandidatesFunction>();
+  registry->RegisterFunction<SetCursorPositionFunction>();
+  registry->RegisterFunction<SetMenuItemsFunction>();
+  registry->RegisterFunction<UpdateMenuItemsFunction>();
+  registry->RegisterFunction<KeyEventHandled>();
 }
 
 InputImeAPI::~InputImeAPI() {
 }
 
 void InputImeAPI::Observe(int type,
                           const content::NotificationSource& source,
                           const content::NotificationDetails& details) {
   if (type == chrome::NOTIFICATION_EXTENSION_LOADED) {
     const Extension* extension =
         content::Details<const Extension>(details).ptr();
-    for (std::vector<Extension::InputComponentInfo>::const_iterator component =
-             extension->input_components().begin();
-         component != extension->input_components().end();
-         ++component) {
-      if (component->type == Extension::INPUT_COMPONENT_TYPE_IME) {
+    if (extensions::InputComponents::GetInputComponents(extension) == NULL)

[Yoyo Zhou, 2012/12/28 21:04:28]

You should assign the InputComponents to a temporary variable to avoid this
unwieldy code. Also, NULL-checks are usually written
if (!foo)

[SanjoyPal, 2012/12/28 22:38:30]

Done.

+      return;
+    for (std::vector<extensions::InputComponentInfo>::const_iterator component =
+        extensions::InputComponents::GetInputComponents(extension)->
+        input_components.begin(); component != extensions::InputComponents::
+        GetInputComponents(extension)->input_components.end(); ++component) {
+      if (component->type == extensions::INPUT_COMPONENT_TYPE_IME) {
         input_ime_event_router()->RegisterIme(
             profile_, extension->id(), *component);
       }
     }
   } else if (type == chrome::NOTIFICATION_EXTENSION_UNLOADED) {
     const Extension* extension =
         content::Details<const UnloadedExtensionInfo>(details)->extension;
-    if (extension->input_components().size() > 0) {
+    if (extensions::InputComponents::GetInputComponents(extension)->
+        input_components.size() > 0) {
       input_ime_event_router()->UnregisterAllImes(profile_, extension->id());
     }
   }
 }
 
 InputImeEventRouter* InputImeAPI::input_ime_event_router() {
   return InputImeEventRouter::GetInstance();
 }
 
+static base::LazyInstance<ProfileKeyedAPIFactory<InputImeAPI> >
+g_factory = LAZY_INSTANCE_INITIALIZER;
+
+template <>
+ProfileKeyedAPIFactory<InputImeAPI>*
+ProfileKeyedAPIFactory<InputImeAPI>::GetInstance() {
+  return &g_factory.Get();
+}
+
 }  // namespace extensions