Bug 25085
| Summary: | Platform layering violations in platform/graphics/MediaPlayer files | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Simon Fraser (smfr) <simon.fraser> |
| Component: | Layout and Rendering | Assignee: | Nobody <webkit-unassigned> |
| Status: | NEW | ||
| Severity: | Normal | CC: | adele, dino, eric.carlson, jer.noble, koivisto, mitz, sam |
| Priority: | P2 | ||
| Version: | 528+ (Nightly build) | ||
| Hardware: | Mac | ||
| OS: | OS X 10.5 | ||
Simon Fraser (smfr)
MediaPlayer.cpp lives in WebCore/platform, yet it knows about FrameView, Frame and Document. This totally breaks the layering that 'platform' is supposed to adhere to.
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Dean Jackson
... and MediaDocument, where it tests for that a lot.
Antti Koivisto
The bug title and description are rather overheated. As far as I see all layering violations here are trivial and easily fixable.