Speedup Observatory elements first time rendering

runtime/observatory/lib/src/elements/helpers/rendering_queue.dart

 // Copyright (c) 2016, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
 import 'dart:html';
 import 'dart:collection';
 import 'dart:async';
 
 /// A generic rendering task that can be scheduled.
 abstract class RenderingTask {
@@ skipping 38 matching lines @@
   RenderingQueue() : this.fromBarrier(new NextAnimationFrameBarrier());
 
   /// Creates a RenderingQueue with a custom synchronization barrier.
   RenderingQueue.fromBarrier(this._barrier) {
     assert(this._barrier != null);
   }
 
   /// Add a task to the queue.
   /// If the current rendering phase is running it will be executed during this
   /// rendering cycle, otherwise it will be queued for the next one.
-  void enqueue(RenderingTask r) {
+  void enqueue(RenderingTask r, {bool realtime: false}) {

[siva, 2017/08/14 20:27:18]

instead of calling enqueue here with a parameter realtime why not just call
scheduleMicrotask(r.render) in the caller.

[cbernaschina, 2017/08/14 21:08:49]

See new Patch.

The execution on a browser which supports custom elements is as follow:
 - the root component schedules the first task and initializes the rendering
stage
 - all the children schedule an extra task that is executed just after the
parent one
   (due to the fact that the render methods are synchronous all the components
will render during a single Microtask)

The execution on a browser with partial support of the custom elements is as
follow:
 - the root component schedules the first task and initializes the rendering
stage
 - all the children schedule an extra task that is executed via Microtask,
possibly packing groups of them together.

Forwarding the decision to the enqueue method guarantees that the extra
Microtasks are created just as a fallback and not as part of the normal
behavior.

     assert(r != null);
     // If no task are in the queue there is no rendering phase scheduled.
-    if (isEmpty) _render();
+    if (isEmpty) {
+      if (realtime) {
+        scheduleMicrotask(r.render);
+        return;
+      } else {
+        _render();
+      }
+    }
     _queue.addLast(r);
   }
 
   Future _render() async {
     await _barrier.next;
     while (_queue.isNotEmpty) {
       _queue.first.render();
       _queue.removeFirst();
     }
   }
 }