Implement CWVWebView.scrollView.

ios/web_view/internal/cwv_web_view.mm

Index: ios/web_view/internal/cwv_web_view.mm
diff --git a/ios/web_view/internal/cwv_web_view.mm b/ios/web_view/internal/cwv_web_view.mm
index 503cadd517ccbf86bb37f8a44104ccd04d337d73..544a327639351c00888847a22c792175895515ac 100644
--- a/ios/web_view/internal/cwv_web_view.mm
+++ b/ios/web_view/internal/cwv_web_view.mm
@@ -17,11 +17,14 @@
 #import "ios/web/public/web_state/context_menu_params.h"
 #import "ios/web/public/web_state/js/crw_js_injection_receiver.h"
 #import "ios/web/public/web_state/ui/crw_web_delegate.h"
+#import "ios/web/public/web_state/ui/crw_web_view_proxy.h"
+#import "ios/web/public/web_state/ui/crw_web_view_scroll_view_proxy.h"
 #import "ios/web/public/web_state/web_state.h"
 #import "ios/web/public/web_state/web_state_delegate_bridge.h"
 #import "ios/web/public/web_state/web_state_observer_bridge.h"
 #import "ios/web_view/internal/cwv_html_element_internal.h"
 #import "ios/web_view/internal/cwv_navigation_action_internal.h"
+#import "ios/web_view/internal/cwv_scroll_view_internal.h"
 #import "ios/web_view/internal/cwv_web_view_configuration_internal.h"
 #import "ios/web_view/internal/translate/web_view_translate_client.h"
 #include "ios/web_view/internal/web_view_browser_state.h"
@@ -85,6 +88,7 @@ static NSString* gUserAgentProduct = nil;
 @synthesize translationDelegate = _translationDelegate;
 @synthesize estimatedProgress = _estimatedProgress;
 @synthesize UIDelegate = _UIDelegate;
+@synthesize scrollView = _scrollView;
 
 + (NSString*)userAgentProduct {
   return gUserAgentProduct;
@@ -114,6 +118,7 @@ static NSString* gUserAgentProduct = nil;
   self = [super initWithFrame:frame];
   if (self) {
     _configuration = [configuration copy];
+    _scrollView = [[CWVScrollView alloc] init];

[michaeldo, 2017/05/01 18:49:53]

This readonly property should be redefined above as writeable which I couldn't
find unless I'm missing it somewhere.

[Hiroshi Ichikawa, 2017/05/08 06:52:41]

Why? Here I directly assign to the underlying instance variable |_scrollView|,
not |self.scrollView|, so it doesn't require the property to be readwrite. And
it's assigned only in the initializer here.

[michaeldo, 2017/05/08 15:18:18]

This is a good point. You're right that we don't need to redefine it. Sorry, I
was under the impression we always do so, but I don't see a reason to change it
here.

     [self resetWebStateWithSessionStorage:nil];
   }
   return self;
@@ -341,6 +346,8 @@ static NSString* gUserAgentProduct = nil;
       base::MakeUnique<ios_web_view::WebViewJavaScriptDialogPresenter>(self,
                                                                        nullptr);
 
+  _scrollView.proxy = _webState.get()->GetWebViewProxy().scrollViewProxy;
+
   // Initialize Translate.
   ios_web_view::WebViewTranslateClient::CreateForWebState(_webState.get());