Remove the separate code path on Mac/Qt for user stylesheet loading. Make sure the cross-platform code path used by everyone else gets data URL support to match Mac/Qt, and then switch everyone over to it.
Created attachment 39088 [details]
Comment on attachment 39088 [details]
> void Page::userStyleSheetLocationChanged()
> -#if !FRAME_LOADS_USER_STYLESHEET
> // FIXME: We should provide a way to load other types of URLs than just
> - // file: (e.g., http:, data:).
> - if (m_settings->userStyleSheetLocation().isLocalFile())
> - m_userStyleSheetPath = m_settings->userStyleSheetLocation().fileSystemPath();
> + // file: (e.g., http:).
I guess you don't agree with this FIXME at all. Maybe you should remove it?
You should remove UserStyleSheetLoader from the Qt project files.
You should explain in your ChangeLog how you made the decision that it's OK to do this.
Do we have regression tests for the user stylesheet code?
Fixed in r48075.