DevTools: migrate DevTools APIs to use WebContents instead of RenderViewHost.

chrome/browser/ui/webui/extensions/extension_error_ui_util.cc

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/ui/webui/extensions/extension_error_ui_util.h"
 
 #include <string>
 
 #include "base/bind.h"
 #include "base/file_util.h"
@@ skipping 139 matching lines @@
   }
 
   content::RenderViewHost* rvh =
       content::RenderViewHost::FromID(render_process_id, render_view_id);
 
   // It's possible that the render view was closed since we last updated the
   // links. Handle this gracefully.
   if (!rvh)
     return;
 
+  // Once we open the inspector, we focus on the appropriate tab...
+  content::WebContents* web_contents =
+      content::WebContents::FromRenderViewHost(rvh);
+  if (!web_contents)
+    return;
+
   // If we include a url, we should inspect it specifically (and not just the
   // render view).
   base::string16 url;
   if (args->GetString(RuntimeError::kUrlKey, &url)) {
     // Line and column numbers are optional; default to the first line.
     int line_number = 1;
     int column_number = 1;
     args->GetInteger(RuntimeError::kLineNumberKey, &line_number);
     args->GetInteger(RuntimeError::kColumnNumberKey, &column_number);
 
     // Line/column numbers are reported in display-friendly 1-based numbers,
     // but are inspected in zero-based numbers.
     DevToolsWindow::OpenDevToolsWindow(
-        rvh,
+        web_contents,
         DevToolsToggleAction::Reveal(url, line_number - 1, column_number - 1));
   } else {
-    DevToolsWindow::OpenDevToolsWindow(rvh);
+    DevToolsWindow::OpenDevToolsWindow(web_contents);
   }
 
-  // Once we open the inspector, we focus on the appropriate tab...
-  content::WebContents* web_contents =
-      content::WebContents::FromRenderViewHost(rvh);
   Browser* browser = chrome::FindBrowserWithWebContents(web_contents);
 
   // ... but some pages (popups and apps) don't have tabs, and some (background
   // pages) don't have an associated browser. For these, the inspector opens in
   // a new window, and our work is done.
   if (!browser || !browser->is_type_tabbed())
     return;
 
   TabStripModel* tab_strip = browser->tab_strip_model();
   tab_strip->ActivateTabAt(tab_strip->GetIndexOfWebContents(web_contents),
                            false);  // Not through direct user gesture.
 }
 
 }  // namespace error_ui_util
 }  // namespace extensions