Bug 116453 - PLATFORM(*) macros used in Source/WebCore/loader/archive/ArchiveFactory.cpp
Summary: PLATFORM(*) macros used in Source/WebCore/loader/archive/ArchiveFactory.cpp
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebCore Misc. (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Zan Dobersek
Depends on:
Blocks: 21354
  Show dependency treegraph
Reported: 2013-05-20 11:42 PDT by Zan Dobersek
Modified: 2013-05-21 09:02 PDT (History)
3 users (show)

See Also:

Patch (1.62 KB, patch)
2013-05-20 23:44 PDT, Zan Dobersek
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Zan Dobersek 2013-05-20 11:42:22 PDT
There's a couple of PLATFORM(*) macros used in ArchiveFactory.cpp, specifying which mime types are supported on what platform.

This should be moved into the platform layer.
Comment 1 Alexey Proskuryakov 2013-05-20 16:03:10 PDT
Can you please elaborate on the reason?

Note that platform directory contains the "platform" on which WebCore builds. While much of that is platform specific, keeping platforms specific code together in one place is not the reason for its existence.
Comment 2 Zan Dobersek 2013-05-20 23:32:54 PDT
I've filed this due to the layering violation reported by the check-webkit-style script in bug #116442.

The patch took on a different path later, removing most of the PLATFORM macros in r150418.

There's still one PLATFORM(QT) left in the file:
I think it can be removed because the Qt port does not enable WebArchive support even when using CoreFoundations.
Comment 3 Zan Dobersek 2013-05-20 23:44:54 PDT
Created attachment 202381 [details]
Comment 4 Alexey Proskuryakov 2013-05-21 08:29:25 PDT
Comment on attachment 202381 [details]

I think that the style bot is wrong complaining about this. Do you know when the check was added?

r=me on this change.
Comment 5 Zan Dobersek 2013-05-21 08:45:06 PDT
That would be r149635.
Comment 6 Zan Dobersek 2013-05-21 09:02:41 PDT
Comment on attachment 202381 [details]

Clearing flags on attachment: 202381

Committed r150444: <http://trac.webkit.org/changeset/150444>
Comment 7 Zan Dobersek 2013-05-21 09:02:52 PDT
All reviewed patches have been landed.  Closing bug.