Timeline


and

03/11/09:

23:39 Changeset [41615] by oliver@apple.com
  • 6 edits
    3 adds in trunk

<rdar://problem/6673852> Widget "Simple World Clock" does not display with Safari 4

Reviewed by Adele Peterson.

A number of dashboard widgets rely on canvas failing silently when given invalid
colors for gradient stops, but both the spec and firefox throw exceptions here.
So we work around this by creating a dashboard quirk that will only suppress the
exception in dashboard compatibility mode.

Test: fast/canvas/canvas-gradient-addStop-error.html

23:02 Changeset [41614] by ddkilzer@apple.com
  • 2 edits in trunk/WebKit/mac

Remove duplicate header include

Rubber-stamped by Mark Rowe.

  • WebView/WebView.mm: Remove duplicate #include <runtime/InitializeThreading.h>. Also realphabetized lowercase #include statements.
22:14 Changeset [41613] by ddkilzer@apple.com
  • 8 edits in trunk

Clarify comments regarding order of FEATURE_DEFINES

Rubber-stamped by Mark Rowe.

JavaScriptCore:

  • Configurations/JavaScriptCore.xcconfig: Added warning about the consequences when FEATURE_DEFINES are not kept in sync.

WebCore:

  • Configurations/WebCore.xcconfig: Added warning about the consequences when FEATURE_DEFINES are not kept in sync.

WebKit/mac:

  • Configurations/WebKit.xcconfig: Added warning about the consequences when FEATURE_DEFINES are not kept in sync.

WebKitTools:

  • Scripts/build-webkit: Added warning about keeping FEATURE_DEFINES in order and the consequences when they are not.
21:39 Changeset [41612] by mrowe@apple.com
  • 2 edits in tags/Safari-6530.3/WebKit/mac

Merge r41592.

21:37 Changeset [41611] by mrowe@apple.com
  • 7 edits
    2 copies in tags/Safari-6530.3

Merge r41576.

21:27 Changeset [41610] by kov@webkit.org
  • 2 edits in trunk/WebCore

2009-03-11 Gustavo Noronha Silva <gns@gnome.org>

Reviewed by Holger Freyther.

https://bugs.webkit.org/show_bug.cgi?id=24515
[GTK] soup backend should use GMappedFile instead of mmap directly

Rewrite mmap code used to upload files to use GMappedFile, so that
it is platform-independent.

  • platform/network/soup/ResourceHandleSoup.cpp: (WebCore::ResourceHandle::startHttp):
21:06 Changeset [41609] by mrowe@apple.com
  • 4 edits in trunk

Versioning.

21:05 Changeset [41608] by mrowe@apple.com
  • 1 copy in tags/Safari-6530.3

New tag.

20:39 Changeset [41607] by mitz@apple.com
  • 8 edits
    2 adds in trunk

JavaScriptCore:

Reviewed by Darin Adler.

  • WTF support for fixing <rdar://problem/3919124> Thai text selection in Safari is incorrect
  • wtf/unicode/icu/UnicodeIcu.h: (WTF::Unicode::hasLineBreakingPropertyComplexContext): Added. Returns whether the character has Unicode line breaking property value SA ("Complex Context").
  • wtf/unicode/qt4/UnicodeQt4.h: (WTF::Unicode::hasLineBreakingPropertyComplexContext): Added an implementation that always returns false.

WebCore:

Reviewed by Darin Adler.

  • WebCore part of fixing <rdar://problem/3919124> Thai text selection in Safari is incorrect

Test: platform/mac/editing/selection/word-thai.html

Tested on Mac, but should work on all ICU platforms (or more precisely
on all platforms that implement
WTF::Unicode::hasLineBreakingPropertyComplexContext() correctly).

  • editing/TextIterator.cpp: (WebCore::BackwardsCharacterIterator::BackwardsCharacterIterator): Added. (WebCore::BackwardsCharacterIterator::range): Added. (WebCore::BackwardsCharacterIterator::advance): Added.
  • editing/TextIterator.h: Added BackwardsCharacterIterator with the minimal functionality required for this patch. (WebCore::BackwardsCharacterIterator::atEnd): Added.
  • editing/visible_units.cpp: (WebCore::firstNonComplexContextLineBreak): Added this helper function that returns the index of the first character in the string whose Unicode line breaking property value is not SA ("Complex Context"), or the length of the string if there is no such character. (WebCore::lastNonComplexContextLineBreak): Added this helper function that returns the index of the last character in the string whose Unicode line breaking property value is not SA ("Complex Context"), or -1 if there is no such character. (WebCore::previousBoundary): Changed the signature of the search function to include an offset parameter. Renamed the 'exception' local variable to 'ec' and changed its type to ExceptionCode. Extend the string forwards until the first character with Unicode line breaking property value other than SA. This gives the boundary search function enough context in the forward direction. Changed to use a BackwardsCharacterIterator for translating the backwards offset into a position. (WebCore::nextBoundary): Changed the signature of the search function to include an offset parameter. Extend the string backwards until the first character with Unicode line breaking property value other than SA. This gives the boundary search function enough context in the backwards direction. Restricted the workaround for <rdar://5192593> only to the line break case, because it was causing an extra character to be selected following a word that ended with a combining mark. (WebCore::startWordBoundary): Added an offset parameter. Check that going backwards from the given offset, there is a character with Unicode line breaking property value other than SA, and otherwise return 0 to request more context. (WebCore::endWordBoundary): Similar, but in reverse. (WebCore::previousWordPositionBoundary): Similar. (WebCore::nextWordPositionBoundary): Similar. (WebCore::startSentenceBoundary): Updated for the new search function signature. (WebCore::endSentenceBoundary): Ditto. (WebCore::previousSentencePositionBoundary): Ditto. (WebCore::nextSentencePositionBoundary): Ditto.

LayoutTests:

Reviewed by Darin Adler.

  • test for <rdar://problem/3919124> Thai text selection in Safari is incorrect
  • platform/mac/editing/selection/word-thai-expected.txt: Added.
  • platform/mac/editing/selection/word-thai.html: Added.
18:29 Changeset [41606] by darin@apple.com
  • 6 edits in trunk/WebCore

2009-03-11 Darin Adler <darin@apple.com>

Reviewed by Mark Rowe.

Don't use "::" in thread names because it is ugly!

  • loader/icon/IconDatabase.cpp: (WebCore::IconDatabase::open): Use "WebCore: " instead of "WebCore::".
  • platform/network/cf/ResourceHandleCFNet.cpp: (WebCore::ResourceHandle::loaderRunLoop): Call this "WebCore: CFNetwork Loader" to make it clear this is a WebCore-created thread.
  • storage/DatabaseThread.cpp: (WebCore::DatabaseThread::start): Use "WebCore: " instead of "WebCore::".
  • storage/LocalStorageThread.cpp: (WebCore::LocalStorageThread::start): Use "WebCore: " instead of "WebCore::".
  • workers/WorkerThread.cpp: (WebCore::WorkerThread::start): Use "WebCore: " instead of "WebCore::".
18:26 Changeset [41605] by darin@apple.com
  • 8 edits in trunk/JavaScriptCore

2009-03-11 Darin Adler <darin@apple.com>

Reviewed by Mark Rowe.

Give threads names on platforms with pthread_setname_np.

  • wtf/Threading.cpp: (WTF::NewThreadContext::NewThreadContext): Initialize thread name. (WTF::threadEntryPoint): Call setThreadNameInternal. (WTF::createThread): Pass thread name.
  • wtf/Threading.h: Added new comments, setThreadNameInternal.
  • wtf/ThreadingGtk.cpp: (WTF::setThreadNameInternal): Added. Empty.
  • wtf/ThreadingNone.cpp: (WTF::setThreadNameInternal): Added. Empty.
  • wtf/ThreadingPthreads.cpp: (WTF::setThreadNameInternal): Call pthread_setname_np when available.
  • wtf/ThreadingQt.cpp: (WTF::setThreadNameInternal): Added. Empty.
  • wtf/ThreadingWin.cpp: (WTF::setThreadNameInternal): Added. Empty.
17:53 Changeset [41604] by mrowe@apple.com
  • 2 edits in trunk/WebCore

<rdar://problem/6653286> WebCore's implementation of screenIsMonochrome is needlessly complex

Rubber-stamped by Darin Adler.

  • platform/mac/PlatformScreenMac.mm:

(WebCore::screenIsMonochrome): Return false.

17:51 Changeset [41603] by simon.fraser@apple.com
  • 2 edits in trunk/WebCore

2009-03-11 Simon Fraser <simon.fraser@apple.com>

Reviewed by Darin Adler

https://bugs.webkit.org/show_bug.cgi?id=24531

Make sure that AnimationControllerPrivate::getAnimatedStyleForRenderer() never
returns a null style if it has a valid renderer, which could happen if a
CompositeAnimation existed, but wasn't running any animations or transitions.

  • page/animation/AnimationController.cpp: (WebCore::AnimationControllerPrivate::getAnimatedStyleForRenderer):
16:25 Changeset [41602] by bdakin@apple.com
  • 3 edits
    4 adds in trunk

WebCore:

2009-03-11 Beth Dakin <bdakin@apple.com>

Reviewed by Dan Bernstein.

Fix for https://bugs.webkit.org/show_bug.cgi?id=14749 percentage
top value on position:relative child not calculated correctly if
parent has percentage height

  • and corresponding - <rdar://problem/6172925>
  • rendering/RenderBoxModelObject.cpp: (WebCore::RenderBoxModelObject::relativePositionOffsetY):

LayoutTests:

2009-03-11 Beth Dakin <bdakin@apple.com>

Reviewed by Dan Bernstein.

Test for https://bugs.webkit.org/show_bug.cgi?id=14749 percentage
top value on position:relative child not calculated correctly if
parent has percentage height

  • and corresponding - <rdar://problem/6172925>
  • fast/css/percent-top-value-with-relative-position.html: Added.
  • platform/mac/fast/css/percent-top-value-with-relative-position-expected.checksum: Added.
  • platform/mac/fast/css/percent-top-value-with-relative-position-expected.png: Added.
  • platform/mac/fast/css/percent-top-value-with-relative-position-expected.txt: Added.
14:54 Changeset [41601] by levin@chromium.org
  • 2 edits in trunk/WebCore

2009-03-11 Jian Li <jianli@chromium.org>

Reviewed by Dimitri Glazkov.

Implement ScriptValue::hasNoValue for V8.
https://bugs.webkit.org/show_bug.cgi?id=24507

  • bindings/v8/ScriptValue.h: (WebCore::ScriptValue::hasNoValue):
14:54 Changeset [41600] by levin@chromium.org
  • 5 edits
    2 copies
    7 adds in trunk/WebCore

2009-03-11 David Levin <levin@chromium.org>

Reviewed by Dimitri Glazkov.

Bug 24459: Add v8 bindings for event.
<https://bugs.webkit.org/show_bug.cgi?id=24459>

Adding custom event bindings.

  • bindings/v8/custom/V8AbstractEventListener.cpp: Added.
  • bindings/v8/custom/V8AbstractEventListener.h: Added.
  • bindings/v8/custom/V8AttrCustom.cpp: Fixed messed up copyright header.
  • bindings/v8/custom/V8CustomEventListener.cpp: Added.
  • bindings/v8/custom/V8CustomEventListener.h: Replaced forwarding header with the real implementation.
  • bindings/v8/custom/V8ElementCustom.cpp:
  • bindings/v8/custom/V8LazyEventListener.cpp: Added.
  • bindings/v8/custom/V8LazyEventListener.h: Added.
  • bindings/v8/custom/V8ObjectEventListener.cpp: Added.
  • bindings/v8/custom/V8ObjectEventListener.h: Added.
  • bindings/v8/custom/V8WorkerContextEventListener.cpp: Added.
  • bindings/v8/custom/V8WorkerContextEventListener.h: Added.
14:40 Changeset [41599] by darin@chromium.org
  • 3 edits in trunk/WebCore

2009-03-11 Darin Fisher <darin@chromium.org>

Reviewed by Dimitri Glazkov.

https://bugs.webkit.org/show_bug.cgi?id=24521
Add some missing includes for the non-JSC build.

  • platform/text/PlatformString.h: added OwnPtr.h
  • xml/XMLHttpRequest.cpp: added SecurityOrigin.h
14:17 Changeset [41598] by pam@chromium.org
  • 1 edit
    2 adds in trunk/LayoutTests

2009-03-11 Pamela Greene <pam@chromium.org>

Reviewed by Alexey Proskuryakov.

Add test for potential crash in window.frames.length
These files were accidentally left out of r41459.

https://bugs.webkit.org/show_bug.cgi?id=24339

  • fast/dom/Window/window-collection-length-no-crash-expected.txt: Added.
  • fast/dom/Window/window-collection-length-no-crash.html: Added.
13:58 Changeset [41597] by andersca@apple.com
  • 8 edits in trunk/WebKit/mac

2009-03-11 Anders Carlsson <andersca@apple.com>

Reviewed by Sam Weinig.

WebKit side of <rdar://problem/6656147>.


  • Plugins/Hosted/NetscapePluginHostManager.mm: (WebKit::NetscapePluginHostManager::instantiatePlugin): Pass the requestID to _WKPHInstantiatePlugin.


  • Plugins/Hosted/NetscapePluginHostProxy.mm: Pass the requestID to setCurrentReply.


  • Plugins/Hosted/NetscapePluginInstanceProxy.h: (WebKit::NetscapePluginInstanceProxy::setCurrentReply): Store the reply in a map with the requestID as the key.


(WebKit::NetscapePluginInstanceProxy::waitForReply):
Wait for a reply that matches the given requestID.


  • Plugins/Hosted/NetscapePluginInstanceProxy.mm: (WebKit::NetscapePluginInstanceProxy::NetscapePluginInstanceProxy): Initialize member variables.


(WebKit::NetscapePluginInstanceProxy::~NetscapePluginInstanceProxy):
Delete all requests.


(WebKit::NetscapePluginInstanceProxy::print):
Pass the requestID to _WKPHPluginInstancePrint.


(WebKit::NetscapePluginInstanceProxy::loadRequest):
Rename m_currentRequestID to m_currentURLRequestID.


(WebKit::NetscapePluginInstanceProxy::processRequestsAndWaitForReply):
Process requests until we find a reply with the right requestID.


(WebKit::NetscapePluginInstanceProxy::createBindingsInstance):
Pass a requestID to the _WKPH function.


(WebKit::NetscapePluginInstanceProxy::nextRequestID):
Ditto.


  • Plugins/Hosted/ProxyInstance.mm: Pass a requestID to the _WKPH functions.


  • Plugins/Hosted/WebKitPluginClient.defs:
  • Plugins/Hosted/WebKitPluginHost.defs: Add requestID parameters.
13:41 Changeset [41596] by simon.fraser@apple.com
  • 2 edits in trunk/WebCore

2009-03-11 Simon Fraser <simon.fraser@apple.com>

Fix Scons build. Patch from Alexander Guy.

  • WebCore.scons:
13:39 Changeset [41595] by aroben@apple.com
  • 3 edits in trunk/WebKit/win

Export functions needed by ThreadSpecific from WebKit

Reviewed by John Sullivan.

  • WebKit.vcproj/WebKit.def:
  • WebKit.vcproj/WebKit_debug.def: Export WTF::tlsKeyCount and WTF::tlsKeys.
13:39 Changeset [41594] by aroben@apple.com
  • 4 edits in trunk/JavaScriptCore

Change the Windows implementation of ThreadSpecific to use functions instead of extern globals

This will make it easier to export ThreadSpecific from WebKit.

Reviewed by John Sullivan.

  • API/JSBase.cpp: (JSEvaluateScript): Touched this file to force ThreadSpecific.h to be copied into $WebKitOutputDir.
  • wtf/ThreadSpecific.h: Replaced g_tls_key_count with tlsKeyCount() and g_tls_keys with tlsKeys().

(WTF::::ThreadSpecific):
(WTF::::~ThreadSpecific):
(WTF::::get):
(WTF::::set):
(WTF::::destroy):
Updated to use the new functions.

  • wtf/ThreadSpecificWin.cpp: (WTF::tlsKeyCount): (WTF::tlsKeys): Added.

(WTF::ThreadSpecificThreadExit): Changed to use the new functions.

12:45 Changeset [41593] by simon.fraser@apple.com
  • 3 edits in trunk/LayoutTests

2009-03-11 Simon Fraser <simon.fraser@apple.com>

Reviewed by Dave Hyatt

https://bugs.webkit.org/show_bug.cgi?id=24366

Make this test more reliable by only calling runAnimationTest() once
the animation starts, not before some setTimeout().

  • animations/change-keyframes-name-expected.txt:
  • animations/change-keyframes-name.html:
12:09 Changeset [41592] by andersca@apple.com
  • 2 edits in trunk/WebKit/mac

2009-03-11 Anders Carlsson <andersca@apple.com>

Reviewed by Darin Adler.

Fix <rdar://problem/6620064>.


  • Plugins/WebPluginContainerPrivate.h:
12:04 Changeset [41591] by simon.fraser@apple.com
  • 6 edits in trunk/LayoutTests

2009-03-11 Simon Fraser <simon.fraser@apple.com>

Reviewed by Dave Hyatt

https://bugs.webkit.org/show_bug.cgi?id=24266

Land new expected results for transform-absolute-in-positioned-container.html,
and new pixel results for reflection-masks.html. Both are progressions.

  • platform/mac/fast/reflections/reflection-masks-expected.checksum:
  • platform/mac/fast/reflections/reflection-masks-expected.png:
  • platform/mac/fast/repaint/transform-absolute-in-positioned-container-expected.checksum:
  • platform/mac/fast/repaint/transform-absolute-in-positioned-container-expected.png:
  • platform/mac/fast/repaint/transform-absolute-in-positioned-container-expected.txt:
12:00 Changeset [41590] by simon.fraser@apple.com
  • 4 edits
    4 adds in trunk

2009-03-11 Simon Fraser <simon.fraser@apple.com>

Reviewed by Dan Bernstein

https://bugs.webkit.org/show_bug.cgi?id=23093

When the cancel button in the search field is shown or hidden,
we need to explicitly setStyle() on the cancel button's renderer,
to ensure that repaint happens. Changing the style without telling
the renderer won't work.

Test: fast/repaint/search-field-cancel.html

  • rendering/RenderTextControlSingleLine.cpp: (WebCore::RenderTextControlSingleLine::subtreeHasChanged): (WebCore::RenderTextControlSingleLine::updateFromElement): (WebCore::RenderTextControlSingleLine::createCancelButtonStyle): (WebCore::RenderTextControlSingleLine::updateCancelButtonVisibility): (WebCore::RenderTextControlSingleLine::visibilityForCancelButton):
  • rendering/RenderTextControlSingleLine.h:
10:58 Changeset [41589] by pam@chromium.org
  • 3 edits in trunk/LayoutTests

2009-03-11 Marc-André Decoste <mad@chromium.org>

Reviewed by Eric Seidel.

Fixed the positions of the list boxes so that the size of their
rendering doesn't affect the test coordinates.
https://bugs.webkit.org/show_bug.cgi?id=24006

  • fast/forms/option-mouseevents-expected.txt:
  • fast/forms/option-mouseevents.html:
09:05 Changeset [41588] by ap@webkit.org
  • 4 edits
    2 adds in trunk

2009-03-11 Hironori Bono <hbono@chromium.org>

Reviewed by Alexey Proskuryakov.

https://bugs.webkit.org/show_bug.cgi?id=15790
Ligatures aren't accounted for when manipulating VisiblePositions

Changed cursorMovementIterator rules to prevent a cursor from moving in the middle of an
Indic ligature which uses a virama sign.


Test: editing/deleting/skip-virama-001.html

  • platform/text/TextBreakIteratorICU.cpp: (WebCore::cursorMovementIterator):
08:46 Changeset [41587] by ap@webkit.org
  • 2 edits in trunk/WebCore

Reviewed by George Staikos.

https://bugs.webkit.org/show_bug.cgi?id=24513
REGRESSION: Cursor movement doesn't work on Tiger (ICU 3.2)

Fixes existing tests on Tiger.

  • platform/text/TextBreakIteratorICU.cpp: (WebCore::cursorMovementIterator): Use characterBreakIterator on Tiger (like we used to do).
07:53 Changeset [41586] by xan@webkit.org
  • 2 edits in trunk/WebKit/gtk

2009-03-10 Xan Lopez <xlopez@igalia.com>

Reviewed by Holger Freyther.

https://bugs.webkit.org/show_bug.cgi?id=24493
[GTK] Misc patches for WebKitWebHistoryItem

Only run code in dispose once per instance.

  • webkit/webkitwebhistoryitem.cpp: (_WebKitWebHistoryItemPrivate::webkit_web_history_item_dispose):
07:53 Changeset [41585] by xan@webkit.org
  • 2 edits in trunk/WebKit/gtk

2009-03-10 Xan Lopez <xlopez@igalia.com>

Reviewed by Holger Freyther.

https://bugs.webkit.org/show_bug.cgi?id=24493
[GTK] Misc patches for WebKitWebHistoryItem

Call deref() on our internal HistoryItem on dispose, as we always
acquire it with a releaseRef() call to a PassRefPtr, which passes
ownership.

  • webkit/webkitwebhistoryitem.cpp: (_WebKitWebHistoryItemPrivate::webkit_web_history_item_dispose):
07:52 Changeset [41584] by xan@webkit.org
  • 2 edits in trunk/WebKit/gtk

2009-03-10 Xan Lopez <xlopez@igalia.com>

Reviewed by Holger Freyther.

https://bugs.webkit.org/show_bug.cgi?id=24493
[GTK] Misc patches for WebKitWebHistoryItem

return foo? foo : NULL == return foo

  • webkit/webkitwebhistoryitem.cpp: (WebKit::core):
05:27 Changeset [41583] by jmalonzo@webkit.org
  • 8 edits in trunk

2009-03-11 Jan Michael Alonzo <jmalonzo@webkit.org>

Reviewed by Holger Freyther.

Gtk] Implement LayoutTestControllerGtk::setPrivateBrowsingEnabled
https://bugs.webkit.org/show_bug.cgi?id=24487

Enable http/tests/security/cross-frame-access-private-browsing.html test.

  • platform/gtk/Skipped:

2009-03-11 Jan Michael Alonzo <jmalonzo@webkit.org>

Reviewed by Holger Freyther.

Gtk] Implement LayoutTestControllerGtk::setPrivateBrowsingEnabled
https://bugs.webkit.org/show_bug.cgi?id=24487

Add private browsing option "enable-private-browsing" to WebKitWebSettings.

  • webkit/webkitwebsettings.cpp: (_WebKitWebSettingsPrivate::): (_WebKitWebSettingsPrivate::webkit_web_settings_class_init): (_WebKitWebSettingsPrivate::webkit_web_settings_set_property): (_WebKitWebSettingsPrivate::webkit_web_settings_get_property): (_WebKitWebSettingsPrivate::webkit_web_settings_copy):
  • webkit/webkitwebview.cpp:

2009-03-11 Jan Michael Alonzo <jmalonzo@webkit.org>

Reviewed by Holger Freyther.

Gtk] Implement LayoutTestControllerGtk::setPrivateBrowsingEnabled
https://bugs.webkit.org/show_bug.cgi?id=24487

Also reset WebSettings to its default state for JavaScript
profiling, Developer Extras and Private Browsing before running
the test (can be after each test but we want to be consistent with
other ports in this regard)

  • DumpRenderTree/gtk/DumpRenderTree.cpp: (resetWebViewToConsistentStateBeforeTesting): (runTest):
  • DumpRenderTree/gtk/LayoutTestControllerGtk.cpp: (LayoutTestController::setPrivateBrowsingEnabled):
05:06 Changeset [41582] by jmalonzo@webkit.org
  • 3 edits in trunk/WebKit/gtk

2009-03-11 Jan Michael Alonzo <jmalonzo@webkit.org>

Reviewed by Holger Freyther.

[GTK] BackForward history leak?
https://bugs.webkit.org/show_bug.cgi?id=19528

Don't ref the history items when returning the back/forward list
Added test_webkit_web_history_item_lifetime test case for this.

  • tests/main.c: (test_webkit_web_history_item_lifetime): (test_webkit_web_back_forward_list_order): Style fix. (test_webkit_web_back_forward_list_add_item): Style fix. (main):
  • webkit/webkitwebbackforwardlist.cpp: (_WebKitWebBackForwardListPrivate::webkit_web_back_forward_list_get_forward_list_with_limit): (_WebKitWebBackForwardListPrivate::webkit_web_back_forward_list_get_back_list_with_limit):
04:58 Changeset [41581] by ap@webkit.org
  • 2 edits in trunk/WebCore

Reviewed by Oliver Hunt.

<rdar://problem/4072827> Downloaded non-ASCII file name becomes garbled

  • platform/network/mac/ResourceRequestMac.mm: (WebCore::ResourceRequest::doUpdatePlatformRequest): Correct selector name in instancesRespondToSelector.
04:57 Changeset [41580] by jmalonzo@webkit.org
  • 2 edits in trunk

2009-03-11 Jan Michael Alonzo <jmalonzo@webkit.org>

Reviewed by Holger Freyther.

[Gtk] Fix make clean targets
https://bugs.webkit.org/show_bug.cgi?id=24450

Fix 'make' clean targets. We shouldn't be removing DerivedSources
if it's only clean. Only remove it if it's distclean or
maintainer-clean. Also remove build-related auxillary files on
dist/maintainer clean.

  • GNUmakefile.am:
04:52 Changeset [41579] by jmalonzo@webkit.org
  • 7 edits in trunk

2009-03-11 Jan Michael Alonzo <jmalonzo@webkit.org>

Reviewed by Holger Freyther

[GTK]DumpRenderTree doesn't compile for non-X11 GTK ports anymore
https://bugs.webkit.org/show_bug.cgi?id=2260

pangoft2 is also used in directfb builds so use it for all targets

  • configure.ac:

2009-03-11 Jan Michael Alonzo <jmalonzo@webkit.org>

Reviewed by Holger Freyther

[GTK]DumpRenderTree doesn't compile for non-X11 GTK ports anymore
https://bugs.webkit.org/show_bug.cgi?id=2260

Add missing stdio.h include to get the build going

  • plugins/gtk/PluginPackageGtk.cpp:

2009-03-11 Jan Michael Alonzo <jmalonzo@webkit.org>

Reviewed by Holger Freyther.

[GTK]DumpRenderTree doesn't compile for non-X11 GTK ports anymore
https://bugs.webkit.org/show_bug.cgi?id=2260

Add plugin support only for X11 builds

  • DumpRenderTree/gtk/DumpRenderTree.cpp: (setDefaultsToConsistentStateValuesForTesting):
  • GNUmakefile.am:

03/10/09:

23:46 Changeset [41578] by justin.garcia@apple.com
  • 2 edits in trunk/WebCore

2009-03-10 Justin Garcia <justin.garcia@apple.com>

Reviewed by Ken Kocienda.

  • dom/Range.cpp: (WebCore::Range::addLineBoxRects): If the range ended at [div, 0], addLineBoxRects would include rects for the divs contents. Set the correct RenderObject to stop at.
23:28 Changeset [41577] by ap@webkit.org
  • 5 edits in trunk

Reviewed by Sam Weinig.

<rdar://problem/6123770> Restrict access to document.cookie when making a cross-site XHR

WebCore:

  • xml/XMLHttpRequest.cpp: (WebCore::XMLHttpRequest::responseXML): Removed an incorrect comment about cookie support. Firefox doesn't expose cookies on responseXML at all, and there are security concerns with exposing them for cross-origin requests, so it's not clear if we want to change anything here.

LayoutTests:

  • http/tests/security/cookies/xmlhttprequest.html: We don't expose cookies for documents retrieved via XMLHttpRequest. Added a check to make sure that we don't forget about cross- origin restrictions if we ever decide to change that.
  • http/tests/xmlhttprequest/resources/get-set-cookie.cgi: While at it, fixed a broken check for Set-Cookie2.
22:43 Changeset [41576] by simon.fraser@apple.com
  • 7 edits
    2 adds in trunk

2009-03-10 Simon Fraser <simon.fraser@apple.com>

Reviewed by Darin Adler

https://bugs.webkit.org/show_bug.cgi?id=24503

Fix hit testing of absolutely positioned single line text controls by
ensuring that we set result.innerNode() correctly. If the hit node is
a descendant of the inner text element or if it is the <input> itself,
then we say we hit the innerTextElement.

Rename hitInnerTextBlock() to hitInnerTextElement() to match the
'innerTextElement' terminology used elsewhere.

Assert that if renderer()->hitTest() returns false, no-one set
result.innerNode().

Test: fast/forms/search-abs-pos-cancel-button.html

  • rendering/RenderLayer.cpp: (WebCore::RenderLayer::hitTestContents):
  • rendering/RenderTextControl.cpp: (WebCore::RenderTextControl::hitInnerTextElement):
  • rendering/RenderTextControl.h:
  • rendering/RenderTextControlMultiLine.cpp: (WebCore::RenderTextControlMultiLine::nodeAtPoint):
  • rendering/RenderTextControlSingleLine.cpp: (WebCore::RenderTextControlSingleLine::nodeAtPoint):
21:03 Changeset [41575] by mrowe@apple.com
  • 4 edits in branches/Safari-6528

Versioning.

21:01 Changeset [41574] by mrowe@apple.com
  • 1 copy in tags/Safari-6528.17

New tag.

19:20 Changeset [41573] by antti@apple.com
  • 2 edits in trunk/WebCore

2009-03-10 Antti Koivisto <antti@apple.com>

Reviewed by Mark Rowe.

<rdar://problem/6668238> WebCore is registering text encodings needlessly from KURL constructor.


Another case where we would trigger extended encoding loading needlessly.

  • platform/text/TextEncoding.cpp: (WebCore::TextEncoding::isNonByteBasedEncoding):
17:19 Changeset [41572] by mrowe@apple.com
  • 13 edits in branches/Safari-6528

Merge r41190.

17:19 Changeset [41571] by mrowe@apple.com
  • 10 edits
    3 adds in branches/Safari-6528

Merge r41232.

17:08 Changeset [41570] by justin.garcia@apple.com
  • 1 edit in trunk/WebCore/page/Frame.cpp

Fix build.

17:05 Changeset [41569] by bdakin@apple.com
  • 118 edits
    4 adds in trunk

WebCore:

2009-03-10 Beth Dakin <bdakin@apple.com>

Reviewed by Darin Adler.

Fix for <rdar://problem/6624769> REGRESSION (Safari 4 PB): No
scroll bar appears for long line of text with NOWRAP set

This is a regression from http://trac.webkit.org/changeset/32226
I talked with Dan about the original change, and we decided that
the best fix was to remove his small potential-optimization that
only created a separate line box for whitespace under certain
circumstances. This new code will always create a separate line
box.

  • rendering/bidi.cpp: (WebCore::RenderBlock::layoutInlineChildren):

LayoutTests:

2009-03-10 Beth Dakin <bdakin@apple.com>

Reviewed by Darin Adler.

Test changes for <rdar://problem/6624769> REGRESSION (Safari 4 PB):
No scroll bar appears for long line of text with NOWRAP set

New test:

  • fast/inline/long-wrapped-line.html: Added.
  • platform/mac/fast/inline/long-wrapped-line-expected.checksum: Added.
  • platform/mac/fast/inline/long-wrapped-line-expected.png: Added.
  • platform/mac/fast/inline/long-wrapped-line-expected.txt: Added.


These tests all have changed results from the patch. Now trailing
whitespace always has its own linebox instead of sometimes sharing
a linebox with text.

  • platform/mac/editing/deleting/5206311-1-expected.txt:
  • platform/mac/editing/deleting/delete-block-merge-contents-018-expected.txt:
  • platform/mac/editing/deleting/delete-block-merge-contents-021-expected.txt:
  • platform/mac/editing/deleting/delete-block-merge-contents-022-expected.txt:
  • platform/mac/editing/deleting/delete-block-table-expected.txt:
  • platform/mac/editing/deleting/delete-leading-ws-001-expected.txt:
  • platform/mac/editing/deleting/delete-line-end-ws-001-expected.txt:
  • platform/mac/editing/deleting/delete-line-end-ws-002-expected.txt:
  • platform/mac/editing/deleting/delete-to-select-table-expected.txt:
  • platform/mac/editing/deleting/delete-ws-fixup-001-expected.txt:
  • platform/mac/editing/deleting/delete-ws-fixup-002-expected.txt:
  • platform/mac/editing/deleting/deletionUI-single-instance-expected.txt:
  • platform/mac/editing/deleting/forward-delete-expected.txt:
  • platform/mac/editing/execCommand/5482023-expected.txt:
  • platform/mac/editing/execCommand/boldSelection-expected.txt:
  • platform/mac/editing/execCommand/italicizeByCharacter-expected.txt:
  • platform/mac/editing/execCommand/modifyForeColorByCharacter-expected.txt:
  • platform/mac/editing/execCommand/print-expected.txt:
  • platform/mac/editing/execCommand/selectAll-expected.txt:
  • platform/mac/editing/execCommand/strikethroughSelection-expected.txt:
  • platform/mac/editing/input/emacs-ctrl-o-expected.txt:
  • platform/mac/editing/inserting/12882-expected.txt:
  • platform/mac/editing/inserting/editable-html-element-expected.txt:
  • platform/mac/editing/inserting/insert-div-019-expected.txt:
  • platform/mac/editing/inserting/insert-div-020-expected.txt:
  • platform/mac/editing/inserting/insert-div-021-expected.txt:
  • platform/mac/editing/inserting/insert-div-022-expected.txt:
  • platform/mac/editing/inserting/insert-div-023-expected.txt:
  • platform/mac/editing/inserting/insert-div-024-expected.txt:
  • platform/mac/editing/inserting/insert-div-025-expected.txt:
  • platform/mac/editing/inserting/insert-div-027-expected.txt:
  • platform/mac/editing/inserting/typing-003-expected.txt:
  • platform/mac/editing/pasteboard/4989774-expected.txt:
  • platform/mac/editing/pasteboard/5028447-expected.txt:
  • platform/mac/editing/pasteboard/merge-end-3-expected.txt:
  • platform/mac/editing/pasteboard/paste-table-003-expected.txt:
  • platform/mac/editing/pasteboard/paste-text-012-expected.txt:
  • platform/mac/editing/pasteboard/paste-text-013-expected.txt:
  • platform/mac/editing/pasteboard/paste-text-014-expected.txt:
  • platform/mac/editing/selection/4947387-expected.txt:
  • platform/mac/editing/selection/4983858-expected.txt:
  • platform/mac/editing/selection/5081257-1-expected.txt:
  • platform/mac/editing/selection/5081257-2-expected.txt:
  • platform/mac/editing/selection/5136696-expected.txt:
  • platform/mac/editing/selection/5234383-1-expected.txt:
  • platform/mac/editing/selection/5234383-2-expected.txt:
  • platform/mac/editing/selection/7152-1-expected.txt:
  • platform/mac/editing/selection/7152-2-expected.txt:
  • platform/mac/editing/selection/after-line-wrap-expected.txt:
  • platform/mac/editing/selection/click-start-of-line-expected.txt:
  • platform/mac/editing/selection/drag-in-iframe-expected.txt:
  • platform/mac/editing/selection/editable-html-element-expected.txt:
  • platform/mac/editing/selection/editable-non-editable-crash-expected.txt:
  • platform/mac/editing/selection/end-of-document-expected.txt:
  • platform/mac/editing/selection/extend-by-character-003-expected.txt:
  • platform/mac/editing/selection/extend-by-character-005-expected.txt:
  • platform/mac/editing/selection/extend-by-word-002-expected.txt:
  • platform/mac/editing/selection/mixed-editability-1-expected.txt:
  • platform/mac/editing/selection/move-by-character-003-expected.txt:
  • platform/mac/editing/selection/move-by-character-005-expected.txt:
  • platform/mac/editing/selection/select-all-001-expected.txt:
  • platform/mac/editing/selection/select-all-002-expected.txt:
  • platform/mac/editing/selection/select-all-003-expected.txt:
  • platform/mac/editing/selection/wrapped-line-caret-1-expected.txt:
  • platform/mac/editing/selection/wrapped-line-caret-2-expected.txt:
  • platform/mac/editing/style/apple-style-editable-mix-expected.txt:
  • platform/mac/editing/style/create-block-for-style-001-expected.txt:
  • platform/mac/editing/style/create-block-for-style-002-expected.txt:
  • platform/mac/editing/style/create-block-for-style-003-expected.txt:
  • platform/mac/editing/style/create-block-for-style-004-expected.txt:
  • platform/mac/editing/style/create-block-for-style-005-expected.txt:
  • platform/mac/editing/style/create-block-for-style-006-expected.txt:
  • platform/mac/editing/style/create-block-for-style-007-expected.txt:
  • platform/mac/editing/style/create-block-for-style-008-expected.txt:
  • platform/mac/editing/style/create-block-for-style-009-expected.txt:
  • platform/mac/editing/style/create-block-for-style-010-expected.txt:
  • platform/mac/editing/style/create-block-for-style-011-expected.txt:
  • platform/mac/editing/style/create-block-for-style-012-expected.txt:
  • platform/mac/editing/style/create-block-for-style-013-expected.txt:
  • platform/mac/editing/style/relative-font-size-change-001-expected.txt:
  • platform/mac/editing/style/relative-font-size-change-002-expected.txt:
  • platform/mac/editing/style/relative-font-size-change-003-expected.txt:
  • platform/mac/editing/style/relative-font-size-change-004-expected.txt:
  • platform/mac/editing/undo/undo-combined-delete-boundary-expected.txt:
  • platform/mac/editing/undo/undo-combined-delete-expected.txt:
  • platform/mac/editing/undo/undo-delete-boundary-expected.txt:
  • platform/mac/editing/undo/undo-delete-expected.txt:
  • platform/mac/editing/undo/undo-forward-delete-boundary-expected.txt:
  • platform/mac/editing/undo/undo-forward-delete-expected.txt:
  • platform/mac/fast/block/float/editable-text-overlapping-float-expected.txt:
  • platform/mac/fast/clip/outline-overflowClip-expected.txt:
  • platform/mac/fast/forms/negativeLineHeight-expected.txt:
  • platform/mac/fast/forms/textAreaLineHeight-expected.txt:
  • platform/mac/fast/forms/textarea-rows-cols-expected.txt:
  • platform/mac/fast/lists/drag-into-marker-expected.txt:
  • platform/mac/fast/parser/comment-in-textarea-expected.txt:
  • platform/mac/fast/parser/entity-comment-in-textarea-expected.txt:
  • platform/mac/fast/parser/open-comment-in-textarea-expected.txt:
  • platform/mac/fast/repaint/inline-outline-repaint-expected.txt:
  • platform/mac/fast/repaint/selection-after-delete-expected.txt:
  • platform/mac/fast/repaint/selection-after-remove-expected.txt:
  • platform/mac/fast/text/embed-at-end-of-pre-wrap-line-expected.txt:
  • platform/mac/fast/text/trailing-white-space-expected.txt:
  • platform/mac/fast/text/whitespace/pre-wrap-line-test-expected.txt:
  • platform/mac/fast/text/whitespace/pre-wrap-overflow-selection-expected.txt:
  • platform/mac/fast/text/whitespace/pre-wrap-spaces-after-newline-expected.txt:
  • platform/mac/fast/text/whitespace/tab-character-basics-expected.txt:
  • platform/mac/http/tests/navigation/anchor-goback-expected.txt:
  • platform/mac/http/tests/navigation/javascriptlink-goback-expected.txt:
  • platform/mac/http/tests/navigation/metaredirect-goback-expected.txt:
  • platform/mac/http/tests/navigation/post-goback2-expected.txt:
  • platform/mac/http/tests/navigation/postredirect-goback2-expected.txt:
  • platform/mac/http/tests/navigation/redirect302-goback-expected.txt:
  • platform/mac/http/tests/navigation/success200-goback-expected.txt:
  • platform/mac/http/tests/navigation/timerredirect-goback-expected.txt:
16:57 Changeset [41568] by weinig@apple.com
  • 3 edits in trunk/WebCore

2009-03-10 Sam Weinig <sam@webkit.org>

Reviewed by Geoffrey Garen.

Fix for <rdar://problem/6166844>
https://bugs.webkit.org/show_bug.cgi?id=24495

Use same rule for loading java applets as we do for images.

  • html/HTMLAppletElement.cpp: (WebCore::HTMLAppletElement::createRenderer):
  • loader/FrameLoader.cpp: (WebCore::FrameLoader::loadItem):
16:23 Changeset [41567] by justin.garcia@apple.com
  • 1 edit in trunk/WebCore/page/Frame.h

Fix the build.

15:59 Changeset [41566] by justin.garcia@apple.com
  • 5 edits in trunk/WebCore

2009-03-10 Justin Garcia <justin.garcia@apple.com>

Reviewed by David Kilzer.

Get rid of revealCaret and use revealSelection instead.

  • WebCore.base.exp:
  • editing/SelectionController.cpp: (WebCore::SelectionController::setSelection):
  • page/Frame.cpp: (WebCore::Frame::revealSelection):
  • page/Frame.h:
15:48 Changeset [41565] by cwzwarich@webkit.org
  • 3 edits
    5 adds in trunk

2009-03-10 Cameron Zwarich <cwzwarich@uwaterloo.ca>

Reviewed by Geoff Garen.

Bug 24291: REGRESSION (r38635): Single line JavaScript comment prevents HTML button click handler execution
<https://bugs.webkit.org/show_bug.cgi?id=24291>
<rdar://problem/6663472>

Add an extra newline to the end of the body of the program text constructed
by the Function constructor for parsing. This allows single line comments to
be handled correctly by the parser.

JavaScriptCore:

  • runtime/FunctionConstructor.cpp: (JSC::constructFunction):

LayoutTests:

  • fast/events/onload-single-line-comment-expected.txt: Added.
  • fast/events/onload-single-line-comment.html: Added.
  • fast/js/function-constructor-single-line-comment-expected.txt: Added.
  • fast/js/function-constructor-single-line-comment.html: Added.
  • fast/js/resources/function-constructor-single-line-comment.js: Added.
14:46 Changeset [41564] by sfalken@apple.com
  • 2 edits in trunk/WebKit/win

<rdar://problem/6662167> When clicking in Top Sites search field, WebHistory::findIndex fails causing crash in BookmarksSearcher::collectHistorySearchResults


Round CFAbsoluteDates to the nearest second when converting to/from the Windows DATE format.


This corrects for inaccuracies introduced by round-tripping between DATE (day based) and CFAbsoluteDate (second based).
The WebKit COM API on Windows uses DATE, while our history storage uses CFAbsoluteTime. This could lead to WebKit
saying there was browsing history for a particular day, and then return no history items when we requested a
list of sites visited that day.


Reviewed by Ada Chan.

  • MarshallingHelpers.cpp: (MarshallingHelpers::DATEToCFAbsoluteTime): (MarshallingHelpers::CFAbsoluteTimeToDATE):
12:49 Changeset [41563] by cwzwarich@webkit.org
  • 3 edits
    2 adds in trunk

2009-03-10 Cameron Zwarich <cwzwarich@uwaterloo.ca>

Reviewed by Geoff Garen.

Bug 23736: WebKit Crashes on http://g-conquest.fr/~server2
<https://bugs.webkit.org/show_bug.cgi?id=23736>
<rdar://problem/6560278>

Before replacing the document, FrameLoader::executeIfJavaScriptURL()
should call stopAllLoaders(), just like continueLoadAfterNavigationPolicy().

WebCore:

  • loader/FrameLoader.cpp: (WebCore::FrameLoader::executeIfJavaScriptURL):

LayoutTests:

  • http/tests/misc/javascript-url-stop-loaders-expected.txt: Added.
  • http/tests/misc/javascript-url-stop-loaders.html: Added.
12:31 Changeset [41562] by darin@apple.com
  • 4 edits
    2 adds in trunk

WebCore:

2009-03-10 Darin Adler <darin@apple.com>

Reviewed by Alexey Proskuryakov.

Bug 24494: crash when deleting at end of document and merging paragraphs
https://bugs.webkit.org/show_bug.cgi?id=24494
rdar://problem/6571537

Test: editing/deleting/merge-at-end-of-document.html

  • dom/Range.cpp: (WebCore::Range::compareBoundaryPoints): Split out assertion. It's better not to use && in assertions since we'd like to know which condition is failing.
  • editing/ApplyStyleCommand.cpp: (WebCore::ApplyStyleCommand::applyInlineStyleToRange): Added a null check before calling compareBoundaryPoints, since a 0 for the node is ambiguous and so the function doesn't know which value to return.

LayoutTests:

2009-03-10 Darin Adler <darin@apple.com>

Reviewed by Alexey Proskuryakov.

Bug 24494: crash when deleting at end of document and merging paragraphs
https://bugs.webkit.org/show_bug.cgi?id=24494
rdar://problem/6571537

  • editing/deleting/merge-at-end-of-document-expected.txt: Added.
  • editing/deleting/merge-at-end-of-document.html: Added.
11:46 Changeset [41561] by krit@webkit.org
  • 5 edits in trunk/WebCore

2009-03-10 Dirk Schulze <krit@webkit.org>

Reviewed by Eric Seidel. Math checked by Oliver Hunt.

Implement the Cairo version of the checkForSolidColor() method. This halfes the
time to draw on 1x1 px background images.
I added two new calls to Color for the pixel manipulation on cairo_surface's.
They are neede to premultiply/unpremultiply the colors of the surface.

[CAIRO] Introduce single-pixel image optimizations
https://bugs.webkit.org/show_bug.cgi?id=17284

  • platform/graphics/Color.cpp: (WebCore::colorFromPremultipliedARGB): (WebCore::premultipliedARGBFromColor):
  • platform/graphics/Color.h:
  • platform/graphics/cairo/ImageBufferCairo.cpp: (WebCore::ImageBuffer::getImageData): (WebCore::ImageBuffer::putImageData):
  • platform/graphics/cairo/ImageCairo.cpp: (WebCore::BitmapImage::checkForSolidColor):
11:38 Changeset [41560] by treat@webkit.org
  • 2 edits in trunk/WebKit/qt

2009-03-10 Adam Treat <adam.treat@torchmobile.com>

Build fix for Qt after r41555.

  • Api/qwebpage.cpp: (QWebPage::acceptNavigationRequest):
11:22 Changeset [41559] by xan@webkit.org
  • 2 edits in trunk/WebKit/gtk

2009-03-10 Xan Lopez <xlopez@igalia.com>

Reviewed by Alexey Proskuryakov.

https://bugs.webkit.org/show_bug.cgi?id=24493
[GTK] Misc patches for WebKitWebHistoryItem

Use g_hash_table_new_full so we can save the manual unref on the
values when removing them from the table.

  • webkit/webkitwebhistoryitem.cpp: (_WebKitWebHistoryItemPrivate::webkit_web_history_item_dispose):
10:34 Changeset [41558] by xan@webkit.org
  • 2 edits in trunk/WebKit/gtk

2009-03-10 Xan Lopez <xlopez@igalia.com>

Reviewed by Alexey Proskuryakov.

https://bugs.webkit.org/show_bug.cgi?id=24493
[GTK] Misc patches for WebKitWebHistoryItem

Use 'if (foo)' instead of 'if (foo != NULL)', per coding style
guidelines.

  • webkit/webkitwebhistoryitem.cpp: (_WebKitWebHistoryItemPrivate::webkit_history_item_remove): (_WebKitWebHistoryItemPrivate::webkit_web_history_item_get_title): (_WebKitWebHistoryItemPrivate::webkit_web_history_item_get_alternate_title): (_WebKitWebHistoryItemPrivate::webkit_web_history_item_get_uri): (_WebKitWebHistoryItemPrivate::webkit_web_history_item_get_original_uri): (_WebKitWebHistoryItemPrivate::webkit_web_history_item_get_last_visited_time): (_WebKitWebHistoryItemPrivate::webkit_web_history_item_get_target): (_WebKitWebHistoryItemPrivate::webkit_web_history_item_is_target_item): (_WebKitWebHistoryItemPrivate::webkit_web_history_item_get_children):
10:00 Changeset [41557] by xan@webkit.org
  • 4 edits in trunk/WebKit

WebKit/mac:

2009-03-10 Xan Lopez <xlopez@igalia.com>

Build fix, no review.

  • WebView/WebFrame.mm: (-[WebFrame _smartDeleteRangeForProposedRange:]):

WebKit/qt:

2009-03-10 Xan Lopez <xlopez@igalia.com>

Build fix, no review.

  • Api/qwebpage.cpp: (QWebPage::inputMethodQuery):
09:38 Changeset [41556] by xan@webkit.org
  • 2 edits in trunk/WebKit/gtk

2009-03-10 Xan Lopez <xlopez@igalia.com>

Reviewed by Alexey Proskuryakov.

Correct return value to false instead of NULL.

  • webkit/webkitwebhistoryitem.cpp: (_WebKitWebHistoryItemPrivate::webkit_web_history_item_is_target_item):
09:23 Changeset [41555] by darin@chromium.org
  • 5 edits in trunk/WebCore

2009-03-06 Darin Fisher <darin@chromium.org>

Reviewed by Sam Weinig.

https://bugs.webkit.org/show_bug.cgi?id=24437

Add support for registering noAccess URL schemes:
1- Add FrameLoader::registerURLSchemeAsNoAccess, and have SecurityOrigin check
that list upon construction (similar to how isLocal is implemented).
2- Make InspectorController call grantUniversalAccess on its Document's
SecurityOrigin at the time when windowScriptObjectAvailable is called.

This enables content such as the inspector to be loaded from a custom (non-file)
URL, which is how Chromium loads the inspector. It also allows other URL schemes
to be treated like data: URLs, which Chromium utilizes for its various HTML-based
UI panels.

  • inspector/InspectorController.cpp: (WebCore::InspectorController::windowScriptObjectAvailable):
  • loader/FrameLoader.cpp: (WebCore::localSchemes): (WebCore::noAccessSchemes): (WebCore::FrameLoader::loadItem):
  • loader/FrameLoader.h:
  • page/SecurityOrigin.cpp: (WebCore::SecurityOrigin::SecurityOrigin): (WebCore::SecurityOrigin::isLocal):
09:19 Changeset [41554] by xan@webkit.org
  • 2 edits in trunk/WebCore

2009-03-10 Xan Lopez <xlopez@igalia.com>

Build fix, no review

  • page/mac/AccessibilityObjectWrapper.mm: (textMarkerForVisiblePosition): (visiblePositionForTextMarker):
09:10 Changeset [41553] by xan@webkit.org
  • 30 edits in trunk/WebCore

2009-03-10 Xan Lopez <xlopez@igalia.com>

Reviewed by Darin Adler.

https://bugs.webkit.org/show_bug.cgi?id=24491
Rename posOffset to m_offset, get rid of accessor

As suggested in dom/Position.h, this patch gets rid of the
'offset()' accessor and renames posOffset to m_offset. I've used
m_offset instead of offset to follow the style guide lines, since
Position is still a class and not a structure. If the long term
plan is still to make it a structure it would be pretty easy to
just s/m_offset/offset/ globally when that is achieved.

  • dom/Position.cpp: (WebCore::Position::previous): (WebCore::Position::next): (WebCore::Position::atStart): (WebCore::Position::atEnd): (WebCore::Position::renderedOffset): (WebCore::Position::isCandidate): (WebCore::Position::inRenderedText): (WebCore::Position::isRenderedCharacter): (WebCore::Position::rendersInDifferentPosition): (WebCore::Position::leadingWhitespacePosition): (WebCore::Position::debugPosition): (WebCore::Position::formatForDebugger):
  • dom/Position.h: (WebCore::Position::Position):
  • dom/PositionIterator.h: (WebCore::PositionIterator::PositionIterator):
  • dom/Range.cpp:
  • dom/RangeBoundaryPoint.h: (WebCore::RangeBoundaryPoint::position): (WebCore::RangeBoundaryPoint::set): (WebCore::RangeBoundaryPoint::setOffset): (WebCore::RangeBoundaryPoint::setToChild): (WebCore::RangeBoundaryPoint::setToStart): (WebCore::RangeBoundaryPoint::setToEnd): (WebCore::RangeBoundaryPoint::childBeforeWillBeRemoved):
  • editing/ApplyStyleCommand.cpp: (WebCore::ApplyStyleCommand::applyRelativeFontStyleChange): (WebCore::ApplyStyleCommand::applyInlineStyleToRange): (WebCore::ApplyStyleCommand::removeInlineStyle): (WebCore::ApplyStyleCommand::nodeFullySelected): (WebCore::ApplyStyleCommand::nodeFullyUnselected): (WebCore::ApplyStyleCommand::splitTextAtStartIfNeeded): (WebCore::ApplyStyleCommand::splitTextAtEndIfNeeded): (WebCore::ApplyStyleCommand::splitTextElementAtStartIfNeeded): (WebCore::ApplyStyleCommand::splitTextElementAtEndIfNeeded): (WebCore::ApplyStyleCommand::mergeStartWithPreviousIfIdentical): (WebCore::ApplyStyleCommand::mergeEndWithNextIfIdentical): (WebCore::ApplyStyleCommand::joinChildTextNodes):
  • editing/BreakBlockquoteCommand.cpp: (WebCore::BreakBlockquoteCommand::doApply):
  • editing/CompositeEditCommand.cpp: (WebCore::CompositeEditCommand::insertNodeAt): (WebCore::CompositeEditCommand::positionOutsideTabSpan): (WebCore::CompositeEditCommand::rebalanceWhitespaceAt): (WebCore::CompositeEditCommand::prepareWhitespaceAtPositionForSplit): (WebCore::CompositeEditCommand::deleteInsignificantText): (WebCore::CompositeEditCommand::removePlaceholderAt): (WebCore::CompositeEditCommand::moveParagraphs): (WebCore::CompositeEditCommand::breakOutOfEmptyMailBlockquotedParagraph):
  • editing/DeleteSelectionCommand.cpp: (WebCore::updatePositionForNodeRemoval): (WebCore::updatePositionForTextRemoval): (WebCore::DeleteSelectionCommand::handleGeneralDelete): (WebCore::DeleteSelectionCommand::fixupWhitespace):
  • editing/Editor.cpp: (WebCore::Editor::setComposition): (WebCore::Editor::advanceToNextMisspelling): (WebCore::Editor::getCompositionSelection):
  • editing/InsertLineBreakCommand.cpp: (WebCore::InsertLineBreakCommand::doApply):
  • editing/InsertParagraphSeparatorCommand.cpp: (WebCore::InsertParagraphSeparatorCommand::doApply):
  • editing/InsertTextCommand.cpp: (WebCore::InsertTextCommand::performTrivialReplace): (WebCore::InsertTextCommand::input): (WebCore::InsertTextCommand::insertTab):
  • editing/MoveSelectionCommand.cpp: (WebCore::MoveSelectionCommand::doApply):
  • editing/ReplaceSelectionCommand.cpp: (WebCore::ReplaceSelectionCommand::doApply):
  • editing/SelectionController.cpp: (WebCore::SelectionController::debugRenderer):
  • editing/TextIterator.cpp: (WebCore::TextIterator::rangeFromLocationAndLength):
  • editing/TypingCommand.cpp: (WebCore::TypingCommand::deleteKeyPressed): (WebCore::TypingCommand::forwardDeleteKeyPressed):
  • editing/VisiblePosition.cpp: (WebCore::VisiblePosition::characterAfter): (WebCore::VisiblePosition::debugPosition): (WebCore::makeRange): (WebCore::setStart): (WebCore::setEnd):
  • editing/VisibleSelection.cpp: (WebCore::VisibleSelection::toNormalizedRange): (WebCore::makeSearchRange): (WebCore::VisibleSelection::debugPosition): (WebCore::VisibleSelection::showTreeForThis):
  • editing/htmlediting.cpp: (WebCore::comparePositions): (WebCore::rangeCompliantEquivalent): (WebCore::isFirstPositionAfterTable): (WebCore::isLastPositionBeforeTable):
  • editing/visible_units.cpp: (WebCore::previousBoundary): (WebCore::nextBoundary): (WebCore::startPositionForLine): (WebCore::startOfLine): (WebCore::endPositionForLine): (WebCore::nextLinePosition): (WebCore::startOfParagraph): (WebCore::endOfParagraph):
  • page/AccessibilityObject.cpp: (WebCore::updateAXLineStartForVisiblePosition):
  • page/AccessibilityRenderObject.cpp: (WebCore::AccessibilityRenderObject::indexForVisiblePosition):
  • page/DOMSelection.cpp: (WebCore::DOMSelection::anchorOffset): (WebCore::DOMSelection::baseOffset): (WebCore::DOMSelection::focusOffset): (WebCore::DOMSelection::extentOffset):
  • page/EventHandler.cpp: (WebCore::EventHandler::handleMousePressEventSingleClick):
  • page/Frame.cpp: (WebCore::Frame::selectionLayoutChanged):
  • rendering/RenderTextControl.cpp: (WebCore::RenderTextControl::indexForVisiblePosition):
  • rendering/RenderTreeAsText.cpp: (WebCore::writeSelection):
07:46 Changeset [41552] by darin@apple.com
  • 3 edits
    2 adds in trunk

WebCore:

2009-03-10 Darin Adler <darin@apple.com>

Reviewed by Dan Bernstein.

Bug 23564: REGRESSION (r39230-39286): crash loading page that changes <input> display type and then calls innerHTML
https://bugs.webkit.org/show_bug.cgi?id=23564
rdar://problem/6537238

Test: fast/dom/HTMLElement/innerHTML-selection-crash.html

  • editing/markup.cpp: (WebCore::createMarkup): Added updateLayoutIgnorePendingStylesheets call to the one of the two overloads of this function that wasn't calling it. This fixes this crash and other possible crashes inside innerHTML.

LayoutTests:

2009-03-10 Darin Adler <darin@apple.com>

Reviewed by Dan Bernstein.

Bug 23564: REGRESSION (r39230-39286): crash loading page that changes <input> display type and then calls innerHTML
https://bugs.webkit.org/show_bug.cgi?id=23564
rdar://problem/6537238

  • fast/dom/HTMLElement/innerHTML-selection-crash-expected.txt: Added.
  • fast/dom/HTMLElement/innerHTML-selection-crash.html: Added.
07:10 Changeset [41551] by ap@webkit.org
  • 16 edits
    2 deletes in trunk/WebCore

Reviewed by Darin Adler.

https://bugs.webkit.org/show_bug.cgi?id=24285
Text resource loading checks for BOM twice

This removes TextDecoder class, since its only purpose was to check for BOM, which is
already done in TextResourceDecoder. Callers that use TextEncoding::decode() won't get
BOM checked, but I didn't find any cases where it would significantly change behavior.

  • GNUmakefile.am:
  • WebCore.pro:
  • WebCore.scons:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • WebCoreSources.bkl: Remove TextDecoder.{h,cpp}.
  • platform/text/TextDecoder.cpp: Removed.
  • platform/text/TextDecoder.h: Removed.
  • loader/TextResourceDecoder.h: Instead of a TextDecoder object, keep TextEncoding and TextCodec separately.
  • loader/TextResourceDecoder.cpp: (WebCore::TextResourceDecoder::TextResourceDecoder): Updated for m_encoding being a member. (WebCore::TextResourceDecoder::setEncoding): Ditto. (WebCore::TextResourceDecoder::checkForBOM): Removed a FIXME saying that a BOM could override even a user-chosen encoding - this is how it already worked due to TextDecoder checking for BOM again. Made this function return the detected BOM length. (WebCore::TextResourceDecoder::decode): Skip the BOM if it was found at the beginning of a text resource. (WebCore::TextResourceDecoder::flush): Reset m_checkedForBOM, so that re-decoding the same resource again (as frequently done by CachedResource subclasses) will skip the BOM correctly.
  • platform/text/TextEncoding.cpp: (WebCore::TextEncoding::decode): Use TextCodec directly without a TextDecoder wrapper. This means that this method no longer checks for BOM, which was a counter-intuitive feature.
  • loader/CachedScript.cpp: (WebCore::CachedScript::CachedScript): (WebCore::CachedScript::setEncoding): (WebCore::CachedScript::encoding): (WebCore::CachedScript::script):
  • loader/CachedScript.h:
  • loader/appcache/ManifestParser.cpp: (WebCore::parseManifest): Use TextResourceDecoder, as TextEncoding::decode() no longer checks for BOM. A side effect of this is that these resources will now be subject to encoding auto-detection.
  • loader/CachedFont.cpp: (WebCore::CachedFont::ensureSVGFontData):
  • page/Page.cpp: (WebCore::Page::userStyleSheet): Be sure to flush TextResourceDecoder, pushing any remaining bytes out, and making the decoder re-usable (for repeated decoding of the same resource).
  • platform/text/TextEncodingRegistry.h: Updated comments for newTextCodec().
05:58 Changeset [41550] by ap@webkit.org
  • 2 edits in trunk/WebCore

Reviewed by Darin Adler.

https://bugs.webkit.org/show_bug.cgi?id=24488
An assertion failure when updating an offline application cache after emptying caches

Emptying appcache cannot be performed automatically, so no test.

  • loader/appcache/ApplicationCacheGroup.cpp: (WebCore::ApplicationCacheGroup::checkIfLoadIsComplete): Ensure that the cache is in disk storage, even if the application wasn't updated server side.
02:44 Changeset [41549] by oliver@apple.com
  • 19 edits
    9 adds in trunk

<rdar://problem/6425807> Implement WorkerUtils.importScripts()
<https://bugs.webkit.org/show_bug.cgi?id=22721>

Reviewed by Alexey Proskuryakov

Implement importScripts, currently uses a series of synchronous loads
to fetch the scripts, but this is simpler than a synchronous load of
multiple loads in parallel. In future we'll want to switch to parallel
loading, but this will do for now.

Test: http/tests/workers/worker-importScripts.html

01:11 Changeset [41548] by ap@webkit.org
  • 9 edits
    4 adds in trunk/WebCore

Reviewed by Darin Adler.

https://bugs.webkit.org/show_bug.cgi?id=24462
Move cross-origin access control code out of XMLHttpRequest

Step 1: move preflight result cache and access control helper functions.

No change in behavior, so no test.

  • GNUmakefile.am:
  • WebCore.pro:
  • WebCore.scons:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • WebCoreSources.bkl:
  • loader/CrossOriginAccessControl.cpp: Added. (WebCore::isOnAccessControlSimpleRequestHeaderWhitelist): (WebCore::isSimpleCrossOriginAccessRequest): (WebCore::createAllowedCrossOriginResponseHeadersSet): (WebCore::isOnAccessControlResponseHeaderWhitelist): (WebCore::passesAccessControlCheck):
  • loader/CrossOriginAccessControl.h: Added.
  • loader/CrossOriginPreflightResultCache.cpp: Added. (WebCore::parseAccessControlMaxAge): (WebCore::addToAccessControlAllowList): (WebCore::parseAccessControlAllowList): (WebCore::CrossOriginPreflightResultCacheItem::parse): (WebCore::CrossOriginPreflightResultCacheItem::allowsCrossOriginMethod): (WebCore::CrossOriginPreflightResultCacheItem::allowsCrossOriginHeaders): (WebCore::CrossOriginPreflightResultCacheItem::allowsRequest): (WebCore::CrossOriginPreflightResultCache::shared): (WebCore::CrossOriginPreflightResultCache::appendEntry): (WebCore::CrossOriginPreflightResultCache::canSkipPreflight):
  • loader/CrossOriginPreflightResultCache.h: Added. (WebCore::CrossOriginPreflightResultCacheItem::CrossOriginPreflightResultCacheItem): (WebCore::CrossOriginPreflightResultCache::CrossOriginPreflightResultCache):
  • xml/XMLHttpRequest.cpp: (WebCore::XMLHttpRequestStaticData::XMLHttpRequestStaticData): (WebCore::XMLHttpRequest::createRequest): (WebCore::XMLHttpRequest::makeCrossOriginAccessRequest): (WebCore::XMLHttpRequest::makeSimpleCrossOriginAccessRequest): (WebCore::XMLHttpRequest::makeCrossOriginAccessRequestWithPreflight): (WebCore::XMLHttpRequest::didReceiveResponse): (WebCore::XMLHttpRequest::didReceiveResponsePreflight):
  • xml/XMLHttpRequest.h:
00:46 Changeset [41547] by ap@webkit.org
  • 5 edits in trunk

Reviewed by Darin Adler.

https://bugs.webkit.org/show_bug.cgi?id=19797
Bring the list of forbidden headers in sync with XMLHttpRequest spec draft

Added new checks to http/tests/xmlhttprequest/set-dangerous-headers.html.

  • xml/XMLHttpRequest.cpp: (WebCore::XMLHttpRequestStaticData::XMLHttpRequestStaticData): Added all headers from XMLHttpRequest 2 draft.
00:03 Changeset [41546] by mitz@apple.com
  • 2 edits in trunk/WebCore

Reviewed by Adele Peterson.

  • rendering/RenderLayer.cpp: (WebCore::RenderLayer::invalidateScrollbarRect): Corrected the vertical scrollbar rect calculation.

03/09/09:

22:50 Changeset [41545] by simon.fraser@apple.com
  • 3 edits
    4 adds in trunk

2009-03-09 Simon Fraser <simon.fraser@apple.com>

Reviewed by Dan Bernstein, Oliver Hunt

https://bugs.webkit.org/show_bug.cgi?id=24475

When repainting replaced elements, need to union the selection
and overflow rects because either may extend outside the other.

Test: fast/repaint/transform-replaced-shadows.html

  • rendering/RenderReplaced.cpp: (WebCore::RenderReplaced::clippedOverflowRectForRepaint):
18:09 Changeset [41544] by oliver@apple.com
  • 5 edits
    3 adds in trunk

Bug 24447: REGRESSION (r41508): Google Maps does not complete initialization
<rdar://problem/6657774>

Reviewed by Gavin Barraclough

r41508 actually exposed a pre-existing bug where we were not invalidating the result
register cache at jump targets. This causes problems when condition loads occur in an

expression -- namely through the ?: and
operators. This patch corrects these issues

by marking the target of all forward jumps as being a jump target, and then clears the
result register cache when ever it starts generating code for a targeted instruction.

I do not believe it is possible to cause this class of failure outside of a single
expression, and expressions only provide forward branches, so this should resolve this
entire class of bug. That said i've included a test case that gets as close as possible
to hitting this bug with a back branch, to hopefully prevent anyone from introducing the
problem in future.

17:38 Changeset [41543] by slewis@apple.com
  • 2 edits in trunk/WebCore

2009-03-09 Stephanie Lewis <slewis@apple.com>

Fix build.

  • editing/Editor.cpp: (WebCore::Editor::advanceToNextMisspelling): (WebCore::markAllMisspellingsAndBadGrammarInRanges):
17:22 Changeset [41542] by darin@chromium.org
  • 2 edits in trunk/WebCore

2009-03-09 Kim Christensen <kimworking@gmail.com>

Reviewed by Darin Fisher.

https://bugs.webkit.org/show_bug.cgi?id=24276
Dropdown / select boxes do not consider the windows task bar in Chromium.

  • platform/chromium/PopupMenuChromium.cpp: (WebCore::PopupContainer::showPopup):
15:49 Changeset [41541] by kov@webkit.org
  • 1 copy in releases/WebKitGTK/webkit-1.1.1

Tagging WebKitGTK+'s 1.1.1 release.

15:40 Changeset [41540] by kov@webkit.org
  • 1 add in releases/WebKitGTK

Creating a place to tag WebKitGTK+ releases.

15:34 Changeset [41539] by weinig@apple.com
  • 2 edits
    14 deletes in trunk/LayoutTests

2009-03-09 Sam Weinig <sam@webkit.org>

Roll out r41393. It has been causing failures on the bots for about a week.

  • fast/block/float/resources/se_arrow.png: Removed.
  • fast/block/float/resources/yinyang.png: Removed.
  • fast/block/float/t0905-c414-flt-fit-01-d-g.html: Removed.
  • fast/block/float/t0905-c5525-fltblck-00-d-ag.html: Removed.
  • fast/block/float/t0905-c5526-flthw-00-c-g.html: Removed.
  • platform/mac/fast/block/float/t0905-c414-flt-fit-01-d-g-expected.checksum: Removed.
  • platform/mac/fast/block/float/t0905-c414-flt-fit-01-d-g-expected.png: Removed.
  • platform/mac/fast/block/float/t0905-c414-flt-fit-01-d-g-expected.txt: Removed.
  • platform/mac/fast/block/float/t0905-c5525-fltblck-00-d-ag-expected.checksum: Removed.
  • platform/mac/fast/block/float/t0905-c5525-fltblck-00-d-ag-expected.png: Removed.
  • platform/mac/fast/block/float/t0905-c5525-fltblck-00-d-ag-expected.txt: Removed.
  • platform/mac/fast/block/float/t0905-c5526-flthw-00-c-g-expected.checksum: Removed.
  • platform/mac/fast/block/float/t0905-c5526-flthw-00-c-g-expected.png: Removed.
  • platform/mac/fast/block/float/t0905-c5526-flthw-00-c-g-expected.txt: Removed.
15:20 Changeset [41538] by mitz@apple.com
  • 4 edits
    2 adds in trunk

WebCore:

Reviewed by Adele Peterson and Dave Hyatt.

  • fix <rdar://problem/6654344> REGRESSION (r41041): Zillow.com has misaligned edit fields

Test: fast/forms/legend-display-none.html

  • css/CSSStyleSelector.cpp: (WebCore::CSSStyleSelector::adjustRenderStyle): Change the display property of legend elements to 'block' unless it is 'none'.
  • css/html4.css: Do not force the display property of legend elements to 'block' using CSS, because that prevents style rules from setting it to 'none'.

LayoutTests:

Reviewed by Adele Peterson and Dave Hyatt.

  • test for <rdar://problem/6654344> REGRESSION (r41041): Zillow.com has misaligned edit fields
  • fast/forms/legend-display-none-expected.txt: Added.
  • fast/forms/legend-display-none.html: Added.
14:01 Changeset [41537] by levin@chromium.org
  • 3 edits
    2 adds in trunk

WebCore:

2009-03-09 Alpha Lam <hclam@chromium.org>

Reviewed by Dave Hyatt.

https://bugs.webkit.org/show_bug.cgi?id=23024

Fix an incorrect criteria in the if statement that leaves references to
deleted floating objects in RenderBlock.

Test: fast/block/float/crash-on-absolute-positioning.html

  • rendering/RenderBlock.cpp: (WebCore::RenderBlock::markAllDescendantsWithFloatsForLayout):

LayoutTests:

2009-03-09 Alpha Lam <hclam@chromium.org>

Reviewed by Dave Hyatt.

https://bugs.webkit.org/show_bug.cgi?id=23024

Added regression test for a bug that causes crashes.

  • fast/block/float/crash-on-absolute-positioning.html: Added.
  • platform/mac/fast/block/float/crash-on-absolute-positioning-expected.txt: Added.
14:01 Changeset [41536] by levin@chromium.org
  • 3 edits
    1 add in trunk/JavaScriptCore

2009-03-09 David Levin <levin@chromium.org>

Reviewed by Darin Adler.

Bug 23175: String and UString should be able to share a UChar* buffer.
<https://bugs.webkit.org/show_bug.cgi?id=23175>

Add CrossThreadRefCounted.

  • wtf/CrossThreadRefCounted.h: Added. (WTF::CrossThreadRefCounted::create): (WTF::CrossThreadRefCounted::isShared): (WTF::CrossThreadRefCounted::dataAccessMustBeThreadSafe): (WTF::CrossThreadRefCounted::mayBePassedToAnotherThread): (WTF::CrossThreadRefCounted::CrossThreadRefCounted): (WTF::CrossThreadRefCounted::~CrossThreadRefCounted): (WTF::CrossThreadRefCounted::ref): (WTF::CrossThreadRefCounted::deref): (WTF::CrossThreadRefCounted::release): (WTF::CrossThreadRefCounted::copy): (WTF::CrossThreadRefCounted::threadSafeDeref):
  • wtf/RefCounted.h:
  • wtf/Threading.h: (WTF::ThreadSafeSharedBase::ThreadSafeSharedBase): (WTF::ThreadSafeSharedBase::derefBase): (WTF::ThreadSafeShared::ThreadSafeShared): (WTF::ThreadSafeShared::deref):
12:48 Changeset [41535] by eric@webkit.org
  • 2 edits in trunk/WebCore

2009-03-09 Mike Belshe <mike@belshe.com>

Reviewed by Eric Seidel.

Unify JSC/V8 idl file.
https://bugs.webkit.org/show_bug.cgi?id=24424

  • page/Location.idl:
12:48 Changeset [41534] by eric@webkit.org
  • 2 edits in trunk/WebCore

2009-03-09 Mike Belshe <mike@belshe.com>

Reviewed by Eric Seidel.

Unify JSC/V8 for History.idl.

  • page/History.idl:
12:48 Changeset [41533] by eric@webkit.org
  • 2 edits in trunk/WebCore

2009-03-09 Mike Belshe <mike@belshe.com>

Reviewed by Eric Seidel.

Unify JSC/V8 implementations of DOMWindow.idl.

  • page/DOMWindow.idl:
11:39 Changeset [41532] by andersca@apple.com
  • 8 edits in trunk/WebKit/mac

2009-03-09 Anders Carlsson <andersca@apple.com>

Reviewed by Kevin Decker.

WebKit side of <rdar://problem/6530007>


  • Plugins/Hosted/NetscapePluginHostProxy.mm: (WKPCEnumerate): Call NetscapePluginInstanceProxy::enumerate.


  • Plugins/Hosted/NetscapePluginInstanceProxy.h:
  • Plugins/Hosted/NetscapePluginInstanceProxy.mm: (WebKit::NetscapePluginInstanceProxy::enumerate): Enumerate the JS object and serialize its values.


  • Plugins/Hosted/ProxyInstance.h:
  • Plugins/Hosted/ProxyInstance.mm: (WebKit::ProxyInstance::getPropertyNames): Ask the plug-in host to get the property names and deserialize them.


  • Plugins/Hosted/WebKitPluginClient.defs:
  • Plugins/Hosted/WebKitPluginHost.defs:
11:06 Changeset [41531] by simon.fraser@apple.com
  • 16 edits in trunk/WebCore

2009-03-09 Simon Fraser <simon.fraser@apple.com>

Reviewed by Dave Hyatt

https://bugs.webkit.org/show_bug.cgi?id=24163

Implement mapping of FloatQuads through 3D transforms.

Removed the redundant localToContainerQuad() methods, which can now
shared code with the old mapLocalToAbsolutePoint(), which was
renamed to mapLocalToContainer(). This can now convert a point,
and optionally a FloatQuad, which are carried along in the TransformState.

Optimized TransformState to reduce to simple FloatPoint.move()
if there are no transforms, and to heap-allocate a transform only if
necessary to accumulate transforms (when using preserve-3d).

Tested by 3d point mapping tests, and the inspector highlight (which now shows
the correct quads for 3d-transformed elements).

10:01 Changeset [41530] by simon.fraser@apple.com
  • 3 edits
    2 adds in trunk

2009-03-09 Simon Fraser <simon.fraser@apple.com>

Reviewed by Dave Hyatt

https://bugs.webkit.org/show_bug.cgi?id=24433

During hit testing with transforms, if a layer has backface-visibility: hidden,
we need to check for it whether or not the layer has a transform, because it
might be an ancestor that is transformed.

Test: transforms/3d/hit-testing/backface-no-transform-hit-test.html

  • rendering/RenderLayer.cpp: (WebCore::RenderLayer::hitTestLayer):
09:53 Changeset [41529] by simon.fraser@apple.com
  • 2 edits in trunk/WebKit/mac

2009-03-09 Simon Fraser <simon.fraser@apple.com>

Reviewed by Oliver Hunt and Cameron Zwarich

https://bugs.webkit.org/show_bug.cgi?id=24440

The sublayer added to WebHTMLView to host accelerated compositing layers needs to
be a subclass of NSView which allows context menu clicks through.

  • WebView/WebHTMLView.mm: (-[WebLayerHostingView rightMouseDown:]): (-[WebHTMLView attachRootLayer:]):
09:37 Changeset [41528] by treat@webkit.org
  • 4 edits in trunk

2009-03-09 Laszlo Gombos <laszlo.1.gombos@nokia.com>

Reviewed by George Staikos.

https://bugs.webkit.org/show_bug.cgi?id=24353
Allow to overrule default build options for Qt build.

  • WebCore.pro: Allow to overrule ENABLE_VIDEO and ENABLE_NETSCAPE_PLUGIN_API
  • JavaScriptCore.pri: Allow to overrule ENABLE_JIT
09:32 Changeset [41527] by treat@webkit.org
  • 2 edits in trunk/WebCore

2009-03-09 Adam Treat <adam.treat@torchmobile.com>

Reviewed by George Staikos.

https://bugs.webkit.org/show_bug.cgi?id=24463
WebCore::qstring is detaching and copying twice for every single
WebCore::TextRun that is processed and drawn. This elevates this method
to one of the top-ten most expensive methods in all of QtWebKit according
to profiling. This changes the method so that QString only detaches
when absolutely necessary.

  • platform/graphics/qt/FontQt.cpp: (WebCore::qstring): (WebCore::fixSpacing): (WebCore::Font::drawComplexText): (WebCore::Font::floatWidthForComplexText): (WebCore::Font::offsetForPositionForComplexText): (WebCore::Font::selectionRectForComplexText):

03/08/09:

23:29 Changeset [41526] by zecke@webkit.org
  • 2 edits in trunk/LayoutTests

2009-03-08 Holger Hans Peter Freyther <zecke@selfish.org>

Rubber-stamped by Mark Rowe.

[Gtk] Skip failing tests that require the eventSender.

The Gtk+ platform is lacking an implementation of the
eventSender. We will enable the tests once there is an
implementation of that class.

  • platform/gtk/Skipped:
22:36 Changeset [41525] by mrowe@apple.com
  • 33 edits
    2 adds in trunk

Split ScrollAlignment and ScrollBehavior out of RenderLayer.h so that
Frame.h no longer needs to include it. This cuts the size of the symbols
for a debug build by around 3%.

Reviewed by Oliver Hunt.

  • dom/Element.cpp:

(WebCore::Element::scrollIntoView):
(WebCore::Element::scrollIntoViewIfNeeded):

  • editing/Editor.cpp:

(WebCore::Editor::insertTextWithoutSendingTextEvent):
(WebCore::Editor::revealSelectionAfterEditingOperation):

  • editing/SelectionController.cpp:

(WebCore::SelectionController::setSelection):

  • loader/FrameLoader.cpp:

(WebCore::FrameLoader::gotoAnchor):

  • page/ContextMenuController.cpp:

(WebCore::ContextMenuController::contextMenuItemSelected):

  • page/Frame.cpp:

(WebCore::Frame::revealSelection):
(WebCore::Frame::revealCaret):

  • page/Frame.h:
  • rendering/RenderLayer.cpp:

(WebCore::RenderLayer::getRectToExpose):
(WebCore::RenderLayer::autoscroll):

  • rendering/RenderLayer.h:
  • rendering/ScrollBehavior.cpp: Added.

(WebCore::):

  • rendering/ScrollBehavior.h: Added.

(WebCore::):
(WebCore::ScrollAlignment::getVisibleBehavior):
(WebCore::ScrollAlignment::getPartialBehavior):
(WebCore::ScrollAlignment::getHiddenBehavior):

Add the new files to the build systems:

  • GNUmakefile.am:
  • WebCore.base.exp:
  • WebCore.pro:
  • WebCore.scons:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • WebCoreSources.bkl:

Add includes that were previously pulled in by way of RenderLayer.h from Frame.h:

  • dom/Clipboard.cpp:
  • editing/EditorCommand.cpp:
  • editing/InsertLineBreakCommand.cpp:
  • editing/TypingCommand.cpp:
  • html/HTMLFormControlElement.cpp:
  • loader/archive/cf/LegacyWebArchive.cpp:
  • loader/icon/IconLoader.cpp:
  • page/animation/AnimationController.cpp:
  • rendering/RenderSlider.cpp:
  • rendering/RenderTextControl.cpp:

WebKit/mac:

  • Plugins/WebNetscapePluginView.mm:
  • WebView/WebFrame.mm:

(-[WebFrame _scrollDOMRangeToVisible:]):
(-[WebFrame _insertParagraphSeparatorInQuotedContent]):
(-[WebFrame _replaceSelectionWithFragment:selectReplacement:smartReplace:matchStyle:]):

  • WebView/WebHTMLView.mm:

(-[WebHTMLView jumpToSelection:]):
(-[WebHTMLView centerSelectionInVisibleArea:]):

WebKit/win:

  • WebView.cpp:

(WebView::centerSelectionInVisibleArea):

20:44 Changeset [41524] by mrowe@apple.com
  • 9 edits in trunk/WebCore

Push the include of Frame.h out of RenderView.h and down to those files
that need it. This cuts the size of symbols for RenderFoo object files
by around 15% in a debug build.

Reviewed Dan Bernstein.

  • bindings/objc/DOM.mm:
  • rendering/RenderFileUploadControl.cpp:
  • rendering/RenderInline.cpp:
  • rendering/RenderObject.cpp:
  • rendering/RenderReplaced.cpp:
  • rendering/RenderText.cpp:
  • rendering/RenderView.cpp:

(WebCore::RenderView::zoomFactor):

  • rendering/RenderView.h:
20:18 Changeset [41523] by mrowe@apple.com
  • 12 edits in trunk/WebCore

Remove some unnecessary or redundant includes from files related to the JS bindings.
This cuts the size of the symbols for a debug build by around 3%.

Rubber-stamped by Dan Bernstein.

  • bindings/js/JSCustomPositionCallback.cpp:
  • bindings/js/JSCustomPositionErrorCallback.cpp:
  • bindings/js/JSCustomSQLStatementCallback.cpp:
  • bindings/js/JSCustomSQLStatementErrorCallback.cpp:
  • bindings/js/JSCustomSQLTransactionCallback.cpp:
  • bindings/js/JSCustomSQLTransactionErrorCallback.cpp:
  • bindings/js/JSCustomVoidCallback.cpp:
  • bindings/js/JSCustomXPathNSResolver.cpp:
  • bindings/js/JSDocumentCustom.cpp:
  • bindings/js/JSPluginElementFunctions.cpp:
  • bindings/js/JSSVGPODTypeWrapper.h:
20:17 Changeset [41522] by zecke@webkit.org
  • 9 edits in trunk

[gtk+] Implement LayoutTestController::setJavaScriptProfilingEnabled

setJavaScriptProfilingEnabled is implemented by enabling the
developer extras using the WebKitWebSettings and adding the
JavaScript Profiling option to the WebKitWebInspector.

16:21 Changeset [41521] by cwzwarich@webkit.org
  • 5 edits in trunk/WebCore

2009-03-08 Cameron Zwarich <cwzwarich@uwaterloo.ca>

Reviewed by Oliver Hunt.

Remove the unused methods previousHistoryItem(), setPreviousHistoryItem()
and provisionalHistoryItem() from FrameLoader.

  • WebCore.base.exp:
  • WebCore.order:
  • loader/FrameLoader.cpp: (WebCore::FrameLoader::frameLoadCompleted): (WebCore::FrameLoader::loadItem):
  • loader/FrameLoader.h:
12:35 Changeset [41520] by kov@webkit.org
  • 3 edits in trunk/WebCore

2009-03-08 Gustavo Noronha Silva <gns@gnome.org>

Reviewed by Holger Freyther.

Implement proper logging for the GTK+ port, inspired by the one in
Qt. We use the WEBKIT_DEBUG environment variable which was already
being used to decide whether to setup a logger for soup or not.

  • platform/gtk/LoggingGtk.cpp: (WebCore::getChannelFromName): (WebCore::InitializeLoggingChannelsIfNecessary):
  • platform/network/soup/ResourceHandleSoup.cpp: (WebCore::ensureSessionIsInitialized):
06:01 Changeset [41519] by oliver@apple.com
  • 2 edits in trunk/JavaScriptCore

Build fix.

Reviewed by NOBODY(Build fix)

03:47 Changeset [41518] by oliver@apple.com
  • 2 edits in trunk/JavaScriptCore

Bug 24268: RuntimeArray is not a fully implemented JSArray
<https://bugs.webkit.org/show_bug.cgi?id=24268>

Reviewed by Cameron Zwarich.

Don't cast a type to JSArray, just because it reportsArray as a supertype
in the JS type system. Doesn't appear feasible to create a testcase
unfortunately as setting up the failure conditions requires internal access
to JSC not present in DRT.

03/07/09:

16:05 Changeset [41517] by mitz@apple.com
  • 13 edits in trunk

WebCore:

Reviewed by Mark Rowe.

  • WebCore part of removing build-time and run-time support for legacy versions of CFNetwork and Core Graphics
  • platform/graphics/win/FontCGWin.cpp: (WebCore::Font::drawGlyphs):
  • platform/graphics/win/FontCustomPlatformData.cpp: (WebCore::FontCustomPlatformData::fontPlatformData):
  • platform/graphics/win/FontPlatformDataCGWin.cpp: (WebCore::FontPlatformData::platformDataInit):
  • platform/network/cf/FormDataStreamCFNet.cpp: (WebCore::setHTTPBody): (WebCore::httpBodyFromRequest):
  • platform/network/cf/ResourceHandleCFNet.cpp: (WebCore::didSendBodyData): (WebCore::shouldUseCredentialStorageCallback): (WebCore::ResourceHandle::start):
  • platform/network/win/CookieJarCFNetWin.cpp: (WebCore::filterCookies):

WebKit/win:

Reviewed by Mark Rowe.

  • WebKit part of removing build-time and run-time support for legacy versions of CFNetwork and Core Graphics
  • WebView.cpp: (WebView::setCacheModel):

WebKitLibraries:

Reviewed by Mark Rowe.

  • WebKitSystemInterface part of removing build-time and run-time support for legacy versions of CFNetwork and Core Graphics
  • win/include/WebKitSystemInterface/WebKitSystemInterface.h:
  • win/lib/WebKitSystemInterface.lib:
  • win/lib/WebKitSystemInterface_debug.lib:
14:16 Changeset [41516] by jmalonzo@webkit.org
  • 2 edits in trunk/LayoutTests

2009-03-07 Jan Michael Alonzo <jmalonzo@webkit.org>

Updated Skipped list per r41464. Not reviewed.
This should remove the warnings that come up everytime we run the gtk layout tests.
http://trac.webkit.org/changeset/41464

The ff. tests have been moved to fast/css/getComputedStyle in the Skipped list:

fast/css/computed-style-without-renderer.html
fast/css/computed-style.html
fast/css/getComputedStyle-border-box.html

The ff. test has been removed from the Skipped list:

fast/css/marginComputedStyle.html

  • platform/gtk/Skipped:
14:15 Changeset [41515] by jmalonzo@webkit.org
  • 2 edits in trunk/LayoutTests

2009-03-07 Jan Michael Alonzo <jmalonzo@webkit.org>

Rubber-stamped by Holger Freyther.

Remove fast/canvas/gradient-empty-path.html from the skipped
list as it was removed by r39288.

  • platform/gtk/Skipped:
14:15 Changeset [41514] by jmalonzo@webkit.org
  • 2 edits in trunk/LayoutTests

2009-03-07 Jan Michael Alonzo <jmalonzo@webkit.org>

Reviewed by Holger Freyther.

[Gtk] Enable http security, plugins and ssl tests
https://bugs.webkit.org/show_bug.cgi?id=24393

Enable http tests for security, plugins and ssl, skipping tests
that are failing/crashing.

  • platform/gtk/Skipped:
14:15 Changeset [41513] by jmalonzo@webkit.org
  • 4 edits in trunk

2009-03-07 Jan Michael Alonzo <jmalonzo@webkit.org>

Reviewed by Holger Freyther.

[Gtk] Enable http history tests
https://bugs.webkit.org/show_bug.cgi?id=24394

Get the index of the current item from the list of (history) items
to print before adding the back history items to the list. This
will make the 'curr' pointer point to the correct item in the
actual results, therefore, passing some of the http/tests/history
tests.

  • DumpRenderTree/gtk/DumpRenderTree.cpp: (dumpBackForwardListForWebView):

2009-03-07 Jan Michael Alonzo <jmalonzo@webkit.org>

Reviewed by Holger Freyther.

[Gtk] Enable http history tests
https://bugs.webkit.org/show_bug.cgi?id=24394

Enable http/tests/history, adding failing tests to the Skipped list.

  • platform/gtk/Skipped:
13:28 Changeset [41512] by mitz@apple.com
  • 2 edits in trunk/WebKit/mac

Reviewed by Alexey Proskuryakov.

  • fix a bug where debug builds were clearing the HTML5 application cache on application termination
  • WebView/WebView.mm: (-[WebView _close]): Call -[WebCache setDisabled:YES] instead of -[WebCache empty].
11:46 Changeset [41511] by adele@apple.com
  • 5 edits
    2 adds in trunk

WebCore:

2009-03-07 Adele Peterson <adele@apple.com>

Reviewed by Dan Bernstein.

Fix for https://bugs.webkit.org/show_bug.cgi?id=24444
No multiple JS property for <input type="file" multiple />

  • html/HTMLInputElement.cpp: (WebCore::HTMLInputElement::multiple): (WebCore::HTMLInputElement::setMultiple):
  • html/HTMLInputElement.h:
  • html/HTMLInputElement.idl:

LayoutTests:

2009-03-07 Adele Peterson <adele@apple.com>

Reviewed by Dan Bernstein.

Test for https://bugs.webkit.org/show_bug.cgi?id=24444
No multiple JS property for <input type="file" multiple />

  • fast/forms/input-multiple-expected.txt: Added.
  • fast/forms/input-multiple.html: Added.
10:42 Changeset [41510] by treat@webkit.org
  • 2 edits in trunk/WebKit/qt

2009-03-07 Adam Treat <adam.treat@torchmobile.com>

Reviewed by Cameron Zwarich.

These methods are clearly returning the wrong values as the two were
returning swapped information.

  • Api/qwebpage.cpp: (QWebPage::totalBytes): (QWebPage::bytesReceived):
04:03 Changeset [41509] by xan@webkit.org
  • 3 edits in trunk/WebCore

2009-03-07 Xan Lopez <xan@gnome.org>

Reviewed by Holger Freyther.

https://bugs.webkit.org/show_bug.cgi?id=24358
[GTK] Scrollbars not clipped correctly

ScrollView scrollbars in subframes are *not* native, so take that
into account again.

  • platform/gtk/ScrollbarGtk.cpp: (ScrollbarGtk::getLocationInParentWindow): (ScrollbarGtk::frameRectsChanged): (ScrollbarGtk::paint):
  • platform/gtk/ScrollbarGtk.h:
Note: See TracTimeline for information about the timeline view.