| OLD | NEW |
|---|
| 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 "base/basictypes.h" | 5 #include "base/basictypes.h" |
| 6 #include "base/memory/scoped_ptr.h" | 6 #include "base/memory/scoped_ptr.h" |
| 7 #include "base/time/time.h" | 7 #include "base/time/time.h" |
| 8 #include "content/browser/devtools/devtools_manager.h" | 8 #include "content/browser/devtools/devtools_manager.h" |
| 9 #include "content/browser/devtools/embedded_worker_devtools_manager.h" | 9 #include "content/browser/devtools/embedded_worker_devtools_manager.h" |
| 10 #include "content/browser/devtools/render_view_devtools_agent_host.h" | 10 #include "content/browser/devtools/render_view_devtools_agent_host.h" |
| (...skipping 188 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
| 199 class DevToolsManagerTest : public RenderViewHostImplTestHarness { | 199 class DevToolsManagerTest : public RenderViewHostImplTestHarness { |
| 200 public: | 200 public: |
| 201 DevToolsManagerTest() | 201 DevToolsManagerTest() |
| 202 : old_browser_client_(NULL) {} | 202 : old_browser_client_(NULL) {} |
| 203 | 203 |
| 204 protected: | 204 protected: |
| 205 void SetUp() override { | 205 void SetUp() override { |
| 206 RenderViewHostImplTestHarness::SetUp(); | 206 RenderViewHostImplTestHarness::SetUp(); |
| 207 TestDevToolsClientHost::ResetCounters(); | 207 TestDevToolsClientHost::ResetCounters(); |
| 208 old_browser_client_ = SetBrowserClientForTesting(&browser_client_); | 208 old_browser_client_ = SetBrowserClientForTesting(&browser_client_); |
| 209 DevToolsManager::GetInstance()->SetUpForTest(DevToolsManager::Scheduler()); |
| 209 } | 210 } |
| 210 | 211 |
| 211 void TearDown() override { | 212 void TearDown() override { |
| 212 SetBrowserClientForTesting(old_browser_client_); | 213 SetBrowserClientForTesting(old_browser_client_); |
| 214 DevToolsManager::GetInstance()->SetUpForTest(DevToolsManager::Scheduler()); |
| 213 RenderViewHostImplTestHarness::TearDown(); | 215 RenderViewHostImplTestHarness::TearDown(); |
| 214 } | 216 } |
| 215 | 217 |
| 216 ContentBrowserClientWithDevTools browser_client_; | 218 ContentBrowserClientWithDevTools browser_client_; |
| 217 ContentBrowserClient* old_browser_client_; | 219 ContentBrowserClient* old_browser_client_; |
| 218 }; | 220 }; |
| 219 | 221 |
| 220 TEST_F(DevToolsManagerTest, OpenAndManuallyCloseDevToolsClientHost) { | 222 TEST_F(DevToolsManagerTest, OpenAndManuallyCloseDevToolsClientHost) { |
| 221 scoped_refptr<DevToolsAgentHost> agent( | 223 scoped_refptr<DevToolsAgentHost> agent( |
| 222 DevToolsAgentHost::GetOrCreateFor(web_contents())); | 224 DevToolsAgentHost::GetOrCreateFor(web_contents())); |
| (...skipping 155 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
| 378 base::Closure closure_; | 380 base::Closure closure_; |
| 379 }; | 381 }; |
| 380 | 382 |
| 381 TEST_F(DevToolsManagerTest, TestObserver) { | 383 TEST_F(DevToolsManagerTest, TestObserver) { |
| 382 GURL url1("data:text/html,<body>Body1</body>"); | 384 GURL url1("data:text/html,<body>Body1</body>"); |
| 383 GURL url2("data:text/html,<body>Body2</body>"); | 385 GURL url2("data:text/html,<body>Body2</body>"); |
| 384 GURL url3("data:text/html,<body>Body3</body>"); | 386 GURL url3("data:text/html,<body>Body3</body>"); |
| 385 | 387 |
| 386 TestDevToolsManagerScheduler scheduler; | 388 TestDevToolsManagerScheduler scheduler; |
| 387 DevToolsManager* manager = DevToolsManager::GetInstance(); | 389 DevToolsManager* manager = DevToolsManager::GetInstance(); |
| 388 manager->SetSchedulerForTest(scheduler.callback()); | 390 manager->SetUpForTest(scheduler.callback()); |
| 389 | 391 |
| 390 contents()->NavigateAndCommit(url1); | 392 contents()->NavigateAndCommit(url1); |
| 391 RunAllPendingInMessageLoop(); | 393 RunAllPendingInMessageLoop(); |
| 392 | 394 |
| 393 scoped_ptr<TestDevToolsManagerObserver> observer( | 395 scoped_ptr<TestDevToolsManagerObserver> observer( |
| 394 new TestDevToolsManagerObserver()); | 396 new TestDevToolsManagerObserver()); |
| 395 manager->AddObserver(observer.get()); | 397 manager->AddObserver(observer.get()); |
| 396 // Added observer should get an update. | 398 // Added observer should get an update. |
| 397 EXPECT_EQ(1, observer->updates_count()); | 399 EXPECT_EQ(1, observer->updates_count()); |
| 398 ASSERT_EQ(1u, observer->hosts().size()); | 400 ASSERT_EQ(1u, observer->hosts().size()); |
| (...skipping 48 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
| 447 EXPECT_EQ(url2.spec(), observer->hosts()[0]->GetURL().spec()); | 449 EXPECT_EQ(url2.spec(), observer->hosts()[0]->GetURL().spec()); |
| 448 | 450 |
| 449 // Check there were no extra updates. | 451 // Check there were no extra updates. |
| 450 scheduler.Run(); | 452 scheduler.Run(); |
| 451 EXPECT_TRUE(scheduler.IsEmpty()); | 453 EXPECT_TRUE(scheduler.IsEmpty()); |
| 452 EXPECT_EQ(4, observer->updates_count()); | 454 EXPECT_EQ(4, observer->updates_count()); |
| 453 | 455 |
| 454 manager->RemoveObserver(observer.get()); | 456 manager->RemoveObserver(observer.get()); |
| 455 | 457 |
| 456 EXPECT_TRUE(scheduler.IsEmpty()); | 458 EXPECT_TRUE(scheduler.IsEmpty()); |
| 457 manager->SetSchedulerForTest(DevToolsManager::Scheduler()); | |
| 458 } | 459 } |
| 459 | 460 |
| 460 } // namespace content | 461 } // namespace content |
| OLD | NEW |
|---|
Chromium Code Reviews
Issue 790723003:
[DevTools] Fix DevToolsManagerTest.TestObserver (Closed)
Base URL: https://chromium.googlesource.com/chromium/src.git@master