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

Issue 60873002: Removing the -webkit prefix from CSS Regions-Shapes tests (Closed)

Created:
7 years, 1 month ago by Zoltan
Modified:
7 years, 1 month ago
Reviewers:
pdr.
CC:
blink-reviews
Visibility:
Public.

Description

Removing the -webkit prefix from CSS Regions-Shapes tests Layout tests should use the new, unprefixed CSS Shapes properties. The prefix has been removed for CSS Shapes in https://src.chromium.org/viewvc/blink?view=rev&revision=160750 Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=161407

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+30 lines, -30 lines) Patch
M LayoutTests/fast/regions/shape-inside/shape-inside-on-additional-regions.html View 1 chunk +1 line, -1 line 0 comments Download
M LayoutTests/fast/regions/shape-inside/shape-inside-on-first-region-block-content.html View 1 chunk +1 line, -1 line 0 comments Download
M LayoutTests/fast/regions/shape-inside/shape-inside-on-first-region-inline-content.html View 1 chunk +1 line, -1 line 0 comments Download
M LayoutTests/fast/regions/shape-inside/shape-inside-on-multiple-autoheight-regions.html View 1 chunk +1 line, -1 line 0 comments Download
M LayoutTests/fast/regions/shape-inside/shape-inside-on-regions.html View 1 chunk +2 lines, -2 lines 0 comments Download
M LayoutTests/fast/regions/shape-inside/shape-inside-on-regions-block-content-basic-overflow-shape-top-offset.html View 1 chunk +1 line, -1 line 0 comments Download
M LayoutTests/fast/regions/shape-inside/shape-inside-on-regions-block-content-overflow-bottom-positioned-multiple-shapes.html View 1 chunk +2 lines, -2 lines 0 comments Download
M LayoutTests/fast/regions/shape-inside/shape-inside-on-regions-block-content-overflow-multiple-shapes.html View 1 chunk +2 lines, -2 lines 0 comments Download
M LayoutTests/fast/regions/shape-inside/shape-inside-on-regions-block-content-polygon-with-adjustment.html View 1 chunk +1 line, -1 line 0 comments Download
M LayoutTests/fast/regions/shape-inside/shape-inside-on-regions-block-content-polygon-with-adjustment-expected.html View 1 chunk +1 line, -1 line 0 comments Download
M LayoutTests/fast/regions/shape-inside/shape-inside-on-regions-expected.html View 1 chunk +2 lines, -2 lines 0 comments Download
M LayoutTests/fast/regions/shape-inside/shape-inside-on-regions-inline-content.html View 1 chunk +2 lines, -2 lines 0 comments Download
M LayoutTests/fast/regions/shape-inside/shape-inside-on-regions-inline-content-basic-overflow-shape-top-offset.html View 1 chunk +1 line, -1 line 0 comments Download
M LayoutTests/fast/regions/shape-inside/shape-inside-on-regions-inline-content-overflow-bottom-positioned-multiple-shapes.html View 1 chunk +2 lines, -2 lines 0 comments Download
M LayoutTests/fast/regions/shape-inside/shape-inside-on-regions-inline-content-overflow-multiple-shapes.html View 1 chunk +2 lines, -2 lines 0 comments Download
M LayoutTests/fast/regions/shape-inside/shape-inside-on-regions-inline-content-polygon-with-adjustment.html View 1 chunk +1 line, -1 line 0 comments Download
M LayoutTests/fast/regions/shape-inside/shape-inside-on-regions-inline-content-polygon-with-adjustment-expected.html View 1 chunk +1 line, -1 line 0 comments Download
M LayoutTests/fast/regions/shape-inside/shape-inside-on-second-region-block-content.html View 1 chunk +1 line, -1 line 0 comments Download
M LayoutTests/fast/regions/shape-inside/shape-inside-on-second-region-inline-content.html View 1 chunk +1 line, -1 line 0 comments Download
M LayoutTests/fast/regions/shape-inside/shape-inside-recursive-layout.html View 1 chunk +1 line, -1 line 0 comments Download
M LayoutTests/fast/regions/shape-inside/shape-inside-recursive-layout-expected.html View 1 chunk +1 line, -1 line 0 comments Download
M LayoutTests/fast/regions/shape-inside/shape-inside-with-region-borders.html View 1 chunk +1 line, -1 line 0 comments Download
M LayoutTests/fast/regions/shape-inside/shape-inside-with-region-padding.html View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 3 (0 generated)
pdr.
LGTM but fix the typo in the title
7 years, 1 month ago (2013-11-05 23:43:50 UTC) #1
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/zoltan@adobe.com/60873002/1
7 years, 1 month ago (2013-11-05 23:47:44 UTC) #2
commit-bot: I haz the power
7 years, 1 month ago (2013-11-06 01:39:59 UTC) #3
Message was sent while issue was closed.
Change committed as 161407

Powered by Google App Engine
This is Rietveld 408576698