Update the contextual search API to include a parameter specifying whether the server shoudl return…

components/search_engines/template_url.h

 // Copyright 2014 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.
 
 #ifndef COMPONENTS_SEARCH_ENGINES_TEMPLATE_URL_H_
 #define COMPONENTS_SEARCH_ENGINES_TEMPLATE_URL_H_
 
 #include <string>
 #include <utility>
 #include <vector>
@@ skipping 63 matching lines @@
 
     struct ContextualSearchParams {
       ContextualSearchParams();
       ContextualSearchParams(const int version,
                              const size_t start,
                              const size_t end,
                              const std::string& selection,
                              const std::string& content,
                              const std::string& base_page_url,
                              const std::string& encoding);
+      ContextualSearchParams(const int version,
+                             const size_t start,
+                             const size_t end,
+                             const std::string& selection,
+                             const std::string& content,
+                             const std::string& base_page_url,
+                             const std::string& encoding,
+                             const bool resolve);
       ~ContextualSearchParams();
 
       // The version of contextual search.
       int version;
 
       // Offset into the page content of the start of the user selection.
       size_t start;
 
       // Offset into the page content of the end of the user selection.
       size_t end;
 
       // The user selection.
       std::string selection;
 
       // The text including and surrounding the user selection.
       std::string content;
 
       // The URL of the page containing the user selection.
       std::string base_page_url;
 
       // The encoding of content.
       std::string encoding;
+
+      // True to have the server generate a search term based on the user
+      // selection and surrounding content.  False to simply return the user
+      // selection as the search term.
+      bool resolve;

[Peter Kasting, 2014/07/31 20:33:49]

Cool, this is much more comprehensible now.

This might be even clearer:

  // If true, the server will generate a search term based on the user selection
  // and context.  Otherwise the user selection will be used as-is as the search
  // term.
  bool generate_search_term;

[jeremycho, 2014/07/31 21:08:39]

Done.

     };
 
     // The search terms (query).
     base::string16 search_terms;
 
     // The original (input) query.
     base::string16 original_query;
 
     // The type the original input query was identified as.
     metrics::OmniboxInputType::Type input_type;
@@ skipping 588 matching lines @@
   TemplateURLRef new_tab_url_ref_;
   TemplateURLRef contextual_search_url_ref_;
   scoped_ptr<AssociatedExtensionInfo> extension_info_;
 
   // TODO(sky): Add date last parsed OSD file.
 
   DISALLOW_COPY_AND_ASSIGN(TemplateURL);
 };
 
 #endif  // COMPONENTS_SEARCH_ENGINES_TEMPLATE_URL_H_