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

Unified Diff: chrome/browser/devtools/browser_list_tabcontents_provider.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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/devtools/browser_list_tabcontents_provider.cc
diff --git a/chrome/browser/devtools/browser_list_tabcontents_provider.cc b/chrome/browser/devtools/browser_list_tabcontents_provider.cc
index 5d98f6ccc0ca43e86b76c4ee7b583622f66b31ec..a4d18b3faaae939a6436c8bb145887d11fd6da44 100644
--- a/chrome/browser/devtools/browser_list_tabcontents_provider.cc
+++ b/chrome/browser/devtools/browser_list_tabcontents_provider.cc
@@ -119,11 +119,8 @@ BrowserListTabContentsProvider::CreateNewTarget(const GURL& url) {
url,
content::PAGE_TRANSITION_LINK);
}
- content::RenderViewHost* rvh = web_contents->GetRenderViewHost();
- if (!rvh)
- return scoped_ptr<DevToolsTarget>();
return scoped_ptr<DevToolsTarget>(
- DevToolsTargetImpl::CreateForRenderViewHost(rvh, true));
+ DevToolsTargetImpl::CreateForWebContents(web_contents, true));
}
void BrowserListTabContentsProvider::EnumerateTargets(TargetCallback callback) {

Powered by Google App Engine
This is Rietveld 408576698