[MD Extensions] make all views within view manager attach only when visible |
|||||
Issue descriptioncurrently all the views (within view-manager in manager.html) are in the dom even when hidden, which makes the page heavier than necessary. They should be behind a dom-if with restamp.
,
Oct 21 2017
CL at https://chromium-review.googlesource.com/c/chromium/src/+/731884
,
Oct 25 2017
The CL reduces the time to page render from 575 ms to 528 ms (on a very fast machine*); A savings of 47 ms or around 8%. *So the improvement will be much greater in total time for average users (and likely similar by percentage).
,
Oct 27 2017
,
Oct 31 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/fc9df5d5ac34fb5c4f706745b4d0dc7172d2ddd6 commit fc9df5d5ac34fb5c4f706745b4d0dc7172d2ddd6 Author: Dave Schuyler <dschuyler@chromium.org> Date: Tue Oct 31 03:03:08 2017 [MD extensions] cr-lazy-render on pages so they are attached on demand This CL wraps the MD extensions pages (details view, error page, and shortcuts page) in cr-lazy-render to accelerate the initial page load. The item list is not lazily rendered because it's the main page. Bug: 770392 Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation Change-Id: I5cc1d58f42c7bf56463d222a9a2dda7e0aa494ce Reviewed-on: https://chromium-review.googlesource.com/731884 Commit-Queue: Dave Schuyler <dschuyler@chromium.org> Reviewed-by: Demetrios Papadopoulos <dpapad@chromium.org> Cr-Commit-Position: refs/heads/master@{#512725} [modify] https://crrev.com/fc9df5d5ac34fb5c4f706745b4d0dc7172d2ddd6/chrome/browser/resources/md_extensions/manager.html [modify] https://crrev.com/fc9df5d5ac34fb5c4f706745b4d0dc7172d2ddd6/chrome/browser/resources/md_extensions/view_manager.js [modify] https://crrev.com/fc9df5d5ac34fb5c4f706745b4d0dc7172d2ddd6/chrome/test/data/webui/extensions/extension_manager_test.js
,
Oct 31 2017
|
|||||
►
Sign in to add a comment |
|||||
Comment 1 by dschuyler@chromium.org
, Oct 18 2017Status: Assigned (was: Available)