Remove shell::Connection* parameter to InterfaceFactory<T>::Create()

services/ui/service.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 SERVICES_UI_SERVICE_H_
 #define SERVICES_UI_SERVICE_H_
 
 #include <stdint.h>
 
 #include <map>
@@ skipping 72 matching lines @@
  private:
   // Holds InterfaceRequests received before the first WindowTreeHost Display
   // has been established.
   struct PendingRequest;
   struct UserState;
 
   using UserIdToUserState = std::map<ws::UserId, std::unique_ptr<UserState>>;
 
   void InitializeResources(shell::Connector* connector);
 
-  // Returns the user specific state for the user id of |connection|. Service
-  // owns the return value.
+  // Returns the user specific state for the user id of |remote_identity|.
+  // Service owns the return value.
   // TODO(sky): if we allow removal of user ids then we need to close anything
   // associated with the user (all incoming pipes...) on removal.
-  UserState* GetUserState(shell::Connection* connection);
+  UserState* GetUserState(const shell::Identity& remote_identity);
 
-  void AddUserIfNecessary(shell::Connection* connection);
+  void AddUserIfNecessary(const shell::Identity& remote_identity);
 
   // shell::Service:
   void OnStart(shell::Connector* connector,
                const shell::Identity& identity,
                uint32_t id) override;
   bool OnConnect(shell::Connection* connection) override;
 
   // WindowServerDelegate:
   void OnFirstDisplayReady() override;
   void OnNoMoreDisplays() override;
   bool IsTestConfig() const override;
   void CreateDefaultDisplays() override;
 
   // shell::InterfaceFactory<mojom::AccessibilityManager> implementation.
-  void Create(shell::Connection* connection,
+  void Create(const shell::Identity& remote_identity,
               mojom::AccessibilityManagerRequest request) override;
 
   // shell::InterfaceFactory<mojom::Clipboard> implementation.
-  void Create(shell::Connection* connection,
+  void Create(const shell::Identity& remote_identity,
               mojom::ClipboardRequest request) override;
 
   // shell::InterfaceFactory<mojom::DisplayManager> implementation.
-  void Create(shell::Connection* connection,
+  void Create(const shell::Identity& remote_identity,
               mojom::DisplayManagerRequest request) override;
 
   // shell::InterfaceFactory<mojom::Gpu> implementation.
-  void Create(shell::Connection* connection,
+  void Create(const shell::Identity& remote_identity,
               mojom::GpuRequest request) override;
 
   // shell::InterfaceFactory<mojom::GpuService> implementation.
-  void Create(shell::Connection* connection,
+  void Create(const shell::Identity& remote_identity,
               mojom::GpuServiceRequest request) override;
 
   // shell::InterfaceFactory<mojom::UserAccessManager> implementation.
-  void Create(shell::Connection* connection,
+  void Create(const shell::Identity& remote_identity,
               mojom::UserAccessManagerRequest request) override;
 
   // shell::InterfaceFactory<mojom::UserActivityMonitor> implementation.
-  void Create(shell::Connection* connection,
+  void Create(const shell::Identity& remote_identity,
               mojom::UserActivityMonitorRequest request) override;
 
   // shell::InterfaceFactory<mojom::WindowManagerWindowTreeFactory>
   // implementation.
-  void Create(shell::Connection* connection,
+  void Create(const shell::Identity& remote_identity,
               mojom::WindowManagerWindowTreeFactoryRequest request) override;
 
   // shell::InterfaceFactory<mojom::WindowTreeFactory>:
-  void Create(shell::Connection* connection,
+  void Create(const shell::Identity& remote_identity,
               mojom::WindowTreeFactoryRequest request) override;
 
   // shell::InterfaceFactory<mojom::WindowTreeHostFactory>:
-  void Create(shell::Connection* connection,
+  void Create(const shell::Identity& remote_identity,
               mojom::WindowTreeHostFactoryRequest request) override;
 
   // shell::InterfaceFactory<mojom::WindowServerTest> implementation.
-  void Create(shell::Connection* connection,
+  void Create(const shell::Identity& remote_identity,
               mojom::WindowServerTestRequest request) override;
 
   // Callback for display configuration. |id| is the identifying token for the
   // configured display that will identify a specific physical display across
   // configuration changes. |bounds| is the bounds of the display in screen
   // coordinates.
   void OnCreatedPhysicalDisplay(int64_t id, const gfx::Rect& bounds);
 
   ws::PlatformDisplayInitParams platform_display_init_params_;
   std::unique_ptr<ws::WindowServer> window_server_;
@@ skipping 17 matching lines @@
   std::unique_ptr<ws::TouchController> touch_controller_;
 
   base::WeakPtrFactory<Service> weak_ptr_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(Service);
 };
 
 }  // namespace ui
 
 #endif  // SERVICES_UI_SERVICE_H_