Skip to content
This repository has been archived by the owner on Sep 6, 2021. It is now read-only.

Show more details of uncaught exceptions #1200

Closed
DennisKehrig opened this issue Jun 29, 2012 · 3 comments
Closed

Show more details of uncaught exceptions #1200

DennisKehrig opened this issue Jun 29, 2012 · 3 comments

Comments

@DennisKehrig
Copy link
Contributor

As an extension developer using the Inspector, I want to not just know that "Some arguments of method 'Debugger.setBreakpointByUrl' can't be processed", I also want to see the part where it says "Parameter 'lineNumber' has wrong type. It must be 'Number'."

Fix: change _onError in LiveDevelopment.js to this:

function _onError(error) {
    var message = error.message;
    // Additional information, like exactly which parameter could not be processed.
    if (error.data) {
        message += "\n" + error.data;
    }
    console.error(message);
}
@redmunds
Copy link
Contributor

Dennis,

Thank you for contributing. This looks like a great idea. Can you submit a pull request for it? This will make it easier for us to see the exact differences when reviewing, make it easier to merge, and also give you credit for the submission.

Thanks,
Randy

@DennisKehrig
Copy link
Contributor Author

Alright, did so! Thanks for replying!

gruehle added a commit that referenced this issue Jul 11, 2012
Improve console output of LiveDevelopment errors (#1200)
@gruehle
Copy link
Member

gruehle commented Jul 11, 2012

Thanks for submitting the pull request - it's been merged. Closing.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants