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

Side by Side Diff: chrome/browser/extensions/event_router_forwarder_unittest.cc

Issue 15984016: Call scoped_refptr<T>::get() rather than relying on implicit "operator T*" (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rebased Created 7 years, 6 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/extensions/event_router_forwarder.h" 5 #include "chrome/browser/extensions/event_router_forwarder.h"
6 6
7 #include "base/bind.h" 7 #include "base/bind.h"
8 #include "base/message_loop.h" 8 #include "base/message_loop.h"
9 #include "base/power_monitor/power_monitor.h" 9 #include "base/power_monitor/power_monitor.h"
10 #include "base/test/thread_test_helper.h" 10 #include "base/test/thread_test_helper.h"
(...skipping 149 matching lines...) Expand 10 before | Expand all | Expand 10 after
160 EXPECT_CALL(*event_router.get(), 160 EXPECT_CALL(*event_router.get(),
161 CallEventRouter(profile1_, "", kEventName, profile1_, url)); 161 CallEventRouter(profile1_, "", kEventName, profile1_, url));
162 EXPECT_CALL(*event_router.get(), 162 EXPECT_CALL(*event_router.get(),
163 CallEventRouter(profile2_, "", kEventName, profile2_, url)); 163 CallEventRouter(profile2_, "", kEventName, profile2_, url));
164 BrowserThread::PostTask(BrowserThread::IO, FROM_HERE, 164 BrowserThread::PostTask(BrowserThread::IO, FROM_HERE,
165 base::Bind( 165 base::Bind(
166 &BroadcastEventToRenderers, base::Unretained(event_router.get()), 166 &BroadcastEventToRenderers, base::Unretained(event_router.get()),
167 kEventName, url)); 167 kEventName, url));
168 168
169 // Wait for IO thread's message loop to be processed 169 // Wait for IO thread's message loop to be processed
170 scoped_refptr<base::ThreadTestHelper> helper( 170 scoped_refptr<base::ThreadTestHelper> helper(new base::ThreadTestHelper(
171 new base::ThreadTestHelper( 171 BrowserThread::GetMessageLoopProxyForThread(BrowserThread::IO).get()));
172 BrowserThread::GetMessageLoopProxyForThread(BrowserThread::IO)));
173 ASSERT_TRUE(helper->Run()); 172 ASSERT_TRUE(helper->Run());
174 173
175 base::MessageLoop::current()->RunUntilIdle(); 174 base::MessageLoop::current()->RunUntilIdle();
176 } 175 }
177 176
178 TEST_F(EventRouterForwarderTest, UnicastRendererUIRestricted) { 177 TEST_F(EventRouterForwarderTest, UnicastRendererUIRestricted) {
179 scoped_refptr<MockEventRouterForwarder> event_router( 178 scoped_refptr<MockEventRouterForwarder> event_router(
180 new MockEventRouterForwarder); 179 new MockEventRouterForwarder);
181 using ::testing::_; 180 using ::testing::_;
182 GURL url; 181 GURL url;
(...skipping 97 matching lines...) Expand 10 before | Expand all | Expand 10 after
280 GURL url; 279 GURL url;
281 EXPECT_CALL(*event_router.get(), 280 EXPECT_CALL(*event_router.get(),
282 CallEventRouter(profile1_, kExt, kEventName, NULL, url)); 281 CallEventRouter(profile1_, kExt, kEventName, NULL, url));
283 EXPECT_CALL(*event_router.get(), CallEventRouter(profile2_, _, _, _, _)) 282 EXPECT_CALL(*event_router.get(), CallEventRouter(profile2_, _, _, _, _))
284 .Times(0); 283 .Times(0);
285 DispatchEventToExtension(event_router.get(), kExt, kEventName, profile1_, 284 DispatchEventToExtension(event_router.get(), kExt, kEventName, profile1_,
286 false, url); 285 false, url);
287 } 286 }
288 287
289 } // namespace extensions 288 } // namespace extensions
OLDNEW
« no previous file with comments | « chrome/browser/extensions/crx_installer.cc ('k') | chrome/browser/extensions/extension_apitest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698