Remove [Client=] annotation from ServiceProvider

mojo/services/window_manager/public/interfaces/window_manager.mojom

 // 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.
 
 module mojo;
 
 import "input_events/public/interfaces/input_events.mojom";
 import "mojo/public/interfaces/application/service_provider.mojom";
 
 [Client=WindowManagerClient]
 interface WindowManager {
   // Requests the WindowManager to embed the app for |url| at an appropriate
-  // View. See ViewMangerService::Embed() for details on |service_provider|.
-  Embed(string url, ServiceProvider& service_provider);
+  // View. See ViewMangerService::Embed() for details on |services| and |exposed_services|.

[sky, 2015/01/21 16:59:00]

nit: > 80

[jamesr, 2015/01/21 18:51:40]

Done.

+  Embed(string url, ServiceProvider&? services, ServiceProvider? exposed_services);

[sky, 2015/01/21 16:59:00]

Again, can we make these not optional?

[jamesr, 2015/01/21 18:51:40]

No

 
   SetCapture(uint32 view_id) => (bool success);
   FocusWindow(uint32 view_id) => (bool success);
   ActivateWindow(uint32 view_id) => (bool success);
 
   GetFocusedAndActiveViews() => (uint32 focused_view_id, uint32 active_view_id);
 };
 
 interface WindowManagerClient {
   OnCaptureChanged(uint32 old_capture_view_id, uint32 new_capture_view_id);
   OnFocusChanged(uint32 old_focused_view_id, uint32 new_focused_view_id);
   OnActiveWindowChanged(uint32 old_focused_view_id, uint32 new_focused_view_id);
 };