Add CWVUIDelegate with method to customize context menu.

ios/web_view/internal/cwv_web_view.mm

 // 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.
 
 #import "ios/web_view/public/cwv_web_view.h"
 
 #include <memory>
 #include <utility>
 
 #import "base/ios/weak_nsobject.h"
 #include "base/memory/ptr_util.h"
 #include "base/strings/sys_string_conversions.h"
 #import "ios/web/public/navigation_manager.h"
 #include "ios/web/public/referrer.h"
+#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/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_website_data_store_internal.h"
 #import "ios/web_view/internal/translate/web_view_translate_client.h"
 #include "ios/web_view/internal/web_view_browser_state.h"
+#import "ios/web_view/public/cwv_html_element.h"
+#import "ios/web_view/public/cwv_ui_delegate.h"
 #import "ios/web_view/public/cwv_web_view_configuration.h"
 #import "ios/web_view/public/cwv_web_view_delegate.h"
 #import "ios/web_view/public/cwv_website_data_store.h"
 #import "net/base/mac/url_conversions.h"
 #include "ui/base/page_transition_types.h"
 #include "url/gurl.h"
 
 #if !defined(__has_feature) || !__has_feature(objc_arc)
 #error "This file requires ARC support."
 #endif
 
 @interface CWVWebView ()<CRWWebStateDelegate, CRWWebStateObserver> {
   CWVWebViewConfiguration* _configuration;
   std::unique_ptr<web::WebState> _webState;
   std::unique_ptr<web::WebStateDelegateBridge> _webStateDelegate;
   std::unique_ptr<web::WebStateObserverBridge> _webStateObserver;
   CGFloat _loadProgress;
 }
 
 @end
 
 @implementation CWVWebView
 
 @synthesize delegate = _delegate;
 @synthesize loadProgress = _loadProgress;
+@synthesize UIDelegate = _UIDelegate;
 
 - (instancetype)initWithFrame:(CGRect)frame
                 configuration:(CWVWebViewConfiguration*)configuration {
   self = [super initWithFrame:frame];
   if (self) {
     _configuration = [configuration copy];
 
     web::WebState::CreateParams webStateCreateParams(
         [configuration.websiteDataStore browserState]);
     _webState = web::WebState::Create(webStateCreateParams);
@@ skipping 120 matching lines @@
         didFinishLoadingWithURL:[self visibleURL]
                     loadSuccess:success];
   }
 }
 
 - (void)webState:(web::WebState*)webState
     didChangeLoadingProgress:(double)progress {
   [self notifyDidUpdateWithChanges:CRIWVWebViewUpdateTypeProgress];
 }
 
+- (BOOL)webState:(web::WebState*)webState
+    handleContextMenu:(const web::ContextMenuParams&)params {
+  if ([_UIDelegate respondsToSelector:@selector

[Eugene But (OOO till 7-30), 2017/02/27 16:34:06]

Optional nit: do you want to reverse the condition and early return NO if
delegate does not respond to selector?

[michaeldo, 2017/02/27 19:02:05]

I agree, I like that better.

+                   (webView:runContextMenuWithTitle:forHTMLElement:inView

[Eugene But (OOO till 7-30), 2017/02/27 16:34:06]

Optional nit: Do you want to create a local variable for selector to improve the
formatting?

[michaeldo, 2017/02/27 19:02:05]

Done.

+                              :userGestureLocation:)]) {
+    NSURL* hyperlink = net::NSURLWithGURL(params.link_url);
+    NSURL* mediaSource = net::NSURLWithGURL(params.src_url);
+    CWVHTMLElement* htmlElement =

[Eugene But (OOO till 7-30), 2017/02/27 16:34:06]

s/htmlElement/HTMLElement

[michaeldo, 2017/02/27 19:02:06]

Done.

+        [[CWVHTMLElement alloc] initWithHyperlink:hyperlink
+                                      mediaSource:mediaSource
+                                             text:params.link_text];
+    [_UIDelegate webView:self
+        runContextMenuWithTitle:params.menu_title
+                 forHTMLElement:htmlElement
+                         inView:params.view
+            userGestureLocation:params.location];
+    return YES;
+  }
+  return NO;
+}
+
 @end