[Webkit-unassigned] [Bug 113260] Add a runtime flag in WebRuntimeFeatures to enable 'image/webp' accept header

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Wed Mar 27 10:47:17 PDT 2013


https://bugs.webkit.org/show_bug.cgi?id=113260


Darin Fisher (:fishd, Google) <fishd at chromium.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #195009|review?                     |review-
               Flag|                            |




--- Comment #9 from Darin Fisher (:fishd, Google) <fishd at chromium.org>  2013-03-27 10:45:27 PST ---
(From update of attachment 195009)
View in context: https://bugs.webkit.org/attachment.cgi?id=195009&action=review

>> Source/WebCore/loader/cache/CachedImage.h:102
>> +    void setCustomAccept(); // Helper method to set the Accept header.
> 
> Would you mind renaming this to something that makes it clear that we're setting the header? I think 'CachedImage::setCustomAcceptHeader' would be clear enough that you could remove the comment.

+1

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.


More information about the webkit-unassigned mailing list