Deprecate kAshDisableNewNetworkStatusArea

Deprecate kAshDisableNewNetworkStatusArea

Eliminates NetworkListDetailedView and dependencies.

This mostly removes dead code, but also fixes the following 
issues related to the old code:
* TrayNetwork::OnWillToggleWifi() -> 
  TrayNetwork::RequestToggleWifi() to toggle wifi directly 
  from Ash.
* Implement TrayNetwork::NetworkManagerChanged() so that 
  detailed views are notified when technology state changes.
* Eliminates unnecessary ash::SystemTrayDelegate
  dependencies on network_menu_ when using new connection
  handler code.

BUG=240859
For about_network.cc:
TBR=jamescook@chromium.org, jennyz@chromium.org

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

[stevenjb, 2013-05-17 23:53:05]

jennyz@ - Primary reviewer
jamescook@ - owner for ash/ and chrome/browser/ui/ash/
gspencer@ - Mostly FYI

[jennyz, 2013-05-20 20:46:17]

lgtm with a nit.

Excellent, so glad to see the code has been simplified so much!

https://codereview.chromium.org/15350002/diff/3001/ash/system/chromeos/networ...
File ash/system/chromeos/network/tray_vpn.h (right):

https://codereview.chromium.org/15350002/diff/3001/ash/system/chromeos/networ...
ash/system/chromeos/network/tray_vpn.h:8: #include
"ash/system/chromeos/network/network_observer.h"
Remove this include?

[James Cook, 2013-05-20 21:06:16]

LGTM with nits. Hooray for code deletion!

https://codereview.chromium.org/15350002/diff/3001/ash/system/chromeos/networ...
File ash/system/chromeos/network/tray_vpn.cc (right):

https://codereview.chromium.org/15350002/diff/3001/ash/system/chromeos/networ...
ash/system/chromeos/network/tray_vpn.cc:7: #include "ash/ash_switches.h"
Remove this?

https://codereview.chromium.org/15350002/diff/3001/ash/system/chromeos/networ...
ash/system/chromeos/network/tray_vpn.cc:17: #include "base/command_line.h"
Remove this. There also might be other files where you don't need it any more.

[stevenjb, 2013-05-20 21:16:50]

https://codereview.chromium.org/15350002/diff/3001/ash/system/chromeos/networ...
File ash/system/chromeos/network/tray_vpn.cc (right):

https://codereview.chromium.org/15350002/diff/3001/ash/system/chromeos/networ...
ash/system/chromeos/network/tray_vpn.cc:7: #include "ash/ash_switches.h"
On 2013/05/20 21:06:16, James Cook (Chromium) wrote:
> Remove this?

Done.

https://codereview.chromium.org/15350002/diff/3001/ash/system/chromeos/networ...
ash/system/chromeos/network/tray_vpn.cc:17: #include "base/command_line.h"
On 2013/05/20 21:06:16, James Cook (Chromium) wrote:
> Remove this. There also might be other files where you don't need it any more.

Done.

https://codereview.chromium.org/15350002/diff/3001/ash/system/chromeos/networ...
File ash/system/chromeos/network/tray_vpn.h (right):

https://codereview.chromium.org/15350002/diff/3001/ash/system/chromeos/networ...
ash/system/chromeos/network/tray_vpn.h:8: #include
"ash/system/chromeos/network/network_observer.h"
On 2013/05/20 20:46:17, jennyz wrote:
> Remove this include?

Done.

[commit-bot: I haz the power, 2013-05-20 23:33:41]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/stevenjb@chromium.org/15350002/27001

[commit-bot: I haz the power, 2013-05-20 23:46:53]

Retried try job too often on chromium_presubmit for step(s) presubmit
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=chromium_p...

[commit-bot: I haz the power, 2013-05-21 00:44:18]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/stevenjb@chromium.org/15350002/27001

[commit-bot: I haz the power, 2013-05-21 01:08:13]

Sorry for I got bad news for ya.
Compile failed with a clobber build on linux_chromeos_clang.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=linux_chro...
Your code is likely broken or HEAD is junk. Please ensure your
code is not broken then alert the build sheriffs.
Look at the try server FAQ for more details.

[stevenjb, 2013-05-21 17:07:24]

Committed patchset #7 manually as r201322 (presubmit successful).