Commit 9beb5e65 authored by jered@chromium.org's avatar jered@chromium.org

InstantExtended: Hide blank query page off NTP.

Otherwise we show a Google logo and identity on third-party websites
when the user backspaces away the query.

BUG=


Review URL: https://chromiumcodereview.appspot.com/11308284

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@170758 0039d316-1c4b-4281-b951-d872f2087c98
parent 1be78145
......@@ -225,12 +225,14 @@ bool InstantController::Update(const AutocompleteMatch& match,
// In non-extended mode, #1 to #7 call Hide(). #8 calls loader_->Update().
//
// In extended mode, #2 and #4 call Hide(). #1 doesn't Hide() as the preview
// may be showing custom NTP content, but doesn't Update() either. #3 and #7
// may be showing custom NTP content, but doesn't Update() either. #3 calls
// Hide() unless on the NTP _and_ sends a blank query; otherwise #3 and #7
// don't Hide(), but send a blank query to Update(). #8 calls Update().
if (extended_enabled_) {
if (!omnibox_popup_is_open) {
if (!full_text.empty()) {
if (!full_text.empty() ||
(user_input_in_progress && !search_mode_.is_origin_ntp())) {
Hide(true);
return false;
}
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment