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

Unified Diff: Source/core/html/canvas/WebGLRenderingContextBase.idl

Issue 262753004: Replace all remaining IDL finitude type checks with [TypeChecking=Unrestricted] (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Remove unnecessary code (std::isfinite not needed for long) Created 6 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: Source/core/html/canvas/WebGLRenderingContextBase.idl
diff --git a/Source/core/html/canvas/WebGLRenderingContextBase.idl b/Source/core/html/canvas/WebGLRenderingContextBase.idl
index 6841cbcfa20d55aca34ed56d062c9e04a7ba3b8a..470fa507c77fe46d10d6f96a5786f7f5f281d7bc 100644
--- a/Source/core/html/canvas/WebGLRenderingContextBase.idl
+++ b/Source/core/html/canvas/WebGLRenderingContextBase.idl
@@ -35,16 +35,15 @@ typedef long long GLsizeiptr;
typedef octet GLubyte; // 'octet' should be an unsigned 8 bit type.
typedef unsigned short GLushort;
typedef unsigned long GLuint;
-// FIXME: Should be unrestricted: http://crbug.com/354298
-typedef /*unrestricted*/ float GLfloat;
-typedef /*unrestricted*/ float GLclampf;
+typedef unrestricted float GLfloat;
+typedef unrestricted float GLclampf;
[
// FIXME: [DoNotCheckConstants] and [TypeChecking=Interface|Nullable] should be applied
// to members and not need to be put on implementing interface
// DoNotCheckConstants, // need to put on implementing interface
NoInterfaceObject, // Always used on target of 'implements'
- // TypeChecking=Interface|Nullable, // need to put on implementing interface
+ // TypeChecking=Interface|Nullable|Unrestricted, // need to put on implementing interface
] interface WebGLRenderingContextBase {
readonly attribute HTMLCanvasElement canvas;

Powered by Google App Engine
This is Rietveld 408576698