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

Jump to bottom

HTML: remove nonsense JSON resources #17574

Merged
merged 2 commits into from Jul 10, 2019
Merged

HTML: remove nonsense JSON resources #17574

merged 2 commits into from Jul 10, 2019

Conversation

annevk
Copy link
Member

@annevk annevk commented Jun 28, 2019

Pretty sure we don't have a use for these.

@annevk
HTML: remove nonsense JSON resources
7c2fba8 
Pretty sure we don't have a use for these.
@zcorpan
Copy link
Member

zcorpan commented Jun 28, 2019 

$ find . -name "contains.json"
./html/webappapis/scripting/events/contains.json
./html/webappapis/scripting/event-loops/contains.json
./html/webappapis/scripting/processing-model-2/contains.json
./html/webappapis/system-state-and-capabilities/the-navigator-object/contains.json
./html/semantics/sections/headings-and-sections/contains.json
./html/semantics/forms/form-control-infrastructure/contains.json
./html/semantics/forms/constraints/contains.json
./html/semantics/forms/introduction-1/contains.json
./html/semantics/forms/form-submission-0/contains.json
./html/semantics/forms/attributes-common-to-form-controls/contains.json
./html/semantics/forms/the-input-element/contains.json
./html/semantics/scripting-1/the-script-element/contains.json
./html/semantics/tabular-data/the-table-element/contains.json
./html/semantics/tabular-data/processing-model-1/contains.json
./html/semantics/links/linktypes/contains.json
./html/semantics/links/downloading-resources/contains.json
./html/semantics/embedded-content/the-canvas-element/contains.json
./html/semantics/embedded-content/the-img-element/contains.json
./html/semantics/embedded-content/image-maps/contains.json
./html/semantics/embedded-content/media-elements/contains.json
./html/semantics/interactive-elements/the-dialog-element/contains.json
./html/semantics/interactive-elements/commands/contains.json
./html/semantics/document-metadata/the-meta-element/contains.json
./html/browsers/history/the-location-interface/contains.json
./html/browsers/windows/nested-browsing-contexts/contains.json
./html/browsers/windows/auxiliary-browsing-contexts/contains.json
./html/browsers/browsing-the-web/unloading-documents/contains.json
./html/browsers/browsing-the-web/history-traversal/contains.json
./html/browsers/offline/introduction-4/contains.json
./html/browsers/offline/manifests/contains.json
./html/dom/elements/content-models/contains.json
./html/dom/elements/element-definitions/contains.json
./html/dom/elements/global-attributes/contains.json
./html/dom/elements/wai-aria/contains.json
./html/syntax/parsing/contains.json
./html/syntax/writing/elements-0/contains.json
./html/syntax/writing/text/contains.json
./html/infrastructure/common-microsyntaxes/numbers/contains.json
./html/infrastructure/common-microsyntaxes/dates-and-times/contains.json
./html/infrastructure/common-dom-interfaces/collections/contains.json

it seems these were added in 2012 by @darobin

@annevk
Copy link
Member Author

annevk commented Jun 28, 2019

Removed those too.

@darobin
Copy link
Contributor

darobin commented Jun 28, 2019

Yeah, I have no idea what they were but they look like artifacts from a big import of tests or something like that.

@Ms2ger
Copy link
Contributor

Ms2ger commented Jul 10, 2019

They were added when @darobin created the directory tree in 818bcab. I doubt they were ever used after that.

@jgraham jgraham merged commit 0f05ba7 into master Jul 10, 2019
@domenic domenic deleted the annevk/html-nonsense-json branch July 10, 2019 13:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jgraham jgraham jgraham approved these changes

@domenic domenic Awaiting requested review from domenic

@foolip foolip Awaiting requested review from foolip

@jdm jdm Awaiting requested review from jdm

@zcorpan zcorpan Awaiting requested review from zcorpan

@zqzhang zqzhang Awaiting requested review from zqzhang

@gsnedders gsnedders Awaiting requested review from gsnedders

@tkent-google tkent-google Awaiting requested review from tkent-google

Assignees

@jgraham jgraham

Labels
html
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
6 participants
@annevk @zcorpan @darobin @Ms2ger @jgraham @wpt-pr-bot