Context Navigation

Q_WS_S60 is not defined for Symbian^{4 devices as Q_WS_S60 used to guard

Avkon UI framework dependencies. Use Q_OS_SYMBIAN everywhere to mark

Symbian dependencies.}

WebCoreSupport/QtFallbackWebPopup.cpp: (WebCore::QtFallbackWebPopup::populate):

2010-08-17 Laszlo Gombos <Laszlo Gombos>

Reviewed by Ariya Hidayat.

[Qt] [Symbian] Consistently use Q_OS_SYMBIAN to guard all Symbian platform dependencies
https://bugs.webkit.org/show_bug.cgi?id=44124

Q_WS_S60 is not defined for Symbian^{4 devices as Q_WS_S60 used to guard

Avkon UI framework dependencies. Use Q_OS_SYMBIAN everywhere to mark

Symbian dependencies.}

DumpRenderTree/qt/DumpRenderTreeQt.cpp: (WebCore::DumpRenderTree::open):
QtTestBrowser/launcherwindow.cpp: (LauncherWindow::init): (LauncherWindow::toggleFullScreenMode): (LauncherWindow::showFPS): (LauncherWindow::updateFPS):
QtTestBrowser/launcherwindow.h: (WindowOptions::WindowOptions):
QtTestBrowser/mainwindow.cpp: (MainWindow::buildUI):

6:20 PM Changeset in webkit [65580] by ariya@webkit.org

2 edits in trunk/WebCore

[Qt] Remove unused ContextShadowParameter class in GraphicsContextQt
https://bugs.webkit.org/show_bug.cgi?id=44142

Patch by Ariya Hidayat <ariya@sencha.com> on 2010-08-17
Reviewed by Antonio Gomes.

The class is not needed anymore after r65425.
We know have dedicated ContextShadow in its own file.

platform/graphics/qt/GraphicsContextQt.cpp:

6:01 PM Changeset in webkit [65579] by Darin Adler

19 edits
12 adds in trunk

Add NetworkingContext to avoid layer violations
https://bugs.webkit.org/show_bug.cgi?id=42292

Patch by Jesus Sanchez-Palencia <jesus.palencia@openbossa.org> on 2010-08-17
Reviewed by Darin Adler.

Preparation: Just add the files to the build system.

GNUmakefile.am: Added new files.

WebCore:

GNUmakefile.am: Added new files.
WebCore.gypi: Ditto.
WebCore.pro: Ditto.
WebCore.vcproj/WebCore.vcproj: Ditto.
WebCore.xcodeproj/project.pbxproj: Ditto.

loader/FrameNetworkingContext.h: Added.
platform/network/NetworkingContext.h: Added.

Placeholders with tentative code that might be changed when landing
the rest of it.

WebKit:

WebKit.xcodeproj/project.pbxproj: Added new files.

WebKit/chromium:

WebKit.gyp: Added new files.
src/FrameNetworkingContextImpl.h: Added.

Placeholder with tentative code that might be changed when landing
the rest of it.

WebKit/gtk:

WebCoreSupport/FrameNetworkingContextGtk.h: Added.

Placeholder with tentative code that might be changed when landing
the rest of it.

WebKit/mac:

WebCoreSupport/WebFrameNetworkingContext.h: Added.

Placeholder with tentative code that might be changed when landing
the rest of it.

WebCoreSupport/WebFrameNetworkingContext.mm: Added.

Empty placeholder for now.

WebKit/qt:

WebCoreSupport/FrameNetworkingContextQt.cpp: Added.

Empty placeholder for now.

WebCoreSupport/FrameNetworkingContextQt.h: Added.

Placeholder with tentative code that might be changed when landing
the rest of it.

WebKit/win:

WebCoreSupport/WebFrameNetworkingContext.cpp: Added.

Empty placeholder for now.

WebCoreSupport/WebFrameNetworkingContext.h: Added.

Placeholder with tentative code that might be changed when landing
the rest of it.

WebKit.vcproj/WebKit.vcproj: Added new files.

WebKit2:

WebKit2.xcodeproj/project.pbxproj: Added new files.
WebProcess/WebCoreSupport/mac/WebFrameNetworkingContext.h: Added.

Placeholder with tentative code that might be changed when landing
the rest of it.

WebProcess/WebCoreSupport/mac/WebFrameNetworkingContext.mm: Added.

Empty placeholder for now.

5:59 PM Changeset in webkit [65578] by dpranke@chromium.org

6 edits in trunk/WebKitTools

2010-08-17 Dirk Pranke <dpranke@chromium.org>

Reviewed by David Levin.

new-run-webkit-tests: remove --show-sources option

--show-sources is pretty much obsolete with --trace everything, so
I'm removing it.

Also rename a couple of methods in the TestTextDiff class to make their
intended visibility (private) more obvious.

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

Scripts/webkitpy/layout_tests/layout_package/printing.py:
Scripts/webkitpy/layout_tests/run_webkit_tests.py:
Scripts/webkitpy/layout_tests/test_types/image_diff.py:
Scripts/webkitpy/layout_tests/test_types/test_type_base.py:
Scripts/webkitpy/layout_tests/test_types/text_diff.py:

5:55 PM Changeset in webkit [65577] by dpranke@chromium.org

3 edits
1 delete in trunk/WebKitTools

2010-08-17 Dirk Pranke <dpranke@chromium.org>

Reviewed by David Levin.

remove --fuzzy-image-diff in new-run-webkit-tests (it doesn't work)

This code bit-rotted at some point more than a year ago, and nobody seems
to miss it. old-run-webkit-tests has a --tolerance flag that new-run-webkit-tests
should support instead, making this flag unnecessary anyway.

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

Scripts/webkitpy/layout_tests/layout_package/test_failures.py:
Scripts/webkitpy/layout_tests/run_webkit_tests.py:
Scripts/webkitpy/layout_tests/test_types/fuzzy_image_diff.py: Removed.

5:41 PM Changeset in webkit [65576] by Beth Dakin

1 edit
3 adds in trunk/WebKitSite

Adding a MathML Demo and screenshot.

Rubber-stamped by Adele.

blog-files/riemann-screenshot.png: Added.
demos/mathml: Added.
demos/mathml/MathMLDemo.xhtml: Added.

4:57 PM Changeset in webkit [65575] by commit-queue@webkit.org

2 edits in trunk/LayoutTests

2010-08-17 Mihai Parparita <mihaip@chromium.org>

Reviewed by Dimitri Glazkov.

transitions/change-values-during-transition.html fails intermittently
https://bugs.webkit.org/show_bug.cgi?id=28461

Make test less flaky during high CPU usage by changing the expectation
from a hardcoded value to an interpolated one based on when the timeout
actually fires. Made the test much more reliable when running with
--iterations=100 while maxing out the CPU on my machine with a
background process.

transitions/change-values-during-transition.html:

4:56 PM Changeset in webkit [65574] by victorw@chromium.org

4 edits in trunk/WebKitTools

2010-08-17 Victor Wang <victorw@chromium.org>

Reviewed by Ojan Vafai.

Update json results generator to have incremental json including
results for tests that pass in current run but failed before.

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

Scripts/webkitpy/layout_tests/layout_package/json_layout_results_generator.py:
Scripts/webkitpy/layout_tests/layout_package/json_results_generator.py:
Scripts/webkitpy/layout_tests/run_webkit_tests.py:

4:40 PM Changeset in webkit [65573] by commit-queue@webkit.org

10 edits
1 move
3 adds in trunk

2010-08-17 Mihai Parparita <mihaip@chromium.org>

Reviewed by Darin Fisher.

Back button after Google search result click broken
https://bugs.webkit.org/show_bug.cgi?id=44086

Added a reduced test case of Google search result click behavior,
renamed gesture-before-onload-target.html to redirect-target.html
since it's useful for this test too.

fast/history/gesture-before-onload-form-submit-expected.txt:
fast/history/gesture-before-onload-form-submit.html:
fast/history/gesture-before-onload-location-href-expected.txt:
fast/history/gesture-before-onload-location-href.html:
fast/history/redirect-via-iframe-expected.txt: Added.
fast/history/redirect-via-iframe.html: Added.
fast/history/resources/iframe-redirect.html: Added.
fast/history/resources/redirect-target.html: Renamed from LayoutTests/fast/history/resources/gesture-before-onload-target.html.

2010-08-17 Mihai Parparita <mihaip@chromium.org>

Reviewed by Darin Fisher.

Back button after Google search result click broken
https://bugs.webkit.org/show_bug.cgi?id=44086

Test: fast/history/redirect-via-iframe.html

isLoadingInAPISense doesn't quite map to onload (it also returns true
if frames are loading, even after the parent frame has fired its onload
handler). Instead, add a flag for the onload event being fired, and
use that.

loader/DocumentLoader.cpp: (WebCore::DocumentLoader::DocumentLoader):
loader/DocumentLoader.h: (WebCore::DocumentLoader::handledOnloadEvents): (WebCore::DocumentLoader::wasOnloadHandled):
loader/FrameLoader.cpp: (WebCore::FrameLoader::handledOnloadEvents):
loader/RedirectScheduler.cpp: (WebCore::RedirectScheduler::mustLockBackForwardList):

4:11 PM Changeset in webkit [65572] by dpranke@chromium.org

10 edits in trunk/WebKitTools

2010-08-17 Dirk Pranke <dpranke@chromium.org>

Reviewed by Eric Seidel.

fix test-webkitpy, add easy way to find a checkout root

test-webkitpy currently doesn't work right if run from someplace other
than the checkout root, and it spews a bunch of debug logging because
the deduplicate_tests tests contaminates the test environment.

This patch cleans up the deduplicate_tests unit tests, and creates
two new methods in scm.py: find_checkout_root() and default_scm(),
both of which use a single algorithm for guessing what checkout root
to use if you aren't explicitly told one from a path.

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

Scripts/deduplicate-tests:
Scripts/webkitpy/common/checkout/scm.py:
Scripts/webkitpy/common/checkout/scm_unittest.py:
Scripts/webkitpy/layout_tests/deduplicate_tests.py:
Scripts/webkitpy/layout_tests/deduplicate_tests_unittest.py:
Scripts/webkitpy/layout_tests/port/test.py:
Scripts/webkitpy/layout_tests/rebaseline_chromium_webkit_tests.py:
Scripts/webkitpy/layout_tests/rebaseline_chromium_webkit_tests_unittest.py:
Scripts/webkitpy/tool/main.py:

4:05 PM Changeset in webkit [65571] by barraclough@apple.com

2 edits in trunk/JavaScriptCore

Bug 44099 - REGRESSION(r65468): Crashes in StringImpl::find

Reviewed by Sam Weinig.

Bug 44080 introuduced a couple of cases in which array bounds could be overrun.
One of these was fixed in r65493, this patch fixes the other and address the
concerns voiced in comment #6 by restructuring the loops to remove the code
dupliction without introducing an additional if check.

wtf/text/StringImpl.cpp:

(WTF::StringImpl::find):
(WTF::StringImpl::findIgnoringCase):
(WTF::StringImpl::reverseFind):
(WTF::StringImpl::reverseFindIgnoringCase):

4:05 PM Changeset in webkit [65570] by steveblock@google.com

4 edits
3 adds in trunk

2010-08-17 Steve Block <steveblock@google.com>

Reviewed by Jeremy Orlow.

Geolocation clearWatch() needs to protect against invalid IDs
https://bugs.webkit.org/show_bug.cgi?id=44096

fast/dom/Geolocation/clear-watch-invalid-id-crash-expected.txt: Added.
fast/dom/Geolocation/clear-watch-invalid-id-crash.html: Added.
fast/dom/Geolocation/script-tests/clear-watch-invalid-id-crash.js: Added.
fast/dom/Geolocation/script-tests/notimer-after-unload.js: (document.body.onload):

2010-08-17 Steve Block <steveblock@google.com>

Reviewed by Jeremy Orlow.

Geolocation clearWatch() needs to protect against invalid IDs
https://bugs.webkit.org/show_bug.cgi?id=44096

If the ID passed to clearWatch() is invalid, we early-out.

Test: fast/dom/Geolocation/clear-watch-invalid-id-crash.html

page/Geolocation.cpp: (WebCore::Geolocation::Watchers::set): (WebCore::Geolocation::Watchers::remove): (WebCore::Geolocation::watchPosition): (WebCore::Geolocation::clearWatch):

4:00 PM Changeset in webkit [65569] by senorblanco@chromium.org

4 edits in trunk/WebCore

2010-08-17 Stephen White <senorblanco@chromium.org>

Reviewed by David Levin.

Allow non-Skia ports to compile with ENABLE(ACCELERATED_2D_CANVAS).
https://bugs.webkit.org/show_bug.cgi?id=44130

No tests; if it compiles, it's good.

html/canvas/CanvasRenderingContext2D.cpp: (WebCore::CanvasRenderingContext2D::CanvasRenderingContext2D): Fix a warning (treated as error).
platform/graphics/GraphicsContext.cpp: (WebCore::GraphicsContext::setGraphicsContext3D): (WebCore::GraphicsContext::syncSoftwareCanvas):
platform/graphics/GraphicsContext.h: Make setGraphicsContext3D() and syncSoftwareCanvas() stubbed out, but still present in non-Skia ports. This allows Chrome Mac to compile with ENABLE(ACCELERATED_2D_CANVAS) (although it isn't hooked up yet, it should at least compile).

3:53 PM Changeset in webkit [65568] by crogers@google.com

3 edits in branches/audio/WebCore

V8-specific fixes for JavaScriptAudioNode

3:46 PM Changeset in webkit [65567] by victorw@chromium.org

4 edits in trunk/WebKitTools

2010-08-17 Victor Wang <victorw@chromium.org>

Reviewed by Ojan Vafai.

Add support to the test results server for downloading json that
contains test list only.

This is for json results generator to generate incremental json
results so that it includes results not only for tests failed in
current run, but also tests failed before.

Also set the results type to "N" (no data) instead of "P" (pass)
if test results cannot be found in incremental json file.

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

TestResultServer/handlers/testfilehandler.py:
TestResultServer/model/jsonresults.py:
TestResultServer/model/jsonresults_unittest.py:

3:44 PM Changeset in webkit [65566] by beidson@apple.com

8 edits
2 adds in trunk

WebCore: Navigating back/forward during a modal dialog causes a crash when the modal dialog is dismissed.
<rdar://problem/8313579> and https://bugs.webkit.org/show_bug.cgi?id=44131

Reviewed by Sam Weinig.

No new tests. (Especially since DRT doesn't spin inner run loops at all, it would be a significant
effort to give it the ability to perform the back/forward navigation in the background to reproduce
this bug)

Manual tests in place of layout tests:

manual-tests/back-forward-during-alert-1.html: Added.
manual-tests/back-forward-during-alert-2.html: Added.

page/Page.cpp:

(WebCore::Page::goToItem):

WebKit: Remove developmentRegion from the project file as it shouldn't have been checked in.

Reviewed by Sam Weinig.

WebKit.xcodeproj/project.pbxproj:

WebKit/mac: Navigating back/forward during a modal dialog causes a crash when the modal dialog is dismissed.
<rdar://problem/8313579> and https://bugs.webkit.org/show_bug.cgi?id=44131

Reviewed by Sam Weinig.

WebView/WebView.mm:

(-[WebView canGoBack]): Return false if loads are deferred.
(-[WebView canGoForward]): Ditto.

WebKit/win: Navigating back/forward during a modal dialog causes a crash when the modal dialog is dismissed.
<rdar://problem/8313579> and https://bugs.webkit.org/show_bug.cgi?id=44131

Reviewed by Sam Weinig.

WebView.cpp:

(WebView::canGoBack): Return false if loads are deferred.
(WebView::canGoForward): Ditto.

3:36 PM Changeset in webkit [65565] by Joseph Pecoraro

1 edit
3 adds in trunk/LayoutTests

2010-08-17 Joseph Pecoraro <Joseph Pecoraro>

Reviewed by Pavel Feldman.

Web Inspector: Add XMLHttpRequest Logging Tests
https://bugs.webkit.org/show_bug.cgi?id=44092

http/tests/inspector/console-xhr-logging-expected.txt: Added.
http/tests/inspector/console-xhr-logging.html: Added.
http/tests/inspector/resources/xhr-exists.html: Added.

3:33 PM Changeset in webkit [65564] by ojan@chromium.org

3 edits
2 adds in trunk

2010-08-17 Ojan Vafai <ojan@chromium.org>

Reviewed by Tony Chang.

selectionStart clobbered if textarea isn't focused
https://bugs.webkit.org/show_bug.cgi?id=43814

fast/forms/text-control-select-blurred-expected.txt: Added.
fast/forms/text-control-select-blurred.html: Added.

2010-08-17 Ojan Vafai <ojan@chromium.org>

Reviewed by Tony Chang.

selectionStart clobbered if textarea isn't focused
https://bugs.webkit.org/show_bug.cgi?id=43814

If a text control is blurred, setting selectionStart/end should use
the cached selection value, which is currently stored on the form
control element.

Test: fast/forms/text-control-select-blurred.html

rendering/RenderTextControl.cpp: (WebCore::RenderTextControl::setSelectionStart): (WebCore::RenderTextControl::setSelectionEnd):

3:29 PM Changeset in webkit [65563] by dimich@chromium.org

5 edits
2 adds in trunk

MessagePort.close() crashes if the owning context was destroyed.
https://bugs.webkit.org/show_bug.cgi?id=43140

Reviewed by David Levin.

WebCore:

Test: fast/events/message-port-context-destroyed.html

dom/MessagePort.cpp:

(WebCore::MessagePort::postMessage):
(WebCore::MessagePort::start):
(WebCore::MessagePort::close):
(WebCore::MessagePort::contextDestroyed):
Use isEntangled() method to gate various operations on MessagePort. This method also takes into account m_closed bit.

dom/ScriptExecutionContext.cpp:

(WebCore::ScriptExecutionContext::stopActiveDOMObjects):
(WebCore::ScriptExecutionContext::closeMessagePorts):

dom/ScriptExecutionContext.h:

Add closeMessagePorts() method that closes message ports at the same moments when other ActiveDOMObjects are stopped.

LayoutTests:

fast/events/message-port-context-destroyed-expected.txt: Added.
fast/events/message-port-context-destroyed.html: Added.

3:22 PM Changeset in webkit [65562] by hclam@chromium.org

6 edits in trunk/LayoutTests

Unreviewed. Build fix.

Submitted for Victoria Kirst <vrk@google.com>.

Rebaseline straggler Mac UI video regressions.
Fixing test expectations to match output from r65215 bug fix.

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

platform/chromium-mac/fast/layers/video-layer-expected.checksum:
platform/chromium-mac/fast/layers/video-layer-expected.png:
platform/chromium-mac/media/video-empty-source-expected.checksum:
platform/chromium-mac/media/video-empty-source-expected.png:
platform/chromium/test_expectations.txt:

3:13 PM Changeset in webkit [65561] by johnnyg@google.com

2 edits in trunk/LayoutTests

2010-08-17 John Gregg <johnnyg@google.com>

Unreviewed, test expectations.

Skip the filltable-*.html tests on mac and windows. They have no expectations, so expecting
them to FAIL is insufficient to make the bots happy.

platform/chromium/test_expectations.txt:

3:10 PM Changeset in webkit [65560] by Adam Roben

2 edits in trunk/WebKitTools

Use the right path style

Scripts/old-run-webkit-tests:

3:08 PM Changeset in webkit [65559] by Adam Roben

2 edits in trunk/WebKitTools

Fix typo

Scripts/old-run-webkit-tests:

3:03 PM Changeset in webkit [65558] by crogers@google.com

505 edits
105 copies
62 deletes in branches/audio/WebKit

merge trunk 61708:65443 into audio branch: WebKit/WebKit

3:02 PM Changeset in webkit [65557] by Adam Roben

2 edits in trunk/WebKitTools

Don't hang when running run-webkit-tests as a non-Administrator on Vista/7

Scripts/old-run-webkit-tests:

(setUpWindowsCrashLogSaving): Use regtool to set NTSD as the
post-mortem debugger, rather than using NTSD itself. The latter waits
for user input when it fails to set the registry values; the former
does not.

2:56 PM Changeset in webkit [65556] by crogers@google.com

10 edits in branches/audio

merge trunk 61708:65443 into audio branch: WebKit remaining files

2:55 PM Changeset in webkit [65555] by crogers@google.com

233 edits
175 copies
17 deletes in branches/audio

merge trunk 61708:65443 into audio branch: WebKit directories 3

2:52 PM Changeset in webkit [65554] by crogers@google.com

157 edits
233 copies
16 deletes in branches/audio/WebKit2

merge trunk 61708:65443 into audio branch: WebKit directories 2

2:51 PM Changeset in webkit [65553] by crogers@google.com

4 edits
149 copies in branches/audio

merge trunk 61708:65443 into audio branch: WebKit directories 1

2:47 PM Changeset in webkit [65552] by crogers@google.com

4 edits
5 copies in branches/audio/JavaScriptGlue

merge trunk 61708:65443 into audio branch: JavaScriptGlue

2:46 PM Changeset in webkit [65551] by crogers@google.com

268 edits
38 copies
6 deletes in branches/audio/JavaScriptCore

merge trunk 61708:65443 into audio branch: JavaScriptCore

2:41 PM Changeset in webkit [65550] by crogers@google.com

30 edits
15 copies
20 deletes in branches/audio/WebCore

merge trunk 61708:65443 into audio branch: WebCore remaining files

2:39 PM Changeset in webkit [65549] by crogers@google.com

533 edits
27 copies in branches/audio/WebCore

merge trunk 61708:65443 into audio branch: WebCore directories 7

2:37 PM Changeset in webkit [65548] by crogers@google.com

248 edits
86 copies
38 deletes in branches/audio/WebCore

merge trunk 61708:65443 into audio branch: WebCore directories 6

2:36 PM Changeset in webkit [65547] by vestbo@webkit.org

2 edits in trunk/LayoutTests

[Qt] Skip media/video-poster-delayed.html

Reviewed by NOBODY.

The QtMultimedia backend has issues with video size hints, skip the test
until this is sorted out.

platform/qt/Skipped:

2:35 PM Changeset in webkit [65546] by crogers@google.com

422 edits
64 copies
12 deletes in branches/audio/WebCore/platform

merge trunk 61708:65443 into audio branch: WebCore directories 5

2:32 PM Changeset in webkit [65545] by crogers@google.com

172 edits
31 copies in branches/audio/WebCore

merge trunk 61708:65443 into audio branch: WebCore directories 4

2:30 PM Changeset in webkit [65544] by crogers@google.com

349 edits
42 copies
15 deletes in branches/audio/WebCore

merge trunk 61708:65443 into audio branch: WebCore directories 3

2:27 PM Changeset in webkit [65543] by crogers@google.com

166 edits
34 copies
2 deletes in branches/audio/WebCore

merge trunk 61708:65443 into audio branch: WebCore directories 2

2:26 PM QtWebKitTriageRoster edited by Laszlo Gombos

(diff)

2:25 PM Changeset in webkit [65542] by crogers@google.com

217 edits
22 copies
15 deletes in branches/audio/WebCore

merge trunk 61708:65443 into audio branch: WebCore directories 1

2:24 PM Changeset in webkit [65541] by crogers@google.com

5 edits in branches/audio/WebCore/audio

merge trunk 61708:65443 into audio branch: WebCore/audio

1:58 PM Changeset in webkit [65540] by Adam Roben

2 edits in trunk/WebKitSite

Instruct developers to (optionally) install Debugging Tools for Windows

These tools are used by some of our scripts, so are useful for
developers to have installed.

Fixes <http://webkit.org/b/44128> webkit.org building instructions
should recommend installing Debugging Tools for Windows

Reviewed by Sam Weinig.

building/tools.html:

1:57 PM Changeset in webkit [65539] by jamesr@google.com

4 edits
25 adds in trunk

2010-08-17 Fady Samuel <fsamuel@chromium.org>

Reviewed by David Hyatt.

Updated table cell hit testing and painting to use binary search instead of linear scan.

Improved Table Hit Testing and Painting Performance
https://bugs.webkit.org/show_bug.cgi?id=43933

Tests: fast/table/simple_paint.html

tables/hittesting/filltable-emptycells.html
tables/hittesting/filltable-levels.html
tables/hittesting/filltable-outline.html
tables/hittesting/filltable-rtl.html
tables/hittesting/filltable-stress.html

rendering/RenderTableSection.cpp: (WebCore::RenderTableSection::paintObject): Dirty rect determined using binary search. (WebCore::RenderTableSection::nodeAtPoint): Cell hit determined using binary search

2010-08-17 Fady Samuel <fsamuel@chromium.org>

Reviewed by David Hyatt.

Updated table cell hit testing and painting to use binary search instead of a linear scan.

Layout tests check that hit testing of empty cells, overlapping cells,
cells with outlines, right-to-left tables, and large
tables all work as expected.

Improved Table Hit Testing and Painting Performance
https://bugs.webkit.org/show_bug.cgi?id=43933

fast/table/simple_paint.html: Added.
platform/chromium-linux/fast/table/simple_paint-expected.checksum: Added.
platform/chromium-linux/fast/table/simple_paint-expected.png: Added.
platform/chromium-linux/fast/table/simple_paint-expected.txt: Added.
platform/chromium-linux/tables/hittesting/filltable-emptycells-expected.txt: Added.
platform/chromium-linux/tables/hittesting/filltable-levels-expected.txt: Added.
platform/chromium-linux/tables/hittesting/filltable-outline-expected.txt: Added.
platform/chromium-linux/tables/hittesting/filltable-rtl-expected.txt: Added.
platform/chromium-linux/tables/hittesting/filltable-stress-expected.txt: Added.
platform/chromium/test_expectations.txt:
platform/gtk/tables/hittesting/filltable-emptycells-expected.txt: Added.
platform/gtk/tables/hittesting/filltable-levels-expected.txt: Added.
platform/gtk/tables/hittesting/filltable-outline-expected.txt: Added.
platform/gtk/tables/hittesting/filltable-rtl-expected.txt: Added.
platform/gtk/tables/hittesting/filltable-stress-expected.txt: Added.
tables/hittesting/filltable-emptycells.html: Added.
tables/hittesting/filltable-levels.html: Added.
tables/hittesting/filltable-outline.html: Added.
tables/hittesting/filltable-rtl.html: Added.
tables/hittesting/filltable-stress.html: Added.

1:49 PM Changeset in webkit [65538] by inferno@chromium.org

3 edits
2 adds in trunk

2010-08-17 Abhishek Arya <inferno@chromium.org>

Reviewed by Dave Hyatt.

Only destroy empty anonymous block when it is columns or column span block
in RenderBlock::removeChild.
https://bugs.webkit.org/show_bug.cgi?id=44035

Test: fast/block/basic/empty-anonymous-block-remove-crash.html

rendering/RenderBlock.cpp: (WebCore::RenderBlock::removeChild):

2010-08-17 Abhishek Arya <inferno@chromium.org>

Reviewed by Dave Hyatt.

Tests that removing an empty anonymous block does not result in crash.
https://bugs.webkit.org/show_bug.cgi?id=44035

fast/block/basic/empty-anonymous-block-remove-crash-expected.txt: Added.
fast/block/basic/empty-anonymous-block-remove-crash.html: Added.

1:45 PM Changeset in webkit [65537] by Adam Roben

4 edits in trunk/WebKitTools

Teach run-webkit-tests, DumpRenderTree, and WebKitTestRunner how to save crash logs on Windows

The crash logs are saved to /tmp/layout-test-results and have names
like CrashLog_02bc_2010-08-17_14-36-20-108.txt.

Unfortunately, crashes in the WebKit2 web process are recorded as
time-outs by run-webkit-tests. Fixing this is covered by
<http://webkit.org/b/44121>.

Fixes <http://webkit.org/b/37859> <rdar://problem/7883560>
DumpRenderTree should save a stack trace and/or dump file when it
crashes

Reviewed by Sam Weinig.

DumpRenderTree/win/DumpRenderTree.cpp:

(exceptionFilter):
(main):

WebKitTestRunner/win/TestControllerWin.cpp:

(WTR::exceptionFilter):
(WTR::TestController::platformInitialize):
These changes set up an exception filter that prints #CRASHED to
stderr, then lets the exception propagate normally. This allows
run-webkit-tests to detect when we've crashed even when a post-mortem
debugger attaches to the process.

Scripts/old-run-webkit-tests:

(top level): Declared some variables used by the crash-log-saving
code.
(openDumpTool): Copy _NT_SYMBOL_PATH into the clean environment so
that the post-mortem debugger has access to it.
(toCygwinPath): Added. This is the opposite of toWindowsPath.
(readFromDumpToolWithTimer): If the dump tool prints #CRASHED to
stderr, consider it a crash.
(setUpWindowsCrashLogSaving): Added. Saves the current post-mortem
debugger, then sets ntsd as the post-mortem debugger. ntsd is
configured to save a crash log and then quit automatically.
(END): Added. Restores the previous post-mortem debugger when the
script exits.

1:17 PM Changeset in webkit [65536] by commit-queue@webkit.org

4 edits
1 delete in trunk/WebKit/chromium

2010-08-17 Sheriff Bot <webkit.review.bot@gmail.com>

Unreviewed, rolling out r65516.
http://trac.webkit.org/changeset/65516
https://bugs.webkit.org/show_bug.cgi?id=44126

broke chromium unit tests on mac and linux (Requested by
johnny_g on #webkit).

WebKit.gyp:
src/WebFrameImpl.cpp: (WebKit::frameContentAsPlainText):
tests/RunAllTests.cpp: (main):
tests/WebFrameTest.cpp: Removed.

1:14 PM Changeset in webkit [65535] by johnnyg@google.com

2 edits in trunk/LayoutTests

2010-08-17 John Gregg <johnnyg@google.com>

Unreviewed, test expectation.

Excluding the new geolocation multiple-requests layout tests from chromium.
See https://bugs.webkit.org/show_bug.cgi?id=44118

platform/chromium/test_expectations.txt:

1:07 PM Changeset in webkit [65534] by darin@chromium.org

3 edits in trunk/WebCore

2010-08-17 Darin Fisher <darin@chromium.org>

Reviewed by Darin Adler.

[Chromium] REGRESSION: Incorrect repaint on scrolling with position:fixed elements
https://bugs.webkit.org/show_bug.cgi?id=42949

This is a partial revert of http://trac.webkit.org/changeset/63907

page/FrameView.cpp: (WebCore::FrameView::scrollPositionChangedViaPlatformWidget): (WebCore::FrameView::scrollPositionChanged):
platform/ScrollView.cpp: (WebCore::ScrollView::valueChanged):

1:06 PM Changeset in webkit [65533] by vestbo@webkit.org

1 edit
2 adds in trunk/LayoutTests

Add testcase for r65514

Reviewed by Eric Carlson.

media/video-poster-delayed-expected.txt: Added.
media/video-poster-delayed.html: Added.

1:00 PM Changeset in webkit [65532] by kov@webkit.org

6 edits in trunk

2010-08-17 Gustavo Noronha Silva <gustavo.noronha@collabora.couk>

Reviewed by Martin Robinson.

WebBackForwardList.get_{back|forward}_list_with_limit not introspectable
https://bugs.webkit.org/show_bug.cgi?id=43054

GNUmakefile.am: Give also API implementation files to the GIR scanner.

WebKit/gtk

2010-08-17 Gustavo Noronha Silva <gustavo.noronha@collabora.co.uk>

Reviewed by Martin Robinson.

WebBackForwardList.get_{back|forward}_list_with_limit not introspectable
https://bugs.webkit.org/show_bug.cgi?id=43054

Annotate WebKitWebBackForwardList and the function that is used to fetch it from
the WebKitWebView. More will follow.

webkit/webkitwebbackforwardlist.cpp: Annotations, and small improvements to the docs.
webkit/webkitwebbackforwardlist.h: Couple empty lines missing.
webkit/webkitwebview.cpp:

12:51 PM Changeset in webkit [65531] by senorblanco@chromium.org

6 edits in trunk/WebCore

2010-08-17 Sheriff Bot <webkit.review.bot@gmail.com>

Unreviewed, rolling out r65528.
http://trac.webkit.org/changeset/65528
https://bugs.webkit.org/show_bug.cgi?id=44123

"Broke a bunch of canvas tests on Chrome win/linux."
(Requested by senorblanco on #webkit).

platform/graphics/skia/BitmapImageSingleFrameSkia.h: (WebCore::BitmapImageSingleFrameSkia::BitmapImageSingleFrameSkia):
platform/graphics/skia/ImageBufferSkia.cpp: (WebCore::ImageBuffer::drawsUsingCopy): (WebCore::ImageBuffer::copyImage): (WebCore::ImageBuffer::draw): (WebCore::ImageBuffer::drawPattern):
platform/graphics/skia/ImageSkia.cpp: (WebCore::BitmapImageSingleFrameSkia::create):
platform/graphics/skia/NativeImageSkia.cpp:
platform/graphics/skia/NativeImageSkia.h:

12:05 PM Changeset in webkit [65530] by Martin Robinson

4 edits
2 adds in trunk/WebCore

2010-08-17 Martin Robinson <mrobinson@igalia.com>

Reviewed by David Hyatt.

[Cairo] Remove unnecessary full-surface copies from ImageCairo and GraphicsContextCairo
https://bugs.webkit.org/show_bug.cgi?id=44076

No new tests as functionality has not changed.

GNUmakefile.am: Added GRefPtrCairo to the sources list.
platform/graphics/cairo/GRefPtrCairo.cpp: Added. (WTF::refGPtr): Added specializations for cairo_t and cairo_surface_t. (WTF::derefGPtr):
platform/graphics/cairo/GRefPtrCairo.h: Added.
platform/graphics/cairo/GraphicsContextCairo.cpp: (WebCore::GraphicsContext::createPlatformShadow): Access the image surface directly from the ImageBuffer instead of wrapping it in an Image.
platform/graphics/cairo/ImageCairo.cpp: (WebCore::Image::drawPattern): Create the temporary surface using cairo primitives instead of through the platform-independent WebCore code.

12:04 PM BuildingOnWindows edited by Adam Roben

Added instructions for getting crash logs from regression tests (diff)

12:00 PM Changeset in webkit [65529] by Martin Robinson

8 edits in trunk

2010-08-17 Martin Robinson <mrobinson@igalia.com>

[GTK] Clean up WebCore/platform/graphics/gtk/ImageGtk.cpp
https://bugs.webkit.org/show_bug.cgi?id=44069

No new tests as functionality has not changed.

GNUmakefile.am: Add WEBKITGTK_API_VERSION_STRING definition, which allows WebCore to use this as well.
platform/graphics/gtk/ImageGtk.cpp: (getWebKitDataDirectory): Added this helper. (WebCore::Image::loadPlatformResource): Use GOwnPtr for gchar pointers and use the new getWebKitDataDirectory helper.
platform/gtk/GOwnPtrGtk.cpp: Move the GtkIconInfo template specialization to this file. (WTF::GtkIconInfo): Added this forward declaration.
platform/gtk/GOwnPtrGtk.h: Declaration for new template specialization.

2010-08-17 Martin Robinson <mrobinson@igalia.com>

Reviewed by Gustavo Noronha Silva.

[GTK] Clean up WebCore/platform/graphics/gtk/ImageGtk.cpp
https://bugs.webkit.org/show_bug.cgi?id=44069

WebCoreSupport/InspectorClientGtk.cpp: (WebKit::InspectorClient::openInspectorFrontend): Switched to using the new WEBKITGTK_API_VERSION_STRING define.
webkit/webkitprivate.cpp: (inspectorGSettings): Ditto.

11:55 AM Changeset in webkit [65528] by senorblanco@chromium.org

6 edits in trunk/WebCore

2010-08-17 Stephen White <senorblanco@chromium.org>

Reviewed by Darin Fisher.

[CHROMIUM] Eliminate a memcpy() from the canvas.drawImage(canvas, ...) path.
This is the Skia followup to Dave Hyatt's patch in r65449.
https://bugs.webkit.org/show_bug.cgi?id=44115

Covered by canvas layout tests.

platform/graphics/skia/BitmapImageSingleFrameSkia.h: Add a "copyPixels" argument to the create() static to indicate if the caller wants the pixels to be copied or ref'ed.
platform/graphics/skia/ImageBufferSkia.cpp: (WebCore::ImageBuffer::drawsUsingCopy): We don't use the always-copy path anymore, so return false. (WebCore::ImageBuffer::copyImage): Shouldn't get here anymore, so ASSERT_NOT_REACHED. (WebCore::ImageBuffer::draw): (WebCore::ImageBuffer::drawPattern): Create a temporary BitampImageSingleFrameSkia wrapped around the canvas's bitmap, and draw with it immediately. If drawing to our own canvas, copy the pixels, otherwise just shallow copy.
platform/graphics/skia/ImageSkia.cpp: (WebCore::BitmapImageSingleFrameSkia::BitmapImageSingleFrameSkia): Make the constructor take an SkBitmap to be wrapped. (WebCore::BitmapImageSingleFrameSkia::create): Implement the "copyPixels" flag: if true, do a deep copy, otherwise do a shallow copy.
platform/graphics/skia/NativeImageSkia.cpp: (WebCore::NativeImageSkia::NativeImageSkia):
platform/graphics/skia/NativeImageSkia.h: Implement a new constructor that shallow-copies the given SkBitmap.

11:47 AM Changeset in webkit [65527] by victorw@chromium.org

3 edits in trunk/WebKitTools

2010-08-17 Victor Wang <victorw@chromium.org>

Reviewed by ojan@chromium.org.

Update test results server:

Normalize test results and times after merging (prune tests where all runs pass or do not have data, truncate all test items to max number of builds)
times should be int not string.
when inserting a new test item, should keep old data regardless whether or not they have same item type with new one.

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

TestResultServer/model/jsonresults.py:
TestResultServer/model/jsonresults_unittest.py:

11:45 AM Changeset in webkit [65526] by jianli@chromium.org

8 edits
6 moves in trunk/WebCore

Move BlobRegistry interface and implementation to platform/network
https://bugs.webkit.org/show_bug.cgi?id=44116

Reviewed by Darin Fisher.

Also move BlobData and BlobStorageData since they're part of BlobRegistry
implementation.

Android.mk:
CMakeLists.txt:
GNUmakefile.am:
WebCore.gypi:
WebCore.pro:
WebCore.vcproj/WebCore.vcproj:
WebCore.xcodeproj/project.pbxproj:
platform/network/BlobData.cpp: Renamed from WebCore/platform/BlobData.cpp.
platform/network/BlobData.h: Renamed from WebCore/platform/BlobData.h.
platform/network/BlobRegistry.h: Renamed from WebCore/platform/BlobRegistry.h.
platform/network/BlobRegistryImpl.cpp: Renamed from WebCore/html/BlobRegistryImpl.cpp.
platform/network/BlobRegistryImpl.h: Renamed from WebCore/html/BlobRegistryImpl.h.
platform/network/BlobStorageData.h: Renamed from WebCore/html/BlobStorageData.h.

11:43 AM Changeset in webkit [65525] by andreas.kling@nokia.com

2 edits in trunk/WebCore

2010-08-17 Andreas Kling <andreas.kling@nokia.com>

Reviewed by Ariya Hidayat.

[Qt] Path: Fast approximation of stroke bounding rects
https://bugs.webkit.org/show_bug.cgi?id=44014

In canvas's stroke(), use QPainterPath::controlPointRect() to calculate
a slightly larger bounding rect in a fraction of the time.

html/canvas/CanvasRenderingContext2D.cpp: (WebCore::CanvasRenderingContext2D::stroke):

11:39 AM Changeset in webkit [65524] by Girish Ramakrishnan

2 edits in trunk/WebCore

Set PluginQuirkRequiresDefaultScreenDepth for all Flash versions and not
just Flash 10.

Reviewed by Kenneth Rohde Christiansen.

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

plugins/PluginPackage.cpp:

(WebCore::PluginPackage::determineQuirks):

11:31 AM Changeset in webkit [65523] by jianli@chromium.org

9 edits in trunk/WebCore

Change FileStream implementation to prepare for blob resource handling.
https://bugs.webkit.org/show_bug.cgi?id=43985

Reviewed by David Levin.

Change FileStream to support synchronous usage only. All the asychronous
logics are moved to FileStreamProxy. Update the existing FileReader code
to adapt to this change.

html/FileReader.cpp: Account for FileStreamProxy changes.

(WebCore::FileReader::didStart):
(WebCore::FileReader::didGetSize):
(WebCore::FileReader::didOpen):
(WebCore::FileReader::didRead):

html/FileReader.h:
html/FileStream.cpp: Remove m_client and make all calls synchronous only.

(WebCore::FileStream::FileStream):
(WebCore::FileStream::start):
(WebCore::FileStream::stop):
(WebCore::FileStream::getSize):
(WebCore::FileStream::openForRead):
(WebCore::FileStream::openForWrite):
(WebCore::FileStream::close):
(WebCore::FileStream::read):
(WebCore::FileStream::write):
(WebCore::FileStream::truncate):

html/FileStream.h: Add getSize() and make all calls synchronous by adding return values.

(WebCore::FileStream::create):

html/FileStreamClient.h: Account for FileStreamProxy changes.

(WebCore::FileStreamClient::didRead):
(WebCore::FileStreamClient::didTruncate):
(WebCore::FileStreamClient::didOpen):

html/FileStreamProxy.cpp: All the logics to handle asynchronous calls are wrapped here.

(WebCore::FileStreamProxy::FileStreamProxy):
(WebCore::FileStreamProxy::create):
(WebCore::FileStreamProxy::fileThread):
(WebCore::didStart):
(WebCore::FileStreamProxy::startOnFileThread):
(WebCore::FileStreamProxy::stop):
(WebCore::derefProxyOnContext):
(WebCore::FileStreamProxy::stopOnFileThread):
(WebCore::didGetSize):
(WebCore::FileStreamProxy::getSize):
(WebCore::FileStreamProxy::getSizeOnFileThread):
(WebCore::didOpen):
(WebCore::FileStreamProxy::openForRead):
(WebCore::FileStreamProxy::openForReadOnFileThread):
(WebCore::FileStreamProxy::openForWrite):
(WebCore::FileStreamProxy::openForWriteOnFileThread):
(WebCore::FileStreamProxy::close):
(WebCore::FileStreamProxy::closeOnFileThread):
(WebCore::didRead):
(WebCore::FileStreamProxy::read):
(WebCore::FileStreamProxy::readOnFileThread):
(WebCore::didWrite):
(WebCore::FileStreamProxy::write):
(WebCore::FileStreamProxy::writeOnFileThread):
(WebCore::didTruncate):
(WebCore::FileStreamProxy::truncate):
(WebCore::FileStreamProxy::truncateOnFileThread):

html/FileStreamProxy.h:
html/FileThreadTask.h: Change all methods to return void since return value is not needed now.

11:28 AM Changeset in webkit [65522] by weinig@apple.com

2 edits in trunk/LayoutTests

Add a few items in the WebKitTestRunner Skipped list.

Rubber-stamped by Anders Carlsson.

platform/mac-wk2/Skipped:

11:22 AM Changeset in webkit [65521] by Laszlo Gombos

2 edits in trunk/WebKit/qt

2010-08-17 Hui Huang <hui.2.huang@nokia.com>

Reviewed by Laszlo Gombos.

[Qt] Fix ARM5 compiling error in FrameLoaderClientQt.cpp
https://bugs.webkit.org/show_bug.cgi?id=43927

In the Symbian build chain QT_USE_FAST_OPERATOR_PLUS is defined,
which means that the '+' will automatically be performed as the
QStringBuilder '%'. Create a QString explicitly to make sure that
toLocal8bit() member is available that the qPrintable Macro uses.

WebCoreSupport/FrameLoaderClientQt.cpp: (WebCore::FrameLoaderClientQt::dispatchDecidePolicyForNavigationAction):

11:06 AM Changeset in webkit [65520] by Dimitri Glazkov

2 edits in trunk/WebCore

2010-08-17 Dimitri Glazkov <Dimitri Glazkov>

Returned WebCore.xcodeproj to default settings after http://trac.webkit.org/changeset/65468/trunk/WebCore/WebCore.xcodeproj/project.pbxproj.

WebCore.xcodeproj/project.pbxproj: Removed developmentRegion = English line.

11:06 AM Changeset in webkit [65519] by eric@webkit.org

1 edit
6 moves in trunk/LayoutTests

2010-08-17 Eric Seidel <eric@webkit.org>

Reviewed by Alexey Proskuryakov.

sputnik S15.9.3.1_A5_T1~T6 fail when run in timezone other than Pacific Time
https://bugs.webkit.org/show_bug.cgi?id=42625

fast/js/sputnik/Conformance/15_Native_Objects/15.9_Date/15.9.3/S15.9.3.1_A5_T1.html-disabled: Renamed from LayoutTests/fast/js/sputnik/Conformance/15_Native_Objects/15.9_Date/15.9.3/S15.9.3.1_A5_T1.html.
fast/js/sputnik/Conformance/15_Native_Objects/15.9_Date/15.9.3/S15.9.3.1_A5_T2.html-disabled: Renamed from LayoutTests/fast/js/sputnik/Conformance/15_Native_Objects/15.9_Date/15.9.3/S15.9.3.1_A5_T2.html.
fast/js/sputnik/Conformance/15_Native_Objects/15.9_Date/15.9.3/S15.9.3.1_A5_T3.html-disabled: Renamed from LayoutTests/fast/js/sputnik/Conformance/15_Native_Objects/15.9_Date/15.9.3/S15.9.3.1_A5_T3.html.
fast/js/sputnik/Conformance/15_Native_Objects/15.9_Date/15.9.3/S15.9.3.1_A5_T4.html-disabled: Renamed from LayoutTests/fast/js/sputnik/Conformance/15_Native_Objects/15.9_Date/15.9.3/S15.9.3.1_A5_T4.html.
fast/js/sputnik/Conformance/15_Native_Objects/15.9_Date/15.9.3/S15.9.3.1_A5_T5.html-disabled: Renamed from LayoutTests/fast/js/sputnik/Conformance/15_Native_Objects/15.9_Date/15.9.3/S15.9.3.1_A5_T5.html.
fast/js/sputnik/Conformance/15_Native_Objects/15.9_Date/15.9.3/S15.9.3.1_A5_T6.html-disabled: Renamed from LayoutTests/fast/js/sputnik/Conformance/15_Native_Objects/15.9_Date/15.9.3/S15.9.3.1_A5_T6.html.

11:04 AM Changeset in webkit [65518] by Martin Robinson

2 edits in trunk

2010-08-17 Martin Robinson <mrobinson@igalia.com>

GTK+ Build fix for those having stable versions of GLib.

configure.ac: Only invoke GLIB_GSETTINGS if we have the necessary gio version.

10:55 AM Changeset in webkit [65517] by Dimitri Glazkov

2 edits in trunk/WebCore

Eliminate extra traversal of the tree when determining ability to lazy-attach a Node.
https://bugs.webkit.org/show_bug.cgi?id=43749

Reviewed by Eric Seidel.

Land the actual change that I intended to land in http://trac.webkit.org/changeset/65508.

Neither SVG nor inputs/counters shadow DOM are traveling down this path (they home-bake their attachment),
so the check for shadow tree existence isn't needed.
Gives a small, but consistent win in Dromaeo's DOM Modification test.

dom/Node.h:

(WebCore::Node::canLazyAttach): Changed to always return true.

10:45 AM Changeset in webkit [65516] by jcivelli@chromium.org

4 edits
1 add in trunk/WebKit/chromium

2010-08-17 Jay Civelli <jcivelli@chromium.org>

Reviewed by Darin Fisher.

Made WebFrame not report the text from hidden frames.
(some pages contain hidden frames with garbage text that
should not be indexed or used to detect the page's language).
https://bugs.webkit.org/show_bug.cgi?id=39456

WebKit.gyp:
public/WebCString.h: (WebKit::operator<):
public/WebURL.h: (WebKit::operator<):
src/WebCString.cpp: (WebKit::WebCString::compare):
src/WebFrameImpl.cpp: (WebKit::frameContentAsPlainText):
tests/RunAllTests.cpp: (main):
tests/WebFrameTest.cpp: Added.

10:24 AM Changeset in webkit [65515] by cmarrin@apple.com

3 edits in trunk/ANGLE

Moved previous link error fix to Base.xcconfig

10:18 AM Changeset in webkit [65514] by vestbo@webkit.org

2 edits in trunk/WebCore

Restore intrinsic size of video elements after loading poster

Reviewed by Eric Carlson.

The element might already have the instrinsic size of the video, in
which case we don't want to override it witht that of the image.

rendering/RenderVideo.cpp: call updateIntrinsicSize() to restore size

10:13 AM Changeset in webkit [65513] by steveblock@google.com

5 edits in trunk/LayoutTests

2010-08-17 Steve Block <steveblock@google.com>

Reviewed by Jeremy Orlow.

[Qt] fast/dom/Geolocation/disconnected-frame-permission-denied.html fails
https://bugs.webkit.org/show_bug.cgi?id=44059

fast/dom/Geolocation/disconnected-frame-permission-denied-expected.txt:
fast/dom/Geolocation/script-tests/disconnected-frame-permission-denied.js: (onIframeLoaded): ():
platform/chromium/test_expectations.txt:
platform/qt/Skipped:

10:05 AM Changeset in webkit [65512] by cmarrin@apple.com

2 edits in trunk/ANGLE

Fixed link errors in WebCore build on Mac

9:54 AM Changeset in webkit [65511] by commit-queue@webkit.org

6 edits
6 adds in trunk

2010-08-17 Mahesh Kulkarni <mahesh.kulkarni@nokia.com>

Reviewed by Steve Block.

Geolocation preemptive permissions policy is buggy
https://bugs.webkit.org/show_bug.cgi?id=42811

Adds cases where permission is niether denied or granted immediately and multiple
requests are waiting.

fast/dom/Geolocation/delayed-permission-allowed-for-multiple-requests-expected.txt: Added.
fast/dom/Geolocation/delayed-permission-allowed-for-multiple-requests.html: Added.
fast/dom/Geolocation/delayed-permission-denied-for-multiple-requests-expected.txt: Added.
fast/dom/Geolocation/delayed-permission-denied-for-multiple-requests.html: Added.
fast/dom/Geolocation/script-tests/delayed-permission-allowed-for-multiple-requests.js: Added. (allowPermission): (): (maybeFinishTest):
fast/dom/Geolocation/script-tests/delayed-permission-denied-for-multiple-requests.js: Added. (denyPermission): (maybeFinishTest):
platform/gtk/Skipped:
platform/qt/Skipped:

2010-08-17 Mahesh Kulkarni <mahesh.kulkarni@nokia.com>

Reviewed by Steve Block.

Geolocation preemptive permissions policy is buggy
https://bugs.webkit.org/show_bug.cgi?id=42811

While waiting for permission, m_startRequestPermissionNotifier was
used to consider only one pending request. This patch implements a set
m_pendingForPermissionNotifiers to maintain set of pending requests.
When user grants/denies permission all listeners will be notified.
Also fixed issue with hasZeroTimeout() where startTimerIfNeeded() has
to start irrespective of permission state

Tests: fast/dom/Geolocation/delayed-permission-allowed-for-multiple-requests.html

fast/dom/Geolocation/delayed-permission-denied-for-multiple-requests.html

WebCore.pro:
page/Geolocation.cpp: (WebCore::Geolocation::startRequest): (WebCore::Geolocation::setIsAllowed): (WebCore::Geolocation::startUpdating): (WebCore::Geolocation::handlePendingPermissionNotifiers):
page/Geolocation.h:

9:27 AM Changeset in webkit [65510] by knorton@google.com

2 edits in trunk/WebKit/chromium

2010-08-17 Kelly Norton <knorton@google.com>

Reviewed by Pavel Feldman.

Web Inspector: Fixes a crash in addTimelineRecord when InspectorTimelineAgent
is enabled through WebDevToolsAgentImpl::setApuEnabled.
https://bugs.webkit.org/show_bug.cgi?id=44105

src/WebDevToolsAgentImpl.cpp: (WebKit::WebDevToolsAgentImpl::setApuAgentEnabled):

9:18 AM Changeset in webkit [65509] by jorlow@chromium.org

19 edits
5 adds in trunk

2010-08-09 Jeremy Orlow <jorlow@chromium.org>

Reviewed by Steve Block.

Beginnings of IndexedDB persistance + IDBDatabase.description fleshed out
https://bugs.webkit.org/show_bug.cgi?id=43744

Add a layout test for IDBDatabase.description.

storage/indexeddb/database-description-expected.txt: Added.
storage/indexeddb/database-description.html: Added.
storage/indexeddb/script-tests/database-description.js: Added. (test): (firstSuccess): (secondSuccess): (thirdSuccess):

2010-08-11 Jeremy Orlow <jorlow@chromium.org>

Beginnings of IndexedDB persistance + IDBDatabase.description fleshed out
https://bugs.webkit.org/show_bug.cgi?id=43744

The beginnings of a SQLite backend for IndexedDB. Right now, only persists
meta-data for the database. The rest is coming in future patches. Adds
a manual test to verify persistance.

Test: storage/indexeddb/database-description.html

+ a manual test

Android.mk:
CMakeLists.txt:
GNUmakefile.am:
WebCore.gypi:
WebCore.vcproj/WebCore.vcproj:
WebCore.xcodeproj/project.pbxproj:
manual-tests/indexed-database.html: Added.
page/SecurityOrigin.cpp: (WebCore::SecurityOrigin::databaseIdentifier):
platform/FileSystem.cpp: Added. (WebCore::): (WebCore::shouldEscapeUChar): (WebCore::encodeForFileName):
platform/FileSystem.h:
storage/IDBDatabase.cpp: (WebCore::IDBDatabase::IDBDatabase):
storage/IDBDatabase.h: (WebCore::IDBDatabase::description):
storage/IDBDatabaseBackendImpl.cpp: (WebCore::extractMetaData): (WebCore::setMetaData): (WebCore::IDBDatabaseBackendImpl::IDBDatabaseBackendImpl): (WebCore::IDBDatabaseBackendImpl::setDescription):
storage/IDBDatabaseBackendImpl.h: (WebCore::IDBDatabaseBackendImpl::create):
storage/IDBFactory.h:
storage/IDBFactory.idl:
storage/IDBFactoryBackendImpl.cpp: (WebCore::IDBFactoryBackendImpl::IDBFactoryBackendImpl): (WebCore::IDBFactoryBackendImpl::~IDBFactoryBackendImpl): (WebCore::openSQLiteDatabase): (WebCore::createTables): (WebCore::IDBFactoryBackendImpl::open):
storage/IDBFactoryBackendImpl.h: (WebCore::IDBFactoryBackendImpl::create):

9:15 AM Changeset in webkit [65508] by Dimitri Glazkov

3 edits in trunk/WebCore

2010-08-17 Dimitri Glazkov <Dimitri Glazkov>

Reviewed by Eric Seidel.

Eliminate extra traversal of the tree when determining ability to lazy-attach a Node.
https://bugs.webkit.org/show_bug.cgi?id=43749

Neither SVG nor inputs/counters shadow DOM are traveling down this path (they home-bake their attachment),
so the check for shadow tree existence isn't needed.

Gives a small, but consisten win in Dromaeo's DOM Modification test.

dom/Node.cpp: Removed canLazyAttach declaration.
dom/Node.h: (WebCore::Node::canLazyAttach): Changed to always return true.

8:59 AM Changeset in webkit [65507] by yurys@chromium.org

2 edits in trunk/LayoutTests

2010-08-17 Yury Semikhatsky <yurys@chromium.org>

Unreviewed. Revert r41142. Unskipped tests are still failing.

platform/qt/Skipped:

8:55 AM Changeset in webkit [65506] by pfeldman@chromium.org

9 edits in trunk

2010-08-17 Pavel Feldman <pfeldman@chromium.org>

Reviewed by Yury Semikhatsky.

Web Inspector: [CRASH] Console formatting crashes after cross-domain navigation.
https://bugs.webkit.org/show_bug.cgi?id=44103

http/tests/inspector-enabled/resources/console-log-frame-before-navigation.html:

2010-08-17 Pavel Feldman <pfeldman@chromium.org>

Reviewed by Yury Semikhatsky.

Web Inspector: [CRASH] Console formatting crashes after cross-domain navigation.
https://bugs.webkit.org/show_bug.cgi?id=44103

bindings/js/JSInjectedScriptHostCustom.cpp: (WebCore::InjectedScriptHost::injectedScriptFor):
bindings/v8/custom/V8InjectedScriptHostCustom.cpp: (WebCore::InjectedScriptHost::injectedScriptFor):
inspector/ConsoleMessage.cpp: (WebCore::ConsoleMessage::addToFrontend):

8:41 AM Changeset in webkit [65505] by yurys@chromium.org

6 edits
7 adds in trunk

2010-08-17 Yury Semikhatsky <yurys@chromium.org>

Reviewed by Pavel Feldman.

Web Inspector: upstream Chromium debugger step tests
https://bugs.webkit.org/show_bug.cgi?id=44106

http/tests/inspector-enabled/console-log-before-frame-navigation.html:
http/tests/inspector-enabled/resources/console-log-before-frame-navigation.js: Added. (doit.callback): (doit): (receiveMessage): (frontend_dumpConsoleMessages):
http/tests/inspector/debugger-test.js: (frontend_captureStackTrace):
inspector/debugger-step-in-expected.txt: Added.
inspector/debugger-step-in.html: Added.
inspector/debugger-step-out-expected.txt: Added.
inspector/debugger-step-out.html: Added.
inspector/debugger-step-over-expected.txt: Added.
inspector/debugger-step-over.html: Added.

2010-08-17 Yury Semikhatsky <yurys@chromium.org>

Reviewed by Pavel Feldman.

Web Inspector: upstream Chromium debugger step tests
https://bugs.webkit.org/show_bug.cgi?id=44106

Tests: inspector/debugger-step-in.html

inspector/debugger-step-out.html
inspector/debugger-step-over.html

bindings/js/ScriptDebugServer.cpp: (WebCore::ScriptDebugServer::returnEvent): check that debugger is still attached after continue to ensure that current call frame is not 0.

2010-08-17 Yury Semikhatsky <yurys@chromium.org>

Reviewed by Pavel Feldman.

Web Inspector: upstream Chromium debugger step tests
https://bugs.webkit.org/show_bug.cgi?id=44106

src/js/Tests.js:

8:35 AM Changeset in webkit [65504] by commit-queue@webkit.org

3 edits in trunk/WebCore

2010-08-17 Ademar de Souza Reis Jr <Ademar Reis>

Reviewed by David Levin.

Trivial fix removing the self-inclusion of a couple of header files.

Self inclusion in Document.h and RenderTheme.h
https://bugs.webkit.org/show_bug.cgi?id=44065

dom/Document.h:
rendering/RenderTheme.h:

8:03 AM Changeset in webkit [65503] by yurys@chromium.org

2 edits in trunk/LayoutTests

2010-08-17 Yury Semikhatsky <yurys@chromium.org>

Reviewed by Pavel Feldman.

[Qt] Web Inspector: inspector/debugger-pause-on-debugger-statement.html times out on Qt
https://bugs.webkit.org/show_bug.cgi?id=41142

platform/qt/Skipped: the tests should pass after r65315

8:00 AM ApplicationsGtk edited by Laszlo Gombos

(diff)

7:54 AM Changeset in webkit [65502] by noam.rosenthal@nokia.com

4 edits in trunk

2010-08-17 No'am Rosenthal <noam.rosenthal@nokia.com>

Reviewed by Ariya Hidayat.

[Qt] Move the accelerated compositing build flag to the right place
https://bugs.webkit.org/show_bug.cgi?id=43882

wtf/Platform.h:

2010-08-17 No'am Rosenthal <noam.rosenthal@nokia.com>

Reviewed by Ariya Hidayat.

[Qt] Move the accelerated compositing build flag to the right place
https://bugs.webkit.org/show_bug.cgi?id=43882

No new tests: build fix.

WebCore.pro:

7:49 AM Changeset in webkit [65501] by loislo@chromium.org

19 edits
1 add
1 delete in trunk

2010-08-17 Sheriff Bot <webkit.review.bot@gmail.com>

Unreviewed, rolling out r65500.
http://trac.webkit.org/changeset/65500
https://bugs.webkit.org/show_bug.cgi?id=44108

Qt bots failed to compile. (Requested by loislo on #webkit).

GNUmakefile.am:

2010-08-17 Sheriff Bot <webkit.review.bot@gmail.com>

Unreviewed, rolling out r65500.
http://trac.webkit.org/changeset/65500
https://bugs.webkit.org/show_bug.cgi?id=44108

Qt bots failed to compile. (Requested by loislo on #webkit).

platform/gtk/Skipped:

2010-08-17 Sheriff Bot <webkit.review.bot@gmail.com>

Unreviewed, rolling out r65500.
http://trac.webkit.org/changeset/65500
https://bugs.webkit.org/show_bug.cgi?id=44108

Qt bots failed to compile. (Requested by loislo on #webkit).

GNUmakefile.am:
WebCore.gyp/WebCore.gyp:
WebCore.gypi:
WebCore.pri:
WebCore.pro:
WebCore.vcproj/copyInspectorFiles.cmd:
WebCore.xcodeproj/project.pbxproj:
combine-javascript-resources:
inspector/CodeGeneratorInspector.pm:
inspector/front-end/InspectorBackendStub.js: Added. (WebInspector.InspectorBackendStub): (WebInspector.InspectorBackendStub.prototype._registerDelegate): (WebInspector.InspectorBackendStub.prototype.sendMessageToBackend):
inspector/front-end/InspectorBackendStub.qrc: Removed.
inspector/front-end/WebKit.qrc:

2010-08-17 Sheriff Bot <webkit.review.bot@gmail.com>

Unreviewed, rolling out r65500.
http://trac.webkit.org/changeset/65500
https://bugs.webkit.org/show_bug.cgi?id=44108

Qt bots failed to compile. (Requested by loislo on #webkit).

WebKit.gyp:

2010-08-17 Sheriff Bot <webkit.review.bot@gmail.com>

Unreviewed, rolling out r65500.
http://trac.webkit.org/changeset/65500
https://bugs.webkit.org/show_bug.cgi?id=44108

Qt bots failed to compile. (Requested by loislo on #webkit).

WebCoreSupport/InspectorClientGtk.cpp: (WebKit::InspectorClient::openInspectorFrontend):

7:33 AM Changeset in webkit [65500] by loislo@chromium.org

19 edits
1 add
1 delete in trunk

2010-08-16 Ilya Tikhonovsky <loislo@chromium.org>

Reviewed by Yury Semikhatsky.

Web Inspector: replace hand written InspectorBackendStub.js by generated one.
New command-line flag was added to combine-javascript-resources.
It specifies the location of generated scripts.
https://bugs.webkit.org/show_bug.cgi?id=43791

GNUmakefile.am:
WebCore.gyp/WebCore.gyp:
WebCore.gypi:
WebCore.pri:
WebCore.pro:
WebCore.vcproj/WebCore.vcproj:
WebCore.xcodeproj/project.pbxproj:
combine-javascript-resources:
inspector/CodeGeneratorInspector.pm:
inspector/front-end/InspectorBackendStub.js: Removed.
inspector/front-end/WebKit.qrc:

2010-08-16 Ilya Tikhonovsky <loislo@chromium.org>

Reviewed by Yury Semikhatsky.

Web Inspector: replace hand written InspectorBackendStub.js by generated one.
https://bugs.webkit.org/show_bug.cgi?id=43791

GNUmakefile.am:

2010-08-17 Ilya Tikhonovsky <loislo@chromium.org>

Reviewed by Yury Semikhatsky.

WebInspector: inspector tests for GTK were added to Skip list.
They should be enabled later.
Bug for GTK https://bugs.webkit.org/show_bug.cgi?id=43977
https://bugs.webkit.org/show_bug.cgi?id=43791

platform/gtk/Skipped:

2010-08-17 Ilya Tikhonovsky <loislo@chromium.org>

Reviewed by Yury Semikhatsky.

Web Inspector: replace hand written InspectorBackendStub.js by generated one.
https://bugs.webkit.org/show_bug.cgi?id=43791

WebKit.gyp:

2010-08-17 Ilya Tikhonovsky <loislo@chromium.org>

Reviewed by Yury Semikhatsky.

Web Inspector: replace hand written InspectorBackendStub.js by generated one.
https://bugs.webkit.org/show_bug.cgi?id=43791

WebCoreSupport/InspectorClientGtk.cpp: (WebKit::InspectorClient::openInspectorFrontend):

6:30 AM Changeset in webkit [65499] by pfeldman@chromium.org

9 edits in trunk/WebCore

2010-08-17 Pavel Feldman <pfeldman@chromium.org>

Reviewed by Pavel Feldman.

Web Inspector: get rid of node and scoped object-specific ids.
https://bugs.webkit.org/show_bug.cgi?id=44097

inspector/front-end/ConsoleView.js:
inspector/front-end/ElementsTreeOutline.js: (WebInspector.ElementsTreeElement.prototype._createTooltipForNode):
inspector/front-end/EventListenersSidebarPane.js:
inspector/front-end/InjectedScript.js: (injectedScriptConstructor.): (injectedScriptConstructor):
inspector/front-end/InjectedScriptAccess.js: (InjectedScriptAccess): (InjectedScriptAccess.getForNode): (InjectedScriptAccess._installHandler.InjectedScriptAccess.prototype.methodName): (InjectedScriptAccess._installHandler):
inspector/front-end/PropertiesSidebarPane.js: (WebInspector.PropertiesSidebarPane.prototype.update):
inspector/front-end/RemoteObject.js: (WebInspector.RemoteObjectId): (WebInspector.RemoteObject.resolveNode):
inspector/front-end/ScriptsPanel.js:

6:09 AM Changeset in webkit [65498] by kov@webkit.org

6 edits
1 add in trunk

2010-08-17 Gustavo Noronha Silva <gustavo.noronha@collabora.co.uk>

Reviewed by Martin Robinson.

[GTK] GSettings support adds annoying warnings on systems with older glib versions
https://bugs.webkit.org/show_bug.cgi?id=44056

Add file containing GLIB_GSETTINGS macro so that
configure doesn't spit warnings in older systems.

autotools/gsettings.m4: Added.
.gitignore: Ignore files that are generated inside autotools explicitely, because we have some that are kept in version control.

WebKit/gtk

2010-08-17 Gustavo Noronha Silva <gustavo.noronha@collabora.co.uk>

Reviewed by Martin Robinson.

[GTK] GSettings support adds annoying warnings on systems with older glib versions
https://bugs.webkit.org/show_bug.cgi?id=44056

Use #ifdef instead of #if for HAVE_GSETTINGS to avoid warnings
when it is not defined.

WebCoreSupport/InspectorClientGtk.cpp:
webkit/webkitprivate.cpp: (webkit_init):
webkit/webkitprivate.h:

5:54 AM QtScript edited by kent.hansen@nokia.com

(diff)

5:23 AM Changeset in webkit [65497] by xan@webkit.org

3 edits in trunk/WebCore

2010-08-17 Xan Lopez <xlopez@igalia.com>

Reviewed by Gustavo Noronha.

[GTK] Fix wrapping of core Events in the bindings
https://bugs.webkit.org/show_bug.cgi?id=44095

Return the proper GObject type instead of hardcoding
WebKitDOMEvent for all Event types.

bindings/gobject/GObjectEventListener.cpp: (WebCore::GObjectEventListener::handleEvent):
bindings/gobject/WebKitDOMBinding.cpp: (WebKit::kit):

5:22 AM Changeset in webkit [65496] by xan@webkit.org

2 edits in trunk/WebCore

2010-08-17 Xan Lopez <xlopez@igalia.com>

Reviewed by Gustavo Noronha.

Revert supposed build-fix which was in fact reverting a previous
build fix which was correct (oh dear).

GNUmakefile.am:

5:15 AM Changeset in webkit [65495] by vestbo@webkit.org

3 edits in trunk/WebCore

[Qt] Add media element support for preload=none

Reviewed by Antonio Gomes.

platform/graphics/qt/MediaPlayerPrivateQt.cpp:
platform/graphics/qt/MediaPlayerPrivateQt.h:

5:15 AM Changeset in webkit [65494] by vestbo@webkit.org

3 edits in trunk/WebCore

[Qt] Pre-roll media when loading media elements

Reviewed by Antonio Gomes.

This ensures we get correct size-hints from QtMultimedia, as well as
the media states Buffering and Buffered instead of Loading and Loaded,
which we use to transition into MediaPlayer::HaveEnoughData.

platform/graphics/qt/MediaPlayerPrivateQt.cpp:
platform/graphics/qt/MediaPlayerPrivateQt.h:

4:54 AM Changeset in webkit [65493] by commit-queue@webkit.org

2 edits in trunk/JavaScriptCore

2010-08-17 Yuta Kitamura <yutak@chromium.org>

Reviewed by Shinichiro Hamaji.

Avoid uninitialized memory read in StringImpl::find().

REGRESSION(r65468): Crashes in StringImpl::find
https://bugs.webkit.org/show_bug.cgi?id=44099

wtf/text/StringImpl.cpp: (WTF::StringImpl::find):

4:38 AM Changeset in webkit [65492] by vestbo@webkit.org

2 edits in trunk/WebCore

[Qt] Fix missing default controls for stand-alone media elements

Reviewed by Kenneth Christiansen.

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

css/mediaControlsQt.css:

4:18 AM Changeset in webkit [65491] by vestbo@webkit.org

2 edits in trunk/WebKit/qt

[Qt] Implement ChromeClient's windowToScreen() and screenToWindow()

Reviewed by Kenneth Christiansen.

WebCoreSupport/ChromeClientQt.cpp:

2:49 AM QtWebKitTriageRoster edited by Csaba Osztrogonác

(diff)

2:30 AM QtWebKitRelease21 edited by Csaba Osztrogonác

typo fix (diff)

2:05 AM Changeset in webkit [65490] by yurys@chromium.org

3 edits
5 adds in trunk

2010-08-17 Yury Semikhatsky <yurys@chromium.org>

Reviewed by Pavel Feldman.

Web Inspector: upstream Chromium console tests
https://bugs.webkit.org/show_bug.cgi?id=44057

inspector/console-eval-expected.txt: Added.
inspector/console-eval-global-expected.txt: Added.
inspector/console-eval-global.html: Added.
inspector/console-eval.html: Added.
platform/chromium/inspector/console-eval-global-expected.txt: Added.

2010-08-17 Yury Semikhatsky <yurys@chromium.org>

Reviewed by Pavel Feldman.

Web Inspector: upstream Chromium console tests and remove a couple
of tests that have already been covered with layout tests.
https://bugs.webkit.org/show_bug.cgi?id=44057

src/js/Tests.js: (.TestSuite.prototype.testConsoleEval): (.TestSuite.prototype.testConsoleLog): (.TestSuite.prototype.testEvalGlobal):

1:55 AM Changeset in webkit [65489] by commit-queue@webkit.org

2 edits in trunk/LayoutTests

2010-08-17 Ademar de Souza Reis Jr <Ademar Reis>

Reviewed by Ariya Hidayat.

[Qt] tests whose bug was closed but are still skipped
https://bugs.webkit.org/show_bug.cgi?id=44068

Enable tests whose bugs have been fixed already:

https://bugs.webkit.org/show_bug.cgi?id=35350 (fixed)
https://bugs.webkit.org/show_bug.cgi?id=28952 (fixed)
https://bugs.webkit.org/show_bug.cgi?id=38502 (see discussion at the end)
https://bugs.webkit.org/show_bug.cgi?id=38604 (fixed)

platform/qt/Skipped:

1:32 AM Changeset in webkit [65488] by ariya@webkit.org

2 edits in trunk/WebCore

2010-08-17 Ariya Hidayat <ariya@sencha.com>

Reviewed by Kenneth Rohde Christiansen.

[Qt] Reduce the size of the shadow buffer to the clip region
https://bugs.webkit.org/show_bug.cgi?id=44091

Instead of allocating the buffer image (for the blur support) as big
as the rectangle which casts the shadow, we limit the size to the
current clip region.

platform/graphics/qt/ContextShadow.cpp: (WebCore::ContextShadow::drawShadowRect):

12:08 AM Changeset in webkit [65487] by mjs@apple.com

2 edits in trunk/LayoutTests

Add/correct a few items in the WebKitTestRunner Skipped list.

Rubber stamped by Sam Weinig.

platform/mac-wk2/Skipped:

Aug 16, 2010:

11:48 PM Changeset in webkit [65486] by Philippe Normand

4 edits in trunk

2010-08-17 Philippe Normand <pnormand@igalia.com>

Reviewed by Martin Robinson.

[GStreamer] media/invalid-media-url-crash.html fails
https://bugs.webkit.org/show_bug.cgi?id=42960

Implemented two ResourceHandleClient callbacks that need to notify
downstream GStreamer elements of errors while accessing the
resource.

platform/graphics/gstreamer/WebKitWebSourceGStreamer.cpp: (StreamingClient::wasBlocked): (StreamingClient::cannotShowURL):

LayoutTests:

2010-08-17 Philippe Normand <pnormand@igalia.com>

Reviewed by Martin Robinson.

[GStreamer] media/invalid-media-url-crash.html fails
https://bugs.webkit.org/show_bug.cgi?id=42960

platform/gtk/Skipped: Unskip fixed test

9:27 PM Changeset in webkit [65485] by kinuko@chromium.org

2 edits in trunk/WebKit/chromium

2010-08-16 Kinuko Yasuda <kinuko@chromium.org>

Unreviewed; attempt to fix Chromium Windows build.

public/WebFileUtilities.h:

9:07 PM Changeset in webkit [65484] by jhoneycutt@apple.com

3 edits in trunk/WebKit2

Allow some tests to pass in WebKitTestRunner before SharedMemory and
PluginInfoStore are implemented on Windows.

Reviewed by Sam Weinig.

Platform/win/SharedMemoryWin.cpp:

Disable not implemented warnings.

UIProcess/Plugins/win/PluginInfoStoreWin.cpp:

Ditto.

8:59 PM Changeset in webkit [65483] by morrita@google.com

2 edits in trunk/LayoutTests

2010-08-16 Yuta Kitamura <yutak@chromium.org>

Unreviewed test expectation update.

Update Chromium test expectations.
https://bugs.webkit.org/show_bug.cgi?id=44090

platform/chromium/test_expectations.txt:

8:53 PM Changeset in webkit [65482] by kinuko@chromium.org

5 edits
1 add in trunk/WebKit/chromium

2010-08-16 Kinuko Yasuda <kinuko@chromium.org>

Reviewed by Jian Li.

[chromium] Prepare to rename WebFileSystem interface to WebFileUtilities
https://bugs.webkit.org/show_bug.cgi?id=44077

We are going to have an implementation for HTML5 FileSystem API and
it would probably be less confusing to use the name "WebFileSystem"
for an WebKit API for the HTML5 API.

This patch adds WebFileUtilities interface that is meant to replace
the existing WebFileSystem.

WebKit.gyp:
public/WebFileSystem.h:
public/WebFileUtilities.h: Added.
public/WebKitClient.h: (WebKit::WebKitClient::fileUtilities): Added.
src/ChromiumBridge.cpp: (WebCore::ChromiumBridge::fileExists): (WebCore::ChromiumBridge::deleteFile): (WebCore::ChromiumBridge::deleteEmptyDirectory): (WebCore::ChromiumBridge::getFileSize): (WebCore::ChromiumBridge::getFileModificationTime): (WebCore::ChromiumBridge::directoryName): (WebCore::ChromiumBridge::pathByAppendingComponent): (WebCore::ChromiumBridge::makeAllDirectories): (WebCore::ChromiumBridge::getAbsolutePath): (WebCore::ChromiumBridge::isDirectory): (WebCore::ChromiumBridge::filePathToURL): (WebCore::ChromiumBridge::openFile): (WebCore::ChromiumBridge::closeFile): (WebCore::ChromiumBridge::seekFile): (WebCore::ChromiumBridge::truncateFile): (WebCore::ChromiumBridge::readFromFile): (WebCore::ChromiumBridge::writeToFile):

8:42 PM Changeset in webkit [65481] by kinuko@chromium.org

2 edits in trunk/WebKit/win

2010-08-16 Kinuko Yasuda <kinuko@chromium.org>

Unreviewed; build fix attempt for Windows.

DefaultDownloadDelegate.cpp:

8:39 PM Changeset in webkit [65480] by jhoneycutt@apple.com

3 edits in trunk/LayoutTests

Add some tests that fail in WebKitTestRunner to the Skipped lists.

Rubber-stamped by Sam Weinig.

platform/mac-wk2/Skipped:
platform/win-wk2/Skipped:

7:21 PM Changeset in webkit [65479] by barraclough@apple.com

3 edits in trunk/JavaScriptCore

Add VectorTraits to String & DefaultHash traits to UString to unify behaviour.

Rubber stamped by Sam Weinig

runtime/UString.h:

(JSC::UStringHash::hash):
(JSC::UStringHash::equal):
(WTF::):

wtf/text/WTFString.h:

(WTF::):

7:13 PM Changeset in webkit [65478] by barraclough@apple.com

15 edits in trunk

Remove unnecessary includes from UString.h, add new includes as necessary.

Rubber stamped by Sam Weinig

JavaScriptCore:

profiler/CallIdentifier.h:
profiler/ProfileNode.h:
runtime/DateConversion.cpp:
runtime/Identifier.h:

(JSC::IdentifierRepHash::hash):

runtime/RegExpCache.h:
runtime/RegExpKey.h:
runtime/UString.cpp:

(JSC::UString::substr):

runtime/UString.h:
wtf/text/WTFString.h:

WebCore:

dom/ScriptExecutionContext.h:
loader/appcache/ApplicationCache.cpp:

WebKit2:

WebProcess/Plugins/Netscape/NetscapePlugin.h:

6:54 PM Changeset in webkit [65477] by commit-queue@webkit.org

2 edits in trunk/LayoutTests

2010-08-16 Ademar de Souza Reis Jr <Ademar Reis>

Reviewed by Ariya Hidayat.

[Qt] REGRESSION: editing/selection/shrink-selection-after-shift-pagedown.html fails after 56567
https://bugs.webkit.org/show_bug.cgi?id=36628

Just enabling the test, it's not failing anymore.

platform/qt/Skipped:

6:19 PM Changeset in webkit [65476] by commit-queue@webkit.org

2 edits in trunk/LayoutTests

2010-08-16 Ademar de Souza Reis Jr <Ademar Reis>

Reviewed by Ariya Hidayat.

[Qt] qt/Skipped needs cleanup on outdaded comments
https://bugs.webkit.org/show_bug.cgi?id=44066

Minor improvements/fixes to the list of Skipped Qt tests

move bug reference to the right place (#35145)
remove outdated bug reference (#32922)
move inspector testcase to the right place
add reference to bug closed on qt-4.7 (QTBUG-9618)

platform/qt/Skipped: cleanups only, no logical changes

6:09 PM Changeset in webkit [65475] by andreas.kling@nokia.com

3 edits in trunk

2010-08-16 Andreas Kling <andreas.kling@nokia.com>

Reviewed by Darin Adler.

CSS: Add fast-path for rgba() color parsing
https://bugs.webkit.org/show_bug.cgi?id=42965

Test: fast/canvas/rgba-parsing.html

css/CSSParser.cpp: (WebCore::CSSParser::parseColor): Extended with support for rgba(). (WebCore::parseAlphaValue): Added, parses an alpha value using WTF::strtod() (if necessary) and clamps between 0 and 1. (WebCore::isTenthAlpha):

2010-08-16 Andreas Kling <andreas.kling@nokia.com>

Reviewed by Darin Adler.

CSS: Add fast-path for rgba() color parsing
https://bugs.webkit.org/show_bug.cgi?id=42965

Add a test of rgba() color parsing.

fast/canvas/rgba-parsing-expected.txt: Added.
fast/canvas/rgba-parsing.html: Added.
fast/canvas/script-tests/rgba-parsing.js: Added.

6:07 PM Changeset in webkit [65474] by kinuko@chromium.org

16 edits
14 adds in trunk/WebCore

2010-08-16 Kinuko Yasuda <kinuko@chromium.org>

Reviewed by Dumitru Daniliuc.

Add idl and mock classes for FileSystem API's {File,Directory}Entry and related interfaces
https://bugs.webkit.org/show_bug.cgi?id=43993

Adding DirectoryEntry, DirectoryReader, EntriesCallback and FileEntry as defined in HTML5 FileSystem API.
http://dev.w3.org/2009/dap/file-system/file-dir-sys.html

Also adding Makefile entries for idl and mock classes added in changeset 64414 (except for Android platforms).

Tests will be added when we add underlying implementation.

CMakeLists.txt:
DerivedSources.cpp:
DerivedSources.make:
GNUmakefile.am:
WebCore.gypi:
WebCore.pri:
WebCore.pro:
WebCore.vcproj/WebCore.vcproj:
WebCore.xcodeproj/project.pbxproj:
storage/DOMFileSystem.cpp: (WebCore::DOMFileSystem::root):
storage/DOMFileSystem.h:
storage/DOMFileSystem.idl:
storage/DirectoryEntry.cpp: Added.
storage/DirectoryEntry.h: Added.
storage/DirectoryEntry.idl: Added.
storage/DirectoryReader.cpp: Added.
storage/DirectoryReader.h: Added.
storage/DirectoryReader.idl: Added.
storage/EntriesCallback.h: Added.
storage/EntriesCallback.idl: Added.
storage/EntryArray.cpp: Added.
storage/EntryArray.h: Added.
storage/EntryArray.idl: Added.
storage/Entry.cpp: (WebCore::Entry::Entry): (WebCore::Entry::getMetadata): (WebCore::Entry::moveTo): (WebCore::Entry::copyTo): (WebCore::Entry::remove): (WebCore::Entry::getParent):
storage/Entry.h: (WebCore::Entry::isFile): (WebCore::Entry::isDirectory):
storage/Entry.idl:
storage/FileEntry.cpp: Added.
storage/FileEntry.h: Added.
storage/FileEntry.idl: Added.

5:46 PM Changeset in webkit [65473] by weinig@apple.com

3 edits in trunk/WebKitTools

Add WebKit2 Mac tester to buildbot.

Reviewed by Mark Rowe.

BuildSlaveSupport/build.webkit.org-config/config.json:
BuildSlaveSupport/build.webkit.org-config/master.cfg:

5:41 PM Changeset in webkit [65472] by andreas.kling@nokia.com

4 edits
3 adds in trunk

2010-08-16 Andreas Kling <andreas.kling@nokia.com>

Reviewed by Ariya Hidayat.

[Qt] Path::closeSubpath() should only close the last subpath if it has >1 point
https://bugs.webkit.org/show_bug.cgi?id=44061

Spec link:
http://www.whatwg.org/specs/web-apps/current-work/#dom-context-2d-closepath

Test: fast/canvas/canvas-closePath-single-point.html

platform/graphics/Path.h: Add a Qt-only member to track the last subpath.
platform/graphics/qt/PathQt.cpp: (WebCore::Path::closeSubpath): Only close the last subpath if it has more than 1 point. Otherwise behave as moveTo(first point in last subpath) (WebCore::Path::Path): (WebCore::Path::operator=): (WebCore::Path::moveTo): (WebCore::Path::transform):

2010-08-16 Andreas Kling <andreas.kling@nokia.com>

Reviewed by Ariya Hidayat.

[Qt] Path::closeSubpath() should only close the last subpath if it has >1 point
https://bugs.webkit.org/show_bug.cgi?id=44061

Add a test to verify behavior of closePath() when path has only 1 point.

fast/canvas/canvas-closePath-single-point-expected.txt: Added.
fast/canvas/canvas-closePath-single-point.html: Added.
fast/canvas/script-tests/canvas-closePath-single-point.js: Added.

5:20 PM Changeset in webkit [65471] by Nate Chapin

2 edits in trunk/WebKit/chromium

2010-08-16 Nate Chapin <Nate Chapin>

Unreviewed, build fix.

Don't include TilingDataTest.cpp, since it doesn't
compile on mac.

WebKit.gyp:

4:57 PM Changeset in webkit [65470] by Nate Chapin

3 edits in trunk

2010-08-16 Nate Chapin <Nate Chapin>

Unreviewed, Chromium expectations update.

Mark 3 tests failing on Chromium Win/Linux after r65449.
They appear to have same root cause as
bugs.webkit.org/show_bug.cgi?id=43912.

platform/chromium/test_expectations.txt:

4:37 PM Changeset in webkit [65469] by Nate Chapin

2 edits in trunk/WebCore

2010-08-16 Nate Chapin <Nate Chapin>

Unreviewed, Chromium mac build fix (with help from jamesr).

platform/chromium/ScrollbarThemeChromiumMac.mm: (WebCore::ScrollbarThemeChromiumMac::paint):

4:31 PM Changeset in webkit [65468] by barraclough@apple.com

58 edits in trunk

Bug 44080 - String find/reverseFind methods need tidying up
These methods have a couple of problems with their interface, and implementation.

Reviewed by Sam Weinig

These methods take and int index, and return an int - however this is problematic
since on 64-bit string indices may have a full 32-bit range. This spills out into
surrounding code, which unsafely casts string indices from unsigned to int. Code
checking the result of these methods check for a mix of "== -1", "< 0", and
"== notFound". Clean this up by changing these methods to take an unsigned
starting index, and return a size_t. with a failed match indicated by notFound.
reverseFind also has a special meaning for the starting index argument, in that a
negative index is interpreted as an offset back from the end of the string. Remove
this functionality, in the (1!) case where it is used we should just calculate the
offset by subtracting from the string's length.

The implementation has a few problems too. The code is not in webkit style, in
using assorted abbreviations in variable names, and implementations of similar
find methods with differing argument types were unnecessarily inconsistent. When
find is passed const char* data the string would be handled as latin1 (zero
extended to UTF-16) for all characters but the first; this is sign extended.
Case-insensitive find is broken for unicode strings; the hashing optimization is
not unicode safe, and could result in false negatives.

Unify UString find methods to match String.

JavaScriptCore:

JavaScriptCore.exp:
bytecode/CodeBlock.cpp:

(JSC::escapeQuotes):

bytecompiler/NodesCodegen.cpp:

(JSC::substitute):

runtime/JSString.cpp:

(JSC::JSString::replaceCharacter):

runtime/RegExp.cpp:

(JSC::RegExp::RegExp):

runtime/RegExpKey.h:

(JSC::RegExpKey::getFlagsValue):

runtime/StringPrototype.cpp:

(JSC::substituteBackreferencesSlow):
(JSC::substituteBackreferences):
(JSC::stringProtoFuncReplace):
(JSC::stringProtoFuncIndexOf):
(JSC::stringProtoFuncLastIndexOf):
(JSC::stringProtoFuncSplit):

runtime/UString.cpp:
runtime/UString.h:

(JSC::UString::find):
(JSC::UString::reverseFind):

wtf/text/AtomicString.h:

(WTF::AtomicString::find):

wtf/text/StringImpl.cpp:

(WTF::StringImpl::find):
(WTF::StringImpl::findCaseInsensitive):
(WTF::StringImpl::reverseFind):
(WTF::StringImpl::reverseFindCaseInsensitive):
(WTF::StringImpl::endsWith):
(WTF::StringImpl::replace):

wtf/text/StringImpl.h:

(WTF::StringImpl::startsWith):

wtf/text/WTFString.cpp:

(WTF::String::split):

wtf/text/WTFString.h:

(WTF::String::find):
(WTF::String::reverseFind):
(WTF::String::findCaseInsensitive):
(WTF::String::reverseFindCaseInsensitive):
(WTF::String::contains):
(WTF::find):
(WTF::reverseFind):

WebCore:

css/CSSSelector.cpp:

(WebCore::CSSSelector::RareData::parseNth):

css/CSSStyleDeclaration.cpp:

(WebCore::CSSStyleDeclaration::setProperty):

css/CSSStyleSelector.cpp:

(WebCore::CSSStyleSelector::SelectorChecker::checkOneSelector):

dom/Document.cpp:

(WebCore::Document::getImageMap):

editing/CompositeEditCommand.cpp:

(WebCore::CompositeEditCommand::inputText):

editing/InsertTextCommand.cpp:

(WebCore::InsertTextCommand::input):

editing/TextIterator.cpp:

(WebCore::TextIterator::handleTextBox):

editing/TypingCommand.cpp:

(WebCore::TypingCommand::insertText):

editing/markup.cpp:

(WebCore::fillContainerFromString):
(WebCore::createFragmentFromText):

html/File.cpp:

(WebCore::File::Init):

html/HTMLAnchorElement.cpp:

(WebCore::HTMLAnchorElement::setHost):

html/HTMLEmbedElement.cpp:

(WebCore::HTMLEmbedElement::parseMappedAttribute):

html/HTMLFormControlElement.cpp:

(WebCore::HTMLTextFormControlElement::isPlaceholderEmpty):

html/HTMLObjectElement.cpp:

(WebCore::HTMLObjectElement::parseMappedAttribute):

inspector/InspectorDOMAgent.cpp:

(WebCore::InspectorDOMAgent::performSearch):

loader/CrossOriginPreflightResultCache.cpp:

(WebCore::parseAccessControlAllowList):

loader/MainResourceLoader.cpp:

(WebCore::MainResourceLoader::substituteMIMETypeFromPluginDatabase):

loader/appcache/ApplicationCacheStorage.cpp:

(WebCore::parseHeader):
(WebCore::parseHeaders):

loader/icon/IconFetcher.cpp:

(WebCore::parseIconLink):

page/DOMWindow.cpp:

(WebCore::DOMWindow::parseModalDialogFeatures):

page/SecurityOrigin.cpp:

(WebCore::SecurityOrigin::createFromDatabaseIdentifier):

page/UserContentURLPattern.cpp:

(WebCore::UserContentURLPattern::parse):

page/XSSAuditor.cpp:

(WebCore::XSSAuditor::findInRequest):

platform/ContentType.cpp:

(WebCore::ContentType::parameter):
(WebCore::ContentType::type):

platform/KURL.cpp:

(WebCore::KURL::lastPathComponent):
(WebCore::KURL::setProtocol):
(WebCore::decodeURLEscapeSequences):
(WebCore::substituteBackslashes):
(WebCore::mimeTypeFromDataURL):

platform/Length.cpp:

(WebCore::newCoordsArray):
(WebCore::newLengthArray):

platform/LinkHash.cpp:

(WebCore::findSlashDotDotSlash):
(WebCore::findSlashSlash):
(WebCore::findSlashDotSlash):
(WebCore::cleanPath):

platform/MIMETypeRegistry.cpp:

(WebCore::MIMETypeRegistry::getMIMETypeForPath):

platform/SchemeRegistry.cpp:

(WebCore::SchemeRegistry::shouldTreatURLAsLocal):

platform/graphics/MediaPlayer.cpp:

(WebCore::MediaPlayer::load):

platform/mac/DragImageMac.mm:

(WebCore::createDragImageIconForCachedImage):

platform/network/CredentialStorage.cpp:

(WebCore::protectionSpaceMapKeyFromURL):
(WebCore::findDefaultProtectionSpaceForURL):

platform/network/HTTPParsers.cpp:

(WebCore::skipWhiteSpace):
(WebCore::skipToken):
(WebCore::parseHTTPRefresh):
(WebCore::filenameFromHTTPContentDisposition):
(WebCore::findCharsetInMediaType):
(WebCore::parseXSSProtectionHeader):
(WebCore::extractReasonPhraseFromHTTPStatusLine):

platform/network/ResourceResponseBase.cpp:

(WebCore::ResourceResponseBase::isAttachment):
(WebCore::parseCacheHeader):

rendering/RenderEmbeddedObject.cpp:

(WebCore::RenderEmbeddedObject::updateWidget):

storage/Entry.cpp:

(WebCore::Entry::Entry):

svg/SVGFont.cpp:

(WebCore::isCompatibleGlyph):

svg/SVGURIReference.cpp:

(WebCore::SVGURIReference::getTarget):

svg/animation/SVGSMILElement.cpp:

(WebCore::SVGSMILElement::parseClockValue):
(WebCore::SVGSMILElement::parseCondition):

xml/XPathFunctions.cpp:

(WebCore::XPath::FunSubstringBefore::evaluate):
(WebCore::XPath::FunSubstringAfter::evaluate):
(WebCore::XPath::FunTranslate::evaluate):
(WebCore::XPath::FunLang::evaluate):

xml/XPathParser.cpp:

(WebCore::XPath::Parser::expandQName):

4:23 PM Changeset in webkit [65467] by rniwa@webkit.org

4 edits
2 adds in trunk

2010-08-16 Ryosuke Niwa <rniwa@webkit.org>

Reviewed by Tony Chang.

REGRESSION(r65208): editing/pasteboard/smart-paste-003.html and smart-paste-004.html crash on Windows Chromium
https://bugs.webkit.org/show_bug.cgi?id=44070

The crash was caused by start or end node being removed in pushDownInlineStyleAroundNode.
It made start and end orphaned, and caused s.node() to be detached from the document.
Fixed the crash by using nodes passed to pushDownInlineStyleAroundNode for start and end if the original
start and end are orphaned because pushDownInlineStyleAroundNode won't prune targetNode.

Test: editing/pasteboard/smart-paste-003-trailing-whitespace.html

editing/ApplyStyleCommand.cpp: (WebCore::ApplyStyleCommand::removeInlineStyle):

2010-08-16 Ryosuke Niwa <rniwa@webkit.org>

Reviewed by Tony Chang.

REGRESSION(r65208): editing/pasteboard/smart-paste-003.html and smart-paste-004.html crash on Windows Chromium
https://bugs.webkit.org/show_bug.cgi?id=44070

The bug only reproduced on Windows when trailing space is selected only on Windows.
Added a crash test to copy & paste a word with trailing space.

editing/pasteboard/smart-paste-003-trailing-whitespace-expected.txt: Added.
editing/pasteboard/smart-paste-003-trailing-whitespace.html: Added.
platform/chromium/test_expectations.txt: editing/pasteboard/smart-paste-003.html and smart-paste-004.html should not crash.

4:06 PM Changeset in webkit [65466] by weinig@apple.com

2 edits in trunk/WebKit2

Fix windows build.

Shared/win/CommandLineWin.cpp:

4:00 PM Changeset in webkit [65465] by weinig@apple.com

46 edits in trunk/WebKit2

Remove #includes of <WebCore/PlatformString.h> in favor of <wtf/text/WTFString.h>

Reviewed by Gavin Barraclough.

Platform/Module.cpp:
Platform/Module.h:
Shared/CommandLine.h:
Shared/WebCoreArgumentCoders.h:
Shared/WebEvent.h:
Shared/WebNavigationDataStore.h:
Shared/WebPreferencesStore.h:
Shared/WebString.h:
Shared/WebURL.h:
Shared/mac/CommandLineMac.cpp:
Shared/qt/UpdateChunk.cpp:
UIProcess/API/C/WKContext.cpp:
UIProcess/API/C/cf/WKStringCF.cpp:
UIProcess/API/C/cf/WKURLCF.cpp:
UIProcess/API/cpp/qt/WKStringQt.cpp:
UIProcess/API/cpp/qt/WKURLQt.cpp:
UIProcess/API/mac/PageClientImpl.mm:
UIProcess/API/qt/qgraphicswkview.cpp:
UIProcess/Launcher/mac/ProcessLauncherMac.mm:
UIProcess/Launcher/qt/ProcessLauncherQt.cpp:
UIProcess/Launcher/win/ProcessLauncherWin.cpp:
UIProcess/WebBackForwardListItem.cpp:
UIProcess/WebBackForwardListItem.h:
UIProcess/WebContext.h:
UIProcess/WebContextInjectedBundleClient.cpp:
UIProcess/WebFrameProxy.cpp:
UIProcess/WebFrameProxy.h:
UIProcess/WebPageProxy.h:
UIProcess/WebPolicyClient.cpp:
UIProcess/WebProcessProxy.cpp:
UIProcess/WebUIClient.cpp:
UIProcess/win/WebView.cpp:
WebProcess/InjectedBundle/API/c/WKBundlePage.cpp:
WebProcess/InjectedBundle/InjectedBundle.h:
WebProcess/InjectedBundle/InjectedBundlePageEditorClient.cpp:
WebProcess/InjectedBundle/InjectedBundlePageLoaderClient.cpp:
WebProcess/InjectedBundle/InjectedBundlePageUIClient.cpp:
WebProcess/InjectedBundle/mac/InjectedBundleMac.cpp:
WebProcess/Plugins/JSNPObject.cpp:
WebProcess/Plugins/NPJSObject.cpp:
WebProcess/Plugins/Netscape/NetscapePluginModule.cpp:
WebProcess/Plugins/Netscape/NetscapePluginModule.h:
WebProcess/WebCoreSupport/WebChromeClient.h:
WebProcess/WebKitMain.cpp:
WebProcess/win/WebProcessMainWin.cpp:

3:54 PM Changeset in webkit [65464] by Nate Chapin

2 edits
1 add in trunk/LayoutTests

2010-08-16 Nate Chapin <Nate Chapin>

Chromium win rebaseline + test expectations tweaks.

platform/chromium-win/svg/custom/absolute-sized-content-with-resources-expected.txt: Added.
platform/chromium/test_expectations.txt:

3:48 PM Changeset in webkit [65463] by jamesr@google.com

4 edits in trunk

fix build

3:35 PM Changeset in webkit [65462] by hyatt@apple.com

1 edit in trunk/WebCore/platform/graphics/GraphicsContext3D.h

Fix Chromium/Qt bustage.

3:32 PM Changeset in webkit [65461] by hyatt@apple.com

1 edit in trunk/WebKit/chromium/src/GraphicsContext3D.cpp

Fix Chromium Bustage.

3:30 PM Changeset in webkit [65460] by hyatt@apple.com

2 edits in trunk

Fix Chromium Bustage.

3:22 PM Changeset in webkit [65459] by weinig@apple.com

2 edits in trunk/WebKit2

Null CFURLRef returned from provisionalURL after searching on zillow.com
<rdar://problem/8261812>
https://bugs.webkit.org/show_bug.cgi?id=44072

Reviewed by Gavin Barraclough.

Conversion from WKURLRef to CFRURLRef was breaking on URLs using characters
such as '{'.

UIProcess/API/C/cf/WKURLCF.cpp:

(WKURLCopyCFURL): Use code similar to that found in KURLCFNet.cpp for conversion.

2:48 PM Changeset in webkit [65458] by hyatt@apple.com

3 edits in trunk/WebCore/platform/graphics

Fix Qt bustage.

2:43 PM Changeset in webkit [65457] by weinig@apple.com

3 edits in trunk/WebKit2

Remove unnecessary copy constructor, assignment operator and swap function.
The defaults (for all except swap) should do just fine.

Reviewed by Gavin Barraclough.

Shared/WebPreferencesStore.cpp:
Shared/WebPreferencesStore.h:

2:37 PM Changeset in webkit [65456] by hyatt@apple.com

1 edit in trunk/WebCore/platform/graphics/qt/ImageBufferQt.cpp

Fix Qt build bustage.

2:36 PM Changeset in webkit [65455] by commit-queue@webkit.org

10 edits
3 adds in trunk

2010-08-16 Vincent Scheib <scheib@chromium.org>

Reviewed by Dimitri Glazkov.

Canvas2D does not support images larger than system's GPU max texture size
https://bugs.webkit.org/show_bug.cgi?id=43864

A new class "TilingData" has been created that provides logic for
splitting a large image into a series of smaller tiles.

GLES2Texture and CLES2Canvas are modified to use the TilingData to
split images on texture upload and render one logical image as many smaller
images.

TilingData is tested via unit tests in WebKit/chromium/tests/TilingDataTest.cpp

WebCore.gypi:
platform/graphics/chromium/GLES2Canvas.cpp: (WebCore::GLES2Canvas::drawTexturedRect): (WebCore::GLES2Canvas::drawTexturedRectTile):
platform/graphics/chromium/GLES2Canvas.h:
platform/graphics/chromium/GLES2Texture.cpp: (WebCore::GLES2Texture::GLES2Texture): (WebCore::GLES2Texture::~GLES2Texture): (WebCore::GLES2Texture::create): (WebCore::copySubRect): (WebCore::GLES2Texture::load): (WebCore::GLES2Texture::bindTile):
platform/graphics/chromium/GLES2Texture.h: (WebCore::GLES2Texture::tiles):
platform/graphics/chromium/TilingData.cpp: Added. (WebCore::computeNumTiles): (WebCore::TilingData::TilingData): (WebCore::TilingData::tileXIndexFromSrcCoord): (WebCore::TilingData::tileYIndexFromSrcCoord): (WebCore::TilingData::tileBounds): (WebCore::TilingData::tileBoundsWithBorder): (WebCore::TilingData::tileBoundsNormalized): (WebCore::TilingData::tilePositionX): (WebCore::TilingData::tilePositionY): (WebCore::TilingData::tileSizeX): (WebCore::TilingData::tileSizeY): (WebCore::TilingData::overlappedTileIndices): (WebCore::TilingData::intersectDrawQuad):
platform/graphics/chromium/TilingData.h: Added. (WebCore::TilingData::maxTextureSize): (WebCore::TilingData::totalSizeX): (WebCore::TilingData::totalSizeY): (WebCore::TilingData::numTiles): (WebCore::TilingData::numTilesX): (WebCore::TilingData::numTilesY): (WebCore::TilingData::tileIndex): (WebCore::TilingData::tileXIndex): (WebCore::TilingData::tileYIndex): (WebCore::TilingData::TilingData): (WebCore::TilingData::assertTile):
platform/graphics/skia/ImageSkia.cpp: (WebCore::BitmapImage::draw):
platform/graphics/skia/PlatformContextSkia.cpp: (WebCore::PlatformContextSkia::uploadSoftwareToHardware):

2010-08-16 Vincent Scheib <scheib@chromium.org>

Reviewed by Dimitri Glazkov.

Canvas2D does not support images larger than system's GPU max texture size
https://bugs.webkit.org/show_bug.cgi?id=43864

Unit tests for TilingData class.

WebKit.gyp:
tests/TilingDataTest.cpp: Added. (WebCore::TEST):

2:34 PM Changeset in webkit [65454] by hyatt@apple.com

1 edit in trunk/WebCore/platform/graphics/cairo/ImageCairo.cpp

Fix GTK build bustage.

2:31 PM Changeset in webkit [65453] by hyatt@apple.com

1 edit in trunk/WebCore/platform/graphics/cairo/ImageBufferCairo.cpp

Fix GTK build bustage.

2:28 PM Changeset in webkit [65452] by hyatt@apple.com

1 edit in trunk/WebCore/platform/graphics/qt/ImageBufferQt.cpp

Fix Qt build bustage.

2:26 PM Changeset in webkit [65451] by hyatt@apple.com

1 edit in trunk/WebCore/platform/graphics/cairo/GraphicsContextCairo.cpp

Fix GTK build bustage.

2:21 PM Changeset in webkit [65450] by cmarrin@apple.com

1 edit in trunk/WebCore/platform/graphics/GraphicsContext3D.h

Put ANGLE include inside PLATFORM(MAC) to fix Chrome and Qt builds

1:44 PM Changeset in webkit [65449] by hyatt@apple.com

49 edits in trunk/WebCore

https://bugs.webkit.org/show_bug.cgi?id=43507, stop ImageBuffer from copying its data when rendering after canvas changes happen.

Reviewed by Anders Carlsson.

This patch renames ImageBuffer::image() to ImageBuffer::copyImage(). The new method always returns a new copy that is a current
snapshot of the ImageBuffer.

To draw an ImageBuffer, you now use new GraphicsContext calls: drawImageBuffer. Platforms can then optimize these calls to draw
efficiently without having to copy the bits of the ImageBuffer into an Image.

WebCore.xcodeproj/project.pbxproj:
css/CSSCanvasValue.cpp:

(WebCore::CSSCanvasValue::image):

html/HTMLCanvasElement.cpp:

(WebCore::HTMLCanvasElement::willDraw):
(WebCore::HTMLCanvasElement::reset):
(WebCore::HTMLCanvasElement::paint):
(WebCore::HTMLCanvasElement::setSurfaceSize):
(WebCore::HTMLCanvasElement::copiedImage):
(WebCore::HTMLCanvasElement::clearCopiedImage):

html/HTMLCanvasElement.h:
html/canvas/CanvasPattern.cpp:

(WebCore::CanvasPattern::CanvasPattern):

html/canvas/CanvasPattern.h:

(WebCore::CanvasPattern::create):

html/canvas/CanvasRenderingContext2D.cpp:

(WebCore::CanvasRenderingContext2D::drawImage):
(WebCore::CanvasRenderingContext2D::createPattern):
(WebCore::CanvasRenderingContext2D::drawTextInternal):

html/canvas/WebGLRenderingContext.cpp:

(WebCore::WebGLRenderingContext::paintRenderingResultsToCanvas):
(WebCore::WebGLRenderingContext::reshape):
(WebCore::WebGLRenderingContext::texImage2D):
(WebCore::WebGLRenderingContext::texSubImage2D):

html/canvas/WebGLRenderingContext.h:

(WebCore::WebGLRenderingContext::paintsIntoCanvasBuffer):

platform/graphics/GeneratedImage.cpp:

(WebCore::GeneratedImage::drawPattern):

platform/graphics/GraphicsContext.cpp:

(WebCore::GraphicsContext::drawImageBuffer):
(WebCore::GraphicsContext::clipToImageBuffer):

platform/graphics/GraphicsContext.h:
platform/graphics/GraphicsContext3D.h:

(WebCore::GraphicsContext3D::paintsIntoCanvasBuffer):

platform/graphics/Image.h:
platform/graphics/ImageBuffer.h:

(WebCore::ImageBuffer::width):
(WebCore::ImageBuffer::height):

platform/graphics/Pattern.cpp:

(WebCore::Pattern::Pattern):

platform/graphics/Pattern.h:

(WebCore::Pattern::create):

platform/graphics/cairo/GraphicsContextCairo.cpp:
platform/graphics/cairo/ImageBufferCairo.cpp:

(WebCore::ImageBuffer::drawsUsingCopy):
(WebCore::ImageBuffer::copyImage):
(WebCore::ImageBuffer::clip):
(WebCore::ImageBuffer::draw):
(WebCore::ImageBuffer::drawPattern):

platform/graphics/cg/GraphicsContextCG.cpp:
platform/graphics/cg/ImageBufferCG.cpp:

(WebCore::ImageBuffer::ImageBuffer):
(WebCore::ImageBuffer::drawsUsingCopy):
(WebCore::ImageBuffer::copyImage):
(WebCore::ImageBuffer::draw):
(WebCore::ImageBuffer::drawPattern):
(WebCore::ImageBuffer::clip):

platform/graphics/cg/ImageBufferData.h:
platform/graphics/filters/FEColorMatrix.cpp:

(WebCore::FEColorMatrix::apply):

platform/graphics/filters/FEComposite.cpp:

(WebCore::FEComposite::apply):

platform/graphics/filters/SourceAlpha.cpp:

(WebCore::SourceAlpha::apply):

platform/graphics/filters/SourceGraphic.cpp:

(WebCore::SourceGraphic::apply):

platform/graphics/mac/GraphicsContext3DMac.mm:
platform/graphics/qt/GraphicsContext3DQt.cpp:
platform/graphics/qt/GraphicsContextQt.cpp:
platform/graphics/qt/ImageBufferData.h:
platform/graphics/qt/ImageBufferQt.cpp:

(WebCore::ImageBufferData::ImageBufferData):
(WebCore::ImageBuffer::drawsUsingCopy):
(WebCore::ImageBuffer::copyImage):
(WebCore::ImageBuffer::draw):
(WebCore::ImageBuffer::drawPattern):
(WebCore::ImageBuffer::clip):

platform/graphics/skia/GraphicsContextSkia.cpp:
platform/graphics/skia/ImageBufferSkia.cpp:

(WebCore::ImageBuffer::drawsUsingCopy):
(WebCore::ImageBuffer::copyImage):
(WebCore::ImageBuffer::clip):
(WebCore::):

platform/graphics/wx/GraphicsContextWx.cpp:
platform/graphics/wx/ImageBufferWx.cpp:

(WebCore::ImageBuffer::drawsUsingCopy):
(WebCore::ImageBuffer::copyImage):
(WebCore::ImageBuffer::clip):
(WebCore::ImageBuffer::draw):
(WebCore::ImageBuffer::drawPattern):

platform/mac/ScrollbarThemeMac.mm:

(WebCore::ScrollbarThemeMac::paint):

rendering/RenderBoxModelObject.cpp:

(WebCore::RenderBoxModelObject::paintFillLayerExtended):

rendering/RenderSVGResourceClipper.cpp:

(WebCore::RenderSVGResourceClipper::applyClippingToContext):

rendering/RenderSVGResourceFilter.cpp:

(WebCore::RenderSVGResourceFilter::postApplyResource):

rendering/RenderSVGResourceGradient.cpp:

(WebCore::clipToTextMask):

rendering/RenderSVGResourceMasker.cpp:

(WebCore::RenderSVGResourceMasker::applyResource):

rendering/RenderSVGResourcePattern.cpp:

(WebCore::RenderSVGResourcePattern::buildPattern):

rendering/RenderThemeMac.mm:

(WebCore::RenderThemeMac::paintProgressBar):

svg/SVGFEImageElement.cpp:

(WebCore::SVGFEImageElement::build):

svg/graphics/SVGImage.cpp:

(WebCore::SVGImage::nativeImageForCurrentFrame):

svg/graphics/SVGImage.h:
svg/graphics/filters/SVGFEMerge.cpp:

(WebCore::FEMerge::apply):

svg/graphics/filters/SVGFEOffset.cpp:

(WebCore::FEOffset::apply):

svg/graphics/filters/SVGFETile.cpp:

(WebCore::FETile::apply):

1:19 PM Changeset in webkit [65448] by cmarrin@apple.com

2 edits in trunk/WebCore

Fixed a warning in ANGLEWebKitBridge.cpp

1:15 PM Changeset in webkit [65447] by cmarrin@apple.com

1 edit in trunk/WebKitTools/Scripts/build-webkit

Build ANGLE for all versions of Mac to fix Leopard build

12:30 PM BuildingQtOnWindows edited by jocelyn.turcotte@nokia.com

python 2.6 -> python 2.x (diff)

12:18 PM Changeset in webkit [65446] by cmarrin@apple.com

19 edits
2 adds in trunk

2010-08-16 Paul Sawaya <psawaya@apple.com>

Reviewed by Chris Marrin.

Added shader validation via ANGLE
https://bugs.webkit.org/show_bug.cgi?id=42405

12:12 PM Changeset in webkit [65445] by kevino@webkit.org

13 edits
1 move in trunk

[wx] Build fix, do not build WebCore as a convenience library as this leads to
errors in the Win build w/export symbols and causes problems with DOM bindings
debugging in gdb.

12:00 PM Changeset in webkit [65444] by commit-queue@webkit.org

2 edits in trunk/LayoutTests

2010-08-16 Mihai Parparita <mihaip@chromium.org>

Reviewed by Nate Chapin.

Some http/tests/history tests now pass on Chromium after r65340
https://bugs.webkit.org/show_bug.cgi?id=44033

Remove expected failures.

platform/chromium/test_expectations.txt:

11:24 AM Changeset in webkit [65443] by loislo@chromium.org

2 edits
2 adds in trunk/WebCore

2010-08-16 Ilya Tikhonovsky <loislo@chromium.org>

Reviewed by Adam Roben.

Current implementation of WebCore post-build step on windows platform is error prone.
It would be better to extract the script into external file.
https://bugs.webkit.org/show_bug.cgi?id=44058

WebCore.vcproj/WebCore.vcproj:
WebCore.vcproj/copyForwardingHeaders.cmd: Added.
WebCore.vcproj/copyInspectorFiles.cmd: Added.

11:09 AM Changeset in webkit [65442] by kevino@webkit.org

2 edits in trunk/WebCore

[wx] Fix typo in previous build fix.

11:04 AM Changeset in webkit [65441] by commit-queue@webkit.org

5 edits in trunk

2010-08-16 Alex Milowski <alex@milowski.com>

Reviewed by David Levin.

Added the configuration of the MathML related files.

WebCore.gyp/WebCore.gyp:
WebCore.gypi:

2010-08-16 Alex Milowski <alex@milowski.com>

Reviewed by David Levin.

Added the ENABLE_MATHML=1 to the feature_defines.

features.gypi:

10:39 AM Changeset in webkit [65440] by pfeldman@chromium.org

2 edits in trunk/WebCore

2010-08-16 Pavel Feldman <pfeldman@chromium.org>

Not reviewed: fix Leopard Release inspector tests.

inspector/front-end/inspector.js:

10:30 AM Changeset in webkit [65439] by bulach@chromium.org

7 edits
3 copies
1 add in trunk

2010-08-16 Marcus Bulach <bulach@chromium.org>

Reviewed by Jeremy Orlow.

Implements IDBKeyPath extractor.
https://bugs.webkit.org/show_bug.cgi?id=43276

Unit-test IDBKeyPathExtractorTest.cpp.
LayoutTests will arrive as IndexedDB infrastructure is fleshed out.

bindings/v8/IDBBindingUtilities.cpp: (WebCore::getValueFrom): (WebCore::createIDBKeyFromSerializedValueAndKeyPath):
bindings/v8/IDBBindingUtilities.h:

2010-08-16 Marcus Bulach <bulach@chromium.org>

Reviewed by Jeremy Orlow.

Implements IDBKeyPath extractor.
https://bugs.webkit.org/show_bug.cgi?id=43276

Unit-test IDBKeyPathExtractorTest.cpp.
LayoutTests will arrive as IndexedDB infrastructure is fleshed out.

WebKit.gyp:
public/WebIDBKey.h:
public/WebIDBKeyPath.h: Added. (WebKit::WebIDBKeyPath::WebIDBKeyPath): (WebKit::WebIDBKeyPath::~WebIDBKeyPath):
public/WebPrivateOwnPtr.h: Added. (WebKit::WebPrivateOwnPtr::WebPrivateOwnPtr): (WebKit::WebPrivateOwnPtr::~WebPrivateOwnPtr): (WebKit::WebPrivateOwnPtr::reset): (WebKit::WebPrivateOwnPtr::get): (WebKit::WebPrivateOwnPtr::operator->):
src/WebIDBKey.cpp: (WebKit::WebIDBKey::createFromValueAndKeyPath):
src/WebIDBKeyPath.cpp: Added. (WebKit::WebIDBKeyPath::create): (WebKit::WebIDBKeyPath::WebIDBKeyPath): (WebKit::WebIDBKeyPath::parseError): (WebKit::WebIDBKeyPath::assign): (WebKit::WebIDBKeyPath::reset): (WebKit::WebIDBKeyPath::operator const WTF::Vector<WebCore::IDBKeyPathElement, 0>&):
tests/IDBBindingUtilitiesTest.cpp: Added. (WebCore::LocalContext::LocalContext): (WebCore::LocalContext::~LocalContext): (WebCore::checkKeyFromValueAndKeyPathInternal): (WebCore::checkKeyPathNullValue): (WebCore::checkKeyPathStringValue): (WebCore::checkKeyPathNumberValue): (WebCore::TEST):

10:10 AM Changeset in webkit [65438] by kov@webkit.org

2 edits in trunk/WebCore

2010-08-13 Gustavo Noronha Silva <gustavo.noronha@collabora.co.uk>

Reviewed by Kenneth Rohde Christiansen and Martin Robinson.

[GTK] GTK+ drawing idle starved
https://bugs.webkit.org/show_bug.cgi?id=43994

Tune down the priority of the shared timers to guarantee GTK+ has
an opportunity to redraw.

platform/gtk/SharedTimerGtk.cpp: (WebCore::setSharedTimerFireTime):

9:44 AM Changeset in webkit [65437] by leandro@webkit.org

8 edits in trunk

2010-08-16 Leandro Pereira <leandro@profusion.mobi>

[EFL] Build fix after r65366.

CMakeLists.txt: Use if (VAR) instead of if (${VAR}) to check if they're empty.
jsc/CMakeLists.txt: Ditto.
wtf/CMakeLists.txt: Ditto.

2010-08-16 Leandro Pereira <leandro@profusion.mobi>

[EFL] Build fix after r65366.

CMakeLists.txt: Use if (VAR) instead of if (${VAR}) to check if they're empty.

2010-08-16 Leandro Pereira <leandro@profusion.mobi>

[EFL] Build fix after r65366.

EFL port does not support automated tests yet.

CMakeLists.txt: Use if (VAR) instead of if (${VAR}) to check if they're empty.

9:40 AM Changeset in webkit [65436] by steveblock@google.com

2 edits in trunk/LayoutTests

Unreviewed build fix.

Add fast/dom/Geolocation/disconnected-frame-permission-denied.html to failing tests for Chromium
https://bugs.webkit.org/show_bug.cgi?id=44059

platform/chromium/test_expectations.txt:

9:35 AM Changeset in webkit [65435] by mitz@apple.com

2 edits in trunk/WebKitTools

Build fix.

WebKitTestRunner/TestController.cpp:

(WTR::createOtherPage): Initialize the contentsSizeChanged member.
(WTR::TestController::initialize): Ditto.

9:33 AM Changeset in webkit [65434] by pfeldman@chromium.org

6 edits
1 delete in trunk

2010-08-16 Pavel Feldman <pfeldman@chromium.org>

Reviewed by Yury Semikhatsky.

Web Inspector: Make InjectedScript proto-based.
https://bugs.webkit.org/show_bug.cgi?id=44028

inspector/front-end/InjectedScript.js: (injectedScriptConstructor): (injectedScriptConstructor.):

2010-08-16 Pavel Feldman <pfeldman@chromium.org>

Reviewed by Yury Semikhatsky.

Web Inspector: Make InjectedScript proto-based.
https://bugs.webkit.org/show_bug.cgi?id=44028
Drive-by test fix.

http/tests/inspector-enabled/console-log-before-frame-navigation.html:
http/tests/inspector-enabled/resources/console-log-before-frame-navigation.js: Removed.
http/tests/inspector-enabled/resources/console-log-frame-before-navigation.html:

9:31 AM Changeset in webkit [65433] by kevino@webkit.org

2 edits in trunk/WebKit/wx

wxMSW build fix. Don't build the Python DOM bindings until we get the export issues
sorted out.

9:29 AM Changeset in webkit [65432] by Csaba Osztrogonác

2 edits in trunk/WebCore

Fix warning in WebCore/plugins/win/PluginPackageWin.cpp
https://bugs.webkit.org/show_bug.cgi?id=43920

Reviewed by Adam Roben.

plugins/win/PluginPackageWin.cpp:

(WebCore::PluginPackage::fetchInfo):

9:19 AM Changeset in webkit [65431] by Csaba Osztrogonác

2 edits in trunk/LayoutTests

Unreviewed.

[Qt] fast/dom/Geolocation/disconnected-frame-permission-denied.html fails
https://bugs.webkit.org/show_bug.cgi?id=44059

platform/qt/Skipped: fast/dom/Geolocation/disconnected-frame-permission-denied.html skipped until fix.

9:15 AM Changeset in webkit [65430] by pfeldman@chromium.org

2 edits in trunk/WebCore

2010-08-16 Pavel Feldman <pfeldman@chromium.org>

Not reviewed. Touch inspector file to force frontend deployment.

inspector/InspectorController.cpp:

9:06 AM Changeset in webkit [65429] by kevino@webkit.org

2 edits in trunk/WebCore

[wx] Build fix for non-Win ports on Windows.

8:16 AM Changeset in webkit [65428] by agl@chromium.org

5 edits in trunk

2010-08-16 Adam Langley <agl@chromium.org>

Reviewed by Tony Chang.

[chromium] Improve spacing support for complex text on Linux

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

Previously, our complex text support ignored word-spacing,
justification and letter-spacing. This fixes the first two issues and
allows us to render Scribd's HTML5 documents much better.

Test: fast/text/atsui-spacing-features.html

platform/graphics/chromium/FontLinux.cpp: (WebCore::TextRunWalker::TextRunWalker): (WebCore::TextRunWalker::setWordSpacingAdjustment): (WebCore::TextRunWalker::setLetterSpacingAdjustment): (WebCore::TextRunWalker::setPadding): (WebCore::TextRunWalker::setGlyphXPositions): (WebCore::TextRunWalker::isCodepointSpace): (WebCore::Font::drawComplexText): (WebCore::Font::floatWidthForComplexText): (WebCore::Font::offsetForPositionForComplexText): (WebCore::Font::selectionRectForComplexText):

8:14 AM Changeset in webkit [65427] by Csaba Osztrogonác

5 edits in trunk/WebCore

Fix warnings: unknown conversion type character 'l' in format
https://bugs.webkit.org/show_bug.cgi?id=43359

Reviewed by Kenneth Rohde Christiansen.

loader/icon/IconDatabase.cpp:

(WebCore::IconDatabase::performURLImport): Replace %zu with %lu, because Windows doesn't understand z modifier.
(WebCore::IconDatabase::pruneUnretainedIcons): Use %I64i intsead of %lli on Windows.

platform/graphics/qt/MediaPlayerPrivatePhonon.cpp:

(WebCore::MediaPlayerPrivate::totalTimeChanged): Use %I64d intsead of %lld on Windows.

platform/sql/SQLiteDatabase.cpp:

(WebCore::SQLiteDatabase::setMaximumSize): Use %I64i intsead of %lli on Windows.

storage/DatabaseTracker.cpp:

(WebCore::DatabaseTracker::setQuota): Use %I64u intsead of %llu on Windows.

8:02 AM Changeset in webkit [65426] by Csaba Osztrogonác

2 edits in trunk/WebCore

Fix warnings: unknown conversion type character 'l' in format
https://bugs.webkit.org/show_bug.cgi?id=43359

Reviewed by Kenneth Rohde Christiansen.

loader/FTPDirectoryParser.cpp: Use %I64u format specifier instead of %llu on Windows.

(WebCore::parseOneFTPLine):

7:32 AM Changeset in webkit [65425] by ariya@webkit.org

6 edits
2 adds in trunk

[Qt] Shadow blur for rectangle fill
https://bugs.webkit.org/show_bug.cgi?id=44025

Patch by Ariya Hidayat <ariya@sencha.com> on 2010-08-16
Reviewed by Kenneth Rohde Christiansen.

WebCore:

The blur implementation follows the approximation of Gaussian blur
with three successive box blurs, working on the alpha channel only.
(see http://www.w3.org/TR/SVG/filters.html#feGaussianBlur).

WebCore.pro: Refer to ContexShadow.* files.
platform/graphics/qt/ContextShadow.cpp: Added.

(WebCore::ContextShadow::ContextShadow):
(WebCore::ContextShadow::clear):
(WebCore::blurHorizontal): Box blurs in horizontal direction, working
on QImage (in-place).
(WebCore::shadowBlur): Main blur function which just calls the
above blurHorizontal function twice, once with transposed image
(equivalent as it would have done vertical box blurs).
(WebCore::ContextShadow::drawShadowRect):

platform/graphics/qt/ContextShadow.h: Added.

(WebCore::ContextShadow::):

platform/graphics/qt/GraphicsContextQt.cpp:

(WebCore::GraphicsContextPlatformPrivate::hasShadow):
(WebCore::GraphicsContext::restorePlatformState):
(WebCore::GraphicsContext::fillRect): Use ContextShadow for the shadow.
(WebCore::GraphicsContext::setPlatformShadow):

LayoutTests:

Update pixel test reference after the blur support.

platform/qt/fast/box-shadow/basic-shadows-expected.checksum:
platform/qt/fast/box-shadow/basic-shadows-expected.png:

7:09 AM Changeset in webkit [65424] by abecsi@webkit.org

2 edits in trunk/WebCore

2010-08-16 Andras Becsi <abecsi@webkit.org>

Rubber-stamped by Gustavo Noronha Silva.

[GTK] Build fix for MathML code generation.

No new tests needed.

GNUmakefile.am:

6:58 AM Changeset in webkit [65423] by commit-queue@webkit.org

2 edits in trunk/WebCore

2010-08-16 Sheriff Bot <webkit.review.bot@gmail.com>

Unreviewed, rolling out r65414.
http://trac.webkit.org/changeset/65414
https://bugs.webkit.org/show_bug.cgi?id=44055

GTL & QT tests failing. (Requested by pfeldman on #webkit).

inspector/front-end/InjectedScript.js: (injectedScriptConstructor): (injectedScriptConstructor.):

6:39 AM Changeset in webkit [65422] by abecsi@webkit.org

12 edits in trunk

2010-08-16 Balazs Kelemen <kb@inf.u-szeged.hu>

Reviewed by Kenneth Rohde Christiansen.

Handle content size change in WebKit2
Re-landing after fix.

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

MiniBrowser/mac/BrowserWindowController.m: (-[BrowserWindowController awakeFromNib]): Initialize WKPageUICallback::contetsSizeChanged to 0.
MiniBrowser/win/BrowserView.cpp: (BrowserView::create): Initialize WKPageUICallback::contetsSizeChanged to 0.

2010-08-16 Balazs Kelemen <kb@inf.u-szeged.hu>

Reviewed by Kenneth Rohde Christiansen.

Handle content size change in WebKit2
Re-landing after fix.

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

Based on the work of Antti Koivisto.
Send message to the UI client when the contents size has changed through the WebChromeClient
and propagate it to the WKPageUIClient.

Shared/CoreIPCSupport/WebPageProxyMessageKinds.h: (WebPageProxyMessage::): Added ContentsSizeChanged message kind.
UIProcess/API/C/WKPage.h: Added WKPageContentsSizeChangedCallback callback to the WKPageUIClient.
UIProcess/API/qt/qwkpage.cpp: (QWKPage::QWKPage): Initialize the new callback to 0.
UIProcess/WebPageProxy.cpp: (WebKit::WebPageProxy::didReceiveMessage): Handle ContentsSizeChanged message. Calls contetsSizeChanged. (WebKit::WebPageProxy::contentsSizeChanged): Added. Propagate the event to the the UI client.
UIProcess/WebPageProxy.h:
UIProcess/WebUIClient.cpp: (WebKit::WebUIClient::contentsSizeChanged): Added. Propagate the event forward to the WKPageUIClient.
UIProcess/WebUIClient.h:
WebProcess/WebCoreSupport/WebChromeClient.cpp: (WebKit::WebChromeClient::contentsSizeChanged): Implemented.

6:28 AM Changeset in webkit [65421] by commit-queue@webkit.org

12 edits in trunk

2010-08-16 Sheriff Bot <webkit.review.bot@gmail.com>

Unreviewed, rolling out r65419.
http://trac.webkit.org/changeset/65419
https://bugs.webkit.org/show_bug.cgi?id=44053

Broke the Windows build (Requested by bbandix on #webkit).

Shared/CoreIPCSupport/WebPageProxyMessageKinds.h: (WebPageProxyMessage::):
UIProcess/API/C/WKPage.h:
UIProcess/API/qt/qwkpage.cpp: (QWKPage::QWKPage):
UIProcess/WebPageProxy.cpp: (WebKit::WebPageProxy::didReceiveMessage):
UIProcess/WebPageProxy.h:
UIProcess/WebUIClient.cpp:
UIProcess/WebUIClient.h:
WebProcess/WebCoreSupport/WebChromeClient.cpp: (WebKit::WebChromeClient::contentsSizeChanged):

2010-08-16 Sheriff Bot <webkit.review.bot@gmail.com>

Unreviewed, rolling out r65419.
http://trac.webkit.org/changeset/65419
https://bugs.webkit.org/show_bug.cgi?id=44053

Broke the Windows build (Requested by bbandix on #webkit).

MiniBrowser/mac/BrowserWindowController.m: (-[BrowserWindowController awakeFromNib]):
MiniBrowser/win/BrowserView.cpp: (BrowserView::create):

5:58 AM Changeset in webkit [65420] by ariya@webkit.org

2 edits in trunk/WebCore

2010-08-16 Ariya Hidayat <ariya@sencha.com>

Reviewed by Antonio Gomes.

[Qt] Save and restore shadow state in GraphicsContextQt
https://bugs.webkit.org/show_bug.cgi?id=44031

This is needed after r65362.
Use a new class ContextShadowParameter to track and keep the shadow
state. We use this rather than using GraphicsContextState to allow
possible optimizations (right now only to determine the shadow type,
but in future it might cover things like cached scratch image,
persistent shader, etc).

platform/graphics/qt/GraphicsContextQt.cpp: (WebCore::ContextShadowParameter::): (WebCore::ContextShadowParameter::ContextShadowParameter): (WebCore::ContextShadowParameter::clear): (WebCore::GraphicsContextPlatformPrivate::hasShadow): (WebCore::GraphicsContextPlatformPrivate::GraphicsContextPlatformPrivate): (WebCore::GraphicsContext::savePlatformState): (WebCore::GraphicsContext::restorePlatformState): (WebCore::GraphicsContext::drawRect): (WebCore::GraphicsContext::drawLine): (WebCore::GraphicsContext::strokeArc): (WebCore::GraphicsContext::drawConvexPolygon): (WebCore::GraphicsContext::fillPath): (WebCore::GraphicsContext::strokePath): (WebCore::GraphicsContext::fillRect): (WebCore::GraphicsContext::fillRoundedRect): (WebCore::GraphicsContext::setPlatformShadow): (WebCore::GraphicsContext::clearPlatformShadow):

5:52 AM Changeset in webkit [65419] by abecsi@webkit.org

12 edits in trunk

2010-08-16 Balazs Kelemen <kb@inf.u-szeged.hu>

Reviewed by Kenneth Rohde Christiansen.

Handle content size change in WebKit2

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

MiniBrowser/mac/BrowserWindowController.m: (-[BrowserWindowController awakeFromNib]): Initialize WKPageUICallback::contetsSizeChanged to 0.
MiniBrowser/win/BrowserView.cpp: (BrowserView::create): Initialize WKPageUICallback::contetsSizeChanged to 0.

2010-08-16 Balazs Kelemen <kb@inf.u-szeged.hu>

Reviewed by Kenneth Rohde Christiansen.

Handle content size change in WebKit2

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

Based on the work of Antti Koivisto.
Send message to the UI client when the contents size has changed through the WebChromeClient
and propagate it to the WKPageUIClient.

Shared/CoreIPCSupport/WebPageProxyMessageKinds.h: (WebPageProxyMessage::): Added ContentsSizeChanged message kind.
UIProcess/API/C/WKPage.h: Added WKPageContentsSizeChangedCallback callback to the WKPageUIClient.
UIProcess/API/qt/qwkpage.cpp: (QWKPage::QWKPage): Initialize the new callback to 0.
UIProcess/WebPageProxy.cpp: (WebKit::WebPageProxy::didReceiveMessage): Handle ContentsSizeChanged message. Calls contetsSizeChanged. (WebKit::WebPageProxy::contentsSizeChanged): Added. Propagate the event to the the UI client.
UIProcess/WebPageProxy.h:
UIProcess/WebUIClient.cpp: (WebKit::WebUIClient::contentsSizeChanged): Added. Propagate the event forward to the WKPageUIClient.
UIProcess/WebUIClient.h:
WebProcess/WebCoreSupport/WebChromeClient.cpp: (WebKit::WebChromeClient::contentsSizeChanged): Implemented.

5:44 AM Changeset in webkit [65418] by ariya@webkit.org

2 edits in trunk/WebKitTools

Add my new email address to committers.py.

Scripts/webkitpy/common/config/committers.py:

5:43 AM Changeset in webkit [65417] by bulach@chromium.org

4 edits
3 adds in trunk

2010-07-28 Marcus Bulach <bulach@chromium.org>

Reviewed by Adam Barth.

Implement IDBKeyPath parser.
https://bugs.webkit.org/show_bug.cgi?id=42976

IDBKeyPath is an internal class, exposed only for unit tests for now.
It'll allow indexed database implementation to traverse a given key path and obtain the corresponding properties
from a SerializedScriptValue.

WebCore.gypi:
storage/IDBKeyPath.cpp: Added. (WebCore::IDBKeyPath::create): (WebCore::IDBKeyPath::IDBKeyPath): (WebCore::IDBKeyPath::~IDBKeyPath): (WebCore::IDBKeyPath::hasNext): (WebCore::IDBKeyPath::next): (WebCore::IDBKeyPath::identifier): (WebCore::IDBKeyPath::hasIndex): (WebCore::IDBKeyPath::indexValue): (WebCore::IDBKeyPath::parserError): (WebCore::IDBKeyPath::parse): (WebCore::IDBKeyPath::Lexer::lex): (WebCore::isSafeIdentifierStartCharacter): (WebCore::isSafeIdentifierCharacter): (WebCore::IDBKeyPath::Lexer::lexIdentifier): (WebCore::IDBKeyPath::Lexer::lexNumber):
storage/IDBKeyPath.h: Added. (WebCore::IDBKeyPath::Token::Token): (WebCore::IDBKeyPath::): (WebCore::IDBKeyPath::Lexer::Lexer): (WebCore::IDBKeyPath::Lexer::next): (WebCore::IDBKeyPath::Lexer::currentToken):

2010-07-28 Marcus Bulach <bulach@chromium.org>

Reviewed by Adam Barth.

Implement IDBKeyPath parser.
https://bugs.webkit.org/show_bug.cgi?id=42976

IDBKeyPath is an internal class, exposed only for unit tests for now.
It'll allow indexed database implementation to traverse a given key path and obtain the corresponding properties
from a SerializedScriptValue.

WebKit.gyp:
tests/IDBKeyPathTest.cpp: Added. (WebCore::ExpectedToken::ExpectedToken): (WebCore::checkKeyPath): (WebCore::TEST):

5:30 AM Changeset in webkit [65416] by steveblock@google.com

5 edits
7 adds in trunk

2010-08-13 Steve Block <steveblock@google.com>

Reviewed by Alexey Proskuryakov.

Geolocation requests in progress when the frame is disconnected should invoke the error callback
https://bugs.webkit.org/show_bug.cgi?id=43974

If requests are ongoing when the Frame is disconnected, we abort them with a fatal error.
To do this, when Geolocation::disconnectFrame() is called we call cancelAllRequests() to
set a fatal error on all ongoing requests. Once the requests have made their error
callbacks, they are deleted. Note that we check at callback time that the script
execution context for the callback is still valid, so it's safe to attempt these
callbacks even after the Geolocation object's Frame has gone.

This requires a change to allow multiple calls to GeoNotifier::setFatalError().
For repeated calls, we do not replace the existing error. This ensures that when
permission has been denied and the frame is then disconnected, the error callback
reports the permission error, as required by the spec.

Tests: fast/dom/Geolocation/disconnected-frame.html

fast/dom/Geolocation/disconnected-frame-permission-denied.html

page/Geolocation.cpp: (WebCore::Geolocation::GeoNotifier::setFatalError): (WebCore::Geolocation::disconnectFrame): (WebCore::Geolocation::startRequest): (WebCore::Geolocation::cancelAllRequests):
page/Geolocation.h:

2010-08-13 Steve Block <steveblock@google.com>

Reviewed by Alexey Proskuryakov.

Geolocation requests in progress when the frame is disconnected should invoke the error callback
https://bugs.webkit.org/show_bug.cgi?id=43974

Added new tests to GTK skipped list.

fast/dom/Geolocation/disconnected-frame.html: Added.
fast/dom/Geolocation/disconnected-frame-expected.txt: Added.
fast/dom/Geolocation/script-tests/disconnected-frame.js: Added.
fast/dom/Geolocation/resources/disconnected-frame-inner.html: Added.
fast/dom/Geolocation/disconnected-frame-permission-denied.html: Added.
fast/dom/Geolocation/disconnected-frame-permission-denied-expected.txt: Added.
fast/dom/Geolocation/script-tests/disconnected-frame-permission-denied.js: Added.
platform/gtk/Skipped:

5:23 AM Changeset in webkit [65415] by pfeldman@chromium.org

5 edits in trunk

2010-08-16 Pavel Feldman <pfeldman@chromium.org>

Reviewed by Joseph Pecoraro.

Web Inspector: upstream frontend-side WebSocket transport.
https://bugs.webkit.org/show_bug.cgi?id=43970

Chromium already has an alternate WebSocket-based communication channel with
the backend. Upstreaming it in this change. We will agree on the URI
of the remote service as the protocol matures.

inspector/front-end/inspector.js: (.WebInspector.socket.onmessage): (.WebInspector.socket.onerror): (.WebInspector.socket.onopen): (WebInspector.loaded): (WebInspector.doLoadedDone): (WebInspector_syncDispatch):

2010-08-16 Pavel Feldman <pfeldman@chromium.org>

Reviewed by Joseph Pecoraro.

Web Inspector: upstream frontend-side WebSocket transport.
https://bugs.webkit.org/show_bug.cgi?id=43970

Chromium already has an alternate WebSocket-based communication channel with
the backend. Upstreaming it in this change. We will agree on the URI
of the remote service as the protocol matures.

src/WebDevToolsFrontendImpl.cpp: (WebKit::WebDevToolsFrontendImpl::dispatchOnInspectorFrontend):
src/js/DevTools.js: (WebInspector.loaded): (devtools.domContentLoaded):

5:19 AM Changeset in webkit [65414] by pfeldman@chromium.org

2 edits in trunk/WebCore

2010-08-16 Pavel Feldman <pfeldman@chromium.org>

Reviewed by Yury Semikhatsky.

Web Inspector: Make InjectedScript proto-based.
https://bugs.webkit.org/show_bug.cgi?id=44028

inspector/front-end/InjectedScript.js: (injectedScriptConstructor): (injectedScriptConstructor.):
inspector/front-end/inspector.js: (WebInspector.loaded):

5:16 AM Changeset in webkit [65413] by yurys@chromium.org

3 edits in trunk/WebKit/chromium

2010-08-16 Yury Semikhatsky <yurys@chromium.org>

Reviewed by Pavel Feldman.

Web Inspector: remove utility context related code from debugger agent
https://bugs.webkit.org/show_bug.cgi?id=44049

src/DebuggerAgentManager.cpp: (WebKit::DebuggerAgentManager::onV8DebugMessage): (WebKit::DebuggerAgentManager::pauseScript):
src/DebuggerAgentManager.h:

5:16 AM Changeset in webkit [65412] by pfeldman@chromium.org

8 edits
2 adds in trunk

2010-08-16 Pavel Feldman <pfeldman@chromium.org>

Reviewed by Joseph Pecoraro.

Web Inspector: [REGRESSION] console's clear command does not work.
https://bugs.webkit.org/show_bug.cgi?id=44027

inspector/console-clear.html:
inspector/console-command-clear-expected.txt: Added.
inspector/console-command-clear.html: Copied from LayoutTests/inspector/console-clear.html.

2010-08-16 Pavel Feldman <pfeldman@chromium.org>

Reviewed by Joseph Pecoraro.

Web Inspector: [REGRESSION] console's clear command does not work.
https://bugs.webkit.org/show_bug.cgi?id=44027

Test: inspector/console-command-clear.html

inspector/Inspector.idl:
inspector/InspectorBackend.cpp:
inspector/InspectorBackend.h:
inspector/InspectorController.cpp: (WebCore::InspectorController::clearConsoleMessages):
inspector/front-end/ConsoleView.js: (WebInspector.ConsoleView.prototype.requestClearMessages):
inspector/front-end/inspector.js: (WebInspector.consoleMessagesCleared):

4:57 AM Changeset in webkit [65411] by hamaji@chromium.org

2 edits in trunk/WebKit/chromium

2010-08-16 Yuta Kitamura <yutak@chromium.org>

Reviewed by Shinichiro Hamaji.

Attempt to fix Chromium's test_shell_tests.

Port Chromium's WebEntities to HTMLEntityTable
https://bugs.webkit.org/show_bug.cgi?id=44036

src/WebEntities.cpp: (WebKit::):

4:26 AM Changeset in webkit [65410] by jorlow@chromium.org

5 edits in trunk/WebCore

2010-08-16 Jeremy Orlow <jorlow@chromium.org>

Ugh, the #if ENABLE() was supposed to be added to Settings not GroupSettings.

page/GroupSettings.cpp: (WebCore::GroupSettings::GroupSettings): (WebCore::GroupSettings::setLocalStorageQuotaBytes):
page/GroupSettings.h:
page/Settings.cpp: (WebCore::Settings::Settings):
page/Settings.h:

4:09 AM Changeset in webkit [65409] by jorlow@chromium.org

2 edits in trunk/WebCore

2010-08-16 Jeremy Orlow <jorlow@chromium.org>

Last build fix?

page/PageGroup.cpp: (WebCore::PageGroup::PageGroup):

3:53 AM Changeset in webkit [65408] by jorlow@chromium.org

3 edits in trunk/WebCore

2010-08-16 Jeremy Orlow <jorlow@chromium.org>

Ugh...another build fix.

page/GroupSettings.cpp: (WebCore::GroupSettings::GroupSettings):
page/GroupSettings.h:

3:40 AM Changeset in webkit [65407] by jorlow@chromium.org

2 edits in trunk/WebCore

2010-08-16 Jeremy Orlow <jorlow@chromium.org>

Build fix.

GNUmakefile.am:

3:20 AM Changeset in webkit [65406] by jorlow@chromium.org

2 edits in trunk/WebCore

2010-08-16 Jeremy Orlow <jorlow@chromium.org>

Build fix.

page/PageGroup.cpp: (WebCore::PageGroup::localStorage):

3:02 AM Changeset in webkit [65405] by jorlow@chromium.org

12 edits
2 adds in trunk/WebCore

2010-08-13 Jeremy Orlow <jorlow@chromium.org>

Some settings are linked to the PageGroup not the Page. Create a new class for those.
https://bugs.webkit.org/show_bug.cgi?id=43794

Since persistient storage is shared between pages, there's no way to modify
some settings related to it on a per page basis. As such, it's not technically
correct for these settings to be on the Page's settings. Create a new class
called GroupSettings, move group-wide local storage setting there (the other we
can't), and add a new setting for IndexedDB's path (which is prompting this change).

No behavior has changed, so no tests.

Android.mk:
CMakeLists.txt:
GNUmakefile.am:
WebCore.gypi:
WebCore.pro:
WebCore.vcproj/WebCore.vcproj:
WebCore.xcodeproj/project.pbxproj:
page/GroupSettings.cpp: Added. (WebCore::GroupSettings::GroupSettings): (WebCore::GroupSettings::setLocalStorageQuotaBytes): (WebCore::GroupSettings::setLocalStorageDatabasePath): (WebCore::GroupSettings::setIndexedDBDatabasePath):
page/GroupSettings.h: Added. (WebCore::GroupSettings::create): (WebCore::GroupSettings::localStorageQuotaBytes): (WebCore::GroupSettings::localStorageDatabasePath): (WebCore::GroupSettings::indexedDBDatabasePath):
page/PageGroup.cpp: (WebCore::PageGroup::PageGroup): (WebCore::PageGroup::localStorage):
page/PageGroup.h: (WebCore::PageGroup::groupSettings):
page/Settings.cpp: (WebCore::Settings::Settings): (WebCore::Settings::setSessionStorageQuota):
page/Settings.h:

2:44 AM Changeset in webkit [65404] by morrita@google.com

2 edits in trunk/LayoutTests

2010-08-16 MORITA Hajime <morrita@google.com>

Unreviewed; test expectations fix for Chromium.

platform/chromium/test_expectations.txt:

2:21 AM Changeset in webkit [65403] by zoltan@webkit.org

3 edits in trunk/WebKit2

Unreviewed Qt WebKit2 build fix.

Patch by Balazs Kelemen <kb@inf.u-szeged.hu> on 2010-08-16

UIProcess/qt/WebContextQt.cpp:

(WebKit::WebContext::platformSetUpWebProcess): Added empty stub implementation.

WebKit2.pro: Enumerate new files, include WebKit.pri for correct define macro set.

2:09 AM Changeset in webkit [65402] by abarth@webkit.org

1 edit
2 adds in trunk/LayoutTests

2010-08-16 Adam Barth <abarth@webkit.org>

Reviewed by Alexey Proskuryakov.

document.write causes the links to appear twice (noscript content is not ignored)
https://bugs.webkit.org/show_bug.cgi?id=16230

Add a test for this case.

fast/parser/document-write-noscript-expected.txt: Added.
fast/parser/document-write-noscript.html: Added.

1:51 AM Changeset in webkit [65401] by abarth@webkit.org

1 edit
2 adds in trunk/LayoutTests

2010-08-16 Adam Barth <abarth@webkit.org>

Reviewed by Alexey Proskuryakov.

document.write() isn't processed immediately before DOM access
https://bugs.webkit.org/show_bug.cgi?id=36881

Test that document.write gets flushed before DOM access takes place.

fast/tokenizer/document-write-racing-dom-access-expected.txt: Added.
fast/tokenizer/document-write-racing-dom-access.html: Added.

1:45 AM Changeset in webkit [65400] by hamaji@chromium.org

15 edits
12 adds in trunk/LayoutTests

2010-08-16 Yuta Kitamura <yutak@chromium.org>

Not reviewed. Chromium test expectations updates and rebaselines.

platform/chromium-linux/fast/canvas/setWidthResetAfterForcedRender-expected.checksum: Added.
platform/chromium-linux/fast/canvas/setWidthResetAfterForcedRender-expected.png: Added.
platform/chromium-linux/fast/text/word-space-expected.checksum:
platform/chromium-linux/fast/text/word-space-expected.png:
platform/chromium-linux/tables/mozilla/other/wa_table_thtd_rowspan-expected.checksum:
platform/chromium-linux/tables/mozilla/other/wa_table_thtd_rowspan-expected.png:
platform/chromium-mac/fast/canvas/setWidthResetAfterForcedRender-expected.checksum: Added.
platform/chromium-mac/fast/canvas/setWidthResetAfterForcedRender-expected.png: Added.
platform/chromium-mac/fast/text/word-space-expected.checksum: Added.
platform/chromium-mac/fast/text/word-space-expected.png: Added.
platform/chromium-mac/tables/mozilla/other/wa_table_thtd_rowspan-expected.checksum: Added.
platform/chromium-mac/tables/mozilla/other/wa_table_thtd_rowspan-expected.png: Added.
platform/chromium-win/fast/canvas/setWidthResetAfterForcedRender-expected.checksum: Added.
platform/chromium-win/fast/canvas/setWidthResetAfterForcedRender-expected.png: Added.
platform/chromium-win/fast/canvas/setWidthResetAfterForcedRender-expected.txt: Added.
platform/chromium-win/fast/text/word-space-expected.checksum:
platform/chromium-win/fast/text/word-space-expected.png:
platform/chromium-win/fast/text/word-space-expected.txt:
platform/chromium-win/svg/W3C-SVG-1.1/animate-elem-32-t-expected.txt:
platform/chromium-win/svg/W3C-SVG-1.1/shapes-intro-01-t-expected.txt:
platform/chromium-win/tables/mozilla/other/wa_table_thtd_rowspan-expected.checksum:
platform/chromium-win/tables/mozilla/other/wa_table_thtd_rowspan-expected.png:
platform/chromium-win/tables/mozilla/other/wa_table_thtd_rowspan-expected.txt:
platform/chromium-win/tables/mozilla/other/wa_table_tr_align-expected.txt:
platform/chromium/test_expectations.txt:

1:45 AM Changeset in webkit [65399] by zoltan@webkit.org

2 edits in trunk/WebKitTools

2010-08-16 Zoltan Horvath <zoltan@webkit.org>

Add my old e-mail addresses to committers.py.

Scripts/webkitpy/common/config/committers.py:

1:22 AM Changeset in webkit [65398] by morrita@google.com

2 edits in trunk/LayoutTests

2010-08-16 MORITA Hajime <morrita@google.com>

Unreviewd, added skip list entries.

This is follow up for r65395. QT port ignores tests under
editing/pasteboard.

platform/qt/Skipped:

1:14 AM Changeset in webkit [65397] by morrita@google.com

2 edits in trunk/LayoutTests

2010-08-16 MORITA Hajime <morrita@google.com>

Unreviewd, added skip list entries.

This is follow up for r65395. GTK port ignores tests under
editing/pasteboard.

platform/gtk/Skipped:

1:04 AM Changeset in webkit [65396] by abarth@webkit.org

3 edits
2 adds in trunk

2010-08-16 Adam Barth <abarth@webkit.org>

Reviewed by Alexey Proskuryakov.

HTML5 fragment parsing doesn't load iframes
https://bugs.webkit.org/show_bug.cgi?id=44038

Test that iframes created in detached documents actually get loaded
when they're adopted into attached documents (or, rather, when they're
inserted into the DOM).

fast/frames/adopt-from-created-document-expected.txt: Added.
fast/frames/adopt-from-created-document.html: Added.

2010-08-16 Adam Barth <abarth@webkit.org>

Reviewed by Alexey Proskuryakov.

HTML5 fragment parsing doesn't load iframes
https://bugs.webkit.org/show_bug.cgi?id=44038

Previously, this code assumed that every source iframe was actually
attached to a render tree. It tried to "keep the iframe alive" as it
moved documents. However, if the iframe wasn't attached to begin with,
it was never "alive," so trying to keep a dead iframe alive lead to a
... zombie iframe.

Test: fast/frames/adopt-from-created-document.html

dom/Document.cpp: (WebCore::Document::adoptNode):

12:30 AM Changeset in webkit [65395] by morrita@google.com

8 edits
6 adds in trunk

2010-08-13 MORITA Hajime <morrita@google.com>

Reviewed by Tony Chang.

https://bugs.webkit.org/show_bug.cgi?id=43778
Dropping should fire textInput event.

editing/pasteboard/drop-text-events-expected.txt: Added.
editing/pasteboard/drop-text-events-sideeffect-expected.txt: Added.
editing/pasteboard/drop-text-events-sideeffect.html: Added.
editing/pasteboard/drop-text-events.html: Added.
editing/pasteboard/script-tests/drop-text-events-sideeffect.js: Added.
editing/pasteboard/script-tests/drop-text-events.js: Added.

2010-08-13 MORITA Hajime <morrita@google.com>

Reviewed by Tony Chang.

https://bugs.webkit.org/show_bug.cgi?id=43778
Dropping should fire textInput event

This change:

added TextEvent::m_isDrop to distinguish drop-initiated events.
added to dispatch TextEvent from DragController before making a side effect and cancel the side effect if the event prevents default

Tests: editing/pasteboard/drop-text-events-sideeffect.html

editing/pasteboard/drop-text-events.html

dom/TextEvent.cpp: (WebCore::TextEvent::createForDrop): Added. (WebCore::TextEvent::TextEvent):
dom/TextEvent.h: (WebCore::TextEvent::isDrop): Added.
editing/Editor.cpp: (WebCore::Editor::handleTextEvent): (WebCore::Editor::findEventTargetFrom): Extracted from findEventTargetFromSelection(). (WebCore::Editor::findEventTargetFromSelection):
editing/Editor.h:
page/DragController.cpp: (WebCore::DragController::dispatchTextInputEventFor): Added. (WebCore::DragController::concludeEditDrag): Added an event dispatch path.
page/DragController.h:

Note: See TracTimeline for information about the timeline view.

Download in other formats:

RSS Feed