Adding support for mount point different from removable devices to MountLibrary

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 5154 matching lines @@
             "type": "boolean",
             "description": "Flag that specifies if volume is mounted in read-only mode."
           },
           "totalSizeKB": {
             "type": "integer",
             "description": "Total disk volume size in KBs"
           }
         }
       },
       {
+        "id": "MountType",

[zel, 2011/07/22 21:12:23]

remove this type, see below

[tbarzic, 2011/07/22 22:04:48]

Done.

+        "type": "string",
+        "enum": ["device", "file", "network"],
+        "description": "Type of the mount."
+        
+      },
+      {
         "id": "VolumeMetadata",
         "type": "object",
         "description": "Mounted disk volume metadata.",
         "properties": {
           "mountPath": {
             "type": "string",
             "description": "Disk volume mount point path."
           },
           "devicePath": {
             "type": "string",
@@ skipping 52 matching lines @@
             "enum": ["added", "removed"],
             "description": "Event type that tells listeners which disk volume even was raised."
           },
           "volumeInfo": {
             "$ref": "VolumeInfo",
             "description":"Volume information that this mount event applies to."
           }
         }
       },
       {
+        "id": "MountCompletedEvent",
+        "type": "object",
+        "description": "Payload data for mount event.",
+        "properties": {
+          "eventType": {

[zel, 2011/07/22 21:12:23]

let's make eventType being just 'mount', 'unmount' and add
another field called 'status' that will have success/error values as:


  success, 
  error_unknown,
  error_internal,
  ...

and make sure you prefix all error conditions with error_*

[tbarzic, 2011/07/22 22:04:48]

Done.

+            "type": "string",
+            "enum": ["success", "unknown", "internal", "unknown_filesystem",
+                     "unsuported_filesystem", "invalid_archive",
+                     "libcros_missing", "path_unmounted"],
+            "description": "Event type that tells listeners if mount was successfull or an error occurred. It also specifies the error."
+          },
+          "sourcePath": {
+            "type": "string",
+            "description": "Path that has been mounted."
+          },
+          "mountPath": {
+            "type": "string",
+            "optional": true,
+            "description": "Path that sourcePath was mounted to."
+          },
+          "mountType": {
+            "$ref": "MountType",

[zel, 2011/07/22 21:12:23]

make this string and put that enum here as :

"type": "string",
"enum": ["device", "file", "network"],

[tbarzic, 2011/07/22 22:04:48]

Done.

+            "description": "Type of the mount."
+          }
+        }
+      },
+      {
         "id": "FileWatchEvent",
         "type": "object",
         "description": "Payload data for disk mount / unmount event.",
         "properties": {
           "eventType": {
             "type": "string",
             "enum": ["changed", "error"],
             "description": "Specifies type of event that is raised."
           },
           "fileUrl": {
@@ skipping 238 matching lines @@
         "description": "Fired when disk mount/unmount event is detected.",
         "parameters": [
           {
             "$ref": "MountEvent",
             "name": "event",
             "description": "Mount event information."
           }
         ]
       },
       {
+        "name": "onMountCompleted",
+        "type": "function",
+        "description": "Fired when mount event is detected.",
+        "parameters": [
+          {
+            "$ref": "MountCompletedEvent",
+            "name": "event",
+            "description": "MountCompleted event information."
+          }
+        ]
+      },
+      {
         "name": "onFileChanged",
         "type": "function",
         "description": "Fired when watched file change event is detected.",
         "parameters": [
           {
             "$ref": "FileWatchEvent",
             "name": "event",
             "description": "File watch event information."
           }
         ]
@@ skipping 1026 matching lines @@
                 "description": "A string result code. The value is non-empty on success only in tests.",
                 "optional": "true"
               }
             ]
           }
         ]
       }
     ]
   }
 ]