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

Issue 26363004: webkit prefix shoudn't be used for text-align-last (Closed)

Created:
7 years, 2 months ago by dw.im
Modified:
7 years, 2 months ago
CC:
blink-reviews, caseq+blink_chromium.org, loislo+blink_chromium.org, eustas+blink_chromium.org, alph+blink_chromium.org, yurys+blink_chromium.org, lushnikov+blink_chromium.org, vsevik+blink_chromium.org, pfeldman+blink_chromium.org, paulirish+reviews_chromium.org, apavlov+blink_chromium.org, devtools-reviews_chromium.org, aandrey+blink_chromium.org
Base URL:
https://chromium.googlesource.com/chromium/blink.git@master
Visibility:
Public.

Description

webkit prefix shoudn't be used for text-align-last When I removed the webkit prefix from the property, I missed some of those. BUG=259332 Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=159707

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+2 lines, -2 lines) Patch
M LayoutTests/fast/css3-text/css3-text-align-last/getComputedStyle/script-tests/getComputedStyle-text-align-last.js View 1 chunk +1 line, -1 line 0 comments Download
M Source/devtools/front_end/CSSMetadata.js View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 9 (0 generated)
dw.im
7 years, 2 months ago (2013-10-15 09:16:25 UTC) #1
Julien - ping for review
lgtm
7 years, 2 months ago (2013-10-15 14:51:05 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/dw.im@samsung.com/26363004/1
7 years, 2 months ago (2013-10-15 14:51:19 UTC) #3
commit-bot: I haz the power
Retried try job too often on blink_presubmit for step(s) presubmit http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=blink_presubmit&number=8895
7 years, 2 months ago (2013-10-15 15:05:46 UTC) #4
dw.im
7 years, 2 months ago (2013-10-15 16:00:43 UTC) #5
Julien - ping for review
@Pavel, we need your rubber-stamp for a trivial change in devtools/
7 years, 2 months ago (2013-10-15 17:17:02 UTC) #6
pfeldman
lgtm
7 years, 2 months ago (2013-10-15 17:33:30 UTC) #7
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/dw.im@samsung.com/26363004/1
7 years, 2 months ago (2013-10-15 18:15:05 UTC) #8
commit-bot: I haz the power
7 years, 2 months ago (2013-10-16 03:17:48 UTC) #9
Message was sent while issue was closed.
Change committed as 159707

Powered by Google App Engine
This is Rietveld 408576698