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: upgrade to V8 4.7.80.24 #4106

Closed
wants to merge 5 commits into from
Closed

Commits on Dec 4, 2015

  1. deps: upgrade V8 to 4.7.80.24

    Pick up the latest branch head for V8 4.7:
    v8/v8@be169f8
    
    Full change history for the 4.7 branch:
    https://chromium.googlesource.com/v8/v8.git/+log/branch-heads/4.7
    
    V8 blog post about what is new on V8 4.7:
    http://v8project.blogspot.de/2015/10/v8-release-47.html
    
    PR-URL: nodejs#4106
    Reviewed-By: bnoordhuis - Ben Noordhuis <info@bnoordhuis.nl>
    Reviewed-By: targos - Michaël Zasso <mic.besace@gmail.com>
    Reviewed-By: rvagg - Rod Vagg <rod@vagg.org>
    ofrobots committed Dec 4, 2015
    Configuration menu
    Copy the full SHA
    e73b0b7 View commit details
    Browse the repository at this point in the history
  2. test: fix test-repl-tab-complete after V8 upgrade

    Array#includes is now available.
    
    Ref: nodejs#3481
    PR-URL: nodejs#4106
    Reviewed-By: bnoordhuis - Ben Noordhuis <info@bnoordhuis.nl>
    Reviewed-By: targos - Michaël Zasso <mic.besace@gmail.com>
    Reviewed-By: rvagg - Rod Vagg <rod@vagg.org>
    ofrobots committed Dec 4, 2015
    Configuration menu
    Copy the full SHA
    44b568e View commit details
    Browse the repository at this point in the history
  3. deps: backport 1ee712a from V8 upstream

    Backport 1ee712ab8687e5f4dec93d45da068d37d28feb8b from V8 upstream.
    
    Original commit message:
    
      Add SetAbortOnUncaughtExceptionCallback API
    
      The --abort-on-uncaught-exception command line switch makes
      Isolate::Throw abort if the error being thrown cannot be caught by a
      try/catch block.
    
      Embedders may want to use other mechanisms than try/catch blocks to
      handle uncaught exceptions. For instance, Node.js has "domain" objects
      that have error handlers that can handle uncaught exception like
      following:
    
      var d = domain.create();
    
      d.on('error', function onError(err) {
        console.log('Handling error');
      });
    
      d.run(function() {
        throw new Error("boom");
      });
    
      These error handlers are called by isolates' message listeners.
    
      If --abort-on-uncaught-exception is *not* used, the isolate's
      message listener will be called, which will in turn call the domain's
      error handler. The process will output 'Handling error' and will exit
      successfully (not due to an uncaught exception). This is the behavior
      that Node.js users expect.
    
      However, if --abort-on-uncaught-exception is used and when throwing an
      error within a domain that has an error handler, the process will abort
      and the domain's error handler will not be called. This is not the
      behavior that Node.js users expect.
    
      Having a SetAbortOnUncaughtExceptionCallback API allows embedders to
      determine when it's not appropriate to abort and instead handle the
      exception via the isolate's message listener.
    
      In the example above, Node.js would set a custom callback with
      SetAbortOnUncaughtExceptionCallback that would be implemented as
      following (the sample code has been simplified to remove what's not
      relevant to this change):
    
      bool ShouldAbortOnUncaughtException(Isolate* isolate) {
        return !IsDomainActive();
      }
    
      Now when --abort-on-uncaught-exception is used, Isolate::Throw would
      call that callback and determine that it should not abort if a domain
      with an error handler is active. Instead, the isolate's message listener
      would be called and the error would be handled by the domain's error
      handler.
    
      I believe this can also be useful for other embedders.
    
      BUG=
    
      R=bmeurer@chromium.org
    
      Review URL: https://codereview.chromium.org/1375933003
    
      Cr-Commit-Position: refs/heads/master@{nodejs#31111}
    
    Ref: nodejs#3036
    Ref: nodejs#3481
    PR-URL: nodejs#4106
    Reviewed-By: bnoordhuis - Ben Noordhuis <info@bnoordhuis.nl>
    Reviewed-By: targos - Michaël Zasso <mic.besace@gmail.com>
    Reviewed-By: rvagg - Rod Vagg <rod@vagg.org>
    Julien Gilli authored and ofrobots committed Dec 4, 2015
    Configuration menu
    Copy the full SHA
    38e3f45 View commit details
    Browse the repository at this point in the history
  4. deps: cherry-pick 68e89fb from v8's upstream

    Original commit message:
    
        This commit adds some postmortem data that is otherwise unavailable.
    
        I have discovered need in those values when writing:
    
        https://github.com/indutny/llnode
    
        BUG=
    
        Review URL: https://codereview.chromium.org/1436473002
    
        Cr-Commit-Position: refs/heads/master@{nodejs#31947}
    
    This postmortem information is useful for both object inspection, and
    function's context variables inspection.
    
    Ref: nodejs#3779
    PR-URL: nodejs#4106
    Reviewed-By: bnoordhuis - Ben Noordhuis <info@bnoordhuis.nl>
    Reviewed-By: targos - Michaël Zasso <mic.besace@gmail.com>
    Reviewed-By: rvagg - Rod Vagg <rod@vagg.org>
    indutny authored and ofrobots committed Dec 4, 2015
    Configuration menu
    Copy the full SHA
    75f07f8 View commit details
    Browse the repository at this point in the history
  5. deps: backport bc2e393 from v8 upstream

    Original commit message:
    
      [tools] Make gen-postmortem-metadata.py more reliable
    
      Instead of basing matches off of whitespace, walk the
      inheritance chain and include any classes that inherit
      from Object.
    
      R=machenbach@chromium.org,jkummerow@chromium.org
      NOTRY=true
    
      Review URL: https://codereview.chromium.org/1435643002
    
      Cr-Commit-Position: refs/heads/master@{nodejs#31964}
    
    This adds some missing classes to postmortem info like
    JSMap and JSSet.
    
    Ref: nodejs#3792
    PR-URL: nodejs#4106
    Reviewed-By: bnoordhuis - Ben Noordhuis <info@bnoordhuis.nl>
    Reviewed-By: targos - Michaël Zasso <mic.besace@gmail.com>
    Reviewed-By: rvagg - Rod Vagg <rod@vagg.org>
    evan.lucas authored and ofrobots committed Dec 4, 2015
    Configuration menu
    Copy the full SHA
    014c27a View commit details
    Browse the repository at this point in the history