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

Patch Function#call and Function#apply together, more robust than single fix. #347

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Commits on Jan 26, 2016

  1. Patch Function#call and Function#apply together, more robust than…

    … single fix.
    
    Ref: es-shims#304
    
    Changed some code back as mentioned in comments.
    
    Changed more code as per comments
    
    Changed more code as per comments.
    
    Changed some variable names to better reflect comments.
    
    Fixed missed invocation.
    
    Added some comments and code changes as discussed.
    
    [tests] Remove unneeded jshint comment.
    
    [Tests] use the preferred it/skip pattern for this strict mode test.
    
    es-shims#345 (comment)
    And some other cleanup
    
    Added `arguments` expectations to tests.
    
    Add tests for `Object#toString` of typed arrays and Symbols, if they exist.
    
    Added note about typed array tests.
    
    Fix `hasToStringTagRegExpBug`
    
    Removed RegExp and Array bug detection as can not test, possible Opera 9.
    Fixed missing `force` on `defineProperties` that caused the patch to not be applied on IE<9.
    Fixed `Uint8ClampedArray` tests for Opera 11 and IE10 that don't have it.
    
    Removed offending test that was moved to detection, but forgotten.
    
    Avoid all possibilities of `call` calling `call`.
    
    Do not pass `undefined` argument, we know IE<9 has unfixable bug.
    
    Final cleanup (hopeully)
    
    Port over work from `apply` fix
    
    Move code so that it is specific to the fix.
    
    Robustness, move before bind.
    
    Remove `Array#slice` tests.
    
    Add notes about `eval` and `apply` avoidance.
    Xotic750 authored and Xotic750 committed Jan 26, 2016
    Configuration menu
    Copy the full SHA
    51a9019 View commit details
    Browse the repository at this point in the history