Add some missing parameter names to Background Fetch code

TBR=rayankans, delphick

Change-Id: I4cd89ee0fbc89efffadc545bf32fcafac0c77636
Reviewed-on: https://chromium-review.googlesource.com/1113312Reviewed-by: Peter Beverloo <peter@chromium.org>
Reviewed-by: Dan Elphick <delphick@chromium.org>
Commit-Queue: Peter Beverloo <peter@chromium.org>
Cr-Commit-Position: refs/heads/master@{#570011}

third_party/blink/renderer/modules/background_fetch/background_fetch_icon_loader.h

@@ -39,7 +39,7 @@ class MODULES_EXPORT BackgroundFetchIconLoader final
   // data, and passes the bitmap to the given callback.
   void Start(BackgroundFetchBridge* bridge,
              ExecutionContext* execution_context,
-             HeapVector<ManifestImageResource>,
+             HeapVector<ManifestImageResource> icons,
              IconCallback callback);
 
   // Cancels the pending load, if there is one. The |icon_callback_| will not

third_party/blink/renderer/modules/background_fetch/background_fetch_settled_event.h

@@ -40,7 +40,7 @@ class MODULES_EXPORT BackgroundFetchSettledEvent : public BackgroundFetchEvent {
   // Web Exposed attribute defined in the IDL file.
   BackgroundFetchSettledFetches* fetches() const;
 
-  void Trace(blink::Visitor*) override;
+  void Trace(blink::Visitor* visitor) override;
 
  protected:
   BackgroundFetchSettledEvent(