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

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

Issue 2652883002: Omnibox results show correctly for Chrome Home (Closed)
Patch Set: nit Created 3 years, 11 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.app.Activity; 7 import android.app.Activity;
8 import android.view.View; 8 import android.view.View;
9 import android.view.ViewGroup; 9 import android.view.ViewGroup;
10 import android.view.Window; 10 import android.view.Window;
11 11
12 import org.chromium.chrome.browser.WindowDelegate; 12 import org.chromium.chrome.browser.WindowDelegate;
13 import org.chromium.chrome.browser.appmenu.AppMenuButtonHelper; 13 import org.chromium.chrome.browser.appmenu.AppMenuButtonHelper;
14 import org.chromium.chrome.browser.ntp.NewTabPage; 14 import org.chromium.chrome.browser.ntp.NewTabPage;
15 import org.chromium.chrome.browser.omnibox.UrlBar.UrlBarDelegate; 15 import org.chromium.chrome.browser.omnibox.UrlBar.UrlBarDelegate;
16 import org.chromium.chrome.browser.profiles.Profile; 16 import org.chromium.chrome.browser.profiles.Profile;
17 import org.chromium.chrome.browser.tab.Tab; 17 import org.chromium.chrome.browser.tab.Tab;
18 import org.chromium.chrome.browser.toolbar.ActionModeController; 18 import org.chromium.chrome.browser.toolbar.ActionModeController;
19 import org.chromium.chrome.browser.toolbar.ActionModeController.ActionBarDelegat e; 19 import org.chromium.chrome.browser.toolbar.ActionModeController.ActionBarDelegat e;
20 import org.chromium.chrome.browser.toolbar.Toolbar; 20 import org.chromium.chrome.browser.toolbar.Toolbar;
21 import org.chromium.chrome.browser.toolbar.ToolbarActionModeCallback; 21 import org.chromium.chrome.browser.toolbar.ToolbarActionModeCallback;
22 import org.chromium.chrome.browser.toolbar.ToolbarDataProvider; 22 import org.chromium.chrome.browser.toolbar.ToolbarDataProvider;
23 import org.chromium.chrome.browser.widget.BottomSheet;
23 import org.chromium.ui.base.WindowAndroid; 24 import org.chromium.ui.base.WindowAndroid;
24 25
25 /** 26 /**
26 * Container that holds the {@link UrlBar} and SSL state related with the curren t {@link Tab}. 27 * Container that holds the {@link UrlBar} and SSL state related with the curren t {@link Tab}.
27 */ 28 */
28 public interface LocationBar extends UrlBarDelegate { 29 public interface LocationBar extends UrlBarDelegate {
29 30
30 /** 31 /**
31 * Handles native dependent initialization for this class. 32 * Handles native dependent initialization for this class.
32 */ 33 */
(...skipping 40 matching lines...) Expand 10 before | Expand all | Expand 10 after
73 * @param updateUrl Whether to update the URL as a result of the this call. 74 * @param updateUrl Whether to update the URL as a result of the this call.
74 */ 75 */
75 void updateLoadingState(boolean updateUrl); 76 void updateLoadingState(boolean updateUrl);
76 77
77 /** 78 /**
78 * Sets the {@link ToolbarDataProvider} to be used for accessing {@link Tool bar} state. 79 * Sets the {@link ToolbarDataProvider} to be used for accessing {@link Tool bar} state.
79 */ 80 */
80 void setToolbarDataProvider(ToolbarDataProvider model); 81 void setToolbarDataProvider(ToolbarDataProvider model);
81 82
82 /** 83 /**
84 * Set the bottom sheet for Chrome Home. This will be null if Chrome Home is disabled.
Ted C 2017/01/25 19:27:02 Would this even need to be called if Chrome Home i
mdjones 2017/01/26 00:34:39 It won't; updated doc.
85 * @param sheet The bottom sheet for Chrome Home if it exists.
86 */
87 void setBottomSheet(BottomSheet sheet);
88
89 /**
83 * Sets the menu helper that should be used if there is a menu button in {@l ink LocationBar}. 90 * Sets the menu helper that should be used if there is a menu button in {@l ink LocationBar}.
84 * @param helper The helper to be used. 91 * @param helper The helper to be used.
85 */ 92 */
86 void setMenuButtonHelper(AppMenuButtonHelper helper); 93 void setMenuButtonHelper(AppMenuButtonHelper helper);
87 94
88 /** 95 /**
89 * @return The anchor view that should be used for the app menu. Null if the re is no menu in 96 * @return The anchor view that should be used for the app menu. Null if the re is no menu in
90 * {@link LocationBar} for the current configuration. 97 * {@link LocationBar} for the current configuration.
91 */ 98 */
92 View getMenuAnchor(); 99 View getMenuAnchor();
(...skipping 70 matching lines...) Expand 10 before | Expand all | Expand 10 after
163 */ 170 */
164 void hideSuggestions(); 171 void hideSuggestions();
165 172
166 /** 173 /**
167 * Sets the callback to be used by default for text editing action bar. 174 * Sets the callback to be used by default for text editing action bar.
168 * @param callback The callback to use. 175 * @param callback The callback to use.
169 */ 176 */
170 void setDefaultTextEditActionModeCallback(ToolbarActionModeCallback callback ); 177 void setDefaultTextEditActionModeCallback(ToolbarActionModeCallback callback );
171 178
172 } 179 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698