android_webview: Let AwContents manage TouchHandleDrawable

android_webview/browser/browser_view_renderer.cc

 // Copyright 2014 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 "android_webview/browser/browser_view_renderer.h"
 
 #include <utility>
 
 #include "android_webview/browser/browser_view_renderer_client.h"
 #include "android_webview/browser/child_frame.h"
@@ skipping 682 matching lines @@
   gfx::Vector2d rounded_overscroll_delta = gfx::ToRoundedVector2d(
       scaled_overscroll_delta + overscroll_rounding_error_);
   overscroll_rounding_error_ =
       scaled_overscroll_delta - rounded_overscroll_delta;
   gfx::Vector2dF fling_velocity_pixels =
       gfx::ScaleVector2d(current_fling_velocity, physical_pixel_scale);
 
   client_->DidOverscroll(rounded_overscroll_delta, fling_velocity_pixels);
 }
 
+ui::TouchHandleDrawable* BrowserViewRenderer::CreateDrawable() {
+  CHECK_GT(dip_scale_, 0.f);

[boliu, 2016/08/24 23:35:30]

hmm, curious, does this really need to be a release check?

[Jinsuk Kim, 2016/08/25 07:32:21]

Nope, meant to use DCHECK_GT like in other places in this class.

+  return client_->CreateDrawable(dip_scale_);
+}
+
 void BrowserViewRenderer::PostInvalidate(
     content::SynchronousCompositor* compositor) {
   TRACE_EVENT_INSTANT0("android_webview", "BrowserViewRenderer::PostInvalidate",
                        TRACE_EVENT_SCOPE_THREAD);
   if (compositor != compositor_)
     return;
 
   client_->PostInvalidate();
 }
 
@@ skipping 19 matching lines @@
   base::StringAppendF(&str,
                       "overscroll_rounding_error_: %s ",
                       overscroll_rounding_error_.ToString().c_str());
   base::StringAppendF(
       &str, "on_new_picture_enable: %d ", on_new_picture_enable_);
   base::StringAppendF(&str, "clear_view: %d ", clear_view_);
   return str;
 }
 
 }  // namespace android_webview