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

chrome/common/extensions/api/extension_api.json

 [
   {
     "namespace": "extension",
     "unprivileged": true,
     "types": [
       {
         "id": "MessageSender",
         "type": "object",
         "description": "An object containing information about the script context that sent a message or request.",
         "properties": {
@@ skipping 5094 matching lines @@
           {
             "$ref": "MountEvent",
             "name": "fullPath",
             "description": "Mount event information."
           }
         ]
       }
     ]
   },
   {
+    "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": [
       {
         "name": "install",
         "description": "Installs the extension corresponding to the given id",
         "parameters": [
           {
             "name": "expected_id",
             "type": "string",
@@ skipping 468 matching lines @@
           {
             "type": "integer",
             "name": "tabId",
             "description": "The id of the tab that was detached."
           }
         ]
       }
     ]
   }
 ]