Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

deps: cherry-pick f4a2b7f3 from V8 upstream. #16053

Closed
wants to merge 1 commit into from

Commits on Oct 6, 2017

  1. deps: cherry-pick f4a2b7f3 from V8 upstream.

    Original commit message:
        should ignore asyncTask* with null
    
        In V8Debugger code we don't expect task_id == null, e.g.
        asyncTaskStartedForStepping will trigger debug break on null as task_id.
        Let's filter task_id == null out.
    
        This issue is originally filed in Node.js:
        nodejs#15464
    
        R=dgozman@chromium.org
    
        Bug: none
        Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel
        Change-Id: Icc9f96105b3c91ee1b102d545a7817f7ee93394c
        Reviewed-on: https://chromium-review.googlesource.com/695808
        Reviewed-by: Dmitry Gozman <dgozman@chromium.org>
        Commit-Queue: Aleksey Kozyatinskiy <kozyatinskiy@chromium.org>
        Cr-Commit-Position: refs/heads/master@{nodejs#48265}
    
    Fixes nodejs#15464
    alexkozy authored and Erin Spiceland committed Oct 6, 2017
    Configuration menu
    Copy the full SHA
    e466e36 View commit details
    Browse the repository at this point in the history