Hiding status area on welcome screen.

Hiding status area on welcome screen.
Avoid confusion when network is connected.

BUG=http://crosbug.com/2322
TEST=Launch for the fist time (or just press Ctrl+Alt+N) and observe that status area is not shown. Status area is shown on other screens.

Committed: http://chrome-svn/viewvc/chromeos?view=rev&revision=46672

[glotov, 2010-05-05 20:07:38]

LGTM

http://codereview.chromium.org/1927002/diff/1/4
File chrome/browser/chromeos/login/wizard_controller.cc (right):

http://codereview.chromium.org/1927002/diff/1/4#newcode272
chrome/browser/chromeos/login/wizard_controller.cc:272: void
WizardController::ShowNetworkScreen() {
I wish these routines could be just one
template <typename ScreenType> void ShowWizardScreen();

But probably it's too late :(

[Nikita (slow), 2010-05-06 09:54:09]

http://codereview.chromium.org/1927002/diff/1/4
File chrome/browser/chromeos/login/wizard_controller.cc (right):

http://codereview.chromium.org/1927002/diff/1/4#newcode272
chrome/browser/chromeos/login/wizard_controller.cc:272: void
WizardController::ShowNetworkScreen() {
On 2010/05/05 20:07:38, glotov wrote:
> I wish these routines could be just one
> template <typename ScreenType> void ShowWizardScreen();
> 
> But probably it's too late :(

Idea of these methods is to encapsulate logic that is performed when switching
to a specific screen:

- hide/show status area
- there could be more in the future

Such template would need to pass that logic in the parameters which is against
initial goal.

[tfarina (gmail-do not use), 2010-05-06 16:56:03]

http://codereview.chromium.org/1927002/diff/1/2
File chrome/browser/chromeos/login/background_view.cc (right):

http://codereview.chromium.org/1927002/diff/1/2#newcode75
chrome/browser/chromeos/login/background_view.cc:75: void
BackgroundView::SetStatusAreaVisible(bool flag) {
nit: please, could you rename this to visible?

[Nikita (slow), 2010-05-06 17:21:56]

http://codereview.chromium.org/1927002/diff/1/2
File chrome/browser/chromeos/login/background_view.cc (right):

http://codereview.chromium.org/1927002/diff/1/2#newcode75
chrome/browser/chromeos/login/background_view.cc:75: void
BackgroundView::SetStatusAreaVisible(bool flag) {
On 2010/05/06 16:56:03, tfarina wrote:
> nit: please, could you rename this to visible?

Done.