Bug 152634 - [SOUP] REGRESSION(r192761): Broke resource URIs for applications that use g_resource_load in a web extension
Summary: [SOUP] REGRESSION(r192761): Broke resource URIs for applications that use g_r...
Alias: None
Product: WebKit
Classification: Unclassified
Component: Page Loading (show other bugs)
Version: Other
Hardware: PC Linux
: P2 Normal
Assignee: Michael Catanzaro
Depends on:
Reported: 2015-12-31 20:35 PST by Michael Catanzaro
Modified: 2016-01-01 13:22 PST (History)
5 users (show)

See Also:

Patch (7.42 KB, patch)
2015-12-31 20:51 PST, Michael Catanzaro
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
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]
Comment 2 Carlos Garcia Campos 2016-01-01 01:13:07 PST
Comment on attachment 268066 [details]

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

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.