Ensure that the OSK does not show up at times when you click on editable fields with the mouse.

content/browser/renderer_host/render_widget_host_view_win.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 "content/browser/renderer_host/render_widget_host_view_win.h"
 
 #include <algorithm>
 #include <map>
 #include <peninputpanel_i.c>
 #include <stack>
@@ skipping 1827 matching lines @@
           parent_msg_lparam = MAKELPARAM(cursor_pos.x, cursor_pos.y);
         }
         if (SendMessage(GetParent(), message, wparam, parent_msg_lparam) != 0) {
           TRACE_EVENT0("browser", "EarlyOut_SentToParent");
           return 1;
         }
       }
     }
   }
 
-  if (message == WM_LBUTTONDOWN && GetBrowserAccessibilityManager())
+  if (message == WM_LBUTTONDOWN && pointer_down_context_ &&
+      GetBrowserAccessibilityManager())
     GetBrowserAccessibilityManager()->GotMouseDown();
 
   if (message == WM_LBUTTONUP && ui::IsMouseEventFromTouch(message) &&
       base::win::IsMetroProcess())
     pointer_down_context_ = false;
 
   ForwardMouseEventToRenderer(message, wparam, lparam);
   return 0;
 }
 
@@ skipping 345 matching lines @@
   size_t total = std::min(static_cast<int>(LOWORD(wparam)),
       static_cast<int>(WebKit::WebTouchEvent::touchesLengthCap));
   TOUCHINPUT points[WebKit::WebTouchEvent::touchesLengthCap];
 
   if (!total || !GetTouchInputInfo((HTOUCHINPUT)lparam, total,
                                    points, sizeof(TOUCHINPUT))) {
     TRACE_EVENT0("browser", "EarlyOut_NothingToDo");
     return 0;
   }
 
+  if (total == 1) {

[sky, 2012/09/20 20:08:14]

combine into a single if.

[ananta, 2012/09/20 22:00:11]

Done.

+    if (points[0].dwFlags & TOUCHEVENTF_DOWN)
+      pointer_down_context_ = true;
+  }
+
   bool has_touch_handler = render_widget_host_->has_touch_handler() &&
       touch_events_enabled_;
 
   // Send a copy of the touch events on to the gesture recognizer.
   for (size_t start = 0; start < total;) {
     start += touch_state_->UpdateTouchPoints(points + start, total - start);
     if (has_touch_handler) {
       if (touch_state_->is_changed()) {
         render_widget_host_->ForwardTouchEvent(touch_state_->touch_event());
         touch_state_->QueueEvents(this, gesture_recognizer_.get());
@@ skipping 459 matching lines @@
   TRACE_EVENT0("browser", "RenderWidgetHostViewWin::OnPointerMessage");
   POINT point = {0};
 
   point.x = GET_X_LPARAM(lparam);
   point.y = GET_Y_LPARAM(lparam);
   ScreenToClient(&point);
 
   lparam = MAKELPARAM(point.x, point.y);
 
   if (message == WM_POINTERDOWN) {
-    pointer_down_context_ = true;
     if (!base::win::IsMetroProcess()) {
+      pointer_down_context_ = true;

[sky, 2012/09/20 20:08:14]

One other question, should we force to false in capture changed?

[ananta, 2012/09/20 22:00:11]

Good point. Done.

       SetFocus();
       received_focus_change_after_pointer_down_ = false;
       MessageLoop::current()->PostDelayedTask(FROM_HERE,
           base::Bind(&RenderWidgetHostViewWin::ResetPointerDownContext,
                       weak_factory_.GetWeakPtr()),
           base::TimeDelta::FromMilliseconds(kPointerDownContextResetDelay));
     }
   }
   handled = FALSE;
   return 0;
@@ skipping 397 matching lines @@
   // receive a focus change in the context of a pointer down message, it means
   // that the pointer down message occurred on the edit field and we should
   // display the on screen keyboard
   if (!received_focus_change_after_pointer_down_ && virtual_keyboard_)
     DisplayOnScreenKeyboardIfNeeded();
   received_focus_change_after_pointer_down_ = false;
   pointer_down_context_ = false;
 }
 
 }  // namespace content