Pass TouchMajor to HitTestResult

android_webview/browser/renderer_host/aw_render_view_host_ext.h

 // 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.
 
 #ifndef ANDROID_WEBVIEW_BROWSER_RENDER_HOST_RENDER_VIEW_HOST_EXT_H_
 #define ANDROID_WEBVIEW_BROWSER_RENDER_HOST_RENDER_VIEW_HOST_EXT_H_
 
 #include "content/public/browser/web_contents_observer.h"
 
 #include "android_webview/common/aw_hit_test_data.h"
@@ skipping 37 matching lines @@
   // |result| will be invoked with the outcome of the request.
   typedef base::Callback<void(bool)> DocumentHasImagesResult;
   void DocumentHasImages(DocumentHasImagesResult result);
 
   // Clear all WebCore memory cache (not only for this view).
   void ClearCache();
 
   // Do a hit test at the view port coordinates and asynchronously update
   // |last_hit_test_data_|. |view_x| and |view_y| are in density independent
   // pixels used by blink::WebView.
-  void RequestNewHitTestDataAt(int view_x, int view_y);
+  void RequestNewHitTestDataAt(int view_x, int view_y, float touch_major);

[jdduke (slow), 2014/10/28 23:09:14]

Could we make this more generic and pass a RectF?

[jdduke (slow), 2014/10/29 02:15:17]

Actually, if this is WebView only I don't really care about the API as long as
it's clear and consistent. Either a rect, or a coordinate and width/height
should be fine.

[hush (inactive), 2014/10/30 00:25:19]

I will pass gfx::RectF like you suggested.

 
   // Optimization to avoid unnecessary Java object creation on hit test.
   bool HasNewHitTestData() const;
   void MarkHitTestDataRead();
 
   // Return |last_hit_test_data_|. Note that this is unavoidably racy;
   // the corresponding public WebView API is as well.
   const AwHitTestData& GetLastHitTestData() const;
 
   // Sets the zoom factor for text only. Used in layout modes other than
@@ skipping 37 matching lines @@
   AwHitTestData last_hit_test_data_;
 
   bool has_new_hit_test_data_;
 
   DISALLOW_COPY_AND_ASSIGN(AwRenderViewHostExt);
 };
 
 }  // namespace android_webview
 
 #endif  // ANDROID_WEBVIEW_BROWSER_RENDER_HOST_RENDER_VIEW_HOST_EXT_H_