Disable uses of ProfileImportProcess on Android.

chrome/utility/chrome_content_utility_client.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/utility/chrome_content_utility_client.h"
 
+#include "base/base64.h"
 #include "base/bind.h"
-#include "base/base64.h"
 #include "base/command_line.h"
 #include "base/json/json_reader.h"
+#include "base/memory/ref_counted.h"
 #include "base/message_loop_proxy.h"
 #include "base/threading/thread.h"
 #include "chrome/browser/importer/external_process_importer_bridge.h"
 #include "chrome/browser/importer/importer.h"
 #include "chrome/browser/importer/profile_import_process_messages.h"
 #include "chrome/common/child_process_logging.h"
 #include "chrome/common/chrome_paths.h"
 #include "chrome/common/chrome_utility_messages.h"
 #include "chrome/common/extensions/extension.h"
 #include "chrome/common/extensions/extension_l10n_util.h"
 #include "chrome/common/extensions/extension_unpacker.h"
 #include "chrome/common/extensions/update_manifest.h"
 #include "chrome/common/web_resource/web_resource_unpacker.h"
 #include "content/public/utility/utility_thread.h"
 #include "printing/backend/print_backend.h"
 #include "printing/page_range.h"
 #include "third_party/skia/include/core/SkBitmap.h"
 #include "ui/base/ui_base_switches.h"
 #include "ui/gfx/rect.h"
 #include "webkit/glue/image_decoder.h"
 
 #if defined(OS_WIN)
 #include "base/file_util.h"
-#include "base/memory/scoped_ptr.h"
 #include "base/path_service.h"
 #include "base/win/iat_patch_function.h"
 #include "base/win/scoped_handle.h"
 #include "content/public/common/content_switches.h"
 #include "printing/emf_win.h"
 #include "ui/gfx/gdi_util.h"
 #endif  // defined(OS_WIN)
 
 namespace chrome {
 
-ChromeContentUtilityClient::ChromeContentUtilityClient() : items_to_import_(0) {
+#if defined(OS_ANDROID)
+
+// Dummy implementation on Android.
+class ChromeContentUtilityClient::ImportHandler {
+ public:
+  void OnImportStart(
+      const importer::SourceProfile& source_profile,
+      uint16 items,
+      const base::DictionaryValue& localized_strings) {
+  }
+  void OnImportCancel() {}
+  void OnImportItemFinished(uint16 item) {}
+};
+
+#else
+
+class ChromeContentUtilityClient::ImportHandler {

[Yaron, 2012/07/17 19:13:53]

Why not split this out to a separate file and then gyp it out for Android?

[Philippe, 2012/07/18 09:34:27]

Good point. I kept the dummy implementation for Android which I moved to its own
file to avoid adding too many #ifdefs here.

+ public:
+  ImportHandler() : items_to_import_(0) {}
+
+  void OnImportStart(
+      const importer::SourceProfile& source_profile,
+      uint16 items,
+      const base::DictionaryValue& localized_strings) {
+    bridge_ = new ExternalProcessImporterBridge(
+        localized_strings, content::UtilityThread::Get(),
+        base::MessageLoopProxy::current());
+    importer_ = importer::CreateImporterByType(source_profile.importer_type);
+    if (!importer_) {
+      Send(new ProfileImportProcessHostMsg_Import_Finished(false,
+          "Importer could not be created."));
+      return;
+    }
+
+    items_to_import_ = items;
+
+    // Create worker thread in which importer runs.
+    import_thread_.reset(new base::Thread("import_thread"));
+    base::Thread::Options options;
+    options.message_loop_type = MessageLoop::TYPE_IO;
+    if (!import_thread_->StartWithOptions(options)) {
+      NOTREACHED();
+      ImporterCleanup();
+    }
+    import_thread_->message_loop()->PostTask(
+        FROM_HERE, base::Bind(&Importer::StartImport, importer_.get(),
+                              source_profile, items, bridge_));
+  }
+
+  void OnImportCancel() {
+    ImporterCleanup();
+  }
+
+  void OnImportItemFinished(uint16 item) {
+    items_to_import_ ^= item;  // Remove finished item from mask.
+    // If we've finished with all items, notify the browser process.
+    if (items_to_import_ == 0) {
+      Send(new ProfileImportProcessHostMsg_Import_Finished(true, ""));
+      ImporterCleanup();
+    }
+  }
+
+ private:
+  // The following are used with out of process profile import:
+  void ImporterCleanup() {
+    importer_->Cancel();
+    importer_ = NULL;
+    bridge_ = NULL;
+    import_thread_.reset();
+    content::UtilityThread::Get()->ReleaseProcessIfNeeded();
+  }
+
+  static bool Send(IPC::Message* message) {
+    return content::UtilityThread::Get()->Send(message);
+  }
+
+  // Thread that importer runs on, while ProfileImportThread handles messages
+  // from the browser process.
+  scoped_ptr<base::Thread> import_thread_;
+
+  // Bridge object is passed to importer, so that it can send IPC calls
+  // directly back to the ProfileImportProcessHost.
+  scoped_refptr<ExternalProcessImporterBridge> bridge_;
+
+  // A bitmask of importer::ImportItem.
+  uint16 items_to_import_;
+
+  // Importer of the appropriate type (Firefox, Safari, IE, etc.)
+  scoped_refptr<Importer> importer_;
+};
+
+#endif
+
+ChromeContentUtilityClient::ChromeContentUtilityClient()
+    : import_handler_(new ImportHandler()) {
 }
 
 ChromeContentUtilityClient::~ChromeContentUtilityClient() {
 }
 
 void ChromeContentUtilityClient::UtilityThreadStarted() {
 #if defined(OS_WIN)
   // Load the pdf plugin before the sandbox is turned on. This is for Windows
   // only because we need this DLL only on Windows.
   FilePath pdf;
@@ skipping 19 matching lines @@
                         OnUnpackWebResource)
     IPC_MESSAGE_HANDLER(ChromeUtilityMsg_ParseUpdateManifest,
                         OnParseUpdateManifest)
     IPC_MESSAGE_HANDLER(ChromeUtilityMsg_DecodeImage, OnDecodeImage)
     IPC_MESSAGE_HANDLER(ChromeUtilityMsg_DecodeImageBase64, OnDecodeImageBase64)
     IPC_MESSAGE_HANDLER(ChromeUtilityMsg_RenderPDFPagesToMetafile,
                         OnRenderPDFPagesToMetafile)
     IPC_MESSAGE_HANDLER(ChromeUtilityMsg_ParseJSON, OnParseJSON)
     IPC_MESSAGE_HANDLER(ChromeUtilityMsg_GetPrinterCapsAndDefaults,
                         OnGetPrinterCapsAndDefaults)
-    IPC_MESSAGE_HANDLER(ProfileImportProcessMsg_StartImport,
-                        OnImportStart)
-    IPC_MESSAGE_HANDLER(ProfileImportProcessMsg_CancelImport,
-                        OnImportCancel)
-    IPC_MESSAGE_HANDLER(ProfileImportProcessMsg_ReportImportItemFinished,
-                        OnImportItemFinished)
+    IPC_MESSAGE_FORWARD(

[Yaron, 2012/07/17 19:13:53]

couldn't you also ask your ImportHandler to register for IPC messages that it
cares about (so it's all localized in one place)?

[Philippe, 2012/07/18 09:34:27]

Good point.

+        ProfileImportProcessMsg_StartImport,
+        import_handler_.get(),
+        ChromeContentUtilityClient::ImportHandler::OnImportStart)
+    IPC_MESSAGE_FORWARD(
+        ProfileImportProcessMsg_CancelImport,
+        import_handler_.get(),
+        ChromeContentUtilityClient::ImportHandler::OnImportCancel)
+    IPC_MESSAGE_FORWARD(
+        ProfileImportProcessMsg_ReportImportItemFinished,
+        import_handler_.get(),
+        ChromeContentUtilityClient::ImportHandler::OnImportItemFinished)
     IPC_MESSAGE_UNHANDLED(handled = false)
   IPC_END_MESSAGE_MAP()
   return handled;
 }
 
 bool ChromeContentUtilityClient::Send(IPC::Message* message) {
   return content::UtilityThread::Get()->Send(message);
 }
 
 void ChromeContentUtilityClient::OnUnpackExtension(
@@ skipping 283 matching lines @@
         printer_name, printer_info));
   } else
 #endif
   {
     Send(new ChromeUtilityHostMsg_GetPrinterCapsAndDefaults_Failed(
         printer_name));
   }
   content::UtilityThread::Get()->ReleaseProcessIfNeeded();
 }
 
-void ChromeContentUtilityClient::OnImportStart(
-    const importer::SourceProfile& source_profile,
-    uint16 items,
-    const DictionaryValue& localized_strings) {
-  bridge_ = new ExternalProcessImporterBridge(
-      localized_strings, content::UtilityThread::Get(),
-      base::MessageLoopProxy::current());
-  importer_ = importer::CreateImporterByType(source_profile.importer_type);
-  if (!importer_) {
-    Send(new ProfileImportProcessHostMsg_Import_Finished(false,
-        "Importer could not be created."));
-    return;
-  }
-
-  items_to_import_ = items;
-
-  // Create worker thread in which importer runs.
-  import_thread_.reset(new base::Thread("import_thread"));
-  base::Thread::Options options;
-  options.message_loop_type = MessageLoop::TYPE_IO;
-  if (!import_thread_->StartWithOptions(options)) {
-    NOTREACHED();
-    ImporterCleanup();
-  }
-  import_thread_->message_loop()->PostTask(
-      FROM_HERE, base::Bind(&Importer::StartImport, importer_.get(),
-                            source_profile, items, bridge_));
-}
-
-void ChromeContentUtilityClient::OnImportCancel() {
-  ImporterCleanup();
-}
-
-void ChromeContentUtilityClient::OnImportItemFinished(uint16 item) {
-  items_to_import_ ^= item;  // Remove finished item from mask.
-  // If we've finished with all items, notify the browser process.
-  if (items_to_import_ == 0) {
-    Send(new ProfileImportProcessHostMsg_Import_Finished(true, ""));
-    ImporterCleanup();
-  }
-}
-
-void ChromeContentUtilityClient::ImporterCleanup() {
-  importer_->Cancel();
-  importer_ = NULL;
-  bridge_ = NULL;
-  import_thread_.reset();
-  content::UtilityThread::Get()->ReleaseProcessIfNeeded();
-}
-
 }  // namespace chrome