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

Issue 6691061: Expose WebGL typed array constructors in the shell sample (Closed)

Created:
9 years, 8 months ago by Jakob Kummerow
Modified:
9 years, 6 months ago
CC:
v8-dev
Visibility:
Public.

Description

Expose WebGL typed array constructors in the shell sample Committed: http://code.google.com/p/v8/source/detail?r=7507

Patch Set 1 #

Total comments: 2

Patch Set 2 : removed unnecessary NOLINT comments #

Unified diffs Side-by-side diffs Delta from patch set Stats (+95 lines, -0 lines) Patch
M samples/shell.cc View 1 3 chunks +95 lines, -0 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
Jakob Kummerow
Please review.
9 years, 8 months ago (2011-04-05 16:49:26 UTC) #1
Mads Ager (chromium)
LGTM Will be very nice to use this to rewrite the test-api external and pixel ...
9 years, 8 months ago (2011-04-05 20:45:33 UTC) #2
Jakob Kummerow
9 years, 8 months ago (2011-04-06 08:08:37 UTC) #3
http://codereview.chromium.org/6691061/diff/1/samples/shell.cc
File samples/shell.cc (right):

http://codereview.chromium.org/6691061/diff/1/samples/shell.cc#newcode515
samples/shell.cc:515: return CreateExternalArray(args, v8::kExternalPixelArray,
sizeof(uint8_t));
On 2011/04/05 20:45:33, Mads Ager wrote:
> Interesting. The linter does not see this type as a type? I wonder if you need
> the above NOLINT comment in that case. Maybe only the float one is actually
> recognized by the linter as a type?

Indeed. I've removed all the //NOLINT tags except for the float case, and
presubmit.py doesn't complain.

Powered by Google App Engine
This is Rietveld 408576698