• adamk's avatar
    Include harmony-string-padding.js in the context snapshot · 02149d89
    adamk authored
    The attached chromium bug suggests that turning this on via a flag
    regressed a perf test. Having previously run into slowdowns due
    to experimental JS natives, my suspicion is that this regression
    can be fixed by including the new builtins in the snapshot.
    My intention is to land this patch to see if the regression
    goes away on the bots.
    
    Given the nature of this feature, it seems low-risk to remove
    the runtime flag. If need be, it would be trivial to craft
    a patch to remove these two methods from String.prototype
    if we run into web compat issues during the release (note
    that no release branch has yet picked up this feature).
    
    BUG=v8:4954, chromium:677444
    
    Review-Url: https://codereview.chromium.org/2610853010
    Cr-Commit-Position: refs/heads/master@{#42111}
    02149d89
harmony-string-pad-start.js 3.16 KB