Hook up inspector highlighting again. This time we use...

webkit/glue/inspector_client_impl.cc

 // Copyright (c) 2006-2008 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 "config.h"
 
 #include "base/compiler_specific.h"
 
 MSVC_PUSH_WARNING_LEVEL(0);
 #include "DOMWindow.h"
@@ skipping 87 matching lines @@
   // Notify the webview delegate of how many resources we're inspecting.
   WebViewDelegate* d = inspected_web_view_->delegate();
   DCHECK(d);
   d->WebInspectorOpened(inspector->resources().size());
 }
 
 void WebInspectorClient::closeWindow() {
   inspector_web_view_ = NULL;
   WebFrameImpl* frame = inspected_web_view_->main_frame();
 
-  if (frame && frame->inspected_node())
+  if (inspected_node_)
     hideHighlight();
 
   if (inspected_web_view_->page())
     inspected_web_view_->page()->inspectorController()->setWindowVisible(false);
 }
 
 bool WebInspectorClient::windowVisible() {
   if (inspector_web_view_ != NULL) {
     Page* page = inspector_web_view_->page();
     ASSERT(page != NULL);
@@ skipping 19 matching lines @@
 static void invalidateNodeBoundingRect(WebViewImpl* web_view) {
   // TODO(ojan): http://b/1143996 Is it important to just invalidate the rect
   // of the node region given that this is not on a critical codepath?
   // In order to do so, we'd have to take scrolling into account.
   gfx::Size size = web_view->size();
   gfx::Rect damaged_rect(0, 0, size.width(), size.height());
   web_view->GetDelegate()->DidInvalidateRect(web_view, damaged_rect);
 }
 
 void WebInspectorClient::highlight(Node* node) {
-  WebFrameImpl* frame = inspected_web_view_->main_frame();
-
-  if (frame->inspected_node())
+  if (inspected_node_)
     hideHighlight();
 
+  inspected_node_ = node;
   invalidateNodeBoundingRect(inspected_web_view_);
-  frame->selectNodeFromInspector(node);
 }
 
 void WebInspectorClient::hideHighlight() {
-  WebFrameImpl* frame = static_cast<WebFrameImpl*>(inspected_web_view_->GetMainFrame());
-
+  inspected_node_ = 0;
   invalidateNodeBoundingRect(inspected_web_view_);
-  frame->selectNodeFromInspector(NULL);
 }
 
 void WebInspectorClient::inspectedURLChanged(const String& newURL) {
   // TODO(jackson): Implement this
 }
 
 String WebInspectorClient::localizedStringsURL() {
   NOTIMPLEMENTED();
   return String();
 }
 
 String WebInspectorClient::hiddenPanels() {
   NOTIMPLEMENTED();
   return String();
 }
 
 void WebInspectorClient::populateSetting(
     const String& key, InspectorController::Setting&) {
   NOTIMPLEMENTED();
 }
 
 void WebInspectorClient::storeSetting(
     const String& key, const InspectorController::Setting&) {
   NOTIMPLEMENTED();
 }
 
 void WebInspectorClient::removeSetting(const String& key) {
   NOTIMPLEMENTED();
 }