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

Side by Side Diff: content/browser/devtools/embedded_worker_devtools_manager_unittest.cc

Issue 678073006: Standardize usage of virtual/override/final specifiers. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fix bogus formatting Created 6 years, 1 month 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
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 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 "content/browser/devtools/embedded_worker_devtools_manager.h" 5 #include "content/browser/devtools/embedded_worker_devtools_manager.h"
6 6
7 #include "base/memory/scoped_ptr.h" 7 #include "base/memory/scoped_ptr.h"
8 #include "base/message_loop/message_loop.h" 8 #include "base/message_loop/message_loop.h"
9 #include "base/run_loop.h" 9 #include "base/run_loop.h"
10 #include "content/browser/browser_thread_impl.h" 10 #include "content/browser/browser_thread_impl.h"
(...skipping 40 matching lines...) Expand 10 before | Expand all | Expand 10 after
51 NULL, 51 NULL,
52 NULL, 52 NULL,
53 NULL, 53 NULL,
54 NULL, 54 NULL,
55 NULL, 55 NULL,
56 NULL, 56 NULL,
57 NULL)), 57 NULL)),
58 partition_id_(*partition_.get()) {} 58 partition_id_(*partition_.get()) {}
59 59
60 protected: 60 protected:
61 virtual void SetUp() override { 61 void SetUp() override {
62 manager_ = EmbeddedWorkerDevToolsManager::GetInstance(); 62 manager_ = EmbeddedWorkerDevToolsManager::GetInstance();
63 } 63 }
64 virtual void TearDown() override { 64 void TearDown() override {
65 EmbeddedWorkerDevToolsManager::GetInstance()->ResetForTesting(); 65 EmbeddedWorkerDevToolsManager::GetInstance()->ResetForTesting();
66 } 66 }
67 67
68 void CheckWorkerState(int worker_process_id, 68 void CheckWorkerState(int worker_process_id,
69 int worker_route_id, 69 int worker_route_id,
70 WorkerState state) { 70 WorkerState state) {
71 const EmbeddedWorkerDevToolsManager::WorkerId id(worker_process_id, 71 const EmbeddedWorkerDevToolsManager::WorkerId id(worker_process_id,
72 worker_route_id); 72 worker_route_id);
73 EmbeddedWorkerDevToolsManager::AgentHostMap::iterator it = 73 EmbeddedWorkerDevToolsManager::AgentHostMap::iterator it =
74 manager_->workers_.find(id); 74 manager_->workers_.find(id);
(...skipping 214 matching lines...) Expand 10 before | Expand all | Expand 10 after
289 manager_->SharedWorkerCreated(3, 2, instance); 289 manager_->SharedWorkerCreated(3, 2, instance);
290 CheckWorkerState(3, 2, WorkerState::WORKER_UNINSPECTED); 290 CheckWorkerState(3, 2, WorkerState::WORKER_UNINSPECTED);
291 agent_host = NULL; 291 agent_host = NULL;
292 CheckWorkerState(3, 2, WorkerState::WORKER_UNINSPECTED); 292 CheckWorkerState(3, 2, WorkerState::WORKER_UNINSPECTED);
293 manager_->WorkerDestroyed(3, 2); 293 manager_->WorkerDestroyed(3, 2);
294 CheckWorkerNotExist(3, 2); 294 CheckWorkerNotExist(3, 2);
295 CheckWorkerCount(0); 295 CheckWorkerCount(0);
296 } 296 }
297 297
298 } // namespace content 298 } // namespace content
OLDNEW
« no previous file with comments | « content/browser/devtools/devtools_manager_unittest.cc ('k') | content/browser/devtools/protocol/tracing_handler.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698