Moving mediaplayer to the chrome filebrowser. Observable behaviour should not change.

chrome/common/extensions/api/extension_api.json

Index: chrome/common/extensions/api/extension_api.json
diff --git a/chrome/common/extensions/api/extension_api.json b/chrome/common/extensions/api/extension_api.json
index f69c49d71027ea4c856a5d2d114933e80f5d7a13..ce91142c167c337f34189f50d037c1764d9f43aa 100644
--- a/chrome/common/extensions/api/extension_api.json
+++ b/chrome/common/extensions/api/extension_api.json
@@ -5112,6 +5112,97 @@
     ]
   },
   {
+    "namespace": "mediaPlayerPrivate",
+    "nodoc": "true",
+    "functions": [
+      {
+        "name": "setPlaylistPositionAndPlay",

[zel, 2011/05/24 17:54:25]

please rename to 'playAt'

[SeRya, 2011/05/25 14:23:49]

Done.

+        "description": "Sets playlist positions and sets flag of pending play request. Then fires the onPlaylistChanged event.",
+        "parameters": [
+          {
+            "name": "position",
+            "type": "integer",
+            "description": "New position in the playlist."
+          }
+        ]
+      },
+      {
+        "name": "playbackError",
+        "description": "Marks the URL as failed to play one. Mediaplayer will visually marks such an item. Then fires the onPlaylistChanged event.",
+        "parameters": [
+          {
+            "name": "url",
+            "type": "string",
+            "description": "URL failed to play."
+          }
+        ]
+      },
+      {
+        "name": "getPlaylist",
+        "type": "function",
+        "description": "Returns current play list, position and flag of penging play request. Resets flag of pending play request.",
+        "parameters": [
+          {
+            "name": "callback",
+            "type": "function",
+            "description": "Callback to retrieve the playlist.",
+            "parameters": [
+              {
+                "name": "playlist",
+                "type": "object",

[zel, 2011/05/24 17:54:25]

this object should be defined outside of the function definition - see "types"
section under fileBrowserPrivate API for an example

[SeRya, 2011/05/25 14:23:49]

Done.

+                "description": "Mediaplayer playlist stored in the browser (it extsts even if the mediaplayer is closed).",
+                "properties": {
+                  "items": {
+                    "name": "items",
+                    "type": "array",
+                    "items": {
+                      "type": "object",
+                      "properties": {
+                        "path": { "type": "string", "description": "URL of mediafile (in 'filesystem:' scheme for local files)." },
+                        "error": { "type": "boolean", "description": "True if the file has failed to play ('playbackError' with its URL called)." }
+                      }
+                    }
+                  },
+                  "position": {
+                    "type": "integer",
+                    "description": "Index of item in the 'items' array to play next."
+                  },
+                  "pending_playback_request": {
+                    "type": "boolean",
+                    "description": "Informs that a request to play current item has been received. This flag is reset automatically."
+                  }
+                }
+              }
+            ]
+          }
+        ]
+      },
+      {
+        "name": "togglePlaylistPanel",
+        "description": "Shows or hides the playlist panel.",
+        "parameters": []
+      },
+      {
+        "name": "toggleFullscreen",
+        "description": "Switches fullscreen mode on/off for the mediaplayer.",
+        "parameters": []
+      },
+      {
+        "name": "showPlaylistPanel",

[zel, 2011/05/24 17:54:25]

Why do have both show and toggle Panel methods? Can we merge these two?

[SeRya, 2011/05/25 14:23:49]

This method was not used. I removed it.

+        "description": "Shows the playlist panel.",
+        "parameters": []
+      }
+    ],
+    "events": [
+      {
+        "name": "onPlaylistChanged",
+        "type": "function",
+        "descrition": "Notifies that playlist content or state has been changed. Data could be retrieved via 'getPlaylist'.",
+        "parameters": []

[zel, 2011/05/24 17:54:25]

we we want to add details about what changed here?

[SeRya, 2011/05/25 14:23:49]

I think we may keep it as is for now. We could pass a flag to inform that
playlist items have not changed (when we just move current position) to avoid
rebuilding playlist. Since we are going to move he playlist from separate panel
I'd postpone that to keep this CL a bit simplier.

[zel, 2011/05/25 14:36:44]

All right. We will remove this event anyway.

+      }
+    ]
+  },
+  {
     "namespace":"webstorePrivate",
     "nodoc": "true",
     "functions": [