Bug 132661

Summary: Use system defaults for hardware jpeg decoding
Product: WebKit Reporter: Pratik Solanki <psolanki>
Component: New BugsAssignee: Pratik Solanki <psolanki>
Status: RESOLVED FIXED    
Severity: Normal CC: psolanki
Priority: P2 Keywords: InRadar
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch thorton: review+

Pratik Solanki
Reported 2014-05-07 13:50:35 PDT
Use system defaults for hardware jpeg decoding
Attachments
Patch (7.25 KB, patch)
2014-05-07 13:52 PDT, Pratik Solanki 		thorton: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Pratik Solanki
Comment 1 2014-05-07 13:52:43 PDT
Created attachment 231019 [details] Patch
Pratik Solanki
Comment 2 2014-05-07 13:53:14 PDT
<rdar://problem/11348201>
Tim Horton
Comment 3 2014-05-07 13:54:27 PDT
Comment on attachment 231019 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=231019&action=review > Source/WebKit/mac/WebView/WebViewPrivate.h:-468 > -+ (void)_setAcceleratedImageDecoding:(BOOL)enabled; > -+ (BOOL)_acceleratedImageDecoding; Please triple check that these aren't used anywhere.
Pratik Solanki
Comment 4 2014-05-07 16:28:14 PDT
(In reply to comment #3) > (From update of attachment 231019 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=231019&action=review > > > Source/WebKit/mac/WebView/WebViewPrivate.h:-468 > > -+ (void)_setAcceleratedImageDecoding:(BOOL)enabled; > > -+ (BOOL)_acceleratedImageDecoding; > > Please triple check that these aren't used anywhere. Good catch. I did find one client. I am going to leave _setAcceleratedImageDecoding as an empty method.
Pratik Solanki
Comment 5 2014-05-07 16:29:47 PDT
Committed r168452: <http://trac.webkit.org/changeset/168452>
Note You need to log in before you can comment on or make changes to this bug.