Fix developer_private.js for closure compiler

Fix developer_private.js for closure compiler

Automated definition used enum types instead of strings, and it looks like we
need to include the two functions which take "any" arguments.

BUG=461039
Committed: https://crrev.com/ceb01b3f0b024d2fc4bc4e19fa79715c8d7c0a43
Cr-Commit-Position: refs/heads/master@{#318143}

[Devlin, 2015-02-25 20:29:38]

rdevlin.cronin@chromium.org changed reviewers:
+ dbeam@chromium.org

[Devlin, 2015-02-25 20:29:39]

Hey Dan, sorry for the noise.

https://codereview.chromium.org/956063002/diff/1/third_party/closure_compiler...
File third_party/closure_compiler/externs/developer_private.js (right):

https://codereview.chromium.org/956063002/diff/1/third_party/closure_compiler...
third_party/closure_compiler/externs/developer_private.js:206:
chrome.developerPrivate.requestFileSource = function(dict, callback) {};
I didn't think we needed these, because closure compiler is totally happy right
now (without any definition).  Any idea why?

[Dan Beam, 2015-02-25 21:42:58]

https://codereview.chromium.org/956063002/diff/1/third_party/closure_compiler...
File third_party/closure_compiler/externs/developer_private.js (left):

https://codereview.chromium.org/956063002/diff/1/third_party/closure_compiler...
third_party/closure_compiler/externs/developer_private.js:37: *   type:
ItemType,
can we just add a @typedef (or @enum if this is one) instead?

[Devlin, 2015-02-25 22:15:43]

https://codereview.chromium.org/956063002/diff/1/third_party/closure_compiler...
File third_party/closure_compiler/externs/developer_private.js (left):

https://codereview.chromium.org/956063002/diff/1/third_party/closure_compiler...
third_party/closure_compiler/externs/developer_private.js:37: *   type:
ItemType,
On 2015/02/25 21:42:57, Dan Beam wrote:
> can we just add a @typedef (or @enum if this is one) instead?

Added @typedefs for them. We can't really use @enum, because we don't generate
enums for these values in the bindings (even though we should).  "enum" as used
in the json/idl files basically just means "we promise it's one of these", but
are always actually strings.

[Dan Beam, 2015-02-25 22:25:46]

lgtm (also we should update that script to this when possible)

[Devlin, 2015-02-25 22:28:35]

The CQ bit was checked by rdevlin.cronin@chromium.org

[commit-bot: I haz the power, 2015-02-25 22:29:11]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/956063002/20001

[commit-bot: I haz the power, 2015-02-25 23:29:00]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-02-25 23:30:05]

Patchset 2 (id:??) landed as
https://crrev.com/ceb01b3f0b024d2fc4bc4e19fa79715c8d7c0a43
Cr-Commit-Position: refs/heads/master@{#318143}