Summary: | Missing include in MediaDescription | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Enrique Ocaña <eocanha> | ||||||
Component: | Platform | Assignee: | Enrique Ocaña <eocanha> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | calvaris, commit-queue | ||||||
Priority: | P2 | ||||||||
Version: | WebKit Local Build | ||||||||
Hardware: | Unspecified | ||||||||
OS: | Unspecified | ||||||||
Bug Depends on: | |||||||||
Bug Blocks: | 157314 | ||||||||
Attachments: |
|
Description
Enrique Ocaña
2016-10-03 11:46:21 PDT
Created attachment 290498 [details]
Patch
Comment on attachment 290498 [details]
Patch
Actually, does <wtf/Forward.h> work?
Created attachment 290585 [details]
Patch
Yes, Forward.h works at the expense of having to include the AtomicString header in the final cpp file. It seems a good compromise. Comment on attachment 290585 [details]
Patch
For this you wouldn't even need to upload the patch for review. It could be an unreviewed build fix.
And it is something you can land now, no need to wait for other MSE changes. Comment on attachment 290585 [details] Patch Clearing flags on attachment: 290585 Committed r206952: <http://trac.webkit.org/changeset/206952> All reviewed patches have been landed. Closing bug. |