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

Sync for validator/cpp/htmlparser #32570

Merged
merged 8 commits into from
Feb 9, 2021
Merged

Sync for validator/cpp/htmlparser #32570

merged 8 commits into from
Feb 9, 2021

Conversation

MichaelRybak
Copy link
Contributor

This is an automatically generated pull request.

@amp-owners-bot
Copy link

amp-owners-bot bot commented Feb 9, 2021

Hey @ampproject/wg-caching! These files were changed:

validator/cpp/htmlparser/doctype.cc
validator/cpp/htmlparser/elements.h
validator/cpp/htmlparser/htmldataset_test.cc
validator/cpp/htmlparser/node.cc
validator/cpp/htmlparser/node.h
validator/cpp/htmlparser/parser.cc
validator/cpp/htmlparser/parser.h
validator/cpp/htmlparser/renderer_test.cc
validator/cpp/htmlparser/strings.cc
validator/cpp/htmlparser/strings_test.cc
validator/cpp/htmlparser/testdata/tree-construction/README.md
validator/cpp/htmlparser/testdata/tree-construction/adoption01.dat
+33 more

…Slash

reported by fuzzer.

PiperOrigin-RevId: 356367850
- Misc fixes and greater html5 compliance. Only 5 tests of 1500 webkit testcases failing. The broken test cases are not critical to amp parsing requirements.

- Updated html5 testdataset (git clone
git@github.com:html5lib/html5lib-tests.git).

- Double the testcase coverage from ~750 to ~1500.

- Corrects column numbers for <?xml|php|doctype etc. tags.

PiperOrigin-RevId: 356380828
Fix fuzz_parser: Crash in fuzz_parser.cc (b/168830101)
Fix fuzz_parser: Crash in htmlparser::Allocator<htmlparser::Node>::FreeBlocks (b/168379832)

All these test cases were pretty large HTML with hundreds of nodes and attributes. Tests were passing for less than 10 runs but failing for 100 runs.

Reduced pressure on stack by allocating objects on heap, plus additional
fix in tokenizer.

PiperOrigin-RevId: 356416503
PiperOrigin-RevId: 356606309
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants