There are a few remaining unimplemented layer hierarchy properties that were left that way because they're more complicated than the others. But, we need them!
Created attachment 214807 [details] patch
Comment on attachment 214807 [details] patch View in context: https://bugs.webkit.org/attachment.cgi?id=214807&action=review > Source/WebKit2/WebProcess/WebPage/mac/PlatformCALayerRemote.cpp:74 > + static_cast<PlatformCALayerRemote*>(layer.get())->m_superlayer = nullptr; Can we add a checked cast function for this (e.g. toPlatformCALayerRemote() that asserts the type is PlatformCALayerRemote)? > Source/WebKit2/WebProcess/WebPage/mac/PlatformCALayerRemote.h:157 > + PlatformCALayer* m_superlayer; Seems like m_superlayer is always a PlatformCALayerRemote. Can we make the type more specific here?
Created attachment 214872 [details] address sam's comments
Created attachment 214876 [details] we'll assert that they're PlatformCALayerRemotes in just one place
Comment on attachment 214876 [details] we'll assert that they're PlatformCALayerRemotes in just one place View in context: https://bugs.webkit.org/attachment.cgi?id=214876&action=review > Source/WebKit2/WebProcess/WebPage/mac/PlatformCALayerRemote.cpp:154 > + PlatformCALayerList& siblings = m_children; I don't think you need to use this local variable. Also, it is misnamed.
http://trac.webkit.org/changeset/157852