RESOLVED FIXED 107854
[WK2][EFL] PagePolicyClientEfl should be based on C WK2 API 
https://bugs.webkit.org/show_bug.cgi?id=107854
Summary [WK2][EFL] PagePolicyClientEfl should be based on C WK2 API
Mikhail Pozdnyakov
Reported 2013-01-24 12:36:37 PST
SSIA
Attachments
patch (3.68 KB, patch)
2013-01-24 13:24 PST, Mikhail Pozdnyakov 		benjamin: review+
DetailsFormatted DiffDiff
patch v2 (3.63 KB, patch)
2013-01-25 01:14 PST, Mikhail Pozdnyakov 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Mikhail Pozdnyakov
Comment 1 2013-01-24 13:24:59 PST
Created attachment 184567 [details] patch
Chris Dumez
Comment 2 2013-01-24 13:42:06 PST
Comment on attachment 184567 [details] patch LGTM.
Benjamin Poulain
Comment 3 2013-01-24 13:59:35 PST
Comment on attachment 184567 [details] patch View in context: https://bugs.webkit.org/attachment.cgi?id=184567&action=review > Source/WebKit2/UIProcess/efl/PagePolicyClientEfl.cpp:94 > + bool isXMLType = WKStringIsEqualToUTF8CStringIgnoringCase(mimeType.get(), "text/xml") || WKStringIsEqualToUTF8CStringIgnoringCase(mimeType.get(), "application/xml"); Using the "IgnoringCase" here seems unnecessary. If ResourceResponse starts returning uppercase mimetype, it would break WebCore.
Mikhail Pozdnyakov
Comment 4 2013-01-25 01:14:35 PST
Created attachment 184694 [details] patch v2 removed "ignoring case"
WebKit Review Bot
Comment 5 2013-01-25 02:46:40 PST
Comment on attachment 184694 [details] patch v2 Clearing flags on attachment: 184694 Committed r140806: <http://trac.webkit.org/changeset/140806>
WebKit Review Bot
Comment 6 2013-01-25 02:46:44 PST
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.