Rename paint related lifecycle methods of FrameView

third_party/WebKit/Source/core/frame/FrameView.h

 /*
    Copyright (C) 1997 Martin Jones (mjones@kde.org)
              (C) 1998 Waldo Bastian (bastian@kde.org)
              (C) 1998, 1999 Torben Weis (weis@kde.org)
              (C) 1999 Lars Knoll (knoll@kde.org)
              (C) 1999 Antti Koivisto (koivisto@kde.org)
    Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009 Apple Inc. All rights
    reserved.
 
    This library is free software; you can redistribute it and/or
@@ skipping 852 matching lines @@
   void updateScrollOffset(const ScrollOffset&, ScrollType) override;
 
   void updateScrollbarEnabledState();
 
   void updateLifecyclePhasesInternal(
       DocumentLifecycle::LifecycleState targetState);
 
   void invalidateTreeIfNeededRecursive();
   void scrollContentsIfNeededRecursive();
   void updateStyleAndLayoutIfNeededRecursive();
-  void updatePaintProperties();
-  void synchronizedPaint();
-  void synchronizedPaintRecursively(GraphicsLayer*);
+  void prePaint();
+  void paint();

[chrishtr, 2017/01/05 17:51:10]

With this CL there are now 3 methods called paint(). Could you you rename
the other two to be more specific? Or rename this to paintAllContent or
similar?

[Xianzhu, 2017/01/05 18:05:09]

One of the other two is Widget override.

paintAllContent seems ambiguous because the word "contents" in the existing
paintContents() means things other than scrollbars.

How about paintTree()?

+  void paintGraphicsLayerRecursively(GraphicsLayer*);
 
   void updateStyleAndLayoutIfNeededRecursiveInternal();
   void invalidateTreeIfNeededRecursiveInternal();
 
   void pushPaintArtifactToCompositor();
 
   void reset();
   void init();
 
   void clearLayoutSubtreeRootsAndMarkContainingBlocks();
@@ skipping 322 matching lines @@
                   widget.isFrameView());
 DEFINE_TYPE_CASTS(FrameView,
                   ScrollableArea,
                   scrollableArea,
                   scrollableArea->isFrameView(),
                   scrollableArea.isFrameView());
 
 }  // namespace blink
 
 #endif  // FrameView_h