Adding mojo MediaSessionClient to support media controls

third_party/WebKit/public/platform/modules/mediasession/media_session.mojom

 // Copyright 2016 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 blink.mojom;
 
 import "mojo/common/common_custom_types.mojom";
 import "ui/gfx/geometry/mojo/geometry.mojom";
 import "url/mojo/url.mojom";
 
 // Album art in MediaMetadata
 // Spec: https://wicg.github.io/mediasession/
 struct MediaImage {
   url.mojom.Url src;
   mojo.common.mojom.String16 type;
   array<gfx.mojom.Size> sizes;
 };
 
 // MediaMetadata
 // Spec: https://wicg.github.io/mediasession/
 struct MediaMetadata {
   mojo.common.mojom.String16 title;
   mojo.common.mojom.String16 artist;
   mojo.common.mojom.String16 album;
   array<MediaImage> artwork;
 };
 
+interface MediaSessionClient {
+  DidReceivedAction(string action);

[whywhat, 2016/10/17 15:38:43]

Could you change action from string to enum?

[Zhiqiang Zhang (Slow), 2016/10/17 21:17:39]

Done. However it seems mapping values enum (or int) from/to AtomicString is ugly
in Blink. Also the tests are bit more unclear since it's using integer as enums.

See the new patch. I'd prefer to revert the change and do the mapping in
content/ in a later patch, WDYT?

+};
+
 interface MediaSessionService {
+  // MediaSessionClient interface is to used to notify Blink MediaSession of
+  // media control actions.
+  SetClient(MediaSessionClient client);
+
   SetMetadata(MediaMetadata? metadata);
+
+  DidSetEventHandlerForAction(string action, bool is_set);

[whywhat, 2016/10/17 15:38:43]

Could you change is_set to two methods?

Maybe the method should not be about the event handler at all but rather
something like:

EnableDisableAction(Action action, bool enable) // again, consider having Enable
and Disable methods

[Zhiqiang Zhang (Slow), 2016/10/17 21:17:39]

Done.

 };