Pass TouchMajor to HitTestResult

content/renderer/render_view_impl.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/renderer/render_view_impl.h"
 
 #include <algorithm>
 #include <cmath>
 
 #include "base/auto_reset.h"
@@ skipping 2532 matching lines @@
       }
     }
   }
 
   return false;
 }
 
 bool RenderViewImpl::NodeContainsPoint(const WebNode& node,
                                        const gfx::Point& point) const {
   blink::WebHitTestResult hit_test =
-      webview()->hitTestResultAt(WebPoint(point.x(), point.y()));
+      webview()->hitTestResultForTap(WebPoint(point.x(), point.y()), WebSize());

[jamesr, 2014/11/08 00:21:08]

this is used for autofill and exposed in the content public API so may be used
for other things. is this a behavior change?

[hush (inactive), 2014/11/08 01:00:43]

This is not a behavior change.
hitTestResultAt does single point hit testing.
hitTestResultForTap does rect-based hit testing. But I'm passing a 0x0 tap area
here. So it is the same as single point hit testing.

   return node.containsIncludingShadowDOM(hit_test.node());
 }
 
 bool RenderViewImpl::ShouldDisplayScrollbars(int width, int height) const {
   return (!send_preferred_size_changes_ ||
           (disable_scrollbars_size_limit_.width() <= width ||
            disable_scrollbars_size_limit_.height() <= height));
 }
 
 int RenderViewImpl::GetEnabledBindings() const {
@@ skipping 1648 matching lines @@
     std::vector<gfx::Size> sizes;
     ConvertToFaviconSizes(icon_urls[i].sizes(), &sizes);
     if (!url.isEmpty())
       urls.push_back(
           FaviconURL(url, ToFaviconType(icon_urls[i].iconType()), sizes));
   }
   SendUpdateFaviconURL(urls);
 }
 
 }  // namespace content