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

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

Issue 2901023002: Disable learning text suggestions if in incognito. (Closed)
Patch Set: Switched to explicit method 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | chrome/android/java/src/org/chromium/chrome/browser/omnibox/UrlBar.java » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/android/java/src/org/chromium/chrome/browser/omnibox/LocationBarLayout.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/omnibox/LocationBarLayout.java b/chrome/android/java/src/org/chromium/chrome/browser/omnibox/LocationBarLayout.java
index 12c8848c7ca170597f804ab9d52277fc560f734d..772ee675b04b7a578a0ff2d9b9a14ba3936d1946 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/omnibox/LocationBarLayout.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/omnibox/LocationBarLayout.java
@@ -2249,15 +2249,18 @@ public class LocationBarLayout extends FrameLayout
updateSecurityIcon(getSecurityLevel());
}
- /**
- * @return The Tab currently showing.
- */
@Override
public Tab getCurrentTab() {
if (mToolbarDataProvider == null) return null;
return mToolbarDataProvider.getTab();
}
+ @Override
+ public boolean allowKeyboardLearning() {
+ if (mToolbarDataProvider == null) return false;
+ return !mToolbarDataProvider.isIncognito();
+ }
+
private void initOmniboxResultsContainer() {
if (mOmniboxResultsContainer != null) return;
« no previous file with comments | « no previous file | chrome/android/java/src/org/chromium/chrome/browser/omnibox/UrlBar.java » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698