Reuse context menu in StaticHTMLViewController

ios/chrome/browser/ui/context_menu/context_menu_delegate.h

Index: ios/chrome/browser/ui/context_menu/context_menu_delegate.h
diff --git a/ios/chrome/browser/ui/context_menu/context_menu_delegate.h b/ios/chrome/browser/ui/context_menu/context_menu_delegate.h
new file mode 100644
index 0000000000000000000000000000000000000000..f76646b62811a9f169bded48bd8baa77488d2e40
--- /dev/null
+++ b/ios/chrome/browser/ui/context_menu/context_menu_delegate.h
@@ -0,0 +1,20 @@
+// Copyright 2017 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 IOS_CHROME_BROWSER_UI_CONTEXT_MENU_CONTEXT_MENU_DELEGATE_H_
+#define IOS_CHROME_BROWSER_UI_CONTEXT_MENU_CONTEXT_MENU_DELEGATE_H_
+
+namespace web {
+class WebState;
+struct ContextMenuParams;
+}
+
+// A delegate that will display a context menu for |params|. User choice on this
+// menu can trigger a navigation on |webState|.
+@protocol ContextMenuDelegate<NSObject>
+- (BOOL)webState:(web::WebState*)webState

[Eugene But (OOO till 7-30), 2017/01/13 07:16:26]

First argument in Objective-C delegate method is always a caller, which is not
really the case here. Should this delegate be like this?:

@protocol OfflinePageNativeContentDelegate
- (BOOL)dfflinePageNativeContent:(OfflinePageNativeContent*)nativeContent
                     handleContextMenu:(const web::ContextMenuParams&)params;
@end

I know that Chrome for iOS code often reuses same delegate protocol for
different classes, but that approach have a few issues:
 - it does not follow Coding Guidelines for Cocoa (which are basis for Google
Style Guide)
 - it increases complexity, because it is harder to reason about the caller

[Olivier, 2017/01/13 10:20:25]

Done.
I used the mode generic "native content" instead of OfflinePageNativeContent as
I don't see any reason to limit to OfflinePageNativeContentDelegate.
There is already a nativeContentDelegate, so I added my method there.
I can change that if needed.

+    handleContextMenu:(const web::ContextMenuParams&)params;
+@end
+
+#endif  // IOS_CHROME_BROWSER_UI_CONTEXT_MENU_CONTEXT_MENU_DELEGATE_H_