Simple event tests for the Bluetooth API.

Simple event tests for the Bluetooth API.

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

[bryeung, 2012-07-09 20:57:14]

Mike: I based this off the serial API tests.  Can you review this?

[bryeung, 2012-07-11 14:25:40]

friendly ping

On 2012/07/09 20:57:14, bryeung wrote:
> Mike: I based this off the serial API tests.  Can you review this?

[miket_OOO, 2012-07-11 16:28:28]

On 2012/07/11 14:25:40, bryeung wrote:
> friendly ping
> 
> On 2012/07/09 20:57:14, bryeung wrote:
> > Mike: I based this off the serial API tests.  Can you review this?

Whoops, sorry, Bryan. You added me as a non-chromium.org reviewer, and
unfortunately that makes the CR nearly invisible to me. I'll review as a
drive-by -- meanwhile, can you fix the reviewer list?

[miket_OOO, 2012-07-11 16:32:53]

LGTM (and I now see that I can edit the reviewers line myself, so I dropped my
doppleganger).

http://codereview.chromium.org/10759006/diff/1/chrome/test/data/extensions/ap...
File chrome/test/data/extensions/api_test/bluetooth/test_events.js (right):

http://codereview.chromium.org/10759006/diff/1/chrome/test/data/extensions/ap...
chrome/test/data/extensions/api_test/bluetooth/test_events.js:11: }
This is just a matter of style, but instead of test.fail() you can do the
slightly more conventional chrome.test.assertTrue(), chrome.test.assertEq(),
etc., which have the potential of giving better failure messages. Up to you,
either's good with me.

[bryeung, 2012-07-11 19:06:34]

http://codereview.chromium.org/10759006/diff/1/chrome/test/data/extensions/ap...
File chrome/test/data/extensions/api_test/bluetooth/test_events.js (right):

http://codereview.chromium.org/10759006/diff/1/chrome/test/data/extensions/ap...
chrome/test/data/extensions/api_test/bluetooth/test_events.js:11: }
On 2012/07/11 16:32:53, miket wrote:
> This is just a matter of style, but instead of test.fail() you can do the
> slightly more conventional chrome.test.assertTrue(), chrome.test.assertEq(),
> etc., which have the potential of giving better failure messages. Up to you,
> either's good with me.

That's a great suggestion: thank you.

[commit-bot: I haz the power, 2012-07-11 19:07:56]

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

[commit-bot: I haz the power, 2012-07-11 20:35:14]

Change committed as 146193