From 3289a82a55e7dbc9badac7d503e11ba5d5cb334c Mon Sep 17 00:00:00 2001 From: "pkasting@chromium.org" Date: Tue, 23 Oct 2012 01:32:15 +0000 Subject: Eliminate duplicate COM and OLE initializations. This is take 2; the original patch ( http://crrev.com/159815 ), which only dealt with OLE, broke a few tests. The OneClickSignIn tests were skipping the RenderViewHostTestHarness SetUp() method, while the Accessibility tests directly muck with Widgets and thus have to manually init OLE. BUG=none TEST=Fewer "Multiple OleInitialize..." messages in tests Review URL: https://codereview.chromium.org/11031059 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@163470 0039d316-1c4b-4281-b951-d872f2087c98 --- chrome/browser/importer/ie_importer.cc | 4 ---- 1 file changed, 4 deletions(-) (limited to 'chrome/browser/importer') diff --git a/chrome/browser/importer/ie_importer.cc b/chrome/browser/importer/ie_importer.cc index 7bef316..55bb95d 100644 --- a/chrome/browser/importer/ie_importer.cc +++ b/chrome/browser/importer/ie_importer.cc @@ -24,7 +24,6 @@ #include "base/utf_string_conversions.h" #include "base/win/registry.h" #include "base/win/scoped_co_mem.h" -#include "base/win/scoped_com_initializer.h" #include "base/win/scoped_comptr.h" #include "base/win/scoped_handle.h" #include "base/win/windows_version.h" @@ -415,9 +414,6 @@ void IEImporter::StartImport(const importer::SourceProfile& source_profile, bridge_->NotifyStarted(); - // Some IE settings (such as Protected Storage) are obtained via COM APIs. - base::win::ScopedCOMInitializer com_initializer; - if ((items & importer::HOME_PAGE) && !cancelled()) ImportHomepage(); // Doesn't have a UI item. // The order here is important! -- cgit v1.1