Move [StrictTypeChecking] to interface (from members), where possible

Move [StrictTypeChecking] to interface (from members), where possible

No changes to generated bindings: just simplification.

kouhei:
there's one case in an SVG IDL that might need fixing,
namely createSVGTransformFromMatrix in
Source/core/svg/SVGSVGElement.idl
(only case of an SVG method not having [StrictTypeChecking]
when others on the interface do)

[StrictTypeChecking] is almost always applied to all members of an interface.
Thus it's clearer and safer to put in on the interface, instead of the members.
This requires minimal CG changes (just check [StrictTypeChecking] on interface),
and allows *massive* simplification of IDL files.

This is ultimately in support of the SVG bindings rewrite,
as we want strict type checking there for null.

There are only 3 IDL files left with [StrictTypeChecking] on members:
(outside of test cases)
* Source/core/html/HTMLMediaElement.idl
* Source/core/svg/SVGSVGElement.idl  ::  kouhei?
* Source/modules/mediastream/RTCPeerConnection.idl

Also some formatting fixes to WebGLRenderingContext.idl,
which is v. long and was quite inconsistent.
(And a few to SVGPathElement.idl, due to long argument lists.)

Followup to:
Add null-correctness checking to [StrictTypeChecking] methods
https://codereview.chromium.org/141523002/

BUG=249598
R=haraken

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=165749

[Nils Barth (inactive), 2014-01-24 06:32:35]

+philipj

[haraken, 2014-01-24 06:33:56]

rubberstamp LGTM.

[commit-bot: I haz the power, 2014-01-24 07:03:31]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/nbarth@chromium.org/146423002/80001

[commit-bot: I haz the power, 2014-01-24 08:14:03]

Failed to apply patch for Source/core/svg/SVGNumber.idl:
While running patch -p1 --forward --force --no-backup-if-mismatch;
  patching file Source/core/svg/SVGNumber.idl
  Hunk #1 FAILED at 20.
  1 out of 1 hunk FAILED -- saving rejects to file
Source/core/svg/SVGNumber.idl.rej

Patch:       Source/core/svg/SVGNumber.idl
Index: Source/core/svg/SVGNumber.idl
diff --git a/Source/core/svg/SVGNumber.idl b/Source/core/svg/SVGNumber.idl
index
c77bbc9035111ddfc3647bf2f7de17b9eb908060..8ed9f43a5d2f3a2cec73770cd5dc824c1978f006
100644
--- a/Source/core/svg/SVGNumber.idl
+++ b/Source/core/svg/SVGNumber.idl
@@ -20,7 +20,9 @@
  * Boston, MA 02110-1301, USA.
  */
 
-interface SVGNumber {
-    [StrictTypeChecking] attribute float value;
+[
+    StrictTypeChecking,
+] interface SVGNumber {
+    attribute float value;
 };

[commit-bot: I haz the power, 2014-01-24 10:21:06]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/nbarth@chromium.org/146423002/190001

[commit-bot: I haz the power, 2014-01-24 13:54:45]

Change committed as 165749