Bug 128080 – WebKit Bot Watcher's Dashboard: Teach JSON.load() to interpret third argument as either an option dictionary or a failure callback

• Status: RESOLVED
• Resolution: FIXED
• Component: Tools / Tests

[Daniel Bates]

For convenience JSON.load() should accept either a Function object or an Object for the data type of the third argument of JSON.load() such that we can call JSON.load() with following forms:

1. JSON.load("http://www.apple.com", function() {/* success */})
2. JSON.load("http://www.apple.com", function() {/* success */}, {withCredentials: true, ...});
3. JSON.load("http://www.apple.com", function() {/* success */}, function() { /* failure */ })
4. JSON.load("http://www.apple.com", function() {/* success */}, function() {/* failure */}, {withCredentials: true, ...})

Currently JSON.load() can be called with forms: (1), (3), and (4).

As implied above, form (2) makes using JSON.load() more convenient when a failure callback function isn't necessary, but a person wants to pass an option dictionary because a person doesn't need to explicitly pass null or a reference to an empty function for the value of the third argument.

[Daniel Bates]

Created attachment 222936 [details]
Patch

[Daniel Bates]

Committed r163313: <http://trac.webkit.org/changeset/163313>