Don't try to create a verbatim match with an empty input.

components/omnibox/browser/clipboard_url_provider.cc

Index: components/omnibox/browser/clipboard_url_provider.cc
diff --git a/components/omnibox/browser/clipboard_url_provider.cc b/components/omnibox/browser/clipboard_url_provider.cc
index e4459e75747fc562b4693b6c554cdd8db61498bd..93d497692b4caa61cde797ebc591a55b5271ec15 100644
--- a/components/omnibox/browser/clipboard_url_provider.cc
+++ b/components/omnibox/browser/clipboard_url_provider.cc
@@ -30,24 +30,29 @@ ClipboardURLProvider::~ClipboardURLProvider() {}
 void ClipboardURLProvider::Start(const AutocompleteInput& input,
                                  bool minimal_changes) {
   matches_.clear();
+
+  // If the user started typing, do not offer clipboard based match.

[Peter Kasting, 2016/07/18 22:16:15]

Nit: I would not add these next two comments.  They simply restate the code
without adding any useful information such as motivation why we do this, so
they're just clutter.

   if (!input.from_omnibox_focus())
     return;
 
   GURL url;
+  // If the clipboard does not contain any URL, or the URL on the page is the
+  // same as the URL in the clipboard, early return.
   if (!clipboard_content_->GetRecentURLFromClipboard(&url) ||
       url == input.current_url())
     return;
 
   DCHECK(url.is_valid());
-  // Adds a default match. This match will be opened when the user presses "Go".
-  AutocompleteMatch verbatim_match = VerbatimMatchForURL(
-      client_, input, input.current_url(), history_url_provider_, -1);
-  if (verbatim_match.destination_url.is_valid())
+  // If the omnibox is not empty, add a default match.

[Peter Kasting, 2016/07/18 22:16:15]

Again, this adds detail that just restates the code.  If you think the old
comment is lacking, focus on describing why it matters that the omnibox is
non-empty, instead of just restating the code.

+  // This match will be opened when the user presses "Enter".
+  if (!input.text().empty()) {
+    AutocompleteMatch verbatim_match = VerbatimMatchForURL(
+        client_, input, input.current_url(), history_url_provider_, -1);
     matches_.push_back(verbatim_match);
+  }
 
-  // Add a clipboard match just below the verbatim match.
-  AutocompleteMatch match(this, verbatim_match.relevance - 1, false,
-                          AutocompleteMatchType::CLIPBOARD);
+  // Add the clipboard match. The relevance is 800 to beat ZeroSuggest results.
+  AutocompleteMatch match(this, 800, false, AutocompleteMatchType::CLIPBOARD);

[Peter Kasting, 2016/07/18 22:16:15]

I don't think we should hardcode 800 here.  If there is a verbatim match, using
its relevance - 1, as before, seems correct.  If there isn't, will there ever be
ZeroSuggest results either?  If not, then something like max(1, verbatim
relevance - 1) would probably be correct.

   match.destination_url = url;
   match.contents.assign(url_formatter::FormatUrl(
       url, url_formatter::kFormatUrlOmitAll, net::UnescapeRule::SPACES,