Make [OriginTrialEnabled] and [RuntimeEnabled] mutually exclusive

third_party/WebKit/Source/bindings/tests/results/core/V8TestInterface2.cpp

Index: third_party/WebKit/Source/bindings/tests/results/core/V8TestInterface2.cpp
diff --git a/third_party/WebKit/Source/bindings/tests/results/core/V8TestInterface2.cpp b/third_party/WebKit/Source/bindings/tests/results/core/V8TestInterface2.cpp
index 2f2f2becae299b4766dbc5161abe89be6aded217..d5551b1dd3497c888c2c493c8ecaf826ebd70486 100644
--- a/third_party/WebKit/Source/bindings/tests/results/core/V8TestInterface2.cpp
+++ b/third_party/WebKit/Source/bindings/tests/results/core/V8TestInterface2.cpp
@@ -17,6 +17,7 @@
 #include "core/dom/Document.h"
 #include "core/dom/Element.h"
 #include "core/frame/LocalDOMWindow.h"
+#include "platform/RuntimeEnabledFeatures.h"

[iclelland, 2016/04/05 13:38:59]

Curious -- do you know why this wasn't included before?

[chasej, 2016/04/05 19:50:22]

I'm pretty sure this was a bug. See line 615 in v8_interface.py, the
constant_context() function (in the original version). There was logic to add
the includes for OriginTrials, but not for RuntimeEnabled. Compare to line 162
in the same file, which has a check to add the include. TestInterface2.idl only
has [RuntimeEnabled] applied to a constant, which would have been affected by
the changes in this CL. Specifically, I moved the logic to add the
RuntimeEnabled include into the runtime_enabled_function_name().

[iclelland, 2016/04/07 14:00:32]

This sort of bug being present and unnoticed makes me more comfortable with the
way you're adding new includes now. The fact that the old method missed cases
like this suggests that the changes in this CL are definite improvements.

 #include "wtf/GetPtr.h"
 #include "wtf/RefPtr.h"