Bug 152634

Summary: [SOUP] REGRESSION(r192761): Broke resource URIs for applications that use g_resource_load in a web extension
Product: WebKit Reporter: Michael Catanzaro <mcatanzaro>
Component: Page LoadingAssignee: Michael Catanzaro <mcatanzaro>
Status: RESOLVED FIXED    
Severity: Normal CC: beidson, bugs-noreply, commit-queue, mario, mcatanzaro
Priority: P2    
Version: Other   
Hardware: PC   
OS: Linux   
Attachments:
Description Flags
Patch none

Description Michael Catanzaro 2015-12-31 20:35:46 PST
Now that network process is mandatory, applications that were using the default WEBKIT_PROCESS_MODEL_SHARED_SECONDARY_PROCESS to allow injecting GResources into the web process have been seriously broken, since the loads occur in the network process but the resources were injected into the web process.

I'm kind of surprised that we support resource URIs, actually (did we somehow inherit that accidentally from libsoup?), but it's a nice feature, so let's keep it working by handling resource URIs in the web process rather than the network process, same as we already do for data URIs and QuickLook URIs (whatever those are ;)
Comment 1 Michael Catanzaro 2015-12-31 20:51:53 PST
Created attachment 268066 [details]
Patch
Comment 2 Carlos Garcia Campos 2016-01-01 01:13:07 PST
Comment on attachment 268066 [details]
Patch

Excellent
Comment 3 WebKit Commit Bot 2016-01-01 13:22:42 PST
Comment on attachment 268066 [details]
Patch

Clearing flags on attachment: 268066

Committed r194480: <http://trac.webkit.org/changeset/194480>
Comment 4 WebKit Commit Bot 2016-01-01 13:22:45 PST
All reviewed patches have been landed.  Closing bug.