Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(330)

Side by Side Diff: chrome/android/java/src/org/chromium/chrome/browser/omnibox/LocationBarPhone.java

Issue 2652883002: Omnibox results show correctly for Chrome Home (Closed)
Patch Set: address comments Created 3 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 package org.chromium.chrome.browser.omnibox; 5 package org.chromium.chrome.browser.omnibox;
6 6
7 import android.animation.Animator; 7 import android.animation.Animator;
8 import android.animation.AnimatorListenerAdapter; 8 import android.animation.AnimatorListenerAdapter;
9 import android.animation.AnimatorSet; 9 import android.animation.AnimatorSet;
10 import android.animation.ObjectAnimator; 10 import android.animation.ObjectAnimator;
(...skipping 243 matching lines...) Expand 10 before | Expand all | Expand 10 after
254 // As the position of the navigation icon has changed, ensure the su ggestions are 254 // As the position of the navigation icon has changed, ensure the su ggestions are
255 // updated to reflect the new position. 255 // updated to reflect the new position.
256 if (getSuggestionList() != null && getSuggestionList().isShown()) { 256 if (getSuggestionList() != null && getSuggestionList().isShown()) {
257 getSuggestionList().invalidateSuggestionViews(); 257 getSuggestionList().invalidateSuggestionViews();
258 } 258 }
259 } 259 }
260 setUrlFocusChangeInProgress(false); 260 setUrlFocusChangeInProgress(false);
261 261
262 NewTabPage ntp = getToolbarDataProvider().getNewTabPageForCurrentTab(); 262 NewTabPage ntp = getToolbarDataProvider().getNewTabPageForCurrentTab();
263 if (hasFocus && ntp != null && ntp.isLocationBarShownInNTP()) { 263 if (hasFocus && ntp != null && ntp.isLocationBarShownInNTP()) {
264 fadeInOmniboxResultsContainerBackground(); 264 showFadingOverlay();
265 } 265 }
266 } 266 }
267 267
268 @Override 268 @Override
269 protected void updateButtonVisibility() { 269 protected void updateButtonVisibility() {
270 updateDeleteButtonVisibility(); 270 updateDeleteButtonVisibility();
271 updateMicButtonVisiblity(mUrlFocusChangePercent); 271 updateMicButtonVisiblity(mUrlFocusChangePercent);
272 } 272 }
273 273
274 @Override 274 @Override
(...skipping 76 matching lines...) Expand 10 before | Expand all | Expand 10 after
351 351
352 mMenuBadgeAnimatorSet.start(); 352 mMenuBadgeAnimatorSet.start();
353 } 353 }
354 354
355 public void cancelAppMenuUpdateBadgeAnimation() { 355 public void cancelAppMenuUpdateBadgeAnimation() {
356 if (mIsMenuBadgeAnimationRunning && mMenuBadgeAnimatorSet != null) { 356 if (mIsMenuBadgeAnimationRunning && mMenuBadgeAnimatorSet != null) {
357 mMenuBadgeAnimatorSet.cancel(); 357 mMenuBadgeAnimatorSet.cancel();
358 } 358 }
359 } 359 }
360 } 360 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698