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

Issue 552123003: Create timing functions directly instead of using parser in KeyframeEffectModelTest (Closed)

Created:
6 years, 3 months ago by Timothy Loh
Modified:
6 years, 3 months ago
CC:
dstockwell, darktears, blink-reviews, blink-reviews-animation_chromium.org, Eric Willigers, Mike Lawther (Google), rjwright, shans, Steve Block, Timothy Loh
Base URL:
https://chromium.googlesource.com/chromium/blink.git@master
Project:
blink
Visibility:
Public.

Description

Create timing functions directly instead of using parser in KeyframeEffectModelTest As the tests in KeyframeEffectModelTest are unit tests, it doesn't make sense for them to use the parser to construct timing functions when they can be constructed directly. Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=181609

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+2 lines, -6 lines) Patch
M Source/core/animation/KeyframeEffectModelTest.cpp View 3 chunks +2 lines, -6 lines 0 comments Download

Messages

Total messages: 7 (3 generated)
Timothy Loh
6 years, 3 months ago (2014-09-09 04:26:24 UTC) #2
dstockwell
lgtm
6 years, 3 months ago (2014-09-09 04:31:09 UTC) #5
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/timloh@chromium.org/552123003/1
6 years, 3 months ago (2014-09-09 04:32:08 UTC) #6
commit-bot: I haz the power
6 years, 3 months ago (2014-09-09 06:11:15 UTC) #7
Message was sent while issue was closed.
Committed patchset #1 (id:1) as 181609

Powered by Google App Engine
This is Rietveld 408576698