Adding ChromeVox as a component extensions (enabled only for ChromeOS, for no...

chrome/browser/profiles/profile_impl.cc

 // Copyright (c) 2011 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/profiles/profile_impl.h"
 
 #include "app/resource_bundle.h"
 #include "base/command_line.h"
 #include "base/environment.h"
 #include "base/file_path.h"
 #include "base/file_util.h"
 #include "base/metrics/histogram.h"
 #include "base/path_service.h"
 #include "base/scoped_ptr.h"
 #include "base/string_number_conversions.h"
 #include "base/string_util.h"
+#include "base/values.h"
+#include "base/version.h"
 #include "chrome/browser/about_flags.h"
 #include "chrome/browser/appcache/chrome_appcache_service.h"
 #include "chrome/browser/autocomplete/autocomplete_classifier.h"
 #include "chrome/browser/autofill/personal_data_manager.h"
 #include "chrome/browser/background_contents_service.h"
 #include "chrome/browser/background_mode_manager.h"
 #include "chrome/browser/bookmarks/bookmark_model.h"
 #include "chrome/browser/browser_list.h"
 #include "chrome/browser/browser_process.h"
 #include "chrome/browser/browser_signin.h"
 #include "chrome/browser/browser_thread.h"
 #include "chrome/browser/chrome_blob_storage_context.h"
 #include "chrome/browser/content_settings/host_content_settings_map.h"
 #include "chrome/browser/dom_ui/ntp_resource_cache.h"
 #include "chrome/browser/download/download_manager.h"
 #include "chrome/browser/extensions/default_apps.h"
+#include "chrome/browser/extensions/extension_accessibility_api.h"
 #include "chrome/browser/extensions/extension_devtools_manager.h"
 #include "chrome/browser/extensions/extension_error_reporter.h"
 #include "chrome/browser/extensions/extension_info_map.h"
 #include "chrome/browser/extensions/extension_event_router.h"
 #include "chrome/browser/extensions/extension_message_service.h"
 #include "chrome/browser/extensions/extension_pref_store.h"
 #include "chrome/browser/extensions/extension_process_manager.h"
 #include "chrome/browser/extensions/extension_service.h"
 #include "chrome/browser/extensions/user_script_master.h"
 #include "chrome/browser/favicon_service.h"
@@ skipping 37 matching lines @@
 #include "chrome/browser/user_style_sheet_watcher.h"
 #include "chrome/browser/visitedlink/visitedlink_event_listener.h"
 #include "chrome/browser/visitedlink/visitedlink_master.h"
 #include "chrome/browser/web_resource/web_resource_service.h"
 #include "chrome/browser/webdata/web_data_service.h"
 #include "chrome/common/chrome_constants.h"
 #include "chrome/common/chrome_paths.h"
 #include "chrome/common/chrome_paths_internal.h"
 #include "chrome/common/chrome_switches.h"
 #include "chrome/common/json_pref_store.h"
+#include "chrome/common/json_value_serializer.h"
 #include "chrome/common/notification_service.h"
 #include "chrome/common/pref_names.h"
 #include "chrome/common/render_messages.h"
+#include "chrome/common/zip.h"
 #include "grit/browser_resources.h"
 #include "grit/locale_settings.h"
 #include "net/base/transport_security_state.h"
 #include "webkit/database/database_tracker.h"
 
 #if defined(TOOLKIT_USES_GTK)
 #include "chrome/browser/gtk/gtk_theme_provider.h"
 #endif
 
 #if defined(OS_WIN)
@@ skipping 272 matching lines @@
 
 void ProfileImpl::RegisterComponentExtensions() {
   // Register the component extensions.
   typedef std::list<std::pair<std::string, int> > ComponentExtensionList;
   ComponentExtensionList component_extensions;
 
   // Bookmark manager.
   component_extensions.push_back(
       std::make_pair("bookmark_manager", IDR_BOOKMARKS_MANIFEST));
 
+#if defined(OS_CHROMEOS)
+  // Register access extensions only if accessibility is enabled.
+  if (ExtensionAccessibilityEventRouter::GetInstance()->

[asargent_no_longer_on_chrome, 2011/01/20 05:55:59]

It seems kind of weird to ask the event router if accessibility is enabled - can
you just look at the pref value here?

[Chaitanya, 2011/01/20 20:04:52]

using prefs now

On 2011/01/20 05:55:59, Antony Sargent wrote:

+      IsAccessibilityEnabled()) {
+    component_extensions.push_back(
+        std::make_pair("access_chromevox", IDR_CHROMEVOX_MANIFEST));
+  }
+#endif
+
 #if defined(TOUCH_UI)
   component_extensions.push_back(
       std::make_pair("keyboard", IDR_KEYBOARD_MANIFEST));
 #endif
 
   // Web Store.
   component_extensions.push_back(
       std::make_pair("web_store", IDR_WEBSTORE_MANIFEST));
 
   for (ComponentExtensionList::iterator iter = component_extensions.begin();
     iter != component_extensions.end(); ++iter) {
     FilePath path;
     if (PathService::Get(chrome::DIR_RESOURCES, &path)) {
       path = path.AppendASCII(iter->first);
     } else {
       NOTREACHED();
     }
 
     std::string manifest =
         ResourceBundle::GetSharedInstance().GetRawDataResource(
             iter->second).as_string();
+
+#if defined(OS_CHROMEOS)
+    if (StartsWithASCII(iter->first, "access_", true)) {
+      path = GetPath().AppendASCII(
+          ExtensionService::kAccessibilityInstallDirectoryName);
+      path = path.AppendASCII(iter->first);
+      UnzipAccessibilityExtensionResources(iter->first, path, manifest);

[asargent_no_longer_on_chrome, 2011/01/20 05:55:59]

nit: it might read more clearly if you factor out the "are we up to date" check,
and then make this something like:

if (!AccessibilityExtensionIsUpToDate(...))
  UnzipAccessibilityExtensionResources(...)

[Chaitanya, 2011/01/20 20:04:52]

Done.

+    }
+#endif
+
     extensions_service_->register_component_extension(
         ExtensionService::ComponentExtensionInfo(manifest, path));
   }
 }
 
 void ProfileImpl::InstallDefaultApps() {
   ExtensionService* extension_service = GetExtensionService();
   DefaultApps* default_apps = extension_service->default_apps();
 
   if (!default_apps->ShouldInstallDefaultApps(extension_service->GetAppIds()))
     return;
 
   const ExtensionIdSet& app_ids = default_apps->default_apps();
   for (ExtensionIdSet::const_iterator iter = app_ids.begin();
        iter != app_ids.end(); ++iter) {
     extension_service->AddPendingExtensionFromDefaultAppList(*iter);
   }
 }
 
+#if defined(OS_CHROMEOS)
+void ProfileImpl::UnzipAccessibilityExtensionResources(
+    const std::string ext_name,
+    const FilePath dir_path,
+    const std::string manifest) {
+  JSONStringValueSerializer serializer(manifest);
+  scoped_ptr<DictionaryValue> dict(
+      static_cast<DictionaryValue*>(serializer.Deserialize(NULL, NULL)));
+  std::string new_version_str = "0.0";
+  dict->GetString("version", &new_version_str);
+  scoped_ptr<Version> new_version(
+      Version::GetVersionFromString(new_version_str));
+  scoped_ptr<Version> curr_version(Version::GetVersionFromString(
+      GetPrefs()->GetString(prefs::kAccessibilityChromeVoxVersion)));
+  if (new_version->CompareTo(*curr_version.get()) == 1 &&
+      Unzip(FilePath().AppendASCII(extension_misc::kAccessExtensionCrxPath)
+          .AppendASCII(ext_name).ReplaceExtension(FILE_PATH_LITERAL(".zip")),

[asargent_no_longer_on_chrome, 2011/01/20 05:55:59]

This is a little hard to read. Can you factor this out into something like

FilePath zipfile_path = ....
bool success = Unzip(zipfile_path, dir_path);
if (success)
   ...

[Chaitanya, 2011/01/20 20:04:52]

Removed UnzipAccessibilityExtensionResources and added
IsAccessibilityExtensionUpToDate

On 2011/01/20 05:55:59, Antony Sargent wrote:

+          dir_path) {
+    GetPrefs()->SetString(prefs::kAccessibilityChromeVoxVersion,
+                          new_version_str);
+  }
+}
+#endif
+
 void ProfileImpl::InitWebResources() {
   if (web_resource_service_)
     return;
 
   web_resource_service_ = new WebResourceService(this);
   web_resource_service_->StartAfterDelay();
 }
 
 NTPResourceCache* ProfileImpl::GetNTPResourceCache() {
   if (!ntp_resource_cache_.get())
@@ skipping 954 matching lines @@
 }
 
 PrerenderManager* ProfileImpl::GetPrerenderManager() {
   CommandLine* cl = CommandLine::ForCurrentProcess();
   if (!cl->HasSwitch(switches::kEnablePagePrerender))
     return NULL;
   if (!prerender_manager_)
     prerender_manager_ = new PrerenderManager(this);
   return prerender_manager_;
 }