[webkit-reviews] review granted: [Bug 73729] [chromium] Remove forwarding headers for WebLayer APIs : [Attachment 117729] Patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Fri Dec 2 23:32:55 PST 2011


Adam Barth <abarth at webkit.org> has granted James Robinson
<jamesr at chromium.org>'s request for review:
Bug 73729: [chromium] Remove forwarding headers for WebLayer APIs
https://bugs.webkit.org/show_bug.cgi?id=73729

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

------- Additional Comments from Adam Barth <abarth at webkit.org>
View in context: https://bugs.webkit.org/attachment.cgi?id=117729&action=review


> Source/WebKit/chromium/src/WebLayerTreeView.cpp:32
> +#include "platform/WebLayer.h"

The main question is whether we want to add platform to the include path.  I
think having the directory name here is helpful, but that's not precisely
webkit style.


More information about the webkit-reviews mailing list