Send URLs on non-zero prefix suggest requests also.

chrome/browser/autocomplete/autocomplete_controller.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 "chrome/browser/autocomplete/autocomplete_controller.h"
 
 #include <set>
 #include <string>
 
 #include "base/command_line.h"
@@ skipping 292 matching lines @@
   }
 }
 
 void AutocompleteController::StartZeroSuggest(
     const GURL& url,
     AutocompleteInput::PageClassification page_classification,
     const string16& permanent_text) {
   if (zero_suggest_provider_ != NULL) {
     DCHECK(!in_start_);  // We should not be already running a query.
     in_zero_suggest_ = true;
     zero_suggest_provider_->StartZeroSuggest(

[Peter Kasting, 2013/09/19 21:20:06]

Doesn't this need to happen _after_ the current page URL gets set?  See comments
below anyway, as I think that will fix things.

[H Fung, 2013/09/21 01:13:25]

OmniboxEditModel::OnSetFocus calls this function, and only if the page has
loaded.

         url, page_classification, permanent_text);
   }
+  if (search_provider_)
+    search_provider_->SetCurrentPageUrl(url);

[Peter Kasting, 2013/09/19 21:20:06]

It's totally cryptic that we're making this seemingly non-zero-suggest-related
call to a non-zero-suggest provider in a function about zero suggest.

I would assume this would happen at some other time, e.g. when the current page
URL changes (which would mean this would need to be in OmniboxEditModel, I
believe).

[H Fung, 2013/09/21 01:13:25]

I moved this into OmniboxEditModel::OnSetFocus(), which I know is not the best
place (since it won't capture back buttons), but I don't see any function in
OmniboxEditModel that gets called when a page URL changes.

[Peter Kasting, 2013/09/21 01:15:55]

OmniboxEditModel::UpdatePermanentText(), which is called by
OmniboxViewXXX::Update().

 }
 
 void AutocompleteController::StopZeroSuggest() {
   if (zero_suggest_provider_ != NULL) {
     DCHECK(!in_start_);  // We should not be already running a query.
     zero_suggest_provider_->Stop(false);
   }
 }
 
 void AutocompleteController::DeleteMatch(const AutocompleteMatch& match) {
@@ skipping 309 matching lines @@
   // the disruptive effect of belated omnibox updates, updates that
   // come after the user has had to time to read the whole dropdown
   // and doesn't expect it to change.
   const int kStopTimeMS = 1500;
   stop_timer_.Start(FROM_HERE,
                     base::TimeDelta::FromMilliseconds(kStopTimeMS),
                     base::Bind(&AutocompleteController::Stop,
                                base::Unretained(this),
                                false));
 }