Chromium Code Reviews
Help | Chromium Project | Gerrit Changes | Sign in
(5)

Issue 2064473002: Don't need to preserve CSS line-height property during editing operation (Closed)

Created:
2 years, 1 month ago by joone
Modified:
2 years, 1 month ago
Reviewers:
yosin_UTC9
CC:
blink-reviews, blink-reviews-style_chromium.org, chromium-reviews
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Don't need to preserve CSS line-height property during editing operation When we merge two paragraphs by typing backspace key at the head of the second paragraph, the styles of the second paragraph can be preserved by using <span> tag. However, we don't need to preserve line-height style because the computed value can be different from the value defined in HTML. BUG=226941 TEST=editing/deleting/backspace-merge-two-paragraphs.html Committed: https://crrev.com/8e411d16171d27612776a2f05356b0ed9f06b848 Cr-Commit-Position: refs/heads/master@{#399410}

Patch Set 1 : #

Patch Set 2 : rebaseline #

Patch Set 3 : rebaseline: webkit_unit_tests #

Messages

Total messages: 13 (7 generated)
joone
Hi yosin@ could you review this CL?
2 years, 1 month ago (2016-06-11 08:35:41 UTC) #4
joone
On 2016/06/11 08:35:41, joone wrote: > Hi yosin@ could you review this CL? There is ...
2 years, 1 month ago (2016-06-13 00:52:06 UTC) #6
yosin_UTC9
lgtm
2 years, 1 month ago (2016-06-13 01:13:52 UTC) #7
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/2064473002/80001
2 years, 1 month ago (2016-06-13 06:32:49 UTC) #9
commit-bot: I haz the power
Committed patchset #3 (id:80001)
2 years, 1 month ago (2016-06-13 06:55:56 UTC) #11
commit-bot: I haz the power
2 years, 1 month ago (2016-06-13 06:58:04 UTC) #13
Message was sent while issue was closed.
Patchset 3 (id:??) landed as
https://crrev.com/8e411d16171d27612776a2f05356b0ed9f06b848
Cr-Commit-Position: refs/heads/master@{#399410}

Powered by Google App Engine
This is Rietveld 408576698