Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(441)

Unified Diff: ui/webui/resources/js/util.js

Issue 604373006: Compile chrome://settings, part 9: yet another final battle (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@K_blockers_from_bookmarks
Patch Set: dbeam@'s review Created 6 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: ui/webui/resources/js/util.js
diff --git a/ui/webui/resources/js/util.js b/ui/webui/resources/js/util.js
index 7c0be72ad5438f1ee893cfe4af54098233010088..dce38b499dff2b60ec7ff2319b8c10f11c9ffb76 100644
--- a/ui/webui/resources/js/util.js
+++ b/ui/webui/resources/js/util.js
@@ -254,12 +254,12 @@ function isRTL() {
* calling getElementById and not checking the result because this lets us
* satisfy the JSCompiler type system.
* @param {string} id The identifier name.
- * @return {!Element} the Element.
+ * @return {!HTMLElement} the Element.
*/
function getRequiredElement(id) {
var element = $(id);
Dan Beam 2014/10/02 02:56:35 ^ remove
Vitaly Pavlenko 2014/10/02 18:18:22 Done.
- assert(element, 'Missing required element: ' + id);
- return /** @type {!Element} */(element);
+ return assertInstanceof($(id), HTMLElement,
+ 'Missing required element: ' + id);
}
// Handle click on a link. If the link points to a chrome: or file: url, then
« ui/webui/resources/js/cr/ui/bubble.js ('K') | « ui/webui/resources/js/cr/ui/touch_handler.js ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698