Fixes style and adds some comments to layers code

Also removes FrameCachePermission::AllowNewest. This should have been removed
as part of https://android-git.corp.google.com/g/#/c/2712.

Bug: 5262656
Change-Id: I4d819cc5087653e4a1180d04ca0b5b20db7e2305

1 files changed, 8 insertions, 3 deletions

diff --git a/Source/WebCore/platform/graphics/android/LayerAndroid.h b/Source/WebCore/platform/graphics/android/LayerAndroid.h
index 7192aaf..a223499 100644
--- a/Source/WebCore/platform/graphics/android/LayerAndroid.h
+++ b/Source/WebCore/platform/graphics/android/LayerAndroid.h
@@ -109,6 +109,10 @@ public:
     void setBackfaceVisibility(bool value) { m_backfaceVisibility = value; }
     void setTransform(const TransformationMatrix& matrix) { m_transform = matrix; }
     FloatPoint translation() const;
+    // Returns a rect describing the bounds of the layer with the local
+    // transformation applied, expressed relative to the parent layer.
+    // FIXME: Currently we use only the translation component of the local
+    // transformation.
     SkRect bounds() const;
     IntRect clippedRect() const;
     bool outsideViewport();
@@ -183,9 +187,10 @@ public:
 
     SkPicture* picture() const { return m_recordingPicture; }
 
-    // remove layers bounds from visible rectangle to show what can be
-    // scrolled into view; returns original minus layer bounds in global space.
-    SkRect subtractLayers(const SkRect& visibleRect) const;
+    // Given a rect in global space, subtracts from it the bounds of this layer
+    // and of all of its children. Returns the bounding rectangle of the result,
+    // in global space.
+    SkRect subtractLayers(const SkRect&) const;
 
     void dumpLayers(FILE*, int indentLevel) const;
     void dumpToLog() const;