Skip to content

Commit

Permalink
Disable showing 3g activation in OOBE.
Browse files Browse the repository at this point in the history
As per the issue, we've decided to not show the activation option for 3g in the OOBE screen, but to show it only on the login screen and started session. This CL disables it in OOBE.

R=sadrul@chromium.org
BUG=134475
TEST=The option to activate the 3g modem doesn't appear in OOBE.


Review URL: https://chromiumcodereview.appspot.com/10837056

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149511 0039d316-1c4b-4281-b951-d872f2087c98
  • Loading branch information
rkc@chromium.org committed Aug 1, 2012
1 parent aef706d commit 7c5cff3
Showing 1 changed file with 5 additions and 0 deletions.
5 changes: 5 additions & 0 deletions chrome/browser/chromeos/status/network_menu.cc
Original file line number Diff line number Diff line change
Expand Up @@ -588,6 +588,11 @@ void MainMenuModel::InitMenuItems(bool should_open_button_options) {
chromeos::ActivationState activation_state =
cell_networks[i]->activation_state();

// This is currently only used in the OOBE screen, do not show
// activating 3G option.
if (activation_state != ACTIVATION_STATE_ACTIVATED)
continue;

// Ampersand is a valid character in a network name, but menu2 uses it
// to mark "mnemonics" for keyboard shortcuts. http://crosbug.com/14697
std::string network_name = EscapeAmpersands(cell_networks[i]->name());
Expand Down

0 comments on commit 7c5cff3

Please sign in to comment.