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

Issue 188773002: Clean up static_cast<CSSFooValue*> in CSSValue.cpp (Closed)

Created:
6 years, 9 months ago by gyuyoung-inactive
Modified:
6 years, 9 months ago
Reviewers:
tkent, eae
CC:
blink-reviews, ed+blinkwatch_opera.com, dglazkov+blink, apavlov+blink_chromium.org, darktears, rune+blink, rwlbuis
Base URL:
https://chromium.googlesource.com/chromium/blink.git@master
Visibility:
Public.

Description

Clean up static_cast<CSSFooValue*> in CSSValue.cpp This cl removes remained static_cast<CSSFooValue*> usage in CSSValue.cpp. BUG=309516 Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=169038

Patch Set 1 #

Patch Set 2 : Rebased #

Patch Set 3 : Rebased #2 #

Patch Set 4 : Rebased #3 #

Patch Set 5 : Rebased #5 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+31 lines, -31 lines) Patch
M Source/core/css/CSSValue.cpp View 1 2 3 4 1 chunk +31 lines, -31 lines 0 comments Download

Messages

Total messages: 26 (0 generated)
gyuyoung-inactive
Add tkent.
6 years, 9 months ago (2014-03-06 15:51:12 UTC) #1
eae
LGTM
6 years, 9 months ago (2014-03-07 01:56:01 UTC) #2
eae
The CQ bit was checked by eae@chromium.org
6 years, 9 months ago (2014-03-07 01:56:04 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/gyuyoung.kim@samsung.com/188773002/1
6 years, 9 months ago (2014-03-07 01:56:13 UTC) #4
commit-bot: I haz the power
The CQ bit was unchecked by commit-bot@chromium.org
6 years, 9 months ago (2014-03-07 01:57:32 UTC) #5
commit-bot: I haz the power
Try jobs failed on following builders: mac_blink_compile
6 years, 9 months ago (2014-03-07 01:57:33 UTC) #6
gyuyoung-inactive
The CQ bit was checked by gyuyoung.kim@samsung.com
6 years, 9 months ago (2014-03-07 05:02:06 UTC) #7
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/gyuyoung.kim@samsung.com/188773002/1
6 years, 9 months ago (2014-03-07 05:02:18 UTC) #8
commit-bot: I haz the power
The CQ bit was unchecked by commit-bot@chromium.org
6 years, 9 months ago (2014-03-07 06:28:13 UTC) #9
commit-bot: I haz the power
Try jobs failed on following builders: linux_blink_rel
6 years, 9 months ago (2014-03-07 06:28:13 UTC) #10
gyuyoung-inactive
The CQ bit was checked by gyuyoung.kim@samsung.com
6 years, 9 months ago (2014-03-07 12:20:46 UTC) #11
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/gyuyoung.kim@samsung.com/188773002/20001
6 years, 9 months ago (2014-03-07 12:20:58 UTC) #12
commit-bot: I haz the power
The CQ bit was unchecked by commit-bot@chromium.org
6 years, 9 months ago (2014-03-07 12:25:39 UTC) #13
commit-bot: I haz the power
Try jobs failed on following builders: linux_blink
6 years, 9 months ago (2014-03-07 12:25:39 UTC) #14
gyuyoung-inactive
It looks this patch doesn't influence on layout test. I don't know why this is ...
6 years, 9 months ago (2014-03-09 05:08:48 UTC) #15
gyuyoung-inactive
The CQ bit was checked by gyuyoung.kim@samsung.com
6 years, 9 months ago (2014-03-09 05:08:52 UTC) #16
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/gyuyoung.kim@samsung.com/188773002/40001
6 years, 9 months ago (2014-03-09 05:08:59 UTC) #17
commit-bot: I haz the power
The CQ bit was unchecked by commit-bot@chromium.org
6 years, 9 months ago (2014-03-09 06:23:00 UTC) #18
commit-bot: I haz the power
Try jobs failed on following builders: linux_blink
6 years, 9 months ago (2014-03-09 06:23:01 UTC) #19
gyuyoung-inactive
The CQ bit was checked by gyuyoung.kim@samsung.com
6 years, 9 months ago (2014-03-10 15:03:45 UTC) #20
gyuyoung-inactive
On 2014/03/09 06:23:01, I haz the power (commit-bot) wrote: > Try jobs failed on following ...
6 years, 9 months ago (2014-03-10 15:04:32 UTC) #21
gyuyoung-inactive
The CQ bit was unchecked by gyuyoung.kim@samsung.com
6 years, 9 months ago (2014-03-10 15:08:37 UTC) #22
gyuyoung-inactive
The CQ bit was checked by gyuyoung.kim@samsung.com
6 years, 9 months ago (2014-03-12 16:13:05 UTC) #23
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/gyuyoung.kim@samsung.com/188773002/80001
6 years, 9 months ago (2014-03-12 16:13:17 UTC) #24
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/gyuyoung.kim@samsung.com/188773002/80001
6 years, 9 months ago (2014-03-12 16:49:11 UTC) #25
commit-bot: I haz the power
6 years, 9 months ago (2014-03-12 17:28:44 UTC) #26
Message was sent while issue was closed.
Change committed as 169038

Powered by Google App Engine
This is Rietveld 408576698