Add title to current page in zero suggest.

components/omnibox/browser/history_url_provider.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 "components/omnibox/browser/history_url_provider.h"
 
 #include <algorithm>
 
 #include "base/bind.h"
 #include "base/command_line.h"
+#include "base/feature_list.h"
 #include "base/location.h"
 #include "base/macros.h"
 #include "base/metrics/histogram_macros.h"
 #include "base/single_thread_task_runner.h"
 #include "base/strings/string_util.h"
 #include "base/strings/utf_string_conversions.h"
 #include "base/threading/sequenced_task_runner_handle.h"
 #include "base/time/time.h"
 #include "base/trace_event/trace_event.h"
 #include "components/bookmarks/browser/bookmark_utils.h"
@@ skipping 588 matching lines @@
                                                match.contents,
                                                ACMatchClassification::URL,
                                                &match.contents_class);
     } else {
       AutocompleteMatch::ClassifyLocationInString(
           best_prefix->prefix.length() - offset, input.text().length(),
           match.contents.length(), ACMatchClassification::URL,
           &match.contents_class);
     }
   }
-
+  if (base::FeatureList::IsEnabled(omnibox::kDisplayTitleForCurrentUrl)) {

[Mark P, 2017/04/12 05:07:56]

I don't this this is right.  SuggestExactInput can be called on things like
user input = "http://foo.com", even if the user is currently on
"http://bar.com".  We don't want to display bar.com's title on foo.com".

I think I was wrong when I suggested pushing this code all the way into
SuggestExactInput.  VerbatimMatchForURL() I guess is the right place in the
hierarchy.  (It must be, as we've now looked at two of the three possibilities!)
 Sorry for giving bad advice on my previous review.

[gcomanici, 2017/04/12 16:16:30]

Done.

+    match.description = input.current_title();
+    AutocompleteMatch::ClassifyLocationInString(
+        base::string16::npos, 0, match.description.length(),
+        ACMatchClassification::NONE, &match.description_class);
+  }
   return match;
 }
 
 void HistoryURLProvider::ExecuteWithDB(HistoryURLProviderParams* params,
                                        history::HistoryBackend* backend,
                                        history::URLDatabase* db) {
   // We may get called with a null database if it couldn't be properly
   // initialized.
   if (!db) {
     params->failed = true;
@@ skipping 557 matching lines @@
     AutocompleteMatch::ClassifyLocationInString(base::string16::npos, 0,
         match.contents.length(), ACMatchClassification::URL,
         &match.contents_class);
   }
   match.description = info.title();
   match.description_class =
       ClassifyDescription(params.input.text(), match.description);
   RecordAdditionalInfoFromUrlRow(info, &match);
   return match;
 }