[MD settings] handle being detached on cr-lazy-render and settings-idle-render

chrome/test/data/webui/cr_elements/cr_elements_browsertest.js

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 /** @fileoverview Tests for shared Polymer elements. */
 
 /** @const {string} Path to source root. */
 var ROOT_PATH = '../../../../../';
 
 // Polymer BrowserTest fixture.
@@ skipping 30 matching lines @@
 
 CrElementsLazyRenderTest.prototype = {
   __proto__: CrElementsBrowserTest.prototype,
 
   /** @override */
   browsePreload:
       'chrome://resources/cr_elements/cr_lazy_render/cr_lazy_render.html',
 
   /** @override */
   extraLibraries: CrElementsBrowserTest.prototype.extraLibraries.concat([
+    ROOT_PATH + 'ui/webui/resources/js/assert.js',

[Dan Beam, 2017/02/08 05:31:36]

please don't add dependencies for production code in tests only.

please, instead, add a <link rel="import"
href="chrome://resources/html/assert.html"> in settings_idle_render.html

backstory: dpapad@ and I had to remove a bunch of "only load this file in tests"
via importHtml(), loadScript(), and extraLibraries because folks were adding
dependencies only in tests when they should just be included in the code that
uses them.

[dschuyler, 2017/02/09 00:42:47]

Thanks. With the other note about detached settings-idle-render having .get()
called, I found that the asserts no longer applied.

     'cr_lazy_render_tests.js'
   ]),
 };
 
 TEST_F('CrElementsLazyRenderTest', 'All', function() {
   mocha.run();
 });
 
 function CrElementsProfileAvatarSelectorTest() {}
 
@@ skipping 64 matching lines @@
   browsePreload: 'chrome://resources/cr_elements/cr_drawer/cr_drawer.html',
 
   /** @override */
   extraLibraries: CrElementsBrowserTest.prototype.extraLibraries.concat(
       ['cr_drawer_tests.js', ROOT_PATH + 'ui/webui/resources/js/util.js']),
 };
 
 TEST_F('CrElementsDrawerTest', 'All', function() {
   mocha.run();
 });