Link MediaSessionTabHelper with native MediaSession [CL is going to be split]

content/browser/media/session/media_session_observer.h

Index: content/browser/media/session/media_session_observer.h
diff --git a/content/browser/media/session/media_session_observer.h b/content/browser/media/session/media_session_observer.h
new file mode 100644
index 0000000000000000000000000000000000000000..ad22725aa8dbea197763c8082f4d787e980a8af7
--- /dev/null
+++ b/content/browser/media/session/media_session_observer.h
@@ -0,0 +1,42 @@
+// 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.
+
+#ifndef CONTENT_BROWSER_MEDIA_SESSION_MEDIA_SESSION_OBSERVER_H_
+#define CONTENT_BROWSER_MEDIA_SESSION_MEDIA_SESSION_OBSERVER_H_
+
+#include "base/macros.h"
+#include "base/optional.h"
+#include "content/public/common/media_metadata.h"
+
+namespace content {
+
+class MediaSession;
+
+class MediaSessionObserver {
+ public:
+  explicit MediaSessionObserver(MediaSession* session);
+  virtual ~MediaSessionObserver() = default;

[whywhat, 2016/10/20 15:38:33]

if you "unobserve" in dtor, you can probably remove the ownership of observers
in MediaSession.

[Zhiqiang Zhang (Slow), 2016/10/20 16:22:32]

Yes. It is now in the new patch.

+
+  MediaSession* session() { return session_; }
+
+  virtual void MediaSessionDisconnected() {}
+
+  virtual void MediaSessionStateChanged(
+      bool is_controllable, bool is_suspended) {}
+
+  virtual void MediaSessionMetadataChanged(
+      const base::Optional<MediaMetadata>& metadata) {}
+
+ private:
+  void Observe(MediaSession* session);
+
+  // Weak pointer to MediaSession, this class is owned by MediaSession.
+  MediaSession* session_;
+
+  DISALLOW_COPY_AND_ASSIGN(MediaSessionObserver);
+};
+
+}  // namespace content
+
+#endif  // CONTENT_BROWSER_MEDIA_SESSION_MEDIA_SESSION_OBSERVER_H_