Inline PrePaintTreeWalk private methods and static functions

third_party/WebKit/Source/core/paint/PaintPropertyTreeBuilder.h

Index: third_party/WebKit/Source/core/paint/PaintPropertyTreeBuilder.h
diff --git a/third_party/WebKit/Source/core/paint/PaintPropertyTreeBuilder.h b/third_party/WebKit/Source/core/paint/PaintPropertyTreeBuilder.h
index a1e7a9f699102c8bfbd4057a8f81d50c1e611ab5..12feaa03fb6b13455ff0c03632ddb6d05e944d3c 100644
--- a/third_party/WebKit/Source/core/paint/PaintPropertyTreeBuilder.h
+++ b/third_party/WebKit/Source/core/paint/PaintPropertyTreeBuilder.h
@@ -115,32 +115,36 @@ class PaintPropertyTreeBuilder {
                                    PaintPropertyTreeBuilderContext&);
 
  private:
-  static void updatePaintOffsetTranslation(const LayoutObject&,
-                                           PaintPropertyTreeBuilderContext&);
-  static void updateTransform(const LayoutObject&,
-                              PaintPropertyTreeBuilderContext&);
-  static void updateTransformForNonRootSVG(const LayoutObject&,
-                                           PaintPropertyTreeBuilderContext&);
-  static void updateEffect(const LayoutObject&,
-                           PaintPropertyTreeBuilderContext&);
-  static void updateCssClip(const LayoutObject&,
-                            PaintPropertyTreeBuilderContext&);
-  static void updateLocalBorderBoxContext(const LayoutObject&,
-                                          PaintPropertyTreeBuilderContext&);
-  static void updateScrollbarPaintOffset(const LayoutObject&,
-                                         PaintPropertyTreeBuilderContext&);
-  static void updateOverflowClip(const LayoutObject&,
-                                 PaintPropertyTreeBuilderContext&);
-  static void updatePerspective(const LayoutObject&,
-                                PaintPropertyTreeBuilderContext&);
-  static void updateSvgLocalToBorderBoxTransform(
+  inline static void updatePaintOffsetTranslation(

[pdr., 2017/01/18 18:51:58]

After seeing your other inline patch, I was experimenting with this exact change
too.

It looks like "static inline" might have no effect in the common case:
http://stackoverflow.com/questions/7762731/whats-the-difference-between-stati...

Do you know if this has any effect on the generated code? Is there an easy way
to test that this change works?

[Xianzhu, 2017/01/18 19:17:33]

I looked into the generated code in a release build. The added 'inline's did
inline these functions and reduces many instructions [1] for each function call.
I think the compiler doesn't inline these static methods by default because they
may be referenced from a friend class outside of the compilation unit.

Each of such functions contributed about 1% of total PrePaint time for
paint-offset-changes.html. I think most of the cost was the overhead of the
function calls as many functions just check condition and return for the test.

I think the stackoverflow article is about local static functions. I think it's
correct if a local static function is small and is called from few places in the
file. We can omit the inlines for the static local functions.

[1] I will add more details soon.

       const LayoutObject&,
       PaintPropertyTreeBuilderContext&);
-  static void updateScrollAndScrollTranslation(
+  inline static void updateTransform(const LayoutObject&,
+                                     PaintPropertyTreeBuilderContext&);
+  inline static void updateTransformForNonRootSVG(
       const LayoutObject&,
       PaintPropertyTreeBuilderContext&);
-  static void updateOutOfFlowContext(const LayoutObject&,
-                                     PaintPropertyTreeBuilderContext&);
+  inline static void updateEffect(const LayoutObject&,
+                                  PaintPropertyTreeBuilderContext&);
+  inline static void updateCssClip(const LayoutObject&,
+                                   PaintPropertyTreeBuilderContext&);
+  inline static void updateLocalBorderBoxContext(
+      const LayoutObject&,
+      PaintPropertyTreeBuilderContext&);
+  inline static void updateScrollbarPaintOffset(
+      const LayoutObject&,
+      PaintPropertyTreeBuilderContext&);
+  inline static void updateOverflowClip(const LayoutObject&,
+                                        PaintPropertyTreeBuilderContext&);
+  inline static void updatePerspective(const LayoutObject&,
+                                       PaintPropertyTreeBuilderContext&);
+  inline static void updateSvgLocalToBorderBoxTransform(
+      const LayoutObject&,
+      PaintPropertyTreeBuilderContext&);
+  inline static void updateScrollAndScrollTranslation(
+      const LayoutObject&,
+      PaintPropertyTreeBuilderContext&);
+  inline static void updateOutOfFlowContext(const LayoutObject&,
+                                            PaintPropertyTreeBuilderContext&);
 };
 
 }  // namespace blink