New issue
Advanced search Search tips

Issue 821947 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Closed: Mar 2018
EstimatedDays: ----
NextAction: ----
OS: Linux , Android , Windows , Mac
Pri: 1
Type: Bug



Sign in to add a comment

Permission.query for "clipboard-read" fails

Project Member Reported by garykac@chromium.org, Mar 14 2018

Issue description

It returns the following error:

"Failed to read the 'query' property from 'Permissions': Async Clipboard flag is not enabled."

The permission code checks the AsyncClipboardAPI feature flag and fails if it is not set. Currently this flag is only set for "experimental" so release channels will not be able to query this permissions.

This feature check should be removed since it is now being used to enable/disable the unshipped portions of the Async Clipboard API.
 
See attached repro + output.
Screen Shot 2018-03-14 at 3.58.13 PM.png
52.6 KB View Download
Labels: -Pri-3 M-66 OS-Android OS-Linux OS-Mac OS-Windows Pri-1
Project Member

Comment 3 by bugdroid1@chromium.org, Mar 15 2018

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/b94978cfefdfbd0eb5f2cc6b6ef667da52763240

commit b94978cfefdfbd0eb5f2cc6b6ef667da52763240
Author: Gary Kacmarcik <garykac@chromium.org>
Date: Thu Mar 15 21:34:55 2018

Remove flag check for clipboard-read/write permission.query

The AsyncClipboardAPI feature flag used to control access to the entire
async clipboard API, but (since we're now shipping the text portions of
the API) its meaning has been changed to control access only to the parts
of the API that are still under development (e.g., image and delayed
generation support).

The Clipboard permission is shipping as part of the text APIs, so it
should no longer be checking this feature flag.

Bug:  677564 , 821947 
Change-Id: Ia57b57df3eae03c2d4372cf5e7472ef1d0532f49
Reviewed-on: https://chromium-review.googlesource.com/963287
Reviewed-by: Daniel Cheng <dcheng@chromium.org>
Commit-Queue: Gary Kacmarcik <garykac@chromium.org>
Cr-Commit-Position: refs/heads/master@{#543504}
[modify] https://crrev.com/b94978cfefdfbd0eb5f2cc6b6ef667da52763240/third_party/WebKit/Source/modules/permissions/Permissions.cpp

Labels: Merge-Request-66
Status: Fixed (was: Started)
Project Member

Comment 6 by sheriffbot@chromium.org, Mar 15 2018

Labels: -Merge-Request-66 Merge-Review-66 Hotlist-Merge-Review
This bug requires manual review: M66 has already been promoted to the beta branch, so this requires manual review
Please contact the milestone owner if you have questions.
Owners: cmasso@(Android), cmasso@(iOS), josafat@(ChromeOS), abdulsyed@(Desktop)

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Labels: TE-Verified-67.0.3372.0 TE-Verified-M67
Able to reproduce the issue on win-10, mac 10.13.3 and ubuntu 14.04 using chrome version #67.0.3371.0.

Verified the fix on Mac 10.13.3, Win-10 and Ubuntu 14.04 using Chrome version #67.0.3372.0 as per the comment #0 and #1.
Attaching screen shot for reference.
Observed that permission.query for "clipboard-read" pass.
Hence, the fix is working as expected. 
Adding the verified labels.

Thanks...!!
821947.png
466 KB View Download

Comment 8 by cmasso@google.com, Mar 19 2018

Labels: -Hotlist-Merge-Review -Merge-Review-66 Merge-Approved-66
Project Member

Comment 9 by bugdroid1@chromium.org, Mar 19 2018

Labels: -merge-approved-66 merge-merged-3359
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/110e2347eff57cf1c97b18663e75ad37f2c96a3f

commit 110e2347eff57cf1c97b18663e75ad37f2c96a3f
Author: Gary Kacmarcik <garykac@chromium.org>
Date: Mon Mar 19 17:50:31 2018

Remove flag check for clipboard-read/write permission.query

The AsyncClipboardAPI feature flag used to control access to the entire
async clipboard API, but (since we're now shipping the text portions of
the API) its meaning has been changed to control access only to the parts
of the API that are still under development (e.g., image and delayed
generation support).

The Clipboard permission is shipping as part of the text APIs, so it
should no longer be checking this feature flag.

Bug:  677564 , 821947 
Change-Id: Ia57b57df3eae03c2d4372cf5e7472ef1d0532f49
Reviewed-on: https://chromium-review.googlesource.com/963287
Reviewed-by: Daniel Cheng <dcheng@chromium.org>
Commit-Queue: Gary Kacmarcik <garykac@chromium.org>
Cr-Original-Commit-Position: refs/heads/master@{#543504}(cherry picked from commit b94978cfefdfbd0eb5f2cc6b6ef667da52763240)
Reviewed-on: https://chromium-review.googlesource.com/969281
Reviewed-by: Gary Kacmarcik <garykac@chromium.org>
Cr-Commit-Position: refs/branch-heads/3359@{#312}
Cr-Branched-From: 66afc5e5d10127546cc4b98b9117aff588b5e66b-refs/heads/master@{#540276}
[modify] https://crrev.com/110e2347eff57cf1c97b18663e75ad37f2c96a3f/third_party/WebKit/Source/modules/permissions/Permissions.cpp

Labels: TE-Verified-M66 TE-Verified-66.0.3359.45
Able to reproduce the issue on win-10, mac 10.13.3 and ubuntu 14.04 using chrome version #67.0.3371.0.

Verified the fix on Mac 10.13.3, Win-10 and Ubuntu 14.04 using Chrome version #66.0.3359.45 as per the comment #0 and #1.
Attaching screen shot for reference.
Observed that permission.query for "clipboard-read" pass.
Hence, the fix is working as expected. 
Adding the verified labels.

Thanks...!!
821947@M66.png
1.0 MB View Download

Sign in to add a comment