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

Unified Diff: extensions/renderer/resources/utils.js

Issue 1915753002: Sanitize inheritance in callers of utils.expose (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: webview: Set proto on prototype. Created 4 years, 8 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: extensions/renderer/resources/utils.js
diff --git a/extensions/renderer/resources/utils.js b/extensions/renderer/resources/utils.js
index fd70c603ddffe295c6dc31d8efe495ca82c0760b..890e13cbfda1f7c3254fb84a845f8170af55ebcd 100644
--- a/extensions/renderer/resources/utils.js
+++ b/extensions/renderer/resources/utils.js
@@ -90,8 +90,7 @@ function loadTypeSchema(typeName, defaultSchema) {
* properties and |readonly| are read-only properties.
*/
function expose(publicClass, privateClass, exposed) {
- // TODO(robwu): Fix callers and uncomment this assertion.
- // DCHECK(!(privateClass instanceof $Object.self));
+ DCHECK(!(privateClass instanceof $Object.self));
$Object.setPrototypeOf(exposed, null);

Powered by Google App Engine
This is Rietveld 408576698