Open Bug 1996797 Opened 2 months ago

[wpt-sync] Sync PR 55699 - Http2WebTestRequestHandler shouldn't subclass http.server.BaseHTTPRequestHandler

Categories

(Testing :: web-platform-tests, task, P4)

Product:
Testing
Component:
web-platform-tests
Type:
task

Tracking

(Not tracked)

People

(Reporter: wpt-sync, Unassigned)

References

(
URL
)

Details

(Whiteboard: [wptsync downstream])

Sync web-platform-tests PR 55699 into mozilla-central (this bug is closed when the sync is complete).

PR: https://github.com/web-platform-tests/wpt/pull/55699
Details from upstream follow.

Sam Sneddon <gsnedders@apple.com> wrote:

Http2WebTestRequestHandler shouldn't subclass http.server.BaseHTTPRequestHandler

H1 and H2 are sufficiently different protocols that it doesn't really
make any sense for this to be a subclass, and we don't actually use
anything from the superclass, which shows how needless this is.

Additionally, move methods from BaseWebTestRequestHandler to
Http1WebTestRequestHandler where they're only used by that subclass.

You need to log in before you can comment on or make changes to this bug.