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

chrome/browser/browser_about_handler.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/browser_about_handler.h"
 
+#include <algorithm>
 #include <string>
 
 #include "base/command_line.h"
 #include "base/logging.h"
 #include "base/memory/singleton.h"
 #include "base/string_util.h"
 #include "chrome/browser/net/url_fixer_upper.h"
 #include "chrome/browser/ui/browser_dialogs.h"
 #include "chrome/common/chrome_switches.h"
 #include "chrome/common/url_constants.h"
@@ skipping 142 matching lines @@
 #endif
 
 #endif  // OFFICIAL_BUILD
 
   return false;
 }
 
 std::vector<std::string> ChromePaths() {
   std::vector<std::string> paths;
   paths.reserve(arraysize(kChromePaths));
-  for (size_t i = 0; i < arraysize(kChromePaths); i++)
+  for (size_t i = 0; i < arraysize(kChromePaths); ++i)
     paths.push_back(kChromePaths[i]);

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

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]

Done.

+  std::sort(paths.begin(), paths.end());
   return paths;
 }