[DevTools] Remove methods on Common.Event.

third_party/WebKit/LayoutTests/inspector/elements/edit/set-outer-html-test.js

 var initialize_SetOuterHTMLTest = function() {
 
 InspectorTest.events = [];
 InspectorTest.containerId;
 
 InspectorTest.setUpTestSuite = function(next)
 {
     InspectorTest.expandElementsTree(step1);
 
     function step1()
     {
         InspectorTest.selectNodeWithId("container", step2);
     }
 
     function step2(node)
     {
         InspectorTest.containerId = node.id;
         InspectorTest.DOMAgent.getOuterHTML(InspectorTest.containerId, step3);
     }
 
     function step3(error, text)
     {
         InspectorTest.containerText = text;
 
         for (var key in SDK.DOMModel.Events) {
             var eventName = SDK.DOMModel.Events[key];
+            if (eventName === SDK.DOMModel.Events.MarkersChanged || eventName === SDK.DOMModel.Events.DOMMutated)
+                continue;
             InspectorTest.domModel.addEventListener(eventName, InspectorTest.recordEvent.bind(InspectorTest, eventName));
         }
 
         next();
     }
 }
 
 InspectorTest.recordEvent = function(eventName, event)
 {
-    if (!event.data || event.type === SDK.DOMModel.Events.MarkersChanged || event.type === SDK.DOMModel.Events.DOMMutated)

[lushnikov, 2016/12/06 20:21:51]

i'm surprised this didn't break a lot of tests

[dgozman, 2016/12/06 21:36:16]

Well, I just don't add the listener in step3().

+    if (!event.data)
         return;
     var node = event.data.node || event.data;
     var parent = event.data.parent;
     for (var currentNode = parent || node; currentNode; currentNode = currentNode.parentNode) {
         if (currentNode.getAttribute("id") === "output")
             return;
     }
     InspectorTest.events.push("Event " + eventName.toString() + ": " + node.nodeName());
 }
 
@@ skipping 55 matching lines @@
         InspectorTest.addResult("==========8<==========");
         InspectorTest.addResult(text);
         InspectorTest.addResult("==========>8==========");
         if (last)
             InspectorTest.addResult("\n\n\n");
         next();
     }
 }
 
 };