• Raymes Khoury's avatar
    Update the UI to reflect the notifications permission for the DSE · 6deaae48
    Raymes Khoury authored
    This updates 2 UI surfaces to reflect notifications permissions set for
    the DSE. Specifically:
    1) In android Site Details, when toggle the DSE permission for
    notififications, it will show the options "Allow/Block for the current
    search engine". 2 cases are handled here: one when that setting links
    directly into Notification Channels on Android O+ and the other in the
    case where the permission is handled directly in Chrome.
    
    2) In Settings>Search Engines, underneath the currently selected search
    engine, it will display a link indicating whether
    notifications/geolocation are allowed. The link will take the user to
    the Site Details for the search engine. Several changes have been made
    to the behavior here:
    -The link used to indicate if location was disabled. Now, if
    notifications are location are disabled, nothing will be shown in the
    text for those permissions.
    -The link used to take users directly to Android Location Settings when
    the text indicated that system location was disabled. But now due to the
    link handling combined permissions, it always takes the user directly to
    Site Details for the search engine, where the user can follow a link
    into Android Location Settings.
    
    Bug: 780344
    Change-Id: I41e0dd1028ad289141d1632875d30373b528f17c
    Reviewed-on: https://chromium-review.googlesource.com/799534
    Commit-Queue: Raymes Khoury <raymes@chromium.org>
    Reviewed-by: default avatarBen Wells <benwells@chromium.org>
    Reviewed-by: default avatarTed Choc <tedchoc@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#523606}
    6deaae48
android_chrome_strings.grd 225 KB