DevTools: enable backwards-compatibility testing

third_party/WebKit/Source/devtools/front_end/protocol/InspectorBackend.js

Index: third_party/WebKit/Source/devtools/front_end/protocol/InspectorBackend.js
diff --git a/third_party/WebKit/Source/devtools/front_end/protocol/InspectorBackend.js b/third_party/WebKit/Source/devtools/front_end/protocol/InspectorBackend.js
index 3c01fe964b7e0c01ff7d14d5ca99c52d340ad991..5fe804e771ca226361cab58cf9f873a65e756974 100644
--- a/third_party/WebKit/Source/devtools/front_end/protocol/InspectorBackend.js
+++ b/third_party/WebKit/Source/devtools/front_end/protocol/InspectorBackend.js
@@ -634,13 +634,6 @@ Protocol.InspectorBackend._AgentPrototype = class {
    * @param {function(*)|function(?Protocol.Error, ?Object)} callback
    */
   dispatchResponse(messageObject, methodName, callback) {
-    if (messageObject.error && messageObject.error.code !== Protocol.InspectorBackend._ConnectionClosedErrorCode &&

[chenwilliam, 2017/01/19 23:08:00]

I removed this after discussing with @pfeldman because missing protocol methods
are being logged into the test results. Each caller that gracefully handles
missing protocol methods should have a customized log which will be recorded in
the test results.

Example:
error: Request Debugger.getPossibleBreakpoints failed.
{"code":-32601,"message":"'Debugger.getPossibleBreakpoints' wasn't found"}

[dgozman, 2017/01/20 17:52:09]

Let's land this separately:
- I'm worried we will miss some new issues due to no logging;
- removing this allows more cleanup on error codes.

[chenwilliam, 2017/01/20 20:19:21]

OK, removing from CL. What kind of cleanup on error codes do you want? (I can do
it?)

-        messageObject.error.code !== Protocol.InspectorBackend.DevToolsStubErrorCode &&
-        !Protocol.InspectorBackend.Options.suppressRequestErrors) {
-      var id = Protocol.InspectorBackend.Options.dumpInspectorProtocolMessages ? ' with id = ' + messageObject.id : '';
-      console.error('Request ' + methodName + id + ' failed. ' + JSON.stringify(messageObject.error));
-    }
-
     var argumentsArray = [];
     argumentsArray[0] = messageObject.error ? messageObject.error.message : null;