Add StrictTypeChecking to CRC2D.{drawFocusIfNeeded,drawCustomFocusRing}

LayoutTests/fast/canvas/draw-focus-if-needed.html

Index: LayoutTests/fast/canvas/draw-focus-if-needed.html
diff --git a/LayoutTests/fast/canvas/draw-focus-if-needed.html b/LayoutTests/fast/canvas/draw-focus-if-needed.html
index f5f48143d1ae27769eed034f123c4e4186120cc2..8bfb2ea3bfefca1cead0a7fbe0dd3cb2a3933937 100644
--- a/LayoutTests/fast/canvas/draw-focus-if-needed.html
+++ b/LayoutTests/fast/canvas/draw-focus-if-needed.html
@@ -16,7 +16,7 @@ document.getElementById("button1").focus();
 
 var canvas = document.getElementById("canvas").getContext("2d");
 
-shouldNotThrow('canvas.drawFocusIfNeeded(null);');
+shouldThrow('canvas.drawFocusIfNeeded(null);');

[Justin Novosad, 2014/04/14 14:33:31]

This fixes a spec compliance bug, which should be documented.  Please create a
bug to that effect in the issue tracker and link the this patch to it.

[fs, 2014/04/14 15:03:28]

I figured that since this API was guarded by the ExperimentalCanvasFeatures
flag/setting it wasn't a big deal, but I can file a bug - no worries.

 shouldThrow('canvas.drawFocusIfNeeded();');
 
 canvas.beginPath();
@@ -24,7 +24,7 @@ canvas.rect(50, 50, 200, 100);
 canvas.fillStyle = "#ccf";
 canvas.fill();
 // re-test null case after having defined a path (regression test for crbug.com/353248)
-shouldNotThrow('canvas.drawFocusIfNeeded(null);');
+shouldThrow('canvas.drawFocusIfNeeded(null);');
 canvas.drawFocusIfNeeded(document.getElementById("button1"));
 
 canvas.beginPath();