Remove code duplication in updateLifecycleToCompositingCleanPlusScrolling.

Source/core/frame/FrameView.h

Index: Source/core/frame/FrameView.h
diff --git a/Source/core/frame/FrameView.h b/Source/core/frame/FrameView.h
index 060098b300bc165c2aada8aa512ebcf8af40dbca..a4d600095db3386793c1fdc187cb71a4dce52fab 100644
--- a/Source/core/frame/FrameView.h
+++ b/Source/core/frame/FrameView.h
@@ -610,7 +610,12 @@ private:
     void setScrollOffset(const IntPoint&, ScrollType) override;
     void setScrollOffset(const DoublePoint&, ScrollType) override;
 
-    void updateAllLifecyclePhasesInternal();
+    enum LifeCycleUpdateOption {
+        AllPhases,
+        OnlyUpToCompositingClean,

[chrishtr, 2015/08/11 16:56:12]

plus scrolling

[ojan, 2015/08/11 17:18:53]

It's weird to me to have a pipeline and list multiple phases in the pipeline.
That said, scrolling isn't an official lifecycle step at the moment. We might
want to make it a lifecycle step. Then this would only be OnlyUptoScrollingClean
as that would imply compositing is clean.

Changed to OnlyUpToCompositingCleanPlusScrolling.

+    };
+
+    void updateLifecyclePhasesInternal(LifeCycleUpdateOption);
     void invalidateTreeIfNeededRecursive();
     void scrollContentsIfNeededRecursive();
     void updateStyleAndLayoutIfNeededRecursive();