Fix an out-of-date issue in chrome.webNavigation document.

chrome/common/extensions/api/web_navigation.json

Index: chrome/common/extensions/api/web_navigation.json
diff --git a/chrome/common/extensions/api/web_navigation.json b/chrome/common/extensions/api/web_navigation.json
index 783ecdddee03f9215b32b2b97c27008c0d85f54e..63f1d9c4b78fedb3e5e58f354389664d1ca7b280 100644
--- a/chrome/common/extensions/api/web_navigation.json
+++ b/chrome/common/extensions/api/web_navigation.json
@@ -149,7 +149,7 @@
               "url": {"type": "string"},
               "processId": {"type": "integer", "description": "The ID of the process runs the renderer for this tab."},
               "frameId": {"type": "integer", "description": "0 indicates the navigation happens in the tab content window; a positive value indicates navigation in a subframe. Frame IDs are unique within a tab."},
-              "transitionType": {"type": "string", "enum": ["link", "typed", "auto_bookmark", "auto_subframe", "manual_subframe", "generated", "start_page", "form_submit", "reload", "keyword", "keyword_generated"], "description": "Cause of the navigation. The same transition types as defined in the history API are used."},
+              "transitionType": {"type": "string", "enum": ["link", "typed", "auto_bookmark", "auto_subframe", "manual_subframe", "generated", "start_page", "form_submit", "reload", "keyword", "keyword_generated"], "description": "Cause of the navigation. The same transition types as defined in the history API are used. These are the same transition types as defined in the <a href=\"history.html#transition_types\">history API</a> except with <code>\"start_page\"</code> in place of <code>\"auto_toplevel\"</code> (for backwards compatibility)."},

[not at google - send to devlin, 2014/01/27 18:22:45]

I thought I made this comment before actually; could you pull this transition
type data out into a type and then for each of these use $ref: TransitionType as
the types.

You may want to annotate that TransitionType type with "inline_doc": true (I
think that's it) so that you don't get the enum generated at the top separtaely,
which I think usually looks a bit weird.

[Haojian Wu, 2014/01/28 06:27:04]

I'm not sure I get your point correctly here. If not, please correct me.

You means we should pull the transitionType into a new type in
web_navigation.json, like:

"types": [
  {
     "id": "TransitionType",
     "type": "string",
     "enum": ["link", "typed", "auto_bookmark", "auto_subframe",
"manual_subframe", "generated", "start_page", "form_submit", "reload",
"keyword", "keyword_generated"],
     "description": "These are the same transition types as defined in the
$ref:[history.VisitItem History API] except with <code>\"start_page\"</code> in
place of <code>\"auto_toplevel\"</code> (for backwards compatibility)."
  }
],

and each usage is like:

"transitionType": {"$ref": "TransitionType", "description": "Cause of the
navigation. The same transition types as defined in the history API are used."},

If we do it in this way, it will introduce a new type structure in webNavigation
API while json schema compiler generates the bundle C++ extension API code,
which will break compiling.

               "transitionQualifiers": {"type": "array", "description": "A list of transition qualifiers.", "items": {"type": "string", "enum": ["client_redirect", "server_redirect", "forward_back", "from_address_bar"]}},
               "timeStamp": {"type": "number", "description": "The time when the navigation was committed, in milliseconds since the epoch."}
             }
@@ -283,7 +283,7 @@
               "url": {"type": "string"},
               "processId": {"type": "integer", "description": "The ID of the process runs the renderer for this tab."},
               "frameId": {"type": "integer", "description": "0 indicates the navigation happens in the tab content window; a positive value indicates navigation in a subframe. Frame IDs are unique within a tab."},
-              "transitionType": {"type": "string", "enum": ["link", "typed", "auto_bookmark", "auto_subframe", "manual_subframe", "generated", "start_page", "form_submit", "reload", "keyword", "keyword_generated"], "description": "Cause of the navigation. The same transition types as defined in the history API are used."},
+              "transitionType": {"type": "string", "enum": ["link", "typed", "auto_bookmark", "auto_subframe", "manual_subframe", "generated", "start_page", "form_submit", "reload", "keyword", "keyword_generated"], "description": "Cause of the navigation. The same transition types as defined in the history API are used. These are the same transition types as defined in the <a href=\"history.html#transition_types\">history API</a> except with <code>\"start_page\"</code> in place of <code>\"auto_toplevel\"</code> (for backwards compatibility)."},
               "transitionQualifiers": {"type": "array", "description": "A list of transition qualifiers.", "items": {"type": "string", "enum": ["client_redirect", "server_redirect", "forward_back", "from_address_bar"]}},
               "timeStamp": {"type": "number", "description": "The time when the navigation was committed, in milliseconds since the epoch."}
             }
@@ -327,7 +327,7 @@
               "url": {"type": "string"},
               "processId": {"type": "integer", "description": "The ID of the process runs the renderer for this tab."},
               "frameId": {"type": "integer", "description": "0 indicates the navigation happens in the tab content window; a positive value indicates navigation in a subframe. Frame IDs are unique within a tab."},
-              "transitionType": {"type": "string", "enum": ["link", "typed", "auto_bookmark", "auto_subframe", "manual_subframe", "generated", "start_page", "form_submit", "reload", "keyword", "keyword_generated"], "description": "Cause of the navigation. The same transition types as defined in the history API are used."},
+              "transitionType": {"type": "string", "enum": ["link", "typed", "auto_bookmark", "auto_subframe", "manual_subframe", "generated", "start_page", "form_submit", "reload", "keyword", "keyword_generated"], "description": "Cause of the navigation. The same transition types as defined in the history API are used. These are the same transition types as defined in the <a href=\"history.html#transition_types\">history API</a> except with <code>\"start_page\"</code> in place of <code>\"auto_toplevel\"</code> (for backwards compatibility)."},
               "transitionQualifiers": {"type": "array", "description": "A list of transition qualifiers.", "items": {"type": "string", "enum": ["client_redirect", "server_redirect", "forward_back", "from_address_bar"]}},
               "timeStamp": {"type": "number", "description": "The time when the navigation was committed, in milliseconds since the epoch."}
             }