Add support for Float/Double types for JS Mojo bindings.

Add support for Float/Double types for JS Mojo bindings.

This is now straightforward using the underlying DataView representation
used by the Mojo JS bindings, given its support for these types.

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=266040

[Tom Sepez, 2014-04-23 22:29:52]

Sky, MP, please review.  I figured I would put this up as long as I was mucking
with this portion of the code.  Thanks.

[sky, 2014-04-23 22:37:05]

LGTM (I only looked at content, I'm assuming Matt will look at the rest).

https://codereview.chromium.org/247013008/diff/20001/content/test/data/web_ui...
File content/test/data/web_ui_test_mojo_bindings.mojom (right):

https://codereview.chromium.org/247013008/diff/20001/content/test/data/web_ui...
content/test/data/web_ui_test_mojo_bindings.mojom:3: struct EchoArgs {
Please document what this is for and what the values are intended for.

[Matt Perry, 2014-04-23 22:37:08]

I have a similar CL that I merged with your codec changes from yesterday:
https://codereview.chromium.org/250253002/

I like your unit tests more. This LGTM - I'll merge my CL with yours when it
lands.

[Tom Sepez, 2014-04-23 22:48:57]

Ok, at which point I think I'm hands off this since I should have enough to get
going with some of the fuzzing (but I couldn't get array of string to work --
maybe you want to look into that, Matt).

[Matt Perry, 2014-04-23 22:53:55]

On 2014/04/23 22:48:57, Tom Sepez wrote:
> Ok, at which point I think I'm hands off this since I should have enough to
get
> going with some of the fuzzing (but I couldn't get array of string to work --
> maybe you want to look into that, Matt).

Sounds good. I *think* array of string should work with my patch, but I'll test
it to be sure.

[Tom Sepez, 2014-04-23 22:55:03]

On 2014/04/23 22:37:05, sky wrote:
> LGTM (I only looked at content, I'm assuming Matt will look at the rest).
> 
>
https://codereview.chromium.org/247013008/diff/20001/content/test/data/web_ui...
> File content/test/data/web_ui_test_mojo_bindings.mojom (right):
> 
>
https://codereview.chromium.org/247013008/diff/20001/content/test/data/web_ui...
> content/test/data/web_ui_test_mojo_bindings.mojom:3: struct EchoArgs {
> Please document what this is for and what the values are intended for.

Done.

[Tom Sepez, 2014-04-24 20:44:06]

The CQ bit was checked by tsepez@chromium.org

[commit-bot: I haz the power, 2014-04-24 21:16:27]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/tsepez@chromium.org/247013008/100001

[commit-bot: I haz the power, 2014-04-24 23:27:32]

Change committed as 266040