Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(376)

Issue 4235005: Push for ers:... (Closed)

Created:
10 years, 1 month ago by Charlie Lee
Modified:
9 years, 7 months ago
Reviewers:
CC:
chromium-reviews, ben+cc_chromium.org, nkostylev+cc_chromium.org, davemoore+watch_chromium.org, Paweł Hajdan Jr.
Visibility:
Public.

Description

Push for ers: Added handling of network ("service" in flimflam parlance) property changes. NetworkLibrary updates its Network objects in response to property changes, then calls any NetworkObservers that have registered for the given network. Use the is_active member of Network in various places to ensure that we are dealing with the correct network. Created separate observer classes for NetworkManager, Network, and CellularDataPlan, so that monitors can be established independently for each of them. Changed the callback method names to prepend "On", i.e., OnNetworkManagerChanged, OnNetworkChanged, OnCellularDataPlanChanged. Wired up signal strength update and network technology update in the status bar, network dropdown menu, and internet settings page. Added new icon badges for 1xRTT radio technology. These are used when EVDO coverage switches to 1xRTT. Just as is the case for the 3G icons, there are three 1x icons: normal, warn, and error. Please consider these icons to be placeholders if the UI folks think they need to be redesigned. Centralized choosing of the network technology badge in NetworkMenu::BadgeForNetworkTechnology. BUG=chromium-os:5612 :chromium-os:7619,chromium-os:5917,chromium-os:4674 TEST=Extensive manual testing with different combinations of wi-fi and cellular networks connected and disconnected. Used a modified version of flimflam that randomly changes reported signal strength, network technology, and roaming state, and verified that the UI correctly reflected current values for the active network (roaming is not handled in the UI at this time). Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=64888

Patch Set 1 #

Patch Set 2 : '' #

Patch Set 3 : '' #

Patch Set 4 : '' #

Unified diffs Side-by-side diffs Delta from patch set Stats (+832 lines, -381 lines) Patch
A chrome/app/theme/statusbar_network_1x.png View Binary file 0 comments Download
A chrome/app/theme/statusbar_network_1x_error.png View Binary file 0 comments Download
A chrome/app/theme/statusbar_network_1x_warn.png View Binary file 0 comments Download
M chrome/app/theme/theme_resources.grd View 1 2 3 1 chunk +3 lines, -0 lines 0 comments Download
M chrome/browser/browser_init.cc View 1 2 3 1 chunk +6 lines, -4 lines 0 comments Download
M chrome/browser/chromeos/cros/cros_mock.cc View 1 2 3 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/chromeos/cros/mock_network_library.h View 1 2 3 1 chunk +6 lines, -4 lines 0 comments Download
M chrome/browser/chromeos/cros/network_library.h View 1 2 3 10 chunks +80 lines, -30 lines 0 comments Download
M chrome/browser/chromeos/cros/network_library.cc View 1 2 3 23 chunks +363 lines, -126 lines 0 comments Download
M chrome/browser/chromeos/dom_ui/internet_options_handler.h View 1 2 3 3 chunks +20 lines, -5 lines 0 comments Download
M chrome/browser/chromeos/dom_ui/internet_options_handler.cc View 1 2 3 4 chunks +52 lines, -8 lines 0 comments Download
M chrome/browser/chromeos/dom_ui/mobile_setup_ui.cc View 1 2 3 6 chunks +33 lines, -43 lines 0 comments Download
M chrome/browser/chromeos/login/login_utils.cc View 1 2 3 1 chunk +6 lines, -6 lines 0 comments Download
M chrome/browser/chromeos/login/network_screen.h View 1 2 3 1 chunk +2 lines, -2 lines 0 comments Download
M chrome/browser/chromeos/login/network_screen.cc View 1 2 3 3 chunks +7 lines, -5 lines 0 comments Download
M chrome/browser/chromeos/login/network_screen_browsertest.cc View 1 2 3 8 chunks +11 lines, -8 lines 0 comments Download
M chrome/browser/chromeos/login/network_screen_delegate.h View 1 2 3 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/chromeos/network_list.h View 1 2 3 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/chromeos/network_list.cc View 1 2 3 1 chunk +2 lines, -1 line 0 comments Download
M chrome/browser/chromeos/network_message_observer.h View 1 2 3 3 chunks +6 lines, -5 lines 0 comments Download
M chrome/browser/chromeos/network_message_observer.cc View 1 2 3 3 chunks +11 lines, -3 lines 0 comments Download
M chrome/browser/chromeos/network_state_notifier.h View 1 2 3 3 chunks +5 lines, -5 lines 0 comments Download
M chrome/browser/chromeos/network_state_notifier.cc View 1 2 3 1 chunk +9 lines, -1 line 0 comments Download
M chrome/browser/chromeos/network_state_notifier_browsertest.cc View 1 2 3 3 chunks +3 lines, -3 lines 0 comments Download
M chrome/browser/chromeos/options/internet_page_view.h View 1 2 3 1 chunk +3 lines, -3 lines 0 comments Download
M chrome/browser/chromeos/options/internet_page_view.cc View 1 2 3 4 chunks +16 lines, -19 lines 0 comments Download
MM chrome/browser/chromeos/status/network_dropdown_button.h View 1 2 3 2 chunks +3 lines, -3 lines 0 comments Download
M chrome/browser/chromeos/status/network_dropdown_button.cc View 1 2 3 3 chunks +20 lines, -21 lines 0 comments Download
M chrome/browser/chromeos/status/network_menu.h View 1 2 3 1 chunk +3 lines, -2 lines 0 comments Download
M chrome/browser/chromeos/status/network_menu.cc View 1 2 3 2 chunks +42 lines, -3 lines 0 comments Download
M chrome/browser/chromeos/status/network_menu_button.h View 1 2 3 4 chunks +16 lines, -4 lines 0 comments Download
M chrome/browser/chromeos/status/network_menu_button.cc View 1 2 3 5 chunks +101 lines, -64 lines 0 comments Download

Powered by Google App Engine
This is Rietveld 408576698