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

Side by Side Diff: chrome/browser/devtools/chrome_devtools_manager_delegate.cc

Issue 442303002: DevTools: migrate DevTools APIs to use WebContents instead of RenderViewHost. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Review comments addressed. Created 6 years, 4 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 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 "chrome/browser/devtools/chrome_devtools_manager_delegate.h" 5 #include "chrome/browser/devtools/chrome_devtools_manager_delegate.h"
6 6
7 #include "base/values.h" 7 #include "base/values.h"
8 #include "chrome/browser/devtools/devtools_network_conditions.h" 8 #include "chrome/browser/devtools/devtools_network_conditions.h"
9 #include "chrome/browser/devtools/devtools_network_controller.h" 9 #include "chrome/browser/devtools/devtools_network_controller.h"
10 #include "chrome/browser/devtools/devtools_protocol.h" 10 #include "chrome/browser/devtools/devtools_protocol.h"
11 #include "chrome/browser/devtools/devtools_protocol_constants.h" 11 #include "chrome/browser/devtools/devtools_protocol_constants.h"
12 #include "chrome/browser/devtools/devtools_window.h" 12 #include "chrome/browser/devtools/devtools_window.h"
13 #include "chrome/browser/profiles/profile.h" 13 #include "chrome/browser/profiles/profile.h"
14 #include "content/public/browser/browser_thread.h" 14 #include "content/public/browser/browser_thread.h"
15 #include "content/public/browser/devtools_agent_host.h" 15 #include "content/public/browser/devtools_agent_host.h"
16 #include "content/public/browser/devtools_manager.h" 16 #include "content/public/browser/devtools_manager.h"
17 #include "content/public/browser/render_process_host.h" 17 #include "content/public/browser/render_process_host.h"
18 #include "content/public/browser/render_view_host.h" 18 #include "content/public/browser/render_view_host.h"
19 #include "content/public/browser/site_instance.h" 19 #include "content/public/browser/site_instance.h"
20 #include "content/public/browser/web_contents.h"
20 21
21 ChromeDevToolsManagerDelegate::ChromeDevToolsManagerDelegate() { 22 ChromeDevToolsManagerDelegate::ChromeDevToolsManagerDelegate() {
22 } 23 }
23 24
24 ChromeDevToolsManagerDelegate::~ChromeDevToolsManagerDelegate() { 25 ChromeDevToolsManagerDelegate::~ChromeDevToolsManagerDelegate() {
25 } 26 }
26 27
27 void ChromeDevToolsManagerDelegate::Inspect( 28 void ChromeDevToolsManagerDelegate::Inspect(
28 content::BrowserContext* browser_context, 29 content::BrowserContext* browser_context,
29 content::DevToolsAgentHost* agent_host) { 30 content::DevToolsAgentHost* agent_host) {
(...skipping 21 matching lines...) Expand all
51 if (method == chrome::devtools::Network::emulateNetworkConditions::kName) 52 if (method == chrome::devtools::Network::emulateNetworkConditions::kName)
52 response = EmulateNetworkConditions(agent_host, command.get()); 53 response = EmulateNetworkConditions(agent_host, command.get());
53 54
54 if (response) 55 if (response)
55 return response->Serialize(); 56 return response->Serialize();
56 return NULL; 57 return NULL;
57 } 58 }
58 59
59 Profile* ChromeDevToolsManagerDelegate::GetProfile( 60 Profile* ChromeDevToolsManagerDelegate::GetProfile(
60 content::DevToolsAgentHost* agent_host) { 61 content::DevToolsAgentHost* agent_host) {
61 content::RenderViewHost* host = agent_host->GetRenderViewHost(); 62 content::WebContents* web_contents = agent_host->GetWebContents();
62 if (!host) 63 if (!web_contents)
63 return NULL; 64 return NULL;
64 return Profile::FromBrowserContext(host->GetSiteInstance()->GetProcess()-> 65 return Profile::FromBrowserContext(web_contents->GetBrowserContext());
65 GetBrowserContext());
66 } 66 }
67 67
68 scoped_ptr<DevToolsProtocol::Response> 68 scoped_ptr<DevToolsProtocol::Response>
69 ChromeDevToolsManagerDelegate::EmulateNetworkConditions( 69 ChromeDevToolsManagerDelegate::EmulateNetworkConditions(
70 content::DevToolsAgentHost* agent_host, 70 content::DevToolsAgentHost* agent_host,
71 DevToolsProtocol::Command* command) { 71 DevToolsProtocol::Command* command) {
72 base::DictionaryValue* params = command->params(); 72 base::DictionaryValue* params = command->params();
73 namespace names = ::chrome::devtools::Network::emulateNetworkConditions; 73 namespace names = ::chrome::devtools::Network::emulateNetworkConditions;
74 74
75 bool offline = false; 75 bool offline = false;
(...skipping 37 matching lines...) Expand 10 before | Expand all | Expand 10 after
113 } 113 }
114 114
115 void ChromeDevToolsManagerDelegate::DevToolsAgentStateChanged( 115 void ChromeDevToolsManagerDelegate::DevToolsAgentStateChanged(
116 content::DevToolsAgentHost* agent_host, 116 content::DevToolsAgentHost* agent_host,
117 bool attached) { 117 bool attached) {
118 scoped_ptr<DevToolsNetworkConditions> conditions; 118 scoped_ptr<DevToolsNetworkConditions> conditions;
119 if (attached) 119 if (attached)
120 conditions.reset(new DevToolsNetworkConditions()); 120 conditions.reset(new DevToolsNetworkConditions());
121 UpdateNetworkState(agent_host, conditions.Pass()); 121 UpdateNetworkState(agent_host, conditions.Pass());
122 } 122 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698