Fix a bug where the inspector would not keep event pages alive.

chrome/browser/debugger/devtools_window.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 <algorithm>
 
 #include "base/command_line.h"
 #include "base/json/json_writer.h"
 #include "base/lazy_instance.h"
 #include "base/string_number_conversions.h"
@@ skipping 625 matching lines @@
     window = Create(profile, inspected_rvh, docked, false);
     manager->RegisterDevToolsClientHostFor(agent, window->frontend_host_);
     do_open = true;
   }
 
   // Update toolbar to reflect DevTools changes.
   window->UpdateBrowserToolbar();
 
   // If window is docked and visible, we hide it on toggle. If window is
   // undocked, we show (activate) it.
-  if (!window->is_docked() || do_open)
+  if (!window->is_docked() || do_open) {

[yurys, 2012/08/21 19:52:55]

Revert this? As I remember style guide doesn't require braces for one line body.

[Matt Perry, 2012/08/21 20:13:46]

Oops, leftover change. Reverted.

     window->Show(action);
-  else
+  } else {
     manager->UnregisterDevToolsClientHostFor(agent);
+  }
 
   return window;
 }
 
 // static
 DevToolsWindow* DevToolsWindow::AsDevToolsWindow(
     DevToolsClientHost* client_host) {
   if (!client_host || g_instances == NULL)
     return NULL;
   DevToolsWindowList& instances = g_instances.Get();
@@ skipping 105 matching lines @@
   FileSelectHelper::RunFileChooser(web_contents, params);
 }
 
 void DevToolsWindow::UpdateBrowserToolbar() {
   if (!inspected_tab_)
     return;
   BrowserWindow* inspected_window = GetInspectedBrowserWindow();
   if (inspected_window)
     inspected_window->UpdateToolbar(inspected_tab_, false);
 }