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

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

Issue 258513002: Introduce WorkerStoragePartitionId and use it in SharedWorkerInstance. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Hold WorkerStoragePartition in SharedWorkerMessageFilter Created 6 years, 8 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
« no previous file with comments | « no previous file | content/browser/shared_worker/shared_worker_instance.h » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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/shared_worker_devtools_manager.h" 5 #include "content/browser/devtools/shared_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 28 matching lines...) Expand all
39 : ui_thread_(BrowserThread::UI, &message_loop_), 39 : ui_thread_(BrowserThread::UI, &message_loop_),
40 browser_context_(new TestBrowserContext()), 40 browser_context_(new TestBrowserContext()),
41 partition_( 41 partition_(
42 new WorkerStoragePartition(browser_context_->GetRequestContext(), 42 new WorkerStoragePartition(browser_context_->GetRequestContext(),
43 NULL, 43 NULL,
44 NULL, 44 NULL,
45 NULL, 45 NULL,
46 NULL, 46 NULL,
47 NULL, 47 NULL,
48 NULL, 48 NULL,
49 NULL)) {} 49 NULL)),
50 partition_id_(*partition_.get()) {}
50 51
51 protected: 52 protected:
52 virtual void SetUp() OVERRIDE { 53 virtual void SetUp() OVERRIDE {
53 manager_ = SharedWorkerDevToolsManager::GetInstance(); 54 manager_ = SharedWorkerDevToolsManager::GetInstance();
54 } 55 }
55 virtual void TearDown() OVERRIDE { 56 virtual void TearDown() OVERRIDE {
56 SharedWorkerDevToolsManager::GetInstance()->ResetForTesting(); 57 SharedWorkerDevToolsManager::GetInstance()->ResetForTesting();
57 } 58 }
58 59
59 void CheckWorkerState(int worker_process_id, 60 void CheckWorkerState(int worker_process_id,
(...skipping 24 matching lines...) Expand all
84 } 85 }
85 86
86 void ClientHostClosing(DevToolsClientHost* client_host) { 87 void ClientHostClosing(DevToolsClientHost* client_host) {
87 DevToolsManagerImpl::GetInstance()->ClientHostClosing(client_host); 88 DevToolsManagerImpl::GetInstance()->ClientHostClosing(client_host);
88 } 89 }
89 90
90 base::MessageLoopForIO message_loop_; 91 base::MessageLoopForIO message_loop_;
91 BrowserThreadImpl ui_thread_; 92 BrowserThreadImpl ui_thread_;
92 scoped_ptr<TestBrowserContext> browser_context_; 93 scoped_ptr<TestBrowserContext> browser_context_;
93 scoped_ptr<WorkerStoragePartition> partition_; 94 scoped_ptr<WorkerStoragePartition> partition_;
95 const WorkerStoragePartitionId partition_id_;
94 SharedWorkerDevToolsManager* manager_; 96 SharedWorkerDevToolsManager* manager_;
95 }; 97 };
96 98
97 TEST_F(SharedWorkerDevToolsManagerTest, BasicTest) { 99 TEST_F(SharedWorkerDevToolsManagerTest, BasicTest) {
98 scoped_refptr<DevToolsAgentHost> agent_host; 100 scoped_refptr<DevToolsAgentHost> agent_host;
99 101
100 SharedWorkerInstance instance1(GURL("http://example.com/w.js"), 102 SharedWorkerInstance instance1(GURL("http://example.com/w.js"),
101 base::string16(), 103 base::string16(),
102 base::string16(), 104 base::string16(),
103 blink::WebContentSecurityPolicyTypeReport, 105 blink::WebContentSecurityPolicyTypeReport,
104 browser_context_->GetResourceContext(), 106 browser_context_->GetResourceContext(),
105 *partition_.get()); 107 partition_id_);
106 108
107 agent_host = manager_->GetDevToolsAgentHostForWorker(1, 1); 109 agent_host = manager_->GetDevToolsAgentHostForWorker(1, 1);
108 EXPECT_FALSE(agent_host.get()); 110 EXPECT_FALSE(agent_host.get());
109 111
110 // Created -> Started -> Destroyed 112 // Created -> Started -> Destroyed
111 CheckWorkerNotExist(1, 1); 113 CheckWorkerNotExist(1, 1);
112 manager_->WorkerCreated(1, 1, instance1); 114 manager_->WorkerCreated(1, 1, instance1);
113 CheckWorkerState(1, 1, SharedWorkerDevToolsManager::WORKER_UNINSPECTED); 115 CheckWorkerState(1, 1, SharedWorkerDevToolsManager::WORKER_UNINSPECTED);
114 manager_->WorkerContextStarted(1, 1); 116 manager_->WorkerContextStarted(1, 1);
115 CheckWorkerState(1, 1, SharedWorkerDevToolsManager::WORKER_UNINSPECTED); 117 CheckWorkerState(1, 1, SharedWorkerDevToolsManager::WORKER_UNINSPECTED);
(...skipping 59 matching lines...) Expand 10 before | Expand all | Expand 10 after
175 177
176 TEST_F(SharedWorkerDevToolsManagerTest, AttachTest) { 178 TEST_F(SharedWorkerDevToolsManagerTest, AttachTest) {
177 scoped_refptr<DevToolsAgentHost> agent_host1; 179 scoped_refptr<DevToolsAgentHost> agent_host1;
178 scoped_refptr<DevToolsAgentHost> agent_host2; 180 scoped_refptr<DevToolsAgentHost> agent_host2;
179 181
180 SharedWorkerInstance instance1(GURL("http://example.com/w1.js"), 182 SharedWorkerInstance instance1(GURL("http://example.com/w1.js"),
181 base::string16(), 183 base::string16(),
182 base::string16(), 184 base::string16(),
183 blink::WebContentSecurityPolicyTypeReport, 185 blink::WebContentSecurityPolicyTypeReport,
184 browser_context_->GetResourceContext(), 186 browser_context_->GetResourceContext(),
185 *partition_.get()); 187 partition_id_);
186 SharedWorkerInstance instance2(GURL("http://example.com/w2.js"), 188 SharedWorkerInstance instance2(GURL("http://example.com/w2.js"),
187 base::string16(), 189 base::string16(),
188 base::string16(), 190 base::string16(),
189 blink::WebContentSecurityPolicyTypeReport, 191 blink::WebContentSecurityPolicyTypeReport,
190 browser_context_->GetResourceContext(), 192 browser_context_->GetResourceContext(),
191 *partition_.get()); 193 partition_id_);
192 194
193 // Created -> GetDevToolsAgentHost -> Register -> Started -> Destroyed 195 // Created -> GetDevToolsAgentHost -> Register -> Started -> Destroyed
194 scoped_ptr<TestDevToolsClientHost> client_host1(new TestDevToolsClientHost()); 196 scoped_ptr<TestDevToolsClientHost> client_host1(new TestDevToolsClientHost());
195 CheckWorkerNotExist(2, 1); 197 CheckWorkerNotExist(2, 1);
196 manager_->WorkerCreated(2, 1, instance1); 198 manager_->WorkerCreated(2, 1, instance1);
197 CheckWorkerState(2, 1, SharedWorkerDevToolsManager::WORKER_UNINSPECTED); 199 CheckWorkerState(2, 1, SharedWorkerDevToolsManager::WORKER_UNINSPECTED);
198 agent_host1 = manager_->GetDevToolsAgentHostForWorker(2, 1); 200 agent_host1 = manager_->GetDevToolsAgentHostForWorker(2, 1);
199 EXPECT_TRUE(agent_host1.get()); 201 EXPECT_TRUE(agent_host1.get());
200 CheckWorkerState(2, 1, SharedWorkerDevToolsManager::WORKER_INSPECTED); 202 CheckWorkerState(2, 1, SharedWorkerDevToolsManager::WORKER_INSPECTED);
201 EXPECT_EQ(agent_host1.get(), manager_->GetDevToolsAgentHostForWorker(2, 1)); 203 EXPECT_EQ(agent_host1.get(), manager_->GetDevToolsAgentHostForWorker(2, 1));
(...skipping 52 matching lines...) Expand 10 before | Expand all | Expand 10 after
254 EXPECT_EQ(agent_host2.get(), manager_->GetDevToolsAgentHostForWorker(2, 5)); 256 EXPECT_EQ(agent_host2.get(), manager_->GetDevToolsAgentHostForWorker(2, 5));
255 ClientHostClosing(client_host2.get()); 257 ClientHostClosing(client_host2.get());
256 CheckWorkerCount(1); 258 CheckWorkerCount(1);
257 agent_host2 = NULL; 259 agent_host2 = NULL;
258 CheckWorkerCount(1); 260 CheckWorkerCount(1);
259 manager_->WorkerDestroyed(2, 5); 261 manager_->WorkerDestroyed(2, 5);
260 CheckWorkerCount(0); 262 CheckWorkerCount(0);
261 } 263 }
262 264
263 } // namespace content 265 } // namespace content
OLDNEW
« no previous file with comments | « no previous file | content/browser/shared_worker/shared_worker_instance.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698