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

Issues Search
    My Issues | Starred     Open | Closed | All

Issue 1343443002: Revert "Allow whitespace inside anonymous table cells" from M45 (Closed)

Created:
5 years, 3 months ago by rhogan
Modified:
5 years, 3 months ago
CC:
blink-reviews, blink-reviews-dom_chromium.org, dglazkov+blink, eae+blinkwatch, rwlbuis, sof
Target Ref:
refs/branch-heads/chromium/2454
Project:
blink
Visibility:
Public.

Description

Revert "Allow whitespace inside anonymous table cells" This reverts commit c773af8cafb48d3ab0786e4214adb1bc0f828a6b. TBR=mstensho@opera.com BUG=528165, 515771, 473832 Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=202161

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -121 lines) Patch
M LayoutTests/fast/block/float/float-not-removed-from-pre-block-expected.txt View 1 chunk +1 line, -1 line 0 comments Download
M LayoutTests/fast/table/table-row-before-after-content-around-table-cell-expected.txt View 1 chunk +0 lines, -2 lines 0 comments Download
M LayoutTests/fast/table/table-row-split2-expected.txt View 2 chunks +0 lines, -6 lines 0 comments Download
M LayoutTests/fast/table/table-section-split2-expected.txt View 2 chunks +0 lines, -6 lines 0 comments Download
D LayoutTests/fast/table/whitespace-in-anonymous-table-cells.html View 1 chunk +0 lines, -19 lines 0 comments Download
D LayoutTests/fast/table/whitespace-in-anonymous-table-cells-expected.html View 1 chunk +0 lines, -25 lines 0 comments Download
M LayoutTests/platform/linux/css2.1/20110323/dynamic-top-change-002-expected.txt View 1 chunk +0 lines, -2 lines 0 comments Download
M LayoutTests/platform/linux/fast/css-generated-content/015-expected.txt View 1 chunk +0 lines, -1 line 0 comments Download
M LayoutTests/platform/linux/fast/css-generated-content/table-before-after-child-add-expected.txt View 1 chunk +0 lines, -1 line 0 comments Download
M LayoutTests/platform/linux/fast/table/add-before-anonymous-child-expected.txt View 1 chunk +0 lines, -4 lines 0 comments Download
M LayoutTests/platform/linux/fast/table/table-after-child-in-table-expected.txt View 1 chunk +0 lines, -1 line 0 comments Download
M LayoutTests/platform/linux/fast/table/table-before-child-in-table-expected.txt View 1 chunk +0 lines, -1 line 0 comments Download
M LayoutTests/platform/linux/fast/table/table-cell-after-child-in-table-expected.txt View 1 chunk +0 lines, -1 line 0 comments Download
M LayoutTests/platform/linux/fast/table/table-display-types-expected.txt View 2 chunks +0 lines, -4 lines 0 comments Download
M LayoutTests/platform/linux/fast/table/table-display-types-strict-expected.txt View 2 chunks +0 lines, -4 lines 0 comments Download
M LayoutTests/platform/linux/fast/table/table-display-types-vertical-expected.txt View 2 chunks +0 lines, -4 lines 0 comments Download
M LayoutTests/platform/linux/fast/table/table-row-after-child-in-table-expected.txt View 1 chunk +0 lines, -1 line 0 comments Download
M LayoutTests/platform/linux/tables/mozilla/bugs/bug2479-1-expected.txt View 2 chunks +0 lines, -15 lines 0 comments Download
M LayoutTests/platform/linux/tables/mozilla/bugs/bug30985-expected.txt View 1 chunk +0 lines, -2 lines 0 comments Download
M LayoutTests/platform/linux/tables/mozilla/bugs/bug72359-expected.txt View 1 chunk +0 lines, -2 lines 0 comments Download
M Source/core/dom/Text.cpp View 1 chunk +0 lines, -19 lines 0 comments Download

Messages

Total messages: 4 (1 generated)
rhogan
5 years, 3 months ago (2015-09-11 19:55:58 UTC) #2
amineer_google
On 2015/09/11 19:55:58, rhogan wrote: Was https://codereview.chromium.org/1338853002/ insufficient? I trust you to be able to ...
5 years, 3 months ago (2015-09-11 19:59:46 UTC) #3
rhogan
5 years, 3 months ago (2015-09-11 20:06:04 UTC) #4
Message was sent while issue was closed. 
Committed patchset #1 (id:1) manually as 202161.

Powered by Google App Engine
This is Rietveld 408576698