commit ed22566dfe53b11a2226c1f5238856f250ec72a7
author Wyatt Allen <wyatta@google.com> Thu Apr 05 10:48:03 2018 -0700
committer Wyatt Allen <wyatta@google.com> Thu Apr 05 18:11:35 2018 -0700
tree 6a875f85f78bbf0ffa54dfcd8724159ba4cb76c6
parent 25bccb7e487caf645d21baec6caff69528143492

Guard comment textarea and overlays behind dom-if templates

Until the thread list was added, we rarely rendered a large number of
diff comments at once. A diff would add them incrementally in an
asynchronous loop, and even then, their number was reduced by being
filtered to to the given path and patch range.

Rendering all comments at once in the thread list exposed inefficiencies
in instantiating gr-diff-comments, and, for changes with many comments,
could temporarily lockup the browser.

The biggest cause of slowness in comment instantiation was the textarea
for editing. Because existing comments are not in an edit state by
default (...and because only the final comment in a thread can be edited
anyway ... assuming the user even chooses to edit ... assuming there
even is an authenticated user...) it's more efficient to put the
textarea in a dom-if until it's needed.

The overlays are also expensive but infrequently used. These are put
inside a dom-if as well.

Change-Id: I50702afe2b178221110ad3200c5e3862a7722c5d