⚠ Archived content — this site is no longer maintained.   Current WebKit documentation is at docs.webkit.org.

Timeline



Mar 27, 2010:

11:09 PM Changeset in webkit [56681] by eric@webkit.org
  • 7 edits in trunk

2010-03-27 Sergio Villar Senin <​svillar@igalia.com>

Reviewed by Eric Seidel.

Removed 5 http/history tests from Skipped file

[GTK] Improve reporting of frame loader callbacks in DRT
​https://bugs.webkit.org/show_bug.cgi?id=36454

  • platform/gtk/Skipped:

2010-03-27 Sergio Villar Senin <​svillar@igalia.com>

Reviewed by Eric Seidel.

FrameLoader emits onload-event when handling
dispatchDidHandleOnloadEvents

[GTK] Improve reporting of frame loader callbacks in DRT
​https://bugs.webkit.org/show_bug.cgi?id=36454

  • WebCoreSupport/FrameLoaderClientGtk.cpp: (WebKit::FrameLoaderClient::dispatchDidHandleOnloadEvents): implemented, now it emits onload-event signal
  • webkit/webkitwebview.cpp: added onload-event signal

2010-03-27 Sergio Villar Senin <​svillar@igalia.com>

Reviewed by Eric Seidel.

Print didHandleOnloadEventsForFrame in the callback of
onload-event signal comming from frame loader

[GTK] Improve reporting of frame loader callbacks in DRT
​https://bugs.webkit.org/show_bug.cgi?id=36454

  • DumpRenderTree/gtk/DumpRenderTree.cpp: (webViewOnloadEvent): (createWebView): added connection to signal::onload-event and signal callback
10:51 PM Changeset in webkit [56680] by eric@webkit.org
  • 2 edits in trunk/WebKitTools

2010-03-27 Sergio Villar Senin <​svillar@igalia.com>

Reviewed by Eric Seidel.

Print didCommitLoadForFrame in the callback of signal::load-committed

[GTK] Improve reporting of frame loader callbacks in DRT
​https://bugs.webkit.org/show_bug.cgi?id=36454

  • DumpRenderTree/gtk/DumpRenderTree.cpp: (webViewLoadCommitted): (createWebView): added connection to sinal::load-committed and signal callback
10:34 PM Changeset in webkit [56679] by eric@webkit.org
  • 2 edits in trunk/WebKitTools

2010-03-27 Sergio Villar Senin <​svillar@igalia.com>

Reviewed by Eric Seidel.

Print didStartProvisionalLoadForFrame in the callback of
notify::load-status property change notification

[GTK] Improve reporting of frame loader callbacks in DRT
​https://bugs.webkit.org/show_bug.cgi?id=36454

  • DumpRenderTree/gtk/DumpRenderTree.cpp: (webInspectorCloseWindow): (webInspectorInspectWebView): (createWebView): added connection to notify::load-status and signal callback
9:16 PM Changeset in webkit [56678] by kbr@google.com
  • 2 edits in trunk/WebKit/chromium

2010-03-27 Kenneth Russell <​kbr@google.com>

Reviewed by Darin Fisher.

Use WebKitClient to instantiate WebGraphicsContext3D
​https://bugs.webkit.org/show_bug.cgi?id=36669

  • src/GraphicsContext3D.cpp: (WebCore::GraphicsContext3DInternal::initialize):
7:50 PM Changeset in webkit [56677] by eric@webkit.org
  • 1 edit
    8 deletes in trunk/WebKitTools

2010-03-27 Chris Jerdonek <Chris Jerdonek>

Reviewed by Adam Barth.

In webkitpy/, deleted the /unittests.py files since test-webkitpy
now auto-detects all *_unittest.py files.

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

  • Scripts/webkitpy/common/net/irc/unittests.py: Removed.
  • Scripts/webkitpy/common/system/unittests.py: Removed.
  • Scripts/webkitpy/common/thread/unittests.py: Removed.
  • Scripts/webkitpy/common/unittests.py: Removed.
  • Scripts/webkitpy/python24/unittests.py: Removed.
  • Scripts/webkitpy/style/unittests.py: Removed.
  • Scripts/webkitpy/tool/unittests.py: Removed.
  • Scripts/webkitpy/unittests.py: Removed.
7:32 PM Changeset in webkit [56676] by eric@webkit.org
  • 2 edits in trunk/WebCore

2010-03-27 Dmitry Gorbik <​socket.h@gmail.com>

Reviewed by Pavel Feldman.

Fix the regression caused by r28078: a global variable
definition masks a local one in an inspector console
​https://bugs.webkit.org/show_bug.cgi?id=32442

  • inspector/front-end/InjectedScript.js: (injectedScriptConstructor):
6:56 PM Changeset in webkit [56675] by mitz@apple.com
  • 5 edits in trunk/LayoutTests

More updated results for r56673.

  • platform/chromium-mac/svg/css/getComputedStyle-basic-expected.txt:
  • platform/chromium-win/svg/css/getComputedStyle-basic-expected.txt:
  • platform/gtk/svg/css/getComputedStyle-basic-expected.txt:
  • svg/css/getComputedStyle-basic-expected.txt:
6:07 PM Changeset in webkit [56674] by eric@webkit.org
  • 2 edits in trunk/WebKitTools

2010-03-27 Sergio Villar Senin <​svillar@igalia.com>

Reviewed by Eric Seidel.

Add a CR after printing didFinishDocumentLoadForFrame

[GTK] Improve reporting of frame loader callbacks in DRT
​https://bugs.webkit.org/show_bug.cgi?id=36454

  • DumpRenderTree/gtk/DumpRenderTree.cpp: (webViewLoadFinished):
5:59 PM Changeset in webkit [56673] by mitz@apple.com
  • 12 edits in trunk

WebCore: <rdar://problem/7801598> REGRESSION (r56182): iWeb shadow drawn around image rectangle, not around opaque part of the image
​https://bugs.webkit.org/show_bug.cgi?id=36700

Reviewed by Darin Adler.

  • css/CSSPrimitiveValue.cpp:

(WebCore::CSSPrimitiveValue::createColor): Reverted to returning an RGB color value rather than an
identifier for the transparent color.

LayoutTests: Updated results and one test for
<rdar://problem/7801598> REGRESSION (r56182): iWeb shadow drawn around image rectangle, not around opaque part of the image
​https://bugs.webkit.org/show_bug.cgi?id=36700

Reviewed by Darin Adler.

  • fast/css/getComputedStyle/computed-style-expected.txt:
  • fast/css/invalid-import-rule-insertion.html:
  • http/tests/mime/standard-mode-does-not-load-stylesheet-with-text-plain-and-css-extension-expected.txt:
  • http/tests/mime/standard-mode-does-not-load-stylesheet-with-text-plain-expected.txt:
  • http/tests/security/cross-origin-css-expected.txt:
  • http/tests/security/cross-origin-css-in-xml-expected.txt:
  • platform/mac/fast/css/getComputedStyle/computed-style-expected.txt:
  • platform/qt/fast/css/getComputedStyle/computed-style-expected.txt:
  • platform/win/fast/css/getComputedStyle/computed-style-expected.txt:
  • svg/css/getComputedStyle-basic-expected.txt:
4:29 PM Changeset in webkit [56672] by Chris Jerdonek
  • 1 edit
    1 move in trunk/WebKitTools

Rubber-stamped by Adam Barth.

Fixed the name of a unit test file in webkitpy.

  • Scripts/webkitpy/tool/steps/updatechangelogswithreview_unittest.py: Copied from WebKitTools/Scripts/webkitpy/tool/steps/updatechangelogswithreview_unittests.py.
  • Scripts/webkitpy/tool/steps/updatechangelogswithreview_unittests.py: Removed.
3:56 PM Changeset in webkit [56671] by Chris Jerdonek
  • 2 edits
    3 adds in trunk/WebKitTools

2010-03-26 Chris Jerdonek <Chris Jerdonek>

Reviewed by Adam Barth.

The test-webkitpy script now automatically detects all unit-test
files in webkitpy/. This lets us eliminate the need to have and
maintain all of the unittests.py files.

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

  • Scripts/test-webkitpy:
    • Replaced the "from webkitpy.unittests import *" with a call to webkitpy.test.main.Tester().run_tests().
  • Scripts/webkitpy/test/init.py: Copied from WebKitTools/QueueStatusServer/filters/init.py.
    • Required file for the new webkitpy/test/ directory.
  • Scripts/webkitpy/test/main.py: Added.
    • Added a Tester class that contains the following methods:
      • _find_unittest_files() to detect all the unit-test files.
      • _modules_from_paths() to convert the paths to the unit-test files to fully-qualified module names.
      • run_tests() which calls the above two methods and then passes the module names to Python's unittest module.
2:42 PM Changeset in webkit [56670] by Joseph Pecoraro
  • 6 edits in trunk/WebCore

2010-03-26 Joseph Pecoraro <Joseph Pecoraro>

Reviewed by Pavel Feldman.

Web Inspector: Converge Inspector ids to be long, not ints
​https://bugs.webkit.org/show_bug.cgi?id=36706

Part 1: Converge on long for callIds, nodeIds, and other Ids.

  • inspector/InspectorController.cpp:
  • inspector/InspectorController.h:
  • inspector/InspectorDOMAgent.cpp:
  • inspector/InspectorFrontend.cpp:
  • inspector/InspectorFrontend.h:
12:51 PM Changeset in webkit [56669] by mrowe@apple.com
  • 2 edits in trunk/LayoutTests

Fix the newly-added test to not trip over a bug that causes some subsequent tests to fail.

Rubber-stamped by Dan Bernstein.

  • fast/dom/script-tests/setPrimitiveValue-exceptions.js:
12:31 PM Changeset in webkit [56668] by robert@webkit.org
  • 5 edits in trunk

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

Reviewed by Simon Fraser.

[Qt/Win] Add support to unix and windows NS plugin for executing scripts on setWindow.

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

  • DumpRenderTree/unix/TestNetscapePlugin/TestNetscapePlugin.cpp: (webkit_test_plugin_new_instance): Add onSetWindow. (webkit_test_plugin_destroy_instance): Add onSetWindow. (webkit_test_plugin_set_window): Add onSetWindow.
  • DumpRenderTree/win/TestNetscapePlugin/main.cpp: (NPP_New): Add onSetWindow. (NPP_Destroy): Add onSetWindow. (NPP_SetWindow): Add onSetWindow.

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

Unreviewed skip failing test.

[Qt] Skip test added in r56646. Tracked by ​https://bugs.webkit.org/show_bug.cgi?id=36702

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

  • platform/qt/Skipped: Add plugins/reentrant-update-widget-positions.html
12:24 PM Changeset in webkit [56667] by Darin Adler
  • 2 edits in trunk/WebKit/mac
  • Misc/WebNSFileManagerExtras.m:

(-[NSFileManager _webkit_pathWithUniqueFilenameForPath:]):
Removed stray "!". How did that get in there?

12:20 PM Changeset in webkit [56666] by Darin Adler
  • 3 edits in trunk/WebKit/mac

WebKit NSFileManager methods don't handle broken symlinks correctly.
Part of <rdar://problem/7574046>.

Reviewed by Mark Rowe.

  • Misc/WebNSFileManagerExtras.h: Removed unused defines and methods.
  • Misc/WebNSFileManagerExtras.m: Removed unused methods.

(fileExists): Added. For use instead of fileExistsAtPath: for cases where we'd like
to treat a broken symlink as a file that does indeed exist.
(-[NSFileManager _webkit_pathWithUniqueFilenameForPath:]): Use fileExists.

12:17 PM Changeset in webkit [56665] by pfeldman@chromium.org
  • 2 edits
    1 add in trunk

2010-03-27 Pavel Feldman <​pfeldman@chromium.org>

Not reviewed: added missing test resource file.

  • inspector/resources/elements-panel-styles.css: Added.
11:24 AM Changeset in webkit [56664] by pfeldman@chromium.org
  • 7 edits
    2 adds in trunk

2010-03-27 Pavel Feldman <​pfeldman@chromium.org>

Reviewed by Timothy Hatcher.

Web Inspector: add layout test for styles panel.

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

Test: inspector/elements-panel-styles.html

  • inspector/front-end/inspector.css: (#elements-content): (#elements-sidebar):
  • inspector/front-end/inspector.js: (WebInspector.displayNameForURL):
  • inspector/front-end/utilities.js: (String.prototype.trimURL):
11:21 AM Changeset in webkit [56663] by mrowe@apple.com
  • 3 edits
    3 adds in trunk

CSSPrimitiveValue::setFloatValue/setStringValue should throw an exception if passed an incorrect unit type.
<​http://webkit.org/b/36680> / <rdar://problem/7801016>

Reviewed by Darin Adler.

WebCore:

Test: fast/dom/setPrimitiveValue-exceptions.html

  • css/CSSPrimitiveValue.cpp:

(WebCore::CSSPrimitiveValue::setFloatValue): Throw an INVALID_ACCESS_ERR if either of our type or the passed-in
type is not a numeric type.
(WebCore::CSSPrimitiveValue::setStringValue): Throw an INVALID_ACCESS_ERR if either of our type or the passed-in
type is not a string type.

LayoutTests:

  • fast/dom/script-tests/setPrimitiveValue-exceptions.js: Added.
  • fast/dom/setPrimitiveValue-exceptions-expected.txt: Added.
  • fast/dom/setPrimitiveValue-exceptions.html: Copied from LayoutTests/fast/dom/event-attribute-availability.html.
6:06 AM Changeset in webkit [56662] by robert@webkit.org
  • 2 edits in trunk/WebCore

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

Reviewed by nobody, fix typo in previous commit.

Allow plugins implemented by the application, such as mimetype 'x-qt-plugin',

when pluginsEnabled is false

Fix parentheses typo in r56661. This happened while rebasing and was not present
in the reviewed patch, so committing unreviewed.

​https://bugs.webkit.org/attachment.cgi?id=49515

  • loader/FrameLoader.cpp: (WebCore::FrameLoader::requestObject):
5:46 AM Changeset in webkit [56661] by robert@webkit.org
  • 18 edits in trunk

2010-03-26 Robert Hogan <​robert@roberthogan.net>

Reviewed by Simon Hausmann.

Allow plugins implemented by the application, such as mimetype 'x-qt-plugin',

when pluginsEnabled is false.

The purpose of disabling plugins is to prevent the execution of third-party code
that may be untrustworthy. Qt plugins are implemented by the client rather than
loaded from an external source, so the client should have the opportunity to
consider them separately from other plugins.

Add a function MimeTypeRegistry::isApplicationPluginMIMEType() that WebKit
uses in conjunction with arePluginsEnabled() to determine if it should attempt
to load a plugin. If isApplicationPluginMIMEType() returns true, WebKit will load
the plugin even if arePluginsEnabled() is false.

Currently, only Qt has application-implemented plugins: these use the mimetype
'x-qt-plugin' and 'x-qt-styled-widget'. This patch permits Qt clients'
reimplementation of QWebPage::createPlugin() to decide whether or not
to create a Qt plugin, even when arePluginsEnabled is false.

For all platforms apart from Qt, isApplicationPluginMIMEType() returns false.

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

Test: plugins/application-plugin-plugins-disabled.html

  • loader/FrameLoader.cpp: (WebCore::FrameLoader::requestObject):
  • platform/MIMETypeRegistry.h:
  • platform/brew/MIMETypeRegistryBrew.cpp: (WebCore::MIMETypeRegistry::isApplicationPluginMIMEType):
  • platform/chromium/MIMETypeRegistryChromium.cpp: (WebCore::MIMETypeRegistry::isApplicationPluginMIMEType):
  • platform/gtk/MIMETypeRegistryGtk.cpp: (WebCore::MIMETypeRegistry::isApplicationPluginMIMEType):
  • platform/haiku/MIMETypeRegistryHaiku.cpp: (WebCore::MIMETypeRegistry::isApplicationPluginMIMEType):
  • platform/mac/MIMETypeRegistryMac.mm: (WebCore::MIMETypeRegistry::isApplicationPluginMIMEType):
  • platform/qt/MIMETypeRegistryQt.cpp: (WebCore::MIMETypeRegistry::isApplicationPluginMIMEType):
  • platform/win/MIMETypeRegistryWin.cpp: (WebCore::MIMETypeRegistry::isApplicationPluginMIMEType):
  • platform/wince/MIMETypeRegistryWince.cpp: (WebCore::MIMETypeRegistry::isApplicationPluginMIMEType):
  • platform/wx/MimeTypeRegistryWx.cpp: (WebCore::MIMETypeRegistry::isApplicationPluginMIMEType):

2010-03-26 Robert Hogan <​robert@roberthogan.net>

Reviewed by Simon Hausmann.

Allow plugins implemented by the application, such as mimetype 'x-qt-plugin',

when pluginsEnabled is false.

For all platforms apart from Qt, isApplicationPluginMIMEType() returns false.

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

  • platform/qt/plugins/application-plugin-plugins-disabled-expected.txt: Added.
  • plugins/application-plugin-plugins-disabled-expected.txt: Added.
  • plugins/application-plugin-plugins-disabled.html: Added.

2010-03-26 Robert Hogan <​robert@roberthogan.net>

Reviewed by Simon Hausmann.

Allow plugins implemented by the application, such as mimetype 'x-qt-plugin',

when pluginsEnabled is false.

Add support for LayoutTestController.WebKitPluginsEnabled

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

  • DumpRenderTree/gtk/DumpRenderTree.cpp: (resetDefaultsToConsistentValues):
  • DumpRenderTree/gtk/LayoutTestControllerGtk.cpp: (copyWebSettingKey):
  • DumpRenderTree/qt/DumpRenderTreeQt.cpp: (WebCore::WebPage::resetSettings):
  • DumpRenderTree/qt/LayoutTestControllerQt.cpp: (LayoutTestController::overridePreference):
5:36 AM Changeset in webkit [56660] by yael.aharon@nokia.com
  • 2 edits in trunk/LayoutTests

Unreviewed.
Skip HTMLProgressElement tests on Windows.

  • platform/win/Skipped:
2:06 AM Changeset in webkit [56659] by mitz@apple.com
  • 2 edits in trunk/WebCore

Fix Acid3 text-shadow rendering regression introduced in r56597.

Reviewed by Mark Rowe.

  • platform/graphics/mac/FontMac.mm:

(WebCore::Font::drawGlyphs): Revert to using the simple shadow logic for zero-blur shadows,
except for color bitmap fonts.

Mar 26, 2010:

10:44 PM Changeset in webkit [56658] by abarth@webkit.org
  • 10 edits
    2 copies in trunk/WebKitTools

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

Reviewed by Eric Seidel.

Add some basic IRC commands to sheriffbot
​https://bugs.webkit.org/show_bug.cgi?id=36684

Adds support for sheriffbot to respond to a "hi" command and a
"last-green-revision" command. It's lame that we're rebuilding
MultiCommandTool, but as discussed in person we'll intergrate the two
once we see what the requirements are.

  • Scripts/webkitpy/tool/bot/irc_command.py: Added.
  • Scripts/webkitpy/tool/bot/queueengine.py:
  • Scripts/webkitpy/tool/bot/queueengine_unittest.py:
  • Scripts/webkitpy/tool/bot/sheriffircbot.py:
  • Scripts/webkitpy/tool/bot/sheriffircbot_unittest.py: Added.
  • Scripts/webkitpy/tool/commands/queues.py:
  • Scripts/webkitpy/tool/commands/queuestest.py:
  • Scripts/webkitpy/tool/commands/sheriffbot.py:
  • Scripts/webkitpy/tool/main.py:
  • Scripts/webkitpy/tool/mocktool.py:
  • Scripts/webkitpy/tool/unittests.py:
10:42 PM Changeset in webkit [56657] by abarth@webkit.org
  • 2 edits in trunk/WebKitTools

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

Unreviewed. More errors in the IRC module. I have no idea how to test
ircbot.py, which is too bad. Hopefully we've abstracted it away enough
that we don't have to touch it very much after this patch.

  • Scripts/webkitpy/common/net/irc/ircbot.py:
10:37 PM Changeset in webkit [56656] by abarth@webkit.org
  • 3 edits
    1 copy in trunk/WebKitTools

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

Unreviewed. Add a missing "_".

  • Scripts/webkitpy/common/net/irc/ircproxy.py:
  • Scripts/webkitpy/common/net/irc/ircproxy_unittest.py: Added.
  • Scripts/webkitpy/common/net/irc/unittests.py:
10:08 PM Changeset in webkit [56655] by dbates@webkit.org
  • 7 edits
    68 adds in trunk

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

Reviewed by Beth Dakin.

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

As per sections 14.2 (​http://www.w3.org/TR/CSS2/colors.html#background)
and 17.5.1 (​http://www.w3.org/TR/CSS2/tables.html#table-layers) of the
CSS 2.1 spec., the background-image of a table-row should span the
entire area of the table-row (i.e cover all the <td>'s). Moreover,
fixes background image painting with respect to row-groups, columns
and column-groups (*).

Currently, the background-image of a table row, row-group, and column-group
is inherited by the table cells. This directly contradicts section 14.2.
Instead, the background-image should cover the entire area of the table row,
row-group, column, or column-group, respectively as described in section 17.5.1.

(*) A follow up patch to compute the absolute content box of a column, and
column-group is needed so that background image positioning for these table
elements works. See bug #36104.

Tests: fast/table/table-background.html

fast/table/table-cell-border-draws-on-top-of-col-background.html
fast/table/table-cell-border-draws-on-top-of-col-group-background.html
fast/table/table-cell-border-draws-on-top-of-row-background.html
fast/table/table-cell-border-draws-on-top-of-row-group-background.html
fast/table/table-cell-overrides-row-background.html
fast/table/table-col-background.html
fast/table/table-col-group-background.html
fast/table/table-col-group-col-span-background.html
fast/table/table-col-group-span-background.html
fast/table/table-col-span-background.html
fast/table/table-row-background-left-50px.html
fast/table/table-row-background-right-100percent.html
fast/table/table-row-background-right-50px.html
fast/table/table-row-background.html
fast/table/table-row-group-background-positioned.html
fast/table/table-row-group-background.html

  • rendering/RenderBoxModelObject.cpp: (WebCore::RenderBoxModelObject::paintFillLayerExtended): (WebCore::RenderBoxModelObject::calculateBackgroundImageGeometry): Modified to call RenderObject::adjustBackgroundImagePosition.
  • rendering/RenderBoxModelObject.h:
  • rendering/RenderObject.h: (WebCore::RenderObject::adjustBackgroundImagePosition): Added.
  • rendering/RenderTableCell.cpp: (WebCore::RenderTableCell::adjustBackgroundImagePosition): Added.
  • rendering/RenderTableCell.h:

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

Reviewed by Beth Dakin.

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

Tests that background-image support for table, table-cell, table-row,
row-group, table-column and column-group (*) conform to section 17.5.1
of the CSS 2.1 spec.

(*) A follow up patch to compute the absolute content box of a column, and
column-group is needed so that background image positioning for these table
elements works. See bug #36104.

  • fast/table/table-background.html: Added.
  • fast/table/table-cell-border-draws-on-top-of-col-background.html: Added.
  • fast/table/table-cell-border-draws-on-top-of-col-group-background.html: Added.
  • fast/table/table-cell-border-draws-on-top-of-row-background.html: Added.
  • fast/table/table-cell-border-draws-on-top-of-row-group-background.html: Added.
  • fast/table/table-cell-overrides-row-background.html: Added.
  • fast/table/table-col-background.html: Added.
  • fast/table/table-col-group-background.html: Added.
  • fast/table/table-col-group-col-span-background.html: Added.
  • fast/table/table-col-group-span-background.html: Added.
  • fast/table/table-col-span-background.html: Added.
  • fast/table/table-row-background-left-50px.html: Added.
  • fast/table/table-row-background-right-100percent.html: Added.
  • fast/table/table-row-background-right-50px.html: Added.
  • fast/table/table-row-background.html: Added.
  • fast/table/table-row-group-background-positioned.html: Added.
  • fast/table/table-row-group-background.html: Added.
  • platform/mac/fast/table/table-background-expected.checksum: Added.
  • platform/mac/fast/table/table-background-expected.png: Added.
  • platform/mac/fast/table/table-background-expected.txt: Added.
  • platform/mac/fast/table/table-cell-border-draws-on-top-of-col-background-expected.checksum: Added.
  • platform/mac/fast/table/table-cell-border-draws-on-top-of-col-background-expected.png: Added.
  • platform/mac/fast/table/table-cell-border-draws-on-top-of-col-background-expected.txt: Added.
  • platform/mac/fast/table/table-cell-border-draws-on-top-of-col-group-background-expected.checksum: Added.
  • platform/mac/fast/table/table-cell-border-draws-on-top-of-col-group-background-expected.png: Added.
  • platform/mac/fast/table/table-cell-border-draws-on-top-of-col-group-background-expected.txt: Added.
  • platform/mac/fast/table/table-cell-border-draws-on-top-of-row-background-expected.checksum: Added.
  • platform/mac/fast/table/table-cell-border-draws-on-top-of-row-background-expected.png: Added.
  • platform/mac/fast/table/table-cell-border-draws-on-top-of-row-background-expected.txt: Added.
  • platform/mac/fast/table/table-cell-border-draws-on-top-of-row-group-background-expected.checksum: Added.
  • platform/mac/fast/table/table-cell-border-draws-on-top-of-row-group-background-expected.png: Added.
  • platform/mac/fast/table/table-cell-border-draws-on-top-of-row-group-background-expected.txt: Added.
  • platform/mac/fast/table/table-cell-overrides-row-background-expected.checksum: Added.
  • platform/mac/fast/table/table-cell-overrides-row-background-expected.png: Added.
  • platform/mac/fast/table/table-cell-overrides-row-background-expected.txt: Added.
  • platform/mac/fast/table/table-col-background-expected.checksum: Added.
  • platform/mac/fast/table/table-col-background-expected.png: Added.
  • platform/mac/fast/table/table-col-background-expected.txt: Added.
  • platform/mac/fast/table/table-col-group-background-expected.checksum: Added.
  • platform/mac/fast/table/table-col-group-background-expected.png: Added.
  • platform/mac/fast/table/table-col-group-background-expected.txt: Added.
  • platform/mac/fast/table/table-col-group-col-span-background-expected.checksum: Added.
  • platform/mac/fast/table/table-col-group-col-span-background-expected.png: Added.
  • platform/mac/fast/table/table-col-group-col-span-background-expected.txt: Added.
  • platform/mac/fast/table/table-col-group-span-background-expected.checksum: Added.
  • platform/mac/fast/table/table-col-group-span-background-expected.png: Added.
  • platform/mac/fast/table/table-col-group-span-background-expected.txt: Added.
  • platform/mac/fast/table/table-col-span-background-expected.checksum: Added.
  • platform/mac/fast/table/table-col-span-background-expected.png: Added.
  • platform/mac/fast/table/table-col-span-background-expected.txt: Added.
  • platform/mac/fast/table/table-row-background-expected.checksum: Added.
  • platform/mac/fast/table/table-row-background-expected.png: Added.
  • platform/mac/fast/table/table-row-background-expected.txt: Added.
  • platform/mac/fast/table/table-row-background-left-50px-expected.checksum: Added.
  • platform/mac/fast/table/table-row-background-left-50px-expected.png: Added.
  • platform/mac/fast/table/table-row-background-left-50px-expected.txt: Added.
  • platform/mac/fast/table/table-row-background-right-100percent-expected.checksum: Added.
  • platform/mac/fast/table/table-row-background-right-100percent-expected.png: Added.
  • platform/mac/fast/table/table-row-background-right-100percent-expected.txt: Added.
  • platform/mac/fast/table/table-row-background-right-50px-expected.checksum: Added.
  • platform/mac/fast/table/table-row-background-right-50px-expected.png: Added.
  • platform/mac/fast/table/table-row-background-right-50px-expected.txt: Added.
  • platform/mac/fast/table/table-row-group-background-expected.checksum: Added.
  • platform/mac/fast/table/table-row-group-background-expected.png: Added.
  • platform/mac/fast/table/table-row-group-background-expected.txt: Added.
  • platform/mac/fast/table/table-row-group-background-positioned-expected.checksum: Added.
  • platform/mac/fast/table/table-row-group-background-positioned-expected.png: Added.
  • platform/mac/fast/table/table-row-group-background-positioned-expected.txt: Added.
10:00 PM Changeset in webkit [56654] by abarth@webkit.org
  • 2 edits in trunk/WebKitTools

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

Unreviewed. Fix a the commit queue after my recent change.

  • Scripts/webkitpy/tool/bot/queueengine.py:
9:31 PM Changeset in webkit [56653] by Laszlo Gombos
  • 2 edits in trunk/WebCore

2010-03-26 Laszlo Gombos <Laszlo Gombos>

Unreviewed, minor build fix.

[Gtk] Guard defined but not used function

Eliminate the "defined but not used" warning after r52684.
No new tests as there is no new functionality.

  • platform/ContextMenu.cpp:
7:38 PM Changeset in webkit [56652] by dpranke@chromium.org
  • 2 edits in trunk/WebKitTools

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

Unreviewed fix.

Fix the return value for port/base.diff_image (changed from 1/0 to
True/False in bug 34826.

  • Scripts/webkitpy/layout_tests/port/base.py:
7:15 PM Changeset in webkit [56651] by abarth@webkit.org
  • 5 edits
    2 adds in trunk

2010-03-26 Justin Schuh <​jschuh@chromium.org>

Reviewed by Adam Barth.

Security: iFrame.src accepts JavaScript URL via nodeValue or textContent
​https://bugs.webkit.org/show_bug.cgi?id=36502

Overrode inherited nodeValue and textContent in Attr.idl so they proxy
to value, which performs a security check.

  • http/tests/security/xss-DENIED-iframe-src-alias-expected.txt: Added.
  • http/tests/security/xss-DENIED-iframe-src-alias.html: Added.

2010-03-26 Justin Schuh <​jschuh@chromium.org>

Reviewed by Adam Barth.

Security: iFrame.src accepts JavaScript URL via nodeValue or textContent
​https://bugs.webkit.org/show_bug.cgi?id=36502

Overrode inherited nodeValue and textContent in Attr.idl so they proxy
to value, which performs a security check.

Test: http/tests/security/xss-DENIED-iframe-src-alias.html

  • bindings/js/JSAttrCustom.cpp: (WebCore::JSAttr::nodeValue): (WebCore::JSAttr::setNodeValue): (WebCore::JSAttr::textContent): (WebCore::JSAttr::setTextContent):
  • bindings/v8/custom/V8AttrCustom.cpp: (WebCore::V8Attr::nodeValueAccessorSetter): (WebCore::V8Attr::nodeValueAccessorGetter): (WebCore::V8Attr::textContentAccessorSetter): (WebCore::V8Attr::textContentAccessorGetter):
  • dom/Attr.idl:
6:20 PM Changeset in webkit [56650] by eric.carlson@apple.com
  • 12 edits in trunk/WebCore

2010-03-26 Eric Carlson <​eric.carlson@apple.com>

<rdar://problem/7320584>
​https://bugs.webkit.org/show_bug.cgi?id=36681
"new Audio()" doesn't work with plug-in backed media engine.

Don't require plug-in backed <video> and <audio> elements to be in the document to play.

  • html/HTMLMediaElement.cpp: (WebCore::HTMLMediaElement::HTMLMediaElement): Initialize m_proxyWidget (WebCore::HTMLMediaElement::createRenderer): Set renderer widget if it has already been allocated. (WebCore::HTMLMediaElement::scheduleLoad): Call createMediaPlayerProxy to make sure we have a plug-in proxy. (WebCore::HTMLMediaElement::loadResource): Ditto. (WebCore::HTMLMediaElement::allocateMediaPlayerIfNecessary): Allocate media player. (WebCore::HTMLMediaElement::setMediaPlayerProxy): Call allocateMediaPlayerIfNecessary. (WebCore::HTMLMediaElement::getPluginProxyParams): New, return the url and params vectors. (WebCore::HTMLMediaElement::finishParsingChildren): Call createMediaPlayerProxy. (WebCore::HTMLMediaElement::createMediaPlayerProxy): New, allocate the proxy widget if necessary.
  • html/HTMLMediaElement.h:
  • loader/EmptyClients.h: (WebCore::EmptyFrameLoaderClient::createMediaPlayerProxyPlugin): Add declaration.
  • loader/FrameLoader.cpp: (WebCore::toPlugInElement): Don't allow an audio or video element to be cast to a plug-in element. (WebCore::FrameLoader::loadPlugin): (WebCore::FrameLoader::loadMediaPlayerProxyPlugin): New, allocate a media player proxy plug-in.
  • loader/FrameLoader.h: Declare loadMediaPlayerProxyPlugin.
  • loader/FrameLoaderClient.h: Declare createMediaPlayerProxyPlugin.
  • rendering/RenderEmbeddedObject.cpp: (WebCore::RenderEmbeddedObject::updateWidget): Call loadMediaPlayerProxyPlugin when passed an audio or video node.
  • rendering/RenderLayerCompositor.cpp: (WebCore::RenderLayerCompositor::requiresCompositingForVideo):
  • rendering/RenderVideo.cpp: (WebCore::RenderVideo::RenderVideo): Get default width and height from defaultSize. (WebCore::RenderVideo::defaultSize): New, return the spec defined default width and height.
  • rendering/RenderVideo.h:
6:11 PM Changeset in webkit [56649] by dpranke@chromium.org
  • 134 edits
    2 adds in trunk/LayoutTests

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

Reviewed by David Levin.

Update a bunch of incorrect image checksums, and suppress all of the
remaining IMAGE failures in platform/mac/test_expectations.txt.

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

  • platform/mac-leopard/compositing/geometry/root-layer-update-expected.checksum:
  • platform/mac-leopard/compositing/overflow/overflow-positioning-expected.checksum:
  • platform/mac-leopard/compositing/repaint/become-overlay-composited-layer-expected.checksum:
  • platform/mac-leopard/compositing/repaint/layer-repaint-rects-expected.checksum:
  • platform/mac-leopard/media/video-zoom-controls-expected.checksum:
  • platform/mac/animations/3d/change-transform-in-end-event-expected.checksum:
  • platform/mac/animations/3d/state-at-end-event-transform-expected.checksum:
  • platform/mac/animations/state-at-end-event-expected.checksum:
  • platform/mac/compositing/animation/state-at-end-event-transform-layer-expected.checksum:
  • platform/mac/compositing/color-matching/pdf-image-match-expected.checksum:
  • platform/mac/compositing/compositing-visible-descendant-expected.checksum:
  • platform/mac/compositing/generated-content-expected.checksum:
  • platform/mac/compositing/geometry/abs-position-inside-opacity-expected.checksum:
  • platform/mac/compositing/geometry/fixed-position-expected.checksum:
  • platform/mac/compositing/geometry/horizontal-scroll-composited-expected.checksum:
  • platform/mac/compositing/geometry/outline-change-expected.checksum:
  • platform/mac/compositing/geometry/partial-layout-update-expected.checksum:
  • platform/mac/compositing/geometry/root-layer-update-expected.checksum:
  • platform/mac/compositing/geometry/transfrom-origin-on-zero-size-layer-expected.checksum:
  • platform/mac/compositing/masks/direct-image-mask-expected.checksum:
  • platform/mac/compositing/masks/masked-ancestor-expected.checksum:
  • platform/mac/compositing/masks/multiple-masks-expected.checksum:
  • platform/mac/compositing/masks/simple-composited-mask-expected.checksum:
  • platform/mac/compositing/overflow/ancestor-overflow-expected.checksum:
  • platform/mac/compositing/overflow/fixed-position-ancestor-clip-expected.checksum:
  • platform/mac/compositing/overflow/overflow-positioning-expected.checksum:
  • platform/mac/compositing/overflow/parent-overflow-expected.checksum:
  • platform/mac/compositing/overflow/scrollbar-painting-expected.checksum:
  • platform/mac/compositing/reflections/reflection-in-composited-expected.checksum:
  • platform/mac/compositing/reflections/reflection-ordering-expected.checksum:
  • platform/mac/compositing/reflections/simple-composited-reflections-expected.checksum:
  • platform/mac/compositing/repaint/become-overlay-composited-layer-expected.checksum:
  • platform/mac/compositing/repaint/content-into-overflow-expected.checksum:
  • platform/mac/compositing/repaint/layer-repaint-expected.checksum:
  • platform/mac/compositing/repaint/layer-repaint-rects-expected.checksum:
  • platform/mac/compositing/repaint/opacity-between-absolute-expected.checksum:
  • platform/mac/compositing/repaint/opacity-between-absolute2-expected.checksum:
  • platform/mac/compositing/repaint/overflow-into-content-expected.checksum:
  • platform/mac/compositing/sibling-positioning-expected.checksum:
  • platform/mac/compositing/transitions/singular-scale-transition-expected.checksum:
  • platform/mac/css2.1/t0905-c5525-flthw-00-c-g-expected.checksum:
  • platform/mac/css2.1/t0905-c5526-flthw-00-c-g-expected.checksum:
  • platform/mac/editing/deleting/delete-to-select-table-expected.checksum:
  • platform/mac/editing/deleting/type-delete-after-quote-expected.checksum:
  • platform/mac/editing/inserting/5418891-expected.checksum:
  • platform/mac/editing/selection/4960116-expected.checksum:
  • platform/mac/editing/selection/caret-rtl-2-expected.checksum:
  • platform/mac/editing/selection/extend-by-word-002-expected.checksum:
  • platform/mac/editing/selection/select-all-002-expected.checksum:
  • platform/mac/editing/selection/select-text-overflow-ellipsis-expected.checksum:
  • platform/mac/editing/selection/transformed-selection-rects-expected.checksum:
  • platform/mac/fast/backgrounds/background-clip-text-expected.checksum:
  • platform/mac/fast/backgrounds/quirks-mode-line-box-backgrounds-expected.checksum:
  • platform/mac/fast/backgrounds/repeat/mask-negative-offset-repeat-expected.checksum:
  • platform/mac/fast/backgrounds/size/backgroundSize15-expected.checksum:
  • platform/mac/fast/backgrounds/size/backgroundSize16-expected.checksum:
  • platform/mac/fast/backgrounds/size/contain-and-cover-expected.checksum: Added.
  • platform/mac/fast/backgrounds/svg-as-background-3-expected.checksum:
  • platform/mac/fast/backgrounds/svg-as-background-5-expected.checksum:
  • platform/mac/fast/backgrounds/svg-as-background-6-expected.checksum:
  • platform/mac/fast/backgrounds/svg-as-mask-expected.checksum:
  • platform/mac/fast/block/float/avoidance-percent-width-compat-expected.checksum:
  • platform/mac/fast/block/float/avoidance-percent-width-strict-expected.checksum:
  • platform/mac/fast/block/margin-collapse/103-expected.checksum:
  • platform/mac/fast/box-shadow/basic-shadows-expected.checksum:
  • platform/mac/fast/box-shadow/transform-fringing-expected.checksum:
  • platform/mac/fast/css/apple-prefix-expected.checksum:
  • platform/mac/fast/css/color-correction-backgrounds-and-text-expected.checksum:
  • platform/mac/fast/css/color-correction-expected.checksum:
  • platform/mac/fast/css/color-correction-on-background-image-expected.checksum:
  • platform/mac/fast/css/color-correction-on-backgrounds-expected.checksum:
  • platform/mac/fast/css/color-correction-on-box-shadow-expected.checksum:
  • platform/mac/fast/css/color-correction-on-text-expected.checksum:
  • platform/mac/fast/css/color-correction-on-text-shadow-expected.checksum:
  • platform/mac/fast/css/color-correction-untagged-images-expected.checksum:
  • platform/mac/fast/css/namespaces/namespaces-comments-expected.checksum:
  • platform/mac/fast/css/namespaces/namespaces-empty-expected.checksum:
  • platform/mac/fast/css/namespaces/namespaces-escapes-expected.checksum:
  • platform/mac/fast/css/namespaces/namespaces-invalid-at-expected.checksum:
  • platform/mac/fast/css/variables/misplaced-import-test-expected.checksum:
  • platform/mac/fast/encoding/invalid-UTF-8-expected.checksum:
  • platform/mac/fast/events/standalone-image-drag-to-editable-expected.checksum:
  • platform/mac/fast/fast-mobile-scrolling/fixed-position-element-expected.checksum:
  • platform/mac/fast/fast-mobile-scrolling/no-fixed-position-elements-expected.checksum:
  • platform/mac/fast/forms/001-expected.checksum:
  • platform/mac/fast/forms/menulist-clip-expected.checksum:
  • platform/mac/fast/forms/menulist-narrow-width-expected.checksum:
  • platform/mac/fast/forms/menulist-option-wrap-expected.checksum:
  • platform/mac/fast/forms/menulist-restrict-line-height-expected.checksum:
  • platform/mac/fast/forms/menulist-style-color-expected.checksum:
  • platform/mac/fast/forms/select-baseline-expected.checksum:
  • platform/mac/fast/forms/textarea-scrollbar-expected.checksum:
  • platform/mac/fast/frames/viewsource-attribute-expected.checksum:
  • platform/mac/fast/inline/long-wrapped-line-expected.checksum:
  • platform/mac/fast/lists/anonymous-items-expected.checksum:
  • platform/mac/fast/lists/markers-in-selection-expected.checksum:
  • platform/mac/fast/multicol/client-rects-expected.checksum: Added.
  • platform/mac/fast/multicol/float-multicol-expected.checksum:
  • platform/mac/fast/overflow/line-clamp-expected.checksum:
  • platform/mac/fast/overflow/position-fixed-transform-clipping-expected.checksum:
  • platform/mac/fast/reflections/abs-position-in-reflection-expected.checksum:
  • platform/mac/fast/reflections/opacity-reflection-transform-expected.checksum:
  • platform/mac/fast/repaint/backgroundSizeRepaint-expected.checksum:
  • platform/mac/fast/repaint/box-shadow-h-expected.checksum:
  • platform/mac/fast/repaint/box-shadow-v-expected.checksum:
  • platform/mac/fast/repaint/line-in-scrolled-clipped-block-expected.checksum:
  • platform/mac/fast/repaint/no-caret-repaint-in-non-content-editable-element-expected.checksum:
  • platform/mac/fast/repaint/rel-positioned-inline-with-overflow-expected.checksum:
  • platform/mac/fast/repaint/selection-gap-overflow-scroll-2-expected.checksum:
  • platform/mac/fast/repaint/subtree-layoutstate-transform-expected.checksum:
  • platform/mac/fast/repaint/table-outer-border-expected.checksum:
  • platform/mac/fast/table/fixed-table-with-percent-inside-percent-table-expected.checksum:
  • platform/mac/fast/table/fixed-table-with-percent-width-inside-auto-table-expected.checksum:
  • platform/mac/fast/table/fixed-table-with-percent-width-inside-div-expected.checksum:
  • platform/mac/fast/table/fixed-table-with-percent-width-inside-extra-large-div-expected.checksum:
  • platform/mac/fast/table/fixed-table-with-percent-width-inside-fixed-width-table-expected.checksum:
  • platform/mac/fast/table/fixed-table-with-small-percent-width-expected.checksum:
  • platform/mac/fast/text/atsui-partial-selection-expected.checksum:
  • platform/mac/fast/text/capitalize-boundaries-expected.checksum:
  • platform/mac/fast/text/drawBidiText-expected.checksum:
  • platform/mac/fast/text/line-breaking-with-kerning-expected.checksum:
  • platform/mac/fast/text/sticky-typesetting-features-expected.checksum:
  • platform/mac/fast/transforms/shadows-expected.checksum:
  • platform/mac/http/tests/misc/slow-loading-mask-expected.checksum:
  • platform/mac/media/video-zoom-controls-expected.checksum:
  • platform/mac/plugins/embed-attributes-style-expected.checksum:
  • platform/mac/plugins/qt-qwidget-plugin-expected.checksum:
  • platform/mac/tables/mozilla/bugs/bug106158-1-expected.checksum:
  • platform/mac/tables/mozilla/bugs/bug106158-2-expected.checksum:
  • platform/mac/tables/mozilla/bugs/bug34176-expected.checksum:
  • platform/mac/tables/mozilla/bugs/bug6674-expected.checksum:
  • platform/mac/test_expectations.txt:
  • platform/mac/transforms/2d/transform-fixed-container-expected.checksum:
  • platform/mac/transitions/move-after-transition-expected.checksum:
  • platform/mac/transitions/transition-end-event-rendering-expected.checksum:
5:15 PM Changeset in webkit [56648] by dpranke@chromium.org
  • 3 edits
    1 add in trunk

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

Reviewed by Eric Seidel.

Add a test_expectations.txt file for the mac platform so
new-run-webkit-tests can track IMAGE failures.

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

  • platform/mac/test_expectations.txt: Added.

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

Reviewed by Eric Seidel.

Change the Mac port of new-run-webkit-tests to look for a
test_expectations.txt file in addition to the Skipped files, so we
can track pixel failures.

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

  • Scripts/webkitpy/layout_tests/port/mac.py:
4:38 PM Changeset in webkit [56647] by dpranke@chromium.org
  • 3 edits
    1 add in trunk/WebKitTools

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

Reviewed by Eric Seidel.

Implement pixel tests (image diff) properly on the Mac port.

This change introduces a new "ServerPocess" class that can be used
to manage processes that the run-webkit-tests harness forks off and
expects to stay up for longer than a single request/response session.
Both DumpRenderTree and ImageDiff use this style of communication,
although the current code forks off a new ImageDiff for each diff
(We need to restructure other parts of the code to be able to do this
safely in a multi-threaded environment).

Also, now that the ServerProcess abstraction exists, we can probably
clean up and simplify some of the thread management logic in
test_shell_thread as well.

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

  • Scripts/webkitpy/layout_tests/port/mac.py:
  • Scripts/webkitpy/layout_tests/port/server_process.py:
  • Scripts/webkitpy/layout_tests/test_types/image_diff.py:
3:23 PM Changeset in webkit [56646] by Simon Fraser
  • 7 edits
    4 adds in trunk

2010-03-26 Simon Fraser <Simon Fraser>

Reviewed by Dan Bernstein, Darin Adler.

Re-entrant layout via plug-ins may cause crashes with bad RenderWidgets
​https://bugs.webkit.org/show_bug.cgi?id=36675
<rdar://problem/7787617>

Fix two places in the code where RenderWidgets can get destroyed while being iterated over.
This can happen when plug-ins execute script from NPP_SetWindow, and that script makes a change
to the page that destroys a RenderWidget.

Tests: plugins/reentrant-update-widget-positions.html

plugins/update-widgets-crash.html

  • page/FrameView.cpp: (WebCore::FrameView::updateWidgets): ref() the RenderEmbeddedObjects that are put into the vector before iterating of them, and deref() them at the end. Rather than checking the m_widgetUpdateSet to see if the RenderWidget is still live, test object->node().
  • rendering/RenderView.cpp: (WebCore::RenderView::RenderView): Initialize some data members to make it more obvious in the debugger that the object is not garbage. (WebCore::RenderView::updateWidgetPositions): Use a Vector of RenderWidget* to keep the RenderWidgets alive during iteration, by reffing and dereffing them.
  • rendering/RenderWidget.h: (WebCore::RenderWidget::ref): Make this and deref() public.
3:16 PM Changeset in webkit [56645] by eric@webkit.org
  • 2 edits in trunk/WebKitTools

2010-03-26 Sergio Villar Senin <​svillar@igalia.com>

Reviewed by Eric Seidel.

Print didFinishLoadForFrame outcome in DRT

[GTK] Improve reporting of frame loader callbacks in DRT
​https://bugs.webkit.org/show_bug.cgi?id=36454

  • DumpRenderTree/gtk/DumpRenderTree.cpp: (webViewLoadCommitted):
3:12 PM Changeset in webkit [56644] by Beth Dakin
  • 4 edits in trunk

WebCore: Partial fix for ​https://bugs.webkit.org/show_bug.cgi?id=36564
Performance regression for setting content of <text> in SVG
-and corresponding-
<rdar://problem/7693963>

Reviewed by Darin Adler.

toClipPath() is extremely inefficient, so this patch removes the
call to it from RenderSVGResourceClipper::resourceBoundingBox()
which is very hot code in SVG with clip-paths. It is sufficient to
call unite with the objectBoundingBox(), which will return early if
objectBoundingBox() is empty.

  • rendering/RenderSVGResourceClipper.cpp:

(WebCore::RenderSVGResourceClipper::resourceBoundingBox):

LayoutTests: Updated test for ​https://bugs.webkit.org/show_bug.cgi?id=36564
Performance regression for setting content of <text> in SVG
-and corresponding-
<rdar://problem/7693963>

Reviewed by Darin Adler.

  • platform/mac/svg/custom/clip-path-referencing-use2-expected.txt:
2:51 PM Changeset in webkit [56643] by eric@webkit.org
  • 2 edits in trunk/WebKitTools

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

Unreviewed test fix.
My change conflicted with one of Adam's causing test-webkitpy to fail.

Move commit_message_for_this_commit from scm to checkout
​https://bugs.webkit.org/show_bug.cgi?id=36629

  • Scripts/webkitpy/common/checkout/api_unittest.py:
    • modified_changelogs is now on Checkout instead of scm.
2:45 PM Changeset in webkit [56642] by abarth@webkit.org
  • 7 edits
    1 copy
    1 add in trunk/WebKitTools

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

Reviewed by Eric Seidel.

Refactor IRCBot controller logic to allow for commands
​https://bugs.webkit.org/show_bug.cgi?id=36676

We need to move the controller logic into the tool package so it can
know about commands. The changes to queueengine could go in a
different patch, but we're going to need it anyway.

  • Scripts/webkitpy/common/config/irc.py: Added.
  • Scripts/webkitpy/common/net/irc/ircbot.py:
  • Scripts/webkitpy/common/net/irc/ircproxy.py:
  • Scripts/webkitpy/tool/bot/queueengine.py:
  • Scripts/webkitpy/tool/bot/sheriffircbot.py: Added.
  • Scripts/webkitpy/tool/commands/sheriffbot.py:
  • Scripts/webkitpy/tool/main.py:
  • Scripts/webkitpy/tool/mocktool.py:
1:44 PM Changeset in webkit [56641] by Joseph Pecoraro
  • 3 edits
    6 adds in trunk

2010-03-26 Joseph Pecoraro <Joseph Pecoraro>

Reviewed by Dave Hyatt.

Generated run-in Content is Mistakenly Getting Deleted
​https://bugs.webkit.org/show_bug.cgi?id=36505
<rdar://problem/7767161>

Test: fast/runin/generated2.html
Test: fast/runin/generated3.html
Test: fast/runin/generated4.html

Do not destroy :before/:after generated content with display run-in
when relaying out different nodes. Have their real owners correctly
handle them.

  • rendering/RenderObjectChildList.cpp: (WebCore::RenderObjectChildList::updateBeforeAfterContent): simplified logic (WebCore::RenderObjectChildList::beforeAfterContainer): skip generated run-ins when checking children, check for them in grandchildren

2010-03-26 Joseph Pecoraro <Joseph Pecoraro>

Reviewed by Dave Hyatt.

Generated run-in Content is Mistakenly Getting Deleted
​https://bugs.webkit.org/show_bug.cgi?id=36505
<rdar://problem/7767161>

Test dynamic update of the inner block, containing the generated
run-in, but not owning it with the :before rule, and owners.

  • fast/runin/generated2.html: Added.
  • platform/mac/fast/runin/generated2-expected.txt: Added.

Test dynamic update of the outer block, this block owns the
generated run-in, but it is actually contained in a child block.

  • fast/runin/generated3.html: Added.
  • platform/mac/fast/runin/generated3-expected.txt: Added.

Test dynamic updates affecting generated run-ins both
in relation to and not in relation to list markers.

  • fast/runin/generated4.html: Added.
  • platform/mac/fast/runin/generated4-expected.txt: Added.
1:22 PM Changeset in webkit [56640] by abarth@webkit.org
  • 5 edits
    4 moves
    3 adds in trunk/WebKitTools

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

Rubber stamped by Eric Seidel.

Move the threading code into its own module. (It has nothing to do with IRC.)

  • Scripts/webkitpy/common/net/irc/ircbot.py:
  • Scripts/webkitpy/common/net/irc/ircproxy.py:
  • Scripts/webkitpy/common/net/irc/messagepump.py: Removed.
  • Scripts/webkitpy/common/net/irc/messagepump_unittest.py: Removed.
  • Scripts/webkitpy/common/net/irc/threadedmessagequeue.py: Removed.
  • Scripts/webkitpy/common/net/irc/threadedmessagequeue_unittest.py: Removed.
  • Scripts/webkitpy/common/net/irc/unittests.py:
  • Scripts/webkitpy/common/thread: Added.
  • Scripts/webkitpy/common/thread/init.py: Added.
  • Scripts/webkitpy/common/thread/messagepump.py: Copied from Scripts/webkitpy/common/net/irc/messagepump.py.
  • Scripts/webkitpy/common/thread/messagepump_unittest.py: Copied from Scripts/webkitpy/common/net/irc/messagepump_unittest.py.
  • Scripts/webkitpy/common/thread/threadedmessagequeue.py: Copied from Scripts/webkitpy/common/net/irc/threadedmessagequeue.py.
  • Scripts/webkitpy/common/thread/threadedmessagequeue_unittest.py: Copied from Scripts/webkitpy/common/net/irc/threadedmessagequeue_unittest.py.
  • Scripts/webkitpy/common/thread/unittests.py: Added.
  • Scripts/webkitpy/common/unittests.py:
1:19 PM Changeset in webkit [56639] by ojan@chromium.org
  • 8 edits in trunk

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

Reviewed by Darin Adler.

shift+home/end and cmd+shift+left/right don't extend the selection correctly
​https://bugs.webkit.org/show_bug.cgi?id=36539

  • editing/selection/extend-after-mouse-selection.html:
  • editing/selection/extend-selection-expected.txt: Extending forward by lineboundary selects the whole line. Now extending backwards by lineboundary after the whole line is selected is a noop instead of collapsed the selection to the start.
  • editing/selection/move-begin-end.html: Just cleaned up this test.
  • platform/mac/editing/selection/extend-after-mouse-selection-expected.txt:

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

Reviewed by Darin Adler.

shift+home/end and cmd+shift+left/right don't extend the selection correctly
​https://bugs.webkit.org/show_bug.cgi?id=36539

  • WebCore.base.exp:
  • editing/SelectionController.cpp: (WebCore::SelectionController::modify):
  • editing/SelectionController.h:
12:47 PM Changeset in webkit [56638] by eric@webkit.org
  • 3 edits
    1 add in trunk/WebKitTools

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

Reviewed by Adam Barth.

Move commit_message_for_this_commit from scm to checkout
​https://bugs.webkit.org/show_bug.cgi?id=36629

Finally add some basic unit testing for Checkout.commit_message_for_this_commit

  • Scripts/webkitpy/common/checkout/api.py:
  • Scripts/webkitpy/common/checkout/api_unittest.py: Added.
  • Scripts/webkitpy/common/unittests.py:
12:40 PM Changeset in webkit [56637] by jhawkins@chromium.org
  • 8 edits
    4 adds in trunk/WebKit/chromium

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

Reviewed by Darin Fisher.

[Chromium] Implement WebFormControlElement and WebSelectElement. Add
a getFormControlElements method to WebFormElement.

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

  • WebKit.gyp:
  • public/WebElement.h:
  • public/WebFormControlElement.h: Added.
  • public/WebFormElement.h:
  • public/WebInputElement.h: (WebKit::WebInputElement::WebInputElement): (WebKit::WebInputElement::operator=): (WebKit::WebInputElement::assign):
  • public/WebSelectElement.h: Added.
  • src/WebElement.cpp: (WebKit::WebElement::isFormControlElement):
  • src/WebFormControlElement.cpp: Added.
  • src/WebFormElement.cpp: (WebKit::WebFormElement::getFormControlElements):
  • src/WebInputElement.cpp: (WebKit::WebInputElement::WebInputElement):
  • src/WebSelectElement.cpp: Added.
12:30 PM Changeset in webkit [56636] by tonikitoo@webkit.org
  • 5 edits
    2 adds in trunk/WebKitTools

User Agent Switcher on QtLauncher ​https://bugs.webkit.org/show_bug.cgi?id=36451

Patch by Diego Gonzalez <​diego.gonzalez@openbossa.org> on 2010-03-26
Reviewed by Simon Hausmann.

Make possible to change the QtLauncher user agent via
a predefined list.

  • QtLauncher/QtLauncher.pro:
  • QtLauncher/QtLauncher.qrc: Added.
  • QtLauncher/main.cpp:

(LauncherWindow::showUserAgentDialog):
(LauncherWindow::createChrome):

  • QtLauncher/useragentlist.txt: Added.
  • QtLauncher/webpage.cpp:

(WebPage::userAgentForUrl):

  • QtLauncher/webpage.h:

(WebPage::setUserAgent):

11:21 AM QtWebKitBackportingFixes edited by Laszlo Gombos
(diff)
11:14 AM Changeset in webkit [56635] by eric@webkit.org
  • 9 edits
    4 adds in trunk/WebKitTools

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

Reviewed by Adam Barth.

Host layout test flakiness dashboard in TestResultServer appengine.

Flakiness dashboard is a tool to monitor layout test status and
help layout test regression diagnostics.

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

  • TestResultServer/handlers/dashboardhandler.py: Added.
    • New handler to handle dashboard request
  • TestResultServer/handlers/menu.py:
    • Add new dashboard links
  • TestResultServer/handlers/testfilehandler.py:
    • Request routes refactory
  • TestResultServer/index.yaml:
  • TestResultServer/main.py:
    • Add new dashboard request routes and refactor test result file rountes.
  • TestResultServer/model/dashboardfile.py: Added.
    • Model to access datastore for dashboard files
  • TestResultServer/model/testfile.py:
  • TestResultServer/stylesheets/dashboardfile.css: Added.
  • TestResultServer/templates/dashboardfilelist.html: Added.
  • TestResultServer/templates/showfilelist.html:
11:06 AM Changeset in webkit [56634] by bweinstein@apple.com
  • 2 edits in trunk/LayoutTests

Add fast/multicol/hit-test-above-or-below.html, this is failing in the same way as Qt, and is being
tracked by the same bug (​https://bugs.webkit.org/show_bug.cgi?id=35973).

  • platform/win/Skipped:
10:38 AM Changeset in webkit [56633] by levin@chromium.org
  • 2 edits in trunk/LayoutTests

Rubber-stamped by Nate Chapin.

[chromium] Updated test expectations due to recent failures,
added bugs, etc.

  • platform/chromium/test_expectations.txt:
10:37 AM FastMalloc Glossary created by ggaren@apple.com
10:32 AM Changeset in webkit [56632] by eric@webkit.org
  • 5 edits in trunk/WebKitTools

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

Reviewed by Eric Seidel.

Remove some evil statics from CommitInfo
​https://bugs.webkit.org/show_bug.cgi?id=36637

These methods should really be on checkout. You can tell because they
know about ChangeLogs and take an SCM as an argument. :)

  • Scripts/webkitpy/common/checkout/api.py:
  • Scripts/webkitpy/common/checkout/changelog.py:
  • Scripts/webkitpy/common/checkout/commitinfo.py:
  • Scripts/webkitpy/tool/commands/queries.py:
10:32 AM WikiStart edited by ggaren@apple.com
(diff)
10:29 AM WikiStart edited by ggaren@apple.com
(diff)
10:04 AM Changeset in webkit [56631] by eric@webkit.org
  • 2 edits in trunk/WebCore

2010-03-26 Janne Koskinen <​janne.p.koskinen@digia.com>

Reviewed by Laszlo Gombos.

Don't undefine SKIP_STATIC_CONSTRUCTORS_ON_GCC for Symbian HW targets.
​https://bugs.webkit.org/show_bug.cgi?id=34081

Defining StringImpl instances as globals will cause a crash on process exit as
StringImpl::Remove expects TLS which was already deleted at time of exiting main and ends up
constructing one exiting thread.

  • config.h:
9:34 AM Changeset in webkit [56630] by eric@webkit.org
  • 8 edits in trunk

2010-03-26 Andrey Kosyakov <​caseq@chromium.org>

Reviewed by Pavel Feldman.

Send worker resource content to inspector to enable display of web
workers in inspector's resource tab.
​https://bugs.webkit.org/show_bug.cgi?id=36658

  • inspector/InspectorController.cpp: (WebCore::InspectorController::resourceRetrievedByXMLHttpRequest): (WebCore::InspectorController::scriptImported):
  • inspector/InspectorResource.cpp: (WebCore::InspectorResource::type): (WebCore::InspectorResource::setOverrideContent): (WebCore::InspectorResource::sourceString):
  • inspector/InspectorResource.h:
  • workers/DefaultSharedWorkerRepository.cpp: (WebCore::SharedWorkerScriptLoader::notifyFinished):
  • workers/Worker.cpp: (WebCore::Worker::notifyFinished):

2010-03-26 Andrey Kosyakov <​caseq@chromium.org>

Reviewed by Pavel Feldman.

Send worker resource content to inspector to enable display of web
workers in inspector's resource tab.
​https://bugs.webkit.org/show_bug.cgi?id=36658

  • src/SharedWorkerRepository.cpp: (WebCore::SharedWorkerScriptLoader::notifyFinished):
8:42 AM Changeset in webkit [56629] by yael.aharon@nokia.com
  • 10 edits in trunk

WebCore: Fix the rendering of HTMLProgressElement
​https://bugs.webkit.org/show_bug.cgi?id=36206

Reviewed by Antti Koivisto.

Update the rendering and css to conform to
​http://www.whatwg.org/specs/web-apps/current-work/complete.html#the-progress-element-0.
Updated the tests and the expected results accordingly.

  • css/html.css:
  • rendering/RenderProgress.cpp:
  • rendering/RenderProgress.h:

LayoutTests: Fix the rendering of HTMLProgressElement
​https://bugs.webkit.org/show_bug.cgi?id=36206
Add dir=rtl test and update the expected results.

Reviewed by Antti Koivisto.

  • fast/dom/HTMLProgressElement/progress-element.html:
  • platform/qt/fast/dom/HTMLProgressElement/progress-element-expected.checksum:
  • platform/qt/fast/dom/HTMLProgressElement/progress-element-expected.png:
  • platform/qt/fast/dom/HTMLProgressElement/progress-element-expected.txt:
8:30 AM Changeset in webkit [56628] by jchaffraix@webkit.org
  • 2 edits in trunk/WebCore

Unreviewed build fix.

FontPlatformDataPango.cpp: prototype for
'WebCore::FontPlatformData::FontPlatformData(cairo_font_face_t*, int, bool, bool)'
does not match any in class
​https://bugs.webkit.org/show_bug.cgi?id=36496

  • platform/graphics/gtk/FontPlatformDataPango.cpp:

(WebCore::FontPlatformData::FontPlatformData): The 'size' parameter
should be a float to match the declaration and the other definitions.

8:22 AM Changeset in webkit [56627] by eric@webkit.org
  • 2 edits in trunk/WebCore

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

Reviewed by Eric Seidel.

[Qt] Prevent referring d->m_job in the future because calling abort()
deletes the instance itself.
​https://bugs.webkit.org/show_bug.cgi?id=36618

Test: http/tests/appcache/fallback.html

  • platform/network/qt/ResourceHandleQt.cpp: (WebCore::ResourceHandle::cancel):
7:49 AM QtWebKit edited by zoltan@webkit.org
(diff)
7:37 AM Changeset in webkit [56626] by Simon Hausmann
  • 2 edits in trunk/WebKit/qt

[Qt] Compile with QT_USE_FAST_OPERATOR_PLUS

Patch by Olivier Goffart <​ogoffart@trolltech.com> on 2010-03-26
Reviewed by Simon Hausmann.

  • WebCoreSupport/InspectorClientQt.cpp:

(WebCore::InspectorClientQt::populateSetting):
(WebCore::InspectorClientQt::storeSetting):

7:35 AM Changeset in webkit [56625] by Simon Hausmann
  • 2 edits in trunk/WebKit/qt

[Qt] Doc: Simplified Commercial Editions for Qt 4.7.

Patch by David Boddie <​dboddie@trolltech.com> on 2010-03-26
Reviewed by Simon Hausmann.

  • docs/qtwebkit.qdoc:
7:26 AM Changeset in webkit [56624] by senorblanco@chromium.org
  • 4 edits
    2 deletes in trunk

2010-03-26 Stephen White <​senorblanco@chromium.org>

Unreviewed; build fix.

Rolling out 56620; too many layout test failures.

Covered by fast/css/*.

  • css/CSSPrimitiveValue.cpp: (WebCore::CSSPrimitiveValue::cleanup): (WebCore::CSSPrimitiveValue::cssText):
  • css/CSSPrimitiveValue.h: (WebCore::CSSPrimitiveValue::):

2010-03-26 Stephen White <​senorblanco@chromium.org>

Unreviewed; build fix.

Rolling out 56620; too many layout test failures.

  • fast/css/cssText-cache-expected.txt: Removed.
  • fast/css/cssText-cache.html: Removed.
7:05 AM Changeset in webkit [56623] by Simon Hausmann
  • 9 edits in trunk

[Qt] Build JavaScriptCore as a static library.
​https://bugs.webkit.org/show_bug.cgi?id=36590

Patch by Jocelyn Turcotte <​jocelyn.turcotte@nokia.com> on 2010-03-26
Reviewed by Simon Hausmann.

This patch takes what was left of the unused JavaScriptCore.pro
and moved the compilation logic from JavaScriptCore.pri to
JavaScriptCore.pro.

.:

  • WebKit.pro:

JavaScriptCore:

WebCore:

  • WebCore.pro:
6:44 AM Changeset in webkit [56622] by Simon Hausmann
  • 2 edits in trunk/WebCore

[Qt] Fix compilation on Windows

Patch by Olivier Goffart <​ogoffart@trolltech.com> on 2010-03-26
Reviewed by Simon Hausmann.

For some reason, the MSVC compiler choose the
operator+(const QString &, const QString &) instead of
operator+(const WebCore::String &, const WebCore::String &)
resulting in errors when QT_USE_FAST_OPERATOR_PLUS is used.

  • dom/XMLTokenizerQt.cpp:

(WebCore::handleElementNamespaces):

6:28 AM Changeset in webkit [56621] by Philippe Normand
  • 2 edits in trunk/LayoutTests

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

fast/parser/xml-declaration-missing-ending-mark.html crashed on Gtk bot
​https://bugs.webkit.org/show_bug.cgi?id=33027

Skipping test that fails on GTK+ Debug bots.

  • platform/gtk/Skipped:
6:18 AM Changeset in webkit [56620] by senorblanco@chromium.org
  • 4 edits
    2 adds in trunk

2010-03-24 Stephen White <​senorblanco@chromium.org>

Reviewed by Darin Adler.

In order to speed up multiple calls to CSSPrimitiveValue::cssText(),
this CL caches the String result. When m_value is changed, the
cached string is cleared. This gives a good speedup on benchmarks
which do a lot of CSS property gets, such as Peacekeeper.
The processing cost should be negligible, since the strings are
refcounted. The memory cost is an additional 4 bytes per
CSSPrimitiveValue, and the extended lifetime of the computed string
(potentially, the same as the lifetime of the CSSPrimitiveValue).

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

Covered by fast/css/large-number-round-trip.html, and more.

  • css/CSSPrimitiveValue.cpp: (WebCore::CSSPrimitiveValue::CSSPrimitiveValue): (WebCore::CSSPrimitiveValue::cleanup): (WebCore::CSSPrimitiveValue::cssText):
  • css/CSSPrimitiveValue.h:

2010-03-25 Stephen White <​senorblanco@chromium.org>

Reviewed by Darin Adler.

Added a test for CSSPrimitiveValue::cssText() string cacheing.

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

  • fast/css/cssText-cache-expected.txt: Added.
  • fast/css/cssText-cache.html: Added.
5:58 AM Changeset in webkit [56619] by eric@webkit.org
  • 4 edits in trunk

2010-03-26 Qi Zhang <​qi.2.zhang@nokia.com>

Reviewed by Laszlo Gombos.

[Qt] fast/canvas/drawImage-with-negative-source-destination.html failed
​https://bugs.webkit.org/show_bug.cgi?id=35005

Enable fast/canvas/drawImage-with-negative-source-destination.html

  • platform/qt/Skipped:

2010-03-26 Qi Zhang <​qi.2.zhang@nokia.com>

Reviewed by Laszlo Gombos.

[Qt] fast/canvas/drawImage-with-negative-source-destination.html failed
​https://bugs.webkit.org/show_bug.cgi?id=35005

To support negative width or height at context.drawImage

  • platform/graphics/qt/StillImageQt.cpp: (WebCore::StillImage::draw):
5:52 AM Changeset in webkit [56618] by tonikitoo@webkit.org
  • 3 edits in trunk/WebKitTools

Reviewed by Simon Hausmann.

Add an option to QtLauncher to enable/disable a QGLWidget as Viewport
of the QGraphicsView when the launcher is running on graphicsview mode.

[Qt] QtLauncher needs an option to Enable/Disable a QGLWidget as Viewport
​https://bugs.webkit.org/show_bug.cgi?id=36270

  • QtLauncher/QtLauncher.pro:
  • QtLauncher/main.cpp:

(LauncherWindow::toggleQGLWidgetViewport):
(LauncherWindow::createChrome):

4:11 AM Changeset in webkit [56617] by ager@chromium.org
  • 2 edits in trunk/LayoutTests

2010-03-26 Mads Ager <​ager@chromium.org>

Reviewed by Jeremy Orlow.

Tweak Chromium test expectations
​https://bugs.webkit.org/show_bug.cgi?id=36651

fast/js/large-expressions.html started failing on Linux in release
mode occassionally. Also fails on other platforms.

  • platform/chromium/test_expectations.txt:
2:21 AM Changeset in webkit [56616] by ukai@chromium.org
  • 2 edits in trunk/LayoutTests

Unreviewed, removing websocket/tests/long-invalid-header.html from Chromium test_expectations.txt

  • platform/chromium/test_expectations.txt: remove websocket/tests/long-invalid-header.html
1:58 AM Changeset in webkit [56615] by Philippe Normand
  • 2 edits in trunk/WebCore

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

Reviewed by Gustavo Noronha.

[GStreamer] soften dependency on libsoup in the http src element
​https://bugs.webkit.org/show_bug.cgi?id=35864

Replaced SoupURI calls with KURL and pause/resume internal soup
messages only if the element is compiled for a port depending on
libsoup.

  • platform/graphics/gstreamer/WebKitWebSourceGStreamer.cpp: (webkit_web_src_init): (webKitWebSrcSetUri): (webKitWebSrcNeedDataMainCb): (webKitWebSrcEnoughDataMainCb):
1:45 AM Changeset in webkit [56614] by ukai@chromium.org
  • 2 edits in trunk/WebKitTools

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

Reviewed by Shinichiro Hamaji.

[Chromium-Win] websocket_server.py failed to start
​https://bugs.webkit.org/show_bug.cgi?id=36586

subproess.Popen() on Python 2.4/Windows with stdout,stdout, but no stdin will fail, because it uses return value of GetStdHandle(STD_INPUT_HANDLE), but DuplicateHandle requires integer, not the handle.
We don't need stdin, so use devnull as stdin.

  • Scripts/webkitpy/layout_tests/port/websocket_server.py: open os.devnull for stdin
1:05 AM Changeset in webkit [56613] by pfeldman@chromium.org
  • 4 edits in trunk/WebCore

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

Reviewed by Pavel Feldman.

Web Inspector: Shot record filter is implemented in Timeline Panel.
​https://bugs.webkit.org/show_bug.cgi?id=36606

  • English.lproj/localizedStrings.js:
  • inspector/front-end/TimelinePanel.js: (WebInspector.TimelinePanel): (WebInspector.TimelinePanel.prototype.get statusBarItems): (WebInspector.TimelinePanel.prototype._createStatusbarButtons): (WebInspector.TimelinePanel.prototype._toggleFilterButtonClicked): (WebInspector.TimelinePanel.prototype._innerAddRecordToTimeline): (WebInspector.TimelinePanel.prototype._addToRecordsWindow): (WebInspector.TimelineRecordGraphRow.prototype.update): (WebInspector.TimelinePanel.FormattedRecord.prototype.get _isLongEvent):
  • inspector/front-end/inspector.css:
12:58 AM Changeset in webkit [56612] by tkent@chromium.org
  • 1 edit
    2 adds in trunk/WebKitTools

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

Reviewed by Dimitri Glazkov.

[DRT/Chromium] Add TestNavigationController and TestWebWorker
​https://bugs.webkit.org/show_bug.cgi?id=36520

Add LayoutTestController class, which is going to be used by
DumpRenderTree Chromium port. These files are based on:

  • src/webkit/tools/test_shell/layout_test_controller.cc
  • src/webkit/tools/test_shell/layout_test_controller.h of Chromium rev.40492.
  • DumpRenderTree/chromium/LayoutTestController.cpp: Added.
  • DumpRenderTree/chromium/LayoutTestController.h: Added.
12:55 AM Changeset in webkit [56611] by Simon Hausmann
  • 2 edits in trunk/WebKit/qt

Symbian build fix.

[Qt] Updated the def file with new exports used by QtLauncher.

  • symbian/eabi/QtWebKitu.def:
12:43 AM Changeset in webkit [56610] by tkent@chromium.org
  • 3 edits in trunk/WebCore

Rollout r56609. It was accidental commit.

12:34 AM Changeset in webkit [56609] by tkent@chromium.org
  • 3 edits in trunk/WebCore

2009-10-20 Mikhail Naganov <​mnaganov@chromium.org>

Reviewed by Pavel Feldman.

Web Inspector: populate child nodes before sorting them.

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

  • inspector/front-end/BottomUpProfileDataGridTree.js: (WebInspector.BottomUpProfileDataGridNode): Swapped with BottomUpProfileDataGridTree to be consistent with TopDownProfileDataGridNode. (WebInspector.BottomUpProfileDataGridNode.prototype._takePropertiesFromProfileDataGridNode): (WebInspector.BottomUpProfileDataGridNode.prototype._keepOnlyChild): (WebInspector.BottomUpProfileDataGridNode.prototype._exclude): (WebInspector.BottomUpProfileDataGridNode.prototype._merge): (WebInspector.BottomUpProfileDataGridNode.prototype._sharedPopulate): (WebInspector.BottomUpProfileDataGridTree.prototype.exclude):
  • inspector/front-end/ProfileDataGridTree.js: (WebInspector.ProfileDataGridNode.prototype.sort): Added missing parentheses. (WebInspector.ProfileDataGridNode.prototype.get _parent): (WebInspector.ProfileDataGridNode.prototype._populate):
  • inspector/front-end/TopDownProfileDataGridTree.js: (WebInspector.TopDownProfileDataGridNode.prototype._sharedPopulate):
12:13 AM Changeset in webkit [56608] by pfeldman@chromium.org
  • 10 edits in trunk/WebCore

2010-03-25 Pavel Feldman <​pfeldman@chromium.org>

Reviewed by Timothy Hatcher.

Web Inspector: Implement CSSStyleSelector::pseudoStyleForElement and use it in the InspectorDOMAgent.

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

This change provides pseudo elements information into the inspector styles sidebar pane.
Changes applied:

  • CSSStyleSelector::pseudoStyleRulesForElement implemented. I changed the signature to accept PseudoId instead of a String since it better reflected the needs;
  • Accompanied elementStyle checks with m_collectRulesOnly checks;
  • Removed pseudoStyleRulesForElement usage from DOMWindow::getMatchedCSSRules. It was anyways returning 0, while I would need to convert String to PseudoId in order to leave the call in place;
  • Needed to introduce AFTER_LAST_INTERNAL_PSEUDOID marker const in RenderStyleConstants' PseudoId enum
  • InspectorDOMAgent is now iterating over all pseudo ids to get all styles (including internal ones).
  • Brushed up front-end code so that proper overriding took place locally within pseudo id categories.
  • English.lproj/localizedStrings.js:
  • css/CSSStyleSelector.cpp: (WebCore::CSSStyleSelector::styleRulesForElement): (WebCore::CSSStyleSelector::pseudoStyleRulesForElement): (WebCore::CSSStyleSelector::SelectorChecker::checkSelector): (WebCore::CSSStyleSelector::SelectorChecker::checkOneSelector):
  • css/CSSStyleSelector.h:
  • inspector/InspectorDOMAgent.cpp: (WebCore::InspectorDOMAgent::getStyles): (WebCore::InspectorDOMAgent::buildObjectForAttributeStyles): (WebCore::InspectorDOMAgent::buildArrayForCSSRules): (WebCore::InspectorDOMAgent::buildArrayForPseudoElements):
  • inspector/InspectorDOMAgent.h:
  • inspector/front-end/StylesSidebarPane.js: (WebInspector.StylesSidebarPane.prototype.update.callback): (WebInspector.StylesSidebarPane.prototype.update): (WebInspector.StylesSidebarPane.prototype._update): (WebInspector.StylesSidebarPane.prototype._refreshStyleRules): (WebInspector.StylesSidebarPane.prototype._rebuildStyleRules.insertInheritedNodeSeparator): (WebInspector.StylesSidebarPane.prototype._rebuildStyleRules): (WebInspector.StylesSidebarPane.prototype._markUsedProperties): (WebInspector.StylesSidebarPane.prototype._refreshSectionsForStyleRules): (WebInspector.StylesSidebarPane.prototype._rebuildSectionsForStyleRules): (WebInspector.StylesSidebarPane.prototype._changeColorFormat): (WebInspector.StylesSidebarPane.prototype.addBlankSection): (WebInspector.StylesSidebarPane.prototype.removeSection):
  • inspector/front-end/inspector.css: (.styles-sidebar-separator):
  • page/DOMWindow.cpp: (WebCore::DOMWindow::getMatchedCSSRules):
  • rendering/style/RenderStyleConstants.h: (WebCore::):
12:11 AM Changeset in webkit [56607] by mrowe@apple.com
  • 7 edits in trunk/WebKitLibraries

Update WebCoreSQLite3 to SQLite v3.6.12.

Rubber-stamped by Dan Bernstein.

  • WebCoreSQLite3/sqlite3.h:
  • WebCoreSQLite3/sqlite3ext.h:
  • libWebCoreSQLite3.a:
  • libWebKitSystemInterfaceLeopard.a:
  • libWebKitSystemInterfaceSnowLeopard.a:
  • libWebKitSystemInterfaceTiger.a:
Note: See TracTimeline for information about the timeline view.