Timeline


and

03/15/10:

23:42 Changeset [56043] by eric@webkit.org
  • 6 edits in trunk

2010-03-15 John Gregg <johnnyg@google.com>

Reviewed by David Levin.

Notification object should expose absolute URL of icon
https://bugs.webkit.org/show_bug.cgi?id=35800

Covered by existing tests.

  • notifications/Notification.cpp: (WebCore::Notification::Notification):
  • notifications/Notification.h: (WebCore::Notification::iconURL):

2010-03-15 John Gregg <johnnyg@google.com>

Reviewed by David Levin.

Notification object should expose absolute URL of icon
https://bugs.webkit.org/show_bug.cgi?id=35800

  • public/WebNotification.h:
  • src/WebNotification.cpp: (WebKit::WebNotification::iconURL):
23:25 Changeset [56042] by eric@webkit.org
  • 1 edit
    1 add in trunk/WebCore

2010-03-15 Leandro Pereira <leandro@profusion.mobi>

Reviewed by Holger Freyther.

Add EFL port files to platform/efl.
http://webkit.org/b/35887

  • platform/efl/ClipboardEfl.cpp: Added.
  • platform/efl/PasteboardEfl.cpp: Added.
22:56 Changeset [56041] by tkent@chromium.org
  • 4 edits
    6 adds in trunk

2010-03-15 Kent Tamura <tkent@chromium.org>

Reviewed by Eric Seidel.

Don't submit disabled menu options.
https://bugs.webkit.org/show_bug.cgi?id=35056

  • fast/forms/menulist-disabled-selected-option-expected.txt: Added.
  • fast/forms/menulist-disabled-selected-option.html: Added.
  • fast/forms/menulist-submit-without-selection-expected.txt: Added.
  • fast/forms/menulist-submit-without-selection.html: Added.
  • fast/forms/script-tests/menulist-disabled-selected-option.js: Added.
  • fast/forms/script-tests/menulist-submit-without-selection.js: Added.

2010-03-15 Kent Tamura <tkent@chromium.org>

Reviewed by Eric Seidel.

Don't submit disabled menu options.
https://bugs.webkit.org/show_bug.cgi?id=35056

Test: fast/forms/menulist-disabled-selected-option.html

  • dom/OptionElement.h:
  • dom/SelectElement.cpp: (WebCore::SelectElement::appendFormData): If a selected option is disabled, skip it. Remove code for non-selected menulist because of compatibility with other browsers.
22:55 Changeset [56040] by abarth@webkit.org
  • 7 edits in trunk/WebKitTools

2010-03-15 Adam Barth <abarth@webkit.org>

Reviewed by Eric Seidel.

Let commit-queue land rollout patches even when the tree is red
https://bugs.webkit.org/show_bug.cgi?id=36155

Now the commit-queue will land patches whose name begins with "ROLLOUT "
even if the tree is red. The patches still go through the usual build
and test process, but they can be landed while the tree is on fire.

  • Scripts/webkitpy/bugzilla.py:
  • Scripts/webkitpy/commands/queues.py:
  • Scripts/webkitpy/commands/queues_unittest.py:
  • Scripts/webkitpy/commands/queuestest.py:
  • Scripts/webkitpy/mock_bugzillatool.py:
22:50 Changeset [56039] by eric@webkit.org
  • 1 edit
    1 add in trunk/WebCore

2010-03-15 Leandro Pereira <leandro@profusion.mobi>

Reviewed by Holger Freyther.

Add EFL port files to platform/efl.
http://webkit.org/b/35884

  • platform/efl/FileSystemEfl.cpp: Added.
22:33 Changeset [56038] by eric@webkit.org
  • 1 edit
    1 add in trunk/WebCore

2010-03-15 Leandro Pereira <leandro@profusion.mobi>

Reviewed by Holger Freyther.

Add EFL port files to platform/efl.
http://webkit.org/b/35882

  • platform/efl/ContextMenuItemEfl.cpp: Added.
22:32 Changeset [56037] by mjs@apple.com
  • 2 edits in trunk/WebKitSite

Tweak canvas perf demo.

  • demos/canvas-perf/canvas.html:
22:15 Changeset [56036] by eric@webkit.org
  • 4 edits in trunk

2010-03-15 Joanmarie Diggs <joanmarie.diggs@gmail.com>

Reviewed by Holger Freyther.

https://bugs.webkit.org/show_bug.cgi?id=35502
[Gtk] Objects of ATK_ROLE_TABLE should not implement AtkText

Simple change to stop tables from implementing AtkText.

  • accessibility/gtk/AccessibilityObjectAtk.cpp: (getInterfaceMaskFromObject):

2010-03-15 Joanmarie Diggs <joanmarie.diggs@gmail.com>

Reviewed by Holger Freyther.

https://bugs.webkit.org/show_bug.cgi?id=35502
[Gtk] Objects of ATK_ROLE_TABLE should not implement AtkText

New test to be sure we do not accidentally implement AtkText for tables

  • tests/testatk.c (testWebkitAtkGetTextInTable): (main):
21:58 Changeset [56035] by eric@webkit.org
  • 6 edits in trunk

2010-03-15 Adam Bergkvist <adam.bergkvist@ericsson.com>

Reviewed by Adam Barth.

According to the updated specification, a data field should always
result in a newline character being appended to the data buffer
regardless if the data buffer contains any data or not. However, upon
event dispatch, the last newline will be removed. This differs from an
older version of the specification where a newline character was
appended before the data value only if the buffer already contained
data. As a result, EventSource now supports receiving events with empty
data or newline characters only. Updated test accordingly.
https://bugs.webkit.org/show_bug.cgi?id=33210

  • http/tests/eventsource/eventsource-parse-event-stream-expected.txt:
  • http/tests/eventsource/eventsource-parse-event-stream.html:
  • http/tests/eventsource/resources/event-stream.php:

2010-03-15 Adam Bergkvist <adam.bergkvist@ericsson.com>

Reviewed by Adam Barth.

According to the updated specification, a data field should always
result in a newline character being appended to the data buffer
regardless if the data buffer contains any data or not. However, upon
event dispatch, the last newline will be removed. This differs from an
older version of the specification where a newline character was
appended before the data value only if the buffer already contained
data. As a result, EventSource now supports receiving events with empty
data or newline characters only.
https://bugs.webkit.org/show_bug.cgi?id=33210

  • page/EventSource.cpp: (WebCore::EventSource::parseEventStreamLine):
21:43 Changeset [56034] by abarth@webkit.org
  • 4 edits
    1 add in trunk/WebKitTools

2010-03-15 Adam Barth <abarth@webkit.org>

Reviewed by Eric Seidel.

Add webkit-patch post-rollout to upload rollouts to bugs.webkit.org for easy committing
https://bugs.webkit.org/show_bug.cgi?id=36154

This new command is a mashup of prepare-rollout and post. This command
will be used by an experimental bot to post rollouts of patches that
break things to bugs.webkit.org where they can be landed with the
greatest of ease.

  • Scripts/webkitpy/commands/download.py:
  • Scripts/webkitpy/commands/download_unittest.py:
  • Scripts/webkitpy/steps/init.py:
  • Scripts/webkitpy/steps/postdiffforrevert.py: Added.
21:41 Changeset [56033] by eric@webkit.org
  • 3 edits
    2 adds in trunk

2010-03-15 Valters Svabe <vsvabe@gmail.com>

Reviewed by Darin Adler.

:after selector displays in wrong place with nested div
https://bugs.webkit.org/show_bug.cgi?id=32288

  • fast/css-generated-content/block-after-expected.txt: Added.
  • fast/css-generated-content/block-after.html: Added.

2010-03-15 Valters Svabe <vsvabe@gmail.com>

Reviewed by Darin Adler.

:after selector displays in wrong place with nested div
https://bugs.webkit.org/show_bug.cgi?id=32288

Make sure we don't append things after :after-generated content, if
it's inside an anonymous block.

Test: fast/css-generated-content/block-after.html

  • rendering/RenderBlock.cpp: (WebCore::RenderBlock::addChild):
21:15 Changeset [56032] by abarth@webkit.org
  • 2 edits in trunk/WebKitTools

2010-03-15 Adam Barth <abarth@webkit.org>

Reviewed by Eric Seidel.

webkit-patch rollout should error out on conflicts
https://bugs.webkit.org/show_bug.cgi?id=36151

Instead of blindingly plowing ahead, we now throw an exception if there
are conflicts after applying a reverse diff.

  • Scripts/webkitpy/scm.py:
21:07 Changeset [56031] by eric@webkit.org
  • 1 edit
    1 add in trunk/JavaScriptCore

2010-03-15 Kwang Yul Seo <skyul@company100.net>

Reviewed by Eric Seidel.

[BREWMP] Add a function to create a BREW instance without local variable declarations.
https://bugs.webkit.org/show_bug.cgi?id=34705

Add a template function to create a BREW instance in one line.

  • wtf/brew/ShellBrew.h: Added. (WTF::createInstance):
20:27 Changeset [56030] by darin@chromium.org
  • 2 edits in trunk/WebKit/chromium

2010-03-15 Darin Fisher <darin@chromium.org>

Fix build bustage. We also need to pull down "third_party/tcmalloc"

  • DEPS:
19:36 Changeset [56029] by ggaren@apple.com
  • 2 edits in trunk/JavaScriptCore

Not reviewed.

Removed a now-incorrect comment I forgot to remove in my last check-in.

  • wtf/FastMalloc.cpp:

(WTF::TCMalloc_PageHeap::scavenge):

19:21 Changeset [56028] by ggaren@apple.com
  • 2 edits in trunk/JavaScriptCore

Fixed a portion of:
<rdar://problem/7165917> | https://bugs.webkit.org/show_bug.cgi?id=28676
Safari 4 does not release memory back to the operating system fast enough (28676)

Reviewed by Sam Weinig.

Every few seconds, release a percentage of the minimum unused page count
during that time period.

SunSpider reports no change, command-line or in-browser, Mac or Windows.

  • wtf/FastMalloc.cpp:

(WTF::TCMalloc_PageHeap::init):
(WTF::TCMalloc_PageHeap::signalScavenger):
(WTF::TCMalloc_PageHeap::initializeScavenger): Renamed shouldContinueScavenging
to shouldScavenge, since scavenging is no longer something that we interrupt.

(WTF::TCMalloc_PageHeap::scavenge): The new scavenging algorithm. Fixes
a bug where the old code would release only one item from each size class
per scavenge, potentially leaving large numbers of large-sized objects
unreleased for a long time.

(WTF::TCMalloc_PageHeap::shouldScavenge):
(WTF::TCMalloc_PageHeap::New):
(WTF::TCMalloc_PageHeap::AllocLarge):
(WTF::TCMalloc_PageHeap::Delete):
(WTF::TCMalloc_PageHeap::GrowHeap):
(WTF::TCMalloc_PageHeap::scavengerThread):
(WTF::TCMalloc_PageHeap::periodicScavenge): Updated to track the minimum
value of free_committed_pages_ during a given scavenge period.

17:06 Changeset [56027] by darin@chromium.org
  • 2 edits in trunk/WebKit/chromium

2010-03-15 Darin Fisher <darin@chromium.org>

Fix build bustage. We also need to pull down "gfx"

  • DEPS:
16:54 Changeset [56026] by cfleizach@apple.com
  • 4 edits in trunk

WebKitTools: Unreviewed layout test fix.

VO not able to perform a VO-spacebar on facebook links
https://bugs.webkit.org/show_bug.cgi?id=36132

GTK needs to implement press for this test to work.

  • DumpRenderTree/gtk/AccessibilityUIElementGtk.cpp:

(AccessibilityUIElement::press):

LayoutTests: Unreviewed.

VO not able to perform a VO-spacebar on facebook links
https://bugs.webkit.org/show_bug.cgi?id=36132

GTK needs to implement press in DRT before this test can work.

  • platform/gtk/Skipped:
16:28 Changeset [56025] by cfleizach@apple.com
  • 4 edits in trunk

Unreviewed layout test fix.

WebKitTools:

VO not able to perform a VO-spacebar on facebook links
https://bugs.webkit.org/show_bug.cgi?id=36132

Windows needs to implement press in DRT.

  • DumpRenderTree/win/AccessibilityUIElementWin.cpp:

(AccessibilityUIElement::press):

LayoutTests:

VO not able to perform a VO-spacebar on facebook links
https://bugs.webkit.org/show_bug.cgi?id=36132

This test must be skipped until https://bugs.webkit.org/show_bug.cgi?id=36145 is done.

  • platform/win/Skipped:
16:28 Changeset [56024] by darin@chromium.org
  • 5 edits in trunk/WebKit/chromium

2010-03-15 Darin Fisher <darin@chromium.org>

Reviewed by David Levin.

https://bugs.webkit.org/show_bug.cgi?id=36131
Update include paths for chromium Rect, Point, and Size types.

Update DEPS to pull latest chromium code that reflects the
new location of these headers.

  • DEPS:
  • public/WebPoint.h:
  • public/WebRect.h:
  • public/WebSize.h:
16:20 Changeset [56023] by eric@webkit.org
  • 1 edit
    1 add in trunk/WebCore

2010-03-15 Leandro Pereira <leandro@profusion.mobi>

Reviewed by Eric Seidel.

Add EFL port files to platform/efl.
http://webkit.org/b/35885

  • platform/efl/KURLEfl.cpp: Added.
16:06 Changeset [56022] by cfleizach@apple.com
  • 3 edits in trunk/WebKitTools

Unreviewed. Fix break of layout tests on win and gtk.

VO not able to perform a VO-spacebar on facebook links
https://bugs.webkit.org/show_bug.cgi?id=36132

Attempting to implement press action for windows and gtk.

  • DumpRenderTree/gtk/AccessibilityUIElementGtk.cpp:

(AccessibilityUIElement::press):

  • DumpRenderTree/win/AccessibilityUIElementWin.cpp:

(AccessibilityUIElement::press):

15:59 Changeset [56021] by barraclough@apple.com
  • 3 edits in trunk/JavaScriptCore

https://bugs.webkit.org/show_bug.cgi?id=35843
Re-land reverted fix to JSString::getIndex()

Reviewed by Sam Weinig.

Calling getIndex() on a JSString in rope form may result in a JSException being thrown
if there is insuficient memory so value(exec) returns UString() with length zero,
which will be passed to jsSingleCharacterSubstring.
Add a slow case function to trap the error & return a safe null value, until the
exception is handled.

  • runtime/JSString.cpp:

(JSC::JSString::getIndexSlowCase):
(JSC::JSString::getStringPropertyDescriptor):

  • runtime/JSString.h:

(JSC::jsSingleCharacterSubstring):
(JSC::JSString::getIndex):
(JSC::jsSingleCharacterString):
(JSC::JSString::getStringPropertySlot):

15:38 Changeset [56020] by cfleizach@apple.com
  • 6 edits in trunk/WebKitTools

VO not able to perform a VO-spacebar on facebook links
https://bugs.webkit.org/show_bug.cgi?id=36132

Reviewed by Beth Dakin.

  • DumpRenderTree/AccessibilityUIElement.cpp:

(pressCallback):
(AccessibilityUIElement::getJSClass):

  • DumpRenderTree/AccessibilityUIElement.h:
  • DumpRenderTree/gtk/AccessibilityUIElementGtk.cpp:

(AccessibilityUIElement::press):

  • DumpRenderTree/mac/AccessibilityUIElementMac.mm:

(AccessibilityUIElement::press):

  • DumpRenderTree/win/AccessibilityUIElementWin.cpp:

(AccessibilityUIElement::press):

15:07 Changeset [56019] by cfleizach@apple.com
  • 3 edits
    2 adds in trunk

VO not able to perform a VO-spacebar on facebook links
https://bugs.webkit.org/show_bug.cgi?id=36132

Reviewed by Beth Dakin.

WebCore:

When a label element is used as a click event handler, and it doesn't have any
corresponding control, it should handle the action itself.

Test: accessibility/label-element-press.html

  • html/HTMLLabelElement.cpp:

(WebCore::HTMLLabelElement::accessKeyAction):

LayoutTests:

  • accessibility/label-element-press-expected.txt: Added.
  • accessibility/label-element-press.html: Added.
14:56 Changeset [56018] by cwzwarich@webkit.org
  • 2 edits in trunk/WebCore

2010-03-15 Cameron Zwarich <zwarich@apple.com>

Reviewed by Geoff Garen.

Bug 36135 - GCController::garbageCollectNow() crashes when called from garbage collection
<https://bugs.webkit.org/show_bug.cgi?id=36135>
<rdar://problem/7752552>

  • bindings/js/GCController.cpp:

(WebCore::GCController::garbageCollectNow): Fail silently if garbage collection is
active rather than hitting the CRASH() reentrancy guard.

14:47 Changeset [56017] by simon.fraser@apple.com
  • 2 edits in trunk/WebCore

2010-03-15 Simon Fraser <simon.fraser@apple.com>

Reviewed by Dan Bernstein.

Drop out of compositing mode when no elements need to composite
https://bugs.webkit.org/show_bug.cgi?id=36134

When no more elements on the page need to be composited, drop out of compositing
mode altogether, including removing the layer-backed view that hosts the layers.

  • rendering/RenderLayerCompositor.cpp: (WebCore::RenderLayerCompositor::updateCompositingLayers): If the list of child layers to attach at the root is empty, we have no composited layers, and can drop out of compositing mode by calling willMoveOffscreen().

(WebCore::RenderLayerCompositor::computeCompositingRequirements): In several places,
use willBeComposited rather than calling needsToBeComposited(), because it avoids
recomputing state that we know already (but assert that we're getting it right).
When we're assessing the root layer in post-order, see if we have any child layers that
are compositing, and also whether the root itself needs to composite. If neither are
true, then we can drop out of compositing mode.

(WebCore::RenderLayerCompositor::needsToBeComposited):
(WebCore::RenderLayerCompositor::requiresCompositingLayer): Move the
"inCompositingMode() && layer->isRootLayer()" test out of requiresCompositingLayer()
and into needsToBeComposited() so that we can call requiresCompositingLayer() separately
on the root layer above.

14:33 Changeset [56016] by eric@webkit.org
  • 1 edit
    1 add in trunk/WebCore

2010-03-15 Leandro Pereira <leandro@profusion.mobi>

Reviewed by Kenneth Rohde Christiansen.

Add EFL port files to platform/efl.
http://webkit.org/b/35888

  • platform/efl/SoundEfl.cpp: Added.
14:21 Changeset [56015] by kenneth@webkit.org
  • 2 edits in trunk/JavaScriptCore

Add a long long version of abs() for MSVC.

Reviewed by Adam Roben.

  • wtf/MathExtras.h:

(abs):

14:02 Changeset [56014] by mrowe@apple.com
  • 2 edits in trunk/WebKitTools

Add a new build slave to replace the existing SnowLeopard Leaks build slave which
appears to be suffering a slow and painful death at the hands of its graphics hardware.

  • BuildSlaveSupport/build.webkit.org-config/config.json:
14:01 Changeset [56013] by simon.fraser@apple.com
  • 19 edits
    2 adds in trunk/LayoutTests

2010-03-15 Simon Fraser <simon.fraser@apple.com>

Rubber-stamped by Dan Bernstein.

Update some pixel results for composting tests; some were incorrect, and others had minor
pixel differences.

  • platform/mac/compositing/color-matching/image-color-matching-expected.checksum:
  • platform/mac/compositing/color-matching/image-color-matching-expected.png:
  • platform/mac/compositing/direct-image-compositing-expected.checksum:
  • platform/mac/compositing/direct-image-compositing-expected.png:
  • platform/mac/compositing/overflow/overflow-scroll-expected.checksum:
  • platform/mac/compositing/overflow/overflow-scroll-expected.png:
  • platform/mac/compositing/reflections/animation-inside-reflection-expected.checksum:
  • platform/mac/compositing/reflections/animation-inside-reflection-expected.png:
  • platform/mac/compositing/reflections/load-video-in-reflection-expected.checksum:
  • platform/mac/compositing/reflections/load-video-in-reflection-expected.png:
  • platform/mac/compositing/reflections/nested-reflection-animated-expected.checksum:
  • platform/mac/compositing/reflections/nested-reflection-animated-expected.png:
  • platform/mac/compositing/reflections/nested-reflection-transition-expected.checksum:
  • platform/mac/compositing/reflections/nested-reflection-transition-expected.png:
  • platform/mac/compositing/reflections/reflection-on-composited-expected.checksum:
  • platform/mac/compositing/reflections/reflection-on-composited-expected.png:
  • platform/mac/compositing/repaint/become-overlay-composited-layer-expected.checksum: Added.
  • platform/mac/compositing/repaint/become-overlay-composited-layer-expected.png: Added.
  • platform/mac/compositing/repaint/layer-repaint-rects-expected.checksum:
  • platform/mac/compositing/repaint/layer-repaint-rects-expected.png:
13:31 Changeset [56012] by enrica@apple.com
  • 18 edits
    4 adds in trunk

WebCore: For continuous scrolling devices on the Mac (trackpads and Mighty/Magic
Mice), set the number of wheel ticks equal to the atomic increment of
the device: a pixel. This ensures that any scrolling action will
generate at least one wheel tick, regardless of the speed/length of
the gesture.

Reviewed by John Sullivan.

https://bugs.webkit.org/show_bug.cgi?id=29601
<rdar://problem/7453254>

Tests: fast/events/continuous-platform-wheelevent-in-scrolling-div.html

fast/events/platform-wheelevent-in-scrolling-div.html

  • platform/mac/WebCoreSystemInterface.h: Modify method signature.
  • platform/mac/WebCoreSystemInterface.mm: Ditto.
  • platform/mac/WheelEventMac.mm:

(WebCore::PlatformWheelEvent::PlatformWheelEvent): Call into
WebKitSystemInterface to determine both wheelTicksX/Y and deltaX/Y.
WKSI will query the event system for number of wheel ticks and
pixels/lines scrolled based on the device type.

WebKit/mac: Updated call to WKGetWheelEventDeltas() to match new method signature.

Reviewed by John Sullivan.

https://bugs.webkit.org/show_bug.cgi?id=29601
<rdar://problem/7453254>

  • WebView/WebDynamicScrollBarsView.mm:

(-[WebDynamicScrollBarsView scrollWheel:]):

WebKitLibraries: Added two new output arguments to WKGetWheelEventDeltas() to return
the number of scroll wheel ticks in the x and y directions.

Reviewed by John Sullivan.

https://bugs.webkit.org/show_bug.cgi?id=29601.
<rdar://problem/7453254>

  • WebKitSystemInterface.h:
  • libWebKitSystemInterfaceLeopard.a:
  • libWebKitSystemInterfaceSnowLeopard.a:
  • libWebKitSystemInterfaceTiger.a:

LayoutTests: These tests verify that scrollwheel events (both continuous and
non-continuous) generate the expected pixel deltas in scrollable
regions, and also generate DOM mousewheel events with expected
wheelDelta values.

Reviewed by John Sullivan.

https://bugs.webkit.org/show_bug.cgi?id=29601
<rdar://problem/7453254>

  • fast/events/continuous-platform-wheelevent-in-scrolling-div-expected.txt: Added.
  • fast/events/continuous-platform-wheelevent-in-scrolling-div.html: Added.
  • fast/events/platform-wheelevent-in-scrolling-div-expected.txt: Added.
  • fast/events/platform-wheelevent-in-scrolling-div.html: Added.
  • fast/events/wheelevent-in-scrolling-div-expected.txt: Rebased

expected results due to change in test output formatting.

  • fast/events/wheelevent-in-scrolling-div.html: Modified to use common

js logging and assertion functions.

  • platform/gtk/Skipped: DRT support needed by tests not yet implemented

in GTK port. See https://bugs.webkit.org/show_bug.cgi?id=36003.

  • platform/mac-tiger/Skipped: DRT support needed by tests not possible

to implement in Tiger.

  • platform/qt/Skipped: DRT support for tests not yet implemented in Qt

port. See https://bugs.webkit.org/show_bug.cgi?id=36004.

  • platform/win/Skipped: DRT support for tests not yet implemented in

Windows port. See https://bugs.webkit.org/show_bug.cgi?id=36002.

13:30 Changeset [56011] by eric@webkit.org
  • 3 edits in trunk/WebKit/chromium

2010-03-15 Marcus Bulach <bulach@chromium.org>

Reviewed by Jeremy Orlow.

Send the frame URL on GeolocationServiceBridgeImpl::startUpdating.
https://bugs.webkit.org/show_bug.cgi?id=36012

  • public/GeolocationServiceBridgeChromium.h:
  • src/GeolocationServiceBridgeChromium.cpp: (WebKit::GeolocationServiceBridgeImpl::startUpdating):
13:18 Changeset [56010] by philn@webkit.org
  • 3 edits in trunk/WebCore

2010-03-15 Philippe Normand <pnormand@igalia.com>

Reviewed by Gustavo Noronha Silva.

[GStreamer] duration query optimizations
https://bugs.webkit.org/show_bug.cgi?id=36116

Don't reattempt duration queries that previously failed and cache
media duration only if it's known.

  • platform/graphics/gstreamer/MediaPlayerPrivateGStreamer.cpp: (WebCore::MediaPlayerPrivate::MediaPlayerPrivate): (WebCore::MediaPlayerPrivate::duration): (WebCore::MediaPlayerPrivate::updateStates): (WebCore::MediaPlayerPrivate::durationChanged):
  • platform/graphics/gstreamer/MediaPlayerPrivateGStreamer.h:
13:11 Changeset [56009] by philn@webkit.org
  • 3 edits in trunk/WebCore

2010-03-15 Philippe Normand <pnormand@igalia.com>

Reviewed by Gustavo Noronha Silva.

[GStreamer] updateStates called for all elements posting a state-change
https://bugs.webkit.org/show_bug.cgi?id=36115

  • platform/graphics/gstreamer/MediaPlayerPrivateGStreamer.cpp: (WebCore::mediaPlayerPrivateMessageCallback): Update player state only for state-change messages coming from playbin2 directly.
  • platform/graphics/gstreamer/MediaPlayerPrivateGStreamer.h: (WebCore::MediaPlayerPrivate::pipeline): public accessor for the playbin element.
13:10 Changeset [56008] by sullivan@apple.com
  • 5 edits in trunk/WebKit/mac

-[WebFrame setAlwaysHideHorizontal/VerticalScroller:] prevents keyboard scrolling
<https://bugs.webkit.org/show_bug.cgi?id=36125>

Reviewed by Adam Roben.

  • WebView/WebDynamicScrollBarsView.h:

Added instance variables horizontalScrollingAllowedButScrollerHidden and
verticalScrollingAllowedButScrollerHidden. Renamed instance variables
hideHorizontal/VerticalScroller to alwaysHideHorizontal/VerticalScroller for clarity.
Declared methods -horizontalScrollingAllowed and -verticalScrollingAllowed.
Added comments.

  • WebView/WebDynamicScrollBarsView.mm:

(-[WebDynamicScrollBarsView setAlwaysHideHorizontalScroller:]):
Updated for instance variable renaming.
(-[WebDynamicScrollBarsView setAlwaysHideVerticalScroller:]):
Ditto.
(-[WebDynamicScrollBarsView horizontalScrollingAllowed]):
New method, returns YES if the scroller is showing or the only reason that the scroller
is not showing is that setAlwaysHideHorizontalScrolling has been called.
(-[WebDynamicScrollBarsView verticalScrollingAllowed]):
New method, returns YES if the scroller is showing or the only reason that the scroller
is not showing is that setAlwaysHideVerticalScrolling has been called.
(-[WebDynamicScrollBarsView updateScrollers]):
Updated for instance variable renamings. Now updates horizontalScrollingAllowedButScrollerHidden
and verticalScrollingAllowedButScrollerHidden. Now takes the always-hidden state into account
in the early-return code path, to avoid taking it into account twice in the regular code path.

  • WebView/WebFrameView.mm:

(-[WebFrameView _scrollToBeginningOfDocument]):
Use _isScrollable instead of _hasScrollBars.
(-[WebFrameView _scrollToEndOfDocument]):
Ditto.
(-[WebFrameView scrollToBeginningOfDocument:]):
Use _largestScrollableChild instead of _largestChildWithScrollBars.
(-[WebFrameView scrollToEndOfDocument:]):
Ditto.
(-[WebFrameView _pageVertically:]):
Use _isScrollable and _largestScrollableChild instead of _hasScrollBars
and _largestChildWithScrollBars.
(-[WebFrameView _pageHorizontally:]):
Ditto.
(-[WebFrameView _scrollLineVertically:]):
Ditto.
(-[WebFrameView _scrollLineHorizontally:]):
Ditto.
(-[WebFrameView keyDown:]):
Use _largestScrollableChild instead of _largestChildWithScrollBars.
(-[WebFrameView _isScrollable]):
New method, calls -[WebDynamicScrollBarsView horizontalScrollingAllowed] and
-[WebDynamicScrollBarsView verticalScrollingAllowed]
(-[WebFrameView _largestScrollableChild]):
New method, like _largestChildWithScrollBars but uses _isScrollable.
(-[WebFrameView _hasScrollBars]):
Added a comment that this is no longer used by Safari, and can thus probably be
deleted once we no longer want to support it for nightly build compatibility with
old versions of Safari.
(-[WebFrameView _largestChildWithScrollBars]):
Ditto.

  • WebView/WebFrameViewPrivate.h:

Declared -_isScrollable and -_largestScrollableChild. Added comments to
_hasScrollBars and _largestChildWithScrollBars saying that they are no longer
used by Safari, and can thus probably be deleted once we no longer want to
support them for nightly build compatibility with old versions of Safari.

13:08 Changeset [56007] by pkasting@chromium.org
  • 10 edits in trunk/WebCore

Clean up more bits of the open-source image decoders. In addition to
simplifying things somewhat, this makes all the decoders the same in
terms of how they construct "image reader" subclasses and init scale
data.

Reviewed by Adam Barth.

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

No functional change, so no tests.

  • platform/image-decoders/ImageDecoder.h: Reorder a few declarations to try and group/order members slightly better.

(WebCore::ImageDecoder::ImageDecoder):

  • platform/image-decoders/gif/GIFImageDecoder.cpp: Move reader construction into decode(). Prep scale data in setSize(). Remove useless comment.

(WebCore::GIFImageDecoder::setData):
(WebCore::GIFImageDecoder::setSize):
(WebCore::GIFImageDecoder::frameBufferAtIndex):
(WebCore::GIFImageDecoder::decode):

  • platform/image-decoders/gif/GIFImageDecoder.h: Prep scale data in setSize().
  • platform/image-decoders/gif/GIFImageReader.cpp: Prep scale data in setSize().

(GIFImageReader::read):

  • platform/image-decoders/ico/ICOImageDecoder.cpp: Shorten code.

(WebCore::ICOImageDecoder::setSize):

  • platform/image-decoders/jpeg/JPEGImageDecoder.cpp: Fix style. Move reader construction into decode().

(WebCore::JPEGImageReader::close):
(WebCore::JPEGImageReader::skipBytes):
(WebCore::JPEGImageDecoder::decode):

  • platform/image-decoders/jpeg/JPEGImageDecoder.h: Move reader construction into decode().

(WebCore::JPEGImageDecoder::filenameExtension):

  • platform/image-decoders/png/PNGImageDecoder.cpp: Move reader construction into decode(). Track completion on decoder, not reader. Prep scale data in setSize(). Remove useless comment.

(WebCore::PNGImageReader::PNGImageReader):
(WebCore::PNGImageReader::close):
(WebCore::PNGImageReader::decode):
(WebCore::PNGImageDecoder::setSize):
(WebCore::PNGImageDecoder::frameBufferAtIndex):
(WebCore::PNGImageDecoder::headerAvailable):
(WebCore::PNGImageDecoder::pngComplete):
(WebCore::PNGImageDecoder::decode):

  • platform/image-decoders/png/PNGImageDecoder.h: Move reader construction into decode(). Track completion on decoder, not reader. Prep scale data in setSize().

(WebCore::PNGImageDecoder::filenameExtension):
(WebCore::PNGImageDecoder::isComplete):

13:00 Changeset [56006] by philn@webkit.org
  • 2 edits in trunk/WebCore

2010-03-15 Philippe Normand <pnormand@igalia.com>

Reviewed by Gustavo Noronha Silva.

[GTK] video playback position query flood when mouse over the video element
https://bugs.webkit.org/show_bug.cgi?id=35333

Don't trigger a position query only to know which play/pause
button to display.

  • platform/gtk/RenderThemeGtk.cpp: (WebCore::RenderThemeGtk::paintMediaPlayButton):
12:57 Changeset [56005] by dpranke@chromium.org
  • 2 edits in trunk/WebKitTools

2010-03-15 Dirk Pranke <dpranke@chromium.org>

Reviewed by Alexey Proskuryakov.

Fix a minor case where we'd deference a null pointer if we tried
to run new-run-webkit-tests on an unsupported platform (e.g.
Cygwin's python version).

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

  • Scripts/webkitpy/layout_tests/port/factory.py:
10:27 Changeset [56004] by jianli@chromium.org
  • 5 edits
    1 add in trunk/WebCore

[v8] Add V8 bindings for DOMFormData.
https://bugs.webkit.org/show_bug.cgi?id=36026

Reviewed by Nate Chapin.

Test: http/tests/local/send-form-data.html

  • Android.v8bindings.mk:
  • WebCore.gypi:
  • bindings/v8/V8Index.h:
  • bindings/v8/custom/V8DOMFormDataCustom.cpp: Added.
  • bindings/v8/custom/V8XMLHttpRequestCustom.cpp:

(WebCore::V8XMLHttpRequest::sendCallback):

10:27 Changeset [56003] by jianli@chromium.org
  • 29 edits
    6 adds in trunk

WebCore: Add DOMFormData.idl to expose FormData interface.
https://bugs.webkit.org/show_bug.cgi?id=36024

Reviewed by Sam Weinig.

The implementation is based on XMLHttpRequest 2 spec:
http://dev.w3.org/2006/webapi/XMLHttpRequest-2/#formdata

Test: http/tests/local/send-form-data.html

  • Android.derived.jscbindings.mk:
  • Android.derived.v8bindings.mk:
  • Android.jscbindings.mk:
  • DerivedSources.cpp:
  • DerivedSources.make:
  • GNUmakefile.am:
  • WebCore.pri:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • bindings/js/JSDOMFormDataCustom.cpp: Added.
  • bindings/js/JSXMLHttpRequestCustom.cpp:

(WebCore::JSXMLHttpRequest::send):

  • bindings/scripts/CodeGeneratorJS.pm:
  • html/DOMFormData.idl: Added.
  • page/DOMWindow.idl:

LayoutTests: Add DOMFormData.idl to expose FormData interface
https://bugs.webkit.org/show_bug.cgi?id=36024

Reviewed by Sam Weinig.

Add a layout test to test sending FormData via XMLHttpRequest. Also
changed all test results that are affected by exposing FormData.

  • fast/dom/Window/window-properties-expected.txt:
  • fast/dom/Window/window-property-descriptors-expected.txt:
  • fast/dom/constructed-objects-prototypes-expected.txt:
  • fast/dom/prototype-inheritance-2-expected.txt:
  • fast/dom/prototype-inheritance-expected.txt:
  • fast/js/global-constructors-expected.txt:
  • http/tests/local/resources/send-form-data.js: Added.
  • http/tests/local/send-form-data-expected.txt: Added.
  • http/tests/local/send-form-data.html: Added.
  • http/tests/xmlhttprequest/resources/multipart-post-echo.php: Added.
  • platform/gtk/Skipped:
  • platform/gtk/fast/dom/Window/window-properties-expected.txt:
  • platform/qt/Skipped:
  • platform/qt/fast/dom/Window/window-properties-expected.txt:
  • platform/qt/fast/dom/Window/window-property-descriptors-expected.txt:
  • platform/qt/fast/dom/prototype-inheritance-expected.txt:
  • platform/qt/fast/js/global-constructors-expected.txt:
  • platform/win/Skipped:
10:14 Changeset [56002] by eric@webkit.org
  • 14 edits in trunk

2010-03-15 Patrik Persson <patrik.j.persson@ericsson.com>

Reviewed by Darin Adler.

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

Revised test of storage/databases in sandboxed iframes.

Verify that SECURITY_ERR is raised on access to
window.localStorage or window.openDatabase(). Also verify that
window.sessionStorage is now allowed in sandboxed iframes.

WebKit would previously return null references in these cases. The
new behavior is in accordance with HTML5:

  • fast/frames/resources/sandboxed-iframe-storage-disallowed.html: expect exceptions, not null
  • fast/frames/sandboxed-iframe-storage-expected.txt: expect exceptions, not null

2010-03-15 Patrik Persson <patrik.j.persson@ericsson.com>

Reviewed by Darin Adler.

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

Revise iframe sandbox behavior to match the updated HTML5 spec.

  • Enables window.sessionStorage in sandboxed iframes.
  • Raises SECURITY_ERR exceptions when window.localStorage or window.openDatabase() is blocked by iframe sandboxing.

Note: window.sessionStorage does not raise exceptions.

WebKit would previously return null references in these cases. The
new behavior is in accordance with HTML5:

http://dev.w3.org/html5/webstorage/ (sections 4.2 and 4.3)
http://dev.w3.org/html5/webdatabase/ (section 4.1)
http://www.mail-archive.com/whatwg@lists.whatwg.org/msg19786.html

  • inspector/InspectorController.cpp: (WebCore::InspectorController::selectDOMStorage): exception handling
  • inspector/InspectorDOMStorageResource.cpp: (WebCore::InspectorDOMStorageResource::handleEvent): exception handling
  • page/DOMWindow.cpp: (WebCore::DOMWindow::sessionStorage): re-enabled in iframe sandbox (WebCore::DOMWindow::localStorage): raise exception rather than return null (WebCore::DOMWindow::openDatabase): raise exception rather than return null
  • page/DOMWindow.h: added exceptions to interface
  • page/DOMWindow.idl: added exceptions to interface
  • page/SecurityOrigin.h: (WebCore::SecurityOrigin::canAccessLocalStorage): renamed function to reflect its purpose
  • storage/StorageEventDispatcher.cpp: (WebCore::StorageEventDispatcher::dispatch): exception handling

2010-03-15 Patrik Persson <patrik.j.persson@ericsson.com>

Reviewed by Darin Adler.

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

Revise iframe sandbox behavior to match the updated HTML5 spec.

  • Enables window.sessionStorage in sandboxed iframes.
  • Raises SECURITY_ERR exceptions when window.localStorage or window.openDatabase() is blocked by iframe sandboxing.

Note: window.sessionStorage does not raise exceptions.

WebKit would previously return null references in these cases. The
new behavior is in accordance with HTML5:

http://dev.w3.org/html5/webstorage/ (sections 4.2 and 4.3)
http://dev.w3.org/html5/webdatabase/ (section 4.1)
http://www.mail-archive.com/whatwg@lists.whatwg.org/msg19786.html

  • src/StorageAreaProxy.cpp: (WebCore::StorageAreaProxy::storageEvent): exception handling
  • src/StorageEventDispatcherImpl.cpp: (WebCore::StorageEventDispatcherImpl::dispatchStorageEvent): exception handling
09:41 Changeset [56001] by darin@apple.com
  • 3 edits in trunk/WebKitTools

Tell Subversion about more directories that expect to have .pyc files.

  • Scripts/webkitpy/layout_tests: Added property svn:ignore.
  • Scripts/webkitpy/layout_tests/port: Added property svn:ignore.
09:33 Changeset [56000] by eric@webkit.org
  • 2 edits in trunk/JavaScriptCore

2010-03-15 Gabor Loki <loki@webkit.org>

Reviewed by Gavin Barraclough.

Combine ctiTrampolines on ARM and Thumb-2
https://bugs.webkit.org/show_bug.cgi?id=36014

  • jit/JITStubs.cpp: (JSC::JITThunks::JITThunks):
09:22 Changeset [55999] by apavlov@chromium.org
  • 9 edits
    1 add in trunk/WebCore

2010-03-15 Alexander Pavlov <apavlov@chromium.org>

Reviewed by Timothy Hatcher.

Web Inspector: Implement a progress indicator in the Audits panel
when resources are reloading
https://bugs.webkit.org/show_bug.cgi?id=35971

  • English.lproj/localizedStrings.js:
  • GNUmakefile.am:
  • WebCore.gypi:
  • inspector/front-end/AuditLauncherView.js: (WebInspector.AuditLauncherView.prototype.updateResourceTrackingState): (WebInspector.AuditLauncherView.prototype.get totalResources): (WebInspector.AuditLauncherView.prototype.set totalResources): (WebInspector.AuditLauncherView.prototype.get loadedResources): (WebInspector.AuditLauncherView.prototype.set loadedResources): (WebInspector.AuditLauncherView.prototype._resetResourceCount): (WebInspector.AuditLauncherView.prototype.resourceStarted): (WebInspector.AuditLauncherView.prototype.resourceFinished): (WebInspector.AuditLauncherView.prototype.reset): (WebInspector.AuditLauncherView.prototype._setAuditRunning): (WebInspector.AuditLauncherView.prototype._launchButtonClicked): (WebInspector.AuditLauncherView.prototype._createCategoryElement): (WebInspector.AuditLauncherView.prototype._createLauncherUI): (WebInspector.AuditLauncherView.prototype._updateResourceProgress): (WebInspector.AuditLauncherView.prototype._updateButton):
  • inspector/front-end/AuditsPanel.js: (WebInspector.AuditsPanel.prototype.resourceStarted): (WebInspector.AuditsPanel.prototype.resourceFinished): (WebInspector.AuditsPanel.prototype.reset):
  • inspector/front-end/Images/spinner.gif: Added.
  • inspector/front-end/WebKit.qrc:
  • inspector/front-end/audits.css: (.audit-launcher-view .resource-progress > img):
  • inspector/front-end/inspector.js: (WebInspector.updateResource):
08:55 QtWebKitBackportingFixes edited by laszlo.1.gombos@nokia.com
(diff)
08:54 Changeset [55998] by eric@webkit.org
  • 4 edits in trunk

2010-03-15 Tony Chang <tony@chromium.org>

Reviewed by Darin Adler.

option-pageup/down should move cursor on chromium mac
https://bugs.webkit.org/show_bug.cgi?id=36108

Remove waitUntilDone/notifyDone from the layout test, it's not needed
and causes timeouts on failure.

  • editing/input/option-page-up-down.html:

2010-03-15 Tony Chang <tony@chromium.org>

Reviewed by Darin Adler.

option-pageup/down should move cursor on chromium mac
https://bugs.webkit.org/show_bug.cgi?id=36108

  • src/EditorClientImpl.cpp: (WebKit::):
08:40 Changeset [55997] by sullivan@apple.com
  • 5 edits in trunk/WebKit/mac

Method name and parameter name mistakes from recent SPI addition
<https://bugs.webkit.org/show_bug.cgi?id=36119>

Reviewed by Dan Bernstein.

  • WebView/WebDynamicScrollBarsView.h:

Renamed instance variable and method name from "setAllowXXX" to "setAllowsXXX".

  • WebView/WebDynamicScrollBarsView.mm:

(-[WebDynamicScrollBarsView setAllowsScrollersToOverlapContent:]):
Updated for renamed instance variable and method.
(-[WebDynamicScrollBarsView setAlwaysHideHorizontalScroller:]):
Changed parameter name from shouldBeVisible to shouldBeHidden.
(-[WebDynamicScrollBarsView setAlwaysHideVerticalScroller:]):
Ditto.
(-[WebDynamicScrollBarsView contentViewFrame]):
Updated for renamed instance variable.
(-[WebDynamicScrollBarsView tile]):
Ditto.
(-[WebDynamicScrollBarsView reflectScrolledClipView:]):
Ditto.

  • WebView/WebFrame.mm:

(-[WebFrame setAllowsScrollersToOverlapContent:]):
Renamed method from setAllowXXX, and updated for same change in WebDynamicScrollBarsView.

  • WebView/WebFramePrivate.h:

Renamed method name from "setAllowXXX" to "setAllowsXXX".

08:34 Changeset [55996] by philn@webkit.org
  • 3 edits in trunk/WebCore

2010-03-15 Philippe Normand <pnormand@igalia.com>

Reviewed by Gustavo Noronha Silva.

[GStreamer] replace g_idle_add / g_timeout_add calls with Timers in the gstreamer player
https://bugs.webkit.org/show_bug.cgi?id=35735

Replaced g_idle_add calls with Timers immediately started once
only. The g_timeout_add was replaced by a periodic Timer.

  • platform/graphics/gstreamer/MediaPlayerPrivateGStreamer.cpp: (WebCore::MediaPlayerPrivate::MediaPlayerPrivate): (WebCore::MediaPlayerPrivate::~MediaPlayerPrivate): (WebCore::MediaPlayerPrivate::volumeChangedTimerFired): (WebCore::MediaPlayerPrivate::volumeChanged): (WebCore::MediaPlayerPrivate::processBufferingStats): (WebCore::MediaPlayerPrivate::fillTimerFired): (WebCore::MediaPlayerPrivate::maxTimeLoaded): (WebCore::MediaPlayerPrivate::updateStates): (WebCore::MediaPlayerPrivate::muteChangedTimerFired): (WebCore::MediaPlayerPrivate::muteChanged):
  • platform/graphics/gstreamer/MediaPlayerPrivateGStreamer.h:
08:20 Changeset [55995] by kov@webkit.org
  • 5 edits in trunk

2010-03-12 Gustavo Noronha Silva <gustavo.noronha@collabora.co.uk>

Reviewed by Eric Carlson.

media/video-preload.html fails
https://bugs.webkit.org/show_bug.cgi?id=35793

Only effectively load, and start buffering when playing, or when
the preload attribute is set.

Test: media/video-preload.html

  • platform/graphics/gstreamer/MediaPlayerPrivateGStreamer.cpp: (WebCore::MediaPlayerPrivate::MediaPlayerPrivate): (WebCore::MediaPlayerPrivate::load): (WebCore::MediaPlayerPrivate::commitLoad): (WebCore::MediaPlayerPrivate::prepareToPlay): (WebCore::MediaPlayerPrivate::setPreload):
  • platform/graphics/gstreamer/MediaPlayerPrivateGStreamer.h:
07:46 Changeset [55994] by laszlo.1.gombos@nokia.com
  • 2 edits in trunk/WebKit/qt

2010-03-15 Laszlo Gombos <laszlo.1.gombos@nokia.com>

Unreviewed.

[Qt] Build fix if TILED_BACKING_STORE is disabled.

  • Api/qwebframe.cpp: (QWebFramePrivate::renderContentsLayerAbsoluteCoords):
07:34 Changeset [55993] by eric@webkit.org
  • 5 edits
    2 adds in trunk

2010-03-15 Shu Chang <Chang.Shu@nokia.com>

Reviewed by Holger Freyther.

[Qt] Change and add expected results after the fix and also remove the test from Skipped.
https://bugs.webkit.org/show_bug.cgi?id=36060

  • platform/qt/Skipped:
  • platform/qt/fast/canvas/fillrect-gradient-zero-stops-expected.checksum: Added.
  • platform/qt/fast/canvas/fillrect-gradient-zero-stops-expected.png: Added.
  • platform/qt/fast/canvas/fillrect-gradient-zero-stops-expected.txt:

2010-03-15 Shu Chang <Chang.Shu@nokia.com>

Reviewed by Holger Freyther.

[Qt] The behavior of QGradient with no stops is defined differently from HTML5 spec,
where the latter requires the gradient to be transparent black. Explicitly added a
transparent black color stop to match the HTML5 behavior.
https://bugs.webkit.org/show_bug.cgi?id=36060

  • platform/graphics/qt/GradientQt.cpp: (WebCore::Gradient::platformGradient):
07:13 Changeset [55992] by tonikitoo@webkit.org
  • 2 edits in trunk/WebCore

Some minor code clean ups in SpatialNavigation.cpp

Rubber-stamped by Holger Freyther.
Patch by Antonio Gomes <tonikitoo@webkit.org>

  • page/SpatialNavigation.cpp:

(WebCore::distanceInDirection):
(WebCore::deflateIfOverlapped):

06:02 Changeset [55991] by pfeldman@chromium.org
  • 2 edits in trunk/WebCore

2010-03-13 Ilya Tikhonovsky <loislo@chromium.org>

Reviewed by Timothy Hatcher.

WebInspector: Position of GraphBar elements is calculating more carefully
and they fully visible at the bottom of Timeline panel.


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

  • inspector/front-end/TimelinePanel.js: (WebInspector.TimelineRecordGraphRow.prototype.update):
02:10 Changeset [55990] by eric@webkit.org
  • 6 edits in trunk

2010-03-15 MORITA Hajime <morrita@google.com>

Reviewed by Alexey Proskuryakov.

Selection.modify extends too far with 'lineboundary'.
https://bugs.webkit.org/show_bug.cgi?id=33413

  • editing/selection/extend-selection-expected.txt:
  • platform/win/editing/selection/extend-selection-expected.txt: Updated result to correct expectation that described the wrong behaviour for the selection expansion with 'lineboundary' granularity.

2010-03-15 MORITA Hajime <morrita@google.com>

Reviewed by Alexey Proskuryakov.

Selection.modify extends too far with 'lineboundary'.
https://bugs.webkit.org/show_bug.cgi?id=33413

Selection.modify() with 'lineboundary' granularity implies just
"Go to the end of the line", unlike selection expansion with
other type of granularities. This change handled LineGranularity
as special case, to look-up end of line with UPSTREAM affinity.
Doing this prevents look-up algorithm to go next line.

Test: editing/selection/extend-selection-expected.txt

  • dom/Position.cpp: (WebCore::Position::getInlineBoxAndOffset): Handled an edge case for node look-up with UPSTREAM.
  • editing/SelectionController.cpp: (WebCore::SelectionController::modifyExtendingForward): Added UPSTREAM tweak for the case for LineGranularity.
01:34 Changeset [55989] by philn@webkit.org
  • 2 edits in trunk/WebKit/gtk

2010-03-09 Philippe Normand <pnormand@igalia.com>

Reviewed by Holger Freyther.

[GTK] GTK_WIDGET_IS_SENSITIVE is deprecated in GTK+ 2.20
https://bugs.webkit.org/show_bug.cgi?id=35909

  • webkit/webkitwebview.cpp: GTK_WIDGET_IS_SENSITIVE has been deprecated in gtk 2.20. Use gtk_widget_is_sensitive when available.
01:33 Changeset [55988] by yuzo@google.com
  • 6 edits
    5 adds in trunk

2010-03-14 Yuzo Fujishima <yuzo@google.com>

Reviewed by Shinichiro Hamaji.

Always call PrintContext.end() from PrintContext::pageNumberForElement().

Without this change, WebKit remains in printing mode after determining the page number if the specified element is found.

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

  • platform/gtk/Skipped:
  • platform/mac/printing/return-from-printing-mode-expected.checksum: Added.
  • platform/mac/printing/return-from-printing-mode-expected.png: Added.
  • platform/mac/printing/return-from-printing-mode-expected.txt: Added.
  • platform/qt/Skipped:
  • platform/win/Skipped:
  • printing/return-from-printing-mode.html: Added.

2010-03-14 Yuzo Fujishima <yuzo@google.com>

Reviewed by Shinichiro Hamaji.

Always call PrintContext.end() from PrintContext::pageNumberForElement().

Without this change, WebKit remains in printing mode after determining the page number if the specified element is found.

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

Test: printing/return-from-printing-mode.html

  • page/PrintContext.cpp: (WebCore::PrintContext::pageNumberForElement):
01:19 Changeset [55987] by philn@webkit.org
  • 2 edits in trunk/WebCore

2010-03-11 Philippe Normand <pnormand@igalia.com>

Reviewed by Gustavo Noronha Silva.

[GStreamer] progressTimer never stopped after EOS
https://bugs.webkit.org/show_bug.cgi?id=36007

Mark the pipeline as paused only if playback hasn't ended
yet. When didEnd() is triggered, the pipeline is paused (but
paused() still returns false) and a timeupdate event is fired,
those 2 conditions allow the HTMLMediaElement to stop the
progressTimer (in updatePlayerState, via mediaPlayerTimeChanged).

  • platform/graphics/gstreamer/MediaPlayerPrivateGStreamer.cpp: (WebCore::MediaPlayerPrivate::updateStates):
00:28 Changeset [55986] by eric@webkit.org
  • 1 edit
    5 adds in trunk/WebKitTools

2010-03-15 Adam Barth <abarth@webkit.org>

Reviewed by Darin Adler.

Operational scripts from running the EWS
https://bugs.webkit.org/show_bug.cgi?id=36097

These are the scripts I use to manage the EWS on EC2. If someone other
than me wants to run the EWS, these scripts might be helpful.

  • EWSTools/boot.sh: Added.
  • EWSTools/create-webkit-git: Added.
  • EWSTools/screen-config: Added.
  • EWSTools/start-queue.sh: Added.
00:04 Changeset [55985] by darin@apple.com
  • 2 edits in trunk/WebKit/mac

2010-03-14 Darin Adler <darin@apple.com>

Reviewed by Mark Rowe.

Remove unneeded dependency on non-string identifier for an NSTableColumn
https://bugs.webkit.org/show_bug.cgi?id=36106

  • WebView/WebTextCompletionController.mm: (-[WebTextCompletionController _buildUI]): Use init instead of initWithIdentifier: because the table has only one column and that column does not need an identifier.

03/14/10:

23:19 Changeset [55984] by mjs@apple.com
  • 1 edit
    3 adds in trunk/WebKitSite

Added canvas perf demo for whatwg.

  • demos/canvas-perf: Added.
  • demos/canvas-perf/canvas.html: Added.
  • demos/canvas-perf/kraken.jpg: Added.
23:16 Changeset [55983] by abarth@webkit.org
  • 3 edits in trunk/WebKitTools

2010-03-14 Adam Barth <abarth@webkit.org>

Unreviewed.

Fix the webkit-patch bots. Turns out they need the path of the main
script to run properly.

  • Scripts/webkit-patch:
  • Scripts/webkitpy/patch/patcher.py:
22:58 Changeset [55982] by darin@apple.com
  • 5 edits in trunk/WebKitTools

Tell Subversion about more directories that expect to have .pyc files.

  • Scripts/webkitpy/init: Added property svn:ignore.
  • Scripts/webkitpy/patch: Added property svn:ignore.
  • Scripts/webkitpy/thirdparty: Added property svn:ignore.
  • Scripts/webkitpy/thirdparty/autoinstalled: Added property svn:ignore.
18:28 Changeset [55981] by eric@webkit.org
  • 5 edits in trunk

2010-03-14 Chang Shu <chang.shu@nokia.com>

Reviewed by Simon Hausmann.

[Qt] [Symbian] Added block for ENABLE_SYMBIAN_DIALOG_PROVIDERS
on Symbian platform.
https://bugs.webkit.org/show_bug.cgi?id=35919

  • WebCore.pro:

2010-03-14 Chang Shu <chang.shu@nokia.com>

Reviewed by Simon Hausmann.

[Qt] [Symbian] Use Symbian native dialog providers for combo pupups.
https://bugs.webkit.org/show_bug.cgi?id=35919

  • WebCoreSupport/QtFallbackWebPopup.cpp: (WebCore::QtFallbackWebPopup::show): (WebCore::ResetAndDestroy): (WebCore::QtFallbackWebPopup::showS60BrowserDialog):
  • WebCoreSupport/QtFallbackWebPopup.h:
18:00 Changeset [55980] by yael.aharon@nokia.com
  • 28 edits
    16 adds in trunk

Support for HTMLProgressElement
https://bugs.webkit.org/show_bug.cgi?id=35937

Reviewed by Darin Adler.

WebCore:

Added support for HTMLProgressElement.
This implementation is enabled only for Qt, because only RenderThemeQt
was modified to actually draw the progress element.
The labels attribute of the progress element will be implemented in a
separate patch.

Tests: fast/dom/HTMLProgressElement/progress-element.html

fast/dom/HTMLProgressElement/set-progress-properties.html

  • DerivedSources.cpp:
  • DerivedSources.make:
  • GNUmakefile.am:
  • WebCore.gypi:
  • WebCore.pri:
  • WebCore.pro:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • css/CSSPrimitiveValueMappings.h:

(WebCore::CSSPrimitiveValue::CSSPrimitiveValue):

  • css/CSSSelector.cpp:

(WebCore::CSSSelector::extractPseudoType):

  • css/CSSValueKeywords.in:
  • css/html.css:
  • html/HTMLElement.cpp:

(WebCore::inlineTagList):

  • html/HTMLElementsAllInOne.cpp:
  • html/HTMLFormControlElement.cpp:
  • html/HTMLFormControlElement.h:
  • html/HTMLProgressElement.cpp: Added.
  • html/HTMLProgressElement.h: Added.
  • html/HTMLProgressElement.idl: Added.
  • html/HTMLTagNames.in:
  • page/DOMWindow.idl:
  • platform/ThemeTypes.h:
  • platform/qt/RenderThemeQt.cpp:
  • platform/qt/RenderThemeQt.h:
  • rendering/RenderObject.h:
  • rendering/RenderProgress.cpp: Added.
  • rendering/RenderProgress.h: Added.
  • rendering/RenderTheme.cpp:
  • rendering/RenderTheme.h:

LayoutTests:

  • fast/dom/HTMLProgressElement: Added.
  • fast/dom/HTMLProgressElement/progress-element.html: Added.
  • fast/dom/HTMLProgressElement/script-tests: Added.
  • fast/dom/HTMLProgressElement/script-tests/TEMPLATE.html: Added.
  • fast/dom/HTMLProgressElement/script-tests/set-progress-properties.js: Added.
  • fast/dom/HTMLProgressElement/set-progress-properties.html: Added.
  • platform/gtk/Skipped:
  • platform/mac/Skipped:
  • platform/qt/fast/dom/HTMLProgressElement: Added.
  • platform/qt/fast/dom/HTMLProgressElement/progress-element-expected.checksum: Added.
  • platform/qt/fast/dom/HTMLProgressElement/progress-element-expected.png: Added.
  • platform/qt/fast/dom/HTMLProgressElement/progress-element-expected.txt: Added.
  • platform/qt/fast/dom/HTMLProgressElement/set-progress-properties-expected.txt: Added.
17:53 Changeset [55979] by antti@apple.com
  • 5 edits in trunk/WebKitTools

https://bugs.webkit.org/show_bug.cgi?id=35146
Support tiled backing store

Reviewed by Simon Hausmann.

QtLauncher support and build flag in build-webkit.

  • QtLauncher/main.cpp:

(LauncherWindow::applyPrefs):
(LauncherWindow::toggleTiledBackingStore):
(LauncherWindow::toggleResizesToContents):
(LauncherWindow::createChrome):
(LauncherApplication::handleUserOptions):

  • QtLauncher/webview.cpp:

(WebViewGraphicsBased::WebViewGraphicsBased):
(WebViewGraphicsBased::setResizesToContents):
(WebViewGraphicsBased::resizeEvent):

  • QtLauncher/webview.h:
  • Scripts/build-webkit:
17:51 Changeset [55978] by antti@apple.com
  • 6 edits in trunk/WebKit/qt

https://bugs.webkit.org/show_bug.cgi?id=35146
Support tiled backing store

Reviewed by Simon Hausmann.

Implements a basic tiled backing store mechanism. Tiles are created and
deleted on demand. The page content is cached to the tiles. Tile content
is kept in sync with the document. Since the backing store covers area
larger than the currently visible viewport, the document can be scrolled
quickly without having to enter rendering tree painting.

The tile management code is platform independent. This patch has simple QPixmap
based tile implementation for Qt.

The feature is behind ENABLE_TILED_BACKING_STORE flag.

  • Api/qgraphicswebview.cpp:

(QGraphicsWebViewPrivate::visibleRect):
(QGraphicsWebView::paint):

  • Api/qwebframe.cpp:

(QWebFramePrivate::renderContentsLayerAbsoluteCoords):

  • Api/qwebframe.h:
  • Api/qwebsettings.cpp:

(QWebSettingsPrivate::apply):

  • Api/qwebsettings.h:
17:01 Changeset [55977] by oliver@apple.com
  • 7 edits in trunk

2010-03-14 Oliver Hunt <oliver@apple.com>

Reviewed by Darin Adler.

REGRESSION(r53287): drop event is not fired if dataTransfer.dropEffect is not explicitly set
https://bugs.webkit.org/show_bug.cgi?id=36095

Correct this test to cover the somewhat bizarre "correct" handling
of undefined dropEffect

  • fast/events/drag-and-drop-expected.txt:
  • fast/events/drag-and-drop.html:

2010-03-14 Oliver Hunt <oliver@apple.com>

Reviewed by Darin Adler.

REGRESSION(r53287): drop event is not fired if dataTransfer.dropEffect is not explicitly set
https://bugs.webkit.org/show_bug.cgi?id=36095

The issue here is that while dropEffect is meant to be initialized
to "none", the behaviour of the drag is differs between dragEffect
not being set and dragEffect being explicitly set to "none"

This patch corrects this behaviour by making Clipboard distinguish
between the initial "none" value of dropEffect and an explicit "none".
This alone is insufficient for correct behaviour, we also need to
resurrect the removed defaultOperationForDrag function, but we now
use the function only when dragEffect is uninitialized. There are a
few tweaks to the behaviour of the defaultOperationForDrag as well
to ensure exactly the same set of outcomes for all cases that we
may hit it.

  • dom/Clipboard.cpp: (WebCore::Clipboard::Clipboard): (WebCore::dragOpFromIEOp): (WebCore::Clipboard::destinationOperation):
  • dom/Clipboard.h: (WebCore::Clipboard::dropEffect): (WebCore::Clipboard::dropEffectIsUninitialized):
  • page/DragController.cpp: (WebCore::defaultOperationForDrag): (WebCore::DragController::tryDHTMLDrag):
16:40 Changeset [55976] by antti@apple.com
  • 8 edits
    5 adds in trunk/WebCore

https://bugs.webkit.org/show_bug.cgi?id=35146
Support tiled backing store

Reviewed by Simon Hausmann.

Implements a basic tiled backing store mechanism. Tiles are created and
deleted on demand. The page content is cached to the tiles. Tile content
is kept in sync with the document. Since the backing store covers area
larger than the currently visible viewport, the document can be scrolled
quickly without having to enter rendering tree painting.

The tile management code is platform independent. This patch has simple QPixmap
based tile implementation for Qt.

The feature is behind ENABLE_TILED_BACKING_STORE flag.

  • WebCore.pri:
  • WebCore.pro:
  • page/Frame.cpp:

(WebCore::Frame::Frame):
(WebCore::Frame::setTiledBackingStoreEnabled):
(WebCore::Frame::tiledBackingStorePaintBegin):
(WebCore::Frame::tiledBackingStorePaint):
(WebCore::Frame::tiledBackingStorePaintEnd):
(WebCore::Frame::tiledBackingStoreContentsRect):

  • page/Frame.h:

(WebCore::Frame::tiledBackingStore):

  • page/FrameView.cpp:

(WebCore::FrameView::repaintContentRectangle):
(WebCore::FrameView::doDeferredRepaints):

  • page/Settings.cpp:

(WebCore::Settings::Settings):
(WebCore::Settings::setTiledBackingStoreEnabled):

  • page/Settings.h:

(WebCore::Settings::tiledBackingStoreEnabled):

  • platform/graphics/Tile.h: Added.

(WebCore::Tile::create):
(WebCore::Tile::coordinate):
(WebCore::Tile::rect):

  • platform/graphics/TiledBackingStore.cpp: Added.

(WebCore::TiledBackingStore::TiledBackingStore):
(WebCore::TiledBackingStore::~TiledBackingStore):
(WebCore::TiledBackingStore::invalidate):
(WebCore::TiledBackingStore::updateTileBuffers):
(WebCore::TiledBackingStore::paint):
(WebCore::TiledBackingStore::viewportChanged):
(WebCore::TiledBackingStore::setContentsScale):
(WebCore::TiledBackingStore::tileDistance):
(WebCore::TiledBackingStore::createTiles):
(WebCore::TiledBackingStore::dropOverhangingTiles):
(WebCore::TiledBackingStore::dropTilesOutsideRect):
(WebCore::TiledBackingStore::tileAt):
(WebCore::TiledBackingStore::setTile):
(WebCore::TiledBackingStore::removeTile):
(WebCore::TiledBackingStore::mapToContents):
(WebCore::TiledBackingStore::mapFromContents):
(WebCore::TiledBackingStore::contentsRect):
(WebCore::TiledBackingStore::tileRectForCoordinate):
(WebCore::TiledBackingStore::tileCoordinateForPoint):
(WebCore::TiledBackingStore::startTileBufferUpdateTimer):
(WebCore::TiledBackingStore::tileBufferUpdateTimerFired):
(WebCore::TiledBackingStore::startTileCreationTimer):
(WebCore::TiledBackingStore::tileCreationTimerFired):
(WebCore::TiledBackingStore::setContentsFrozen):

  • platform/graphics/TiledBackingStore.h: Added.

(WebCore::TiledBackingStore::contentsScale):
(WebCore::TiledBackingStore::contentsFrozen):

  • platform/graphics/TiledBackingStoreClient.h: Added.
  • platform/graphics/qt/TileQt.cpp: Added.

(WebCore::checkeredPixmap):
(WebCore::Tile::Tile):
(WebCore::Tile::~Tile):
(WebCore::Tile::isDirty):
(WebCore::Tile::isReadyToPaint):
(WebCore::Tile::invalidate):
(WebCore::Tile::updateBackBuffer):
(WebCore::Tile::swapBackBufferToFront):
(WebCore::Tile::paint):
(WebCore::Tile::paintCheckerPattern):

14:44 Changeset [55975] by mitz@apple.com
  • 16 edits
    2 deletes in trunk

WebCore: WebCore part of removing support for legacy versions of Core Graphics

Reviewed by Darin Adler.

  • WebCore.vcproj/WebCore.vcproj: Removed FontDatabase.{cpp,h}
  • platform/graphics/win/FontCGWin.cpp:

(WebCore::Font::drawGlyphs): Removed call to wkCanCreateCGFontWithLOGFONT(),
as it is now always true.

  • platform/graphics/win/FontCacheWin.cpp:

(WebCore::FontCache::createFontPlatformData): Ditto.

  • platform/graphics/win/FontCustomPlatformData.cpp:

(WebCore::FontCustomPlatformData::~FontCustomPlatformData): Updated for
the removal of m_cgFont.
(WebCore::FontCustomPlatformData::fontPlatformData): Removed call to
wkCanCreateCGFontWithLOGFONT(), as it is now always true.
(WebCore::createFontCustomPlatformData): Ditto. Also updated for change to
the FontCustomPlatformData constructor.

  • platform/graphics/win/FontCustomPlatformData.h: Removed m_cgFont member.

(WebCore::FontCustomPlatformData::FontCustomPlatformData): Removed cgFont
parameter.

  • platform/graphics/win/FontDatabase.cpp: Removed.
  • platform/graphics/win/FontDatabase.h: Removed.
  • platform/graphics/win/FontPlatformDataCGWin.cpp:

(WebCore::FontPlatformData::platformDataInit): Removed call to
wkCanCreateCGFontWithLOGFONT(), as it is now always true.

  • platform/win/TemporaryLinkStubs.cpp:

(WebCore::populateFontDatabase): Removed stub.

WebKit/win: WebKit part of removing support for legacy versions of Core Graphics

Reviewed by Darin Adler.

  • WebKitClassFactory.cpp:

(WebKitClassFactory::WebKitClassFactory): Removed call to populateFontDatabase().

  • WebKitGraphics.cpp:

(makeFont): Ditto.

  • WebTextRenderer.cpp:

(WebTextRenderer::registerPrivateFont): Removed call to wkAddFontsAtPath().

WebKitLibraries: WebKitSystemInterface part of removing support for legacy versions of Core Graphics

Reviewed by Darin Adler.

  • win/include/WebKitSystemInterface/WebKitSystemInterface.h: Removed

wkCanCreateCGFontWithLOGFONT(), wkSetFontPlatformInfo(), wkAddFontsInDirectory(),
wkAddFontsAtPath(), wkAddFontsFromRegistry(), wkAddFontsFromPlist(), and
wkCreateFontsPlist().

  • win/lib/WebKitSystemInterface.lib: Updated.
  • win/lib/WebKitSystemInterface_debug.lib: Updated.
08:43 Changeset [55974] by jberlin@webkit.org
  • 2 edits in trunk/WebCore

"event.ctrlKey" is always false when dragging an element with "ctrl" key down
https://bugs.webkit.org/show_bug.cgi?id=17113

Reviewed by Sam Weinig.

No new tests.

  • page/DragController.cpp:

(WebCore::createMouseEvent):
Use the current state of the shift, ctrl, alt, and meta keys when creating the drag mouse event.

05:58 Changeset [55973] by tonikitoo@webkit.org
  • 3 edits in trunk/WebCore

2010-03-13 Antonio Gomes <tonikitoo@webkit.org>

Not reviewed identation fix.

  • manual-tests/spatial-navigation/spatial-navigation-test-cases.html:
01:51 QtWebKitJournal edited by hausmann@webkit.org
(diff)

03/13/10:

16:23 Changeset [55972] by cjerdonek@webkit.org
  • 2 edits
    1 copy
    2 adds in trunk/WebKitTools

The webkit-patch script now displays a warning if run using
a version of Python less than 2.5. This will help users
understand why webkit-patch is erroring out if they are
using Python 2.4, for example.

Reviewed by Adam Barth.

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

  • Scripts/webkit-patch:
    • Moved most of the file contents to webkitpy/patch/patcher.py so the Python version can be checked before interpreting any code that can cause the script to error out.
    • Added a configure_logging() method to enable any version warnings to show up.
    • Added a main() method with calls to configure_logging(), check_version(), and the main webkit patch method.
  • Scripts/webkitpy/patch/init.py: Copied from WebKitTools/QueueStatusServer/filters/init.py.
    • This file is required to make a folder a package.
  • Scripts/webkitpy/patch/patcher.py: Added.
    • Moved code from Scripts/webkit-patch.
15:48 Changeset [55971] by cjerdonek@webkit.org
  • 4 edits in trunk/WebKitTools

Changed test-webkitpy so that messages logged as a side-effect
of unit-testing code do not get displayed to the screen. These
messages clutter up the unit test results if not filtered out.

Reviewed by Adam Barth.

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

  • Scripts/test-webkitpy:
    • Adjusted the configure_logging() method to filter out any log messages from webkitpy.
    • Also added an INFO message stating that most console logging is getting suppressed.
  • Scripts/webkitpy/init/versioning.py:
    • Added a log parameter to the check_version() method.
  • Scripts/webkitpy/init/versioning_unittest.py:
    • Qualified a call to check_version() with the parameter names.
14:36 Changeset [55970] by cjerdonek@webkit.org
  • 3 edits
    1 copy
    3 adds in trunk/WebKitTools

The test-webkitpy script now warns the user if the script is
being run using a Python version different from the minimum
version the webkitpy package was meant to support.

Reviewed by Adam Barth.

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

Warning developers if their Python version is too low will help
them understand why test-webkitpy is failing. Secondly, warning
developers if their Python version is higher than the minimum will
help them understand that their changes may not be okay for the
minimum supported version, even if test-webkitpy is passing.

  • Scripts/test-webkitpy:
    • Moved the "from ..._unittest import *" lines to the new file Scripts/webkitpy/unittests.py. This will allow the version-check warning to be displayed even if an error occurs while interpreting (i.e. importing) the unit test code.
    • Added configure_logging() to configur logging for test-webkitpy.
    • Added an init() method to configure logging and check the current Python version.
  • Scripts/webkitpy/init/unittests.py: Added.
    • Added a file to import all unit test modules in the webkitpy.init package.
  • Scripts/webkitpy/init/versioning.py: Added.
    • Added a _MINIMUM_SUPPORTED_PYTHON_VERSION variable and set it equal to 2.5.
    • Added a compare_version() method to compare the current Python version against a target version.
    • Added a check_version() method to check the current Python version against the current minimum supported version, and to log a warning message if the check fails.
  • Scripts/webkitpy/init/versioning_unittest.py: Added.
    • Added unit tests for the functions in versioning.py.
  • Scripts/webkitpy/style/unittests.py:
    • Fixed/updated a code comment.
  • Scripts/webkitpy/unittests.py: Added.
    • Moved the "from ..._unittest import *" lines from test-webkitpy.
14:10 Changeset [55969] by cjerdonek@webkit.org
  • 6 edits
    1 copy
    1 add in trunk/WebKitTools

Moved all code in webkitpy/init.py to another location.

Reviewed by Adam Barth.

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

Keeping webkitpy/init.py free of non-trivial code allows
calling code to import initialization code from webkitpy
before any errors or log messages occur due to code in
init.py. Such initialization code can include things like
version checking code and logging configuration code. This
also lets us move the autoinstall initialization code to a
location where it only executes if it is needed -- something
we have done in this patch.

  • Scripts/webkitpy/init.py:
    • Moved all executable code to the following location: webkitpy/thirdparty/autoinstalled/init.py
    • Added a code comment to keep this file free of non-trivial code.
  • Scripts/webkitpy/bugzilla.py:
    • Updated mechanize import statement.
  • Scripts/webkitpy/networktransaction.py:
    • Updated mechanize import statement.
  • Scripts/webkitpy/networktransaction_unittest.py:
    • Updated mechanize import statement.
  • Scripts/webkitpy/statusserver.py:
    • Updated mechanize import statement.
  • Scripts/webkitpy/thirdparty/autoinstalled/init.py: Added.
    • Copied the code from webkitpy/init.py and updated it as necessary.
13:28 Changeset [55968] by cjerdonek@webkit.org
  • 16 edits
    1 move
    2 adds in trunk/WebKitTools

Moved webkitpy/mock.py into webkitpy/thirdparty since it is
third-party code.

Reviewed by Adam Barth.

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

Updated the import statement in all of the below except where noted.

  • Scripts/webkitpy/bugzilla_unittest.py:
  • Scripts/webkitpy/commands/commandtest.py:
  • Scripts/webkitpy/commands/download_unittest.py:
  • Scripts/webkitpy/commands/early_warning_system_unittest.py:
  • Scripts/webkitpy/commands/queries_unittest.py:
  • Scripts/webkitpy/commands/queues_unittest.py:
  • Scripts/webkitpy/commands/queuestest.py:
  • Scripts/webkitpy/commands/upload.py:
  • Scripts/webkitpy/commands/upload_unittest.py:
  • Scripts/webkitpy/commands_references.py: Added.
    • Added a file containing an absolute import of Mock so that the imports in the commands folder can import from this file (similar to style_references.py). This helps limit the number of affected files in future refactorings.
  • Scripts/webkitpy/credentials_unittest.py:
  • Scripts/webkitpy/mock.py: Removed.
    • Moved to Scripts/webkitpy/thirdparty.
  • Scripts/webkitpy/mock_bugzillatool.py:
  • Scripts/webkitpy/patchcollection_unittest.py:
  • Scripts/webkitpy/steps/closebugforlanddiff_unittest.py:
  • Scripts/webkitpy/steps/steps_unittest.py:
  • Scripts/webkitpy/steps/updatechangelogswithreview_unittests.py:
  • Scripts/webkitpy/steps_references.py: Added.
    • Added a file containing an absolute import of Mock so that the imports in the steps folder can import from this file (similar to style_references.py). This helps limit the number of affected files in future refactorings.
  • Scripts/webkitpy/thirdparty/mock.py: Copied from WebKitTools/Scripts/webkitpy/mock.py.
    • Also eliminated trailing white space and carriage returns.
11:28 Changeset [55967] by eric@webkit.org
  • 2 edits in trunk/WebCore

2010-03-13 Kim Grönholm <kim.gronholm@nomovok.com>

Reviewed by Kenneth Rohde Christiansen.

[Qt] GraphicsLayer: Opacity change from zero to non-zero doesn't always have effect with AC
https://bugs.webkit.org/show_bug.cgi?id=36034

  • platform/graphics/qt/GraphicsLayerQt.cpp: (WebCore::OpacityAnimationQt::applyFrame):
10:54 Changeset [55966] by eric@webkit.org
  • 2 edits in trunk/WebCore

2010-03-13 Kim Grönholm <kim.gronholm@nomovok.com>

Reviewed by Kenneth Rohde Christiansen.

[Qt] GraphicsLayer: Opacity transitions end with begin value
https://bugs.webkit.org/show_bug.cgi?id=36019

  • platform/graphics/qt/GraphicsLayerQt.cpp: (WebCore::AnimationQt::updateCurrentTime):
10:35 Changeset [55965] by eric@webkit.org
  • 1 edit
    2 adds in trunk/WebKit

2010-03-13 Leandro Pereira <leandro@profusion.mobi>

Reviewed by Kenneth Rohde Christiansen.

Add EFL port files to efl/ewk.
http://webkit.org/b/35925

  • efl/ewk/ewk_history.cpp: Added.
  • efl/ewk/ewk_history.h: Added.
10:17 Changeset [55964] by eric@webkit.org
  • 1 edit
    2 adds in trunk/WebKit

2010-03-13 Leandro Pereira <leandro@profusion.mobi>

Reviewed by Kenneth Rohde Christiansen.

Add EFL port files to efl/ewk.
http://webkit.org/b/35928

  • efl/ewk/ewk_main.cpp: Added.
  • efl/ewk/ewk_main.h: Added.
10:12 Changeset [55963] by ossy@webkit.org
  • 2 edits in trunk/WebKit/qt

[Qt] Enable accelerated compositing by default
https://bugs.webkit.org/show_bug.cgi?id=35866

Unreviewed. Roll-out r55955, because it broke 3 tests:

  • animations/fill-mode-transform.html
  • animations/play-state.html
  • animations/simultaneous-start-left.html
  • Api/qwebsettings.cpp:

(QWebSettings::QWebSettings):

07:25 QtWebKitJournal edited by hausmann@webkit.org
(diff)
05:43 Changeset [55962] by krit@webkit.org
  • 3 edits in trunk/WebCore

2010-03-13 Dirk Schulze <krit@webkit.org>

No review, rolling out r55927.
http://trac.webkit.org/changeset/55927
https://bugs.webkit.org/show_bug.cgi?id=35793

Breaks Gtk build bots.

  • platform/graphics/gstreamer/MediaPlayerPrivateGStreamer.cpp: (WebCore::MediaPlayerPrivate::MediaPlayerPrivate): (WebCore::MediaPlayerPrivate::load): (WebCore::MediaPlayerPrivate::setPreload):
  • platform/graphics/gstreamer/MediaPlayerPrivateGStreamer.h:
02:59 Changeset [55961] by robert@webkit.org
  • 2 edits in trunk/WebKitTools

2010-03-12 Robert Hogan <robert@webkit.org>

Reviewed by Adam Barth.

Amend incorrect typo patch for QtLauncher.

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

  • QtLauncher/webview.cpp: (WebViewGraphicsBased::setFrameRateMeasurementEnabled):
02:44 Changeset [55960] by eric@webkit.org
  • 1 edit
    19 adds in trunk/WebKitTools

2010-03-13 Victor Wang <victorw@chromium.org>

Add appengine app to host and serve webkit layout test results.

The app allows you post test result files (json) and serve them up.
Chromium flakiness dashboard will first use this app to host results.json
and expectations.json, but the files hosted by this app are not limited
to chromium results or json files. It can be used to host other files if needed.

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

  • TestResultServer: Added.
  • TestResultServer/app.yaml: Added.
  • TestResultServer/handlers: Added.
  • TestResultServer/handlers/init.py: Added.
  • TestResultServer/handlers/menu.py: Added.
  • TestResultServer/handlers/testfilehandler.py: Added.
  • TestResultServer/index.yaml: Added.
  • TestResultServer/main.py: Added.
  • TestResultServer/model: Added.
  • TestResultServer/model/init.py: Added.
  • TestResultServer/model/testfile.py: Added.
  • TestResultServer/stylesheets: Added.
  • TestResultServer/stylesheets/form.css: Added.
  • TestResultServer/stylesheets/menu.css: Added.
  • TestResultServer/stylesheets/testfile.css: Added.
  • TestResultServer/templates: Added.
  • TestResultServer/templates/menu.html: Added.
  • TestResultServer/templates/showfilelist.html: Added.
  • TestResultServer/templates/uploadform.html: Added.
02:28 Changeset [55959] by eric@webkit.org
  • 2 edits in trunk/WebKitTools

2010-03-13 Dimitri Glazkov <dglazkov@chromium.org>

Reviewed by David Levin.

new-run-webkit-tests fails with --debug option.
https://bugs.webkit.org/show_bug.cgi?id=36067

  • Scripts/webkitpy/layout_tests/port/mac.py:
02:11 Changeset [55958] by eric@webkit.org
  • 1 edit
    2 adds in trunk/WebKit

2010-03-13 Leandro Pereira <leandro@profusion.mobi>

Reviewed by Kenneth Rohde Christiansen.

Add EFL port files to efl/ewk.
http://webkit.org/b/35924

  • efl/ewk/ewk_frame.cpp: Added.
  • efl/ewk/ewk_frame.h: Added.
01:54 Changeset [55957] by eric@webkit.org
  • 1 edit
    1 add in trunk/WebKit

2010-03-13 Leandro Pereira <leandro@profusion.mobi>

Reviewed by Kenneth Rohde Christiansen.

Add EFL port files to efl/ewk.
http://webkit.org/b/35936

  • efl/ewk/ewk_view_single.c: Added.
01:38 Changeset [55956] by eric@webkit.org
  • 2 edits in trunk/WebKitTools

2010-03-13 Jesus Sanchez-Palencia <jesus.palencia@openbossa.org>

Reviewed by Kenneth Rohde Christiansen.

Add "Show FPS" menu option to QtLauncher.

[Qt] QtLauncher need a menu option to show/hide FPS
https://bugs.webkit.org/show_bug.cgi?id=35794

  • QtLauncher/main.cpp: (LauncherWindow::showFPS): (LauncherWindow::createChrome):
01:21 Changeset [55955] by eric@webkit.org
  • 2 edits in trunk/WebKit/qt

2010-03-13 Simon Hausmann <simon.hausmann@nokia.com>

Reviewed by Kenneth Rohde Christiansen.

[Qt] Enable accelerated compositing by default
https://bugs.webkit.org/show_bug.cgi?id=35866

  • Api/qwebsettings.cpp: (QWebSettings::QWebSettings):
01:04 Changeset [55954] by eric@webkit.org
  • 1 edit
    2 adds in trunk/WebKit

2010-03-13 Leandro Pereira <leandro@profusion.mobi>

Reviewed by Kenneth Rohde Christiansen.

Add EFL port files to efl/ewk.
http://webkit.org/b/35929

  • efl/ewk/ewk_util.cpp: Added.
  • efl/ewk/ewk_util.h: Added.
00:47 Changeset [55953] by eric@webkit.org
  • 1 edit
    5 adds in trunk/WebKit

2010-03-13 Leandro Pereira <leandro@profusion.mobi>

Reviewed by Kenneth Rohde Christiansen.

Add EFL port files to efl/ewk.
http://webkit.org/b/35934

  • efl/ewk/EWebKit.h: Added.
  • efl/ewk/ewk_eapi.h: Added.
  • efl/ewk/ewk_logging.h: Added.
  • efl/ewk/ewk_private.h: Added.
00:31 Changeset [55952] by eric@webkit.org
  • 3 edits in trunk/WebKit/chromium

2010-03-13 Eric Roman <eroman@chromium.org>

Reviewed by Adam Barth.

Add a firstPartyForCookies() property to WebDocument.
https://bugs.webkit.org/show_bug.cgi?id=35592

  • public/WebDocument.h:
  • src/WebDocument.cpp: (WebKit::WebDocument::firstPartyForCookies):
00:14 Changeset [55951] by eric@webkit.org
  • 2 edits in trunk/WebKitTools

2010-03-13 Jesus Sanchez-Palencia <jesus.palencia@openbossa.org>

Reviewed by Kenneth Rohde Christiansen.

Add a "Toggle FullScreen" option to QtLauncher Menu.

[Qt] QtLauncher needs an option to toggle FullScreen Mode
https://bugs.webkit.org/show_bug.cgi?id=35755

  • QtLauncher/main.cpp: (LauncherWindow::init): (LauncherWindow::eventFilter): (LauncherWindow::initializeView): (LauncherWindow::toggleFullScreenMode): (LauncherWindow::createChrome):

03/12/10:

23:57 Changeset [55950] by eric@webkit.org
  • 3 edits
    2 adds in trunk/LayoutTests

2010-03-12 Shu Chang <chang.shu@nokia.com>

Reviewed by David Levin.

[Qt] Rebaseline the test and remove it from Skipped.
https://bugs.webkit.org/show_bug.cgi?id=36057

  • platform/qt/Skipped:
  • platform/qt/fast/canvas/canvas-size-change-after-layout-expected.checksum: Added.
  • platform/qt/fast/canvas/canvas-size-change-after-layout-expected.png: Added.
  • platform/qt/fast/canvas/canvas-size-change-after-layout-expected.txt:
23:07 Changeset [55949] by krit@webkit.org
  • 19 edits in trunk

2010-03-12 Dirk Schulze <krit@webkit.org>

Reviewed by Nikolas Zimmermann.

SVG Glyphs - transform path not context
https://bugs.webkit.org/show_bug.cgi?id=36070

SVGFont should transform the path of a glyph, not the context. Modifying
the context causes wrong gradient transformations. This bug doesn't
influence CG because we generally fill or stroke texts with a mask image.
All other platforms provide a direct way to make the drawings.

  • svg/SVGFont.cpp: (WebCore::Font::drawTextUsingSVGFont):

2010-03-12 Dirk Schulze <krit@webkit.org>

Reviewed by Nikolas Zimmermann.

SVG Glyphs - transform path not context
https://bugs.webkit.org/show_bug.cgi?id=36070

Some tests don't look pixel perfect after the SVGFont changes (difference of 0.1%).
They just need an update.
Tests for the changes on other platforms than CG are

  • svg/batik/text/textEffect.svg
  • svg/batik/text/textEffect3.svg
  • svg/W3C-SVG-1.1/pservers-grad-08-b.svg
  • platform/mac-leopard/svg/custom/svg-fonts-in-html-expected.checksum:
  • platform/mac-leopard/svg/custom/svg-fonts-in-html-expected.png:
  • platform/mac/svg/W3C-SVG-1.1/animate-elem-03-t-expected.checksum:
  • platform/mac/svg/W3C-SVG-1.1/animate-elem-03-t-expected.png:
  • platform/mac/svg/W3C-SVG-1.1/animate-elem-36-t-expected.checksum:
  • platform/mac/svg/W3C-SVG-1.1/animate-elem-36-t-expected.png:
  • platform/mac/svg/W3C-SVG-1.1/animate-elem-40-t-expected.checksum:
  • platform/mac/svg/W3C-SVG-1.1/animate-elem-40-t-expected.png:
  • platform/mac/svg/W3C-SVG-1.1/pservers-grad-08-b-expected.checksum:
  • platform/mac/svg/W3C-SVG-1.1/pservers-grad-08-b-expected.png:
  • platform/mac/svg/batik/text/textEffect-expected.checksum:
  • platform/mac/svg/batik/text/textEffect-expected.png:
  • platform/mac/svg/batik/text/textEffect3-expected.checksum:
  • platform/mac/svg/batik/text/textEffect3-expected.png:
  • platform/mac/svg/text/selection-doubleclick-expected.png:
  • platform/mac/svg/text/selection-tripleclick-expected.png:
18:10 Changeset [55948] by barraclough@apple.com
  • 2 edits in trunk/JavaScriptCore

Reviewed by NOBODY (build fix).

17:56 Changeset [55947] by enrica@apple.com
  • 2 edits in trunk/WebKit/win

Fixed broken build on Windows.
Added contditional compilation for accelerated compositing.

  • WebView.cpp:

(WebView::deleteBackingStore):
(WebView::addToDirtyRegion):
(WebView::updateBackingStore):

17:39 Changeset [55946] by bdakin@apple.com
  • 14 edits in trunk

Fix for https://bugs.webkit.org/show_bug.cgi?id=34942 Fullscreen
API naming is inconsistent
-and corresponding-
<rdar://problem/7729165>

Reviewed by Simon Fraser.

WebCore:

This patch changes all occurrences of "fullScreen" to the more
popular "fullscreen." webkitEnterFullScreen and
webkitExitFullScreen have been maintained for now for backwards
compatibility.

  • html/HTMLVideoElement.cpp:

(WebCore::HTMLVideoElement::webkitEnterFullscreen):
(WebCore::HTMLVideoElement::webkitExitFullscreen):

  • html/HTMLVideoElement.h:

(WebCore::HTMLVideoElement::webkitEnterFullScreen):
(WebCore::HTMLVideoElement::webkitExitFullScreen):

  • html/HTMLVideoElement.idl:
  • platform/graphics/mac/MediaPlayerProxy.h:

WebKit/mac:

This patch changes all occurrences of "fullScreen" to the more
popular "fullscreen."

  • Plugins/Hosted/NetscapePluginHostProxy.h:

(WebKit::NetscapePluginHostProxy::isFullscreenWindowShowing):

  • Plugins/Hosted/NetscapePluginHostProxy.mm:

(WebKit::NetscapePluginHostProxy::NetscapePluginHostProxy):
(WebKit::NetscapePluginHostProxy::didEnterFullscreen):
(WebKit::NetscapePluginHostProxy::didExitFullscreen):
(WebKit::NetscapePluginHostProxy::setFullscreenWindowIsShowing):
(WKPCSetFullscreenWindowIsShowing):

  • Plugins/Hosted/WebKitPluginClient.defs:
  • Plugins/WebNetscapePluginView.mm:

(-[WebNetscapePluginView _workaroundSilverlightFullscreenBug:]):
(-[WebNetscapePluginView _createPlugin]):
(-[WebNetscapePluginView _destroyPlugin]):

  • WebView/WebVideoFullscreenHUDWindowController.mm:

(-[WebVideoFullscreenHUDWindowController windowDidLoad]):

WebKit/win:

This patch changes all occurrences of "fullScreen" to the more
popular "fullscreen."

  • FullscreenVideoController.cpp:

(FullscreenVideoController::onMouseDown):
(FullscreenVideoController::onMouseMove):
(FullscreenVideoController::onMouseUp):

  • FullscreenVideoController.h:

(FullscreenVideoController::fullscreenToHUDCoordinates):

17:02 Changeset [55945] by jamesr@google.com
  • 2 edits in trunk/WebCore

2010-03-12 James Robinson <jamesr@chromium.org>

Reviewed by Adam Barth.

Add virtual destructor to DOMWrapperWorld
https://bugs.webkit.org/show_bug.cgi?id=36077

DOMWrapperWorld is RefCounted<DOMWrapperWorld>. IsolatedWorld
inherits from DOMWrapperWorld and has member variables
with destructors, so DOMWrapperWorld needs to have a declared virtual
d'tor to ensure that its derived class's destructors are called.

No new tests, no change in behavior

  • bindings/v8/DOMWrapperWorld.h: (WebCore::DOMWrapperWorld::~DOMWrapperWorld):
16:46 Changeset [55944] by barraclough@apple.com
  • 2 edits in trunk/JavaScriptCore

Reviewed by NOBODY (build fix).

15:49 Changeset [55943] by barraclough@apple.com
  • 10 edits in trunk

Bug 36075 - Clean up screwyness re static string impls & Identifiers.

Reviewed by Oliver Hunt.

JavaScriptCore:

  • API/JSClassRef.cpp:

(OpaqueJSClass::~OpaqueJSClass): Classname may be null/empty, and these are an identifer. This is okay, since the null/empty strings are shared across all threads.

(JSC::Identifier::add): No need to explicitly hash null reps, this is done in the ststic UStringImpl constructor.
(JSC::Identifier::addSlowCase): UStringImpl::empty() handled & checkCurrentIdentifierTable now called in the header.
(JSC::Identifier::checkCurrentIdentifierTable): Replaces checkSameIdentifierTable (this no longer checked the rep since the identifierTable pointer was removed from UString::Rep long ago).

  • runtime/Identifier.h:

(JSC::Identifier::add): Replace call to checkSameIdentifierTable with call to checkCurrentIdentifierTable at head of function.

  • runtime/UStringImpl.cpp:

(JSC::UStringImpl::~UStringImpl): Remove call to checkConsistency - this function no longer checks anything interesting.

  • runtime/UStringImpl.h:

(JSC::UStringOrRopeImpl::UStringOrRopeImpl): Set s_refCountFlagIsIdentifier in static constructor.
(JSC::UStringImpl::UStringImpl): remove calls to checkConsistency (see above), add new ASSERT to substring constructor.
(JSC::UStringImpl::setHash): ASSERT not static (static strings set the hash in their constructor, should not reach this code path).
(JSC::UStringImpl::create): Add missing ASSERT.
(JSC::UStringImpl::setIsIdentifier): ASSERT !isStatic() (static strings hash set in constructor).

WebCore:

  • platform/text/StringImpl.cpp:

(WebCore::StringImpl::~StringImpl): Add ASSERT
(WebCore::StringImpl::sharedBuffer): Add ASSERT

  • platform/text/StringImpl.h:

(WebCore::StringImpl::setHash): Add ASSERT
(WebCore::StringImpl::isStatic): added.

15:17 Changeset [55942] by bdakin@apple.com
  • 2 edits in trunk/WebKit/win

Fix for https://bugs.webkit.org/show_bug.cgi?id=33739 Fullscreen
video HUD stays on top when switching to another window (e.g. via
Alt-Tab)
-and corresponding-
<rdar://problem/7547574>

Reviewed by Adam Roben.

The HUD was always on top because it had the WS_EX_TOPMOST style.
So I removed the style and made m_videoWindow the owner of
m_hudWindow. This keeps m_hudWindow on top only when m_videoWindow
is the focused window.

  • FullscreenVideoController.cpp:

(FullscreenVideoController::exitFullscreen): ASSERT that movie()->exitFullscreen() also destroyed the hud.
(FullscreenVideoController::createHUDWindow):

15:11 Changeset [55941] by enrica@apple.com
  • 6 edits
    10 adds
    1 delete in trunk

Content of 3D tests appears at the bottom right corner sometimes.
<rdar://problem/7556244>
<https://bugs.webkit.org/show_bug.cgi?id=36027>

Reviewed by Simon Fraser.

WebCore:

There were two problems to solve here:

  • the incorrect anchoring of the rootChildLayer that was causing the composited content to be positioned incorrectly
  • the failure to paint the non composited content into the backing store when animating composited content.


The first problem has been solved by leaving the original anchor point for the
rootChildLayer and splitting the tasks of clipping and scrolling using two separate layers.
The second problem has been solved leveraging the knowledge that WebView has of the dirty region
of the backing store to pass this information to the layer renderer. This allows the renderer to force
a paint into the backing store before moving to the compositing.

Tests: compositing/geometry/horizontal-scroll-composited.html

compositing/geometry/vertical-scroll-composited.html

  • manual-tests/win/horizontal-scroll-composited.html: Removed. This is now a layout test.
  • manual-tests/win/milliondollar.html: Added.
  • platform/graphics/win/WKCACFLayerRenderer.cpp:

(WebCore::WKCACFLayerRenderer::WKCACFLayerRenderer): Added initialization of dirty flag.
(WebCore::WKCACFLayerRenderer::setScrollFrame):
(WebCore::WKCACFLayerRenderer::updateScrollFrame): Updated to resize and position the clip and scroll layers.
(WebCore::WKCACFLayerRenderer::setRootChildLayer):
(WebCore::WKCACFLayerRenderer::createRenderer): Added new layer hierarchy.
(WebCore::WKCACFLayerRenderer::destroyRenderer): Remove clip layer on destroy.
(WebCore::WKCACFLayerRenderer::resize):
(WebCore::WKCACFLayerRenderer::paint): Forcing paint massage to trigger paint into the backing store.

  • platform/graphics/win/WKCACFLayerRenderer.h:

(WebCore::WKCACFLayerRenderer::setBackingStoreDirty): Added.

WebKit/win:

See detailed comments in WebCore/ChangeLog.

  • WebView.cpp:

(WebView::deleteBackingStore): Reset the dirty flag when deleting the backing store.
(WebView::addToDirtyRegion): Set the dirty flag when adding dirty rectangles to the
backing store dirty region.
(WebView::updateBackingStore): Reset the dirty flag after painting into the backing store.
(WebView::setAcceleratedCompositing): Removed unnecessary call to updateRootLayerContents.
(WebView::updateRootLayerContents): Changed the way we pass parameters to setScrollFrame.
We are passing width and height of the view content together with the offset for the scrolling.
It was confusing to pass it all as a rectangle, when it is not a rectangle.

LayoutTests:

  • compositing/geometry/horizontal-scroll-composited.html: Added.
  • compositing/geometry/vertical-scroll-composited.html: Added.
  • compositing/resources/apple.jpg: Added.
  • platform/mac/compositing/geometry/horizontal-scroll-composited-expected.checksum: Added.
  • platform/mac/compositing/geometry/horizontal-scroll-composited-expected.png: Added.
  • platform/mac/compositing/geometry/horizontal-scroll-composited-expected.txt: Added.
  • platform/mac/compositing/geometry/vertical-scroll-composited-expected.checksum: Added.
  • platform/mac/compositing/geometry/vertical-scroll-composited-expected.png: Added.
  • platform/mac/compositing/geometry/vertical-scroll-composited-expected.txt: Added.
14:58 Changeset [55940] by dpranke@chromium.org
  • 2 edits in trunk/WebKitTools

2010-03-12 Dirk Pranke <dpranke@chromium.org>

Reviewed by Dimitri Glazkov.

Fix typo in websocket_server (path_from_base instead of
path_from_chromium_base).

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

  • Scripts/webkitpy/layout_tests/port/websocket_server.py:
14:49 Changeset [55939] by robert@webkit.org
  • 2 edits in trunk/WebCore

2010-03-12 Robert Hogan <robert@webkit.org>

Not reviewed, build fix.

Revert http://trac.webkit.org/projects/webkit/changeset/55374 which broke
the !ENABLE(DATABASE) build on all platforms when attempting to fix the
--minimal build on Qt.

Support for SQLite now seems to be non-negotiable for the Qt build but making
it mandatory requires review, so re-break --minimal Qt build for now.

Qt build issue now tracked at https://bugs.webkit.org/show_bug.cgi?id=36073

  • page/GeolocationPositionCache.cpp:
14:43 Changeset [55938] by dpranke@chromium.org
  • 5 edits in trunk/WebKitTools

2010-03-12 Dirk Pranke <dpranke@chromium.org>

Reviewed by Adam Barth.

new-run-webkit-tests --new-baseline doesn't work at all.

It attempts to call a method that isn't defined. To fix it, I
removed the unnecessary and unnecessarily confusing 'platform'
argument to the test_type constructor and use the Port object that
is passed in instead, since we are only ever generating a baseline
from the port that is currently executing.

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

  • Scripts/webkitpy/layout_tests/port/mac.py:
  • Scripts/webkitpy/layout_tests/rebaseline_chromium_webkit_tests.py:
  • Scripts/webkitpy/layout_tests/run_webkit_tests.py:
  • Scripts/webkitpy/layout_tests/test_types/test_type_base.py:
14:42 Changeset [55937] by dpranke@chromium.org
  • 2 edits in trunk/WebKitTools

2010-03-12 Dirk Pranke <dpranke@chromium.org>

Reviewed by Adam Barth.

Fix new-run-webkit-tests --run-singly

This script option is currently broken - the script attempts to
dereference methods and variables that don't exist, which causes
the Chromium Linux valgrind bot to be quite unhappy. This has been
broken since r54449 when I renamed Port.start_test_driver to
Port.start_driver.

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

  • Scripts/webkitpy/layout_tests/layout_package/test_shell_thread.py:
14:31 Changeset [55936] by krit@webkit.org
  • 2 edits in trunk/LayoutTests

2010-03-12 Dirk Schulze <krit@webkit.org>

Unreviewed LayoutTest update.

Result was not updated with r55930.

  • platform/mac/svg/custom/dominant-baseline-hanging-expected.txt:
13:23 Changeset [55935] by mitz@apple.com
  • 2 edits in trunk/WebCore

<rdar://problem/7709115> REGRESSION: toolbar is missing at http://www.glom.org/
https://bugs.webkit.org/show_bug.cgi?id=35507

Reviewed by Alexey Proskuryakov.

  • css/CSSImportRule.cpp:

(WebCore::CSSImportRule::setCSSStyleSheet): Extend the change made in
<http://trac.webkit.org/changeset/48818> to detect the two variants of
KHTMLFixes.css in @import rules as well as in <link> elements.

13:16 Changeset [55934] by barraclough@apple.com
  • 2 edits in trunk/WebCore

Bug 36052 - [Qt] REGRESSION(55878) 63 test cases crash

Reviewed by Geoff Garen.

r55878 changed UStringImpl::empty()->characters() to be non-null,
so TextBreakIteratorQt.cpp now should check the length of strings
(previously was assuming all strings with a non-null data pointer
had a length of at least 1).

  • platform/text/qt/TextBreakIteratorQt.cpp:

(WebCore::wordBreakIterator):
(WebCore::characterBreakIterator):
(WebCore::lineBreakIterator):
(WebCore::sentenceBreakIterator):

13:15 Changeset [55933] by abarth@webkit.org
  • 2 edits in trunk/WebKit/chromium

2010-03-12 Garret Kelly <gdk@chromium.org>

Unreiviewed.

Fix Chromium Mac build. Last patch did not apply properly.
(Garret assures me this patch will actually work.)

  • src/WebInputEventConversion.cpp: (WebKit::toPlatformTouchEventType): (WebKit::toPlatformTouchPointState):
12:48 Changeset [55932] by abarth@webkit.org
  • 2 edits in trunk/WebKit/chromium

2010-03-12 Garret Kelly <gdk@chromium.org>

Unreviewed.

Fix Chromium Mac build. This time, with actual fixing goodness.

  • src/WebInputEventConversion.cpp: (WebKit::toPlatformTouchEventType): (WebKit::toPlatformTouchPointState):
12:47 Changeset [55931] by beidson@apple.com
  • 4 edits in trunk/WebKit/mac

Expose WebPDFView's underlying PDFDocument.

Patch by Andy Estes <aestes@apple.com> on 2010-03-12
Reviewed by Brady Eidson.

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

  • WebView/WebDocumentPrivate.h: Create a new protocol called

WebDocumentPDF.

  • WebView/WebPDFView.h: Have WebPDFView implement said protocol.
  • WebView/WebPDFView.mm:

(-[WebPDFView PDFDocument]): Expose WebPDFView's underlying
PDFDocument by implementing -(PDFDocument*)PDFDocument from
WebDocumentPDF.

12:43 Changeset [55930] by krit@webkit.org
  • 5 edits
    4 adds in trunk

2010-03-12 Dirk Schulze <krit@webkit.org>

Reviewed by Nikolas Zimmermann.

SVG fallback color doesn't work for bogus gradients.
https://bugs.webkit.org/show_bug.cgi?id=35479

Use a given fallback color on ignored gradients if present. Gradients
must be ignored, if one dimension of the objects boundingBox is zero.

Test: svg/custom/gradient-with-1d-boundingbox.svg

  • svg/graphics/SVGPaintServer.cpp: (WebCore::SVGPaintServer::strokePaintServer):
  • svg/graphics/SVGPaintServerGradient.cpp: (WebCore::SVGPaintServerGradient::setup):

2010-03-12 Dirk Schulze <krit@webkit.org>

Reviewed by Nikolas Zimmermann.

SVG fallback color doesn't work for bogus gradients.
https://bugs.webkit.org/show_bug.cgi?id=35479

We now use a fallback color on ignored gradients for one dimensional objects,
either a given fallback color or black as default.
This causes a change of one LayoutTest, because it was done manually in the
gradient PaintServer, without respecting a given fallback color.

  • platform/mac/svg/W3C-SVG-1.1/pservers-grad-17-b-expected.txt:
  • platform/mac/svg/custom/gradient-with-1d-boundingbox-expected.checksum: Added.
  • platform/mac/svg/custom/gradient-with-1d-boundingbox-expected.png: Added.
  • platform/mac/svg/custom/gradient-with-1d-boundingbox-expected.txt: Added.
  • svg/custom/gradient-with-1d-boundingbox.svg: Added.
12:36 Changeset [55929] by abarth@webkit.org
  • 2 edits in trunk/WebKit/chromium

2010-03-12 Garret Kelly <gdk@chromium.org>

Unreviewed.

Fix Chromium Mac build.

  • src/WebInputEventConversion.cpp: (WebKit::toPlatformTouchEventType): (WebKit::toPlatformTouchPointState):
12:29 Changeset [55928] by hyatt@apple.com
  • 16 edits
    1 delete in trunk/WebCore

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

Reviewed by Dan Bernstein.

Eliminate InlineRunBox.

  • WebCore.xcodeproj/project.pbxproj:
  • rendering/InlineFlowBox.cpp:

(WebCore::InlineFlowBox::adjustPosition):
(WebCore::InlineFlowBox::paintFillLayer):
(WebCore::InlineFlowBox::paintBoxDecorations):
(WebCore::InlineFlowBox::paintMask):

  • rendering/InlineFlowBox.h:

(WebCore::InlineFlowBox::InlineFlowBox):
(WebCore::InlineFlowBox::prevLineBox):
(WebCore::InlineFlowBox::nextLineBox):
(WebCore::InlineFlowBox::setNextLineBox):
(WebCore::InlineFlowBox::setPreviousLineBox):

  • rendering/InlineRunBox.h: Removed.
  • rendering/InlineTextBox.h:

(WebCore::InlineTextBox::InlineTextBox):
(WebCore::InlineTextBox::prevTextBox):
(WebCore::InlineTextBox::nextTextBox):
(WebCore::InlineTextBox::setNextTextBox):
(WebCore::InlineTextBox::setPreviousTextBox):

  • rendering/RenderBlock.cpp:

(WebCore::RenderBlock::destroy):
(WebCore::RenderBlock::rightmostPosition):
(WebCore::RenderBlock::leftmostPosition):

  • rendering/RenderInline.cpp:

(WebCore::RenderInline::destroy):
(WebCore::RenderInline::absoluteRects):
(WebCore::RenderInline::absoluteQuads):
(WebCore::RenderInline::linesBoundingBox):
(WebCore::RenderInline::linesVisibleOverflowBoundingBox):
(WebCore::RenderInline::addFocusRingRects):
(WebCore::RenderInline::paintOutline):

  • rendering/RenderLayer.cpp:

(WebCore::RenderLayer::localBoundingBox):

  • rendering/RenderLineBoxList.cpp:

(WebCore::RenderLineBoxList::deleteLineBoxTree):
(WebCore::RenderLineBoxList::extractLineBox):
(WebCore::RenderLineBoxList::attachLineBox):
(WebCore::RenderLineBoxList::removeLineBox):
(WebCore::RenderLineBoxList::deleteLineBoxes):
(WebCore::RenderLineBoxList::dirtyLineBoxes):
(WebCore::RenderLineBoxList::paint):
(WebCore::RenderLineBoxList::hitTest):
(WebCore::RenderLineBoxList::dirtyLinesFromChangedChild):
(WebCore::RenderLineBoxList::checkConsistency):

  • rendering/RenderSVGInline.cpp:

(WebCore::RenderSVGInline::absoluteRects):
(WebCore::RenderSVGInline::absoluteQuads):

  • rendering/RenderSVGText.cpp:

(WebCore::RenderSVGText::absoluteRects):
(WebCore::RenderSVGText::absoluteQuads):
(WebCore::RenderSVGText::objectBoundingBox):

  • rendering/RenderText.cpp:

(WebCore::RenderText::extractTextBox):
(WebCore::RenderText::attachTextBox):
(WebCore::RenderText::removeTextBox):
(WebCore::RenderText::createInlineTextBox):
(WebCore::RenderText::positionLineBox):

  • rendering/RootInlineBox.h:

(WebCore::RootInlineBox::nextRootBox):
(WebCore::RootInlineBox::prevRootBox):

12:11 Changeset [55927] by kov@webkit.org
  • 3 edits in trunk/WebCore

2010-03-12 Gustavo Noronha Silva <gustavo.noronha@collabora.co.uk>

Reviewed by Eric Carlson.

media/video-preload.html fails
https://bugs.webkit.org/show_bug.cgi?id=35793

Only effectively load, and start buffering when playing, or when
the preload attribute is set.

Test: media/video-preload.html

  • platform/graphics/gstreamer/MediaPlayerPrivateGStreamer.cpp: (WebCore::MediaPlayerPrivate::MediaPlayerPrivate): (WebCore::MediaPlayerPrivate::load): (WebCore::MediaPlayerPrivate::commitLoad): (WebCore::MediaPlayerPrivate::prepareToPlay): (WebCore::MediaPlayerPrivate::setPreload):
  • platform/graphics/gstreamer/MediaPlayerPrivateGStreamer.h:
12:06 Changeset [55926] by dimich@chromium.org
  • 30 edits
    7 deletes in trunk

Not reviewed, build fix.

Reverts 55920 and 55921. Landing for Jian Li.

WebCore:

  • Android.derived.jscbindings.mk:
  • Android.derived.v8bindings.mk:
  • DerivedSources.cpp:
  • DerivedSources.make:
  • GNUmakefile.am:
  • WebCore.gypi:
  • WebCore.pri:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • bindings/js/JSDOMFormDataCustom.cpp: Removed.
  • bindings/js/JSXMLHttpRequestCustom.cpp:

(WebCore::JSXMLHttpRequest::send):

  • bindings/scripts/CodeGeneratorJS.pm:
  • bindings/v8/V8Index.h:
  • bindings/v8/custom/V8DOMFormDataCustom.cpp: Removed.
  • bindings/v8/custom/V8XMLHttpRequestCustom.cpp:

(WebCore::V8XMLHttpRequest::sendCallback):

  • html/DOMFormData.idl: Removed.
  • page/DOMWindow.idl:

LayoutTests:

  • fast/dom/Window/window-properties-expected.txt:
  • fast/dom/Window/window-property-descriptors-expected.txt:
  • fast/dom/constructed-objects-prototypes-expected.txt:
  • fast/dom/prototype-inheritance-2-expected.txt:
  • fast/dom/prototype-inheritance-expected.txt:
  • fast/js/global-constructors-expected.txt:
  • http/tests/local/resources/send-form-data.js: Removed.
  • http/tests/local/send-form-data-expected.txt: Removed.
  • http/tests/local/send-form-data.html: Removed.
  • http/tests/xmlhttprequest/resources/multipart-post-echo.php: Removed.
  • platform/gtk/Skipped:
  • platform/gtk/fast/dom/Window/window-properties-expected.txt:
  • platform/qt/Skipped:
  • platform/qt/fast/dom/Window/window-properties-expected.txt:
  • platform/qt/fast/dom/Window/window-property-descriptors-expected.txt:
  • platform/qt/fast/dom/prototype-inheritance-expected.txt:
  • platform/qt/fast/js/global-constructors-expected.txt:
  • platform/win/Skipped:
12:03 Changeset [55925] by kov@webkit.org
  • 2 edits in trunk/WebKitTools

2010-03-12 Gustavo Noronha Silva <gustavo.noronha@collabora.co.uk>

Fix critical being printed to stderr on every test. This is
because the jar is only being created when soup hits the HTTP
path. We should reconsider the time of its creation.

  • DumpRenderTree/gtk/DumpRenderTree.cpp: (resetDefaultsToConsistentValues):
11:57 Changeset [55924] by tonikitoo@webkit.org
  • 2 edits in trunk/LayoutTests

Make the timeout in 'snav-unit-overflow-and-scroll-in-direction.html' 10x shorter (from 500ms to 50ms)

Rubber-stamped by Kenneth Christiansen.
Patch by Antonio Gomes <tonikitoo@webkit.org>

  • fast/events/spatial-navigation/snav-unit-overflow-and-scroll-in-direction.html:
11:45 Changeset [55923] by jpetsovits@rim.com
  • 3 edits in trunk/WebCore

2010-03-12 Jakob Petsovits <jpetsovits@rim.com>

Reviewed by Dirk Schulze.

[OpenVG] Add support for drawing text to PainterOpenVG
https://bugs.webkit.org/show_bug.cgi?id=35581

Doesn't come with any actual font classes, as OpenVG
by itself doesn't provide any access to platform fonts
but just the means to draw glyphs that have been loaded
manually before.

  • platform/graphics/openvg/PainterOpenVG.cpp: (WebCore::PlatformPainterState::PlatformPainterState): (WebCore::PlatformPainterState::copyPaintState): (WebCore::PainterOpenVG::textDrawingMode): (WebCore::PainterOpenVG::setTextDrawingMode): (WebCore::PainterOpenVG::drawText):
  • platform/graphics/openvg/PainterOpenVG.h:
11:25 Changeset [55922] by jpetsovits@rim.com
  • 6 edits in trunk/WebCore

2010-03-12 Jakob Petsovits <jpetsovits@rim.com>

Reviewed by Dirk Schulze.

[OpenVG] Use masks to implement non-rectilinear clipping
https://bugs.webkit.org/show_bug.cgi?id=35544

Requires some additional context switching logic to
make sure the right context is current when dealing
with the mask, because we don't store it by ourselves.

Initial version of this code was written by
Eli Fidler <efidler@rim.com>, I did a couple of
bug fixes and efficiency improvements since then.

  • platform/graphics/openvg/GraphicsContextOpenVG.cpp: (WebCore::GraphicsContext::clipPath): (WebCore::GraphicsContext::clip): (WebCore::GraphicsContext::clipOut): (WebCore::GraphicsContext::clipOutEllipseInRect): (WebCore::GraphicsContext::addInnerRoundedRectClip):
  • platform/graphics/openvg/PainterOpenVG.cpp: (WebCore::PlatformPainterState::PlatformPainterState): (WebCore::PlatformPainterState::~PlatformPainterState): (WebCore::PlatformPainterState::maskingEnabled): (WebCore::PlatformPainterState::applyState): (WebCore::PlatformPainterState::saveMaskIfNecessary): (WebCore::PainterOpenVG::intersectClipRect): (WebCore::PainterOpenVG::clipPath): (WebCore::PainterOpenVG::save):
  • platform/graphics/openvg/PainterOpenVG.h: (WebCore::PainterOpenVG::):
  • platform/graphics/openvg/SurfaceOpenVG.cpp: (WebCore::SurfaceOpenVG::makeCurrent): (WebCore::SurfaceOpenVG::makeCompatibleCurrent):
  • platform/graphics/openvg/SurfaceOpenVG.h: (WebCore::SurfaceOpenVG::):
11:17 Changeset [55921] by jianli@chromium.org
  • 4 edits
    1 add in trunk/WebCore

[v8] Add V8 bindings for DOMFormData.
https://bugs.webkit.org/show_bug.cgi?id=36026

Reviewed by Nate Chapin.

Test: http/tests/local/send-form-data.html

  • WebCore.gypi:
  • bindings/v8/V8Index.h:
  • bindings/v8/custom/V8DOMFormDataCustom.cpp: Added.

(WebCore::V8DOMFormData::appendCallback):

  • bindings/v8/custom/V8XMLHttpRequestCustom.cpp:

(WebCore::V8XMLHttpRequest::sendCallback):

11:17 Changeset [55920] by jianli@chromium.org
  • 27 edits
    6 adds in trunk

WebCore: Add DOMFormData.idl to expose FormData interface.
https://bugs.webkit.org/show_bug.cgi?id=36024

Reviewed by Sam Weinig.

The implementation is based on XMLHttpRequest 2 spec:
http://dev.w3.org/2006/webapi/XMLHttpRequest-2/#formdata

Test: http/tests/local/send-form-data.html

  • Android.derived.jscbindings.mk:
  • Android.derived.v8bindings.mk:
  • DerivedSources.cpp:
  • DerivedSources.make:
  • GNUmakefile.am:
  • WebCore.pri:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • bindings/js/JSDOMFormDataCustom.cpp: Added.
  • bindings/js/JSXMLHttpRequestCustom.cpp:

(WebCore::JSXMLHttpRequest::send):

  • bindings/scripts/CodeGeneratorJS.pm:
  • html/DOMFormData.idl: Added.
  • page/DOMWindow.idl:

LayoutTests: Add DOMFormData.idl to expose FormData interface
https://bugs.webkit.org/show_bug.cgi?id=36024

Reviewed by Sam Weinig.

Add a layout test to test sending FormData via XMLHttpRequest. Also
changed all test results that are affected by exposing FormData.

  • fast/dom/Window/window-properties-expected.txt:
  • fast/dom/Window/window-property-descriptors-expected.txt:
  • fast/dom/constructed-objects-prototypes-expected.txt:
  • fast/dom/prototype-inheritance-2-expected.txt:
  • fast/dom/prototype-inheritance-expected.txt:
  • fast/js/global-constructors-expected.txt:
  • http/tests/local/resources/send-form-data.js: Added.
  • http/tests/local/send-form-data-expected.txt: Added.
  • http/tests/local/send-form-data.html: Added.
  • http/tests/xmlhttprequest/resources/multipart-post-echo.php: Added.
  • platform/gtk/Skipped:
  • platform/gtk/fast/dom/Window/window-properties-expected.txt:
  • platform/qt/Skipped:
  • platform/qt/fast/dom/Window/window-properties-expected.txt:
  • platform/qt/fast/dom/Window/window-property-descriptors-expected.txt:
  • platform/qt/fast/dom/prototype-inheritance-expected.txt:
  • platform/qt/fast/js/global-constructors-expected.txt:
  • platform/win/Skipped:
11:11 Changeset [55919] by jpetsovits@rim.com
  • 5 edits in trunk/WebCore

2010-03-12 Jakob Petsovits <jpetsovits@rim.com>

Reviewed by Dirk Schulze.

[OpenVG] Add a SurfaceOpenVG constructor for EGL client buffer surfaces
https://bugs.webkit.org/show_bug.cgi?id=35538

SurfaceOpenVG can now not only encapsulate pbuffer
and window surfaces but also VGImage-based ones.

  • platform/graphics/openvg/EGLDisplayOpenVG.cpp: (WebCore::EGLDisplayOpenVG::createPbufferFromClientBuffer):
  • platform/graphics/openvg/EGLDisplayOpenVG.h:
  • platform/graphics/openvg/SurfaceOpenVG.cpp: (WebCore::SurfaceOpenVG::SurfaceOpenVG):
  • platform/graphics/openvg/SurfaceOpenVG.h:
11:06 Changeset [55918] by aroben@apple.com
  • 2 edits in trunk/WebKitTools

Teach prepare-ChangeLog to find modified selectors in CSS files

Reviewed by Tim Hatcher.

Fixes <http://webkit.org/b/36064> prepare-ChangeLog should extract
modified selectors from CSS files

  • Scripts/prepare-ChangeLog:

(get_function_line_ranges): Call get_selector_line_ranges_for_css for
.css files.
(get_selector_line_ranges_for_css): Added. Finds selectors and their
line ranges and returns them.

11:03 Changeset [55917] by eric@webkit.org
  • 3 edits
    2 adds in trunk

2010-03-12 Alpha Lam <hclam@chromium.org>

Reviewed by Eric Carlson.

HTMLMediaElement crash when it is used after resource loading is canceled.
https://bugs.webkit.org/show_bug.cgi?id=35992

Adding a test to access the duration property of a video element after
resource loading was canceled.

  • http/tests/media/video-cancel-load-expected.txt: Added.
  • http/tests/media/video-cancel-load.html: Added.

2010-03-12 Alpha Lam <hclam@chromium.org>

Reviewed by Eric Carlson.

Fix a crash when resource loading of media element is canceled.
https://bugs.webkit.org/show_bug.cgi?id=35992

Use of HTMLMediaElement::duration() after resource loading was canceled
will cause a crash. This is because HTMLMediaElement::m_player is used
when NULL.
Test: http/tests/media/video-cancel-load.html

  • html/HTMLMediaElement.cpp: (WebCore::HTMLMediaElement::duration): Avoid calling to m_player when it is null. (WebCore::HTMLMediaElement::userCancelledLoad): Set m_readyState to HAVE_NOTHING.
10:45 Changeset [55916] by eric@webkit.org
  • 2 edits in trunk/WebKit/chromium

2010-03-12 Garret Kelly <gdk@chromium.org>

Reviewed by Eric Seidel.

Enable touch events in Chromium features gypi file.
https://bugs.webkit.org/show_bug.cgi?id=35994

  • features.gypi:
10:36 Changeset [55915] by kov@webkit.org
  • 9 edits in trunk/WebKit/gtk

2010-03-12 Gustavo Noronha Silva <gustavo.noronha@collabora.co.uk>

Rubber-stamped by Kenneth Rohde Christiansen.

Misc documentation fixes. Fixes (almost) all warnings emitted by
the documentation build process. Only the broken references to JSC
objects remain.

  • webkit/webkitsecurityorigin.cpp:
  • webkit/webkitwebbackforwardlist.cpp:
  • webkit/webkitwebdatasource.cpp:
  • webkit/webkitwebframe.cpp:
  • webkit/webkitwebhistoryitem.cpp:
  • webkit/webkitwebinspector.cpp: (webkit_web_inspector_class_init):
  • webkit/webkitwebsettings.cpp: (webkit_web_settings_class_init):
  • webkit/webkitwebview.cpp: (DNDContentsRequest::webkit_web_view_class_init):
10:28 Changeset [55914] by mitz@apple.com
  • 2 edits in trunk/WebCore

<rdar://problem/7725534> CSSPrimitiveValue::parserValue() returns deleted memory
https://bugs.webkit.org/show_bug.cgi?id=20069

Reviewed by Darin Adler.

No test added, since with the CSS variables feature disabled, the pointer
to the freed memory is never dereferenced.

  • css/CSSPrimitiveValue.cpp:

(WebCore::valueOrPropertyName): Changed to return a const AtomicString& from
a static table.
(WebCore::CSSPrimitiveValue::parserValue): Updated for the above change.

10:18 Changeset [55913] by ojan@chromium.org
  • 38 edits
    4 copies
    7 adds in trunk/LayoutTests

2010-03-05 Ojan Vafai <ojan@chromium.org>

Reviewed by David Levin.

Tests for smartdelete should only occur after double-click
https://bugs.webkit.org/show_bug.cgi?id=35314

Fixes the tests to make selections using the mouse in order
to force smart-delete. It also adds/modifies tests to have them correctly
identify cases where we are smart-deleting, but shouldn't be.

Fixing to smart-delete only on mouse-based selections will be a followup patch.

  • editing/deleting/5390681-2.html:
  • editing/deleting/non-smart-delete.html: Added.
  • editing/deleting/smart-delete-001.html:
  • editing/deleting/smart-delete-002.html:
  • editing/deleting/smart-delete-003.html:
  • editing/deleting/smart-delete-004.html:
  • editing/editing.js: (doubleClick): (doubleClickAtSelectionStart):
  • editing/pasteboard/4944770-2.html:
  • editing/pasteboard/drag-drop-modifies-page.html:
  • editing/pasteboard/smart-drag-drop.html: Added.
  • editing/pasteboard/smart-paste-001.html:
  • editing/pasteboard/smart-paste-002.html:
  • editing/pasteboard/smart-paste-003.html:
  • editing/pasteboard/smart-paste-004.html:
  • editing/pasteboard/smart-paste-005.html:
  • editing/pasteboard/smart-paste-007.html:
  • editing/pasteboard/smart-paste-008.html:
  • editing/selection/delete-word-granularity-text-control-expected.txt: Added.
  • editing/selection/delete-word-granularity-text-control.html: Added.
  • editing/selection/script-tests/delete-word-granularity-text-control.js: Added.
  • editing/style/style-boundary-005.html:
  • platform/mac/editing/deleting/non-smart-delete-expected.checksum: Added.
  • platform/mac/editing/deleting/non-smart-delete-expected.png: Added.
  • platform/mac/editing/deleting/non-smart-delete-expected.txt: Added.
  • platform/mac/editing/deleting/smart-delete-001-expected.txt:
  • platform/mac/editing/deleting/smart-delete-002-expected.txt:
  • platform/mac/editing/deleting/smart-delete-003-expected.txt:
  • platform/mac/editing/deleting/smart-delete-004-expected.txt:
  • platform/mac/editing/pasteboard/drag-drop-modifies-page-expected.checksum:
  • platform/mac/editing/pasteboard/drag-drop-modifies-page-expected.png:
  • platform/mac/editing/pasteboard/drag-drop-modifies-page-expected.txt:
  • platform/mac/editing/pasteboard/smart-drag-drop-expected.checksum: Added.
  • platform/mac/editing/pasteboard/smart-drag-drop-expected.png: Added.
  • platform/mac/editing/pasteboard/smart-drag-drop-expected.txt: Added.
  • platform/mac/editing/pasteboard/smart-paste-001-expected.txt:
  • platform/mac/editing/pasteboard/smart-paste-002-expected.txt:
  • platform/mac/editing/pasteboard/smart-paste-003-expected.txt:
  • platform/mac/editing/pasteboard/smart-paste-004-expected.txt:
  • platform/mac/editing/pasteboard/smart-paste-005-expected.txt:
  • platform/mac/editing/pasteboard/smart-paste-007-expected.txt:
  • platform/mac/editing/pasteboard/smart-paste-008-expected.txt:
  • platform/mac/editing/style/style-boundary-005-expected.checksum:
  • platform/mac/editing/style/style-boundary-005-expected.png:
  • platform/mac/editing/style/style-boundary-005-expected.txt:
10:14 Changeset [55912] by darin@chromium.org
  • 2 edits in trunk/WebKit/chromium

2010-03-12 Darin Fisher <darin@chromium.org>

Fix Chromium build bustage.

Wrap usage of WebPrivatePtr from WebNode.h in a #if
WEBKIT_IMPLEMENTATION guard.

  • public/WebNode.h:
10:03 Changeset [55911] by mitz@apple.com
  • 2 edits in trunk/WebCore

Build fix.

  • platform/chromium/PlatformKeyboardEventChromium.cpp:
09:54 Changeset [55910] by ap@apple.com
  • 1 edit in trunk/WebKitTools/ChangeLog

Fixing an embarrassing typo in ChangeLog.

09:06 Changeset [55909] by mitz@apple.com
  • 16 edits
    2 adds in trunk

<rdar://problem/7694674> Hover states not updated when overflow section scrolls under stationary mouse pointer
https://bugs.webkit.org/show_bug.cgi?id=35949

Reviewed by Darin Adler.

WebCore:

Test: fast/events/overflow-scroll-fake-mouse-move.html

Soon after an overflow section scrolls under the mouse pointer, dispatch
a fake mouse move event. This is similar to how frame scrolling is handled
in WebKit, and has the effect of updating hover state, dispatching DOM mouse
events, and updating the tool tip.

  • page/EventHandler.cpp:

(WebCore::EventHandler::EventHandler): Initialize m_fakeMouseMoveEventTimer.
(WebCore::EventHandler::~EventHandler): Assert that the timer is not active.
(WebCore::EventHandler::clear): Stop the timer.
(WebCore::EventHandler::handleMousePressEvent): Cancel pending fake mouse
move events.
(WebCore::EventHandler::handleMouseMoveEvent): Ditto.
(WebCore::EventHandler::dispatchFakeMouseMoveEventSoonInQuad): If the mouse
is in the passed-in quad, ensure that a fake mouse move event is scheduled
to fire soon.
(WebCore::EventHandler::cancelFakeMouseMoveEvent): Does what the name says.
(WebCore::EventHandler::fakeMouseMoveEventTimerFired): Constructs a
PlatformMouseEvent with the current mouse location, modifier key state and
time stamp and calls mouseMoved().

  • page/EventHandler.h:
  • platform/PlatformKeyboardEvent.h: Declared getCurrentModifierState().
  • platform/android/KeyEventAndroid.cpp:

(WebCore::PlatformKeyboardEvent::getCurrentModifierState): Stubbed out.

  • platform/brew/PlatformKeyboardEventBrew.cpp:

(WebCore::PlatformKeyboardEvent::getCurrentModifierState): Ditto.

  • platform/chromium/PlatformKeyboardEventChromium.cpp:

(WebCore::PlatformKeyboardEvent::getCurrentModifierState): Added.

  • platform/efl/PlatformKeyboardEventEfl.cpp:

(WebCore::PlatformKeyboardEvent::getCurrentModifierState): Stubbed out.

  • platform/gtk/KeyEventGtk.cpp:

(WebCore::PlatformKeyboardEvent::getCurrentModifierState): Ditto.

  • platform/haiku/PlatformKeyboardEventHaiku.cpp:

(WebCore::PlatformKeyboardEvent::getCurrentModifierState): Added.

  • platform/mac/KeyEventMac.mm:

(WebCore::PlatformKeyboardEvent::getCurrentModifierState): Ditto.

  • platform/qt/PlatformKeyboardEventQt.cpp:

(WebCore::PlatformKeyboardEvent::getCurrentModifierState): Stubbed out.

  • platform/win/KeyEventWin.cpp:

(WebCore::PlatformKeyboardEvent::getCurrentModifierState): Added.

  • platform/wx/KeyboardEventWx.cpp:

(WebCore::PlatformKeyboardEvent::getCurrentModifierState): Ditto.

  • rendering/RenderLayer.cpp:

(WebCore::RenderLayer::scrollToOffset): Call
EventHandler::dispatchFakeMouseMoveEventSoonInQuad(). Moved things around
a little to avoid computing the repaint rect twice.

LayoutTests:

  • fast/events/overflow-scroll-fake-mouse-move-expected.txt: Added.
  • fast/events/overflow-scroll-fake-mouse-move.html: Added.
  • fast/events/touch/basic-multi-touch-events.html:
  • fast/events/touch/basic-single-touch-events.html:
09:03 Changeset [55908] by eric@webkit.org
  • 2 edits in trunk/WebKit/chromium

2010-03-12 Nicolas Weber <thakis@chromium.org>

Reviewed by David Levin.

[Chromium] Cmd-clicking submit buttons should submit in new tab
https://bugs.webkit.org/show_bug.cgi?id=36023

Take modifiers into account when clicking form buttons. E.g.
cmd-clicking a submit button will submit in a new background tab,
cmd-shift-clicking in a new foreground tab, shift-clicking in a new
window. (On windows/linux, it's ctrl instead of cmd.)

  • src/FrameLoaderClientImpl.cpp: (WebKit::FrameLoaderClientImpl::actionSpecifiesNavigationPolicy):
08:56 Changeset [55907] by ossy@webkit.org
  • 2 edits in trunk/LayoutTests

2010-03-12 Csaba Osztrogonác <ossy@webkit.org>

[Qt] css1/box_properties/acid_test.html failed in release mode (32 bit).
But it passes in debug mode (32 and 64 bit) and in release mode (64 bit).

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

  • platform/qt/Skipped: css1/box_properties/acid_test.html skipped again.
08:36 Changeset [55906] by eric@webkit.org
  • 2 edits in trunk/WebKit/chromium

2010-03-12 Kavita Kanetkar <kkanetkar@chromium.org>

Reviewed by Darin Fisher.

[chromium] Fix memory leak in WebImageDecoder API
https://bugs.webkit.org/show_bug.cgi?id=35946

  • src/WebImageDecoder.cpp: (WebKit::WebImageDecoder::setData): (WebKit::WebImageDecoder::getFrameAtIndex):
08:15 Changeset [55905] by kov@webkit.org
  • 3 edits in trunk/WebKitTools

Build fix (for EWS). Make sure the new code builds on older soup.

Thanks to Dan Winship.

08:05 Changeset [55904] by kov@webkit.org
  • 6 edits in trunk

2010-03-12 Gustavo Noronha Silva <gustavo.noronha@collabora.co.uk>

Reviewed by Eric Carlson.

[GTK] DRT does not handle cookie policy setting
https://bugs.webkit.org/show_bug.cgi?id=36056

Implement cookie accept policy setting for GTK+'s LayoutTestController.

  • DumpRenderTree/gtk/DumpRenderTree.cpp: (resetDefaultsToConsistentValues):
  • DumpRenderTree/gtk/LayoutTestControllerGtk.cpp: (LayoutTestController::setAlwaysAcceptCookies):

LayoutTests

2010-03-12 Gustavo Noronha Silva <gustavo.noronha@collabora.co.uk>

Reviewed by Eric Carlson.

[GTK] DRT does not handle cookie policy setting
https://bugs.webkit.org/show_bug.cgi?id=36056

Fix tests to request that the cookie acceptance policy always allows cookies.

  • http/tests/media/video-cookie.html:
  • http/tests/media/video-referer.html:
08:02 Changeset [55903] by eric@webkit.org
  • 2 edits in trunk/WebKitTools

2010-03-12 Adam Langley <agl@chromium.org>

Reviewed by Dimitri Glazkov.

[chromium]: update Linux layout test scripts for RedHat like systems.

(Tested on Fedora 12.)

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

  • Scripts/webkitpy/layout_tests/port/chromium_linux.py:
07:53 Changeset [55902] by ossy@webkit.org
  • 2 edits in trunk/LayoutTests

[Qt] Now css1/box_properties/acid_test.html passes, so we can unskip it.
original bug: https://bugs.webkit.org/show_bug.cgi?id=33834

  • platform/qt/Skipped: css1/box_properties/acid_test.html unskipped.
07:45 Changeset [55901] by eric@webkit.org
  • 2 edits in trunk/WebKit/mac

2010-03-12 Andy Estes <aestes@apple.com>

Reviewed by Brady Eidson.

Remove an unused method.

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

  • Plugins/WebPluginController.mm: removed - (void)showURL:(NSURL *) inFrame:(NSString *)
07:28 Changeset [55900] by eric@webkit.org
  • 2 edits in trunk/JavaScriptCore

2010-03-12 Peter Varga <pvarga@inf.u-szeged.hu>

Reviewed by David Levin.

Two functions tryConsumeCharacter() and tryConsumeCharacterClass() are
removed from yarr/RegexInterpreter.cpp because they are never called.

  • yarr/RegexInterpreter.cpp:
07:16 Changeset [55899] by ossy@webkit.org
  • 2 edits in trunk/LayoutTests

[Qt] DRT failing on test introduced in r55499
https://bugs.webkit.org/show_bug.cgi?id=35729

Now fast/events/zoom-dblclick.html passes, so we can unskip it.

  • platform/qt/Skipped: fast/events/zoom-dblclick.html unskipped.
07:12 Changeset [55898] by aroben@apple.com
  • 2 edits in trunk/WebKitTools

Make svn-create-patch's diffs of ObjC header files more readable

Fixes <http://webkit.org/b/36055>.

Reviewed by John Sullivan.

  • Scripts/svn-create-patch:

(diffOptionsForFile): Added. Returns the options that should be passed
to diff for the given file. All the options are the same for all
files, except for the option to specify which lines should be used as
hunk headers.
(generateDiff): Use diffOptionsForFile to get the options to pass to
diff.
(hunkHeaderLineRegExForFile): Added. Returns the regular expression
that should be used by diff to identify lines that should be included
after the "@@" in the hunk header lines of the diff. For ObjC[++]
source files, we use any lines starting with -, +, or
@implementation/@interface/@protocol. For ObjC[++] header files (which
we assume to be any .h files in a mac/ or objc/ directory), we use any
lines starting with @implementation/@interface/@protocol.

07:10 Changeset [55897] by eric@webkit.org
  • 5 edits in trunk

2010-03-12 Scott Byer <scottbyer@chromium.org>

Reviewed by David Levin.

Popup font size needs to be exposed to clients.
https://bugs.webkit.org/show_bug.cgi?id=35990

Add function to expose the popup menu font size, add a field to
WebPopupMenuInfo that receives that information to convey that to
the web view client's createPopupMenu() call.

  • WebCore/platform/chromium/PopupMenuChromium.cpp:
  • WebCore/platform/chromium/PopupMenuChromium.h:
  • WebKit/chromium/public/WebPopupMenuInfo.h:
  • WebKit/chromium/src/ChromeClientImpl.cpp:
06:53 Changeset [55896] by eric@webkit.org
  • 2 edits in trunk/WebCore

2010-03-12 Kent Hansen <kent.hansen@nokia.com>

Reviewed by Darin Adler.

Webkit doesn't build with workers on and database off
https://bugs.webkit.org/show_bug.cgi?id=35997

Added missing ENABLE(DATABASE) guards.

  • workers/WorkerThread.cpp: (WebCore::WorkerThreadShutdownStartTask::performTask):
06:25 Changeset [55895] by kov@webkit.org
  • 2 edits in trunk/LayoutTests

2010-03-12 Gustavo Noronha Silva <gustavo.noronha@collabora.co.uk>

[GTK] Fails fast/dom/Geolocation/maximum-age.htm
https://bugs.webkit.org/show_bug.cgi?id=36053

Skip new failing test.

  • platform/gtk/Skipped:
06:25 Changeset [55894] by kov@webkit.org
  • 4 edits in trunk

2010-03-12 Dan Winship <danw@gnome.org>

Reviewed by Gustavo Noronha.

Make the defaultCookieJar use a no-third-party policy. Most
applications set their own cookie jar, but DumpRenderTree doesn't,
so it was failing the new third-party-cookie test.

  • platform/network/soup/CookieJarSoup.cpp: (WebCore::defaultCookieJar):

LayoutTests

2010-03-12 Dan Winship <danw@gnome.org>

Reviewed by Gustavo Noronha.

Revert previous; the test passes now.

  • platform/gtk/Skipped:
06:06 Changeset [55893] by eric@webkit.org
  • 8 edits
    4 adds in trunk

2010-03-12 Jochen Eisinger <jochen@chromium.org>

Reviewed by Jeremy Orlow.

Test for referrer information being stripped when the header is removed in willSendRequest
https://bugs.webkit.org/show_bug.cgi?id=35920

  • http/tests/security/no-referrer-expected.txt: Added.
  • http/tests/security/no-referrer.html: Added.
  • http/tests/security/resources/no-referrer-frame.php: Added.
  • http/tests/security/resources/no-referrer.php: Added.
  • platform/gtk/Skipped:
  • platform/qt/Skipped:

2010-03-12 Jochen Eisinger <jochen@chromium.org>

Reviewed by Jeremy Orlow.

Introduce setWillSendRequestClearHeader to LayoutTestController to selectively remove headers in willSendRequest. Used in http/tests/security/no-referrer.html
https://bugs.webkit.org/show_bug.cgi?id=35920

  • DumpRenderTree/LayoutTestController.cpp: (setWillSendRequestClearHeaderCallback): (LayoutTestController::staticFunctions):
  • DumpRenderTree/LayoutTestController.h: (LayoutTestController::willSendRequestClearHeaders): (LayoutTestController::setWillSendRequestClearHeader):
  • DumpRenderTree/mac/ResourceLoadDelegate.mm: (-[ResourceLoadDelegate webView:resource:willSendRequest:redirectResponse:fromDataSource:]):
  • DumpRenderTree/win/ResourceLoadDelegate.cpp: (ResourceLoadDelegate::willSendRequest):
05:44 Changeset [55892] by zimmermann@webkit.org
  • 5 edits in trunk/LayoutTests

2010-03-12 Nikolas Zimmermann <nzimmermann@rim.com>

Rubber-stamped by Dirk Schulze.

Update two tests, whose pixel hash fail with --tolerance 0. No visible differences.

  • platform/mac/svg/custom/dominant-baseline-hanging-expected.checksum:
  • platform/mac/svg/custom/dominant-baseline-hanging-expected.png:
  • platform/mac/svg/custom/gradient-stop-corner-cases-expected.checksum:
  • platform/mac/svg/custom/gradient-stop-corner-cases-expected.png:

03/11/10:

23:24 Changeset [55891] by eric@webkit.org
  • 4 edits in trunk/WebCore

2010-03-11 Simon Hausmann <simon.hausmann@nokia.com>

Reviewed by Laszlo Gombos.

[Qt] Enable network state notifier when compiling against Qt 4.7
https://bugs.webkit.org/show_bug.cgi?id=35983

  • WebCore.pri:
  • platform/network/qt/NetworkStateNotifierPrivate.h:
  • platform/network/qt/NetworkStateNotifierQt.cpp:
23:07 Changeset [55890] by eric@webkit.org
  • 8 edits in trunk/WebCore

2010-03-11 Benjamin Poulain <benjamin.poulain@nokia.com>

Reviewed by Adam Treat.

Do not render the full frame when there is some elements with fixed positioning
https://bugs.webkit.org/show_bug.cgi?id=33150

The frame view take into acount the list of fixed object when scrolling
the view. If the number of object is lower than a certain threshold, the pixel
are blitted, and the invalidated area updated.

  • page/FrameView.cpp: (WebCore::FrameView::FrameView): (WebCore::FrameView::useSlowRepaints): (WebCore::FrameView::useSlowRepaintsIfNotOverlapped): (WebCore::FrameView::addFixedObject): (WebCore::FrameView::removeFixedObject): (WebCore::FrameView::scrollContentsFastPath):
  • page/FrameView.h:
  • platform/ScrollView.cpp: (WebCore::ScrollView::scrollContents): (WebCore::ScrollView::scrollContentsFastPath):
  • platform/ScrollView.h:
  • rendering/RenderBlock.h: (WebCore::RenderBlock::positionedObjects):
  • rendering/RenderBox.cpp: (WebCore::RenderBox::styleWillChange):
  • rendering/RenderObject.cpp: (WebCore::RenderObject::styleWillChange):
22:46 Changeset [55889] by eric@webkit.org
  • 2 edits in trunk/JavaScriptCore

2010-03-11 Kent Hansen <kent.hansen@nokia.com>

Reviewed by Simon Hausmann.

Mac OS X: Use deployment target to determine whether memory tagging should be enabled
https://bugs.webkit.org/show_bug.cgi?id=34888

Use TARGETING_TIGER rather than BUILDING_ON_TIGER to determine the tags.

  • wtf/VMTags.h:
22:11 Changeset [55888] by eric@webkit.org
  • 12 edits in trunk

2010-03-11 Aaron Boodman <aa@chromium.org>

Kill WebDocument::applicationID() (part 1).

Modify interface to WebCore::NotificationPresenter::checkPermission()
and remove implementation of WebDocument::applicationID(). Breaking
API changes will be in a subsequent change.
https://bugs.webkit.org/show_bug.cgi?id=35846

  • notifications/Notification.cpp: (WebCore::Notification::Notification):
  • notifications/NotificationCenter.cpp: (WebCore::NotificationCenter::checkPermission):
  • notifications/NotificationPresenter.h:

2010-03-11 Aaron Boodman <aa@chromium.org>

Kill WebDocument::applicationID() (part 1).

Modify interface to WebCore::NotificationPresenter::checkPermission()
and remove implementation of WebDocument::applicationID(). Breaking
API changes will be in a subsequent change.
https://bugs.webkit.org/show_bug.cgi?id=35846

  • public/WebNotificationPresenter.h:
  • src/NotificationPresenterImpl.cpp: (WebKit::NotificationPresenterImpl::checkPermission):
  • src/NotificationPresenterImpl.h:
  • src/WebDocument.cpp: (WebKit::WebDocument::applicationID):

2010-03-11 Aaron Boodman <aa@chromium.org>

Kill WebDocument::applicationID() (part 1).

Modify interface to WebCore::NotificationPresenter::checkPermission()
and remove implementation of WebDocument::applicationID(). Breaking
API changes will be in a subsequent change.
https://bugs.webkit.org/show_bug.cgi?id=35846

  • WebCoreSupport/WebDesktopNotificationsDelegate.cpp: (WebDesktopNotificationsDelegate::checkPermission):
  • WebCoreSupport/WebDesktopNotificationsDelegate.h:
21:41 Changeset [55887] by eric@webkit.org
  • 2 edits in trunk/JavaScriptCore

2010-03-11 Jedrzej Nowacki <jedrzej.nowacki@nokia.com>

Reviewed by Simon Hausmann.

The JSNative state was renamed to JSPrimitive. The new name better
coresponds to the ECMAScript standard.

Enum QScriptValuePrivate::States was renamed to State to obey Qt
coding style rules ("States" name suggests that a state could
mixed together with an other state using bitwise logic operators.

[Qt] QScriptValuePrivate::States has naming issues
https://bugs.webkit.org/show_bug.cgi?id=35968

  • qt/api/qscriptvalue_p.h: (QScriptValuePrivate::): (QScriptValuePrivate::QScriptValuePrivate): (QScriptValuePrivate::isBool): (QScriptValuePrivate::isNumber): (QScriptValuePrivate::isNull): (QScriptValuePrivate::isString): (QScriptValuePrivate::isUndefined): (QScriptValuePrivate::toString): (QScriptValuePrivate::toNumber): (QScriptValuePrivate::toBool): (QScriptValuePrivate::assignEngine): (QScriptValuePrivate::refinedJSValue):
21:24 Changeset [55886] by eric@webkit.org
  • 3 edits in trunk/LayoutTests

2010-03-11 Shu Chang <Chang.Shu@nokia.com>

Reviewed by David Levin.

[Qt] Rebaseline the test and remove it from Skipped.
https://bugs.webkit.org/show_bug.cgi?id=35969

  • platform/qt/Skipped:
  • platform/qt/fast/canvas/canvas-resize-reset-expected.txt:
20:39 Changeset [55885] by barraclough@apple.com
  • 2 edits in trunk/JavaScriptCore

Add export.

Reviewed by NOBODY (Windows build fix).

20:36 Changeset [55884] by ukai@chromium.org
  • 2 edits in trunk/WebKitTools

2010-03-11 Fumitoshi Ukai <ukai@chromium.org>

Unreviewed.

Fix typo in websocket_server.py

  • Scripts/webkitpy/layout_tests/port/websocket_server.py:
20:07 Changeset [55883] by barraclough@apple.com
  • 2 edits in trunk/JavaScriptCore

Add export.

Reviewed by NOBODY (Windows build fix).

19:51 Changeset [55882] by dimich@chromium.org
  • 9 edits in trunk/WebCore

[v8] Remove obsolete code for delayed dereferencing of DOM objects for single-heap, multithread v8 usage.
https://bugs.webkit.org/show_bug.cgi?id=36043

Reviewed by David Levin.

No new tests, no changes in functionality.

  • bindings/v8/DOMData.cpp:

(WebCore::DOMData::DOMData):

  • bindings/v8/DOMData.h:

(WebCore::DOMData::handleWeakObject):

  • bindings/v8/DOMDataStore.cpp:
  • bindings/v8/DOMDataStore.h:

(WebCore::DOMDataStore::domObjectMap):
(WebCore::DOMDataStore::activeDomObjectMap):
(WebCore::DOMDataStore::domSvgElementInstanceMap):
(WebCore::DOMDataStore::domSvgObjectWithContextMap):

  • bindings/v8/ScopedDOMDataStore.cpp:

(WebCore::ScopedDOMDataStore::ScopedDOMDataStore):

  • bindings/v8/StaticDOMDataStore.cpp:

(WebCore::StaticDOMDataStore::StaticDOMDataStore):

  • bindings/v8/StaticDOMDataStore.h:
  • bindings/v8/V8DOMMap.cpp:

(WebCore::removeAllDOMObjectsInCurrentThread):

19:25 Changeset [55881] by barraclough@apple.com
  • 4 edits in trunk

Rubber stamped by Oliver Hunt.

Remove nonsense comments used in development & commited in error.

JavaScriptCore:

  • runtime/UStringImpl.h:

WebCore:

  • platform/text/StringImpl.h:
19:22 Changeset [55880] by eric@webkit.org
  • 2 edits in trunk/WebKitTools

2010-03-11 Garret Kelly <gdk@chromium.org>

Reviewed by Darin Adler.

Fixing minor typo in the commit queue status page.
https://bugs.webkit.org/show_bug.cgi?id=35979

  • Scripts/webkitpy/commands/queues.py:
19:20 Changeset [55879] by barraclough@apple.com
  • 2 edits in trunk/JavaScriptCore

Remove export.

Reviewed by NOBODY (Windows build fix).

19:14 Changeset [55878] by barraclough@apple.com
  • 8 edits in trunk

https://bugs.webkit.org/show_bug.cgi?id=36041
Remove unnecessary differences in common code between WebCore::StringImpl & JSC::UStringImpl

Reviewed by Oliver Hunt.

Much of the code in WebCore::StringImpl and JSC::UStringImpl is now very similar,
but has trivial and unnecessary formatting differences, such as the exact wording
of comments, missing ASSERTs, functions implemented in the .h vs .cpp etc.

JavaScriptCore:

  • runtime/Identifier.cpp:

(JSC::Identifier::add): UStringImpl::empty() now automatically hashes, uas per WebCore strings.
(JSC::Identifier::addSlowCase): UStringImpl::empty() now automatically hashes, uas per WebCore strings.

  • runtime/UStringImpl.cpp:

(JSC::UStringImpl::~UStringImpl): Only call bufferOwnership() once, add missing ASSERTs.
(JSC::UStringImpl::createUninitialized): Move from .h, not commonly called, no need to inline.
(JSC::UStringImpl::create): Move from .h, not commonly called, no need to inline.
(JSC::UStringImpl::sharedBuffer): Rewritten to more closely match WebCore implementation, remove need for separate baseSharedBuffer() method.

  • runtime/UStringImpl.h:

(JSC::UStringImpl::UStringImpl): Automatically hash static strings, ASSERT m_data & m_length are non-null/non-zero in non-static strings.
(JSC::UStringImpl::setHash): Add missing ASSERT.
(JSC::UStringImpl::create): Moved to .cpp / added missing check for empty string creation.
(JSC::UStringImpl::adopt): Vector.size() returns size_t, not unsigned.
(JSC::UStringImpl::cost): Renamed m_bufferSubstring -> m_substringBuffer
(JSC::UStringImpl::hash): Reordered in file.
(JSC::UStringImpl::existingHash): Reordered in file.
(JSC::UStringImpl::computeHash): Reordered in file, renamed parameter.
(JSC::UStringImpl::checkConsistency): rewrote ASSERT.
(JSC::UStringImpl::bufferOwnership): Return type should be BufferOwnership.
(JSC::UStringImpl::): Moved friends to head of class.

WebCore:

  • platform/text/StringImpl.cpp:

(WebCore::StringImpl::empty): Reordered in file, made empty()->characters() return a non-null value to match JSC.
(WebCore::StringImpl::createUninitialized): Added overflow check.
(WebCore::StringImpl::create): Reordered in file.
(WebCore::StringImpl::sharedBuffer): Reordered in file.

  • platform/text/StringImpl.h:

(WebCore::StringImpl::): Remove ThreadGlobalData as friend, move SharableUChar & SharedUChar to WebCore namespace.
(WebCore::StringImpl::StringImpl): Made static constructor method (used to create empty string) take arguments, to match JSC & prevent accidental use.
(WebCore::StringImpl::setHash): Added missing ASSERT.
(WebCore::StringImpl::adopt): Make adpot work with Vectors with a non-zero inline capacity.
(WebCore::StringImpl::characters): Mark as const to match JSC.
(WebCore::StringImpl::hash): Use !m_hash instead of m_hash == 0.
(WebCore::StringImpl::computeHash): Remove redundant 'inline'.

19:04 Changeset [55877] by eric@webkit.org
  • 2 edits in trunk/WebKit/chromium

2010-03-11 Stuart Morgan <stuartmorgan@chromium.org>

Reviewed by Darin Fisher.

Fix the screen coordinate conversion of Chromium Mac mouse events for
multiple-moniter setups. Also changes the global coordinate values
to be computed from the event, rather than the instantaneous mouse
location, and reduces code duplication for location conversion.

Dual-sided patch with http://codereview.chromium.org/751002

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

  • src/mac/WebInputEventFactory.mm: (WebKit::setWebEventLocationFromEventInView): (WebKit::WebInputEventFactory::mouseEvent): (WebKit::WebInputEventFactory::mouseWheelEvent):
18:47 Changeset [55876] by eric@webkit.org
  • 1 edit
    2 adds in trunk/LayoutTests

2010-03-11 Ryosuke Niwa <rniwa@webkit.org>

Reviewed by Darin Adler.

Applying h1 format to text node after HR tag fails to include first letter of text node.
https://bugs.webkit.org/show_bug.cgi?id=30148

This patch adds a test case to the above bug, which has been fixed by
http://trac.webkit.org/changeset/55705.

  • editing/style/heading-text-after-hr-expected.txt: Added.
  • editing/style/heading-text-after-hr.html: Added.
18:37 Changeset [55875] by ukai@chromium.org
  • 2 edits in trunk/WebKitTools

2010-03-11 Fumitoshi Ukai <ukai@chromium.org>

Unreviewed.

Fix for WebSocket layout test runner on chromium/win port.

  • Scripts/webkitpy/layout_tests/port/websocket_server.py: register_cygwin and set CYGWIN_PATH
18:20 Changeset [55874] by eric@webkit.org
  • 1 edit
    2 adds in trunk/WebKit

2010-03-11 Leandro Pereira <leandro@profusion.mobi>

Reviewed by Gustavo Noronha Silva.

Add EFL port files to efl/WebCoreSupport.
http://webkit.org/b/35917

  • efl/WebCoreSupport/InspectorClientEfl.h: Added.
  • efl/WebCoreSupport/InspectorClientEfl.cpp: Added.
18:15 Changeset [55873] by jianli@chromium.org
  • 4 edits
    3 adds in trunk/LayoutTests

Add more test cases for Blob.slice.
https://bugs.webkit.org/show_bug.cgi?id=36038

Reviewed by Dmitry Titov.

  • http/tests/local/resources/abe.png: Copied from LayoutTests/editing/resources/abe.png.
  • http/tests/local/resources/empty.txt: Copied from LayoutTests/http/tests/appcache/resources/empty.txt.
  • http/tests/local/resources/send-sliced-dragged-file.js:
  • http/tests/local/send-sliced-dragged-file-expected.txt:
  • http/tests/resources/post-and-verify.cgi: Added.
  • http/tests/resources/write-temp-file.php:
18:03 Changeset [55872] by eric@webkit.org
  • 1 edit
    2 adds in trunk/WebKit

2010-03-11 Leandro Pereira <leandro@profusion.mobi>

Reviewed by Gustavo Noronha Silva.

Add EFL port files to efl/WebCoreSupport.
http://webkit.org/b/35914

  • efl/WebCoreSupport/DragClientEfl.h: Added.
  • efl/WebCoreSupport/DragClientEfl.cpp: Added.
17:53 Changeset [55871] by mrowe@apple.com
  • 10 edits in trunk

<rdar://problem/7745082> Make it possible to build WebKit for older Mac OS X versions from the current Mac OS X version

Reviewed by David Kilzer.

Default to using the appropriate SDK if the target Mac OS X version is not the current Mac OS X version.

JavaScriptCore:

  • Configurations/Base.xcconfig:

JavaScriptGlue:

  • Configurations/Base.xcconfig:

WebCore:

  • Configurations/Base.xcconfig:

WebKit/mac:

  • Configurations/Base.xcconfig:

WebKitTools:

  • DumpRenderTree/mac/Configurations/Base.xcconfig:
17:37 Changeset [55870] by eric@webkit.org
  • 4 edits
    4 adds in trunk

2010-03-11 Jungshik Shin <jshin@chromium.org>

[Chromium]: Plane 2 characters are rendered "blank"

(not even empty boxes) on Windows even when there are fonts to
cover them.

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

Test:LayoutTests/fast/text/international/plane2.html

  • platform/graphics/chromium/FontCacheChromiumWin.cpp: (WebCore::LookupAltName): Add two ExtB fonts to the array (namePairs) that are used for Plane 2 character rendering. (WebCore::FontCache::getFontDataForCharacters): Add two more fonts to the fallback font list
  • platform/graphics/chromium/FontUtilsChromiumWin.cpp: (WebCore::getFallbackFamily): Fix the fallback font lookup to cover Plane 2 (CJK ExtB).
17:20 Changeset [55869] by eric@webkit.org
  • 1 edit
    1 add in trunk/WebCore

2010-03-11 Leandro Pereira <leandro@profusion.mobi>

Reviewed by Holger Freyther.

Add EFL port files to platform/efl.
http://webkit.org/b/35889

  • platform/efl/SystemTimeEfl.cpp: Added.
16:50 Changeset [55868] by victorw@chromium.org
  • 2 edits in trunk/WebKitTools

2010-03-11 Victor Wang <victorw@chromium.org>

Reviewed by dglazkov@chromium.org.

rebaseline_chromium_webkit_tests can generate new baselines for
all platforms so it needs to know two ports in order to work correctly:
the port that the script is running on and the port that it generates
new baselines for. Update rebaselining tool to handle both port correctly.

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

  • Scripts/webkitpy/layout_tests/rebaseline_chromium_webkit_tests.py:
16:44 Changeset [55867] by mrowe@apple.com
  • 26 edits in trunk

<rdar://problem/7745082> Make it possible to build WebKit for older Mac OS X versions from the current Mac OS X version

Reviewed by Tim Hatcher.

Introduce TARGET_MAC_OS_X_VERSION_MAJOR to represent the Mac OS X version that is being targeted. It defaults to the
current Mac OS X version unless otherwise specified.

Key off TARGET_MAC_OS_X_VERSION_MAJOR where we'd previously been keying off MAC_OS_X_VERSION_MAJOR.

Explicitly map from the target Mac OS X version to the preferred compiler since Xcode's default compiler choice
may not be usable when targetting a different Mac OS X version.

Key off TARGET_GCC_VERSION rather than MAC_OS_X_VERSION_MAJOR in locations where we'd previously been keying off
MAC_OS_X_VERSION_MAJOR but the decision is really related to the compiler version being used.

JavaScriptCore:

  • Configurations/Base.xcconfig:
  • Configurations/DebugRelease.xcconfig:
  • Configurations/FeatureDefines.xcconfig:
  • Configurations/JavaScriptCore.xcconfig:
  • Configurations/Version.xcconfig:

JavaScriptGlue:

  • Configurations/Base.xcconfig:
  • Configurations/DebugRelease.xcconfig:
  • Configurations/Version.xcconfig:
  • JavaScriptGlue.xcodeproj/project.pbxproj:

WebCore:

  • Configurations/Base.xcconfig:
  • Configurations/DebugRelease.xcconfig:
  • Configurations/FeatureDefines.xcconfig:
  • Configurations/Version.xcconfig:
  • Configurations/WebCore.xcconfig:

WebKit/mac:

  • Configurations/Base.xcconfig:
  • Configurations/DebugRelease.xcconfig:
  • Configurations/FeatureDefines.xcconfig:
  • Configurations/Version.xcconfig:

WebKitTools:

  • DumpRenderTree/mac/Configurations/Base.xcconfig:
  • DumpRenderTree/mac/Configurations/DebugRelease.xcconfig:
  • DumpRenderTree/mac/DumpRenderTree.mm: Wrap the include of mach-o/getsect.h in 'extern "C"' as some versions of the

header in older SDKs do not do this inside the header.

16:40 Changeset [55866] by darin@chromium.org
  • 14 edits in trunk/WebKit/chromium

2010-03-10 Darin Fisher <darin@chromium.org>

Reviewed by Jeremy Orlow.

Use WebPrivatePtr for WebNode
https://bugs.webkit.org/show_bug.cgi?id=36005

  • public/WebElement.h:
  • public/WebFormElement.h:
  • public/WebInputElement.h:
  • public/WebLabelElement.h:
  • public/WebNode.h: (WebKit::WebNode::WebNode): (WebKit::WebNode::isNull): (WebKit::WebNode::unwrap): (WebKit::WebNode::constUnwrap):
  • public/WebPrivatePtr.h: (WebKit::WebPrivatePtr::assign):
  • src/WebDocument.cpp: (WebKit::WebDocument::WebDocument): (WebKit::WebDocument::operator=): (WebKit::WebDocument::operator PassRefPtr<Document>):
  • src/WebElement.cpp: (WebKit::WebElement::hasTagName): (WebKit::WebElement::WebElement): (WebKit::WebElement::operator=): (WebKit::WebElement::operator PassRefPtr<Element>):
  • src/WebFormElement.cpp: (WebKit::WebFormElement::WebFormElement): (WebKit::WebFormElement::operator=): (WebKit::WebFormElement::operator PassRefPtr<HTMLFormElement>):
  • src/WebInputElement.cpp: (WebKit::WebInputElement::WebInputElement): (WebKit::WebInputElement::operator=): (WebKit::WebInputElement::operator PassRefPtr<HTMLInputElement>):
  • src/WebLabelElement.cpp: (WebKit::WebLabelElement::correspondingControl): (WebKit::WebLabelElement::operator=): (WebKit::WebLabelElement::operator PassRefPtr<HTMLLabelElement>):
  • src/WebNode.cpp: (WebKit::WebNode::reset): (WebKit::WebNode::assign): (WebKit::WebNode::equals): (WebKit::WebNode::parentNode): (WebKit::WebNode::createMarkup): (WebKit::WebNode::addEventListener): (WebKit::WebNode::removeEventListener): (WebKit::WebNode::simulateClick): (WebKit::WebNode::WebNode): (WebKit::WebNode::operator=): (WebKit::WebNode::operator PassRefPtr<Node>):
16:27 Changeset [55865] by jianli@chromium.org
  • 2 edits in trunk/LayoutTests

Unreviewed. Added the test that time out on Tiger bot to mac-tiger/Skipped.

  • platform/mac-tiger/Skipped:
16:09 Changeset [55864] by andersca@apple.com
  • 27 edits in trunk

Remove invalidateContents, it isn't used and it never makes sense to only invalidate the contents.

Reviewed by David Hyatt.

WebCore:

  • loader/EmptyClients.h:
  • page/Chrome.cpp:
  • page/Chrome.h:
  • page/ChromeClient.h:
  • platform/HostWindow.h:

WebKit/chromium:

  • src/ChromeClientImpl.cpp:
  • src/ChromeClientImpl.h:

WebKit/gtk:

  • WebCoreSupport/ChromeClientGtk.cpp:
  • WebCoreSupport/ChromeClientGtk.h:

WebKit/haiku:

  • WebCoreSupport/ChromeClientHaiku.cpp:
  • WebCoreSupport/ChromeClientHaiku.h:

WebKit/mac:

  • WebCoreSupport/WebChromeClient.h:
  • WebCoreSupport/WebChromeClient.mm:

WebKit/qt:

  • WebCoreSupport/ChromeClientQt.cpp:
  • WebCoreSupport/ChromeClientQt.h:

WebKit/win:

  • WebCoreSupport/WebChromeClient.cpp:
  • WebCoreSupport/WebChromeClient.h:

WebKit/wx:

  • WebKitSupport/ChromeClientWx.cpp:
  • WebKitSupport/ChromeClientWx.h:
16:03 Changeset [55863] by simon.fraser@apple.com
  • 3 edits in trunk/LayoutTests

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

No review.

Need to turn on WebGL explicitly for these tests for Leopard.

  • compositing/webgl/webgl-background-color.html:
  • compositing/webgl/webgl-reflection.html:
16:03 Changeset [55862] by japhet@chromium.org
  • 9 edits
    1 delete in trunk/WebCore

2010-03-11 Nate Chapin <japhet@chromium.org>

Reviewed by Dimitri Glazkov.

[V8] Remove DOMObjectsInclude.h and update headers accordingly.

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

  • WebCore.gypi:
  • bindings/v8/DOMData.h:
  • bindings/v8/DOMDataStore.h:
  • bindings/v8/DOMObjectsInclude.h: Removed.
  • bindings/v8/V8DOMMap.cpp:
  • bindings/v8/V8DOMWindowShell.cpp:
  • bindings/v8/V8DOMWrapper.cpp:
  • bindings/v8/V8GCController.cpp:
  • bindings/v8/V8Proxy.cpp:
16:02 Changeset [55861] by ap@apple.com
  • 7 edits
    2 adds in trunk

Reviewed by Geoff Garen.

https://bugs.webkit.org/show_bug.cgi?id=35965
<rdar://problem/7742771> Crash when passing an object returned from plug-in back to the plug-in

Test: plugins/round-trip-npobject.html

  • Plugins/Hosted/NetscapePluginInstanceProxy.mm: (WebKit::NetscapePluginInstanceProxy::retainLocalObject): Corrected the check - there is now a separate ProxyRuntimeObject class for proxy pbjects. (WebKit::NetscapePluginInstanceProxy::releaseLocalObject): Ditto.
  • Plugins/Hosted/ProxyInstance.mm: (WebKit::ProxyInstance::invoke): Check if m_instanceProxy is still non-zero. The plug-in could have crashed while we were waiting for response. (WebKit::ProxyInstance::setFieldValue): Ditto.
15:57 Changeset [55860] by simon.fraser@apple.com
  • 2 edits in trunk/WebCore

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

Reviewed by Mark Rowe.

Remove duplicate entries revealed after sorting.

  • WebCore.xcodeproj/project.pbxproj:
15:35 Changeset [55859] by simon.fraser@apple.com
  • 6 edits in trunk

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

Reviewed by Mark Rowe.

Sort the project files, also fixing some bogus spaces in the WebCore project.

15:14 Changeset [55858] by dimich@chromium.org
  • 6 edits in trunk/LayoutTests

Make appcache layouttests top-frame-*.html more robust for async checking event
https://bugs.webkit.org/show_bug.cgi?id=32047

Reviewed by Alexey Proskuryakov.

Modified the tests to watch for all termination conditions, eliminating the hardcoded
assumed order of the events from separate async event sources.
Also added check for "noupdate" event.

  • http/tests/appcache/top-frame-2.html: Modified to be consistent with the other 2 tests.
  • http/tests/appcache/top-frame-3-expected.txt:
  • http/tests/appcache/top-frame-3.html: Used logic similar to top-frame-2.html to avoid assumption of the events order.
  • http/tests/appcache/top-frame-4-expected.txt:
  • http/tests/appcache/top-frame-4.html: Ditto.
15:13 Changeset [55857] by simon.fraser@apple.com
  • 2 edits in trunk/WebKitTools

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

Reviewed by Oliver Hunt.

https://bugs.webkit.org/show_bug.cgi?id=35905
REGRESSION(55699?): media/video-no-autoplay.html times out on Leopard Commit Bot

Make sure we reset the WebGL preference, so that WebGL doesn't get left
on after being enabled via layoutTestController.overridePreference(),
which in turn causes accelerated compositing to be enabled on Leopard
when we don't want it to be.

  • DumpRenderTree/mac/DumpRenderTree.mm: (resetDefaultsToConsistentValues):
14:42 Changeset [55856] by japhet@chromium.org
  • 2 edits in trunk/WebCore

2010-03-11 Nate Chapin <japhet@chromium.org>

Reviewed by Dimitri Glazkov.

Fix style issues and improve comments from
http://trac.webkit.org/changeset/55853.

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

  • platform/animation/TimingFunction.h: (WebCore::TimingFunction::TimingFunction):
14:24 Changeset [55855] by cfleizach@apple.com
  • 16 edits in trunk/WebCore

accessibilityIsIgnoredBase() needs to respect when platform says include
https://bugs.webkit.org/show_bug.cgi?id=36025

Reviewed by Beth Dakin.

Changed accessibilityIsIgnoredBase() to return a policy instead of a yes/no
answer. This allows the platform to make a yes decision on an element.

  • accessibility/AccessibilityList.cpp:

(WebCore::AccessibilityList::accessibilityIsIgnored):

  • accessibility/AccessibilityListBox.cpp:

(WebCore::AccessibilityListBox::accessibilityIsIgnored):

  • accessibility/AccessibilityObject.h:

(WebCore::):
(WebCore::AccessibilityObject::accessibilityIgnoreAttachment):
(WebCore::AccessibilityObject::accessibilityPlatformIncludesObject):

  • accessibility/AccessibilityRenderObject.cpp:

(WebCore::AccessibilityRenderObject::accessibilityIsIgnoredBase):
(WebCore::AccessibilityRenderObject::accessibilityIsIgnored):

  • accessibility/AccessibilityRenderObject.h:
  • accessibility/AccessibilitySlider.cpp:

(WebCore::AccessibilitySlider::accessibilityIsIgnored):

  • accessibility/AccessibilityTable.cpp:

(WebCore::AccessibilityTable::accessibilityIsIgnored):

  • accessibility/AccessibilityTableCell.cpp:

(WebCore::AccessibilityTableCell::accessibilityIsIgnored):

  • accessibility/AccessibilityTableRow.cpp:

(WebCore::AccessibilityTableRow::accessibilityIsIgnored):

  • accessibility/chromium/AccessibilityObjectChromium.cpp:

(WebCore::AccessibilityObject::accessibilityPlatformIncludesObject):

  • accessibility/gtk/AccessibilityObjectAtk.cpp:

(WebCore::AccessibilityObject::accessibilityPlatformIncludesObject):

  • accessibility/mac/AccessibilityObjectMac.mm:

(WebCore::AccessibilityObject::accessibilityPlatformIncludesObject):

  • accessibility/qt/AccessibilityObjectQt.cpp:

(WebCore::AccessibilityObject::accessibilityPlatformIncludesObject):

  • accessibility/win/AccessibilityObjectWin.cpp:

(WebCore::AccessibilityObject::accessibilityPlatformIncludesObject):

  • accessibility/wx/AccessibilityObjectWx.cpp:

(WebCore::AccessibilityObject::accessibilityPlatformIncludesObject):

13:39 Changeset [55854] by kbr@google.com
  • 3 edits in trunk/WebCore

2010-03-11 Kenneth Russell <kbr@google.com>

Reviewed by Dimitri Glazkov.

Fix build breakage with ENABLE_3D_CANVAS=0
https://bugs.webkit.org/show_bug.cgi?id=35995

No new tests; verified in Chromium that WebGL is disabled in
ENABLE_3D_CANVAS=0 builds.

  • bindings/v8/custom/V8DocumentCustom.cpp:
  • bindings/v8/custom/V8HTMLCanvasElementCustom.cpp:
12:24 Changeset [55853] by japhet@chromium.org
  • 2 edits in trunk/WebCore

2010-03-11 Nate Chapin <japhet@chromium.org>

Unreviewed, Chromium mac build fix.

Add an explicit copy constructor to TimingFunction.
http://trac.webkit.org/changeset/55835 trigged an inlining
bug in gcc that the copy constructor resolves.

  • platform/animation/TimingFunction.h: (WebCore::TimingFunction::TimingFunction):
12:18 Changeset [55852] by mrobinson@webkit.org
  • 3 edits
    1 add
    4 deletes in trunk/LayoutTests

2010-03-11 Martin Robinson <mrobinson@webkit.org>

Reviewed by Adam Barth

Convert editing/selection/5136696.html to dumpAsText
https://bugs.webkit.org/show_bug.cgi?id=34306

  • editing/selection/5136696-expected.txt: Added.
  • editing/selection/5136696.html: Use dumpAsText now.
  • platform/gtk/Skipped: Unskip this test in GTK+ now that it has expected result.
  • platform/mac/editing/selection/5136696-expected.checksum: Removed.
  • platform/mac/editing/selection/5136696-expected.png: Removed.
  • platform/mac/editing/selection/5136696-expected.txt: Removed.
  • platform/qt/editing/selection/5136696-expected.txt: Removed.
  • platform/win/editing/selection/5136696-expected.txt: Removed.
12:01 Changeset [55851] by aroben@apple.com
  • 2 edits in trunk

Teach git about ObjC files

Fixes <http://webkit.org/b/36015>.

Reviewed by Tim Hatcher.

  • .gitattributes: Set the diff attribute for .m and .mm files, and .h

files in Mac-specific directories. This can be used to generate
more-readable diffs of ObjC files.

11:56 Changeset [55850] by dpranke@chromium.org
  • 4 edits in trunk/WebKitTools

2010-03-10 Dirk Pranke <dpranke@chromium.org>

Reviewed by Dimitri Glazkov.

rebaseline_chromium_webkit_tests doesn't handle other plaforms
correctly (e.g., if you run on the Mac platform and try to
rebaseline the WIN results, the result gets written into
platform/mac instead of platform/chromium-win). Also, this script
doesn't work on non-Chromium ports, so we need to fix that at some
point.

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

  • Scripts/webkitpy/layout_tests/rebaseline_chromium_webkit_tests.py:
11:28 Changeset [55849] by cfleizach@apple.com
  • 2 edits in trunk/WebCore

Fixing GTK. No review.

support lang attribute on <option> elements
https://bugs.webkit.org/show_bug.cgi?id=36021

We don't need to cast to AccessibilityRenderObject to get language().

  • accessibility/gtk/AccessibilityObjectWrapperAtk.cpp:

(webkit_accessible_document_get_locale):

11:19 Changeset [55848] by cfleizach@apple.com
  • 8 edits
    2 adds in trunk

support lang attribute on <option> elements
https://bugs.webkit.org/show_bug.cgi?id=36021

Reviewed by Darin Adler.

WebCore:

Allow non AccessibilityRenderObject classes to access the lang
attribute by moving the useful code into AccessibilityObject.

Test: platform/mac/accessibility/option-with-lang.html

  • accessibility/AccessibilityListBoxOption.cpp:

(WebCore::AccessibilityListBoxOption::language):

  • accessibility/AccessibilityListBoxOption.h:
  • accessibility/AccessibilityObject.cpp:

(WebCore::AccessibilityObject::language):

  • accessibility/AccessibilityObject.h:
  • accessibility/AccessibilityRenderObject.cpp:

(WebCore::AccessibilityRenderObject::language):

  • accessibility/AccessibilityRenderObject.h:

LayoutTests:

  • platform/mac/accessibility/option-with-lang-expected.txt: Added.
  • platform/mac/accessibility/option-with-lang.html: Added.
10:48 Changeset [55847] by tonikitoo@webkit.org
  • 4 edits in trunk

[Qt] Access key modifier should be Ctrl+Alt for Darwin derived OS and Alt for the others
https://bugs.webkit.org/show_bug.cgi?id=35993

Patch by Diego Gonzalez <diego.gonzalez@openbossa.org> on 2010-03-11
Reviewed by Simon Hausmann.

WebCore:

  • page/qt/EventHandlerQt.cpp:

(WebCore::EventHandler::accessKeyModifiers):

LayoutTests:

  • platform/qt/Skipped:
10:38 Changeset [55846] by brettw@chromium.org
  • 3 edits in trunk

010-03-08 Brett Wilson <brettw@chromium.org>

Reviewed by Adam Barth.

Remove the deprecated argument to url_util::IsStandard in preparation
for deleting that version of the function. Pull the latest googleurl
with the new version for the Chromium builder.

  • platform/KURLGoogle.cpp: (WebCore::KURL::isHierarchical):
09:02 BuildingQtOnSymbian edited by siddharth.mathur@nokia.com
(diff)
09:00 BuildingQtOnSymbian edited by siddharth.mathur@nokia.com
running on HW issues (diff)
08:03 Changeset [55845] by hausmann@webkit.org
  • 2 edits in trunk/WebCore

[Qt] Make it possible to do builds with separate debug info in packages

Reviewed by Tor Arne Vestbø.

  • WebCore.pro:
08:03 Changeset [55844] by hausmann@webkit.org
  • 2 edits in trunk/WebCore

[Qt] Avoid double-buffering with Qt image decoders

Reviewed by Tor Arne Vestbø.

Pass QIODevice::Unbuffered when opening the QBuffer that
wraps the image data, to hint to Qt that no extra buffering
is needed.

  • platform/graphics/qt/ImageDecoderQt.cpp:

(WebCore::ImageDecoderQt::setData):

07:25 Changeset [55843] by benm@google.com
  • 4 edits in trunk/WebCore

[Android] The platform touch events on Android are missing support
for key modifiers.
https://bugs.webkit.org/show_bug.cgi?id=35521

Reviewed by Jeremy Orlow.

Add support in PlatformToucHEventAndroid for the platform supplying
key modifiers with touch events.

Fixes Android so it now passes basic-single-touch-events.html.

  • page/EventHandler.cpp:

(WebCore::EventHandler::handleTouchEvent): Fix two compiler warnings.

  • platform/PlatformTouchEvent.h:

(WebCore::PlatformTouchEvent::PlatformTouchEvent): Add support for

key modifiers on Android.

  • platform/android/PlatformTouchEventAndroid.cpp:

(WebCore::PlatformTouchEvent::PlatformTouchEvent): ditto.

06:40 Changeset [55842] by cfleizach@apple.com
  • 2 edits in trunk/LayoutTests

Unreviewed.

VoiceOver can navigate to hidden content in widget (WAI-ARIA)
https://bugs.webkit.org/show_bug.cgi?id=35986

Wrong files were placed in skipped lists.

  • platform/gtk/Skipped:
03:38 Changeset [55841] by steveblock@google.com
  • 5 edits
    3 adds in trunk

2010-03-10 Steve Block <steveblock@google.com>

Reviewed by Jeremy Orlow.

Implements Geolocation maximumAge property
https://bugs.webkit.org/show_bug.cgi?id=30676

Test: fast/dom/Geolocation/maximum-age.html

  • WebCore.xcodeproj/project.pbxproj: Modified. Adds GeolocationPositionCache.h to Private headers
  • page/Geolocation.cpp: Modified. (WebCore::Geolocation::GeoNotifier::setUseCachedPosition): Added. (WebCore::Geolocation::GeoNotifier::runSuccessCallback): Added. (WebCore::Geolocation::GeoNotifier::timerFired): Modified. Added logic to handle using a cached position (WebCore::Geolocation::Watchers::contains): Added. Required to determine if a notifier is a watch request (WebCore::Geolocation::startRequest): Modified. Added logic to check for a cached position (WebCore::Geolocation::requestUsesCachedPosition): Added. Callback to Geolocation object when notifier uses a cached position (WebCore::Geolocation::makeCachedPositionCallbacks): Added. (WebCore::Geolocation::haveSuitableCachedPosition): Added. (WebCore::Geolocation::setIsAllowed): Modified. (WebCore::Geolocation::positionChanged): Modified. Make callbacks using cached position where appropriate (WebCore::Geolocation::geolocationServiceErrorOccurred): Modified. Make callbacks using cached position where appropriate
  • page/Geolocation.h: Modified.

2010-03-10 Steve Block <steveblock@google.com>

Reviewed by Jeremy Orlow.

Implements Geolocation maximumAge property
https://bugs.webkit.org/show_bug.cgi?id=30676

  • fast/dom/Geolocation/resources/maximum-age.js: Added. Checks that a cached position is used only when it should be.
  • fast/dom/Geolocation/maximum-age.html: Added. HTML wraper for above test.
  • fast/dom/Geolocation/maximum-age-expected.txt: Added. Expected result for above test.
02:57 Changeset [55840] by philn@webkit.org
  • 2 edits in trunk/LayoutTests

2010-03-11 Philippe Normand <pnormand@igalia.com>

Unreviewed.

Unskip the fixed test that was (accidently?) re-skipped in r54205.

  • platform/gtk/Skipped:
02:25 Changeset [55839] by ossy@webkit.org
  • 2 edits in trunk/LayoutTests

[Qt] storage/open-database-creation-callback-isolated-world.html was introduced in r55823,
but layoutTestController.evaluateScriptInIsolatedWorld() is unimplemented now.

  • platform/qt/Skipped: storage/open-database-creation-callback-isolated-world.html skipped.
02:16 Changeset [55838] by ossy@webkit.org
  • 2 edits in trunk/WebCore

Unreviewed buildfix after r55823. (To fix Qt --minimal build.)
I have déjà vu, I'm sure that I did it before. (r55598)

  • bindings/js/JSDOMWindowCustom.cpp: Missing #if ENABLE(DATABASE) guard added.
01:43 Changeset [55837] by ossy@webkit.org
  • 3 edits in trunk/WebCore

[Qt] Unreviewed buildfix after r55833.

Rename all instances of data() to characters()

  • bridge/qt/qt_class.cpp:

(JSC::Bindings::QtClass::fieldNamed):

  • bridge/qt/qt_runtime.cpp:

(JSC::Bindings::convertValueToQVariant):

01:29 Changeset [55836] by zoltan@webkit.org
  • 2 edits in trunk/WebCore

2010-03-11 Zoltan Horvath <zoltan@webkit.org>

Reviewed by Simon Hausmann.

Allow custom memory allocation control for RenderLayerBacking class
https://bugs.webkit.org/show_bug.cgi?id=35857

Inherits the following class from Noncopyable because it is
instantiated by 'new' and no need to be copyable:

class name - instantiated at: WebCore/'location'
RenderLayerBacking - rendering/RenderLayer.cpp:3047

  • rendering/RenderLayerBacking.h:
00:41 Changeset [55835] by zoltan@webkit.org
  • 2 edits in trunk/WebCore

2010-03-11 Zoltan Horvath <zoltan@webkit.org>

Reviewed by Simon Hausmann.

Allow custom memory allocation control for TimingFunction struct
https://bugs.webkit.org/show_bug.cgi?id=35855

Inherits the following struct from FastAllocBase because it is
instantiated by 'new':

class name - instantiated at: WebCore/'location'
TimingFuction - platform/graphics/GraphicsLayer.h:89

  • platform/animation/TimingFunction.h:

03/10/10:

23:53 Changeset [55834] by loki@webkit.org
  • 3 edits in trunk/JavaScriptCore

Buildfix for Thumb-2 after r55684. Add branch8 and branchTest8 functions.
https://bugs.webkit.org/show_bug.cgi?id=35892

Reviewed by Gavin Barraclough.

  • assembler/ARMv7Assembler.h:

(JSC::ARMv7Assembler::):
(JSC::ARMv7Assembler::ldrb):

  • assembler/MacroAssemblerARMv7.h:

(JSC::MacroAssemblerARMv7::load8):
(JSC::MacroAssemblerARMv7::branch8):
(JSC::MacroAssemblerARMv7::branchTest8):
(JSC::MacroAssemblerARMv7::setTest8):

23:19 Changeset [55833] by barraclough@apple.com
  • 15 edits in trunk

Rubber stamped by Oliver Hunt.

Rename JSC::UStringImpl::data() to characters(), to match WebCore::StringImpl.

JavaScriptCore:

  • API/JSClassRef.cpp:

(OpaqueJSClassContextData::OpaqueJSClassContextData):

  • bytecompiler/BytecodeGenerator.cpp:

(JSC::keyForCharacterSwitch):

  • bytecompiler/NodesCodegen.cpp:

(JSC::processClauseList):

  • interpreter/Interpreter.cpp:

(JSC::Interpreter::privateExecute):

  • jit/JITStubs.cpp:

(JSC::DEFINE_STUB_FUNCTION):

  • runtime/ArrayPrototype.cpp:

(JSC::arrayProtoFuncToString):

  • runtime/Identifier.cpp:

(JSC::Identifier::equal):
(JSC::Identifier::addSlowCase):

  • runtime/JSString.cpp:

(JSC::JSString::resolveRope):

  • runtime/UString.cpp:

(JSC::UString::toStrictUInt32):
(JSC::equal):

  • runtime/UString.h:

(JSC::UString::data):

  • runtime/UStringImpl.h:

(JSC::UStringImpl::characters):
(JSC::UStringImpl::hash):
(JSC::UStringImpl::setHash):

WebCore:

  • bridge/jni/jsc/JavaStringJSC.h:

(JSC::Bindings::JavaStringImpl::uchars):

  • platform/text/AtomicString.cpp:

(WebCore::AtomicString::add):
(WebCore::AtomicString::find):

23:07 Changeset [55832] by barraclough@apple.com
  • 2 edits in trunk/WebCore

Fix for r55825, threadsafeCopy no longer needs to special-case for
empty strings (in fact, doing so results in leaks).

Reviewed by Mark Rowe.

  • platform/text/StringImpl.cpp:

(WebCore::StringImpl::threadsafeCopy):

Note: See TracTimeline for information about the timeline view.