Remove hack in HTMLParserScheduler for deferring to layout while parsing.

Source/core/html/parser/HTMLParserScheduler.cpp

 /*
  * Copyright (C) 2010 Google, Inc. All Rights Reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
  *    notice, this list of conditions and the following disclaimer in the
@@ skipping 70 matching lines @@
 }
 
 HTMLParserScheduler::~HTMLParserScheduler()
 {
     m_continueNextChunkTimer.stop();
 }
 
 void HTMLParserScheduler::continueNextChunkTimerFired(Timer<HTMLParserScheduler>* timer)
 {
     ASSERT_UNUSED(timer, timer == &m_continueNextChunkTimer);
-    // FIXME: The timer class should handle timer priorities instead of this code.
-    // If a layout is scheduled, wait again to let the layout timer run first.
-    // FIXME: We should fix this by reducing the max-parse-time instead of
-    // artificially forcing the parser to yield agressively before first layout.
-    if (m_parser->document()->shouldParserYieldAgressivelyBeforeScriptExecution()) {

[abarth-chromium, 2014/03/29 20:20:35]

Can we delete shouldParserYieldAgressivelyBeforeScriptExecution?

-        m_continueNextChunkTimer.startOneShot(0, FROM_HERE);
-        return;
-    }
     m_parser->resumeParsingAfterYield();
 }
 
 void HTMLParserScheduler::checkForYieldBeforeScript(PumpSession& session)
 {
     // If we've never painted before and a layout is pending, yield prior to running
     // scripts to give the page a chance to paint earlier.
     Document* document = m_parser->document();
     bool needsFirstPaint = document->view() && !document->view()->hasEverPainted();
     if (needsFirstPaint && document->shouldParserYieldAgressivelyBeforeScriptExecution())
         session.needsYield = true;
     session.didSeeScript = true;
 }
 
 void HTMLParserScheduler::scheduleForResume()
 {
     m_continueNextChunkTimer.startOneShot(0, FROM_HERE);
 }
 
-
 void HTMLParserScheduler::suspend()
 {
     ASSERT(!m_isSuspendedWithActiveTimer);
     if (!m_continueNextChunkTimer.isActive())
         return;
     m_isSuspendedWithActiveTimer = true;
     m_continueNextChunkTimer.stop();
 }
 
 void HTMLParserScheduler::resume()
 {
     ASSERT(!m_continueNextChunkTimer.isActive());
     if (!m_isSuspendedWithActiveTimer)
         return;
     m_isSuspendedWithActiveTimer = false;
     m_continueNextChunkTimer.startOneShot(0, FROM_HERE);
 }
 
 }