Meta refresh parsing logic is not interoperable yet |
|||
Issue descriptionChrome Version: 61 Canary OS: All but iOS What steps will reproduce the problem? (1) Open http://w3c-test.org/html/semantics/document-metadata/the-meta-element/pragma-directives/attr-meta-http-equiv-refresh/parsing.html (2) Wait until test results are rendered. What is the expected result? All tests pass. What happens instead? 3 Fail, 10 Timeout Especially, the following tests pass on both of Edge and Firefox. We should fix them. "1\nurl=foo" "1\turl=foo" "1\rurl=foo" "1.9..5.; url=foo" "1 url=foo" " 1 url = foo" Please use labels and text to provide additional information. Specification: https://html.spec.whatwg.org/multipage/semantics.html#attr-meta-http-equiv-refresh
,
Jul 18 2017
,
Jul 18 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/e6cbbc3c522a73a9e8e664c3f7a62da8c3103115 commit e6cbbc3c522a73a9e8e664c3f7a62da8c3103115 Author: Kent Tamura <tkent@chromium.org> Date: Tue Jul 18 12:14:00 2017 <meta>: Correct number parsing for meta refresh. We should ignore trailing "." and digits after a number. We should reject a leading "-", a leading "+", and scientific notation. Bug: 742104 Change-Id: I4e5ed15e0a07f361fd86680a60430cb2d760dc83 Reviewed-on: https://chromium-review.googlesource.com/574973 Commit-Queue: Kent Tamura <tkent@chromium.org> Reviewed-by: Takayoshi Kochi <kochi@chromium.org> Cr-Commit-Position: refs/heads/master@{#487446} [modify] https://crrev.com/e6cbbc3c522a73a9e8e664c3f7a62da8c3103115/third_party/WebKit/LayoutTests/TestExpectations [add] https://crrev.com/e6cbbc3c522a73a9e8e664c3f7a62da8c3103115/third_party/WebKit/LayoutTests/external/wpt/html/semantics/document-metadata/the-meta-element/pragma-directives/attr-meta-http-equiv-refresh/parsing-expected.txt [modify] https://crrev.com/e6cbbc3c522a73a9e8e664c3f7a62da8c3103115/third_party/WebKit/Source/platform/network/HTTPParsers.cpp [modify] https://crrev.com/e6cbbc3c522a73a9e8e664c3f7a62da8c3103115/third_party/WebKit/Source/platform/network/HTTPParsersTest.cpp
,
Jul 18 2017
,
Jul 19 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/c02f55b15d42f7c36e41c1115b6853d179f66acb commit c02f55b15d42f7c36e41c1115b6853d179f66acb Author: Kent Tamura <tkent@chromium.org> Date: Wed Jul 19 04:57:06 2017 [html] Mark html/.../attr-meta-http-equiv-refresh/parsing.html slow. It times out sometimes with Release builds, frequently with Debug builds. Bug: 742104 Change-Id: I57d727cb37885a02700624f9a36b90191735f975 Reviewed-on: https://chromium-review.googlesource.com/576598 Commit-Queue: Kent Tamura <tkent@chromium.org> Commit-Queue: Takayoshi Kochi <kochi@chromium.org> Reviewed-by: Takayoshi Kochi <kochi@chromium.org> Cr-Commit-Position: refs/heads/master@{#487756} [modify] https://crrev.com/c02f55b15d42f7c36e41c1115b6853d179f66acb/third_party/WebKit/LayoutTests/external/wpt/html/semantics/document-metadata/the-meta-element/pragma-directives/attr-meta-http-equiv-refresh/parsing.html |
|||
►
Sign in to add a comment |
|||
Comment 1 by bugdroid1@chromium.org
, Jul 13 2017