Bluetooth API: test discovery callback.

Bluetooth API: test discovery callback.

TEST=added
BUG=132616
Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=146271

[miket_OOO, 2012-07-11 21:49:46]

LGTM % nits.

http://codereview.chromium.org/10695161/diff/2001/chrome/test/data/extensions...
File chrome/test/data/extensions/api_test/bluetooth/test_discovery.js (right):

http://codereview.chromium.org/10695161/diff/2001/chrome/test/data/extensions...
chrome/test/data/extensions/api_test/bluetooth/test_discovery.js:16:
chrome.test.runTests([ testDiscovery ]);
We're not super-consistent about whitespace on single-line arrays (see
http://google-styleguide.googlecode.com/svn/trunk/javascriptguide.xml?showone...),
but it'd be nice at least to be consistent with the other usage in this file.

http://codereview.chromium.org/10695161/diff/2001/chrome/test/data/extensions...
chrome/test/data/extensions/api_test/bluetooth/test_discovery.js:31:
console.log('Got device: ' + device.name);
oops -- if you want to keep this in, please indent properly.

[bryeung, 2012-07-12 01:02:40]

Sorry: that was a bit sloppy of me.  Thanks for catching those things.

http://codereview.chromium.org/10695161/diff/2001/chrome/test/data/extensions...
File chrome/test/data/extensions/api_test/bluetooth/test_discovery.js (right):

http://codereview.chromium.org/10695161/diff/2001/chrome/test/data/extensions...
chrome/test/data/extensions/api_test/bluetooth/test_discovery.js:16:
chrome.test.runTests([ testDiscovery ]);
On 2012/07/11 21:49:46, miket wrote:
> We're not super-consistent about whitespace on single-line arrays (see
>
http://google-styleguide.googlecode.com/svn/trunk/javascriptguide.xml?showone...),
> but it'd be nice at least to be consistent with the other usage in this file.

Oops.  Good catch.

http://codereview.chromium.org/10695161/diff/2001/chrome/test/data/extensions...
chrome/test/data/extensions/api_test/bluetooth/test_discovery.js:31:
console.log('Got device: ' + device.name);
On 2012/07/11 21:49:46, miket wrote:
> oops -- if you want to keep this in, please indent properly.

Yep: another oops.  Thanks.

[commit-bot: I haz the power, 2012-07-12 01:02:58]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/bryeung@chromium.org/10695161/8001

[commit-bot: I haz the power, 2012-07-12 02:13:49]

Change committed as 146271