[webkit-reviews] review granted: [Bug 58287] [chromium] WebCrossOriginRequestPolicy enum should use 'Web' prefix for values : [Attachment 89201] Proposed Patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Tue Apr 12 09:18:54 PDT 2011


Darin Fisher (:fishd, Google) <fishd at chromium.org> has granted Bill Budge
<bbudge at gmail.com>'s request for review:
Bug 58287: [chromium] WebCrossOriginRequestPolicy enum should use 'Web' prefix
for values
https://bugs.webkit.org/show_bug.cgi?id=58287

Attachment 89201: Proposed Patch
https://bugs.webkit.org/attachment.cgi?id=89201&action=review

------- Additional Comments from Darin Fisher (:fishd, Google)
<fishd at chromium.org>
R=me

Since this is likely to break Chromium, what is the plan for landing this
change?

A couple ideas:

1- You could add a #define in WebURLLoader.h, and then in Chromium code you can
land #ifdefs that switch on that #define to determine which definition of
WebURLLoaderOptions and CrossOriginRequestPolicy to use.  Then, land that
Chromium patch before landing this WebKit patch.

2- You could change this WebKit patch to make WebURLLoader.h #include
WebURLLoaderOptions.h, and you could also keep the old enum around, defined in
terms of the new one.


More information about the webkit-reviews mailing list