Sort ChromePaths alphabetically for chrome://chrome-urls, etc.

Sort ChromePaths alphabetically for chrome://chrome-urls, etc.

BUG=NONE
TEST=The big "List of Chrome URLs" at chrome://chrome-urls (about:about) is sorted alphabetically.
R=thakis@chromium.org

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

[msw, 2012-09-25 03:33:04]

Hey Nico, please take a look; thanks!
This arose from: http://codereview.chromium.org/10980008

[Nico, 2012-09-25 04:12:22]

thanks!

https://codereview.chromium.org/10978015/diff/1/chrome/browser/browser_about_...
File chrome/browser/browser_about_handler.cc (right):

https://codereview.chromium.org/10978015/diff/1/chrome/browser/browser_about_...
chrome/browser/browser_about_handler.cc:171: paths.push_back(kChromePaths[i]);
Can you initialize paths like this:

  std::vector<std::string> paths(kChromePaths, kChromePaths +
arraysize(kChromePaths));

? That saves you 3 lines.

[msw, 2012-09-25 05:00:51]

Please take another look; thanks!

https://codereview.chromium.org/10978015/diff/1/chrome/browser/browser_about_...
File chrome/browser/browser_about_handler.cc (right):

https://codereview.chromium.org/10978015/diff/1/chrome/browser/browser_about_...
chrome/browser/browser_about_handler.cc:171: paths.push_back(kChromePaths[i]);
On 2012/09/25 04:12:22, Nico wrote:
> Can you initialize paths like this:
> 
>   std::vector<std::string> paths(kChromePaths, kChromePaths +
> arraysize(kChromePaths));
> 
> ? That saves you 3 lines.

Done.

[Nico, 2012-09-25 05:03:53]

lgtm

[commit-bot: I haz the power, 2012-09-25 05:37:30]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/msw@chromium.org/10978015/6001

[commit-bot: I haz the power, 2012-09-25 13:10:11]

Change committed as 158558