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

Side by Side Diff: test/inspector/protocol-test.js

Issue 2767323002: [inspector] better isArrayLike for injected-script-source.js (Closed)
Patch Set: length on array should be own property Created 3 years, 9 months 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 // Copyright 2016 the V8 project authors. All rights reserved. 1 // Copyright 2016 the V8 project authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 InspectorTest = {}; 5 InspectorTest = {};
6 InspectorTest._dispatchTable = new Map(); 6 InspectorTest._dispatchTable = new Map();
7 InspectorTest._requestId = 0; 7 InspectorTest._requestId = 0;
8 InspectorTest._dumpInspectorProtocolMessages = false; 8 InspectorTest._dumpInspectorProtocolMessages = false;
9 InspectorTest._eventHandler = {}; 9 InspectorTest._eventHandler = {};
10 InspectorTest._commandsForLogging = new Set(); 10 InspectorTest._commandsForLogging = new Set();
(...skipping 326 matching lines...) Expand 10 before | Expand all | Expand 10 after
337 defineProperty(ObjectPrototype, '${getter}', { 337 defineProperty(ObjectPrototype, '${getter}', {
338 set() { debugger; throw 42; }, get() { debugger; throw 42; }, 338 set() { debugger; throw 42; }, get() { debugger; throw 42; },
339 __proto__: null 339 __proto__: null
340 }); 340 });
341 `).join('\n') + '})();'; 341 `).join('\n') + '})();';
342 InspectorTest.addScript(scriptSource); 342 InspectorTest.addScript(scriptSource);
343 343
344 if (debug) { 344 if (debug) {
345 InspectorTest.log('WARNING: InspectorTest.makeContextDirty with debug flag f or debugging only and should not be landed.'); 345 InspectorTest.log('WARNING: InspectorTest.makeContextDirty with debug flag f or debugging only and should not be landed.');
346 InspectorTest.log('WARNING: run test with --expose-inspector-scripts flag to get more details.'); 346 InspectorTest.log('WARNING: run test with --expose-inspector-scripts flag to get more details.');
347 InspectorTest.log('WARNING: you can additionally comment \'input_text = rjsm in.jsmin(input_text)\' line in xxd.py to get unminified injected-script-source.j s.');
dgozman 2017/03/23 21:10:20 Let's not be too specific (or it will be outdated)
kozy 2017/03/24 00:40:37 Done.
347 InspectorTest.setupScriptMap(); 348 InspectorTest.setupScriptMap();
348 Protocol.Debugger.enable(); 349 Protocol.Debugger.enable();
349 Protocol.Debugger.onPaused(message => { 350 Protocol.Debugger.onPaused(message => {
350 let callFrames = message.params.callFrames; 351 let callFrames = message.params.callFrames;
351 InspectorTest.logSourceLocations(callFrames.map(frame => frame.location)); 352 InspectorTest.logSourceLocations(callFrames.map(frame => frame.location));
352 }) 353 })
353 } 354 }
354 } 355 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698