[webkit-reviews] review denied: [Bug 56575] REGRESSION: "Hidden" NPAPI plugin not loaded : [Attachment 88992] Tabs removed
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Wed May 4 01:18:33 PDT 2011
Eric Seidel <eric at webkit.org> has denied Pere Martir <pere.martir4 at gmail.com>'s
request for review:
Bug 56575: REGRESSION: "Hidden" NPAPI plugin not loaded
https://bugs.webkit.org/show_bug.cgi?id=56575
Attachment 88992: Tabs removed
https://bugs.webkit.org/attachment.cgi?id=88992&action=review
------- Additional Comments from Eric Seidel <eric at webkit.org>
View in context: https://bugs.webkit.org/attachment.cgi?id=88992&action=review
> Source/WebCore/html/HTMLObjectElement.cpp:139
> + // FIXME: Not dynamic, since we add this but don't remove it,
but it may be OK for now
> + // that this rarely-used attribute won't work properly if you
remove it.
> + addCSSLength(attr, CSSPropertyWidth, "0");
> + addCSSLength(attr, CSSPropertyHeight, "0");
I'm really not a big fan of this hack.
It's not clear to me why this is needed. When did behavior change? Do we know
why it changed? It should be relatively trivial to run
Tools/Scripts/bisect-builds or git bisect and find the revision at which the
bahavior changed in WebKIt. Then we would better understand what change may be
necessary to restore old behavior (or if the behavior change was intentional).
More information about the webkit-reviews
mailing list