Replace Chrome IPC with Mojo IPC for querying BatteryStatus service

Replace Chrome IPC with Mojo IPC for querying BatteryStatus service

[darin (slow to review), 2014-08-13 23:02:00]

Here's what it looks like with the battery service moved to device/battery/.
That code has no dependency back on content/. I've only ported the ChromeOS
implementation!

Notice how trivial the registration code is in content/browser/ to expose the
battery status service to the renderer.

[timvolodine, 2014-08-14 17:58:19]

On 2014/08/13 23:02:00, darin wrote:
> Here's what it looks like with the battery service moved to device/battery/.
> That code has no dependency back on content/. I've only ported the ChromeOS
> implementation!
> 
> Notice how trivial the registration code is in content/browser/ to expose the
> battery status service to the renderer.

I think this looks great. It would significantly simplify the content/renderer
code we currently have, especially if we move the other APIs to mojo at some
point. Why is it that "std::numeric_limits<double>::infinity()" cannot be used
in the .mojom file?

[darin (slow to review), 2014-08-14 19:37:16]

On 2014/08/14 17:58:19, timvolodine wrote:
...
> I think this looks great. It would significantly simplify the content/renderer
> code we currently have, especially if we move the other APIs to mojo at some
> point. Why is it that "std::numeric_limits<double>::infinity()" cannot be used
> in the .mojom file?

See https://code.google.com/p/chromium/issues/detail?id=402216. We just need
to teach the Mojom Parser to understand "inf" as a valid value for a double
field.