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

chrome/browser/extensions/api/app_window/app_window_api.cc

 // Copyright (c) 2012 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/extensions/api/app_window/app_window_api.h"
 
 #include "apps/app_window.h"
 #include "apps/app_window_contents.h"
 #include "apps/app_window_registry.h"
 #include "apps/ui/apps_client.h"
@@ skipping 55 matching lines @@
 // AppWindowCreateFunction until the DevToolsWindow has finished loading, and is
 // ready to stop on breakpoints in the callback.
 class DevToolsRestorer : public base::RefCounted<DevToolsRestorer> {
  public:
   DevToolsRestorer(AppWindowCreateFunction* delayed_create_function,
                    content::RenderViewHost* created_view)
       : delayed_create_function_(delayed_create_function) {
     AddRef();  // Balanced in LoadCompleted.
     DevToolsWindow* devtools_window =
         DevToolsWindow::OpenDevToolsWindow(
-            created_view,
+            content::WebContents::FromRenderViewHost(created_view),
             DevToolsToggleAction::ShowConsole());
     devtools_window->SetLoadCompletedCallback(
         base::Bind(&DevToolsRestorer::LoadCompleted, this));
   }
 
  private:
   friend class base::RefCounted<DevToolsRestorer>;
   ~DevToolsRestorer() {}
 
   void LoadCompleted() {
@@ skipping 213 matching lines @@
   base::DictionaryValue* result = new base::DictionaryValue;
   result->Set("viewId", new base::FundamentalValue(view_id));
   result->Set("injectTitlebar",
       new base::FundamentalValue(inject_html_titlebar_));
   result->Set("id", new base::StringValue(app_window->window_key()));
   app_window->GetSerializedState(result);
   SetResult(result);
 
   if (apps::AppWindowRegistry::Get(browser_context())
           ->HadDevToolsAttached(created_view)) {
     new DevToolsRestorer(this, created_view);

[dgozman, 2014/08/06 13:06:09]

Passing WebContents here would be more clear.

[pfeldman, 2014/08/06 16:26:35]

Done.

     return true;
   }
 
   SendResponse(true);
   app_window->WindowEventsReady();
 
   return true;
 }
 
 bool AppWindowCreateFunction::GetBoundsSpec(
@@ skipping 171 matching lines @@
 
   if (options.frame->as_frame_options->inactive_color.get()) {
     error_ = app_window_constants::kInactiveColorWithoutColor;
     return false;
   }
 
   return true;
 }
 
 }  // namespace extensions