Remove rendundant WebLocalFrame parameter in various plugin code.

third_party/WebKit/public/web/WebFrameClient.h

 /*
  * Copyright (C) 2011, 2012 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 99 matching lines @@
 struct WebRect;
 struct WebURLError;
 
 class BLINK_EXPORT WebFrameClient {
  public:
   virtual ~WebFrameClient() {}
 
   // Factory methods -----------------------------------------------------
 
   // May return null.
-  virtual WebPlugin* CreatePlugin(WebLocalFrame*, const WebPluginParams&) {
-    return 0;
-  }
+  virtual WebPlugin* CreatePlugin(const WebPluginParams&) { return nullptr; }
 
   // May return null.
   // WebContentDecryptionModule* may be null if one has not yet been set.
   virtual WebMediaPlayer* CreateMediaPlayer(const WebMediaPlayerSource&,
                                             WebMediaPlayerClient*,
                                             WebMediaPlayerEncryptedMediaClient*,
                                             WebContentDecryptionModule*,
                                             const WebString& sink_id) {
-    return 0;
+    return nullptr;

[jochen (gone - plz use gerrit), 2017/05/05 14:32:40]

nit. adding many but unrelated style fixes like this makes reviewing such CLs a
bit hard :/

[Lei Zhang, 2017/05/05 17:18:29]

I'll try to make a separate cleanup pass next time.

   }
 
   // May return null.
-  virtual WebMediaSession* CreateMediaSession() { return 0; }
+  virtual WebMediaSession* CreateMediaSession() { return nullptr; }
 
   // May return null.
   virtual std::unique_ptr<WebApplicationCacheHost> CreateApplicationCacheHost(
       WebApplicationCacheHostClient*) {
     return nullptr;
   }
 
   // May return null.
   virtual std::unique_ptr<WebServiceWorkerProvider>
   CreateServiceWorkerProvider() {
     return nullptr;
   }
 
   // May return null.
   virtual WebWorkerContentSettingsClientProxy*
   CreateWorkerContentSettingsClientProxy() {
-    return 0;
+    return nullptr;
   }
 
   // Returns a new WebWorkerFetchContext for a dedicated worker. Ownership of
   // the returned object is transferred to the caller. This is used only when
   // off-main-thread-fetch is enabled.
   virtual std::unique_ptr<WebWorkerFetchContext> CreateWorkerFetchContext() {
     return nullptr;
   }
 
   // Create a new WebPopupMenu. In the "createExternalPopupMenu" form, the
   // client is responsible for rendering the contents of the popup menu.
   virtual WebExternalPopupMenu* CreateExternalPopupMenu(
       const WebPopupMenuInfo&,
       WebExternalPopupMenuClient*) {
-    return 0;
+    return nullptr;
   }
 
   // Services ------------------------------------------------------------
 
   // A frame specific cookie jar.  May return null, in which case
   // WebKitPlatformSupport::cookieJar() will be called to access cookies.
-  virtual WebCookieJar* CookieJar() { return 0; }
+  virtual WebCookieJar* CookieJar() { return nullptr; }
 
   // Returns a blame context for attributing work belonging to this frame.
   virtual BlameContext* GetFrameBlameContext() { return nullptr; }
 
   // General notifications -----------------------------------------------
 
   // Indicates if creating a plugin without an associated renderer is supported.
   virtual bool CanCreatePluginWithoutRenderer(const WebString& mime_type) {
     return false;
   }
@@ skipping 130 matching lines @@
     WebNavigationPolicy default_policy;
     bool replaces_current_history_item;
     bool is_history_navigation_in_new_child_frame;
     bool is_client_redirect;
     WebFormElement form;
     bool is_cache_disabled;
     WebSourceLocation source_location;
     WebContentSecurityPolicyDisposition
         should_check_main_world_content_security_policy;
 
-    NavigationPolicyInfo(WebURLRequest& url_request)
+    explicit NavigationPolicyInfo(WebURLRequest& url_request)
         : extra_data(nullptr),
           url_request(url_request),
           navigation_type(kWebNavigationTypeOther),
           default_policy(kWebNavigationPolicyIgnore),
           replaces_current_history_item(false),
           is_history_navigation_in_new_child_frame(false),
           is_client_redirect(false),
           is_cache_disabled(false),
           should_check_main_world_content_security_policy(
               kWebContentSecurityPolicyDispositionCheck) {}
@@ skipping 137 matching lines @@
     return false;
   }
 
   // PlzNavigate
   // Called to abort a navigation that is being handled by the browser process.
   virtual void AbortClientNavigation() {}
 
   // Push API ---------------------------------------------------
 
   // Used to access the embedder for the Push API.
-  virtual WebPushClient* PushClient() { return 0; }
+  virtual WebPushClient* PushClient() { return nullptr; }
 
   // Presentation API ----------------------------------------------------
 
   // Used to access the embedder for the Presentation API.
-  virtual WebPresentationClient* PresentationClient() { return 0; }
+  virtual WebPresentationClient* PresentationClient() { return nullptr; }
 
   // InstalledApp API ----------------------------------------------------
 
   // Used to access the embedder for the InstalledApp API.
   virtual WebRelatedAppsFetcher* GetRelatedAppsFetcher() { return nullptr; }
 
   // Editing -------------------------------------------------------------
 
   // These methods allow the client to intercept and overrule editing
   // operations.
@@ skipping 13 matching lines @@
   // This method opens the color chooser and returns a new WebColorChooser
   // instance. If there is a WebColorChooser already from the last time this
   // was called, it ends the color chooser by calling endChooser, and replaces
   // it with the new one. The given list of suggestions can be used to show a
   // simple interface with a limited set of choices.
 
   virtual WebColorChooser* CreateColorChooser(
       WebColorChooserClient*,
       const WebColor&,
       const WebVector<WebColorSuggestion>&) {
-    return 0;
+    return nullptr;
   }
 
   // Displays a modal alert dialog containing the given message. Returns
   // once the user dismisses the dialog.
   virtual void RunModalAlertDialog(const WebString& message) {}
 
   // Displays a modal confirmation dialog with the given message as
   // description and OK/Cancel choices. Returns true if the user selects
   // 'OK' or false otherwise.
   virtual bool RunModalConfirmDialog(const WebString& message) { return false; }
@@ skipping 131 matching lines @@
   virtual void RequestStorageQuota(WebStorageQuotaType,
                                    unsigned long long new_quota_in_bytes,
                                    WebStorageQuotaCallbacks) {}
 
   // MediaStream -----------------------------------------------------
 
   // A new WebRTCPeerConnectionHandler is created.
   virtual void WillStartUsingPeerConnectionHandler(
       WebRTCPeerConnectionHandler*) {}
 
-  virtual WebUserMediaClient* UserMediaClient() { return 0; }
+  virtual WebUserMediaClient* UserMediaClient() { return nullptr; }
 
   // Encrypted Media -------------------------------------------------
 
-  virtual WebEncryptedMediaClient* EncryptedMediaClient() { return 0; }
+  virtual WebEncryptedMediaClient* EncryptedMediaClient() { return nullptr; }
 
   // User agent ------------------------------------------------------
 
   // Asks the embedder if a specific user agent should be used. Non-empty
   // strings indicate an override should be used. Otherwise,
   // Platform::current()->userAgent() will be called to provide one.
   virtual WebString UserAgentOverride() { return WebString(); }
 
   // Do not track ----------------------------------------------------
 
   // Asks the embedder what value the network stack will send for the DNT
   // header. An empty string indicates that no DNT header will be send.
   virtual WebString DoNotTrackValue() { return WebString(); }
 
   // WebGL ------------------------------------------------------
 
   // Asks the embedder whether WebGL is allowed for the WebFrame. This call is
   // placed here instead of WebContentSettingsClient because this class is
   // implemented in content/, and putting it here avoids adding more public
   // content/ APIs.
   virtual bool AllowWebGL(bool default_value) { return default_value; }
 
   // Screen Orientation --------------------------------------------------
 
   // Access the embedder API for (client-based) screen orientation client .
   virtual WebScreenOrientationClient* GetWebScreenOrientationClient() {
-    return 0;
+    return nullptr;
   }
 
   // Accessibility -------------------------------------------------------
 
   // Notifies embedder about an accessibility event.
   virtual void PostAccessibilityEvent(const WebAXObject&, WebAXEvent) {}
 
   // Provides accessibility information about a find in page result.
   virtual void HandleAccessibilityFindInPageResult(
       int identifier,
@@ skipping 68 matching lines @@
   // Overwrites the given URL to use an HTML5 embed if possible.
   // An empty URL is returned if the URL is not overriden.
   virtual WebURL OverrideFlashEmbedWithHTML(const WebURL& url) {
     return WebURL();
   }
 };
 
 }  // namespace blink
 
 #endif