Support options page handling into ComponentExtensionManager.

chromeos/ime/component_extension_ime_manager.h

 // Copyright 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 CHROMEOS_IME_COMPONENT_EXTENSION_IME_MANAGER_H_
 #define CHROMEOS_IME_COMPONENT_EXTENSION_IME_MANAGER_H_
 
 #include "base/files/file_path.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/observer_list.h"
 #include "chromeos/chromeos_export.h"
 #include "chromeos/dbus/ibus/ibus_component.h"
 #include "chromeos/ime/input_method_descriptor.h"
 
 namespace chromeos {
 
 // Represents a component extension IME.
 struct CHROMEOS_EXPORT ComponentExtensionIME {
   ComponentExtensionIME();
   ~ComponentExtensionIME();
   std::string id;  // extension id.
   std::string manifest;  // the contents of manifest.json
   std::string description;  // description of extension.
+  std::string options_page_url; // We can't use GURL due to deps restriction.

[satorux1, 2013/04/15 04:16:08]

Hmm, I think it's ok to use GURL in 'chromeos'. For now, please make it a TODO.

I think you can add GURL in chromeos/DEPS at a later time. I think no one will
object.

[Seigo Nonaka, 2013/04/15 04:25:53]

Sure,

On 2013/04/15 04:16:08, satorux1 wrote:

   base::FilePath path;
   std::vector<IBusComponent::EngineDescription> engines;
 };
 
 // Provides an interface to list/load/unload for component extension IME.
 class CHROMEOS_EXPORT ComponentExtensionIMEManagerDelegate {
  public:
   ComponentExtensionIMEManagerDelegate();
   virtual ~ComponentExtensionIMEManagerDelegate();
 
@@ skipping 97 matching lines @@
   ObserverList<Observer> observers_;
 
   bool is_initialized_;
 
   DISALLOW_COPY_AND_ASSIGN(ComponentExtensionIMEManager);
 };
 
 }  // namespace chromeos
 
 #endif  // CHROMEOS_IME_COMPONENT_EXTENSION_IME_MANAGER_H_