Remove redundant call to UpdateGeometriesIfNeeded

This already gets called from PerformPostLayoutTasks. That's a better
place for it, because it means the call happens before we recurse
into child frames and run layout in them, since the geometry update
call invalidate layout in the child frames.

Change-Id: I145f054b7516e95545d68402fae38519c6706557
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1800922Reviewed-by: Philip Rogers <pdr@chromium.org>
Commit-Queue: Stefan Zager <szager@chromium.org>
Cr-Commit-Position: refs/heads/master@{#697828}

third_party/blink/renderer/core/frame/local_frame_view.cc

@@ -2902,8 +2902,6 @@ void LocalFrameView::UpdateStyleAndLayoutIfNeededRecursive() {
   frame_->GetDocument()->GetLayoutView()->AssertLaidOut();
 #endif
 
-  UpdateGeometriesIfNeeded();
-
   if (Lifecycle().GetState() < DocumentLifecycle::kLayoutClean)
     Lifecycle().AdvanceTo(DocumentLifecycle::kLayoutClean);