IDL compiler: [Conditional] for setters

IDL compiler: [Conditional] for setters

Simple change, with some whitespace changes on Perl for consistency
and simplicity.

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

[Nils Barth (inactive), 2013-10-18 05:44:04]

Quick one.

[haraken, 2013-10-18 05:49:44]

LGTM

[commit-bot: I haz the power, 2013-10-18 06:53:33]

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

[commit-bot: I haz the power, 2013-10-18 18:13:52]

Retried try job too often on blink_presubmit for step(s) presubmit
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=blink_pres...

[commit-bot: I haz the power, 2013-10-19 03:03:28]

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

[commit-bot: I haz the power, 2013-10-19 03:57:58]

Sorry for I got bad news for ya.
Compile failed with a clobber build on win_blink_rel.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win_blink_...
Your code is likely broken or HEAD is junk. Please ensure your
code is not broken then alert the build sheriffs.
Look at the try server FAQ for more details.

[commit-bot: I haz the power, 2013-10-19 04:13:24]

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

[commit-bot: I haz the power, 2013-10-19 04:42:33]

Sorry for I got bad news for ya.
Compile failed with a clobber build on win_blink_rel.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win_blink_...
Your code is likely broken or HEAD is junk. Please ensure your
code is not broken then alert the build sheriffs.
Look at the try server FAQ for more details.

[commit-bot: I haz the power, 2013-10-19 05:12:18]

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

[commit-bot: I haz the power, 2013-10-19 05:15:31]

Sorry for I got bad news for ya.
Compile failed with a clobber build on win_layout.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win_layout...
Your code is likely broken or HEAD is junk. Please ensure your
code is not broken then alert the build sheriffs.
Look at the try server FAQ for more details.

[commit-bot: I haz the power, 2013-10-19 11:07:50]

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

[commit-bot: I haz the power, 2013-10-19 11:07:54]

Failed to apply patch for Source/bindings/tests/idls/TestObjectPython.idl:
While running patch -p1 --forward --force --no-backup-if-mismatch;
  patching file Source/bindings/tests/idls/TestObjectPython.idl
  Hunk #1 FAILED at 103.
  1 out of 1 hunk FAILED -- saving rejects to file
Source/bindings/tests/idls/TestObjectPython.idl.rej

Patch:       Source/bindings/tests/idls/TestObjectPython.idl
Index: Source/bindings/tests/idls/TestObjectPython.idl
diff --git a/Source/bindings/tests/idls/TestObjectPython.idl
b/Source/bindings/tests/idls/TestObjectPython.idl
index
660cc38e2e0144ca33bb3bdac1d88acfb1170398..ff1c3ee724c563496eb80394a71df3a5aaa84eab
100644
--- a/Source/bindings/tests/idls/TestObjectPython.idl
+++ b/Source/bindings/tests/idls/TestObjectPython.idl
@@ -103,10 +103,10 @@ interface TestObjectPython {
     [ActivityLogging=Getter] readonly attribute long
activityLogGetterReadonlyLongAttribute;
     [CachedAttribute=isValueDirty] attribute any cachedAttributeAnyAttribute;
     [CallWith=ExecutionContext] readonly attribute any
callWithExecutionContextReadonlyAnyAttribute;
-    [CheckSecurityForNode] readonly attribute Document
checkSecurityForNodeReadonlyDocumentAttribute;
-    [Conditional=Condition] readonly attribute long
conditionalReadonlyLongAttribute;
-    [Conditional=Condition1&Condition2] readonly attribute long
conditionalAndReadonlyLongAttribute;
-    [Conditional=Condition1|Condition2] readonly attribute long
conditionalOrReadonlyLongAttribute;
+    [CheckSecurityForNode] readonly attribute Document
checkSecurityForNodeReadonlyDocumentAttribute;  // All uses are readonly
+    [Conditional=Condition] attribute long conditionalLongAttribute;
+    [Conditional=Condition1&Condition2] attribute long
conditionalAndLongAttribute;
+    [Conditional=Condition1|Condition2] attribute long
conditionalOrLongAttribute;
     [Custom] readonly attribute long customReadonlyLongAttribute;
     [CustomGetter] readonly attribute long customGetterReadonlyLongAttribute;
     [Conditional=Condition, Custom] readonly attribute long
customReadonlyLongAttribute;

[commit-bot: I haz the power, 2013-10-19 11:12:40]

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

[commit-bot: I haz the power, 2013-10-19 13:38:07]

Change committed as 160004