Updating PageClickTracker to Notify listeners When Text Input Loses Focus.

chrome/renderer/page_click_tracker.h

 // Copyright (c) 2010 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.
 
 #ifndef CHROME_RENDERER_PAGE_CLICK_TRACKER_H_
 #define CHROME_RENDERER_PAGE_CLICK_TRACKER_H_
 
 #include <vector>
 
 #include "base/basictypes.h"
@@ skipping 32 matching lines @@
 
  private:
   // RenderView::Observer implementation.
   virtual void DidFinishDocumentLoad(WebKit::WebFrame* frame) OVERRIDE;
   virtual void FrameDetached(WebKit::WebFrame* frame) OVERRIDE;
   virtual void DidHandleMouseEvent(const WebKit::WebMouseEvent& event) OVERRIDE;
 
   // WebKit::WebDOMEventListener implementation.
   virtual void handleEvent(const WebKit::WebDOMEvent& event);
 
+  // Checks to see if a text field is losing focus and inform listeners if
+  // it is.
+  void HandleTextFieldLosingFocus(const WebKit::WebNode& newly_clicked_node);

[Ilya Sherman, 2011/10/31 21:23:12]

nit: "Handle" implies that the event being handled definitely happened.  That
is, in this case, the method name suggests that it will only be called when a
text field is losing focus.  Hence, I would prefer something like
"CheckForTextFieldLosingFocus", or "HandleTextFieldMaybeLosingFocus".  

[csharp, 2011/11/01 14:49:01]

Done.

+
   // The last node that was clicked and had focus.
   WebKit::WebNode last_node_clicked_;
 
   // Whether the last clicked node had focused before it was clicked.
   bool was_focused_;
 
   // The frames we are listening to for mouse events.
   typedef std::vector<WebKit::WebFrame*> FrameList;
   FrameList tracked_frames_;
 
   // The listener getting the actual notifications.
   ObserverList<PageClickListener> listeners_;
 
   DISALLOW_COPY_AND_ASSIGN(PageClickTracker);
 };
 
 #endif  // CHROME_RENDERER_PAGE_CLICK_TRACKER_H_