summaryrefslogtreecommitdiffstats
path: root/chromeos/ime/component_extension_ime_manager.h
diff options
context:
space:
mode:
authorshuchen@chromium.org <shuchen@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2014-03-11 11:30:45 +0000
committershuchen@chromium.org <shuchen@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2014-03-11 11:30:45 +0000
commitdd9e3673c168c9ed98daa020113f52f4489dabce (patch)
tree550c0b27f0ea11cee21ba200d762efdec871f0d0 /chromeos/ime/component_extension_ime_manager.h
parent5e75b7a0a2dc13f19e77647bb2decbedcdd1df57 (diff)
downloadchromium_src-dd9e3673c168c9ed98daa020113f52f4489dabce.zip
chromium_src-dd9e3673c168c9ed98daa020113f52f4489dabce.tar.gz
chromium_src-dd9e3673c168c9ed98daa020113f52f4489dabce.tar.bz2
[IME] Use ExtensionSystem to check extension's availability instead of holding an internal set in ComponentExtensionIMEManagerImpl. This is to fix the problem of xkb extension doesn't expose its resource to chrome-extension://... The root cause of the problem is that ComponentExtensionIMEManagerImpl object is created before login, but ExtensionSystem is initialized after login. Therefore, xkb extension which is loaded before login won't be in ExtensionSystem but ComponentExtensionIMEManagerImpl thinks it's already been loaded.
BUG=351200 TEST=Verified on pixel device. Review URL: https://codereview.chromium.org/194183003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@256201 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chromeos/ime/component_extension_ime_manager.h')
-rw-r--r--chromeos/ime/component_extension_ime_manager.h3
1 files changed, 1 insertions, 2 deletions
diff --git a/chromeos/ime/component_extension_ime_manager.h b/chromeos/ime/component_extension_ime_manager.h
index fad81d3..bc26ec6 100644
--- a/chromeos/ime/component_extension_ime_manager.h
+++ b/chromeos/ime/component_extension_ime_manager.h
@@ -55,8 +55,7 @@ class CHROMEOS_EXPORT ComponentExtensionIMEManagerDelegate {
const base::FilePath& path) = 0;
// Unloads component extension IME associated with |extension_id|.
- // Returns false if it fails, otherwise returns true;
- virtual bool Unload(const std::string& extension_id,
+ virtual void Unload(const std::string& extension_id,
const base::FilePath& path) = 0;
};