Move Web Process part of the Remote Layer Tree code out of a Mac-specific directory
Created attachment 319699 [details] Patch
Attachment 319699 [details] did not pass style-queue: ERROR: Source/WebKit/WebProcess/WebPage/Cocoa/RemoteLayerTree/PlatformCALayerRemoteCustom.mm:114: No space between ^ and block definition. [whitespace/brackets] [4] ERROR: Source/WebKit/WebProcess/WebPage/Cocoa/RemoteLayerTree/PlatformCALayerRemoteTiledBacking.h:26: Use #pragma once instead of #ifndef for header guard. [build/header_guard] [5] ERROR: Source/WebKit/WebProcess/WebPage/Cocoa/RemoteLayerTree/PlatformCALayerRemote.h:198: The parameter name "context" adds no information, so it should be removed. [readability/parameter_name] [5] ERROR: Source/WebKit/WebProcess/WebPage/Cocoa/RemoteLayerTree/PlatformCAAnimationRemote.h:26: Use #pragma once instead of #ifndef for header guard. [build/header_guard] [5] ERROR: Source/WebKit/WebProcess/WebPage/Cocoa/RemoteLayerTree/RemoteLayerTreeContext.h:26: Use #pragma once instead of #ifndef for header guard. [build/header_guard] [5] ERROR: Source/WebKit/WebProcess/WebPage/Cocoa/RemoteLayerTree/RemoteScrollingCoordinator.h:26: Use #pragma once instead of #ifndef for header guard. [build/header_guard] [5] ERROR: Source/WebKit/WebProcess/WebPage/Cocoa/RemoteLayerTree/GraphicsLayerCARemote.cpp:27: You should add a blank line after implementation file's own header. [build/include_order] [4] ERROR: Source/WebKit/WebProcess/WebPage/Cocoa/RemoteLayerTree/PlatformCALayerRemote.cpp:280: 'layerProtector' is incorrectly named. It should be named 'protector' or 'protectedLayer'. [readability/naming/protected] [4] ERROR: Source/WebKit/WebProcess/WebPage/Cocoa/RemoteLayerTree/PlatformCALayerRemote.cpp:290: 'layerProtector' is incorrectly named. It should be named 'protector' or 'protectedLayer'. [readability/naming/protected] [4] ERROR: Source/WebKit/WebProcess/WebPage/Cocoa/RemoteLayerTree/PlatformCALayerRemote.cpp:301: 'layerProtector' is incorrectly named. It should be named 'protector' or 'protectedLayer'. [readability/naming/protected] [4] ERROR: Source/WebKit/WebProcess/WebPage/Cocoa/RemoteLayerTree/PlatformCAAnimationRemote.mm:603: Multi line control clauses should use braces. [whitespace/braces] [4] ERROR: Source/WebKit/WebProcess/WebPage/Cocoa/RemoteLayerTree/PlatformCAAnimationRemote.mm:617: Multi line control clauses should use braces. [whitespace/braces] [4] ERROR: Source/WebKit/WebProcess/WebPage/Cocoa/RemoteLayerTree/PlatformCAAnimationRemote.mm:631: Multi line control clauses should use braces. [whitespace/braces] [4] ERROR: Source/WebKit/WebProcess/WebPage/Cocoa/RemoteLayerTree/PlatformCAAnimationRemote.mm:647: Multi line control clauses should use braces. [whitespace/braces] [4] ERROR: Source/WebKit/WebProcess/WebPage/Cocoa/RemoteLayerTree/RemoteScrollingCoordinator.mm:39: Alphabetical sorting problem. [build/include_order] [4] ERROR: Source/WebKit/WebProcess/WebPage/Cocoa/RemoteLayerTree/RemoteScrollingCoordinator.mm:40: Alphabetical sorting problem. [build/include_order] [4] ERROR: Source/WebKit/WebProcess/WebPage/Cocoa/RemoteLayerTree/RemoteLayerTreeDrawingArea.mm:349: 'retainedPage' is incorrectly named. It should be named 'protector' or 'protectedWebPage'. [readability/naming/protected] [4] Total errors found: 17 in 21 files If any of these errors are false positives, please file a bug against check-webkit-style.
Comment on attachment 319699 [details] Patch rs=me
Anyone have thoughts about moving to WebProcess/WebPage/RemoteLayerTree instead? That would make it easier to parallel the UI process patch, which has platform-specific things in it (so then you can have UIProcess/RemoteLayerTree/ios instead of UIProcess/ios/RemoteLayerTree in addition to UIProcess/RemoteLayerTree -- locality is better in the former).
Weird, builds for me, but it ought not.
Created attachment 319706 [details] Patch
Attachment 319706 [details] did not pass style-queue: ERROR: Source/WebKit/WebProcess/WebPage/RemoteLayerTree/GraphicsLayerCARemote.cpp:27: You should add a blank line after implementation file's own header. [build/include_order] [4] ERROR: Source/WebKit/WebProcess/WebPage/RemoteLayerTree/PlatformCAAnimationRemote.mm:603: Multi line control clauses should use braces. [whitespace/braces] [4] ERROR: Source/WebKit/WebProcess/WebPage/RemoteLayerTree/PlatformCAAnimationRemote.mm:617: Multi line control clauses should use braces. [whitespace/braces] [4] ERROR: Source/WebKit/WebProcess/WebPage/RemoteLayerTree/PlatformCAAnimationRemote.mm:631: Multi line control clauses should use braces. [whitespace/braces] [4] ERROR: Source/WebKit/WebProcess/WebPage/RemoteLayerTree/PlatformCAAnimationRemote.mm:647: Multi line control clauses should use braces. [whitespace/braces] [4] ERROR: Source/WebKit/WebProcess/WebPage/RemoteLayerTree/PlatformCALayerRemote.h:198: The parameter name "context" adds no information, so it should be removed. [readability/parameter_name] [5] ERROR: Source/WebKit/WebProcess/WebPage/RemoteLayerTree/PlatformCAAnimationRemote.h:26: Use #pragma once instead of #ifndef for header guard. [build/header_guard] [5] ERROR: Source/WebKit/WebProcess/WebPage/RemoteLayerTree/RemoteLayerTreeContext.h:26: Use #pragma once instead of #ifndef for header guard. [build/header_guard] [5] ERROR: Source/WebKit/WebProcess/WebPage/RemoteLayerTree/PlatformCALayerRemoteCustom.mm:114: No space between ^ and block definition. [whitespace/brackets] [4] ERROR: Source/WebKit/WebProcess/WebPage/RemoteLayerTree/PlatformCALayerRemote.cpp:280: 'layerProtector' is incorrectly named. It should be named 'protector' or 'protectedLayer'. [readability/naming/protected] [4] ERROR: Source/WebKit/WebProcess/WebPage/RemoteLayerTree/PlatformCALayerRemote.cpp:290: 'layerProtector' is incorrectly named. It should be named 'protector' or 'protectedLayer'. [readability/naming/protected] [4] ERROR: Source/WebKit/WebProcess/WebPage/RemoteLayerTree/PlatformCALayerRemote.cpp:301: 'layerProtector' is incorrectly named. It should be named 'protector' or 'protectedLayer'. [readability/naming/protected] [4] ERROR: Source/WebKit/WebProcess/WebPage/RemoteLayerTree/RemoteLayerTreeDrawingArea.mm:349: 'retainedPage' is incorrectly named. It should be named 'protector' or 'protectedWebPage'. [readability/naming/protected] [4] ERROR: Source/WebKit/WebProcess/WebPage/RemoteLayerTree/RemoteScrollingCoordinator.mm:39: Alphabetical sorting problem. [build/include_order] [4] ERROR: Source/WebKit/WebProcess/WebPage/RemoteLayerTree/RemoteScrollingCoordinator.mm:40: Alphabetical sorting problem. [build/include_order] [4] ERROR: Source/WebKit/WebProcess/WebPage/RemoteLayerTree/PlatformCALayerRemoteTiledBacking.h:26: Use #pragma once instead of #ifndef for header guard. [build/header_guard] [5] ERROR: Source/WebKit/WebProcess/WebPage/RemoteLayerTree/RemoteScrollingCoordinator.h:26: Use #pragma once instead of #ifndef for header guard. [build/header_guard] [5] Total errors found: 17 in 23 files If any of these errors are false positives, please file a bug against check-webkit-style.
Created attachment 319710 [details] Patch
Attachment 319710 [details] did not pass style-queue: ERROR: Source/WebKit/WebProcess/WebPage/RemoteLayerTree/GraphicsLayerCARemote.cpp:27: You should add a blank line after implementation file's own header. [build/include_order] [4] ERROR: Source/WebKit/WebProcess/WebPage/RemoteLayerTree/PlatformCAAnimationRemote.mm:603: Multi line control clauses should use braces. [whitespace/braces] [4] ERROR: Source/WebKit/WebProcess/WebPage/RemoteLayerTree/PlatformCAAnimationRemote.mm:617: Multi line control clauses should use braces. [whitespace/braces] [4] ERROR: Source/WebKit/WebProcess/WebPage/RemoteLayerTree/PlatformCAAnimationRemote.mm:631: Multi line control clauses should use braces. [whitespace/braces] [4] ERROR: Source/WebKit/WebProcess/WebPage/RemoteLayerTree/PlatformCAAnimationRemote.mm:647: Multi line control clauses should use braces. [whitespace/braces] [4] ERROR: Source/WebKit/WebProcess/WebPage/RemoteLayerTree/PlatformCALayerRemote.h:198: The parameter name "context" adds no information, so it should be removed. [readability/parameter_name] [5] ERROR: Source/WebKit/WebProcess/WebPage/RemoteLayerTree/PlatformCAAnimationRemote.h:26: Use #pragma once instead of #ifndef for header guard. [build/header_guard] [5] ERROR: Source/WebKit/WebProcess/WebPage/RemoteLayerTree/RemoteLayerTreeContext.h:26: Use #pragma once instead of #ifndef for header guard. [build/header_guard] [5] ERROR: Source/WebKit/WebProcess/WebPage/RemoteLayerTree/PlatformCALayerRemoteCustom.mm:114: No space between ^ and block definition. [whitespace/brackets] [4] ERROR: Source/WebKit/WebProcess/WebPage/RemoteLayerTree/PlatformCALayerRemote.cpp:280: 'layerProtector' is incorrectly named. It should be named 'protector' or 'protectedLayer'. [readability/naming/protected] [4] ERROR: Source/WebKit/WebProcess/WebPage/RemoteLayerTree/PlatformCALayerRemote.cpp:290: 'layerProtector' is incorrectly named. It should be named 'protector' or 'protectedLayer'. [readability/naming/protected] [4] ERROR: Source/WebKit/WebProcess/WebPage/RemoteLayerTree/PlatformCALayerRemote.cpp:301: 'layerProtector' is incorrectly named. It should be named 'protector' or 'protectedLayer'. [readability/naming/protected] [4] ERROR: Source/WebKit/WebProcess/WebPage/RemoteLayerTree/RemoteLayerTreeDrawingArea.mm:349: 'retainedPage' is incorrectly named. It should be named 'protector' or 'protectedWebPage'. [readability/naming/protected] [4] ERROR: Source/WebKit/WebProcess/WebPage/RemoteLayerTree/RemoteScrollingCoordinator.mm:39: Alphabetical sorting problem. [build/include_order] [4] ERROR: Source/WebKit/WebProcess/WebPage/RemoteLayerTree/RemoteScrollingCoordinator.mm:40: Alphabetical sorting problem. [build/include_order] [4] ERROR: Source/WebKit/WebProcess/WebPage/RemoteLayerTree/PlatformCALayerRemoteTiledBacking.h:26: Use #pragma once instead of #ifndef for header guard. [build/header_guard] [5] ERROR: Source/WebKit/WebProcess/WebPage/RemoteLayerTree/RemoteScrollingCoordinator.h:26: Use #pragma once instead of #ifndef for header guard. [build/header_guard] [5] Total errors found: 17 in 23 files If any of these errors are false positives, please file a bug against check-webkit-style.
Comment on attachment 319710 [details] Patch Clearing flags on attachment: 319710 Committed r221536: <http://trac.webkit.org/changeset/221536>
All reviewed patches have been landed. Closing bug.
Comment on attachment 319710 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=319710&action=review > Source/WebKit/ChangeLog:15 > + * WebProcess/WebPage/RemoteLayerTree/GraphicsLayerCARemote.cpp: Renamed from WebProcess/WebPage/mac/GraphicsLayerCARemote.cpp. The WebPage directory doesn't really make any sense. Would it be better to parallel WebCore platform/ and do something like WebProcess/graphics/, WebProcess/scrolling/ ?
(In reply to Simon Fraser (smfr) from comment #12) > Comment on attachment 319710 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=319710&action=review > > > Source/WebKit/ChangeLog:15 > > + * WebProcess/WebPage/RemoteLayerTree/GraphicsLayerCARemote.cpp: Renamed from WebProcess/WebPage/mac/GraphicsLayerCARemote.cpp. > > The WebPage directory doesn't really make any sense. Would it be better to > parallel WebCore platform/ and do something like WebProcess/graphics/, > WebProcess/scrolling/ ? The UI process and Shared directories have a lot of stuff directly in them, so maybe that is the place to go (just move up one level)... I'm not sure why historically stuff has gotten stuck inside WebProcess/WebPage/... in this patch I left it there because e.g. DrawingArea itself is in WebProcess/WebPage, and it didn't seem to make sense to move a DrawingArea subclass to a directory above the base class. We could follow WebCore... would we then fold e.g. the CoordinatedGraphics directory into WebProcess/graphics? (Mostly the motivation here was to get it out of mac/, but we can do more cleanup!)
<rdar://problem/34693744>