Bug 126614

Summary: Web Inspector: Remove support for overriding the page CSS media type
Product: WebKit Reporter: Seokju Kwon <seokju>
Component: Web InspectorAssignee: Seokju Kwon <seokju>
Status: RESOLVED WONTFIX    
Severity: Normal CC: commit-queue, eric.carlson, glenn, graouts, jer.noble, joepeck, timothy, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Bug Depends on:    
Bug Blocks: 126236    
Attachments:
Description Flags
Patch timothy: review-

Seokju Kwon
Reported 2014-01-07 20:33:32 PST
Remove support for overriding the page CSS media type.
Attachments
Patch (16.92 KB, patch)
2014-01-07 20:47 PST, Seokju Kwon 		timothy: review-
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Radar WebKit Bug Importer
Comment 1 2014-01-07 20:33:43 PST
<rdar://problem/15769511>
Seokju Kwon
Comment 2 2014-01-07 20:47:30 PST
Created attachment 220587 [details] Patch
Timothy Hatcher
Comment 3 2014-01-07 21:25:26 PST
Comment on attachment 220587 [details] Patch Thanks for tackling these removals. However, I think this media emulation is something we likely want to keep and expose in the Style details sidebar or Safari Decelop menu. Let's keep this support for now.
Seokju Kwon
Comment 4 2014-01-07 21:54:02 PST
(In reply to comment #3) > (From update of attachment 220587 [details]) > Thanks for tackling these removals. However, I think this media emulation is something we likely want to keep and expose in the Style details sidebar or Safari Decelop menu. Let's keep this support for now. Ok. I will let you know before filing a bug related to removing protocols. Thanks.
Joseph Pecoraro
Comment 5 2014-01-08 10:11:06 PST
(In reply to comment #3) > (From update of attachment 220587 [details]) > Thanks for tackling these removals. However, I think this media emulation is something we likely want to keep and expose in the Style details sidebar I agree as well. This would be something worth adding to our frontend instead of removing from the backend.
Note You need to log in before you can comment on or make changes to this bug.