[Webkit-unassigned] [Bug 21807] Add the ability for embedders to use Google-URL without changing shared headers

[bugzilla-daemon at webkit.org]

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


brettw at chromium.org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         AssignedTo|webkit-                     |brettw at chromium.org
                   |unassigned at lists.webkit.org |
             Status|NEW                         |ASSIGNED
  Attachment #24564|                            |review?
               Flag|                            |




------- Comment #1 from brettw at chromium.org  2008-10-22 12:54 PDT -------
Created an attachment (id=24564)
 --> (https://bugs.webkit.org/attachment.cgi?id=24564&action=view)
Patch

Changes to KURL.h to allow it to bring in Google-URL. No actual code using
Google-URL is present in this patch, it merely changes the shared files.

The goal of this patch is not to be a long-term solution, but something that
lets Chromium unfork in the short term, and allows the WebKit community to
evaluate the library. This will allow the community to make a better
determination of the future direction.

Technically, I tried to impack KURL.h as little as possible. I conditionally
include a header, and use the object defined in that header to replace most of
the private stuff in KURL. This allows us not to pollute the file with a lot of
experimental code.

I moved a handful of functions to the .ccp file since their implementation
would be different and this avoids having to add more ifdefs to the header
file. I kept a few functions in the header that should make a difference being
inlined like the string getter, and used ifdefs for them. I'm happy to use more
or less inlining if you prefer (with corresponding change in ugliness in the
header).


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