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

Side by Side Diff: chrome/browser/search_engines/template_url.cc

Issue 164493005: Move SearchTermsData::ForceInstantResultsParam() to chrome::ForceInstantResultsParam(). (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Addressed nits Created 6 years, 10 months 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 unified diff | Download patch | Annotate | Revision Log
« no previous file with comments | « chrome/browser/search_engines/template_url.h ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/search_engines/template_url.h" 5 #include "chrome/browser/search_engines/template_url.h"
6 6
7 #include <string> 7 #include <string>
8 #include <vector> 8 #include <vector>
9 9
10 #include "base/basictypes.h" 10 #include "base/basictypes.h"
(...skipping 873 matching lines...) Expand 10 before | Expand all | Expand 10 after
884 HandleReplacement( 884 HandleReplacement(
885 "cp", 885 "cp",
886 base::StringPrintf("%" PRIuS, search_terms_args.cursor_position), 886 base::StringPrintf("%" PRIuS, search_terms_args.cursor_position),
887 *i, 887 *i,
888 &url); 888 &url);
889 break; 889 break;
890 890
891 case GOOGLE_FORCE_INSTANT_RESULTS: 891 case GOOGLE_FORCE_INSTANT_RESULTS:
892 DCHECK(!i->is_post_param); 892 DCHECK(!i->is_post_param);
893 HandleReplacement(std::string(), 893 HandleReplacement(std::string(),
894 search_terms_data.ForceInstantResultsParam( 894 chrome::ForceInstantResultsParam(
895 search_terms_args.force_instant_results), 895 search_terms_args.force_instant_results),
896 *i, 896 *i,
897 &url); 897 &url);
898 break; 898 break;
899 899
900 case GOOGLE_INSTANT_EXTENDED_ENABLED: 900 case GOOGLE_INSTANT_EXTENDED_ENABLED:
901 DCHECK(!i->is_post_param); 901 DCHECK(!i->is_post_param);
902 HandleReplacement(std::string(), 902 HandleReplacement(std::string(),
903 chrome::InstantExtendedEnabledParam(type_ == SEARCH), 903 chrome::InstantExtendedEnabledParam(type_ == SEARCH),
904 *i, 904 *i,
(...skipping 427 matching lines...) Expand 10 before | Expand all | Expand 10 after
1332 // patterns. This means that given patterns 1332 // patterns. This means that given patterns
1333 // [ "http://foo/#q={searchTerms}", "http://foo/?q={searchTerms}" ], 1333 // [ "http://foo/#q={searchTerms}", "http://foo/?q={searchTerms}" ],
1334 // calling ExtractSearchTermsFromURL() on "http://foo/?q=bar#q=' would 1334 // calling ExtractSearchTermsFromURL() on "http://foo/?q=bar#q=' would
1335 // return false. This is important for at least Google, where such URLs 1335 // return false. This is important for at least Google, where such URLs
1336 // are invalid. 1336 // are invalid.
1337 return !search_terms->empty(); 1337 return !search_terms->empty();
1338 } 1338 }
1339 } 1339 }
1340 return false; 1340 return false;
1341 } 1341 }
OLDNEW
« no previous file with comments | « chrome/browser/search_engines/template_url.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698