Rename flags for Origin Trials framework

third_party/WebKit/public/web/WebRuntimeFeatures.h

Index: third_party/WebKit/public/web/WebRuntimeFeatures.h
diff --git a/third_party/WebKit/public/web/WebRuntimeFeatures.h b/third_party/WebKit/public/web/WebRuntimeFeatures.h
index 00593d56058bfaca81642f1c84059f04d6342cec..3262e77fb112e91b192ea1d25bf6795ded1a0a12 100644
--- a/third_party/WebKit/public/web/WebRuntimeFeatures.h
+++ b/third_party/WebKit/public/web/WebRuntimeFeatures.h
@@ -64,8 +64,8 @@ public:
     BLINK_EXPORT static void forceDisplayList2dCanvas(bool);
     BLINK_EXPORT static void forceDisable2dCanvasCopyOnWrite(bool);
 
-    BLINK_EXPORT static void enableExperimentalFramework(bool);
-    BLINK_EXPORT static bool isExperimentalFrameworkEnabled();
+    BLINK_EXPORT static void enableOriginTrials(bool);

[iclelland, 2016/05/11 16:23:44]

Just curious -- why are these declarations in a different order than the
implementation? You moved them in the .cpp file to be still roughly
alphabetical, but not here.

[chasej, 2016/05/11 19:05:39]

I was trying to match the existing order in both files. The source file isn't
following the convention to match the declaration order. So, the order is
different in each file.

As far as I can tell, it looks like the header file groups flags that have a
getter (or other custom accessor) separately from those that only have a setter.
Within those groups, the order looks alphabetical. Based on that order (group,
then alphabetical), the renamed methods are actually (still) in the right place.


On the other hand, the source file appears to be mostly ordered alphabetically,
by the name of the setter. So, I moved the methods to what seemed like the right
place.

[iclelland, 2016/05/11 19:14:50]

Acknowledged. I'm satisfied with this :)

+    BLINK_EXPORT static bool isOriginTrialsEnabled();
 
     BLINK_EXPORT static void enableApplicationCache(bool);
     BLINK_EXPORT static void enableAudioOutputDevices(bool);