Split MediaContentType and AudioFocusType

media/base/media_content_type.h

Index: media/base/media_content_type.h
diff --git a/media/base/media_content_type.h b/media/base/media_content_type.h
new file mode 100644
index 0000000000000000000000000000000000000000..1c790e9ae8c7fbbcc9d6bfbcb80101d78f4b0e17
--- /dev/null
+++ b/media/base/media_content_type.h
@@ -0,0 +1,35 @@
+// 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 MEDIA_BASE_MEDIA_CONTENT_TYPE_H_
+#define MEDIA_BASE_MEDIA_CONTENT_TYPE_H_
+
+#include "base/time/time.h"
+#include "media/base/media_export.h"
+
+namespace media {
+
+// The content type of a media player, which will be used by MediaSession to
+// control its players.
+enum class MediaContentType {
+  // Type indicating that a player is primary content on the page.
+  Content,

[whywhat, 2016/08/18 20:33:42]

nit: MediaContentType::Content sounds a bit odd, also, primary content sounds a
bit misleading - the page may have 10 videos on it, are they all primary?

What about describing the action the session needs to do with the player?

smth like 

enum class FocusAction {
  GainPause,
  GainDuck,  // do we need a separate one for Transient vs Unpausable at this
point?
  Ignore
};

Maybe we don't need to split it into two then even? Just add ignore to
AudioFocusType and be it?

[Zhiqiang Zhang (Slow), 2016/08/19 15:05:55]

Maybe we call them "Persistent" as opposed to "Transient"?
We should consider Pepper, which will be specially treated in MediaSession in
follow-up patches. Thus I think we should split it.

Also with Pepper, "FocusAction" might not be a good name :(

Renamed "Unpausable" to "Pepper"

+  // Type indicating that a player only plays a transient sound.
+  Transient,
+  // Type indicating that a player cannot be paused. MediaSession may duck the
+  // player instead of pausing it.
+  Unpausable,
+  // Type indicating that a player cannot be controlled. MediaSession wil ignore
+  // this player.
+  Uncontrollable
+};
+
+// Utility function for deciding whether a player is treated as Content type
+// based on its duration.
+MEDIA_EXPORT MediaContentType
+DurationToMediaContentType(base::TimeDelta duration);
+
+}  // namespace media
+
+#endif  // MEDIA_BASE_MEDIA_CONTENT_TYPE_H_