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

Issue 23876008: Add Clock Sample. (Closed)

Created:
7 years, 3 months ago by jvanverth1
Modified:
7 years, 3 months ago
Reviewers:
reed1
CC:
skia-review_googlegroups.com, tfarina
Visibility:
Public.

Description

Add Clock Sample. There are two possible rendering methods: paths (default) or primitives. At the moment the rrect primitive only supports axis-aligned rrects for GPU acceleration, so that will not be a win on GPU. R=reed@google.com Committed: https://code.google.com/p/skia/source/detail?r=11330

Patch Set 1 #

Total comments: 3

Patch Set 2 : Pre-commit cleanup #

Patch Set 3 : Clean up for commit #

Unified diffs Side-by-side diffs Delta from patch set Stats (+230 lines, -0 lines) Patch
M gyp/SampleApp.gyp View 1 chunk +1 line, -0 lines 0 comments Download
A samplecode/SampleClock.cpp View 1 2 1 chunk +229 lines, -0 lines 0 comments Download

Messages

Total messages: 4 (0 generated)
jvanverth1
7 years, 3 months ago (2013-09-10 20:06:54 UTC) #1
reed1
lgtm
7 years, 3 months ago (2013-09-10 20:13:13 UTC) #2
tfarina
https://codereview.chromium.org/23876008/diff/1/samplecode/SampleClock.cpp File samplecode/SampleClock.cpp (right): https://codereview.chromium.org/23876008/diff/1/samplecode/SampleClock.cpp#newcode1 samplecode/SampleClock.cpp:1: rm this blank line. https://codereview.chromium.org/23876008/diff/1/samplecode/SampleClock.cpp#newcode3 samplecode/SampleClock.cpp:3: * Copyright 2011 ...
7 years, 3 months ago (2013-09-10 21:59:02 UTC) #3
jvanverth1
7 years, 3 months ago (2013-09-17 19:52:12 UTC) #4
Message was sent while issue was closed.
Committed patchset #3 manually as r11330 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698