Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(240)

Side by Side Diff: third_party/WebKit/LayoutTests/http/tests/inspector/inspector-test.js

Issue 2574603002: DevTools: Fix Elements tab event listener removal (Closed)
Patch Set: add test Created 4 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 if (window.GCController) 1 if (window.GCController)
2 GCController.collectAll(); 2 GCController.collectAll();
3 var initialize_InspectorTest = function() { 3 var initialize_InspectorTest = function() {
4 4
5 var results = []; 5 var results = [];
6 6
7 function consoleOutputHook(messageType) 7 function consoleOutputHook(messageType)
8 { 8 {
9 InspectorTest.addResult(messageType + ": " + Array.prototype.slice.call(argu ments, 1)); 9 InspectorTest.addResult(messageType + ": " + Array.prototype.slice.call(argu ments, 1));
10 } 10 }
(...skipping 301 matching lines...) Expand 10 before | Expand all | Expand 10 after
312 InspectorTest.addResult(expandedSubstring + " " + treeElement.listItemElemen t.deepTextContent()); 312 InspectorTest.addResult(expandedSubstring + " " + treeElement.listItemElemen t.deepTextContent());
313 313
314 for (var i = 0; i < treeElement.childCount(); ++i) { 314 for (var i = 0; i < treeElement.childCount(); ++i) {
315 var property = treeElement.childAt(i).property; 315 var property = treeElement.childAt(i).property;
316 var key = property.name; 316 var key = property.name;
317 var value = property.value._description; 317 var value = property.value._description;
318 InspectorTest.addResult(" " + key + ": " + value); 318 InspectorTest.addResult(" " + key + ": " + value);
319 } 319 }
320 } 320 }
321 321
322 InspectorTest.expandAndDumpEventListeners = function(eventListenersView, callbac k) 322 InspectorTest.expandAndDumpEventListeners = function(eventListenersView, callbac k, force)
323 { 323 {
324 InspectorTest.addSniffer(Components.EventListenersView.prototype, "_eventLis tenersArrivedForTest", listenersArrived); 324 InspectorTest.addSniffer(Components.EventListenersView.prototype, "_eventLis tenersArrivedForTest", listenersArrived);
325 325
326 function listenersArrived() 326 function listenersArrived()
327 { 327 {
328 var listenerTypes = eventListenersView._treeOutline.rootElement().childr en(); 328 var listenerTypes = eventListenersView._treeOutline.rootElement().childr en();
329 for (var i = 0; i < listenerTypes.length; ++i) { 329 for (var i = 0; i < listenerTypes.length; ++i) {
330 listenerTypes[i].expand(); 330 listenerTypes[i].expand();
331 var listenerItems = listenerTypes[i].children(); 331 var listenerItems = listenerTypes[i].children();
332 for (var j = 0; j < listenerItems.length; ++j) 332 for (var j = 0; j < listenerItems.length; ++j)
(...skipping 12 matching lines...) Expand all
345 InspectorTest.addResult(""); 345 InspectorTest.addResult("");
346 InspectorTest.addResult("======== " + eventType + " ========"); 346 InspectorTest.addResult("======== " + eventType + " ========");
347 var listenerItems = listenerTypes[i].children(); 347 var listenerItems = listenerTypes[i].children();
348 for (var j = 0; j < listenerItems.length; ++j) { 348 for (var j = 0; j < listenerItems.length; ++j) {
349 InspectorTest.addResult("== " + listenerItems[j].eventListener() .listenerType()); 349 InspectorTest.addResult("== " + listenerItems[j].eventListener() .listenerType());
350 InspectorTest.dumpObjectPropertyTreeElement(listenerItems[j]); 350 InspectorTest.dumpObjectPropertyTreeElement(listenerItems[j]);
351 } 351 }
352 } 352 }
353 callback(); 353 callback();
354 } 354 }
355
356 if (force)
lushnikov 2016/12/15 22:33:16 let's rather do: if (force) lstenersArrived
phulce 2016/12/16 18:41:56 Went back and forth and that actually, good point
357 Components.EventListenersView.prototype._eventListenersArrivedForTest();
355 } 358 }
356 359
357 InspectorTest.dumpNavigatorView = function(navigatorView) 360 InspectorTest.dumpNavigatorView = function(navigatorView)
358 { 361 {
359 dumpNavigatorTreeOutline(navigatorView._scriptsTree); 362 dumpNavigatorTreeOutline(navigatorView._scriptsTree);
360 363
361 function dumpNavigatorTreeElement(prefix, treeElement) 364 function dumpNavigatorTreeElement(prefix, treeElement)
362 { 365 {
363 var titleText; 366 var titleText;
364 if (treeElement.title instanceof Element) 367 if (treeElement.title instanceof Element)
(...skipping 853 matching lines...) Expand 10 before | Expand all | Expand 10 after
1218 _output("[page] " + text); 1221 _output("[page] " + text);
1219 } 1222 }
1220 1223
1221 function _output(result) 1224 function _output(result)
1222 { 1225 {
1223 if (!outputElement) 1226 if (!outputElement)
1224 createOutputElement(); 1227 createOutputElement();
1225 outputElement.appendChild(document.createTextNode(result)); 1228 outputElement.appendChild(document.createTextNode(result));
1226 outputElement.appendChild(document.createElement("br")); 1229 outputElement.appendChild(document.createElement("br"));
1227 } 1230 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698