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

Issues Search
    My Issues | Starred     Open | Closed | All

Issue 269053005: Remove CSS media features -webkit-transform-2d, -webkit-animation and -webkit-view-mode (Closed)

Created:
6 years, 7 months ago by philipj_slow
Modified:
6 years, 7 months ago
CC:
blink-reviews, kenneth.christiansen, Yoav Weiss, blink-reviews-css, ed+blinkwatch_opera.com, dglazkov+blink, apavlov+blink_chromium.org, darktears, rune+blink, rwlbuis
Visibility:
Public.

Description

Remove CSS media features -webkit-transform-2d, -webkit-animation and -webkit-view-mode Intent to Deprecate and Remove: https://groups.google.com/a/chromium.org/d/msg/blink-dev/lYZC-Xqf_Gw/D6Or7mZHwr4J BUG=none Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=173494

Patch Set 1 #

Patch Set 2 : update tests #

Unified diffs Side-by-side diffs Delta from patch set Stats (+3 lines, -172 lines) Patch
M LayoutTests/fast/media/matchmedium-query-api.html View 1 1 chunk +0 lines, -3 lines 0 comments Download
M LayoutTests/fast/media/matchmedium-query-api-expected.txt View 1 1 chunk +0 lines, -2 lines 0 comments Download
M LayoutTests/fast/media/media-query-list-01.html View 1 1 chunk +0 lines, -3 lines 0 comments Download
M LayoutTests/fast/media/media-query-list-01-expected.txt View 1 1 chunk +0 lines, -2 lines 0 comments Download
D LayoutTests/fast/media/mq-animation.html View 1 1 chunk +0 lines, -23 lines 0 comments Download
D LayoutTests/fast/media/mq-animation-expected.html View 1 1 chunk +0 lines, -7 lines 0 comments Download
D LayoutTests/fast/media/mq-transform-01.html View 1 1 chunk +0 lines, -14 lines 0 comments Download
D LayoutTests/fast/media/mq-transform-01-expected.html View 1 1 chunk +0 lines, -5 lines 0 comments Download
D LayoutTests/fast/media/mq-transform-03.html View 1 1 chunk +0 lines, -14 lines 0 comments Download
D LayoutTests/fast/media/mq-transform-03-expected.html View 1 1 chunk +0 lines, -5 lines 0 comments Download
D LayoutTests/fast/media/mq-transform-04.html View 1 1 chunk +0 lines, -15 lines 0 comments Download
D LayoutTests/fast/media/mq-transform-04-expected.html View 1 1 chunk +0 lines, -5 lines 0 comments Download
M LayoutTests/fast/media/mq-unknown-ident-values.html View 1 2 chunks +2 lines, -6 lines 0 comments Download
M LayoutTests/fast/media/mq-unknown-ident-values-expected.html View 1 1 chunk +0 lines, -1 line 0 comments Download
M Source/core/css/CSSValueKeywords.in View 1 chunk +0 lines, -7 lines 0 comments Download
M Source/core/css/MediaFeatureNames.in View 1 chunk +0 lines, -3 lines 0 comments Download
M Source/core/css/MediaQueryEvaluator.cpp View 2 chunks +0 lines, -34 lines 0 comments Download
M Source/core/css/MediaQueryExp.cpp View 4 chunks +1 line, -20 lines 0 comments Download
M Source/core/frame/UseCounter.h View 1 1 chunk +0 lines, -3 lines 0 comments Download

Messages

Total messages: 4 (0 generated)
jochen (gone - plz use gerrit)
lgtm
6 years, 7 months ago (2014-05-07 08:06:35 UTC) #1
philipj_slow
The CQ bit was checked by philipj@opera.com
6 years, 7 months ago (2014-05-07 08:09:24 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/philipj@opera.com/269053005/20001
6 years, 7 months ago (2014-05-07 08:10:41 UTC) #3
commit-bot: I haz the power
6 years, 7 months ago (2014-05-07 08:24:38 UTC) #4
Message was sent while issue was closed. 
Change committed as 173494

Powered by Google App Engine
This is Rietveld 408576698