SharedWorker: Remove unnecessary indirection in SharedWorkerMessageFilter

content/browser/shared_worker/shared_worker_service_impl.h

 // 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.
 
 #ifndef CONTENT_BROWSER_SHARED_WORKER_SHARED_WORKER_SERVICE_IMPL_H_
 #define CONTENT_BROWSER_SHARED_WORKER_SHARED_WORKER_SERVICE_IMPL_H_
 
 #include <map>
 #include <memory>
 #include <set>
@@ skipping 38 matching lines @@
   void AddObserver(WorkerServiceObserver* observer) override;
   void RemoveObserver(WorkerServiceObserver* observer) override;
 
   // These methods correspond to worker related IPCs.
   blink::WebWorkerCreationError CreateWorker(
       const ViewHostMsg_CreateWorker_Params& params,
       int route_id,
       SharedWorkerMessageFilter* filter,
       ResourceContext* resource_context,
       const WorkerStoragePartitionId& partition_id);
-  void ConnectToWorker(int route_id,
-                       int sent_message_port_id,
-                       SharedWorkerMessageFilter* filter);
-  void DocumentDetached(unsigned long long document_id,
-                        SharedWorkerMessageFilter* filter);
-  void WorkerContextClosed(int worker_route_id,
-                           SharedWorkerMessageFilter* filter);
-  void WorkerContextDestroyed(int worker_route_id,
-                              SharedWorkerMessageFilter* filter);
-  void WorkerReadyForInspection(int worker_route_id,
-                                SharedWorkerMessageFilter* filter);
-  void WorkerScriptLoaded(int worker_route_id,
-                          SharedWorkerMessageFilter* filter);
-  void WorkerScriptLoadFailed(int worker_route_id,
-                              SharedWorkerMessageFilter* filter);
-  void WorkerConnected(int message_port_id,
+  void ConnectToWorker(SharedWorkerMessageFilter* filter,
+                       int route_id,
+                       int sent_message_port_id);
+  void DocumentDetached(SharedWorkerMessageFilter* filter,
+                        unsigned long long document_id);
+  void WorkerContextClosed(SharedWorkerMessageFilter* filter,
+                           int worker_route_id);
+  void WorkerContextDestroyed(SharedWorkerMessageFilter* filter,
+                              int worker_route_id);
+  void WorkerReadyForInspection(SharedWorkerMessageFilter* filter,
+                                int worker_route_id);
+  void WorkerScriptLoaded(SharedWorkerMessageFilter* filter,
+                          int worker_route_id);
+  void WorkerScriptLoadFailed(SharedWorkerMessageFilter* filter,
+                              int worker_route_id);
+  void WorkerConnected(SharedWorkerMessageFilter* filter,
+                       int message_port_id,
+                       int worker_route_id);
+  void AllowFileSystem(SharedWorkerMessageFilter* filter,
                        int worker_route_id,
-                       SharedWorkerMessageFilter* filter);
-  void AllowFileSystem(int worker_route_id,
                        const GURL& url,
-                       IPC::Message* reply_msg,
-                       SharedWorkerMessageFilter* filter);
-  void AllowIndexedDB(int worker_route_id,
+                       IPC::Message* reply_msg);
+  void AllowIndexedDB(SharedWorkerMessageFilter* filter,
+                      int worker_route_id,
                       const GURL& url,
                       const base::string16& name,
-                      bool* result,
-                      SharedWorkerMessageFilter* filter);
+                      bool* result);
 
   void OnSharedWorkerMessageFilterClosing(
       SharedWorkerMessageFilter* filter);
 
   // Removes the references to shared workers from all the documents in the
   // renderer frame. And shuts down any shared workers that are no longer
   // referenced by active documents.
   void RenderFrameDetached(int render_process_id, int render_frame_id);
 
   // Checks the worker dependency of renderer processes and calls
@@ skipping 77 matching lines @@
   int next_pending_instance_id_;
 
   base::ObserverList<WorkerServiceObserver> observers_;
 
   DISALLOW_COPY_AND_ASSIGN(SharedWorkerServiceImpl);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_BROWSER_SHARED_WORKER_SHARED_WORKER_SERVICE_IMPL_H_