Cf (form control) and default ignorable characters are converted to 'nothing' before being submitted via form |
|||||||||||
Issue descriptionI'm putting Blink>Forms for now, but it may be an html parsing issue. bug 626396 and bug 626400 share the same root cause. When one of characters listed in https://goo.gl/WXRxCa are is submitted via form (text field), it's turned to nothing. See https://github.com/whatwg/encoding/issues/59 and https://github.com/whatwg/encoding/issues/62
,
Sep 16 2016
,
Sep 16 2016
We should start to investigate core/html/FormData.* and platform/network/FormDataEncoder.*.
,
Sep 16 2016
,
Sep 18 2016
,
Sep 18 2016
,
Sep 18 2016
,
Sep 18 2016
,
Sep 18 2016
,
Sep 18 2016
,
Sep 18 2016
,
Sep 18 2016
,
Sep 21 2016
It seems ICU removes them. ICU code: https://cs.chromium.org/chromium/src/third_party/icu/source/common/ucnv_err.c?q=UCNV_FROM_U_CALLBACK_ESCAPE&sq=package:chromium&dr=C&l=223 Blink side caller is TextCodecICU::encodeInternal(input, EntitiesForUnencodables) in wtf/text/TextCodecICU.cpp.
,
Sep 21 2016
,
Sep 26 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/65267c78144e27387dcc19b75c79407b904f8f1c commit 65267c78144e27387dcc19b75c79407b904f8f1c Author: tkent <tkent@chromium.org> Date: Mon Sep 26 15:57:44 2016 FormData should not drop ignorable code points. UCNV_FROM_U_CALLBACK_ESCAPE() used for EntitiesForUnencodables removes "ignorable" characters. We should avoid to use it. BUG= 647568 Review-Url: https://codereview.chromium.org/2354533005 Cr-Commit-Position: refs/heads/master@{#420897} [modify] https://crrev.com/65267c78144e27387dcc19b75c79407b904f8f1c/third_party/WebKit/Source/wtf/BUILD.gn [modify] https://crrev.com/65267c78144e27387dcc19b75c79407b904f8f1c/third_party/WebKit/Source/wtf/text/TextCodecICU.cpp [modify] https://crrev.com/65267c78144e27387dcc19b75c79407b904f8f1c/third_party/WebKit/Source/wtf/text/TextCodecICU.h [add] https://crrev.com/65267c78144e27387dcc19b75c79407b904f8f1c/third_party/WebKit/Source/wtf/text/TextCodecICUTest.cpp
,
Sep 26 2016
Issue 626402 has been merged into this issue.
,
Sep 26 2016
|
|||||||||||
►
Sign in to add a comment |
|||||||||||
Comment 1 by js...@chromium.org
, Sep 16 2016