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

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

Issue 560323005: [DevTools] Move target-related methods from DevToolsHttpHandlerDelegate to DevToolsManagerDelegate. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: rebase Created 6 years, 3 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 | « chrome/browser/devtools/chrome_devtools_manager_delegate.h ('k') | chrome/chrome_browser.gypi » ('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 "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_target_impl.h"
8 #include "chrome/browser/devtools/devtools_window.h" 9 #include "chrome/browser/devtools/devtools_window.h"
10 #include "chrome/browser/history/top_sites.h"
9 #include "chrome/browser/profiles/profile.h" 11 #include "chrome/browser/profiles/profile.h"
12 #include "chrome/browser/profiles/profile_manager.h"
13 #include "chrome/browser/ui/browser.h"
14 #include "chrome/browser/ui/browser_iterator.h"
15 #include "content/public/browser/browser_thread.h"
10 #include "content/public/browser/devtools_agent_host.h" 16 #include "content/public/browser/devtools_agent_host.h"
17 #include "content/public/browser/web_contents.h"
11 18
12 ChromeDevToolsManagerDelegate::ChromeDevToolsManagerDelegate() 19 ChromeDevToolsManagerDelegate::ChromeDevToolsManagerDelegate()
13 : network_protocol_handler_(new DevToolsNetworkProtocolHandler()) { 20 : network_protocol_handler_(new DevToolsNetworkProtocolHandler()) {
14 } 21 }
15 22
16 ChromeDevToolsManagerDelegate::~ChromeDevToolsManagerDelegate() { 23 ChromeDevToolsManagerDelegate::~ChromeDevToolsManagerDelegate() {
17 } 24 }
18 25
19 void ChromeDevToolsManagerDelegate::Inspect( 26 void ChromeDevToolsManagerDelegate::Inspect(
20 content::BrowserContext* browser_context, 27 content::BrowserContext* browser_context,
21 content::DevToolsAgentHost* agent_host) { 28 content::DevToolsAgentHost* agent_host) {
22 if (!agent_host->IsWorker()) { 29 if (!agent_host->IsWorker()) {
23 // TODO(horo): Support other types of DevToolsAgentHost when necessary. 30 // TODO(horo): Support other types of DevToolsAgentHost when necessary.
24 NOTREACHED() << "Inspect() only supports workers."; 31 NOTREACHED() << "Inspect() only supports workers.";
25 } 32 }
26 #if !defined(OS_ANDROID)
27 if (Profile* profile = Profile::FromBrowserContext(browser_context)) 33 if (Profile* profile = Profile::FromBrowserContext(browser_context))
28 DevToolsWindow::OpenDevToolsWindowForWorker(profile, agent_host); 34 DevToolsWindow::OpenDevToolsWindowForWorker(profile, agent_host);
29 #endif
30 } 35 }
31 36
32 base::DictionaryValue* ChromeDevToolsManagerDelegate::HandleCommand( 37 base::DictionaryValue* ChromeDevToolsManagerDelegate::HandleCommand(
33 content::DevToolsAgentHost* agent_host, 38 content::DevToolsAgentHost* agent_host,
34 base::DictionaryValue* command_dict) { 39 base::DictionaryValue* command_dict) {
35 return network_protocol_handler_->HandleCommand(agent_host, command_dict); 40 return network_protocol_handler_->HandleCommand(agent_host, command_dict);
36 } 41 }
37 42
38 void ChromeDevToolsManagerDelegate::DevToolsAgentStateChanged( 43 void ChromeDevToolsManagerDelegate::DevToolsAgentStateChanged(
39 content::DevToolsAgentHost* agent_host, 44 content::DevToolsAgentHost* agent_host,
40 bool attached) { 45 bool attached) {
41 network_protocol_handler_->DevToolsAgentStateChanged(agent_host, attached); 46 network_protocol_handler_->DevToolsAgentStateChanged(agent_host, attached);
42 } 47 }
48
49 std::string ChromeDevToolsManagerDelegate::GetPageThumbnailData(
50 const GURL& url) {
51 for (chrome::BrowserIterator it; !it.done(); it.Next()) {
52 Profile* profile = (*it)->profile();
53 history::TopSites* top_sites = profile->GetTopSites();
54 if (!top_sites)
55 continue;
56 scoped_refptr<base::RefCountedMemory> data;
57 if (top_sites->GetPageThumbnail(url, false, &data))
58 return std::string(data->front_as<char>(), data->size());
59 }
60 return std::string();
61 }
62
63 scoped_ptr<content::DevToolsTarget>
64 ChromeDevToolsManagerDelegate::CreateNewTarget(const GURL& url) {
65 chrome::NavigateParams params(ProfileManager::GetLastUsedProfile(),
66 url, ui::PAGE_TRANSITION_AUTO_TOPLEVEL);
67 params.disposition = NEW_FOREGROUND_TAB;
68 chrome::Navigate(&params);
69 if (!params.target_contents)
70 return scoped_ptr<content::DevToolsTarget>();
71 return scoped_ptr<content::DevToolsTarget>(
72 DevToolsTargetImpl::CreateForWebContents(params.target_contents, true));
73 }
74
75 void ChromeDevToolsManagerDelegate::EnumerateTargets(TargetCallback callback) {
76 DevToolsTargetImpl::EnumerateAllTargets(
77 *reinterpret_cast<DevToolsTargetImpl::Callback*>(&callback));
78 }
79
OLDNEW
« no previous file with comments | « chrome/browser/devtools/chrome_devtools_manager_delegate.h ('k') | chrome/chrome_browser.gypi » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698