DescriptionRevert of IDL: Properly support [Exposed] on static methods (patchset #2 id:20001 of https://codereview.chromium.org/793243004/)
Reason for revert:
seems to be causing compile failures: http://build.chromium.org/p/chromium.webkit/builders/WebKit%20Mac%20Builder%20%28dbg%29/builds/101401/steps/compile/logs/stdio
Original issue's description:
> IDL: Properly support [Exposed] on static methods
>
> They were conditionally installed as indicated by the [Exposed] attribute,
> but they were installed on the prototype object rather than on the
> interface object.
>
> A static method with [Exposed] fulfills the requirements for being treated
> both as a conditionally enabled method (because of [Exposed]) and a method
> with custom registration (because of being static), and due to the order
> in which those requirements were checked, it ended up being treated as a
> conditionally enabled method, only.
>
> Fix by giving custom registration priority and by adding the same
> condition filters around that registration code, so that such methods can
> still also be all sorts of conditionally enabled.
>
> BUG=434741
>
> Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=187161
TBR=haraken@chromium.org,jl@opera.com
NOTREECHECKS=true
NOTRY=true
BUG=434741
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=187182
Patch Set 1 #
Created: 6 years ago
(Patch set is too large to download)
Messages
Total messages: 8 (0 generated)
|