Fix constant integers style in persistent_tab_restore_service.cc.

Fix constant integers style in persistent_tab_restore_service.cc.

This was unnecessarily changed by r164641.

TEST=browser_tests


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

[Nico, 2012-11-09 15:55:18]

Lgtm
On Nov 9, 2012 7:35 AM, <pliard@chromium.org> wrote:

> Reviewers: sky, Nico,
>
> Description:
> Fix constant integers style in persistent_tab_restore_**service.cc.
>
> This was unnecessarily changed by r164641.
>
> TEST=browser_tests
>
>
> Please review this at
http://codereview.chromium.**org/11377070/<http://codereview.chromium.org/113...
>
> SVN Base:
svn://svn.chromium.org/chrome/**trunk/src<http://svn.chromium.org/chrome/trunk/src>
>
> Affected files:
>   M chrome/browser/sessions/**persistent_tab_restore_**service.cc
>
>
> Index: chrome/browser/sessions/**persistent_tab_restore_**service.cc
> diff --git a/chrome/browser/sessions/**persistent_tab_restore_**service.cc
> b/chrome/browser/sessions/**persistent_tab_restore_**service.cc
> index 02424eb62aed794c63725cab627de1**201f0c45b9..**
> 357964762037202a6fc537b139a07e**5fac6c50fa 100644
> --- a/chrome/browser/sessions/**persistent_tab_restore_**service.cc
> +++ b/chrome/browser/sessions/**persistent_tab_restore_**service.cc
> @@ -102,13 +102,11 @@ const SessionCommand::id_type
> kCommandSetExtensionAppID = 6;
>  const SessionCommand::id_type kCommandSetWindowAppName = 7;
>  const SessionCommand::id_type kCommandSetTabUserAgentOverrid**e = 8;
>
> -enum {
> -  // Number of entries (not commands) before we clobber the file and write
> -  // everything.
> -  kEntriesPerReset = 40,
> +// Number of entries (not commands) before we clobber the file and write
> +// everything.
> +const int kEntriesPerReset = 40;
>
> -  kMaxEntries = TabRestoreServiceHelper::**kMaxEntries,
> -};
> +const size_t kMaxEntries = TabRestoreServiceHelper::**kMaxEntries;
>
>  }  // namespace
>
>
>
>

[sky, 2012-11-09 18:14:44]

LGTM

[Philippe, 2012-11-09 18:17:40]

On 2012/11/09 18:14:44, sky wrote:
> LGTM

Thanks guys!

[commit-bot: I haz the power, 2012-11-09 18:17:56]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/pliard@chromium.org/11377070/1

[commit-bot: I haz the power, 2012-11-13 14:34:20]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/pliard@chromium.org/11377070/1

[commit-bot: I haz the power, 2012-11-13 14:47:38]

Change committed as 167394