1. 23 Jul, 2009 40 commits
    • mrowe@apple.com's avatar
      Fix layout tests that incorrectly attempt to detect whether they're running... · b5cf863a
      mrowe@apple.com authored
      Fix layout tests that incorrectly attempt to detect whether they're running inside DumpRenderTree using "if (layoutTestController)".
      
      Rubber-stamped by Brady Eidson.
      
      * editing/execCommand/25256.html:
      * editing/execCommand/25320.html:
      * fast/canvas/arc-crash.html:
      * fast/css/emptyStyleTag.html:
      * fast/events/js-keyboard-event-creation.html:
      * fast/events/scrollbar-double-click.html:
      * fast/forms/select-no-name.html:
      * fast/history/window-open.html:
      * platform/mac/fast/events/objc-event-api.html:
      * platform/mac/fast/events/objc-keyboard-event-creation.html:
      * scrollbars/scrollbar-miss-mousemove-disabled.html:
      * scrollbars/scrollbar-miss-mousemove.html:
      * storage/domstorage/localstorage/clear.html:
      * storage/domstorage/localstorage/delete-removal.html:
      * storage/domstorage/localstorage/enumerate-storage.html:
      * storage/domstorage/localstorage/index-get-and-set.html:
      * storage/domstorage/localstorage/onstorage-attribute-markup.html:
      * storage/domstorage/localstorage/onstorage-attribute-setattribute.html:
      * storage/domstorage/localstorage/private-browsing-affects-storage.html:
      * storage/domstorage/localstorage/resources/window-open-second.html:
      * storage/domstorage/localstorage/simple-events.html:
      * storage/domstorage/localstorage/simple-usage.html:
      * storage/domstorage/localstorage/string-conversion.html:
      * storage/domstorage/localstorage/window-open.html:
      * storage/domstorage/sessionstorage/clear.html:
      * storage/domstorage/sessionstorage/delete-removal.html:
      * storage/domstorage/sessionstorage/enumerate-storage.html:
      * storage/domstorage/sessionstorage/index-get-and-set.html:
      * storage/domstorage/sessionstorage/onstorage-attribute-markup.html:
      * storage/domstorage/sessionstorage/onstorage-attribute-setattribute.html:
      * storage/domstorage/sessionstorage/private-browsing-affects-storage.html:
      * storage/domstorage/sessionstorage/resources/window-open-second.html:
      * storage/domstorage/sessionstorage/simple-events.html:
      * storage/domstorage/sessionstorage/simple-usage.html:
      * storage/domstorage/sessionstorage/string-conversion.html:
      * storage/domstorage/sessionstorage/window-open.html:
      * storage/domstorage/window-attributes-exist.html:
      * svg/custom/stop-crash.svg:
      
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@46314 268f45cc-cd09-0410-ab3c-d52691b4dbfc
      b5cf863a
    • mrowe@apple.com's avatar
      And now with some valid JavaScript syntax! · 0e1ee81c
      mrowe@apple.com authored
      * http/tests/misc/redirect-to-about-blank.html:
      
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@46313 268f45cc-cd09-0410-ab3c-d52691b4dbfc
      0e1ee81c
    • mrowe@apple.com's avatar
      Remove unnecessary check for eventSender in an attempt to please the GTK build bot. · 8d61c1f4
      mrowe@apple.com authored
      * fast/css/getComputedStyle/getComputedStyle-text-overflow.html:
      
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@46312 268f45cc-cd09-0410-ab3c-d52691b4dbfc
      8d61c1f4
    • mrowe@apple.com's avatar
      Update build.webkit.org config for new machine. · 6b3df756
      mrowe@apple.com authored
      * BuildSlaveSupport/build.webkit.org-config/config.json:
      
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@46311 268f45cc-cd09-0410-ab3c-d52691b4dbfc
      6b3df756
    • mrowe@apple.com's avatar
      Add a workaround for <rdar://problem/6925892> to the test that it affects. · 13d187ef
      mrowe@apple.com authored
      Reviewed by Brady Eidson.
      
      * http/tests/misc/redirect-to-about-blank.html:
      
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@46310 268f45cc-cd09-0410-ab3c-d52691b4dbfc
      13d187ef
    • beidson@apple.com's avatar
      2009-07-23 Brady Eidson <beidson@apple.com> · 8bb49f2d
      beidson@apple.com authored
              Remove accidentally checked in code.
      
              * DumpRenderTree/mac/LayoutTestControllerMac.mm:
              (waitUntilDoneWatchdogFired):
              (LayoutTestController::setWaitToDump):
      
      
      
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@46309 268f45cc-cd09-0410-ab3c-d52691b4dbfc
      8bb49f2d
    • beidson@apple.com's avatar
      WebCore: · df488b50
      beidson@apple.com authored
      2009-07-23  Brady Eidson  <beidson@apple.com>
      
              Reviewed by Geoff Garen.
      
              WebCore has a few places that don't gracefully handle a null request returned from willSendRequest.
              https://bugs.webkit.org/show_bug.cgi?id=27595
      
              Test: http/tests/misc/will-send-request-returns-null-on-redirect.html
              
              * inspector/InspectorController.cpp:
              (WebCore::InspectorController::removeResource): Null-check the request URL.
              
              * platform/network/cf/ResourceHandleCFNet.cpp: Ditto, and return null instead of creating an empty one.
              (WebCore::willSendRequest):
      
      WebKit/win:
      
      2009-07-23  Brady Eidson  <beidson@apple.com>
      
              Reviewed by Geoff Garen.
      
              WebFrameLoaderClient on Windows doesn't gracefully handle a null request returned from willSendRequest.
              https://bugs.webkit.org/show_bug.cgi?id=27595
      
              * WebCoreSupport/WebFrameLoaderClient.cpp:
              (WebFrameLoaderClient::dispatchWillSendRequest):
      
      WebKitTools:
      
      2009-07-23  Brady Eidson  <beidson@apple.com>
      
              Reviewed by Geoff Garen.
      
              WebCore has a few places that don't gracefully handle a null request returned from willSendRequest.
              https://bugs.webkit.org/show_bug.cgi?id=27595
      
              Add the ability to tell the LayoutTestController to return a null request for willSendRequest
              redirect callbacks.
              
              * DumpRenderTree/LayoutTestController.cpp:
              (LayoutTestController::LayoutTestController):
              (setWillSendRequestReturnsNullOnRedirectCallback):
              (LayoutTestController::staticFunctions):
              * DumpRenderTree/LayoutTestController.h:
              (LayoutTestController::willSendRequestReturnsNullOnRedirect):
              (LayoutTestController::setWillSendRequestReturnsNullOnRedirect):
      
              * DumpRenderTree/mac/ResourceLoadDelegate.mm:
              (-[ResourceLoadDelegate webView:resource:willSendRequest:redirectResponse:fromDataSource:]):
              * DumpRenderTree/win/ResourceLoadDelegate.cpp:
              (ResourceLoadDelegate::willSendRequest):
      
      LayoutTests:
      
      2009-07-23  Brady Eidson  <beidson@apple.com>
      
              Reviewed by Geoff Garen.
      
              WebCore has a few places that don't gracefully handle a null request returned from willSendRequest.
              https://bugs.webkit.org/show_bug.cgi?id=27595
      
              * http/tests/misc/resources/redirect-to-http-url.php: Added.
              * http/tests/misc/will-send-request-returns-null-on-redirect-expected.txt: Added.
              * http/tests/misc/will-send-request-returns-null-on-redirect.html: Added.
              * platform/gtk/Skipped:
              * platform/qt/Skipped:
      
      
      
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@46308 268f45cc-cd09-0410-ab3c-d52691b4dbfc
      df488b50
    • pdherbemont@apple.com's avatar
      2009-07-23 Pierre d'Herbemont <pdherbemont@apple.com> · 02ff2d94
      pdherbemont@apple.com authored
              Fix layout tests, no review.
      
              Actually add the video-layer tests results for leopard, that were
              forgotten in r46302. Also add test results for windows.
      
              * platform/mac-leopard/fast/layers/video-layer-expected.png: Added.
              * platform/mac-leopard/fast/layers/video-layer-expected.txt: Added.
              * platform/win/fast/layers/video-layer-expected.txt: Added.
      
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@46307 268f45cc-cd09-0410-ab3c-d52691b4dbfc
      02ff2d94
    • mrowe@apple.com's avatar
      Fix DumpRenderTree to not fail an assertion on launch on a new machine. · be919533
      mrowe@apple.com authored
      Reviewed Oliver Hunt.
      
      * DumpRenderTree/mac/DumpRenderTree.mm:
      (setDefaultsToConsistentValuesForTesting): Disable the XSS auditor.  It being enabled
      breaks all of the JavaScript tests within testStringByEvaluatingJavaScriptFromString,
      which causes us to exit due to an assertion failure.  It's not clear why the XSS auditor
      decides to interfere with these tests.
      
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@46304 268f45cc-cd09-0410-ab3c-d52691b4dbfc
      be919533
    • pdherbemont@apple.com's avatar
      2009-07-23 Pierre d'Herbemont <pdherbemont@apple.com> · c2737dd4
      pdherbemont@apple.com authored
              Reviewed by Simon Fraser.
      
              Following changes on the video controls, video-layer tests results need to
              be updated in order to fix the layout tests.
      
              * platform/mac-leopard/fast/layers/video-layer-expected.checksum: Copied from LayoutTests/platform/mac/fast/layers/video-layer-expected.checksum.
              * platform/mac-leopard/fast/layers/video-layer-expected.png: Copied from LayoutTests/platform/mac/fast/layers/video-layer-expected.png.
              * platform/mac-leopard/fast/layers/video-layer-expected.txt: Copied from LayoutTests/platform/mac/fast/layers/video-layer-expected.txt.
              * platform/mac/fast/layers/video-layer-expected.checksum: Removed.
              * platform/mac/fast/layers/video-layer-expected.png: Removed.
              * platform/mac/fast/layers/video-layer-expected.txt: Updated.
      
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@46302 268f45cc-cd09-0410-ab3c-d52691b4dbfc
      c2737dd4
    • cfleizach@apple.com's avatar
    • darin@apple.com's avatar
      2009-07-23 Darin Adler <darin@apple.com> · 0395090d
      darin@apple.com authored
              * WebView/WebDataSource.mm:
              (-[WebDataSource _makeRepresentation]): Landed a comment I forgot last time.
      
      
      
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@46300 268f45cc-cd09-0410-ab3c-d52691b4dbfc
      0395090d
    • mrowe@apple.com's avatar
      Update layout test results for three tests related to text fields as their metrics · b43babdd
      mrowe@apple.com authored
      have changed slightly since the results were last updated.
      
      Rubber-stamped by Dan Bernstein.
      
      * platform/mac-leopard/fast/forms/text-control-intrinsic-widths-expected.txt: Copied from LayoutTests/fast/forms/text-control-intrinsic-widths-expected.txt.
      * platform/mac/fast/forms/input-text-maxlength-expected.txt:
      * platform/mac/fast/forms/input-text-paste-maxlength-expected.txt:
      * platform/mac/fast/forms/text-control-intrinsic-widths-expected.txt: Renamed from LayoutTests/fast/forms/text-control-intrinsic-widths-expected.txt.
      
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@46299 268f45cc-cd09-0410-ab3c-d52691b4dbfc
      b43babdd
    • darin@apple.com's avatar
      WebCore: · 719d29cb
      darin@apple.com authored
      2009-07-23  Darin Adler  <darin@apple.com>
      
              Reviewed by Brady Eidson.
      
              URL appears in back/forward button menu instead of title for items with custom representation
              https://bugs.webkit.org/show_bug.cgi?id=27586
              rdar://problem/5060337
      
              * WebCore.base.exp: Exported DocumentLoader::setTitle for use by Mac WebKit.
              * loader/FrameLoader.cpp:
              (WebCore::FrameLoader::didChangeTitle): Tightened code to check if the document
              loader is the correct one; previously this would never happen because we'd
              commit the load before any title changes could be registered, but now we can
              encounter a case where we get a title during a provisional load.
      
      WebKit/mac:
      
      2009-07-23  Darin Adler  <darin@apple.com>
      
              Reviewed by Brady Eidson.
      
              URL appears in back/forward button menu instead of title for items with custom representation
              https://bugs.webkit.org/show_bug.cgi?id=27586
              rdar://problem/5060337
      
              The problem is that DocumentLoader expects to store a title, but for custom representations
              it is never passed to the document loader.
      
              * WebView/WebDataSource.mm:
              (-[WebDataSource _makeRepresentation]): Added a call to DocumentLoader::setTitle.
              Works as long as the title does not change during the document's lifetime, which
              is good enough for the simple cases in Safari.
      
      
      
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@46298 268f45cc-cd09-0410-ab3c-d52691b4dbfc
      719d29cb
    • mrowe@apple.com's avatar
      Update default state of the XSS auditor in GTK DumpRenderTree to match Mac and Windows · 062ce4a1
      mrowe@apple.com authored
      in hopes of fixing some layout test failures seen on the build bot.
      
      Reviewed by Oliver Hunt.
      
      * DumpRenderTree/gtk/DumpRenderTree.cpp:
      (resetWebViewToConsistentStateBeforeTesting):
      
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@46297 268f45cc-cd09-0410-ab3c-d52691b4dbfc
      062ce4a1
    • mrowe@apple.com's avatar
      Windows appears to match the Leopard results for the CSS 2.1 counter tests that were failing. · 23c2cc53
      mrowe@apple.com authored
      * platform/win/css2.1/t1202-counter-04-b-expected.txt: Copied from LayoutTests/platform/mac-leopard/css2.1/t1202-counter-04-b-expected.txt.
      * platform/win/css2.1/t1202-counters-04-b-expected.txt: Copied from LayoutTests/platform/mac-leopard/css2.1/t1202-counters-04-b-expected.txt.
      
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@46296 268f45cc-cd09-0410-ab3c-d52691b4dbfc
      23c2cc53
    • mitz@apple.com's avatar
      WebCore: · fde1b4b0
      mitz@apple.com authored
              Reviewed by Dave Hyatt.
      
              [CSS3 Backgrounds and Borders] Add support for inset box shadows
              https://bugs.webkit.org/show_bug.cgi?id=27582
      
              Test: fast/box-shadow/inset.html
      
              * css/CSSComputedStyleDeclaration.cpp:
              (WebCore::valueForShadow): Set the ShadowValue’s shadow style to 'inset'
                  as needed.
      
              * css/CSSParser.cpp:
              (WebCore::ShadowParseContext::ShadowParseContext): Added style and allowStyle
                  members. Initialize the allowStyle member.
              (WebCore::ShadowParseContext::commitValue): Pass the style value to the
                  ShadowValue constructor. Reset allowStyle.
              (WebCore::ShadowParseContext::commitLength): Update allowStyle.
              (WebCore::ShadowParseContext::commitColor): Ditto.
              (WebCore::ShadowParseContext::commitStyle): Added. Sets the style member and
                  updates the state.
              (WebCore::CSSParser::parseShadow): Parse the 'inset' keyword.
      
              * css/CSSStyleSelector.cpp:
              (WebCore::CSSStyleSelector::applyProperty): Get the style value from the
                  shadow value and pass it to the ShadowData constructor.
      
              * css/ShadowValue.cpp:
              (WebCore::ShadowValue::ShadowValue): Added style.
              (WebCore::ShadowValue::cssText): Added style.
      
              * css/ShadowValue.h:
              (WebCore::ShadowValue::create): Added style.
      
              * page/animation/AnimationBase.cpp:
              (WebCore::blendFunc): Added a blend function for shadow styles. When blending
                  between normal and inset shadows, all intermediate values map to normal.
              (WebCore::PropertyWrapperShadow::blend): Added normal style to the default
                  shadow.
                  
              * rendering/InlineFlowBox.cpp:
              (WebCore::InlineFlowBox::paintBoxShadow): Added a shadow style parameter,
                  which is passed through to RenderBoxModelObject::paintBoxShadow().
      
              (WebCore::InlineFlowBox::paintBoxDecorations): Paint inset shadows on top of
                  the background.
      
              * rendering/InlineFlowBox.h:
      
              * rendering/RenderBox.cpp:
              (WebCore::RenderBox::paintBoxDecorations): Paint inset shadows on top of the
                  background.
      
              * rendering/RenderBoxModelObject.cpp:
              (WebCore::RenderBoxModelObject::paintBoxShadow): Added a shadow style
                  parameter, and code to paint inset shadows.
      
              * rendering/RenderBoxModelObject.h:
      
              * rendering/RenderFieldset.cpp:
              (WebCore::RenderFieldset::paintBoxDecorations): Paint inset shadows on top of
                  the background.
      
              * rendering/RenderTable.cpp:
              (WebCore::RenderTable::paintBoxDecorations): Ditto.
      
              * rendering/RenderTableCell.cpp:
              (WebCore::RenderTableCell::paintBoxDecorations): Ditto.
      
              * rendering/style/ShadowData.h:
              Added a ShadowStyle enum.
              (WebCore::ShadowData::ShadowData): Add and initialize a style member.
      
      LayoutTests:
      
              Reviewed by Dave Hyatt.
      
              [CSS3 Backgrounds and Borders] Add support for inset box shadows
              https://bugs.webkit.org/show_bug.cgi?id=27582
      
              * fast/box-shadow/inset.html: Added.
              * platform/mac/fast/box-shadow/inset-expected.checksum: Added.
              * platform/mac/fast/box-shadow/inset-expected.png: Added.
              * platform/mac/fast/box-shadow/inset-expected.txt: Added.
      
      
      
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@46295 268f45cc-cd09-0410-ab3c-d52691b4dbfc
      fde1b4b0
    • treat@webkit.org's avatar
      2009-07-23 Jakob Petsovits <jakob.petsovits@torchmobile.com> · 4629ee3a
      treat@webkit.org authored
               Reviewed by Adam Treat.
      
               Fix false positives for switch statement indentation check in cpplint.
               https://bugs.webkit.org/show_bug.cgi?id=27615
      
               Makes one-line case statements (e.g. "case foo: bar();") work.
               Also a few general improvements to the robustness and readability of
               the check, and more test cases.
      
               * Scripts/modules/cpplint.py:
               * Scripts/modules/cpplint_unittest.py:
      
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@46294 268f45cc-cd09-0410-ab3c-d52691b4dbfc
      4629ee3a
    • simon.fraser@apple.com's avatar
      2009-07-23 Simon Fraser <simon.fraser@apple.com> · 69070f36
      simon.fraser@apple.com authored
              Fix the build with UNUSED_PARAM(frame) for when ENABLE(3D_RENDERING) is not defined.
      
              * css/MediaQueryEvaluator.cpp:
              (WebCore::transform_3dMediaFeatureEval):
      
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@46293 268f45cc-cd09-0410-ab3c-d52691b4dbfc
      69070f36
    • treat@webkit.org's avatar
      2009-07-23 Jakob Petsovits <jakob.petsovits@torchmobile.com> · 2fe25536
      treat@webkit.org authored
               Reviewed by Adam Treat.
      
               Add check for line-breaking rule #3 to cpplint.
               https://bugs.webkit.org/show_bug.cgi?id=27610
      
               "An else if statement should be written as an if statement when
               the prior if concludes with a return statement."
      
               Implemented by a multi-line (kind of back-tracking) algorithm.
               Comes with loads of unit tests. Fixes the check for label indentation
               to be not so overzealous, as it didn't allow for completely unindented
               goto labels (at the very start of a line).
      
               * Scripts/modules/cpplint.py:
               * Scripts/modules/cpplint_unittest.py:
      
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@46292 268f45cc-cd09-0410-ab3c-d52691b4dbfc
      2fe25536
    • simon.fraser@apple.com's avatar
      2009-07-23 Simon Fraser <simon.fraser@apple.com> · ec4b4f9e
      simon.fraser@apple.com authored
              Reviewed by Adele Peterson.
      
              3d-transforms media query needs to look check that accelerated compositing is enabled
              https://bugs.webkit.org/show_bug.cgi?id=27621
      
              When evaluating a media query with '-webkit-transform-3d', we need to check the
              runtime switch that toggles accererated compositing, and therefore 3D.
      
              No test because we can't disable the pref dynamically in DRT.
      
              * css/MediaQueryEvaluator.cpp:
              (WebCore::transform_3dMediaFeatureEval):
      
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@46291 268f45cc-cd09-0410-ab3c-d52691b4dbfc
      ec4b4f9e
    • mrowe@apple.com's avatar
      Update layout test results for two international text tests that are failing... · 5752f6e1
      mrowe@apple.com authored
      Update layout test results for two international text tests that are failing due to font differences.
      
      Rubber-stamped by Dan Bernstein.
      
      * platform/mac-leopard/fast/text/international/thai-baht-space-expected.checksum: Renamed from LayoutTests/platform/mac/fast/text/international/thai-baht-space-expected.checksum.
      * platform/mac-leopard/fast/text/international/thai-baht-space-expected.png: Renamed from LayoutTests/platform/mac/fast/text/international/thai-baht-space-expected.png.
      * platform/mac-leopard/fast/text/international/thai-baht-space-expected.txt: Copied from LayoutTests/platform/mac/fast/text/international/thai-baht-space-expected.txt.
      * platform/mac/fast/text/international/bidi-neutral-directionality-paragraph-start-expected.txt:
      * platform/mac/fast/text/international/thai-baht-space-expected.txt:
      
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@46290 268f45cc-cd09-0410-ab3c-d52691b4dbfc
      5752f6e1
    • mrowe@apple.com's avatar
      Update layout test results for two CSS 2.1 counter tests that are failing due to font differences. · 9e962c0b
      mrowe@apple.com authored
      Rubber-stamped by Dan Bernstein.
      
      * platform/mac-leopard/css2.1/t1202-counter-04-b-expected.txt: Copied from LayoutTests/platform/mac/css2.1/t1202-counter-04-b-expected.txt.
      * platform/mac-leopard/css2.1/t1202-counters-04-b-expected.txt: Copied from LayoutTests/platform/mac/css2.1/t1202-counters-04-b-expected.txt.
      * platform/mac/css2.1/t1202-counter-04-b-expected.txt:
      * platform/mac/css2.1/t1202-counters-04-b-expected.txt:
      
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@46289 268f45cc-cd09-0410-ab3c-d52691b4dbfc
      9e962c0b
    • pdherbemont@apple.com's avatar
      2009-07-23 Pierre d'Herbemont <pdherbemont@apple.com> · b1a382bb
      pdherbemont@apple.com authored
              Layout test fix so no review.
      
              Add audio-controls-rendering layout test results for windows and leopard.
      
              * platform/mac-leopard/media/audio-controls-rendering-expected.txt: Added.
              * platform/win/media/audio-controls-rendering-expected.txt: Added.
      
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@46288 268f45cc-cd09-0410-ab3c-d52691b4dbfc
      b1a382bb
    • mrowe@apple.com's avatar
      Update layout test results for the addition of a new computed style property in r46285. · d8a4e4af
      mrowe@apple.com authored
      * fast/css/getComputedStyle/computed-style-expected.txt:
      * fast/css/getComputedStyle/computed-style-without-renderer-expected.txt:
      * platform/mac/fast/css/getComputedStyle/computed-style-expected.txt:
      * platform/mac/fast/css/getComputedStyle/computed-style-without-renderer-expected.txt:
      * platform/qt/fast/css/getComputedStyle/computed-style-expected.txt:
      * platform/qt/fast/css/getComputedStyle/computed-style-without-renderer-expected.txt:
      * platform/win/fast/css/getComputedStyle/computed-style-expected.txt:
      * platform/win/fast/css/getComputedStyle/computed-style-without-renderer-expected.txt:
      * svg/css/getComputedStyle-basic-expected.txt:
      
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@46287 268f45cc-cd09-0410-ab3c-d52691b4dbfc
      d8a4e4af
    • rniwa@webkit.org's avatar
      WebCore: · 5bd84ef3
      rniwa@webkit.org authored
      2009-07-22  Ryosuke Niwa  <rniwa@webkit.org>
      
              Reviewed by Eric Seidel.
      
              execCommand('underline') can't remove <U> underlines
              https://bugs.webkit.org/show_bug.cgi?id=20215
      
              This patch adds support for u, s, and strike to implicitlyStyledElementShouldBeRemovedWhenApplyingStyle so that
              WebKit can remove those presentational tags when necessary.
              It also modifies StyleChange::init not to add "text-decoration: none".  Not only is this style meaningless
              (does not override inherited styles) but it was also causing WebKit to generate extra spans with this style.
      
              * css/CSSValueList.cpp:
              (WebCore::CSSValueList::hasValue): True if the property contains the specified value
              * css/CSSValueList.h: Updated prototype
              * editing/ApplyStyleCommand.cpp:
              (WebCore::StyleChange::init): No longer adds "text-decoration: none"
              (WebCore::ApplyStyleCommand::implicitlyStyledElementShouldBeRemovedWhenApplyingStyle): Supports text-decoration-related elements
      
      LayoutTests:
      
      2009-07-22  Ryosuke Niwa  <rniwa@webkit.org>
      
              Reviewed by Eric Seidel.
      
              execCommand('underline') can't remove <U> underlines
              https://bugs.webkit.org/show_bug.cgi?id=20215
      
              This patch rebaselines toggle-styles.html because WebKit now passes three tests it used to fail.
              toggle-style-2.html is added to test cases in which multiple styles are specified with tags.
              We still fail some tests because WebKit doesn't properly support non-CSS mode but they are visually correct.
      
              * editing/execCommand/resources/toggle-style-2.js: Added.
              (testSingleToggle):
              (testDoubleToggle):
              * editing/execCommand/toggle-style-2-expected.txt: Added.
              * editing/execCommand/toggle-style-2.html: Added.
              * editing/execCommand/toggle-styles-expected.txt: Passes all the tests
      
      
      
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@46286 268f45cc-cd09-0410-ab3c-d52691b4dbfc
      5bd84ef3
    • jberlin@apple.com's avatar
      2009-07-23 Jessie Berlin <jberlin@apple.com> · e5f53553
      jberlin@apple.com authored
              https://bugs.webkit.org/show_bug.cgi?id=27554
              Expose the value of text-overflow in getComputedStyle.
              
              Reviewed by Dan Bernstein.
      
              Test: fast/css/getComputedStyle/getComputedStyle-text-overflow.html
      
              * css/CSSComputedStyleDeclaration.cpp:
              (WebCore::):
              Add text-overflow to the list of computedProperties.
              (WebCore::CSSComputedStyleDeclaration::getPropertyCSSValue):
              Return the value of the text-overflow property.
      
              * fast/css/getComputedStyle/getComputedStyle-text-overflow-expected.txt: Added.
              * fast/css/getComputedStyle/getComputedStyle-text-overflow.html: Added.
      
      
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@46285 268f45cc-cd09-0410-ab3c-d52691b4dbfc
      e5f53553
    • hausmann@webkit.org's avatar
      2009-07-23 Yongjun Zhang <yongjun.zhang@nokia.com> · 096c6a95
      hausmann@webkit.org authored
              Reviewed by Simon Hausmann.
      
              https://bugs.webkit.org/show_bug.cgi?id=27510
      
              [S60 QtWebKit] Don't put some intermediate generated files into the final mmp project
              file for linking. This is a temporary workaround for qmake bug in Symbian port, should
              be reverted after qmake is fixed.
      
              * WebCore.pro:
      
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@46284 268f45cc-cd09-0410-ab3c-d52691b4dbfc
      096c6a95
    • jianli@chromium.org's avatar
      2009-07-23 Jian Li <jianli@chromium.org> · 10b1eaaf
      jianli@chromium.org authored
              Reviewed by David Levin.
      
              [V8] Fix an assert in running workers in Chrome.
              https://bugs.webkit.org/show_bug.cgi?id=27620
      
              The fix is to change V8DOMMap::removeAllDOMObjectsInCurrentThreadHelper
              to do not call removeObjectsFromWrapperMap for certain types of DOM
              objects that exist only in main thread.
      
              * bindings/v8/V8DOMMap.cpp:
              (WebCore::removeAllDOMObjectsInCurrentThreadHelper):
      
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@46283 268f45cc-cd09-0410-ab3c-d52691b4dbfc
      10b1eaaf
    • rniwa@webkit.org's avatar
      2009-07-23 Ryosuke Niwa <rniwa@webkit.org> · eee58410
      rniwa@webkit.org authored
              Reviewed by David Hyatt.
      
              Rebaseline for http://trac.webkit.org/changeset/46274
              (The changeset rebaselined the expected results for mac but not for windows)
      
              * platform/win/fast/css/getComputedStyle/computed-style-expected.txt:
              * platform/win/fast/css/getComputedStyle/computed-style-without-renderer-expected.txt:
      
      
      
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@46282 268f45cc-cd09-0410-ab3c-d52691b4dbfc
      eee58410
    • hyatt@apple.com's avatar
      WebCore: · b3f921e5
      hyatt@apple.com authored
      2009-07-23  David Hyatt  <hyatt@apple.com>
      
              Reviewed by Dan Bernstein.
      
              https://bugs.webkit.org/show_bug.cgi?id=27572
              Implement support for background-attachment:local.
      
              Added new test fast/overflow/overflow-with-local-attachment.html.
      
              * css/CSSComputedStyleDeclaration.cpp:
              (WebCore::CSSComputedStyleDeclaration::getPropertyCSSValue):
              * css/CSSParser.cpp:
              (WebCore::CSSParser::parseFillProperty):
              * css/CSSPrimitiveValueMappings.h:
              (WebCore::CSSPrimitiveValue::CSSPrimitiveValue):
              (WebCore::CSSPrimitiveValue::operator EFillAttachment):
              * css/CSSStyleSelector.cpp:
              (WebCore::CSSStyleSelector::mapFillAttachment):
              * css/CSSValueKeywords.in:
              * rendering/RenderBoxModelObject.cpp:
              (WebCore::RenderBoxModelObject::paintFillLayerExtended):
              (WebCore::RenderBoxModelObject::calculateBackgroundImageGeometry):
              * rendering/style/FillLayer.h:
              (WebCore::FillLayer::attachment):
              (WebCore::FillLayer::setAttachment):
              (WebCore::FillLayer::hasFixedImage):
              (WebCore::FillLayer::initialFillAttachment):
              * rendering/style/RenderStyle.h:
              (WebCore::InheritedFlags::backgroundAttachment):
              (WebCore::InheritedFlags::maskAttachment):
              * rendering/style/RenderStyleConstants.h:
              (WebCore::):
      
      LayoutTests:
      
      2009-07-23  David Hyatt  <hyatt@apple.com>
      
              Reviewed by Dan Bernstein.
      
              https://bugs.webkit.org/show_bug.cgi?id=27572
              Implement support for background-attachment:local.
      
              Added new test fast/overflow/overflow-with-local-attachment.html.
      
              * fast/overflow/overflow-with-local-background-attachment.html: Added.
              * platform/mac/fast/overflow/overflow-with-local-background-attachment-expected.checksum: Added.
              * platform/mac/fast/overflow/overflow-with-local-background-attachment-expected.png: Added.
              * platform/mac/fast/overflow/overflow-with-local-background-attachment-expected.txt: Added.
      
      
      
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@46281 268f45cc-cd09-0410-ab3c-d52691b4dbfc
      b3f921e5
    • rniwa@webkit.org's avatar
      2009-07-23 Ryosuke Niwa <rniwa@webkit.org> · e49e6734
      rniwa@webkit.org authored
              Reviewed by Eric Seidel.
      
              copyInheritableProperties and removeComputedInheritablePropertiesFrom should be deprecated
              https://bugs.webkit.org/show_bug.cgi?id=27325
      
              This patch deprecates copyInheritableProperties because it has been used for two different purposes:
              1.  Calculating the typing style.
              2.  Moving HTML subtrees and seeking to remove redundant styles.
              These tasks should be broken out into two separate functions.  New code should not use this function.
      
              It deletes removeComputedInheritablePropertiesFrom because it hasn't been used anywhere.
      
              There is no test since the patch does not change any behavior.
      
              * css/CSSComputedStyleDeclaration.cpp: removeComputedInheritablePropertiesFrom has been removed
              (WebCore::CSSComputedStyleDeclaration::deprecatedCopyInheritableProperties): has been renamed from copyInheritableProperties
              * css/CSSComputedStyleDeclaration.h: ditto
              * editing/DeleteSelectionCommand.cpp: ditto
              (WebCore::removeEnclosingAnchorStyle): ditto
              (WebCore::DeleteSelectionCommand::saveTypingStyleState): ditto
              * editing/EditCommand.cpp: ditto
              (WebCore::EditCommand::styleAtPosition): ditto
              * editing/RemoveFormatCommand.cpp: ditto
              (WebCore::RemoveFormatCommand::doApply): ditto
              * editing/ReplaceSelectionCommand.cpp: ditto
              (WebCore::handleStyleSpansBeforeInsertion): ditto
              (WebCore::ReplaceSelectionCommand::handleStyleSpans): ditto
              * editing/markup.cpp: ditto
              (WebCore::removeEnclosingMailBlockquoteStyle): ditto
              (WebCore::removeDefaultStyles): ditto
              (WebCore::createMarkup): ditto
      
      
      
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@46280 268f45cc-cd09-0410-ab3c-d52691b4dbfc
      e49e6734
    • pdherbemont@apple.com's avatar
      WebCore: · 512d4de7
      pdherbemont@apple.com authored
          2009-07-22  Pierre d'Herbemont  <pdherbemont@apple.com>
      
              Reviewed by Simon Fraser.
      
              Audio element at default width shouldn't have time field.
              https://bugs.webkit.org/show_bug.cgi?id=27589
      
              * rendering/MediaControlElements.cpp:
              (WebCore::MediaControlTimeDisplayElement::setVisible): Make sure we don't
              forget to remember the visibility if there is no renderer.
      
      LayoutTests:
      
      2009-07-22  Pierre d'Herbemont  <pdherbemont@apple.com>
      
              Reviewed by Simon Fraser.
      
              Audio element at default width shouldn't have time field.
              https://bugs.webkit.org/show_bug.cgi?id=27589
      
              * platform/mac-snowleopard/Skipped: Enables back audio-controls-rendering.
              * platform/mac/media/audio-controls-rendering-expected.checksum:
              * platform/mac/media/audio-controls-rendering-expected.png:
              * platform/mac/media/audio-controls-rendering-expected.txt:
      
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@46279 268f45cc-cd09-0410-ab3c-d52691b4dbfc
      512d4de7
    • kenneth@webkit.org's avatar
      2009-07-23 Kenneth Rohde Christiansen <kenneth@webkit.org> · c38c84ee
      kenneth@webkit.org authored
              Reviewed by Adam Treat.
      
              Added expected results for tests that we pass.
      
              * platform/qt/editing/inserting/break-blockquote-after-delete-expected.txt: Added.
              * platform/qt/fast/backgrounds/body-generated-image-propagated-to-root-expected.txt: Added.
              * platform/qt/fast/box-shadow/transform-fringing-expected.txt: Added.
              * platform/qt/fast/canvas/drawImage-with-globalAlpha-expected.txt: Added.
              * platform/qt/fast/css/attribute-selector-dynamic-expected.txt: Added.
              * platform/qt/fast/css/nth-child-dynamic-expected.txt: Added.
              * platform/qt/fast/dom/HTMLTableColElement/resize-table-using-col-width-expected.txt: Added.
              * platform/qt/fast/dynamic/anchor-lock-expected.txt: Added.
              * platform/qt/fast/frames/frameset-style-recalc-expected.txt: Added.
              * platform/qt/fast/frames/iframe-text-contents-expected.txt: Added, equal to mac.
              * platform/qt/fast/frames/viewsource-on-image-file-expected.txt: Added.
              * platform/qt/fast/repaint/background-generated-expected.txt: Added, equal to mac.
              * platform/qt/fast/repaint/background-misaligned-expected.txt: Added, equal to mac.
              * platform/qt/fast/repaint/float-in-new-block-with-layout-delta-expected.txt: Added, equal to mac.
              * platform/qt/fast/text/international/hindi-whitespace-expected.txt: Added.
              * platform/qt/fast/transforms/bounding-rect-zoom-expected.txt: Added.
      
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@46278 268f45cc-cd09-0410-ab3c-d52691b4dbfc
      c38c84ee
    • kevino@webkit.org's avatar
      wx build fix, adding missing header. · 60f67ce9
      kevino@webkit.org authored
      
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@46277 268f45cc-cd09-0410-ab3c-d52691b4dbfc
      60f67ce9
    • bdakin@apple.com's avatar
      2009-07-23 Beth Dakin <bdakin@apple.com> · ab05fc54
      bdakin@apple.com authored
              Reviewed by Darin Adler.
      
              Fix for https://bugs.webkit.org/show_bug.cgi?id=27598 Clean up the 
              AccessibilityObject class
      
              Mostly this is just moving empty stubs into the header rather than 
              muddying the cpp file with them. A few functions were made pure 
              virtual.
      
              * accessibility/AccessibilityObject.cpp:
              (WebCore::AccessibilityObject::isARIAControl):
              (WebCore::AccessibilityObject::clickPoint):
              (WebCore::AccessibilityObject::documentFrameView):
              (WebCore::AccessibilityObject::actionVerb):
              * accessibility/AccessibilityObject.h:
              (WebCore::AccessibilityObject::intValue):
              (WebCore::AccessibilityObject::layoutCount):
              (WebCore::AccessibilityObject::doAccessibilityHitTest):
              (WebCore::AccessibilityObject::focusedUIElement):
              (WebCore::AccessibilityObject::firstChild):
              (WebCore::AccessibilityObject::lastChild):
              (WebCore::AccessibilityObject::previousSibling):
              (WebCore::AccessibilityObject::nextSibling):
              (WebCore::AccessibilityObject::parentObjectIfExists):
              (WebCore::AccessibilityObject::observableObject):
              (WebCore::AccessibilityObject::linkedUIElements):
              (WebCore::AccessibilityObject::titleUIElement):
              (WebCore::AccessibilityObject::ariaRoleAttribute):
              (WebCore::AccessibilityObject::isPresentationalChildOfAriaRole):
              (WebCore::AccessibilityObject::ariaRoleHasPresentationalChildren):
              (WebCore::AccessibilityObject::roleValue):
              (WebCore::AccessibilityObject::ariaAccessiblityName):
              (WebCore::AccessibilityObject::ariaLabeledByAttribute):
              (WebCore::AccessibilityObject::ariaDescribedByAttribute):
              (WebCore::AccessibilityObject::accessibilityDescription):
              (WebCore::AccessibilityObject::ariaSelectedTextDOMRange):
              (WebCore::AccessibilityObject::axObjectCache):
              (WebCore::AccessibilityObject::axObjectID):
              (WebCore::AccessibilityObject::setAXObjectID):
              (WebCore::AccessibilityObject::anchorElement):
              (WebCore::AccessibilityObject::actionElement):
              (WebCore::AccessibilityObject::boundingBoxRect):
              (WebCore::AccessibilityObject::selectedTextRange):
              (WebCore::AccessibilityObject::selectionStart):
              (WebCore::AccessibilityObject::selectionEnd):
              (WebCore::AccessibilityObject::url):
              (WebCore::AccessibilityObject::selection):
              (WebCore::AccessibilityObject::stringValue):
              (WebCore::AccessibilityObject::title):
              (WebCore::AccessibilityObject::helpText):
              (WebCore::AccessibilityObject::textUnderElement):
              (WebCore::AccessibilityObject::text):
              (WebCore::AccessibilityObject::textLength):
              (WebCore::AccessibilityObject::selectedText):
              (WebCore::AccessibilityObject::accessKey):
              (WebCore::AccessibilityObject::widget):
              (WebCore::AccessibilityObject::widgetForAttachmentView):
              (WebCore::AccessibilityObject::setFocused):
              (WebCore::AccessibilityObject::setSelectedText):
              (WebCore::AccessibilityObject::setSelectedTextRange):
              (WebCore::AccessibilityObject::setValue):
              (WebCore::AccessibilityObject::setSelected):
              (WebCore::AccessibilityObject::makeRangeVisible):
              (WebCore::AccessibilityObject::childrenChanged):
              (WebCore::AccessibilityObject::addChildren):
              (WebCore::AccessibilityObject::hasChildren):
              (WebCore::AccessibilityObject::selectedChildren):
              (WebCore::AccessibilityObject::visibleChildren):
              (WebCore::AccessibilityObject::visiblePositionRange):
              (WebCore::AccessibilityObject::visiblePositionRangeForLine):
              (WebCore::AccessibilityObject::boundsForVisiblePositionRange):
              (WebCore::AccessibilityObject::setSelectedVisiblePositionRange):
              (WebCore::AccessibilityObject::visiblePositionForPoint):
              (WebCore::AccessibilityObject::nextVisiblePosition):
              (WebCore::AccessibilityObject::previousVisiblePosition):
              (WebCore::AccessibilityObject::visiblePositionForIndex):
              (WebCore::AccessibilityObject::indexForVisiblePosition):
              (WebCore::AccessibilityObject::index):
              (WebCore::AccessibilityObject::doAXRangeForLine):
              (WebCore::AccessibilityObject::doAXRangeForIndex):
              (WebCore::AccessibilityObject::doAXStringForRange):
              (WebCore::AccessibilityObject::doAXBoundsForRange):
              (WebCore::AccessibilityObject::updateBackingStore):
      
      
      
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@46276 268f45cc-cd09-0410-ab3c-d52691b4dbfc
      ab05fc54
    • bweinstein@apple.com's avatar
      Fix of <rdar://4877658> Dragging from the area between the horizontal/vertical... · 9f624e55
      bweinstein@apple.com authored
      Fix of <rdar://4877658> Dragging from the area between the horizontal/vertical scrollbars when status bar is showing starts a selection and autoscroll.
      
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@46275 268f45cc-cd09-0410-ab3c-d52691b4dbfc
      9f624e55
    • hyatt@apple.com's avatar
      WebCore: · 2dde470b
      hyatt@apple.com authored
      2009-07-23  David Hyatt  <hyatt@apple.com>
      
              Reviewed by Dan Bernstein.
      
              https://bugs.webkit.org/show_bug.cgi?id=27581
              Drop the prefix from the box-shadow property.
      
              * css/CSSComputedStyleDeclaration.cpp:
              (WebCore::):
              (WebCore::CSSComputedStyleDeclaration::getPropertyCSSValue):
              * css/CSSParser.cpp:
              (WebCore::CSSParser::parseValue):
              (WebCore::ShadowParseContext::commitLength):
              (WebCore::cssPropertyID):
              * css/CSSPropertyNames.in:
              * css/CSSStyleSelector.cpp:
              (WebCore::CSSStyleSelector::applyProperty):
              * page/animation/AnimationBase.cpp:
              (WebCore::ensurePropertyMap):
      
      LayoutTests:
      
      2009-07-23  David Hyatt  <hyatt@apple.com>
      
              Reviewed by Dan Bernstein.
              
              https://bugs.webkit.org/show_bug.cgi?id=27581
              Drop the prefix from the box-shadow property.
      
              * platform/mac/fast/css/getComputedStyle/computed-style-expected.txt:
              * platform/mac/fast/css/getComputedStyle/computed-style-without-renderer-expected.txt:
              * svg/css/getComputedStyle-basic-expected.txt:
      
      
      
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@46274 268f45cc-cd09-0410-ab3c-d52691b4dbfc
      2dde470b
    • treat@webkit.org's avatar
      WebCore: · bc00ddcf
      treat@webkit.org authored
      2009-07-22  Viet-Trung Luu  <viettrungluu@gmail.com>
      
              Reviewed by David Hyatt.
      
              https://bugs.webkit.org/show_bug.cgi?id=27289
              When a mouse click occurs on a scrollbar without a preceding mouse move
              onto it, the release isn't handled correctly (since
              EventHandler::m_lastScrollbarUnderMouse isn't set on mouse down, but
              only on mouse move). (Side comment: That scrollbar-handling code
              in EventHandler is ugly. It should be fixed properly.)
      
              Tests: scrollbars/scrollbar-miss-mousemove.html
                     scrollbars/scrollbar-miss-mousemove-disabled.html
      
              * page/EventHandler.cpp:
              (WebCore::EventHandler::handleMousePressEvent):
              (WebCore::EventHandler::handleMouseMoveEvent):
              (WebCore::EventHandler::updateLastScrollbarUnderMouse):
              * page/EventHandler.h:
      
      LayoutTests:
      2009-07-22  Viet-Trung Luu  <viettrungluu@gmail.com>
      
              Reviewed by David Hyatt.
      
              https://bugs.webkit.org/show_bug.cgi?id=27289
              Tests that mouse clicks/releases are handled properly on scrollbars
              even when there is no mouse move onto the scrollbar (two cases: enabled
              and disabled controls).
      
              * scrollbars/scrollbar-miss-mousemove-disabled-expected.txt: Added.
              * scrollbars/scrollbar-miss-mousemove-disabled.html: Added.
              * scrollbars/scrollbar-miss-mousemove-expected.txt: Added.
              * scrollbars/scrollbar-miss-mousemove.html: Added.
      
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@46273 268f45cc-cd09-0410-ab3c-d52691b4dbfc
      bc00ddcf
    • levin@chromium.org's avatar
      2009-07-23 Mike Fenton <mike.fenton@torchmobile.com> · 86403917
      levin@chromium.org authored
              Reviewed by David Levin.
      
              Update WebCore/page/BarInfo.cpp to conform to WebKit
              Style Guidelines as identified by cpplint.py.
              https://bugs.webkit.org/show_bug.cgi?id=27606
      
              * page/BarInfo.cpp:
              (WebCore::BarInfo::visible):
      
      git-svn-id: http://svn.webkit.org/repository/webkit/trunk@46272 268f45cc-cd09-0410-ab3c-d52691b4dbfc
      86403917