[Local NTP] Cleanup: Don't create HTML elements dynamically

chrome/test/data/local_ntp_browsertest.js

 // Copyright 2013 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 the local NTP.
  */
 
 
@@ skipping 48 matching lines @@
         window.console.log(testName + ' ' + err);
         pass = false;
       }
     }
   }
   return pass;
 }
 
 
 /**
+ * Checks whether a given HTMLElement exists and is visible.
+ * @param {HTMLElement|undefined} elem An HTMLElement.
+ * @return {boolean} True if the element exists and is visible.
+ */
+function elementIsVisible(elem) {
+  return elem && elem.offsetWidth > 0 && elem.offsetHeight > 0;
+}
+
+
+/**
  * Tests that Google NTPs show a fakebox and logo.
  */
 function testShowsFakeboxAndLogoIfGoogle() {
   var localNTP = LocalNTP();
   configData.isGooglePage = true;
   localNTP.init();
-  assert($('fakebox'));
-  assert($('logo'));
+  assert(elementIsVisible($('fakebox')));
+  assert(elementIsVisible($('logo')));
 }
 
 
 /**
  * Tests that non-Google NTPs do not show a fakebox.
  */
 function testDoesNotShowFakeboxIfNotGoogle() {
   var localNTP = LocalNTP();
   configData.isGooglePage = false;
   localNTP.init();
-  assert(!$('fakebox'));
-  assert(!$('logo'));
+  assert(!elementIsVisible($('fakebox')));
+  assert(!elementIsVisible($('logo')));

[Marc Treib, 2017/02/07 14:37:11]

Before, the elements didn't exist at all; now they do exist but have "display:
none". IMO that's perfectly fine.

 }