[inspector] Refactor protocol-test.js

test/inspector/console/destroy-context-during-log.js

 // Copyright 2016 the V8 project authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
+let {session, contextGroup, Protocol} = InspectorTest.start('Tests that destroying context from inside of console.log does not crash');

[kozy, 2017/05/18 23:44:54]

If you already have a script, could you change it to:
let {session, contextGroup, Protocol}
  = InspectorTest.start..

to not exceed 80 characters line limit.

[dgozman, 2017/05/19 00:02:45]

Acknowledged.

+
 const expression = `
   Object.defineProperty(Object.prototype, 'RemoteObject', {
     configurable: true,
     set(v) {
       console.log("Should never be called");
       delete Object.prototype.RemoteObject;
       this.RemoteObject = v;
 
       inspector.fireContextDestroyed();
       setTimeout(function() {
@@ skipping 16 matching lines @@
 
 Protocol.Runtime.enable();
 Protocol.Runtime.evaluate({ expression: expression });
 
 Protocol.Runtime.onConsoleAPICalled(function(result) {
   InspectorTest.logObject(result.params.args[0]);
   if (result.params.args[0].value == "End of test") {
     InspectorTest.completeTest();
   }
 });