Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(91)

Side by Side Diff: content/browser/media/session/media_session_service_impl.h

Issue 2583463002: [MediaSession] Add playbackState attribute to Blink MediaSession and use it to determine playback s… (Closed)
Patch Set: browser tests Created 4 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef CONTENT_BROWSER_MEDIA_SESSION_MEDIA_SESSION_SERVICE_IMPL_H_ 5 #ifndef CONTENT_BROWSER_MEDIA_SESSION_MEDIA_SESSION_SERVICE_IMPL_H_
6 #define CONTENT_BROWSER_MEDIA_SESSION_MEDIA_SESSION_SERVICE_IMPL_H_ 6 #define CONTENT_BROWSER_MEDIA_SESSION_MEDIA_SESSION_SERVICE_IMPL_H_
7 7
8 #include "base/optional.h" 8 #include "base/optional.h"
9 #include "content/public/common/media_metadata.h" 9 #include "content/public/common/media_metadata.h"
10 #include "mojo/public/cpp/bindings/binding.h" 10 #include "mojo/public/cpp/bindings/binding.h"
(...skipping 10 matching lines...) Expand all
21 class CONTENT_EXPORT MediaSessionServiceImpl 21 class CONTENT_EXPORT MediaSessionServiceImpl
22 : public blink::mojom::MediaSessionService { 22 : public blink::mojom::MediaSessionService {
23 public: 23 public:
24 ~MediaSessionServiceImpl() override; 24 ~MediaSessionServiceImpl() override;
25 25
26 static void Create(RenderFrameHost* render_frame_host, 26 static void Create(RenderFrameHost* render_frame_host,
27 blink::mojom::MediaSessionServiceRequest request); 27 blink::mojom::MediaSessionServiceRequest request);
28 const blink::mojom::MediaSessionClientPtr& GetClient() { return client_; } 28 const blink::mojom::MediaSessionClientPtr& GetClient() { return client_; }
29 RenderFrameHost* GetRenderFrameHost() { return render_frame_host_; } 29 RenderFrameHost* GetRenderFrameHost() { return render_frame_host_; }
30 30
31 blink::mojom::MediaSessionPlaybackState playback_state() const {
32 return playback_state_;
33 }
mlamouri (slow - plz ping) 2016/12/16 15:49:14 It's generally better to avoid inline implementati
Zhiqiang Zhang (Slow) 2016/12/16 18:26:15 Acknowledged.
31 const base::Optional<MediaMetadata>& metadata() const { return metadata_; } 34 const base::Optional<MediaMetadata>& metadata() const { return metadata_; }
32 const std::set<blink::mojom::MediaSessionAction>& actions() const { 35 const std::set<blink::mojom::MediaSessionAction>& actions() const {
33 return actions_; 36 return actions_;
34 } 37 }
35 38
36 // blink::mojom::MediaSessionService implementation. 39 // blink::mojom::MediaSessionService implementation.
37 void SetClient(blink::mojom::MediaSessionClientPtr client) override; 40 void SetClient(blink::mojom::MediaSessionClientPtr client) override;
38 41
42 void SetPlaybackState(blink::mojom::MediaSessionPlaybackState state) override;
39 void SetMetadata(const base::Optional<MediaMetadata>& metadata) override; 43 void SetMetadata(const base::Optional<MediaMetadata>& metadata) override;
40 44
41 void EnableAction(blink::mojom::MediaSessionAction action) override; 45 void EnableAction(blink::mojom::MediaSessionAction action) override;
42 void DisableAction(blink::mojom::MediaSessionAction action) override; 46 void DisableAction(blink::mojom::MediaSessionAction action) override;
43 47
44 protected: 48 protected:
45 explicit MediaSessionServiceImpl(RenderFrameHost* render_frame_host); 49 explicit MediaSessionServiceImpl(RenderFrameHost* render_frame_host);
46 50
47 private: 51 private:
48 MediaSessionImpl* GetMediaSession(); 52 MediaSessionImpl* GetMediaSession();
49 53
50 void Bind(blink::mojom::MediaSessionServiceRequest request); 54 void Bind(blink::mojom::MediaSessionServiceRequest request);
51 55
52 RenderFrameHost* render_frame_host_; 56 RenderFrameHost* render_frame_host_;
53 57
54 // RAII binding of |this| to an MediaSessionService interface request. 58 // RAII binding of |this| to an MediaSessionService interface request.
55 // The binding is removed when binding_ is cleared or goes out of scope. 59 // The binding is removed when binding_ is cleared or goes out of scope.
56 std::unique_ptr<mojo::Binding<blink::mojom::MediaSessionService>> binding_; 60 std::unique_ptr<mojo::Binding<blink::mojom::MediaSessionService>> binding_;
57 blink::mojom::MediaSessionClientPtr client_; 61 blink::mojom::MediaSessionClientPtr client_;
62 blink::mojom::MediaSessionPlaybackState playback_state_;
58 base::Optional<MediaMetadata> metadata_; 63 base::Optional<MediaMetadata> metadata_;
59 std::set<blink::mojom::MediaSessionAction> actions_; 64 std::set<blink::mojom::MediaSessionAction> actions_;
60 65
61 DISALLOW_COPY_AND_ASSIGN(MediaSessionServiceImpl); 66 DISALLOW_COPY_AND_ASSIGN(MediaSessionServiceImpl);
62 }; 67 };
63 68
64 } // namespace content 69 } // namespace content
65 70
66 #endif // CONTENT_BROWSER_MEDIA_SESSION_MEDIA_SESSION_SERVICE_IMPL_H_ 71 #endif // CONTENT_BROWSER_MEDIA_SESSION_MEDIA_SESSION_SERVICE_IMPL_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698