Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(2605)

Unified Diff: ceee/ie/broker/window_api_module_unittest.cc

Issue 6063001: ceee: Include base/win/scope_comptr.h instead of base/scoped_comptr_win.h (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/ceee
Patch Set: fix chrome_frame Created 10 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « ceee/ie/broker/tab_api_module_unittest.cc ('k') | ceee/ie/plugin/bho/http_negotiate.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ceee/ie/broker/window_api_module_unittest.cc
diff --git a/ceee/ie/broker/window_api_module_unittest.cc b/ceee/ie/broker/window_api_module_unittest.cc
index 590bf02e47ea8e8e9f04053cb11472740b1fb499..13456e8245073ae616e66a5523ed73e7804f9f99 100644
--- a/ceee/ie/broker/window_api_module_unittest.cc
+++ b/ceee/ie/broker/window_api_module_unittest.cc
@@ -11,8 +11,8 @@
#include <iepmapi.h>
#include <set>
-#include "base/scoped_comptr_win.h"
#include "base/scoped_ptr.h"
+#include "base/win/scoped_comptr.h"
#include "ceee/common/process_utils_win.h"
#include "ceee/ie/broker/chrome_postman.h"
#include "ceee/ie/broker/window_api_module.h"
@@ -275,8 +275,8 @@ class WindowApiTests: public testing::Test {
testing::MockWindowExecutor* mock_window_executor_;
testing::MockTabExecutor* mock_tab_executor_;
// To control the life span of the executors.
- ScopedComPtr<ICeeeWindowExecutor> mock_window_executor_keeper_;
- ScopedComPtr<ICeeeTabExecutor> mock_tab_executor_keeper_;
+ base::win::ScopedComPtr<ICeeeWindowExecutor> mock_window_executor_keeper_;
+ base::win::ScopedComPtr<ICeeeTabExecutor> mock_tab_executor_keeper_;
// Lifespan controlled by Singleton template.
StrictMock<testing::MockExecutorsManager>* executors_manager_;
@@ -738,7 +738,7 @@ TEST_F(WindowApiTests, CreateWindowErrorHandling) {
CComObject<StrictMock<testing::MockIWebBrowser2>>* browser;
CComObject<StrictMock<testing::MockIWebBrowser2>>::CreateInstance(&browser);
DCHECK(browser != NULL);
- ScopedComPtr<IWebBrowser2> browser_keeper(browser);
+ base::win::ScopedComPtr<IWebBrowser2> browser_keeper(browser);
EXPECT_CALL(mock_ie_create, CoCreateInstance(_, _, _, _, _)).
WillRepeatedly(DoAll(SetArgumentPointee<4>(browser_keeper.get()),
AddRef(browser_keeper.get()), Return(S_OK)));
@@ -803,7 +803,7 @@ TEST_F(WindowApiTests, CreateWindowStraightline) {
CComObject<StrictMock<testing::MockIWebBrowser2>>* browser;
CComObject<StrictMock<testing::MockIWebBrowser2>>::CreateInstance(&browser);
DCHECK(browser != NULL);
- ScopedComPtr<IWebBrowser2> browser_keeper(browser);
+ base::win::ScopedComPtr<IWebBrowser2> browser_keeper(browser);
MockIeWindowCreation mock_ie_create;
// TODO(mad@chromium.org): Test behavior with protected on too.
EXPECT_CALL(mock_ie_create, IEIsProtectedModeURL(_)).
« no previous file with comments | « ceee/ie/broker/tab_api_module_unittest.cc ('k') | ceee/ie/plugin/bho/http_negotiate.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698