WebKit-https.git
12 years agoWebCore:
justin.garcia@apple.com [Wed, 23 Jan 2008 21:51:13 +0000 (21:51 +0000)]
WebCore:

        Reviewed by Alice Liu.

        <rdar://problem/5658727> Undo/redo for pasted or typed text in yahoo mail becomes disabled after one round of being selected

        * rendering/RenderTextControl.cpp:
        (WebCore::RenderTextControl::updateFromElement): If value == text() and
        m_innerText is empty, then value and text() are "". In that case both DOM
        mutations are no-ops and there is no reason to clear the Undo stack.

LayoutTests:

        Reviewed by Alice Liu.

        <rdar://problem/5658727> Undo/redo for pasted or typed text in yahoo mail becomes disabled after one round of being selected

        * editing/undo/5658727-expected.txt: Added.
        * editing/undo/5658727.html: Added.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@29748 268f45cc-cd09-0410-ab3c-d52691b4dbfc

12 years ago Fix <rdar://5698502> Inspector window reverts to default position/size unexp...
aroben@apple.com [Wed, 23 Jan 2008 21:08:15 +0000 (21:08 +0000)]
     Fix <rdar://5698502> Inspector window reverts to default position/size unexpectedly

         Reviewed by Hyatt.

         * WebInspectorClient.cpp:
         (defaultWindowRect): Added.
         (WebInspectorClient::createPage):
             - Use defaultWindowRect() to set the initial size/position of the
               Inspector window.
             - Pass the Inspector window's client rect to
               IWebView::initWithFrame so that the WebView will be initially
               sized to fill the window.
         (WebInspectorClient::showWindow): Don't move/resize the window back to
         the defaults every time this method is called.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@29747 268f45cc-cd09-0410-ab3c-d52691b4dbfc

12 years ago Fix <rdar://5700620> REGRESSION (r28190): "Open frame in new tab" instead of "Open...
aroben@apple.com [Wed, 23 Jan 2008 21:07:47 +0000 (21:07 +0000)]
 Fix <rdar://5700620> REGRESSION (r28190): "Open frame in new tab" instead of "Open link in new tab" @ digg.com

 WebKit/win:

         Fix <rdar://5700620> REGRESSION (r28190): "Open frame in new tab" instead of "Open link in new tab" @ digg.com

         Test: fast/events/contextmenu-scrolled-page-with-frame.html

         Reviewed by Dan.

         * WebView.cpp:
         (WebView::handleContextMenuEvent): Translate the mouse coordinates
         into document coordinates before performing the hit-test. This is what
         EventHandler does internally.

 LayoutTests:

         Add a test for <rdar://5700620> REGRESSION (r28190): "Open frame in
         new tab" instead of "Open link in new tab" @ digg.com

         Reviewed by Dan.

         * fast/events/contextmenu-scrolled-page-with-frame-expected.txt: Added.
         * fast/events/contextmenu-scrolled-page-with-frame.html: Added.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@29746 268f45cc-cd09-0410-ab3c-d52691b4dbfc

12 years agoWebCore:
sfalken@apple.com [Wed, 23 Jan 2008 19:43:14 +0000 (19:43 +0000)]
WebCore:

        <rdar://problem/5699509> Allow file upload dialog to be localized.

        Reviewed by Darin.

        * platform/LocalizedStrings.h:
        * platform/win/FileChooserWin.cpp:
        (WebCore::FileChooser::openFileChooser):

WebKit:

        Update localization exceptions.

        Reviewed by Darin.

        * English.lproj/Localizable.strings:
        * StringsNotToBeLocalized.txt:

WebKit/win:

        <rdar://problem/5699509> Allow file upload dialog to be localized.

        Reviewed by Darin.

        * English.lproj/Localizable.strings: Updated.
        * WebCoreLocalizedStrings.cpp:
        (WebCore::uploadFileText): Added.
        (WebCore::allFilesText): Added.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@29745 268f45cc-cd09-0410-ab3c-d52691b4dbfc

12 years ago Reviewed by Darin.
dsmith@webkit.org [Wed, 23 Jan 2008 17:49:23 +0000 (17:49 +0000)]
    Reviewed by Darin.

        Added a modified version of the MooTools SlickSpeed Selector Test/Benchmark to show off querySelectorAll. MIT licensed.

        * perf/slickspeed: Added.
        * perf/slickspeed/config.ini: Added.
        * perf/slickspeed/footer.html: Added.
        * perf/slickspeed/frameworks: Added.
        * perf/slickspeed/frameworks/DomQuery.js: Added.
        * perf/slickspeed/frameworks/dummy.js: Added.
        * perf/slickspeed/frameworks/jquery-1.2.1.js: Added.
        * perf/slickspeed/frameworks/prototype.js: Added.
        * perf/slickspeed/header.html: Added.
        * perf/slickspeed/index.php: Added.
        * perf/slickspeed/instructions.txt: Added.
        * perf/slickspeed/logo.png: Added.
        * perf/slickspeed/selectors.list: Added.
        * perf/slickspeed/style.css: Added.
        * perf/slickspeed/system: Added.
        * perf/slickspeed/system/index.php: Added.
        * perf/slickspeed/system/slickspeed.js: Added.
        * perf/slickspeed/system/template.php: Added.
        * perf/slickspeed/template.html: Added.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@29744 268f45cc-cd09-0410-ab3c-d52691b4dbfc

12 years ago Fix behavior of type-to-select in <select> menus
aroben@apple.com [Wed, 23 Jan 2008 17:38:17 +0000 (17:38 +0000)]
     Fix behavior of type-to-select in <select> menus

         In r27825 we started posting both a WM_KEYDOWN and WM_CHAR message
         whenever a key was pressed in the dropdown menu. However, since posted
         messages go through TranslateMessage, the WM_KEYDOWN was generating
         another WM_CHAR message, so two WM_CHAR messages in a row would reach
         the WebView. This caused the searching to happen twice for each key,
         so typing "dog" would search for "ddoogg".

         Reviewed by Darin.

         * platform/win/PopupMenuWin.cpp:
         (WebCore::PopupWndProc): Repost the WM_KEYDOWN message to the WebView.
         This is will generate a WM_CHAR message which will trigger the
         type-to-select behavior. This is very similar to our behavior prior to
         r27825, except that we're now calling PostMessage instead of
         SendMessage.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@29743 268f45cc-cd09-0410-ab3c-d52691b4dbfc

12 years ago - updated results following <http://trac.webkit.org/projects/webkit/changeset...
mitz@apple.com [Wed, 23 Jan 2008 16:18:24 +0000 (16:18 +0000)]
    - updated results following <trac.webkit.org/projects/webkit/changeset/29721>

        * platform/mac/fast/frames/viewsource-attribute-expected.checksum:
        * platform/mac/fast/frames/viewsource-attribute-expected.png:
        * platform/mac/fast/frames/viewsource-attribute-expected.txt:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@29742 268f45cc-cd09-0410-ab3c-d52691b4dbfc

12 years ago2008-01-23 Alp Toker <alp@atoker.com>
alp@webkit.org [Wed, 23 Jan 2008 15:55:41 +0000 (15:55 +0000)]
2008-01-23  Alp Toker  <alp@atoker.com>

        Rubber-stamped by Mark Rowe.

        Remove whitespace after -I in automake include lists.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@29741 268f45cc-cd09-0410-ab3c-d52691b4dbfc

12 years agoHolger Hans Peter Freyther <holger.freyther@trolltech.com>
hausmann@webkit.org [Wed, 23 Jan 2008 11:16:45 +0000 (11:16 +0000)]
Holger Hans Peter Freyther <holger.freyther@trolltech.com>

* Fix leaking of sub frames (WebCore::Frame). We keep one reference too many.
  This was found while working on the page-cache, other ports are not affected.
* Rubber stamped by Simon

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@29740 268f45cc-cd09-0410-ab3c-d52691b4dbfc

12 years agoHolger Hans Peter Freyther <holger.freyther@trolltech.com>
hausmann@webkit.org [Wed, 23 Jan 2008 11:14:06 +0000 (11:14 +0000)]
Holger Hans Peter Freyther <holger.freyther@trolltech.com>

* Load images incrementally. This will show warnings on the console
  and we want to fix them for Qt4.4. If that happens we have to remove
  the comment from the enum inside the ImageDecoderQt.cpp

* Rubber stamped by Simon

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@29739 268f45cc-cd09-0410-ab3c-d52691b4dbfc

12 years agoRohan McGovern <rohan.mcgovern@trolltech.com>
hausmann@webkit.org [Wed, 23 Jan 2008 11:11:27 +0000 (11:11 +0000)]
Rohan McGovern <rohan.mcgovern@trolltech.com>

Fix Qtopia compilation with QT_NO_TOOLTIP

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@29738 268f45cc-cd09-0410-ab3c-d52691b4dbfc

12 years agoGeir Vattekar <gvatteka@trolltech.com>
hausmann@webkit.org [Wed, 23 Jan 2008 11:11:13 +0000 (11:11 +0000)]
Geir Vattekar <gvatteka@trolltech.com>

Doc: Replaced \code with snippets in the docs

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@29737 268f45cc-cd09-0410-ab3c-d52691b4dbfc

12 years agoLars Knoll <lars@trolltech.com>
hausmann@webkit.org [Wed, 23 Jan 2008 11:10:57 +0000 (11:10 +0000)]
Lars Knoll <lars@trolltech.com>

Fix rendering of the Scrollbar as well as mouse handling for some styles.

The QStyle expects that that painter is set up to clip to the scrollbar bounds
and some of the styles seem to paint somewhat outside of these bounds. Clipping to
the scrollbar bounds before drawing removes some artifacts.

Also set m_opt.rect.topLeft to (0/0) in the mouse handlers. Fixes some issues with
the oxygen style.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@29736 268f45cc-cd09-0410-ab3c-d52691b4dbfc

12 years agoMichael Brasser <michael.brasser@trolltech.com>
hausmann@webkit.org [Wed, 23 Jan 2008 09:55:46 +0000 (09:55 +0000)]
Michael Brasser <michael.brasser@trolltech.com>

Rendering fixes for Qtopia.

Make the focus ring work with line breaks (until we have QPainterPath::simplify) and
call QStyle with better arguments when painting themed elements.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@29735 268f45cc-cd09-0410-ab3c-d52691b4dbfc

12 years agoHolger Hans Peter Freyther <holger.freyther@trolltech.com>
hausmann@webkit.org [Wed, 23 Jan 2008 09:55:28 +0000 (09:55 +0000)]
Holger Hans Peter Freyther <holger.freyther@trolltech.com>

* FormData::flatten does not include to be uploaded files. Create our own QIODevice
  for the QNetworkAccessManager and feed the complete content of the FormData* to
  the (HTTP) backend.
* Try to optimize the ::readData implementation to fill the buffer as best as possible

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@29734 268f45cc-cd09-0410-ab3c-d52691b4dbfc

12 years agoHolger Hans Peter Freyther <holger.freyther@trolltech.com>
hausmann@webkit.org [Wed, 23 Jan 2008 09:55:08 +0000 (09:55 +0000)]
Holger Hans Peter Freyther <holger.freyther@trolltech.com>

* Coding-Style fixes

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@29733 268f45cc-cd09-0410-ab3c-d52691b4dbfc

12 years agoMichael Brasser <michael.brasser@trolltech.com>
hausmann@webkit.org [Wed, 23 Jan 2008 09:48:22 +0000 (09:48 +0000)]
Michael Brasser <michael.brasser@trolltech.com>

use PopupClient's font for popup menu. Otherwise, in Qtopia, the popup's font is much bigger (QApplication::font()?) and the text doesn't fit.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@29732 268f45cc-cd09-0410-ab3c-d52691b4dbfc

12 years agoBenjamin Meyer <bmeyer@trolltech.com>
hausmann@webkit.org [Wed, 23 Jan 2008 09:44:09 +0000 (09:44 +0000)]
Benjamin Meyer <bmeyer@trolltech.com>

Tweak key presses even to match commonly expected behavior
- space key == page down
- page down moved down not a page, but slightly less then a page so you don't loose your spot when reading.
- Use font height rather then a hard coded "10" for left, right, up, down
- Ctrl-Up moves to the top of the page
- Ctrl-Down move to the bottom of the page
- Backspace == GoBack
- Shift-Backspace == GoForward

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@29731 268f45cc-cd09-0410-ab3c-d52691b4dbfc

12 years agoImplemented FrameLoaderClient::startDownload() and FrameLoaderClient::download().
hausmann@webkit.org [Wed, 23 Jan 2008 09:43:52 +0000 (09:43 +0000)]
Implemented FrameLoaderClient::startDownload() and FrameLoaderClient::download().

Added two signals to QWebPage to handle downloading of links and handling of
unsupported content.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@29730 268f45cc-cd09-0410-ab3c-d52691b4dbfc

12 years agoMichael Goddard <michael.goddard@trolltech.com>
hausmann@webkit.org [Wed, 23 Jan 2008 09:39:04 +0000 (09:39 +0000)]
Michael Goddard <michael.goddard@trolltech.com>

Reworked the JavaScriptCore Qt bindings:

* Add initial support for string and variant arrays, as well
  as sub QObjects in the JS bindings.

* Don't expose fields marked as not scriptable by moc.

* Add support for dynamic properties and accessing named
  QObject children of an object (like QtScript and older
  IE DOM style JS).
* Add support for custom toString methods.

* Fine tune some bindings to be closer to QtScript.
  Make void functions return undefined, and empty/
  null QStrings return a zero length string.

* Create framework for allowing more direct method calls.
  Since RuntimeMethod doesn't allow us to add additional
  methods/properties to a function, add these classes.
  Start prototyping object.signal.connect(...).

* Add signal support to the Qt bindings.
  Allow connecting to signals (object.signal.connect(slot)),
  disconnecting, and emitting signals.  Currently chooses
  the first signal that matches the name, so this will need
  improvement.

* Add property names, and resolve signals closer to use.
  Enumerating properties now returns some of the Qt properties
  and signals.  Slots and methods aren't quite present.  Also,
  resolve signal connections etc. closer to the time of use, so
  we can do more dynamic resolution based on argument type etc.
  Still picks the first one with the same name, at the moment.

* Make signature comparison code consistent.
  Use the same code for checking meta signatures in
  the method and fallback getters, and avoid a
  QByteArray construction when we can.

* Fix minor memory leak, and handle pointers better.
  Delete the private object in the dtors, and use RefPtrs
  for holding Instances etc.

* Handle method lookup better.
  Allow invocation time method lookup based on the arguments,
  which is closer to QtScript behaviour.  Also, cache the
  method lists and delete them in the QtClass dtor (stops
  a memory leak).

* Improve JS to Qt data type conversions.
  Add some support for Date & RegExp JS objects,
  and provide some metrics on the quality of the
  conversion.

* A couple of fixes for autotest failures.
  Better support for converting lists, read/write only
  QMetaProperty support, modified slot search order...)

* Update JS DRT controller for Qt JS binding changes.
  There were two functions that needed some changes
  so that the layout tests would work, so this makes
  a few tests pass again.

* Bump the timeout for layout tests up to 11s.
  At least some tests have an internal timeout of
  10 seconds, so make the waitUntilDone approach
  wait at least 11s.  fast/dom/open-and-close-by-DOM.html
  is one of these - now the failure message is more
  accurate.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@29729 268f45cc-cd09-0410-ab3c-d52691b4dbfc

12 years ago Reviewed by Adele Peterson.
mitz@apple.com [Wed, 23 Jan 2008 04:01:09 +0000 (04:01 +0000)]
    Reviewed by Adele Peterson.

        - fix http://bugs.webkit.org/show_bug.cgi?id=16905
          <rdar://problem/5692407> REGRESSION (3.0.4-TOT): "menu" pseudocolor is badly chosen

        The regression test for this is fast/css/css2-system-color.html which
        is currently disabled.

        * rendering/RenderThemeMac.mm:
        (WebCore::menuBackgroundColor): Added. Uses HIThemeDrawMenuBackground
        to draw the menu item background into a bitmap graphics context and
        returns the color of the pixel at (0, 0).
        (WebCore::RenderThemeMac::systemColor): Changed to call
        menuBackgroundColor for the CSS2 menu color.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@29728 268f45cc-cd09-0410-ab3c-d52691b4dbfc

12 years ago Reviewed by Oliver Hunt.
darin@apple.com [Wed, 23 Jan 2008 00:29:49 +0000 (00:29 +0000)]
    Reviewed by Oliver Hunt.

        - turn full repainting back on for <canvas> until we get incremental
          repaint working properly

        Ollie and I came up with a plan for testing this, but for now we should
        just fall back on the "repaint everything" behavior we have always had.

        * html/HTMLCanvasElement.cpp:
        (WebCore::HTMLCanvasElement::willDraw): Just call repaint.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@29727 268f45cc-cd09-0410-ab3c-d52691b4dbfc

12 years agoJavaScriptCore:
andersca@apple.com [Tue, 22 Jan 2008 23:16:37 +0000 (23:16 +0000)]
JavaScriptCore:

        Reviewed by Darin and Adam.

        <rdar://problem/5688975>
        div element on microsoft site has wrong left offset.

        Return true even if NPN_GetProperty returns null or undefined. This matches Firefox
        (and is what the Silverlight plug-in expects).

        * bindings/NP_jsobject.cpp:
        (_NPN_GetProperty):

WebKitTools:

        Reviewed by Darin and Adam.

        <rdar://problem/5688975>
        div element on microsoft site has wrong left offset.

        Add new method for testing the return vale of NPN_GetProperty.

        * DumpRenderTree/TestNetscapePlugIn.subproj/PluginObject.cpp:
        (pluginInvoke):

LayoutTests:

        Reviewed by Darin and Adam.

        <rdar://problem/5688975>
        div element on microsoft site has wrong left offset.

        Add test.

        * plugins/netscape-get-property-return-value-expected.txt: Added.
        * plugins/netscape-get-property-return-value.html: Added.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@29726 268f45cc-cd09-0410-ab3c-d52691b4dbfc

12 years ago2008-01-22 Alp Toker <alp@atoker.com>
alp@webkit.org [Tue, 22 Jan 2008 21:23:44 +0000 (21:23 +0000)]
2008-01-22  Alp Toker  <alp@atoker.com>

        GTK+/qmake build fix.

        * WebCore.pro:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@29725 268f45cc-cd09-0410-ab3c-d52691b4dbfc

12 years ago Reviewed by Adam Roben.
ap@webkit.org [Tue, 22 Jan 2008 20:46:16 +0000 (20:46 +0000)]
    Reviewed by Adam Roben.

        <rdar://problem/5670257> editing/selection/4895428-3.html makes editing/selection/5131716-2.html fail

        * DumpRenderTree/win/EventSender.cpp: (makeEventSender): Reset static variables for a new test.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@29724 268f45cc-cd09-0410-ab3c-d52691b4dbfc

12 years ago2008-01-22 Christian Dywan <christian@imendio.com>
alp@webkit.org [Tue, 22 Jan 2008 20:45:18 +0000 (20:45 +0000)]
2008-01-22  Christian Dywan  <christian@imendio.com>

        Reviewed by Alp Toker.

        [GTK] API: WebKitWebSettings is not usable
        http://bugs.webkit.org/show_bug.cgi?id=16219

        Implement WebKitWebSettings.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@29723 268f45cc-cd09-0410-ab3c-d52691b4dbfc

12 years ago Fix <rdar://5693599> REGRESSION (r29369): scroll bars are drawing when they shouldn...
aroben@apple.com [Tue, 22 Jan 2008 19:33:26 +0000 (19:33 +0000)]
 Fix <rdar://5693599> REGRESSION (r29369): scroll bars are drawing when they shouldn't be on an ad on cnn.com

 WebCore:

         Add HTMLFrameOwnerElement::scrollingMode

         This new virtual method is overridden by HTMLFrameElementBase to
         provide the scrolling mode set on the frame or iframe element. Object
         elements always have scrolling=auto, so there's no need to override
         this method for HTMLPlugInElement.

         Reviewed by Hyatt.

         * html/HTMLFrameElementBase.h: Declare scrollingMode as virtual.
         * html/HTMLFrameOwnerElement.h:
         (WebCore::HTMLFrameOwnerElement::scrollingMode): Added.

 WebKit/win:

         Fix <rdar://5693599> REGRESSION (r29369): scroll bars are drawing when they shouldn't be on an ad on cnn.com

         Now that we have multiple FrameViews per Frame on Windows, we need to
         make sure each FrameView gets its scrolling mode set correctly. On Mac
         this works because WebFrameView holds the scrolling mode and a single
         WebFrameView is shared by many FrameViews. On Windows, we have no such
         shared object, so we just ask the Frame's owner element for the
         scrolling mode.

         Tests: fast/frames/frame-scrolling-attribute.html
                fast/frames/iframe-scrolling-attribute.html

         Reviewed by Hyatt.

         * WebFrame.cpp:
         (WebFrame::transitionToCommittedForNewPage): Set the scrolling mode
         on the FrameView based on the scrolling mode of the Frame's owner
         element.

 LayoutTests:

         Tests for <rdar://5693599> REGRESSION (r29369): scroll bars are drawing when they shouldn't be on an ad on cnn.com

         Reviewed by Hyatt.

         * fast/frames/frame-scrolling-attribute.html: Added.
         * fast/frames/iframe-scrolling-attribute.html: Added.
         * platform/mac/fast/frames/frame-scrolling-attribute-expected.checksum: Added.
         * platform/mac/fast/frames/frame-scrolling-attribute-expected.png: Added.
         * platform/mac/fast/frames/frame-scrolling-attribute-expected.txt: Added.
         * platform/mac/fast/frames/iframe-scrolling-attribute-expected.checksum: Added.
         * platform/mac/fast/frames/iframe-scrolling-attribute-expected.png: Added.
         * platform/mac/fast/frames/iframe-scrolling-attribute-expected.txt: Added.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@29722 268f45cc-cd09-0410-ab3c-d52691b4dbfc

12 years ago Fix for <rdar://problem/5698481> REGRESSION: Web Inspector source view gutter...
hyatt@apple.com [Tue, 22 Jan 2008 17:36:26 +0000 (17:36 +0000)]
    Fix for <rdar://problem/5698481> REGRESSION: Web Inspector source view gutter double divider and no padding

        Now that cellpadding cascades properly on <td>s, we need to make the view source padding rules in the user
        agent sheet have !important in order to override author settings.

        Reviewed by Tim

        * css/view-source.css:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@29721 268f45cc-cd09-0410-ab3c-d52691b4dbfc

12 years agoReviewed by Alp.
zimmermann@webkit.org [Tue, 22 Jan 2008 16:09:01 +0000 (16:09 +0000)]
Reviewed by Alp.
Allow to use SVGPaintServerSolid without RenderObject/RenderStyle. Used later to draw SVG Fonts in HTML.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@29720 268f45cc-cd09-0410-ab3c-d52691b4dbfc

12 years agoReviewed by Alp.
zimmermann@webkit.org [Tue, 22 Jan 2008 14:57:47 +0000 (14:57 +0000)]
Reviewed by Alp.

Initialize missing variables, m_lineGap / m_lineSpacing.
Only noticeable when trying to render HTML text using SVG Fonts (which is not possible with trunk, but with my local version).

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@29719 268f45cc-cd09-0410-ab3c-d52691b4dbfc

12 years ago2008-01-22 Alp Toker <alp@atoker.com>
alp@webkit.org [Tue, 22 Jan 2008 13:53:37 +0000 (13:53 +0000)]
2008-01-22  Alp Toker  <alp@atoker.com>

        Reviewed by Nikolas.

        Fix Cairo SVG fonts following changes in r29700.

        Set the font description's details as needed in FontPlatformData and
        eliminate GraphicsContext::setPlatformFont() since it was redundant
        and assumed all fonts were Cairo fonts.

        * platform/graphics/GraphicsContext.cpp:
        * platform/graphics/cairo/GraphicsContextCairo.cpp:
        * platform/graphics/gtk/FontPlatformDataGtk.cpp:
        (WebCore::FontPlatformData::FontPlatformData):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@29718 268f45cc-cd09-0410-ab3c-d52691b4dbfc

12 years agoReviewed by Eric.
zimmermann@webkit.org [Tue, 22 Jan 2008 11:10:45 +0000 (11:10 +0000)]
Reviewed by Eric.

- fix http://bugs.webkit.org/show_bug.cgi?id=16967
<rdar://problem/5699344> Reproducible crash when navigating back to a page using SVG fonts

Test: svg/custom/font-platformDestroy-crash.svg

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@29717 268f45cc-cd09-0410-ab3c-d52691b4dbfc

12 years agoLars Knoll <lars@trolltech.com>
hausmann@webkit.org [Tue, 22 Jan 2008 10:03:54 +0000 (10:03 +0000)]
Lars Knoll <lars@trolltech.com>

setup a family for cursive and fantasy fonts as well.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@29716 268f45cc-cd09-0410-ab3c-d52691b4dbfc

12 years agoLars Knoll <lars@trolltech.com>
hausmann@webkit.org [Tue, 22 Jan 2008 10:03:40 +0000 (10:03 +0000)]
Lars Knoll <lars@trolltech.com>

cleanup the CookieJarQt implementation and implement cookiesEnabled.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@29715 268f45cc-cd09-0410-ab3c-d52691b4dbfc

12 years agoSimon Hausmann <hausmann@webkit.org>
hausmann@webkit.org [Tue, 22 Jan 2008 10:03:26 +0000 (10:03 +0000)]
Simon Hausmann <hausmann@webkit.org>

Don't compile the ICO plugin when building against Qt >= 4.4

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@29714 268f45cc-cd09-0410-ab3c-d52691b4dbfc

12 years agoBenjamin Meyer <bmeyer@trolltech.com>
hausmann@webkit.org [Tue, 22 Jan 2008 10:03:14 +0000 (10:03 +0000)]
Benjamin Meyer <bmeyer@trolltech.com>

Implement ChromeClientQt::setToolTip
Implement ChromeClientQt::mouseDidMoveOverElement

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@29713 268f45cc-cd09-0410-ab3c-d52691b4dbfc

12 years agoLars Knoll <lars@trolltech.com>
hausmann@webkit.org [Tue, 22 Jan 2008 08:38:52 +0000 (08:38 +0000)]
Lars Knoll <lars@trolltech.com>

Ported the font implementation to use the new QFont features of Qt 4.4.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@29712 268f45cc-cd09-0410-ab3c-d52691b4dbfc

12 years ago Reviewed by Nikolas & Oliver.
eric@webkit.org [Tue, 22 Jan 2008 06:48:57 +0000 (06:48 +0000)]
    Reviewed by Nikolas & Oliver.

        Tested by svg/custom/svg-features.html

        * dom/DOMImplementation.cpp:
        (WebCore::isSVG10Feature): use ENABLE(SVG_*) conditionals
        (WebCore::isSVG11Feature): use ENABLE(SVG_*) conditionals

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@29711 268f45cc-cd09-0410-ab3c-d52691b4dbfc

12 years agoJavaScriptCore:
ggaren@apple.com [Tue, 22 Jan 2008 06:18:10 +0000 (06:18 +0000)]
JavaScriptCore:

        Reviewed by Maciej Stachowiak.

        Fixed http://bugs.webkit.org/show_bug.cgi?id=16909
        REGRESSION: Amazon.com crash (ActivationImp)

        (and a bunch of other crashes)

        Plus, a .7% SunSpider speedup to boot.

        Replaced the buggy currentExec and savedExec mechanisms with an
        explicit ExecState stack.

        * kjs/collector.cpp:
        (KJS::Collector::collect): Explicitly mark the ExecState stack.

        (KJS::Collector::reportOutOfMemoryToAllExecStates): Slight change in
        behavior: We no longer throw an exception in any global ExecStates,
        since global ExecStates are more like pseudo-ExecStates, and aren't
        used for script execution. (It's unclear what would happen if you left
        an exception waiting around in a global ExecState, but it probably
        wouldn't be good.)

WebCore:

        Reviewed by Maciej Stachowiak.

        Adapted WebCore to the fix for http://bugs.webkit.org/show_bug.cgi?id=16909
        REGRESSION: Amazon.com crash (ActivationImp)

        * bindings/js/kjs_proxy.cpp:
        (WebCore::KJSProxy::~KJSProxy): No convenient way to make this assertion
        anymore. (It wasn't firing for anyone, anyway, so it's no big loss.)

        * bindings/objc/WebScriptObject.mm:
        (+[WebScriptObject throwException:]): Use the ExecState stack, instead
        of currentExec.
        (-[WebScriptObject setException:]): ditto. Also, a slight change in
        behavior: If no ExecStates are active, we no longer throw an exception
        in the global ExecState. The JavaScriptCore ChangeLog explains why.
        This also matches the behavior of +throwException.

LayoutTests:

        Layout test for http://bugs.webkit.org/show_bug.cgi?id=16909
        REGRESSION: Amazon.com crash (ActivationImp)

        * fast/js/exec-state-marking-expected.txt: Added.
        * fast/js/exec-state-marking.html: Added.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@29710 268f45cc-cd09-0410-ab3c-d52691b4dbfc

12 years agoNot reviewed. Try to fix Qt build, after the rmdir() fixes.
zimmermann@webkit.org [Tue, 22 Jan 2008 00:52:38 +0000 (00:52 +0000)]
Not reviewed. Try to fix Qt build, after the rmdir() fixes.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@29709 268f45cc-cd09-0410-ab3c-d52691b4dbfc

12 years agoNot reviewed. Another Wx build fix.
zimmermann@webkit.org [Tue, 22 Jan 2008 00:49:41 +0000 (00:49 +0000)]
Not reviewed. Another Wx build fix.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@29708 268f45cc-cd09-0410-ab3c-d52691b4dbfc

12 years ago - try to fix Qt build
darin@apple.com [Tue, 22 Jan 2008 00:30:41 +0000 (00:30 +0000)]
    - try to fix Qt build

        * platform/qt/FileSystemQt.cpp:
        (WebCore::deleteEmptyDirectory): QDir::root().rmdir() instead of QDir::rmdir().

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@29707 268f45cc-cd09-0410-ab3c-d52691b4dbfc

12 years agoNot reviewed. Yet another win/gtk build fix.
zimmermann@webkit.org [Tue, 22 Jan 2008 00:05:28 +0000 (00:05 +0000)]
Not reviewed. Yet another win/gtk build fix.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@29706 268f45cc-cd09-0410-ab3c-d52691b4dbfc

12 years agoNot reviewed. Another build fix for Gtk/Linux & Wx/Mac.
zimmermann@webkit.org [Mon, 21 Jan 2008 23:43:32 +0000 (23:43 +0000)]
Not reviewed. Another build fix for Gtk/Linux & Wx/Mac.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@29705 268f45cc-cd09-0410-ab3c-d52691b4dbfc

12 years agoNot reviewed. Build fix for Qt/Gtk & Wx.
zimmermann@webkit.org [Mon, 21 Jan 2008 23:26:06 +0000 (23:26 +0000)]
Not reviewed. Build fix for Qt/Gtk & Wx.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@29704 268f45cc-cd09-0410-ab3c-d52691b4dbfc

12 years agoNot reviewed. Build fix for ports that don't build SVG: wx.
zimmermann@webkit.org [Mon, 21 Jan 2008 23:16:59 +0000 (23:16 +0000)]
Not reviewed. Build fix for ports that don't build SVG: wx.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@29703 268f45cc-cd09-0410-ab3c-d52691b4dbfc

12 years agoNot reviewed. (First) Build fix for Qt/Gtk.
zimmermann@webkit.org [Mon, 21 Jan 2008 23:12:25 +0000 (23:12 +0000)]
Not reviewed. (First) Build fix for Qt/Gtk.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@29702 268f45cc-cd09-0410-ab3c-d52691b4dbfc

12 years ago2008-01-21 Jan Michael Alonzo <jmalonzo@unpluggable.com>
alp@webkit.org [Mon, 21 Jan 2008 23:06:23 +0000 (23:06 +0000)]
2008-01-21  Jan Michael Alonzo  <jmalonzo@unpluggable.com>

        Reviewed by Alp Toker.

        http://bugs.webkit.org/show_bug.cgi?id=16955
        Get errors when cross-compile webkit-gtk

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@29701 268f45cc-cd09-0410-ab3c-d52691b4dbfc

12 years agoReviewed by Eric. Older parts reviewed by Dan.
zimmermann@webkit.org [Mon, 21 Jan 2008 22:56:53 +0000 (22:56 +0000)]
Reviewed by Eric. Older parts reviewed by Dan.

Fixes: http://bugs.webkit.org/show_bug.cgi?id=16880 (SVGCSSFontFace should die, instead integrate within the FontCache.)
Fixes: http://bugs.webkit.org/show_bug.cgi?id=16784 (In-document fonts work only as the first child to font-face-src)

Proper integration of SVG Fonts within the existing custom font concept, removing the need for a custom SVGCSSFontFace.
Integrate within the CSSFontFaceSource logic for caching support, as well as the framework for external SVG Fonts.

Proper handling of missing glyphs, using <missing-glyph> element. If that is not specified in a font, fallback to non-SVG font rendering.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@29700 268f45cc-cd09-0410-ab3c-d52691b4dbfc

12 years ago2008-01-21 Alp Toker <alp@atoker.com>
alp@webkit.org [Mon, 21 Jan 2008 22:43:52 +0000 (22:43 +0000)]
2008-01-21  Alp Toker  <alp@atoker.com>

        GTK+ build fix for breakage introduced in r29698.

        * WebCoreSupport/ChromeClientGtk.h:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@29699 268f45cc-cd09-0410-ab3c-d52691b4dbfc

12 years agoWebCore:
darin@apple.com [Mon, 21 Jan 2008 22:23:28 +0000 (22:23 +0000)]
WebCore:

        Reviewed by John Sullivan.

        - fix <rdar://problem/5644324> Delegate-less WebKit clients should have no databases

        - remove the default database quota setting

        - simplify the quota-related client calls by merging the one for a new database with
          the one for an existing database, adding a feature where you can get details about
          a database being created using the same functions that work on the other databases,
          and removing the parameters and return values, instead having clients use a function
          to set the quota

        - fix unsafe multi-thread access to the database tracker's quota map
        - fix bug in deleteAllDatabases where it would iterate a HashMap while modifying it

        The tracker database is now only created when we set the quota for a database origin.
        Thus asking for info about databases won't cause anything to be written to disk.

        * WebCore.base.exp: Updated.

        * bindings/js/GCController.cpp: Added an #if to get rid of an unused function warning.

        * page/Chrome.cpp: Removed database-related functions. There's no problem having the
        code deal directly with the client.
        * page/Chrome.h: Ditto. Also made Chrome inherit from Noncopyable.

        * page/ChromeClient.h: Replaced the two quota-related functions with a single one.
        The details about the state of databases in the origin are now available by asking
        for database details. There's also no need to pass the security origin, since
        it's easy to get that from the frame's document.

        * page/Settings.cpp: Removed the default quota setting.
        * page/Settings.h: Ditto.

        * platform/FileSystem.h: Added deleteEmptyDirectory.
        * platform/gtk/FileSystemGtk.cpp:
        (WebCore::deleteEmptyDirectory): Added.
        * platform/posix/FileSystemPOSIX.cpp:
        (WebCore::deleteEmptyDirectory): Added.
        * platform/qt/FileSystemQt.cpp:
        (WebCore::deleteEmptyDirectory): Added.
        * platform/win/FileSystemWin.cpp:
        (WebCore::deleteEmptyDirectory): Added.
        * platform/wx/FileSystemWx.cpp:
        (WebCore::deleteEmptyDirectory): Added placeholder.

        * storage/DatabaseDetails.h: Removed the isValid() function since its name is
        confusing -- we removed our other isValid() functions. For the few callers that
        need this, it's fine to just check name().isEmpty(). Made the member functions
        all const.

        * storage/DatabaseTracker.cpp:
        (WebCore::DatabaseTracker::DatabaseTracker): Removed m_defaultQuota initialization.
        Added m_proposedDatabase and m_thread.
        (WebCore::DatabaseTracker::setDatabaseDirectoryPath): Got rid of code that would
        open the tracker database here. This might slightly speed up launch time, since
        we won't do the I/O until we have to, and before we were actually creating an SQL
        database in this code that's run when we go to the first webpage.
        (WebCore::DatabaseTracker::trackerDatabasePath): Added.
        (WebCore::DatabaseTracker::openTrackerDatabase): Added a boolean parameter telling
        this function whether to create the database. Made this function safe to call if
        the database is already open. Also made the function fail quietly if the path has
        not been set yet.
        (WebCore::DatabaseTracker::canEstablishDatabase): Got rid of the call to the
        establishEntryForOrigin function. The origin is now put in the tracker database
        when the quota is set to a non-zero value. When judging if there's enough space
        for the new database, require at least one byte even if estimatedSize is 0, and
        check for overflow. Also added code here to populate the origins map, which
        guarantees it will be ready when the database uses it on another thread later.
        Also changed this to call the new ChromeClient function.
        (WebCore::DatabaseTracker::hasEntryForDatabase): Added code to open the
        tracker database, since that's no longer done by setDatabaseDirectoryPath.
        (WebCore::DatabaseTracker::originPath): Added.
        (WebCore::DatabaseTracker::fullPathForDatabase): Added code so that this will
        return a null string for the proposed database if called from within the
        ChromeClient function. Also switched from empty string to null string for the
        error cases.
        (WebCore::DatabaseTracker::populateOrigins): Added code to open the tracker
        database.
        (WebCore::DatabaseTracker::databaseNamesForOrigin): Ditto.
        (WebCore::DatabaseTracker::detailsForNameAndOrigin): Ditto. Also added code that
        will return the details of the proposed database if called from within the
        ChromeClient function. This is how the client can learn of the display name and
        the estimated size of the new database.
        (WebCore::DatabaseTracker::setDatabaseDetails): Added code to open the tracker
        database.
        (WebCore::DatabaseTracker::quotaForOrigin): Made this code OK to call on a non-main
        thread by using m_quotaMapGuard to guard access to the map. Other code runs on the
        main thread only, and only functions that write to the map use the lock.
        (WebCore::DatabaseTracker::setQuota): Changed this function so it can insert the
        initial quota as well as updating an existing quota. Added locking since this
        function modifies the quota map. Added code to open the tracker database. Added
        an early exit if the quota is already correct, which is guarantees that if you
        set a quota to 0 it won't trigger creation of a tracker database.
        (WebCore::DatabaseTracker::addDatabase): Added code to open the tracker database.
        (WebCore::DatabaseTracker::deleteAllDatabases): Made a copy of the quota map before
        iterating it to find all the origins. This fixes a problem with the old code where
        it would modify the map while iterating it, which gives assertions in debug builds
        and unpredictable results.
        (WebCore::DatabaseTracker::deleteOrigin): Replaced deleteDatabasesWithOrigin with
        this function. Added code to open the tracker database if needed. Added code to
        delete the origin from the tracker database, and to close the tracker database
        and delete files and directories as needed if we are deleting the final origin.
        (WebCore::DatabaseTracker::deleteDatabase): Added code to open the tracker database
        if needed.

        * storage/DatabaseTracker.h: Renamed databasePath to databaseDirectoryPath for
        clarity, including the data member, and the getter and setter functions.
        Replaced deleteDatabasesWithOrigin with deleteOrigin. Removed the functions
        dealing with default origin quota. There is no default any more; origins start
        with no quota and the client must set a quota. Added trackerDatabasePath and
        originPath helper functions. Added a boolean parameter to openTrackerDatabase
        to tell it whether to create the database or not. Removed the
        establishEntryForOrigin function. Renamed m_originQuotaMap to just m_quotaMap,
        and added m_quotaMapGuard. Added a QuotaMap typedef. Added m_proposedDatabase,
        which holds the origin and details for the current proposed database during
        the client callback function that must decide whether to grant quota. Added a
        m_thread data member for debugging use to assert if functions that can only
        be called on a single thread are misused.

        * storage/SQLTransaction.cpp:
        (WebCore::SQLTransaction::deliverQuotaIncreaseCallback): Changed to call the
        new exceededDatabaseQuota function instead of the old one.

        * svg/graphics/SVGImageEmptyClients.h: Updated for the change to ChromeClient.

WebKit/gtk:

        Reviewed by John Sullivan.

        - updated for changes to database functions

        * WebCoreSupport/ChromeClientGtk.cpp:
        (WebKit::ChromeClient::exceededDatabaseQuota):
        * WebCoreSupport/ChromeClientGtk.h:
        * WebView/webkitprivate.cpp:
        (webkit_init):

WebKit/mac:

        Reviewed by John Sullivan.

        - fix <rdar://problem/5644324> Delegate-less WebKit clients should have no databases
        - add a missing export of WebDatabaseExpectedSizeKey
        - implement deleteOrigin: and remove deleteDatabasesWithOrigin:

        * Storage/WebDatabaseManager.mm:
        (-[WebDatabaseManager detailsForDatabase:withOrigin:]): Updated to check for a null
        name instead of calling isValid().
        (-[WebDatabaseManager deleteOrigin:]): Implemented.
        (WebKitInitializeDatabasesIfNecessary): Updated for name change.

        * Storage/WebDatabaseManagerPrivate.h: Removed deleteDatabasesWithOrigin:.

        * WebCoreSupport/WebChromeClient.h: Updated for changes to ChromeClient.
        * WebCoreSupport/WebChromeClient.mm:
        (WebChromeClient::exceededDatabaseQuota): Replaced the two different client functions
        we had before with a single one.

        * WebKit.exp: Added missing export for WebDatabaseExpectedSizeKey.

        * WebView/WebPreferenceKeysPrivate.h: Removed WebKitDefaultDatabaseQuotaKey.
        * WebView/WebPreferences.m:
        (+[WebPreferences initialize]): Removed the default for WebKitDefaultDatabaseQuotaKey.
        * WebView/WebPreferencesPrivate.h: Removed defaultDatabaseQuota and
        setDefaultDatabaseQuota:.

        * WebView/WebUIDelegatePrivate.h: Replaced the two different database quota delegate
        methods we had before with a single one.

        * WebView/WebView.mm:
        (-[WebView _preferencesChangedNotification:]): Removed the code to set the
        default database origin quota in WebCore::Settings based on WebPreferences.

        * WebView/WebViewInternal.h: Removed delegate method dispatch functions for unusual
        types of parameters that the database UI delegate methods had before.

WebKit/qt:

        Reviewed by John Sullivan.

        - updated for changes to ChromeClient database functions

        * WebCoreSupport/ChromeClientQt.cpp:
        (WebCore::ChromeClientQt::exceededDatabaseQuota):
        * WebCoreSupport/ChromeClientQt.h:

WebKit/win:

        Reviewed by John Sullivan.

        - fix <rdar://problem/5644324> Delegate-less WebKit clients should have no databases
        - fix incorrect value for WebDatabaseDisplayNameKey in header

        * Interfaces/IWebDatabaseManager.idl: Renamed detailsForDatabaseWithOrigin to
        detailsForDatabase. Replaced deleteDatabasesWithOrigin with deleteOrigin.
        Renamed deleteDatabaseWithOrigin to deleteDatabase. Fixed incorrect value for
        WebDatabaseDisplayNameKey.
        * Interfaces/IWebPreferencesPrivate.idl: Removed defaultDatabaseQuota and
        setDefaultDatabaseQuota.
        * Interfaces/IWebUIDelegatePrivate.idl: Replaced the two database quota functions
        with a single one.

        * WebChromeClient.cpp:
        (WebChromeClient::exceededDatabaseQuota): Replaced the two database quota functions
        with a single one.
        * WebChromeClient.h: Ditto.

        * WebDatabaseManager.cpp:
        (WebDatabaseManager::detailsForDatabase): Changed to check for a name of null to
        detect a nonexistent database rather than using isValid.
        (WebDatabaseManager::deleteOrigin): Updated for name change (actually a semantic
        change too, but both have the same name).
        (WebDatabaseManager::deleteDatabase): Updated for name change.
        (WebKitSetWebDatabasesPathIfNecessary): Ditto.

        * WebDatabaseManager.h: Updated for name changes.

        * WebPreferenceKeysPrivate.h: Removed WebKitDefaultDatabaseQuotaKey.
        * WebPreferences.cpp:
        (WebPreferences::initializeDefaultSettings): Removed the default for
        WebKitDefaultDatabaseQuotaKey.
        * WebPreferences.h: Removed defaultDatabaseQuota and setDefaultDatabaseQuota.
        * WebView.cpp:
        (WebView::notifyPreferencesChanged): Removed the code to set the
        default database origin quota in WebCore::Settings based on IWebPreferencesPrivate.

WebKit/wx:

        Reviewed by John Sullivan.

        - updated for changes to ChromeClient database functions

        * WebKitSupport/ChromeClientWx.cpp:
        (WebCore::ChromeClientWx::exceededDatabaseQuota):
        * WebKitSupport/ChromeClientWx.h:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@29698 268f45cc-cd09-0410-ab3c-d52691b4dbfc

12 years agoRubber stamped by Oliver.
zimmermann@webkit.org [Mon, 21 Jan 2008 21:52:09 +0000 (21:52 +0000)]
Rubber stamped by Oliver.
Add missing layout test results.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@29697 268f45cc-cd09-0410-ab3c-d52691b4dbfc

12 years agoWebCore:
hyatt@apple.com [Mon, 21 Jan 2008 21:19:30 +0000 (21:19 +0000)]
WebCore:

        Fix for http://bugs.webkit.org/show_bug.cgi?id=16935

        Fix a bug in layers where the positioned ancestor was being computed incorrectly (the root should not
        automatically be included).

        Reviewed by olliej

        Added fast/layers/positioned-inside-root-with-margins.html

        * rendering/RenderLayer.cpp:
        (WebCore::RenderLayer::stackingContext):
        (WebCore::RenderLayer::enclosingPositionedAncestor):

LayoutTests:

        http://bugs.webkit.org/show_bug.cgi?id=16935

        Reviewed by olliej

        * fast/layers/positioned-inside-root-with-margins.html: Added.
        * platform/mac/fast/layers/positioned-inside-root-with-margins-expected.checksum: Added.
        * platform/mac/fast/layers/positioned-inside-root-with-margins-expected.png: Added.
        * platform/mac/fast/layers/positioned-inside-root-with-margins-expected.txt: Added.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@29696 268f45cc-cd09-0410-ab3c-d52691b4dbfc

12 years ago Adding media/{audio,video}-controls-rendering.html back to the skipped list...
adele@apple.com [Mon, 21 Jan 2008 19:00:08 +0000 (19:00 +0000)]
    Adding media/{audio,video}-controls-rendering.html back to the skipped list, since they're still failing on some machines.

        * platform/win/Skipped:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@29695 268f45cc-cd09-0410-ab3c-d52691b4dbfc

12 years ago Reviewed by Mitz and Adam.
darin@apple.com [Mon, 21 Jan 2008 16:59:24 +0000 (16:59 +0000)]
    Reviewed by Mitz and Adam.

        * Scripts/svn-create-patch: Sort ChangeLog files first. Also slightly improved the
        sorting speed by doing all sort criteria in a single pass instead of three sorts.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@29694 268f45cc-cd09-0410-ab3c-d52691b4dbfc

12 years agoBenjamin Meyer <bmeyer@trolltech.com>
hausmann@webkit.org [Mon, 21 Jan 2008 11:06:20 +0000 (11:06 +0000)]
Benjamin Meyer <bmeyer@trolltech.com>

Change QWebHistoryInterface::addHistoryEntry() from const to non-const

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@29693 268f45cc-cd09-0410-ab3c-d52691b4dbfc

12 years agoAdam Treat <treat@kde.org>
hausmann@webkit.org [Mon, 21 Jan 2008 11:06:06 +0000 (11:06 +0000)]
Adam Treat <treat@kde.org>

* Don't leak the d-pointer in QWebSettings.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@29692 268f45cc-cd09-0410-ab3c-d52691b4dbfc

12 years agoSimon Hausmann <hausmann@webkit.org>
hausmann@webkit.org [Mon, 21 Jan 2008 10:47:15 +0000 (10:47 +0000)]
Simon Hausmann <hausmann@webkit.org>

Update the WebKit version number from WebKit/win/WebKit.vcproj/VERSION.

This has the fortunate side-effect that gmail sends us sensible HTML/JS again :)

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@29691 268f45cc-cd09-0410-ab3c-d52691b4dbfc

12 years agoFix KURL to QUrl conversion.
hausmann@webkit.org [Mon, 21 Jan 2008 10:41:43 +0000 (10:41 +0000)]
Fix KURL to QUrl conversion.

Stick to the encoded version of the URL and in addition to KURL we encode the
characters mentioned in section 2.4.3 of RFC 2396 as QUrl requires these, too.

This fixes fast/css/import-rule-regression-11590.html,
fast/css/import-style-update.html, svg/hixie/processing-model/003.xml and
svg/hixie/processing-model/004.xml.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@29690 268f45cc-cd09-0410-ab3c-d52691b4dbfc

12 years agoFix focus chain handling and cycling through focusable objects (links) using tab...
hausmann@webkit.org [Mon, 21 Jan 2008 10:40:53 +0000 (10:40 +0000)]
Fix focus chain handling and cycling through focusable objects (links) using tab/backtab.

* Fix GraphicsContext::drawFocusRing to also draw single focus rects.
* Implemented QWebPage::focusNextPrevChild by sending fake tab/shift-tab events
  and make the return value depend on whether we successfully determined a focusable
  node or not.
* Changed QWebView::focusNextPrevChild() to call the base QWidget implementation correctly
  if we could not handle the focus chain ourselves.
* Changed the focus policy of QWebView to correctly use WheelFocus instead of ClickFocus.
* Made ChromeClientQt::canTakeFocus() and takeFocus() dummy method since they are only
  used to control the situation of stepping out of the focus chain inside the page.
* Made inclusion of links in the focus chain configurable through QWebSettings::LinksIncludedInFocusChain.
  The layout tests expect this to be disabled but for the user it seems sensible to have it
  on by default, hence the default in qwebsettings.cpp

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@29689 268f45cc-cd09-0410-ab3c-d52691b4dbfc

12 years agoFix access key support and fast/forms/legend-access-key.html
hausmann@webkit.org [Mon, 21 Jan 2008 10:40:16 +0000 (10:40 +0000)]
Fix access key support and fast/forms/legend-access-key.html

SVN revision 26664 changed the default access key for the non-mac build to Alt
while this test relies on Ctrl.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@29688 268f45cc-cd09-0410-ab3c-d52691b4dbfc

12 years agoFix error reporting when parsing X(HT)ML fragments.
hausmann@webkit.org [Mon, 21 Jan 2008 10:39:57 +0000 (10:39 +0000)]
Fix error reporting when parsing X(HT)ML fragments.

We use the regular XMLTokenizer write()/.../end() sequence to parse those
fragments, but we should not report any parsing errors inline inside the
content. Instead we should just return the error and the DOM layer will take
care of throwing an exception.

This fixes fast/innerHTML/innerHTML-changing-document-properties.xhtml

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@29687 268f45cc-cd09-0410-ab3c-d52691b4dbfc

12 years agoHolger Freyther <holger.freyther@trolltech.com>
hausmann@webkit.org [Mon, 21 Jan 2008 09:10:10 +0000 (09:10 +0000)]
Holger Freyther <holger.freyther@trolltech.com>

Fix some issues with redirections.

* Ensure that we deliver the response to the ResourceHandleClient only once by
  setting m_responseSent back to false only before calling start() for the
  redirection
* Added 307 as another HTTP status code that causes a redirection after POST to
  become get, just like in MainResourceLoader::isPostOrRedirectAfterPost
* Also set the HTTP method on the ResourceRequest to GET before passing it to
  willSendRequest.
* willSendRequest's newRequest argument is actually an in/out argument and could be
  modified in theory, so set m_request accordingly after the call.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@29686 268f45cc-cd09-0410-ab3c-d52691b4dbfc

12 years agoHolger Freyther <holger.freyther@trolltech.com>
hausmann@webkit.org [Mon, 21 Jan 2008 09:09:44 +0000 (09:09 +0000)]
Holger Freyther <holger.freyther@trolltech.com>

Fixed WebCore::cookies() to return all cookies, not only the first one.

Fixes login into mail.yahoo.com.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@29685 268f45cc-cd09-0410-ab3c-d52691b4dbfc

12 years ago Try to make the test less timing dependent to fix occasional test bot failures.
antti@apple.com [Mon, 21 Jan 2008 09:01:09 +0000 (09:01 +0000)]
    Try to make the test less timing dependent to fix occasional test bot failures.

        * media/video-seek-past-end-paused-expected.txt:
        * media/video-seek-past-end-paused.html:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@29684 268f45cc-cd09-0410-ab3c-d52691b4dbfc

12 years ago2008-01-20 Alp Toker <alp@atoker.com>
alp@webkit.org [Mon, 21 Jan 2008 07:07:20 +0000 (07:07 +0000)]
2008-01-20 Alp Toker <alp@atoker.com>

        Reviewed by Mark Rowe.

        Attach the widget's style to its window and set the background to the
        base background to avoid black flicker when repainting. This is
        similar to what GtkTextView does.

        * WebView/webkitwebview.cpp:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@29683 268f45cc-cd09-0410-ab3c-d52691b4dbfc

12 years agoRemove code bracketed by REMOVE_SAFARI_DOM_TREE_DEBUG_ITEM as we are no longer intere...
mrowe@apple.com [Mon, 21 Jan 2008 05:34:54 +0000 (05:34 +0000)]
Remove code bracketed by REMOVE_SAFARI_DOM_TREE_DEBUG_ITEM as we are no longer interested in supporting Safari 2 with TOT WebKit.

Reviewed by Dan Bernstein.

* WebKit.xcodeproj/project.pbxproj: Don't define REMOVE_SAFARI_DOM_TREE_DEBUG_ITEM as it is no longer used.
* mac/WebView/WebView.mm:
(+[WebView initialize]):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@29682 268f45cc-cd09-0410-ab3c-d52691b4dbfc

12 years agoFix http://bugs.webkit.org/show_bug.cgi?id=16816 , rdar://problem/5682985
oliver@apple.com [Mon, 21 Jan 2008 04:30:09 +0000 (04:30 +0000)]
Fix bugs.webkit.org/show_bug.cgi?id=16816 , rdar://problem/5682985

Reviewed by Mitz

Correctly trigger willPerformDragDestinationAction when a drop causes a
load to occur.  The logic that originally did this was lost during the
great drag migration of '07.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@29681 268f45cc-cd09-0410-ab3c-d52691b4dbfc

12 years ago - update Tiger-specific results for r29667
mitz@apple.com [Mon, 21 Jan 2008 04:04:41 +0000 (04:04 +0000)]
    - update Tiger-specific results for r29667

        * platform/mac/editing/pasteboard/paste-RTFD-expected.txt:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@29680 268f45cc-cd09-0410-ab3c-d52691b4dbfc

12 years ago Reviewed by Darin Adler.
mitz@apple.com [Mon, 21 Jan 2008 02:01:26 +0000 (02:01 +0000)]
    Reviewed by Darin Adler.

        - fix http://bugs.webkit.org/show_bug.cgi?id=16951
          Assertion failure in FrameView::scheduleRelayout (!m_frame->document() || !m_frame->document()->inPageCache()) when going back from a page with a focused popup

        * dom/Document.cpp:
        (WebCore::Document::setFocusedNode): Bail out if the document is in the
        page cache. Documents in the back/forward cache are "frozen" and should
        not change state.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@29679 268f45cc-cd09-0410-ab3c-d52691b4dbfc

12 years agoWebCore:
weinig@apple.com [Sun, 20 Jan 2008 23:18:32 +0000 (23:18 +0000)]
WebCore:

        Reviewed by Sam Weinig.

        Fix for http://bugs.webkit.org/show_bug.cgi?id=16775

        We now use frame()->loader()->url() for postMessage, preventing a
        malicious sender from overwriting the uri property (using a <base> tag,
        for example). Also, use frame->loader()->url().host() instead of
        instead of document()->SecurityOrigin()->domain() to reflect a recent
        clarification in the HTML5 spec.

        Tests: http/tests/security/postMessage/domain-affected-by-document-domain.html
               http/tests/security/postMessage/domain-and-uri-unaffected-by-base-tag.html
               http/tests/security/postMessage/javascript-page-still-sends-domain.html

        * bindings/js/JSDOMWindowCustom.cpp:
        (WebCore::JSDOMWindow::postMessage):

LayoutTests:

        Reviewed by Sam Weinig.

        Tests for http://bugs.webkit.org/show_bug.cgi?id=16775

        * http/tests/security/postMessage: Added.
        * http/tests/security/postMessage/domain-and-uri-unaffected-by-base-tag-expected.txt: Added.
        * http/tests/security/postMessage/domain-and-uri-unaffected-by-base-tag.html: Added.
        * http/tests/security/postMessage/domain-unaffected-by-document-domain-expected.txt: Added.
        * http/tests/security/postMessage/domain-unaffected-by-document-domain.html: Added.
        * http/tests/security/postMessage/javascript-page-still-sends-domain-expected.txt: Added.
        * http/tests/security/postMessage/javascript-page-still-sends-domain.html: Added.
        * http/tests/security/postMessage/resources: Added.
        * http/tests/security/postMessage/resources/javascript-post-message-sender.html: Added.
        * http/tests/security/postMessage/resources/post-message-listener.html: Added.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@29678 268f45cc-cd09-0410-ab3c-d52691b4dbfc

12 years agoWebCore:
antti@apple.com [Sun, 20 Jan 2008 21:22:06 +0000 (21:22 +0000)]
WebCore:

        Reviewed by Adele.

        Fix <rdar://problem/5695451>
        Middle part of the media timeline missing on Windows if load failed

        Add some null checking to avoid getting garbage results from the functions.

        * platform/graphics/MediaPlayer.cpp:
        * platform/graphics/win/QTMovieWin.cpp:
        (QTMovieWin::duration):
        (QTMovieWin::currentTime):
        (QTMovieWin::setCurrentTime):
        (QTMovieWin::maxTimeLoaded):

LayoutTests:

        Reviewed by Adele.

        Update test so it checks the state of time related properties after the failure.

        * media/video-error-does-not-exist-expected.txt:
        * media/video-error-does-not-exist.html:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@29677 268f45cc-cd09-0410-ab3c-d52691b4dbfc

12 years agoFix 'Reviewed by' typo in ChangeLog
alp@webkit.org [Sun, 20 Jan 2008 04:53:25 +0000 (04:53 +0000)]
Fix 'Reviewed by' typo in ChangeLog

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@29676 268f45cc-cd09-0410-ab3c-d52691b4dbfc

12 years ago2008-01-19 Alp Toker <alp@atoker.com>
alp@webkit.org [Sun, 20 Jan 2008 04:49:11 +0000 (04:49 +0000)]
2008-01-19  Alp Toker  <alp@atoker.com>

        Reviewed by Alp Toker.

        http://bugs.webkit.org/show_bug.cgi?id=16944
        Use of GST_PLUGIN_DEFINE_STATIC results in a module-level constructor

        http://bugzilla.gnome.org/show_bug.cgi?id=510547
        Epiphany/Webkit fails to start due to initializing threads twice

        Don't call GST_PLUGIN_DEFINE_STATIC() as it performs g_thread_init()
        in the global initializer, breaking subsequent calls to
        g_thread_init() in applications.

        * platform/graphics/gtk/VideoSinkGStreamer.cpp:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@29675 268f45cc-cd09-0410-ab3c-d52691b4dbfc

12 years agoAllow the --http flag to run-webkit-tests to override
mrowe@apple.com [Sat, 19 Jan 2008 18:51:36 +0000 (18:51 +0000)]
Allow the --http flag to run-webkit-tests to override
the default behaviour of disabling HTTP tests for Qt,
Gtk and Wx.

Reviewed by Alp Toker.

* Scripts/run-webkit-tests:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@29674 268f45cc-cd09-0410-ab3c-d52691b4dbfc

12 years ago2008-01-19 Alp Toker <alp@atoker.com>
alp@webkit.org [Sat, 19 Jan 2008 18:34:01 +0000 (18:34 +0000)]
2008-01-19  Alp Toker  <alp@atoker.com>

        Reviewed by Mark Rowe.

        Remove any fragment part from the URL to be requested just before
        passing it to curl, otherwise curl sends it as part of the HTTP/local
        file request, causing page loads to fail.

        * platform/network/curl/ResourceHandleManager.cpp:
        (WebCore::ResourceHandleManager::startJob):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@29673 268f45cc-cd09-0410-ab3c-d52691b4dbfc

12 years agoSet svn:eol-style=native on .idl and .c files that were missed in r29663.
mrowe@apple.com [Sat, 19 Jan 2008 18:27:17 +0000 (18:27 +0000)]
Set svn:eol-style=native on .idl and .c files that were missed in r29663.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@29672 268f45cc-cd09-0410-ab3c-d52691b4dbfc

12 years ago2008-01-19 Mark Rowe <mrowe@apple.com>
mrowe@apple.com [Sat, 19 Jan 2008 18:10:55 +0000 (18:10 +0000)]
2008-01-19  Mark Rowe  <mrowe@apple.com>

        Fix inconsistent line endings and set svn:eol-style=native
        on IDL files that did not have it set.

        * Interfaces/IGEN_DOMObject.idl:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@29671 268f45cc-cd09-0410-ab3c-d52691b4dbfc

12 years ago2008-01-19 Jan Michael Alonzo <jmalonzo@unpluggable.com>
alp@webkit.org [Sat, 19 Jan 2008 15:46:23 +0000 (15:46 +0000)]
2008-01-19  Jan Michael Alonzo  <jmalonzo@unpluggable.com>

        Reviewed by Alp Toker.

        Add svg foreign object and svg experimental to the GTK+/autotools
        build system.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@29670 268f45cc-cd09-0410-ab3c-d52691b4dbfc

12 years ago2008-01-19 Christian Dywan <christian@imendio.com>
alp@webkit.org [Sat, 19 Jan 2008 13:58:44 +0000 (13:58 +0000)]
2008-01-19  Christian Dywan  <christian@imendio.com>

        Reviewed by Alp Toker.

        http://bugs.webkit.org/show_bug.cgi?id=16882
        [GTK] ChromeClientGtk is incompete

        Implement these functions.

        * WebCoreSupport/ChromeClientGtk.cpp:
        (WebKit::ChromeClient::windowRect):
        (WebKit::ChromeClient::pageRect):
        (WebKit::ChromeClient::scaleFactor):
        (WebKit::ChromeClient::focus):
        (WebKit::ChromeClient::unfocus):
        (WebKit::ChromeClient::canTakeFocus):
        (WebKit::ChromeClient::takeFocus):
        (WebKit::ChromeClient::canRunBeforeUnloadConfirmPanel):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@29669 268f45cc-cd09-0410-ab3c-d52691b4dbfc

12 years ago <rdar://problem/5695344> check-for-global-initializers script never checks...
ddkilzer@apple.com [Sat, 19 Jan 2008 10:43:32 +0000 (10:43 +0000)]
    <rdar://problem/5695344> check-for-global-initializers script never checks any object files

        Reviewed by Darin.

        We now touch a check-for-global-initializers.timestamp file in
        the TARGET_TEMP_DIR directory to determine when new object files
        have been compiled and thus need to be checked.  If the timestamp
        file doesn't exist, all object files will be checked.

        Previously the modification time of the "executable" (the
        framework binary, e.g., WebKit.framework/WebKit) was used, but
        since this was the last file modified at the end of the compile
        phase, no object files would ever get checked!

        Also added JSCustomSQLTransactionCallback.o to the list of files
        since it has static initializers in Debug builds of WebCore.

        * Scripts/check-for-global-initializers:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@29668 268f45cc-cd09-0410-ab3c-d52691b4dbfc

12 years agoWebCore:
mitz@apple.com [Sat, 19 Jan 2008 10:12:48 +0000 (10:12 +0000)]
WebCore:

        Reviewed by Maciej Stachowiak.

        - fix <rdar://problem/5645813> CrashTracer: [USER] 6 crashes in Safari at com.apple.WebCore: WebCore::RenderBox::destroy + 116

        Test: editing/selection/inconsistent-in-removeChildNode.html

        * editing/SelectionController.cpp:
        (WebCore::SelectionController::nodeWillBeRemoved): If the selection
        base or extent are not visible any more, adjust the selection.

LayoutTests:

        Reviewed by Maciej Stachowiak.

        - test and updated results for <rdar://problem/5645813> CrashTracer: [USER] 6 crashes in Safari at com.apple.WebCore: WebCore::RenderBox::destroy + 116

        * editing/selection/inconsistent-in-removeChildNode.html: Added.
        * platform/mac-leopard/editing/pasteboard/paste-RTFD-expected.txt:
        * platform/mac-leopard/editing/selection: Added.
        * platform/mac-leopard/editing/selection/inconsistent-in-removeChildNode-expected.checksum: Added.
        * platform/mac-leopard/editing/selection/inconsistent-in-removeChildNode-expected.png: Added.
        * platform/mac/editing/deleting/collapse-whitespace-3587601-fix-expected.txt:
        * platform/mac/editing/deleting/delete-3608462-fix-expected.txt:
        * platform/mac/editing/deleting/delete-4083333-fix-expected.txt:
        * platform/mac/editing/deleting/delete-leading-ws-001-expected.txt:
        * platform/mac/editing/deleting/delete-line-011-expected.txt:
        * platform/mac/editing/execCommand/find-after-replace-expected.txt:
        * platform/mac/editing/execCommand/paste-1-expected.txt:
        * platform/mac/editing/execCommand/paste-2-expected.txt:
        * platform/mac/editing/pasteboard/emacs-ctrl-k-y-001-expected.txt:
        * platform/mac/editing/selection/inconsistent-in-removeChildNode-expected.txt: Added.
        * platform/mac/editing/selection/move-between-blocks-no-001-expected.txt:
        * platform/mac/editing/selection/replace-selection-1-expected.txt:
        * platform/mac/editing/style/remove-underline-across-paragraph-expected.txt:
        * platform/mac/editing/style/remove-underline-across-paragraph-in-bold-expected.txt:
        * platform/mac/editing/style/remove-underline-after-paragraph-in-bold-expected.txt:
        * platform/mac/editing/style/remove-underline-expected.txt:
        * platform/mac/editing/style/remove-underline-in-bold-expected.txt:
        * platform/mac/editing/style/unbold-in-bold-expected.txt:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@29667 268f45cc-cd09-0410-ab3c-d52691b4dbfc

12 years agoBuild fix for --svg-foreign-object path
oliver@apple.com [Sat, 19 Jan 2008 09:43:52 +0000 (09:43 +0000)]
Build fix for --svg-foreign-object path

Reviewed by Eric Seidel

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@29666 268f45cc-cd09-0410-ab3c-d52691b4dbfc

12 years ago2008-01-18 Luca Bruno <lethalman88@gmail.com>
alp@webkit.org [Sat, 19 Jan 2008 07:49:53 +0000 (07:49 +0000)]
2008-01-18  Luca Bruno  <lethalman88@gmail.com>

        Reviewed by Alp Toker.

        http://bugs.webkit.org/show_bug.cgi?id=16910
        [GTK] REGRESSION: keyboard cursor doesn't blink

        http://bugs.webkit.org/show_bug.cgi?id=16863
        [GTK] REGRESSION: tab focusing doesn't work

        * WebView/webkitwebview.cpp:
        (webkit_web_view_focus_in_event): Added. Set the active frame.
        (webkit_web_view_class_init):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@29665 268f45cc-cd09-0410-ab3c-d52691b4dbfc

12 years ago2008-01-18 Stephanie <slewis@apple.com>
slewis@apple.com [Sat, 19 Jan 2008 06:01:28 +0000 (06:01 +0000)]
2008-01-18  Stephanie  <slewis@apple.com>

        Reviewed by Oliver.

        Workaround for rdar:5695848.  Set mime_type for local files based on file extension.

        * platform/network/cf/ResourceResponseCFNet.cpp:
        (WebCore::ResourceResponse::doUpdateResourceResponse):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@29664 268f45cc-cd09-0410-ab3c-d52691b4dbfc

12 years agoset eol-style to native on tons of files to head off future mixed-line-ending problems
darin@apple.com [Sat, 19 Jan 2008 05:49:02 +0000 (05:49 +0000)]
set eol-style to native on tons of files to head off future mixed-line-ending problems

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@29663 268f45cc-cd09-0410-ab3c-d52691b4dbfc

12 years ago Reviewed by Oliver.
adele@apple.com [Sat, 19 Jan 2008 01:44:02 +0000 (01:44 +0000)]
    Reviewed by Oliver.

        Fix for <rdar://5683527> media/{audio,video|-controls-rendering.html failing

        Remove platform specific results from media directory, since they should match the mac.  Also, remove these tests from the skipped list.

        * platform/win/Skipped:
        * platform/win/media: Removed.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@29662 268f45cc-cd09-0410-ab3c-d52691b4dbfc

12 years ago Updating these results again. I accidently checked in the other results...
adele@apple.com [Sat, 19 Jan 2008 01:30:01 +0000 (01:30 +0000)]
    Updating these results again.  I accidently checked in the other results with some local changes.

        * platform/mac/media/audio-controls-rendering-expected.checksum:
        * platform/mac/media/audio-controls-rendering-expected.png:
        * platform/mac/media/video-controls-rendering-expected.checksum:
        * platform/mac/media/video-controls-rendering-expected.png:
        * platform/mac/media/video-display-toggle-expected.checksum:
        * platform/mac/media/video-display-toggle-expected.png:
        * platform/mac/media/video-seek-past-end-playing-expected.checksum:
        * platform/mac/media/video-seek-past-end-playing-expected.png:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@29661 268f45cc-cd09-0410-ab3c-d52691b4dbfc

12 years ago - Build fix.
kmccullough@apple.com [Sat, 19 Jan 2008 01:11:07 +0000 (01:11 +0000)]
    - Build fix.

        * kjs/ustring.h:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@29660 268f45cc-cd09-0410-ab3c-d52691b4dbfc

12 years ago Rubber stamped by Darin.
bdakin@apple.com [Sat, 19 Jan 2008 00:58:19 +0000 (00:58 +0000)]
    Rubber stamped by Darin.

        These are updated test results after http://bugs.webkit.org/
        show_bug.cgi?id=15765 ASSERTION FAILED: m_frame->page() in
        FrameLoader::tokenizerProcessedData using the new GMail interface
        was checked in. This is a TEMPORARY fix until http://
        bugs.webkit.org/show_bug.cgi?id=16853 really addresses this
        problem.

        * http/tests/loading/gmail-assert-on-load-expected.txt:
        * http/tests/loading/onload-vs-immediate-refresh-expected.txt:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@29659 268f45cc-cd09-0410-ab3c-d52691b4dbfc

12 years ago Rubber stamped by Geoff.
kmccullough@apple.com [Sat, 19 Jan 2008 00:55:02 +0000 (00:55 +0000)]
    Rubber stamped by Geoff.

        - <rdar://5667267> editing/selection/5131716-1.html is failing
        The problem is when the fonts are not present, but they should be
        available now.  This may have failed for other reasons in the past
        that are now fixed.

        * platform/win/Skipped:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@29658 268f45cc-cd09-0410-ab3c-d52691b4dbfc

12 years ago <rdar://problem/5682340> REGRESSION (r28188): Context menu appears at wrong...
adachan@apple.com [Sat, 19 Jan 2008 00:24:41 +0000 (00:24 +0000)]
    <rdar://problem/5682340> REGRESSION (r28188): Context menu appears at wrong place when clicking in iframe whose parent is scrolled (16827)
        Calling absolutePosition() in EventHandler::hitTestResultAtPoint() is extremely inefficient and passing in
        true as fixed before was wrong.  Use HitTestResult::localPoint() instead, and take into account border and padding widths.

        Reviewed by Hyatt and Darin.

        * page/EventHandler.cpp:
        (WebCore::EventHandler::hitTestResultAtPoint):
        * rendering/HitTestResult.cpp:
        (WebCore::HitTestResult::operator=):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@29657 268f45cc-cd09-0410-ab3c-d52691b4dbfc

12 years ago Reviewed by Antti.
adele@apple.com [Sat, 19 Jan 2008 00:09:40 +0000 (00:09 +0000)]
    Reviewed by Antti.

        Adding pixel results for media tests.

        * platform/mac/media/audio-constructor-expected.checksum: Added.
        * platform/mac/media/audio-constructor-expected.png: Added.
        * platform/mac/media/audio-constructor-src-expected.checksum: Added.
        * platform/mac/media/audio-constructor-src-expected.png: Added.
        * platform/mac/media/broken-video-expected.checksum: Added.
        * platform/mac/media/broken-video-expected.png: Added.
        * platform/mac/media/constructors-expected.checksum: Added.
        * platform/mac/media/constructors-expected.png: Added.
        * platform/mac/media/fallback-expected.checksum: Added.
        * platform/mac/media/fallback-expected.png: Added.
        * platform/mac/media/progress-event-expected.checksum: Added.
        * platform/mac/media/progress-event-expected.png: Added.
        * platform/mac/media/remove-from-document-expected.checksum: Added.
        * platform/mac/media/remove-from-document-expected.png: Added.
        * platform/mac/media/unsupported-rtsp-expected.checksum: Added.
        * platform/mac/media/unsupported-rtsp-expected.png: Added.
        * platform/mac/media/unsupported-tracks-expected.checksum: Added.
        * platform/mac/media/unsupported-tracks-expected.png: Added.
        * platform/mac/media/video-append-source-expected.checksum: Added.
        * platform/mac/media/video-append-source-expected.png: Added.
        * platform/mac/media/video-autoplay-expected.checksum: Added.
        * platform/mac/media/video-autoplay-expected.png: Added.
        * platform/mac/media/video-buffered-expected.checksum: Added.
        * platform/mac/media/video-buffered-expected.png: Added.
        * platform/mac/media/video-controls-expected.checksum: Added.
        * platform/mac/media/video-controls-expected.png: Added.
        * platform/mac/media/video-currentTime-expected.checksum: Added.
        * platform/mac/media/video-currentTime-expected.png: Added.
        * platform/mac/media/video-currentTime-set-expected.checksum: Added.
        * platform/mac/media/video-currentTime-set-expected.png: Added.
        * platform/mac/media/video-currentTime-set2-expected.checksum: Added.
        * platform/mac/media/video-currentTime-set2-expected.png: Added.
        * platform/mac/media/video-dom-autoplay-expected.checksum: Added.
        * platform/mac/media/video-dom-autoplay-expected.png: Added.
        * platform/mac/media/video-dom-end-expected.checksum: Added.
        * platform/mac/media/video-dom-end-expected.png: Added.
        * platform/mac/media/video-dom-loopcount-expected.checksum: Added.
        * platform/mac/media/video-dom-loopcount-expected.png: Added.
        * platform/mac/media/video-dom-loopend-expected.checksum: Added.
        * platform/mac/media/video-dom-loopend-expected.png: Added.
        * platform/mac/media/video-dom-loopstart-expected.checksum: Added.
        * platform/mac/media/video-dom-loopstart-expected.png: Added.
        * platform/mac/media/video-dom-src-expected.checksum: Added.
        * platform/mac/media/video-dom-src-expected.png: Added.
        * platform/mac/media/video-dom-start-expected.checksum: Added.
        * platform/mac/media/video-dom-start-expected.png: Added.
        * platform/mac/media/video-end-expected.checksum: Added.
        * platform/mac/media/video-end-expected.png: Added.
        * platform/mac/media/video-error-abort-expected.checksum: Added.
        * platform/mac/media/video-error-abort-expected.png: Added.
        * platform/mac/media/video-error-does-not-exist-expected.checksum: Added.
        * platform/mac/media/video-error-does-not-exist-expected.png: Added.
        * platform/mac/media/video-load-networkState-expected.checksum: Added.
        * platform/mac/media/video-load-networkState-expected.png: Added.
        * platform/mac/media/video-load-readyState-expected.checksum: Added.
        * platform/mac/media/video-load-readyState-expected.png: Added.
        * platform/mac/media/video-loopcount-expected.checksum: Added.
        * platform/mac/media/video-loopcount-expected.png: Added.
        * platform/mac/media/video-loopend-expected.checksum: Added.
        * platform/mac/media/video-loopend-expected.png: Added.
        * platform/mac/media/video-loopstart-expected.checksum: Added.
        * platform/mac/media/video-loopstart-expected.png: Added.
        * platform/mac/media/video-muted-expected.checksum: Added.
        * platform/mac/media/video-muted-expected.png: Added.
        * platform/mac/media/video-no-autoplay-expected.checksum: Added.
        * platform/mac/media/video-no-autoplay-expected.png: Added.
        * platform/mac/media/video-pause-empty-events-expected.checksum: Added.
        * platform/mac/media/video-pause-empty-events-expected.png: Added.
        * platform/mac/media/video-play-empty-events-expected.checksum: Added.
        * platform/mac/media/video-play-empty-events-expected.png: Added.
        * platform/mac/media/video-play-pause-events-expected.checksum: Added.
        * platform/mac/media/video-play-pause-events-expected.png: Added.
        * platform/mac/media/video-play-pause-exception-expected.checksum: Added.
        * platform/mac/media/video-play-pause-exception-expected.png: Added.
        * platform/mac/media/video-poster-expected.checksum: Added.
        * platform/mac/media/video-poster-expected.png: Added.
        * platform/mac/media/video-seek-past-end-paused-expected.checksum: Added.
        * platform/mac/media/video-seek-past-end-paused-expected.png: Added.
        * platform/mac/media/video-seek-past-end-playing-expected.checksum: Added.
        * platform/mac/media/video-seek-past-end-playing-expected.png: Added.
        * platform/mac/media/video-seekable-expected.checksum: Added.
        * platform/mac/media/video-seekable-expected.png: Added.
        * platform/mac/media/video-seeking-expected.checksum: Added.
        * platform/mac/media/video-seeking-expected.png: Added.
        * platform/mac/media/video-size-expected.checksum: Added.
        * platform/mac/media/video-size-expected.png: Added.
        * platform/mac/media/video-source-expected.checksum: Added.
        * platform/mac/media/video-source-expected.png: Added.
        * platform/mac/media/video-source-media-expected.checksum: Added.
        * platform/mac/media/video-source-media-expected.png: Added.
        * platform/mac/media/video-source-type-expected.checksum: Added.
        * platform/mac/media/video-source-type-expected.png: Added.
        * platform/mac/media/video-src-change-expected.checksum: Added.
        * platform/mac/media/video-src-change-expected.png: Added.
        * platform/mac/media/video-src-expected.checksum: Added.
        * platform/mac/media/video-src-expected.png: Added.
        * platform/mac/media/video-src-remove-expected.checksum: Added.
        * platform/mac/media/video-src-remove-expected.png: Added.
        * platform/mac/media/video-src-set-expected.checksum: Added.
        * platform/mac/media/video-src-set-expected.png: Added.
        * platform/mac/media/video-src-source-expected.checksum: Added.
        * platform/mac/media/video-src-source-expected.png: Added.
        * platform/mac/media/video-start-expected.checksum: Added.
        * platform/mac/media/video-start-expected.png: Added.
        * platform/mac/media/video-volume-expected.checksum: Added.
        * platform/mac/media/video-volume-expected.png: Added.
        * platform/mac/media/video-width-height-expected.checksum: Added.
        * platform/mac/media/video-width-height-expected.png: Added.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@29656 268f45cc-cd09-0410-ab3c-d52691b4dbfc

12 years agoWebCore:
bdakin@apple.com [Sat, 19 Jan 2008 00:04:08 +0000 (00:04 +0000)]
WebCore:

        Reviewed by Darin, landed by Beth.

        - fixed http://bugs.webkit.org/show_bug.cgi?id=15765

        The call to HTMLTokenizer::write might result in a call to ::end which will invoke
        HTMLParser::finished() or Document::finishedParsing(). HTMLParser::finished() will eventually
        call Document::finishedParsing. The Document will delete the calling HTMLTokenizer and from the deleted
        tokenizer we will call into FrameLoader::tokenizerProcessedData.
        -) FrameLoader::tokenizerProcessedData calls FrameLoader::checkCompleted which gets called from the Document::finishedParsing
           (FrameLoader::finishedParsing).

        * html/HTMLTokenizer.cpp:
        (WebCore::HTMLTokenizer::timerFired):

LayoutTests:

        Reviewed by Darin, landed by Beth.

        -test for http://bugs.webkit.org/show_bug.cgi?id=15765
        Gmail hitting an assert in FrameLoader::tokenizerProcessedData

        * http/tests/loading/gmail-assert-on-load-expected.txt: Added.
        * http/tests/loading/gmail-assert-on-load.html: Added.
        * http/tests/loading/resources/gmail-assert-on-load-frame.html: Added.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@29655 268f45cc-cd09-0410-ab3c-d52691b4dbfc

12 years ago Reviewed by Beth Dakin.
weinig@apple.com [Sat, 19 Jan 2008 00:01:01 +0000 (00:01 +0000)]
    Reviewed by Beth Dakin.

        Fix for <rdar://problem/5683524> http/tests/security/cross-frame-access-get.html is failing

        - Guard against possible timing related failures using polling.

        * http/tests/security/:
        * http/tests/security/resources/cross-frame-iframe-for-get-test.html:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@29654 268f45cc-cd09-0410-ab3c-d52691b4dbfc

12 years ago - Build fix.
kmccullough@apple.com [Sat, 19 Jan 2008 00:00:25 +0000 (00:00 +0000)]
    - Build fix.

        * kjs/ustring.cpp:
        * kjs/ustring.h:
        (KJS::UString::cost):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@29653 268f45cc-cd09-0410-ab3c-d52691b4dbfc

12 years agoRemove incorrect addition of ChangeLog as changed in the ChangeLog.
weinig@apple.com [Fri, 18 Jan 2008 23:33:07 +0000 (23:33 +0000)]
Remove incorrect addition of ChangeLog as changed in the ChangeLog.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@29652 268f45cc-cd09-0410-ab3c-d52691b4dbfc

12 years ago Update inline results after previous checkins. New results match Firefox...
hyatt@apple.com [Fri, 18 Jan 2008 23:20:30 +0000 (23:20 +0000)]
    Update inline results after previous checkins.  New results match Firefox more closely.

        * platform/mac/fast/inline/001-expected.checksum:
        * platform/mac/fast/inline/001-expected.png:
        * platform/mac/fast/inline/002-expected.checksum:
        * platform/mac/fast/inline/002-expected.png:
        * platform/mac/fast/inline/br-text-decoration-expected.checksum:
        * platform/mac/fast/inline/br-text-decoration-expected.png:
        * platform/mac/fast/inline/continuation-outlines-expected.checksum:
        * platform/mac/fast/inline/continuation-outlines-expected.png:
        * platform/mac/fast/inline/continuation-outlines-with-layers-expected.checksum:
        * platform/mac/fast/inline/continuation-outlines-with-layers-expected.png:
        * platform/mac/fast/inline/dirtyLinesForInline-expected.checksum:
        * platform/mac/fast/inline/dirtyLinesForInline-expected.png:
        * platform/mac/fast/inline/drawStyledEmptyInlines-expected.checksum:
        * platform/mac/fast/inline/drawStyledEmptyInlines-expected.png:
        * platform/mac/fast/inline/drawStyledEmptyInlines-expected.txt:
        * platform/mac/fast/inline/drawStyledEmptyInlinesWithWS-expected.checksum:
        * platform/mac/fast/inline/drawStyledEmptyInlinesWithWS-expected.png:
        * platform/mac/fast/inline/emptyInlinesWithinLists-expected.checksum:
        * platform/mac/fast/inline/emptyInlinesWithinLists-expected.png:
        * platform/mac/fast/inline/emptyInlinesWithinLists-expected.txt:
        * platform/mac/fast/inline/inline-borders-with-bidi-override-expected.checksum:
        * platform/mac/fast/inline/inline-borders-with-bidi-override-expected.png:
        * platform/mac/fast/inline/outline-continuations-expected.checksum:
        * platform/mac/fast/inline/outline-continuations-expected.png:
        * platform/mac/fast/inline/positionedLifetime-expected.checksum:
        * platform/mac/fast/inline/positionedLifetime-expected.png:
        * platform/mac/fast/inline/styledEmptyInlinesWithBRs-expected.checksum:
        * platform/mac/fast/inline/styledEmptyInlinesWithBRs-expected.png:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@29651 268f45cc-cd09-0410-ab3c-d52691b4dbfc

12 years ago Don't apply the border/margin/padding check to root line boxes, since their...
hyatt@apple.com [Fri, 18 Jan 2008 23:10:51 +0000 (23:10 +0000)]
    Don't apply the border/margin/padding check to root line boxes, since their renderers
        are blocks.

        * rendering/InlineFlowBox.cpp:
        (WebCore::InlineFlowBox::computeLogicalBoxHeights):
        (WebCore::InlineFlowBox::placeBoxesVertically):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@29650 268f45cc-cd09-0410-ab3c-d52691b4dbfc

12 years agoWebCore:
hyatt@apple.com [Fri, 18 Jan 2008 22:59:29 +0000 (22:59 +0000)]
WebCore:

        Not reviewed: rolling out a previous patch.

        Fixed <rdar://problem/5695439> Crash during GCController destructor on
        quitting browser

        Used svn merge to roll out r29603 because it introduced some crashes
        on quit.

        GC relies on static hash tables, so it's not safe to GC from a static
        destructor, which might run after the static hash tables' destructors.

        * bindings/js/GCController.cpp:
        (WebCore::GCController::garbageCollectNow):
        * bindings/js/GCController.h:

LayoutTests:

        Fix for http://bugs.webkit.org/show_bug.cgi?id=15665

        Reviewed by Beth

        * fast/inline/inline-padding-disables-text-quirk.html: Added.
        * platform/mac/fast/inline/inline-padding-disables-text-quirk-expected.checksum: Added.
        * platform/mac/fast/inline/inline-padding-disables-text-quirk-expected.png: Added.
        * platform/mac/fast/inline/inline-padding-disables-text-quirk-expected.txt: Added.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@29649 268f45cc-cd09-0410-ab3c-d52691b4dbfc