Convert ViewMsg_NetworkStateChanged from routed -> control, allowing

Convert ViewMsg_NetworkStateChanged from routed -> control
allowing it to be sent from BrowserRenderProcessHost::OnProcessLaunched 
and minimizing the number of required IPCs (since the online state
is a per-WebKit singleton).

In doing so, I've disentangled online state management from TabContents: it now resides off by itself in BrowserOnlineStateObserver, a tiny class owned by BrowserProcessImpl.

BUG=7469, 86538
TEST=begin with network disconnected, open new tab, load test page attached to 7469, then re-connect network

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=91390

[darin (slow to review), 2011-07-01 18:19:15]

LGTM

http://codereview.chromium.org/7259004/diff/1001/content/renderer/render_thre...
File content/renderer/render_thread.cc (right):

http://codereview.chromium.org/7259004/diff/1001/content/renderer/render_thre...
content/renderer/render_thread.cc:676:
WebKit::WebNetworkStateNotifier::setOnLine(online);
nit: convention is to add a using directive at the top of the file to import
WebNetworkStateNotifier.  then, just call it directly without the WebKit::
prefix.

[adamk, 2011-07-01 18:48:06]

http://codereview.chromium.org/7259004/diff/1001/content/renderer/render_thre...
File content/renderer/render_thread.cc (right):

http://codereview.chromium.org/7259004/diff/1001/content/renderer/render_thre...
content/renderer/render_thread.cc:676:
WebKit::WebNetworkStateNotifier::setOnLine(online);
On 2011/07/01 18:19:15, darin wrote:
> nit: convention is to add a using directive at the top of the file to import
> WebNetworkStateNotifier.  then, just call it directly without the WebKit::
> prefix.

Done.