Pass current value of attributes to WebDOMActivityLogger Setter logs.

Source/bindings/IDLExtendedAttributes.txt

Index: Source/bindings/IDLExtendedAttributes.txt
diff --git a/Source/bindings/IDLExtendedAttributes.txt b/Source/bindings/IDLExtendedAttributes.txt
index bdcdb16f893a3e5ff2efd1afc819fa0e47055486..9c5062aaf4a4080d2af045e4ce8b8d42680e7ecf 100644
--- a/Source/bindings/IDLExtendedAttributes.txt
+++ b/Source/bindings/IDLExtendedAttributes.txt
@@ -34,7 +34,7 @@
 #
 
 ActiveDOMObject
-ActivityLogging=ForAllWorlds|GetterForAllWorlds|SetterForAllWorlds|ForIsolatedWorlds|GetterForIsolatedWorlds|SetterForIsolatedWorlds
+ActivityLogging=ForAllWorlds|GetterForAllWorlds|SetterForAllWorlds|ForIsolatedWorlds|GetterForIsolatedWorlds|SetterForIsolatedWorlds|ForAllWorlsIncludeOldValueForSetter|SetterForAllWorldsIncludeOldValueForSetter|ForIsolatedWorldsIncludeOldValueForSetter|SetterForIsolatedWorldsIncludeOldValueForSetter

[Devlin, 2014/04/22 17:47:09]

This is getting a bit crazy - do we want to split these up?

[abarth-chromium, 2014/04/22 23:35:51]

Yeah, there seem to be several bits conflated here.  Can you split these up into
separate attributes?

[Devlin, 2014/04/23 01:10:03]

Done, per offline conversation, decided on one attribute each for the type of
activity, the worlds we log, and whether or not to include the old value.

As expected, this resulted in a bit of noise in the CL.  Apologies.

[Nils Barth (inactive), 2014/04/23 01:12:02]

Combinatorial explosion (>.<)
We can keep one attribute but have a list instead:
ActivityLogging=Getter|Setter|ForAllWorlds|ForIsolatedWorlds|OldValueForSetter

(I'm happy to do splitting in separate CL if that's desired.)
haraken@?

[Devlin, 2014/04/23 01:31:43]

Already fixed per Adam's comment.  Does the new format look okay?

[Nils Barth (inactive), 2014/04/23 03:04:42]

New format is fine, thanks!
Could you please post this in a separate CL, to simplify this CL?
(i.e., the extended attribute renaming, without adding new feature)

[Devlin, 2014/04/23 18:28:02]

Done, https://codereview.chromium.org/249833002/

[Nils Barth (inactive), 2014/04/24 01:00:41]

Thanks!

 CachedAttribute=*
 CallWith=ExecutionContext|ScriptState|NewScriptState|ScriptArguments|ActiveWindow|FirstWindow|ThisValue
 CheckSecurity=Frame|Node