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

Unified Diff: chrome/browser/extensions/extension_event_router_forwarder_unittest.cc

Issue 7346024: Get rid of the ProfileId. It was added for ceee. I reverted the original change, since it led to ... (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: '' Created 9 years, 5 months 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
Index: chrome/browser/extensions/extension_event_router_forwarder_unittest.cc
===================================================================
--- chrome/browser/extensions/extension_event_router_forwarder_unittest.cc (revision 92309)
+++ chrome/browser/extensions/extension_event_router_forwarder_unittest.cc (working copy)
@@ -145,8 +145,7 @@
EXPECT_CALL(*event_router,
CallExtensionEventRouter(profile2_, _, _, _, _, _)).Times(0);
event_router->DispatchEventToRenderers(kEventName, kEventArgs,
- profile1_->GetRuntimeId(),
- true, url);
+ profile1_, true, url);
}
TEST_F(ExtensionEventRouterForwarderTest,
@@ -164,8 +163,7 @@
EXPECT_CALL(*event_router,
CallExtensionEventRouter(profile2_, _, _, _, _, _)).Times(0);
event_router->DispatchEventToRenderers(kEventName, kEventArgs,
- profile1_->GetRuntimeId(),
- true, url);
+ profile1_, true, url);
}
TEST_F(ExtensionEventRouterForwarderTest,
@@ -183,8 +181,7 @@
EXPECT_CALL(*event_router,
CallExtensionEventRouter(profile2_, _, _, _, _, _)).Times(0);
event_router->DispatchEventToRenderers(kEventName, kEventArgs,
- incognito->GetRuntimeId(),
- true, url);
+ incognito, true, url);
}
TEST_F(ExtensionEventRouterForwarderTest, UnicastRendererUIUnrestricted) {
@@ -198,8 +195,7 @@
EXPECT_CALL(*event_router,
CallExtensionEventRouter(profile2_, _, _, _, _, _)).Times(0);
event_router->DispatchEventToRenderers(kEventName, kEventArgs,
- profile1_->GetRuntimeId(),
- false, url);
+ profile1_, false, url);
}
TEST_F(ExtensionEventRouterForwarderTest,
@@ -217,8 +213,7 @@
EXPECT_CALL(*event_router,
CallExtensionEventRouter(profile2_, _, _, _, _, _)).Times(0);
event_router->DispatchEventToRenderers(kEventName, kEventArgs,
- profile1_->GetRuntimeId(),
- false, url);
+ profile1_, false, url);
}
TEST_F(ExtensionEventRouterForwarderTest, BroadcastExtensionUI) {
@@ -245,8 +240,7 @@
EXPECT_CALL(*event_router,
CallExtensionEventRouter(profile2_, _, _, _, _, _)).Times(0);
event_router->DispatchEventToExtension(kExt, kEventName, kEventArgs,
- profile1_->GetRuntimeId(),
- true, url);
+ profile1_, true, url);
}
TEST_F(ExtensionEventRouterForwarderTest, UnicastExtensionUIUnrestricted) {
@@ -260,6 +254,5 @@
EXPECT_CALL(*event_router,
CallExtensionEventRouter(profile2_, _, _, _, _, _)).Times(0);
event_router->DispatchEventToExtension(kExt, kEventName, kEventArgs,
- profile1_->GetRuntimeId(),
- false, url);
+ profile1_, false, url);
}
« no previous file with comments | « chrome/browser/extensions/extension_event_router_forwarder.cc ('k') | chrome/browser/extensions/extension_function.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698