From 8d464436b97de1aace612fbcb4cb40af6f3eec83 Mon Sep 17 00:00:00 2001 From: Ricardo Cerqueira Date: Sat, 18 Jan 2014 19:22:47 +0000 Subject: Implement client-side incognito mode Since WebView deprecated privateBrowsing in API 17 (and killed it for good in API 19), we were left without incognito in the Browser app. Follow the docs' recommendations and implement it with "manual" control of privacy-related engine options * Internally track incognito state for tabs * Disable all forms of storage and cache on private webviews * Disable all cookie activity while an incognito tab is active * Stop trying to use the deprecated "privateBrowsing" argument when creating webViews, always set to false Change-Id: I23f2e34ee125635bba8981f0711ba4986a9beaab --- src/com/android/browser/BrowserWebView.java | 32 +++++++++++++++++++++++++++-- 1 file changed, 30 insertions(+), 2 deletions(-) (limited to 'src/com/android/browser/BrowserWebView.java') diff --git a/src/com/android/browser/BrowserWebView.java b/src/com/android/browser/BrowserWebView.java index 3e3672c9d..eabb0e0ab 100644 --- a/src/com/android/browser/BrowserWebView.java +++ b/src/com/android/browser/BrowserWebView.java @@ -21,6 +21,7 @@ import android.graphics.Canvas; import android.util.AttributeSet; import android.view.View; import android.webkit.WebChromeClient; +import android.webkit.WebSettings; import android.webkit.WebStorage; import android.webkit.WebView; import android.webkit.WebViewClient; @@ -41,6 +42,7 @@ public class BrowserWebView extends WebView { private OnScrollChangedListener mOnScrollChangedListener; private WebChromeClient mWebChromeClient; private WebViewClient mWebViewClient; + private boolean mPrivateBrowsing = false; /** * @param context @@ -50,7 +52,9 @@ public class BrowserWebView extends WebView { */ public BrowserWebView(Context context, AttributeSet attrs, int defStyle, Map javascriptInterfaces, boolean privateBrowsing) { - super(context, attrs, defStyle, javascriptInterfaces, privateBrowsing); + super(context, attrs, defStyle, javascriptInterfaces, false); + // WebView doesn't support it, but save it here for internal use + mPrivateBrowsing = privateBrowsing; } /** @@ -60,7 +64,9 @@ public class BrowserWebView extends WebView { */ public BrowserWebView( Context context, AttributeSet attrs, int defStyle, boolean privateBrowsing) { - super(context, attrs, defStyle, privateBrowsing); + super(context, attrs, defStyle, false); + // WebView doesn't support it, but save it here for internal use + mPrivateBrowsing = privateBrowsing; } /** @@ -153,4 +159,26 @@ public class BrowserWebView extends WebView { super.destroy(); } + @Override + public boolean isPrivateBrowsingEnabled() { + return mPrivateBrowsing; + } + + /* Make sure the local webview remains in sync, since the engine won't retain that data */ + public void setPrivateBrowsing(boolean state) { + mPrivateBrowsing = state; + if (state) { + // Disable ALL the things + this.getSettings().setCacheMode(WebSettings.LOAD_NO_CACHE); + this.getSettings().setAppCacheEnabled(false); + this.getSettings().setDomStorageEnabled(false); + this.getSettings().setDatabaseEnabled(false); + this.getSettings().setGeolocationEnabled(false); + this.getSettings().setSaveFormData(false); + this.getSettings().setSavePassword(false); + this.getSettings().setSupportMultipleWindows(false); + this.getSettings().setAppCacheMaxSize(0); + this.clearHistory(); + } + } } -- cgit v1.2.3