diff options
author | Attila Dusnoki <dati91@gmail.com> | 2017-02-01 09:38:16 +0100 |
---|---|---|
committer | Attila Dusnoki <dati91@gmail.com> | 2017-02-13 14:58:10 +0100 |
commit | cce362dc39fea02c3817d5b8623aaa03275781c8 (patch) | |
tree | 4ee6949e28596dbc0a982d16b96d93a65d2ac864 /components/script/dom/permissions.rs | |
parent | 74c1350518c2da765344137be1e0021eef4b6198 (diff) | |
download | servo-cce362dc39fea02c3817d5b8623aaa03275781c8.tar.gz servo-cce362dc39fea02c3817d5b8623aaa03275781c8.zip |
Fix popup build error
Diffstat (limited to 'components/script/dom/permissions.rs')
-rw-r--r-- | components/script/dom/permissions.rs | 29 |
1 files changed, 18 insertions, 11 deletions
diff --git a/components/script/dom/permissions.rs b/components/script/dom/permissions.rs index 104376e2e0c..5279f844962 100644 --- a/components/script/dom/permissions.rs +++ b/components/script/dom/permissions.rs @@ -256,17 +256,7 @@ pub fn get_descriptor_permission_state(permission_name: PermissionName , // if the feature is not allowed in non-secure contexcts, // and let the user decide to grant the permission or not. if !allowed_in_nonsecure_contexts(&permission_name) { - if cfg!(target_os = "linux") { - match tinyfiledialogs::message_box_yes_no(DIALOG_TITLE, - &format!("{} {:?} ?", QUERY_DIALOG_MESSAGE, permission_name), - MessageBoxIcon::Question, - YesNo::No) { - YesNo::Yes => return PermissionState::Granted, - YesNo::No => return PermissionState::Denied, - }; - } else { - return PermissionState::Denied; - } + return prompt_user(permission_name); } // TODO: Step 3: Store the invocation results @@ -276,6 +266,23 @@ pub fn get_descriptor_permission_state(permission_name: PermissionName , PermissionState::Granted } +#[cfg(target_os = "linux")] +fn prompt_user(permission_name: PermissionName) -> PermissionState { + match tinyfiledialogs::message_box_yes_no(DIALOG_TITLE, + &format!("{} {:?} ?", QUERY_DIALOG_MESSAGE, permission_name), + MessageBoxIcon::Question, + YesNo::No) { + YesNo::Yes => return PermissionState::Granted, + YesNo::No => return PermissionState::Denied, + }; +} + +#[cfg(not(target_os = "linux"))] +fn prompt_user(_permission_name: PermissionName) -> PermissionState { + // TODO popup only supported on linux + PermissionState::Denied +} + // https://w3c.github.io/permissions/#allowed-in-non-secure-contexts fn allowed_in_nonsecure_contexts(permission_name: &PermissionName) -> bool { match *permission_name { |