Adds an experiment that enabled SuggestionsService suggestions (MostLikely) on LocalNTP

chrome/common/instant_types.cc

 // Copyright 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/common/instant_types.h"
 
 #include "base/strings/utf_string_conversions.h"
 #include "net/base/escape.h"
 
 namespace {
@@ skipping 70 matching lines @@
       has_attribution == rhs.has_attribution &&
       logo_alternate == rhs.logo_alternate;
 }
 
 const char kSearchQueryKey[] = "q";
 const char kOriginalQueryKey[] = "oq";
 const char kRLZParameterKey[] = "rlz";
 const char kInputEncodingKey[] = "ie";
 const char kAssistedQueryStatsKey[] = "aqs";
 
+InstantMostVisitedItem::InstantMostVisitedItem() {}
+
+InstantMostVisitedItem::~InstantMostVisitedItem() {}

[kmadhusu, 2015/07/30 21:57:02]

Any specific reason to define an empty constructor and destructor?

[fserb, 2015/07/31 17:08:17]

The build complains that the struct has more than 3 items and must have a
constructor/destructor.

+
 EmbeddedSearchRequestParams::EmbeddedSearchRequestParams() {
 }
 
 EmbeddedSearchRequestParams::EmbeddedSearchRequestParams(const GURL& url) {
   const std::string& url_params(url.ref().empty()? url.query() : url.ref());
   url::Component query, key, value;
   query.len = static_cast<int>(url_params.size());
 
   const net::UnescapeRule::Type unescape_rules =
       net::UnescapeRule::SPOOFING_AND_CONTROL_CHARS |
@@ skipping 20 matching lines @@
           value_param, net::UnescapeRule::NORMAL);
     } else if (key_param == kAssistedQueryStatsKey) {
       assisted_query_stats = net::UnescapeAndDecodeUTF8URLComponent(
           value_param, net::UnescapeRule::NORMAL);
     }
   }
 }
 
 EmbeddedSearchRequestParams::~EmbeddedSearchRequestParams() {
 }