Bug 163508 – Move user agent quirks to cross-platform location

• Status: RESOLVED
• Resolution: FIXED
• Component: WebCore Misc.

[Michael Catanzaro]

A downstream port wants to use GTK's list of user agent quirks. It would surely be beneficial for EFL as well, and actually probably for pretty much any port except macOS and iOS (which are big enough that hopefully UA quirks are not needed).

[Michael Catanzaro]

Created attachment 291787 [details]
Patch

[Michael Catanzaro]

Created attachment 291788 [details]
Patch

[Michael Catanzaro]

Created attachment 291789 [details]
Patch

[Carlos Garcia Campos]

Comment on attachment 291789 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=291789&action=review

Please don't do the "Standard" rename.

> Source/WebCore/ChangeLog:13
> +        * platform/StandardUserAgent.h: Renamed from Source/WebCore/platform/gtk/UserAgentGtk.h.
> +        Why not UserAgent.h? Because it clashes with a Cocoa-specific header, and would require
> +        editing XCode build files. Maybe a bad reason to rename a bunch of files, but there you go!

I know adding new files for mac is a pain, but I really think this should be called UserAgent.h. The cocoa file is in page/cocoa/ dir, and I don't think we have that in our include dir list, so it's probably safe to add it in platform. We can then file a sepàrate bug for the cocoa implementation to be merged in the new one, and we don't need to deal with Xcode in this bug.

> Source/WebCore/ChangeLog:29
> +        * platform/efl/StandardUserAgentEfl.cpp: Renamed from Source/WebCore/platform/efl/UserAgentEfl.cpp.
> +        (WebCore::standardUserAgentForURL):
> +        * platform/gtk/StandardUserAgentGtk.cpp: Renamed from Source/WebCore/platform/gtk/UserAgentGtk.cpp.

And then we don't need to rename these.

[Michael Catanzaro]

(In reply to comment #4)
> Please don't do the "Standard" rename.

Yeah, this was dumb.

> I know adding new files for mac is a pain, but I really think this should be
> called UserAgent.h. The cocoa file is in page/cocoa/ dir, and I don't think
> we have that in our include dir list, so it's probably safe to add it in
> platform. We can then file a sepàrate bug for the cocoa implementation to be
> merged in the new one, and we don't need to deal with Xcode in this bug.

But cocoa ports do include Source/WebCore/platform. And the header files do get tracked in the XCode project files in several places unfortunately.

I should just make the necessary changes (remove page/cocoa/UserAgentCocoa.h and move page/cocoa/UserAgentCocoa.cpp to platform/cocoa) without touching the XCode files, and then we can ask someone on IRC into fixing up the XCode project files.

[Michael Catanzaro]

Created attachment 291808 [details]
Patch

[Michael Catanzaro]

Comment on attachment 291808 [details]
Patch

I played a trick with #include_next.

[Michael Catanzaro]

Let's land this manually, EWS is choking on unrelated tests.

[Michael Catanzaro]

Committed r207406: <http://trac.webkit.org/changeset/207406>