• Shimi Zhang's avatar
    Revert "[JJI] Add validation for jsObjectName" · 48ff1397
    Shimi Zhang authored
    This reverts commit a72a5f6b.
    
    Reason for revert: After a discussion with Torne and Changwan, we feel it is less meaningful to put a restriction there. Hence the revert.
    
    Original change's description:
    > [JJI] Add validation for jsObjectName
    > 
    > Since we are not using jsObjectName until the next navigation, it will
    > only until then we could tell if the parameter is valid or not, if it
    > is not valid, the injection will fail sliently. We want to provide a
    > early exception for developer to know if the object name is valid or
    > not.
    > 
    > Since it is hard for us to valid the full set of the allowed JavaScript
    > variable name, we limited the scope to a manageable subset of it.
    > 
    > Bug: 918065
    > Change-Id: I74909768fc9695e6d33302d880eb211a4b250c62
    > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1894181
    > Commit-Queue: Shimi Zhang <ctzsm@chromium.org>
    > Reviewed-by: Changwan Ryu <changwan@chromium.org>
    > Cr-Commit-Position: refs/heads/master@{#711795}
    
    TBR=torne@chromium.org,changwan@chromium.org,ctzsm@chromium.org
    
    Change-Id: Ic86d9b1681f7c806e085679b75cb713cc3b6a5c6
    No-Presubmit: true
    No-Tree-Checks: true
    No-Try: true
    Bug: 918065
    Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1896048Reviewed-by: default avatarShimi Zhang <ctzsm@chromium.org>
    Commit-Queue: Shimi Zhang <ctzsm@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#711875}
    48ff1397
post_message_repeat.html 197 Bytes