Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(570)

Unified Diff: chrome/renderer/page_click_tracker.cc

Issue 8351009: Updating PageClickTracker to Notify listeners When Text Input Loses Focus. (Closed) Base URL: http://git.chromium.org/git/chromium.git@trunk
Patch Set: Created 9 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/renderer/page_click_tracker.cc
diff --git a/chrome/renderer/page_click_tracker.cc b/chrome/renderer/page_click_tracker.cc
index 2bbb6d0c4deff16e5dff5f0cef33ef336fae73c2..c026a9ea4a78cd381243b4a5a2c2859964f7d6e6 100644
--- a/chrome/renderer/page_click_tracker.cc
+++ b/chrome/renderer/page_click_tracker.cc
@@ -79,8 +79,6 @@ void PageClickTracker::DidHandleMouseEvent(const WebMouseEvent& event) {
if (listener->InputElementClicked(input_element, was_focused_, is_focused))
break;
}
-
- last_node_clicked_.reset();
}
void PageClickTracker::AddListener(PageClickListener* listener) {
@@ -109,8 +107,6 @@ void PageClickTracker::FrameDetached(WebKit::WebFrame* frame) {
}
void PageClickTracker::handleEvent(const WebDOMEvent& event) {
- last_node_clicked_.reset();
-
if (!event.isMouseEvent())
return;
@@ -123,10 +119,28 @@ void PageClickTracker::handleEvent(const WebDOMEvent& event) {
// We'll get a notification once the mouse event has been processed
// (DidHandleMouseEvent), we'll notify the listener at that point.
WebNode node = mouse_event.target();
+
+ // The selected text field has lost focus.
+ if (!last_node_clicked_.isNull() && last_node_clicked_ != node) {
+ SelectedTextFieldLostFocus();
+ }
Ilya Sherman 2011/10/24 21:30:46 nit: No need for curly braces around a single line
+
// We are only interested in text field clicks.
- if (GetTextWebInputElement(node).isNull())
+ if (GetTextWebInputElement(node).isNull()) {
+ last_node_clicked_.reset();
+
return;
+ }
last_node_clicked_ = node;
was_focused_ = (render_view()->GetFocusedNode() == last_node_clicked_);
}
+
+void PageClickTracker::SelectedTextFieldLostFocus() {
+ ObserverListBase<PageClickListener>::Iterator it(listeners_);
+ PageClickListener* listener;
+ while ((listener = it.GetNext()) != NULL) {
+ if (listener->InputElementLostFocus())
+ break;
+ }
+}

Powered by Google App Engine
This is Rietveld 408576698