Bug 134022

Summary: [Soup] Image Data URIs not displaying if media-type is not specified
Product: WebKit Reporter: Shanmuga Pandi M <shanmuga.m>
Component: PlatformAssignee: Nobody <webkit-unassigned>
Status: RESOLVED WORKSFORME    
Severity: Normal CC: berto, cdumez, cgarcia, commit-queue, danw, gustavo, gyuyoung.kim, mcatanzaro, mrobinson
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Linux   
Attachments:
Description Flags
Sample test file
none
Patch
none
Patch none

Description Shanmuga Pandi M 2014-06-18 02:00:31 PDT
Created attachment 233299 [details]
Sample test file

On Webkit2 EFL port,

Steps to Reproduce:

1. Open attached sample html file.
2. Observe

Actual Result:
    Nothing is displayed.

Expected Result:
    One directory icon should get displayed


Firefox and chrome working as expected.

If Data-URI doesn't contain media-type, then ImageDecoder creation fails due to improper data it receives.
Comment 1 Shanmuga Pandi M 2014-06-18 02:59:52 PDT
Created attachment 233301 [details]
Patch
Comment 2 WebKit Commit Bot 2014-06-18 03:00:58 PDT
Attachment 233301 [details] did not pass style-queue:


ERROR: Source/WebCore/ChangeLog:8:  You should remove the 'No new tests' and either add and list tests, or explain why no new tests were possible.  [changelog/nonewtests] [5]
Total errors found: 1 in 4 files


If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 3 Shanmuga Pandi M 2014-06-18 03:17:22 PDT
Created attachment 233303 [details]
Patch
Comment 4 Dan Winship 2014-06-18 06:24:04 PDT
This seems like something that should be fixed in libsoup
Comment 5 Shanmuga Pandi M 2014-06-18 06:28:45 PDT
(In reply to comment #4)
> This seems like something that should be fixed in libsoup


Yes. Can we push this patch until libsoup fixes this issue?
Comment 6 Gustavo Noronha (kov) 2014-06-19 13:02:08 PDT
Perhaps you could patch libsoup instead? Shouldn't be much harder than fixing it here.
Comment 7 Michael Catanzaro 2016-09-17 07:14:11 PDT
Comment on attachment 233303 [details]
Patch

Hi,

Apologies that your patch was not reviewed in a timely manner. Since it's now quite old, I am removing it from the review request queue. Please consider rebasing it on trunk and resubmitting.

To increase the chances of getting a review, consider using 'Tools/Scripts/webkit-patch upload --suggest-reviewers' to CC reviewers who might be interested in this bug.
Comment 8 Michael Catanzaro 2016-09-17 07:14:37 PDT
I see a directory icon, so I guess it's already been fixed.