diff options
author | Tao Bai <michaelbai@google.com> | 2014-05-13 13:55:52 -0700 |
---|---|---|
committer | Tao Bai <michaelbai@google.com> | 2014-05-14 11:04:42 -0700 |
commit | e1b6b3b2f9c0dbf201234162101023770835df9e (patch) | |
tree | 5169a99decf4690eda805a408eb81e44899e518a /src/com/android/browser/Tab.java | |
parent | 86b21dc331237980b9df44df19e6f202e6049cbb (diff) | |
download | packages_apps_Browser-e1b6b3b2f9c0dbf201234162101023770835df9e.zip packages_apps_Browser-e1b6b3b2f9c0dbf201234162101023770835df9e.tar.gz packages_apps_Browser-e1b6b3b2f9c0dbf201234162101023770835df9e.tar.bz2 |
Use general permission API.
Geolocation related code is untouched.
PermissionsPrompt is almost same as GeolocationPermissionPrompt, except
- Remember checkbox is hidden as we don't has this feature in framework now.
- The resource name is appended according the request.
- Remove toast.
BUG: 13699047
Change-Id: I1f3f1908bf9c7131ecf2d060fc79d3e5c8842f94
Diffstat (limited to 'src/com/android/browser/Tab.java')
-rw-r--r-- | src/com/android/browser/Tab.java | 33 |
1 files changed, 33 insertions, 0 deletions
diff --git a/src/com/android/browser/Tab.java b/src/com/android/browser/Tab.java index ed20e67..668786c 100644 --- a/src/com/android/browser/Tab.java +++ b/src/com/android/browser/Tab.java @@ -50,7 +50,9 @@ import android.webkit.BrowserDownloadListener; import android.webkit.ClientCertRequest; import android.webkit.ConsoleMessage; import android.webkit.GeolocationPermissions; +import android.webkit.GeolocationPermissions.Callback; import android.webkit.HttpAuthHandler; +import android.webkit.PermissionRequest; import android.webkit.SslErrorHandler; import android.webkit.URLUtil; import android.webkit.ValueCallback; @@ -135,6 +137,8 @@ class Tab implements PictureListener { // The Geolocation permissions prompt private GeolocationPermissionsPrompt mGeolocationPermissionsPrompt; + // The permissions prompt + private PermissionsPrompt mPermissionsPrompt; // Main WebView wrapper private View mContainer; // Main WebView @@ -912,6 +916,19 @@ class Tab implements PictureListener { } } + @Override + public void onPermissionRequest(PermissionRequest request) { + if (!mInForeground) return; + getPermissionsPrompt().show(request); + } + + @Override + public void onPermissionRequestCanceled(PermissionRequest request) { + if (mInForeground && mPermissionsPrompt != null) { + mPermissionsPrompt.hide(); + } + } + /* Adds a JavaScript error message to the system log and if the JS * console is enabled in the about:debug options, to that console * also. @@ -1232,6 +1249,10 @@ class Tab implements PictureListener { mGeolocationPermissionsPrompt.hide(); } + if (mPermissionsPrompt != null) { + mPermissionsPrompt.hide(); + } + mWebViewController.onSetWebView(this, w); if (mMainView != null) { @@ -1539,6 +1560,18 @@ class Tab implements PictureListener { } /** + * @return The permissions prompt for this tab. + */ + PermissionsPrompt getPermissionsPrompt() { + if (mPermissionsPrompt == null) { + ViewStub stub = (ViewStub) mContainer + .findViewById(R.id.permissions_prompt); + mPermissionsPrompt = (PermissionsPrompt) stub.inflate(); + } + return mPermissionsPrompt; + } + + /** * @return The application id string */ String getAppId() { |