Added option to bookmarkManagerPrivate.getMetaInfo() to get meta info from all bookmarks in a one f…

chrome/common/extensions/api/bookmark_manager_private.json

Index: chrome/common/extensions/api/bookmark_manager_private.json
diff --git a/chrome/common/extensions/api/bookmark_manager_private.json b/chrome/common/extensions/api/bookmark_manager_private.json
index a965d17f9e17e37d082ffcb58f15165053a99c98..ef802b19a7cb60a192140accbfde7f8dc8fc1a49 100644
--- a/chrome/common/extensions/api/bookmark_manager_private.json
+++ b/chrome/common/extensions/api/bookmark_manager_private.json
@@ -49,6 +49,12 @@
         "type": "object",
         "description": "Collection of meta info fields.",
         "additionalProperties": {"type": "string"}
+      },
+      {
+        "id": "MapMetaInfoFields",
+        "type": "object",
+        "description": "Map Id to MetaInfoFields.",
+        "additionalProperties": {"type": "any"}
       }
     ],
     "functions": [
@@ -269,10 +275,12 @@
         "name": "getMetaInfo",
         "type": "function",
         "description": "Gets meta info from a bookmark node",
+        "allowAmbiguousOptionalArguments": true,
         "parameters": [
           {
             "name": "id",
-            "description": "The id of the bookmark to retrieve meta info from",
+            "description": "The id of the bookmark to retrieve meta info from. If omitted meta info for all nodes is returned.",
+            "optional": true,
             "type": "string"
           },
           {
@@ -287,13 +295,13 @@
             "parameters": [
               {
                 "name": "value",
-                "description": "If a key was given, the value of the specified field, if present. Otherwise an object containing all meta info fields for the node.",
+                "description": "If a key was given, the value of the specified field, if present. Otherwise an object containing all meta info fields for the node. If id is not given then meta info for all nodes as an object with node id to meta info.",

[not at google - send to devlin, 2014/06/16 21:49:27]

Ah, yeah, this is a pretty confusing API no matter how you phrase it... 1 type,
2 types, no types... 2 methods would have been better I think.

e.g. what happens if you specify a key value, but no ID? like

chrome.bookmarkManagerPrivate.getMetaInfo(null, 'foo', function(whatIsThis) {});

[yefimt, 2014/06/16 21:56:20]

Originally it was ignoring second parameter if first one is null. After talking
to Mike and extension dev changed it to return "undefined" in callback. That is
consistent with behavior when key is not found.

On 2014/06/16 21:49:27, kalman wrote:

[not at google - send to devlin, 2014/06/16 21:59:47]

undefined... and set error? sounds like it should be an error in that case.

[yefimt, 2014/06/16 22:32:12]

extension developers don't want error as they don't get callback in this case.

On 2014/06/16 21:59:47, kalman wrote:

[not at google - send to devlin, 2014/06/16 22:33:54]

they don't get a callback..? then where does the undefined go? I'm confusd.

                 "optional": true,
                 // TODO(rfevang): Convert this to always return MetaInfoFields
                 // if the API is made public.
                 "choices": [
                   {"type": "string"},
-                  {"$ref": "MetaInfoFields"}
+                  {"$ref": "MapMetaInfoFields"}

[not at google - send to devlin, 2014/06/16 21:49:27]

if you're going to do this, might as well inline the definition of
MapMetaInfoFields

[yefimt, 2014/06/16 21:56:20]

Do you mean remove "choices", yes it is possible
On 2014/06/16 21:49:27, kalman wrote:

[not at google - send to devlin, 2014/06/16 21:59:47]

No I mean make this

"choices": [
  {"type": "string"},
  {"type": "object", "additionalPropreties": "any"}
]

[yefimt, 2014/06/16 22:00:42]

I think it will be too confusing. Combining MetaInfoFields and MapMetaInfoFileds
was semi OK because they both "additionalProperties", adding "string" there
sounds too much:)

On 2014/06/16 21:56:20, yefimt wrote:

[not at google - send to devlin, 2014/06/16 22:02:01]

I don't understand. this is exactly what you already have, except instead of
{"$ref": "MapMetaInfoFields"} it's {"type": "object", ...}

[yefimt, 2014/06/16 22:32:12]

Done.

                 ]
               }
             ]