Introduce NavigatorOnLine interface

Introduce NavigatorOnLine interface

Introduce NavigatorOnLine interface and have Navigator and WorkerNavigator
implement it to avoid duplication and match the specification:
http://www.whatwg.org/specs/web-apps/current-work/multipage/offline.html#navigatoronline
http://www.whatwg.org/specs/web-apps/current-work/multipage/workers.html#workernavigator
http://www.whatwg.org/specs/web-apps/current-work/multipage/timers.html#navigator

R=haraken@chromium.org

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=156392

[haraken, 2013-08-20 10:28:51]

LGTM

https://codereview.chromium.org/23308006/diff/4001/Source/core/page/Navigator...
File Source/core/page/NavigatorOnLine.h (right):

https://codereview.chromium.org/23308006/diff/4001/Source/core/page/Navigator...
Source/core/page/NavigatorOnLine.h:38: class NavigatorOnLine {

Nit: Previously you used a namespace for this kind of simple class :) Both are
fine.

[do-not-use, 2013-08-20 10:31:48]

https://codereview.chromium.org/23308006/diff/4001/Source/core/page/Navigator...
File Source/core/page/NavigatorOnLine.h (right):

https://codereview.chromium.org/23308006/diff/4001/Source/core/page/Navigator...
Source/core/page/NavigatorOnLine.h:38: class NavigatorOnLine {
On 2013/08/20 10:28:51, haraken wrote:
> 
> Nit: Previously you used a namespace for this kind of simple class :) Both are
> fine.

I used a namespace first then got linking errors about having functions defined
multiple times (because I did not add a .cpp file). So I decided not to bother
and use a class with static functions.

[haraken, 2013-08-20 10:32:30]

On 2013/08/20 10:31:48, Christophe Dumez wrote:
> > Nit: Previously you used a namespace for this kind of simple class :) Both
are
> > fine.
> 
> I used a namespace first then got linking errors about having functions
defined
> multiple times (because I did not add a .cpp file). So I decided not to bother
> and use a class with static functions.

Makes sense.

[do-not-use, 2013-08-20 12:43:44]

Committed patchset #2 manually as r156392 (presubmit successful).

[Peter Beverloo, 2013-08-20 12:45:42]

NavigatorOnLine.idl should probably have a license header.

[do-not-use, 2013-08-20 12:46:45]

On 2013/08/20 12:45:42, Peter Beverloo wrote:
> NavigatorOnLine.idl should probably have a license header.

Sorry about that, I'll fix it straight away