Timeline



Jul 7, 2010:

11:23 PM Changeset in webkit [62763] by commit-queue@webkit.org
  • 2 edits in trunk/WebCore

2010-07-07 Pavel Podivilov <​podivilov@chromium.org>

Reviewed by Nate Chapin.

[V8] Fix document wrapper memory leak in bindings.
​https://bugs.webkit.org/show_bug.cgi?id=41771

  • bindings/v8/custom/V8HTMLDocumentCustom.cpp: (WebCore::V8HTMLDocument::WrapInShadowObject): Do not create static persistent handle to shadowConstructor because it keeps the first context alive forever.
11:09 PM Changeset in webkit [62762] by commit-queue@webkit.org
  • 2 edits in trunk/WebCore

2010-07-07 Andreas Kling <​andreas.kling@nokia.com>

Reviewed by Tor Arne Vestbø.

[Qt] Enable smooth pixmap transforms by default
​https://bugs.webkit.org/show_bug.cgi?id=41774

  • platform/graphics/qt/GraphicsContextQt.cpp: (WebCore::GraphicsContextPlatformPrivate::GraphicsContextPlatformPrivate): (WebCore::GraphicsContext::setImageInterpolationQuality):
10:57 PM Changeset in webkit [62761] by jesus@webkit.org
  • 2 edits in trunk/WebCore

2010-07-07 Jesus Sanchez-Palencia <​jesus@webkit.org>

Reviewed by Antti Koivisto.

[Qt] Missing include to build QtWebKit with WebKit2 in MediaPlayerPrivatePhonon.cpp
​https://bugs.webkit.org/show_bug.cgi?id=41767

  • platform/graphics/qt/MediaPlayerPrivatePhonon.cpp: Add Logging.h include
10:45 PM Changeset in webkit [62760] by zoltan@webkit.org
  • 2 edits in trunk/WebKit2

2010-07-07 Zoltan Horvath <​zoltan@webkit.org>

Reviewed by Kenneth Rohde Christiansen.

[Qt] Terminate QtWebProcess on exit
​https://bugs.webkit.org/show_bug.cgi?id=41766

Call WKPageTerminate to terminate QtWebProcess on exit.

  • UIProcess/API/qt/qwkpage.cpp: (QWKPage::~QWKPage):
10:09 PM Changeset in webkit [62759] by sfalken@apple.com
  • 2 edits in tags/Safari-534.2.1/WebKit2

Merge r62756.

10:06 PM Changeset in webkit [62758] by sfalken@apple.com
  • 5 edits in tags/Safari-534.2.1

Versioning.

9:59 PM Changeset in webkit [62757] by sfalken@apple.com
  • 1 copy in tags/Safari-534.2.1

New tag.

9:54 PM Changeset in webkit [62756] by sfalken@apple.com
  • 2 edits in trunk/WebKit2

Windows release build fix.
Don't generate public symbols since this is exceeding address space on our builders.

  • win/WebKit2.vcproj:
9:37 PM Changeset in webkit [62755] by dbates@webkit.org
  • 3 edits in trunk/WebKitTools

2010-07-07 Daniel Bates <​dbates@rim.com>

Reviewed by Dumitru Daniliuc.

VCSUtils.pm complains about uninitialized value $newLine
​https://bugs.webkit.org/show_bug.cgi?id=41333

Fixes an issue where VCSUtils::fixChangeLogPatch() may read off the end of
an array when fixing a change log entry that overlaps with an earlier
entry. In particular, when a patch contains a change log entry inserted
earlier in the change log file, but after an entry with the same author
and date.

  • Scripts/VCSUtils.pm:
    • Added for-loop constraint to fixChangeLogPatch() so that it does not read off the end of the @overlappingLines array.
  • Scripts/webkitperl/VCSUtils_unittest/fixChangeLogPatch.pl: Added unit test.
9:20 PM Changeset in webkit [62754] by commit-queue@webkit.org
  • 4 edits in trunk

2010-07-07 Pavel Podivilov <​podivilov@chromium.org>

Reviewed by Yury Semikhatsky.

Web Inspector: Move v8-related script offset conversion to DebuggerScript.js
​https://bugs.webkit.org/show_bug.cgi?id=41755

  • bindings/v8/ScriptDebugServer.cpp: (WebCore::ScriptDebugServer::dispatchDidParseSource):

2010-07-07 Pavel Podivilov <​podivilov@chromium.org>

Reviewed by Yury Semikhatsky.

Web Inspector: convert script offset to webkit format
​https://bugs.webkit.org/show_bug.cgi?id=41755

  • src/js/DebuggerScript.js: ():
9:07 PM Changeset in webkit [62753] by commit-queue@webkit.org
  • 3 edits
    1 add in trunk

2010-07-07 Nicolas Weber <​thakis@chromium.org>

Reviewed by Dimitri Glazkov.
​https://bugs.webkit.org/show_bug.cgi?id=41580

Fix rendering of radial gradients in skia if both points of the
gradient are the same and r0 > 0.

  • fast/gradients/radial-centered.html: Added.

2010-07-07 Nicolas Weber <​thakis@chromium.org>

Reviewed by Dimitri Glazkov.
​https://bugs.webkit.org/show_bug.cgi?id=41580

Fix rendering of radial gradients in skia if both points of the
gradient are the same and r0 > 0.

Test: fast/gradients/radial-centered.html

  • platform/graphics/skia/GradientSkia.cpp: (WebCore::Gradient::platformGradient):
8:55 PM Changeset in webkit [62752] by andersca@apple.com
  • 2 edits in trunk/WebKitTools

Comment out a part of TestNetscapePlugin that caused tests to fail on Tiger and Leopard.

  • DumpRenderTree/TestNetscapePlugIn/main.cpp:

(NPP_New):

8:41 PM Changeset in webkit [62751] by tkent@chromium.org
  • 4 edits in trunk/WebKitTools

[DRT/Chromium] Support for indeterminate checkbox
​https://bugs.webkit.org/show_bug.cgi?id=41747

Reviewed by Adam Barth.

Import ​http://src.chromium.org/viewvc/chrome?view=rev&revision=51499

  • DumpRenderTree/chromium/WebThemeControlDRT.cpp:

(WebThemeControlDRT::draw):

  • DumpRenderTree/chromium/WebThemeControlDRT.h:

(WebThemeControlDRT::):

  • DumpRenderTree/chromium/WebThemeEngineDRT.cpp:

(WebThemeEngineDRT::paintButton):

8:37 PM Changeset in webkit [62750] by weinig@apple.com
  • 4 edits
    2 moves
    7 adds
    1 delete in trunk/WebKitTools

Clean up MiniBrowser Xcode project.

Reviewed by Mark Rowe.

  • Use xcconfig files.
  • Rename plists to canonical Info.plist
  • Remove localizations.
  • MiniBrowser/Configurations: Added.
  • MiniBrowser/Configurations/Base.xcconfig: Added.
  • MiniBrowser/Configurations/DebugRelease.xcconfig: Added.
  • MiniBrowser/Configurations/MiniBrowser.xcconfig: Added.
  • MiniBrowser/Configurations/WebBundle.xcconfig: Added.
  • MiniBrowser/MiniBrowser.xcodeproj/project.pbxproj:
  • MiniBrowser/WebBundle-Info.plist: Removed.
  • MiniBrowser/mac/BrowserStatisticsWindowController.m:

(-[BrowserStatisticsWindowController initWithThreadedWKContextRef:processWKContextRef:]):

  • MiniBrowser/mac/BrowserWindow.xib: Added.
  • MiniBrowser/mac/BrowserWindowController.m:

(-[BrowserWindowController initWithPageNamespace:]):

  • MiniBrowser/mac/English.lproj: Removed.
  • MiniBrowser/mac/English.lproj/BrowserWindow.xib: Removed.
  • MiniBrowser/mac/English.lproj/InfoPlist.strings: Removed.
  • MiniBrowser/mac/English.lproj/MainMenu.xib: Removed.
  • MiniBrowser/mac/Info.plist: Copied from MiniBrowser/mac/MiniBrowser-Info.plist.
  • MiniBrowser/mac/MainMenu.xib: Added.
  • MiniBrowser/mac/MiniBrowser-Info.plist: Removed.
  • MiniBrowser/mac/WebBundle/Info.plist: Copied from MiniBrowser/WebBundle-Info.plist.
8:34 PM Changeset in webkit [62749] by tkent@chromium.org
  • 2 edits in trunk/WebKitTools

2010-07-07 Kent Tamura <​tkent@chromium.org>

Reviewed by Dimitri Glazkov.

[DRT/Chromium] Fix two regressions
​https://bugs.webkit.org/show_bug.cgi?id=41745

  • DumpRenderTree/chromium/WebViewHost.cpp: (WebViewHost::paintInvalidatedRegion): Repaint three times to fix svg/W3C-SVG-1.1/struct-use-01-t.svg and svg/custom/use-on-g-containing-foreignObject-and-image.svg though test_shell does it twice. Probably DRT needs it because DRT delays painting as possible.
8:32 PM Changeset in webkit [62748] by tkent@chromium.org
  • 2 edits in trunk/WebKit/chromium

2010-07-07 Kent Tamura <​tkent@chromium.org>

Reviewed by Adam Barth.

[DRT/Chromium] DumpRenderTree should depend on ImageDiff
​https://bugs.webkit.org/show_bug.cgi?id=41749

We usually use DumpRenderTree with ImageDiff. So ImageDiff should
be built with DumpRenderTree. Without this change, ImageDiff is not
built in a case of Chromium tree build.
test_shell in Chromium tree also depends on image_diff.

  • WebKit.gyp:
8:13 PM Changeset in webkit [62747] by andersca@apple.com
  • 2 edits in trunk/WebKitTools

Try to fix build.

  • DumpRenderTree/unix/TestNetscapePlugin/TestNetscapePlugin.cpp:

(webkit_test_plugin_destroy_stream):

8:07 PM Changeset in webkit [62746] by andersca@apple.com
  • 2 edits in trunk/WebKitTools

Try to fix the GTK+ and Qt test failures by merging the newly added code from TestNetscapePlugin/main.cpp

  • DumpRenderTree/unix/TestNetscapePlugin/TestNetscapePlugin.cpp:

(webkit_test_plugin_new_stream):
(webkit_test_plugin_destroy_stream):
(webkit_test_plugin_write_ready):
(webkit_test_plugin_write):

8:04 PM Changeset in webkit [62745] by weinig@apple.com
  • 2 edits in trunk/WebKitTools

Don't hard code the Mac OS X 10.6 SDK for MiniBrowser.

Reviewed by Mark Rowe.

  • MiniBrowser/MiniBrowser.xcodeproj/project.pbxproj:
7:58 PM Changeset in webkit [62744] by andersca@apple.com
  • 2 edits in trunk/WebKitTools

Use the correct XP_MACOSX define instead of XP_MAC. This was causing test failures on some platforms.

  • DumpRenderTree/TestNetscapePlugIn/main.cpp:

(NPP_New):
(NPP_HandleEvent):

7:49 PM Changeset in webkit [62743] by jam@chromium.org
  • 8 edits in trunk/WebKit/chromium

2010-07-07 John Abd-El-Malek <​jam@chromium.org>

Reviewed by Darin Fisher.

[chromium] Add a few more methods to WebPlugin so that Pepper v2 plugins can support copy/zoom/find
​https://bugs.webkit.org/show_bug.cgi?id=41649

  • public/WebPlugin.h: (WebKit::WebPlugin::hasSelection): (WebKit::WebPlugin::selectionAsText): (WebKit::WebPlugin::selectionAsMarkup): (WebKit::WebPlugin::setZoomFactor): (WebKit::WebPlugin::supportsFind): (WebKit::WebPlugin::startFind): (WebKit::WebPlugin::selectFindResult): (WebKit::WebPlugin::stopFind):
  • src/ContextMenuClientImpl.cpp: (WebKit::ContextMenuClientImpl::getCustomMenuFromDefaultItems):
  • src/WebFrameImpl.cpp: (WebKit::WebFrameImpl::pluginContainerFromFrame): (WebKit::ChromePluginPrintContext::end): (WebKit::ChromePluginPrintContext::computePageRects): (WebKit::ChromePluginPrintContext::spoolPage): (WebKit::WebFrameImpl::executeCommand):
  • src/WebFrameImpl.h:
  • src/WebPluginContainerImpl.cpp: (WebKit::WebPluginContainerImpl::copy): (WebKit::WebPluginContainerImpl::handleKeyboardEvent):
  • src/WebPluginContainerImpl.h:
  • src/WebViewImpl.cpp: (WebKit::WebViewImpl::setZoomLevel):
7:26 PM Changeset in webkit [62742] by bweinstein@apple.com
  • 3 edits in trunk/WebKitTools

Need to have a way to specify different results for Windows XP and 7
​https://bugs.webkit.org/show_bug.cgi?id=41776

Reviewed by Adam Roben.

Add the ability to have platform/win-xp, platform/win-vista, and platform/win-7
subdirectories, so we can have different results for tests on Windows XP, Windows
Vista, and Windows 7.

Windows XP will try the order: win-xp, win-vista, win-7, win, mac-snowleopard, mac.
Windows Vista will try the order: win-vista, win-7, win, mac-snowleopard, mac.
Windows 7 will try the order: win-7, win, mac-snowleopard, mac.

This matches the behavior of the mac.

  • Scripts/old-run-webkit-tests: Add multiple platforms for Windows, and if we are on XP

or Vista, try looking in other win-* subdirectories before win.

  • Scripts/webkitdirs.pm: Add some helper functions to determine what version of Windows

we are running.

7:13 PM Changeset in webkit [62741] by morrita@google.com
  • 3 edits
    3 adds in trunk

2010-06-18 MORITA Hajime <​morrita@google.com>

Reviewed by Ojan Vafai.

​https://bugs.webkit.org/show_bug.cgi?id=26526
Add support for input events (oninput) to contentEditable elements

  • fast/events/event-input-contentEditable-expected.txt: Added.
  • fast/events/event-input-contentEditable.html: Added.
  • fast/events/script-tests/event-input-contentEditable.js: Added.

2010-06-18 MORITA Hajime <​morrita@google.com>

Reviewed by Ojan Vafai.

​https://bugs.webkit.org/show_bug.cgi?id=26526
Add support for input events (oninput) to contentEditable elements

Made a default event handler on the Node to dispatch an input event when
webkitEditableContentChanged arrived.

Test: fast/events/event-input-contentEditable.html

  • dom/Node.cpp: (WebCore::Node::defaultEventHandler):
6:52 PM Changeset in webkit [62740] by tkent@chromium.org
  • 2 edits in trunk/WebKitTools

[DRT/Chromium] Remove ASSERT_NOT_REACHED in some functions
​https://bugs.webkit.org/show_bug.cgi?id=41753

Reviewed by Dimitri Glazkov.

The original code of the following functions in
test_shell_devtools_client.cc have NOTIMPLEMENTED(), not
ASSERT_NOT_REACHED(). The program shouldn't stop at these
functions.

  • DumpRenderTree/chromium/DRTDevToolsClient.cpp:

(DRTDevToolsClient::activateWindow):
(DRTDevToolsClient::closeWindow):
(DRTDevToolsClient::dockWindow):
(DRTDevToolsClient::undockWindow):

6:47 PM Changeset in webkit [62739] by andersca@apple.com
  • 6 edits
    1 copy
    1 add in trunk

Test that we call NPP_DestroyStream if a plug-in returns -1 from its NPP_Write function
​https://bugs.webkit.org/show_bug.cgi?id=41821

Reviewed by Sam Weinig.

WebKitTools:

  • DumpRenderTree/TestNetscapePlugIn/PluginObject.cpp:

(pluginGetProperty):
(pluginSetProperty):
(pluginAllocate):

  • DumpRenderTree/TestNetscapePlugIn/PluginObject.h:

Add and initialize the returnNegativeOneFromWrite property.

  • DumpRenderTree/TestNetscapePlugIn/main.cpp:

(NPP_NewStream):
Set the stream type to NP_NORMAL so we'll get write callbacks.

(NPP_DestroyStream):
Treat the onstreamdestroy attribute as a function name and not a string.

(NPP_WriteReady):
Have this return a nonzero value.

(NPP_Write):
If returnNegativeOneFromWrite is true return -1.

LayoutTests:

  • plugins/return-error-from-new-stream-doesnt-invoke-destroy-stream.html:

Update for changes to onstreamdestroy.

  • plugins/return-negative-one-from-write-expected.txt: Added.
  • plugins/return-negative-one-from-write.html: Added.

Add new test.

6:46 PM Changeset in webkit [62738] by andersca@apple.com
  • 2 edits in trunk/WebKit

Unreviewed build fix after r62676.

Patch by Lucas De Marchi <​lucas.demarchi@profusion.mobi> on 2010-07-07
[EFL] Fix build after Page constructor has changed. By now, the
introduced BackForwardControllerClient is not used.
​https://bugs.webkit.org/show_bug.cgi?id=41825

  • efl/ewk/ewk_view.cpp:

(_ewk_view_priv_new): Add missing parameter to WebCore::Page
constructor.

6:22 PM Changeset in webkit [62737] by kevino@webkit.org
  • 2 edits in trunk/WebKitTools

[wx] Build fix for machines with both MSVC 2005 and 2008 installed. Ensures the
latest version is selected by default, and allows the version to be specified.

6:19 PM Changeset in webkit [62736] by Darin Adler
  • 5 edits in trunk/WebCore

2010-07-07 Darin Adler <Darin Adler>

Reviewed by Anders Carlsson.

Fix adoptRef assertion failures caused by stack-allocated ResourceHandle objects
​https://bugs.webkit.org/show_bug.cgi?id=41823

  • platform/network/android/ResourceHandleAndroid.cpp: (WebCore::ResourceHandle::loadResourceSynchronously): Use adoptRef and new instead of allocating an object on the stack.
  • platform/network/curl/ResourceHandleCurl.cpp: (WebCore::ResourceHandle::loadResourceSynchronously): Ditto.
  • platform/network/qt/ResourceHandleQt.cpp: (WebCore::ResourceHandle::loadResourceSynchronously): Ditto.
  • platform/network/soup/ResourceHandleSoup.cpp: (WebCore::ResourceHandle::loadResourceSynchronously): Use create instead of allocating an object on the stack.
6:04 PM Changeset in webkit [62735] by Chris Fleizach
  • 5 edits
    2 adds in trunk

WebCore: AX: TextArea should return AXSelectedTextRange of 0,0 if the cursor is not in the text area
​https://bugs.webkit.org/show_bug.cgi?id=41810

WebKitTools: AX: TextArea should return AXSelectedTextRange of 0,0 if the cursor is not in the text area
​https://bugs.webkit.org/show_bug.cgi?id=41810

Reviewed by Darin Adler.

  • DumpRenderTree/mac/AccessibilityUIElementMac.mm:

(AccessibilityUIElement::selectedTextRange):

LayoutTests: AX: TextArea should return AXSelectedTextRange of 0,0 if the cursor is not in the text area
​https://bugs.webkit.org/show_bug.cgi?id=41810

Reviewed by Darin Adler.

  • platform/mac/accessibility/selected-text-range-for-empty-textarea-expected.txt: Added.
  • platform/mac/accessibility/selected-text-range-for-empty-textarea.html: Added.
5:17 PM Changeset in webkit [62734] by Darin Adler
  • 4 edits in trunk/JavaScriptCore

Fixed build in configurations like PowerPC.

  • runtime/RegExpConstructor.cpp: Added include of PassOwnPtr.h.
  • runtime/RegExpObject.cpp: Ditto.
  • wtf/SizeLimits.cpp: Changed compile time assertion to work

even on platforms where two bool members do not end up taking
the same size as one int member!

5:16 PM Changeset in webkit [62733] by Chris Fleizach
  • 3 edits in trunk/LayoutTests

Bug 41806 - AX: Data table heuristics: assume data table if at least one row or column of TH cells exist
​https://bugs.webkit.org/show_bug.cgi?id=41806

No review. Fixing skipped layout test name.

  • platform/mac-leopard/Skipped:
  • platform/mac-tiger/Skipped:
5:12 PM Changeset in webkit [62732] by ojan@chromium.org
  • 3 edits in trunk/WebKitTools

2010-07-07 Martin Robinson <​mrobinson@igalia.com>

Unreviewed.

Build fix after r62700.

  • GNUmakefile.am: Update the source list to reflect the new TestNetscapePlugIn source file locations.
5:09 PM Changeset in webkit [62731] by Joseph Pecoraro
  • 2 edits in trunk/WebCore

2010-07-07 Joseph Pecoraro <Joseph Pecoraro>

Rubber-stamped by Pavel Feldman.

Web Inspector: All RefPtr arguments and return values should be changed to PassRefPtr according to common practice.
​https://bugs.webkit.org/show_bug.cgi?id=41759

  • inspector/CodeGeneratorInspector.pm:
5:03 PM Changeset in webkit [62730] by Martin Robinson
  • 2 edits in trunk/WebKitTools

2010-07-07 Martin Robinson <​mrobinson@igalia.com>

Unreviewed.

Build fix after r62700.

  • GNUmakefile.am: Update the source list to reflect the new TestNetscapePlugIn source file locations.
4:53 PM Changeset in webkit [62729] by mrowe@apple.com
  • 1 edit in branches/safari-533.17-branch/WebCore/websockets/WebSocketHandshake.cpp

Build fix.

4:48 PM Changeset in webkit [62728] by ojan@chromium.org
  • 4 edits in trunk/WebKitTools

2010-07-07 Ojan Vafai <​ojan@chromium.org>

Reviewed by David Levin.

add --exit-after-n-crashes to run-webkit-tests
​https://bugs.webkit.org/show_bug.cgi?id=41811

Change the bots at build.webkit.org to use --exit-after-n-crashes
instead of --exit-after-n-failures since we want to be able to have
more than 20 failures on those bots, but more than 20 crashes means
something should be rolled out.

  • Scripts/old-run-webkit-tests:
  • Scripts/webkitpy/layout_tests/run_webkit_tests.py:
4:27 PM Changeset in webkit [62727] by oliver@apple.com
  • 3 edits in trunk/JavaScriptCore

2010-07-07 Oliver Hunt <​oliver@apple.com>

Reviewed by Geoffrey Garen.

Lazy mode of parser allows invalid syntax in object literals.
​https://bugs.webkit.org/show_bug.cgi?id=41809

Make the parser itself validate getter and setter syntax rather
than offloading it to the AST builder.

  • parser/ASTBuilder.h: (JSC::ASTBuilder::createGetterOrSetterProperty):
  • parser/JSParser.cpp: (JSC::JSParser::parseProperty):
4:01 PM Changeset in webkit [62726] by mrowe@apple.com
  • 10 edits in branches/safari-533.17-branch

Merge r62625.

3:53 PM Changeset in webkit [62725] by mrowe@apple.com
  • 2 edits in branches/safari-533.17-branch/WebCore

Merge r62477.

3:51 PM Changeset in webkit [62724] by mrowe@apple.com
  • 2 edits in branches/safari-533.17-branch/WebCore

Merge r62632.

3:46 PM Changeset in webkit [62723] by pkasting@chromium.org
  • 2 edits in trunk/LayoutTests

Not reviewed. Chromium test expectation update.

Mark as passing a test which has not failed in a while.

  • platform/chromium/test_expectations.txt:
3:46 PM Changeset in webkit [62722] by mrowe@apple.com
  • 3 edits
    4 adds in branches/safari-533.17-branch

Merge r62482.

3:43 PM Changeset in webkit [62721] by mrowe@apple.com
  • 4 edits
    4 adds in branches/safari-533.17-branch

Merge r62662.

3:42 PM Changeset in webkit [62720] by crogers@google.com
  • 2 edits in branches/audio/WebCore/audio

try to use more optimized copyWithGainFrom() method in a few places where possible

3:42 PM Changeset in webkit [62719] by crogers@google.com
  • 3 edits in branches/audio/WebCore/audio

Take cone and distance gains into account in AudioPannerNode

3:42 PM Changeset in webkit [62718] by crogers@google.com
  • 2 edits in branches/audio/WebCore/audio

Add AudioBus::copyWithGainFrom() to avoid summing to bus in certain cases

3:41 PM Changeset in webkit [62717] by bweinstein@apple.com
  • 2 edits in trunk/WebKitTools

Fix the Windows build by removing win/TestNetscapePlugin/main.cpp from the
TestNetscapePlugin project, because this was removed from the tree.

  • DumpRenderTree/TestNetscapePlugIn/win/TestNetscapePlugin.vcproj:
3:38 PM Changeset in webkit [62716] by mrowe@apple.com
  • 6 edits
    35 deletes in branches/safari-533.17-branch/WebKitTools

Merge r59350.

3:38 PM Changeset in webkit [62715] by mrowe@apple.com
  • 7 edits in branches/safari-533.17-branch

Merge r59903.

3:36 PM Changeset in webkit [62714] by mrowe@apple.com
  • 4 edits
    4 adds in branches/safari-533.17-branch

Merge r62664.

3:36 PM Changeset in webkit [62713] by Chris Fleizach
  • 5 edits
    2 adds in trunk

AX: Data table heuristics: assume data table if at least one row or column of TH cells exist
​https://bugs.webkit.org/show_bug.cgi?id=41806

Reviewed by Beth Dakin.

WebCore:

Test: platform/mac/accessibility/table-with-row-col-of-headers.html

  • accessibility/AccessibilityTable.cpp:

(WebCore::AccessibilityTable::isTableExposableThroughAccessibility):

LayoutTests:

  • platform/mac-leopard/Skipped:
  • platform/mac-tiger/Skipped:
  • platform/mac/accessibility/table-with-row-col-of-headers-expected.txt: Added.
  • platform/mac/accessibility/table-with-row-col-of-headers.html: Added.
3:33 PM Changeset in webkit [62712] by mrowe@apple.com
  • 7 edits in branches/safari-533.17-branch

Merge r62391.

3:30 PM Changeset in webkit [62711] by mrowe@apple.com
  • 3 edits
    3 adds in branches/safari-533.17-branch

Merge r62301.

3:25 PM Changeset in webkit [62710] by mrowe@apple.com
  • 3 edits
    3 adds in branches/safari-533.17-branch

Merge r62386.

3:21 PM Changeset in webkit [62709] by pkasting@chromium.org
  • 2 edits
    4 deletes in trunk/LayoutTests

Not reviewed. Chromium test expectation update.

Remove erroneously-added platform-specific expectations and instead mark
the tests as failing.

  • platform/chromium-mac/fast/js/function-apply-expected.txt: Removed.
  • platform/chromium-mac/fast/js/function-apply-many-args-expected.txt: Removed.
  • platform/chromium-win/fast/js/function-apply-expected.txt: Removed.
  • platform/chromium-win/fast/js/function-apply-many-args-expected.txt: Removed.
  • platform/chromium/test_expectations.txt:
3:20 PM Changeset in webkit [62708] by dumi@chromium.org
  • 4 edits in trunk

JavaScriptCore: Revert r62689.
​https://bugs.webkit.org/show_bug.cgi?id=41804

Reviewed by Adam Roben.

  • runtime/Collector.cpp:

(JSC::Heap::freeBlocks):

WebKitLibraries: Disable MSVC warning 4288.
​https://bugs.webkit.org/show_bug.cgi?id=41804

Reviewed by Adam Roben.

MSVC has a non-standard extension that allows variables declared
in for-loops to remain visible in the same scope even after
exiting the for-loop
(​http://msdn.microsoft.com/en-us/library/bk5hc10s.aspx). The /Ze
option (turned on by default) enables all MSVC extensions, and
/Zc:forScope- tells the compiler to issue a C4288 warning when the
same variable is declared in the for-loop and re-declared later in
the same scope.

There seems to be a bug in VS2005 that erroneously enables
/Zc:forScope- even when that option is not specified
(​http://connect.microsoft.com/VisualStudio/feedback/details/338010/bogus-compiler-warning-c4288). Looks
like our build got hit by that bug, so we need to disable warning
4288 to fix it.

  • win/tools/vsprops/common.vsprops:
3:18 PM Changeset in webkit [62707] by mrowe@apple.com
  • 2 edits in trunk/WebCore

Fix the build.

Xcode decided to change the reference type from "Relative to Build Product" to "Relative to Group"
when I moved the file between folders. That's incredibly confusing so I'll file a bug against Xcode.

  • WebCore.xcodeproj/project.pbxproj:
3:13 PM Changeset in webkit [62706] by mrowe@apple.com
  • 2 edits in trunk/WebCore

Fix the MathML build.

  • html/HTMLTreeBuilder.cpp:

(WebCore::HTMLTreeBuilder::resetInsertionModeAppropriately): Fix a typo in a variable name.

3:00 PM Changeset in webkit [62705] by jhawkins@chromium.org
  • 5 edits in trunk

2010-07-07 James Hawkins <​jhawkins@chromium.org>

Reviewed by Dimitri Glazkov.

[Chromium] Add setSuggestedValue to WebInputElement
​https://bugs.webkit.org/show_bug.cgi?id=41785

  • public/WebInputElement.h:
  • src/WebInputElement.cpp: (WebKit::WebInputElement::setSuggestedValue): (WebKit::WebInputElement::suggestedValue):

2010-07-07 James Hawkins <​jhawkins@chromium.org>

Reviewed by Dimitri Glazkov.

Move setSuggestedValue() and suggestedValue() back to public as it's
now used by WebInputElement.
​https://bugs.webkit.org/show_bug.cgi?id=41785

No new tests, as the method is only being used by Chromium's WebKit
API.

  • html/HTMLInputElement.h:
2:53 PM Changeset in webkit [62704] by mitz@apple.com
  • 1 edit
    6 copies in trunk/LayoutTests

Requested by Mark Rowe.

Reverted the part of r62675 in which Snow Leopard-and-earlier results for a mac-only
test were mistakenly removed.

  • platform/mac-snowleopard/platform: Copied from platform/mac-snowleopard/platform.
  • platform/mac-snowleopard/platform/mac: Copied from platform/mac-snowleopard/platform/mac.
  • platform/mac-snowleopard/platform/mac/fast: Copied from platform/mac-snowleopard/platform/mac/fast.
  • platform/mac-snowleopard/platform/mac/fast/loader: Copied from platform/mac-snowleopard/platform/mac/fast/loader.
  • platform/mac-snowleopard/platform/mac/fast/loader/file-url-mimetypes-2-expected.txt: Copied from platform/mac-snowleopard/platform/mac/fast/loader/file-url-mimetypes-2-expected.txt.
  • platform/mac-snowleopard/platform/mac/fast/loader/file-url-mimetypes-3-expected.txt: Copied from platform/mac-snowleopard/platform/mac/fast/loader/file-url-mimetypes-3-expected.txt.
2:52 PM Changeset in webkit [62703] by mrowe@apple.com
  • 2 edits in trunk/WebCore

Clean up the project file slightly.

Rubber-stamped by Sam Weinig.

Remove obsolete .exp files, add the new .exp.in file, and move ExportFileGenerator.cpp in to the Exports collection.

  • WebCore.xcodeproj/project.pbxproj:
2:48 PM Changeset in webkit [62702] by jamesr@google.com
  • 5 edits in trunk

2010-07-07 James Robinson <​jamesr@chromium.org> and Vincent Scheib <​schieb@chromium.org>

Reviewed by Dimitri Glazkov.

WebCore::GLES2Context should allow creating onscreen and offscreen contexts
​https://bugs.webkit.org/show_bug.cgi?id=41492

You should be able to create a GLES2Context for rendering both on screen
and off screen (to a texture). Currently WebGL does the latter by side
stepping the WebCore::GLES2Context API completely. Longer term it should
move over to using this.

  • src/GLES2Context.cpp: (WebCore::GLES2ContextInternal::initializeOnscreen): (WebCore::GLES2ContextInternal::initializeOffscreen): (WebCore::GLES2Context::createOnscreen): (WebCore::GLES2Context::createOffscreen):

2010-07-07 James Robinson <​jamesr@chromium.org> and Vincent Scheib <​schieb@chromium.org>

Reviewed by Dimitri Glazkov.

WebCore::GLES2Context should allow creating onscreen and offscreen contexts
​https://bugs.webkit.org/show_bug.cgi?id=41492

You should be able to create a GLES2Context for rendering both on screen
and off screen (to a texture). Chromium's layer renderer (used for compositing)
does the former. Currently WebGL does the latter by side stepping the
WebCore::GLES2Context API completely. Longer term it should move over to using this.

  • platform/chromium/GLES2Context.h:
  • platform/graphics/chromium/LayerRendererChromium.cpp: (WebCore::LayerRendererChromium::initGL):
2:42 PM Changeset in webkit [62701] by Simon Fraser
  • 3 edits in trunk/WebCore

2010-07-07 Simon Fraser <Simon Fraser>

Reviewed by Dan Bernstein.

Document::ownerElement() should return an HTMLFrameOwnerElement
​https://bugs.webkit.org/show_bug.cgi?id=41789

Change return type of RenderLayerCompositor::enclosingIFrameElement() to
HTMLFrameOwnerElement*. No behavioral changes.

  • rendering/RenderLayerCompositor.cpp: (WebCore::RenderLayerCompositor::shouldPropagateCompositingToEnclosingIFrame): (WebCore::RenderLayerCompositor::enclosingIFrameElement):
  • rendering/RenderLayerCompositor.h:
2:36 PM Changeset in webkit [62700] by andersca@apple.com
  • 4 edits
    1 delete in trunk/WebKitTools

Unify Mac and Windows TestNetscapePlugin main.cpp files
​https://bugs.webkit.org/show_bug.cgi?id=41798

Reviewed by Simon Fraser.

  • DumpRenderTree/DumpRenderTree.sln:
  • DumpRenderTree/TestNetscapePlugIn/main.cpp:

(strcasecmp):
(NP_Initialize):
(NP_GetEntryPoints):
(NP_Shutdown):
(NPP_New):
(NPP_Destroy):
(NPP_HandleEvent):

  • DumpRenderTree/TestNetscapePlugIn/win/TestNetscapePlugin.vcproj:
  • DumpRenderTree/win/TestNetscapePlugin/main.cpp: Removed.
2:25 PM Changeset in webkit [62699] by crogers@google.com
  • 6 edits in branches/audio/WebCore

More windows build fixes (courtesy of Ian Lewis)

2:22 PM Changeset in webkit [62698] by crogers@google.com
  • 8 edits in branches/audio/WebCore/audio

More fully implement doppler shift

2:21 PM Changeset in webkit [62697] by abarth@webkit.org
  • 5 edits in trunk

2010-07-07 Adam Barth <​abarth@webkit.org>

Reviewed by Sam Weinig.

Add reverseFind to Vector and deploy in HTML5 parser
​https://bugs.webkit.org/show_bug.cgi?id=41778

This method seems generally useful. I'm slightly surprised we don't
have it already.

  • wtf/Vector.h: (WTF::::reverseFind):

2010-07-07 Adam Barth <​abarth@webkit.org>

Reviewed by Sam Weinig.

Add reverseFind to Vector and deploy in HTML5 parser
​https://bugs.webkit.org/show_bug.cgi?id=41778

This patch moves reverseFind from begin an
HTMLFormattingElementList-specific concept to begin a general Vector
concept. Also, instead of using Entry as the type for operator==, we
now use elements directly. The old code compiled because the Entry
constructor wasn't explicit, which means we were churning refs on every
comparison!

  • html/HTMLFormattingElementList.cpp: (WebCore::HTMLFormattingElementList::find): (WebCore::HTMLFormattingElementList::bookmarkFor): (WebCore::HTMLFormattingElementList::insertAt): (WebCore::HTMLFormattingElementList::remove):
  • html/HTMLFormattingElementList.h: (WebCore::HTMLFormattingElementList::Entry::operator==): (WebCore::HTMLFormattingElementList::Entry::operator!=):
1:55 PM Changeset in webkit [62696] by Darin Adler
  • 16 edits in trunk

2010-07-07 Darin Adler <Darin Adler>

Reviewed by Adam Barth.

Turn on adoptRef assertion for RefCounted
​https://bugs.webkit.org/show_bug.cgi?id=41547

  • wtf/CrossThreadRefCounted.h: Fixed include style. Includes of other WTF headers should use "" includes; consistent in most WTF headers. Added a call to relaxAdoptionRequirement.
  • wtf/RefCounted.h: Fixed include style. Removed LOOSE_REF_COUNTED. Added relaxAdoptionRequirement.

2010-07-07 Darin Adler <Darin Adler>

Reviewed by Adam Barth.

Turn on adoptRef assertion for RefCounted
​https://bugs.webkit.org/show_bug.cgi?id=41547

The WebCore part of this fixes all the assertions I saw in testing.

  • html/FileReader.cpp: (WebCore::FileReader::readAsBinaryString): Added null checks. Callers from JavaScript can pass the wrong type, which becomes null. (WebCore::FileReader::readAsText): Ditto. (WebCore::FileReader::readAsDataURL): Ditto.
  • html/FileStreamClient.h: Removed unneeded include.
  • html/FileStreamProxy.cpp: (WebCore::FileStreamProxy::FileStreamProxy): Made inline and moved some of the code, including the ref, into the create function. (WebCore::FileStreamProxy::create): Moved some of the code from the constructor here. It's safe to ref once the object has been created and adopted.
  • html/FileStreamProxy.h: Changed create function to no longer be inlined. Also removed an unneeded include.
  • page/EventSource.cpp: (WebCore::EventSource::EventSource): Made inline, changed arguments and moved code that involves the need to ref this object into the create function. Also moved failure handling out there since it's cleaner to have a function that fails than a constructor. For example, the function can return 0. (WebCore::EventSource::create): Moved some of the code from the constructor here.
  • page/EventSource.h: Removed unneeded includes. Made the creation function non-inline. Changed the arguments to the constructor.
  • storage/StorageAreaSync.cpp: (WebCore::StorageAreaSync::StorageAreaSync): Made inline. Moved code that requires ref'ing this object out to the create function. (WebCore::StorageAreaSync::create): Moved some of the code from the constructor here.
  • storage/StorageAreaSync.h: Removed unneeded includes. Changed the type of one of the constructor arguments from String to const String&.
  • workers/SharedWorker.cpp: (WebCore::SharedWorker::SharedWorker): Made inline. Moved most of the setup code out of here into the create function. (WebCore::SharedWorker::create): Moved the code here.
  • workers/SharedWorker.h: Removed unneeded includes. Made the create function non-inline. Marked the toSharedWorker override private to catch people doing an unnecessary virtual function call if they already have a SharedWorker*.
  • workers/Worker.cpp: (WebCore::Worker::Worker): Made inline. Moved most of the setup code out of here into the create function. (WebCore::Worker::create): Moved the code here.
  • workers/Worker.h: Made the create function non-inline. Changed the arguments to the constructor.
1:32 PM Changeset in webkit [62695] by kevino@webkit.org
  • 2 edits in trunk/WebKitTools

[wx] Build fix. pageProperty needs to return a value.

1:26 PM Changeset in webkit [62694] by Chris Fleizach
  • 3 edits
    2 adds in trunk

AX: when a node's role changes, the AX tree might need to be updated
​https://bugs.webkit.org/show_bug.cgi?id=41784

Reviewed by Beth Dakin.

WebCore:

Test: platform/mac/accessibility/update-children-when-aria-role-changes.html

  • accessibility/AccessibilityRenderObject.cpp:

(WebCore::AccessibilityRenderObject::AccessibilityRenderObject):
(WebCore::AccessibilityRenderObject::updateAccessibilityRole):

LayoutTests:

  • platform/mac/accessibility/update-children-when-aria-role-changes-expected.txt: Added.
  • platform/mac/accessibility/update-children-when-aria-role-changes.html: Added.
1:04 PM Changeset in webkit [62693] by Simon Fraser
  • 8 edits in trunk/WebCore

2010-07-07 Simon Fraser <Simon Fraser>

Reviewed by Darin Adler.

Document::ownerElement() should return an HTMLFrameOwnerElement
​https://bugs.webkit.org/show_bug.cgi?id=41789

Change the return type of Document::ownerElement() to an HTMLFrameOwnerElement, for better
type checking. No behavioral changes.

  • accessibility/mac/AccessibilityObjectWrapper.mm:
  • dom/Document.cpp: (WebCore::Document::ownerElement):
  • dom/Document.h:
  • editing/SelectionController.cpp:
  • rendering/RenderBoxModelObject.cpp:
  • rendering/RenderLayer.cpp:
  • rendering/RenderView.cpp:
1:02 PM Changeset in webkit [62692] by ddkilzer@apple.com
  • 2 edits in trunk/WebKitTools

<​http://webkit.org/b/41788> commit-log-editor: wrong ChangeLog read when invoked from subdir with git

Reviewed by Anders Carlsson.

For both svn and git, commit-log-editor is invoked from the root
of the working directory. Unlike svn, git returns a list of
changed files that are relative to the directory where the
command was invoked. This caused the ChangeLog file in the root
directory to be read instead of the ChangeLog in the current
directory.

The fix is to use $ENV{PWD} as the base directory when fixing
the path to the ChangeLog files. With svn, this has no net
effect since $ENV{PWD} is the root of the working directory and
the ChangeLog paths are already relative to that directory.
With git, $ENV{PWD} is the directory that the commit was invoked
from, which fixes the ChangeLog paths so that the correct files
are read when creating the commit log entry.

Note that the call to makeFilePathRelative() was supposed to
address this issue, but it doesn't because (a) it does nothing
with svn working directories by design, and (b) it does nothing
with git working directories because it's invoked when the
current directory is the root of the working directory, thus
giving no relative path.

  • Scripts/commit-log-editor: Removed call to

makeFilePathRelative() since since it does nothing. Moved code
to fix up $changeLog path so that it's fixed before trying to
open the file, and use $ENV{PWD} as the base path. Also use
canonicalizePath() to clean up paths with "../" in them.

12:47 PM Changeset in webkit [62691] by Chris Fleizach
  • 3 edits
    2 adds in trunk

AX: when an element uses role="text" but no aria-label, it should default to textUnderElement()
​https://bugs.webkit.org/show_bug.cgi?id=41780

Reviewed by Beth Dakin.

WebCore:

Test: platform/mac/accessibility/static-text-role-uses-text-under-element.html

  • accessibility/AccessibilityRenderObject.cpp:

(WebCore::AccessibilityRenderObject::stringValue):

Fallback to textUnderElement() if text() does not return any explicitly set text.

(WebCore::AccessibilityRenderObject::accessibilityIsIgnored):

Remove erroneous check to ignore text elements that didn't have a explicit text value.

LayoutTests:

  • platform/mac/accessibility/static-text-role-uses-text-under-element-expected.txt: Added.
  • platform/mac/accessibility/static-text-role-uses-text-under-element.html: Added.
12:38 PM Changeset in webkit [62690] by kevino@webkit.org
  • 2 edits in trunk/WebKit/wx

[wx] Build fix after change in Page constructor.

12:27 PM Changeset in webkit [62689] by andersca@apple.com
  • 2 edits in trunk/JavaScriptCore

Try to fix the Windows build.

  • runtime/Collector.cpp:

(JSC::Heap::freeBlocks):

11:51 AM Changeset in webkit [62688] by andersca@apple.com
  • 2 edits in trunk/WebKitTools

Another attempt at fixing the Qt build.

  • DumpRenderTree/qt/TestNetscapePlugin/TestNetscapePlugin.pro:
11:51 AM Changeset in webkit [62687] by Simon Fraser
  • 5 edits in trunk/WebCore

2010-07-07 Simon Fraser <Simon Fraser>

Reviewed by Dan Bernstein.

REGRESSION: GMail becomes blank after closing other tabs
​https://bugs.webkit.org/show_bug.cgi?id=40421

RenderLayerCompositor's attach/detachRootPlatformLayer methods use
setNeedsStyleRecalc() in order to trigger acclerated compositing layers
to be hooked together across iframe boundaries. However, it was possible
for these to get called while inside of Document::recalcStyle(), which
is bad because it can cause the recalc to fail to get processed.

Fix this by using the existing queuePostAttachCallback() functionality
to delay the call to setNeedsStyleRecalc() if post-attach callbacks
are suspended (indicating that we're inside recalcStyle()).

No new tests because I wasn't able to make a test that shows the problem.

  • dom/ContainerNode.h: Make queuePostAttachCallback() public.
  • dom/ContainerNode.cpp: (WebCore::ContainerNode::postAttachCallbacksAreSuspended): Added; returns whether s_attachDepth is non-zero.
  • rendering/RenderLayerCompositor.cpp: (WebCore::RenderLayerCompositor::attachRootPlatformLayer): Call scheduleNeedsStyleRecalc() instead of setNeedsStyleRecalc(). (WebCore::RenderLayerCompositor::detachRootPlatformLayer): Ditto. (WebCore::needsStyleRecalcCallback): Here we call setNeedsStyleRecalc(). (WebCore::RenderLayerCompositor::scheduleNeedsStyleRecalc): (WebCore::RenderLayerCompositor::notifyIFramesOfCompositingChange):
  • rendering/RenderLayerCompositor.h: Add scheduleNeedsStyleRecalc().
11:50 AM Changeset in webkit [62686] by andersca@apple.com
  • 2 edits in trunk/WebKitTools

Try to fix Qt build.

  • DumpRenderTree/qt/TestNetscapePlugin/TestNetscapePlugin.pro:
11:42 AM Changeset in webkit [62685] by andersca@apple.com
  • 4 edits
    11 moves
    3 adds in trunk

Rename TestNetscapePlugin.subproj and move platform specific files to subdirectories
​https://bugs.webkit.org/show_bug.cgi?id=41781

Reviewed by Simon Fraser.

WebKit/win:

  • WebKit.vcproj/WebKit.sln:

WebKitTools:

  • DumpRenderTree/DumpRenderTree.xcodeproj/project.pbxproj:
  • DumpRenderTree/TestNetscapePlugIn/PluginObject.cpp: Renamed from WebKitTools/DumpRenderTree/TestNetscapePlugIn.subproj/PluginObject.cpp.
  • DumpRenderTree/TestNetscapePlugIn/PluginObject.h: Renamed from WebKitTools/DumpRenderTree/TestNetscapePlugIn.subproj/PluginObject.h.
  • DumpRenderTree/TestNetscapePlugIn/TestObject.cpp: Renamed from WebKitTools/DumpRenderTree/TestNetscapePlugIn.subproj/TestObject.cpp.
  • DumpRenderTree/TestNetscapePlugIn/TestObject.h: Renamed from WebKitTools/DumpRenderTree/TestNetscapePlugIn.subproj/TestObject.h.
  • DumpRenderTree/TestNetscapePlugIn/mac/Info.plist: Renamed from WebKitTools/DumpRenderTree/TestNetscapePlugIn.subproj/Info.plist.
  • DumpRenderTree/TestNetscapePlugIn/main.cpp: Renamed from WebKitTools/DumpRenderTree/TestNetscapePlugIn.subproj/main.cpp.
  • DumpRenderTree/TestNetscapePlugIn/win/TestNetscapePlugin.def: Renamed from WebKitTools/DumpRenderTree/win/TestNetscapePlugin/TestNetscapePlugin.def.
  • DumpRenderTree/TestNetscapePlugIn/win/TestNetscapePlugin.rc: Renamed from WebKitTools/DumpRenderTree/win/TestNetscapePlugin/TestNetscapePlugin.rc.
  • DumpRenderTree/TestNetscapePlugIn/win/TestNetscapePlugin.vcproj: Added.
  • DumpRenderTree/TestNetscapePlugIn/win/TestNetscapePlugin_debug.def: Renamed from WebKitTools/DumpRenderTree/win/TestNetscapePlugin/TestNetscapePlugin_debug.def.
  • DumpRenderTree/TestNetscapePlugIn/win/resource.h: Renamed from WebKitTools/DumpRenderTree/win/TestNetscapePlugin/resource.h.
  • DumpRenderTree/win/TestNetscapePlugin/TestNetscapePlugin.vcproj: Removed.
11:25 AM Changeset in webkit [62684] by ap@apple.com
  • 1 edit
    1 copy in trunk/LayoutTests

Not reviewed, just updating test expectations.

​https://bugs.webkit.org/show_bug.cgi?id=41156
Cross origin XMLHttpRequest should log the reason why connection failed

Synchronous resource loader on Tiger doesn't return a response if authentication failed.
It only returns an error, so WebKit cannot find out that the cross origin 401 response can
be passed to caller.

Observable behavior is still the same (failure ot load), but an incorrect error is logged
to console.

  • platform/mac-tiger/http/tests/xmlhttprequest/cross-origin-authorization-with-embedder-expected.txt: Copied from http/tests/xmlhttprequest/cross-origin-authorization-with-embedder-expected.txt.
10:48 AM Changeset in webkit [62683] by Simon Fraser
  • 3 edits
    5 adds in trunk

2010-07-06 Simon Fraser <Simon Fraser>

Reviewed by Dan Bernstein.

Incorrect compositing order with negative z-index
​https://bugs.webkit.org/show_bug.cgi?id=38959

When painting the contents of compositing layers whose phase is "GraphicsLayerPaintBackground"
(indicating that they are used for the background of elements with negative z-index children),
we need to paint the non-composited negative-z-order descendants at the end of the background phase, so
they appear behind composited negative-z-order descendants.

Test: compositing/z-order/negative-z-index.html

  • rendering/RenderLayerBacking.cpp: (WebCore::RenderLayerBacking::paintIntoLayer):
10:48 AM Changeset in webkit [62682] by Simon Fraser
  • 2 edits in trunk/WebCore

2010-07-06 Simon Fraser <Simon Fraser>

Reviewed by Dan Bernstein.

Incorrect compositing order with negative z-index
​https://bugs.webkit.org/show_bug.cgi?id=38959

Prerequisite: adopt new paintList() method that was added when painting compositing layer contents.
No behavioral changes.

  • rendering/RenderLayerBacking.cpp: (WebCore::RenderLayerBacking::paintIntoLayer):
10:35 AM Changeset in webkit [62681] by abarth@webkit.org
  • 2 edits in trunk/WebCore

2010-07-07 Adam Barth <​abarth@webkit.org>

Unreviewed.

Fix ASSERT in Eric's previous patch. I think he tested his patch in
Release and not Debug.

  • html/HTMLFormattingElementList.h: (WebCore::HTMLFormattingElementList::Entry::Entry): (WebCore::HTMLFormattingElementList::findIndex):
10:15 AM Changeset in webkit [62680] by abarth@webkit.org
  • 3 edits in trunk/WebCore

2010-07-07 Eric Seidel <​eric@webkit.org>

Reviewed by Adam Barth.

Grease the TreeBuilder's lightning
​https://bugs.webkit.org/show_bug.cgi?id=41756

Brings the new TreeBuilder from 7s to 3s on the parser benchmark.
This makes performance comparable to the old parser.

We have not begun to fight! There is so much fat left on these bones.

  • html/HTMLFormattingElementList.cpp: (WebCore::HTMLFormattingElementList::find): (WebCore::HTMLFormattingElementList::bookmarkFor): (WebCore::HTMLFormattingElementList::insertAt): (WebCore::HTMLFormattingElementList::remove):
  • html/HTMLFormattingElementList.h: (WebCore::HTMLFormattingElementList::findIndex):
10:11 AM Changeset in webkit [62679] by andersca@apple.com
  • 3 edits in trunk/WebKit/mac

Change requestURL to a KURL.
​https://bugs.webkit.org/show_bug.cgi?id=41775

Reviewed by Sam Weinig.

  • Plugins/WebNetscapePluginStream.h:

(WebNetscapePluginStream::setRequestURL):

  • Plugins/WebNetscapePluginStream.mm:

(WebNetscapePluginStream::pluginCancelledConnectionError):
(WebNetscapePluginStream::errorForReason):
(WebNetscapePluginStream::startStream):
(WebNetscapePluginStream::didReceiveResponse):
(WebNetscapePluginStream::destroyStream):

10:11 AM Changeset in webkit [62678] by abarth@webkit.org
  • 3 edits in trunk/WebCore

2010-07-07 Eric Seidel <​eric@webkit.org>

Reviewed by Adam Barth.

HTMLTreeBuilder is way too slow
​https://bugs.webkit.org/show_bug.cgi?id=41754

This takes us from 14s to 7s on our parsing benchmark.
That's still much slower than the old tree builder, but there
is a huge amount of fat left to trim.

Vector<T> wasn't able to inline all the Entry functions when
they were buried in the cpp. Turns out the active formatting elements
list is very hot.

I'm not sure Vector<T> is going to be the right data structure for us
in the end, but it has done alright for bring-up.

  • html/HTMLFormattingElementList.cpp:
  • html/HTMLFormattingElementList.h: (WebCore::HTMLFormattingElementList::Entry::Entry): (WebCore::HTMLFormattingElementList::Entry::~Entry): (WebCore::HTMLFormattingElementList::Entry::isMarker): (WebCore::HTMLFormattingElementList::Entry::element): (WebCore::HTMLFormattingElementList::Entry::replaceElement): (WebCore::HTMLFormattingElementList::Entry::operator==): (WebCore::HTMLFormattingElementList::Entry::operator!=):
9:53 AM Changeset in webkit [62677] by Darin Adler
  • 34 edits in trunk

2010-07-07 Darin Adler <Darin Adler>

Reviewed by Adam Barth.

More OwnPtr work
​https://bugs.webkit.org/show_bug.cgi?id=41727

  • API/JSCallbackObject.h: (JSC::JSCallbackObjectData::setPrivateProperty): Use adoptPtr.
  • API/JSCallbackObjectFunctions.h: (JSC::JSCallbackObject::JSCallbackObject): Ditto.
  • bytecode/CodeBlock.cpp: (JSC::CodeBlock::CodeBlock): Ditto.
  • bytecode/CodeBlock.h: (JSC::CodeBlock::createRareDataIfNecessary): Ditto.
  • parser/Nodes.cpp: (JSC::ScopeNode::ScopeNode): Ditto.
  • parser/ParserArena.cpp: (JSC::ParserArena::ParserArena): Ditto.
  • runtime/Arguments.h: (JSC::Arguments::Arguments): Ditto.
  • runtime/Executable.cpp: (JSC::EvalExecutable::compile): Ditto. (JSC::ProgramExecutable::compile): Ditto. (JSC::FunctionExecutable::compileForCall): Ditto. (JSC::FunctionExecutable::compileForConstruct): Ditto. (JSC::FunctionExecutable::reparseExceptionInfo): Ditto. (JSC::EvalExecutable::reparseExceptionInfo): Ditto.
  • runtime/JSArray.cpp: (JSC::JSArray::sort): Ditto.
  • runtime/RegExpConstructor.cpp: (JSC::RegExpConstructor::RegExpConstructor): Ditto.
  • runtime/RegExpObject.cpp: (JSC::RegExpObject::RegExpObject): Ditto.
  • runtime/SmallStrings.cpp: (JSC::SmallStrings::createSingleCharacterString): Ditto. (JSC::SmallStrings::singleCharacterStringRep): Ditto.
  • wtf/unicode/icu/CollatorICU.cpp: (WTF::Collator::userDefault): Use adoptPtr.
  • yarr/RegexInterpreter.cpp: (JSC::Yarr::ByteCompiler::ByteCompiler): Ditto. (JSC::Yarr::ByteCompiler::compile): Ditto. (JSC::Yarr::ByteCompiler::regexBegin): Ditto. (JSC::Yarr::byteCompileRegex): Ditto.
  • yarr/RegexInterpreter.h: (JSC::Yarr::BytecodePattern::BytecodePattern): Ditto.

2010-07-06 Darin Adler <Darin Adler>

Reviewed by Adam Barth.

More OwnPtr work, including making clear set the pointer to 0 before deletion
​https://bugs.webkit.org/show_bug.cgi?id=41727

  • WebCore.exp.in: Updated.
  • css/CSSSelector.h: (WebCore::CSSSelector::RareData::RareData): Use adoptPtr. (WebCore::CSSSelector::createRareData): Ditto.
  • dom/SpaceSplitString.h: (WebCore::SpaceSplitString::SpaceSplitString): Ditto. (WebCore::SpaceSplitString::set): Ditto.
  • history/CachedFrame.cpp: (WebCore::CachedFrame::CachedFrame): Ditto. (WebCore::CachedFrame::setCachedFramePlatformData): Ditto.
  • history/CachedFrame.h: Use PassOwnPtr.
  • loader/appcache/ApplicationCacheGroup.cpp: (WebCore::CallCacheListenerTask::create): Use adoptPtr.
  • loader/appcache/ApplicationCacheStorage.cpp: (WebCore::ApplicationCacheStorage::storeCopyOfCache): Ditto.
  • platform/PurgeableBuffer.h: (WebCore::PurgeableBuffer::create): Ditto.
  • platform/graphics/GlyphMetricsMap.h: (WebCore::::locatePageSlowCase): Ditto.
  • platform/graphics/GraphicsLayer.h: (WebCore::AnimationValue::AnimationValue): Ditto. (WebCore::TransformAnimationValue::TransformAnimationValue): Ditto.
  • platform/graphics/MediaPlayer.h: (WebCore::MediaPlayer::create): Ditto.
  • platform/graphics/SimpleFontData.h: (WebCore::SimpleFontData::boundsForGlyph): Ditto.
  • platform/mac/PurgeableBufferMac.cpp: (WebCore::PurgeableBuffer::create): Ditto.
  • rendering/InlineFlowBox.h: (WebCore::InlineFlowBox::setHorizontalOverflowPositions): Ditto. (WebCore::InlineFlowBox::setVerticalOverflowPositions): Ditto.
  • rendering/RootInlineBox.h: (WebCore::RootInlineBox::floats): Ditto.
  • rendering/style/RenderStyle.h: (WebCore::InheritedFlags::inheritAnimations): Ditto. (WebCore::InheritedFlags::inheritTransitions): Ditto.
  • rendering/style/SVGRenderStyleDefs.h: Use PassOwnPtr.
9:44 AM Changeset in webkit [62676] by weinig@apple.com
  • 22 edits
    2 adds in trunk

Patch for ​https://bugs.webkit.org/show_bug.cgi?id=41772
Add basic piping for BackForwardControllerClient.

Reviewed by Anders Carlsson.

WebCore:

  • WebCore.exp.in:
  • WebCore.xcodeproj/project.pbxproj:
  • history/BackForwardController.cpp:

(WebCore::BackForwardController::BackForwardController):

  • history/BackForwardControllerClient.h:
  • page/Page.cpp:

(WebCore::Page::Page):

  • page/Page.h:
  • svg/graphics/SVGImage.cpp:

(WebCore::SVGImage::dataChanged):

WebKit/chromium:

  • src/WebViewImpl.cpp:

(WebKit::WebViewImpl::WebViewImpl):

WebKit/gtk:

  • webkit/webkitwebview.cpp:

(webkit_web_view_init):

WebKit/mac:

  • WebView/WebView.mm:

(-[WebView _commonInitializationWithFrameName:groupName:usesDocumentViews:]):

WebKit/qt:

  • Api/qwebpage.cpp:

(QWebPagePrivate::QWebPagePrivate):

WebKit/win:

  • WebView.cpp:

(WebView::initWithFrame):

WebKit2:

  • Add very basic WebBackForwardControllerClient.
  • WebKit2.xcodeproj/project.pbxproj:
  • WebProcess/WebCoreSupport/WebBackForwardControllerClient.cpp: Added.

(WebKit::WebBackForwardControllerClient::backForwardControllerDestroyed):
(WebKit::WebBackForwardControllerClient::createBackForwardList):

  • WebProcess/WebCoreSupport/WebBackForwardControllerClient.h: Added.

(WebKit::WebBackForwardControllerClient::WebBackForwardControllerClient):

  • WebProcess/WebPage/WebPage.cpp:

(WebKit::WebPage::WebPage):

  • win/WebKit2.vcproj:
9:43 AM Changeset in webkit [62675] by mitz@apple.com
  • 1 edit
    2 copies
    3 deletes in trunk/LayoutTests

Move current Mac results from platform/mac-snowleopard to platform/mac,
move Leopard results from platform/mac to platform/mac-leopard, and remove
stray files and directories from platform/mac-snowleopard.

Reviewed by Sam Weinig.

  • platform/mac-leopard/svg/custom/use-font-face-crash-expected.txt: Copied from platform/mac/svg/custom/use-font-face-crash-expected.txt.
  • platform/mac-snowleopard/platform: Removed.
  • platform/mac-snowleopard/platform/mac: Removed.
  • platform/mac-snowleopard/platform/mac/fast: Removed.
  • platform/mac-snowleopard/platform/mac/fast/loader: Removed.
  • platform/mac-snowleopard/platform/mac/fast/loader/file-url-mimetypes-2-expected.txt: Removed.
  • platform/mac-snowleopard/platform/mac/fast/loader/file-url-mimetypes-3-expected.txt: Removed.
  • platform/mac-snowleopard/svg: Removed.
  • platform/mac-snowleopard/svg/custom: Removed.
  • platform/mac-snowleopard/svg/custom/use-font-face-crash-expected.txt: Removed.
  • platform/mac-snowleopard/tables: Removed.
  • platform/mac-snowleopard/tables/mozilla: Removed.
  • platform/mac-snowleopard/tables/mozilla/images: Removed.
  • platform/mac/svg/custom/use-font-face-crash-expected.txt: Replaced with platform/mac-snowleopard/svg/custom/use-font-face-crash-expected.txt.
9:07 AM Changeset in webkit [62674] by Darin Adler
  • 8 edits in trunk/JavaScriptCore

Make clear set the pointer to 0 before deletion
​https://bugs.webkit.org/show_bug.cgi?id=41727

Reviewed by Adam Barth.

  • wtf/OwnArrayPtr.h: Changed code so we always set the pointer to its new

value before deleting the old one, including in the set function and the
clear function. This required changing safeDelete.

  • wtf/OwnPtr.h: Ditto. Also removed some extra null checks.
  • wtf/PassOwnPtr.h: Ditto.
  • wtf/PassRefPtr.h: Changed code so we always set the pointer to its new

value before deref'ing the old one in the clear function. Also added a
leakRef function for NonNullPassRefPtr.

  • wtf/RefPtr.h: Ditto.
  • wtf/gobject/GOwnPtr.h: More of the same.
  • wtf/gobject/GRefPtr.h: Ditto.
8:58 AM Changeset in webkit [62673] by andersca@apple.com
  • 4 edits
    2 moves in trunk/WebKit

Rename the WebBaseNetscapePluginStream.cpp and .h files to WebNetscapePluginStream.cpp and .h to match the class name.

Reviewed by Sam Weinig.

WebKit:

  • WebKit.xcodeproj/project.pbxproj:

WebKit/mac:

  • Plugins/WebNetscapePluginStream.h: Renamed from WebKit/mac/Plugins/WebBaseNetscapePluginStream.h.
  • Plugins/WebNetscapePluginStream.mm: Renamed from WebKit/mac/Plugins/WebBaseNetscapePluginStream.mm.
8:39 AM Changeset in webkit [62672] by tonikitoo@webkit.org
  • 2 edits in trunk/WebCore

Unreviewed build fix after r62577.

Patch by Lucas De Marchi <​lucas.demarchi@profusion.mobi> on 2010-07-07
[EFL] build fix after r62577
​https://bugs.webkit.org/show_bug.cgi?id=41764

No new functionality so no new tests.

8:32 AM Changeset in webkit [62671] by kov@webkit.org
  • 2 edits in trunk/WebCore

2010-07-07 Gustavo Noronha Silva <​gustavo.noronha@collabora.co.uk>

Reviewed by Xan Lopez.

[GTK] Scrollbars sometimes go dead and stop scrolling the view
​https://bugs.webkit.org/show_bug.cgi?id=41711

Rework the setGtkAdjustment function slightly, make it have an
early return, and be more readable.

  • platform/gtk/ScrollViewGtk.cpp: (WebCore::ScrollView::setGtkAdjustments):
8:15 AM Changeset in webkit [62670] by vitalyr@chromium.org
  • 2 edits in trunk/LayoutTests

2010-07-07 Vitaly Repeshko <​vitalyr@chromium.org>

Unreviewed.

[chromium] Update more expectations.

  • platform/chromium/test_expectations.txt:
8:05 AM Changeset in webkit [62669] by vitalyr@chromium.org
  • 4 edits
    3 adds in trunk/LayoutTests

2010-07-07 Vitaly Repeshko <​vitalyr@chromium.org>

Unreviewed.

[chromium] Update expectations.

  • platform/chromium-mac/html5lib/runner-expected.txt:
  • platform/chromium-win/html5lib/runner-expected.txt:
  • platform/chromium/test_expectations.txt:
8:01 AM Changeset in webkit [62668] by commit-queue@webkit.org
  • 2 edits in trunk/WebCore

2010-07-07 Kristian Monsen <​kristianm@google.com>

Reviewed by Steve Block.

Updating Android to use the UserGestureIndicator class instead of Android specific code.
The rest of the changes are in parts of the Android port which are yet to be upstreamed.

No change in behavior, so no new test.

  • platform/network/android/ResourceRequest.h: (WebCore::ResourceRequest::ResourceRequest): (WebCore::ResourceRequest::doUpdateResourceRequest):
7:50 AM Changeset in webkit [62667] by Nikolas Zimmermann
  • 1 edit
    3 adds in trunk/LayoutTests

2010-07-07 Nikolas Zimmermann <​nzimmermann@rim.com>

Not reviewed. Add snow-leopard specific result for svg/custom/use-font-face-crash.svg

  • platform/mac-snowleopard/svg: Added.
  • platform/mac-snowleopard/svg/custom: Added.
  • platform/mac-snowleopard/svg/custom/use-font-face-crash-expected.txt: Added.
7:49 AM Changeset in webkit [62666] by commit-queue@webkit.org
  • 9 edits in trunk/WebKit

2010-07-07 Gyuyoung Kim <​gyuyoung.kim@samsung.com>

Reviewed by Kenneth Rohde Christiansen.

[EFL] EFLWebKit doesn't support viewport meta tag.
Support viewport meta tag on EFL Port.
​https://bugs.webkit.org/show_bug.cgi?id=40278

  • efl/EWebLauncher/main.c: Process 'viewport,changed' signal. (viewport_set): (on_viewport_changed): Add a callback function for the 'viewport,changed' signal. (browserCreate): Register the callback function for the 'viewport,changed' signal.
  • efl/WebCoreSupport/ChromeClientEfl.cpp: Implement method to process viewport tag. (WebCore::ChromeClientEfl::didReceiveViewportArguments):
  • efl/WebCoreSupport/ChromeClientEfl.h: Define method to process viewport tag.
  • efl/WebCoreSupport/FrameLoaderClientEfl.cpp: (WebCore::FrameLoaderClientEfl::FrameLoaderClientEfl): Implement a function to set layout when website doesn't contain viewport tag. (WebCore::FrameLoaderClientEfl::dispatchDidCommitLoad): (WebCore::FrameLoaderClientEfl::dispatchDidFirstLayout):
  • efl/WebCoreSupport/FrameLoaderClientEfl.h: (WebCore::FrameLoaderClientEfl::setInitLayoutCompleted): (WebCore::FrameLoaderClientEfl::getInitLayoutCompleted):
  • efl/ewk/ewk_private.h: Added internal APIs to process viewport tag.
  • efl/ewk/ewk_view.cpp: Added APIs to process and to handle viewport tag. (_ewk_view_priv_new): (ewk_view_fixed_layout_size_set): (ewk_view_zoom_set): (ewk_view_zoom_weak_set): (ewk_view_zoom_animated_set): (ewk_view_viewport_set): Reports that viewport has changed. (ewk_view_viewport_get): Reports that viewport has changed. (ewk_view_zoom_range_set): Sets the zoom range. (ewk_view_zoom_range_min_get): Gets minimum value of zoom range. (ewk_view_zoom_range_max_get): Gets maximum value of zoom range. (ewk_view_user_scalable_set): Sets if zoom is enabled. (ewk_view_user_scalable_get): Gets if zoom is enabled.
  • efl/ewk/ewk_view.h:
7:41 AM Changeset in webkit [62665] by steveblock@google.com
  • 2 edits in trunk/LayoutTests

Unreviewed fix for failing Chromium test

7:37 AM Changeset in webkit [62664] by aestes@apple.com
  • 4 edits
    4 adds in trunk

2010-07-07 Andy Estes <​aestes@apple.com>

Reviewed by Adam Barth.

Test that loading of images in <object> tags can be cancelled by
beforeload event handlers.
​https://bugs.webkit.org/show_bug.cgi?id=41027
<rdar://problem/8120596>

  • fast/dom/beforeload/image-object-before-load-expected.txt: Added.
  • fast/dom/beforeload/image-object-before-load-innerHTML-expected.txt: Added.
  • fast/dom/beforeload/image-object-before-load-innerHTML.html: Added.
  • fast/dom/beforeload/image-object-before-load.html: Added.

2010-07-07 Andy Estes <​aestes@apple.com>

Reviewed by Adam Barth.

Allow a beforeload listener to prevent loading of images in <object> tags.
​https://bugs.webkit.org/show_bug.cgi?id=41027
<rdar://problem/8120596>

Tests: fast/dom/beforeload/image-object-before-load-innerHTML.html

fast/dom/beforeload/image-object-before-load.html

  • html/HTMLObjectElement.cpp: (WebCore::HTMLObjectElement::attach): Do not call RenderImage::setCachedImage() at the end of attach(). Instead, allow this to happen conditionally after beforeload is dispatched.
  • loader/ImageLoader.cpp: (WebCore::ImageLoader::dispatchPendingBeforeLoadEvent): Render fallback content if an object's load was cancelled.
7:23 AM Changeset in webkit [62663] by commit-queue@webkit.org
  • 5 edits in trunk

2010-07-07 Rafael Antognolli <​antognolli@profusion.mobi>

Reviewed by Adam Barth.

[EFL] Fix cmake build and libsoup detection
Some changes should be done to cmake build system to fix the detection of libsoup and correct some link paths:

  • LINK_FLAGS should be set using quotes, otherwise some wrong substitution takes place;
  • we should use LIBSOUP24_* instead of LIBSOUP_* since this is the version we are using;
  • need to set HAVE_LIBSOUP_2_29_90 if we find a version equal or newer than that. ​https://bugs.webkit.org/show_bug.cgi?id=41717
  • cmake/OptionsEfl.cmake:

2010-07-07 Rafael Antognolli <​antognolli@profusion.mobi>

Reviewed by Adam Barth.

[EFL] Fix cmake build and libsoup detection
Some changes should be done to cmake build system to fix the detection of libsoup and correct some link paths:

  • LINK_FLAGS should be set using quotes, otherwise some wrong substitution takes place;
  • we should use LIBSOUP24_* instead of LIBSOUP_* since this is the version we are using;
  • need to set HAVE_LIBSOUP_2_29_90 if we find a version equal or newer than that. ​https://bugs.webkit.org/show_bug.cgi?id=41717
  • CMakeLists.txt:
  • CMakeListsEfl.txt:
7:17 AM Changeset in webkit [62662] by Nikolas Zimmermann
  • 4 edits
    4 adds in trunk

2010-07-06 Nikolas Zimmermann <​nzimmermann@rim.com>

Reviewed by Dirk Schulze.

<use> on <font-face> causes crashes, if SVGUseElement gets detached
​https://bugs.webkit.org/show_bug.cgi?id=41621

Do not call removeFromMappedElementSheet() from the SVGFontFaceElement destructor,
as that can potentially cause the element to be reattached while destructing.

In order to fix the crash in the testcase, the order of calling the base-class detach
method in SVGUseElement and the instance/shadow tree destruction has to be reversed,
matching the order in removedFromDocument().

Test: svg/custom/use-font-face-crash.svg

  • svg/SVGFontFaceElement.cpp: (WebCore::SVGFontFaceElement::~SVGFontFaceElement): Remove removeFromMappedElementSheet() call.
  • svg/SVGUseElement.cpp: (WebCore::SVGUseElement::detach): Reverse order of calling base-class detach method and instance/shadow tree destruction.
7:10 AM Changeset in webkit [62661] by commit-queue@webkit.org
  • 7 edits in trunk/JavaScriptCore/qt

2010-07-07 Caio Marcelo de Oliveira Filho <​caio.oliveira@openbossa.org>

Reviewed by Kenneth Rohde Christiansen.

Implementation of QScriptValue::isArray()
​https://bugs.webkit.org/show_bug.cgi?id=41713

Since we don't have access to the Class? internal property of
builtins (including Array), the solution was to keep the original 'Array'
(constructor) and 'Array.prototype' objects and use them to identify
if a given object is an Array.

Also uncomment some tests and add some tests of newArray() that
depended on isArray().

  • api/qscriptengine_p.cpp: (QScriptEnginePrivate::QScriptEnginePrivate): (QScriptEnginePrivate::~QScriptEnginePrivate):
  • api/qscriptengine_p.h: (QScriptEnginePrivate::isArray):
  • api/qscriptvalue.cpp: (QScriptValue::isArray):
  • api/qscriptvalue.h:
  • api/qscriptvalue_p.h: (QScriptValuePrivate::isArray):
  • tests/qscriptengine/tst_qscriptengine.cpp: (tst_QScriptEngine::newArray):
7:01 AM Changeset in webkit [62660] by Adam Roben
  • 2 edits in trunk/WebKit/win

Windows build fix

  • Interfaces/WebKit.idl: Touched to force a build.
6:58 AM Changeset in webkit [62659] by steveblock@google.com
  • 4 edits
    3 adds in trunk

2010-07-07 Steve Block <​steveblock@google.com>

Reviewed by Adam Barth.

document.createEvent() should support DeviceOrientationEvent
​https://bugs.webkit.org/show_bug.cgi?id=41618

  • fast/dom/DeviceOrientation/create-event-expected.txt: Added.
  • fast/dom/DeviceOrientation/create-event.html: Added.
  • fast/dom/DeviceOrientation/script-tests/create-event.js: Added.
  • platform/gtk/Skipped:

2010-07-07 Steve Block <​steveblock@google.com>

Reviewed by Adam Barth.

document.createEvent() should support DeviceOrientationEvent
​https://bugs.webkit.org/show_bug.cgi?id=41618

Tests: fast/dom/DeviceOrientation/create-event.html

  • dom/Document.cpp: (WebCore::Document::createEvent):
6:06 AM Changeset in webkit [62658] by yurys@chromium.org
  • 3 edits in trunk/WebCore

2010-07-07 Yury Semikhatsky <​yurys@chromium.org>

Reviewed by Pavel Feldman.

Web Inspector: show functions from internal browser scripts on the call stack if they
are there.
​https://bugs.webkit.org/show_bug.cgi?id=41762

  • English.lproj/localizedStrings.js:
  • inspector/front-end/CallStackSidebarPane.js: (WebInspector.CallStackSidebarPane.prototype.update):
6:04 AM Changeset in webkit [62657] by Csaba Osztrogonác
  • 2 edits
    4 adds in trunk/LayoutTests

Unreviewed fix.

[Qt] Remove non-existent files from Skipped list, and
add Qt specific expected result for svg/hixie/text/001.xml

  • platform/qt/Skipped: Remove non-existent files.
    • svg/hixie/text/001-broken.xml, because it was renamed by r61393.
    • tables/mozilla/images/adforce_imgis_com.html, because it was removed by r61289.
    • inspector/debugger-breakpoints-not-activated-on-reload.html, because it was removed by r62651.
  • platform/qt/svg/hixie/text: Added.
  • platform/qt/svg/hixie/text/001-expected.checksum: Added.
  • platform/qt/svg/hixie/text/001-expected.png: Added.
  • platform/qt/svg/hixie/text/001-expected.txt: Added.
6:01 AM Changeset in webkit [62656] by loislo@chromium.org
  • 6 edits in trunk/WebCore

2010-07-07 Ilya Tikhonovsky <​loislo@chromium.org>

Reviewed by Pavel Feldman.

Web Inspector: All RefPtr arguments and return values should be
changed to PassRefPtr whenever it is possible according to common practice.
​https://bugs.webkit.org/show_bug.cgi?id=41759

  • inspector/CodeGeneratorInspector.pm:
  • inspector/InspectorTimelineAgent.cpp: (WebCore::InspectorTimelineAgent::pushGCEventRecords): (WebCore::InspectorTimelineAgent::didInstallTimer): (WebCore::InspectorTimelineAgent::didRemoveTimer): (WebCore::InspectorTimelineAgent::didScheduleResourceRequest): (WebCore::InspectorTimelineAgent::willSendResourceRequest): (WebCore::InspectorTimelineAgent::didFinishLoadingResource): (WebCore::InspectorTimelineAgent::didMarkTimeline): (WebCore::InspectorTimelineAgent::didMarkDOMContentEvent): (WebCore::InspectorTimelineAgent::didMarkLoadEvent): (WebCore::InspectorTimelineAgent::addRecordToTimeline): (WebCore::InspectorTimelineAgent::setHeapSizeStatistic): (WebCore::InspectorTimelineAgent::pushCurrentRecord):
  • inspector/InspectorTimelineAgent.h: (WebCore::InspectorTimelineAgent::TimelineRecordEntry::TimelineRecordEntry):
  • inspector/TimelineRecordFactory.cpp: (WebCore::TimelineRecordFactory::createGenericRecord): (WebCore::TimelineRecordFactory::createGCEventData): (WebCore::TimelineRecordFactory::createFunctionCallData): (WebCore::TimelineRecordFactory::createEventDispatchData): (WebCore::TimelineRecordFactory::createGenericTimerData): (WebCore::TimelineRecordFactory::createTimerInstallData): (WebCore::TimelineRecordFactory::createXHRReadyStateChangeData): (WebCore::TimelineRecordFactory::createXHRLoadData): (WebCore::TimelineRecordFactory::createEvaluateScriptData): (WebCore::TimelineRecordFactory::createMarkTimelineData): (WebCore::TimelineRecordFactory::createScheduleResourceRequestData): (WebCore::TimelineRecordFactory::createResourceSendRequestData): (WebCore::TimelineRecordFactory::createResourceReceiveResponseData): (WebCore::TimelineRecordFactory::createResourceFinishData): (WebCore::TimelineRecordFactory::createReceiveResourceData): (WebCore::TimelineRecordFactory::createPaintData): (WebCore::TimelineRecordFactory::createParseHTMLData):
  • inspector/TimelineRecordFactory.h:
5:41 AM Changeset in webkit [62655] by antti.j.koivisto@nokia.com
  • 2 edits in trunk/WebKit2

[Qt] Initial WebKit2 implementation
​https://bugs.webkit.org/show_bug.cgi?id=40233

Reviewed by Kenneth Rohde Christiansen.

Add Qt section to the prefix header.

  • WebKit2Prefix.h:
5:37 AM Changeset in webkit [62654] by antti.j.koivisto@nokia.com
  • 3 edits in trunk/WebCore

[Qt] Initial WebKit2 implementation
​https://bugs.webkit.org/show_bug.cgi?id=40233

Reviewed by Kenneth Rohde Christiansen.

Export some event conversion functions needed for WebKit2.

  • platform/PlatformKeyboardEvent.h:
  • platform/qt/PlatformKeyboardEventQt.cpp:

(WebCore::keyIdentifierForQtKeyCode):
(WebCore::windowsKeyCodeForKeyEvent):

4:51 AM Changeset in webkit [62653] by steveblock@google.com
  • 10 edits
    2 moves in trunk

2010-07-06 Steve Block <​steveblock@google.com>

Reviewed by Adam Barth.

DeviceOrientationEventConstructor should be exposed at window.DeviceOrientationEvent
​https://bugs.webkit.org/show_bug.cgi?id=41685

Test: fast/dom/Window/window-properties-device-orientation.html

fast/dom/DeviceOrientation/window-property.txt

  • page/DOMWindow.idl:

2010-07-06 Steve Block <​steveblock@google.com>

Reviewed by Adam Barth.

DeviceOrientationEventConstructor should be exposed at window.DeviceOrientationEvent
​https://bugs.webkit.org/show_bug.cgi?id=41685

This required fast/dom/Window/window-properties-on-device-orientation.html to be renamed
to fast/dom/Window/window-properties-device-orientation.html and updated.

  • fast/dom/DeviceOrientation/script-tests/window-property.js:
  • fast/dom/DeviceOrientation/window-property-expected.txt:
  • fast/dom/Window/window-properties-device-orientation-expected.txt: Copied from LayoutTests/fast/dom/Window/window-properties-on-device-orientation-expected.txt.
  • fast/dom/Window/window-properties-device-orientation.html: Copied from LayoutTests/fast/dom/Window/window-properties-on-device-orientation.html.
  • fast/dom/Window/window-properties-on-device-orientation-expected.txt: Removed.
  • fast/dom/Window/window-properties-on-device-orientation.html: Removed.
  • fast/dom/Window/window-properties.html:
  • platform/gtk/Skipped:
  • platform/mac/Skipped:
  • platform/qt/Skipped:
  • platform/win/Skipped:
4:39 AM Changeset in webkit [62652] by commit-queue@webkit.org
  • 2 edits in trunk/WebKit/qt

2010-07-07 Marc Mutz <​marc.mutz@kdab.com>

Reviewed by Adam Barth.

[Qt] QWebSettings: Don't leak native pixmaps

When Q_GLOBAL_STATIC gets around to deleting the QHash with the
pixmaps in, the QApplication destructor has already been
completed. This makes it impossible for ~QPixmap to return the
native pixmap to the OS, thus leaking them.

This patch adds a post routine (executed as part of ~QApplication)
that clears the hash so Q_GLOBAL_STATIC's destruction doesn't have
to execute ~QPixmap's anymore.

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

  • Api/qwebsettings.cpp: (earlyClearGraphics): New. Clears the pixmap hash. (graphics): add earlyClearGraphics as a qAddPostRoutine()
4:27 AM Changeset in webkit [62651] by commit-queue@webkit.org
  • 7 edits
    2 deletes in trunk

2010-07-07 Sheriff Bot <​webkit.review.bot@gmail.com>

Unreviewed, rolling out r62645.
​http://trac.webkit.org/changeset/62645
​https://bugs.webkit.org/show_bug.cgi?id=41757

"Broken inspector tests on several platforms" (Requested by
eseidel on #webkit).

  • inspector/debugger-breakpoints-not-activated-on-reload-expected.txt: Removed.
  • inspector/debugger-breakpoints-not-activated-on-reload.html: Removed.

2010-07-07 Sheriff Bot <​webkit.review.bot@gmail.com>

Unreviewed, rolling out r62645.
​http://trac.webkit.org/changeset/62645
​https://bugs.webkit.org/show_bug.cgi?id=41757

"Broken inspector tests on several platforms" (Requested by
eseidel on #webkit).

  • inspector/front-end/ScriptView.js: (WebInspector.ScriptView.prototype._addBreakpoint):
  • inspector/front-end/ScriptsPanel.js: (WebInspector.ScriptsPanel): (WebInspector.ScriptsPanel.prototype._resourceLoadingFinished): (WebInspector.ScriptsPanel.prototype._breakpointAdded): (WebInspector.ScriptsPanel.prototype._scriptOrResourceForURLAndLine): (WebInspector.ScriptsPanel.prototype._showScriptOrResource): (WebInspector.ScriptsPanel.prototype._toggleBreakpointsClicked):
  • inspector/front-end/SourceView.js: (WebInspector.SourceView.prototype._addBreakpoint):

2010-07-07 Sheriff Bot <​webkit.review.bot@gmail.com>

Unreviewed, rolling out r62645.
​http://trac.webkit.org/changeset/62645
​https://bugs.webkit.org/show_bug.cgi?id=41757

"Broken inspector tests on several platforms" (Requested by
eseidel on #webkit).

  • src/js/DebuggerScript.js: ():
3:27 AM Changeset in webkit [62650] by steveblock@google.com
  • 3 edits in trunk/WebCore

2010-07-07 Steve Block <​steveblock@google.com>

Reviewed by Adam Barth.

JavaInstanceV8 needs to push a local reference frame to avoid table overflow.
​https://bugs.webkit.org/show_bug.cgi?id=41516

This change is required to help prevent the Java virtual machine from
running out of local references. The Java virtual machine supports only
a limited number of local references. Normally, local references are
cleared when the native method returns.

This change adds calls to PushLocalFrame() and PopLocalFrame() around
each invocation of a method on JavaInstance. These calls instruct the
virtual machine to clear local references when the frame is popped.
This means that in the case where many calls to JavaInstance methods
are made within the same native call, local references are cleared as
soon as they are no longer needed, thus conserving references.

This change mirrors exactly JavaInstanceJSC.

No new tests.

  • bridge/jni/v8/JavaInstanceV8.cpp: (JavaInstance::virtualBegin): (JavaInstance::virtualEnd):
  • bridge/jni/v8/JavaInstanceV8.h:
3:20 AM Changeset in webkit [62649] by yurys@chromium.org
  • 2 edits in trunk/LayoutTests

2010-07-07 Yury Semikhatsky <​yurys@chromium.org>

Unreviewed. Add new debugger test to the skip list on Qt.

  • platform/qt/Skipped:
3:19 AM Changeset in webkit [62648] by antti.j.koivisto@nokia.com
  • 2 edits in trunk/WebKit2

Not reviewed.

Fix build, remove some left over debugging code.

  • Platform/CoreIPC/qt/ConnectionQt.cpp:

(CoreIPC::Connection::readyReadHandler):
(CoreIPC::Connection::sendOutgoingMessage):

3:10 AM Changeset in webkit [62647] by yurys@chromium.org
  • 12 edits in trunk/WebCore

2010-07-06 Yury Semikhatsky <​yurys@chromium.org>

Reviewed by Pavel Feldman.

Inspector should remember the size of sidebars set by the user
​https://bugs.webkit.org/show_bug.cgi?id=19207

  • inspector/front-end/AbstractTimelinePanel.js: (WebInspector.AbstractTimelinePanel):
  • inspector/front-end/AuditsPanel.js: (WebInspector.AuditsPanel):
  • inspector/front-end/ConsolePanel.js: (WebInspector.ConsolePanel):
  • inspector/front-end/ElementsPanel.js: (WebInspector.ElementsPanel.prototype.rightSidebarResizerDragEnd):
  • inspector/front-end/Panel.js: (WebInspector.Panel): (WebInspector.Panel.prototype.get toolbarItemClass): (WebInspector.Panel.prototype.show): (WebInspector.Panel.prototype.createSidebar): (WebInspector.Panel.prototype.get _sidebarWidthSettingName): (WebInspector.Panel.prototype._endSidebarDragging): (WebInspector.Panel.prototype.updateSidebarWidth): (WebInspector.Panel.prototype.restoreSidebarWidth): (WebInspector.Panel.prototype.saveSidebarWidth):
  • inspector/front-end/ProfilesPanel.js: (WebInspector.ProfilesPanel):
  • inspector/front-end/ResourcesPanel.js: (WebInspector.ResourcesPanel): (WebInspector.ResourcesPanel.prototype.resourceTrackingWasEnabled):
  • inspector/front-end/ScriptsPanel.js: (WebInspector.ScriptsPanel): (WebInspector.ScriptsPanel.prototype._endSidebarResizeDrag): (WebInspector.ScriptsPanel.prototype._sidebarResizeDrag): (WebInspector.ScriptsPanel.prototype.setSidebarWidth):
  • inspector/front-end/Settings.js: (WebInspector.populateApplicationSettings):
  • inspector/front-end/StoragePanel.js: (WebInspector.StoragePanel):
  • inspector/front-end/TimelinePanel.js: (WebInspector.TimelinePanel):
2:52 AM Changeset in webkit [62646] by steveblock@google.com
  • 10 edits
    2 moves in trunk/WebCore

2010-07-05 Steve Block <​steveblock@google.com>

Reviewed by Adam Barth.

DeviceOrientation should be renamed to DeviceOrientationController
​https://bugs.webkit.org/show_bug.cgi?id=41608

No new tests, renaming only.

  • Android.mk:
  • CMakeLists.txt:
  • GNUmakefile.am:
  • WebCore.gypi:
  • WebCore.pro:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • dom/DeviceOrientation.cpp: Removed.
  • dom/DeviceOrientation.h: Removed.
  • dom/DeviceOrientationController.cpp: Copied from WebCore/dom/DeviceOrientation.cpp. (WebCore::DeviceOrientationController::DeviceOrientationController): (WebCore::DeviceOrientationController::onDeviceOrientationChange):
  • dom/DeviceOrientationController.h: Copied from WebCore/dom/DeviceOrientation.h.
  • page/Page.cpp: (WebCore::Page::Page):
  • page/Page.h: (WebCore::Page::deviceOrientationController):
2:30 AM Changeset in webkit [62645] by yurys@chromium.org
  • 7 edits
    2 adds in trunk

2010-07-07 Pavel Podivilov <​podivilov@chromium.org>

Reviewed by Yury Semikhatsky.

Web Inspector: do not activate all breakpoints on page reload
​https://bugs.webkit.org/show_bug.cgi?id=41461

  • inspector/debugger-breakpoints-not-activated-on-reload-expected.txt: Added.
  • inspector/debugger-breakpoints-not-activated-on-reload.html: Added.

2010-07-07 Pavel Podivilov <​podivilov@chromium.org>

Reviewed by Yury Semikhatsky.

Web Inspector: do not activate all breakpoints on page reload
​https://bugs.webkit.org/show_bug.cgi?id=41461

Test: inspector/debugger-breakpoints-not-activated-on-reload.html

  • inspector/front-end/ScriptView.js: (WebInspector.ScriptView.prototype._addBreakpoint):
  • inspector/front-end/ScriptsPanel.js: (WebInspector.ScriptsPanel): (WebInspector.ScriptsPanel.prototype._breakpointAdded): (WebInspector.ScriptsPanel.prototype.toggleBreakpointsClicked):
  • inspector/front-end/SourceView.js: (WebInspector.SourceView.prototype._addBreakpoint):

2010-07-07 Pavel Podivilov <​podivilov@chromium.org>

Reviewed by Yury Semikhatsky.

Web Inspector: if breakpoints were deactivated, add new breakpoints as disabled.
​https://bugs.webkit.org/show_bug.cgi?id=41461

  • src/js/DebuggerScript.js: ():
1:58 AM Changeset in webkit [62644] by Nikolas Zimmermann
  • 3 edits
    4 deletes in trunk

2010-07-07 Nikolas Zimmermann <​nzimmermann@rim.com>

Not reviewed. Rollout r62633 - test crashes after some last-minute modifications.

  • svg/SVGFontFaceElement.cpp: (WebCore::SVGFontFaceElement::~SVGFontFaceElement):
1:52 AM Changeset in webkit [62643] by eric@webkit.org
  • 4 edits in trunk

2010-07-07 Eric Seidel <​eric@webkit.org>

Reviewed by Adam Barth.

Fix </html> handling in before head mode
​https://bugs.webkit.org/show_bug.cgi?id=41752

  • html5lib/runner-expected-html5.txt:

2010-07-07 Eric Seidel <​eric@webkit.org>

Reviewed by Adam Barth.

Fix </html> handling in before head mode
​https://bugs.webkit.org/show_bug.cgi?id=41752

Covered by html5lib/runner.html.

  • html/HTMLTreeBuilder.cpp: (WebCore::HTMLTreeBuilder::processEndTag):
1:50 AM Changeset in webkit [62642] by eric@webkit.org
  • 8 edits in trunk

2010-07-07 Eric Seidel <​eric@webkit.org>

Reviewed by Adam Barth.

</div> pops too many <div> elements in body
​https://bugs.webkit.org/show_bug.cgi?id=41751

  • html5lib/runner-expected-html5.txt:

2010-07-07 Eric Seidel <​eric@webkit.org>

Reviewed by Adam Barth.

</div> pops too many <div> elements in body
​https://bugs.webkit.org/show_bug.cgi?id=41751

Another missing return!

While tracking this down I added some debugging code to both the
open elements stack and the active formatting elements list.

I also unwrapped a very long
chain to make it more readable.
  • html/HTMLElementStack.cpp: (WebCore::HTMLElementStack::show):
  • html/HTMLElementStack.h:
  • html/HTMLFormattingElementList.cpp: (WebCore::HTMLFormattingElementList::show):
  • html/HTMLFormattingElementList.h:
  • html/HTMLTreeBuilder.cpp: (WebCore::HTMLTreeBuilder::processEndTagForInBody):
1:50 AM Changeset in webkit [62641] by abecsi@webkit.org
  • 2 edits in trunk/WebKitTools

Unreviewed trivial fix.

Remove DUMPRENDERTREE_TEMP environment variable from NRWT
which was checked in accidentally in r62635.

  • Scripts/webkitpy/layout_tests/port/webkit.py:
1:48 AM Changeset in webkit [62640] by eric@webkit.org
  • 4 edits in trunk

2010-07-06 Eric Seidel <​eric@webkit.org>

Reviewed by Adam Barth.

HTMLTreeBuilder needs to handle <table><input>
​https://bugs.webkit.org/show_bug.cgi?id=41744

  • html5lib/runner-expected-html5.txt:

2010-07-06 Eric Seidel <​eric@webkit.org>

Reviewed by Adam Barth.

HTMLTreeBuilder needs to handle <table><input>
​https://bugs.webkit.org/show_bug.cgi?id=41744

  • html/HTMLTreeBuilder.cpp: (WebCore::HTMLTreeBuilder::processStartTagForInTable):
1:46 AM Changeset in webkit [62639] by antti.j.koivisto@nokia.com
  • 1 edit
    1 add in trunk/WebKit2

[Qt] Initial WebKit2 implementation
​https://bugs.webkit.org/show_bug.cgi?id=40233

Reviewed by Kenneth Rohde Christiansen.

Implement WebPage for Qt.

  • WebProcess/WebPage/qt/WebPageQt.cpp: Added.

(WebKit::WebPage::platformInitialize):
(WebKit::):
(WebKit::WebPage::interpretKeyEvent):

1:45 AM Changeset in webkit [62638] by eric@webkit.org
  • 4 edits in trunk

2010-07-06 Eric Seidel <​eric@webkit.org>

Reviewed by Adam Barth.

Teach HTMLTreeBuilder how to handle <button><button>
​https://bugs.webkit.org/show_bug.cgi?id=41743

  • html5lib/runner-expected-html5.txt:

2010-07-06 Eric Seidel <​eric@webkit.org>

Reviewed by Adam Barth.

Teach HTMLTreeBuilder how to handle <button><button>
​https://bugs.webkit.org/show_bug.cgi?id=41743

  • html/HTMLTreeBuilder.cpp: (WebCore::HTMLTreeBuilder::processStartTagForInBody):
1:42 AM Changeset in webkit [62637] by eric@webkit.org
  • 4 edits in trunk

2010-07-06 Eric Seidel <​eric@webkit.org>

Reviewed by Adam Barth.

Teach TreeBuilder how to handle <form><form>
​https://bugs.webkit.org/show_bug.cgi?id=41741

  • html5lib/runner-expected-html5.txt:

2010-07-06 Eric Seidel <​eric@webkit.org>

Reviewed by Adam Barth.

Teach TreeBuilder how to handle <form><form>
​https://bugs.webkit.org/show_bug.cgi?id=41741

  • html/HTMLTreeBuilder.cpp: (WebCore::HTMLTreeBuilder::processStartTagForInBody):
1:41 AM Changeset in webkit [62636] by eric@webkit.org
  • 4 edits in trunk

2010-07-06 Eric Seidel <​eric@webkit.org>

Reviewed by Adam Barth.

<table> should auto-close <p> when not in quirks mode
​https://bugs.webkit.org/show_bug.cgi?id=41740

  • html5lib/runner-expected-html5.txt:

2010-07-06 Eric Seidel <​eric@webkit.org>

Reviewed by Adam Barth.

<table> should auto-close <p> when not in quirks mode
​https://bugs.webkit.org/show_bug.cgi?id=41740

This is already covered by multiple tests in html5lib/runner.html

  • html/HTMLTreeBuilder.cpp: (WebCore::HTMLTreeBuilder::processStartTagForInBody):
1:38 AM Changeset in webkit [62635] by commit-queue@webkit.org
  • 3 edits in trunk/WebKitTools

2010-07-07 Gabor Rapcsanyi <​rgabor@inf.u-szeged.hu>

Reviewed by Eric Seidel.

Pass port specific environment to server process.
​https://bugs.webkit.org/show_bug.cgi?id=41593

  • Scripts/webkitpy/layout_tests/port/qt.py:
  • Scripts/webkitpy/layout_tests/port/webkit.py:
1:37 AM Changeset in webkit [62634] by eric@webkit.org
  • 4 edits in trunk

2010-07-06 Eric Seidel <​eric@webkit.org>

Reviewed by Adam Barth.

Make "in body" character handling reconstruct active formatting elements
​https://bugs.webkit.org/show_bug.cgi?id=41739

  • html5lib/runner-expected-html5.txt:

2010-07-06 Eric Seidel <​eric@webkit.org>

Reviewed by Adam Barth.

Make "in body" character handling reconstruct active formatting elements
​https://bugs.webkit.org/show_bug.cgi?id=41739

This fixes a bunch of tests, and due to some other bug in our
formatting code, regresses two. I'll fix that bug in a separate
patch.

  • html/HTMLTreeBuilder.cpp: (WebCore::HTMLTreeBuilder::processCharacter):
1:35 AM Changeset in webkit [62633] by Nikolas Zimmermann
  • 3 edits
    4 adds in trunk

2010-07-06 Nikolas Zimmermann <​nzimmermann@rim.com>

Reviewed by Darin Adler.

<use> on <font-face> causes crashes, if SVGUseElement gets detached
​https://bugs.webkit.org/show_bug.cgi?id=41621

Do not call removeFromMappedElementSheet() from the destructor, as the call to document()->updateStyleSelector() that can potentially
cause the element to be reattached while destructing. It's not needed at all, because removedFromDocument() is called before destruction,
which already calls removeFromMappedElementSheet() - at this point it's still safe to update the style selector.

The crash is reproducable when using <use> on <font-face>.

Test: svg/custom/use-font-face-crash.svg

  • svg/SVGFontFaceElement.cpp: (WebCore::SVGFontFaceElement::~SVGFontFaceElement):
1:06 AM Changeset in webkit [62632] by commit-queue@webkit.org
  • 5 edits in trunk

2010-07-07 Hayato Ito <​hayato@chromium.org>

Reviewed by Darin Adler.

Rolling out 'page-break-inside:avoid' part of the r54929.
Rebased the related layout tests, which are now expected to fail, as well.

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

  • fast/multicol/break-properties-expected.txt:
  • printing/page-break-inside-avoid-expected.txt:

2010-07-07 Hayato Ito <​hayato@chromium.org>

Reviewed by Darin Adler.

Rolling out 'page-break-inside:avoid' part of the r54929.
Rebased the related layout tests, which are now expected to fail, as well.

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

  • rendering/RenderBlock.cpp: (WebCore::RenderBlock::paintChildren):
12:46 AM Changeset in webkit [62631] by mrowe@apple.com
  • 2 edits in trunk/WebCore

Fix failures in a handful of Java-related tests.

  • WebCore.exp.in: Remove an extraneous ENABLE_ from an #if so that the condition

evalutes to true when we expect it to.

12:41 AM Changeset in webkit [62630] by pfeldman@chromium.org
  • 20 edits
    3 adds in trunk

2010-07-06 Pavel Feldman <​pfeldman@chromium.org>

Reviewed by Joseph Pecoraro.

Web Inspector: provide line numbers for inline styles.

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

Test: inspector/styles-source-lines-inline.html

  • css/CSSParser.cpp: (WebCore::CSSParser::parseSheet):
  • css/CSSParser.h:
  • css/CSSStyleSheet.cpp: (WebCore::CSSStyleSheet::parseString): (WebCore::CSSStyleSheet::parseStringAtLine):
  • css/CSSStyleSheet.h:
  • dom/StyleElement.cpp: (WebCore::StyleElement::process): (WebCore::StyleElement::createSheet):
  • dom/StyleElement.h:
  • html/HTMLStyleElement.cpp: (WebCore::HTMLStyleElement::HTMLStyleElement): (WebCore::HTMLStyleElement::finishParsingChildren):
  • html/HTMLStyleElement.h:
  • inspector/InspectorCSSStore.cpp: (WebCore::InspectorCSSStore::getRuleRangesForStyleSheet):
  • inspector/InspectorDOMAgent.cpp: (WebCore::InspectorDOMAgent::buildObjectForRule):
  • inspector/front-end/DOMAgent.js: (WebInspector.CSSStyleDeclaration.parseRule):
  • inspector/front-end/StylesSidebarPane.js: (WebInspector.StylePropertiesSection.else.linkifyUncopyable): (WebInspector.StylePropertiesSection):

2010-07-06 Pavel Feldman <​pfeldman@chromium.org>

Reviewed by Joseph Pecoraro.

Web Inspector: provide line numbers for inline styles.

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

  • http/tests/inspector/inspector-test.js: (runAfterIframeIsLoaded.step): (runAfterIframeIsLoaded):
  • inspector/elements-tests.js:
  • inspector/resources/styles-source-lines-inline-iframe.html: Added.
  • inspector/styles-iframe.html:
  • inspector/styles-source-lines-expected.txt: Removed.
  • inspector/styles-source-lines-inline-expected.txt: Added.
  • inspector/styles-source-lines-inline.html: Added.
  • inspector/styles-source-lines.html:
12:12 AM Changeset in webkit [62629] by abarth@webkit.org
  • 3 edits in trunk/LayoutTests

2010-07-07 Adam Barth <​abarth@webkit.org>

Unreviewed.

Clean up Alexey's mess from ​http://trac.webkit.org/changeset/62576
​https://bugs.webkit.org/show_bug.cgi?id=41156

I think these results are correct, but I don't really know for sure.
What I do know is that he left this failure in the tree for 12 hours.

  • fast/xmlhttprequest/xmlhttprequest-no-file-access-expected.txt:
  • fast/xmlhttprequest/xmlhttprequest-nonexistent-file-expected.txt:
12:04 AM Changeset in webkit [62628] by zherczeg@webkit.org
  • 3 edits in trunk/JavaScriptCore

Refactored string parsing inside the lexer
​https://bugs.webkit.org/show_bug.cgi?id=41606

Reviewed by Oliver Hunt.

Does not use goto. Although the last sunspider
parse-only tests yields 1.044x speedup, I think the
patch can have a slight improvement at most.

  • parser/Lexer.cpp:

(JSC::singleEscape):
(JSC::Lexer::parseString):
(JSC::Lexer::lex):

  • parser/Lexer.h:

Jul 6, 2010:

11:58 PM Changeset in webkit [62627] by mitz@apple.com
  • 2 edits in trunk/WebCore

<rdar://problem/8163651> Wrong value type being supplied for kCTTypesetterOptionForcedEmbeddingLevel key

Reviewed by Jon Honeycutt.

No change in behavior, so no new test.

  • platform/graphics/mac/ComplexTextControllerCoreText.cpp:

(WebCore::ComplexTextController::collectComplexTextRunsForCharactersCoreText): Use CFNumber instances
as the values of the kCTTypesetterOptionForcedEmbeddingLevel key, instead of CFBooleans.

10:53 PM Changeset in webkit [62626] by mrowe@apple.com
  • 3 edits in trunk/WebCore

Fix production builds on Mac OS X.

  • DerivedSources.make: Don't try and generate the export file during the installhdrs build phase.

It's not useful until much later in the build process, and it's not possible to generate it until
after the tool has been compiled anyway.

  • WebCore.xcodeproj/project.pbxproj: Tweak how the ICU and forwarding headers are copied so that it

works in production builds once more.

10:42 PM Changeset in webkit [62625] by sfalken@apple.com
  • 10 edits in trunk

WebCore: Expose URL matching from WebUserContentURLPattern
​https://bugs.webkit.org/show_bug.cgi?id=41726
<rdar://problem/7910144>

Reviewed by Simon Fraser.

  • WebCore.exp.in: Export UserContentURLPattern::matches for use in WebKit.

WebKit/mac: Expose URL matching from WebUserContentURLPattern
​https://bugs.webkit.org/show_bug.cgi?id=41726
<rdar://problem/7910144>

Reviewed by Simon Fraser.

We previously had a way to construct WebUserContentURLPattern
instances via WebKit, but no way for callers to perform matching.
This patch adds the matchesURL functionality to allow for this.

  • Misc/WebUserContentURLPattern.h: Added matchesURL.
  • Misc/WebUserContentURLPattern.mm:

(-[WebUserContentURLPattern matchesURL:]): Added. Calls through to WebCore::UserContentURLPattern::matches.

WebKit/win: Expose URL matching from WebUserContentURLPattern
​https://bugs.webkit.org/show_bug.cgi?id=41726
<rdar://problem/7910144>

Reviewed by Simon Fraser.

We previously had a way to construct WebUserContentURLPattern
instances via WebKit, but no way for callers to perform matching.
This patch adds the matchesURL functionality to allow for this.

  • Interfaces/IWebUserContentURLPattern.idl: Added matchesURL.
  • Interfaces/IWebView.idl: Touch to force a build.
  • WebUserContentURLPattern.cpp:

(WebUserContentURLPattern::matchesURL): Added. Calls through to WebCore::UserContentURLPattern::matches.

  • WebUserContentURLPattern.h: Added matchesURL.
10:08 PM Changeset in webkit [62624] by abarth@webkit.org
  • 5 edits in trunk

2010-07-06 Eric Seidel <​eric@webkit.org>

Reviewed by Adam Barth.

Fix <nobr><nobr> case in HTMLTreeBuilder
​https://bugs.webkit.org/show_bug.cgi?id=41735

We were both not handling <nobr> correctly, as well as
never hitting the <nobr> case because our formatting
elements check was overzealous.

  • html/HTMLTreeBuilder.cpp: (WebCore::HTMLTreeBuilder::processStartTagForInBody):

2010-07-06 Adam Barth <​abarth@webkit.org>

Reviewed by Eric Seidel.

Implement start table tag in table mode
​https://bugs.webkit.org/show_bug.cgi?id=41736

  • html5lib/runner-expected-html5.txt:
10:05 PM Changeset in webkit [62623] by eric@webkit.org
  • 7 edits in trunk

2010-07-06 Eric Seidel <​eric@webkit.org>

Reviewed by Adam Barth.

Fix <nobr><nobr> case in HTMLTreeBuilder
​https://bugs.webkit.org/show_bug.cgi?id=41735

  • html5lib/runner-expected-html5.txt:

2010-07-06 Eric Seidel <​eric@webkit.org>

Reviewed by Adam Barth.

Fix <nobr><nobr> case in HTMLTreeBuilder
​https://bugs.webkit.org/show_bug.cgi?id=41735

We were both not handling <nobr> correctly, as well as
never hitting the <nobr> case because our formatting
elements check was overzealous.

  • html/HTMLTreeBuilder.cpp: (WebCore::HTMLTreeBuilder::processStartTagForInBody):
9:50 PM Changeset in webkit [62622] by tkent@chromium.org
  • 6 edits in trunk/LayoutTests

2010-07-06 Kent Tamura <​tkent@chromium.org>

Unreviewed, test expectation update.

Update expectation files for fast/forms/indeterminate.html. Chromium
started to support them on Windows and Linux recently.

  • platform/chromium-linux/fast/forms/indeterminate-expected.checksum:
  • platform/chromium-linux/fast/forms/indeterminate-expected.png:
  • platform/chromium-win/fast/forms/indeterminate-expected.checksum:
  • platform/chromium-win/fast/forms/indeterminate-expected.png:
  • platform/chromium/test_expectations.txt:
9:43 PM Changeset in webkit [62621] by abarth@webkit.org
  • 7 edits in trunk

2010-07-06 Adam Barth <​abarth@webkit.org>

Reviewed by Eric Seidel.

Implement "act as if the banana has been peeled" (for lack of a better term)
​https://bugs.webkit.org/show_bug.cgi?id=41734

The spec has a tricky passage that says "whenever a node would be
inserted into the current node, it must instead be foster parented."
This patch attempts to implement that requirement.

  • html/HTMLConstructionSite.cpp: (WebCore::HTMLConstructionSite::attach): (WebCore::HTMLConstructionSite::HTMLConstructionSite): (WebCore::HTMLConstructionSite::insertHTMLHtmlElement): (WebCore::HTMLConstructionSite::insertHTMLHeadElement): (WebCore::HTMLConstructionSite::insertHTMLBodyElement): (WebCore::HTMLConstructionSite::insertScriptElement): (WebCore::HTMLConstructionSite::fosterParent):
  • html/HTMLConstructionSite.h: (WebCore::HTMLConstructionSite::RedirectToFosterParentGuard::RedirectToFosterParentGuard): (WebCore::HTMLConstructionSite::RedirectToFosterParentGuard::~RedirectToFosterParentGuard):
  • html/HTMLTreeBuilder.cpp: (WebCore::HTMLTreeBuilder::processStartTagForInTable): (WebCore::HTMLTreeBuilder::processEndTagForInTable):
  • html/HTMLTreeBuilder.h:

2010-07-06 Adam Barth <​abarth@webkit.org>

Reviewed by Eric Seidel.

Implement "act as if the banana has been peeled" (for lack of a better term)
​https://bugs.webkit.org/show_bug.cgi?id=41734

Massive test progression.

  • html5lib/runner-expected-html5.txt:
9:41 PM Changeset in webkit [62620] by tkent@chromium.org
  • 5 edits in trunk/WebKitTools

2010-07-06 Kent Tamura <​tkent@chromium.org>

Reviewed by Shinichiro Hamaji.

Fix a wrong dup detection of rebaseline-chromium-webkit-tests
​https://bugs.webkit.org/show_bug.cgi?id=41644

rebaseline-chromium-webkit-tests used ImageDiff with 0.1%
tolerance. We don't need tolerance for rebaseline.

  • Scripts/webkitpy/layout_tests/port/base.py: Add optional tolerance parameter to diff_image().
  • Scripts/webkitpy/layout_tests/port/test.py: Add optional tolerance parameter to diff_image().
  • Scripts/webkitpy/layout_tests/port/webkit.py: Add optional tolerance parameter to diff_image(), and pass it to ImageDiff command.
  • Scripts/webkitpy/layout_tests/test_types/image_diff.py: diff_files() always calls diff_image() with tolerance=0. diff_files() is used only by rebaseline-chromium-webkit-tests.
9:32 PM Changeset in webkit [62619] by tkent@chromium.org
  • 2 edits in trunk/LayoutTests

2010-07-06 Kent Tamura <​tkent@chromium.org>

Unreviewed, test expectation update.

  • platform/chromium/test_expectations.txt: Remove menulist-no-overflow.html, width100percent-checkbox.html, width100percent-radio.html, and button-sizes.html. They have been passed since Chromium r51096, test_shell flip issue.
9:23 PM Changeset in webkit [62618] by eric@webkit.org
  • 7 edits in trunk

2010-07-06 Eric Seidel <​eric@webkit.org>

Reviewed by Adam Barth.

Fix </optgroup> in "in select" mode
​https://bugs.webkit.org/show_bug.cgi?id=41733

  • html5lib/runner-expected-html5.txt:

2010-07-06 Eric Seidel <​eric@webkit.org>

Reviewed by Adam Barth.

Fix </optgroup> in "in select" mode
​https://bugs.webkit.org/show_bug.cgi?id=41733

I had to add a oneBelowTop() accessor.
I added some ASSERTs after looking at the fragment case
documentation. I'm now convinced that top() and oneBeforeTop()
should never be NULL, so we just ASSERT they aren't.

This is a very obscure corner of the HTML spec, but at least
we have a test for it. This makes one more html5lib test pass.

  • html/HTMLConstructionSite.h: (WebCore::HTMLConstructionSite::oneBelowTop):
  • html/HTMLElementStack.cpp: (WebCore::HTMLElementStack::topRecord): (WebCore::HTMLElementStack::top): (WebCore::HTMLElementStack::oneBelowTop):
  • html/HTMLElementStack.h:
  • html/HTMLTreeBuilder.cpp: (WebCore::HTMLTreeBuilder::processEndTag):
8:48 PM Changeset in webkit [62617] by eric@webkit.org
  • 3 edits in trunk

2010-07-06 Eric Seidel <​eric@webkit.org>

Reviewed by Adam Barth.

Fix "in row" "Anything else" handling of end tags
​https://bugs.webkit.org/show_bug.cgi?id=41731

I also fixed </br> handling to follow the spec more closely
since it implementation was old and didn't use all our
new processFake* hotness.

This is already covered by 3 tests, however we don't
pass them yet, due to lack of table foster parenting
code (which Adam is working on as we speak).

  • html/HTMLTreeBuilder.cpp: (WebCore::HTMLTreeBuilder::processEndTagForInBody): (WebCore::HTMLTreeBuilder::processEndTag):
8:40 PM Changeset in webkit [62616] by mrowe@apple.com
  • 2 edits in trunk/WebCore

Fix clean builds on Mac OS X.

Since "Copy Forwarding and ICU Headers" was split in to a separate target
from the WebCore framework it no longer had automatica access to the
PRIVATE_HEADERS_FOLDER_PATH variable. This caused it to copy the header
files in to the wrong location. Fix that by giving PRIVATE_HEADERS_FOLDER_PATH
its correct value and ensuring that the path exists.

  • WebCore.xcodeproj/project.pbxproj:
7:16 PM Changeset in webkit [62615] by weinig@apple.com
  • 2 edits in trunk/WebCore

Fix Gtk build.

  • GNUmakefile.am:
7:12 PM Changeset in webkit [62614] by mrowe@apple.com
  • 2 edits in trunk/WebCore

Tiger build fix.

Don't use WebCore's LDFLAGS when building WebCoreExportFileGenerator.

  • WebCore.xcodeproj/project.pbxproj:
6:49 PM Changeset in webkit [62613] by eric@webkit.org
  • 6 edits in trunk

2010-07-06 Eric Seidel <​eric@webkit.org>

Reviewed by Adam Barth.

"In cell" does not correctly handle <td><tr> or <td><td>
​https://bugs.webkit.org/show_bug.cgi?id=41729

  • html5lib/resources/tables01.dat:
    • Added a case for <td><tr> since it was missing from the rest of the suite.
  • html5lib/runner-expected-html5.txt:
  • html5lib/runner-expected.txt:

2010-07-06 Eric Seidel <​eric@webkit.org>

Reviewed by Adam Barth.

"In cell" does not correctly handle <td><tr> or <td><td>
​https://bugs.webkit.org/show_bug.cgi?id=41729

This change is mostly cleanup to try and prevent forgetting
tag name checks in the future by using inlines to reduce
copy/paste code.

3 little bugs in InCellMode:

  • Missing trTag from the long or statement (reason for the cleanup)
Used
instead of &&
  • Forgot to reprocess the tag after closeTheCell()
  • html/HTMLTreeBuilder.cpp: (WebCore::HTMLTreeBuilder::processStartTagForInBody): (WebCore::HTMLTreeBuilder::processStartTagForInTable): (WebCore::HTMLTreeBuilder::processStartTag): (WebCore::HTMLTreeBuilder::processEndTagForInTable): (WebCore::HTMLTreeBuilder::processEndTag):
6:35 PM Changeset in webkit [62612] by oliver@apple.com
  • 12 edits in trunk/JavaScriptCore

2010-07-06 Oliver Hunt <​oliver@apple.com>

Reviewed by Maciej Stachowiak.

Make it possible to have both the JIT and Interpreter available in a single build
​https://bugs.webkit.org/show_bug.cgi?id=41722

Separate the concept of !ENABLE(JIT) and ENABLE(INTERPRETER) and make it possible
to have both JIT and INTERPRETER enabled at the same time. This doesn't add
support for mix mode execution, but it does allow a single build to contain all
the code needed to use either the interpreter or the jit.

If both ENABLE(INTERPRETER) and ENABLE(JIT) are true then setting the environment
variable JSC_FORCE_INTERPRETER will force JSC to use the interpreter.

This patch basically consists of replacing !ENABLE(JIT) with ENABLE(INTERPRETER),
or converting #if ENABLE(JIT) ... #else ... into #if ENABLE(JIT) ... #endif
#if ENABLE(INTERPRETER), etc. There are also a few functions that need to be
renamed to resolve return type ambiguity.

  • bytecode/CodeBlock.cpp: (JSC::CodeBlock::~CodeBlock): (JSC::CodeBlock::shrinkToFit):
  • bytecode/CodeBlock.h:
  • interpreter/CallFrame.h: (JSC::ExecState::returnVPC):
  • interpreter/Interpreter.cpp: (JSC::Interpreter::unwindCallFrame): (JSC::Interpreter::throwException): (JSC::Interpreter::execute): (JSC::Interpreter::executeCall): (JSC::Interpreter::executeConstruct): (JSC::Interpreter::prepareForRepeatCall): (JSC::Interpreter::privateExecute): (JSC::Interpreter::retrieveLastCaller):
  • interpreter/Interpreter.h:
  • runtime/ArrayPrototype.cpp: (JSC::isNumericCompareFunction):
  • runtime/Executable.cpp: (JSC::EvalExecutable::generateJITCode): (JSC::ProgramExecutable::generateJITCode): (JSC::FunctionExecutable::generateJITCodeForCall): (JSC::FunctionExecutable::generateJITCodeForConstruct): (JSC::FunctionExecutable::reparseExceptionInfo): (JSC::EvalExecutable::reparseExceptionInfo):
  • runtime/JSFunction.cpp:
  • runtime/JSGlobalData.cpp: (JSC::JSGlobalData::JSGlobalData):
  • runtime/JSGlobalData.h: (JSC::JSGlobalData::canUseJIT):
  • wtf/Platform.h:
6:34 PM Changeset in webkit [62611] by abarth@webkit.org
  • 4 edits in trunk/WebCore

2010-07-06 Adam Barth <​abarth@webkit.org>

Reviewed by Eric Seidel.

Move fosterParent to HTMLConstructionSite
​https://bugs.webkit.org/show_bug.cgi?id=41728

This paves the way for handling the default case of InTableMode
properly.

  • html/HTMLConstructionSite.cpp: (WebCore::HTMLConstructionSite::fosterParent):
  • html/HTMLConstructionSite.h:
  • html/HTMLTreeBuilder.cpp: (WebCore::HTMLTreeBuilder::callTheAdoptionAgency):
6:16 PM Changeset in webkit [62610] by abarth@webkit.org
  • 10 edits
    2 copies in trunk/WebCore

2010-07-06 Adam Barth <​abarth@webkit.org>

Rubber-stamped by Eric Seidel.

Move HTMLConstructionSite to its own file. No behavior change.

  • Android.mk:
  • CMakeLists.txt:
  • GNUmakefile.am:
  • WebCore.gypi:
  • WebCore.pro:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • html/HTMLConstructionSite.cpp: Copied from WebCore/html/HTMLTreeBuilder.cpp. (WebCore::HTMLNames::hasImpliedEndTag): (WebCore::HTMLConstructionSite::attach): (WebCore::HTMLConstructionSite::HTMLConstructionSite): (WebCore::HTMLConstructionSite::~HTMLConstructionSite): (WebCore::HTMLConstructionSite::insertHTMLHtmlStartTagBeforeHTML): (WebCore::HTMLConstructionSite::mergeAttributesFromTokenIntoElement): (WebCore::HTMLConstructionSite::insertHTMLHtmlStartTagInBody): (WebCore::HTMLConstructionSite::insertHTMLBodyStartTagInBody):
  • html/HTMLConstructionSite.h: Copied from WebCore/html/HTMLTreeBuilder.h.
  • html/HTMLTreeBuilder.cpp:
  • html/HTMLTreeBuilder.h:
6:08 PM Changeset in webkit [62609] by mrowe@apple.com
  • 2 edits in trunk/WebCore

Build fix.

The path to the generated file needs to be relative to the built product directory,
not to the project file.

  • WebCore.xcodeproj/project.pbxproj:
6:01 PM Changeset in webkit [62608] by mrowe@apple.com
  • 3 edits
    1 move
    1 add
    20 deletes in trunk/WebCore

<​http://webkit.org/b/41725> REGRESSION(r62283): No longer possible to build for Tiger from SnowLeopard due to export file shenanigans

Reviewed by Darin Adler.

Change how the exports file is generated. Rather than having the logic for which symbols
to include be encoded in a Makefile that concatenated separate files, the logic is now
encoded as #if's in the base exports file. This ensures that the exact same preprocessor
settings are used when determining which symbols to export as when compiling the code that
defines the symbols.

  • DerivedSources.make: Remove logic that determines which export files to concatenate.

Replace with an invocation of WebCoreExportFileGenerator.

  • WebCore.3DRendering.exp: Removed.
  • WebCore.AcceleratedCompositing.exp: Removed.
  • WebCore.ClientBasedGeolocation.exp: Removed.
  • WebCore.ContextMenus.exp: Removed.
  • WebCore.DashboardSupport.exp: Removed.
  • WebCore.DragSupport.exp: Removed.
  • WebCore.Geolocation.exp: Removed.
  • WebCore.Inspector.exp: Removed.
  • WebCore.JNI.exp: Removed.
  • WebCore.NPAPI.exp: Removed.
  • WebCore.OrientationEvents.exp: Removed.
  • WebCore.PluginHostProcess.exp: Removed.
  • WebCore.ProtectionSpaceAuthCallback.exp: Removed.
  • WebCore.SVG.Animation.exp: Removed.
  • WebCore.SVG.Filters.exp: Removed.
  • WebCore.SVG.ForeignObject.exp: Removed.
  • WebCore.SVG.exp: Removed.
  • WebCore.Tiger.exp: Removed.
  • WebCore.Video.exp: Removed.
  • WebCore.VideoProxy.exp: Removed.
  • WebCore.exp.in: Renamed from WebCore/WebCore.base.exp. Merge the individual .exp

files in, wrapping the appropriate parts with the necessary #if's.

  • WebCore.xcodeproj/project.pbxproj:
  • make-exports-file-generator: Added.
5:46 PM Changeset in webkit [62607] by weinig@apple.com
  • 10 edits
    3 adds in trunk/WebCore

Patch for ​https://bugs.webkit.org/show_bug.cgi?id=41723
Add BackForwardController class

Reviewed by Anders Carlsson.

  • Add BackForwardController which currently only forwards to the existing BackForwardList implementation. This is a first step toward allowing for a proxying BackForwardList implementation.
  • CMakeLists.txt:
  • GNUmakefile.am:
  • WebCore.base.exp:
  • WebCore.gypi:
  • WebCore.pro:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • history/BackForwardController.cpp: Added.

(WebCore::BackForwardController::BackForwardController):
(WebCore::BackForwardController::~BackForwardController):

  • history/BackForwardController.h: Added.

(WebCore::BackForwardController::client):
(WebCore::BackForwardController::list):

  • history/BackForwardControllerClient.h: Added.

(WebCore::BackForwardControllerClient::~BackForwardControllerClient):

  • page/Page.cpp:

(WebCore::Page::Page):
(WebCore::Page::~Page):
(WebCore::Page::backForwardList):
(WebCore::Page::goBack):
(WebCore::Page::goForward):
(WebCore::Page::canGoBackOrForward):
(WebCore::Page::goBackOrForward):
(WebCore::Page::getHistoryLength):

  • page/Page.h:
5:17 PM Changeset in webkit [62606] by vitalyr@chromium.org
  • 4 edits in trunk/LayoutTests

2010-07-06 Vitaly Repeshko <​vitalyr@chromium.org>

Unreviewed.

[chromium] Update more expectations.

  • platform/chromium-mac/html5lib/runner-expected.txt:
  • platform/chromium-win/html5lib/runner-expected.txt:
  • platform/chromium/test_expectations.txt:
4:53 PM Changeset in webkit [62605] by andersca@apple.com
  • 2 edits in trunk/WebKit2

Fix Windows WebKit2 build.

  • UIProcess/Plugins/win/PluginInfoStoreWin.cpp:

(WebKit::PluginInfoStore::mimeTypeFromExtension):

4:20 PM Changeset in webkit [62604] by kov@webkit.org
  • 2 edits in trunk/WebCore

2010-07-06 Gustavo Noronha Silva <​gustavo.noronha@collabora.co.uk>

Unreviewed.

Fixes regression on API test by disabling the scrollbars before
configuring the adjustments.

  • platform/gtk/ScrollViewGtk.cpp: (WebCore::ScrollView::setGtkAdjustments):
4:08 PM Changeset in webkit [62603] by eric@webkit.org
  • 7 edits in trunk

2010-07-06 Eric Seidel <​eric@webkit.org>

Reviewed by Adam Barth.

Add support for <li>, <dd> and <dt> nested tag closing
​https://bugs.webkit.org/show_bug.cgi?id=41720

  • html5lib/resources/webkit01.dat:
    • Add test for <dd><dt> since this wasn't well covered.
  • html5lib/runner-expected-html5.txt:
  • html5lib/runner-expected.txt:

2010-07-06 Eric Seidel <​eric@webkit.org>

Reviewed by Adam Barth.

Add support for <li>, <dd> and <dt> nested tag closing
​https://bugs.webkit.org/show_bug.cgi?id=41720

li, dd, and dt have nearly identical logic in HTML5, so I
attempted to make them share as much code as possible.

This fixes a bunch of subtests for html5lib/runner.html

  • html/HTMLTreeBuilder.cpp: (WebCore::HTMLTreeBuilder::processCloseWhenNestedTag): (WebCore::HTMLTreeBuilder::processStartTagForInBody): (WebCore::HTMLTreeBuilder::processAnyOtherEndTagForInBody): (WebCore::HTMLTreeBuilder::furthestBlockForFormattingElement):
  • html/HTMLTreeBuilder.h:
4:03 PM Changeset in webkit [62602] by Csaba Osztrogonác
  • 2 edits in trunk/WebKit/qt

Client errors should set "WebKitErrorDomain" domain instead of "WebKit" to match Mac.

Rubber-stamped by Kenneth Rohde Christiansen.

  • WebCoreSupport/FrameLoaderClientQt.cpp:

(WebCore::FrameLoaderClientQt::blockedError):
(WebCore::FrameLoaderClientQt::cannotShowURLError):
(WebCore::FrameLoaderClientQt::interruptForPolicyChangeError):
(WebCore::FrameLoaderClientQt::cannotShowMIMETypeError):

4:02 PM Changeset in webkit [62601] by vitalyr@chromium.org
  • 2 edits in trunk/LayoutTests

2010-07-06 Vitaly Repeshko <​vitalyr@chromium.org>

Unreviewed.

[chromium] Update expectations.

  • platform/chromium/test_expectations.txt:
3:52 PM Changeset in webkit [62600] by andersca@apple.com
  • 4 edits in trunk/WebKit2

Add PluginInfoStore::findPlugin
​https://bugs.webkit.org/show_bug.cgi?id=41719

Reviewed by Sam Weinig.

  • UIProcess/Plugins/PluginInfoStore.cpp:

(WebKit::PluginInfoStore::getPlugins):
(WebKit::PluginInfoStore::findPluginForMIMEType):
(WebKit::PluginInfoStore::findPluginForExtension):
(WebKit::pathExtension):
(WebKit::PluginInfoStore::findPlugin):

  • UIProcess/Plugins/PluginInfoStore.h:
  • UIProcess/Plugins/mac/PluginInfoStoreMac.mm:

(WebKit::PluginInfoStore::mimeTypeFromExtension):

3:51 PM Changeset in webkit [62599] by abarth@webkit.org
  • 4 edits in trunk/WebCore

2010-07-06 Adam Barth <​abarth@webkit.org>

Reviewed by Eric Seidel.

Factor HTMLConstructionSite out of HTMLTreeBuilder
​https://bugs.webkit.org/show_bug.cgi?id=41716

The HTMLContructionSite is the model object on which the
HTMLTreeBuilder (a controller) acts.

No behavior change. I'll move this class into its own file in a
followup patch.

  • html/HTMLFormattingElementList.h: (WebCore::HTMLFormattingElementList::at):
  • html/HTMLTreeBuilder.cpp: (WebCore::HTMLTreeBuilder::HTMLTreeBuilder): (WebCore::HTMLConstructionSite::HTMLConstructionSite): (WebCore::HTMLTreeBuilder::passTokenToLegacyParser): (WebCore::HTMLTreeBuilder::processDoctypeToken): (WebCore::HTMLConstructionSite::insertHTMLHtmlStartTagBeforeHTML): (WebCore::HTMLConstructionSite::mergeAttributesFromTokenIntoElement): (WebCore::HTMLConstructionSite::insertHTMLHtmlStartTagInBody): (WebCore::HTMLConstructionSite::insertHTMLBodyStartTagInBody): (WebCore::HTMLTreeBuilder::processFakePEndTagIfPInScope): (WebCore::HTMLTreeBuilder::processIsindexStartTagForInBody): (WebCore::HTMLTreeBuilder::processStartTagForInBody): (WebCore::HTMLTreeBuilder::processColgroupEndTagForInColumnGroup): (WebCore::HTMLTreeBuilder::closeTheCell): (WebCore::HTMLTreeBuilder::processStartTagForInTable): (WebCore::HTMLTreeBuilder::processStartTag): (WebCore::HTMLTreeBuilder::processBodyEndTagForInBody): (WebCore::HTMLTreeBuilder::processAnyOtherEndTagForInBody): (WebCore::HTMLTreeBuilder::furthestBlockForFormattingElement): (WebCore::HTMLTreeBuilder::findFosterParentFor): (WebCore::HTMLTreeBuilder::callTheAdoptionAgency): (WebCore::HTMLTreeBuilder::resetInsertionModeAppropriately): (WebCore::HTMLTreeBuilder::processEndTagForInBody): (WebCore::HTMLTreeBuilder::processCaptionEndTagForInCaption): (WebCore::HTMLTreeBuilder::processTrEndTagForInRow): (WebCore::HTMLTreeBuilder::processEndTagForInTable): (WebCore::HTMLTreeBuilder::processEndTag): (WebCore::HTMLTreeBuilder::processComment): (WebCore::HTMLTreeBuilder::processCharacter): (WebCore::HTMLTreeBuilder::processEndOfFile): (WebCore::HTMLTreeBuilder::processDefaultForBeforeHTMLMode): (WebCore::HTMLTreeBuilder::processStartTagForInHead): (WebCore::HTMLConstructionSite::insertDoctype): (WebCore::HTMLConstructionSite::insertComment): (WebCore::HTMLConstructionSite::insertCommentOnDocument): (WebCore::HTMLConstructionSite::insertCommentOnHTMLHtmlElement): (WebCore::HTMLConstructionSite::createElementAndAttachToCurrent): (WebCore::HTMLConstructionSite::insertHTMLHtmlElement): (WebCore::HTMLConstructionSite::insertHTMLHeadElement): (WebCore::HTMLConstructionSite::insertHTMLBodyElement): (WebCore::HTMLConstructionSite::insertElement): (WebCore::HTMLConstructionSite::insertSelfClosingElement): (WebCore::HTMLConstructionSite::insertFormattingElement): (WebCore::HTMLTreeBuilder::processGenericRCDATAStartTag): (WebCore::HTMLTreeBuilder::processGenericRawTextStartTag): (WebCore::HTMLConstructionSite::insertScriptElement): (WebCore::HTMLTreeBuilder::processScriptStartTag): (WebCore::HTMLConstructionSite::insertTextNode): (WebCore::HTMLConstructionSite::createElement): (WebCore::HTMLTreeBuilder::indexOfFirstUnopenFormattingElement): (WebCore::HTMLTreeBuilder::reconstructTheActiveFormattingElements): (WebCore::HTMLTreeBuilder::generateImpliedEndTagsWithExclusion): (WebCore::HTMLTreeBuilder::generateImpliedEndTags):
  • html/HTMLTreeBuilder.h: (WebCore::HTMLConstructionSite::currentElement): (WebCore::HTMLConstructionSite::openElements): (WebCore::HTMLConstructionSite::activeFormattingElements): (WebCore::HTMLConstructionSite::head): (WebCore::HTMLConstructionSite::form): (WebCore::HTMLConstructionSite::releaseForm): (WebCore::HTMLConstructionSite::setForm): (WebCore::HTMLConstructionSite::fragmentScriptingPermission): (WebCore::HTMLConstructionSite::attach):
3:32 PM Changeset in webkit [62598] by kov@webkit.org
  • 2 edits in trunk/WebCore

2010-07-06 Gustavo Noronha Silva <​gustavo.noronha@collabora.co.uk>

Reviewed by Xan Lopez.

[GTK] Scrollbars sometimes go dead and stop scrolling the view
​https://bugs.webkit.org/show_bug.cgi?id=41711

Fixes GtkScrolledWindow scrollbars not actually scrolling the page
in certain conditions. No tests because it is hard to reproduce,
and depends on interaction with a widget that is outside of the
WebView, which is tricky.

  • platform/gtk/ScrollViewGtk.cpp: (WebCore::ScrollView::setGtkAdjustments):
2:26 PM Changeset in webkit [62597] by vitalyr@chromium.org
  • 3 edits
    11 adds in trunk/LayoutTests

2010-07-06 Vitaly Repeshko <​vitalyr@chromium.org>

Unreviewed.

[chromium] Update expectations.

  • platform/chromium-linux/svg/custom/svg-curve-with-relative-cordinates-expected.checksum: Added.
  • platform/chromium-linux/svg/custom/svg-curve-with-relative-cordinates-expected.png: Added.
  • platform/chromium-mac/fast/xmlhttprequest/xmlhttprequest-no-file-access-expected.txt: Added.
  • platform/chromium-mac/fast/xmlhttprequest/xmlhttprequest-nonexistent-file-expected.txt: Added.
  • platform/chromium-mac/html5lib/runner-expected.txt:
  • platform/chromium-mac/svg/custom/svg-curve-with-relative-cordinates-expected.checksum: Added.
  • platform/chromium-mac/svg/custom/svg-curve-with-relative-cordinates-expected.png: Added.
  • platform/chromium-win/fast/xmlhttprequest/xmlhttprequest-no-file-access-expected.txt: Added.
  • platform/chromium-win/fast/xmlhttprequest/xmlhttprequest-nonexistent-file-expected.txt: Added.
  • platform/chromium-win/html5lib/runner-expected.txt:
  • platform/chromium-win/svg/custom/svg-curve-with-relative-cordinates-expected.checksum: Added.
  • platform/chromium-win/svg/custom/svg-curve-with-relative-cordinates-expected.png: Added.
  • platform/chromium-win/svg/custom/svg-curve-with-relative-cordinates-expected.txt: Added.
2:21 PM Changeset in webkit [62596] by Darin Adler
  • 8 edits in trunk/WebCore

2010-07-06 Darin Adler <Darin Adler>

Reviewed by Geoffrey Garen.

Simplify access to element attribute map, removing unneeded functions
​https://bugs.webkit.org/show_bug.cgi?id=41703

  • css/CSSStyleSelector.cpp: (WebCore::CSSStyleSelector::canShareStyleWithElement): Use the attributeMap function instead of the mappedAttributes function. (WebCore::CSSStyleSelector::styleForElement): Ditto.
  • dom/Element.cpp: (WebCore::Element::~Element): Renamed namedAttrMap to m_attributeMap. (WebCore::Element::removeAttribute): Ditto. (WebCore::Element::getAttribute): Ditto. (WebCore::Element::setAttribute): Ditto. (WebCore::Element::setAttributeMap): Ditto. (WebCore::Element::hasAttributes): Ditto. (WebCore::Element::createAttributeMap): Ditto. (WebCore::Element::insertedIntoDocument): Ditto. (WebCore::Element::removedFromDocument): Ditto. (WebCore::Element::getURLAttribute): Ditto.
  • dom/Element.h: Removed unneeded virtual attributes function. Made read-only argument to attributes function default to false. (Later, we'll eliminate this.) Renamed namedAttrMap to m_attributeMap and made it private instead of protected.
  • dom/StyledElement.cpp: (WebCore::StyledElement::attributeChanged): Use attributeMap function instead of namedAttrMap data member or mappedAttributes function. (WebCore::StyledElement::classAttributeChanged): Ditto. (WebCore::StyledElement::parseMappedAttribute): Ditto.
  • dom/StyledElement.h: Removed mappedAttributes function and changed callers to use attributeMap function instead.
  • html/HTMLInputElement.cpp: (WebCore::HTMLInputElement::setInputType): Use attributeMap instead of mappedAttributes.
  • svg/SVGStyledElement.cpp: (WebCore::SVGStyledElement::getPresentationAttribute): Ditto.
2:13 PM Changeset in webkit [62595] by weinig@apple.com
  • 2 edits in trunk/WebKitTools

Fix MiniBrowser build.

  • MiniBrowser/mac/WebBundle/WebBundleMain.m:
2:02 PM Changeset in webkit [62594] by andersca@apple.com
  • 2 edits in trunk/WebCore

Add a build rule that removes the old generated Obj-C bindings for PluginArray and MimeTypeArray.

Reviewed by Sam Weinig.

  • DerivedSources.make:
1:58 PM Changeset in webkit [62593] by abarth@webkit.org
  • 2 edits in trunk/WebCore

2010-07-06 Adam Barth <​abarth@webkit.org>

Unreviewed.

Make the remaining insertion modes explicit.

  • html/HTMLTreeBuilder.cpp: (WebCore::HTMLTreeBuilder::processStartTag): (WebCore::HTMLTreeBuilder::processEndTag): (WebCore::HTMLTreeBuilder::processCharacter): (WebCore::HTMLTreeBuilder::processEndOfFile):
1:51 PM Changeset in webkit [62592] by ap@apple.com
  • 4 edits in trunk

Not reviewed, fixing a simple copy/paste mistake.

​https://bugs.webkit.org/show_bug.cgi?id=41156
Cross origin XMLHttpRequest should log the reason why connection failed

Covered by xmlhttprequest/access-control-basic-non-simple-deny-cached.html

  • loader/CrossOriginPreflightResultCache.cpp: (WebCore::CrossOriginPreflightResultCacheItem::allowsCrossOriginHeaders): Fix incorrectly copy/pasted header field name.
1:51 PM Changeset in webkit [62591] by weinig@apple.com
  • 3 edits in trunk/WebKitTools

Add support for dumping the contents of messages to the console to WebKitTestRunner

Reviewed by Anders Carlsson.

  • WebKitTestRunner/InjectedBundle/InjectedBundlePage.cpp:

(WTR::InjectedBundlePage::InjectedBundlePage):
(WTR::InjectedBundlePage::_addMessageToConsole):
(WTR::InjectedBundlePage::addMessageToConsole):

  • WebKitTestRunner/InjectedBundle/InjectedBundlePage.h:
1:43 PM Changeset in webkit [62590] by weinig@apple.com
  • 10 edits
    2 moves
    2 adds in trunk/WebKit2

Add injected bundle API for exposing console messages.

Reviewed by Anders Carlsson.

  • Split WKBundlePageClient into multiple clients matching WKPage. (For now we only have a loader client and a UI client)
  • WebKit2.xcodeproj/project.pbxproj:
  • WebProcess/InjectedBundle/API/c/WKBundlePage.cpp:

(WKBundlePageSetLoaderClient):
(WKBundlePageSetUIClient):

  • WebProcess/InjectedBundle/API/c/WKBundlePage.h:
  • WebProcess/InjectedBundle/InjectedBundlePageClient.cpp: Removed.
  • WebProcess/InjectedBundle/InjectedBundlePageClient.h: Removed.
  • WebProcess/InjectedBundle/InjectedBundlePageLoaderClient.cpp: Copied from WebProcess/InjectedBundle/InjectedBundlePageClient.cpp.

(WebKit::InjectedBundlePageLoaderClient::InjectedBundlePageLoaderClient):
(WebKit::InjectedBundlePageLoaderClient::initialize):
(WebKit::InjectedBundlePageLoaderClient::didStartProvisionalLoadForFrame):
(WebKit::InjectedBundlePageLoaderClient::didReceiveServerRedirectForProvisionalLoadForFrame):
(WebKit::InjectedBundlePageLoaderClient::didFailProvisionalLoadWithErrorForFrame):
(WebKit::InjectedBundlePageLoaderClient::didCommitLoadForFrame):
(WebKit::InjectedBundlePageLoaderClient::didFinishLoadForFrame):
(WebKit::InjectedBundlePageLoaderClient::didFailLoadWithErrorForFrame):
(WebKit::InjectedBundlePageLoaderClient::didReceiveTitleForFrame):
(WebKit::InjectedBundlePageLoaderClient::didClearWindowObjectForFrame):

  • WebProcess/InjectedBundle/InjectedBundlePageLoaderClient.h: Copied from WebProcess/InjectedBundle/InjectedBundlePageClient.h.
  • WebProcess/InjectedBundle/InjectedBundlePageUIClient.cpp: Added.

(WebKit::InjectedBundlePageUIClient::InjectedBundlePageUIClient):
(WebKit::InjectedBundlePageUIClient::initialize):
(WebKit::InjectedBundlePageUIClient::addMessageToConsole):

  • WebProcess/InjectedBundle/InjectedBundlePageUIClient.h: Added.
  • WebProcess/WebCoreSupport/WebChromeClient.cpp:

(WebKit::WebChromeClient::addMessageToConsole):

  • WebProcess/WebCoreSupport/WebFrameLoaderClient.cpp:

(WebKit::WebFrameLoaderClient::dispatchDidReceiveServerRedirectForProvisionalLoad):
(WebKit::WebFrameLoaderClient::dispatchDidStartProvisionalLoad):
(WebKit::WebFrameLoaderClient::dispatchDidReceiveTitle):
(WebKit::WebFrameLoaderClient::dispatchDidCommitLoad):
(WebKit::WebFrameLoaderClient::dispatchDidFailProvisionalLoad):
(WebKit::WebFrameLoaderClient::dispatchDidFailLoad):
(WebKit::WebFrameLoaderClient::dispatchDidFinishLoad):
(WebKit::WebFrameLoaderClient::dispatchDidClearWindowObjectInWorld):

  • WebProcess/WebPage/WebPage.cpp:

(WebKit::WebPage::initializeInjectedBundleLoaderClient):
(WebKit::WebPage::initializeInjectedBundleUIClient):

  • WebProcess/WebPage/WebPage.h:

(WebKit::WebPage::injectedBundleLoaderClient):
(WebKit::WebPage::injectedBundleUIClient):

  • mac/WebKit2.exp:
  • win/WebKit2.vcproj:
1:31 PM Changeset in webkit [62589] by Darin Adler
  • 2 edits in trunk/WebKit/chromium

Try to fix build.

Why do the Chromium bots stop after the first error they encounter?

  • src/WebWorkerImpl.cpp:

(WebKit::WebWorkerImpl::postMessageToWorkerContextTask): Removed
unneeded release called on a PassOwnPtr to pass it as an argument of
type PassOwnPtr.

1:30 PM Changeset in webkit [62588] by kov@webkit.org
  • 1 edit in trunk/WebCore/ChangeLog

Adding webkit bugzilla reference for 62546

1:27 PM Changeset in webkit [62587] by Darin Adler
  • 2 edits in trunk/WebKit/qt

Fix crash caused by my fix to the Qt build.

  • WebCoreSupport/InspectorClientQt.cpp:

(WebCore::InspectorFrontendClientQt::closeWindow): Release pointer into
a local OwnPtr instead of calling clear. Another fix, probably better,
would be to change OwnPtr::clear to zero out the pointer before deleting,
but that affects many more call sites, so I will do that separately.

1:23 PM Changeset in webkit [62586] by xan@webkit.org
  • 2 edits in trunk/WebCore

2010-07-06 Xan Lopez <​xlopez@igalia.com>

GTK+ fixes for the DOM plug-in renaming Apocalypse.

  • GNUmakefile.am:
1:18 PM Changeset in webkit [62585] by Darin Adler
  • 3 edits in trunk/WebKit/chromium

Try to fix build.

  • src/WebWorkerBase.cpp:

(WebKit::WebWorkerBase::dispatchTaskToMainThread): Use leakPtr instead
of release.

  • src/WebWorkerClientImpl.cpp:

(WebKit::WebWorkerClientImpl::postMessageToWorkerObjectTask): Removed
unneeded release called on a PassOwnPtr to pass it as an argument of
type PassOwnPtr.

1:14 PM UpdatingBugzilla edited by ddkilzer@webkit.org
(diff)
1:05 PM Changeset in webkit [62584] by andersca@apple.com
  • 2 edits in trunk/WebCore

Yet another Windows build fix.

  • DerivedSources.cpp:
12:57 PM Changeset in webkit [62583] by andersca@apple.com
  • 2 edits in trunk/WebCore

Fix Windows build.

  • WebCore.vcproj/WebCore.vcproj:
12:39 PM Changeset in webkit [62582] by andersca@apple.com
  • 3 edits in trunk/WebCore

More build fixes.

  • WebCore.gypi:
  • WebCore.pro:
12:33 PM Changeset in webkit [62581] by andersca@apple.com
  • 2 edits in trunk/WebCore

Try to fix Qt build.

  • WebCore.pro:
12:32 PM Changeset in webkit [62580] by andersca@apple.com
  • 2 edits in trunk/WebCore

Attempt to fix GTK+ build.

  • GNUmakefile.am:
12:29 PM Changeset in webkit [62579] by abarth@webkit.org
  • 2 edits in trunk/WebCore

2010-07-06 Adam Barth <​abarth@webkit.org>

Address reviewer comments from ​https://bugs.webkit.org/show_bug.cgi?id=41671

Deploy isTableBodyContextTag where possible. No behavior change.

  • html/HTMLTreeBuilder.cpp: (WebCore::HTMLTreeBuilder::processStartTagForInBody): (WebCore::HTMLTreeBuilder::processStartTag): (WebCore::HTMLTreeBuilder::processEndTagForInTable): (WebCore::HTMLTreeBuilder::processEndTag):
12:27 PM Changeset in webkit [62578] by andersca@apple.com
  • 3 edits in trunk/WebCore

Build fixes.

  • WebCore.gypi:
  • bindings/js/JSBindingsAllInOne.cpp:
12:13 PM Changeset in webkit [62577] by andersca@apple.com
  • 12 edits
    15 moves in trunk/WebCore

DOM plug-in related renames
​https://bugs.webkit.org/show_bug.cgi?id=41709

Reviewed by Sam Weinig.

This is in preparation for using "Plugin" for something other than a DOM object.

MimeType => DOMMimeType
MimeTypeArray => DOMMimeTypeArray
Plugin => DOMPlugin
PluginArray => DOMPluginArray

  • CMakeLists.txt:
  • DerivedSources.make:
  • GNUmakefile.am:
  • WebCore.pri:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • bindings/js/JSDOMMimeTypeArrayCustom.cpp: Added.

(WebCore::JSDOMMimeTypeArray::canGetItemsForName):
(WebCore::JSDOMMimeTypeArray::nameGetter):

  • bindings/js/JSDOMPluginArrayCustom.cpp: Added.

(WebCore::JSDOMPluginArray::canGetItemsForName):
(WebCore::JSDOMPluginArray::nameGetter):

  • bindings/js/JSDOMPluginCustom.cpp: Added.

(WebCore::JSDOMPlugin::canGetItemsForName):
(WebCore::JSDOMPlugin::nameGetter):

  • bindings/js/JSMimeTypeArrayCustom.cpp: Removed.
  • bindings/js/JSPluginArrayCustom.cpp: Removed.
  • bindings/js/JSPluginCustom.cpp: Removed.
  • bindings/scripts/CodeGeneratorJS.pm:
  • page/DOMWindow.idl:
  • page/Navigator.cpp:

(WebCore::Navigator::plugins):
(WebCore::Navigator::mimeTypes):

  • page/Navigator.h:
  • page/Navigator.idl:
  • plugins/DOMMimeType.cpp: Added.

(WebCore::DOMMimeType::DOMMimeType):
(WebCore::DOMMimeType::~DOMMimeType):
(WebCore::DOMMimeType::type):
(WebCore::DOMMimeType::suffixes):
(WebCore::DOMMimeType::description):
(WebCore::DOMMimeType::enabledPlugin):

  • plugins/DOMMimeType.h: Added.

(WebCore::DOMMimeType::create):
(WebCore::DOMMimeType::mimeClassInfo):

  • plugins/DOMMimeType.idl: Added.
  • plugins/DOMMimeTypeArray.cpp: Added.

(WebCore::DOMMimeTypeArray::DOMMimeTypeArray):
(WebCore::DOMMimeTypeArray::~DOMMimeTypeArray):
(WebCore::DOMMimeTypeArray::length):
(WebCore::DOMMimeTypeArray::item):
(WebCore::DOMMimeTypeArray::canGetItemsForName):
(WebCore::DOMMimeTypeArray::namedItem):
(WebCore::DOMMimeTypeArray::getPluginData):

  • plugins/DOMMimeTypeArray.h: Added.

(WebCore::DOMMimeTypeArray::create):
(WebCore::DOMMimeTypeArray::disconnectFrame):

  • plugins/DOMMimeTypeArray.idl: Added.
  • plugins/DOMPlugin.cpp: Added.

(WebCore::DOMPlugin::DOMPlugin):
(WebCore::DOMPlugin::~DOMPlugin):
(WebCore::DOMPlugin::name):
(WebCore::DOMPlugin::filename):
(WebCore::DOMPlugin::description):
(WebCore::DOMPlugin::length):
(WebCore::DOMPlugin::item):
(WebCore::DOMPlugin::canGetItemsForName):
(WebCore::DOMPlugin::namedItem):

  • plugins/DOMPlugin.h: Added.

(WebCore::DOMPlugin::create):
(WebCore::DOMPlugin::pluginInfo):

  • plugins/DOMPlugin.idl: Added.
  • plugins/DOMPluginArray.cpp: Added.

(WebCore::DOMPluginArray::DOMPluginArray):
(WebCore::DOMPluginArray::~DOMPluginArray):
(WebCore::DOMPluginArray::length):
(WebCore::DOMPluginArray::item):
(WebCore::DOMPluginArray::canGetItemsForName):
(WebCore::DOMPluginArray::namedItem):
(WebCore::DOMPluginArray::refresh):
(WebCore::DOMPluginArray::pluginData):

  • plugins/DOMPluginArray.h: Added.

(WebCore::DOMPluginArray::create):
(WebCore::DOMPluginArray::disconnectFrame):

  • plugins/DOMPluginArray.idl: Added.
  • plugins/MimeType.cpp: Removed.
  • plugins/MimeType.h: Removed.
  • plugins/MimeType.idl: Removed.
  • plugins/MimeTypeArray.cpp: Removed.
  • plugins/MimeTypeArray.h: Removed.
  • plugins/MimeTypeArray.idl: Removed.
  • plugins/Plugin.cpp: Removed.
  • plugins/Plugin.h: Removed.
  • plugins/Plugin.idl: Removed.
  • plugins/PluginArray.cpp: Removed.
  • plugins/PluginArray.h: Removed.
  • plugins/PluginArray.idl: Removed.
12:10 PM Changeset in webkit [62576] by ap@apple.com
  • 35 edits in trunk

Reviewed by Darin Adler.

​https://bugs.webkit.org/show_bug.cgi?id=41156
Cross origin XMLHttpRequest should log the reason why connection failed

Covered by multiple existing tests.

  • loader/CrossOriginAccessControl.cpp: (WebCore::passesAccessControlCheck):
  • loader/CrossOriginAccessControl.h:
  • loader/CrossOriginPreflightResultCache.cpp: (WebCore::CrossOriginPreflightResultCacheItem::parse): (WebCore::CrossOriginPreflightResultCacheItem::allowsCrossOriginMethod): (WebCore::CrossOriginPreflightResultCacheItem::allowsCrossOriginHeaders): (WebCore::CrossOriginPreflightResultCacheItem::allowsRequest):
  • loader/CrossOriginPreflightResultCache.h: Functions that check requests now take a string argument for error explanation.
  • loader/DocumentThreadableLoader.cpp: (WebCore::DocumentThreadableLoader::DocumentThreadableLoader): Make an informative error if crossOriginRequestPolicy is DenyCrossOriginRequests. This doesn't currently go anywhere, since XMLHttpRequest is the only client that logs from didFail(), and it of course supports cross origin requests. (WebCore::DocumentThreadableLoader::makeSimpleCrossOriginAccessRequest): Make an informative error when trying to e.g. send a cross origin XMLHttpRequest to ​ftp://. (WebCore::DocumentThreadableLoader::didReceiveResponse): Pass error explanation from cross origin access control code. (WebCore::DocumentThreadableLoader::preflightFailure): Ditto.
  • loader/DocumentThreadableLoader.h: preflightFailure() now takes arguments.
  • platform/network/ResourceErrorBase.cpp:
  • platform/network/ResourceErrorBase.h: Added a constant for WebKit error domain.
  • xml/XMLHttpRequest.cpp: (WebCore::XMLHttpRequest::didFail): Report the error to console. One day we'll be able to also provide script file and line number here, so it's best to pass error all the way down to XHR for reporting.
11:52 AM Changeset in webkit [62575] by commit-queue@webkit.org
  • 2 edits in trunk/WebCore

2010-07-06 Pavel Podivilov <​podivilov@chromium.org>

Reviewed by Pavel Feldman.

Web Inspector: convert script offset to webkit format in v8 ScriptDebugServer
​https://bugs.webkit.org/show_bug.cgi?id=41696

  • bindings/v8/ScriptDebugServer.cpp: (WebCore::ScriptDebugServer::dispatchDidParseSource):
11:34 AM Changeset in webkit [62574] by Darin Adler
  • 2 edits in trunk/WebKit/chromium

Fix build.

  • src/WebSharedWorkerImpl.cpp:

(WebKit::WebSharedWorkerImpl::connectTask): Removed unneeded release
called on a PassOwnPtr to pass it as an argument of type PassOwnPtr.

11:33 AM Changeset in webkit [62573] by abarth@webkit.org
  • 6 edits in trunk

2010-07-06 Adam Barth <​abarth@webkit.org>

Reviewed by Eric Seidel.

Implement InTableBodyMode
​https://bugs.webkit.org/show_bug.cgi?id=41671

In the process of implementing this patch, Eric and I discussed some of
the high-level organization of the HTMLTreeBuilder class. We'd like to
split it up into smaller pieces. In the meantime, I've reordered some
of the declarations in the header to make the relations between the
different kinds of functions clearer.

  • html/HTMLElementStack.cpp:
  • html/HTMLTreeBuilder.cpp: (WebCore::HTMLTreeBuilder::processIsindexStartTagForInBody): (WebCore::HTMLTreeBuilder::processStartTagForInBody): (WebCore::HTMLTreeBuilder::processStartTagForInTable): (WebCore::HTMLTreeBuilder::processStartTag): (WebCore::HTMLTreeBuilder::processEndTagForInTable): (WebCore::HTMLTreeBuilder::processEndTag): (WebCore::HTMLTreeBuilder::processCharacter): (WebCore::HTMLTreeBuilder::processEndOfFile):
  • html/HTMLTreeBuilder.h:

2010-07-06 Adam Barth <​abarth@webkit.org>

Reviewed by Eric Seidel.

Implement InTableBodyMode
​https://bugs.webkit.org/show_bug.cgi?id=41671

  • html5lib/runner-expected-html5.txt:
11:25 AM Changeset in webkit [62572] by weinig@apple.com
  • 3 edits in trunk/WebKitTools

Patch for ​https://bugs.webkit.org/show_bug.cgi?id=41708
Add dumpAsText support for WebKitTestRunner

Reviewed by Anders Carlsson.

  • WebKitTestRunner/InjectedBundle/InjectedBundle.cpp:

(WTR::InjectedBundle::InjectedBundle):
(WTR::InjectedBundle::reset): Reset the LayoutTestController for each test.

  • WebKitTestRunner/InjectedBundle/InjectedBundlePage.cpp:

(WTR::InjectedBundlePage::didFinishLoadForFrame): Use the new WKBundleFrameCopyInnerText
API to dump the main frames text.

11:18 AM Changeset in webkit [62571] by leandro@webkit.org
  • 4 edits in trunk

2010-07-06 Leandro Pereira <​leandro@profusion.mobi>

Unreviewed build fix.

[EFL] Generate code for RemoteInspector.

  • CMakeLists.txt:

2010-07-06 Leandro Pereira <​leandro@profusion.mobi>

Unreviewed build fix.

[EFL] Add macro to generate inspector code.

  • cmake/WebKitGenerators.cmake:
11:12 AM Changeset in webkit [62570] by Darin Adler
  • 2 edits in trunk/WebKitTools

Fix Chromium build.

  • DumpRenderTree/chromium/WebViewHost.cpp:

(WebViewHost::didCreateDataSource): Call leakPtr instead of release.
(WebViewHost::didNavigateWithinPage): Ditto.
(WebViewHost::updateURL): Ditto.

11:10 AM Changeset in webkit [62569] by Darin Adler
  • 2 edits in trunk/WebKit/chromium

Fix build.

  • src/WebDataSourceImpl.cpp:

(WebKit::WebDataSourceImpl::setNextPluginLoadObserver): Call leakPtr
here instead of release. Comment makes it really unclear why it's OK!

11:10 AM Changeset in webkit [62568] by commit-queue@webkit.org
  • 3 edits
    4 adds in trunk

2010-07-06 Varun Jain <​varunjain@chromium.org>

Reviewed by Oliver Hunt.

Implementing CURVE_TO_CUBIC specification for SVG Path Segments. Currently,
curves specified in relative cordinates such as using createCurvetoCubicRel()
javascript function are not displayed.

SVG CurvetoCubic Path not implemented for relative cordinates
​https://bugs.webkit.org/show_bug.cgi?id=41294

  • platform/mac/svg/custom/svg-curve-with-relative-cordinates-expected.checksum: Added.
  • platform/mac/svg/custom/svg-curve-with-relative-cordinates-expected.png: Added.
  • platform/mac/svg/custom/svg-curve-with-relative-cordinates-expected.txt: Added.
  • svg/custom/svg-curve-with-relative-cordinates.html: Added.

2010-07-06 Varun Jain <​varunjain@chromium.org>

Reviewed by Oliver Hunt.

Implementing CURVE_TO_CUBIC specification for SVG Path Segments. Currently,
curves specified in relative cordinates such as using createCurvetoCubicRel()
javascript function are not displayed.

SVG CurvetoCubic Path not implemented for relative cordinates
​https://bugs.webkit.org/show_bug.cgi?id=41294

Test: svg/custom/svg-curve-with-relative-cordinates.html

  • svg/SVGPathSegList.cpp: (WebCore::SVGPathSegList::toPathData):
11:09 AM Changeset in webkit [62567] by weinig@apple.com
  • 6 edits in trunk/WebKit2

Patch for ​https://bugs.webkit.org/show_bug.cgi?id=41707
Add ability to dump frame inner text for the test runner

Reviewed by Anders Carlsson.

  • WebProcess/InjectedBundle/API/c/WKBundleFrame.cpp:

(WKBundleFrameCopyInnerText):

  • WebProcess/InjectedBundle/API/c/WKBundleFrame.h:
  • WebProcess/WebPage/WebFrame.cpp:

(WebKit::WebFrame::innerText):

  • WebProcess/WebPage/WebFrame.h:
  • mac/WebKit2.exp:
10:52 AM Changeset in webkit [62566] by ap@apple.com
  • 3 edits in trunk/WebCore

Roll out patch for ​https://bugs.webkit.org/show_bug.cgi?id=41348 "Remove global variables
from XSLTProcessorLibxslt.cpp", as it causes crashes on buildbot.

  • xml/XSLTProcessor.h: (WebCore::XSLTProcessor::xslStylesheet): (WebCore::XSLTProcessor::XSLTProcessor):
  • xml/XSLTProcessorLibxslt.cpp: (WebCore::docLoaderFunc): (WebCore::setXSLTLoadCallBack): (WebCore::xsltStylesheetPointer): (WebCore::XSLTProcessor::transformToString):
10:51 AM Changeset in webkit [62565] by xan@webkit.org
  • 2 edits in trunk/LayoutTests

2010-07-06 Xan Lopez <​xlopez@igalia.com>

Skip flakey a11y test.

  • platform/gtk/Skipped:
10:44 AM Changeset in webkit [62564] by Martin Robinson
  • 2 edits in trunk/WebKitTools

2010-07-06 Martin Robinson <​mrobinson@igalia.com>

Reviewed by Xan Lopez.

[GTK] build-webkit should not run autogen.sh unconditionally
​https://bugs.webkit.org/show_bug.cgi?id=41704

  • Scripts/webkitdirs.pm: Only run autogen.sh during a build if GNUmakefile doesn't exist. Autotools should re-run autogen.sh when necessary. Also do some very minor cleanup.
10:28 AM Changeset in webkit [62563] by abarth@webkit.org
  • 10 edits in trunk

2010-07-06 Eric Seidel <​eric@webkit.org>

Reviewed by Adam Barth.

Add processing for "in cell" mode and end tag processing for "in row"
​https://bugs.webkit.org/show_bug.cgi?id=41688

I added several new table tests, not all of which pass yet.
Remaining failures exist due to lack of full "in table body" mode
support, which Adam has an outstanding patch for.
I believe all of the various branches added by this change are
covered now by our tests.

  • html/HTMLElementStack.cpp:
    • Added QualifiedName versions of inScope functions. Using tagName.localName() is wrong for non-HTML elements, in preparation for supporting foreign content we should centralize our handling of QualifiedName in these functions instead of sprinkling more .localName() calls around the code.

(WebCore::HTMLElementStack::inScope):
(WebCore::HTMLElementStack::inListItemScope):
(WebCore::HTMLElementStack::inTableScope):

  • html/HTMLElementStack.h:
  • html/HTMLFormattingElementList.cpp: (WebCore::HTMLFormattingElementList::Entry::operator==): (WebCore::HTMLFormattingElementList::Entry::operator!=):
    • Calling element() on markers will assert, so use m_element.
  • html/HTMLTreeBuilder.cpp: (WebCore::HTMLTreeBuilder::closeTheCell):
    • Implemented per the spec.

(WebCore::HTMLTreeBuilder::processStartTag):

  • Use isTableBodyContextTag where possible.
  • Add InCellMode.

(WebCore::HTMLTreeBuilder::resetInsertionModeAppropriately):

  • Use isTableBodyContextTag where possible.

(WebCore::HTMLTreeBuilder::processTrEndTagForInRow):

  • This code is needed from at least two callsites.

(WebCore::HTMLTreeBuilder::processEndTag):

  • Add InCellMode and InRowMode

(WebCore::HTMLTreeBuilder::processCharacter):
(WebCore::HTMLTreeBuilder::processEndOfFile):

  • html/HTMLTreeBuilder.h:

2010-07-06 Eric Seidel <​eric@webkit.org>

Reviewed by Adam Barth.

Add processing for "in cell" mode and end tag processing for "in row"
​https://bugs.webkit.org/show_bug.cgi?id=41688

  • html5lib/resources/tables01.dat:
  • html5lib/runner-expected-html5.txt:
  • html5lib/runner-expected.txt:
10:25 AM Changeset in webkit [62562] by abarth@webkit.org
  • 4 edits in trunk

2010-07-06 Eric Seidel <​eric@webkit.org>

Reviewed by Adam Barth.

Make <a> auto-close any parent <a> tag
​https://bugs.webkit.org/show_bug.cgi?id=41684

I first fixed <a> to auto-close surrounding <a> tags.
That caused about 3 more test passes and one new test failure.
The test failure was due to <marque> not adding a marker
to the active formatting elements.

So I made a one-line change to fix <marque>. However that hit an
ASSERT due to a bug in indexOfFirstUnopenFormattingElement.
Fixing indexOfFirstUnopenFormattingElement fixed another bunch of
Adoption Agency related tests (clearly this was the second typo
which was causing all the adoption agency failures).


However fixing <marque> and indexOfFirstUnopenFormattingElement,
added yet another failure due to missing support for </p>
automatically adding an implicit <p> tag. Fixing </p> finally
got the tests to a stable (all improving) state.

In the end, these 4 minor (and very well tested) tweaks fixed a total
of 9 subtests in html5lib/runner.html.
Another subtest progressed, but did not fully pass due to missing
<table> foster-parenting support.

  • html/HTMLTreeBuilder.cpp: (WebCore::HTMLTreeBuilder::processStartTagForInBody): (WebCore::HTMLTreeBuilder::processEndTagForInBody): (WebCore::HTMLTreeBuilder::indexOfFirstUnopenFormattingElement):

2010-07-06 Eric Seidel <​eric@webkit.org>

Reviewed by Adam Barth.

Make <a> auto-close any parent <a> tag
​https://bugs.webkit.org/show_bug.cgi?id=41684

  • html5lib/runner-expected-html5.txt:
    • Update expectations now that we pass 9 more subtests.
10:16 AM Changeset in webkit [62561] by Philippe Normand
  • 2 edits in trunk/WebCore

2010-07-02 Philippe Normand <​pnormand@igalia.com>

Reviewed by Xan Lopez

[GStreamer] can't seek past maxTimeLoaded value
​https://bugs.webkit.org/show_bug.cgi?id=40526

Extended the seekable range to the whole media. This allows video
played with progressive download to be seeked past the current
buffered position.

  • platform/graphics/gstreamer/MediaPlayerPrivateGStreamer.cpp: (WebCore::MediaPlayerPrivateGStreamer::maxTimeSeekable): (WebCore::MediaPlayerPrivateGStreamer::updateStates):
10:09 AM Changeset in webkit [62560] by andersca@apple.com
  • 7 edits
    4 copies
    3 moves
    1 add
    1 delete in trunk/WebKit2

Only export a single WebKitMain function for the Web Process to call
​https://bugs.webkit.org/show_bug.cgi?id=41702

Reviewed by Sam Weinig.

  • Shared/CommandLine.h:

Add windows CommandLine::parse overload.

  • Shared/win/CommandLineWin.cpp: Added.

(WebKit::CommandLine::parse):
Move parse overload from WebProcess/win/WebProcessMain.cpp.

  • UIProcess/Launcher/win/ProcessLauncherWin.cpp:

(WebKit::ProcessLauncher::launchProcess):
Pass the right mode to.

  • WebKit2.xcodeproj/project.pbxproj:

Update for moves.

  • WebProcess/Launching/mac/WebProcessMain.mm: Removed.
  • WebProcess/Launching/win/WebProcessWinMain.cpp: Removed.
  • WebProcess/WebKitMain.cpp: Added.

(WebKitMain):
(enableTerminationOnHeapCorruption):
Add WebKitMain functions for Mac and Windows.

  • WebProcess/WebProcessMain.h: Added.
  • WebProcess/mac/WebProcessMainMac.mm: Added.

(WebKit::WebProcessMain):
Add Mac version of WebProcessMain.

  • WebProcess/win/WebProcessMain.cpp: Removed.
  • WebProcess/win/WebProcessMain.h: Removed.
  • WebProcess/win/WebProcessMainWin.cpp: Added.

(WebKit::WebProcessMain):
Add Windows version of WebProcessMain.

  • mac/MainMac.cpp: Added.

(main):
Add Mac WebProcess main function.

  • mac/WebKit2.exp:

Remove functions that we don't need to export.

  • win/MainWin.cpp: Added.

(wWinMain):
Add Windows WebProcess main function.

  • win/WebKit2.vcproj:
  • win/WebKit2WebProcess.vcproj:

Update for moves.

10:05 AM Changeset in webkit [62559] by ap@apple.com
  • 3 edits in trunk/WebCore

2010-07-06 Andreas Wictor <​andreas.wictor@xcerion.com>

Reviewed by Alexey Proskuryakov.

Remove global variables from XSLTProcessorLibxslt.cpp
​https://bugs.webkit.org/show_bug.cgi?id=41348

Remove the globalProcessor and globalDocLoader global variables
by using the _private field that exists on most libxml structs.

No new tests, existing tests covers this.

  • xml/XSLTProcessor.h: (WebCore::XSLTProcessor::sourceNode): (WebCore::XSLTProcessor::XSLTProcessor):
  • xml/XSLTProcessorLibxslt.cpp: (WebCore::registredXSLTProcessors): (WebCore::registredXSLStyleSheets): (WebCore::docLoaderFunc): (WebCore::xsltStylesheetPointer): (WebCore::XSLTProcessor::transformToString):
9:45 AM Changeset in webkit [62558] by Darin Adler
  • 2 edits in trunk/WebKit/chromium

Fix build.

  • src/SharedWorkerRepository.cpp:

(WebCore::SharedWorkerRepository::connect): Remove unneeded release
called on a PassOwnPtr to pass it as an argument of type PassOwnPtr.

9:38 AM Changeset in webkit [62557] by Darin Adler
  • 2 edits in trunk/WebCore

Fix Cairo build.

  • platform/graphics/cairo/GraphicsContextCairo.cpp:

(WebCore::GraphicsContext::createPlatformShadow): Removed unnneded release
in code that passes a PassOwnPtr to another PassOwnPtr.

9:36 AM Changeset in webkit [62556] by Darin Adler
  • 2 edits in trunk/WebKitTools

Try to fix Chromium build.

  • DumpRenderTree/chromium/WebViewHost.cpp: Added include of PassOwnPtr.h.
9:35 AM Changeset in webkit [62555] by Darin Adler
  • 2 edits in trunk/WebKitTools

Fix build.

  • WebKitAPITest/TestsController.cpp:

(WebKitAPITest::TestsController::addTest): Use leakPtr instead of release.

9:32 AM Changeset in webkit [62554] by Darin Adler
  • 2 edits in trunk/WebKit/qt

Fix Qt build.

  • WebCoreSupport/InspectorClientQt.cpp:

(WebCore::InspectorFrontendClientQt::closeWindow): Call clear instead of
calling delete on the result of leakPtr.

9:31 AM Changeset in webkit [62553] by Darin Adler
  • 2 edits in trunk/WebCore

Fix Chromium build.

  • platform/graphics/skia/SkiaUtils.cpp:

(WebCore::scratchContext): Use leakPtr instead of release.

9:28 AM Changeset in webkit [62552] by Martin Robinson
  • 4 edits in trunk

2010-07-06 Martin Robinson <​mrobinson@igalia.com>

Unreviewed.

Build fix after r62549.

  • GNUmakefile.am: Remove new variable that is no longer used.

2010-07-06 Martin Robinson <​mrobinson@igalia.com>

Unreviewed.

Build fix after r62549.

  • GNUmakefile.am: Generate RemoteInspector files by manually listing them instead of building them into libWebCoreJS.la.
9:14 AM Changeset in webkit [62551] by Darin Adler
  • 32 edits in trunk

2010-07-06 Darin Adler <Darin Adler>

Reviewed by Adam Barth.

Add adoptPtr and leakPtr functions for OwnPtr and PassOwnPtr
​https://bugs.webkit.org/show_bug.cgi?id=41320

  • bytecode/CodeBlock.cpp: (JSC::CodeBlock::reparseForExceptionInfoIfNecessary): Use assignment instead of set since the result of reparseExceptionInfo is now a PassOwnPtr.
  • bytecode/CodeBlock.h: Change extractExceptionInfo to return a PassOwnPtr instead of a raw pointer.
  • runtime/Executable.cpp: (JSC::FunctionExecutable::reparseExceptionInfo): Return a PassOwnPtr. (JSC::EvalExecutable::reparseExceptionInfo): Ditto. (JSC::ProgramExecutable::reparseExceptionInfo): Added. This was in the header before, but it's better to not have it there to reduce header dependencies. Return a PassOwnPtr.
  • runtime/Executable.h: Made reparseExceptionInfo return a PassOwnPtr, and put it in the private sections of classes other than the base class.
  • wtf/MessageQueue.h: (WTF::MessageQueue::append): Use leakPtr instead of release. (WTF::MessageQueue::appendAndCheckEmpty): Ditto. (WTF::MessageQueue::prepend): Ditto.
  • wtf/OwnPtr.h: Tweaked formatting. Changed the release function to return a PassOwnPtr rather than a raw pointer. Added a leakPtr function that returns a raw pointer. Put the constructor that takes a raw pointer and the set function into a section guarded by LOOSE_OWN_PTR. Adapted to the new adoptPtr function from PassOwnPtr.h.
  • wtf/PassOwnPtr.h: Tweaked formatting. Renamed the release function to leakPtr. Added an adoptPtr function that creates a new PassOwnPtr. Put the constructor and assignment operators that take a raw pointer into a section guarded by LOOSE_PASS_OWN_PTR.

2010-07-06 Darin Adler <Darin Adler>

Reviewed by Adam Barth.

Add adoptPtr and leakPtr functions for OwnPtr and PassOwnPtr
​https://bugs.webkit.org/show_bug.cgi?id=41320

Made code changes required because of the change to the release function.
The equivalent to the old release function is now named leakPtr and
should be used sparingly. The new release function returns a PassOwnPtr.

  • css/CSSFontFaceSource.cpp: (WebCore::CSSFontFaceSource::getFontData): Changed code to call leakPtr instead of release.
  • css/CSSParser.cpp: (WebCore::CSSParser::addProperty): Ditto.
  • css/CSSSegmentedFontFace.cpp: (WebCore::CSSSegmentedFontFace::getFontData): Removed unneeded type casting. Not sure why this changed the type to FontData* and then casted back to SimpleFontData*.
  • css/MediaQuery.cpp: (WebCore::MediaQuery::MediaQuery): Removed call to release on a PassOwnPtr, since the data member is now an OwnPtr. (WebCore::MediaQuery::~MediaQuery): Removed now-unneeded delete.
  • css/MediaQuery.h: Changed m_expressions to be an OwnPtr.
  • html/HTMLToken.h: (WebCore::AtomicHTMLToken::AtomicHTMLToken): Use assignment instead of the set function since there are no raw pointers involved.
  • loader/CachedResource.cpp: (WebCore::CachedResource::makePurgeable): Ditto.
  • loader/CrossOriginPreflightResultCache.cpp: (WebCore::CrossOriginPreflightResultCache::appendEntry): Use leakPtr instead of release, and also add FIXME about deleting the old value if the original and URL are already in the map. I believe dealing with this FIXME may fix a storage leak.
  • loader/CrossOriginPreflightResultCache.h: Change the argument to be PassOwnPtr instead of a raw pointer, since this function does take ownership.
  • loader/DocumentThreadableLoader.cpp: (WebCore::DocumentThreadableLoader::DocumentThreadableLoader): Use assignment instead of the set function since there are no raw pointers involved.
  • loader/FrameLoader.cpp: (WebCore::FrameLoader::startIconLoader): Ditto.
  • loader/TextResourceDecoder.cpp: (WebCore::TextResourceDecoder::decode): Ditto. (WebCore::TextResourceDecoder::flush): Ditto.
  • page/DOMTimer.cpp: (WebCore::DOMTimer::fired): Use OwnPtr instead of an explicit delete.
  • platform/CrossThreadCopier.h: Removed explicit code that tried to copy PassOwnPtr in a complicated way. It did nothing different from just returning the PassOwnPtr. This presumably was done because PassRefPtr has issues when copied cross-thread, but there are no similar issues for PassOwnPtr. Someone with more experience than I might be able to remove the specialization altogether, because CrossThreadCopierPassThrough does the right thing in this case.
  • platform/SharedBuffer.cpp: (WebCore::SharedBuffer::adoptPurgeableBuffer): Changed argument to be a PassOwnPtr. (WebCore::SharedBuffer::releasePurgeableBuffer): Changed result to be a PassOwnPtr.
  • platform/SharedBuffer.h: Updated for changes above.
  • rendering/RenderSVGResourceFilter.cpp: (WebCore::RenderSVGResourceFilter::applyResource): Changed one site to use assignment instead of the set function since there are no raw pointers involved. Changed another site to use leakPtr instead of release.
  • rendering/RenderSVGResourceGradient.cpp: (WebCore::createMaskAndSwapContextForTextGradient): Use assignment instead of the set function since there are no raw pointers involved.
  • rendering/style/RenderStyle.cpp: (WebCore::RenderStyle::setContent): Use leakPtr instead of release. (WebCore::RenderStyle::setBoxShadow): Ditto.
  • workers/DefaultSharedWorkerRepository.cpp: (WebCore::DefaultSharedWorkerRepository::connectToWorker): Removed unneeded call to release function in a code path that passes a PassOwnPtr to a function that takes a PassOwnPtr.
  • workers/WorkerContext.cpp: (WebCore::WorkerContext::setTimeout): Changed argument type to PassOwnPtr. (WebCore::WorkerContext::setInterval): Ditto.
  • workers/WorkerContext.h: Updated for changes above.
  • workers/WorkerMessagingProxy.cpp: (WebCore::WorkerMessagingProxy::postMessageToWorkerObject): (WebCore::WorkerMessagingProxy::postMessageToWorkerContext): Removed unneeded call to release function in code paths that pass a PassOwnPtr to a function that takes a PassOwnPtr.
8:47 AM Changeset in webkit [62550] by weinig@apple.com
  • 2 edits in trunk/JavaScriptCore

Update comment in StringExtras.h to be more accurate.

Reviewed by Darin Adler

  • wtf/StringExtras.h:
8:04 AM Changeset in webkit [62549] by loislo@chromium.org
  • 21 edits
    2 adds in trunk

2010-07-06 Ilya Tikhonovsky <​loislo@chromium.org>

Reviewed by Yury Semikhatsky.

WebInspector: generator part of the patch for bug 40675.
On the way to Remote Debugging we want to support JSON serialization
on both sides of WebInspector transport.
As far as InspectorFrontend class is a simple proxy to WebInspector
it would be better to generate it from an IDL file.
We have generator infrastructure for binding and will reuse it for
new generator.
​https://bugs.webkit.org/show_bug.cgi?id=41692

  • CMakeLists.txt:
  • DerivedSources.make:
  • GNUmakefile.am:
  • WebCore.gyp/WebCore.gyp:
  • WebCore.gyp/scripts/rule_binding.py:
  • WebCore.gypi:
  • WebCore.pri:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • bindings/scripts/CodeGenerator.pm:
  • bindings/scripts/IDLParser.pm:
  • bindings/scripts/generate-bindings.pl:
  • inspector/CodeGeneratorInspector.pm: Added.
  • inspector/InspectorController.cpp: (WebCore::InspectorController::connectFrontend): (WebCore::InspectorController::startTimelineProfiler):
  • inspector/InspectorController.h:
  • inspector/InspectorFrontend.cpp:
  • inspector/InspectorFrontend.h:
  • inspector/InspectorFrontend2.idl: Added.
  • inspector/InspectorTimelineAgent.cpp: (WebCore::InspectorTimelineAgent::InspectorTimelineAgent): (WebCore::InspectorTimelineAgent::resetFrontendProxyObject): (WebCore::InspectorTimelineAgent::addRecordToTimeline):
  • inspector/InspectorTimelineAgent.h:
7:58 AM Changeset in webkit [62548] by vitalyr@chromium.org
  • 4 edits in trunk/LayoutTests

2010-07-06 Vitaly Repeshko <​vitalyr@chromium.org>

Unreviewed.

[chromium] Update expectations.

  • platform/chromium-mac/html5lib/runner-expected.txt:
  • platform/chromium-win/html5lib/runner-expected.txt:
  • platform/chromium-win/svg/hixie/error/013-expected.txt:
6:55 AM Changeset in webkit [62547] by commit-queue@webkit.org
  • 8 edits in trunk/JavaScriptCore/qt

2010-07-06 Jedrzej Nowacki <​jedrzej.nowacki@nokia.com>

Reviewed by Kenneth Rohde Christiansen.

Implementation of QScriptValue properties accessors.

The patch contains implementation of the QScriptValue::property() and
the QScriptValue::setProperty(). It is not full functionality, as these
method are too complex for one patch, but it is enough to cover about
95% of use cases.

Missing functionality:

  • Few of the PropertyFlags are ignored.
  • Only a public part of the ResolveFlags can be used (ResolveLocal, ResolvePrototype).

A lot of new test cases were added.

[Qt] QScriptValue should have API for accessing object properties
​https://bugs.webkit.org/show_bug.cgi?id=40903

  • api/qscriptconverter_p.h: (QScriptConverter::toPropertyFlags):
  • api/qscriptstring_p.h: (QScriptStringPrivate::operator JSStringRef):
  • api/qscriptvalue.cpp: (QScriptValue::property): (QScriptValue::setProperty):
  • api/qscriptvalue.h: (QScriptValue::):
  • api/qscriptvalue_p.h: (QScriptValuePrivate::assignEngine): (QScriptValuePrivate::property): (QScriptValuePrivate::hasOwnProperty): (QScriptValuePrivate::setProperty): (QScriptValuePrivate::deleteProperty):
  • tests/qscriptvalue/tst_qscriptvalue.cpp: (tst_QScriptValue::getPropertySimple_data): (tst_QScriptValue::getPropertySimple): (tst_QScriptValue::setPropertySimple): (tst_QScriptValue::getPropertyResolveFlag): (tst_QScriptValue::getSetProperty): (tst_QScriptValue::setProperty_data): (tst_QScriptValue::setProperty):
  • tests/qscriptvalue/tst_qscriptvalue.h:
5:52 AM Changeset in webkit [62546] by kov@webkit.org
  • 2 edits in trunk/WebCore

2010-07-06 Gustavo Noronha Silva <​gustavo.noronha@collabora.co.uk>

Rubber-stamped by Xan Lopez.

Crash when hitting back on some pages
​http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=575236

Also NULL-check document, in hopes of fixing this hard to
reproduce crash that we are getting reported quite a bit.

  • page/FrameView.cpp: (WebCore::FrameView::windowClipRect):
5:11 AM Changeset in webkit [62545] by rolandsteiner@chromium.org
  • 7 edits
    3 copies
    3 adds in trunk/WebKitTools

2010-07-06 Roland Steiner <​rolandsteiner@chromium.org>

Reviewed by Kent Tamura.

Bug 40558 - [DRT/Chromium] Upstream TestShellDevTools for Chromium DRT
​https://bugs.webkit.org/show_bug.cgi?id=40558

Upstream DevTools for Chromium DRT.
(original Chromium files rev. 51287)

  • DumpRenderTree/DumpRenderTree.gypi:
  • DumpRenderTree/chromium/DRTDevToolsAgent.cpp: Added. (DRTDevToolsAgent::DRTDevToolsAgent): (DRTDevToolsAgent::setWebView): (DRTDevToolsAgent::sendMessageToFrontend): (DRTDevToolsAgent::forceRepaint): (DRTDevToolsAgent::runtimeFeatureStateChanged): (DRTDevToolsAgent::injectedScriptSource): (DRTDevToolsAgent::injectedScriptDispatcherSource): (DRTDevToolsAgent::debuggerScriptSource): (DRTDevToolsAgent::asyncCall): (DRTDevToolsAgent::call): (DRTDevToolsAgent::webDevToolsAgent): (DRTDevToolsAgent::attach): (DRTDevToolsAgent::detach): (DRTDevToolsAgent::setTimelineProfilingEnabled): (DRTDevToolsAgent::evaluateInWebInspector): (DRTDevToolsAgent::dispatchMessageLoop):
  • DumpRenderTree/chromium/DRTDevToolsAgent.h: Added. (DRTDevToolsAgent::~DRTDevToolsAgent): (DRTDevToolsAgent::hostIdentifier):
  • DumpRenderTree/chromium/DRTDevToolsCallArgs.cpp:
  • DumpRenderTree/chromium/DRTDevToolsCallArgs.h: (DRTDevToolsCallArgs::DRTDevToolsCallArgs): (DRTDevToolsCallArgs::~DRTDevToolsCallArgs): (DRTDevToolsCallArgs::callsCount):
  • DumpRenderTree/chromium/DRTDevToolsClient.cpp: Added. (DRTDevToolsClient::DRTDevToolsClient): (DRTDevToolsClient::~DRTDevToolsClient): (DRTDevToolsClient::sendMessageToAgent): (DRTDevToolsClient::sendDebuggerCommandToAgent): (DRTDevToolsClient::activateWindow): (DRTDevToolsClient::closeWindow): (DRTDevToolsClient::dockWindow): (DRTDevToolsClient::undockWindow): (DRTDevToolsClient::asyncCall): (DRTDevToolsClient::call): (DRTDevToolsClient::allMessagesProcessed):
  • DumpRenderTree/chromium/DRTDevToolsClient.h:
  • DumpRenderTree/chromium/EventSender.cpp:
  • DumpRenderTree/chromium/LayoutTestController.cpp: (LayoutTestController::LayoutTestController): (LayoutTestController::closeWebInspector): (LayoutTestController::setTimelineProfilingEnabled): (LayoutTestController::evaluateInWebInspector):
  • DumpRenderTree/chromium/LayoutTestController.h:
  • DumpRenderTree/chromium/TestShell.cpp: (TestShell::TestShell): (TestShell::~TestShell): (TestShell::createDRTDevToolsClient): (TestShell::showDevTools): (TestShell::closeDevTools): (TestShell::runFileTest): (TestShell::createNewWindow):
  • DumpRenderTree/chromium/TestShell.h: (TestShell::drtDevToolsAgent): (TestShell::drtDevToolsClient):
  • DumpRenderTree/chromium/config.h:
4:34 AM Changeset in webkit [62544] by commit-queue@webkit.org
  • 7 edits in trunk/JavaScriptCore

2010-07-06 Sheriff Bot <​webkit.review.bot@gmail.com>

Unreviewed, rolling out r62511.
​http://trac.webkit.org/changeset/62511
​https://bugs.webkit.org/show_bug.cgi?id=41686

Breaks Linux/64bit compilation (Requested by xan_ on #webkit).

  • jit/ExecutableAllocator.cpp:
  • jit/ExecutableAllocatorFixedVMPool.cpp: (JSC::FixedVMPoolAllocator::FixedVMPoolAllocator): (JSC::FixedVMPoolAllocator::free): (JSC::ExecutablePool::systemAlloc):
  • jit/ExecutableAllocatorPosix.cpp: (JSC::ExecutableAllocator::reprotectRegion): (JSC::ExecutableAllocator::cacheFlush):
  • jit/ExecutableAllocatorSymbian.cpp:
  • jit/ExecutableAllocatorWin.cpp:
  • wtf/Platform.h:
4:12 AM Changeset in webkit [62543] by Csaba Osztrogonác
  • 5 edits
    6 adds in trunk/LayoutTests

Unreviewed.

[Qt] png/checksums for tests need to be updated after r62444
​https://bugs.webkit.org/show_bug.cgi?id=41568

  • platform/qt/fast/dynamic/anchor-lock-expected.checksum: Added.
  • platform/qt/fast/dynamic/anchor-lock-expected.png: Added.
  • platform/qt/fast/overflow/position-fixed-transform-clipping-expected.checksum: Added.
  • platform/qt/fast/overflow/position-fixed-transform-clipping-expected.png: Added.
  • platform/qt/fast/repaint/fixed-tranformed-expected.checksum: Updated.
  • platform/qt/fast/repaint/fixed-tranformed-expected.png: Updated.
  • platform/qt/fast/repaint/repaint-during-scroll-expected.checksum: Updated.
  • platform/qt/fast/repaint/repaint-during-scroll-expected.png: Updated.
  • platform/qt/transforms/2d/transform-fixed-container-expected.checksum: Added.
  • platform/qt/transforms/2d/transform-fixed-container-expected.png: Added.
3:33 AM Changeset in webkit [62542] by loislo@chromium.org
  • 16 edits in trunk

2010-07-05 Ilya Tikhonovsky <​loislo@chromium.org>

Reviewed by Yury Semikhatsky.

Web Inspector: On the way to Remote Debuging we want to support JSON
serialization on both sides of WebInspector transport. As far as
InspectorFrontend class is a simple proxy to WebInspector it would
be better to generate it from an IDL file. We have generator
infostructure for binding and will reuse it for new generator.
​https://bugs.webkit.org/show_bug.cgi?id=40675

  • bindings/js/ScriptCallStack.cpp: (WebCore::ScriptCallStack::stackTrace):
  • bindings/js/ScriptCallStack.h:
  • bindings/v8/ScriptCallStack.cpp: (WebCore::ScriptCallStack::stackTrace):
  • bindings/v8/ScriptCallStack.h:
  • inspector/InspectorFrontend.cpp: (WebCore::InspectorFrontend::addRecordToTimeline):
  • inspector/InspectorFrontend.h:
  • inspector/InspectorTimelineAgent.cpp: (WebCore::InspectorTimelineAgent::pushGCEventRecords): (WebCore::InspectorTimelineAgent::willCallFunction): (WebCore::InspectorTimelineAgent::willDispatchEvent): (WebCore::InspectorTimelineAgent::willLayout): (WebCore::InspectorTimelineAgent::willRecalculateStyle): (WebCore::InspectorTimelineAgent::willPaint): (WebCore::InspectorTimelineAgent::willWriteHTML): (WebCore::InspectorTimelineAgent::didWriteHTML): (WebCore::InspectorTimelineAgent::didInstallTimer): (WebCore::InspectorTimelineAgent::didRemoveTimer): (WebCore::InspectorTimelineAgent::willFireTimer): (WebCore::InspectorTimelineAgent::willChangeXHRReadyState): (WebCore::InspectorTimelineAgent::willLoadXHR): (WebCore::InspectorTimelineAgent::willEvaluateScript): (WebCore::InspectorTimelineAgent::didScheduleResourceRequest): (WebCore::InspectorTimelineAgent::willSendResourceRequest): (WebCore::InspectorTimelineAgent::willReceiveResourceData): (WebCore::InspectorTimelineAgent::willReceiveResourceResponse): (WebCore::InspectorTimelineAgent::didFinishLoadingResource): (WebCore::InspectorTimelineAgent::didMarkTimeline): (WebCore::InspectorTimelineAgent::didMarkDOMContentEvent): (WebCore::InspectorTimelineAgent::didMarkLoadEvent): (WebCore::InspectorTimelineAgent::addRecordToTimeline): (WebCore::InspectorTimelineAgent::setHeapSizeStatistic): (WebCore::InspectorTimelineAgent::didCompleteCurrentRecord): (WebCore::InspectorTimelineAgent::pushCurrentRecord):
  • inspector/InspectorTimelineAgent.h: (WebCore::InspectorTimelineAgent::TimelineRecordEntry::TimelineRecordEntry):
  • inspector/TimelineRecordFactory.cpp: (WebCore::TimelineRecordFactory::createGenericRecord): (WebCore::TimelineRecordFactory::createGCEventData): (WebCore::TimelineRecordFactory::createFunctionCallData): (WebCore::TimelineRecordFactory::createEventDispatchData): (WebCore::TimelineRecordFactory::createGenericTimerData): (WebCore::TimelineRecordFactory::createTimerInstallData): (WebCore::TimelineRecordFactory::createXHRReadyStateChangeData): (WebCore::TimelineRecordFactory::createXHRLoadData): (WebCore::TimelineRecordFactory::createEvaluateScriptData): (WebCore::TimelineRecordFactory::createMarkTimelineData): (WebCore::TimelineRecordFactory::createScheduleResourceRequestData): (WebCore::TimelineRecordFactory::createResourceSendRequestData): (WebCore::TimelineRecordFactory::createResourceReceiveResponseData): (WebCore::TimelineRecordFactory::createResourceFinishData): (WebCore::TimelineRecordFactory::createReceiveResourceData): (WebCore::TimelineRecordFactory::createPaintData): (WebCore::TimelineRecordFactory::createParseHTMLData):
  • inspector/TimelineRecordFactory.h:
2:43 AM Changeset in webkit [62541] by commit-queue@webkit.org
  • 2 edits in trunk/WebKitTools

2010-07-06 Gabor Rapcsanyi <​rgabor@inf.u-szeged.hu>

Reviewed by Eric Seidel.

Don't pass image hash to DRT when pixel tests are disabled.
​https://bugs.webkit.org/show_bug.cgi?id=41597

  • Scripts/webkitpy/layout_tests/layout_package/dump_render_tree_thread.py:
2:20 AM Changeset in webkit [62540] by abarth@webkit.org
  • 5 edits in trunk

2010-07-06 Adam Barth <​abarth@webkit.org>

Reviewed by Eric Seidel.

Implement InColgroupMode
​https://bugs.webkit.org/show_bug.cgi?id=41663

  • html5lib/runner-expected-html5.txt:

2010-07-06 Adam Barth <​abarth@webkit.org>

Reviewed by Eric Seidel.

Implement InColgroupMode
​https://bugs.webkit.org/show_bug.cgi?id=41663

  • html/HTMLTreeBuilder.cpp: (WebCore::HTMLTreeBuilder::processColgroupEndTagForInColumnGroup): (WebCore::HTMLTreeBuilder::processStartTag): (WebCore::HTMLTreeBuilder::processEndTag): (WebCore::HTMLTreeBuilder::processCharacter): (WebCore::HTMLTreeBuilder::processEndOfFile):
  • html/HTMLTreeBuilder.h:
1:54 AM Changeset in webkit [62539] by eric@webkit.org
  • 2 edits in trunk/LayoutTests

2010-07-06 Eric Seidel <​eric@webkit.org>

Unreviewed. Update test expectations for the LegacyTreeBuilder codepath.

Implement bookmarking for HTML5 Adoption Agency algorithm
​https://bugs.webkit.org/show_bug.cgi?id=41659

  • html5lib/runner-expected.txt:
1:27 AM Changeset in webkit [62538] by yuzo@google.com
  • 2 edits in trunk/LayoutTests

2010-07-06 Yuzo Fujishima <​yuzo@google.com>

Unreviewed test expectation update.

unicode-range doesn't seem to be implemented for Qt.

  • platform/qt/Skipped:
1:16 AM Changeset in webkit [62537] by eric@webkit.org
  • 7 edits in trunk

2010-07-06 Eric Seidel <​eric@webkit.org>

Reviewed by Adam Barth.

Implement bookmarking for HTML5 Adoption Agency algorithm
​https://bugs.webkit.org/show_bug.cgi?id=41659

Added 3 new adoption tests because the suite didn't
cover all the bookmarking cases.

  • html5lib/runner-expected-html5.txt:
    • Fixes tests!
  • html5lib/resources/adoption01.dat

2010-07-06 Eric Seidel <​eric@webkit.org>

Reviewed by Adam Barth.

Implement bookmarking for HTML5 Adoption Agency algorithm
​https://bugs.webkit.org/show_bug.cgi?id=41659

Was easier than I expected it to be.
Fixes a bunch of html5lib/runner.html tests.

  • html/HTMLFormattingElementList.cpp: (WebCore::HTMLFormattingElementList::bookmarkFor): (WebCore::HTMLFormattingElementList::insertAt):
  • html/HTMLFormattingElementList.h: (WebCore::HTMLFormattingElementList::Bookmark::Bookmark): (WebCore::HTMLFormattingElementList::Bookmark::moveToAfter): (WebCore::HTMLFormattingElementList::Bookmark::elementBefore): (WebCore::HTMLFormattingElementList::Bookmark::elementAfter):
  • html/HTMLTreeBuilder.cpp: (WebCore::HTMLTreeBuilder::callTheAdoptionAgency):
1:06 AM Changeset in webkit [62536] by abarth@webkit.org
  • 2 edits in trunk/WebCore

2010-07-06 Adam Barth <​abarth@webkit.org>

Reviewed by Eric Seidel.

Implement more of InTableMode
​https://bugs.webkit.org/show_bug.cgi?id=41652

By the time we got around to landing this patch, it turns out to be
just a bit of cleanup (the functional changes where landed already.)

  • html/HTMLTreeBuilder.cpp: (WebCore::HTMLTreeBuilder::processCharacter): (WebCore::HTMLTreeBuilder::processEndOfFile):
12:54 AM Changeset in webkit [62535] by commit-queue@webkit.org
  • 11 edits in trunk/WebCore

2010-07-06 Sheriff Bot <​webkit.review.bot@gmail.com>

Unreviewed, rolling out r62529.
​http://trac.webkit.org/changeset/62529
​https://bugs.webkit.org/show_bug.cgi?id=41661

http/tests/inspector/resource-har-conversion.html is broken
(Requested by yutak on #webkit).

  • inspector/InspectorController.cpp: (WebCore::InspectorController::addResource): (WebCore::InspectorController::removeResource):
  • inspector/InspectorController.h:
  • inspector/InspectorResource.cpp: (WebCore::InspectorResource::InspectorResource): (WebCore::InspectorResource::updateScriptObject): (WebCore::InspectorResource::cachedResource): (WebCore::InspectorResource::type): (WebCore::InspectorResource::resourceData):
  • inspector/InspectorResource.h: (WebCore::InspectorResource::): (WebCore::InspectorResource::create):
  • inspector/front-end/Resource.js: (WebInspector.Resource.Type.toString): (WebInspector.Resource.prototype.set type): (WebInspector.Resource.prototype._mimeTypeIsConsistentWithType):
  • inspector/front-end/ResourceView.js: (WebInspector.ResourceView.prototype._refreshRequestHeaders): (WebInspector.ResourceView.prototype._refreshResponseHeaders): (WebInspector.ResourceView.prototype._refreshHeaders):
  • inspector/front-end/inspector.css: (.resources-category-scripts, .resources-category-xhr, .resources-category-fonts, .resources-category-other):
  • inspector/front-end/inspector.js: (WebInspector.loaded): (WebInspector.updateResource):
  • websockets/WebSocketChannel.cpp: (WebCore::WebSocketChannel::WebSocketChannel): (WebCore::WebSocketChannel::disconnect): (WebCore::WebSocketChannel::didOpen): (WebCore::WebSocketChannel::didClose): (WebCore::WebSocketChannel::processBuffer):
  • websockets/WebSocketChannel.h:
12:53 AM Changeset in webkit [62534] by Nikolas Zimmermann
  • 1 edit
    1 add in trunk/LayoutTests

2010-07-06 Nikolas Zimmermann <​nzimmermann@rim.com>

Not reviewed. Add missing layout test result.

  • svg/css/svg-ellipse-render-crash-expected.txt: Added.
12:39 AM Changeset in webkit [62533] by abarth@webkit.org
  • 5 edits in trunk

2010-07-06 Adam Barth <​abarth@webkit.org>

Reviewed by Eric Seidel.

Implement InCaptionMode
​https://bugs.webkit.org/show_bug.cgi?id=41660

Implementing this mode also turned up a subtle bug in the adoption
agency code.

  • html/HTMLTreeBuilder.cpp: (WebCore::HTMLTreeBuilder::processStartTag): (WebCore::HTMLTreeBuilder::processCaptionEndTagForInCaption): (WebCore::HTMLTreeBuilder::processEndTag): (WebCore::HTMLTreeBuilder::processCharacter): (WebCore::HTMLTreeBuilder::indexOfFirstUnopenFormattingElement):
    • We're supposed to reutrn the first *unopened* formatting element. The old code returned the first *opened* formatting element.
  • html/HTMLTreeBuilder.h:

2010-07-06 Adam Barth <​abarth@webkit.org>

Reviewed by Eric Seidel.

Implement InCaptionMode
​https://bugs.webkit.org/show_bug.cgi?id=41660

Massive test progression due to fixing Mr. Adoption Agency.

  • html5lib/runner-expected-html5.txt:
12:35 AM Changeset in webkit [62532] by abarth@webkit.org
  • 5 edits in trunk

2010-07-06 Adam Barth <​abarth@webkit.org>

Reviewed by Eric Seidel.

Implement defaut cases for InTableMode
​https://bugs.webkit.org/show_bug.cgi?id=41656

  • html/HTMLTreeBuilder.cpp: (WebCore::HTMLTreeBuilder::processStartTag): (WebCore::HTMLTreeBuilder::processEndTagForInBody): (WebCore::HTMLTreeBuilder::processEndTag):
  • html/HTMLTreeBuilder.h:

2010-07-06 Adam Barth <​abarth@webkit.org>

Reviewed by Eric Seidel.

Implement defaut cases for InTableMode
​https://bugs.webkit.org/show_bug.cgi?id=41656

  • html5lib/runner-expected-html5.txt:
12:28 AM Changeset in webkit [62531] by Nikolas Zimmermann
  • 12 edits in trunk

2010-07-05 Nikolas Zimmermann <​nzimmermann@rim.com>

Reviewed by Dirk Schulze.

SVGRenderContainer forces too many kids to relayout
​https://bugs.webkit.org/show_bug.cgi?id=15391

Fixing a long-standing performance issue. We should only ever need to relayout container children when the parent bounds change.
The bounds of a container can only change, if the outermost RenderSVGRoot container uses relative length values and its size changes.
This can either happen when the window resizes for standalone SVG documents, or if an enclosing RenderBox changes width/height values.

Only relayout container children if the container has selfNeedsLayout() true, or if the parent bounds change.
Lively Kernel doesn't do any relayouts anymore, except if you change the Safari window size, this is a great progression.

  • rendering/RenderPath.cpp: (WebCore::RenderPath::layout): No need to special case Path updates, if the element uses relative lengths. SVGRenderSupport now handles this case.
  • rendering/RenderSVGContainer.cpp: (WebCore::RenderSVGContainer::layout): Fix some style issues.
  • rendering/RenderSVGContainer.h: (WebCore::RenderSVGContainer::setDrawsContents): Inlined for speed. (WebCore::RenderSVGContainer::drawsContents): Ditto.
  • rendering/RenderSVGRoot.cpp: (WebCore::RenderSVGRoot::RenderSVGRoot): Initialize m_isLayoutSizeChanged to false. (WebCore::RenderSVGRoot::layout): Set m_isLayoutSizeChanged=true when the RenderSVGRoot size changes during layout. (WebCore::RenderSVGRoot::calcViewport): Remove hasRelativeLengths() special case.
  • rendering/RenderSVGRoot.h: (WebCore::RenderSVGRoot::isLayoutSizeChanged): New function, which returns true during layout() if the outermost <svg> size changes.
  • rendering/RenderSVGViewportContainer.cpp: (WebCore::RenderSVGViewportContainer::calcViewport): Cleanup code, and remove obsolete hasRelativeLengths() special case.
  • rendering/SVGRenderSupport.cpp: (WebCore::svgRootTreeObject): Added helper function, that returns the RenderSVGRoot for a given RenderObject. (WebCore::SVGRenderSupport::layoutChildren): Remove FIXME, only relayout container children, if the parent bounds change and the child uses relative lengths.
  • svg/SVGStyledElement.cpp: (WebCore::SVGStyledElement::updateRelativeLengthsInformation): Implemented this function. Keeps track of relative lengths elements, so that the

hasRelativeLengths() information is always up2date.

  • svg/SVGStyledElement.h: (WebCore::SVGStyledElement::hasRelativeLengths): Don't call the virtual selfHasRelativeLengths() information, just return wheter m_elementsWithRelativeLengths is not empty.
12:09 AM Changeset in webkit [62530] by yuzo@google.com
  • 8 edits
    2 adds in trunk

2010-07-05 Yuzo Fujishima <​yuzo@google.com>

Reviewed by Dan Bernstein.

Fix for Bug 41509 - Ranges for @font-face unicode-range must be separated by commas
​https://bugs.webkit.org/show_bug.cgi?id=41509

  • fast/css/beforeSelectorOnCodeElement.html: Fixed the space-separated ranges.
  • fast/css/font-face-multiple-ranges-for-unicode-range-expected.txt: Added.
  • fast/css/font-face-multiple-ranges-for-unicode-range.html: Added.
  • fast/css/text-security.html: Fixed the space-separated ranges.
  • fast/events/updateLayoutForHitTest.html: Fixed the space-separated ranges.
  • fast/text/capitalize-boundaries.html: Fixed the space-separated ranges.
  • platform/win/css2.1/resources/Mac-compatible-font-fallback.css: (@font-face): Fixed the space-separated ranges.

2010-07-05 Yuzo Fujishima <​yuzo@google.com>

Reviewed by Dan Bernstein.

Fix for Bug 41509 - Ranges for @font-face unicode-range must be separated by commas
​https://bugs.webkit.org/show_bug.cgi?id=41509

Test: fast/css/font-face-multiple-ranges-for-unicode-range.html

  • css/CSSParser.cpp: (WebCore::CSSParser::parseFontFaceUnicodeRange):

Jul 5, 2010:

11:48 PM Changeset in webkit [62529] by hamaji@chromium.org
  • 11 edits in trunk/WebCore

2010-07-05 Yuta Kitamura <​yutak@chromium.org>

Reviewed by Pavel Feldman.

Add WebSocket resource type to Web Inspector.

When a new WebSocket connection is established, a line for that connection
will appear in Web Inspector's Resources tab. If the resource name is
clicked, the details of handshake request and response will be shown.

Web Inspector: WebSocket in Resources tab
​https://bugs.webkit.org/show_bug.cgi?id=40768

  • inspector/InspectorController.cpp: (WebCore::InspectorController::addResource): WebSocket resource does not have an associated loader, thus frame might be null. Need to check it. (WebCore::InspectorController::removeResource): Ditto. (WebCore::InspectorController::didCreateWebSocket): (WebCore::InspectorController::willSendWebSocketHandshakeRequest): (WebCore::InspectorController::didReceiveWebSocketHandshakeResponse): (WebCore::InspectorController::didCloseWebSocket):
  • inspector/InspectorController.h:
  • inspector/InspectorResource.cpp: Add null checks of m_loader and m_frame, because WebSocket does not have a loader and we need to allow null for these variables. (WebCore::createReadableStringFromBinary): (WebCore::InspectorResource::InspectorResource): (WebCore::InspectorResource::create): Factory function of regular (non-WebSocket) resources. (WebCore::InspectorResource::createWebSocket): Factory function of WebSocket resources. (WebCore::InspectorResource::updateWebSocketRequest): (WebCore::InspectorResource::updateWebSocketResponse): (WebCore::InspectorResource::updateScriptObject): (WebCore::InspectorResource::cachedResource): (WebCore::InspectorResource::type): (WebCore::InspectorResource::resourceData):
  • inspector/InspectorResource.h: (WebCore::InspectorResource::): (WebCore::InspectorResource::markWebSocket):
  • inspector/front-end/Resource.js: (WebInspector.Resource.Type.toString): (WebInspector.Resource.prototype.set type): (WebInspector.Resource.prototype._mimeTypeIsConsistentWithType):
  • inspector/front-end/ResourceView.js: (WebInspector.ResourceView.prototype._refreshRequestHeaders): (WebInspector.ResourceView.prototype._refreshResponseHeaders): (WebInspector.ResourceView.prototype._refreshHeaders):
  • inspector/front-end/inspector.css: (.resources-category-websockets, .resources-category-other): (.resources-category-websockets .resources-graph-bar): (.resources-category-websockets.resource-cached .resources-graph-bar):
  • inspector/front-end/inspector.js: (WebInspector.loaded): (WebInspector.updateResource):
  • websockets/WebSocketChannel.cpp: (WebCore::WebSocketChannel::WebSocketChannel): (WebCore::WebSocketChannel::disconnect): (WebCore::WebSocketChannel::didOpen): (WebCore::WebSocketChannel::didClose): (WebCore::WebSocketChannel::processBuffer): (WebCore::WebSocketChannel::identifier):
  • websockets/WebSocketChannel.h:
11:43 PM Changeset in webkit [62528] by yurys@chromium.org
  • 2 edits in trunk/WebCore

2010-07-05 Yury Semikhatsky <​yurys@chromium.org>

Unreviewed. Fix Chromium build.

  • inspector/InspectorApplicationCacheAgent.cpp: (WebCore::InspectorApplicationCacheAgent::fillResourceList): (WebCore::InspectorApplicationCacheAgent::getApplicationCaches):
11:29 PM Changeset in webkit [62527] by abarth@webkit.org
  • 3 edits in trunk/WebCore

2010-07-05 Adam Barth <​abarth@webkit.org>

Unreviewed.

Move processStartTagForInBody to its own function.

  • html/HTMLTreeBuilder.cpp: (WebCore::HTMLTreeBuilder::processStartTagForInBody): (WebCore::HTMLTreeBuilder::processStartTag):
  • html/HTMLTreeBuilder.cpp.orig: Added.
  • html/HTMLTreeBuilder.h:
11:29 PM Changeset in webkit [62526] by Csaba Osztrogonác
  • 2 edits in trunk/WebKitTools

2010-07-05 Csaba Osztrogonác <Csaba Osztrogonác>

Reviewed by Eric Seidel.

REGRESSION(r60652): WebKitTools/Scripts/ensure-valid-python should cleanup temporary directory
​https://bugs.webkit.org/show_bug.cgi?id=41612

  • Scripts/ensure-valid-python: File::Temp::tempdir call fixed.
11:14 PM Changeset in webkit [62525] by eric@webkit.org
  • 4 edits in trunk

2010-07-05 Eric Seidel <​eric@webkit.org>

Reviewed by Adam Barth.

Fix one more notImplemented in h1-h6 start tag handling
​https://bugs.webkit.org/show_bug.cgi?id=41654

  • html5lib/runner-expected-html5.txt:
    • Fix two sub-tests!

2010-07-05 Eric Seidel <​eric@webkit.org>

Reviewed by Adam Barth.

Fix one more notImplemented in h1-h6 start tag handling
​https://bugs.webkit.org/show_bug.cgi?id=41654

  • html/HTMLTreeBuilder.cpp: (WebCore::HTMLTreeBuilder::processStartTag):
11:05 PM Changeset in webkit [62524] by eric@webkit.org
  • 3 edits in trunk/WebCore

2010-07-05 Eric Seidel <​eric@webkit.org>

Unreviewed. Just fixing proess to process.
Clearly Adam and I can't spell. Thankfully MikeSmith can.

Add <isindex> support, per HTML5
​https://bugs.webkit.org/show_bug.cgi?id=41650

  • html/HTMLTreeBuilder.cpp: (WebCore::HTMLTreeBuilder::processFakeStartTag): (WebCore::HTMLTreeBuilder::processFakeEndTag): (WebCore::HTMLTreeBuilder::processIsindexStartTagForBody): (WebCore::HTMLTreeBuilder::processStartTag):
  • html/HTMLTreeBuilder.h:
11:01 PM Changeset in webkit [62523] by eric@webkit.org
  • 9 edits
    1 add in trunk

2010-07-05 Eric Seidel <​eric@webkit.org>

Reviewed by Adam Barth.

Add <isindex> support, per HTML5
​https://bugs.webkit.org/show_bug.cgi?id=41650

  • html5lib/resources/tests2.dat:
    • Make the expected wording match the HTML5 spec. Not sure why the wording in this test diverged.
  • html5lib/resources/isindex.dat:
    • Cover more isindex cases.
  • html5lib/runner.html:

2010-07-05 Eric Seidel <​eric@webkit.org>

Reviewed by Adam Barth.

Add <isindex> support, per HTML5
​https://bugs.webkit.org/show_bug.cgi?id=41650

Covered by html5lib/runner.html including a new
isindex.dat test suite.

  • html/HTMLToken.h: (WebCore::AtomicHTMLToken::AtomicHTMLToken):
    • Support passing attributes to the constructor.

(WebCore::AtomicHTMLToken::name):
(WebCore::AtomicHTMLToken::setName):
(WebCore::AtomicHTMLToken::getAttributeItem):
(WebCore::AtomicHTMLToken::attributes):
(WebCore::AtomicHTMLToken::takeAtributes):

  • Reduces ref-churn, and makes it possible for callers to modify attributes w/o affecting future uses of the attributes.

(WebCore::AtomicHTMLToken::usesName):

  • Used by ASSERTS.

(WebCore::AtomicHTMLToken::usesAttributes):

  • Used by ASSERTS.
  • html/HTMLTreeBuilder.cpp: (WebCore::convertToOldStyle):
    • Can't be const, now that we use takeAttributes()

(WebCore::HTMLTreeBuilder::insertHTMLStartTagBeforeHTML):
(WebCore::HTMLTreeBuilder::proesssFakeStartTag):

  • New function. I'm not sure this is the perfect design (I'd kinda like AtomicHTMLToken to be copyable so we can have create functions for it), but this makes the callsites using fake tokens much more readable.

(WebCore::HTMLTreeBuilder::proesssFakeEndTag):
(WebCore::HTMLTreeBuilder::processFakeCharacters):
(WebCore::HTMLTreeBuilder::attributesForIsindexInput):
(WebCore::HTMLTreeBuilder::processIsindexStartTagForBody):
(WebCore::HTMLTreeBuilder::processStartTag):
(WebCore::HTMLTreeBuilder::insertScriptElement):

  • Use takeAttributes() for less ref-churn.

(WebCore::HTMLTreeBuilder::createElement): ditto
(WebCore::HTMLTreeBuilder::finished):

  • Remove bogus use of AtomicHTMLToken constructor which wasn't even being used now that we support emitting EOF tokens from the Tokenizer directly.
  • html/HTMLTreeBuilder.h:
10:38 PM Changeset in webkit [62522] by abarth@webkit.org
  • 2 edits in trunk/WebCore

2010-07-05 Adam Barth <​abarth@webkit.org>

Reviewed by Eric Seidel.

ASSERT that we're processing the correct type of token
​https://bugs.webkit.org/show_bug.cgi?id=41647

Making these asserts work required a small (non-observable) tweak to
some old code.

  • html/HTMLTreeBuilder.cpp: (WebCore::HTMLTreeBuilder::processDoctypeToken): (WebCore::HTMLTreeBuilder::processStartTag): (WebCore::HTMLTreeBuilder::processBodyEndTagForInBody): (WebCore::HTMLTreeBuilder::processAnyOtherEndTagForInBody): (WebCore::HTMLTreeBuilder::processEndTag): (WebCore::HTMLTreeBuilder::processComment): (WebCore::HTMLTreeBuilder::processCharacter): (WebCore::HTMLTreeBuilder::processEndOfFile): (WebCore::HTMLTreeBuilder::processStartTagForInHead):
10:31 PM Changeset in webkit [62521] by abarth@webkit.org
  • 6 edits in trunk

2010-07-05 Adam Barth <​abarth@webkit.org>

Reviewed by Eric Seidel.

Implement in select in table
​https://bugs.webkit.org/show_bug.cgi?id=41646

Add some tests for this mode. We fail all these tests currently
because there's no way to get into this mode. :(

  • html5lib/resources/tables01.dat:
  • html5lib/runner-expected-html5.txt:
  • html5lib/runner-expected.txt:

2010-07-05 Adam Barth <​abarth@webkit.org>

Reviewed by Eric Seidel.

Implement in select in table
​https://bugs.webkit.org/show_bug.cgi?id=41646

This mode is mostly a fall-through to the InSelectMode.

  • html/HTMLTreeBuilder.cpp: (WebCore::HTMLTreeBuilder::processStartTag): (WebCore::HTMLTreeBuilder::processEndTag): (WebCore::HTMLTreeBuilder::processCharacter): (WebCore::HTMLTreeBuilder::processEndOfFile):
10:14 PM Changeset in webkit [62520] by abarth@webkit.org
  • 9 edits in trunk

2010-07-05 Adam Barth <​abarth@webkit.org>

Reviewed by Eric Seidel.

Implement basic text node coalescing
​https://bugs.webkit.org/show_bug.cgi?id=41623

  • html5lib/resources/webkit01.dat:
    • This test had incorrect expected results according to the spec and to Minefield.
  • html5lib/runner-expected-html5.txt:
    • Update expected results to show test progression. In some cases, we continue to fail tests, but in new (also wrong) ways. I think that's ok though.
  • html5lib/runner-expected.txt:
    • Update expected results w.r.t. new webkit01.dat baseline.
  • html5lib/webkit-resumer-expected.txt:
    • Update expected results w.r.t. new webkit01.dat baseline.

2010-07-05 Adam Barth <​abarth@webkit.org>

Reviewed by Eric Seidel.

Implement basic text node coalescing
​https://bugs.webkit.org/show_bug.cgi?id=41623

This patch isn't the end-all, be-all of text node coalescing, but it's
a good start.

  • dom/CharacterData.cpp: (WebCore::CharacterData::parserAppendData): (WebCore::CharacterData::appendData):
  • dom/CharacterData.h:
    • Added a new method to dance around mutation events.
  • html/HTMLTreeBuilder.cpp: (WebCore::HTMLTreeBuilder::insertTextNode):
9:58 PM Changeset in webkit [62519] by abarth@webkit.org
  • 4 edits in trunk

2010-07-05 Adam Barth <​abarth@webkit.org>

Reviewed by Eric Seidel.

Implement InSelectMode
​https://bugs.webkit.org/show_bug.cgi?id=41627

Massive test progression.

  • html5lib/runner-expected-html5.txt:

2010-07-05 Adam Barth <​abarth@webkit.org>

Reviewed by Eric Seidel.

Implement InSelectMode
​https://bugs.webkit.org/show_bug.cgi?id=41627

  • html/HTMLTreeBuilder.cpp: (WebCore::HTMLTreeBuilder::processStartTag): (WebCore::HTMLTreeBuilder::processEndTag): (WebCore::HTMLTreeBuilder::processCharacter): (WebCore::HTMLTreeBuilder::processEndOfFile):
9:57 PM Changeset in webkit [62518] by abarth@webkit.org
  • 3 edits in trunk/WebKitTools

2010-07-05 Adam Barth <​abarth@webkit.org>

Reviewed by Eric Seidel.

We should be able to specify a bug to block for webkit-patch upload
​https://bugs.webkit.org/show_bug.cgi?id=41648

This will be useful for working on the HTML5 parser.

  • Scripts/webkitpy/tool/steps/createbug.py:
  • Scripts/webkitpy/tool/steps/options.py:
9:21 PM Changeset in webkit [62517] by yuzo@google.com
  • 2 edits in trunk/LayoutTests

2010-07-05 Yuzo Fujishima <​yuzo@google.com>

Unreviewed test expectation update.

Fix: On Leopard run-webkit-tests complains: Skipped list contained
'compositing/iframes/composited-iframe.html', but no file of that name
could be found

  • platform/mac-leopard/Skipped: Removed the above test.
8:34 PM Changeset in webkit [62516] by abarth@webkit.org
  • 5 edits in trunk/LayoutTests

2010-07-05 Adam Barth <​abarth@webkit.org>

Unreviewed.

We were discussing this test case, so I figured I might as well add it
to the suite.

  • html5lib/resources/webkit01.dat:
  • html5lib/runner-expected-html5.txt:
  • html5lib/runner-expected.txt:
  • html5lib/webkit-resumer-expected.txt:
8:11 PM Changeset in webkit [62515] by commit-queue@webkit.org
  • 2 edits in trunk/WebKit

2010-07-05 Gyuyoung Kim <​gyuyoung.kim@samsung.com>

Reviewed by Kenneth Rohde Christiansen.

[EFL] Implement downloadURL in ContextMenuClientEfl
When user selects a download menu from context menu, send the download
request to application.
​https://bugs.webkit.org/show_bug.cgi?id=41149

  • efl/WebCoreSupport/ContextMenuClientEfl.cpp: (WebCore::ContextMenuClientEfl::downloadURL): Get url for downloading file and Send it to application.
  • efl/WebCoreSupport/ContextMenuClientEfl.h:
7:03 PM Changeset in webkit [62514] by commit-queue@webkit.org
  • 3 edits
    2 adds in trunk

2010-07-05 Fady Samuel <​fsamuel@chromium.org>

Reviewed by Darin Adler.

Fixed a svg crash when setting class of an svg ellipse object.

Altering the CSS class of an attached SVG element causes WebKit to crash
​https://bugs.webkit.org/show_bug.cgi?id=40857

  • platform/chromium-linux/svg/css/svg-ellipse-render-crash-expected.txt: Added.
  • svg/css/svg-ellipse-render-crash.html: Added.

2010-07-05 Fady Samuel <​fsamuel@chromium.org>

Reviewed by Darin Adler.

Fixed a svg crash when setting class of an svg ellipse object.

Altering the CSS class of an attached SVG element causes WebKit to crash
​https://bugs.webkit.org/show_bug.cgi?id=40857

Test: svg/css/svg-ellipse-render-crash.html

  • dom/StyledElement.cpp: (WebCore::StyledElement::classAttributeChanged):
6:06 PM Changeset in webkit [62513] by mitz@apple.com
  • 4 edits
    2 adds in trunk

Reproducible crash with Optimize Legibility extension
​https://bugs.webkit.org/show_bug.cgi?id=41585

Reviewed by Sam Weinig.

WebCore:

Test: fast/css/text-rendering-priority.html

Moved the text-rendering CSS property to the “high priority” group, because applying it
during style selection invalidates the font.

  • css/CSSPropertyNames.in: Moved text-rendering to the “high priority” section at the top.
  • css/CSSStyleSelector.cpp:

(WebCore::CSSStyleSelector::applyDeclarations): Updated compile-time assertion.

LayoutTests:

  • fast/css/text-rendering-priority-expected.txt: Added.
  • fast/css/text-rendering-priority.html: Added.
5:53 PM Changeset in webkit [62512] by alice.liu@apple.com
  • 1 edit
    2 adds in trunk/LayoutTests

Crash reading past end of block in UniscribeController::shapeAndPlaceItem
​https://bugs.webkit.org/show_bug.cgi?id=41554

Reviewed by Dan Bernstein.

  • platform/win/fast/text/uniscribe-item-boundary-crash-expected.txt: Added.
  • platform/win/fast/text/uniscribe-item-boundary-crash.html: Added.
5:51 PM Applications using WebKit edited by onny_silvester@yahoo.com
(diff)
5:22 PM Changeset in webkit [62511] by barraclough@apple.com
  • 7 edits in trunk/JavaScriptCore

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

Reviewed by Sam Weinig.

Update compile flags to allow use of ExecutableAllocatorFixedVMPool on platforms
other than x86-64 (this may be useful on 32-bit platforms, too).

Simplify ifdefs by dividing into thwo broad allocation strategies
(ENABLE_EXECUTABLE_ALLOCATOR_FIXED & ENABLE_EXECUTABLE_ALLOCATOR_DEMAND).

Rename constant used in the code to have names descriptive of their purpose,
rather than their specific value on a given platform.

  • jit/ExecutableAllocator.cpp:

(JSC::ExecutableAllocator::reprotectRegion):
(JSC::ExecutableAllocator::cacheFlush):

  • jit/ExecutableAllocatorFixedVMPool.cpp:

(JSC::FixedVMPoolAllocator::FixedVMPoolAllocator):
(JSC::FixedVMPoolAllocator::free):
(JSC::ExecutablePool::systemAlloc):

  • jit/ExecutableAllocatorPosix.cpp:
  • jit/ExecutableAllocatorSymbian.cpp:
  • jit/ExecutableAllocatorWin.cpp:
  • wtf/Platform.h:
5:11 PM Changeset in webkit [62510] by mitz@apple.com
  • 2 edits in trunk/WebCore

optimizeLegibility doesn't play nice with fonts that do not have a space glyph
​https://bugs.webkit.org/show_bug.cgi?id=41599

Reviewed by Sam Weinig.

No test because none of the fonts available to DumpRenderTree are missing a space glyph.

  • rendering/RenderBlockLineLayout.cpp:

(WebCore::RenderBlock::findNextLineBreak): When the font does not map the space character
to a glyph, a fallback font is used for space. Therefore, wordTrailingSpaceWidth must be
initialized with the width of a space as measured by the Font rather than with the cached
space width.

2:39 PM Changeset in webkit [62509] by abarth@webkit.org
  • 5 edits in trunk

2010-07-05 Eric Seidel <​eric@webkit.org>

Reviewed by Adam Barth.

Implement HTML5's "reset insertion mode appropriately"
​https://bugs.webkit.org/show_bug.cgi?id=41628

This has some minimal testing. One from my previous
</table> patch, and a few from the main suite.

Mostly resetInsertionModeAppropriately isn't used yet
but we're about to add a bunch of states which do use it
and our test coverage will expand further as we do.

  • html/HTMLTreeBuilder.cpp: (WebCore::HTMLTreeBuilder::HTMLTreeBuilder): (WebCore::HTMLTreeBuilder::setInsertionModeAndEnd): (WebCore::HTMLTreeBuilder::resetInsertionModeAppropriately):
  • html/HTMLTreeBuilder.h:

2010-07-05 Eric Seidel <​eric@webkit.org>

Reviewed by Adam Barth.

Implement HTML5's "reset insertion mode appropriately"
​https://bugs.webkit.org/show_bug.cgi?id=41628

Fixes a bunch of subtests!

  • html5lib/runner-expected-html5.txt:
2:34 PM Changeset in webkit [62508] by Joseph Pecoraro
  • 2 edits in trunk/WebCore

2010-07-05 Joseph Pecoraro <Joseph Pecoraro>

Unreviewed build fix for r62503.

Chromium missing method. Filed bug 41632 to track.

  • loader/appcache/ApplicationCacheHost.h: (WebCore::ApplicationCacheHost::applicationCache):
2:25 PM Changeset in webkit [62507] by abarth@webkit.org
  • 7 edits in trunk

2010-07-04 Eric Seidel <​eric@webkit.org>

Reviewed by Adam Barth.

Add </table> support for "in table" insertion mode
​https://bugs.webkit.org/show_bug.cgi?id=41591

resetInsertionModeAppropriately isn't implemented yet, however
I've added a test for the usage I added.

  • html/HTMLTreeBuilder.cpp: (WebCore::HTMLTreeBuilder::resetInsertionModeAppropriately): (WebCore::HTMLTreeBuilder::processEndTag):
  • html/HTMLTreeBuilder.h:

2010-07-05 Eric Seidel <​eric@webkit.org>

Reviewed by Adam Barth.

Add </table> support for "in table" insertion mode
​https://bugs.webkit.org/show_bug.cgi?id=41591

Add tests for </table> in "in table" mode, and
one for ignored close tags.

  • html5lib/resources/tables01.dat:
  • html5lib/runner-expected-html5.txt:
  • html5lib/runner-expected.txt:
2:08 PM Changeset in webkit [62506] by Joseph Pecoraro
  • 2 edits in trunk/WebCore

2010-07-05 Joseph Pecoraro <Joseph Pecoraro>

Unreviewed build fix for r62503.

Missed GTK's build file.

  • GNUmakefile.am:
2:00 PM Changeset in webkit [62505] by Joseph Pecoraro
  • 2 edits in trunk/WebCore

2010-07-05 Joseph Pecoraro <Joseph Pecoraro>

Unreviewed build fix for r62503.

Pass a blank ResourceResponse instead of passing 0.

  • loader/appcache/ApplicationCacheGroup.cpp: (WebCore::ApplicationCacheGroup::createResourceHandle):
1:47 PM Changeset in webkit [62504] by Joseph Pecoraro
  • 2 edits in trunk/WebCore

2010-07-05 Joseph Pecoraro <Joseph Pecoraro>

Unreviewed build fix for r62503.

Forward declare ApplicationCache for Chromium.

  • loader/appcache/ApplicationCacheHost.h:
1:32 PM Changeset in webkit [62503] by Joseph Pecoraro
  • 16 edits in trunk/WebCore

2010-07-05 Joseph Pecoraro <Joseph Pecoraro>

Reviewed by Pavel Feldman.

WebInspector: HTML5 Offline Web Applications Support (ApplicationCache)
​https://bugs.webkit.org/show_bug.cgi?id=24529

Parts of this patch were written by Kavita Kanetkar <​kkanetkar@chromium.org>.

Part 2: Pulling ApplicationCache Resources to Display in the Inspector.

The InspectorApplicationCacheAgent gathers the information it
needs and forwards it on to the inspector.

  • inspector/InspectorApplicationCacheAgent.cpp: (WebCore::InspectorApplicationCacheAgent::fillResourceList): get information about the resources. (WebCore::InspectorApplicationCacheAgent::getApplicationCaches): gathers all the information from the ApplicationCacheHost. (WebCore::InspectorApplicationCacheAgent::buildObjectForApplicationCache): (WebCore::InspectorApplicationCacheAgent::buildArrayForApplicationCacheResources): (WebCore::InspectorApplicationCacheAgent::buildObjectForApplicationCacheResource):
  • inspector/InspectorApplicationCacheAgent.h: defines structures to hold information about caches and resources. (WebCore::InspectorApplicationCacheAgent::ApplicationCacheInfo::ApplicationCacheInfo): (WebCore::InspectorApplicationCacheAgent::ResourceInfo::ResourceInfo): (WebCore::InspectorApplicationCacheAgent::~InspectorApplicationCacheAgent):
  • loader/appcache/ApplicationCacheHost.h: (WebCore::ApplicationCacheHost::applicationCacheForInspector): new convention, public cache accessor for the inspector. (WebCore::ApplicationCacheHost::documentLoader): added const.

The User Interface uses a DataGrid, like Cookies.

  • inspector/front-end/ApplicationCacheItemsView.js: (WebInspector.ApplicationCacheItemsView.prototype._update): (WebInspector.ApplicationCacheItemsView.prototype._updateCallback): (WebInspector.ApplicationCacheItemsView.prototype._createDataGrid): (WebInspector.ApplicationCacheItemsView.prototype._populateDataGrid.numberCompare): (WebInspector.ApplicationCacheItemsView.prototype._populateDataGrid.localeCompare): (WebInspector.ApplicationCacheItemsView.prototype._populateDataGrid):

The usual frontend pull workflow, except this goes through
InspectorApplicationCacheAgent instead of InspectorController.

  • inspector/InspectorBackend.cpp: (WebCore::InspectorBackend::getApplicationCaches): (WebCore::InspectorBackend::inspectorApplicationCacheAgent):
  • inspector/InspectorBackend.h:
  • inspector/InspectorBackend.idl:
  • inspector/InspectorFrontend.cpp: (WebCore::InspectorFrontend::didGetApplicationCaches):
  • inspector/InspectorFrontend.h:
  • inspector/front-end/DOMAgent.js: (WebInspector.DOMNode.prototype.hasChildNodes): style fix. (WebInspector.DOMAgent.prototype.nodeForId): style fix. (WebInspector.ApplicationCache.getApplicationCachesAsync): pull. (WebInspector.Cookies.getCookiesAsync):
  • inspector/front-end/StoragePanel.js: (WebInspector.StoragePanel.prototype.updateManifest): (WebInspector.ApplicationCacheSidebarTreeElement.prototype.onselect):
  • inspector/front-end/inspector.js: (WebInspector.updateResource): (WebInspector._addAppCacheDomain): (WebInspector.reset):

Miscellaneous. Localized Strings and fixes.

  • English.lproj/localizedStrings.js: "Type", "%s (%s)"
  • inspector/InspectorController.cpp: (WebCore::InspectorController::deleteCookie):
  • inspector/InspectorController.h:
1:32 PM Changeset in webkit [62502] by Joseph Pecoraro
  • 19 edits
    4 adds in trunk/WebCore

2010-07-04 Joseph Pecoraro <Joseph Pecoraro>

Reviewed by Pavel Feldman.

WebInspector: HTML5 Offline Web Applications Support (ApplicationCache)
​https://bugs.webkit.org/show_bug.cgi?id=24529

Part 1: Backend -> Frontend Messages. ApplicationCache Status and Connectivity Status.

This patch adds an InspectorApplicationCacheAgent to monitor application
cache changes, starts a UI in the Storage panel, handles the boilerplate
of adding new files.

Added an agent to encapsulate and handle the application cache logic.
This is similar to the timeline agent.

  • inspector/InspectorApplicationCacheAgent.cpp: Added. (WebCore::InspectorApplicationCacheAgent::InspectorApplicationCacheAgent): (WebCore::InspectorApplicationCacheAgent::~InspectorApplicationCacheAgent): (WebCore::InspectorApplicationCacheAgent::didReceiveManifestResponse): (WebCore::InspectorApplicationCacheAgent::updateApplicationCacheStatus): (WebCore::InspectorApplicationCacheAgent::updateNetworkState):
  • inspector/InspectorApplicationCacheAgent.h: Added.

InspectorController owns an InspectorApplicationCacheAgent. This
handles its lifetime management.

  • inspector/InspectorController.cpp: (WebCore::InspectorController::~InspectorController): (WebCore::InspectorController::connectFrontend): create the agent with a frontend. (WebCore::InspectorController::disconnectFrontend): remove the agent when closing. (WebCore::InspectorController::releaseFrontendLifetimeAgents):
  • inspector/InspectorController.h:

User Interface for ApplicationCache in the StoragePanel. This follows
very closely to Cookies, it will have a sortable table of items. The
status bar contains connectivity and application cache status
indicators which update when backend messages are received. There
are some stubs which the next part will implement.

  • inspector/front-end/ApplicationCacheItemsView.js: Added. (WebInspector.ApplicationCacheItemsView): (WebInspector.ApplicationCacheItemsView.prototype.get statusBarItems): refresh, delete, connectivity, application cache status. (WebInspector.ApplicationCacheItemsView.prototype.show): (WebInspector.ApplicationCacheItemsView.prototype.hide): (WebInspector.ApplicationCacheItemsView.prototype.updateStatus): this is the application cache status indicator. (WebInspector.ApplicationCacheItemsView.prototype.updateNetworkState): this is the online/offline connectivity indicator. (WebInspector.ApplicationCacheItemsView.prototype._update): (WebInspector.ApplicationCacheItemsView.prototype._updateCallback): (WebInspector.ApplicationCacheItemsView.prototype._createDataGrid): (WebInspector.ApplicationCacheItemsView.prototype._populateDataGrid): (WebInspector.ApplicationCacheItemsView.prototype.resize): (WebInspector.ApplicationCacheItemsView.prototype._deleteButtonClicked): (WebInspector.ApplicationCacheItemsView.prototype._deleteCallback): (WebInspector.ApplicationCacheItemsView.prototype._refreshButtonClicked):

The usual dispatch flow from the backend, to the frontend, to the
panel, and then to the visible view. Some slight refactoring to
eliminate duplicated code.

  • inspector/InspectorFrontend.cpp: (WebCore::InspectorFrontend::updateDOMStorage): (WebCore::InspectorFrontend::didGetApplicationCaches): (WebCore::InspectorFrontend::updateApplicationCacheStatus):
  • inspector/InspectorFrontend.h:
  • inspector/front-end/StoragePanel.js: (WebInspector.StoragePanel): (WebInspector.StoragePanel.prototype.reset): (WebInspector.StoragePanel.prototype.addApplicationCache): (WebInspector.StoragePanel.prototype.showDatabase): (WebInspector.StoragePanel.prototype.showDOMStorage): (WebInspector.StoragePanel.prototype.showCookies): (WebInspector.StoragePanel.prototype.showApplicationCache): (WebInspector.StoragePanel.prototype._genericViewSetup): (WebInspector.StoragePanel.prototype.updateApplicationCacheStatus): (WebInspector.StoragePanel.prototype.updateNetworkState): (WebInspector.CookieSidebarTreeElement.prototype.set subtitle): (WebInspector.ApplicationCacheSidebarTreeElement): (WebInspector.ApplicationCacheSidebarTreeElement.prototype.onselect): (WebInspector.ApplicationCacheSidebarTreeElement.prototype.get mainTitle): (WebInspector.ApplicationCacheSidebarTreeElement.prototype.set mainTitle): (WebInspector.ApplicationCacheSidebarTreeElement.prototype.get subtitle): (WebInspector.ApplicationCacheSidebarTreeElement.prototype.set subtitle):
  • inspector/front-end/inspector.js: (WebInspector.dispatch.delayDispatch): (WebInspector.dispatch): (WebInspector._addAppCacheDomain): (WebInspector.addDOMStorage): (WebInspector.updateDOMStorage):

Notify the InspectorApplicationCacheAgent on application cache changes
or specifics. Notify the InspectorController on generic resource events.
That is because ApplicationCacheController is a ResourceClient and needs
to trigger the resource events normally handled by ResourceLoader.

  • loader/appcache/ApplicationCacheGroup.cpp: (WebCore::inspectorUpdateApplicationCacheStatus): helper method to prevent duplicated code. (WebCore::ApplicationCacheGroup::setNewestCache): status change. (WebCore::ApplicationCacheGroup::makeObsolete): status change. (WebCore::ApplicationCacheGroup::update): status change. (WebCore::ApplicationCacheGroup::createResourceHandle): resource event. (WebCore::ApplicationCacheGroup::willSendRequest): resource event. (WebCore::ApplicationCacheGroup::didReceiveResponse): resource event. (WebCore::ApplicationCacheGroup::didReceiveData): resource event. (WebCore::ApplicationCacheGroup::didFinishLoading): resource event. (WebCore::ApplicationCacheGroup::didFail): resource event. (WebCore::ApplicationCacheGroup::didFinishLoadingManifest): resource event. (WebCore::ApplicationCacheGroup::manifestNotFound): status change. (WebCore::ApplicationCacheGroup::checkIfLoadIsComplete): status change. (WebCore::ApplicationCacheGroup::setUpdateStatus): single access point for status changes.
  • loader/appcache/ApplicationCacheGroup.h:
  • page/Page.cpp: (WebCore::networkStateChanged): connectivity status change.

Final inspector details to add the new file, style new elements,
images, and localized strings.

  • inspector/front-end/inspector.css: (.application-cache-sidebar-tree-item .icon): (.timeline-records-counter, .storage-application-cache-status, .storage-application-cache-connectivity): (.storage-application-cache-status-icon, .storage-application-cache-connectivity-icon): (.status-bar-divider): (.storage-application-cache-status, .storage-application-cache-connectivity):
  • inspector/front-end/inspector.html:
  • inspector/front-end/Images/applicationCache.png: Added.
  • English.lproj/localizedStrings.js: "APPLICATION CACHE", "No Application Cache information available.", "Online", "Offline"

Updated build files.

  • CMakeLists.txt:
  • WebCore.gypi:
  • WebCore.pro:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • inspector/front-end/WebKit.qrc:
1:17 PM Changeset in webkit [62501] by commit-queue@webkit.org
  • 4 edits in trunk/WebKit2

2010-07-05 Luiz Agostini <​luiz.agostini@openbossa.org>

Reviewed by Kenneth Rohde Christiansen.

[Qt] WebKit2 triple click
​https://bugs.webkit.org/show_bug.cgi?id=41629

QtWebkit2 triple click implementation.

  • UIProcess/API/qt/qwkpage.cpp: (QWKPagePrivate::mousePressEvent): (QWKPagePrivate::mouseDoubleClickEvent): (QWKPage::timerEvent):
  • UIProcess/API/qt/qwkpage.h:
  • UIProcess/API/qt/qwkpage_p.h:
1:03 PM Changeset in webkit [62500] by abarth@webkit.org
  • 6 edits in trunk

2010-07-05 Eric Seidel <​eric@webkit.org>

Reviewed by Adam Barth.

Add basic <col> support to the treebuilder
​https://bugs.webkit.org/show_bug.cgi?id=41590

  • html/HTMLTreeBuilder.cpp: (WebCore::HTMLTreeBuilder::processStartTag):

2010-07-05 Eric Seidel <​eric@webkit.org>

Reviewed by Adam Barth.

Add basic <col> support to the treebuilder
​https://bugs.webkit.org/show_bug.cgi?id=41590

Test the new <col> support. Turns out the old treebuilder doesn't
handle <table><col> correctly.

  • html5lib/resources/tables01.dat:
  • html5lib/runner-expected-html5.txt:
  • html5lib/runner-expected.txt:
1:02 PM Changeset in webkit [62499] by abarth@webkit.org
  • 8 edits
    1 add in trunk

2010-07-05 Eric Seidel <​eric@webkit.org>

Reviewed by Adam Barth.

Add basic "in row" mode to support <td> and <th> insertion
​https://bugs.webkit.org/show_bug.cgi?id=41588

Also fixed <td> or <th> as a direct child of <table>.

Most of this was covered by html5lib/runner.html tests,
but I had to add a new tables01.dat to cover the <th> cases.

  • html/HTMLElementStack.cpp: (WebCore::HTMLNames::isTableRowScopeMarker): (WebCore::HTMLElementStack::popUntilTableRowScopeMarker):
  • html/HTMLElementStack.h:
  • html/HTMLTreeBuilder.cpp: (WebCore::HTMLTreeBuilder::processStartTag):

2010-07-05 Eric Seidel <​eric@webkit.org>

Reviewed by Adam Barth.

Add basic "in row" mode to support <td> insertion
​https://bugs.webkit.org/show_bug.cgi?id=41588

Added tables01.dat to test a previously untested
<table><th> case. I expect we'll add many more table
tests there as we complete our implementation.

  • html5lib/runner-expected-html5.txt:
  • html5lib/runner-expected.txt:
  • html5lib/runner.html:
  • html5lib/resources/tables01.dat: Added.
12:42 PM Changeset in webkit [62498] by eric@webkit.org
  • 7 edits
    1 add in trunk

2010-07-05 Eric Seidel <​eric@webkit.org>

Reviewed by Adam Barth.

Finish implementing "any other end tag" for "in body" mode
​https://bugs.webkit.org/show_bug.cgi?id=41582

  • html5lib/resources/inbody01.dat: Added.
  • html5lib/runner-expected-html5.txt:
    • Update the one result which was affected by this. That test would pass if we had text node coalescing.
  • html5lib/runner-expected.txt:
    • Update to add the extra test suite.
  • html5lib/runner.html:

2010-07-05 Eric Seidel <​eric@webkit.org>

Reviewed by Adam Barth.

Finish implementing "any other end tag" for "in body" mode
​https://bugs.webkit.org/show_bug.cgi?id=41582

I believe I found a "bug" in the HTML5 spec when writing this:
​http://www.w3.org/Bugs/Public/show_bug.cgi?id=10080

  • html/HTMLTreeBuilder.cpp: (WebCore::HTMLTreeBuilder::processAnyOtherEndTagForInBody): (WebCore::HTMLTreeBuilder::processEndTag):
  • html/HTMLTreeBuilder.h:
11:32 AM Changeset in webkit [62497] by Martin Robinson
  • 2 edits in trunk/WebCore

2010-07-05 Martin Robinson <​mrobinson@igalia.com>

Unreviewed.

Add a missing slash to the list of generated sources. This
was missing from a previous build fix.

  • GNUmakefile.am:
11:04 AM Changeset in webkit [62496] by pfeldman@chromium.org
  • 4 edits in trunk/WebCore

2010-07-05 Pavel Feldman <​pfeldman@chromium.org>

Reviewed by Joseph Pecoraro.

Web Inspector: preserve scroll positions in source frame when switching between panes.

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

  • inspector/front-end/ScriptsPanel.js: (WebInspector.ScriptsPanel.prototype.hide):
  • inspector/front-end/SourceFrame.js: (WebInspector.SourceFrame.prototype.set visible):
  • inspector/front-end/SourceView.js: (WebInspector.SourceView.prototype.hide):
10:57 AM Changeset in webkit [62495] by yurys@chromium.org
  • 1 edit
    17 adds in trunk/LayoutTests

2010-07-05 Yury Semikhatsky <​yury@yurys-imac.local>

Unreviewed. Provide Chromium specific expectations for new svg tests.

  • platform/chromium-linux/svg/custom/clone-element-with-animated-svg-properties-expected.checksum: Added.
  • platform/chromium-linux/svg/custom/relative-sized-inner-svg-expected.checksum: Added.
  • platform/chromium-linux/svg/custom/relative-sized-use-without-attributes-on-symbol-expected.checksum: Added.
  • platform/chromium-linux/svg/filters/filter-width-update-expected.checksum: Added.
  • platform/chromium-mac/svg/filters/filter-width-update-expected.checksum: Added.
  • platform/chromium-win/svg/custom/clone-element-with-animated-svg-properties-expected.checksum: Added.
  • platform/chromium-win/svg/custom/clone-element-with-animated-svg-properties-expected.png: Added.
  • platform/chromium-win/svg/custom/clone-element-with-animated-svg-properties-expected.txt: Added.
  • platform/chromium-win/svg/custom/relative-sized-inner-svg-expected.checksum: Added.
  • platform/chromium-win/svg/custom/relative-sized-inner-svg-expected.png: Added.
  • platform/chromium-win/svg/custom/relative-sized-inner-svg-expected.txt: Added.
  • platform/chromium-win/svg/custom/relative-sized-use-without-attributes-on-symbol-expected.checksum: Added.
  • platform/chromium-win/svg/custom/relative-sized-use-without-attributes-on-symbol-expected.png: Added.
  • platform/chromium-win/svg/custom/relative-sized-use-without-attributes-on-symbol-expected.txt: Added.
  • platform/chromium-win/svg/filters/filter-width-update-expected.checksum: Added.
  • platform/chromium-win/svg/filters/filter-width-update-expected.png: Added.
  • platform/chromium-win/svg/filters/filter-width-update-expected.txt: Added.
10:50 AM Changeset in webkit [62494] by commit-queue@webkit.org
  • 2 edits
    2 copies
    2 adds in trunk/WebKit2

2010-07-05 Kenneth Rohde Christiansen <​kenneth.christiansen@openbossa.org>

Reviewed by Antti Koivisto.

[Qt] Initial WebKit2 implementation
​https://bugs.webkit.org/show_bug.cgi?id=40233

Add stubs

  • UIProcess/Plugins/qt/PluginInfoStoreQt.cpp: Copied from WebKit2/UIProcess/Plugins/win/PluginInfoStoreWin.cpp. (WebKit::PluginInfoStore::pluginDirectories): (WebKit::PluginInfoStore::pluginPathsInDirectory): (WebKit::PluginInfoStore::getPluginInfo): (WebKit::PluginInfoStore::shouldUsePlugin):
  • WebProcess/InjectedBundle/InjectedBundle.h:
  • WebProcess/InjectedBundle/qt/InjectedBundleQt.cpp: Added. (WebKit::InjectedBundle::load):
10:10 AM Changeset in webkit [62493] by steveblock@google.com
  • 2 edits in trunk/JavaScriptCore

2010-07-05 Steve Block <​steveblock@google.com>

Reviewed by Darin Adler.

ThreadingPthreads.cpp should use JNIUtility.h on Android, not outdated jni_utility.h
​https://bugs.webkit.org/show_bug.cgi?id=41594

  • wtf/ThreadingPthreads.cpp:
10:01 AM Changeset in webkit [62492] by rwlbuis@webkit.org
  • 7 edits
    3 adds in trunk

2010-07-05 Rob Buis <​rwlbuis@gmail.com>

Reviewed by Dirk Schulze.

IE SVG test fails
​https://bugs.webkit.org/show_bug.cgi?id=41619

Make SVGSVGElement.createSVGTransform create a SVGTransform with the
right type, thereby fixing the IE test.

Test: svg/custom/svg-createsvgtransform-type.html

  • svg/SVGSVGElement.cpp: (WebCore::SVGSVGElement::createSVGTransform):
9:54 AM Changeset in webkit [62491] by commit-queue@webkit.org
  • 1 edit
    2 adds in trunk/WebKit2

2010-07-05 Kenneth Rohde Christiansen <​kenneth.christiansen@openbossa.org>

Reviewed by Antti Koivisto.

[Qt] Initial WebKit2 implementation
​https://bugs.webkit.org/show_bug.cgi?id=40233

Add WebErrorsQt

  • WebProcess/WebCoreSupport/qt/WebErrorsQt.cpp: Added. (WebKit::): (WebKit::cancelledError): (WebKit::blockedError): (WebKit::cannotShowURLError): (WebKit::interruptForPolicyChangeError): (WebKit::cannotShowMIMETypeError): (WebKit::fileDoesNotExistError):
8:29 AM Changeset in webkit [62490] by Martin Robinson
  • 3 edits in trunk/WebKitTools

2010-07-05 Martin Robinson <​mrobinson@igalia.com>

Reviewed by Oliver Hunt.

The style checker exempts gtk2drawing.h when it should exempt gtkdrawing.h
​https://bugs.webkit.org/show_bug.cgi?id=41017

  • Scripts/webkitpy/style/checker.py: Exempt gtkdrawing.h instead of gtk2drawing.h
  • Scripts/webkitpy/style/checker_unittest.py: Update the unit test to reflect the change.
8:12 AM Changeset in webkit [62489] by pfeldman@chromium.org
  • 2 edits in trunk/WebCore

2010-07-05 Pavel Feldman <​pfeldman@chromium.org>

Reviewed by Yury Semikhatsky.

Web Inspector: computed style pane is not updated when styles pane is collapsed.

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

  • inspector/front-end/ElementsPanel.js: (WebInspector.ElementsPanel.prototype.updateStyles):
7:32 AM Changeset in webkit [62488] by Nikolas Zimmermann
  • 44 edits
    12 adds in trunk

2010-07-05 Nikolas Zimmermann <​nzimmermann@rim.com>

Reviewed by Dirk Schulze.

Logic to track whether elements are using relative lengths is incomplete
​https://bugs.webkit.org/show_bug.cgi?id=41566

Add logic to all SVG elements which create renderes to expose a method
"bool selfHasRelativeLengths()", that returns whether the element uses relative
lengths (eg. <rect x="50%"...). This will be used soon to avoid relayouts of
container children, when the bounds have not changed.

A new method SVGStyledElement::updateRelativeLengthsInformation() is added,
which is called from the various svgAttributeChanged() methods and insertedIntoDocument/removedFromDocument.
It will be implemented in a follow-up patch. This patch does not affect any test behaviour related
to relayouting. As SVGFilterElement finally got a proper svgAttributeChanged() method, it now
properly invalidates clients on attribute changes - covered by a new test.

Tests: svg/custom/relative-sized-inner-svg.xhtml

svg/custom/relative-sized-use-without-attributes-on-symbol.xhtml
svg/filters/filter-width-update.svg


  • rendering/RenderPath.cpp: (WebCore::RenderPath::layout): Rename hasRelativeValues to hasRelativeLengths.
  • rendering/RenderSVGRoot.cpp: (WebCore::RenderSVGRoot::layout): Ditto. (WebCore::RenderSVGRoot::calcViewport): Ditto.
  • rendering/RenderSVGViewportContainer.cpp: (WebCore::RenderSVGViewportContainer::calcViewport): Ditto
  • rendering/SVGRenderSupport.cpp: (WebCore::SVGRenderSupport::layoutChildren): Ditto.
  • svg/SVGCircleElement.cpp: (WebCore::SVGCircleElement::svgAttributeChanged): Call updateRelativeLengthsInformation() if any attribute which may contain relative lengths changes. (WebCore::SVGCircleElement::selfHasRelativeLengths): Ditto. Same for all other SVG*Elements below.
  • svg/SVGCircleElement.h:
  • svg/SVGEllipseElement.cpp: (WebCore::SVGEllipseElement::svgAttributeChanged): (WebCore::SVGEllipseElement::selfHasRelativeLengths):
  • svg/SVGEllipseElement.h:
  • svg/SVGFilterElement.cpp: (WebCore::SVGFilterElement::svgAttributeChanged): (WebCore::SVGFilterElement::selfHasRelativeLengths):
  • svg/SVGFilterElement.h:
  • svg/SVGForeignObjectElement.cpp: (WebCore::SVGForeignObjectElement::svgAttributeChanged): (WebCore::SVGForeignObjectElement::selfHasRelativeLengths):
  • svg/SVGForeignObjectElement.h:
  • svg/SVGImageElement.cpp: (WebCore::SVGImageElement::svgAttributeChanged): (WebCore::SVGImageElement::selfHasRelativeLengths):
  • svg/SVGImageElement.h:
  • svg/SVGLineElement.cpp: (WebCore::SVGLineElement::svgAttributeChanged): (WebCore::SVGLineElement::selfHasRelativeLengths):
  • svg/SVGLineElement.h:
  • svg/SVGLinearGradientElement.cpp: (WebCore::SVGLinearGradientElement::svgAttributeChanged): (WebCore::SVGLinearGradientElement::selfHasRelativeLengths):
  • svg/SVGLinearGradientElement.h:
  • svg/SVGMarkerElement.cpp: (WebCore::SVGMarkerElement::svgAttributeChanged): (WebCore::SVGMarkerElement::selfHasRelativeLengths):
  • svg/SVGMarkerElement.h:
  • svg/SVGMaskElement.cpp: (WebCore::SVGMaskElement::svgAttributeChanged): (WebCore::SVGMaskElement::selfHasRelativeLengths):
  • svg/SVGMaskElement.h:
  • svg/SVGPatternElement.cpp: (WebCore::SVGPatternElement::svgAttributeChanged): (WebCore::SVGPatternElement::selfHasRelativeLengths):
  • svg/SVGPatternElement.h:
  • svg/SVGRadialGradientElement.cpp: (WebCore::SVGRadialGradientElement::svgAttributeChanged): (WebCore::SVGRadialGradientElement::selfHasRelativeLengths):
  • svg/SVGRadialGradientElement.h:
  • svg/SVGRectElement.cpp: (WebCore::SVGRectElement::svgAttributeChanged): (WebCore::SVGRectElement::selfHasRelativeLengths):
  • svg/SVGRectElement.h:
  • svg/SVGSVGElement.cpp: (WebCore::SVGSVGElement::svgAttributeChanged): (WebCore::SVGSVGElement::selfHasRelativeLengths):
  • svg/SVGSVGElement.h:
  • svg/SVGStyledElement.cpp: (WebCore::SVGStyledElement::insertedIntoDocument): Call updateRelativeLengthsInformation(). (WebCore::SVGStyledElement::removedFromDocument): Ditto. (WebCore::SVGStyledElement::updateRelativeLengthsInformation): Not implemented so far. Will land in a follow-up patch, together with the render tree changes.
  • svg/SVGStyledElement.h: (WebCore::SVGStyledElement::hasRelativeLengths): Devirtualized. For now, just call selfHasRelativeLengths() - this will change in a follow-up patch. (WebCore::SVGStyledElement::updateRelativeLengthsInformation): (WebCore::SVGStyledElement::selfHasRelativeLengths): Renamed from hasRelativeValues().
  • svg/SVGSymbolElement.cpp: (WebCore::SVGSymbolElement::svgAttributeChanged): (WebCore::SVGSymbolElement::selfHasRelativeLengths):
  • svg/SVGSymbolElement.h:
  • svg/SVGTextContentElement.cpp: (WebCore::SVGTextContentElement::selfHasRelativeLengths):
  • svg/SVGTextContentElement.h:
  • svg/SVGTextPathElement.cpp: (WebCore::SVGTextPathElement::svgAttributeChanged): (WebCore::SVGTextPathElement::insertedIntoDocument): Call right base class' method. Skipped one in the hierachy before. (WebCore::SVGTextPathElement::selfHasRelativeLengths):
  • svg/SVGTextPathElement.h:
  • svg/SVGTextPositioningElement.cpp: (WebCore::SVGTextPositioningElement::svgAttributeChanged): (WebCore::listContainsRelativeValue): New helper funtion that checks wheter a SVGLengthList contains relative lengths. (WebCore::SVGTextPositioningElement::selfHasRelativeLengths):
  • svg/SVGTextPositioningElement.h:
  • svg/SVGUseElement.cpp: (WebCore::SVGUseElement::insertedIntoDocument): Call right base class' method. Skipped on in the hierachy before. (WebCore::SVGUseElement::removedFromDocument): Ditto. (WebCore::SVGUseElement::svgAttributeChanged): (WebCore::SVGUseElement::buildShadowAndInstanceTree): Call updateRelativeLengthsInformation() after building the shadow tree. (WebCore::SVGUseElement::selfHasRelativeLengths): Same as all other methods, except that it includes the shadow tree root element.
  • svg/SVGUseElement.h:

2010-07-05 Nikolas Zimmermann <​nzimmermann@rim.com>

Reviewed by Dirk Schulze.

Logic to track whether elements are using relative lengths is incomplete
​https://bugs.webkit.org/show_bug.cgi?id=41566

Add two new relative-sized-*.svg tests that would break, when relative sized kids aren't layout correctly.
Add new test covering that filters update their clients, if any attribute changes.

  • platform/mac/svg/custom/relative-sized-inner-svg-expected.checksum: Added.
  • platform/mac/svg/custom/relative-sized-inner-svg-expected.png: Added.
  • platform/mac/svg/custom/relative-sized-inner-svg-expected.txt: Added.
  • platform/mac/svg/custom/relative-sized-use-without-attributes-on-symbol-expected.checksum: Added.
  • platform/mac/svg/custom/relative-sized-use-without-attributes-on-symbol-expected.png: Added.
  • platform/mac/svg/custom/relative-sized-use-without-attributes-on-symbol-expected.txt: Added.
  • platform/mac/svg/filters/filter-width-update-expected.checksum: Added.
  • platform/mac/svg/filters/filter-width-update-expected.png: Added.
  • platform/mac/svg/filters/filter-width-update-expected.txt: Added.
  • svg/custom/relative-sized-inner-svg.xhtml: Added.
  • svg/custom/relative-sized-use-without-attributes-on-symbol.xhtml: Added.
  • svg/filters/filter-width-update.svg: Added.
7:23 AM Changeset in webkit [62487] by pfeldman@chromium.org
  • 2 edits in trunk/WebCore

2010-07-05 Pavel Feldman <​pfeldman@chromium.org>

Reviewed by Yury Semikhatsky.

Web Inspector: Problem with copying a code from Scripts panel.

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

  • inspector/front-end/TextViewer.js: (WebInspector.TextViewer.prototype._getSelection): (WebInspector.TextViewer.prototype._selectionToPosition):
7:07 AM Changeset in webkit [62486] by commit-queue@webkit.org
  • 1 edit
    5 adds in trunk/WebKit2

2010-07-05 Kenneth Rohde Christiansen <​kenneth.christiansen@openbossa.org>

Reviewed by Antti Koivisto.

[Qt] Initial WebKit2 implementation
​https://bugs.webkit.org/show_bug.cgi?id=40233

Add Qt C API integration.

  • UIProcess/API/cpp/qt/WKStringQt.cpp: Added. (WKStringCreateWithQString): (WKStringCopyQString):
  • UIProcess/API/cpp/qt/WKStringQt.h: Added.
  • UIProcess/API/cpp/qt/WKURLQt.cpp: Added. (WKURLCreateWithQUrl): (WKURLCopyQUrl):
  • UIProcess/API/cpp/qt/WKURLQt.h: Added.
6:41 AM Changeset in webkit [62485] by commit-queue@webkit.org
  • 1 edit
    2 adds in trunk/WebKit2

2010-07-05 Kenneth Rohde Christiansen <​kenneth.christiansen@openbossa.org>

Reviewed by Antti Koivisto.

[Qt] Initial WebKit2 implementation
​https://bugs.webkit.org/show_bug.cgi?id=40233

Add the Qt process launcher.

  • UIProcess/Launcher/qt/ProcessLauncherQt.cpp: Added. (WebKit::ProcessLauncher::launchProcess): (WebKit::ProcessLauncher::terminateProcess): (WebKit::webThreadBody): (WebKit::ProcessLauncher::createWebThread): (webProcessMain):
5:48 AM Changeset in webkit [62484] by Nikolas Zimmermann
  • 3 edits
    4 adds in trunk

2010-07-05 Nikolas Zimmermann <​nzimmermann@rim.com>

Reviewed by Dirk Schulze / Darin Adler.

Node.cloneNode does not work on SVG nodes
​https://bugs.webkit.org/show_bug.cgi?id=41421

Be sure to synchronize animated SVG properties before checking whether NamedNodeMap exists.
When creating a SVG element from JS, and setting all attributes via SVG DOM, and not using setAttribute
the NamedNodeMap does not exist. When cloning such an element, be sure to synchronize SVG <-> XML DOM
attributes before attempting to clone, otherwhise the SVG animated properties are lost while cloning.

Test: svg/custom/clone-element-with-animated-svg-properties.html

  • dom/Element.cpp: (WebCore::Element::cloneElementWithoutChildren):
5:31 AM Changeset in webkit [62483] by antti.j.koivisto@nokia.com
  • 5 edits in trunk

Revert unplanned project file change.

.:

  • WebKit.pro:

WebCore:

  • WebCore.pri:
  • WebCore.pro:
5:27 AM Changeset in webkit [62482] by Nikolas Zimmermann
  • 3 edits
    4 adds in trunk

2010-07-05 Nikolas Zimmermann <​nzimmermann@rim.com>

Reviewed by Darin Adler.

Memory corruption with SVG <use> element
​https://bugs.webkit.org/show_bug.cgi?id=40994

Fix race condition in svgAttributeChanged. Never call svgAttributeChanged() from attributeChanged()
when we're synchronizing SVG attributes. It leads to either unnecessary extra work being done or
crashes. Especially together with <polyline>/<polygon> which always synchronize the SVGAnimatedPoints
datastructure with the points attribute, no matter if there are changes are not. This should be
furhter optimized, but this fix is sane and fixes the root of the evil races.

Test: svg/custom/use-property-synchronization-crash.svg

  • svg/SVGElement.cpp: (WebCore::SVGElement::attributeChanged):
5:26 AM Changeset in webkit [62481] by antti.j.koivisto@nokia.com
  • 5 edits
    4 adds in trunk

[Qt] Initial WebKit2 implementation
​https://bugs.webkit.org/show_bug.cgi?id=40233

Reviewed by Kenneth Rohde Christiansen.

Implement ChunkedUpdateDrawingArea + Proxy for Qt. Not built yet.

  • UIProcess/ChunkedUpdateDrawingAreaProxy.h:
  • UIProcess/qt: Added.
  • UIProcess/qt/ChunkedUpdateDrawingAreaProxyQt.cpp: Added.

(WebKit::ChunkedUpdateDrawingAreaProxy::page):
(WebKit::ChunkedUpdateDrawingAreaProxy::ensureBackingStore):
(WebKit::ChunkedUpdateDrawingAreaProxy::invalidateBackingStore):
(WebKit::ChunkedUpdateDrawingAreaProxy::platformPaint):
(WebKit::ChunkedUpdateDrawingAreaProxy::drawUpdateChunkIntoBackingStore):

  • WebProcess/WebPage/qt: Added.
  • WebProcess/WebPage/qt/ChunkedUpdateDrawingAreaQt.cpp: Added.

(WebKit::ChunkedUpdateDrawingArea::paintIntoUpdateChunk):

3:50 AM Changeset in webkit [62480] by morrita@google.com
  • 3 edits in trunk/WebKitTools

2010-07-04 MORITA Hajime <​morrita@google.com>

rebaseline-chromium-webkit-tests: UnicodeDecodeError
​https://bugs.webkit.org/show_bug.cgi?id=41589

  • run() method can result non-utf-8 bytes, that causes utf-8 decoding fail. Fixed to disable decoding.
  • Fixed Git.find_checkout_root() to make a test pass.
  • Scripts/webkitpy/common/checkout/scm.py:
  • Scripts/webkitpy/common/checkout/scm_unittest.py:
3:05 AM Changeset in webkit [62479] by yurys@chromium.org
  • 4 edits in trunk

2010-07-05 Yury Semikhatsky <​yurys@chromium.org>

Reviewed by Pavel Feldman.

[v8] Web Inspector: remove v8-specific code dealing with getOwnPropertyNames from InjectedScript.js
​https://bugs.webkit.org/show_bug.cgi?id=41595

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

2010-07-05 Yury Semikhatsky <​yurys@chromium.org>

Reviewed by Pavel Feldman.

[v8] Web Inspector: remove v8-specific code dealing with getOwnPropertyNames from InjectedScript.js
​https://bugs.webkit.org/show_bug.cgi?id=41595

  • src/js/DebuggerScript.js:
3:03 AM Changeset in webkit [62478] by rwlbuis@webkit.org
  • 7 edits
    3 adds in trunk

2010-07-04 Rob Buis <​rwlbuis@gmail.com>

Reviewed by Dirk Schulze.

SVG polygons should draw polygons up to the first parsing error
​https://bugs.webkit.org/show_bug.cgi?id=41140

Render polygons up until the first parsing error.

Test: svg/custom/poly-parsing-error.html

  • svg/SVGPolyElement.cpp: (WebCore::SVGPolyElement::parseMappedAttribute):
2:49 AM QtWebKitTriageRoster edited by Simon Hausmann
(diff)
12:32 AM Changeset in webkit [62477] by alice.liu@apple.com
  • 2 edits in trunk/WebCore

Crash reading past end of block in UniscribeController::shapeAndPlaceItem
​https://bugs.webkit.org/show_bug.cgi?id=41554

Reviewed by Dan Bernstein.

Test: platform/win/fast/text/uniscribe-item-boundary-crash.html

  • platform/graphics/win/UniscribeController.cpp:

(WebCore::UniscribeController::shapeAndPlaceItem):
Don't look one past the end of str. Instead look to the next item, if applicable.

12:18 AM Changeset in webkit [62476] by Philippe Normand
  • 4 edits in trunk/WebKit/mac

2010-06-23 Philippe Normand <​pnormand@igalia.com>

Reviewed by Eric Carlson.

[GStreamer] on Mac OS use a runloop observer to process the glib context iterations
​https://bugs.webkit.org/show_bug.cgi?id=35747

If VIDEO and GSTREAMER are enabled, make the WebView start a
runloop observer that will trigger the glib main context
iterations. Those are needed by the GStreamer player only at the
moment but could later be used for other GLib-dependant components
if required.

  • WebView/WebView.mm: (-[WebView _commonInitializationWithFrameName:groupName:usesDocumentViews:]): (-[WebView _close]): (-[WebView _clearGlibLoopObserver]): (glibContextIterationCallback): (-[WebView _scheduleGlibContextIterations]):
  • WebView/WebViewData.h:
  • WebView/WebViewInternal.h:

Jul 4, 2010:

11:56 PM Changeset in webkit [62475] by yurys@chromium.org
  • 2 edits in trunk/LayoutTests

2010-07-04 Yury Semikhatsky <​yurys@chromium.org>

Unreviewed. Mark fast/js/function-apply-many-args.html as
having wrong output in debug mode.

  • platform/chromium/test_expectations.txt:
9:45 PM Changeset in webkit [62474] by eric@webkit.org
  • 6 edits in trunk

2010-07-04 Eric Seidel <​eric@webkit.org>

Reviewed by Adam Barth.

Add basic "in table body" mode to support <tr> insertion
​https://bugs.webkit.org/show_bug.cgi?id=41587

  • html5lib/runner-expected-html5.txt:
    • <tr> tags now show up, imagine that.

2010-07-04 Eric Seidel <​eric@webkit.org>

Reviewed by Adam Barth.

Add basic "in table body" mode to support <tr> insertion
​https://bugs.webkit.org/show_bug.cgi?id=41587

This also adds a (currently untestable?)
popUntilTableBodyScopeMarker code path.
Any tags which would be between a tbody a <tr> would end up
foster parented outside the <table>. I think the spec was
just being over-cautious with popUntilTableBodyScopeMarker.

  • html/HTMLElementStack.cpp: (WebCore::HTMLNames::isTableScopeMarker): (WebCore::HTMLNames::isTableBodyScopeMarker): (WebCore::HTMLElementStack::popUntilTableBodyScopeMarker):
  • html/HTMLElementStack.h:
  • html/HTMLTreeBuilder.cpp: (WebCore::HTMLTreeBuilder::processStartTag): (WebCore::HTMLTreeBuilder::callTheAdoptionAgency):
7:37 PM Changeset in webkit [62473] by pkasting@chromium.org
  • 4 edits in trunk/LayoutTests

Unreviewed. Update Chromium test expectations.

  • platform/chromium-mac/html5lib/runner-expected.txt:
  • platform/chromium-win/html5lib/runner-expected.txt:
  • platform/chromium/test_expectations.txt:
4:27 PM Changeset in webkit [62472] by andersca@apple.com
  • 2 edits in trunk/WebCore

Fix Windows build.

  • WebCore.vcproj/WebCore.vcproj:
4:15 PM Changeset in webkit [62471] by andersca@apple.com
  • 5 edits
    2 adds in trunk/WebKit2

Add simple command line parser and pass mode argument to the web process
​https://bugs.webkit.org/show_bug.cgi?id=41586

Reviewed by Dan Bernstein.

  • Shared/CommandLine.h:

(WebKit::CommandLine::operator[]):
Add CommandLine class.

  • Shared/mac/CommandLineMac.cpp:

(WebKit::CommandLine::parse):
Implement Mac version of CommandLine::parse.

  • UIProcess/Launcher/mac/ProcessLauncherMac.mm:

(WebKit::ProcessLauncher::launchProcess):
Add "legacywebprocess" mode to the posix_spawn call.

  • WebKit2.xcodeproj/project.pbxproj:

Add CommandLine to the xcode project.

  • WebProcess/Launching/mac/WebProcessMain.mm:

(webProcessMain):
(main):
Parse the command line and check the mode.

3:06 PM Changeset in webkit [62470] by eric@webkit.org
  • 2 edits in trunk/WebCore

2010-07-04 Eric Seidel <​eric@webkit.org>

Reviewed by Adam Barth.

HTMLTreeBuilder needs an adoption agency
​https://bugs.webkit.org/show_bug.cgi?id=41453

Fix Qt Minimal build. SVGNames.h should always
be generated, even when SVG is off, however that's
not how things currently work.

  • html/HTMLTreeBuilder.cpp:
2:54 PM Changeset in webkit [62469] by eric@webkit.org
  • 9 edits in trunk

2010-07-04 Eric Seidel <​eric@webkit.org>

Reviewed by Adam Barth.

Add a very basic InTable insertion mode
​https://bugs.webkit.org/show_bug.cgi?id=41581

  • html5lib/runner-expected-html5.txt:
    • Update expectations now that we pass 6 more tests.

2010-07-04 Eric Seidel <​eric@webkit.org>

Reviewed by Adam Barth.

Add a very basic InTable insertion mode
​https://bugs.webkit.org/show_bug.cgi?id=41581

There is still a bunch of low-hanging fruit left for this
mode, but even this most-basic support lets us pass 6 more tests. :)

It's a progression, ship it! :)

  • html/HTMLElementStack.cpp: (WebCore::HTMLNames::isScopeMarker): (WebCore::HTMLNames::isListItemScopeMarker): (WebCore::HTMLNames::isTableScopeMarker): (WebCore::HTMLElementStack::popUntilTableScopeMarker):
  • html/HTMLElementStack.h:
  • html/HTMLFormattingElementList.cpp: (WebCore::HTMLFormattingElementList::appendMarker):
  • html/HTMLFormattingElementList.h:
  • html/HTMLTreeBuilder.cpp: (WebCore::HTMLTreeBuilder::processStartTag):
  • html/HTMLTreeBuilder.h:
2:15 PM Changeset in webkit [62468] by eric@webkit.org
  • 13 edits
    1 add in trunk

2010-07-02 Eric Seidel <​eric@webkit.org>

Reviewed by Adam Barth.

HTMLTreeBuilder needs an adoption agency
​https://bugs.webkit.org/show_bug.cgi?id=41453

Added new adoption01 suite for testing adoption agency
bugs. Right now only the simplest adoption test passes.
I'll be adding more in future commits.

  • html5lib/resources/adoption01.dat: Added.
  • html5lib/runner-expected-html5.txt:
  • html5lib/runner-expected.txt:
  • html5lib/runner.html:

2010-07-01 Eric Seidel <​eric@webkit.org>

Reviewed by Adam Barth.

HTMLTreeBuilder needs an adoption agency
​https://bugs.webkit.org/show_bug.cgi?id=41453

This changes some test results, but only makes the simplest
adoption agency cases pass. I think the code is likely
very close, but further iteration to make this change larger
seems counter-productive. I recommend we check in this
progression and work from here.

  • dom/ContainerNode.cpp: (WebCore::ContainerNode::addChildCommon):
    • Make sure callers don't assume this will reparent.

(WebCore::ContainerNode::parserAddChild):

  • Update comment to document lack of reparenting behavior.
  • html/HTMLElementStack.cpp: (WebCore::HTMLElementStack::ElementRecord::ElementRecord): (WebCore::HTMLElementStack::ElementRecord::~ElementRecord): (WebCore::HTMLElementStack::ElementRecord::replaceElement): (WebCore::HTMLElementStack::ElementRecord::isAbove):
    • Added for debugging.

(WebCore::HTMLElementStack::pushHTMLHtmlElement):
(WebCore::HTMLElementStack::insertAbove):

  • Needed for the adoption agency.

(WebCore::HTMLElementStack::topRecord):
(WebCore::HTMLElementStack::bottom):
(WebCore::HTMLElementStack::removeHTMLHeadElement):
(WebCore::HTMLElementStack::remove):
(WebCore::HTMLElementStack::find):
(WebCore::HTMLElementStack::topmost):
(WebCore::HTMLElementStack::contains):
(WebCore::HTMLElementStack::htmlElement):
(WebCore::HTMLElementStack::headElement):
(WebCore::HTMLElementStack::bodyElement):
(WebCore::HTMLElementStack::pushCommon):
(WebCore::HTMLElementStack::removeNonTopCommon):

  • Fix the name to match top/bottom.
  • html/HTMLElementStack.h: (WebCore::HTMLElementStack::ElementRecord::element): (WebCore::HTMLElementStack::ElementRecord::next): (WebCore::HTMLElementStack::ElementRecord::releaseNext): (WebCore::HTMLElementStack::ElementRecord::setNext):
  • html/HTMLFormattingElementList.cpp: (WebCore::HTMLFormattingElementList::closestElementInScopeWithName): (WebCore::HTMLFormattingElementList::contains): (WebCore::HTMLFormattingElementList::find): (WebCore::HTMLFormattingElementList::remove):
  • html/HTMLFormattingElementList.h: (WebCore::HTMLFormattingElementList::isEmpty): (WebCore::HTMLFormattingElementList::size):
  • html/HTMLTreeBuilder.cpp: (WebCore::HTMLTreeBuilder::processStartTag): (WebCore::HTMLTreeBuilder::furthestBlockForFormattingElement):
    • Part of the Adoption Agency algorithm.

(WebCore::HTMLTreeBuilder::findFosterParentFor):

  • Used to move mis-nested content out of tables. This doesn't seem to work quite right yet.

(WebCore::HTMLTreeBuilder::reparentChildren):
(WebCore::HTMLTreeBuilder::callTheAdoptionAgency):

  • The ridiculously long/complicated adoption agency algorithm from HTML5.

(WebCore::HTMLTreeBuilder::processEndTag):

  • html/HTMLTreeBuilder.h:
10:44 AM Changeset in webkit [62467] by jschuh@chromium.org
  • 2 edits in trunk/WebCore

2010-07-04 Justin Schuh <​jschuh@chromium.org>

Reviewed by Darin Adler.

Remove custom src bindings for HTMLFrameElement and HTMLIFrameElement
​https://bugs.webkit.org/show_bug.cgi?id=41578

Remove bindings obsoleted by: ​http://trac.webkit.org/changeset/59866
No new tests because behavior is not changed.

  • Android.jscbindings.mk:
  • Android.v8bindings.mk:
  • CMakeLists.txt:
  • GNUmakefile.am:
  • WebCore.gypi:
  • WebCore.pro:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • bindings/js/JSBindingsAllInOne.cpp:
  • bindings/js/JSHTMLFrameElementCustom.cpp:
  • bindings/js/JSHTMLIFrameElementCustom.cpp: Removed.
  • bindings/v8/custom/V8HTMLFrameElementCustom.cpp:
  • bindings/v8/custom/V8HTMLIFrameElementCustom.cpp: Removed.
  • html/HTMLFrameElement.idl:
  • html/HTMLIFrameElement.idl:
9:22 AM Changeset in webkit [62466] by jschuh@chromium.org
  • 14 edits
    2 deletes in trunk/WebCore

2010-07-04 Justin Schuh <​jschuh@chromium.org>

Reviewed by Darin Adler.

Remove custom src bindings for HTMLFrameElement and HTMLIFrameElement
​https://bugs.webkit.org/show_bug.cgi?id=41578

Remove bindings obsoleted by: ​http://trac.webkit.org/changeset/59866
No new tests because behavior is not changed.

  • Android.jscbindings.mk:
  • Android.v8bindings.mk:
  • CMakeLists.txt:
  • GNUmakefile.am:
  • WebCore.gypi:
  • WebCore.pro:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • bindings/js/JSBindingsAllInOne.cpp:
  • bindings/js/JSHTMLFrameElementCustom.cpp:
  • bindings/js/JSHTMLIFrameElementCustom.cpp: Removed.
  • bindings/v8/custom/V8HTMLFrameElementCustom.cpp:
  • bindings/v8/custom/V8HTMLIFrameElementCustom.cpp: Removed.
  • html/HTMLFrameElement.idl:
  • html/HTMLIFrameElement.idl:
9:06 AM UpdatingBugzilla edited by ddkilzer@webkit.org
(diff)
9:04 AM UpdatingBugzilla edited by ddkilzer@webkit.org
(diff)
6:38 AM Changeset in webkit [62465] by robert@webkit.org
  • 2 edits in trunk/LayoutTests

2010-07-04 Robert Hogan <​robert@webkit.org>

[Qt] Triage failing HTTP tests in Qt Skipped list

Unreviewed, sorting tests and adding comments to skipped list.

  • platform/qt/Skipped:
4:48 AM Changeset in webkit [62464] by mrowe@apple.com
  • 2 edits in trunk/JavaScriptCore

Build fix after r62456.

  • interpreter/Interpreter.cpp:

(JSC::Interpreter::privateExecute): Be slightly more consistent in using uint32_t to prevent
warnings about comparisons between signed and unsigned types, and attempts to call an overload
of std::min that doesn't exist.

3:38 AM Changeset in webkit [62463] by robert@webkit.org
  • 2 edits in trunk/LayoutTests

[Qt] NPP_SetWindow seems to not be called when TestNetscapePlugin is moved

Unreviewed, accidental omission from r61505

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

Forgot to unskip plugins/reentrant-update-widget-positions.html
as part of ​http://trac.webkit.org/changeset/61505

3:16 AM Changeset in webkit [62462] by yurys@chromium.org
  • 3 edits
    4 adds in trunk/LayoutTests

2010-07-04 Yury Semikhatsky <​yury@yurys-imac.local>

Unreviewed. Update Chromium test expectations.

  • platform/chromium-mac/fast/js/function-apply-expected.txt: Added.
  • platform/chromium-mac/fast/js/function-apply-many-args-expected.txt: Added.
  • platform/chromium-mac/html5lib/runner-expected.txt:
  • platform/chromium-win/fast/js/function-apply-expected.txt: Added.
  • platform/chromium-win/fast/js/function-apply-many-args-expected.txt: Added.
  • platform/chromium-win/html5lib/runner-expected.txt:
2:41 AM Changeset in webkit [62461] by mjs@apple.com
  • 2 edits in trunk/WebKit2

2010-07-04 Maciej Stachowiak <​mjs@apple.com>

Reviewed by Mark Rowe.

WebProcess crashes in release for simple layout tests
​https://bugs.webkit.org/show_bug.cgi?id=41575

  • WebProcess/InjectedBundle/API/c/WKBundlePage.cpp: (WKBundlePageCopyRenderTreeExternalRepresentation): ref external representation StringImpl* before taking it out of its parent String, to avoid returning freed memory.

Jul 3, 2010:

10:09 PM Changeset in webkit [62460] by abarth@webkit.org
  • 7 edits in trunk

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

Reviewed by Maciej Stachowiak.

Implement AfterAfterFramesetMode
​https://bugs.webkit.org/show_bug.cgi?id=41561

Test coverage of this mode was a bit ... lacking, shall we say? This
patch adds a test for it.

  • html5lib/resources/webkit01.dat:
  • html5lib/runner-expected-html5.txt:
  • html5lib/runner-expected.txt:
  • html5lib/webkit-resumer-expected.txt:

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

Reviewed by Maciej Stachowiak.

Implement AfterAfterFramesetMode
​https://bugs.webkit.org/show_bug.cgi?id=41561

This mode is almost unobservable. The main way to observe it seems to
be seeing where comment nodes get attached to the DOM.

  • html/HTMLTreeBuilder.cpp: (WebCore::HTMLTreeBuilder::processStartTag): (WebCore::HTMLTreeBuilder::processEndTag): (WebCore::HTMLTreeBuilder::processComment): (WebCore::HTMLTreeBuilder::processCharacter): (WebCore::HTMLTreeBuilder::processEndOfFile):
9:53 PM Changeset in webkit [62459] by mjs@apple.com
  • 2 edits in trunk/WebKit2

Add stdint.h include to fix WebKitTestRunner build.

Rubber stamped by Anders Carlsson.

  • UIProcess/API/C/WKPage.h:
6:33 PM Changeset in webkit [62458] by zmo@google.com
  • 5 edits in trunk

2010-07-03 Zhenyao Mo <​zmo@google.com>

Reviewed by Dimitri Glazkov.

WebGLRenderingContext::vertexAttrib* leads to possible out-of-range vector member visit
​https://bugs.webkit.org/show_bug.cgi?id=41572

  • platform/mac-leopard/Skipped: Adding back the program-test.html test since the failure cause is fixed in this patch.

2010-07-03 Zhenyao Mo <​zmo@google.com>

Reviewed by Dimitri Glazkov.

WebGLRenderingContext::vertexAttrib* leads to possible out-of-range vector member visit
​https://bugs.webkit.org/show_bug.cgi?id=41572

  • html/canvas/WebGLRenderingContext.cpp: Fix the out-of-range vector member visit, also refactor the code. (WebCore::WebGLRenderingContext::vertexAttrib1f): (WebCore::WebGLRenderingContext::vertexAttrib1fv): (WebCore::WebGLRenderingContext::vertexAttrib2f): (WebCore::WebGLRenderingContext::vertexAttrib2fv): (WebCore::WebGLRenderingContext::vertexAttrib3f): (WebCore::WebGLRenderingContext::vertexAttrib3fv): (WebCore::WebGLRenderingContext::vertexAttrib4f): (WebCore::WebGLRenderingContext::vertexAttrib4fv): (WebCore::WebGLRenderingContext::vertexAttribfImpl): (WebCore::WebGLRenderingContext::vertexAttribfvImpl):
  • html/canvas/WebGLRenderingContext.h: Helper function declaration.
5:11 PM Changeset in webkit [62457] by weinig@apple.com
  • 4 edits in trunk/JavaScriptCore

Patch for ​https://bugs.webkit.org/show_bug.cgi?id=41553
Make StringExtras.h versions of snprintf and vsnprintf match the unix versions.

Reviewed by Darin Adler.

  • MSVC does not ensure the buffers are null terminated as the unix versions do.
  • runtime/JSGlobalObjectFunctions.cpp: Cleanup includes.
  • runtime/UString.cpp: Clean up includes.

(JSC::UString::from): Don't pass sizeof(buf) - 1, that is wrong.

  • wtf/StringExtras.h:

(snprintf): Ensure null termination of buffer.
(vsnprintf): Ditto.

4:49 PM Changeset in webkit [62456] by Darin Adler
  • 5 edits in trunk

2010-07-03 Yong Li <​yoli@rim.com>

Reviewed by Darin Adler.

Make Arguments::MaxArguments clamping work for numbers >= 0x80000000 in
the interpreter as well as the JIT.

​https://bugs.webkit.org/show_bug.cgi?id=41351
rdar://problem/8142141

  • interpreter/Interpreter.cpp: (JSC::Interpreter::privateExecute): Fix signed integer overflow problem in op_load_varargs handling. 0xFFFFFFFF was read as -1.

2010-07-03 Darin Adler <Darin Adler>

Added test cases for edge cases in apply function on arrays.
​https://bugs.webkit.org/show_bug.cgi?id=41351

  • fast/js/function-apply-expected.txt: Updated to expect success.
  • fast/js/script-tests/function-apply.js: Added test cases.
4:02 PM Changeset in webkit [62455] by jorlow@chromium.org
  • 2 edits in trunk/WebCore

Ugh. Have to put the destructor in the .h file since the .ccp isn't compiled yet.

  • storage/IDBKey.h:

(WebCore::IDBKey::~IDBKey):

4:00 PM Changeset in webkit [62454] by commit-queue@webkit.org
  • 3 edits in trunk/WebKitTools

2010-07-03 Patrick Gansterer <​paroga@paroga.com>

Reviewed by Simon Hausmann.

[Qt] Fix DumpRenderTree userStyleSheet handling.
​https://bugs.webkit.org/show_bug.cgi?id=41570

DumpRenderTree did only set the userStyleSheet at
layoutTestController.setUserStyleSheetEnabled().

  • DumpRenderTree/qt/LayoutTestControllerQt.cpp: (LayoutTestController::reset): (LayoutTestController::setUserStyleSheetLocation): (LayoutTestController::setUserStyleSheetEnabled):
  • DumpRenderTree/qt/LayoutTestControllerQt.h:
3:48 PM UpdatingBugzilla edited by ddkilzer@webkit.org
(diff)
3:48 PM Changeset in webkit [62453] by jorlow@chromium.org
  • 2 edits in trunk/WebCore

Build fix. Forgot destructor.

  • storage/IDBKey.cpp:

(WebCore::IDBKey::~IDBKey):

3:46 PM UpdatingBugzilla edited by ddkilzer@webkit.org
(diff)
3:45 PM UpdatingBugzilla edited by ddkilzer@webkit.org
Updated Update section for bzr (diff)
3:37 PM UpdatingBugzilla edited by ddkilzer@webkit.org
Switched from CVS to bzr for upstream Bugzilla repository. (diff)
3:18 PM Changeset in webkit [62452] by jorlow@chromium.org
  • 20 edits
    6 copies
    5 adds in trunk

2010-06-26 Jeremy Orlow <​jorlow@chromium.org>

Reviewed by Dumitru Daniliuc.

Support for keys and in-memory storage for IndexedDB
​https://bugs.webkit.org/show_bug.cgi?id=41252

Set the role to Private.

2010-06-26 Jeremy Orlow <​jorlow@chromium.org>

Reviewed by Dumitru Daniliuc.

Support for keys and in-memory storage for IndexedDB
​https://bugs.webkit.org/show_bug.cgi?id=41252

It'll take some time to get data persistence working for IndexedDB,
so until then, we'll just store everything in an in memory tree.
The tree uses WTF::AVLTree and is a template so that it can be used by
object stores (IDBKey -> SerializedScriptValue) and indexes (IDBKey ->
IDBKey). This class will be used in a subsequent patch.

Also add an IDBKey type that represents one of these keys. We use a
custom toJS function in a way similar to IDBAny to convert from WebCore
to a JS value. For converting the other way, we have to teach the code
generators what to do (unfortunately). This is done in a way similar
to serialized script value. Unlike serialized script value, IDBKey is
in WebCore and only a helper function is JS engine specific.

This code is not accessable from layout tests. (Will fix in
​https://bugs.webkit.org/show_bug.cgi?id=41250) The bindings tests
show us that the generated bindings are what we expect.

  • ForwardingHeaders/wtf/AVLTree.h: Added.
  • bindings/js/IDBBindingUtilities.cpp: Added. (WebCore::createIDBKeyFromValue):
  • bindings/js/IDBBindingUtilities.h: Added.
  • bindings/js/JSIDBKeyCustom.cpp: Added. (WebCore::toJS):
  • bindings/scripts/CodeGeneratorJS.pm:
  • bindings/scripts/CodeGeneratorV8.pm:
  • bindings/scripts/test/CPP/WebDOMTestObj.cpp: (WebDOMTestObj::idbKey):
  • bindings/scripts/test/CPP/WebDOMTestObj.h:
  • bindings/scripts/test/GObject/WebKitDOMTestObj.cpp: (webkit_dom_test_obj_idb_key):
  • bindings/scripts/test/GObject/WebKitDOMTestObj.h:
  • bindings/scripts/test/JS/JSTestObj.cpp: (WebCore::): (WebCore::jsTestObjPrototypeFunctionIdbKey):
  • bindings/scripts/test/JS/JSTestObj.h:
  • bindings/scripts/test/ObjC/DOMTestObj.h:
  • bindings/scripts/test/ObjC/DOMTestObj.mm: (-[DOMTestObj idbKey:]):
  • bindings/scripts/test/TestObj.idl:
  • bindings/scripts/test/V8/V8TestObj.cpp: (WebCore::TestObjInternal::idbKeyCallback): (WebCore::):
  • bindings/v8/IDBBindingUtilities.cpp: Added. (WebCore::createIDBKeyFromValue):
  • bindings/v8/IDBBindingUtilities.h: Added.
  • bindings/v8/custom/V8IDBKeyCustom.cpp: Added. (WebCore::toV8):
  • storage/IDBCallbacks.h:
  • storage/IDBKey.cpp: Added. (WebCore::IDBKey::IDBKey): (WebCore::IDBKey::~IDBKey):
  • storage/IDBKey.h: Added. (WebCore::IDBKey::create): (WebCore::IDBKey::): (WebCore::IDBKey::type): (WebCore::IDBKey::string): (WebCore::IDBKey::number):
  • storage/IDBKey.idl: Added.
  • storage/IDBKeyTree.h: Added. (WebCore::IDBKeyTree::create): (WebCore::IDBKeyTree::AVLTreeAbstractor::get_less): (WebCore::IDBKeyTree::AVLTreeAbstractor::set_less): (WebCore::IDBKeyTree::AVLTreeAbstractor::get_greater): (WebCore::IDBKeyTree::AVLTreeAbstractor::set_greater): (WebCore::IDBKeyTree::AVLTreeAbstractor::get_balance_factor): (WebCore::IDBKeyTree::AVLTreeAbstractor::set_balance_factor): (WebCore::IDBKeyTree::AVLTreeAbstractor::null): (WebCore::IDBKeyTree::AVLTreeAbstractor::compare_key_node): (WebCore::IDBKeyTree::AVLTreeAbstractor::compare_node_node): (WebCore::::IDBKeyTree): (WebCore::::~IDBKeyTree): (WebCore::::AVLTreeAbstractor::compare_key_key): (WebCore::::get): (WebCore::::insert): (WebCore::::remove):
2:57 PM Changeset in webkit [62451] by jhoneycutt@apple.com
  • 14 edits in trunk

The missing plug-in indicator should be clickable

​https://bugs.webkit.org/show_bug.cgi?id=41550
<rdar://problem/8132162>

From an original patch by Kevin Decker.

Reviewed by Darin Adler.

WebCore:

  • html/HTMLPlugInElement.cpp:

(WebCore::HTMLPlugInElement::defaultEventHandler):
If the renderer is a RenderEmbeddedWidget showing the missing plug-in
indicator, and the event is a click even, call the ChromeClient's
missingPluginButtonClicked() function.

  • page/ChromeClient.h:

(WebCore::ChromeClient::missingPluginButtonClicked):
Declare missingPluginButtonClicked(), and stub the default
implementation.

  • rendering/RenderEmbeddedObject.cpp:

(WebCore::RenderEmbeddedObject::RenderEmbeddedObject):
Initialize m_showsMissingPluginIndicator.
(WebCore::RenderEmbeddedObject::setShowsMissingPluginIndicator):
Assert that we're not currently showing any replacement text. Set
m_showsMissingPluginIndicator after setting the replacement text.
(WebCore::RenderEmbeddedObject::setShowsCrashedPluginIndicator):
Add the same assert as above.

  • rendering/RenderEmbeddedObject.h:

(WebCore::RenderEmbeddedObject::showsMissingPluginIndicator):
Getter for m_showsMissingPluginIndicator.

WebKit/mac:

  • WebCoreSupport/WebChromeClient.h:

Declare an override of missingPluginButtonClicked().

  • WebCoreSupport/WebChromeClient.mm:

(WebChromeClient::missingPluginButtonClicked):
Call the UIDelegate's method.

  • WebView/WebUIDelegatePrivate.h:

Declare didPressMissingPluginButton.

WebKit/win:

  • Interfaces/IWebUIDelegatePrivate.idl:

Added a new delegate interface, and declare a function
didPressMissingPluginButton().

  • Interfaces/WebKit.idl:

Touch this file to force interfaces to be rebuilt.

  • WebCoreSupport/WebChromeClient.cpp:

(WebChromeClient::missingPluginButtonClicked):
Get the UI delegate, and query it for IWebUIDelegatePrivate3. Call its
didPressMissingPluginButton() function.

  • WebCoreSupport/WebChromeClient.h:

Declare an override of missingPluginButtonClicked().

2:36 PM Changeset in webkit [62450] by robert@webkit.org
  • 4 edits in trunk

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

Reviewed by Kenneth Rohde Christiansen.

[Qt] Return usable property names from DRT's computedStyleIncludingVisitedInfo()

Unskip fast/history/self-is-visited.html

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

  • platform/qt/Skipped:

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

Reviewed by Kenneth Rohde Christiansen.

[Qt] Return usable property names from DRT's computedStyleIncludingVisitedInfo()

Object property names such as 'background-color' need to be returned in camel
case, i.e. backgroundColor so that JS can reference them. Add support for this
to DumpRenderTreeSupportQt.

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

  • WebCoreSupport/DumpRenderTreeSupportQt.cpp: (convertToPropertyName): (DumpRenderTreeSupportQt::computedStyleIncludingVisitedInfo):
1:30 PM Changeset in webkit [62449] by oliver@apple.com
  • 9 edits in trunk

Move BOM handling out of the lexer and parser
​https://bugs.webkit.org/show_bug.cgi?id=41539

Reviewed by Geoffrey Garen.

JavaScriptCore:

Doing the BOM stripping in the lexer meant that we could
end up having to strip the BOMs from a source multiple times.
To deal with this we now require all strings provided by
a SourceProvider to already have had the BOMs stripped.
This also simplifies some of the lexer logic.

  • parser/Lexer.cpp:

(JSC::Lexer::setCode):
(JSC::Lexer::sourceCode):

  • parser/SourceProvider.h:

(JSC::SourceProvider::SourceProvider):
(JSC::UStringSourceProvider::create):
(JSC::UStringSourceProvider::getRange):
(JSC::UStringSourceProvider::UStringSourceProvider):

  • wtf/text/StringImpl.h:

(WebCore::StringImpl::copyStringWithoutBOMs):

WebCore:

Update WebCore to ensure that SourceProviders don't
produce strings with BOMs in them.

  • bindings/js/ScriptSourceProvider.h:

(WebCore::ScriptSourceProvider::ScriptSourceProvider):

  • bindings/js/StringSourceProvider.h:

(WebCore::StringSourceProvider::StringSourceProvider):

  • loader/CachedScript.cpp:

(WebCore::CachedScript::CachedScript):
(WebCore::CachedScript::script):

  • loader/CachedScript.h:

(WebCore::CachedScript::):

CachedScript now stores decoded data with the BOMs stripped,
and caches the presence of BOMs across memory purges.

12:20 PM Changeset in webkit [62448] by xan@webkit.org
  • 2 edits in trunk/WebCore

2010-07-03 Xan Lopez <​xlopez@igalia.com>

Include DerivedSources/WebCore before DerivedSources/

Recently DerivedSources generation was changed, with some files no
longer being generated in the toplevel DerivedSources
directory. Since that directory is first in the -I flags the build
can be broken in some cases by including old files unless a 'make
clean' is done. Change the -I order to fix the build in the 32 bit
Release bot.

  • GNUmakefile.am:
7:08 AM Changeset in webkit [62447] by arv@chromium.org
  • 7 edits
    1 add in trunk

2010-07-03 Erik Arvidsson <​arv@chromium.org>

Reviewed by Ojan Vafai.

Fix issue where a contextmenu event was reporting the wrong target if
the context menu was shown due to pressing the context menu key
(or Shift+F10).

Split sendContextMenuForEvent into one case for keyboard events and use
that when the contextmenu event should be dispatched due to a keypress.

For the keboard case we now use the focused node as the target for the
event and use the clipped rect to determine the position of the menu.

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

Use manual test since DRT does not handle context menu keys.

  • manual-tests/win/contextmenu-key.html: Added.
  • page/EventHandler.cpp: (WebCore::EventHandler::sendContextMenuEvent): (WebCore::EventHandler::sendContextMenuEventForKey):
  • page/EventHandler.h:

2010-07-03 Erik Arvidsson <​arv@chromium.org>

Reviewed by Ojan Vafai.

Fix issue where a contextmenu event was reporting the wrong target
if the context menu was shown due to pressing the context menu key
(or Shift+F10).

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

  • src/WebViewImpl.cpp: (WebKit::WebViewImpl::sendContextMenuEvent):

2010-07-03 Erik Arvidsson <​arv@chromium.org>

Reviewed by Ojan Vafai.

Fix issue where a contextmenu event was reporting the wrong target
if the context menu was shown due to pressing the context menu key
(or Shift+F10).

  • WebView.cpp: (WebView::handleContextMenuEvent):
5:03 AM Changeset in webkit [62446] by krit@webkit.org
  • 2 edits in trunk/WebCore

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

Unreviewed sort of XCode project file.

  • WebCore.xcodeproj/project.pbxproj:
3:42 AM Changeset in webkit [62445] by robert@webkit.org
  • 10 edits in trunk/LayoutTests

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

[Qt] Update Qt expected results after r62444

Unreviewed, update test results.

​http://trac.webkit.org/changeset/62444
​https://bugs.webkit.org/show_bug.cgi?id=41088

Qt expected results for various fast/ tests updated to reflect
support for dumpChildFrameScrollPositions(). Checked against Mac results
and they are all good.

I have not updated the .png and .checksum files for the tests. This is
because I suspect I will not get the same pixel results as the Qt build
bot.

  • platform/qt/fast/dynamic/anchor-lock-expected.txt:
  • platform/qt/fast/overflow/position-fixed-transform-clipping-expected.txt:
  • platform/qt/fast/repaint/fixed-child-move-after-scroll-expected.txt:
  • platform/qt/fast/repaint/fixed-child-of-fixed-move-after-scroll-expected.txt:
  • platform/qt/fast/repaint/fixed-child-of-transformed-move-after-scroll-expected.txt:
  • platform/qt/fast/repaint/fixed-move-after-scroll-expected.txt:
  • platform/qt/fast/repaint/fixed-tranformed-expected.txt:
  • platform/qt/fast/repaint/repaint-during-scroll-expected.txt:
  • platform/qt/transforms/2d/transform-fixed-container-expected.txt:
3:00 AM Changeset in webkit [62444] by robert@webkit.org
  • 7 edits in trunk

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

Reviewed by Kenneth Rohde Christiansen.

[Qt] support dumpChildFrameScrollPositions

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

Unskip:

http/tests/navigation/anchor-subframeload.html
http/tests/navigation/relativeanchor-frames.html

  • platform/qt/Skipped:

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

Reviewed by Kenneth Rohde Christiansen.

[Qt] support dumpChildFrameScrollPositions

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

Unskip:

http/tests/navigation/anchor-subframeload.html
http/tests/navigation/relativeanchor-frames.html

  • DumpRenderTree/qt/DumpRenderTreeQt.cpp: (WebCore::DumpRenderTree::dumpFrameScrollPosition): (WebCore::DumpRenderTree::dump):
  • DumpRenderTree/qt/DumpRenderTreeQt.h:
  • DumpRenderTree/qt/LayoutTestControllerQt.cpp: (LayoutTestController::reset):
  • DumpRenderTree/qt/LayoutTestControllerQt.h: (LayoutTestController::shouldDumpChildFrameScrollPositions): (LayoutTestController::dumpChildFrameScrollPositions):
1:45 AM Changeset in webkit [62443] by commit-queue@webkit.org
  • 2 edits
    1 add in trunk/WebCore

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

Reviewed by Kent Tamura.

[BREWMP] Port Widget
​https://bugs.webkit.org/show_bug.cgi?id=41538

Make Widget a dummy class. WebKit Brew MP uses the full screen mode
and does not use the window system introduced in Brew MP.

  • platform/Widget.h:
  • platform/brew/WidgetBrew.cpp: Added. (WebCore::Widget::Widget): (WebCore::Widget::~Widget): (WebCore::Widget::frameRect): (WebCore::Widget::setFrameRect): (WebCore::Widget::setFocus): (WebCore::Widget::setCursor): (WebCore::Widget::show): (WebCore::Widget::hide): (WebCore::Widget::paint): (WebCore::Widget::setIsSelected):
1:30 AM Changeset in webkit [62442] by commit-queue@webkit.org
  • 3 edits in trunk/JavaScriptCore

2010-07-03 Patrick Gansterer <​paroga@paroga.com>

Reviewed by Kent Tamura.

[WINCE] Implement Unicode::isAlphanumeric and Unicode::isArabicChar.
​https://bugs.webkit.org/show_bug.cgi?id=41411

  • wtf/unicode/wince/UnicodeWince.cpp: (WTF::Unicode::isAlphanumeric):
  • wtf/unicode/wince/UnicodeWince.h: (WTF::Unicode::isArabicChar):
1:17 AM Changeset in webkit [62441] by commit-queue@webkit.org
  • 2 edits in trunk/JavaScriptCore

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

Reviewed by Kent Tamura.

[BREWMP] Change the CRASH() macro to print "WebKit CRASH" log.
​https://bugs.webkit.org/show_bug.cgi?id=41524

Print "WebKit CRASH" before crashing.

  • wtf/Assertions.h:
12:17 AM Changeset in webkit [62440] by abarth@webkit.org
  • 4 edits in trunk

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

Reviewed by Eric Seidel.

Implement AfterFramesetMode
​https://bugs.webkit.org/show_bug.cgi?id=41560

It's somewhat hard to observe this state, so I'm not surprised that the
diff to the expected results are small. We still fail this test
because we don't handle whitespace correctly.

  • html5lib/runner-expected-html5.txt:

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

Reviewed by Eric Seidel.

Implement AfterFramesetMode
​https://bugs.webkit.org/show_bug.cgi?id=41560

  • html/HTMLTreeBuilder.cpp: (WebCore::HTMLTreeBuilder::processStartTag): (WebCore::HTMLTreeBuilder::processEndTag): (WebCore::HTMLTreeBuilder::processCharacter): (WebCore::HTMLTreeBuilder::processEndOfFile):
12:03 AM Changeset in webkit [62439] by abarth@webkit.org
  • 4 edits in trunk

2010-07-02 Adam Barth <​abarth@webkit.org>

Reviewed by Eric Seidel.

Implement InFramesetMode
​https://bugs.webkit.org/show_bug.cgi?id=41559

And the tests keep progressing.

  • html5lib/runner-expected-html5.txt:

2010-07-02 Adam Barth <​abarth@webkit.org>

Reviewed by Eric Seidel.

Implement InFramesetMode
​https://bugs.webkit.org/show_bug.cgi?id=41559

Pretty straighforward. We still don't handle character tokens
correctly.

  • html/HTMLTreeBuilder.cpp: (WebCore::HTMLTreeBuilder::processStartTag): (WebCore::HTMLTreeBuilder::processEndTag): (WebCore::HTMLTreeBuilder::processCharacter): (WebCore::HTMLTreeBuilder::processEndOfFile):
Note: See TracTimeline for information about the timeline view.