2009-01-27 Darin Adler <darin@apple.com>
authordarin@apple.com <darin@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 28 Jan 2009 01:54:50 +0000 (01:54 +0000)
committerdarin@apple.com <darin@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 28 Jan 2009 01:54:50 +0000 (01:54 +0000)
commit182507b36d297f31b9986c9fbecc526ff083ffd4
tree15184991996b376b1345a48ec4085ea7757c7350
parentab45b2bb2067aa12b4956ad5fa285e0636d7f938
2009-01-27  Darin Adler  <darin@apple.com>

        Reviewed by Sam Weinig.

        REGRESSION (r40234): Launching iTunes from a link displays a
        "Safari can't find the Internet plug-in" sheet in Safari
        rdar://problem/6533678

        * rendering/RenderPartObject.cpp:
        (WebCore::havePlugin): Put the check for empty string in here.
        (WebCore::serviceTypeForClassId): Only return activeXType() if
        we have a plug-in that can handle that type. While it might be logical
        to return an Active X type for any <object> with a classId in it,
        even if we don't have a plug-in that implements Active X, existing
        clients, such as Safari, don't expect that.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@40305 268f45cc-cd09-0410-ab3c-d52691b4dbfc
WebCore/ChangeLog
WebCore/rendering/RenderPartObject.cpp