Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(4715)

Unified Diff: chrome/browser/autocomplete/builtin_provider.cc

Issue 8568039: Revert 110235 - chrome: Remove 14 exit time destructors and 2 static initializers. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Created 9 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | chrome/browser/autocomplete/search_provider.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/autocomplete/builtin_provider.cc
===================================================================
--- chrome/browser/autocomplete/builtin_provider.cc (revision 110237)
+++ chrome/browser/autocomplete/builtin_provider.cc (working copy)
@@ -63,13 +63,13 @@
(input.matches_requested() == AutocompleteInput::BEST_MATCH))
return;
- const string16 kAbout = ASCIIToUTF16(chrome::kAboutScheme) +
+ static const string16 kAbout = ASCIIToUTF16(chrome::kAboutScheme) +
ASCIIToUTF16(chrome::kStandardSchemeSeparator);
- const string16 kChrome = ASCIIToUTF16(chrome::kChromeUIScheme) +
+ static const string16 kChrome = ASCIIToUTF16(chrome::kChromeUIScheme) +
ASCIIToUTF16(chrome::kStandardSchemeSeparator);
- const int kUrl = ACMatchClassification::URL;
- const int kMatch = kUrl | ACMatchClassification::MATCH;
+ static const int kUrl = ACMatchClassification::URL;
+ static const int kMatch = kUrl | ACMatchClassification::MATCH;
string16 text = input.text();
bool starting_chrome = StartsWith(kChrome, text, false);
@@ -77,7 +77,7 @@
ACMatchClassifications styles;
// Highlight the input portion matching "chrome://"; or if the user has
// input "about:" (with optional slashes), highlight the whole "chrome://".
- const size_t kAboutSchemeLength = strlen(chrome::kAboutScheme);
+ static const size_t kAboutSchemeLength = strlen(chrome::kAboutScheme);
bool highlight = starting_chrome || text.length() > kAboutSchemeLength;
styles.push_back(ACMatchClassification(0, highlight ? kMatch : kUrl));
size_t offset = starting_chrome ? text.length() : kChrome.length();
« no previous file with comments | « no previous file | chrome/browser/autocomplete/search_provider.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698