Make power button in status view invisible until we are sure battery is supported on device.

Make power button in status view invisible until we are sure battery is supported on device.

BUG=chromium-os:21360
TEST=Verified power button is visible on Cr48
     Verified power button is invisible when there is no battery
     Verified power button is invisible in vm
     PowerMenuButtonBrowserTest.*

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

[tbarzic, 2011-11-15 01:25:26]

Dave, could you take a look at this?

[DaveMoore, 2011-11-16 23:35:27]

lgtm if you remove the string resource.

http://codereview.chromium.org/8536048/diff/2001/chrome/browser/chromeos/stat...
File chrome/browser/chromeos/status/power_menu_button.cc (left):

http://codereview.chromium.org/8536048/diff/2001/chrome/browser/chromeos/stat...
chrome/browser/chromeos/status/power_menu_button.cc:340: tooltip_text =
l10n_util::GetStringUTF16(IDS_STATUSBAR_NO_BATTERY);
Please remove the string too

[tbarzic, 2011-11-16 23:56:49]

On 2011/11/16 23:35:27, DaveMoore wrote:
> lgtm if you remove the string resource.
> 
>
http://codereview.chromium.org/8536048/diff/2001/chrome/browser/chromeos/stat...
> File chrome/browser/chromeos/status/power_menu_button.cc (left):
> 
>
http://codereview.chromium.org/8536048/diff/2001/chrome/browser/chromeos/stat...
> chrome/browser/chromeos/status/power_menu_button.cc:340: tooltip_text =
> l10n_util::GetStringUTF16(IDS_STATUSBAR_NO_BATTERY);
> Please remove the string too

I'd rather remove resources in separate CL, since this is supposed to be merged
into R16 branch.

[achuithb, 2011-11-17 09:39:40]

lgtm