Reviewed by darin.
authorggaren <ggaren@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 7 Feb 2006 21:32:55 +0000 (21:32 +0000)
committerggaren <ggaren@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 7 Feb 2006 21:32:55 +0000 (21:32 +0000)
commit00d4e456bb4a8de71f33b28204d0f9dfc901f79b
treeda22088041fc4650223fe56f6cf2c6d8746a9743
parent7817c566135d1d1d8d68199f2f625f8523327a16
    Reviewed by darin.

        - Fixed <rdar://problem/4425269> REGRESSION: wrong size pop-up when
        downloading attachment at webmail.mac.com (6882)

        The bug was that the feature string parser didn't always initialize the
        *Set variables, so WebCore assumed a height was set when it wasn't,
        and used a garbage height variable.

        While there, I fixed two other bugs: (1) The parser wasn't chewing up
        as many characters as I thought it was. (Chewing up extra characters
        is necessary to match Win IE.) (2) We considered \t and \f to be
        whitespace, but Win IE doesn't.

        * manual-tests/window-open-features-parsing.html: Added these cases,
        fixed up comments.

        * bridge/BrowserExtension.h: Cleaned up declaration order a bit.
        * khtml/ecma/kjs_window.cpp:
        (KJS::isSeparator): New function, tells you if a character is a
        separator
        (KJS::parseWindowFeatures): Always initialize *Set to false. Drive
        parsing based on invalid characters ('separators') rather than valid
        ones, to match Win IE.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@12647 268f45cc-cd09-0410-ab3c-d52691b4dbfc
WebCore/ChangeLog
WebCore/bridge/BrowserExtension.h
WebCore/khtml/ecma/kjs_window.cpp
WebCore/manual-tests/window-open-features-parsing.html