DescriptionRevert of Fix the Copresence permission. (patchset #1 of https://codereview.chromium.org/523433002/)
Reason for revert:
This change causes a test failure:
http://build.chromium.org/p/chromium.chromiumos/builders/Linux%20ChromiumOS%20Tests%20%281%29/builds/48008/steps/unit_tests/logs/stdio
Error:
[ RUN ] PermissionsTest.PermissionMessages
../../chrome/common/extensions/permissions/permission_set_unittest.cc:781: Failure
Value of: permission_info->message_id()
Actual: 65
Expected: PermissionMessage::kNone
Which is: 1
unexpected message_id for copresence
Looks like this change is the cause.
Original issue's description:
> Fix the Copresence permission.
>
> The initializer for the Copresence permission is malformed. It specifies the
> message id instead of the flag.
>
> R=kalman@chromium.org
> BUG=408843
>
> Committed to pending queue: https://chromium.googlesource.com/chromium/src/+/c2a41e8
TBR=kalman@chromium.org,rkc@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=408843
Committed: https://crrev.com/7b723d5ca9bcd0b3e7619c08b4284e97ffc25d31
Cr-Commit-Position: refs/heads/master@{#292532}
Patch Set 1 #
Created: 6 years, 3 months ago
(Patch set is too large to download)
Messages
Total messages: 4 (0 generated)
|