[dom] support iterable<> NodeList

third_party/WebKit/Source/web/WebRuntimeFeatures.cpp

 /*
  * Copyright (C) 2013 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 131 matching lines @@
 void WebRuntimeFeatures::enableFileSystem(bool enable)
 {
     RuntimeEnabledFeatures::setFileSystemEnabled(enable);
 }
 
 void WebRuntimeFeatures::enableImageColorProfiles(bool enable)
 {
     RuntimeEnabledFeatures::setImageColorProfilesEnabled(enable);
 }
 
+void WebRuntimeFeatures::enableIterableNodeLists(bool enable)

[jsbell, 2015/10/26 22:53:34]

Why is this necessary?

[caitp (gmail), 2015/10/26 23:11:46]

For procedures like putting something behind a flag, I usually look for similar
CLs doing the same thing, and appropriate the changes. Build worked, so I didn't
question it. Per comments on the other related CL, it's not necessary

[caitp (gmail), 2015/10/27 02:30:58]

Anyways, got rid of this stuff

+{
+    RuntimeEnabledFeatures::setIterableNodeListsEnabled(enable);
+}
+
 void WebRuntimeFeatures::enableMediaPlayer(bool enable)
 {
     RuntimeEnabledFeatures::setMediaEnabled(enable);
 }
 
 void WebRuntimeFeatures::enableMediaCapture(bool enable)
 {
     RuntimeEnabledFeatures::setMediaCaptureEnabled(enable);
 }
 
@@ skipping 161 matching lines @@
 {
     RuntimeEnabledFeatures::setNewMediaPlaybackUiEnabled(enable);
 }
 
 void WebRuntimeFeatures::enablePresentationAPI(bool enable)
 {
     RuntimeEnabledFeatures::setPresentationEnabled(enable);
 }
 
 } // namespace blink