Split MediaSessionStateChanged() and MediaSessionMetadataChanged()

content/browser/web_contents/web_contents_impl.h

Index: content/browser/web_contents/web_contents_impl.h
diff --git a/content/browser/web_contents/web_contents_impl.h b/content/browser/web_contents/web_contents_impl.h
index d1fc9a1bc034c62de03ce089918abc06c9aa7d6b..0958b4dcd6ada05813c6f3e391c90ff173354db3 100644
--- a/content/browser/web_contents/web_contents_impl.h
+++ b/content/browser/web_contents/web_contents_impl.h
@@ -415,6 +415,7 @@ class CONTENT_EXPORT WebContentsImpl
   void ExitFullscreen(bool will_cause_resize) override;
   void ResumeLoadingCreatedWebContents() override;
   void OnMediaSessionStateChanged();

[clamy, 2016/10/12 12:25:04]

It seems that these methods are badly located in the header: in the middle of
WebContents implementation when they appear not to be part of the WebContents
interface. If that's the case, could you move them below GetAllowOtherViews and
add a description for each of them?

[Zhiqiang Zhang (Slow), 2016/10/12 12:41:16]

Done.

+  void OnMediaSessionMetadataChanged();
   void ResumeMediaSession() override;
   void SuspendMediaSession() override;
   void StopMediaSession() override;