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

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

Issue 2891423002: Disable NTP URL bar while in VR on Android. (Closed)
Patch Set: javadoc Created 3 years, 7 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 static org.chromium.chrome.browser.toolbar.ToolbarPhone.URL_FOCUS_CHANGE_ ANIMATION_DURATION_MS; 7 import static org.chromium.chrome.browser.toolbar.ToolbarPhone.URL_FOCUS_CHANGE_ ANIMATION_DURATION_MS;
8 8
9 import android.Manifest; 9 import android.Manifest;
10 import android.animation.Animator; 10 import android.animation.Animator;
(...skipping 808 matching lines...) Expand 10 before | Expand all | Expand 10 after
819 */ 819 */
820 public void showCachedZeroSuggestResultsIfAvailable() { 820 public void showCachedZeroSuggestResultsIfAvailable() {
821 if (!mShowCachedZeroSuggestResults || mSuggestionList == null) return; 821 if (!mShowCachedZeroSuggestResults || mSuggestionList == null) return;
822 setSuggestionsListVisibility(true); 822 setSuggestionsListVisibility(true);
823 mSuggestionList.updateLayoutParams(); 823 mSuggestionList.updateLayoutParams();
824 } 824 }
825 825
826 /** 826 /**
827 * @param focusable Whether the url bar should be focusable. 827 * @param focusable Whether the url bar should be focusable.
828 */ 828 */
829 @Override
829 public void setUrlBarFocusable(boolean focusable) { 830 public void setUrlBarFocusable(boolean focusable) {
830 if (mUrlBar == null) return; 831 if (mUrlBar == null) return;
831 mUrlBar.setFocusable(focusable); 832 mUrlBar.setFocusable(focusable);
832 mUrlBar.setFocusableInTouchMode(focusable); 833 mUrlBar.setFocusableInTouchMode(focusable);
833 } 834 }
834 835
835 /** 836 /**
836 * @return The WindowDelegate for the LocationBar. This should be used for a ll Window related 837 * @return The WindowDelegate for the LocationBar. This should be used for a ll Window related
837 * state queries. 838 * state queries.
838 */ 839 */
(...skipping 1673 matching lines...) Expand 10 before | Expand all | Expand 10 after
2512 public void setTitleToPageTitle() { } 2513 public void setTitleToPageTitle() { }
2513 2514
2514 @Override 2515 @Override
2515 public void setShowTitle(boolean showTitle) { } 2516 public void setShowTitle(boolean showTitle) { }
2516 2517
2517 @Override 2518 @Override
2518 public boolean mustQueryUrlBarLocationForSuggestions() { 2519 public boolean mustQueryUrlBarLocationForSuggestions() {
2519 return DeviceFormFactor.isTablet(); 2520 return DeviceFormFactor.isTablet();
2520 } 2521 }
2521 } 2522 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698