Move most of FrameLoader::CheckCompleted() to Document

third_party/WebKit/Source/core/svg/SVGSVGElement.cpp

Index: third_party/WebKit/Source/core/svg/SVGSVGElement.cpp
diff --git a/third_party/WebKit/Source/core/svg/SVGSVGElement.cpp b/third_party/WebKit/Source/core/svg/SVGSVGElement.cpp
index f0c7555c5909eb139bfd659708da82d5e8dbe4ec..a46c2b807bb39cdd98618e49b0222b1cf0250a49 100644
--- a/third_party/WebKit/Source/core/svg/SVGSVGElement.cpp
+++ b/third_party/WebKit/Source/core/svg/SVGSVGElement.cpp
@@ -504,8 +504,8 @@ Node::InsertionNotificationRequest SVGSVGElement::InsertedInto(
       // the load event, but if we miss that train (deferred programmatic
       // element insertion for example) we need to initialize the time container
       // here.
-      if (!GetDocument().Parsing() && !GetDocument().ProcessingLoadEvent() &&

[Nate Chapin, 2017/04/11 23:25:50]

If LoadEventFinished() is true, ProcessingLoadEvent() must be false, since they
are driven by the same enum. I think this logic predates the enum.

-          GetDocument().LoadEventFinished() && !TimeContainer()->IsStarted())
+      if (!GetDocument().Parsing() && GetDocument().LoadEventFinished() &&
+          !TimeContainer()->IsStarted())
         TimeContainer()->Start();
     }
   }