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.

Sign up for GitHub

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

Jump to bottom

Replace the implementation of escapeTextContentForBrowser with escape-html #6862

Merged
merged 4 commits into from
Jun 2, 2016
Merged

Replace the implementation of escapeTextContentForBrowser with escape-html #6862

merged 4 commits into from
Jun 2, 2016

Commits on May 24, 2016

  1. Replacing the implementation of escapeTextContentForBrowser with esca… 

    …pe-html for performance
    @aickin
    aickin committed May 24, 2016
    Configuration menu
    Copy the full SHA
    9baaf0a View commit details
    Browse the repository at this point in the history

Commits on Jun 1, 2016

  1. Addressing @spicyj's code review comment here: facebook#6862 (comment) … 

    …. Pulled the code of escape-html in to react and changed the encoding of single quote to &#x27.
    @aickin
    aickin committed Jun 1, 2016
    Configuration menu
    Copy the full SHA
    28b49ea View commit details
    Browse the repository at this point in the history

  2. Addressing code review comment facebook#6862 (comment) to make code m… 

    …ore inlinable for v8. Thanks, @spicyj.
    @aickin
    aickin committed Jun 1, 2016
    Configuration menu
    Copy the full SHA
    97e359a View commit details
    Browse the repository at this point in the history

  3. Merge branch 'master' into use-escape-html

    @aickin
    aickin committed Jun 1, 2016
    Configuration menu
    Copy the full SHA
    7cc0c80 View commit details
    Browse the repository at this point in the history