Docserver manifest follow up (rewrite)

chrome/common/extensions/docs/templates/json/manifest.json

 {
-  "required": [
-    {
-      "documentation": "manifest/name.html",
-      "example": "My Extension",
-      "name": "name"
-    },
-    {
-      "documentation": "manifest/app.html",
-      "example": {},
-      "name": "app"
-    },
-    {
-      "documentation": "manifest/version.html",
-      "example": "versionString",
-      "name": "version"
-    },
-    {
-      "documentation": "manifest/manifest_version.html",
-      "example": 2,
-      "name": "manifest_version"
-    }
-  ],
-  "recommended": [
-    {
-      "documentation": "manifest/description.html",
-      "example": "A plain text description",
-      "name": "description"
-    },
-    {
-      "documentation": "manifest/icons.html",
-      "example": {},
-      "name": "icons"
-    },
-    {
-      "documentation": "manifest/default_locale.html",
-      "example": "en",
-      "name": "default_locale"
-    }
-  ],
-  "only_one": [
-    {
-      "documentation": "browserAction.html",
-      "example": {},
-      "name": "browser_action"
-    },
-    {
-      "documentation": "pageAction.html",
-      "example": {},
-      "name": "page_action"
-    },
-    {
-      "documentation": "themes.html",
-      "example": {},
-      "name": "theme"
-    }
-  ],
-  "optional": [
-    {
-      "documentation": "event_pages.html",
-      "example": {
-        "persistent": false
-      },
-      "name": "background"
-    },
-    {
-      "documentation": "background_pages.html",
-      "example": {
+  "app": {
+    "documentation": "manifest/app.html",
+    "example": {
+      "value": {}

[not at google - send to devlin, 2013/07/24 21:45:56]

let's make sure for empty object to generate {...} in the documentation not {}

likewise [...] not [].

[jshumway, 2013/07/26 00:36:46]

Done.

+    },

[not at google - send to devlin, 2013/07/24 21:45:56]

also - I think a better way to solve the problem of "example: false" being
detected as "no example" would be for manifest_data_source to insert a flag like
"has_example" for those APIs that have an example.

I.e. only exposed to templates, no need to specify in here.

[jshumway, 2013/07/26 00:36:46]

I like that a lot more, changed it.

+    "level":  "required"
+  },
+  "background": {
+    "documentation": "background_pages.html",
+    "example": {
+      "value": {
         "persistent": true
-      },
-      "name": "background.persistent"
-    },
-    {
-      "documentation": "override.html",
-      "example": {},
-      "name": "chrome_url_overrides"
-    },
-    {
-      "documentation": "content_scripts.html",
-      "example": {},
-      "name": "content_scripts"
-    },
-    {
-      "documentation": "contentSecurityPolicy.html",
-      "example": "policyString",
-      "name": "content_security_policy"
-    },
-    {
-      "documentation": "fileBrowserHandler.html",
-      "example": [],
-      "name": "file_browser_handlers"
-    },
-    {
-      "documentation": "manifest/file_handlers.html",
-      "example": {},
-      "name": "file_handlers"
-    },
-    {
-      "documentation": "manifest/homepage_url.html",
-      "example": "http://path/to/homepage",
-      "name": "homepage_url"
-    },
-    {
-      "documentation": "manifest/incognito.html",
-      "example": "spanning or split",
-      "name": "incognito"
-    },
-    {
-      "documentation": "manifest/key.html",
-      "example": "publicKey",
-      "name": "key"
-    },
-    {
-      "documentation": "manifest/minimum_chrome_version.html",
-      "example": "versionString",
-      "name": "minimum_chrome_version"
-    },
-    {
-      "documentation": "manifest/nacl_modules.html",
-      "example": [],
-      "name": "nacl_modules"
-    },
-    {
-      "documentation": "manifest/kiosk_enabled.html",
-      "example": true,
-      "name": "kiosk_enabled"
-    },
-    {
-      "documentation": "manifest/offline_enabled.html",
-      "example": true,
-      "name": "offline_enabled"
-    },
-    {
-      "documentation": "omnibox.html",
-      "example": {
+      }
+    },
+    "level": "optional"
+  },
+  "background.persistent": {
+    "documentation": "event_pages.html",
+    "example": {
+      "value": false
+    },
+    "level": "optional"
+  },
+  "chrome_url_overrides": {
+    "documentation": "override.html",
+    "example": {
+      "value": {}
+    },
+    "level": "optional"
+  },
+  "content_scripts": {
+    "documentation": "content_scripts.html",
+    "example": {
+      "value": {}
+    },
+    "level": "optional"
+  },
+  "content_security_policy": {
+    "documentation": "contentSecurityPolicy.html",
+    "example": {
+      "value": "policyString"
+    },
+    "level": "optional"
+  },
+  "default_locale": {
+    "documentation": "manifest/default_locale.html",
+    "example": {
+      "value": "en"
+    },
+    "level": "recommended"
+  },
+  "description": {
+    "documentation": "manifest/description.html",
+    "example": {
+      "value": "A plain text description"
+    },
+    "level": "recommended"
+  },
+  "browser_action": {
+    "documentation": "browserAction.html",
+    "example": {
+      "value": {}
+    },
+    "level": "only_one"
+  },
+  "file_browser_handlers": {
+    "documentation": "fileBrowserHandler.html",
+    "example": {
+      "value": []
+    },
+    "level": "optional"
+  },
+  "file_handlers": {
+    "documentation": "manifest/file_handlers.html",
+    "example": {
+      "value": {}
+    },
+    "level": "optional"
+  },
+  "homepage_url": {
+    "documentation": "manifest/homepage_url.html",
+    "example": {
+      "value": "http://path/to/homepage"
+    },
+    "level": "optional"
+  },
+  "icons": {
+    "documentation": "manifest/icons.html",
+    "example": {
+      "value": {}
+    },
+    "level": "recommended"
+  },
+  "incognito": {
+    "documentation": "manifest/incognito.html",
+    "example": {
+      "value": "spanning or split"
+    },
+    "level": "optional"
+  },
+  "key": {
+    "documentation": "manifest/key.html",
+    "example": {
+      "value": "publicKey"
+    },
+    "level": "optional"
+  },
+  "kiosk_enabled": {
+    "documentation": "manifest/kiosk_enabled.html",
+    "example": {
+      "value": true
+    },
+    "level": "optional"
+  },
+  "manifest_version": {
+    "documentation": "manifest/manifest_version.html",
+    "example": {
+      "value": 2
+    },
+    "level":  "required"
+  },
+  "minimum_chrome_version": {
+    "documentation": "manifest/minimum_chrome_version.html",
+    "example": {
+      "value": "versionString"
+    },
+    "level": "optional"
+  },
+  "nacl_modules": {
+    "documentation": "manifest/nacl_modules.html",
+    "example": {
+      "value": []
+    },
+    "level": "optional"
+  },
+  "name": {
+    "documentation": "manifest/name.html",
+    "example": {
+      "value": "My Extension"
+    },
+    "level":  "required"
+  },
+  "offline_enabled": {
+    "documentation": "manifest/offline_enabled.html",
+    "example": {
+      "value": true
+    },
+    "level": "optional"
+  },
+  "omnibox": {
+    "documentation": "omnibox.html",
+    "example": {
+      "value": {
         "keyword": "aString"
-      },
-      "name": "omnibox"
-    },
-    {
-      "documentation": "options.html",
-      "example": "aFile.html",
-      "name": "options_page"
-    },
-    {
-      "documentation": "declare_permissions.html",
-      "example": [],
-      "name": "permissions"
-    },
-    {
-      "documentation": "npapi.html",
-      "example": [],
-      "name": "plugins"
-    },
-    {
-      "documentation": "manifest/requirements.html",
-      "example": {},
-      "name": "requirements"
-    },
-    {
-      "documentation": "autoupdate.html",
-      "example": "http://path/to/updateInfo.xml",
-      "name": "update_url"
-    },
-    {
-      "documentation": "manifest/web_accessible_resources.html",
-      "example": [],
-      "name": "web_accessible_resources"
-    },
-    {
-      "documentation": "manifest/sandbox.html",
-      "example": [],
-      "name": "sandbox"
-    }
-  ]
+      }
+    },
+    "level": "optional"
+  },
+  "options_page": {
+    "documentation": "options.html",
+    "example": {
+      "value": "aFile.html"
+    },
+    "level": "optional"
+  },
+  "page_action": {
+    "documentation": "pageAction.html",
+    "example": {
+      "value": {}
+    },
+    "level": "only_one"
+  },
+  "permissions": {
+    "documentation": "declare_permissions.html",
+    "example": {
+      "value": []
+    },
+    "level": "optional"
+  },
+  "plugins": {
+    "documentation": "npapi.html",
+    "example": {
+      "value": []
+    },
+    "level": "optional"
+  },
+  "requirements": {
+    "documentation": "manifest/requirements.html",
+    "example": {
+      "value": {}
+    },
+    "level": "optional"
+  },
+  "sandbox": {
+    "documentation": "manifest/sandbox.html",
+    "example": {
+      "value": []
+    },
+    "level": "optional"

[not at google - send to devlin, 2013/07/24 21:45:56]

let's just make optional the default and apply that to the manifest keys that
come from _manifest_features.json.

[jshumway, 2013/07/26 00:36:46]

Done.

+  },
+  "theme": {
+    "documentation": "themes.html",
+    "example": {
+      "value": {}
+    },
+    "level": "only_one"
+  },
+  "update_url": {
+    "documentation": "autoupdate.html",
+    "example": {
+      "value": "http://path/to/updateInfo.xml"
+    },
+    "level": "optional"
+  },
+  "version": {
+    "documentation": "manifest/version.html",
+    "example": {
+      "value": "versionString"
+    },
+    "level":  "required"
+  },
+  "web_accessible_resources": {
+    "documentation": "manifest/web_accessible_resources.html",
+    "example": {
+      "value": []
+    },
+    "level": "optional"
+  }
 }
+