Change the size in the App Info dialog to always display in MB

Change the size in the App Info dialog to always display in MB

Changed the size in the App Info dialog to always display in MB, no
matter the size of the app. If the app is less than 1 MB, display
'< 1 MB' instead of the size.

BUG=398717
Committed: https://crrev.com/142d2bdbbe41338204e6c99b5f43a1b9d61e3e95
Cr-Commit-Position: refs/heads/master@{#297365}

[sashab, 2014-09-29 08:31:28]

sashab@chromium.org changed reviewers:
+ tapted@chromium.org

[sashab, 2014-09-29 08:31:29]

Approved by UI (ainslie@). See bug for details.

[tapted, 2014-09-30 01:36:51]

lgtm

https://codereview.chromium.org/615453002/diff/1/chrome/browser/ui/views/apps...
File chrome/browser/ui/views/apps/app_info_dialog/app_info_summary_panel.cc
(right):

https://codereview.chromium.org/615453002/diff/1/chrome/browser/ui/views/apps...
chrome/browser/ui/views/apps/app_info_dialog/app_info_summary_panel.cc:326:
const int kOneMegabyteInBytes = 1024 * 1024;
nit: -> one_mebibyte_in_bytes, or just one_mebibyte (kFooStyle isn't for
function-local consts, and "mebi" to be consistent with the unit below)

[sashab, 2014-09-30 02:58:36]

https://codereview.chromium.org/615453002/diff/1/chrome/browser/ui/views/apps...
File chrome/browser/ui/views/apps/app_info_dialog/app_info_summary_panel.cc
(right):

https://codereview.chromium.org/615453002/diff/1/chrome/browser/ui/views/apps...
chrome/browser/ui/views/apps/app_info_dialog/app_info_summary_panel.cc:326:
const int kOneMegabyteInBytes = 1024 * 1024;
On 2014/09/30 01:36:50, tapted wrote:
> nit: -> one_mebibyte_in_bytes, or just one_mebibyte (kFooStyle isn't for
> function-local consts, and "mebi" to be consistent with the unit below)

Done.

[sashab, 2014-09-30 02:58:47]

The CQ bit was checked by sashab@chromium.org

[commit-bot: I haz the power, 2014-09-30 03:05:18]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/615453002/20001

[commit-bot: I haz the power, 2014-09-30 05:07:39]

Committed patchset #2 (id:20001) as 80bbd0e5905ac4cc0b233e8f7e59c68f99cc24d3

[commit-bot: I haz the power, 2014-09-30 05:08:16]

Patchset 2 (id:??) landed as
https://crrev.com/142d2bdbbe41338204e6c99b5f43a1b9d61e3e95
Cr-Commit-Position: refs/heads/master@{#297365}