NOT FOR LANDING Implement WebMediaSession

content/renderer/media/renderer_webmediasession_impl.h

+// Copyright (c) 2015 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_RENDERER_MEDIA_RENDERER_WEBMEDIASESSION_IMPL_H_
+#define CONTENT_RENDERER_MEDIA_RENDERER_WEBMEDIASESSION_IMPL_H_
+
+#include "base/macros.h"
+#include "third_party/WebKit/public/platform/WebMediaSession.h"
+
+namespace content {
+
+class RendererWebMediaSessionImpl : public blink::WebMediaSession {

[mcasas, 2015/07/31 14:14:28]

I don't think the prefix "renderer" is needed in either the
file name nor the class, and the same applies for the
"impl" suffix - see other files around the same folder:
media_stream_track for WebMediaStreamTrack in Blink 
etc.

+ public:
+  RendererWebMediaSessionImpl();
+  virtual ~RendererWebMediaSessionImpl();
+
+  // blink::WebMediaSession implementation.
+  virtual void activate();

[whywhat, 2015/07/30 14:31:22]

nit: we use "override" instead of "virtual" in Chromium (and also in Blink I
believe) for virtual method overrides and also dtors.

[philipj_slow, 2015/07/30 14:47:04]

Oops, copypasta.

[mlamouri (slow - plz ping), 2015/08/05 15:50:16]

We don't actually use override when overriding Blink interfaces because it would
break Chromium if they were to change.

+  virtual void deactivate();
+
+ private:
+

[mlamouri (slow - plz ping), 2015/08/05 15:50:16]

nit: remove empty line

+  DISALLOW_COPY_AND_ASSIGN(RendererWebMediaSessionImpl);
+};
+
+}  // namespace content
+
+#endif  // CONTENT_RENDERER_MEDIA_RENDERER_WEBMEDIASESSION_IMPL_H_