Timeline


and

03/05/10:

23:28 Changeset [55613] by eric@webkit.org

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

Reviewed by Eric Seidel.

https://bugs.webkit.org/show_bug.cgi?id=33247
Backwards cursor movement incorrect when previous block ends with <br>.

If the cursor is trying to move into a node that has a height of 0,
skip over it.

  • editing/execCommand/move-selection-back-line-expected.txt: Added.
  • editing/execCommand/move-selection-back-line.html: Added.

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

Reviewed by Eric Seidel.

https://bugs.webkit.org/show_bug.cgi?id=33247
Backwards cursor movement incorrect when previous block ends with <br>.

If the cursor is trying to move into a node that has a height of 0,
skip over it.

Test: editing/execCommand/move-selection-back-line.html

  • editing/visible_units.cpp: (WebCore::previousLinePosition):
23:11 Changeset [55612] by eric@webkit.org

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

Reviewed by Eric Seidel.

Enable resource loading callback while loading worker scripts
https://bugs.webkit.org/show_bug.cgi?id=35744

  • workers/WorkerScriptLoader.cpp: (WebCore::WorkerScriptLoader::loadSynchronously): (WebCore::WorkerScriptLoader::loadAsynchronously):
22:45 Changeset [55611] by eric@webkit.org

2010-03-05 Chris Fleizach <cfleizach@apple.com>

Reviewed by Eric Seidel.

WAI-ARIA live region doesn't appear when <div> only has a <div> child
https://bugs.webkit.org/show_bug.cgi?id=35751

  • platform/mac/accessibility/div-containing-div-with-aria-expected.txt: Added.
  • platform/mac/accessibility/div-containing-div-with-aria.html: Added.

2010-03-05 Chris Fleizach <cfleizach@apple.com>

Reviewed by Eric Seidel.

WAI-ARIA live region doesn't appear when <div> only has a <div> child
https://bugs.webkit.org/show_bug.cgi?id=35751

Elements should not be ignored if they carry ARIA attributes in them.

Test: platform/mac/accessibility/div-containing-div-with-aria.html

  • accessibility/AccessibilityObject.cpp: (WebCore::AccessibilityObject::supportsARIAAttributes):
  • accessibility/AccessibilityObject.h: (WebCore::AccessibilityObject::supportsARIADropping): (WebCore::AccessibilityObject::supportsARIADragging):
  • accessibility/AccessibilityRenderObject.cpp: (WebCore::AccessibilityRenderObject::supportsARIADropping): (WebCore::AccessibilityRenderObject::supportsARIADragging): (WebCore::AccessibilityRenderObject::accessibilityIsIgnored):
  • accessibility/AccessibilityRenderObject.h:
19:48 Changeset [55610] by eric@webkit.org

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

Reviewed by Eric Seidel.

Fix Print option on QtLauncher by calling print directly from QWebFrame.

[Qt] QtLauncher Print option is not working on QGraphicsView mode
https://bugs.webkit.org/show_bug.cgi?id=35769

  • QtLauncher/main.cpp: (LauncherWindow::print):
19:16 Changeset [55609] by dimich@chromium.org

Add a manual test for crash in DOMWindow::clearTimeout when DOMWindow is not connected to Frame
https://bugs.webkit.org/show_bug.cgi?id=32353

Reviewed by Oliver Hunt.

  • manual-tests/clearTimeout-crash-bug29832.html: Added.
19:04 Changeset [55608] by dimich@chromium.org

[v8] Remove wrong assert in GC callback.
https://bugs.webkit.org/show_bug.cgi?id=35757

Reviewed by David Levin.

WebCore:

Test: fast/workers/wrapper-map-gc.html

  • bindings/v8/DOMData.h:

(WebCore::DOMData::handleWeakObject): remove ASSERT(isMainThread()), move another assert up to verify we are on the right thread.

LayoutTests:

  • fast/workers/resources/wrapper-map-gc.js: Added.
  • fast/workers/wrapper-map-gc.html: Added.
  • fast/workers/wrapper-map-gc-expected.txt: Added.
18:52 Changeset [55607] by eric@webkit.org

2010-03-05 Alex Milowski <alex@milowski.com>

Reviewed by Kenneth Rohde Christiansen.

Added tests for row and operator stretching.
Updated over, underover, and subsup due to row/operators in use.

  • mathml/presentation/mo.xhtml: Added.
  • mathml/presentation/row.xhtml: Added.
  • platform/mac/mathml/presentation/mo-expected.checksum: Added.
  • platform/mac/mathml/presentation/mo-expected.png: Added.
  • platform/mac/mathml/presentation/mo-expected.txt: Added.
  • platform/mac/mathml/presentation/over-expected.checksum:
  • platform/mac/mathml/presentation/over-expected.png:
  • platform/mac/mathml/presentation/over-expected.txt:
  • platform/mac/mathml/presentation/row-expected.checksum: Added.
  • platform/mac/mathml/presentation/row-expected.png: Added.
  • platform/mac/mathml/presentation/row-expected.txt: Added.
  • platform/mac/mathml/presentation/subsup-expected.txt:
  • platform/mac/mathml/presentation/underover-expected.checksum:
  • platform/mac/mathml/presentation/underover-expected.png:
  • platform/mac/mathml/presentation/underover-expected.txt:

2010-03-05 Alex Milowski <alex@milowski.com>

Reviewed by Kenneth Rohde Christiansen.

Added support for row layout with stretchy operators and
adjusted the over spacing for over and underover accordingly.

Tests: mathml/presentation/mo.xhtml

mathml/presentation/row.xhtml

  • WebCore.xcodeproj/project.pbxproj:
  • mathml/MathMLInlineContainerElement.cpp:
  • mathml/MathMLTextElement.cpp:
  • mathml/RenderMathMLMath.cpp: Added.
  • mathml/RenderMathMLMath.h: Added.
  • mathml/RenderMathMLOperator.cpp: Added.
  • mathml/RenderMathMLOperator.h: Added.
  • mathml/RenderMathMLRow.cpp: Added.
  • mathml/RenderMathMLRow.h: Added.
  • mathml/RenderMathMLUnderOver.cpp:
  • mathml/mathtags.in:
18:34 Changeset [55606] by eric@webkit.org

2010-03-05 John Abd-El-Malek <jam@chromium.org>

Reviewed by Darin Fisher.

Take out code hack that flips the result of NPP_HandleEvent. This needs to move to
the NPAPI code so that Pepper plugins don't hit it.
https://bugs.webkit.org/show_bug.cgi?id=35779

  • public/WebPluginContainer.h:
  • src/WebPluginContainerImpl.cpp: (WebKit::WebPluginContainerImpl::handleMouseEvent): (WebKit::WebPluginContainerImpl::handleKeyboardEvent):
18:27 Changeset [55605] by dino@apple.com

2010-03-05 Dean Jackson <dino@apple.com>

Reviewed by Simon Fraser.

https://bugs.webkit.org/show_bug.cgi?id=35772
Animation fill modes should be supported in the shorthand property

Add CSSPropertyWebkitAnimationFillMode to the list of properties
evaluated in the -webkit-animation shorthand.

Test: animations/animation-shorthand.html

  • css/CSSParser.cpp: (WebCore::CSSParser::parseAnimationShorthand):
17:09 Changeset [55604] by eric@webkit.org

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

No review. Skipping recently regressed test to unblock the commit-queue (20 patches waiting).

REGRESSION(55593?): fast/workers/worker-cloneport.html is timing out on Leopard
https://bugs.webkit.org/show_bug.cgi?id=35819

  • platform/mac-leopard/Skipped:
16:52 Changeset [55603] by cjerdonek@webkit.org

Changed the logging code for new-run-webkit-tests to use
module-specific loggers rather than the root logger. This is
a standard practice that allows logging specific to a package
to be configured independently of other modules.

Reviewed by Eric Seidel.

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

Added a line of the form "_log = logging.getLogger(<module>)"
to each module below, where <module> is the fully-qualified
name of the module, and updated the log lines to use the new
_log logger.

  • Scripts/webkitpy/layout_tests/layout_package/json_results_generator.py:
  • Scripts/webkitpy/layout_tests/layout_package/test_expectations.py:
  • Scripts/webkitpy/layout_tests/layout_package/test_shell_thread.py:
  • Scripts/webkitpy/layout_tests/port/apache_http_server.py:
  • Scripts/webkitpy/layout_tests/port/chromium.py:
  • Scripts/webkitpy/layout_tests/port/chromium_linux.py:
  • Scripts/webkitpy/layout_tests/port/chromium_mac.py:
  • Scripts/webkitpy/layout_tests/port/chromium_win.py:
  • Scripts/webkitpy/layout_tests/port/http_server.py:
  • Scripts/webkitpy/layout_tests/port/http_server_base.py:
  • Scripts/webkitpy/layout_tests/port/mac.py:
  • Scripts/webkitpy/layout_tests/port/websocket_server.py:
  • Scripts/webkitpy/layout_tests/rebaseline_chromium_webkit_tests.py:
  • Scripts/webkitpy/layout_tests/run_webkit_tests.py:
  • Scripts/webkitpy/layout_tests/test_types/fuzzy_image_diff.py:
  • Scripts/webkitpy/layout_tests/test_types/image_diff.py:
  • Scripts/webkitpy/layout_tests/test_types/test_type_base.py:
  • Scripts/webkitpy/layout_tests/test_types/text_diff.py:
16:38 Changeset [55602] by dpranke@chromium.org

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

Reviewed by Eric Siedel.

Split the command-line invocation of the Chromium/python LigHTTPd
server implementation out into its own top level script to make it
a more "public" interface and to resolve some awkward layering
issues. This script will be called directly by other test scripts in
the Chromium tree.

At some point this script should be made to work with Apache-based
implementations and on other ports. I have filed
https://bugs.webkit.org/show_bug.cgi?id=35820 for this.

Also fix a bug in port/factory where options.chromium could be
dereferenced even if it wasn't set, raising an exception.

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

  • Scripts/webkitpy/layout_tests/port/factory.py:
  • Scripts/webkitpy/layout_tests/port/http_server.py:
  • Scripts/new-run-webkit-httpd: Added
16:21 Changeset [55601] by dino@apple.com

2010-03-05 Dean Jackson <dino@apple.com>

Reviewed by NOBODY (OOPS!).

https://bugs.webkit.org/show_bug.cgi?id=35815
Animation Fill Modes fail on Windows

  • css/CSSComputedStyleDeclaration.cpp:
  • css/CSSStyleSelector.cpp:
  • platform/graphics/mac/GraphicsLayerCA.mm:
    • use new enum value
  • platform/animation/Animation.h:
    • change bitfield type
  • rendering/style/RenderStyleConstants.h:
    • add new enum for fill mode
15:44 Changeset [55600] by oliver@apple.com

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

Reviewed by NOBODY (build fix).

Add enw exports to windows

15:29 Changeset [55599] by oliver@apple.com

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

Reviewed by Gavin Barraclough.

JSC should cache int to Identifier conversion as it does for ordinary strings
https://bugs.webkit.org/show_bug.cgi?id=35814

Make the NumericStrings cache cache unsigned ints in addition to signed.
We keep them separate from the int cache as it both simplifies code, and
also because the unsigned path is exclusive to property access and therefore
seems to have different usage patterns.

The primary trigger for the unsigned to Identifier propertyName conversion
is the construction of array-like objects out of normal objects. Given these
tend to be relative small numbers, and the array-like behaviour lends itself
to sequential values this patch also adds a non-colliding cache for all small
numbers.

  • JavaScriptCore.exp:
  • runtime/Identifier.cpp: (JSC::Identifier::from):
  • runtime/Identifier.h:
  • runtime/NumericStrings.h: (JSC::NumericStrings::add): (JSC::NumericStrings::lookup): (JSC::NumericStrings::lookupSmallString):
15:26 Changeset [55598] by ossy@webkit.org

Unreviewed buildfix after r55593. (To fix Qt --minimal build.)

  • bindings/js/JSDOMWindowCustom.cpp: Missing #if ENABLE(DATABASE) guard added.
14:55 Changeset [55597] by simon.fraser@apple.com

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

Reviewed by Darin Adler.

ASSERTION FAILED: rootLayer == m_clipRectsRoot at hulu.com
https://bugs.webkit.org/show_bug.cgi?id=34065
<rdar://problem/7573509>

Fix one instance of this assertion (not necessarily the one seen on hulu.com).

If the layer that clippingRoot() is being called on is itself compositing,
then it acts as the clipping root. Without this, calls to RenderLayer::childrenClipRect()
and RenderLayer::selfClipRect() via FrameView::windowClipRectForLayer(), for plug-ins,
caused the caching of an incorrect clip rects root.

Test: compositing/geometry/object-clip-rects-assertion.html

  • rendering/RenderLayer.cpp: (WebCore::RenderLayer::clippingRoot):
14:53 Changeset [55596] by dino@apple.com

2010-03-05 Dean Jackson <dino@apple.com>

No review.

Disabling animations/fill-mode-transform.html on Leopard
due to https://bugs.webkit.org/show_bug.cgi?id=35714

  • platform/mac-leopard/Skipped:
14:46 Changeset [55595] by simon.fraser@apple.com

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

Reviewed by Dan Bernstein.

Child clipping compositing layers don't show the blue debug color
https://bugs.webkit.org/show_bug.cgi?id=35807

Set a layer owner on the GraphicsLayers created for clipping children,
so that they can get to the debug settings that give them the blue color.

  • rendering/RenderLayerBacking.cpp: (WebCore::RenderLayerBacking::updateClippingLayers):
14:43 Changeset [55594] by simon.fraser@apple.com

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

Reviewed by Dan Bernstein.

MobileMe movie page is missing playback controls
https://bugs.webkit.org/show_bug.cgi?id=35805
<rdar://problem/7653169>

On pages with video or plug-ins, we run overlap tests to determine which layers
need to be composited. There was an ordering dependency bug in
RenderLayerCompositor::computeCompositingRequirements() that caused us to fail
to detect that a layer needed to be composited, resulting in page content
not being visible.

Specifically, layer->setHasCompositingDescendant() can change the answer
to needsToBeComposited(), when a layer has to clip compositing descendants.
So if we change setHasCompositingDescendant(), then we need to re-test
clipsCompositingDescendants() and add the layer to the overlap map if so.

Test: compositing/overflow/overflow-compositing-descendant.html

  • rendering/RenderLayerCompositor.cpp: (WebCore::RenderLayerCompositor::computeCompositingRequirements):
14:17 Changeset [55593] by dumi@chromium.org

WebCore: Adding support for the optional creation callback that could be
passed to openDatabase().

Reviewed by Adam Barth.

Test: storage/open-database-creation-callback.html

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

  • Android.jscbindings.mk
  • Android.v8bindings.mk
  • GNUmakefile.am:
  • WebCore.gypi:
  • WebCore.pro:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • bindings/js/JSDOMWindowCustom.cpp:

(WebCore::JSDOMWindow::openDatabase):

  • bindings/js/JSDatabaseCallback.cpp: Added.

(WebCore::JSDatabaseCallback::JSDatabaseCallback):
(WebCore::JSDatabaseCallback::~JSDatabaseCallback):
(WebCore::JSDatabaseCallback::handleEvent):

  • bindings/js/JSDatabaseCallback.h: Added.

(WebCore::JSDatabaseCallback::create):

  • bindings/v8/custom/V8DOMWindowCustom.cpp:

(WebCore::V8DOMWindow::openDatabaseCallback):

  • bindings/v8/custom/V8DatabaseCallback.cpp: Added.

(WebCore::V8DatabaseCallback::V8DatabaseCallback):
(WebCore::V8DatabaseCallback::~V8DatabaseCallback):
(WebCore::V8DatabaseCallback::handleEvent):

  • bindings/v8/custom/V8DatabaseCallback.h: Added.

(WebCore::V8DatabaseCallback::create):

  • page/DOMWindow.cpp:

(WebCore::DOMWindow::openDatabase):

  • page/DOMWindow.h:
  • page/DOMWindow.idl:
  • storage/Database.cpp:

(WebCore::DatabaseCreationCallbackTask::create):
(WebCore::DatabaseCreationCallbackTask::performTask):
(WebCore::DatabaseCreationCallbackTask::DatabaseCreationCallbackTask):
(WebCore::Database::openDatabase):
(WebCore::Database::Database):
(WebCore::Database::performOpenAndVerify):
(WebCore::Database::performCreationCallback):

  • storage/Database.h:

(WebCore::Database::isNew):

  • storage/DatabaseCallback.h: Added.

(WebCore::DatabaseCallback::~DatabaseCallback):

  • workers/WorkerContext.cpp:

(WebCore::WorkerContext::openDatabase):

  • workers/WorkerContext.h:

LayoutTests: Adding a test to test the creation callback passed to
openDatabase() and another one to make sure it is executed in the
correct world.

Reviewed by Adam Barth.

Also, fixing fast/frames/sandboxed-iframe-storage.html: calling
openDatabase() without at least 4 parameters should always fail,
because the database name, version, description and estimated size
are not optional according to the spec.

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

  • fast/frames/resources/sandboxed-iframe-storage-allowed.html:
  • fast/frames/resources/sandboxed-iframe-storage-disallowed.html:
  • fast/frames/sandboxed-iframe-storage-expected.txt:
  • platform/qt/Skipped:
  • storage/open-database-creation-callback-expected.txt: Added.
  • storage/open-database-creation-callback.html: Added.
  • storage/open-database-creation-callback-isolated-world-expected.txt: Added.
  • storage/open-database-creation-callback-isolated-world.html: Added.
13:04 Changeset [55592] by cmarrin@apple.com

Fixed crash due to infinite recursion when viewing composited video on Windows
https://bugs.webkit.org/show_bug.cgi?id=35798

10:33 Changeset [55591] by bweinstein@apple.com

Update getComputedStyle results on Windows.

  • platform/win/fast/css/getComputedStyle/computed-style-expected.txt:
  • platform/win/fast/css/getComputedStyle/computed-style-without-renderer-expected.txt:
10:26 Changeset [55590] by dglazkov@chromium.org

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

Reviewed by David Levin.

[Chromium] Missing commas in WebCore.gypi file make building Geo unpossible.
https://bugs.webkit.org/show_bug.cgi?id=35797

No new tests, this is a latent compile break.

  • WebCore.gypi:
10:12 Changeset [55589] by eric.carlson@apple.com

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

Unreviewed, build fix.

Use a movie that will hopefully fail to open less often on a headless
SnowLeopard bot.

  • media/video-preload-expected.txt:
  • media/video-preload.html: Use test.mp4 instead of silence.mpg.
09:39 QtWebKitBugs edited by vestbo@webkit.org
(diff)
09:32 Changeset [55588] by eric.carlson@apple.com

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

Unreviewed, build fix.

media/video-preload.html fails on GTK, skip it.

  • platform/gtk/Skipped:
09:13 Changeset [55587] by eric.carlson@apple.com

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

Reviewed by Adele Peterson.

<rdar://problem/7718442> Implement 'preload=none'
https://bugs.webkit.org/show_bug.cgi?id=35789

Don't load any media data when preload is 'none'.

Test: media/video-preload.html

  • html/HTMLMediaElement.cpp: (WebCore::HTMLMediaElement::loadResource): Don't call player's setPreload method

when autoplay is set.

  • platform/graphics/MediaPlayer.cpp: (WebCore::MediaPlayer::setPreload): Set m_preload so we have the correct value when

the media engine is created.

  • platform/graphics/mac/MediaPlayerPrivateQTKit.h:
  • platform/graphics/mac/MediaPlayerPrivateQTKit.mm: (WebCore::MediaPlayerPrivate::MediaPlayerPrivate): Initialize m_preload. (WebCore::MediaPlayerPrivate::resumeLoad): New, kick off postponed a load. (WebCore::MediaPlayerPrivate::load): Do nothing if preload is 'none' (WebCore::MediaPlayerPrivate::loadInternal): New, complete loading. (WebCore::MediaPlayerPrivate::prepareToPlay): New, resume a postponed load as someone

has called play().

(WebCore::MediaPlayerPrivate::setPreload): New, set m_preload.

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

Reviewed by Adele Peterson.

<rdar://problem/7718442> Implement 'preload=none'
https://bugs.webkit.org/show_bug.cgi?id=35789

Functional test of the 'proload' attribute.

  • media/video-preload-expected.txt: Added.
  • media/video-preload.html: Added.
  • platform/win/Skipped: Skip on Windows for now.
08:53 Changeset [55586] by japhet@chromium.org

2010-03-04 Nate Chapin <japhet@chromium.org>

Reviewed by Dimitri Glazkov.

[V8] Give CodeGeneratorV8.pm a much-needed spring cleaning.

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

  • bindings/scripts/CodeGeneratorV8.pm: Remove unused code, fix style issues, make less hard-coded.
  • bindings/v8/custom/V8DOMWindowCustom.cpp: (WebCore::V8DOMWindow::addEventListenerCallback): (WebCore::V8DOMWindow::removeEventListenerCallback):
08:16 Changeset [55585] by tonikitoo@webkit.org

[Gtk] Implement setSpatialNavigationEnabled in DRT.
https://bugs.webkit.org/show_bug.cgi?id=35705

Reviewed by Holger Freyther
Patch by Antonio Gomes <tonikitoo@webkit.org>

WebKitTools:

  • DumpRenderTree/gtk/DumpRenderTree.cpp:

(resetDefaultsToConsistentValues):

  • DumpRenderTree/gtk/LayoutTestControllerGtk.cpp:

(LayoutTestController::setSpatialNavigationEnabled):

LayoutTests:

  • platform/gtk/Skipped:
08:13 Changeset [55584] by pfeldman@chromium.org

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

Not reviewed. Roll back r55522 since it regreses performance
according to chromium's page cycler.

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

08:07 Changeset [55583] by tonikitoo@webkit.org

[Gtk] Add 'enable-spatial-navigation' setting for Spatial Navigation on/off.
https://bugs.webkit.org/show_bug.cgi?id=35701

Reviewed by Gustavo Noronha.
Patch by Antonio Gomes <tonikitoo@webkit.org>

  • webkit/webkitwebsettings.cpp:

(webkit_web_settings_class_init):
(webkit_web_settings_set_property):
(webkit_web_settings_get_property):
(webkit_web_settings_copy):

  • webkit/webkitwebview.cpp:

(DNDContentsRequest::webkit_web_view_update_settings):
(DNDContentsRequest::webkit_web_view_settings_notify):

08:04 Changeset [55582] by tonikitoo@webkit.org

Add toggle on/off stub methods for Spatial Navigation in win, gtk and mac LayoutTestController class implementations.
https://bugs.webkit.org/show_bug.cgi?id=35699

Reviewed by Kenneth Christiansen.
Patch by Antonio Gomes <tonikitoo@webkit.org>

  • DumpRenderTree/LayoutTestController.cpp:

(setSpatialNavigationEnabledCallback):
(LayoutTestController::staticFunctions):

  • DumpRenderTree/LayoutTestController.h:
  • DumpRenderTree/gtk/LayoutTestControllerGtk.cpp:

(LayoutTestController::setSpatialNavigationEnabled):

  • DumpRenderTree/mac/LayoutTestControllerMac.mm:

(LayoutTestController::setSpatialNavigationEnabled):

  • DumpRenderTree/win/LayoutTestControllerWin.cpp:

(LayoutTestController::setSpatialNavigationEnabled):

08:01 Changeset [55581] by tonikitoo@webkit.org

[Qt] Add toggle on/off mechanism for Spatial Navigation in QtLauncher.

Reviewed by Kenneth Christiansen.
Patch by Antonio Gomes <tonikitoo@webkit.org>

  • QtLauncher/main.cpp:

(LauncherWindow::toggleSpatialNavigation):
(LauncherWindow::setupUI):

07:58 Changeset [55580] by tonikitoo@webkit.org

[Qt] Add setSpatialNavigationEnabled method DRT
https://bugs.webkit.org/show_bug.cgi?id=33715

Reviewed by Simon Hausmann.
Patch by Antonio Gomes <tonikitoo@webkit.org>

Added setSpatialNavigationEnabled support to Qt DRT.

  • DumpRenderTree/qt/DumpRenderTreeQt.cpp:

(WebCore::WebPage::WebPage):
(WebCore::WebPage::resetSettings):

  • DumpRenderTree/qt/LayoutTestControllerQt.cpp:

(LayoutTestController::setSpatialNavigationEnabled):

  • DumpRenderTree/qt/LayoutTestControllerQt.h:

Unskipped fast/events/spatial-navigation/

07:49 Changeset [55579] by tonikitoo@webkit.org

[Qt] QWebSettings attribute for toggle Spatial Navigation on/off
https://bugs.webkit.org/show_bug.cgi?id=33714 (Qt API part)

Reviewed by Simon Hausmann.
Patch by Antonio Gomes <tonikitoo@webkit.org>

Added 'SpatialNavigationEnabled' attribute to QWebSettings.

  • Api/qwebsettings.cpp:

(QWebSettingsPrivate::apply):

  • Api/qwebsettings.h:
07:48 Changeset [55578] by tonikitoo@webkit.org

Extend keyboard navigation to allow directional navigation (LayoutTests - part II)
https://bugs.webkit.org/show_bug.cgi?id=18662

Reviewed by Kenneth Christiansen.
Patch by Antonio Gomes <tonikitoo@webkit.org>

This patch adds two LayoutTest to ensure the basic table traversal correctness of
Spatial Navigation (SNav) algorithm: focusable elements in a <table> should be accessible.

  • fast/events/spatial-navigation/snav-table-traversal.html: Added. Makes sure all elements in a <table> are accessible via Spatial Navigation.
  • fast/events/spatial-navigation/snav-tiny-table-traversal.html: Added. Tests the similar table traversal scenario as above, but in this case the elements in the <table> are styled with 'font-size: small'. That way this test also validades that overlapping rects of elements too closed from each other are correctly handled.
07:46 Changeset [55577] by tonikitoo@webkit.org

Extend keyboard navigation to allow directional navigation (LayoutTests - part I)
https://bugs.webkit.org/show_bug.cgi?id=18662

Reviewed by Kenneth Christiansen.
Patch by Antonio Gomes <tonikitoo@webkit.org>

This patch adds support for auto testing the Spatial Navigation feature by:

1) Creating the directory where the tests are going to live in, named

fast/events/spatial-navigation/.

2) Skipping this directory by Mac, Qt, Gtk and Win.
3) Adding a JS file drives the test execution by processing, for each test,

a given set of arrow key actions and verify if the right expected Element
got the focus after each event processed. This file is
fast/events/spatial-navigation/resources/spatial-navigation-utils.js

  • LayoutTests/fast/events/spatial-navigation/snav-vertical-superprecedence.html: Added.
07:42 QtWebKitJournal edited by hausmann@webkit.org
(diff)
07:31 QtWebKitJournal edited by hausmann@webkit.org
(diff)
07:31 QtWebKitJournal edited by hausmann@webkit.org
(diff)
07:30 QtWebKitJournal edited by hausmann@webkit.org
(diff)
07:28 QtWebKitJournal edited by hausmann@webkit.org
(diff)
07:27 QtWebKitJournal edited by hausmann@webkit.org
(diff)
07:23 QtWebKitJournal edited by hausmann@webkit.org
(diff)
07:23 QtWebKitJournal edited by hausmann@webkit.org
(diff)
07:22 QtWebKitJournal edited by hausmann@webkit.org
(diff)
07:17 QtWebKit edited by hausmann@webkit.org
(diff)
07:11 QtWebKitBugs edited by vestbo@webkit.org
(diff)
06:06 QtWebKit edited by hausmann@webkit.org
(diff)
06:06 QtWebKit edited by hausmann@webkit.org
(diff)
04:58 Changeset [55576] by dino@apple.com

2010-03-05 Dean Jackson <dino@apple.com>

Reviewed by Simon Fraser and Chris Marrin.

Bug 26869: Add fill modes for CSS Animations
https://bugs.webkit.org/show_bug.cgi?id=26869

Tests: animations/fill-mode-removed.html

animations/fill-mode-transform.html
animations/fill-mode.html

  • css/CSSComputedStyleDeclaration.cpp:
  • css/CSSParser.cpp:
  • css/CSSParser.h:
  • css/CSSPropertyNames.in:
  • css/CSSStyleSelector.cpp:
  • css/CSSStyleSelector.h:
  • css/CSSValueKeywords.in:
    • parse, assign and retrieve the value of the new -webkit-animation-fill-mode property
  • page/animation/AnimationBase.cpp:
  • page/animation/AnimationBase.h:
    • new state in animation engine for a finished animation that is "filling" forwards in time. This allows the engine to keep the animation around and not revert to the old style.
    • update the timer code to indicate it doesn't need to keep animating if it is filling
    • now that animations can extend beyond their elapsed time, make sure progress works correctly with iteration counts
  • page/animation/KeyframeAnimation.cpp: (WebCore::KeyframeAnimation::animate):
    • ensure correct style value is returned at the right time by checking for fill mode

(WebCore::KeyframeAnimation::onAnimationEnd):

  • continue to send the end event, but only remove the animation if it isn't filling forwards
  • platform/animation/Animation.cpp:
  • platform/animation/Animation.h:
    • new fill mode member property
  • platform/animation/AnimationList.cpp:
    • ensure the fill mode is propagated to a list of style valus
  • platform/graphics/mac/GraphicsLayerCA.mm:
    • make hardware layers use Core Animation's fill mode
02:09 Changeset [55575] by pfeldman@chromium.org

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

Reviewed by Pavel Feldman.

Web Inspector: Do not show link helper in popovers and/or for external resources.

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

  • English.lproj/localizedStrings.js:
  • inspector/front-end/ResourcesPanel.js: (WebInspector.ResourcesPanel.prototype.canShowSourceLine):
  • inspector/front-end/ScriptsPanel.js: (WebInspector.ScriptsPanel.prototype._scriptOrResourceForURLAndLine):
  • inspector/front-end/TimelinePanel.js: (WebInspector.TimelinePanel.prototype.hide):
  • inspector/front-end/inspector.js: (WebInspector.documentMouseOver): (WebInspector.documentClick.followLink): (WebInspector.documentClick): (WebInspector.addMainEventListeners):
01:51 Changeset [55574] by ukai@chromium.org

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

Reviewed by Alexey Proskuryakov.

WebSocket test to transfer null character
https://bugs.webkit.org/show_bug.cgi?id=35715

  • websocket/tests/null-character-expected.txt: Added.
  • websocket/tests/null-character.html: Added.
  • websocket/tests/script-tests/null-character.js: Added.
01:35 Changeset [55573] by ukai@chromium.org

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

Reviewed by Alexey Proskuryakov.

WebSocket onmessageerror event handler
https://bugs.webkit.org/show_bug.cgi?id=35570

  • websocket/tests/error-detect-expected.txt: Added.
  • websocket/tests/error-detect.html: Added.
  • websocket/tests/script-tests/error-detect.js: Added.
  • websocket/tests/unknown-frame-type_wsh.py: Added.

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

Reviewed by Alexey Proskuryakov.

WebSocket onmessageerror event handler
https://bugs.webkit.org/show_bug.cgi?id=35570

Test: websocket/tests/error-detect.html

  • websockets/WebSocket.cpp: (WebCore::WebSocketChannel::didConnect): assert scriptExecutionContext. (WebCore::WebSocketChannel::didReceiveMessage): assert scriptExecutionContext. (WebCore::WebSocket::didReceiveMessageError): (WebCore::WebSocket::didClose): assert scriptExecutionContext.
  • websockets/WebSocket.h:
  • websockets/WebSocket.idl: Add onerror event listener
  • websockets/WebSocketChannel.cpp: (WebCore::WebSocketChannel::didReceiveData): call didReceiveMessageError if unknown frame tye is detected, or frame length overflowed.
  • websockets/WebSocketChannelClient.h: (WebCore::WebSocketChannelClient::didReceiveMessageError):

03/04/10:

22:09 Changeset [55572] by eric@webkit.org

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

Reviewed by Darin Fisher.

Changing private members from PlatformTouchEvent and PlatformTouchPoint
to be protected, so that Chromium's PlatformTouchEventBuilder and
PlatformTouchPointBuilder can access them. Exercised by the
fast/events/touch tests.
https://bugs.webkit.org/show_bug.cgi?id=35760

  • platform/PlatformTouchEvent.h: (WebCore::PlatformTouchEvent::~PlatformTouchEvent):
  • platform/PlatformTouchPoint.h: (WebCore::PlatformTouchPoint::~PlatformTouchPoint):

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

Reviewed by Darin Fisher.

Adding PlatformTouchEventBuilder and PlatformTouchPointBuilder for
converting Chromium WebTouchEvent and WebTouchPoint types to
corresponding WebCore types.
https://bugs.webkit.org/show_bug.cgi?id=35760

  • src/WebInputEventConversion.cpp: (WebKit::toPlatformTouchEventType): (WebKit::toPlatformTouchPointState): (WebKit::PlatformTouchPointBuilder::PlatformTouchPointBuilder): (WebKit::PlatformTouchEventBuilder::PlatformTouchEventBuilder):
  • src/WebInputEventConversion.h:
21:52 Changeset [55571] by eric@webkit.org

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

Reviewed by Darin Fisher.

[chromium] WebDocument:applicationID() crashes on <meta> tag with no http-equiv attribute
https://bugs.webkit.org/show_bug.cgi?id=35771

  • src/WebDocument.cpp: (WebKit::WebDocument::applicationID): check for presence of http-equiv attribute before lower()ing it.
21:50 Changeset [55570] by ukai@chromium.org

WebSocket crash bug when reloading the page while the WebSocket is busy
https://bugs.webkit.org/show_bug.cgi?id=35732

Reviewed by Alexey Proskuryakov.

WebCore:

  • websockets/WebSocketChannel.cpp:

(WebCore::WebSocketChannel::disconnect): clear m_context from WebSocketChannel and WebSocketHandshake.
(WebCore::WebSocketChannel::didOpen): check m_context
(WebCore::WebSocketChannel::didClose): clear m_context
(WebCore::WebSocketChannel::didReceiveData): check m_context, and check m_client early before appending to buffer

  • websockets/WebSocketHandshake.cpp:

(WebCore::WebSocketHandshake::clearScriptExecutionContext): Added.

  • websockets/WebSocketHandshake.h:

LayoutTests:

  • websocket/tests/reload-crash-expected.txt: Added.
  • websocket/tests/reload-crash.html: Added.
  • websocket/tests/resoures/reload-crash-iframe.html: Added.
21:35 Changeset [55569] by eric@webkit.org

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

Reviewed by Alexey Proskuryakov.

Refactoring: XMLHTTPRequest.open() should have all overloaded implementations
https://bugs.webkit.org/show_bug.cgi?id=35630

  • http/tests/xmlhttprequest/open-async-overload-expected.txt: Added.
  • http/tests/xmlhttprequest/open-async-overload.html: Added.

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

Reviewed by Alexey Proskuryakov.

Refactoring: XMLHTTPRequest.open() should have all overloaded implementations
https://bugs.webkit.org/show_bug.cgi?id=35630

Test: http/tests/xmlhttprequest/open-async-overload.html

  • bindings/js/JSXMLHttpRequestCustom.cpp: (WebCore::JSXMLHttpRequest::open):
  • bindings/v8/custom/V8XMLHttpRequestCustom.cpp: (WebCore::V8XMLHttpRequest::openCallback):
  • xml/XMLHttpRequest.cpp: (WebCore::XMLHttpRequest::open):
  • xml/XMLHttpRequest.h:
21:18 Changeset [55568] by eric@webkit.org

2010-03-04 James Robinson <jamesr@google.com>

Reviewed by Eric Seidel.

Styles do not have to be synchronously rematched after every event dispatch

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

  • bindings/js/JSEventListener.cpp: (WebCore::JSEventListener::handleEvent):
  • bindings/v8/V8AbstractEventListener.cpp: (WebCore::V8AbstractEventListener::handleEvent):
21:01 Changeset [55567] by eric@webkit.org

2010-03-04 Luiz Agostini <luiz.agostini@openbossa.org>

Reviewed by Kenneth Rohde Christiansen.

maemo spelled as mameo in WebCore.pro
https://bugs.webkit.org/show_bug.cgi?id=35765

Spelling corrected.

  • WebCore.pro:
20:34 Changeset [55566] by tony@chromium.org

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

Reviewed by Eric Seidel.

https://bugs.webkit.org/show_bug.cgi?id=18819
Pressing option+page{down,up} should move the cursor and scroll in
content editable areas. On other platforms, pressing page{down,up}
should move the cursor and scroll in content editable areas.

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

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

Reviewed by Eric Seidel.

https://bugs.webkit.org/show_bug.cgi?id=18819
Pressing option+page{down,up} should move the cursor and scroll in
content editable areas. On other platforms, pressing page{down,up}
should move the cursor and scroll in content editable areas.

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

  • editing/EditorCommand.cpp: (WebCore::verticalScrollDistance):
19:42 Changeset [55565] by mrowe@apple.com

Find the framework relative to TARGET_BUILD_DIR as that has a more obvious value during production builds.

Rubber-stamped by Alice Liu.

  • Scripts/check-for-webkit-framework-include-consistency:
17:33 Changeset [55564] by oliver@apple.com

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

Reviewed by Gavin Barraclough.

Allow static property getters to interact with JSCs caching
https://bugs.webkit.org/show_bug.cgi?id=35716

Add new opcodes for handling cached lookup of static value getters.
More or less the same as with JS getters, all that changes is that
instead of calling through a JSFunction we always know that we have
a C function to call.

For the patching routines in the JIT we now need to pass a few
new parameters to allow us to pass enough information to the stub
function to allow us to call the C function correctly. Logically
this shouldn't actually be necessary as all of these functions ignore
the identifier, but removing the ident parameter would require
somewhat involved changes to the way we implement getOwnPropertySlot,
etc.

  • bytecode/CodeBlock.cpp: (JSC::CodeBlock::dump): (JSC::CodeBlock::derefStructures): (JSC::CodeBlock::refStructures):
  • bytecode/Instruction.h: (JSC::Instruction::Instruction): (JSC::Instruction::):
  • bytecode/Opcode.h:
  • interpreter/Interpreter.cpp: (JSC::Interpreter::tryCacheGetByID): (JSC::Interpreter::privateExecute):
  • jit/JIT.cpp: (JSC::JIT::privateCompileMainPass):
  • jit/JIT.h: (JSC::JIT::compileGetByIdProto): (JSC::JIT::compileGetByIdSelfList): (JSC::JIT::compileGetByIdProtoList): (JSC::JIT::compileGetByIdChainList): (JSC::JIT::compileGetByIdChain):
  • jit/JITPropertyAccess.cpp: (JSC::JIT::privateCompileGetByIdProto): (JSC::JIT::privateCompileGetByIdSelfList): (JSC::JIT::privateCompileGetByIdProtoList): (JSC::JIT::privateCompileGetByIdChainList): (JSC::JIT::privateCompileGetByIdChain):
  • jit/JITPropertyAccess32_64.cpp: (JSC::JIT::privateCompileGetByIdProto): (JSC::JIT::privateCompileGetByIdSelfList): (JSC::JIT::privateCompileGetByIdProtoList): (JSC::JIT::privateCompileGetByIdChainList): (JSC::JIT::privateCompileGetByIdChain):
  • jit/JITStubs.cpp: (JSC::JITThunks::tryCacheGetByID): (JSC::DEFINE_STUB_FUNCTION):
  • jit/JITStubs.h: (JSC::):
  • runtime/JSFunction.cpp: (JSC::JSFunction::getOwnPropertySlot):
  • runtime/Lookup.h: (JSC::getStaticPropertySlot): (JSC::getStaticValueSlot):
  • runtime/PropertySlot.h: (JSC::PropertySlot::): (JSC::PropertySlot::PropertySlot): (JSC::PropertySlot::cachedPropertyType): (JSC::PropertySlot::isCacheable): (JSC::PropertySlot::isCacheableValue): (JSC::PropertySlot::setValueSlot): (JSC::PropertySlot::setCacheableCustom): (JSC::PropertySlot::setGetterSlot): (JSC::PropertySlot::setCacheableGetterSlot): (JSC::PropertySlot::clearOffset): (JSC::PropertySlot::customGetter):

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

Reviewed by Gavin Barraclough.

Allow static property getters to interact with JSCs caching
https://bugs.webkit.org/show_bug.cgi?id=35716

Add tests to ensure nothing horrifying happens to static property
getters if they're in a path where we end up caching lookups.

  • fast/js/pic/cached-named-property-getter-expected.txt: Added.
  • fast/js/pic/cached-named-property-getter.html: Added.

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

Reviewed by Gavin Barraclough.

Allow static property getters to interact with JSCs caching
https://bugs.webkit.org/show_bug.cgi?id=35716

Update the obviously safe getters to allow caching

Test: fast/js/pic/cached-named-property-getter.html

  • bridge/runtime_array.cpp: (JSC::RuntimeArray::getOwnPropertySlot):
  • bridge/runtime_method.cpp: (JSC::RuntimeMethod::getOwnPropertySlot):
16:50 Changeset [55563] by tkent@chromium.org

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

Reviewed by Dimitri Glazkov.

[DRT/Chromium] Add CppVariant and CppBoundClass
https://bugs.webkit.org/show_bug.cgi?id=35634

Add CppVariant and CppBoundClass classes, which are going to be
used by DumpRenderTree Chromium port. These files are based on:

  • src/webkit/glue/cpp_variant.{cc,h} and
  • src/webkit/glue/cpp_bound_class.{cc,h} of Chromium rev.40492.
  • DumpRenderTree/chromium/CppBoundClass.cpp: Added.
  • DumpRenderTree/chromium/CppBoundClass.h: Added.
  • DumpRenderTree/chromium/CppVariant.cpp: Added.
  • DumpRenderTree/chromium/CppVariant.h: Added.
16:29 Changeset [55562] by mrowe@apple.com

Build fix for older versions of Ruby.

  • Scripts/check-for-webkit-framework-include-consistency:
16:23 Changeset [55561] by mrowe@apple.com

Add a script to verify that WebKit framework headers are internally consistent.

Reviewed by Sam Weinig.

WebKit:

  • WebKit.xcodeproj/project.pbxproj: Run the script during the build and fail with

an error should the consistency check fail.

WebKitTools:

This script detects the following problematic situations:

  • An exported WebKit header that includes a header from WebCore.
  • An exported WebKit header that includes a header that does not exist in the WebKit framework.
  • A public WebKit header that includes a private WebKit header.
  • Scripts/check-for-webkit-framework-include-consistency: Added.
16:23 Changeset [55560] by mrowe@apple.com

<rdar://problem/7717249> DOMSVG.h includes a non-existent DOMSVGFEMorphologyElement.h

Reviewed by Sam Weinig.

WebCore:

  • WebCore.xcodeproj/project.pbxproj: Copy DOMSVGFEMorphologyElement.h and DOMSVGFEMorphologyElementInternal.h

in to the framework wrapper.

WebKit/mac:

  • MigrateHeaders.make: Migrate DOMSVGFEMorphologyElement.h and DOMSVGFEMorphologyElementInternal.h.
16:23 Changeset [55559] by mrowe@apple.com

Use a framework-style include to pull in WebInspector.h for consistency
with other parts of WebKit.

Reviewed by Sam Weinig.

  • WebInspector/WebInspectorPrivate.h:
15:34 Changeset [55558] by dumi@chromium.org

Making sure that the correct path is set before invoking
DumpRenderTree on cygwin.

Reviewed by Adam Roben.

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

  • Scripts/run-webkit-tests:
15:29 Changeset [55557] by jamesr@google.com

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

Reviewed by Dimitri Glazkov.

Handles setting HTMLSelectElement.length with mutation handlers present
https://bugs.webkit.org/show_bug.cgi?id=33983

When setting an HTMLSelectElement's length attribute, option elements have to be added or removed to the select
as appropriate. This is a little tricky with mutation events since they might add, remove, or reorder elements
while option elements are being added or deleted.

Tests: fast/forms/select-set-length-optgroup.html

fast/forms/select-set-length-with-mutation-remove.html
fast/forms/select-set-length-with-mutation-reorder.html
fast/forms/select-set-length-with-mutation-reparent.html
fast/forms/select-set-length-with-mutation.html
fast/forms/select-set-length.html

  • html/HTMLSelectElement.cpp: (WebCore::HTMLSelectElement::setLength):

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

Reviewed by Dimitri Glazkov.

Tests for setting an select's options.length attribute.
https://bugs.webkit.org/show_bug.cgi?id=33983

  • fast/forms/script-tests/select-set-length-optgroup.js: Added.
  • fast/forms/script-tests/select-set-length-with-mutation-remove.js: Added. (gc): (onRemove):
  • fast/forms/script-tests/select-set-length-with-mutation-reorder.js: Added. (onRemove):
  • fast/forms/script-tests/select-set-length-with-mutation-reparent.js: Added. (onRemove):
  • fast/forms/script-tests/select-set-length-with-mutation.js: Added. (sel):
  • fast/forms/script-tests/select-set-length.js: Added.
  • fast/forms/select-set-length-expected.txt: Added.
  • fast/forms/select-set-length-optgroup-expected.txt: Added.
  • fast/forms/select-set-length-optgroup.html: Added.
  • fast/forms/select-set-length-with-mutation-expected.txt: Added.
  • fast/forms/select-set-length-with-mutation-remove-expected.txt: Added.
  • fast/forms/select-set-length-with-mutation-remove.html: Added.
  • fast/forms/select-set-length-with-mutation-reorder-expected.txt: Added.
  • fast/forms/select-set-length-with-mutation-reorder.html: Added.
  • fast/forms/select-set-length-with-mutation-reparent-expected.txt: Added.
  • fast/forms/select-set-length-with-mutation-reparent.html: Added.
  • fast/forms/select-set-length-with-mutation.html: Added.
  • fast/forms/select-set-length.html: Added.
15:21 Changeset [55556] by ossy@webkit.org

[Qt] Unreviewed buildfix after r55542 on Windows.

  • WebCore.pro:
    • plugins/win/PluginDatabaseWin.cpp removed.
    • platform/win/WebCoreInstanceHandle.cpp added.
  • platform/win/WebCoreInstanceHandle.cpp: Remove explicit qualification in namespace declaration to make gcc happy.
15:01 Changeset [55555] by japhet@chromium.org

2010-03-04 Nate Chapin <japhet@chromium.org>

Reviewed by Kenneth Rohde Christiansen.

Fix the case where we do a same document navigation, scroll,
then repeat the same document navigation. Currently, the second
navigation does nothing.

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

Test: fast/loader/repeat-same-document-navigation.html

  • loader/FrameLoader.cpp: (WebCore::FrameLoader::loadInSameDocument): Call scrollToFragment() whether or not the hash changed.
14:44 QtWebKitBugs edited by jocelyn.turcotte@nokia.com
(diff)
14:31 Changeset [55554] by simon.fraser@apple.com

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

Build fix.

const long long cMaxDistance = numeric_limits<long long>::max() created global
initializers at the call sites, so replace with an inline function.

  • page/FocusController.cpp: (WebCore::updateFocusCandidateIfCloser):
  • page/SpatialNavigation.cpp: (WebCore::distanceInDirection):
  • page/SpatialNavigation.h: (WebCore::maxDistance): (WebCore::FocusCandidate::FocusCandidate):
14:29 QtWebKitBugs edited by jocelyn.turcotte@nokia.com
Repeat reporting link. Fill out the crashes part (diff)
14:15 Changeset [55553] by simon.fraser@apple.com

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

Build fix: add exceptions to the check-for-global-initializers script
for FocusController and SpatialNavigation, and improve the script
to actually print out the globals found.

  • Scripts/check-for-global-initializers:
13:52 Changeset [55552] by tonikitoo@webkit.org

Unreviewed attempt to fix Windows build.

  • page/SpatialNavigation.cpp:

(WebCore::spatialDistance):

13:40 Changeset [55551] by dpranke@chromium.org

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

Reviewed by Dimitri Glazkov.

Fix dangling reference to "port" instead of "self._port_obj" that
was preventing the http_server from starting on Windows.

  • Scripts/webkitpy/layout_tests/port/http_server.py:
13:35 Changeset [55550] by simon.fraser@apple.com

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

Build fix.

Delcare updateFocusCandidateIfCloser static to avoid warning.

  • page/FocusController.cpp: (WebCore::updateFocusCandidateIfCloser):
13:11 Changeset [55549] by darin@chromium.org

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

Reviewed by Dmitry Titov.

[chromium] make history.{push,replace}State enabled at runtime
https://bugs.webkit.org/show_bug.cgi?id=35753

  • bindings/generic/RuntimeEnabledFeatures.cpp:
  • bindings/generic/RuntimeEnabledFeatures.h: (WebCore::RuntimeEnabledFeatures::setPushStateEnabled): (WebCore::RuntimeEnabledFeatures::pushStateEnabled): (WebCore::RuntimeEnabledFeatures::replaceStateEnabled):
  • page/History.idl:
13:09 Changeset [55548] by tonikitoo@webkit.org

Unreviewed attempt to (again) fix Mac build.

  • page/SpatialNavigation.cpp:

(WebCore::spatialDistance):

13:03 Changeset [55547] by mitz@apple.com

Fix typos

13:02 Changeset [55546] by mitz@apple.com

WebCore: Based on a pacth from Nick Jong.

Reviewed by Simon Fraser.

Improve selection in multi-column blocks when hitting points above or
below a column rect.

Test: fast/multicol/hit-test-above-or-below.html

  • rendering/RenderBlock.cpp:

(WebCore::RenderBlock::adjustPointToColumnContents): If the point lies
within the horizontal range for a column, constrain it to the column (if
it is above) or the next column (if it is below).

LayoutTests: Added test for hit-testing points above or below the column rect.

Reviewed by Simon Fraser.

  • fast/multicol/hit-test-above-or-below-expected.txt: Added.
  • fast/multicol/hit-test-above-or-below.html: Added.
12:57 Changeset [55545] by tonikitoo@webkit.org

Attempt to fix Mac build.

Reviewed by Simon Frase.

  • page/SpatialNavigation.cpp:
12:20 Changeset [55544] by tonikitoo@webkit.org

Add SpatialNavigation header and cpp to Mac XCode project file

Reviewed by Kenneth Christiansen.
Patch by Jesus Palencia <jesus.palencia@openbossa.org>

  • WebCore.xcodeproj/project.pbxproj:
12:19 Changeset [55543] by tonikitoo@webkit.org

Extend keyboard navigation to allow directional navigation
https://bugs.webkit.org/show_bug.cgi?id=18662

Reviewed by Simon Fraser, Eric Seidel and Darin Adler.
Patch by Antonio Gomes <tonikitoo@webkit.org>
Based on the initial work of Marco Barisione <marco.barisione@collabora.co.uk>

This patch implements the core logic of the 'Spatial Navigation' feature [1].
It improves the accessibility support of WebCore by extending the basic keyboard
navigation currently available (based on Tab forward and backward) with the
addition of a two-dimensional directional navigation by using Left, Right, Up and
Down arrow keys to move to the "nearest" element in the corresponding direction.

Highlights:

  • Feature is turned off by default in Settings. Port specific APIs need to be added for toggling it on/off.
  • Only elements viewed in the current viewport can have focus move to it. If the "nearest" is not in viewport dimensions, then a scroll-in-direction action is performed.
  • The layout tests added run on Qt's DRT only for now (skipped for Mac, Win and Gtk).

Known issues (to be covered in follow-up bugs):

  • Add port specific hooks to each DRT to enable/disable Spatial Navigation.
  • Support for spatial navigation through form elements (<input>, <select>, etc) is be added.
  • Make navigation keys customizable. It currently works with arrows keys only (up, down, right and left).
  • Make it support modifiers (Alt, Ctrl and Shift).

[1] http://en.wikipedia.org/wiki/Spatial_navigation

  • Android.mk:
  • GNUmakefile.am:
  • WebCore.gypi:
  • WebCore.pro:
  • WebCore.vcproj/WebCore.vcproj:
  • page/EventHandler.cpp:

(WebCore::EventHandler::defaultKeyboardEventHandler):
(WebCore::EventHandler::focusDirectionForKey):
(WebCore::EventHandler::defaultArrowEventHandler):

  • page/EventHandler.h:
  • page/FocusController.cpp:

(WebCore::FocusController::advanceFocus):
(WebCore::FocusController::advanceFocusInDocumentOrder):
(WebCore::FocusController::advanceFocusDirectionally):
(WebCore::updateFocusCandidateIfCloser):
(WebCore::FocusController::findFocusableNodeInDirection):
(WebCore::FocusController::deepFindFocusableNodeInDirection):

  • page/FocusController.h:
  • page/FocusDirection.h:

(WebCore::):

  • page/Settings.cpp:

(WebCore::Settings::Settings):
(WebCore::Settings::setSpatialNavigationEnabled):

  • page/Settings.h:

(WebCore::Settings::isSpatialNavigationEnabled):

  • page/SpatialNavigation.cpp: Added.

(WebCore::distanceInDirection):
(WebCore::renderRectRelativeToRootDocument):
(WebCore::alignmentForRects):
(WebCore::isHorizontalMove):
(WebCore::areRectsFullyAligned):
(WebCore::areRectsPartiallyAligned):
(WebCore::spatialDistance):
(WebCore::isRectInDirection):
(WebCore::hasOffscreenRect):
(WebCore::scrollInDirection):
(WebCore::isInRootDocument):
(WebCore::deflateIfOverlapped):

  • page/SpatialNavigation.h: Added.

(WebCore::):
(WebCore::FocusCandidate::FocusCandidate):

12:08 Changeset [55542] by bdakin@apple.com

WebCore: This fixes the layering violation I committed yesterday by moving
Page::instanceHandle into its own file just in the WebCore
namespace.

Reviewed by Anders Carlsson.

Added WebCoreInstanceHandle.h + .cpp and removed PageWin.cpp

  • WebCore.vcproj/WebCore.vcproj:
  • page/win/PageWin.cpp: Removed.

Remove all instance handle code from Page.

  • page/Page.h:

New files.

  • platform/win/WebCoreInstanceHandle.cpp: Added.
  • platform/win/WebCoreInstanceHandle.h: Added.

(WebCore::setInstanceHandle):
(WebCore::instanceHandle):

Switch to WebCore::instanceHandle() instead of
Page::instanceHandle() and include the new header.

  • platform/graphics/win/WKCACFLayerRenderer.cpp:

(WebCore::WKCACFLayerRenderer::acceleratedCompositingAvailable):

  • platform/win/PasteboardWin.cpp:

(WebCore::Pasteboard::Pasteboard):

  • platform/win/PopupMenuWin.cpp:

(WebCore::PopupMenu::show):
(WebCore::PopupMenu::registerClass):

  • platform/win/SharedTimerWin.cpp:

(WebCore::initializeOffScreenTimerWindow):

  • plugins/win/PluginViewWin.cpp:

(WebCore::registerPluginView):
(WebCore::PluginView::platformStart):

WebKit/win: WebCore::Page::setInstanceHandle() is now just
WebCore::setInstanceHandle()

Reviewed by Anders Carlsson.

  • WebKitDLL.cpp:

(DllMain):

11:30 Changeset [55541] by eric@webkit.org

2010-03-04 Diego Gonzalez <diego.gonzalez@openbossa.org>

Reviewed by Simon Hausmann.

Make Qt DumpRenderTree EventSender able to send double click events

LayoutTests:

fast/events/dblclick-addEventListener.html

[Qt] DRT: Send double click event from EventSender
https://bugs.webkit.org/show_bug.cgi?id=35255

  • platform/qt/Skipped:

2010-03-04 Diego Gonzalez <diego.gonzalez@openbossa.org>

Reviewed by Simon Hausmann.

Make Qt DumpRenderTree EventSender able to send double click events

LayoutTests:

fast/events/dblclick-addEventListener.html

[Qt] DRT: Send double click event from EventSender
https://bugs.webkit.org/show_bug.cgi?id=35255

  • DumpRenderTree/qt/DumpRenderTreeQt.cpp: (WebCore::DumpRenderTree::resetToConsistentStateBeforeTesting):
  • DumpRenderTree/qt/EventSenderQt.cpp: (EventSender::EventSender): (EventSender::mouseDown):
  • DumpRenderTree/qt/EventSenderQt.h: (EventSender::resetClickCount):
11:15 Changeset [55540] by tonikitoo@webkit.org

[Gtk] Unreviewed build fix.
Patch by Antonio Gomes <tonikitoo@webkit.org>

'glib_file_name' renamed to 'gligFileName'.

  • platform/graphics/gtk/ImageGtk.cpp:

(WebCore::Image::loadPlatformResource):

11:12 Changeset [55539] by eric@webkit.org

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

Reviewed by Kenneth Rohde Christiansen.

[Qt] qwebelement.h does not include QtCore headers correctly
https://bugs.webkit.org/show_bug.cgi?id=35748

The header files of QtCore must be included as QtCore/foo.h.

See also http://bugreports.qt.nokia.com/browse/QTBUG-8661

  • Api/qwebelement.h:
11:03 Changeset [55538] by pfeldman@chromium.org

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

Not reviewed: chromium tests fix.

  • src/js/Tests.js:
10:56 Changeset [55537] by pfeldman@chromium.org

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

Reviewed by Timothy Hatcher.

Web Inspector: Remove two last synchronous calls from front-end to InspectorBackend.

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

  • inspector/InspectorBackend.cpp: (WebCore::InspectorBackend::enableSearchingForNode): (WebCore::InspectorBackend::disableSearchingForNode): (WebCore::InspectorBackend::setPauseOnExceptionsState):
  • inspector/InspectorBackend.h:
  • inspector/InspectorBackend.idl:
  • inspector/InspectorController.cpp: (WebCore::InspectorController::setWindowVisible): (WebCore::InspectorContorller::setSearchingForNode): (WebCore::InspectorController::populateScriptObjects):
  • inspector/InspectorController.h:
  • inspector/InspectorFrontend.cpp: (WebCore::InspectorFrontend::searchingForNodeWasEnabled): (WebCore::InspectorFrontend::searchingForNodeWasDisabled): (WebCore::InspectorFrontend::updatePauseOnExceptionsState):
  • inspector/InspectorFrontend.h:
  • inspector/front-end/ElementsPanel.js: (WebInspector.ElementsPanel): (WebInspector.ElementsPanel.prototype.get statusBarItems): (WebInspector.ElementsPanel.prototype.hide): (WebInspector.ElementsPanel.prototype.reset): (WebInspector.ElementsPanel.prototype.searchingForNodeWasEnabled): (WebInspector.ElementsPanel.prototype.searchingForNodeWasDisabled): (WebInspector.ElementsPanel.prototype._nodeSearchButtonClicked):
  • inspector/front-end/InspectorBackendStub.js: (.WebInspector.InspectorBackendStub.prototype.enableSearchingForNode): (.WebInspector.InspectorBackendStub.prototype.disableSearchingForNode): (.WebInspector.InspectorBackendStub.prototype.setPauseOnExceptionsState):
  • inspector/front-end/ScriptsPanel.js: (WebInspector.ScriptsPanel): (WebInspector.ScriptsPanel.prototype.get statusBarItems): (WebInspector.ScriptsPanel.prototype.updatePauseOnExceptionsState): (WebInspector.ScriptsPanel.prototype._updateDebuggerButtons): (WebInspector.ScriptsPanel.prototype._togglePauseOnExceptions):
  • inspector/front-end/inspector.js: (WebInspector.searchingForNodeWasEnabled): (WebInspector.searchingForNodeWasDisabled): (WebInspector.updatePauseOnExceptionsState): WebKit/chromium:
  • src/js/InspectorControllerImpl.js: (devtools.InspectorBackendImpl):
10:48 Changeset [55536] by pfeldman@chromium.org

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

Reviewed by Timothy Hatcher.

Web Inspector: Clicking on an error should take me to the error

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

  • inspector/front-end/ResourceView.js: (WebInspector.ResourceView): (WebInspector.ResourceView.prototype._selectTab): (WebInspector.ResourceView.prototype.selectContentTab):
  • inspector/front-end/ResourcesPanel.js: (WebInspector.ResourcesPanel.prototype.showResource):
10:46 Changeset [55535] by pfeldman@chromium.org

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

Reviewed by Timothy Hatcher.

Web Inspector: Remove sync access to resourceTrackingEnabled.

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

  • inspector/InspectorBackend.cpp:
  • inspector/InspectorBackend.h:
  • inspector/InspectorBackend.idl:
  • inspector/InspectorController.cpp: (WebCore::InspectorController::populateScriptObjects):
  • inspector/front-end/AuditsPanel.js: (WebInspector.AuditsPanel.prototype._reloadResources): (WebInspector.AuditsPanel.prototype.show):
  • inspector/front-end/InspectorBackendStub.js: (.WebInspector.InspectorBackendStub): (.WebInspector.InspectorBackendStub.prototype.enableResourceTracking): (.WebInspector.InspectorBackendStub.prototype.disableResourceTracking): (.WebInspector.InspectorBackendStub.prototype.enableDebugger): (.WebInspector.InspectorBackendStub.prototype.disableDebugger): (.WebInspector.InspectorBackendStub.prototype.enableProfiler): (.WebInspector.InspectorBackendStub.prototype.disableProfiler):
  • inspector/front-end/ResourcesPanel.js: (WebInspector.ResourcesPanel): (WebInspector.ResourcesPanel.prototype.get resourceTrackingEnabled): (WebInspector.ResourcesPanel.prototype.get visibleView): (WebInspector.ResourcesPanel.prototype.resourceTrackingWasEnabled): (WebInspector.ResourcesPanel.prototype.resourceTrackingWasDisabled): (WebInspector.ResourcesPanel.prototype.reset): (WebInspector.ResourcesPanel.prototype.showResource): (WebInspector.ResourcesPanel.prototype._enableResourceTracking): (WebInspector.ResourcesPanel.prototype._toggleResourceTracking):
10:43 Changeset [55534] by jam@chromium.org

Update test expectations from my recent checkin

10:43 Changeset [55533] by chang.shu@nokia.com

Committed by Chang Shu: commit manually for bug 35687.

10:17 Changeset [55532] by eric@webkit.org

2010-03-04 Antoine Quint <ml@graougraou.com>

Reviewed by Darin Adler.

DOM insertion mutation events should dispatch after a node is attached to the render tree
https://bugs.webkit.org/show_bug.cgi?id=35590

  • fast/events/domnodeinsertedintodocument-dispatched-post-rendering-expected.txt: Added.
  • fast/events/domnodeinsertedintodocument-dispatched-post-rendering.html: Added.

2010-03-04 Antoine Quint <ml@graougraou.com>

Reviewed by Darin Adler.

DOM insertion mutation events should dispatch after a node is attached to the render tree
https://bugs.webkit.org/show_bug.cgi?id=35590

Test: fast/events/domnodeinsertedintodocument-dispatched-post-rendering.html

Split off the internal-to-WebCore node insertion notification code from the DOM mutation
event dispatching, originally in dispatchChildInsertionEvents(), to a new static function
called notifyChildInserted(). This allows us to dispatch the mutation events at a later
time upon insertion of a child into to the tree, specifically _after_ attachment to the render
tree.

  • dom/ContainerNode.cpp: (WebCore::ContainerNode::insertBefore): (WebCore::ContainerNode::replaceChild): (WebCore::ContainerNode::appendChild): (WebCore::notifyChildInserted): (WebCore::dispatchChildInsertionEvents):
10:00 Changeset [55531] by eric@webkit.org

2010-03-04 Fridrich Strba <fridrich.strba@bluewin.ch>

Reviewed by Holger Freyther.

Make paths relocatable on runtime on Windows
https://bugs.webkit.org/show_bug.cgi?id=32711

  • platform/graphics/gtk/ImageGtk.cpp: (get_webkit_datadir): (WebCore::Image::loadPlatformResource):
09:35 Changeset [55530] by pfeldman@chromium.org

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

Reviewed by Timothy Hatcher.

Web Inspector: Popup for Timeline panel will work in a tooltip mode

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

09:25 Changeset [55529] by beidson@apple.com

REGRESSION (31281): -[WebArchive initWithCoder:] leaks
<rdar://problem/7702420> and https://bugs.webkit.org/show_bug.cgi?id=35534

Reviewed by Geoff Garen.

  • WebView/WebArchive.mm:

(-[WebArchive initWithCoder:]): Don't retain objects we don't own.

09:18 Changeset [55528] by laszlo.1.gombos@nokia.com

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

Unreviewed build fix.

Build fix after r55464.

No new tests, no new functionality.

  • inspector/InspectorController.cpp: (WebCore::InspectorController::populateScriptObjects):
08:14 Changeset [55527] by ossy@webkit.org

Unreviewed.

[Qt] storage/open-database-creation-callback-isolated-world.html was removed
from tree by r55488, so it should be removed from Qt Skipped list too.

  • platform/qt/Skipped: storage/open-database-creation-callback-isolated-world.html removed.
07:41 Changeset [55526] by eric@webkit.org

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

Reviewed by Kenneth Rohde Christiansen.

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

  • platform/graphics/efl/FloatRectEfl.cpp: Added.
  • platform/graphics/efl/FontEfl.cpp: Added.
  • platform/graphics/efl/IconEfl.cpp: Added.
  • platform/graphics/efl/ImageEfl.cpp: Added.
  • platform/graphics/efl/IntPointEfl.cpp: Added.
  • platform/graphics/efl/IntRectEfl.cpp: Added.
07:25 Changeset [55525] by eric@webkit.org

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

Reviewed by Kenneth Rohde Christiansen.

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

  • platform/text/efl/TextBreakIteratorInternalICUEfl.cpp: Added.
07:03 Changeset [55524] by eric@webkit.org

2010-03-04 Fridrich Strba <fridrich.strba@bluewin.ch>

Reviewed by Holger Freyther.

https://bugs.webkit.org/show_bug.cgi?id=35726
Remove orphaned #ifdef WTF_USE_GLIB_ICU_UNICODE_HYBRID

Removing orphaned #if USE.

  • GNUmakefile.am:

2010-03-04 Fridrich Strba <fridrich.strba@bluewin.ch>

Reviewed by Holger Freyther.

https://bugs.webkit.org/show_bug.cgi?id=35726
Remove orphaned #ifdef WTF_USE_GLIB_ICU_UNICODE_HYBRID

Removing orphaned #if USE.

  • platform/ThreadGlobalData.h:
06:24 Changeset [55523] by jorlow@chromium.org

Trottle sync requests sent to the LocalStorage background thread
https://bugs.webkit.org/show_bug.cgi?id=34943

Reviewed by Darin Fisher.

Currently, once a second LocalStorage takes all keys/values which have
been changed and sends them to a background thread to sync. The problem
is that this background thread can get overwhelmed and stop being
responsive. This means that if any other page tries to start using
LocalStorage (and thus initiates the initial import) that'll block on
all the previous syncs completing.

To mitigate this, I'm adding code so that we never schedule another
sync task when another is still running. In order to keep the sync
tasks from growing exponentially when they do take longer than the
storage sync interval, I've also added a basic rate limiter. No effort
is made to ensure fairness/ordering of what gets synced nor is there
any way for this rate to be changed because most normal uses of
LocalStorage really shouldn't be hitting these types of limits anyway.

The only behavioral change that's observible in JavaScript is time based
and thus it's not practical to make new tests that aren't racy. The
existing layout tests cover LocalStorage pretty well, though.

  • storage/StorageAreaSync.cpp:

(WebCore::StorageAreaSync::StorageAreaSync):
(WebCore::StorageAreaSync::scheduleFinalSync):
(WebCore::StorageAreaSync::syncTimerFired):
(WebCore::StorageAreaSync::performSync):

  • storage/StorageAreaSync.h:
06:00 Changeset [55522] by pfeldman@chromium.org

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

Reviewed by Pavel Feldman.

Added support for worker instrumentation in inspector
(display list of active workers, allow debugging of workers
by injecting fake JS implementation)

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

  • English.lproj/localizedStrings.js:
  • WebCore.gypi:
  • WebCore.vcproj/WebCore.vcproj:
  • bindings/js/JSInjectedScriptHostCustom.cpp: (WebCore::InjectedScriptHost::createInjectedScript): (WebCore::InjectedScriptHost::injectedScriptFor):
  • bindings/v8/custom/V8InjectedScriptHostCustom.cpp: (WebCore::InjectedScriptHost::createInjectedScript): (WebCore::InjectedScriptHost::injectedScriptFor):
  • dom/Document.cpp: (WebCore::Document::inspectorController):
  • dom/Document.h:
  • dom/ScriptExecutionContext.h: (WebCore::ScriptExecutionContext::inspectorController):
  • inspector/InjectedScriptHost.cpp: (WebCore::InjectedScriptHost::InjectedScriptHost): (WebCore::InjectedScriptHost::injectScript): (WebCore::InjectedScriptHost::nextWorkerId): (WebCore::InjectedScriptHost::didCreateWorker): (WebCore::InjectedScriptHost::willDestroyWorker):
  • inspector/InjectedScriptHost.h:
  • inspector/InjectedScriptHost.idl:
  • inspector/InspectorController.cpp: (WebCore::InspectorController::populateScriptObjects): (WebCore::InspectorController::resetScriptObjects): (WebCore::InspectorController::didCommitLoad): (WebCore::InspectorController::didCreateWorker): (WebCore::InspectorController::willDestroyWorker):
  • inspector/InspectorController.h:
  • inspector/InspectorFrontend.cpp: (WebCore::InspectorFrontend::didCreateWorker): (WebCore::InspectorFrontend::willDestroyWorker):
  • inspector/InspectorFrontend.h:
  • inspector/InspectorWorkerResource.h: Added. (WebCore::InspectorWorkerResource::create): (WebCore::InspectorWorkerResource::id): (WebCore::InspectorWorkerResource::url): (WebCore::InspectorWorkerResource::isSharedWorker): (WebCore::InspectorWorkerResource::InspectorWorkerResource):
  • inspector/front-end/Checkbox.js: Added. (WebInspector.Checkbox.callbackWrapper): (WebInspector.Checkbox): (WebInspector.Checkbox.prototype.checked):
  • inspector/front-end/InjectedFakeWorker.js: (InjectedFakeWorker.FakeWorker): (InjectedFakeWorker.FakeWorker.prototype.terminate): (InjectedFakeWorker.FakeWorker.prototype._handleException): (InjectedFakeWorker.FakeWorker.prototype._importScripts): (InjectedFakeWorker.FakeWorker.prototype._loadScript): (InjectedFakeWorker.FakeWorker.prototype._expandURLAndCheckOrigin):
  • inspector/front-end/ScriptsPanel.js: (WebInspector.ScriptsPanel): (WebInspector.ScriptsPanel.prototype.debuggerWasEnabled): (WebInspector.ScriptsPanel.prototype.debuggerWasDisabled): (WebInspector.ScriptsPanel.prototype.reset):
  • inspector/front-end/WebKit.qrc:
  • inspector/front-end/WorkersSidebarPane.js: Added. (WebInspector.WorkersSidebarPane): (WebInspector.WorkersSidebarPane.prototype.addWorker): (WebInspector.WorkersSidebarPane.prototype.removeWorker): (WebInspector.WorkersSidebarPane.prototype.setInstrumentation): (WebInspector.WorkersSidebarPane.prototype.reset): (WebInspector.WorkersSidebarPane.prototype._onTriggerInstrument): (WebInspector.Worker): (WebInspector.didCreateWorker): (WebInspector.willDestroyWorker):
  • inspector/front-end/inspector.css:
  • inspector/front-end/inspector.html:
  • workers/AbstractWorker.cpp: (WebCore::AbstractWorker::AbstractWorker): (WebCore::AbstractWorker::~AbstractWorker): (WebCore::AbstractWorker::onDestroyWorker): (WebCore::AbstractWorker::contextDestroyed):
  • workers/AbstractWorker.h: (WebCore::AbstractWorker::id):
  • workers/SharedWorker.cpp: (WebCore::SharedWorker::SharedWorker):
  • workers/Worker.cpp: (WebCore::Worker::Worker):
05:58 Changeset [55521] by hamaji@chromium.org

2010-03-04 Shinichiro Hamaji <hamaji@chromium.org>

Unreviewed. Remove a non-ASCII character introduced in the following bug.

put_by_id does will incorrectly cache writes where a specific value exists, where at the point of caching the same value is being written.
https://bugs.webkit.org/show_bug.cgi?id=35537

  • runtime/JSObject.h: (JSC::JSObject::putDirectInternal):
05:39 Changeset [55520] by jocelyn.turcotte@nokia.com

[Qt] Make the OUTPUT_DIR variable in qmake projects independent of build-webkit's logic.

Reviewed by Tor Arne Vestbø.

This also allows shadow builds relying only on qmake to work properly.

.:

  • WebKit.pri:

JavaScriptCore:

  • jsc.pro:
  • qt/api/QtScript.pro:
  • qt/tests/qscriptengine/qscriptengine.pro:
  • qt/tests/qscriptvalue/qscriptvalue.pro:
  • qt/tests/tests.pri:

WebCore:

  • WebCore.pro:

WebKit\qt:

  • tests/benchmarks/loading/loading.pro:
  • tests/benchmarks/painting/painting.pro:
  • tests/hybridPixmap/hybridPixmap.pro:
  • tests/qgraphicswebview/qgraphicswebview.pro:
  • tests/qwebelement/qwebelement.pro:
  • tests/qwebframe/qwebframe.pro:
  • tests/qwebhistory/qwebhistory.pro:
  • tests/qwebhistoryinterface/qwebhistoryinterface.pro:
  • tests/qwebinspector/qwebinspector.pro:
  • tests/qwebpage/qwebpage.pro:
  • tests/qwebplugindatabase/qwebplugindatabase.pro:

WebKitTools:

  • DumpRenderTree/qt/DumpRenderTree.pro:
  • DumpRenderTree/qt/ImageDiff.pro:
  • DumpRenderTree/qt/TestNetscapePlugin/TestNetscapePlugin.pro:
  • QtLauncher/QtLauncher.pro:
05:17 QtWebKitBugs edited by hausmann@webkit.org
(diff)
04:46 Changeset [55519] by ossy@webkit.org

Enable to use built-product-archive on Qt platform.

Patch by Gabor Rapcsanyi <rgabor@inf.u-szeged.hu> on 2010-03-04
Reviewed by Kenneth Rohde Christiansen.

  • BuildSlaveSupport/built-product-archive:
04:29 Changeset [55518] by zecke@webkit.org

[Qt] Avoid calling QFont::detach too often from FontQt

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

The letter- and wordspacing is inside the WebCore::Font class,
our QFont is located in the WebCore::FontPlatformData. Everytime
we need to use a QFont inside WebCore::Font we are calling the
font method which gets the QFont from the WebCore::FontPlatformData
and is applying the letter- and wordspacing. Internally this
will attempt to detach the QFont...

Avoid calling setLetterSpacing and setWordSpacing on QFont if
the WebCore::Font has the default applied.

  • platform/graphics/qt/FontQt.cpp:

(WebCore::Font::font):

04:21 Changeset [55517] by apavlov@chromium.org

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

Reviewed by Pavel Feldman.

Web Inspector: Nothing happens on navigation to resource via a link if resource tracking is disabled
https://bugs.webkit.org/show_bug.cgi?id=35574

03:04 Changeset [55516] by abecsi@webkit.org

Rubber-stamped by Kenneth Rohde Christiansen.

[Qt] DRT is failing on new test introduced in r55499
Skipping it until fix.
https://bugs.webkit.org/show_bug.cgi?id=35729

  • platform/qt/Skipped: fast/events/zoom-dblclick.html skipped
02:45 Changeset [55515] by vestbo@webkit.org

[Qt] Fix generation of forwaring headers

Reviewed by Kenneth Rohde Christiansen.

The dependencies were a bit wrong, so we ended up not generating
forwarding headers for qwebkitversion.h and qwebkitglobal.h

The forwardning headers are now the only targets depending on the
real headers. All other targets either depend on the generated
class headers, or the forwarding headers.

  • Api/DerivedSources.pro:
01:41 Changeset [55514] by eric@webkit.org

2010-03-04 Vangelis Kokkevis <vangelis@chromium.org>

Reviewed by Oliver Hunt.

Update existing test to check that getUniformLocation() returns
null if the uniform requested is not found.
https://bugs.webkit.org/show_bug.cgi?id=34669

  • fast/canvas/webgl/script-tests/uniform-location.js:
  • fast/canvas/webgl/uniform-location-expected.txt:

2010-03-04 Vangelis Kokkevis <vangelis@chromium.org>

Reviewed by Oliver Hunt.

getUniformLocation() now returns null if uniform requested
is not found.
https://bugs.webkit.org/show_bug.cgi?id=34669

Test:LayoutTests/fast/canvas/webgl/uniform-location.html
(added missing test)

  • html/canvas/WebGLRenderingContext.cpp: (WebCore::WebGLRenderingContext::getUniformLocation):
01:24 Changeset [55513] by eric@webkit.org

2010-03-04 John Abd-El-Malek <jam@chromium.org>

Reviewed by Adam Barth.

Fix incorrect setup of DateExtension.
https://bugs.webkit.org/show_bug.cgi?id=35710

  • fast/dom/Window/resources/slow_unload_handler1.html:

2010-03-04 John Abd-El-Malek <jam@chromium.org>

Reviewed by Adam Barth.

Fix incorrect setup of DateExtension.
https://bugs.webkit.org/show_bug.cgi?id=35710

  • bindings/v8/DateExtension.cpp: (WebCore::DateExtension::setAllowSleep):
01:13 Changeset [55512] by darin@chromium.org

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

Reviewed by Pavel Feldman.

Fix failing DevToolsSanityTest.TestResourceContentLength
https://bugs.webkit.org/show_bug.cgi?id=35725

http://trac.webkit.org/changeset/55466 missed one more spot
where contentLength needed to be renamed resourceSize.

  • src/js/Tests.js:
00:57 Changeset [55511] by oliver@apple.com

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

Reviewed by Maciej Stachowiak.

https://bugs.webkit.org/show_bug.cgi?id=35603
SVG incorrectly allows nested <use> elements in a <use> shadow tree

Simplify the handling of invalid or missing use-targets by cloning them
to empty <g> elements rather than just allowing the <use> element to
be copied into the shadow tree, as this violates a number of assumptions
in the shadow tree handling code.

Tests: svg/custom/use-nested-disallowed-target.svg

svg/custom/use-nested-missing-target-added.svg
svg/custom/use-nested-missing-target-removed.svg
svg/custom/use-nested-missing-target.svg
svg/custom/use-nested-notarget.svg

  • svg/SVGUseElement.cpp: (WebCore::SVGUseElement::buildPendingResource): (WebCore::SVGUseElement::buildShadowAndInstanceTree): (WebCore::SVGUseElement::expandUseElementsInShadowTree):

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

Reviewed by Maciej Stachowiak.

https://bugs.webkit.org/show_bug.cgi?id=35603
SVG incorrectly allows nested <use> elements in a <use> shadow tree

Add test cases for various combinations of <use> and targets that aren't present.

  • platform/mac/svg/custom/use-empty-reference-expected.txt: Added.
  • svg/custom/use-nested-disallowed-target-expected.txt: Added.
  • svg/custom/use-nested-disallowed-target.svg: Added.
  • svg/custom/use-nested-missing-target-added-expected.txt: Added.
  • svg/custom/use-nested-missing-target-added.svg: Added.
  • svg/custom/use-nested-missing-target-expected.txt: Added.
  • svg/custom/use-nested-missing-target-removed-expected.txt: Added.
  • svg/custom/use-nested-missing-target-removed.svg: Added.
  • svg/custom/use-nested-missing-target.svg: Added.
  • svg/custom/use-nested-notarget-expected.txt: Added.
  • svg/custom/use-nested-notarget.svg: Added.
00:51 Changeset [55510] by eric@webkit.org

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

Reviewed by Gustavo Noronha Silva.

Move Cairo-related font code from platform/graphics/gtk to
platform/graphics/cairo, so other ports may use them.
http://webkit.org/b/35539

  • GNUmakefile.am:
  • platform/graphics/cairo/FontCacheCairo.cpp: Copied from WebCore/platform/graphics/gtk/FontCacheGtk.cpp.
  • platform/graphics/cairo/FontCustomPlatformData.cpp: Copied from WebCore/platform/graphics/gtk/FontCustomPlatformData.cpp.
  • platform/graphics/cairo/FontCustomPlatformData.h: Copied from WebCore/platform/graphics/gtk/FontCustomPlatformData.h.
  • platform/graphics/cairo/FontPlatformData.h: Copied from WebCore/platform/graphics/gtk/FontPlatformData.h.
  • platform/graphics/cairo/FontPlatformDataCairo.cpp: Copied from WebCore/platform/graphics/gtk/FontPlatformDataGtk.cpp.
  • platform/graphics/cairo/GlyphPageTreeNodeCairo.cpp: Copied from WebCore/platform/graphics/gtk/GlyphPageTreeNodeGtk.cpp.
  • platform/graphics/cairo/SimpleFontDataCairo.cpp: Copied from WebCore/platform/graphics/gtk/SimpleFontDataGtk.cpp.
  • platform/graphics/gtk/FontCacheGtk.cpp: Removed.
  • platform/graphics/gtk/FontCustomPlatformData.cpp: Removed.
  • platform/graphics/gtk/FontCustomPlatformData.h: Removed.
  • platform/graphics/gtk/FontPlatformData.h: Removed.
  • platform/graphics/gtk/FontPlatformDataGtk.cpp: Removed.
  • platform/graphics/gtk/GlyphPageTreeNodeGtk.cpp: Removed.
  • platform/graphics/gtk/SimpleFontDataGtk.cpp: Removed.
00:34 Changeset [55509] by eric@webkit.org

2010-03-04 Evan Stade <estade@chromium.org>

Reviewed by David Levin.

https://bugs.webkit.org/show_bug.cgi?id=35261
[skia] crash when attempting to render certain SVGs with Skia

This test passes if it doesn't crash.

  • platform/mac/Skipped: Skip the test on Mac, where it crashes.
  • svg/custom/tiling-regular-hexagonal-crash-expected.txt: Added.
  • svg/custom/tiling-regular-hexagonal-crash.svg: Added.

2010-03-04 Evan Stade <estade@chromium.org>

Reviewed by David Levin.

https://bugs.webkit.org/show_bug.cgi?id=35261
[skia] crash when attempting to render certain SVGs

This fixes the crash, but the SVG still doesn't render properly.

Test: svg/custom/tiling-regular-hexagonal-crash.svg

  • platform/graphics/skia/ImageSkia.cpp: (WebCore::BitmapImageSingleFrameSkia::create): don't return 0 when the copy fails; instead return a blank bitmap. The caller doesn't check for 0 before dereferencing.
00:03 Changeset [55508] by tony@chromium.org

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

Reviewed by Darin Fisher.

Initialize m_isMultipartPayload in other ResourceResponse ctor.
In r55451 (bug 35628) I added an extra bool, but only initialized
it in one constructor.
https://bugs.webkit.org/show_bug.cgi?id=35719

  • platform/network/chromium/ResourceResponse.h: (WebCore::ResourceResponse::ResourceResponse):
00:01 Changeset [55507] by eric@webkit.org

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

Reviewed by Darin Fisher.

Adding touch event type and point type. Tested against the try bots.
https://bugs.webkit.org/show_bug.cgi?id=35691

  • public/WebInputEvent.h: (WebKit::WebInputEvent::): (WebKit::WebInputEvent::isTouchEventType): (WebKit::WebTouchEvent::WebTouchEvent):
  • public/WebTouchPoint.h: Added. (WebKit::WebTouchPoint::WebTouchPoint): (WebKit::WebTouchPoint::):

03/03/10:

23:18 Changeset [55506] by bweinstein@apple.com

Touch WebKit.idl to force a rebuild of Interfaces

22:10 Changeset [55505] by ukai@chromium.org

2010-03-03 Yuta Kitamura <yutak@chromium.org>

Reviewed by Alexey Proskuryakov.

Add a new class that stores information about Web Socket handshake request.

Instances of this class contain the necessary information to send a Web Socket
handshake request. In the future, this class will provide request information
to the Web Inspector.

WebSocketHandshake needs to provide request information
https://bugs.webkit.org/show_bug.cgi?id=34784

No new tests, since the current tests will suffice (LayoutTests/websocket/*).

  • GNUmakefile.am:
  • WebCore.gypi:
  • WebCore.pro:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • websockets/WebSocketChannel.cpp:
  • websockets/WebSocketHandshake.cpp: (WebCore::WebSocketHandshake::clientHandshakeMessage): (WebCore::WebSocketHandshake::clientHandshakeRequest):
  • websockets/WebSocketHandshake.h:
  • websockets/WebSocketHandshakeRequest.cpp: Added. (WebCore::WebSocketHandshakeRequest::WebSocketHandshakeRequest): (WebCore::WebSocketHandshakeRequest::~WebSocketHandshakeRequest): (WebCore::WebSocketHandshakeRequest::addExtraHeaderField): (WebCore::WebSocketHandshakeRequest::headerFields): (WebCore::WebSocketHandshakeRequest::host):
  • websockets/WebSocketHandshakeRequest.h: Added.
21:19 Changeset [55504] by eric@webkit.org

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

Reviewed by Kenneth Rohde Christiansen.

Build fix after r55452.

No new tests, this is a build fix.

  • page/Settings.cpp: (WebCore::Settings::setDatabasesEnabled):
21:17 Changeset [55503] by alice.liu@apple.com

Add a way to get an iframe's content frame

Reviewed by Jon Honeycutt.

  • DOMCoreClasses.cpp:

(DOMElement::createInstance):
Added case for DOMHTMLIFrameElement

  • DOMHTMLClasses.cpp:

Adding the few DOMHTMLIFrameElement functions definitions that have
distinct implementations (all others just call parent implementation)
(DOMHTMLIFrameElement::QueryInterface):
(DOMHTMLIFrameElement::contentFrame):

  • DOMHTMLClasses.h:

Most of these function declarations have definitions that just call the parent implementation
(DOMHTMLIFrameElement::DOMHTMLIFrameElement):
(DOMHTMLIFrameElement::AddRef):
(DOMHTMLIFrameElement::Release):
(DOMHTMLIFrameElement::throwException):
(DOMHTMLIFrameElement::callWebScriptMethod):
(DOMHTMLIFrameElement::evaluateWebScript):
(DOMHTMLIFrameElement::removeWebScriptKey):
(DOMHTMLIFrameElement::stringRepresentation):
(DOMHTMLIFrameElement::webScriptValueAtIndex):
(DOMHTMLIFrameElement::setWebScriptValueAtIndex):
(DOMHTMLIFrameElement::setException):
(DOMHTMLIFrameElement::nodeName):
(DOMHTMLIFrameElement::nodeValue):
(DOMHTMLIFrameElement::setNodeValue):
(DOMHTMLIFrameElement::nodeType):
(DOMHTMLIFrameElement::parentNode):
(DOMHTMLIFrameElement::childNodes):
(DOMHTMLIFrameElement::firstChild):
(DOMHTMLIFrameElement::lastChild):
(DOMHTMLIFrameElement::previousSibling):
(DOMHTMLIFrameElement::nextSibling):
(DOMHTMLIFrameElement::attributes):
(DOMHTMLIFrameElement::ownerDocument):
(DOMHTMLIFrameElement::insertBefore):
(DOMHTMLIFrameElement::replaceChild):
(DOMHTMLIFrameElement::removeChild):
(DOMHTMLIFrameElement::appendChild):
(DOMHTMLIFrameElement::hasChildNodes):
(DOMHTMLIFrameElement::cloneNode):
(DOMHTMLIFrameElement::normalize):
(DOMHTMLIFrameElement::isSupported):
(DOMHTMLIFrameElement::namespaceURI):
(DOMHTMLIFrameElement::prefix):
(DOMHTMLIFrameElement::setPrefix):
(DOMHTMLIFrameElement::localName):
(DOMHTMLIFrameElement::hasAttributes):
(DOMHTMLIFrameElement::isSameNode):
(DOMHTMLIFrameElement::isEqualNode):
(DOMHTMLIFrameElement::textContent):
(DOMHTMLIFrameElement::setTextContent):
(DOMHTMLIFrameElement::tagName):
(DOMHTMLIFrameElement::getAttribute):
(DOMHTMLIFrameElement::setAttribute):
(DOMHTMLIFrameElement::removeAttribute):
(DOMHTMLIFrameElement::getAttributeNode):
(DOMHTMLIFrameElement::setAttributeNode):
(DOMHTMLIFrameElement::removeAttributeNode):
(DOMHTMLIFrameElement::getElementsByTagName):
(DOMHTMLIFrameElement::getAttributeNS):
(DOMHTMLIFrameElement::setAttributeNS):
(DOMHTMLIFrameElement::removeAttributeNS):
(DOMHTMLIFrameElement::getAttributeNodeNS):
(DOMHTMLIFrameElement::setAttributeNodeNS):
(DOMHTMLIFrameElement::getElementsByTagNameNS):
(DOMHTMLIFrameElement::hasAttribute):
(DOMHTMLIFrameElement::hasAttributeNS):
(DOMHTMLIFrameElement::focus):
(DOMHTMLIFrameElement::blur):
(DOMHTMLIFrameElement::idName):
(DOMHTMLIFrameElement::setIdName):
(DOMHTMLIFrameElement::title):
(DOMHTMLIFrameElement::setTitle):
(DOMHTMLIFrameElement::lang):
(DOMHTMLIFrameElement::setLang):
(DOMHTMLIFrameElement::dir):
(DOMHTMLIFrameElement::setDir):
(DOMHTMLIFrameElement::className):
(DOMHTMLIFrameElement::setClassName):
(DOMHTMLIFrameElement::innerHTML):
(DOMHTMLIFrameElement::setInnerHTML):
(DOMHTMLIFrameElement::innerText):
(DOMHTMLIFrameElement::setInnerText):

  • Interfaces/DOMHTML.idl:

Added IDOMHTMLIFrameElement interface

21:02 Changeset [55502] by eric@webkit.org

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

Reviewed by Simon Hausmann.

QScriptValue::isObject fix.

Fix broken internal state evaluation from JSValue to JSNative / JSObject.
New function was introduced which should take care about promoting
JSValue state inside QScriptValuePrivate. It should be used instead of a
direct JSC C API call.

The bug exposed a weakness in autotest suite, as the QScriptValuePrivate
is based on state machine with lazy state evaluation, there is a possibility
that serial sequencial calls to the same public const function could return
different results. The patch fix the issue.

[Qt] Sometimes QScriptValue::isObject returns an incorrect value
https://bugs.webkit.org/show_bug.cgi?id=35387

  • qt/api/qscriptvalue_p.h: (QScriptValuePrivate::isBool): (QScriptValuePrivate::isNumber): (QScriptValuePrivate::isNull): (QScriptValuePrivate::isString): (QScriptValuePrivate::isUndefined): (QScriptValuePrivate::isError): (QScriptValuePrivate::isObject): (QScriptValuePrivate::isFunction): (QScriptValuePrivate::call): (QScriptValuePrivate::refineJSValue):
  • qt/tests/qscriptvalue/tst_qscriptvalue_generated.cpp: (tst_QScriptValue::initScriptValues): (tst_QScriptValue::isValid_makeData): (tst_QScriptValue::isValid_test): (tst_QScriptValue::isBool_makeData): (tst_QScriptValue::isBool_test): (tst_QScriptValue::isBoolean_makeData): (tst_QScriptValue::isBoolean_test): (tst_QScriptValue::isNumber_makeData): (tst_QScriptValue::isNumber_test): (tst_QScriptValue::isFunction_test): (tst_QScriptValue::isNull_makeData): (tst_QScriptValue::isNull_test): (tst_QScriptValue::isString_makeData): (tst_QScriptValue::isString_test): (tst_QScriptValue::isUndefined_makeData): (tst_QScriptValue::isUndefined_test): (tst_QScriptValue::isObject_makeData): (tst_QScriptValue::isObject_test): (tst_QScriptValue::toString_makeData): (tst_QScriptValue::toString_test): (tst_QScriptValue::toNumber_makeData): (tst_QScriptValue::toNumber_test): (tst_QScriptValue::toBool_makeData): (tst_QScriptValue::toBool_test): (tst_QScriptValue::toBoolean_makeData): (tst_QScriptValue::toBoolean_test): (tst_QScriptValue::toInteger_makeData): (tst_QScriptValue::toInteger_test): (tst_QScriptValue::toInt32_makeData): (tst_QScriptValue::toInt32_test): (tst_QScriptValue::toUInt32_makeData): (tst_QScriptValue::toUInt32_test): (tst_QScriptValue::toUInt16_makeData): (tst_QScriptValue::toUInt16_test):
20:45 Changeset [55501] by eric@webkit.org

2010-03-03 Diego Gonzalez <diego.gonzalez@openbossa.org>

Reviewed by Kenneth Rohde Christiansen.

Make keyIdentifierForQtKeyCode return the unicode backspace value on Qt PlatformKeyboardEvent

LayoutTests:

fast/events/key-events-in-input-text.html
fast/events/special-key-events-in-input-text.html

[Qt] Return the unicode backspace value in keyIdentifierForQtKeyCode on Qt PlatformKeyboardEvent
https://bugs.webkit.org/show_bug.cgi?id=35694

  • platform/qt/Skipped:

2010-03-03 Diego Gonzalez <diego.gonzalez@openbossa.org>

Reviewed by Kenneth Rohde Christiansen.

Make keyIdentifierForQtKeyCode return the unicode backspace value on Qt PlatformKeyboardEvent

LayoutTests:

fast/events/key-events-in-input-text.html
fast/events/special-key-events-in-input-text.html

[Qt] Return the unicode backspace value in keyIdentifierForQtKeyCode on Qt PlatformKeyboardEvent
https://bugs.webkit.org/show_bug.cgi?id=35694

  • platform/qt/PlatformKeyboardEventQt.cpp: (WebCore::keyIdentifierForQtKeyCode):
19:57 Changeset [55500] by eric@webkit.org

2010-03-03 Chao-ying Fu <fu@mips.com>

Reviewed by Gavin Barraclough.

MIPS JIT Supports
https://bugs.webkit.org/show_bug.cgi?id=30144

The following changes enable MIPS YARR and YARR_JIT.

  • assembler/AbstractMacroAssembler.h: (JSC::AbstractMacroAssembler::Imm32::Imm32):
  • assembler/MIPSAssembler.h: Added. (JSC::MIPSRegisters::): (JSC::MIPSAssembler::MIPSAssembler): (JSC::MIPSAssembler::): (JSC::MIPSAssembler::JmpSrc::JmpSrc): (JSC::MIPSAssembler::JmpDst::JmpDst): (JSC::MIPSAssembler::JmpDst::isUsed): (JSC::MIPSAssembler::JmpDst::used): (JSC::MIPSAssembler::emitInst): (JSC::MIPSAssembler::nop): (JSC::MIPSAssembler::loadDelayNop): (JSC::MIPSAssembler::copDelayNop): (JSC::MIPSAssembler::move): (JSC::MIPSAssembler::li): (JSC::MIPSAssembler::lui): (JSC::MIPSAssembler::addiu): (JSC::MIPSAssembler::addu): (JSC::MIPSAssembler::subu): (JSC::MIPSAssembler::mult): (JSC::MIPSAssembler::mfhi): (JSC::MIPSAssembler::mflo): (JSC::MIPSAssembler::mul): (JSC::MIPSAssembler::andInsn): (JSC::MIPSAssembler::andi): (JSC::MIPSAssembler::nor): (JSC::MIPSAssembler::orInsn): (JSC::MIPSAssembler::ori): (JSC::MIPSAssembler::xorInsn): (JSC::MIPSAssembler::xori): (JSC::MIPSAssembler::slt): (JSC::MIPSAssembler::sltu): (JSC::MIPSAssembler::sltiu): (JSC::MIPSAssembler::sll): (JSC::MIPSAssembler::sllv): (JSC::MIPSAssembler::sra): (JSC::MIPSAssembler::srav): (JSC::MIPSAssembler::lw): (JSC::MIPSAssembler::lwl): (JSC::MIPSAssembler::lwr): (JSC::MIPSAssembler::lhu): (JSC::MIPSAssembler::sw): (JSC::MIPSAssembler::jr): (JSC::MIPSAssembler::jalr): (JSC::MIPSAssembler::jal): (JSC::MIPSAssembler::bkpt): (JSC::MIPSAssembler::bgez): (JSC::MIPSAssembler::bltz): (JSC::MIPSAssembler::beq): (JSC::MIPSAssembler::bne): (JSC::MIPSAssembler::bc1t): (JSC::MIPSAssembler::bc1f): (JSC::MIPSAssembler::newJmpSrc): (JSC::MIPSAssembler::appendJump): (JSC::MIPSAssembler::addd): (JSC::MIPSAssembler::subd): (JSC::MIPSAssembler::muld): (JSC::MIPSAssembler::lwc1): (JSC::MIPSAssembler::ldc1): (JSC::MIPSAssembler::swc1): (JSC::MIPSAssembler::sdc1): (JSC::MIPSAssembler::mtc1): (JSC::MIPSAssembler::mfc1): (JSC::MIPSAssembler::truncwd): (JSC::MIPSAssembler::cvtdw): (JSC::MIPSAssembler::ceqd): (JSC::MIPSAssembler::cngtd): (JSC::MIPSAssembler::cnged): (JSC::MIPSAssembler::cltd): (JSC::MIPSAssembler::cled): (JSC::MIPSAssembler::cueqd): (JSC::MIPSAssembler::coled): (JSC::MIPSAssembler::coltd): (JSC::MIPSAssembler::culed): (JSC::MIPSAssembler::cultd): (JSC::MIPSAssembler::label): (JSC::MIPSAssembler::align): (JSC::MIPSAssembler::getRelocatedAddress): (JSC::MIPSAssembler::getDifferenceBetweenLabels): (JSC::MIPSAssembler::size): (JSC::MIPSAssembler::executableCopy): (JSC::MIPSAssembler::getCallReturnOffset): (JSC::MIPSAssembler::linkJump): (JSC::MIPSAssembler::linkCall): (JSC::MIPSAssembler::linkPointer): (JSC::MIPSAssembler::relinkJump): (JSC::MIPSAssembler::relinkCall): (JSC::MIPSAssembler::repatchInt32): (JSC::MIPSAssembler::repatchPointer): (JSC::MIPSAssembler::repatchLoadPtrToLEA): (JSC::MIPSAssembler::relocateJumps): (JSC::MIPSAssembler::linkWithOffset): (JSC::MIPSAssembler::linkCallInternal):
  • assembler/MacroAssembler.h:
  • assembler/MacroAssemblerMIPS.h: Added. (JSC::MacroAssemblerMIPS::MacroAssemblerMIPS): (JSC::MacroAssemblerMIPS::): (JSC::MacroAssemblerMIPS::add32): (JSC::MacroAssemblerMIPS::and32): (JSC::MacroAssemblerMIPS::lshift32): (JSC::MacroAssemblerMIPS::mul32): (JSC::MacroAssemblerMIPS::not32): (JSC::MacroAssemblerMIPS::or32): (JSC::MacroAssemblerMIPS::rshift32): (JSC::MacroAssemblerMIPS::sub32): (JSC::MacroAssemblerMIPS::xor32): (JSC::MacroAssemblerMIPS::load32): (JSC::MacroAssemblerMIPS::load32WithUnalignedHalfWords): (JSC::MacroAssemblerMIPS::load32WithAddressOffsetPatch): (JSC::MacroAssemblerMIPS::loadPtrWithPatchToLEA): (JSC::MacroAssemblerMIPS::loadPtrWithAddressOffsetPatch): (JSC::MacroAssemblerMIPS::load16): (JSC::MacroAssemblerMIPS::store32WithAddressOffsetPatch): (JSC::MacroAssemblerMIPS::store32): (JSC::MacroAssemblerMIPS::supportsFloatingPoint): (JSC::MacroAssemblerMIPS::supportsFloatingPointTruncate): (JSC::MacroAssemblerMIPS::pop): (JSC::MacroAssemblerMIPS::push): (JSC::MacroAssemblerMIPS::move): (JSC::MacroAssemblerMIPS::swap): (JSC::MacroAssemblerMIPS::signExtend32ToPtr): (JSC::MacroAssemblerMIPS::zeroExtend32ToPtr): (JSC::MacroAssemblerMIPS::branch32): (JSC::MacroAssemblerMIPS::branch32WithUnalignedHalfWords): (JSC::MacroAssemblerMIPS::branch16): (JSC::MacroAssemblerMIPS::branchTest32): (JSC::MacroAssemblerMIPS::jump): (JSC::MacroAssemblerMIPS::branchAdd32): (JSC::MacroAssemblerMIPS::branchMul32): (JSC::MacroAssemblerMIPS::branchSub32): (JSC::MacroAssemblerMIPS::breakpoint): (JSC::MacroAssemblerMIPS::nearCall): (JSC::MacroAssemblerMIPS::call): (JSC::MacroAssemblerMIPS::ret): (JSC::MacroAssemblerMIPS::set32): (JSC::MacroAssemblerMIPS::setTest32): (JSC::MacroAssemblerMIPS::moveWithPatch): (JSC::MacroAssemblerMIPS::branchPtrWithPatch): (JSC::MacroAssemblerMIPS::storePtrWithPatch): (JSC::MacroAssemblerMIPS::tailRecursiveCall): (JSC::MacroAssemblerMIPS::makeTailRecursiveCall): (JSC::MacroAssemblerMIPS::loadDouble): (JSC::MacroAssemblerMIPS::storeDouble): (JSC::MacroAssemblerMIPS::addDouble): (JSC::MacroAssemblerMIPS::subDouble): (JSC::MacroAssemblerMIPS::mulDouble): (JSC::MacroAssemblerMIPS::convertInt32ToDouble): (JSC::MacroAssemblerMIPS::insertRelaxationWords): (JSC::MacroAssemblerMIPS::branchTrue): (JSC::MacroAssemblerMIPS::branchFalse): (JSC::MacroAssemblerMIPS::branchEqual): (JSC::MacroAssemblerMIPS::branchNotEqual): (JSC::MacroAssemblerMIPS::branchDouble): (JSC::MacroAssemblerMIPS::branchTruncateDoubleToInt32): (JSC::MacroAssemblerMIPS::linkCall): (JSC::MacroAssemblerMIPS::repatchCall):
  • jit/ExecutableAllocator.h: (JSC::ExecutableAllocator::cacheFlush):
  • wtf/Platform.h:
  • yarr/RegexJIT.cpp: (JSC::Yarr::RegexGenerator::generateEnter): (JSC::Yarr::RegexGenerator::generateReturn):
19:31 Changeset [55499] by eric@webkit.org

2010-03-03 Chad Faragher <wyck@chromium.org>

Reviewed by Darin Adler.

Changed the double-click framework code to pass adjusted page
coordinates for zoomed pages. Added a new test to expose
mistreatment of mouse click coordinates during a double-click.

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

  • fast/events/zoom-dblclick-expected.txt: Added.
  • fast/events/zoom-dblclick.html: Added.

2010-03-03 Chad Faragher <wyck@chromium.org>

Reviewed by Darin Adler.
Changed the double-click framework code to pass adjusted page
coordinates for zoomed pages. Added a new test to expose
mistreatment of mouse click coordinates during a double-click.

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

Test: fast/events/zoom-dblclick.html

  • dom/Node.cpp: (WebCore::Node::dispatchMouseEvent):
19:13 Changeset [55498] by eric@webkit.org

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

Reviewed by Simon Hausmann.

Add a missing 'm_' to class variables names.

[Qt] QtLauncher is not respecting class variable names
https://bugs.webkit.org/show_bug.cgi?id=35542

  • QtLauncher/main.cpp: (LauncherWindow::LauncherWindow): (LauncherWindow::init): (LauncherWindow::sendTouchEvent): (LauncherWindow::eventFilter): (LauncherWindow::zoomIn): (LauncherWindow::zoomOut): (LauncherWindow::resetZoom): (LauncherWindow::setEditable): (LauncherWindow::setTouchMocking): (LauncherWindow::initializeView): (LauncherWindow::createChrome):
19:00 Changeset [55497] by sfalken@apple.com

Remove extraneous ChangeLog entries.

18:56 Changeset [55496] by sfalken@apple.com

Merged r55495.

18:52 Changeset [55495] by sfalken@apple.com

Windows build fix.

18:42 Changeset [55494] by sfalken@apple.com

Merge r55492.

18:35 Changeset [55493] by sfalken@apple.com

New tag.

18:33 Changeset [55492] by sfalken@apple.com

Windows build fix.

17:23 Changeset [55491] by levin@chromium.org

No review, rolling out r55474.

The patch broke fast/frames/sandboxed-iframe-storage.html

Last one (I hope).

  • WebCore.xcodeproj/project.pbxproj:
17:20 Changeset [55490] by dumi@chromium.org

Making database-lock-after-reload.html a bit more robust and
cleaning up some style issues.

Reviewed by Dimitri Glazkov.

  • storage/database-lock-after-reload.html:
  • storage/resources/database-lock-after-reload-2.html:
17:14 Changeset [55489] by ap@apple.com

Rubber-stamped by Mark Rowe.

Exclude leaks in Java that build bot complains about.

  • Scripts/run-webkit-tests:
17:09 Changeset [55488] by levin@chromium.org

No review, rolling out r55474.

The patch broke fast/frames/sandboxed-iframe-storage.html

Unfortunately, (the webkit-patch rollout and) I missed these in r55485.

WebCore:

  • bindings/js/JSDatabaseCallback.cpp: Removed.
  • bindings/js/JSDatabaseCallback.h: Removed.
  • bindings/v8/custom/V8DatabaseCallback.cpp: Removed.
  • bindings/v8/custom/V8DatabaseCallback.h: Removed.
  • storage/DatabaseCallback.h: Removed.

LayoutTests:

  • storage/open-database-creation-callback-expected.txt: Removed.
  • storage/open-database-creation-callback-isolated-world-expected.txt: Removed.
  • storage/open-database-creation-callback-isolated-world.html: Removed.
  • storage/open-database-creation-callback.html: Removed.
17:01 Changeset [55487] by dumi@chromium.org

Fix a bug that could lead to a crash. Some parts of
SQLTransaction::checkAndHandleClosedDatabase() should only be run
when that method is invoked on the DB thread.

Reviewed by Dimitri Glazkov.

We cannot test this fix with a test, because the crash happens
only when all of the following conditions are met:

  1. A database is closing.
  2. A transaction on that database is in progress.
  3. The transaction is in a state where a statement/transaction

success/error callback needs to be invoked (so there's a task for
this transaction pending on the main thread).

  1. The DB thread finished processing all its tasks and called

SQLTransactionCoordinator::shutdown() before the main thread go to
that task.

The closest thing we have to a test is running
LayoutTests/storage/database-lock-after-reload.html 1000 times in
a row. Without the patch, the probability of a crash happening in
one of the runs is very high. With the patch, the test should
reliably run 1000 times in a row without a single crash.

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

  • storage/SQLTransaction.cpp:

(WebCore::SQLTransaction::checkAndHandleClosedDatabase):

16:47 Changeset [55486] by darin@chromium.org

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

Reviewed by Mark Rowe.

Page should not care about Chromium plug-in implementation details
https://bugs.webkit.org/show_bug.cgi?id=35623

  • WebCore.gypi: Compile PluginViewNone.cpp
  • page/Page.cpp: (WebCore::Page::privateBrowsingStateChanged): Remove conditional compilation for PLATFORM(CHROMIUM).
16:38 Changeset [55485] by levin@chromium.org

2010-03-03 David Levin <levin@chromium.org>

No review, rolling out r55474.
http://trac.webkit.org/changeset/55480

The patch broke fast/frames/sandboxed-iframe-storage.html

  • Android.jscbindings.mk:
  • Android.v8bindings.mk:
  • GNUmakefile.am:
  • WebCore.gypi:
  • WebCore.pro:
  • WebCore.vcproj/WebCore.vcproj:
  • bindings/js/JSDOMWindowCustom.cpp:
  • bindings/v8/custom/V8DOMWindowCustom.cpp:
  • dom/Document.cpp: (WebCore::Document::postTask):
  • page/DOMWindow.cpp: (WebCore::DOMWindow::openDatabase):
  • page/DOMWindow.h:
  • page/DOMWindow.idl:
  • storage/Database.cpp: (WebCore::Database::openDatabase): (WebCore::Database::Database): (WebCore::Database::performOpenAndVerify):
  • storage/Database.h:
  • workers/WorkerContext.cpp: (WebCore::WorkerContext::openDatabase):
  • workers/WorkerContext.h:
16:36 Changeset [55484] by levin@chromium.org

2010-03-03 David Levin <levin@chromium.org>

No review, rolling out r55480.
http://trac.webkit.org/changeset/55480

The patch broke fast/frames/sandboxed-iframe-storage.html

  • storage/Database.cpp:
  • storage/Database.h:
16:22 Changeset [55483] by mrowe@apple.com

Add virtual memory tags for TCMalloc and WebCore's purgeable buffers.

Reviewed by Geoff Garen.

JavaScriptCore:

  • wtf/TCSystemAlloc.cpp:

(TryMmap): Use the VM tag.

  • wtf/VMTags.h: Make use of VM_MEMORY_TCMALLOC and VM_MEMORY_WEBCORE_PURGEABLE_BUFFERS.

WebCore:

  • platform/mac/PurgeableBufferMac.cpp:

(WebCore::PurgeableBuffer::create): Use the VM tag.

16:20 Changeset [55482] by ap@apple.com

Not reviewed.

Try better to fix Java tests on Tiger bot.

  • platform/mac-tiger/Skipped:
  • platform/mac-tiger/java/lc3/JavaArray/ToString-001-expected.txt: Removed. This test gets different resultson each run when it fails, so it needs to be disabled on Tiger.
  • platform/mac-tiger/java/lc3/JavaObject/JavaObjectToString-001-expected.txt: Added a newline I didn't get when copying actual results from buildbot.
16:05 Changeset [55481] by cmarrin@apple.com

Additional change to separate Accelerated Compositing test from 3D Rendering test
https://bugs.webkit.org/show_bug.cgi?id=35610

15:03 Changeset [55480] by dumi@chromium.org

Move #include "DatabaseCallback.h" from Database.h to
Database.cpp.

Reviewed by nobody, build fix.

  • storage/Database.cpp:
  • storage/Database.h:
14:55 Changeset [55479] by ossy@webkit.org

[Qt] Skip new test introduced in r55474, because of
missing layoutTestController.evaluateScriptInIsolatedWorld()

  • platform/qt/Skipped: storage/open-database-creation-callback-isolated-world.html skipped.
14:24 Changeset [55478] by darin@apple.com

Fixed Mac build.

  • WebCore.xcodeproj/project.pbxproj: Marked a couple of headers "private" so they can

be used in the WebKit project.

14:22 Changeset [55477] by ap@apple.com

Rubber-stamped by Dan Bernstein.

https://bugs.webkit.org/show_bug.cgi?id=35700
Some of the newly added Java tests fail randomly on buildbot

  • fast/js/resources/js-test-post-async.js: Added a version that calls notifyDone(). I first wanted to add a notifyDone() call to js-test-post.js, but it turns out that multiple tests dump results after "TEST COMPLETE", and we need to get bots back to green fast.
  • java/lc3: (Many files). Don't load two scripts dynamically, WebKit has issues with order of execution in this case.
14:06 Changeset [55476] by sfalken@apple.com

Rubber stamped by Adam Roben.

Fix bogus xcopy that was polluting source tree at build time.

14:05 Changeset [55475] by ap@apple.com

Reviewed by Adele Peterson.

https://bugs.webkit.org/show_bug.cgi?id=35044
Crash in XML tokenizer reloading zoom-coords-viewattr-01-b.svg

I'm often getting a crash even when opening the test for the first time in Safari, but it
doesn't seem to crash in DumpRenderTree. Still, I can't think of a stronger way to test for
this condition, so no new regression test.

  • dom/XMLTokenizer.cpp: (WebCore::XMLTokenizer::end): Be prepared that parsing remaining input will pause parsing.
13:48 Changeset [55474] by dumi@chromium.org

WebCore: Adding support for the optional creation callback that could be
passed to openDatabase().

Reviewed by Adam Barth.

Test: storage/open-database-creation-callback.html

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

  • Android.jscbindings.mk
  • Android.v8bindings.mk
  • GNUmakefile.am:
  • WebCore.gypi:
  • WebCore.pro:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • bindings/js/JSDOMWindowCustom.cpp:

(WebCore::JSDOMWindow::openDatabase):

  • bindings/js/JSDatabaseCallback.cpp: Added.

(WebCore::JSDatabaseCallback::JSDatabaseCallback):
(WebCore::JSDatabaseCallback::~JSDatabaseCallback):
(WebCore::JSDatabaseCallback::handleEvent):

  • bindings/js/JSDatabaseCallback.h: Added.

(WebCore::JSDatabaseCallback::create):

  • bindings/v8/custom/V8DOMWindowCustom.cpp:

(WebCore::V8DOMWindow::openDatabaseCallback):

  • bindings/v8/custom/V8DatabaseCallback.cpp: Added.

(WebCore::V8DatabaseCallback::V8DatabaseCallback):
(WebCore::V8DatabaseCallback::~V8DatabaseCallback):
(WebCore::V8DatabaseCallback::handleEvent):

  • bindings/v8/custom/V8DatabaseCallback.h: Added.

(WebCore::V8DatabaseCallback::create):

  • page/DOMWindow.cpp:

(WebCore::DOMWindow::openDatabase):

  • page/DOMWindow.h:
  • page/DOMWindow.idl:
  • storage/Database.cpp:

(WebCore::DatabaseCreationCallbackTask::create):
(WebCore::DatabaseCreationCallbackTask::performTask):
(WebCore::DatabaseCreationCallbackTask::DatabaseCreationCallbackTask):
(WebCore::Database::openDatabase):
(WebCore::Database::Database):
(WebCore::Database::performOpenAndVerify):
(WebCore::Database::performCreationCallback):

  • storage/Database.h:

(WebCore::Database::isNew):

  • storage/DatabaseCallback.h: Added.

(WebCore::DatabaseCallback::~DatabaseCallback):

  • workers/WorkerContext.cpp:

(WebCore::WorkerContext::openDatabase):

  • workers/WorkerContext.h:

LayoutTests: Adding a test to test the creation callback passed to
openDatabase() and another one to make sure it is executed in the
correct world.

Reviewed by Adam Barth.

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

  • storage/open-database-creation-callback-expected.txt: Added.
  • storage/open-database-creation-callback.html: Added.
  • storage/open-database-creation-callback-isolated-world-expected.txt: Added.
  • storage/open-database-creation-callback-isolated-world.html: Added.
13:33 Changeset [55473] by cmarrin@apple.com

Useless change to kick the Windows build into recompiling the idl files

13:25 Changeset [55472] by bweinstein@apple.com

Touch WebKit.idl to force a rebuild

12:45 Changeset [55471] by ap@apple.com

Not reviewed.

Two newly added Java tests fail on Tiger build bot.

Just land actual results as expected. I don't even know which behavior is more correct.

  • platform/mac-tiger/java: Added.
  • platform/mac-tiger/java/lc3: Added.
  • platform/mac-tiger/java/lc3/JavaArray: Added.
  • platform/mac-tiger/java/lc3/JavaArray/ToString-001-expected.txt: Added.
  • platform/mac-tiger/java/lc3/JavaObject: Added.
  • platform/mac-tiger/java/lc3/JavaObject/JavaObjectToString-001-expected.txt: Added.
11:34 Changeset [55470] by ap@apple.com

Reviewed by Timothy Hatcher.

https://bugs.webkit.org/show_bug.cgi?id=35692
<rdar://problem/7703622> Crash when calling abort() on an XHR while in a windowless WebView

  • Panels/WebAuthenticationPanel.m: (-[WebAuthenticationPanel runAsModalDialogWithChallenge:]): Retain the challenge, just like it's done for sheet.
11:15 Changeset [55469] by ap@apple.com

Reviewed by Darin Adler.

https://bugs.webkit.org/show_bug.cgi?id=35617
Import Mozilla LiveConnect 3 test suite

  • java/lc3/sources: Added (many files). Sources for Mozilla test applet.
  • java/jstests.jar: Added in compiled form, since it's unlikely that we'll ever need to modify the above sources.
  • java/lc3.java: Added. (lc3): This is a small wrapper around jstests.jar. The latter only adds some classes, but WebKit needs an actual applet to call into Java. (lc3.createQAObject): Instantiate a class defined in jstests.jar by name. (lc3.stringToByteArray): A helper to make a byte array out of string. Tests originally relied on window.java package interface to do this, but it's not available in WebKit. We need more functions like this to work around other uses of window.java, but this is one I wrote for now.
  • java/Makefile: Link test applets to Mozilla's jstests.jar. This is not needed for tests other than ones imported here, but it's also harmless, and I don't know how to write good makefiles.
  • java/lc3: Added (many files). Tests were heavily reworked to run to completion, but PASS/FAIL results should not be generally relied upon. All html wrappers are identical.
11:05 Changeset [55468] by eric.carlson@apple.com

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

Unreviewed, build fix.

Update expected results for fast/js/custom-constructors-expected.txt

10:51 Changeset [55467] by cmarrin@apple.com

Enable LayoutTests/compositing for Windows when compositing is available
https://bugs.webkit.org/show_bug.cgi?id=35610

10:20 Changeset [55466] by apavlov@chromium.org

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

Reviewed by Pavel Feldman.

Web Inspector: resources panel only shows uncompressed resource size.
https://bugs.webkit.org/show_bug.cgi?id=35403

  • English.lproj/localizedStrings.js:
  • inspector/InspectorResource.cpp: (WebCore::InspectorResource::updateScriptObject):
  • inspector/front-end/AbstractTimelinePanel.js: (WebInspector.AbstractTimelinePanel.prototype.refresh):
  • inspector/front-end/AuditRules.js: (WebInspector.AuditRules.GzipRule.prototype.doRun): (WebInspector.AuditRules.GzipRule.prototype._shouldCompress):
  • inspector/front-end/ImageView.js: (WebInspector.ImageView):
  • inspector/front-end/Resource.js: (WebInspector.Resource.prototype.get resourceSize): (WebInspector.Resource.prototype.set resourceSize): (WebInspector.Resource.prototype.get transferSize): (WebInspector.Resource.CompareBySize): (WebInspector.Resource.CompareByTransferSize):
  • inspector/front-end/ResourcesPanel.js: (WebInspector.ResourcesPanel.prototype.populateSidebar): (WebInspector.ResourceTimeCalculator.prototype.computeBarGraphLabels): (WebInspector.ResourceTransferSizeCalculator.prototype.computeBarGraphLabels): (WebInspector.ResourceTransferSizeCalculator.prototype.computeBarGraphPercentages): (WebInspector.ResourceTransferSizeCalculator.prototype._value): (WebInspector.ResourceTransferSizeCalculator.prototype._networkBytes): (WebInspector.ResourceSidebarTreeElement.CompareByDescendingTransferSize): (WebInspector.ResourceGraph.prototype.refreshLabelPositions): (WebInspector.ResourceGraph.prototype.refresh):
  • inspector/front-end/inspector.css:
  • inspector/front-end/inspector.js: (WebInspector.updateResource):
09:42 Changeset [55465] by mitz@apple.com

Build fix.

  • dom/ContainerNode.cpp:

(WebCore::ContainerNode::removeChildren):

09:24 Changeset [55464] by pfeldman@chromium.org

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

Reviewed by Timothy Hatcher.

Web Inspector: Get rid of synchronous debuggerEnabled, profilerEnabled calls.

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

  • inspector/InspectorBackend.cpp:
  • inspector/InspectorBackend.h:
  • inspector/InspectorBackend.idl:
  • inspector/front-end/BreakpointsSidebarPane.js: (WebInspector.BreakpointsSidebarPane.prototype.addBreakpoint): (WebInspector.BreakpointsSidebarPane.prototype.removeBreakpoint): (WebInspector.BreakpointsSidebarPane.prototype._breakpointEnableChanged):
  • inspector/front-end/ProfilesPanel.js: (WebInspector.ProfilesPanel): (WebInspector.ProfilesPanel.prototype.profilerWasEnabled): (WebInspector.ProfilesPanel.prototype.profilerWasDisabled): (WebInspector.ProfilesPanel.prototype._updateInterface): (WebInspector.ProfilesPanel.prototype._enableProfiling): (WebInspector.ProfilesPanel.prototype._toggleProfiling):
  • inspector/front-end/ScriptsPanel.js: (WebInspector.ScriptsPanel): (WebInspector.ScriptsPanel.prototype.debuggerWasEnabled): (WebInspector.ScriptsPanel.prototype.debuggerWasDisabled): (WebInspector.ScriptsPanel.prototype.reset): (WebInspector.ScriptsPanel.prototype.canShowSourceLine): (WebInspector.ScriptsPanel.prototype._updateDebuggerButtons): (WebInspector.ScriptsPanel.prototype._enableDebugging): (WebInspector.ScriptsPanel.prototype._toggleDebugging):
09:19 Changeset [55463] by eric.carlson@apple.com

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

Reviewed by Adele Peterson.

Replace 'autobuffer' attribute with 'preload' to align with spec revision 4811.
https://bugs.webkit.org/show_bug.cgi?id=35385
rdar://problem/7689602

Tests: media/audio-constructor-preload.html

media/video-dom-preload.html

  • html/HTMLAttributeNames.in: Remove autobuffer, add preload
  • html/HTMLAudioElement.cpp: (WebCore::HTMLAudioElement::createForJSConstructor): set preload to 'auto' instead of

autobuffer to true.

  • html/HTMLMediaElement.cpp: (WebCore::HTMLMediaElement::HTMLMediaElement): Initialize m_preload. (WebCore::HTMLMediaElement::parseMappedAttribute): Deal with 'preload' attribute. (WebCore::HTMLMediaElement::loadResource): Call setPreload() on the newly created MediaPlayer,

before calling load() so it can pass the setting through to the media engine.

(WebCore::HTMLMediaElement::preload): New.
(WebCore::HTMLMediaElement::setPreload): Ditto

  • html/HTMLMediaElement.h:
  • html/HTMLMediaElement.idl:
  • platform/graphics/MediaPlayer.cpp: (WebCore::MediaPlayer::MediaPlayer): Initialize m_preload. (WebCore::MediaPlayer::load): Pass m_preload to newly created media engine. (WebCore::MediaPlayer::preload): New, return m_preload. (WebCore::MediaPlayer::setPreload): New, set m_preload.
  • platform/graphics/MediaPlayer.h: (WebCore::MediaPlayer::):
  • platform/graphics/MediaPlayerPrivate.h: (WebCore::MediaPlayerPrivateInterface::setPreload):
  • platform/graphics/gtk/MediaPlayerPrivateGStreamer.cpp: (WebCore::MediaPlayerPrivate::setPreload): Renamed from setAutoplay, fix logic for preload.
  • platform/graphics/gtk/MediaPlayerPrivateGStreamer.h:

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

Reviewed by Adele Peterson.

Replace 'autobuffer' attribute with 'preload' to align with spec revision 4811.
https://bugs.webkit.org/show_bug.cgi?id=35385
rdar://problem/7689602

  • media/audio-constructor-autobuffer-expected.txt: Removed.
  • media/audio-constructor-autobuffer.html: Removed.
  • media/audio-constructor-preload-expected.txt: Added.
  • media/audio-constructor-preload.html: Added.
  • media/video-dom-autobuffer-expected.txt: Removed.
  • media/video-dom-autobuffer.html: Removed.
  • media/video-dom-preload-expected.txt: Added.
  • media/video-dom-preload.html: Added.
07:21 QtWebKitBugs edited by hausmann@webkit.org
(diff)
07:02 Changeset [55462] by mitz@apple.com

<rdar://problem/7682756> Assertion failure when replacing the contents of a <select>

Reviewed by Sam Weinig.

WebCore:

Test: fast/dom/remove-children-notification-order.html

  • dom/ContainerNode.cpp:

(WebCore::ContainerNode::removeChildren): Changed to call childrenChanged()
before calling removedFromDocument() on each removed child, which matches
the order removeChild() does things, and avoids the assertion. This required
temporarily storing the removed children in a vector.
Also added comments about other discrepancies between this function and
removeChild().

LayoutTests:

  • fast/dom/remove-children-notification-order-expected.txt: Added.
  • fast/dom/remove-children-notification-order.html: Added.
06:48 Changeset [55461] by abarth@webkit.org

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

Reviewed by Darin Adler.

Document cookieURL and firstPartyForCookies
https://bugs.webkit.org/show_bug.cgi?id=35613

Some folks asked what the cookieURL and the firstPartyForCookies were
on IRC. This patch documents these properties in the code so folks
don't have to ask on IRC anymore.

  • dom/Document.h:
06:42 Changeset [55460] by eric@webkit.org

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

Reviewed by Simon Hausmann.

Refactor QtLauncher so it respects command line arguments
and inherits settings when you clone or create a new window.

Implemented with help of Kenneth Rohde Christiansen.

[Qt] QtLauncher must be refactored to fix command line arguments usage
https://bugs.webkit.org/show_bug.cgi?id=35536

  • QtLauncher/main.cpp: (LauncherWindow::LauncherWindow): (LauncherWindow::init): (LauncherWindow::isGraphicsBased): (applySetting): (LauncherWindow::applyPrefs): (LauncherWindow::initializeView): (LauncherWindow::newWindow): (LauncherWindow::cloneWindow): (LauncherWindow::createChrome): (main):
  • QtLauncher/webview.cpp: (WebViewGraphicsBased::WebViewGraphicsBased): (WebViewGraphicsBased::setFrameRateMeasurementEnabled):
  • QtLauncher/webview.h: (WebViewGraphicsBased::itemCacheMode): (WebViewGraphicsBased::frameRateMeasurementEnabled):
06:40 Google Summer of Code 2010 edited by aroben@apple.com
(diff)
06:00 Google Summer of Code 2010 edited by alex@milowski.com
(diff)
05:30 Changeset [55459] by eric@webkit.org

2010-03-03 Arno Renevier <arno@renevier.net>

Reviewed by Gustavo Noronha Silva.

[Gtk]: assertion triggered when geolocation getCurrentPosition without an option argument.
https://bugs.webkit.org/show_bug.cgi?id=35666

  • platform/gtk/GeolocationServiceGtk.cpp: (WebCore::GeolocationServiceGtk::startUpdating):
05:17 Changeset [55458] by kov@webkit.org

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

[GTK] build slaves need working geoclue service
https://bugs.webkit.org/show_bug.cgi?id=35674

Skip tests we're not really able to run for the time being.

  • platform/gtk/Skipped:
04:44 Changeset [55457] by eric@webkit.org

2010-03-03 Fridrich Strba <fridrich.strba@bluewin.ch>

Reviewed by Xan Lopez.

Miscellaneous little fixes for the windows build of webkit-gtk
https://bugs.webkit.org/show_bug.cgi?id=35640

  • GNUmakefile.am: On Windows with GCC, presence of declspec(dllexport) on some symbols disables the autoexport/autoimport feature for all others. Using regex here assures that all symbols that need to be exported in the dll are actually exported.

2010-03-03 Fridrich Strba <fridrich.strba@bluewin.ch>

Reviewed by Xan Lopez.

Miscellaneous little fixes for the windows build of webkit-gtk
https://bugs.webkit.org/show_bug.cgi?id=35640

  • GNUmakefile.am: dist two new files concerning mathml support.

2010-03-03 Fridrich Strba <fridrich.strba@bluewin.ch>

Reviewed by Xan Lopez.

Miscellaneous little fixes for the windows build of webkit-gtk
https://bugs.webkit.org/show_bug.cgi?id=35640

  • webkit/webkitdownload.cpp: Windows headers define ERROR which breaks the build. Undef ERROR if it is defined.
04:09 Changeset [55456] by xan@webkit.org

2010-03-03 Xan Lopez <xlopez@igalia.com>

Unreviewed distcheck fix.

Add new file to the build.

  • GNUmakefile.am:
04:02 Changeset [55455] by antti@apple.com

Make IntPoint usable as a hash key

Reviewed by Oliver Hunt.

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

  • WebCore.pro:
  • WebCore.xcodeproj/project.pbxproj:
  • platform/graphics/IntPointHash.h: Added.

(WTF::IntPointHash::hash):
(WTF::IntPointHash::equal):
(WTF::):

00:24 Changeset [55454] by philn@webkit.org

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

Unreviewed, build fix after r55452.

  • webkit/webkitwebview.cpp: (DNDContentsRequest::webkit_web_view_settings_notify): added missing braces.
00:07 Changeset [55453] by eric@webkit.org

2010-03-02 Fridrich Strba <fridrich.strba@bluewin.ch>

Reviewed by Oliver Hunt.

Allow building smoothly on win32 and win64 using GCC
https://bugs.webkit.org/show_bug.cgi?id=35607

  • jit/JITStubs.h:
  • runtime/Collector.cpp: (JSC::Heap::allocateBlock): (JSC::Heap::freeBlockPtr): (JSC::currentThreadStackBase):

03/02/10:

23:43 Changeset [55452] by eric@webkit.org

2010-03-02 Eric Uhrhane <ericu@chromium.org>

Reviewed by David Levin.

Move database enable bit fully out of settings
This is stage one of a three-stage commit [webkit, then chromium, then
webkit again]. In this change I'm adding calls to
Database::setIsAvailable inside Settings::setDatabaseEnabled and
anywhere else that called it, and switching webkit fully over to using
that flag [added in a previous checkin]. Phase two will remove
Chromium's use of Settings for the Database, and phase three will remove
the Setting for the Database enable entirely, leaving only
Database::isAvailable/setIsAvailable.

No new tests; tested by existing storage tests.

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

  • WebCore.base.exp: Export Database::setIsAvailable
  • WebCore.xcodeproj/project.pbxproj: Export needed headers as Private
  • page/DOMWindow.cpp: Read isAvailable, not Settings::isDatabaseEnabled (WebCore::DOMWindow::openDatabase):
  • page/Settings.cpp: Add a call to Database::setIsAvailable. (WebCore::Settings::setDatabasesEnabled):

2010-03-02 Eric Uhrhane <ericu@chromium.org>

Reviewed by David Levin.

Move database enable bit fully out of settings
This is stage one of a three-stage commit [webkit, then chromium, then
webkit again]. In this change I'm adding calls to
Database::setIsAvailable inside Settings::setDatabaseEnabled and
anywhere else that called it, and switching webkit fully over to using
that flag [added in a previous checkin]. Phase two will remove
Chromium's use of Settings for the Database, and phase three will remove
the Setting for the Database enable entirely, leaving only
Database::isAvailable/setIsAvailable.

No new tests; tested by existing storage tests.

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

  • webkit/webkitwebview.cpp: Add calls to Database::setIsAvailable (DNDContentsRequest::webkit_web_view_update_settings): (DNDContentsRequest::webkit_web_view_settings_notify):

2010-03-02 Eric Uhrhane <ericu@chromium.org>

Reviewed by David Levin.

Move database enable bit fully out of settings
This is stage one of a three-stage commit [webkit, then chromium, then
webkit again]. In this change I'm adding calls to
Database::setIsAvailable inside Settings::setDatabaseEnabled and
anywhere else that called it, and switching webkit fully over to using
that flag [added in a previous checkin]. Phase two will remove
Chromium's use of Settings for the Database, and phase three will remove
the Setting for the Database enable entirely, leaving only
Database::isAvailable/setIsAvailable.

No new tests; tested by existing storage tests.

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

  • WebView/WebView.mm: Added a call to Database::setIsAvailable. (-[WebView _preferencesChangedNotification:]):

2010-03-02 Eric Uhrhane <ericu@chromium.org>

Reviewed by David Levin.

Move database enable bit fully out of settings
This is stage one of a three-stage commit [webkit, then chromium, then
webkit again]. In this change I'm adding calls to
Database::setIsAvailable inside Settings::setDatabaseEnabled and
anywhere else that called it, and switching webkit fully over to using
that flag [added in a previous checkin]. Phase two will remove
Chromium's use of Settings for the Database, and phase three will remove
the Setting for the Database enable entirely, leaving only
Database::isAvailable/setIsAvailable.

No new tests; tested by existing storage tests.

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

  • Api/qwebsettings.cpp: (QWebSettingsPrivate::apply): Add a call to Database::setIsAvailable
23:22 Changeset [55451] by tony@chromium.org

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

Reviewed by Darin Fisher.

add a flag to WebURLResponse so we can identify multipart content
https://bugs.webkit.org/show_bug.cgi?id=35628

  • platform/network/chromium/ResourceResponse.h: (WebCore::ResourceResponse::ResourceResponse): (WebCore::ResourceResponse::isMultipartPayload): (WebCore::ResourceResponse::setIsMultipartPayload):

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

Reviewed by Darin Fisher.

add a flag to WebURLResponse so we can identify multipart content
https://bugs.webkit.org/show_bug.cgi?id=35628

  • public/WebURLResponse.h:
  • src/WebURLResponse.cpp: (WebKit::WebURLResponse::isMultipartPayload): (WebKit::WebURLResponse::setIsMultipartPayload):
22:46 Changeset [55450] by tony@chromium.org

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

Not reviewed, test fix.

Revert r55447 because the new layout test is crashing consistently
on Leopard Intel Debug (tests).
https://bugs.webkit.org/show_bug.cgi?id=35261

  • svg/custom/tiling-regular-hexagonal-crash-expected.txt: Removed.
  • svg/custom/tiling-regular-hexagonal-crash.svg: Removed.

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

Not reviewed, test fix.

Revert r55447 because the new layout test is crashing consistently
on Leopard Intel Debug (tests).
https://bugs.webkit.org/show_bug.cgi?id=35261

  • platform/graphics/skia/ImageSkia.cpp: (WebCore::BitmapImageSingleFrameSkia::create):
19:16 Changeset [55449] by eric@webkit.org

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

Reviewed by Darin Fisher.

Google Analytics triggers "blocked plugin" UI
https://bugs.webkit.org/show_bug.cgi?id=35565

Just like for running script, we need to distinguish between querying
whether plug-ins are enabled and actually blocking a page from
instantiating a plugin. We need to issue different callbacks to the
FrameLoaderClient so that the client can inform us that plug-ins are
disabled in some cases without showing the "plug-in blocked" UI.

  • dom/DOMImplementation.cpp: (WebCore::DOMImplementation::createDocument):
  • loader/FrameLoader.cpp: (WebCore::FrameLoader::requestObject): (WebCore::FrameLoader::allowPlugins):
  • loader/FrameLoader.h: (WebCore::):
  • loader/FrameLoaderClient.h: (WebCore::FrameLoaderClient::didNotAllowPlugins):
  • loader/MainResourceLoader.cpp: (WebCore::MainResourceLoader::substituteMIMETypeFromPluginDatabase):
  • loader/PluginDocument.cpp: (WebCore::PluginTokenizer::writeRawData):
  • page/Page.cpp: (WebCore::Page::pluginData):
  • plugins/MimeType.cpp: (WebCore::MimeType::enabledPlugin):

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

Reviewed by Darin Fisher.

Google Analytics triggers "blocked plugin" UI
https://bugs.webkit.org/show_bug.cgi?id=35565

Plumb didNotAllowPlugins to the client.

  • public/WebFrameClient.h: (WebKit::WebFrameClient::didNotAllowPlugins):
  • src/FrameLoaderClientImpl.cpp: (WebKit::FrameLoaderClientImpl::didNotAllowPlugins):
  • src/FrameLoaderClientImpl.h:
18:53 Changeset [55448] by eric@webkit.org

2010-03-02 Andreas Kling <andreas.kling@nokia.com>

Reviewed by Simon Hausmann.

[Qt] Support the HTTP OPTIONS verb (needed for preflight requests)

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

  • platform/network/qt/QNetworkReplyHandler.cpp: (WebCore::QNetworkReplyHandler::QNetworkReplyHandler): (WebCore::QNetworkReplyHandler::start):
18:37 Changeset [55447] by eric@webkit.org

2010-03-02 Evan Stade <estade@chromium.org>

Reviewed by David Levin.

https://bugs.webkit.org/show_bug.cgi?id=35261
[skia] crash when attempting to render certain SVGs with Skia

This test passes if it doesn't crash.

  • svg/custom/tiling-regular-hexagonal-crash-expected.txt: Added.
  • svg/custom/tiling-regular-hexagonal-crash.svg: Added.

2010-03-02 Evan Stade <estade@chromium.org>

Reviewed by David Levin.

https://bugs.webkit.org/show_bug.cgi?id=35261
[skia] crash when attempting to render certain SVGs

This fixes the crash, but the SVG still doesn't render properly.

Test: svg/custom/tiling-regular-hexagonal-crash.svg

  • platform/graphics/skia/ImageSkia.cpp: (WebCore::BitmapImageSingleFrameSkia::create): don't return 0 when the copy fails; instead return a blank bitmap. The caller doesn't check for 0 before dereferencing.
18:20 Changeset [55446] by eric@webkit.org

2010-03-02 Arno Renevier <arno@renevier.net>

Reviewed by Gustavo Noronha Silva.

[Gtk] use geoclue providers with don't provide update
https://bugs.webkit.org/show_bug.cgi?id=35191

No new tests, behaviour depends on system.

  • platform/gtk/GeolocationServiceGtk.cpp: (WebCore::GeolocationServiceGtk::startUpdating):
17:43 Changeset [55445] by eric@webkit.org

2010-03-02 John Abd-El-Malek <jam@chromium.org>

Reviewed by Darin Adler.

Remove unnecessary check.
https://bugs.webkit.org/show_bug.cgi?id=35513

  • platform/ScrollView.cpp: (WebCore::ScrollView::updateScrollbars):
17:27 Changeset [55444] by eric@webkit.org

2010-03-02 Arno Renevier <arno@renevier.net>

Reviewed by Gustavo Noronha Silva.

[Gtk] implements ChromeClient::requestGeolocationPermissionForFrame
https://bugs.webkit.org/show_bug.cgi?id=35210

  • GNUmakefile.am:

2010-03-02 Arno Renevier <arno@renevier.net>

Reviewed by Gustavo Noronha Silva.

[Gtk] implements ChromeClient::requestGeolocationPermissionForFrame
https://bugs.webkit.org/show_bug.cgi?id=35210

  • platform/gtk/Skipped:

2010-03-02 Arno Renevier <arno@renevier.net>

Reviewed by Gustavo Noronha Silva.

[Gtk] implements ChromeClient::requestGeolocationPermissionForFrame
https://bugs.webkit.org/show_bug.cgi?id=35210

  • WebCoreSupport/ChromeClientGtk.cpp:
  • WebCoreSupport/ChromeClientGtk.h:
  • docs/webkitgtk-sections.txt:
  • webkit/webkit.h:
  • webkit/webkitdefines.h:
  • webkit/webkitgeolocationpolicydecision.cpp: Added. (webkit_geolocation_policy_decision_class_init): (webkit_geolocation_policy_decision_init): (webkit_geolocation_policy_decision_new): (webkit_geolocation_policy_allow): (webkit_geolocation_policy_deny):
  • webkit/webkitgeolocationpolicydecision.h: Added.
  • webkit/webkitprivate.h:
  • webkit/webkitwebview.cpp: (DNDContentsRequest::webkit_web_view_class_init):
  • webkitmarshal.list:

2010-03-02 Arno Renevier <arno@renevier.net>

Reviewed by Gustavo Noronha Silva.

[Gtk] implements ChromeClient::requestGeolocationPermissionForFrame
https://bugs.webkit.org/show_bug.cgi?id=35210

  • DumpRenderTree/gtk/DumpRenderTree.cpp: (geolocationPolicyDecisionRequested): (createWebView):
16:59 Changeset [55443] by darin@chromium.org

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

Fix chromium build bustage.

  • page/Page.cpp: (WebCore::Page::privateBrowsingStateChanged): PluginView methods may not be called in the Chromium port.
16:48 Changeset [55442] by bdakin@apple.com

WebCore: Fix for <rdar://problem/7485289> WebKit crashes on systems that
don't support CoreAnimation

Reviewed by Darin Adler and Adam Roben.

WKCACFLayerRenderer::acceleratedCompositingAvailable() now creates
a dummy HWND so that it can step through the whole setHostWindow()
and createRenderer() process. createRenderer() also calls a new
function, hardwareCapabilitiesIndicateCoreAnimationSupport() which
can only be called once we have a d3dDevice.

setHostWindow() and createRenderer() now both return bools that
indicate whether or not they have succeeded.

  • platform/graphics/win/WKCACFLayerRenderer.cpp:

(WebCore::hardwareCapabilitiesIndicateCoreAnimationSupport):
(WebCore::CoreAnimationTesterWindowWndProc):
(WebCore::WKCACFLayerRenderer::acceleratedCompositingAvailable):
(WebCore::WKCACFLayerRenderer::shared):
(WebCore::WKCACFLayerRenderer::createRenderer):

  • platform/graphics/win/WKCACFLayerRenderer.h:

(WebCore::WKCACFLayerRenderer::setHostWindow):

WebKit/win: Tiny WebKit portion of fix for <rdar://problem/7485289> WebKit
crashes on systems that don't support CoreAnimation

Reviewed by Darin Adler and Adam Roben.

setHostWindow() no longer calls createRenderer(), so now that has
to be called manually.

  • WebView.cpp:

(WebView::setAcceleratedCompositing):

16:43 Changeset [55441] by mrowe@apple.com

ScriptDebugServer shouldn't care that Mac does not use PluginView for plug-ins.

Rubber-stamped by Jon Honeycutt.

  • bindings/js/ScriptDebugServer.cpp:
  • plugins/PluginViewNone.cpp:

(WebCore::PluginView::setJavaScriptPaused): Add an empty implementation of setJavaScriptPaused.

16:36 Changeset [55440] by mrowe@apple.com

Clean up the build fix r55437 by adding an empty implementation of privateBrowsingStateChanged in PluginViewNone.cpp

Rubber-stamped by Jon Honeycutt.

  • page/Page.cpp:
  • plugins/PluginViewNone.cpp:

(WebCore::PluginView::privateBrowsingStateChanged):

16:16 Changeset [55439] by dpranke@chromium.org

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

Reviewed by David Levin.

Revert r55339 - the Chromium codebase downstream had a temporary
need for WebKitTools/pywebsocket to still exist after Chris Jerdonek
had moved it (scripts still referenced the old location). Those
scripts have been updated to the new location, so it should be safe
to delete this now.

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

  • pywebsocket/mod_pywebsocket/standalone.py: Removed.
16:09 Changeset [55438] by andersca@apple.com

NPWindow is now null in the Cocoa event model.

Reviewed by Kevin Decker.

  • NetscapeInputMethodPlugin/main.m:

(handleDraw):
(NPP_HandleEvent):

15:39 Changeset [55437] by mrowe@apple.com

Fix the Mac build.

  • page/Page.cpp:

(WebCore::Page::privateBrowsingStateChanged): Mac doesn't use WebCore's PluginView class
for plug-ins, so provide an empty implementation of privateBrowsingStateChanged for Mac.

15:33 Changeset [55436] by mrowe@apple.com

WebCore: Add the capability to create and dispatch a WheelEvent in JavaScript.
Ensure the event's default handler is triggered in the same way as it is
during a PlatformWheelEvent.

Patch by Andy Estes <aestes@apple.com> on 2010-03-02
Reviewed by Maciej Stachowiak.

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

Test: fast/events/wheelevent-in-scrolling-div.html

  • dom/Node.cpp: Ensure that the default behavior (scrolling) occurs for

wheel events originating both from the platform and from
JavaScript/ObjC.
(WebCore::Node::dispatchWheelEvent): Instantiate new WheelEvent with
the graunularity of the PlatformWheelEvent.
(WebCore::Node::defaultEventHandler): Add support for mousewheel events.

  • dom/WheelEvent.cpp: Add three new member variables: m_deltaX, m_deltaY

and m_granularity. m_deltaX and m_deltaY differ from m_wheelDeltaX and
m_wheelDeltaY, which are the number of wheel ticks multiplied by 120 for
IE compatibility.
(WebCore::WheelEvent::WheelEvent): Initialize new member variables.
(WebCore::WheelEvent::initWheelEvent): Same.
(WebCore::WheelEvent::initWebKitWheelEvent): Same.

  • dom/WheelEvent.h: See WheelEvent.cpp.

(WebCore::WheelEvent::): Add Granularity enum (Pixel, Line, Page).
(WebCore::WheelEvent::create): Add new arguments.
(WebCore::WheelEvent::deltaX): Amount of scroll in x direction.
(WebCore::WheelEvent::deltaY): Amount of scroll in y direction.
(WebCore::WheelEvent::granularity): Units of deltaX and deltaY.

  • dom/WheelEvent.idl: Add initWebKitWheelEvent() to JavaScript. This is

the same as the initWheelEvent ObjC method. As the DOM Level 3 Events
specification is still a working draft and subject to change, prefix
'WebKit' to the method signature to indicate experimental support.

  • page/EventHandler.cpp: Move the scroll handling from

handleWheelEvent() to defaultWheelEventHandler(), which is executed on
both PlatformWheelEvents and JavaScript WheelEvents.
(WebCore::scrollNode): Renamed from scrollAndAcceptEvent(). Remove
the PlatformWheelEvent from the argument list and instead return a
boolean indicating if the scroll event was accepted.
(WebCore::EventHandler::handleWheelEvent): Move scrolling code from here
(WebCore::EventHandler::defaultWheelEventHandler): ...to here.

  • page/EventHandler.h: Add function signature.

LayoutTests: Add a test for the patch to https://bugs.webkit.org/show_bug.cgi?id=35566.
These can be run manually or from DRT.

Patch by Andy Estes <aestes@apple.com> on 2010-03-02
Reviewed by Maciej Stachowiak.

  • fast/events/wheelevent-in-scrolling-div-expected.txt: Added.
  • fast/events/wheelevent-in-scrolling-div.html: Added.
15:32 Changeset [55435] by dpranke@chromium.org

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

Reviewed by Dimitri Glazkov.

new-run-webkit-tests on chromium-linux tests to see if
layout_test_helper exists, but we don't use layout_test_helper on
linux. The test derefences a None object, and we crash. This fixes
that.

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

  • Scripts/webkitpy/layout_tests/port/chromium.py:

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

Reviewed by Dimitri Glazkov.

r55388 (bug 35553) worked around a bug in Python's subprocess.Popen()
that was causing DRT to hang on exit in new-run-webkit-tests.
Unfortunately, that workaround doesn't work on chromium-win
(and the script fails completely). The good news is that the check
isn't actually necessary, and so this change makes it conditional.

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

  • Scripts/webkitpy/layout_tests/port/chromium.py:
15:29 Changeset [55434] by dpranke@chromium.org

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

Reviewed by Dimitri Glazkov.

r55388 (bug 35553) worked around a bug in Python's subprocess.Popen()
that was causing DRT to hang on exit in new-run-webkit-tests.
Unfortunately, that workaround doesn't work on chromium-win
(and the script fails completely). The good news is that the check
isn't actually necessary, and so this change makes it conditional.

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

  • Scripts/webkitpy/layout_tests/port/chromium.py:
15:13 Changeset [55433] by mrowe@apple.com

Bug 35576: WebKit should tell plug-in instances when private browsing state changes
<http://webkit.org/b/35576>

Reviewed by Darin Adler.

WebCore:

Notify plug-in instances when the private browsing state changes to match the behavior of the
Mac plug-in code.

  • page/Page.cpp:

(WebCore::Page::privateBrowsingStateChanged): Walk the frame tree and notify each PluginView that
the private browsing state has changed.

  • page/Page.h:
  • page/Settings.cpp:

(WebCore::Settings::setPrivateBrowsingEnabled): Notify the page that the private browsing state
has changed.

  • plugins/PluginView.cpp:

(WebCore::PluginView::privateBrowsingStateChanged): Notify the plug-in instance of the new private
browsing state.

  • plugins/PluginView.h:

WebKitTools:

TestNetscapePlugin is another bit of plug-in code where copy-paste was heavily used
when porting. Update the Windows and UNIX implementations of NPP_New and NPP_SetValue
to provide the expected behavior related to NPNVprivateModeBool. Hopefully this code
duplication can be cleaned up in the future.

  • DumpRenderTree/unix/TestNetscapePlugin/TestNetscapePlugin.cpp:

(webkit_test_plugin_new_instance):
(webkit_test_plugin_set_value):

  • DumpRenderTree/win/TestNetscapePlugin/main.cpp:

(NPP_New):
(NPP_SetValue):

LayoutTests:

  • platform/win/Skipped: Remove plugins/private-browsing-mode.html now that it passes.
15:13 Changeset [55432] by mrowe@apple.com

Bug 30348: Implement private mode for plug-ins on Windows
<http://webkit.org/b/30348> / <rdar://problem/7562261>

Reviewed by Oliver Hunt.

WebCore:

Rework PluginView::getValue and PluginView::getValueStatic to remove the amount of code that
was duplicated across platforms. getValue and getValueStatic now call in to platform-specific
variants that indicate whether they handled the query. If the query is not handled by the
platform-specific variants then the cross-platform handler has a chance to handle it.

  • plugins/PluginView.cpp:

(WebCore::PluginView::getValueStatic): Give the platform-specific variant a chance to handle the
variable lookup. If it does not handle it, return an error.
(WebCore::PluginView::getValue): Give the platform-specific variant and platform-specific static
variant a chance to handle the variable lookup. If they do not handle it, apply the cross-platform
handler. At the moment the cross-platform code handles NPNVWindowNPObject, NPNVPluginElementNPObject,
and NPNVprivateModeBool as they have an identical implementation across ports.

  • plugins/PluginView.h:
  • plugins/PluginViewNone.cpp:

(WebCore::PluginView::platformGetValue): PluginViewNone does not handle any lookups.
(WebCore::PluginView::platformGetValueStatic): Ditto.

  • plugins/gtk/PluginViewGtk.cpp:

(WebCore::PluginView::platformGetValueStatic):
(WebCore::PluginView::platformGetValue):
platform-independent implementation.

  • plugins/mac/PluginViewMac.cpp:

(WebCore::PluginView::platformGetValueStatic):
(WebCore::PluginView::platformGetValue):

  • plugins/qt/PluginViewQt.cpp:

(WebCore::PluginView::platformGetValueStatic):
(WebCore::PluginView::platformGetValue): Fix a bug noticed while updating this code.
The Qt implementation of the handler for NPNVToolkit was relying on case fall-through
to have some values handled by the static handler. When NPNVprivateModeBool was added
it was placed before the default case, interferring with this fall-through. It now
explicitly indicates in this situation that it was not handled.

  • plugins/symbian/PluginViewSymbian.cpp:

(WebCore::PluginView::platformGetValueStatic):
(WebCore::PluginView::platformGetValue):

  • plugins/win/PluginViewWin.cpp:

(WebCore::PluginView::platformGetValueStatic):
(WebCore::PluginView::platformGetValue):

LayoutTests:

  • platform/win/Skipped: Remove plugins/private-browsing-mode-2.html now that it passes.
14:27 Changeset [55431] by andersca@apple.com

Make the default constructor available to all platforms.

Reviewed by Sam Weinig.

  • platform/PlatformKeyboardEvent.h:

(WebCore::PlatformKeyboardEvent::PlatformKeyboardEvent):

  • platform/mac/KeyEventMac.mm:
14:24 Changeset [55430] by ap@apple.com

Reviewed by Adam Roben.

https://bugs.webkit.org/show_bug.cgi?id=35600
shouldBe() fails when comparing an array to null.

  • fast/js/resources/js-test-pre.js: (areArraysEqual): This function is expected to be called with weird arguments, as it's used with array-like objects like WebGLByteArray. Made it resilient against exceptions raised when dealing with those.
14:21 Changeset [55429] by eric@webkit.org

2010-03-02 Jeremy Orlow <jorlow@chromium.org>

Reviewed by David Levin.

Revert database thread changes that are no longer required
https://bugs.webkit.org/show_bug.cgi?id=35519

Jochen Eisinger created 55214 and 55247 to track which database
owns which thread. Dmitry suggested that this could also
be done via TLS, though. After exploring the options, Jochen
chose to go the TLS route, so these patches are no longer needed.

  • wtf/Threading.h:
  • wtf/ThreadingNone.cpp: (WTF::isMainThread):
  • wtf/ThreadingPthreads.cpp: (WTF::identifierByPthreadHandle): (WTF::establishIdentifierForPthreadHandle): (WTF::pthreadHandleForIdentifier): (WTF::createThreadInternal): (WTF::currentThread):
  • wtf/ThreadingWin.cpp: (WTF::threadMap): (WTF::storeThreadHandleByIdentifier): (WTF::threadHandleForIdentifier): (WTF::createThreadInternal):

2010-03-02 Jeremy Orlow <jorlow@chromium.org>

Reviewed by David Levin.

Revert database thread changes that are no longer required
https://bugs.webkit.org/show_bug.cgi?id=35519

Jochen Eisinger created 55214 and 55247 to track which database
owns which thread. Dmitry suggested that this could also
be done via TLS, though. After exploring the options, Jochen
chose to go the TLS route, so these patches are no longer needed.

  • storage/DatabaseThread.cpp: (WebCore::DatabaseThread::DatabaseThread): (WebCore::DatabaseThread::databaseThread):
  • storage/DatabaseThread.h: (WebCore::DatabaseThread::getThreadID):
13:46 Changeset [55428] by beidson@apple.com

Followup for REGRESSION(r51097) - Unable to log in to statefarm.com
<rdar://problem/7672667> and https://bugs.webkit.org/show_bug.cgi?id=35556

Reviewed by Sam Weinig.

WebCore:

  • dom/ScriptElement.cpp:

(WebCore::ScriptElementData::shouldExecuteAsJavaScript): To more perfectly match Gecko's rule,

strip whitespace from the attribute values before comparing to window/onload/onload().

LayoutTests:

  • fast/loader/for-window-event-onload-scripts-expected.txt:
  • fast/loader/for-window-event-onload-scripts.html:
13:39 Changeset [55427] by eric@webkit.org

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

Reviewed by Simon Hausmann.

Fix QScriptValue::toString().

More ECMA Script compliance, especially for values as NaN, Inifinite
and really big/small numbers.

[Qt] QScriptValue::toString() returns incorrect values
https://bugs.webkit.org/show_bug.cgi?id=34850

  • qt/api/qscriptconverter_p.h: (QScriptConverter::toString):
  • qt/api/qscriptvalue_p.h: (QScriptValuePrivate::toString):
  • qt/tests/qscriptvalue/tst_qscriptvalue.cpp:
  • qt/tests/qscriptvalue/tst_qscriptvalue.h:
  • qt/tests/qscriptvalue/tst_qscriptvalue_generated.cpp: (tst_QScriptValue::toString_initData): (tst_QScriptValue::toString_makeData): (tst_QScriptValue::toString_test):
13:20 Changeset [55426] by eric@webkit.org

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

Reviewed by Simon Hausmann.

Introduce a new class; QScriptString.

The QScriptString class should act as a handle to "interned"
strings in a QScriptEngine.

[Qt] QtScript should provide QScriptString
https://bugs.webkit.org/show_bug.cgi?id=34843

  • qt/api/QtScript.pro:
  • qt/api/qscriptengine.cpp: (QScriptEngine::toStringHandle):
  • qt/api/qscriptengine.h:
  • qt/api/qscriptengine_p.h: (QScriptEnginePrivate::toStringHandle):
  • qt/api/qscriptstring.cpp: Added. (QScriptString::QScriptString): (QScriptString::~QScriptString): (QScriptString::operator=): (QScriptString::isValid): (QScriptString::operator==): (QScriptString::operator!=): (QScriptString::toArrayIndex): (QScriptString::toString): (QScriptString::operator QString): (qHash):
  • qt/api/qscriptstring.h: Added.
  • qt/api/qscriptstring_p.h: Added. (QScriptStringPrivate::QScriptStringPrivate): (QScriptStringPrivate::~QScriptStringPrivate): (QScriptStringPrivate::get): (QScriptStringPrivate::isValid): (QScriptStringPrivate::operator==): (QScriptStringPrivate::operator!=): (QScriptStringPrivate::toArrayIndex): (QScriptStringPrivate::toString): (QScriptStringPrivate::id):
  • qt/tests/qscriptstring/qscriptstring.pro: Added.
  • qt/tests/qscriptstring/tst_qscriptstring.cpp: Added. (tst_QScriptString::tst_QScriptString): (tst_QScriptString::~tst_QScriptString): (tst_QScriptString::test): (tst_QScriptString::hash): (tst_QScriptString::toArrayIndex_data): (tst_QScriptString::toArrayIndex):
  • qt/tests/tests.pro:
13:15 Changeset [55425] by cfleizach@apple.com

aria-label isn't respected on option elements
https://bugs.webkit.org/show_bug.cgi?id=35400

Reviewed by Beth Dakin.

WebCore:

When aria-label is used on an <option> element, it can take three forms.
An option in a multi-select list, a popup button and the menu that is displayed
from the popup button. This patches the three requisite locations so that if
aria-label is used, the correct accessibility text is returned.

Test: platform/mac/accessibility/option-with-arialabel.html

  • accessibility/AccessibilityListBoxOption.cpp:

(WebCore::AccessibilityListBoxOption::stringValue):

  • accessibility/AccessibilityObject.cpp:

(WebCore::AccessibilityObject::getAttribute):

  • accessibility/AccessibilityObject.h:
  • accessibility/AccessibilityRenderObject.cpp:

(WebCore::AccessibilityRenderObject::getAttribute):
(WebCore::AccessibilityRenderObject::stringValue):

  • platform/PopupMenuClient.h:
  • platform/mac/PopupMenuMac.mm:

(WebCore::PopupMenu::populate):

  • rendering/RenderMenuList.cpp:

(WebCore::RenderMenuList::itemAccessibilityText):

  • rendering/RenderMenuList.h:
  • rendering/RenderTextControlSingleLine.h:

(WebCore::RenderTextControlSingleLine::itemAccessibilityText):

WebKit/chromium:

  • src/SuggestionsPopupMenuClient.h:

(WebKit::SuggestionsPopupMenuClient::itemAccessibilityText):

LayoutTests:

  • platform/mac/accessibility/option-with-arialabel-expected.txt: Added.
  • platform/mac/accessibility/option-with-arialabel.html: Added.
13:11 QtWebKitBugs edited by jocelyn.turcotte@nokia.com
(diff)
13:08 QtWebKitBugs edited by jocelyn.turcotte@nokia.com
title size adjustment (diff)
13:04 QtWebKit edited by jocelyn.turcotte@nokia.com
link to the QtWebKitBugs page instead (diff)
13:01 QtWebKitContrib edited by jocelyn.turcotte@nokia.com
Move bugs information to QtWebKitBugs (diff)
13:01 QtWebKitBugs created by jocelyn.turcotte@nokia.com
12:48 Changeset [55424] by eric@webkit.org

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

Reviewed by Adam Barth.

[V8] V8 should be notified of context disposals
https://bugs.webkit.org/show_bug.cgi?id=35526

Notify V8 of context disposals to allow it to clean up memory from those
contexts when idle. When disposing a context, start a timer that will
give V8 an idle notification after a while to force cleanup. Use a timer
to avoid performing an idle notification in the middle of navigation where
we know we are not idle.

  • WebCore.gypi:
  • bindings/v8/V8DOMWindowShell.cpp: (WebCore::V8DOMWindowShell::disposeContextHandles):
  • bindings/v8/V8GCForContextDispose.cpp: Added. (WebCore::V8GCForContextDispose::V8GCForContextDispose): (WebCore::V8GCForContextDispose::notifyContextDisposed): (WebCore::V8GCForContextDispose::notifyIdleSooner): (WebCore::V8GCForContextDispose::instance): (WebCore::V8GCForContextDispose::pseudoIdleTimerFired):
  • bindings/v8/V8GCForContextDispose.h: Added.
  • bindings/v8/custom/V8DOMWindowCustom.cpp: (WebCore::WindowSetTimeoutImpl):

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

Reviewed by Adam Barth.

[V8] V8 should be notified of context disposals
https://bugs.webkit.org/show_bug.cgi?id=35526

Update V8 DEPS in order to get V8 API changes needed for context
disposal notifications.

  • DEPS:
12:31 Changeset [55423] by eric@webkit.org

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

Reviewed by Shinichiro Hamaji.

Merge ValidityState-stepMismatch-{number,range,unsupported}.html and
input-step-{number,range,unsupported}.html to other tests
https://bugs.webkit.org/show_bug.cgi?id=35435

  • Merge ValidityState-stepMismatch-{number,range,unsupported}.html to ValidityState-stepMismatch.html
  • Merge input-step--{number,range,unsupported}.html to input-step.html
  • fast/forms/ValidityState-stepMismatch-expected.txt:
  • fast/forms/ValidityState-stepMismatch-number-expected.txt: Removed.
  • fast/forms/ValidityState-stepMismatch-number.html: Removed.
  • fast/forms/ValidityState-stepMismatch-range-expected.txt: Removed.
  • fast/forms/ValidityState-stepMismatch-range.html: Removed.
  • fast/forms/ValidityState-stepMismatch-unsupported-expected.txt: Removed.
  • fast/forms/ValidityState-stepMismatch-unsupported.html: Removed.
  • fast/forms/input-step-number-expected.txt: Removed.
  • fast/forms/input-step-number.html: Removed.
  • fast/forms/input-step-range-expected.txt: Removed.
  • fast/forms/input-step-range.html: Removed.
  • fast/forms/input-step-unsupported-expected.txt: Removed.
  • fast/forms/input-step-unsupported.html: Removed.
  • fast/forms/input-stepup-stepdown-expected.txt:
  • fast/forms/script-tests/ValidityState-stepMismatch-number.js: Removed.
  • fast/forms/script-tests/ValidityState-stepMismatch-range.js: Removed.
  • fast/forms/script-tests/ValidityState-stepMismatch-unsupported.js: Removed.
  • fast/forms/script-tests/ValidityState-stepMismatch.js:
  • fast/forms/script-tests/input-step-number.js: Removed.
  • fast/forms/script-tests/input-step-range.js: Removed.
  • fast/forms/script-tests/input-step-unsupported.js: Removed.
  • fast/forms/script-tests/input-stepup-stepdown.js:
12:18 Changeset [55422] by pfeldman@chromium.org

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

Not reviewed: adding missing image files to WebKit.qrc and gypi.

  • WebCore.gypi:
  • inspector/front-end/WebKit.qrc:
12:15 Changeset [55421] by eric@webkit.org

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

Reviewed by Simon Hausmann.

[Qt] GraphicsLayer: Video element with 3d transform crashes when AC is enabled.
https://bugs.webkit.org/show_bug.cgi?id=35516

No new tests.

  • platform/graphics/qt/GraphicsLayerQt.cpp: (WebCore::GraphicsLayerQt::setContentsToMedia):
11:50 Changeset [55420] by eric@webkit.org

2010-03-02 Fridrich Strba <fridrich.strba@bluewin.ch>

Reviewed by Xan Lopez.

Use unsigned instead of uint which does not exist on windows
https://bugs.webkit.org/show_bug.cgi?id=35546

  • platform/graphics/gtk/ImageGtk.cpp: (WebCore::getCairoSurfacePixel): (WebCore::getGdkPixbufPixel):
11:33 Changeset [55419] by eric@webkit.org

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

Reviewed by Darin Fisher.

Add EnabledAtRuntime attribute to WebGLArray constructors
https://bugs.webkit.org/show_bug.cgi?id=35558

New functionality verified manually in Chromium; not possible to
write layout test. Ran WebGL tests in WebKit as well.

  • bindings/generic/RuntimeEnabledFeatures.cpp:
  • bindings/generic/RuntimeEnabledFeatures.h: (WebCore::RuntimeEnabledFeatures::setWebGLEnabled): (WebCore::RuntimeEnabledFeatures::webGLRenderingContextEnabled): (WebCore::RuntimeEnabledFeatures::webGLArrayBufferEnabled): (WebCore::RuntimeEnabledFeatures::webGLByteArrayEnabled): (WebCore::RuntimeEnabledFeatures::webGLUnsignedByteArrayEnabled): (WebCore::RuntimeEnabledFeatures::webGLShortArrayEnabled): (WebCore::RuntimeEnabledFeatures::webGLUnsignedShortArrayEnabled): (WebCore::RuntimeEnabledFeatures::webGLIntArrayEnabled): (WebCore::RuntimeEnabledFeatures::webGLUnsignedIntArrayEnabled): (WebCore::RuntimeEnabledFeatures::webGLFloatArrayEnabled):
  • page/DOMWindow.idl:

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

Reviewed by Darin Fisher.

Add EnabledAtRuntime attribute to WebGLArray constructors
https://bugs.webkit.org/show_bug.cgi?id=35558

  • public/WebRuntimeFeatures.h:
  • src/WebRuntimeFeatures.cpp: (WebKit::WebRuntimeFeatures::enableWebGL): (WebKit::WebRuntimeFeatures::isWebGLEnabled):
11:12 Changeset [55418] by dimich@chromium.org

Ignore compiled Java test cases in .gitignore.
https://bugs.webkit.org/show_bug.cgi?id=35559

Reviewed by Alexey Proskuryakov.

  • .gitignore:
11:10 Changeset [55417] by japhet@chromium.org

2010-03-02 Nate Chapin <japhet@chromium.org>

Reviewed by Dimitri Glazkov.

[V8] Fail gracefully if NPN_GetProperty tries to retrieve a property that doesn't exist.

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

Required for passing LayoutTests/plugins/netscape-plugin-property-access-exception.html on Chromium.

  • bindings/v8/NPV8Object.cpp: (_NPN_GetProperty): If the result is empty, don't try to convert it to an NPVariant and return false.
10:59 Changeset [55416] by jianli@chromium.org

[V8] DOMCoreException should be visible as DOMException
https://bugs.webkit.org/show_bug.cgi?id=35552

Reviewed by Nate Chapin.

Fix V8 code generator to use the correct visible name.

  • bindings/scripts/CodeGeneratorV8.pm:
10:34 Changeset [55415] by kov@webkit.org

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

Reviewed by Alexey Proskuryakov.

Test plugin still has issues with releasing objects, and variants
https://bugs.webkit.org/show_bug.cgi?id=35587

Fix the conditions for releasing the variants after calling
invoke, and avoid having a number of objects leak.

  • DumpRenderTree/TestNetscapePlugIn.subproj/PluginObject.cpp: (testDocumentOpen): (testWindowOpen):
10:33 Changeset [55414] by beidson@apple.com

REGRESSION(r51097) - Unable to log in to statefarm.com
<rdar://problem/7672667> and https://bugs.webkit.org/show_bug.cgi?id=35556

Reviewed by Sam Weinig.

WebCore:

Test: fast/loader/for-window-event-onload-scripts.html

Match Gecko's rules for executing "for/event" scripts:

-If there's only a 'for' attribute, execute it.
-If there's only an 'event' attribute, execute it.
-If there's a 'for=window' and 'event=onload', execute it.
-If there's a 'for=window' and 'event=onload()', execute it.
-If there's any other combination of both 'for' and 'event', don't execute it.

  • dom/ScriptElement.cpp:

(WebCore::ScriptElementData::shouldExecuteAsJavaScript):

  • dom/ScriptElement.h:
  • html/HTMLScriptElement.cpp:

(WebCore::HTMLScriptElement::eventAttributeValue):

  • html/HTMLScriptElement.h:
  • svg/SVGScriptElement.cpp:

(WebCore::SVGScriptElement::eventAttributeValue):

  • svg/SVGScriptElement.h:

Add the event attribute name:

  • html/HTMLAttributeNames.in:

LayoutTests:

  • fast/loader/for-window-event-onload-scripts-expected.txt: Added.
  • fast/loader/for-window-event-onload-scripts.html: Added.
10:19 Changeset [55413] by eric@webkit.org

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

Reviewed by Darin Fisher.

[Chromium] Implement WebNode::equals and add inline operators for ==
and !=.

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

  • public/WebNode.h: (WebKit::operator==): (WebKit::operator!=):
  • src/WebNode.cpp: (WebKit::WebNode::equals):
09:52 Changeset [55412] by simon.fraser@apple.com

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

Reviewed by Dan Bernstein.

https://bugs.webkit.org/show_bug.cgi?id=35555
QuickTime plugin content can spill outside the <object> tag

Set -masksToBounds on the layer that is handed to us by plug-ins, to ensure that sublayers
of that layer don't spill outside the <object> contents rect.

Manual test because it relies on QuickTime, and pixel results depend on movie loading timing.

  • manual-tests/plugins/object-clipping.html: Added.
  • platform/graphics/mac/GraphicsLayerCA.mm: (WebCore::GraphicsLayerCA::setupContentsLayer):
09:25 Changeset [55411] by cjerdonek@webkit.org

Refactored the StyleChecker class's file-processing method
_process_file(). This will make it easier to add new
file-processing capabilities to check-webkit-style.

Reviewed by Shinichiro Hamaji.

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

  • Scripts/webkitpy/style/checker.py:
    • Added a _read_lines() method to the StyleChecker class that extracts the lines from a file.
    • Replaced part of _process_file() with a call to the new _read_lines() method.
    • Replaced another part of _process_file() with a call to the new CarriageReturnProcessor.process() method.
  • Scripts/webkitpy/style/processors/common.py:
    • Replaced the check_no_carriage_return() function with a new CarriageReturnProcessor class.
  • Scripts/webkitpy/style/processors/common_unittest.py:
    • Renamed the CarriageReturnTest class to CarriageReturnProcessorTest and updated it as necessary.
09:25 Changeset [55410] by cfleizach@apple.com

Fixing layout test problems. No review.

AX: changes to WAI-ARIA grid aren't perceived correctly by VoiceOver
https://bugs.webkit.org/show_bug.cgi?id=35514

AXTable tests crash on Leopard/Tiger because tables don't exist. This test needs to be skipped
on those playtforms.

  • platform/mac-leopard/Skipped:
  • platform/mac-tiger/Skipped:
09:11 QtWebKit edited by vestbo@webkit.org
(diff)
08:48 Changeset [55409] by cjerdonek@webkit.org

2010-03-02 Chris Jerdonek <cjerdonek@webkit.org>

Reviewed by Shinichiro Hamaji.

Started using the logging module in check-webkit-style.
This provides more options for debugging and a more flexible,
uniform way to report messages to the end-user.

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

Also included classes in a central location to facilitate
the unit testing of logging code (setUp and tearDown of unit
test logging configurations, etc).

  • Scripts/check-webkit-style:
    • Added a call to configure_logging() in the beginning of main().
    • Replaced two calls to sys.stderr.write() with appropriate logging calls.
  • Scripts/webkitpy/init/init.py: Copied from WebKitTools/QueueStatusServer/filters/init.py.
  • Scripts/webkitpy/init/logtesting.py: Added.
    • Added a UnitTestLogStream class to capture log output during unit tests.
    • Added a UnitTestLog class that provides convenience methods for unit-testing logging code.
  • Scripts/webkitpy/style/checker.py:
    • Added a configure_logging() method.
    • Added a _LevelLoggingFilter class to filter out log messages above a certain logging level.
    • Removed the _stderr_write() method from the StyleChecker class and replaced its use with appropriate logging calls.
  • Scripts/webkitpy/style/checker_unittest.py:
    • Added a ConfigureLoggingTest class to unit test the configure_logging() method.
    • Updated the StyleCheckerCheckFileTest class as necessary.
  • Scripts/webkitpy/style_references.py:
    • Added references to logtesting.UnitTestLog and logtesting.UnitTestLogStream.
08:22 Changeset [55408] by aroben@apple.com

Expose SecurityOrigin::registerURLSchemeAsSecure as WebKit SPI

Fixes <http://webkit.org/b/35580> <rdar://problem/7706407>

Reviewed by Tim Hatcher.

WebCore:

Export SecurityOrigin::registerURLSchemeAsSecure

  • WebCore.base.exp: Added symbol, sorted file.

WebKit/mac:

Add -[WebView _registerURLSchemeAsSecure:]

  • WebView/WebView.mm:

(+[WebView _registerURLSchemeAsSecure:]):

  • WebView/WebViewPrivate.h:

Added. Calls through to SecurityOrigin::registerURLSchemeAsSecure.

WebKit/win:

Add IWebViewPrivate::registerURLSchemeAsSecure

  • Interfaces/WebKit.idl: Touched to force a build.
  • Interfaces/IWebViewPrivate.idl:
  • WebView.cpp:

(WebView::registerURLSchemeAsSecure):

  • WebView.h:

Added. Calls through to SecurityOrigin::registerURLSchemeAsSecure.

08:15 QtWebKit edited by vestbo@webkit.org
(diff)
08:14 QtWebKit edited by vestbo@webkit.org
(diff)
08:10 QtWebKitContrib edited by vestbo@webkit.org
(diff)
08:09 QtWebKitContrib edited by vestbo@webkit.org
(diff)
06:52 QtWebKitContrib edited by vestbo@webkit.org
(diff)
06:50 QtWebKitContrib edited by vestbo@webkit.org
(diff)
06:44 QtWebKitContrib edited by vestbo@webkit.org
(diff)
06:37 Changeset [55407] by pfeldman@chromium.org

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

Reviewed by Timothy Hatcher.

Web Inspector: render breakpoints as border images instead of canvas.

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

  • inspector/front-end/Images/breakpointBorder.png: Added.
  • inspector/front-end/Images/breakpointConditionalBorder.png: Added.
  • inspector/front-end/Images/breakpointConditionalCounterBorder.png: Added.
  • inspector/front-end/Images/breakpointCounterBorder.png: Added.
  • inspector/front-end/Images/programCounterBorder.png: Added.
  • inspector/front-end/SourceFrame.js: (WebInspector.SourceFrame.prototype._updateExecutionLine): (WebInspector.SourceFrame.prototype._addBreakpointToSource): (WebInspector.SourceFrame.prototype.resize):
  • inspector/front-end/TextViewer.js: (WebInspector.TextChunk):
  • inspector/front-end/textViewer.css:
06:09 CommitterTips edited by cjerdonek@webkit.org
Added section about unreviewed commits. (diff)
06:06 CommitterTips edited by cjerdonek@webkit.org
Added section about rolling out. (diff)
02:16 Changeset [55406] by zecke@webkit.org

[Qt] Special case Font::floatWidthForComplexText for single space

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

For a single space we can go through the QFontMetric::width routine
instead of converting the WebCore::String to a QString and then
going through the QTextLine.

  • platform/graphics/qt/FontQt.cpp:

(WebCore::Font::floatWidthForComplexText):

01:59 Changeset [55405] by zecke@webkit.org

[Gtk] Support private browsing mode in plugins.

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

Integrate the PluginViewQt.cpp changes from
r55358 into Gtk+ and unskip the test.

  • plugins/gtk/PluginViewGtk.cpp:

(WebCore::PluginView::getValue):

  • platform/gtk/Skipped:
01:12 Changeset [55404] by oliver@apple.com

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

Reviewed by NOBODY (Build fix).

Update Qt bridge to new named getter signature

  • bridge/qt/qt_runtime.cpp: (JSC::Bindings::QtRuntimeMetaMethod::lengthGetter): (JSC::Bindings::QtRuntimeMetaMethod::connectGetter): (JSC::Bindings::QtRuntimeMetaMethod::disconnectGetter): (JSC::Bindings::QtRuntimeConnectionMethod::lengthGetter):
  • bridge/qt/qt_runtime.h:
01:01 PythonGuidelines created by cjerdonek@webkit.org
Created Python page.
00:32 Changeset [55403] by oliver@apple.com

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

Reviewed by NOBODY (Build fix).

Export function on windows.

00:31 WikiStart edited by cjerdonek@webkit.org
Added link to Python Guidelines page. (diff)
00:30 Changeset [55402] by philn@webkit.org

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

Reviewed by Gustavo Noronha.

[GTK] Move gstreamer-related source files to platform/graphics/gstreamer
https://bugs.webkit.org/show_bug.cgi?id=35518

Moved GStreamer-related files to platform/graphics/gstreamer so
other ports could benefit from this media player implementation.

  • GNUmakefile.am:
  • platform/graphics/gstreamer/DataSourceGStreamer.cpp: Renamed from WebCore/platform/graphics/gtk/DataSourceGStreamer.cpp. (_do_init): (webkit_data_src_base_init): (webkit_data_src_class_init): (webkit_data_src_reset): (webkit_data_src_init): (webkit_data_src_finalize): (webkit_data_src_change_state): (webkit_data_src_uri_get_type): (webkit_data_src_uri_get_protocols): (webkit_data_src_uri_get_uri): (webkit_data_src_uri_set_uri): (webkit_data_src_uri_handler_init):
  • platform/graphics/gstreamer/DataSourceGStreamer.h: Renamed from WebCore/platform/graphics/gtk/DataSourceGStreamer.h.
  • platform/graphics/gstreamer/MediaPlayerPrivateGStreamer.cpp: Renamed from WebCore/platform/graphics/gtk/MediaPlayerPrivateGStreamer.cpp. (WebCore::greatestCommonDivisor): (WebCore::mediaPlayerPrivateMessageCallback): (WebCore::mediaPlayerPrivateSourceChangedCallback): (WebCore::mediaPlayerPrivateVolumeChangedCallback): (WebCore::notifyVolumeIdleCallback): (WebCore::mediaPlayerPrivateMuteChangedCallback): (WebCore::notifyMuteIdleCallback): (WebCore::bufferingTimeoutCallback): (WebCore::playbackPosition): (WebCore::mediaPlayerPrivateRepaintCallback): (WebCore::MediaPlayerPrivate::create): (WebCore::MediaPlayerPrivate::registerMediaEngine): (WebCore::doGstInit): (WebCore::MediaPlayerPrivate::isAvailable): (WebCore::MediaPlayerPrivate::MediaPlayerPrivate): (WebCore::MediaPlayerPrivate::~MediaPlayerPrivate): (WebCore::MediaPlayerPrivate::load): (WebCore::MediaPlayerPrivate::changePipelineState): (WebCore::MediaPlayerPrivate::play): (WebCore::MediaPlayerPrivate::pause): (WebCore::MediaPlayerPrivate::duration): (WebCore::MediaPlayerPrivate::currentTime): (WebCore::MediaPlayerPrivate::seek): (WebCore::MediaPlayerPrivate::startEndPointTimerIfNeeded): (WebCore::MediaPlayerPrivate::cancelSeek): (WebCore::MediaPlayerPrivate::endPointTimerFired): (WebCore::MediaPlayerPrivate::paused): (WebCore::MediaPlayerPrivate::seeking): (WebCore::MediaPlayerPrivate::naturalSize): (WebCore::MediaPlayerPrivate::hasVideo): (WebCore::MediaPlayerPrivate::hasAudio): (WebCore::MediaPlayerPrivate::setVolume): (WebCore::MediaPlayerPrivate::volumeChangedCallback): (WebCore::MediaPlayerPrivate::volumeChanged): (WebCore::MediaPlayerPrivate::setRate): (WebCore::MediaPlayerPrivate::networkState): (WebCore::MediaPlayerPrivate::readyState): (WebCore::MediaPlayerPrivate::buffered): (WebCore::MediaPlayerPrivate::processBufferingStats): (WebCore::MediaPlayerPrivate::queryBufferingStats): (WebCore::MediaPlayerPrivate::maxTimeSeekable): (WebCore::MediaPlayerPrivate::maxTimeLoaded): (WebCore::MediaPlayerPrivate::bytesLoaded): (WebCore::MediaPlayerPrivate::totalBytes): (WebCore::MediaPlayerPrivate::cancelLoad): (WebCore::MediaPlayerPrivate::updateStates): (WebCore::MediaPlayerPrivate::mediaLocationChanged): (WebCore::MediaPlayerPrivate::loadNextLocation): (WebCore::MediaPlayerPrivate::loadStateChanged): (WebCore::MediaPlayerPrivate::sizeChanged): (WebCore::MediaPlayerPrivate::timeChanged): (WebCore::MediaPlayerPrivate::didEnd): (WebCore::MediaPlayerPrivate::durationChanged): (WebCore::MediaPlayerPrivate::supportsMuting): (WebCore::MediaPlayerPrivate::setMuted): (WebCore::MediaPlayerPrivate::muteChangedCallback): (WebCore::MediaPlayerPrivate::muteChanged): (WebCore::MediaPlayerPrivate::loadingFailed): (WebCore::MediaPlayerPrivate::setSize): (WebCore::MediaPlayerPrivate::setVisible): (WebCore::MediaPlayerPrivate::repaint): (WebCore::MediaPlayerPrivate::paint): (WebCore::mimeTypeCache): (WebCore::MediaPlayerPrivate::getSupportedTypes): (WebCore::MediaPlayerPrivate::supportsType): (WebCore::MediaPlayerPrivate::hasSingleSecurityOrigin): (WebCore::MediaPlayerPrivate::supportsFullscreen): (WebCore::MediaPlayerPrivate::setAutobuffer): (WebCore::MediaPlayerPrivate::createGSTPlayBin):
  • platform/graphics/gstreamer/MediaPlayerPrivateGStreamer.h: Renamed from WebCore/platform/graphics/gtk/MediaPlayerPrivateGStreamer.h. (WebCore::MediaPlayerPrivate::pipelineReset):
  • platform/graphics/gstreamer/VideoSinkGStreamer.cpp: Renamed from WebCore/platform/graphics/gtk/VideoSinkGStreamer.cpp. (webkit_video_sink_base_init): (webkit_video_sink_init): (webkit_video_sink_timeout_func): (webkit_video_sink_render): (webkit_video_sink_dispose): (unlock_buffer_mutex): (webkit_video_sink_unlock): (webkit_video_sink_unlock_stop): (webkit_video_sink_stop): (webkit_video_sink_start): (marshal_VOIDMINIOBJECT): (webkit_video_sink_class_init): (webkit_video_sink_new):
  • platform/graphics/gstreamer/VideoSinkGStreamer.h: Renamed from WebCore/platform/graphics/gtk/VideoSinkGStreamer.h.
  • platform/graphics/gstreamer/WebKitWebSourceGStreamer.cpp: Renamed from WebCore/platform/graphics/gtk/WebKitWebSourceGStreamer.cpp. (doInit): (webkit_web_src_base_init): (webkit_web_src_class_init): (webkit_web_src_init): (webKitWebSrcFinalize): (webKitWebSrcSetProperty): (webKitWebSrcGetProperty): (webKitWebSrcStop): (webKitWebSrcStart): (webKitWebSrcChangeState): (webKitWebSrcUriGetType): (webKitWebSrcGetProtocols): (webKitWebSrcGetUri): (webKitWebSrcSetUri): (webKitWebSrcUriHandlerInit): (webKitWebSrcNeedDataMainCb): (webKitWebSrcNeedDataCb): (webKitWebSrcEnoughDataMainCb): (webKitWebSrcEnoughDataCb): (webKitWebSrcSeekMainCb): (webKitWebSrcSeekDataCb): (webKitWebSrcSetFrame): (StreamingClient::StreamingClient): (StreamingClient::~StreamingClient): (StreamingClient::willSendRequest): (StreamingClient::didReceiveResponse): (StreamingClient::didReceiveData): (StreamingClient::didFinishLoading): (StreamingClient::didFail): (StreamingClient::wasBlocked): (StreamingClient::cannotShowURL):
  • platform/graphics/gstreamer/WebKitWebSourceGStreamer.h: Renamed from WebCore/platform/graphics/gtk/WebKitWebSourceGStreamer.h.
00:20 Changeset [55401] by oliver@apple.com

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

Reviewed by Maciej Stachowiak.

Refactor named getter function signature to be in line with indexing getter signature
https://bugs.webkit.org/show_bug.cgi?id=35563

This removes the PropertySlot argument from getter functions, and makes them directly
pass the slot base. This makes the semantics for the functions match that of the
indexing getters.

On the down side, this means that we can no longer simply use a proxy function for
JS getters, so we now add another marker value to indicate that a getter is present
and branch accordingly.

Against all rationality sunspider reports this as a perf win, but i suspect it's just noise.

  • API/JSCallbackObject.h:
  • API/JSCallbackObjectFunctions.h: (JSC::::staticValueGetter): (JSC::::staticFunctionGetter): (JSC::::callbackGetter):
  • JavaScriptCore.exp:
  • runtime/JSActivation.cpp: (JSC::JSActivation::argumentsGetter):
  • runtime/JSActivation.h:
  • runtime/JSFunction.cpp: (JSC::JSFunction::argumentsGetter): (JSC::JSFunction::callerGetter): (JSC::JSFunction::lengthGetter):
  • runtime/JSFunction.h:
  • runtime/NumberConstructor.cpp: (JSC::numberConstructorNaNValue): (JSC::numberConstructorNegInfinity): (JSC::numberConstructorPosInfinity): (JSC::numberConstructorMaxValue): (JSC::numberConstructorMinValue):
  • runtime/PropertySlot.cpp: (JSC::PropertySlot::functionGetter):
  • runtime/PropertySlot.h: (JSC::PropertySlot::getValue): (JSC::PropertySlot::setGetterSlot): (JSC::PropertySlot::setCacheableGetterSlot):
  • runtime/RegExpConstructor.cpp: (JSC::regExpConstructorDollar1): (JSC::regExpConstructorDollar2): (JSC::regExpConstructorDollar3): (JSC::regExpConstructorDollar4): (JSC::regExpConstructorDollar5): (JSC::regExpConstructorDollar6): (JSC::regExpConstructorDollar7): (JSC::regExpConstructorDollar8): (JSC::regExpConstructorDollar9): (JSC::regExpConstructorInput): (JSC::regExpConstructorMultiline): (JSC::regExpConstructorLastMatch): (JSC::regExpConstructorLastParen): (JSC::regExpConstructorLeftContext): (JSC::regExpConstructorRightContext):
  • runtime/RegExpObject.cpp: (JSC::regExpObjectGlobal): (JSC::regExpObjectIgnoreCase): (JSC::regExpObjectMultiline): (JSC::regExpObjectSource): (JSC::regExpObjectLastIndex):

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

Reviewed by Maciej Stachowiak.

Refactor named getter function signature to be in line with indexing getter signature
https://bugs.webkit.org/show_bug.cgi?id=35563

Fix method signature and update code as appropriate

  • UserObjectImp.cpp: (UserObjectImp::userObjectGetter):
  • UserObjectImp.h:

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

Reviewed by Maciej Stachowiak.

Refactor named getter function signature to be in line with indexing getter signature
https://bugs.webkit.org/show_bug.cgi?id=35563

Fix up WebCore to use the new named getter function signature, update the
codegenerator to the new calling convention, and fix the custom bindings.

  • bindings/js/JSCSSStyleDeclarationCustom.cpp: (WebCore::JSCSSStyleDeclaration::nameGetter):
  • bindings/js/JSDOMBinding.cpp: (WebCore::objectToStringFunctionGetter):
  • bindings/js/JSDOMBinding.h:
  • bindings/js/JSDOMWindowCustom.cpp: (WebCore::nonCachingStaticFunctionGetter): (WebCore::childFrameGetter): (WebCore::namedItemGetter):
  • bindings/js/JSDataGridColumnListCustom.cpp: (WebCore::JSDataGridColumnList::nameGetter):
  • bindings/js/JSHTMLAllCollectionCustom.cpp: (WebCore::JSHTMLAllCollection::nameGetter):
  • bindings/js/JSHTMLCollectionCustom.cpp: (WebCore::JSHTMLCollection::nameGetter):
  • bindings/js/JSHTMLDocumentCustom.cpp: (WebCore::JSHTMLDocument::nameGetter):
  • bindings/js/JSHTMLFormElementCustom.cpp: (WebCore::JSHTMLFormElement::nameGetter):
  • bindings/js/JSHTMLFrameSetElementCustom.cpp: (WebCore::JSHTMLFrameSetElement::nameGetter):
  • bindings/js/JSHistoryCustom.cpp: (WebCore::nonCachingStaticBackFunctionGetter): (WebCore::nonCachingStaticForwardFunctionGetter): (WebCore::nonCachingStaticGoFunctionGetter):
  • bindings/js/JSLocationCustom.cpp: (WebCore::nonCachingStaticReplaceFunctionGetter): (WebCore::nonCachingStaticReloadFunctionGetter): (WebCore::nonCachingStaticAssignFunctionGetter):
  • bindings/js/JSMimeTypeArrayCustom.cpp: (WebCore::JSMimeTypeArray::nameGetter):
  • bindings/js/JSNamedNodeMapCustom.cpp: (WebCore::JSNamedNodeMap::nameGetter):
  • bindings/js/JSNodeListCustom.cpp: (WebCore::JSNodeList::nameGetter):
  • bindings/js/JSPluginArrayCustom.cpp: (WebCore::JSPluginArray::nameGetter):
  • bindings/js/JSPluginCustom.cpp: (WebCore::JSPlugin::nameGetter):
  • bindings/js/JSPluginElementFunctions.cpp: (WebCore::runtimeObjectPropertyGetter):
  • bindings/js/JSPluginElementFunctions.h:
  • bindings/js/JSStorageCustom.cpp: (WebCore::JSStorage::nameGetter):
  • bindings/js/JSStyleSheetListCustom.cpp: (WebCore::JSStyleSheetList::nameGetter):
  • bindings/scripts/CodeGeneratorJS.pm:
  • bridge/runtime_array.cpp: (JSC::RuntimeArray::lengthGetter):
  • bridge/runtime_array.h:
  • bridge/runtime_method.cpp: (JSC::RuntimeMethod::lengthGetter):
  • bridge/runtime_method.h:
  • bridge/runtime_object.cpp: (JSC::Bindings::RuntimeObject::fallbackObjectGetter): (JSC::Bindings::RuntimeObject::fieldGetter): (JSC::Bindings::RuntimeObject::methodGetter):
  • bridge/runtime_object.h:
00:10 Changeset [55400] by hausmann@webkit.org

Symbian build fix.

[Qt] Updated the def file with one new export, to fix
QtLauncher linkage.

  • symbian/eabi/QtWebKitu.def:

03/01/10:

20:14 Changeset [55399] by jhoneycutt@apple.com

Remove Windows line endings from some files.

Rubber-stamped by Alice Liu.

  • Interfaces/IWebEmbeddedView.idl:
  • WebCoreSupport/EmbeddedWidget.cpp:

(EmbeddedWidget::create):
(EmbeddedWidget::~EmbeddedWidget):
(EmbeddedWidget::createWindow):
(EmbeddedWidget::invalidateRect):
(EmbeddedWidget::setFrameRect):
(EmbeddedWidget::frameRectsChanged):
(EmbeddedWidget::setFocus):
(EmbeddedWidget::show):
(EmbeddedWidget::hide):
(EmbeddedWidget::windowClipRect):
(EmbeddedWidget::setParent):
(EmbeddedWidget::attachToWindow):
(EmbeddedWidget::detachFromWindow):
(EmbeddedWidget::didReceiveResponse):
(EmbeddedWidget::didReceiveData):
(EmbeddedWidget::didFinishLoading):
(EmbeddedWidget::didFail):

  • WebCoreSupport/EmbeddedWidget.h:

(EmbeddedWidget::EmbeddedWidget):

20:14 Changeset [55398] by jhoneycutt@apple.com

Some WebKit DOMNode API is unimplemented.
https://bugs.webkit.org/show_bug.cgi?id=35554

Reviewed by Alice Liu.

  • DOMCoreClasses.cpp:

(DOMNode::nextSibling):
Create a DOMNode to wrap m_node's next sibling, and assign it to the
out param 'result'.
(DOMNode::insertBefore):
Query for the DOMNode for newChild, and return early if we fail. Query
refChild for DOMNode. Call insertBefore(), passing the newChild's
WebCore node and refChild's WebCore node (if refChild is non-null). If
we successfully insert the child, fill the result out param with
newChild, ref it, and return S_OK. Otherwise, return E_FAIL.
(DOMNode::removeChild):
Query oldChild for DOMNode. If we fail, return E_FAIL. Call
removeChild(), passing the node's WebCore node. If this fails, return
E_FAIL. Otherwise, fill the result out param with oldChild, ref it, and
return S_OK.

20:08 Changeset [55397] by oliver@apple.com

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

Reviewed by Gavin Barraclough.

PropertySlot::getValue(ExecState, unsigned) unnecessarily converts index to an Identifier
https://bugs.webkit.org/show_bug.cgi?id=35561

Fix this by defining a separate property getter function for index getters. This allows
us to pass an unsigned number without the conversion to an Identifier. We then update
setCustomIndex to take this new getter type.

  • runtime/PropertySlot.h: (JSC::PropertySlot::getValue): (JSC::PropertySlot::setCustom): (JSC::PropertySlot::setCustomIndex):

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

Reviewed by Gavin Barraclough.

PropertySlot::getValue(ExecState, unsigned) unnecessarily converts index to an Identifier
https://bugs.webkit.org/show_bug.cgi?id=35561

Update bindings generation and the few manual indexing getters we have to use
the new PropertySlot API.

  • bindings/js/JSDOMWindowCustom.cpp: (WebCore::indexGetter):
  • bindings/scripts/CodeGeneratorJS.pm:
  • bridge/runtime_array.cpp: (JSC::RuntimeArray::indexGetter):
  • bridge/runtime_array.h:
18:24 Changeset [55396] by ap@apple.com

Reviewed by Darin Adler.

https://bugs.webkit.org/show_bug.cgi?id=26520
Accessing a DOM node from a parent window in a child iframe moves the prototype to the child
iframe and resets it in the parent window

Adding a test for a bug that no longer occurs in ToT.

  • fast/dom/cross-frame-node-prototype-expected.txt: Added.
  • fast/dom/cross-frame-node-prototype.html: Added.
  • fast/dom/resources/cross-frame-node-prototype-iframe.html: Added.
17:58 Changeset [55395] by cfleizach@apple.com

Fixing broken DRT on Leopard/Tiger. Second try.

AX: changes to WAI-ARIA grid aren't perceived correctly by VoiceOver
https://bugs.webkit.org/show_bug.cgi?id=35514

  • DumpRenderTree/mac/AccessibilityUIElementMac.mm:
17:50 Changeset [55394] by cfleizach@apple.com

Fixing broken DRT on Leopard/Tiger.

AX: changes to WAI-ARIA grid aren't perceived correctly by VoiceOver
https://bugs.webkit.org/show_bug.cgi?id=35514

  • DumpRenderTree/mac/AccessibilityUIElementMac.mm:
17:40 Changeset [55393] by kov@webkit.org

2010-03-01 Gustavo Noronha Silva <gns@gnome.org>

Rebaseline the two editing tests we stopped skipping - we failed
to take the new caret position result into account while
unskipping these.

  • platform/gtk/editing/selection/5109817-expected.txt:
  • platform/gtk/editing/selection/5354455-2-expected.txt:
17:35 Changeset [55392] by kov@webkit.org

WebKitTools

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

Reviewed by Eric Seidel.

[GTK] plugins/setProperty.html fails on 64bit Release
https://bugs.webkit.org/show_bug.cgi?id=35425

Check invoke's return code before releasing the variant, since
there's a chance it won't be properly initialized, leading to
memory corruption, in some cases.

  • DumpRenderTree/TestNetscapePlugIn.subproj/PluginObject.cpp: (testCallback): (testEnumerate): (testDocumentOpen): (testWindowOpen): (handleCallback):

LayoutTests

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

Reviewed by Eric Seidel.

[GTK] plugins/setProperty.html fails on 64bit Release
https://bugs.webkit.org/show_bug.cgi?id=35425

Unskip the tests that failed as a consequence of the memory
corruption.

  • platform/gtk/Skipped:
17:29 Changeset [55391] by dpranke@chromium.org

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

Reviewed by nobody. Build bustage :(

Fix stupid typo that I committed even after David Levin pointed
it out to me :(

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

  • Scripts/webkitpy/layout_tests/port/mac.py:
17:19 Changeset [55390] by cfleizach@apple.com

AX: changes to WAI-ARIA grid aren't perceived correctly by VoiceOver
https://bugs.webkit.org/show_bug.cgi?id=35514

Reviewed by Darin Adler.

WebCore:

When a table's DOM is changed and an AX Table is not asked first for its children,
it would return wrong information. A table needs to make sure children are up to date in
all methods that can be called from the outside.

Test: platform/mac/accessibility/stale-table-rows.html

  • accessibility/AccessibilityARIAGrid.cpp:

(WebCore::AccessibilityARIAGrid::cellForColumnAndRow):

  • accessibility/AccessibilityObject.cpp:

(WebCore::AccessibilityObject::clearChildren):

  • accessibility/AccessibilityRenderObject.cpp:

(WebCore::AccessibilityRenderObject::clearChildren):
(WebCore::AccessibilityRenderObject::updateChildrenIfNecessary):
(WebCore::AccessibilityRenderObject::children):

  • accessibility/AccessibilityRenderObject.h:

(WebCore::AccessibilityRenderObject::needsToUpdateChildren):
(WebCore::AccessibilityRenderObject::setNeedsToUpdateChildren):

  • accessibility/AccessibilityTable.cpp:

(WebCore::AccessibilityTable::clearChildren):
(WebCore::AccessibilityTable::columns):
(WebCore::AccessibilityTable::rows):
(WebCore::AccessibilityTable::rowHeaders):
(WebCore::AccessibilityTable::columnHeaders):
(WebCore::AccessibilityTable::cells):
(WebCore::AccessibilityTable::columnCount):
(WebCore::AccessibilityTable::rowCount):
(WebCore::AccessibilityTable::cellForColumnAndRow):

WebKitTools:

Add rowCount, columnCount for tables.

  • DumpRenderTree/AccessibilityUIElement.cpp:

(rowCountCallback):
(columnCountCallback):
(AccessibilityUIElement::getJSClass):

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

(AccessibilityUIElement::rowCount):
(AccessibilityUIElement::columnCount):

  • DumpRenderTree/mac/AccessibilityUIElementMac.mm:

(AccessibilityUIElement::rowCount):
(AccessibilityUIElement::columnCount):

  • DumpRenderTree/win/AccessibilityUIElementWin.cpp:

(AccessibilityUIElement::rowCount):
(AccessibilityUIElement::columnCount):

LayoutTests:

  • platform/mac/accessibility/stale-table-rows-expected.txt: Added.
  • platform/mac/accessibility/stale-table-rows.html: Added.
17:12 Changeset [55389] by eric@webkit.org

2010-03-01 José Millán Soto <jmillan@igalia.com>

Reviewed by Gustavo Noronha Silva.

[GTK] Right click does not activate text entry
https://bugs.webkit.org/show_bug.cgi?id=29177

Stop skipping some tests which where failing because of right click
not being correct handled.

  • platform/gtk/Skipped:

2010-03-01 José Millán Soto <jmillan@igalia.com>

Reviewed by Gustavo Noronha Silva.

[GTK] Right click does not activate text entry
https://bugs.webkit.org/show_bug.cgi?id=29177

Makes the frame handle the mouse click event before sending the
context menu event.

  • webkit/webkitwebview.cpp: (PopupMenuPositionFunc): Function created to make the popup menu appear in the correct position, especially when invoked from the keyboard. (webkit_web_view_forward_context_menu_event): Mouse click event is sent to frame before creating context menu, PopupMenuPositionFunc used to determine the position where the menu should appear. (webkit_web_view_popup_menu_handler): Improved focused node position detection. Event button set to right button.
17:07 Changeset [55388] by dpranke@chromium.org

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

Reviewed by Eric Seidel.

Work around a bug in Python's subprocess.Popen() that keeps us from
cleaning up DumpRenderTree / test_shell properly when we finish the
tests in new-run-webkit-tests.

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

  • Scripts/webkitpy/layout_tests/port/chromium.py:
  • Scripts/webkitpy/layout_tests/port/mac.py:
16:56 Changeset [55387] by eric@webkit.org

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

Reviewed by Adam Barth.

Convert the zoom mode (page vs. text-only) into a proper enum.
https://bugs.webkit.org/show_bug.cgi?id=35347

  • GNUmakefile.am:
  • WebCore.base.exp: Substituted symbols ZN7WebCore5Frame13setZoomFactorEfNS_8ZoomModeE and ZN7WebCore8Settings11setZoomModeENS_8ZoomModeE for ZN7WebCore5Frame13setZoomFactorEfb and ZN7WebCore8Settings16setZoomsTextOnlyEb, respectively.
  • WebCore.gypi:
  • WebCore.pro:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • page/Frame.cpp: (WebCore::Frame::zoomMode): (WebCore::Frame::shouldApplyTextZoom): (WebCore::Frame::shouldApplyPageZoom): (WebCore::Frame::setZoomFactor):
  • page/Frame.h:
  • page/Settings.cpp: (WebCore::Settings::Settings): (WebCore::Settings::setZoomMode):
  • page/Settings.h: (WebCore::Settings::zoomMode):
  • page/ZoomMode.h: Added. (WebCore::):
  • svg/SVGSVGElement.cpp: (WebCore::SVGSVGElement::setCurrentScale):

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

Reviewed by Adam Barth.

Adapt to the new ZoomMode enum.
https://bugs.webkit.org/show_bug.cgi?id=35347

  • src/WebViewImpl.cpp: (WebKit::WebViewImpl::setZoomLevel):

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

Reviewed by Adam Barth.

Adapt to the new ZoomMode enum.
https://bugs.webkit.org/show_bug.cgi?id=35347

  • webkit/webkitwebview.cpp: (DNDContentsRequest::webkit_web_view_apply_zoom_level):

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

Reviewed by Adam Barth.

Adapt to the new ZoomMode enum.
https://bugs.webkit.org/show_bug.cgi?id=35347

  • WebView/WebView.mm: (-[WebView _preferencesChangedNotification:]): (-[WebView _setZoomMultiplier:isTextOnly:]): (-[WebView _realZoomMultiplierIsTextOnly]):

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

Reviewed by Adam Barth.

Adapt to the new ZoomMode enum.
https://bugs.webkit.org/show_bug.cgi?id=35347

  • Api/qwebframe.cpp: (QWebFrame::setTextSizeMultiplier): (QWebFrame::setZoomFactor):
  • Api/qwebsettings.cpp: (QWebSettingsPrivate::apply):

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

Reviewed by Adam Barth.

Adapt to the new ZoomMode enum.
https://bugs.webkit.org/show_bug.cgi?id=35347

  • WebFrame.cpp: (WebFrame::setTextSizeMultiplier):
  • WebView.cpp: (WebView::setZoomMultiplier): (WebView::zoomMultiplier): (WebView::canMakeTextLarger): (WebView::makeTextLarger): (WebView::canMakeTextSmaller): (WebView::makeTextSmaller): (WebView::notifyPreferencesChanged):

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

Reviewed by Adam Barth.

Adapt to the new ZoomMode enum.
https://bugs.webkit.org/show_bug.cgi?id=35347

  • WebFrame.cpp: (wxWebFrame::IncreaseTextSize): (wxWebFrame::DecreaseTextSize): (wxWebFrame::ResetTextSize):
16:22 Changeset [55386] by eric@webkit.org

2010-03-01 Alex Milowski <alex@milowski.com>

Reviewed by Kenneth Rohde Christiansen.

Added test for basic msubsup support.

  • mathml/presentation/subsup.xhtml: Added.
  • platform/mac/mathml/presentation/subsup-expected.checksum: Added.
  • platform/mac/mathml/presentation/subsup-expected.png: Added.
  • platform/mac/mathml/presentation/subsup-expected.txt: Added.

2010-03-01 Alex Milowski <alex@milowski.com>

Reviewed by Kenneth Rohde Christiansen.

Added support for the msubsup element that also handles the msup and msub elements.

Test: mathml/presentation/subsup.xhtml

  • WebCore.xcodeproj/project.pbxproj:
  • mathml/MathMLInlineContainerElement.cpp: (WebCore::MathMLInlineContainerElement::createRenderer):
  • mathml/RenderMathMLSubSup.cpp: Added. (WebCore::RenderMathMLSubSup::RenderMathMLSubSup): (WebCore::RenderMathMLSubSup::addChild): (WebCore::RenderMathMLSubSup::stretchToHeight): (WebCore::RenderMathMLSubSup::nonOperatorHeight): (WebCore::RenderMathMLSubSup::layout): (WebCore::RenderMathMLSubSup::baselinePosition):
  • mathml/RenderMathMLSubSup.h: Added. (WebCore::RenderMathMLSubSup::hasBase): (WebCore::RenderMathMLSubSup::):
16:18 Changeset [55385] by jhoneycutt@apple.com

<rdar://problem/7703368> IWebUIDelegatePrivate::embeddedViewWithArguments
is passed wrong arguments

Reviewed by Adam Roben.

  • Interfaces/IWebUIDelegatePrivate.idl:

Update copyright strings. Added a new key for the plug-in source URL.

  • Interfaces/WebKit.idl:

Update copyright strings.

  • WebCoreSupport/WebFrameLoaderClient.cpp:

(WebFrameLoaderClient::createPlugin):
Pass the URL of the plug-in as the source URL. Pass the document's
base URI for the base URL.

16:05 Changeset [55384] by eric@webkit.org

2010-03-01 Arno Renevier <arno@renevier.net>

Reviewed by Xan Lopez.

webkit-build could pass unknown arguments to autogen.sh
https://bugs.webkit.org/show_bug.cgi?id=35454

  • Scripts/build-webkit:
15:49 Changeset [55383] by eric@webkit.org

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

Reviewed by Timothy Hatcher.

Web Inspector: use dejavu sans mono 11px on linux.

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

  • inspector/front-end/inspector.css:
15:17 Changeset [55382] by dpranke@chromium.org

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

Rubber-stamped by Dimitri Glazkov.

Fix breakage from r55372.

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

  • Scripts/webkitpy/layout_tests/port/chromium.py:
14:51 Changeset [55381] by eric@webkit.org

2010-03-01 Thatcher Ulrich <tulrich@google.com>

Reviewed by Darin Fisher.

Fix chromium iframe shims. Add another test case to the
iframes-shims test. After r53637, the plugin widget doesn't get
moved every paint. This used to hide the bug that if an iframe
gets added, the plugin's cutout rectangles don't get updated until
a layout happens.
https://bugs.webkit.org/show_bug.cgi?id=35184

  • plugins/iframe-shims.html: test case that adds an iframe shim dynamically

2010-03-01 Thatcher Ulrich <tulrich@google.com>

Reviewed by Darin Fisher.

Fix chromium iframe shims. Add another test case to the
iframes-shims test. After r53637, the plugin widget doesn't get
moved every paint. This used to hide the bug that if an iframe
gets added, the plugin's cutout rectangles don't get updated until
a layout happens.
https://bugs.webkit.org/show_bug.cgi?id=35184

  • platform/Widget.h: (WebCore::Widget::widgetPositionsUpdated): new virtual method

widgetPositionsUpdated() to notify widgets when other widgets on
the page have been repositioned.

  • rendering/RenderView.cpp: (WebCore::RenderView::updateWidgetPositions): call widgetPositionsUpdated() on the widgets
  • rendering/RenderWidget.cpp: (WebCore::RenderWidget::widgetPositionsUpdated): call widgetPositionsUpdated() on the widget
  • rendering/RenderWidget.h:

2010-03-01 Thatcher Ulrich <tulrich@google.com>

Reviewed by Darin Fisher.

Fix chromium iframe shims. Add another test case to the
iframes-shims test. After r53637, the plugin widget doesn't get
moved every paint. This used to hide the bug that if an iframe
gets added, the plugin's cutout rectangles don't get updated until
a layout happens.
https://bugs.webkit.org/show_bug.cgi?id=35184

  • src/WebPluginContainerImpl.cpp: (WebKit::WebPluginContainerImpl::widgetPositionsUpdated): do reportGeometry() to ensure that

the plugin is aware of the positions of cutouts on the page (for iframe shim behavior).

  • src/WebPluginContainerImpl.h:
14:27 Changeset [55380] by abarth@webkit.org

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

Reviewed by Eric Seidel.

EWS can hang for five hours when compile output is too big
https://bugs.webkit.org/show_bug.cgi?id=35545

  • Scripts/webkitpy/commands/queues.py: Limit uploads to 1MB instead of 5MB. AppEngine seems to not like 5MB uploads. I'm not sure what the limit is. Let's try 1MB for a while and see how it goes.
  • Scripts/webkitpy/networktransaction.py: Tune the default parameters to our exponential backoff. I'm not sure why I picked five hours as the retry limit. That seems way too large.
14:14 Changeset [55379] by barraclough@apple.com

Bug 35537 - put_by_id does will incorrectly cache writes where a specific value exists,

where at the point of caching the same value is being written.

Reviewed by Oliver Hunt.

JavaScriptCore:

When performing a put_by_id that is replacing a property already present on the object,
there are three interesting cases regarding the state of the specific value:

(1) No specific value set - nothing to do, leave the structure in it's current state,

can cache.

(2) A specific value was set, the new put is not of a specified value (i.e. function),

or is of a different specific value - in these cases we need to perform a despecifying
transition to clear the specific value in the structure, but having done so this is a
normal property so as such we can again cache normally.

(3) A specific value was set, and we are overwriting with the same value - in these cases

leave the structure unchanged, but since a specific value is set we cannot cache this
put (we would need the JIT to dynamically check the value being written matched).

Unfortunately, the current behaviour does not match this. the checks for a specific value
being present & the value matching are combined in such a way that in case (2), above we
will unnecessarily prevent the transition being cached, but in case (3) we will incorrectly
fail to prevent caching.

The bug exposes itself if multiple puts of the same specific value are performed to a
property, and erroneously the put is allowed to be cached by the JIT. Method checks may be
generated caching calls of this structure. Subsequent puts performed from JIT code may
write different values without triggering a despecify transition, and as such cached method
checks will continue to pass, despite the value having changed.

  • runtime/JSObject.h:

(JSC::JSObject::putDirectInternal):

LayoutTests:

Add test case.

  • fast/js/method-check-expected.txt:
  • fast/js/script-tests/method-check.js:

(addOne):
(addOneHundred):
(totalizer.makeCall):

14:07 Changeset [55378] by eric@webkit.org

2010-03-01 Kalle Vahlman <zuh@iki.fi>

Reviewed by Gustavo Noronha Silva.

Queue a resize when either of the content dimensions change
https://bugs.webkit.org/show_bug.cgi?id=35489

The check for size changes only queued a resize if both of the content
dimensions change, leaving the widget size out-of-sync if eg. only the
width changes.

  • WebCoreSupport/ChromeClientGtk.cpp:
13:51 Changeset [55377] by eric@webkit.org

2010-03-01 Fridrich Strba <fridrich.strba@bluewin.ch>

Reviewed by Holger Freyther.

Dist some mathml related files

  • GNUmakefile.am:
13:20 Changeset [55376] by kbr@google.com

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

Reviewed by Oliver Hunt.

Integer overflow in WebGL arrays
https://bugs.webkit.org/show_bug.cgi?id=35241

Test: fast/canvas/webgl/webgl-array-invalid-ranges.html

  • bindings/js/JSWebGLArrayBufferConstructor.cpp: (WebCore::constructCanvasArrayBuffer):
  • bindings/js/JSWebGLArrayBufferConstructor.h: (WebCore::construct):
  • bindings/js/JSWebGLArrayHelper.h: (WebCore::setWebGLArrayFromArray):
  • bindings/js/JSWebGLByteArrayConstructor.cpp: (WebCore::constructCanvasByteArray):
  • bindings/js/JSWebGLFloatArrayConstructor.cpp: (WebCore::constructCanvasFloatArray):
  • bindings/js/JSWebGLIntArrayConstructor.cpp: (WebCore::constructCanvasIntArray):
  • bindings/js/JSWebGLShortArrayConstructor.cpp: (WebCore::constructCanvasShortArray):
  • bindings/js/JSWebGLUnsignedByteArrayConstructor.cpp: (WebCore::constructCanvasUnsignedByteArray):
  • bindings/js/JSWebGLUnsignedIntArrayConstructor.cpp: (WebCore::constructCanvasUnsignedIntArray):
  • bindings/js/JSWebGLUnsignedShortArrayConstructor.cpp: (WebCore::constructCanvasUnsignedShortArray):
  • bindings/v8/V8Binding.cpp: (WebCore::toUInt32):
  • bindings/v8/V8Binding.h: (WebCore::toUInt32):
  • bindings/v8/custom/V8WebGLArrayBufferCustom.cpp: (WebCore::V8WebGLArrayBuffer::constructorCallback):
  • bindings/v8/custom/V8WebGLArrayCustom.h: (WebCore::constructWebGLArray): (WebCore::getWebGLArrayElement): (WebCore::setWebGLArrayFromArray): (WebCore::setWebGLArray):
  • bindings/v8/custom/V8WebGLByteArrayCustom.cpp: (WebCore::V8WebGLByteArray::constructorCallback):
  • bindings/v8/custom/V8WebGLFloatArrayCustom.cpp: (WebCore::V8WebGLFloatArray::constructorCallback):
  • bindings/v8/custom/V8WebGLIntArrayCustom.cpp: (WebCore::V8WebGLIntArray::constructorCallback):
  • bindings/v8/custom/V8WebGLShortArrayCustom.cpp: (WebCore::V8WebGLShortArray::constructorCallback):
  • bindings/v8/custom/V8WebGLUnsignedByteArrayCustom.cpp: (WebCore::V8WebGLUnsignedByteArray::constructorCallback):
  • bindings/v8/custom/V8WebGLUnsignedIntArrayCustom.cpp: (WebCore::V8WebGLUnsignedIntArray::constructorCallback):
  • bindings/v8/custom/V8WebGLUnsignedShortArrayCustom.cpp: (WebCore::V8WebGLUnsignedShortArray::constructorCallback):
  • html/canvas/WebGLArray.cpp: (WebCore::WebGLArray::setImpl):
  • html/canvas/WebGLArray.h: (WebCore::WebGLArray::verifySubRange): (WebCore::WebGLArray::clampOffsetAndNumElements):
  • html/canvas/WebGLArrayBuffer.cpp: (WebCore::WebGLArrayBuffer::create): (WebCore::WebGLArrayBuffer::WebGLArrayBuffer): (WebCore::WebGLArrayBuffer::tryAllocate):
  • html/canvas/WebGLArrayBuffer.h:
  • html/canvas/WebGLByteArray.cpp: (WebCore::WebGLByteArray::create): (WebCore::WebGLByteArray::WebGLByteArray): (WebCore::WebGLByteArray::slice):
  • html/canvas/WebGLByteArray.h:
  • html/canvas/WebGLFloatArray.cpp: (WebCore::WebGLFloatArray::create): (WebCore::WebGLFloatArray::WebGLFloatArray): (WebCore::WebGLFloatArray::slice):
  • html/canvas/WebGLFloatArray.h:
  • html/canvas/WebGLIntArray.cpp: (WebCore::WebGLIntArray::create): (WebCore::WebGLIntArray::WebGLIntArray): (WebCore::WebGLIntArray::slice):
  • html/canvas/WebGLIntArray.h:
  • html/canvas/WebGLShortArray.cpp: (WebCore::WebGLShortArray::create): (WebCore::WebGLShortArray::WebGLShortArray): (WebCore::WebGLShortArray::slice):
  • html/canvas/WebGLShortArray.h:
  • html/canvas/WebGLUnsignedByteArray.cpp: (WebCore::WebGLUnsignedByteArray::create): (WebCore::WebGLUnsignedByteArray::WebGLUnsignedByteArray): (WebCore::WebGLUnsignedByteArray::slice):
  • html/canvas/WebGLUnsignedByteArray.h:
  • html/canvas/WebGLUnsignedIntArray.cpp: (WebCore::WebGLUnsignedIntArray::create): (WebCore::WebGLUnsignedIntArray::WebGLUnsignedIntArray): (WebCore::WebGLUnsignedIntArray::slice):
  • html/canvas/WebGLUnsignedIntArray.h:
  • html/canvas/WebGLUnsignedShortArray.cpp: (WebCore::WebGLUnsignedShortArray::create): (WebCore::WebGLUnsignedShortArray::WebGLUnsignedShortArray): (WebCore::WebGLUnsignedShortArray::slice):
  • html/canvas/WebGLUnsignedShortArray.h:

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

Reviewed by Oliver Hunt.

Integer overflow in WebGL arrays
https://bugs.webkit.org/show_bug.cgi?id=35241

  • fast/canvas/webgl/array-unit-tests-expected.txt:
  • fast/canvas/webgl/array-unit-tests.html:
  • fast/canvas/webgl/webgl-array-invalid-ranges-expected.txt: Added.
  • fast/canvas/webgl/webgl-array-invalid-ranges.html: Added.
13:10 Changeset [55375] by beidson@apple.com

REGRESSION: Telling a WebView to go to its current WebHistoryItem is broken.
<rdar://problem/7699371> and https://bugs.webkit.org/show_bug.cgi?id=35532

Reviewed by Sam Weinig.

WebCore:

Test: fast/loader/api-test-go-to-current-back-forward-item.html

  • loader/FrameLoader.cpp:

(WebCore::FrameLoader::loadItem): If the current item is the same as the target item, don't

consider this to be a same document navigation.

  • page/Page.cpp:

(WebCore::Page::goToItem): Hard code the "going to the same item as the current item" relationship

as a precondition for stopping all loaders, as that will be a new document load.

WebKitTools:

  • DumpRenderTree/LayoutTestController.cpp:

(apiTestGoToCurrentBackForwardItemCallback):
(LayoutTestController::staticFunctions):

  • DumpRenderTree/LayoutTestController.h:
  • DumpRenderTree/mac/LayoutTestControllerMac.mm:

(LayoutTestController::apiTestGoToCurrentBackForwardItem):

Stubs for now:

  • DumpRenderTree/gtk/LayoutTestControllerGtk.cpp:

(LayoutTestController::apiTestGoToCurrentBackForwardItem):

  • DumpRenderTree/win/LayoutTestControllerWin.cpp:

(LayoutTestController::apiTestGoToCurrentBackForwardItem):

  • DumpRenderTree/wx/LayoutTestControllerWx.cpp:

(LayoutTestController::apiTestGoToCurrentBackForwardItem):

LayoutTests:

  • fast/loader/api-test-go-to-current-back-forward-item-expected.txt: Added.
  • fast/loader/api-test-go-to-current-back-forward-item.html: Added.
  • platform/gtk/Skipped:
  • platform/qt/Skipped:
  • platform/win/Skipped:
13:04 Changeset [55374] by robert@webkit.org

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

Reviewed by nobody, build fix.

Fix --minimal build. Add ENABLE(DATABASE) as compile time condition
for building file GeolocationPositionCache.cpp.

  • page/GeolocationPositionCache.cpp:
12:41 Changeset [55373] by dpranke@chromium.org

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

Reviewed by David Levin.

General cleanup of error handling in new-run-webkit-tests.

Add Port.check_build() call that is separate from Port.check_sys_deps()
(and add a --nocheck-build flag to skip). This breaks a circular
dependency where you would start the layout test helper before
checking sys deps, but checking sys deps was the thing that told
you if your binaries where there.

Also, made Port.check_sys_deps(), start_helper() and stop_helper()
optional by providing default implementations in the base class
rather than requiring ports to implement the routines regardless
of whether or not they were needed.

Lastly, tweak a bunch of log messages to be cleaner, including
changing messages in thirdparty/autoinstall.py to be silent at
the default log level.

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

  • Scripts/webkitpy/layout_tests/layout_package/test_expectations.py:
  • Scripts/webkitpy/layout_tests/port/base.py:
  • Scripts/webkitpy/layout_tests/port/chromium.py:
  • Scripts/webkitpy/layout_tests/port/chromium_linux.py:
  • Scripts/webkitpy/layout_tests/port/chromium_mac.py:
  • Scripts/webkitpy/layout_tests/port/chromium_win.py:
  • Scripts/webkitpy/layout_tests/port/mac.py:
  • Scripts/webkitpy/layout_tests/port/passing.py:
  • Scripts/webkitpy/layout_tests/port/test.py:
  • Scripts/webkitpy/layout_tests/run_webkit_tests.py:
  • Scripts/webkitpy/thirdparty/autoinstall.py
11:36 Changeset [55372] by dpranke@chromium.org

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

Reviewed by David Levin.

new-chromium-webkit-tests --platform=mac-leopard diffs are backwards
https://bugs.webkit.org/show_bug.cgi?id=35265

Some parts of the code passed arguments as
"actual, expected" and some passed as "expected, actual".
As you might imagine, this lead to great confusion and wrongness.
Standardize on "expected, actual" as that's the order which is
passed to the underlying diff tool.

Based on a patch by Eric Siedel <eric@webkit.org>.

  • Scripts/webkitpy/layout_tests/port/base.py:
  • Scripts/webkitpy/layout_tests/port/chromium.py:
  • Scripts/webkitpy/layout_tests/port/test.py:
  • Scripts/webkitpy/layout_tests/test_types/image_diff.py
11:01 UsingGitWithWebKit edited by robert@webkit.org
(diff)
09:41 Changeset [55371] by jpetsovits@rim.com

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

Reviewed by Dirk Schulze.

[OpenVG] Implement support for paths
https://bugs.webkit.org/show_bug.cgi?id=34366

Adds an implementation of the Path class -
*almost* complete, but not quite because OpenVG
does not provide access to the points in a VGPath
unless one keeps track of all the points by
themselves, which we decided not to do.

Also hooked up to PainterOpenVG and GraphicsContext.

Further introduced is SharedResourceOpenVG, which is
intended as base class for paths and other OpenVG
resources (images, fonts) that WebKit creates as
long-lived objects. We are at a slight disadvantage
here as WebKit doesn't have the concept of resources
belonging to a specific (hardware graphics) context,
which is the reason why EGLDisplayOpenVG had to
provide a current display singleton; this class is
what actually requires that functionality.

Path::addArcTo() uses code by Yong Li <yoli@rim.com>.

  • platform/graphics/Path.cpp:
  • platform/graphics/Path.h:
  • platform/graphics/openvg/GraphicsContextOpenVG.cpp: (WebCore::GraphicsContext::fillPath): (WebCore::GraphicsContext::strokePath): (WebCore::GraphicsContext::drawPath): (WebCore::GraphicsContext::beginPath): (WebCore::GraphicsContext::addPath):
  • platform/graphics/openvg/PainterOpenVG.cpp: (WebCore::PainterOpenVG::PainterOpenVG): (WebCore::PainterOpenVG::~PainterOpenVG): (WebCore::PainterOpenVG::transformPath): (WebCore::PainterOpenVG::beginPath): (WebCore::PainterOpenVG::addPath): (WebCore::PainterOpenVG::currentPath): (WebCore::PainterOpenVG::drawPath):
  • platform/graphics/openvg/PainterOpenVG.h:
  • platform/graphics/openvg/PathOpenVG.cpp: Added. (WebCore::PlatformPathOpenVG::PlatformPathOpenVG): (WebCore::PlatformPathOpenVG::operator=): (WebCore::PlatformPathOpenVG::~PlatformPathOpenVG): (WebCore::PlatformPathOpenVG::clear): (WebCore::PlatformPathOpenVG::createPath): (WebCore::Path::Path): (WebCore::Path::~Path): (WebCore::Path::operator=): (WebCore::Path::contains): (WebCore::Path::strokeContains): (WebCore::Path::translate): (WebCore::Path::boundingRect): (WebCore::Path::strokeBoundingRect): (WebCore::Path::moveTo): (WebCore::Path::addLineTo): (WebCore::Path::addQuadCurveTo): (WebCore::Path::addBezierCurveTo): (WebCore::Path::addArcTo): (WebCore::Path::closeSubpath): (WebCore::Path::addArc): (WebCore::Path::addRect): (WebCore::Path::addEllipse): (WebCore::Path::clear): (WebCore::Path::isEmpty): (WebCore::Path::hasCurrentPoint): (WebCore::Path::debugString): (WebCore::Path::apply): (WebCore::Path::transform): (WebCore::Path::length): (WebCore::Path::pointAtLength): (WebCore::Path::normalAngleAtLength):
  • platform/graphics/openvg/PlatformPathOpenVG.h: Added. (WebCore::PlatformPathOpenVG::vgPath):
  • platform/graphics/openvg/SharedResourceOpenVG.cpp: Added. (WebCore::SharedResourceOpenVG::makeSharedContextCurrent): (WebCore::SharedResourceOpenVG::makeCompatibleContextCurrent):
  • platform/graphics/openvg/SharedResourceOpenVG.h: Added.
08:18 Changeset [55370] by pfeldman@chromium.org

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

Not reviewed, bring English.lproj/locallizedStrings.js back to binary (UTF) mode.

  • English.lproj/localizedStrings.js:
07:51 Changeset [55369] by jpetsovits@rim.com

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

Reviewed by Nikolas Zimmermann.

[OpenVG] Convert from TransformationMatrix to AffineTransform.
https://bugs.webkit.org/show_bug.cgi?id=35445

Dirk Schulze previously introduced AffineTransform as
replacement for most TransformationMatrix occurrences
in GraphicsContext & Co., but did not update the OpenVG
graphics backend as there's no publicly available way
to build it. This commit now takes care of that.

  • platform/graphics/openvg/GraphicsContextOpenVG.cpp: (WebCore::GraphicsContext::getCTM): (WebCore::GraphicsContext::roundToDevicePixels): (WebCore::GraphicsContext::origin): (WebCore::GraphicsContext::concatCTM):
  • platform/graphics/openvg/PainterOpenVG.cpp: (WebCore::isNonRotatedAffineTransformation): (WebCore::PlatformPainterState::PlatformPainterState): (WebCore::PlatformPainterState::applyState): (WebCore::PlatformPainterState::applyTransformation): (WebCore::PainterOpenVG::transformation): (WebCore::PainterOpenVG::concatTransformation): (WebCore::PainterOpenVG::setTransformation): (WebCore::PainterOpenVG::scale): (WebCore::PainterOpenVG::rotate): (WebCore::PainterOpenVG::translate): (WebCore::PainterOpenVG::intersectClipRect):
  • platform/graphics/openvg/PainterOpenVG.h:
  • platform/graphics/openvg/VGUtils.cpp: (WebCore::VGMatrix::VGMatrix): (WebCore::VGMatrix::operator AffineTransform): (WebCore::VGMatrix::operator TransformationMatrix): (WebCore::AffineTransform::operator VGMatrix):
  • platform/graphics/openvg/VGUtils.h:
  • platform/graphics/transforms/AffineTransform.h:
07:34 Changeset [55368] by vestbo@webkit.org

[Qt] Fix segfault when drawing NPAPI plugins on Mac

Reviewed by Simon Hausmann.

Don't try to get a contexctRef for a null-pixmap. If the pixmap
size is 0,0 the private pixmap data has not yet been initialized.

07:34 Changeset [55367] by vestbo@webkit.org

Fix the Qt build on Mac OS X/Cocoa 64-bit

Add missing implementation file to resolve JSC symbols.

Use the proper event/drawing-model guards instead of global 64-bit
guard for the NPAPI plugin implementation (view and package).

06:34 QtWebKitTableOfFeatures20 edited by henry.haverinen@nokia.com
(diff)
06:06 QtWebKitTableOfFeatures20 edited by henry.haverinen@nokia.com
(diff)
06:00 QtWebKitTableOfFeatures20 edited by henry.haverinen@nokia.com
(diff)
05:58 QtWebKitTableOfFeatures20 edited by henry.haverinen@nokia.com
(diff)
05:48 QtWebKitTableOfFeatures20 edited by henry.haverinen@nokia.com
(diff)
05:46 QtWebKitTableOfFeatures20 edited by henry.haverinen@nokia.com
(diff)
05:42 QtWebKitTableOfFeatures46 edited by henry.haverinen@nokia.com
(diff)
05:41 QtWebKitTableOfFeatures20 created by henry.haverinen@nokia.com
05:41 QtWebKitFeatures20 edited by henry.haverinen@nokia.com
(diff)
05:40 QtWebKitFeatures20 created by henry.haverinen@nokia.com
05:38 QtWebKit edited by henry.haverinen@nokia.com
(diff)
04:54 Changeset [55366] by kenneth@webkit.org

Fix ChangeLog entries not showing the reviewer due to the ChangeLogs
having been made manually without the (OOPS!!!) part.

Rubberstamped by Gustavo Noronha

02:17 Changeset [55365] by pfeldman@chromium.org

2010-02-27 Ilya Tikhonovsky <loislo@chromium.org>

Reviewed by Pavel Feldman.

Web Inspector: callerScriptName and callerScriptLine should be obfuscated for timeline tests.
https://bugs.webkit.org/show_bug.cgi?id=35478

  • ../../LayoutTests/inspector/timeline-test.js:
02:12 Changeset [55364] by pfeldman@chromium.org

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

Reviewed by Timothy Hatcher.

Web Inspector: disable breakpoint upon Shift-click.

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

  • inspector/front-end/ConsoleView.js: (WebInspector.ConsoleView.prototype._format):
  • inspector/front-end/SourceFrame.js: (WebInspector.SourceFrame.prototype._mouseDown): (WebInspector.SourceFrame.prototype._mouseMove):
  • inspector/front-end/textViewer.css: Chromium: remove DevTools part that has been upstreamed.
  • src/js/DevTools.js:
02:07 Changeset [55363] by cjerdonek@webkit.org

Unreviewed.

Rolling out: http://trac.webkit.org/changeset/55348

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

Rolling out since the changes to autoinstall do not work
with Python 2.4. In particular, ZipFile.extractall() was
added in Python 2.6.

  • Scripts/webkitpy/init.py:
  • Scripts/webkitpy/bugzilla.py:
  • Scripts/webkitpy/init/init.py: Removed.
  • Scripts/webkitpy/init/autoinstall.py: Removed.
  • Scripts/webkitpy/networktransaction.py:
  • Scripts/webkitpy/networktransaction_unittest.py:
  • Scripts/webkitpy/statusserver.py:
  • Scripts/webkitpy/thirdparty/autoinstall.py: Added.
01:44 Changeset [55362] by cjerdonek@webkit.org

Unreviewed.

Fixing up ChangeLog after: http://trac.webkit.org/changeset/55361

Restoring deleted ChangeLog entry, and added new ChangeLog entry
for r55361.

01:19 Changeset [55361] by cjerdonek@webkit.org

Unreviewed. Rolling out r55350: http://trac.webkit.org/changeset/55350

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

Need to roll out because this patch (pep8) depends on the newly rewritten
autoinstall.py (r55348), which is breaking for people with Python 2.4:

https://bugs.webkit.org/show_bug.cgi?id=35163#c21

That also needs to be rolled out and will be rolled out next.

Note: See TracTimeline for information about the timeline view.