Ignore:
Timestamp:
Aug 24, 2015, 4:58:31 PM (10 years ago)
Author:
BJ Burg
Message:

Web Inspector: add protocol test for existing error handling performed by the backend
​https://bugs.webkit.org/show_bug.cgi?id=147097

Reviewed by Joseph Pecoraro.

Source/JavaScriptCore:

A new test revealed that the protocol "method" parameter was being parsed in a naive way.
Rewrite it to use String::split and improve error checking to avoid failing later.

  • inspector/InspectorBackendDispatcher.cpp:

(Inspector::BackendDispatcher::dispatch):

Source/WebInspectorUI:

Add a way to send raw messages to the backend while still awaiting on responses.
This is necessary to test protocol error handling in the inspector backend.

  • UserInterface/Test/InspectorProtocol.js:

(InspectorProtocol.sendCommand):
(InspectorProtocol.awaitCommand): Use awaitMessage internally.
(InspectorProtocol.awaitMessage): Added. Use a dummy requestId if none is supplied.
(InspectorProtocol._sendMessage): Added.
(InspectorProtocol.dispatchMessageFromBackend):
Reject with the error object instead of the error message, so error code/data can be checked.
(InspectorProtocol.sendMessage): Deleted, it is now a private method.

LayoutTests:

Add a bunch of test cases to cover existing error handling by the backend dispatcher.

  • inspector/protocol/backend-dispatcher-argument-errors-expected.txt: Added.
  • inspector/protocol/backend-dispatcher-argument-errors.html: Added.
  • inspector/protocol/backend-dispatcher-malformed-message-errors-expected.txt: Added.
  • inspector/protocol/backend-dispatcher-malformed-message-errors.html: Added.
File:
1 edited

Legend:

Unmodified
Added
Removed

  • trunk/Source/WebInspectorUI/UserInterface/Test/InspectorProtocol.js

    r188639 r188897  
    2727InspectorProtocol = {};
    2828InspectorProtocol._dispatchTable = [];
     29InspectorProtocol._placeholderRequestIds = [];
    2930InspectorProtocol._requestId = -1;
    3031InspectorProtocol.eventHandler = {};
    … …  
    3940    this._dispatchTable[++this._requestId] = handler;
    4041    let messageObject = {method, params, id: this._requestId};
    41     this.sendMessage(messageObject);
     42    this._sendMessage(messageObject);
    4243
    4344    return this._requestId;
    … …  
    4748{
    4849    let {method, params} = args;
     50    let messageObject = {method, params, id: ++this._requestId};
     51
     52    return this.awaitMessage(messageObject);
     53}
     54
     55InspectorProtocol.awaitMessage = function(messageObject)
     56{
     57    // Send a raw message to the backend. Mostly used to test the backend's error handling.
    4958    return new Promise((resolve, reject) => {
    50         this._dispatchTable[++this._requestId] = {resolve, reject};
    51         let messageObject = {method, params, id: this._requestId};
    52         this.sendMessage(messageObject);
     59        let requestId = messageObject.id;
     60
     61        // If the caller did not provide an id, then make one up so that the response
     62        // can be used to settle a promise.
     63        if (typeof requestId !== "number") {
     64            requestId = ++this._requestId;
     65            this._placeholderRequestIds.push(requestId);
     66        }
     67
     68        this._dispatchTable[requestId] = {resolve, reject};
     69        this._sendMessage(messageObject);
    5370    });
    5471}
    … …  
    6683        }
    6784    });
     85}
     86
     87InspectorProtocol._sendMessage = function(messageObject)
     88{
     89    let messageString = typeof messageObject !== "string" ? JSON.stringify(messageObject) : messageObject;
     90
     91    if (ProtocolTest.dumpInspectorProtocolMessages)
     92        console.log(`frontend: ${messageString}`);
     93
     94    InspectorFrontendHost.sendMessageToBackend(messageString);
    6895}
    6996
    … …  
    94121}
    95122
    96 InspectorProtocol.sendMessage = function(messageObject)
    97 {
    98     // This matches the debug dumping in InspectorBackend, which is bypassed
    99     // by InspectorProtocol. Return messages should be dumped by InspectorBackend.
    100     if (ProtocolTest.dumpInspectorProtocolMessages)
    101         console.log("frontend: " + JSON.stringify(messageObject));
    102 
    103     InspectorFrontendHost.sendMessageToBackend(JSON.stringify(messageObject));
    104 }
    105 
    106123InspectorProtocol.checkForError = function(responseObject)
    107124{
    … …  
    122139    // If the message has an id, then it is a reply to a command.
    123140    let messageId = messageObject.id;
     141
     142    // If the id is 'null', then it may be an error response.
     143    if (messageId === null)
     144        messageId = InspectorProtocol._placeholderRequestIds.shift();
     145
     146    // If we could figure out a requestId, then dispatch the message.
    124147    if (typeof messageId === "number") {
    125148        let handler = InspectorProtocol._dispatchTable[messageId];
    … …  
    132155            let {resolve, reject} = handler;
    133156            if ("error" in messageObject)
    134                 reject(messageObject.error.message);
     157                reject(messageObject.error);
    135158            else
    136159                resolve(messageObject.result);
    … …  
    150173            let {resolve, reject} = handler;
    151174            if ("error" in messageObject)
    152                 reject(messageObject.error.message);
     175                reject(messageObject.error);
    153176            else
    154177                resolve(messageObject.result);
Note: See TracChangeset for help on using the changeset viewer.