Make the HTMLDocumentParser yield more aggressively

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 75 matching lines @@
 }
 
 void HTMLParserScheduler::continueNextChunkTimerFired(Timer<HTMLParserScheduler>* timer)
 {
     ASSERT_UNUSED(timer, timer == &m_continueNextChunkTimer);
     m_parser->resumeParsingAfterYield();
 }
 
 void HTMLParserScheduler::scheduleForResume()
 {
+    // FIXME: Currently Timers use the SharedTimer queue, so scheduling
+    // a 0ms timer here may cause the parser to resume again before
+    // returning to the Chromium MessageLoop, thus starving other tasks
+    // in the MessageLoop. This will be fixed by moving Blink's Timer
+    // class ontop of postTask, but until then we're probably missing
+    // frames with this implementation.
     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);
 }
 
 }