[Android] A prototype of the interactive media notification.

chrome/android/java/src/org/chromium/chrome/browser/media/ui/MediaInfo.java

+// Copyright 2013 The Chromium Authors. All rights reserved.

[mlamouri (slow - plz ping), 2015/06/18 16:43:22]

2015

[whywhat, 2015/06/19 16:00:34]

Done.

+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+package org.chromium.chrome.browser.media.ui;
+
+import android.text.TextUtils;
+
+/**
+ * Exposes information about the current media to the external clients.
+ */
+public class MediaInfo {
+
+    // Playback state enum.
+    public static final int PLAYING = 0;
+    public static final int PAUSED = 1;
+
+    /**
+     * The title of the media
+     */
+    public String title;
+
+    /**
+     * The current state of the media
+     */
+    public int state;

[mlamouri (slow - plz ping), 2015/06/18 16:43:22]

Why not a boolean? We could move to an enum later if needed.

[whywhat, 2015/06/19 16:00:34]

Done.

+
+    /**
+     * The eTLD+1 of the media

[mlamouri (slow - plz ping), 2015/06/18 16:43:22]

I believe this is the origin, not eTLD+1.

Also, not the origin of the media but the origin of the top frame of the tab
containing the media.

[whywhat, 2015/06/19 16:00:34]

Well, the origin doesn't have a simple definition, right?
https://tools.ietf.org/html/rfc6454#section-3.2

Since I get it from originForDisplay though, I could name it that.
Yes, it's tab's/page's origin for display.

[mlamouri (slow - plz ping), 2015/06/22 14:34:17]

Origin has a fairly simple definition:
scheme + subdomains + domain + port.

Maybe getOriginForDisplay doesn't show the port?

[whywhat, 2015/06/23 19:39:10]

It doesn't show the default port for https.

Just for the sake of argument (I changed the name to |origin|)

Still, the origin of a URI can be a globally unique identifier instead:
https://tools.ietf.org/html/rfc6454#section-4
E.g. for file or data URIs (which can still play audio and have our notification
show up).

+     */
+    public String broadDomain;

[mlamouri (slow - plz ping), 2015/06/18 16:43:22]

Why not call it origin?

[whywhat, 2015/06/19 16:00:34]

See above. Also, the MediaCapture notification uses the following code to obtain
the "URL" to display:

https://code.google.com/p/chromium/codesearch#chromium/src/chrome/android/jav...

It's the scheme and host, which roughly matches getOriginForDisplay(..., true);

I guess it could be origin after all :)

+
+    /**
+     * The id of the tab containing the media
+     */
+    public int tabId;
+
+    /**
+     * Create a new MediaInfo
+     * @param title
+     * @param state
+     * @param broadDomain
+     * @param tabId
+     */
+    public MediaInfo(
+            String title,
+            int state,
+            String broadDomain,
+            int tabId) {
+        this.title = title;
+        this.state = state;
+        this.broadDomain = broadDomain;
+        this.tabId = tabId;
+    }
+
+    /**
+     * Copy a media info
+     * @param other the source.
+     */
+    public MediaInfo(MediaInfo other) {
+        this(other.title, other.state, other.broadDomain, other.tabId);
+    }
+
+    @Override
+    public boolean equals(Object obj) {
+        if (obj == this) return true;
+        if (!(obj instanceof MediaInfo)) return false;
+
+        MediaInfo other = (MediaInfo) obj;
+        return state == other.state
+                && tabId == other.tabId
+                && TextUtils.equals(title, other.title)
+                && TextUtils.equals(broadDomain, other.broadDomain);
+    }
+
+    @Override
+    public int hashCode() {
+        int result = state;
+        result = 31 * result + (title == null ? 0 : title.hashCode());
+        result = 31 * result + (broadDomain == null ? 0 : broadDomain.hashCode());
+        result = 31 * result + tabId;
+        return result;
+    }
+}