New issue
Advanced search Search tips

Issue 830022 link

Starred by 1 user

Issue metadata

Status: Assigned
Owner:
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug


Participants' hotlists:
Hotlist-Bindings-IDLCompiler


Sign in to add a comment

Bindings: InstallMethodInternal() should not allow constructor behavior on non-constructor methods

Project Member Reported by l...@chromium.org, Apr 6 2018

Issue description

V8DOMConfiguration.cpp [1] is responsible for installing methods, accessors, etc.  Creating new FunctionTemplates has v8::ConstructorBehavior::kAllow as the default ConstructorBehavior.  This likely should be stricter, and we should pass kThrow for any non-constructor method.

[1] https://cs.chromium.org/chromium/src/third_party/WebKit/Source/bindings/core/v8/V8DOMConfiguration.cpp?q=v8domconfiguration.cpp&sq=package:chromium&dr
 
Cc: peria@chromium.org
peria@, feel free to take this.  :)

Labels: Hotlist-Bindings-IDLCompiler
Owner: peria@chromium.org

Sign in to add a comment