[Webkit-unassigned] [Bug 58287] [chromium] WebCrossOriginRequestPolicy enum should use 'Web' prefix for values

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=58287





--- Comment #10 from Bill Budge <bbudge at gmail.com>  2011-04-12 10:26:38 PST ---
I don't think it's necessary to include WebURLLoaderOptions.h.

WebURLLoaderOptions really has nothing to do with the WebURLLoader abstraction, just with instantiating AssociatedURLLoaders (for now). In fact, at first I called these AssociatedURLLoaderOptions. Later I thought it was bad to expose a class with that name, which is really an implementation detail, and WebURLLoaderOptions seemed like a natural name.

So it seems incorrect (though perhaps convenient) to include the header in WebURLLoader. Anyone who wants to set up options has to include an extra file. Of course, I have to change my 2 patches to PPAPI to adapt to these changes.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.