Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(295)

Issue 2334333002: Enable GeometryInterfaces with CompositorWorker (Closed)

Created:
4 years, 3 months ago by flackr
Modified:
4 years, 3 months ago
Reviewers:
pdr.
CC:
blink-reviews, chrishtr, chromium-reviews, Ian Vollick
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Enable GeometryInterfaces with CompositorWorker CompositorWorker uses DOMMatrix to represent transforms so we should enable GeometryInterfaces with CompositorWorker. Since CompositorWorker is already enabled on experimental, GeometryInterfaces will be enabled with experimental as well. Ideally we could restrict the visibility to just CompositorWorker, though that would affect explicitly enabling GeometryInterfaces so we're just enabling it in general with experimental web platform features. BUG=None Committed: https://crrev.com/28000ee372791e81ed566cb8716f8cf27367e3c3 Cr-Commit-Position: refs/heads/master@{#418414}

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -1 line) Patch
M third_party/WebKit/Source/platform/RuntimeEnabledFeatures.in View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 9 (4 generated)
flackr
Hey, I've chatted with Ian about this already, since CompositorWorker uses DOMMatrix we should enable ...
4 years, 3 months ago (2016-09-13 20:03:36 UTC) #2
pdr.
On 2016/09/13 at 20:03:36, flackr wrote: > Hey, > > I've chatted with Ian about ...
4 years, 3 months ago (2016-09-13 21:53:42 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2334333002/1
4 years, 3 months ago (2016-09-13 22:07:25 UTC) #6
commit-bot: I haz the power
Committed patchset #1 (id:1)
4 years, 3 months ago (2016-09-13 23:38:32 UTC) #7
commit-bot: I haz the power
4 years, 3 months ago (2016-09-13 23:41:47 UTC) #9
Message was sent while issue was closed.
Patchset 1 (id:??) landed as
https://crrev.com/28000ee372791e81ed566cb8716f8cf27367e3c3
Cr-Commit-Position: refs/heads/master@{#418414}

Powered by Google App Engine
This is Rietveld 408576698