Timeline


and

01/29/10:

21:37 Changeset [54103] by mrowe@apple.com
  • 2 edits in trunk/WebCore

Stop copying an IDL file in to the framework wrapper.

  • WebCore.xcodeproj/project.pbxproj:
21:35 Changeset [54102] by mrowe@apple.com
  • 6 edits in trunk

Sort Xcode projects.

JavaScriptCore:

WebCore:

  • WebCore.xcodeproj/project.pbxproj:

WebKit:

  • WebKit.xcodeproj/project.pbxproj:
21:32 Changeset [54101] by mrowe@apple.com
  • 3 edits
    3 copies in trunk

Roll over a few oversized ChangeLogs.

21:30 Changeset [54100] by mrowe@apple.com
  • 7 edits
    2 deletes in trunk

Roll out r54073 as it introduced many thousands of leaks.

21:21 Changeset [54099] by mrowe@apple.com
  • 2 edits in trunk/WebKitTools

Fix run-leaks with newer versions of the leaks tool.

Rubber-stamped by Stephanie Lewis.

  • Scripts/run-leaks:
21:00 Changeset [54098] by mrowe@apple.com
  • 3 edits in trunk/WebKitLibraries

Keep the Windows feature defines in sync with FeatureDefines.xcconfig.

  • win/tools/vsprops/FeatureDefines.vsprops:
  • win/tools/vsprops/FeatureDefinesCairo.vsprops:
20:51 Changeset [54097] by mrowe@apple.com
  • 6 edits in trunk

Fix the Mac build.

Disable ENABLE_INDEXED_DATABASE since it is "completely non-functional".

As the comment in FeatureDefines.xcconfig notes, the list of feature defines
needs to be kept in sync across the various files. The default values also
need to be kept in sync between these files and build-webkit.

JavaScriptCore:

  • Configurations/FeatureDefines.xcconfig:

WebCore:

  • Configurations/FeatureDefines.xcconfig:

WebKit/mac:

  • Configurations/FeatureDefines.xcconfig:
19:04 Changeset [54096] by sfalken@apple.com
  • 2 edits in trunk/WebCore

2010-01-29 Steve Falkenburg <sfalken@apple.com>

Reviewed by Darin Adler.

Timeout for client-based Geolocation shouldn't start until user gives consent
https://bugs.webkit.org/show_bug.cgi?id=34352

  • page/Geolocation.cpp: (WebCore::Geolocation::startRequest): Don't start timer if we're blocked on user consent. (WebCore::Geolocation::setIsAllowed): Start timer after user gives consent.
18:17 Changeset [54095] by cmarrin@apple.com
  • 4 edits in trunk/WebCore

Fixed changed virtual function in GraphicsLayerCACF and call order issues
https://bugs.webkit.org/show_bug.cgi?id=34348


The correct virtual function in GraphicsLayerCACF is now being
called. We also fixed an issue in QTMovieWin where the size
of the movie was not being set correctly because the call order
was changed.

17:25 Changeset [54094] by dpranke@chromium.org
  • 1 edit
    4 adds in trunk/WebKitTools

2010-01-29 Dirk Pranke <dpranke@chromium.org>

Reviewed by Eric Siedel.

Top-level test drivers for running the Chromium port of run-webkit-tests
and being able to rebaseline test results from the test bots. The
files in the Scripts directory are simply wrappers around the files
in webkitpy/layout_tests for convenience.


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

  • Scripts/rebaseline-chromium-webkit-tests: Added.
  • Scripts/run-chromium-webkit-tests: Added.
  • Scripts/webkitpy/layout_tests/rebaseline_chromium_webkit_tests.py: Added.
  • Scripts/webkitpy/layout_tests/run_chromium_webkit_tests.py: Added.
17:17 Changeset [54093] by dpranke@chromium.org
  • 1 edit
    6 adds in trunk/WebKitTools

2010-01-29 Dirk Pranke <dpranke@chromium.org>

Reviewed by Eric Siedel.

Add in the second block of python code for the Chromium port
of run-webkit-tests. These files execute different diffs to classify
the various types of failures from a test.

  • Scripts/webkitpy/layout_tests/test_types: Added.
  • Scripts/webkitpy/layout_tests/test_types/init.py: Added.
  • Scripts/webkitpy/layout_tests/test_types/fuzzy_image_diff.py: Added.
  • Scripts/webkitpy/layout_tests/test_types/image_diff.py: Added.
  • Scripts/webkitpy/layout_tests/test_types/test_type_base.py: Added.
  • Scripts/webkitpy/layout_tests/test_types/text_diff.py: Added.
17:12 Changeset [54092] by mrowe@apple.com
  • 4 edits in branches/safari-4-branch

Versioning.

17:06 Changeset [54091] by dpranke@chromium.org
  • 1 edit
    21 adds in trunk/WebKitTools

2010-01-29 Dirk Pranke <dpranke@chromium.org>

Reviewed by Eric Siedel.

Check in the first part of the Chromium Python port of the
run-webkit-tests test driver. The files under
layout_tests/layout_layout constitute most of the implementation;
they can be roughly divided into code that parses the
"test_expectations.txt" file that describes how we expect tests to
pass or fail, platform-specific hooks for the different Chromium
ports (in platform_utils*), code for parsing the output of the
tests and generating results files and HTML and JSON for the
dashboards, auxiliary scripts for starting and stopping HTTP and
Web Socket servers, and then one of the actual driver files
(test_shell_thread). Code for actually parsing test output for
failures and the top-level driver scripts will follow shortly.

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

  • Scripts/webkitpy/layout_tests: Added.
  • Scripts/webkitpy/layout_tests/layout_package: Added.
  • Scripts/webkitpy/layout_tests/layout_package/init.py: Added.
  • Scripts/webkitpy/layout_tests/layout_package/apache_http_server.py: Added.
  • Scripts/webkitpy/layout_tests/layout_package/http_server.py: Added.
  • Scripts/webkitpy/layout_tests/layout_package/http_server_base.py: Added.
  • Scripts/webkitpy/layout_tests/layout_package/httpd2.pem: Added.
    • scripts to start and stop apache. Note that the apache file generates a conf file dynamically, and we should switch to using the same static conf file that the regular run-webkit-tests uses, and we can also use the same httpd2.pem file.
  • Scripts/webkitpy/layout_tests/layout_package/json_layout_results_generator.py: Added.
  • Scripts/webkitpy/layout_tests/layout_package/json_results_generator.py: Added.
    • scripts to generate the JSON layout test dashboard and the flakiness dashboard
  • Scripts/webkitpy/layout_tests/layout_package/lighttpd.conf: Added.
    • default configuration for LigHTTPd (used on Windows)
  • Scripts/webkitpy/layout_tests/layout_package/metered_stream.py: Added.
    • utility class that implements progress bars on the console to be displayed while the tests are running
  • Scripts/webkitpy/layout_tests/layout_package/path_utils.py: Added.
    • various routines for manipulating paths and URIs
  • Scripts/webkitpy/layout_tests/layout_package/platform_utils.py: Added.
  • Scripts/webkitpy/layout_tests/layout_package/platform_utils_linux.py: Added.
  • Scripts/webkitpy/layout_tests/layout_package/platform_utils_mac.py: Added.
  • Scripts/webkitpy/layout_tests/layout_package/platform_utils_win.py: Added.
    • platform-specific aspects of the drivers (binary names, paths, process control, etc.)
  • Scripts/webkitpy/layout_tests/layout_package/test_expectations.py: Added.
    • code for parsing the 'test_expectations.txt' file to determine which tests are expected to fail (and how) on which platforms
  • Scripts/webkitpy/layout_tests/layout_package/test_failures.py: Added.
    • code for handling different kinds of failures (generating output in the results, etc.)
  • Scripts/webkitpy/layout_tests/layout_package/test_files.py: Added.
    • code to gather the lists of tests
  • Scripts/webkitpy/layout_tests/layout_package/test_shell_thread.py: Added.
    • code to actually execute tests via TestShell and process the output
  • Scripts/webkitpy/layout_tests/layout_package/websocket_server.py: Added.
    • scripts to start and stop the pywebsocket server
16:59 Changeset [54090] by bweinstein@apple.com
  • 4 edits in trunk/WebKit/win

Roll out r54088 because the functions were already in
IWebPreferencesPrivate, so they should stay there.

16:48 Changeset [54089] by jpetsovits@rim.com
  • 4 edits in trunk/WebCore

2010-01-29 Jakob Petsovits <jpetsovits@rim.com>

Reviewed by Nikolas Zimmermann.

[OpenVG] Implement more graphics primitives
https://bugs.webkit.org/show_bug.cgi?id=34339

Adds lines, arcs, ellipses, polygons and rounded
rectangles to PainterOpenVG and GraphicsContext.

Rounded rects support by Eli Fidler <efidler@rim.com>.

  • platform/graphics/openvg/GraphicsContextOpenVG.cpp: (WebCore::GraphicsContext::drawLine): (WebCore::GraphicsContext::drawEllipse): (WebCore::GraphicsContext::strokeArc): (WebCore::GraphicsContext::drawConvexPolygon): (WebCore::GraphicsContext::fillRect): (WebCore::GraphicsContext::fillRoundedRect): (WebCore::GraphicsContext::drawFocusRing): (WebCore::GraphicsContext::drawLineForText): (WebCore::GraphicsContext::clearRect): (WebCore::GraphicsContext::strokeRect):
  • platform/graphics/openvg/PainterOpenVG.cpp: (WebCore::PainterOpenVG::drawRect): (WebCore::PainterOpenVG::drawRoundedRect): (WebCore::PainterOpenVG::drawLine): (WebCore::PainterOpenVG::drawArc): (WebCore::PainterOpenVG::drawEllipse): (WebCore::PainterOpenVG::drawPolygon):
  • platform/graphics/openvg/PainterOpenVG.h:
16:48 Changeset [54088] by bweinstein@apple.com
  • 4 edits in trunk/WebKit/win

Rubber-stamped by Adam Roben.

Move customDragCursorEnabled preference functions from
IWebPreferences to IWebPreferencesPrivate.

  • Interfaces/IWebPreferences.idl: Moved customDragFunctions from here...
  • Interfaces/IWebPreferencesPrivate.idl: To here.
  • Interfaces/WebKit.idl: Touched to make sure we rebuild.
16:34 Changeset [54087] by dpranke@chromium.org
  • 1 edit
    9 adds in trunk/WebKitTools

2010-01-29 Dirk Pranke <dpranke@chromium.org>

Reviewed by Eric Seidel.

Check in a copy of the simplejson library; it will be used by
the Chromium port of run-webkit-tests.


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

  • simplejson: Added.
  • simplejson/LICENSE.txt: Added.
  • simplejson/README.txt: Added.
  • simplejson/init.py: Added.
  • simplejson/_speedups.c: Added. (ascii_escape_char): (ascii_escape_unicode): (ascii_escape_str): (py_encode_basestring_ascii): (init_speedups):
  • simplejson/decoder.py: Added.
  • simplejson/encoder.py: Added.
  • simplejson/jsonfilter.py: Added.
  • simplejson/scanner.py: Added.
16:27 Changeset [54086] by dpranke@chromium.org
  • 2 edits in trunk/WebKitTools

2010-01-29 Dirk Pranke <dpranke@chromium.org>

No review

Add myself to the committers list

  • Scripts/webkitpy/committers.py:
16:18 Changeset [54085] by jorlow@chromium.org
  • 23 edits
    12 adds in trunk

2010-01-29 Jeremy Orlow <jorlow@chromium.org>

Reviewed by Dimitri Glazkov.

A first step towards the Indexed Database API
https://bugs.webkit.org/show_bug.cgi?id=34342

Add runtime enable support for Indexed Database API.

  • features.gypi:
  • public/WebRuntimeFeatures.h:
  • src/WebRuntimeFeatures.cpp: (WebKit::WebRuntimeFeatures::enableIndexedDatabase): (WebKit::WebRuntimeFeatures::isIndexedDatabaseEnabled):

2010-01-29 Jeremy Orlow <jorlow@chromium.org>

Reviewed by Dimitri Glazkov.

A first step towards the Indexed Database API
https://bugs.webkit.org/show_bug.cgi?id=34342

Flesh out the first part of Indexed Database API.
Currently only compiles with v8 + chromium for now.
Completely non-functional, but it seems best to do
this in chunks.

No tests because nothing works yet.

  • Configurations/FeatureDefines.xcconfig:
  • WebCore.gypi:
  • bindings/scripts/CodeGeneratorV8.pm:
  • bindings/v8/DOMObjectsInclude.h:
  • bindings/v8/DerivedSourcesAllInOne.cpp:
  • bindings/v8/RuntimeEnabledFeatures.cpp:
  • bindings/v8/RuntimeEnabledFeatures.h: (WebCore::RuntimeEnabledFeatures::setIndexedDatabaseEnabled): (WebCore::RuntimeEnabledFeatures::indexedDatabaseEnabled):
  • bindings/v8/V8Index.cpp:
  • bindings/v8/V8Index.h:
  • bindings/v8/custom/V8DOMWindowCustom.cpp: (WebCore::V8DOMWindow::IndexedDBEnabled):
  • bindings/v8/custom/V8IDBRequestCustom.cpp: Added. (WebCore::V8IDBRequest::resultAccessorGetter):
  • bindings/v8/custom/V8IndexedDatabaseRequestCustom.cpp: Added. (WebCore::V8IndexedDatabaseRequest::openCallback):
  • dom/EventNames.h:
  • page/DOMWindow.cpp: (WebCore::DOMWindow::indexedDB):
  • page/DOMWindow.h:
  • page/DOMWindow.idl:
  • storage/IDBDatabaseError.h: Added. (WebCore::IDBDatabaseError::create): (WebCore::IDBDatabaseError::~IDBDatabaseError): (WebCore::IDBDatabaseError::code): (WebCore::IDBDatabaseError::setCode): (WebCore::IDBDatabaseError::message): (WebCore::IDBDatabaseError::setMessage): (WebCore::IDBDatabaseError::IDBDatabaseError):
  • storage/IDBDatabaseError.idl: Added.
  • storage/IDBDatabaseException.h: Added. (WebCore::IDBDatabaseException::create): (WebCore::IDBDatabaseException::~IDBDatabaseException): (WebCore::IDBDatabaseException::code): (WebCore::IDBDatabaseException::setCode): (WebCore::IDBDatabaseException::message): (WebCore::IDBDatabaseException::setMessage): (WebCore::IDBDatabaseException::IDBDatabaseException):
  • storage/IDBDatabaseException.idl: Added.
  • storage/IDBRequest.cpp: Added. (WebCore::IDBRequest::IDBRequest): (WebCore::IDBRequest::~IDBRequest): (WebCore::IDBRequest::abort): (WebCore::IDBRequest::eventTargetData): (WebCore::IDBRequest::ensureEventTargetData):
  • storage/IDBRequest.h: Added. (WebCore::IDBRequest::create): (WebCore::IDBRequest::readyState): (WebCore::IDBRequest::error): (WebCore::IDBRequest::result): (WebCore::IDBRequest::scriptExecutionContext): (WebCore::IDBRequest::toIDBRequest): (WebCore::IDBRequest::refEventTarget): (WebCore::IDBRequest::derefEventTarget):
  • storage/IDBRequest.idl: Added.
  • storage/IndexedDatabaseRequest.cpp: Added. (WebCore::IndexedDatabaseRequest::IndexedDatabaseRequest): (WebCore::IndexedDatabaseRequest::~IndexedDatabaseRequest): (WebCore::IndexedDatabaseRequest::open):
  • storage/IndexedDatabaseRequest.h: Added. (WebCore::IndexedDatabaseRequest::create): (WebCore::IndexedDatabaseRequest::request):
  • storage/IndexedDatabaseRequest.idl: Added.

2010-01-29 Jeremy Orlow <jorlow@chromium.org>

Reviewed by Dimitri Glazkov.

A first step towards the Indexed Database API
https://bugs.webkit.org/show_bug.cgi?id=34342

Add Indexed Database API

  • configure.ac:

2010-01-29 Jeremy Orlow <jorlow@chromium.org>

Reviewed by Dimitri Glazkov.

A first step towards the Indexed Database API
https://bugs.webkit.org/show_bug.cgi?id=34342

Add indexed database API.

  • Scripts/build-webkit:
16:00 Changeset [54084] by abecsi@webkit.org
  • 2 edits in trunk/WebKitTools

Correct openHTTPD() to print requests to stdout if run-webkit-httpd is used.
This fixes a regression introduced in r53559.

Reviewed by Alexey Proskuryakov.

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

  • Scripts/webkitperl/httpd.pm:
16:00 Changeset [54083] by bweinstein@apple.com
  • 9 edits in trunk

WebCore: Drag and Drop: Windows uses "stop" sign as cursor when dragging
https://bugs.webkit.org/show_bug.cgi?id=34305
<rdar://problem/7589672>

Reviewed by Adam Roben.

Add a FIXME for the code that needs to be changed to support full
custom cursors.

  • page/EventHandler.cpp:

(WebCore::EventHandler::handleDrag):

WebKit/win: Drag and Drop: Windows uses "stop" sign as cursor when dragging
https://bugs.webkit.org/show_bug.cgi?id=34305
<rdar://problem/7589672>

Reviewed by Adam Roben.

Add a preference in WebKit (that defaults to false), for whether or not
we should show the custom cursors during drag and drop. However, this is
currently only used on Windows, and only used to hide the "drop not allowed"
icon inside the WebView is the preference is set to true.

This will be off by default, so no change in behavior.

  • Interfaces/IWebPreferencesPrivate.idl: Added new functions.
  • Interfaces/WebKit.idl: Touched to force Interfaces build.
  • WebDropSource.cpp:

(WebDropSource::GiveFeedback): Implementation of conditional showing cursor

logic.

  • WebPreferenceKeysPrivate.h: Added new preference key.
  • WebPreferences.cpp: Added new functions.

(WebPreferences::setCustomDragCursorsEnabled):
(WebPreferences::customDragCursorsEnabled):

  • WebPreferences.h: Added new functions.
15:39 Changeset [54082] by victorw@chromium.org
  • 4 edits
    2 adds in trunk
14:26 Changeset [54081] by ap@apple.com
  • 9 edits in trunk

Reviewed by Dan Bernstein.

<rdar://problem/7441982> REGRESSION (TOT): Adobe CS4: Installer alerts are displayed as
blank windows

We probably shouldn't be deferring loads below modal dialogs and alerts, because that's not
what being modal means. But making such a change for general Web content would require fixes
in other parts of code, which I'm not ready to implement right now, so making it application
specific.

WebCore:

  • page/Page.cpp: (WebCore::Page::setDefersLoading): Do nothing if load deferring is not enabled in page settings.
  • WebCore.base.exp:
  • page/Settings.cpp: (WebCore::Settings::Settings): (WebCore::Settings::setLoadDeferringEnabled):
  • page/Settings.h: (WebCore::Settings::loadDeferringEnabled): Add the ability for client to disable page deferring (still enabled by default).
  • platform/mac/RuntimeApplicationChecks.h:
  • platform/mac/RuntimeApplicationChecks.mm: (WebCore::applicationIsAdobeInstaller): Added a bundle ID test for Adobe installer.

WebKit/mac:

  • WebView/WebView.mm: (shouldEnableLoadDeferring): Load deferring is enabled unless the application is Adobe Installer. (-[WebView _preferencesChangedNotification:]): Call setLoadDeferringEnabled().
14:16 Changeset [54080] by sfalken@apple.com
  • 2 edits in trunk/WebCore

Build fix.

  • page/Geolocation.cpp:

(WebCore::Geolocation::startUpdating):

14:08 Changeset [54079] by sfalken@apple.com
  • 3 edits in trunk/WebCore

2010-01-29 Steve Falkenburg <sfalken@apple.com>

Reviewed by Darin Adler.

Client-based Geolocation starts updating before getting consent from the user
https://bugs.webkit.org/show_bug.cgi?id=34343

  • page/Geolocation.cpp: (WebCore::Geolocation::startRequest): Pass notifier instead of options to startUpdating. (WebCore::Geolocation::setIsAllowed): Add the observer or notify of error for deferred startUpdating. (WebCore::Geolocation::startUpdating): Pass notifier instead of options, since we may need to call it if we fail to get user consent. Defer adding the observer if we don't yet have user consent, since this could kick off server-based wifi Geolocation requests.
  • page/Geolocation.h:
13:55 Changeset [54078] by jhoneycutt@apple.com
  • 17 edits
    1 copy
    2 adds in trunk

MSAA: Crash when posting a notification for a detached object

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

Reviewed by Darin Adler.

WebCore:

Test: platform/win/accessibility/detached-object-notification-crash.html

  • accessibility/AccessibilityRenderObject.cpp:

(AccessibilityRenderObject::document):
Null check m_renderer. This is the bug fix; the other changes in the
patch are for the test.

  • accessibility/win/AXObjectCacheWin.cpp:

(WebCore::AXObjectCache::postPlatformNotification):
Map AXValueChanged to EVENT_OBJECT_VALUECHANGED, so we'll post a
notification when AXValueChanged is posted. Receiving an event of this
type tells us that the test passed.

WebKit/win:

  • AccessibleBase.cpp:

(AccessibleBase::QueryService):
If an unrecognized service ID is passed, return early. Otherwise, return
the result of QueryInterface.
(AccessibleBase::QueryInterface):
Add static_casts. Check for new UUIDs.
(AccessibleBase::isSameObject):
Query the object for AccessibleBase. Return whether the pointers or the
wrapped objects match.

  • AccessibleBase.h:

Give the class a UUID so we can query for it in isSameObject(). Inherit
from IAccessibleComparable; inherit from IServiceProvider so clients can
use QueryService to query for a custom interface.

  • Interfaces/AccessibleComparable.idl: Added. Declares a function that

can be called to compare to accessible objects.

  • Interfaces/WebKit.idl:

Include the new IDL.

  • WebKit.vcproj/Interfaces.vcproj:

Add the new IDL to the project.

WebKitTools:

  • DumpRenderTree/AccessibilityController.h:

Declare new functions. Add new members to store the event hook and the
mapping of accessibility elements to their JS callbacks.

  • DumpRenderTree/gtk/AccessibilityControllerGtk.cpp:

(AccessibilityController::notificationReceived):
Stubbed.
(AccessibilityController::addNotificationListener):
Stubbed.

  • DumpRenderTree/mac/AccessibilityControllerMac.mm:

(AccessibilityController::notificationReceived):
Stubbed.
(AccessibilityController::addNotificationListener):
Stubbed.

  • DumpRenderTree/win/AccessibilityControllerWin.cpp:

(AccessibilityController::AccessibilityController):
Initialize the event hook.
(AccessibilityController::~AccessibilityController):
Remove the event hook. Unprotect all of the JS functions that are stored
in the map.
(logEventProc):
Clean-up a variable.
(stringEvent):
Return a string description of the MSAA event code.
(notificationListenerProc):
Get the accessible object from the event, and query it for IAccessible.
Call the AccessibilityController's notificationReceived().
(comparableObject):
Use QueryService to obtain the IAccessibleComparable for the
IServiceProvider.
(AccessibilityController::notificationReceived):
Iterate the map of objects that have registered for notification
callbacks. Query each for IServiceProvider, then use comparableObject()
to get an IAccessibleComparable. If we find an object matching the
notified object, call its callback, passing the event that was received.
(AccessibilityController::addNotificationListener):
If we have not created the event hook, create it. Protect the JS
callback function object, and add the object and its callback to our
map.

  • DumpRenderTree/win/AccessibilityUIElementWin.cpp:

(AccessibilityUIElement::addNotificationListener):
Call through to the AccessibilityController's addNotificationListener().

  • DumpRenderTree/win/DumpRenderTreeWin.h:

Add an extern declaration for the shared FrameLoadDelegate extern, so we
can access it from AccessibilityController.

  • DumpRenderTree/win/FrameLoadDelegate.h:

(FrameLoadDelegate::accessibilityController):
Return the AccessibilityController.

LayoutTests:

  • platform/win/accessibility/detached-object-notification-crash-expected.txt: Added.
  • platform/win/accessibility/detached-object-notification-crash.html: Added.
13:32 Changeset [54077] by kenneth@webkit.org
  • 2 edits in trunk/WebKitTools

Rubberstamped by Simon Hausmann.

[Qt] Separate implementation from class definition.

  • QtLauncher/main.cpp:

(LauncherWindow::webView):
(LauncherWindow::LauncherWindow):
(LauncherWindow::~LauncherWindow):
(LauncherWindow::keyPressEvent):
(LauncherWindow::grabZoomKeys):
(LauncherWindow::sendTouchEvent):
(LauncherWindow::eventFilter):
(LauncherWindow::loadStarted):
(LauncherWindow::loadFinished):
(LauncherWindow::showLinkHover):
(LauncherWindow::zoomIn):
(LauncherWindow::zoomOut):
(LauncherWindow::resetZoom):
(LauncherWindow::toggleZoomTextOnly):
(LauncherWindow::print):
(LauncherWindow::screenshot):
(LauncherWindow::setEditable):
(LauncherWindow::dumpHtml):
(LauncherWindow::selectElements):
(LauncherWindow::setTouchMocking):
(LauncherWindow::newWindow):
(LauncherWindow::setupUI):

13:12 Changeset [54076] by eric@webkit.org
  • 2 edits in trunk/WebKitTools

2010-01-29 Shinichiro Hamaji <hamaji@chromium.org>

Reviewed by Darin Adler.

prepare-ChangeLog outputs useless messages for a nonexistent bug ID
https://bugs.webkit.org/show_bug.cgi?id=34313

  • Scripts/prepare-ChangeLog:
12:21 Changeset [54075] by darin@chromium.org
  • 9 edits in trunk

2010-01-29 Darin Fisher <darin@chromium.org>

Okayed by Oliver Hunt.

Rollout r53949, r53951 and r54013 due to a Chromium regression that it
causes. Somehow this code change is triggering an endless repaint loop.

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

  • platform/graphics/GraphicsContext.cpp: (WebCore::GraphicsContext::drawTiledImage):
  • platform/graphics/GraphicsContext.h:
  • platform/graphics/transforms/TransformationMatrix.h: (WebCore::TransformationMatrix::isIdentityOrTranslation):
  • rendering/RenderBoxModelObject.cpp: (WebCore::RenderBoxModelObject::~RenderBoxModelObject): (WebCore::RenderBoxModelObject::paintFillLayerExtended):
  • rendering/RenderBoxModelObject.h:
11:47 Changeset [54074] by kenneth@webkit.org
  • 3 edits in trunk/WebKitTools

[Qt] On Maemo5, a click/touch in the location bar (when unfocused)
should result in all text selected.

Reviewed by Ariya Hidayat.

  • QtLauncher/locationedit.cpp:

(LocationEdit::focusInEvent):

  • QtLauncher/locationedit.h:
11:46 Changeset [54073] by oliver@apple.com
  • 7 edits
    2 adds in trunk

2010-01-29 Oliver Hunt <oliver@apple.com>

Reviewed by Darin Adler.

JSC is failing to propagate anonymous slot count on some transitions
https://bugs.webkit.org/show_bug.cgi?id=34321

Remove the unsafe two argument Structure::create method, and correct
the uses of it to propagate the anonymous slot count.

  • runtime/JSObject.h: (JSC::JSObject::setStructure):
  • runtime/Structure.cpp: (JSC::Structure::addPropertyTransition): (JSC::Structure::changePrototypeTransition): (JSC::Structure::despecifyFunctionTransition): (JSC::Structure::getterSetterTransition): (JSC::Structure::toDictionaryTransition):
  • runtime/Structure.h:

2010-01-29 Oliver Hunt <oliver@apple.com>

Reviewed by Darin Adler.

JSC is failing to propagate anonymous slot count on some transitions
https://bugs.webkit.org/show_bug.cgi?id=34321

Add a test for modification of a type with anonymous slots.

  • fast/dom/Window/anonymous-slot-with-changes-expected.txt: Added.
  • fast/dom/Window/anonymous-slot-with-changes.html: Added.

2010-01-29 Oliver Hunt <oliver@apple.com>

Reviewed by Darin Adler.

JSC is failing to propagate anonymous slot count on some transitions
https://bugs.webkit.org/show_bug.cgi?id=34321

Make code generator add assertions for anonymous slot count.

Test: fast/dom/Window/anonymous-slot-with-changes.html

  • bindings/scripts/CodeGeneratorJS.pm:
11:46 Changeset [54072] by kenneth@webkit.org
  • 3 edits in trunk/WebKitTools

[Qt] Show progress reaching 100% for loads.

Reviewed by Ariya Hidayat.

  • QtLauncher/locationedit.cpp:

(LocationEdit::LocationEdit):
(LocationEdit::setProgress):
(LocationEdit::reset):
(LocationEdit::paintEvent):

  • QtLauncher/locationedit.h:
10:32 Changeset [54071] by eric@webkit.org
  • 4 edits in trunk/WebCore

2010-01-29 Tony Chang <tony@chromium.org>

Reviewed by Darin Adler.

Add gdb helper methods for Ranges like we have for VisibleSelections.
https://bugs.webkit.org/show_bug.cgi?id=34308

No new tests, these are only for debugging.

  • dom/Position.cpp: (WebCore::Position::showTreeForThis):
  • dom/Range.cpp: (showTree):
  • dom/Range.h:
10:13 Changeset [54070] by eric@webkit.org
  • 2 edits in trunk/WebCore

2010-01-29 Drew Wilson <atwilson@chromium.org>

Reviewed by Adam Barth.

V8 implementation of MessageEvent.initMessageEvent() does not set source correctly.
https://bugs.webkit.org/show_bug.cgi?id=34292

Test: fast/events/init-events.html will pass in Chrome now.

  • bindings/v8/custom/V8MessageEventCustom.cpp: (WebCore::V8MessageEvent::initMessageEventCallback): Now properly extracts the reference to the DOMWindow object from the passed-in window parameter.
09:56 Changeset [54069] by eric@webkit.org
  • 7 edits in trunk

2010-01-29 Ben Murdoch <benm@google.com>

Reviewed by Dimitri Glazkov.

[Android] Android needs functionality in the ChromeClient to be informed when touch events are and are not needed by the webpage.
https://bugs.webkit.org/show_bug.cgi?id=34215

Add a function on the ChromeClient that WebCore can use to inform the platform when it needs touch events. This way the platform can optimise by not forwarding the events if they are not required.

No new tests as the only implementation is specific to Android.

  • dom/Document.cpp: (WebCore::Document::detach): Check if this is the top level document and if so, stop forwarding touch events. (WebCore::Document::addListenerTypeIfNeeded): Inform the ChromeClient it should start forwarding touch events and guard touch event code properly.
  • history/CachedFrame.cpp: (WebCore::CachedFrameBase::restore): If the document uses touch events, inform the ChromeClient to start forwarding them. (WebCore::CachedFrame::CachedFrame): If the document uses touch events, inform the ChromeClient to stop forwarding them, as the document is being put into the page cache.
  • loader/EmptyClients.h: (WebCore::EmptyChromeClient::needTouchEvents): Add an empty implementation.
  • page/ChromeClient.h: Add the needTouchEvents() function.

2010-01-29 Ben Murdoch <benm@google.com>

Reviewed by Dimitri Glazkov.

[Android] Android needs functionality in the ChromeClient to be informed when touch events are and are not needed by the webpage.
https://bugs.webkit.org/show_bug.cgi?id=34215

Add needTouchEvents() to the ChromeClient which is called when the page decides it needs or no longer needs to be informed of touch events.

  • WebCoreSupport/ChromeClientQt.h: (WebCore::ChromeClientQt::needTouchEvents): Add an empty implementation.
09:38 Changeset [54068] by eric@webkit.org
  • 2 edits in trunk/WebCore

2010-01-29 Alexander Pavlov <apavlov@chromium.org>

Reviewed by Pavel Feldman.

Fix toolbar gradient to match the window's titlebar on Snow Leopard
https://bugs.webkit.org/show_bug.cgi?id=34320

  • inspector/front-end/inspector.css:
09:22 Changeset [54067] by eric@webkit.org
  • 2 edits in trunk/WebCore

2010-01-29 Alexander Pavlov <apavlov@chromium.org>

Reviewed by Pavel Feldman.

Fix MacOS X version detection

MacOS X version can have 2 or 3 segments.
https://bugs.webkit.org/show_bug.cgi?id=34322

  • inspector/front-end/inspector.js: (WebInspector._detectPlatform):
09:06 Changeset [54066] by eric@webkit.org
  • 2 edits in trunk/WebKitTools

2010-01-29 Andreas Kling <andreas.kling@nokia.com>

Reviewed by Kenneth Rohde Christiansen.

[Qt] Disable the QtLauncher statusbar on Maemo

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

  • QtLauncher/main.cpp: (LauncherWindow::showLinkHover): (LauncherWindow::selectElements):
08:49 Changeset [54065] by eric@webkit.org
  • 4 edits
    2 copies in trunk/WebKitTools

2010-01-29 Andreas Kling <andreas.kling@nokia.com>

Reviewed by Kenneth Rohde Christiansen.

[Qt] Display page loading progress inside the QtLauncher location bar

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

  • QtLauncher/QtLauncher.pro:
  • QtLauncher/locationedit.cpp: Added. (LocationEdit::LocationEdit): (LocationEdit::setProgress): (LocationEdit::paintEvent):
  • QtLauncher/locationedit.h: Added.
  • QtLauncher/mainwindow.cpp: (MainWindow::buildUI):
  • QtLauncher/mainwindow.h:
08:25 Changeset [54064] by kenneth@webkit.org
  • 2 edits in trunk/WebKit/qt

Disable auto-uppercase and predictive text on Maemo5, just like the
build-in MicroB Browser.

Reviewed by Simon Hausmann

  • WebCoreSupport/EditorClientQt.cpp:

(WebCore::EditorClientQt::setInputMethodState):

07:43 Changeset [54063] by jpetsovits@rim.com
  • 8 edits
    4 adds in trunk/WebCore

2010-01-29 Jakob Petsovits <jpetsovits@rim.com>

Reviewed by Nikolas Zimmermann.

[OpenVG] Implement a basic GraphicsContext on top of a new PainterOpenVG class
https://bugs.webkit.org/show_bug.cgi?id=33405

PainterOpenVG provides painter state management that works
on a single (EGL) context, and apart from painter state
implements a single drawing operation (for now), drawRect().

It is a generic painter class that cooperates with
SurfaceOpenVG to enable robust surface/context switching
(given that EGL doesn't notify us if somebody switches
the surface/context behind our backs), and provides
painting operations close to OpenVG's way of working
that GraphicsContext, Path, Image, Font and other WebKit
platform classes can use to draw on.

Initial code for state management and OpenVG scissoring by
Eli Fidler <efidler@rim.com>. VGRect/VGMatrix and the bulk
of the transformations code by Adam Treat <atreat@rim.com>.
Preliminary drawFocusRing() implementation by Yong Li <yoli@rim.com>.

  • platform/graphics/FloatRect.h:
  • platform/graphics/GraphicsContext.cpp:
  • platform/graphics/GraphicsContext.h:
  • platform/graphics/openvg/GraphicsContextOpenVG.cpp: Added. (WebCore::GraphicsContextPlatformPrivate::GraphicsContextPlatformPrivate): (WebCore::GraphicsContext::GraphicsContext): (WebCore::GraphicsContext::~GraphicsContext): (WebCore::GraphicsContext::platformContext): (WebCore::GraphicsContext::getCTM): (WebCore::GraphicsContext::savePlatformState): (WebCore::GraphicsContext::restorePlatformState): (WebCore::GraphicsContext::drawRect): (WebCore::GraphicsContext::drawLine): (WebCore::GraphicsContext::drawEllipse): (WebCore::GraphicsContext::strokeArc): (WebCore::GraphicsContext::drawConvexPolygon): (WebCore::GraphicsContext::fillPath): (WebCore::GraphicsContext::strokePath): (WebCore::GraphicsContext::fillRect): (WebCore::GraphicsContext::fillRoundedRect): (WebCore::GraphicsContext::beginPath): (WebCore::GraphicsContext::addPath): (WebCore::GraphicsContext::clip): (WebCore::GraphicsContext::clipPath): (WebCore::GraphicsContext::drawFocusRing): (WebCore::GraphicsContext::drawLineForText): (WebCore::GraphicsContext::drawLineForMisspellingOrBadGrammar): (WebCore::GraphicsContext::roundToDevicePixels): (WebCore::GraphicsContext::setPlatformShadow): (WebCore::GraphicsContext::clearPlatformShadow): (WebCore::GraphicsContext::beginTransparencyLayer): (WebCore::GraphicsContext::endTransparencyLayer): (WebCore::GraphicsContext::clearRect): (WebCore::GraphicsContext::strokeRect): (WebCore::GraphicsContext::setLineCap): (WebCore::GraphicsContext::setLineDash): (WebCore::GraphicsContext::setLineJoin): (WebCore::GraphicsContext::setMiterLimit): (WebCore::GraphicsContext::setAlpha): (WebCore::GraphicsContext::setCompositeOperation): (WebCore::GraphicsContext::canvasClip): (WebCore::GraphicsContext::clipOut): (WebCore::GraphicsContext::scale): (WebCore::GraphicsContext::rotate): (WebCore::GraphicsContext::translate): (WebCore::GraphicsContext::origin): (WebCore::GraphicsContext::clipOutEllipseInRect): (WebCore::GraphicsContext::clipToImageBuffer): (WebCore::GraphicsContext::addInnerRoundedRectClip): (WebCore::GraphicsContext::concatCTM): (WebCore::GraphicsContext::setURLForRect): (WebCore::GraphicsContext::setPlatformStrokeColor): (WebCore::GraphicsContext::setPlatformStrokeStyle): (WebCore::GraphicsContext::setPlatformStrokeThickness): (WebCore::GraphicsContext::setPlatformFillColor): (WebCore::GraphicsContext::setPlatformShouldAntialias): (WebCore::GraphicsContext::setImageInterpolationQuality): (WebCore::GraphicsContext::imageInterpolationQuality):
  • platform/graphics/openvg/PainterOpenVG.cpp: Added. (WebCore::isNonRotatedAffineTransformation): (WebCore::toVGCapStyle): (WebCore::toVGJoinStyle): (WebCore::toVGFillRule): (WebCore::colorToVGColor): (WebCore::setVGSolidColor): (WebCore::PlatformPainterState::PlatformPainterState): (WebCore::PlatformPainterState::copyPaintState): (WebCore::PlatformPainterState::applyState): (WebCore::PlatformPainterState::applyBlending): (WebCore::PlatformPainterState::applyTransformationMatrix): (WebCore::PlatformPainterState::applyScissorRect): (WebCore::PlatformPainterState::applyStrokeStyle): (WebCore::PlatformPainterState::strokeDisabled): (WebCore::PlatformPainterState::fillDisabled): (WebCore::PainterOpenVG::PainterOpenVG): (WebCore::PainterOpenVG::~PainterOpenVG): (WebCore::PainterOpenVG::begin): (WebCore::PainterOpenVG::end): (WebCore::PainterOpenVG::destroyPainterStates): (WebCore::PainterOpenVG::applyState): (WebCore::PainterOpenVG::blitToSurface): (WebCore::PainterOpenVG::transformationMatrix): (WebCore::PainterOpenVG::concatTransformationMatrix): (WebCore::PainterOpenVG::setTransformationMatrix): (WebCore::PainterOpenVG::compositeOperation): (WebCore::PainterOpenVG::setCompositeOperation): (WebCore::PainterOpenVG::opacity): (WebCore::PainterOpenVG::setOpacity): (WebCore::PainterOpenVG::strokeThickness): (WebCore::PainterOpenVG::setStrokeThickness): (WebCore::PainterOpenVG::strokeStyle): (WebCore::PainterOpenVG::setStrokeStyle): (WebCore::PainterOpenVG::setLineDash): (WebCore::PainterOpenVG::setLineCap): (WebCore::PainterOpenVG::setLineJoin): (WebCore::PainterOpenVG::setMiterLimit): (WebCore::PainterOpenVG::strokeColor): (WebCore::PainterOpenVG::setStrokeColor): (WebCore::PainterOpenVG::fillColor): (WebCore::PainterOpenVG::setFillColor): (WebCore::PainterOpenVG::antialiasingEnabled): (WebCore::PainterOpenVG::setAntialiasingEnabled): (WebCore::PainterOpenVG::scale): (WebCore::PainterOpenVG::rotate): (WebCore::PainterOpenVG::translate): (WebCore::PainterOpenVG::intersectScissorRect): (WebCore::PainterOpenVG::intersectClipRect): (WebCore::PainterOpenVG::drawRect): (WebCore::PainterOpenVG::save): (WebCore::PainterOpenVG::restore):
  • platform/graphics/openvg/PainterOpenVG.h: Added. (WebCore::PainterOpenVG::): (WebCore::PainterOpenVG::surface):
  • platform/graphics/openvg/SurfaceOpenVG.cpp: (WebCore::SurfaceOpenVG::SurfaceOpenVG): (WebCore::SurfaceOpenVG::~SurfaceOpenVG): (WebCore::SurfaceOpenVG::makeCurrent): (WebCore::SurfaceOpenVG::makeCompatibleCurrent): (WebCore::SurfaceOpenVG::setActivePainter): (WebCore::SurfaceOpenVG::activePainter):
  • platform/graphics/openvg/SurfaceOpenVG.h: (WebCore::SurfaceOpenVG::):
  • platform/graphics/openvg/VGUtils.cpp: Added. (WebCore::VGMatrix::VGMatrix): (WebCore::VGMatrix::operator TransformationMatrix): (WebCore::TransformationMatrix::operator VGMatrix): (WebCore::VGRect::VGRect): (WebCore::VGRect::operator FloatRect): (WebCore::FloatRect::operator VGRect):
  • platform/graphics/openvg/VGUtils.h: (WebCore::VGMatrix::toVGfloat): (WebCore::VGRect::toVGfloat):
  • platform/graphics/transforms/TransformationMatrix.h:
07:42 Changeset [54062] by philn@webkit.org
  • 5 edits in trunk/LayoutTests

2010-01-29 Philippe Normand <pnormand@igalia.com>

Unreviewed, build fix.

http/tests/media/video-referer.html fails on mac
https://bugs.webkit.org/show_bug.cgi?id=34331

Skipping the failing test. Requires further investigation.

  • platform/mac-leopard/Skipped:
  • platform/mac-snowleopard/Skipped:
  • platform/mac-tiger/Skipped:
  • platform/mac/Skipped:
07:04 Changeset [54061] by philn@webkit.org
  • 3 edits
    4 adds in trunk

2010-01-29 Philippe Normand <pnormand@igalia.com>

Reviewed by Gustavo Noronha Silva.

[Gtk] Vimeo HTML5 player doesn't work
https://bugs.webkit.org/show_bug.cgi?id=34327

Send Referer when requesting media over HTTP.

Test: http/tests/media/video-referer.html

  • platform/graphics/gtk/MediaPlayerPrivateGStreamer.cpp: (WebCore::mediaPlayerPrivateSourceChangedCallback):

2010-01-29 Philippe Normand <pnormand@igalia.com>

Reviewed by Gustavo Noronha Silva.

[Gtk] Vimeo HTML5 player doesn't work
https://bugs.webkit.org/show_bug.cgi?id=34327

Test for HTTP Referer checking when playing remote media.

  • http/tests/media/resources/setCookieAndReferer.cgi: Added.
  • http/tests/media/resources/video-referer-check-referer.php: Added.
  • http/tests/media/video-referer-expected.txt: Added.
  • http/tests/media/video-referer.html: Added.
06:33 Changeset [54060] by hausmann@webkit.org
  • 3 edits in trunk/WebCore

[Qt] Speed up the WebCore::String -> QString conversion

Patch by Oswald Buddenhagen <oswald.buddenhagen@nokia.com> on 2010-01-29
Reviewed by Simon Hausmann.

Use QString(const QChar *, int len) constructor instead of QString::fromUtf16 to
avoid BOM checks and byteswapping.

  • bridge/qt/qt_class.cpp:

(JSC::Bindings::QtClass::fieldNamed):

  • bridge/qt/qt_runtime.cpp:

(JSC::Bindings::convertValueToQVariant):

06:10 Changeset [54059] by pfeldman@chromium.org
  • 3 edits in trunk/WebCore

2010-01-29 Alexander Pavlov <apavlov@chromium.org>

Reviewed by Timothy Hatcher.

Web Inspector: Source view displays text using wrong fonts
https://bugs.webkit.org/show_bug.cgi?id=34269

  • inspector/front-end/TextEditor.js: (WebInspector.TextEditor): (WebInspector.TextEditor.prototype._initFont):
  • inspector/front-end/inspector.js:
06:01 QtWebKitBackportingFixes edited by hausmann@webkit.org
(diff)
06:00 Changeset [54058] by eric@webkit.org
  • 2 edits in trunk/WebKitTools

2010-01-29 Andreas Kling <andreas.kling@nokia.com>

Reviewed by Kenneth Rohde Christiansen.

[Qt] Add support for Maemo zoom keys in QtLauncher

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

  • QtLauncher/main.cpp: (LauncherWindow::LauncherWindow): (LauncherWindow::~LauncherWindow): (LauncherWindow::keyPressEvent): (LauncherWindow::grabZoomKeys):
05:48 QtWebKitBackportingFixes edited by hausmann@webkit.org
(diff)
05:46 QtWebKitBackportingFixes edited by hausmann@webkit.org
(diff)
05:41 Changeset [54057] by yurys@chromium.org
  • 2 edits in trunk/WebKit/chromium

2010-01-29 Yury Semikhatsky <yurys@chromium.org>

Reviewed by Pavel Feldman.

Add API methods for loading inspector utility scripts.

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

  • public/WebDevToolsAgentClient.h: (WebKit::WebDevToolsAgentClient::injectedScriptSource): (WebKit::WebDevToolsAgentClient::injectedScriptDispatcherSource):
05:36 QtWebKitBackportingFixes edited by hausmann@webkit.org
(diff)
05:32 QtWebKitBackportingFixes edited by hausmann@webkit.org
(diff)
05:29 Changeset [54056] by eric@webkit.org
  • 2 edits in trunk/WebCore

2010-01-29 Philippe Normand <pnormand@igalia.com>

Reviewed by Gustavo Noronha Silva.

[Gtk] Apple trailers don't play anymore
https://bugs.webkit.org/show_bug.cgi?id=34316

Fake QuickTime when accessing movie trailers.

  • platform/graphics/gtk/MediaPlayerPrivateGStreamer.cpp: (WebCore::mediaPlayerPrivateSourceChangedCallback):
05:02 Changeset [54055] by abecsi@webkit.org
  • 2 edits in trunk/LayoutTests

Unreviewed.

[Qt] Remove disabled tests from Skiplist. The tests were disabled in r53701.

  • platform/qt/Skipped: -css2.1/t1204-increment-00-c-o.html -css2.1/t1204-increment-01-c-o.html -css2.1/t1204-increment-02-c-o.html -css2.1/t1204-reset-00-c-o.html -css2.1/t1204-reset-01-c-o.html -css2.1/t1204-reset-02-c-o.html
04:17 Changeset [54054] by eric@webkit.org
  • 5 edits in trunk/WebKitTools

2010-01-29 Benjamin Poulain <benjamin.poulain@nokia.com>

Reviewed by Kenneth Rohde Christiansen.

[Qt] Implement the display() method of the layout test controller
https://bugs.webkit.org/show_bug.cgi?id=34258

  • DumpRenderTree/qt/DumpRenderTreeQt.cpp: (WebCore::DumpRenderTree::DumpRenderTree): (WebCore::DumpRenderTree::showPage): (WebCore::DumpRenderTree::hidePage):
  • DumpRenderTree/qt/DumpRenderTreeQt.h:
  • DumpRenderTree/qt/LayoutTestControllerQt.cpp: (LayoutTestController::reset): (LayoutTestController::display):
  • DumpRenderTree/qt/LayoutTestControllerQt.h:
03:04 Changeset [54053] by pfeldman@chromium.org
  • 19 edits
    2 adds
    3 deletes in trunk

2010-01-29 Pavel Feldman <pfeldman@chromium.org>

Reviewed by Timothy Hatcher.

Web Inspector: migrate to tokenizer-based highlighting in the Elements panel.

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

  • WebCore.gypi:
  • WebCore.vcproj/WebCore.vcproj:
  • inspector/front-end/CSSSourceSyntaxHighlighter.js: Removed.
  • inspector/front-end/DOMSyntaxHighlighter.js: Added. (WebInspector.DOMSyntaxHighlighter): (WebInspector.DOMSyntaxHighlighter.prototype.createSpan): (WebInspector.DOMSyntaxHighlighter.prototype.syntaxHighlightNode):
  • inspector/front-end/ElementsTreeOutline.js:
  • inspector/front-end/JavaScriptSourceSyntaxHighlighter.js: Removed.
  • inspector/front-end/SourceCSSTokenizer.js: (WebInspector.SourceCSSTokenizer): (WebInspector.SourceCSSTokenizer.prototype.nextToken):
  • inspector/front-end/SourceCSSTokenizer.re2js:
  • inspector/front-end/SourceHTMLTokenizer.js: (WebInspector.SourceHTMLTokenizer): (WebInspector.SourceHTMLTokenizer.prototype.nextToken):
  • inspector/front-end/SourceHTMLTokenizer.re2js:
  • inspector/front-end/SourceJavaScriptTokenizer.js: (WebInspector.SourceJavaScriptTokenizer): (WebInspector.SourceJavaScriptTokenizer.prototype.nextToken):
  • inspector/front-end/SourceJavaScriptTokenizer.re2js:
  • inspector/front-end/SourceSyntaxHighlighter.js: Removed.
  • inspector/front-end/SourceTokenizer.js: Added. (WebInspector.SourceTokenizer): (WebInspector.SourceTokenizer.prototype.set line): (WebInspector.SourceTokenizer.prototype.set condition): (WebInspector.SourceTokenizer.prototype.get condition): (WebInspector.SourceTokenizer.prototype.hasCondition): (WebInspector.SourceTokenizer.prototype.getLexCondition): (WebInspector.SourceTokenizer.prototype.setLexCondition): (WebInspector.SourceTokenizer.prototype._charAt): (WebInspector.SourceTokenizer.Registry): (WebInspector.SourceTokenizer.Registry.getInstance): (WebInspector.SourceTokenizer.Registry.prototype.getTokenizer):
  • inspector/front-end/TextEditorHighlighter.js: (WebInspector.TextEditorHighlighter): (WebInspector.TextEditorHighlighter.prototype.set mimeType):
  • inspector/front-end/WebKit.qrc:
  • inspector/front-end/inspector.html: LayoutTests:
  • inspector/syntax-highlight-css-expected.txt:
  • inspector/syntax-highlight-css.html:
  • inspector/syntax-highlight-javascript-expected.txt:
  • inspector/syntax-highlight-javascript.html:
  • inspector/syntax-highlight.js: (frontend_dumpSyntaxHighlight):
03:02 QtWebKitBackportingFixes edited by andreas.kling@nokia.com
(diff)
02:38 Changeset [54052] by pfeldman@chromium.org
  • 8 edits in trunk/WebCore

2010-01-29 Pavel Feldman <pfeldman@chromium.org>

Reviewed by Timothy Hatcher.

Web Inspector: No need to render background sources
when performing search.

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

01:41 Changeset [54051] by yurys@chromium.org
  • 19 edits in trunk/WebCore

2010-01-29 Yury Semikhatsky <yurys@chromium.org>

Reviewed by Adam Barth.

Remove ScriptState::frame which is only used for reporting exceptions
in some cases in a way that diverges from the regular exception reporting in
v8 bindings.

Cache ScriptState directly on v8::Context instead of providing
implementations specific for isolated worlds and ScriptController.

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

  • bindings/js/ScriptController.cpp:
  • bindings/js/ScriptController.h:
  • bindings/js/ScriptState.cpp: (WebCore::mainWorldScriptState):
  • bindings/js/ScriptState.h:
  • bindings/v8/ScriptCallStack.cpp: (WebCore::ScriptCallStack::ScriptCallStack):
  • bindings/v8/ScriptController.cpp: (WebCore::ScriptController::evaluate): (WebCore::ScriptController::clearWindowShell):
  • bindings/v8/ScriptController.h:
  • bindings/v8/ScriptScope.cpp: (WebCore::ScriptScope::ScriptScope): (WebCore::ScriptScope::success):
  • bindings/v8/ScriptScope.h:
  • bindings/v8/ScriptState.cpp: (WebCore::ScriptState::ScriptState): (WebCore::ScriptState::forContext): (WebCore::ScriptState::current): (WebCore::ScriptState::weakReferenceCallback): (WebCore::mainWorldScriptState): (WebCore::scriptStateFromPage):
  • bindings/v8/ScriptState.h: (WebCore::ScriptState::ScriptState): (WebCore::EmptyScriptState::EmptyScriptState): (WebCore::EmptyScriptState::~EmptyScriptState):
  • bindings/v8/V8HiddenPropertyName.h:
  • bindings/v8/V8IsolatedContext.cpp:
  • bindings/v8/V8IsolatedContext.h:
  • bindings/v8/V8Utilities.cpp: (WebCore::getScriptExecutionContext): (WebCore::reportException):
  • bindings/v8/custom/V8NodeIteratorCustom.cpp: (WebCore::V8NodeIterator::nextNodeCallback): (WebCore::V8NodeIterator::previousNodeCallback):
  • bindings/v8/custom/V8TreeWalkerCustom.cpp: (WebCore::V8TreeWalker::parentNodeCallback): (WebCore::V8TreeWalker::firstChildCallback): (WebCore::V8TreeWalker::lastChildCallback): (WebCore::V8TreeWalker::nextNodeCallback): (WebCore::V8TreeWalker::previousNodeCallback): (WebCore::V8TreeWalker::nextSiblingCallback): (WebCore::V8TreeWalker::previousSiblingCallback):
  • inspector/InspectorController.cpp: (WebCore::InspectorController::injectedScriptForNodeId):
01:23 Changeset [54050] by hausmann@webkit.org
  • 2 edits in trunk/JavaScriptCore

Rubber-stamped by Maciej Stachowiak.

Fix the ARM build.

  • runtime/JSNumberCell.h:

(JSC::JSNumberCell::createStructure): Call the right Structure::create overload.

00:43 Changeset [54049] by darin@chromium.org
  • 13 edits in trunk

2010-01-29 Darin Fisher <darin@chromium.org>

Reviewed by Adam Barth.

Add FrameLoaderClient::allowImages method to allow the client to
overrule image loading policy on a per frame basis.

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

For completeness, this patch also adds Settings::areImagesEnabled.
This is different from loadsImagesAutomatically as is explained in
Settings.h.

  • loader/DocLoader.cpp: (WebCore::DocLoader::requestImage):
  • loader/FrameLoaderClient.h: (WebCore::FrameLoaderClient::allowImages):
  • loader/ImageDocument.cpp: (WebCore::ImageTokenizer::writeRawData):
  • page/Settings.cpp: (WebCore::Settings::Settings): (WebCore::Settings::setImagesEnabled):
  • page/Settings.h: (WebCore::Settings::areImagesEnabled):

01/28/10:

23:42 Changeset [54048] by eric@webkit.org
  • 3 edits in trunk/WebCore

2010-01-28 Nicholas Young <nicholas.young@nokia.com>

Reviewed by Eric Carlson.

Prefer provided video element width/height properties to hard coded
defaults for intrinsic size when natural video size is unavailable.
https://bugs.webkit.org/show_bug.cgi?id=34302

No new tests needed.

  • rendering/RenderVideo.cpp: Attempt to use width/height properties (WebCore::RenderVideo::RenderVideo):
  • rendering/RenderVideo.h: More appropriate constructor signature
23:16 Changeset [54047] by oliver@apple.com
  • 5 edits in trunk

2010-01-28 Oliver Hunt <oliver@apple.com>

Reviewed by Gavin Barraclough.

MessageEvent.data should not be repeated deserialised
https://bugs.webkit.org/show_bug.cgi?id=34311

Add test to ensure we get the same value back when accessing event.data multiple times

  • fast/dom/Window/window-postmessage-clone.html:

2010-01-28 Oliver Hunt <oliver@apple.com>

Reviewed by Gavin Barraclough.

MessageEvent.data should not be repeated deserialised
https://bugs.webkit.org/show_bug.cgi?id=34311

Cache the result of deserialising the event data

  • bindings/scripts/CodeGeneratorJS.pm:
  • dom/MessageEvent.idl:
23:15 Changeset [54046] by jorlow@chromium.org
  • 3 edits in trunk/WebKit/chromium

2010-01-28 Jeremy Orlow <jorlow@chromium.org>

Reviewed by Darin Fisher.

StorageNamespaceProxy::copy() should be a no-op
https://bugs.webkit.org/show_bug.cgi?id=34298

The comment in the code should make clear why.

  • public/WebViewClient.h:
  • src/StorageNamespaceProxy.cpp: (WebCore::StorageNamespaceProxy::copy):
23:01 Changeset [54045] by eric@webkit.org
  • 5 edits
    3 adds in trunk

2010-01-28 Hayato Ito <hayato@chromium.org>

Reviewed by Darin Adler.

Change CSS 'page-break-inside' property from inherited one to non-inherited one because CSS specification defines so.

Note: Currently, the 'page-break-inside' property is used only when parsing the CSS. That property is not used when rendering.

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

  • fast/css/getComputedStyle/computed-style-page-break-inside-expected.txt: Added.
  • fast/css/getComputedStyle/computed-style-page-break-inside.html: Added.
  • fast/css/getComputedStyle/script-tests/computed-style-page-break-inside.js: Added.

2010-01-28 Hayato Ito <hayato@chromium.org>

Reviewed by Darin Adler.

Change CSS 'page-break-inside' property from inherited one to non-inherited one because CSS specification defines so.

Note: Currently, the 'page-break-inside' property is used only when parsing the CSS. That property is not used when rendering.

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

  • rendering/style/RenderStyle.h: (WebCore::InheritedFlags::NonInheritedFlags::operator==): (WebCore::InheritedFlags::setBitDefaults): (WebCore::InheritedFlags::pageBreakInside): (WebCore::InheritedFlags::setPageBreakInside):
  • rendering/style/StyleInheritedData.cpp: (WebCore::StyleInheritedData::StyleInheritedData): (WebCore::StyleInheritedData::operator==):
  • rendering/style/StyleInheritedData.h:
22:41 Changeset [54044] by eric@webkit.org
  • 7 edits
    4 adds in trunk

2010-01-28 Michael Nordman <michaeln@google.com>

Reviewed by Alexey Proskuryakov.

ApplicationCache events should be deferred until after Document onload has fired.
https://bugs.webkit.org/show_bug.cgi?id=29690

  • http/tests/appcache/deferred-events-expected.txt: Added.
  • http/tests/appcache/deferred-events.html: Added.
  • http/tests/appcache/deferred-events-delete-while-raising-expected.txt: Added.
  • http/tests/appcache/deferred-events-delete-while-raising.html: Added.

2010-01-28 Michael Nordman <michaeln@google.com>

Reviewed by Alexey Proskuryakov.

ApplicationCache events should be deferred until after Document onload has fired.
https://bugs.webkit.org/show_bug.cgi?id=29690

Test: http/tests/appcache/deferred-events.html

  • loader/FrameLoader.cpp: (WebCore::FrameLoader::handledOnloadEvents): Tells the ApplicationCacheHost to stop deferring events.
  • loader/appcache/ApplicationCacheHost.cpp: (WebCore::ApplicationCacheHost::ApplicationCacheHost): Initialize m_isDeferringEvents to true. (WebCore::ApplicationCacheHost::notifyDOMApplicationCache): Depending, defer or raise the event. (WebCore::ApplicationCacheHost::stopDeferringEvents): Raise any deferred events and reset the flag.
  • loader/appcache/ApplicationCacheHost.h: Declare new data members and method.

2010-01-28 Michael Nordman <michaeln@google.com>

Reviewed by Alexey Proskuryakov.

ApplicationCache events should be deferred until after Document onload has fired.
https://bugs.webkit.org/show_bug.cgi?id=29690

  • src/ApplicationCacheHost.cpp: (WebCore::ApplicationCacheHost::ApplicationCacheHost): Initialize m_isDeferringEvents to true. (WebCore::ApplicationCacheHost::notifyDOMApplicationCache): Depending, defer or raise the event. (WebCore::ApplicationCacheHost::stopDeferringEvents): Raise any deferred events and reset the flag.
22:11 Changeset [54043] by kevino@webkit.org
  • 5 edits in trunk

[wx] Build fix for MSW, use ThreadingWin.cpp as the Windows pthreads implementation
implements pthread_t in a way that makes it impossible to check its validity,
which is needed by ThreadingPthreads.cpp.

20:49 Changeset [54042] by eric@webkit.org
  • 41 edits
    3 deletes in trunk/WebCore

2010-01-28 Kavita Kanetkar <kkanetkar@chromium.org>

Reviewed by Dimitri Glazkov.

[V8] Generate header declaration for custom constructor callbacks
https://bugs.webkit.org/show_bug.cgi?id=33680

Added handling of 'CanBeConstructed' and 'OmitConstructor' and a new extended attribute
CustomConstructor'.

Deleted implementation .cpp files for corresponding 'CanBeConstructed'
extended attributes. These are now generated via CodeGeneratorV8.pm
Treating 'OmitConstructor' and 'CustomConstructor' to be the same in
CodeGeneratorJS.pm
Cleaned idls that had 'CustomConstructor' and 'OmitConstructor' together, and
same with CustomConstructor and CanBeConstructed.

  • Android.v8bindings.mk:
  • WebCore.gypi: Removed deps for classes being generated.
  • bindings/scripts/CodeGeneratorJS.pm: OmitConstructor and CustomConstructor have the same behavior in generator.
  • bindings/scripts/CodeGeneratorV8.pm: Generating the callbacks for constructors.
  • bindings/v8/V8DOMWrapper.cpp: Removed manual calls to SetCallHandler() for constructor callbacks. (WebCore::V8DOMWrapper::getTemplate):
  • bindings/v8/custom/V8CustomBinding.h: Removed manual declarations of constructor callbacks.
  • bindings/v8/custom/V8DOMParserConstructor.cpp:
  • bindings/v8/custom/V8EventSourceConstructor.cpp: (WebCore::V8EventSource::constructorCallback):
  • bindings/v8/custom/V8MessageChannelConstructor.cpp: (WebCore::V8MessageChannel::constructorCallback):
  • bindings/v8/custom/V8SharedWorkerCustom.cpp: (WebCore::V8SharedWorker::constructorCallback):
  • bindings/v8/custom/V8WebGLArrayBufferCustom.cpp: (WebCore::V8WebGLArrayBuffer::constructorCallback):
  • 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):
  • bindings/v8/custom/V8WebKitCSSMatrixConstructor.cpp: (WebCore::V8WebKitCSSMatrix::constructorCallback):
  • bindings/v8/custom/V8WebKitPointConstructor.cpp: (WebCore::V8WebKitPoint::constructorCallback):
  • bindings/v8/custom/V8WebSocketCustom.cpp: (WebCore::V8WebSocket::constructorCallback):
  • bindings/v8/custom/V8WorkerCustom.cpp: (WebCore::V8Worker::constructorCallback):
  • bindings/v8/custom/V8XMLHttpRequestConstructor.cpp: (WebCore::V8XMLHttpRequest::constructorCallback):
  • bindings/v8/custom/V8XMLSerializerConstructor.cpp:
  • bindings/v8/custom/V8XPathEvaluatorConstructor.cpp: Deleted the file. Implementation now being generated.
  • bindings/v8/custom/V8XSLTProcessorCustom.cpp: Deleted the file. Implementation now being generated. (WebCore::V8XSLTProcessor::constructorCallback): Deleted the file. Implementation now being generated.
  • css/WebKitCSSMatrix.idl: Added CustomConstructor attribute.
  • dom/MessageChannel.idl: Added CustomConstructor attribute.
  • html/canvas/WebGLArrayBuffer.idl: Added CustomConstructor attribute.
  • html/canvas/WebGLByteArray.idl: Added CustomConstructor attribute.
  • html/canvas/WebGLFloatArray.idl: Added CustomConstructor attribute.
  • html/canvas/WebGLIntArray.idl: Added CustomConstructor attribute.
  • html/canvas/WebGLShortArray.idl: Added CustomConstructor attribute.
  • html/canvas/WebGLUnsignedByteArray.idl: Added CustomConstructor attribute.
  • html/canvas/WebGLUnsignedIntArray.idl: Added CustomConstructor attribute.
20:27 Changeset [54041] by eric@webkit.org
  • 3 edits in trunk/WebCore

2010-01-28 Steve Block <steveblock@google.com>

Reviewed by David Levin.

Add stub for AXObjectCache::contentChanged when HAVE(ACCESSIBILITY) is not defined
https://bugs.webkit.org/show_bug.cgi?id=34271

No new tests, build fix only.

  • accessibility/AXObjectCache.cpp: Modified. Added HAVE(ACCESSIBILITY) guards around contentChanged
  • accessibility/AXObjectCache.h: Modified. (WebCore::AXObjectCache::contentChanged): Added stub
20:00 Changeset [54040] by oliver@apple.com
  • 4 edits in trunk

2010-01-28 Oliver Hunt <oliver@apple.com>

Reviewed by Gavin Barraclough.

DOM Objects shouldn't all require custom mark functions
https://bugs.webkit.org/show_bug.cgi?id=34291

Make getAnonymousValue const-friendly

  • runtime/JSObject.h: (JSC::JSObject::getAnonymousValue):

2010-01-28 Steve Block <steveblock@google.com>

Reviewed by David Levin.

Use long long rather than int64 in V8Binding.h
https://bugs.webkit.org/show_bug.cgi?id=34270

No new tests, build fix only.

  • bindings/v8/V8Binding.h: Modified. (WebCore::toInt64): Returns long long
19:47 WebKit Team edited by abecsi@webkit.org
(diff)
19:40 Changeset [54039] by abecsi@webkit.org
  • 2 edits in trunk/WebKitTools

Adding myself to the committer list. No review needed.

  • Scripts/webkitpy/committers.py:
18:57 Changeset [54038] by eric@webkit.org
  • 2 edits in trunk/WebCore

2010-01-28 Steve Block <steveblock@google.com>

Reviewed by David Levin.

Use long long rather than int64 in V8Binding.h
https://bugs.webkit.org/show_bug.cgi?id=34270

No new tests, build fix only.

  • bindings/v8/V8Binding.h: Modified. (WebCore::toInt64): Returns long long
18:41 Changeset [54037] by eric@webkit.org
  • 2 edits in trunk/WebCore

2010-01-28 Avi Drissman <avi@chromium.org>

Reviewed by David Levin.

Unused variable in ThemeMac (followup to change 49103).
https://bugs.webkit.org/show_bug.cgi?id=34274

Covered by existing tests.

  • platform/mac/ThemeMac.mm: (WebCore::setupButtonCell):
18:10 Changeset [54036] by enrica@apple.com
  • 6 edits in trunk

Huge plain text pastes are slow
<rdar://problem/5195056>
https://bugs.webkit.org/show_bug.cgi?id=34237

Reviewed by Maciej Stachowiak.

WebCore:

No new tests. It is a performance improvement.

The performance of the plain text paste is now linear.
We treat as a special case, a fragment that has been
created from plain text. Finding VisiblePositions and
applying style is greatly simplified, given the nature of
the fragment.

  • editing/ReplaceSelectionCommand.cpp:

(WebCore::ReplaceSelectionCommand::doApply):

  • editing/markup.cpp:

(WebCore::isPlainTextMarkup):

  • editing/markup.h:

WebKit/mac:

Mail is ignoring the fragment created from plain text or HTML that is
passed to the delegate function, that creates a new one.
This fix avoids creating the fragment twice.

  • WebView/WebHTMLView.mm:

(-[WebHTMLView _pasteWithPasteboard:allowPlainText:]):

17:57 Changeset [54035] by jorlow@chromium.org
  • 2 edits in trunk/WebCore

Initialize DOM Storage's quota's current length parameter when we clone it.
https://bugs.webkit.org/show_bug.cgi?id=34294

Patch by Jeremy Orlow <jorlow@chromium.org> on 2010-01-28
Reviewed by Adam Barth.

  • storage/StorageMap.cpp:

(WebCore::StorageMap::copy):

17:49 Changeset [54034] by eric@webkit.org
  • 2 edits in trunk/WebKit/qt

2010-01-28 Andreas Kling <andreas.kling@nokia.com>

Reviewed by Kenneth Rohde Christiansen.

[Qt] Support kinetic scrolling on Maemo 5

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

Patch by Ralf Engels <ralf.engels@nokia.com> and
Robert Griebl <rgriebl@trolltech.com>

  • Api/qwebview.cpp: (QWebViewKineticScroller::QWebViewKineticScroller): (QWebViewKineticScroller::eventFilter): (QWebViewKineticScroller::currentFrame): (QWebViewKineticScroller::scrollingFrameAt): (QWebViewKineticScroller::attachToWidget): (QWebViewKineticScroller::removeFromWidget): (QWebViewKineticScroller::positionRange): (QWebViewKineticScroller::position): (QWebViewKineticScroller::viewportSize): (QWebViewKineticScroller::setPosition): (QWebView::QWebView):
17:32 Changeset [54033] by eric@webkit.org
  • 2 edits in trunk/WebKitTools

2010-01-28 Hayato Ito <hayato@chromium.org>

Reviewed by David Levin.

Remove NULL char from input JS file because 'grep' fails if the file contains NULL char.

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

  • Scripts/make-script-test-wrappers:
17:28 Changeset [54032] by bfulgham@webkit.org
  • 2 edits in trunk/WebCore

Unreviewed build fix after r53411 on WinCairo.

  • platform/image-decoders/gif/GIFImageDecoder.cpp:

(WebCore::GIFImageDecoder::frameCount):

17:21 Changeset [54031] by mrowe@apple.com
  • 4 edits in branches/safari-4-branch

Versioning.

17:15 Changeset [54030] by eric@webkit.org
  • 2 edits in trunk/WebCore

2010-01-28 Nayan Kumar K <nayankk@gmail.com>

Reviewed by Laszlo Gombos.

Fix for compilation error in WML enabled build introduced because of the changeset 53514.

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

  • wml/WMLDocument.cpp: (WebCore::WMLDocument::WMLDocument):
16:57 Changeset [54029] by eric@webkit.org
  • 8 edits in trunk

2010-01-28 Alex Milowski <alex@milowski.com>

Reviewed by Eric Seidel.

Updates from CSS change for mathml.css

  • platform/mac/mathml/presentation/sub-expected.txt:
  • platform/mac/mathml/presentation/sup-expected.txt:
  • platform/mac/mathml/presentation/tables-expected.checksum:
  • platform/mac/mathml/presentation/tables-expected.png:
  • platform/mac/mathml/presentation/tables-expected.txt:

2010-01-28 Alex Milowski <alex@milowski.com>

Reviewed by Eric Seidel.

Update to CSS to remove unnecessary line-height settings

  • css/mathml.css:
16:31 Changeset [54028] by steveblock@google.com
  • 4 edits in trunk/WebCore

Add missing includes for HashMap/HashSet
https://bugs.webkit.org/show_bug.cgi?id=34276

Reviewed by David Levin.

No new tests, build fix only.

  • platform/KURL.cpp: Modified. Include HashMap.h
  • platform/network/CredentialStorage.cpp: Modified. Include HashMap.h and HashSet.h
  • platform/network/ProtectionSpaceHash.h: Modified. Include HashTraits.h
16:20 Changeset [54027] by kov@webkit.org
  • 1 edit in trunk/LayoutTests/http/tests/media/resources/setCookie.cgi

Add missing svn:executable property on LayoutTests/http/tests/media/resources/setCookie.cgi

15:56 Changeset [54026] by kov@webkit.org
  • 5 edits
    4 adds in trunk

2010-01-28 Sebastian Dröge <sebastian.droege@collabora.co.uk>

Reviewed by Gustavo Noronha.

Pass cookies to the GStreamer HTTP source
https://bugs.webkit.org/show_bug.cgi?id=34003

Test: http/tests/media/video-cookie.html

  • platform/graphics/gtk/MediaPlayerPrivateGStreamer.cpp: (WebCore::mediaPlayerPrivateSourceChangedCallback): When using an appropiate GStreamer HTTP source element, pass the cookies for the media URI to it. This fixes playback of the YouTube HTML5 videos.
15:50 Changeset [54025] by ossy@webkit.org
  • 4 edits in trunk/WebCore

[Qt] Unreviewed buildfix after r54022.

  • bridge/qt/qt_instance.cpp:

(JSC::Bindings::QtRuntimeObjectImp::createStructure):

  • bridge/qt/qt_pixmapruntime.cpp:

(JSC::Bindings::QtPixmapRuntimeObjectImp::createStructure):

  • bridge/qt/qt_runtime.h:

(JSC::Bindings::QtRuntimeMethod::createStructure):

15:19 Changeset [54024] by ossy@webkit.org
  • 2 edits in trunk/LayoutTests

[Qt] Incorrect small caps vs. normal text handling
https://bugs.webkit.org/show_bug.cgi?id=34286

  • platform/qt/Skipped: css2.1/t1505-c524-font-var-00-b.html skipped
15:03 Changeset [54023] by aroben@apple.com
  • 2 edits in trunk/WebCore

Fix an assertion beneath SerializedScriptValue::deserialize on 32-bit builds (e.g., on Windows)

Passing a JSValue to toRef on 32-bit builds can perform an allocation,
so we need to make sure we hold a JSLock when that occurs.

Reviewed by Oliver Hunt.

  • bindings/js/SerializedScriptValue.cpp:

(WebCore::SerializedScriptValueData::serialize):
(WebCore::SerializedScriptValueData::deserialize):
Moved JSLocks from here...

(WebCore::SerializedScriptValue::create):
(WebCore::SerializedScriptValue::deserialize):
...to this higher level, from which toRef can be called.

14:51 Changeset [54022] by oliver@apple.com
  • 49 edits in trunk

2010-01-28 Oliver Hunt <oliver@apple.com>

Reviewed by Gavin Barraclough.

Simplify anonymous slot implementation
https://bugs.webkit.org/show_bug.cgi?id=34282

A class must now specify the number of slots it needs at construction time
rather than later on with a transition. This makes many things simpler,
we no longer need to need an additional transition on object creation to
add the anonymous slots, and we remove the need for a number of transition
type checks.

  • API/JSCallbackConstructor.h: (JSC::JSCallbackConstructor::createStructure):
  • API/JSCallbackFunction.h: (JSC::JSCallbackFunction::createStructure):
  • API/JSCallbackObject.h: (JSC::JSCallbackObject::createStructure):
  • JavaScriptCore.exp:
  • debugger/DebuggerActivation.h: (JSC::DebuggerActivation::createStructure):
  • runtime/Arguments.h: (JSC::Arguments::createStructure):
  • runtime/BooleanObject.h: (JSC::BooleanObject::createStructure):
  • runtime/DateInstance.h: (JSC::DateInstance::createStructure):
  • runtime/DatePrototype.h: (JSC::DatePrototype::createStructure):
  • runtime/FunctionPrototype.h: (JSC::FunctionPrototype::createStructure):
  • runtime/GetterSetter.h: (JSC::GetterSetter::createStructure):
  • runtime/GlobalEvalFunction.h: (JSC::GlobalEvalFunction::createStructure):
  • runtime/InternalFunction.h: (JSC::InternalFunction::createStructure):
  • runtime/JSAPIValueWrapper.h: (JSC::JSAPIValueWrapper::createStructure):
  • runtime/JSActivation.h: (JSC::JSActivation::createStructure):
  • runtime/JSArray.h: (JSC::JSArray::createStructure):
  • runtime/JSByteArray.cpp: (JSC::JSByteArray::createStructure):
  • runtime/JSCell.h: (JSC::JSCell::createDummyStructure):
  • runtime/JSFunction.h: (JSC::JSFunction::createStructure):
  • runtime/JSGlobalObject.h: (JSC::JSGlobalObject::createStructure):
  • runtime/JSNotAnObject.h: (JSC::JSNotAnObject::createStructure):
  • runtime/JSONObject.h: (JSC::JSONObject::createStructure):
  • runtime/JSObject.h: (JSC::JSObject::createStructure): (JSC::JSObject::putAnonymousValue): (JSC::JSObject::getAnonymousValue):
  • runtime/JSPropertyNameIterator.h: (JSC::JSPropertyNameIterator::createStructure):
  • runtime/JSStaticScopeObject.h: (JSC::JSStaticScopeObject::createStructure):
  • runtime/JSString.h: (JSC::Fiber::createStructure):
  • runtime/JSVariableObject.h: (JSC::JSVariableObject::createStructure):
  • runtime/JSWrapperObject.h: (JSC::JSWrapperObject::createStructure): (JSC::JSWrapperObject::JSWrapperObject):
  • runtime/MathObject.h: (JSC::MathObject::createStructure):
  • runtime/NumberConstructor.h: (JSC::NumberConstructor::createStructure):
  • runtime/NumberObject.h: (JSC::NumberObject::createStructure):
  • runtime/RegExpConstructor.h: (JSC::RegExpConstructor::createStructure):
  • runtime/RegExpObject.h: (JSC::RegExpObject::createStructure):
  • runtime/StringObject.h: (JSC::StringObject::createStructure):
  • runtime/StringObjectThatMasqueradesAsUndefined.h: (JSC::StringObjectThatMasqueradesAsUndefined::createStructure):
  • runtime/Structure.cpp: (JSC::Structure::~Structure): (JSC::Structure::materializePropertyMap):
  • runtime/Structure.h: (JSC::Structure::create): (JSC::Structure::anonymousSlotCount):
  • runtime/StructureTransitionTable.h:

2010-01-28 Oliver Hunt <oliver@apple.com>

Reviewed by Gavin Barraclough.

Simplify anonymous slot implementation
https://bugs.webkit.org/show_bug.cgi?id=34282

Update JSGlue Structure usage to pass the anonymous slot count.

  • UserObjectImp.h: (UserObjectImp::createStructure):

2010-01-28 Oliver Hunt <oliver@apple.com>

Reviewed by Gavin Barraclough.

Simplify anonymous slot implementation
https://bugs.webkit.org/show_bug.cgi?id=34282

Update the WebCore JS DOM bindings to correctly pass and
propagate the anonymous slot count information.

  • bindings/js/JSDOMBinding.h: (WebCore::DOMObjectWithGlobalPointer::createStructure): (WebCore::DOMConstructorObject::createStructure):
  • bindings/js/JSDOMWindowShell.h: (WebCore::JSDOMWindowShell::createStructure):
  • bindings/scripts/CodeGeneratorJS.pm:
  • bridge/objc/objc_runtime.h: (JSC::Bindings::ObjcFallbackObjectImp::createStructure):
  • bridge/runtime_array.h: (JSC::RuntimeArray::createStructure):
  • bridge/runtime_method.h: (JSC::RuntimeMethod::createStructure):
  • bridge/runtime_object.h: (JSC::RuntimeObjectImp::createStructure):
14:37 QtWebKitBackportingFixes edited by laszlo.1.gombos@nokia.com
(diff)
14:26 Changeset [54021] by kevino@webkit.org
  • 2 edits in trunk/WebKitTools

[wx] Build fix after removal of the zlib image decoder.

14:17 Changeset [54020] by evan@chromium.org
  • 4 edits
    4 adds in trunk

2010-01-27 Evan Martin <evan@chromium.org>

Reviewed by David Levin.

[chromium] hebrew vowel marks incorrectly positioned
https://bugs.webkit.org/show_bug.cgi?id=34234

Add a layout test containing some Hebrew vowels.

  • fast/text/international/hebrew-vowels.html: Added.

2010-01-27 Evan Martin <evan@chromium.org>

Reviewed by David Levin.

[chromium] hebrew vowel marks incorrectly positioned
https://bugs.webkit.org/show_bug.cgi?id=34234

Fix glyph metrics computation such that the glyph offsets
are correct, so we can now make use of them.

Test: fast/text/international/hebrew-vowels.html

  • platform/graphics/chromium/FontLinux.cpp: (WebCore::TextRunWalker::createGlyphArrays): initialize per-glyph offset table. (WebCore::TextRunWalker::setGlyphXPositions): use per-glyph offsets.
  • platform/graphics/chromium/HarfbuzzSkia.cpp: (WebCore::getGlyphMetrics): fix metrics computation such that per-glyph offsets are computed correctly.
14:08 Changeset [54019] by laszlo.1.gombos@nokia.com
  • 2 edits in trunk/WebCore

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

Unreviewed build fix.

[Qt] Build fix if websockets support is turned off after r53947.

  • WebCore.pro:
13:47 Changeset [54018] by mrowe@apple.com
  • 4 edits in branches/safari-4-branch

Versioning.

13:46 Changeset [54017] by mrowe@apple.com
  • 1 copy in tags/Safari-6531.22.2

New tag.

13:44 Changeset [54016] by dglazkov@chromium.org
  • 19 edits in trunk/WebCore

2010-01-28 Dimitri Glazkov <dglazkov@chromium.org>

No review, rolling out r54012.
http://trac.webkit.org/changeset/54012
https://bugs.webkit.org/show_bug.cgi?id=34266

Introduced 33 crashes in V8 traversal/ tests.

  • bindings/js/ScriptController.cpp: (WebCore::ScriptController::mainWorldScriptState):
  • bindings/js/ScriptController.h:
  • bindings/js/ScriptState.cpp:
  • bindings/js/ScriptState.h:
  • bindings/v8/ScriptCallStack.cpp: (WebCore::ScriptCallStack::ScriptCallStack):
  • bindings/v8/ScriptController.cpp: (WebCore::ScriptController::mainWorldScriptState): (WebCore::ScriptController::currentScriptState): (WebCore::ScriptController::clearWindowShell):
  • bindings/v8/ScriptController.h:
  • bindings/v8/ScriptScope.cpp: (WebCore::ScriptScope::ScriptScope): (WebCore::ScriptScope::success):
  • bindings/v8/ScriptScope.h:
  • bindings/v8/ScriptState.cpp: (WebCore::ScriptState::ScriptState): (WebCore::scriptStateFromPage):
  • bindings/v8/ScriptState.h: (WebCore::ScriptState::ScriptState): (WebCore::ScriptState::frame):
  • bindings/v8/V8HiddenPropertyName.h:
  • bindings/v8/V8IsolatedContext.cpp: (WebCore::V8IsolatedContext::scriptState):
  • bindings/v8/V8IsolatedContext.h:
  • bindings/v8/V8Utilities.cpp: (WebCore::getScriptExecutionContext): (WebCore::reportException):
  • bindings/v8/custom/V8NodeIteratorCustom.cpp: (WebCore::V8NodeIterator::nextNodeCallback): (WebCore::V8NodeIterator::previousNodeCallback):
  • bindings/v8/custom/V8TreeWalkerCustom.cpp: (WebCore::V8TreeWalker::parentNodeCallback): (WebCore::V8TreeWalker::firstChildCallback): (WebCore::V8TreeWalker::lastChildCallback): (WebCore::V8TreeWalker::nextNodeCallback): (WebCore::V8TreeWalker::previousNodeCallback): (WebCore::V8TreeWalker::nextSiblingCallback): (WebCore::V8TreeWalker::previousSiblingCallback):
  • inspector/InspectorController.cpp: (WebCore::InspectorController::injectedScriptForNodeId):
13:36 Changeset [54015] by aa@chromium.org
  • 12 edits in trunk

Reapply 54008: Expand the NotificationPresenter::checkPermission() interface
to send the full URL of the requesting context, as well as a pointer to the
document if that context was a document.

13:34 Changeset [54014] by eric@webkit.org
  • 4 edits in trunk/WebCore

2010-01-28 Steve Block <steveblock@google.com>

Reviewed by Adam Barth.

Fix missing forward-declarations and includes in V8 bindings
https://bugs.webkit.org/show_bug.cgi?id=34272

No new tests, build fix only.

  • bindings/v8/ScriptController.h: Modified. Forward-declare NPObject
  • bindings/v8/V8DOMWindowShell.cpp: Modified. Include CString.h and PlatformBridge.h
  • bindings/v8/V8DOMWrapper.h: Modified. Forward-declare V8Proxy
13:24 Changeset [54013] by dglazkov@chromium.org
  • 3 edits in trunk/LayoutTests

2010-01-28 Dimitri Glazkov <dglazkov@chromium.org>

Unreviewed, trivial baseline update.

Update baseline for "Animated scaling of background-image is too slow"
https://bugs.webkit.org/show_bug.cgi?id=33808

  • platform/mac/fast/backgrounds/size/backgroundSize16-expected.checksum:
  • platform/mac/fast/backgrounds/size/backgroundSize16-expected.png:
13:05 Changeset [54012] by yurys@chromium.org
  • 19 edits in trunk/WebCore

2010-01-28 Yury Semikhatsky <yurys@chromium.org>

Reviewed by Adam Barth.

Remove ScriptState::frame which is only used for reporting exceptions
in some cases in a way that diverges from the regular exception reporting in
v8 bindings.


Cache ScriptState directly on v8::Context instead of providing
implementations specific for isolated worlds and ScriptController.

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

  • bindings/js/ScriptController.cpp:
  • bindings/js/ScriptController.h:
  • bindings/js/ScriptState.cpp: (WebCore::mainWorldScriptState):
  • bindings/js/ScriptState.h:
  • bindings/v8/ScriptCallStack.cpp: (WebCore::ScriptCallStack::ScriptCallStack):
  • bindings/v8/ScriptController.cpp: (WebCore::ScriptController::clearWindowShell):
  • bindings/v8/ScriptController.h:
  • bindings/v8/ScriptScope.cpp: (WebCore::ScriptScope::ScriptScope): (WebCore::ScriptScope::success):
  • bindings/v8/ScriptScope.h:
  • bindings/v8/ScriptState.cpp: (WebCore::ScriptState::ScriptState): (WebCore::ScriptState::forContext): (WebCore::ScriptState::current): (WebCore::ScriptState::empty): (WebCore::ScriptState::weakReferenceCallback): (WebCore::mainWorldScriptState): (WebCore::scriptStateFromPage):
  • bindings/v8/ScriptState.h:
  • bindings/v8/V8HiddenPropertyName.h:
  • bindings/v8/V8IsolatedContext.cpp:
  • bindings/v8/V8IsolatedContext.h:
  • bindings/v8/V8Utilities.cpp: (WebCore::getScriptExecutionContext): (WebCore::reportException):
  • bindings/v8/custom/V8NodeIteratorCustom.cpp: (WebCore::V8NodeIterator::nextNodeCallback): (WebCore::V8NodeIterator::previousNodeCallback):
  • bindings/v8/custom/V8TreeWalkerCustom.cpp: (WebCore::V8TreeWalker::parentNodeCallback): (WebCore::V8TreeWalker::firstChildCallback): (WebCore::V8TreeWalker::lastChildCallback): (WebCore::V8TreeWalker::nextNodeCallback): (WebCore::V8TreeWalker::previousNodeCallback): (WebCore::V8TreeWalker::nextSiblingCallback): (WebCore::V8TreeWalker::previousSiblingCallback):
  • inspector/InspectorController.cpp: (WebCore::InspectorController::injectedScriptForNodeId):
12:38 Changeset [54011] by aa@chromium.org
  • 12 edits in trunk

revert

12:34 Changeset [54010] by abarth@webkit.org
  • 3 edits
    3 adds in trunk

2010-01-28 Adam Barth <abarth@webkit.org>

Reviewed by David Levin.

Remove XSSAuditor false positive for Google Translate
https://bugs.webkit.org/show_bug.cgi?id=34242

Add a test that we allow attackers to inject directly into the href
property of the base tag.

  • http/tests/security/xssAuditor/base-href-direct-expected.txt: Added.
  • http/tests/security/xssAuditor/base-href-direct.html: Added.
  • http/tests/security/xssAuditor/resources/echo-head-base-href-direct.pl: Added.

2010-01-28 Adam Barth <abarth@webkit.org>

Reviewed by David Levin.

Remove XSSAuditor false positive for Google Translate
https://bugs.webkit.org/show_bug.cgi?id=34242

Google translate takes a base URL as a parameter, causing a false
positive in the XSS filter. This patch removes the false positive by
allowing direct injections into the href property of the base tag.

Test: http/tests/security/xssAuditor/base-href-direct.html

  • page/XSSAuditor.cpp: (WebCore::XSSAuditor::canSetBaseElementURL):
12:27 Changeset [54009] by zimmermann@webkit.org
  • 17 edits
    1 add in trunk/WebCore

2010-01-28 Nikolas Zimmermann <nzimmermann@rim.com>

Reviewed by Dirk Schulze.

Save additional memory in SVG*Element classes, by introducing SVGElementRareData
https://bugs.webkit.org/show_bug.cgi?id=34268

Introduce SVGElementRareData, to remove the need to store any members in SVGElement/SVGStyledElement.
This saves some additional MB for the svg-node-count-vs-scroll.xhtml testcase.

SVGElementRareData works just like NodeRareData. Unlike ElementRareData, we're not inheriting from
NodeRareData in SVG, as we don't need all the rare data stored there, unlike Element.

  • GNUmakefile.am: Add SVGElementRareData.h to build.
  • WebCore.gypi: Ditto.
  • WebCore.pro: Ditto.
  • WebCore.vcproj/WebCore.vcproj: Ditto.
  • WebCore.xcodeproj/project.pbxproj: Ditto.
  • dom/Node.cpp: Use Nodes bitfield to keep track wheter a SVGElement has a SVGElementRareData object associated. (WebCore::Node::Node):
  • dom/Node.h: Add m_hasRareSVGData flag, merged with the existing bitfield, 10 bits remaining now. (WebCore::Node::hasRareSVGData):
  • svg/SVGAnimateMotionElement.cpp: SVGElement::instancesForElement() now returns a const-reference to the HashMap instead of copying. (WebCore::SVGAnimateMotionElement::applyResultsToTarget):
  • svg/SVGAnimateTransformElement.cpp: Ditto. (WebCore::SVGAnimateTransformElement::applyResultsToTarget):
  • svg/SVGAnimationElement.cpp: Ditto. (WebCore::SVGAnimationElement::setTargetAttributeAnimatedValue):
  • svg/SVGElement.cpp: Move the element instances HashSet and two cursor-related pointers into SVGElementRareData. (WebCore::SVGElement::SVGElement): (WebCore::SVGElement::~SVGElement): (WebCore::SVGElement::rareSVGData): Modelled just like Node::rareData(). (WebCore::SVGElement::ensureRareSVGData): Dito. (WebCore::SVGElement::accessDocumentSVGExtensions): (WebCore::SVGElement::mapInstanceToElement): Route call through SVGElementRareData. (WebCore::SVGElement::removeInstanceMapping): Ditto. (WebCore::SVGElement::instancesForElement): Ditto. (WebCore::SVGElement::setCursorElement): Ditto. (WebCore::SVGElement::setCursorImageValue): Ditto.
  • svg/SVGElement.h:
  • svg/SVGElementInstance.cpp: Ditto. (WebCore::SVGElementInstance::invalidateAllInstancesOfElement):
  • svg/SVGElementRareData.h: Added. Based on NodeRareData - works exactly the same. (WebCore::SVGElementRareData::SVGElementRareData): (WebCore::SVGElementRareData::rareDataMap): (WebCore::SVGElementRareData::rareDataFromMap): (WebCore::SVGElementRareData::elementInstances): (WebCore::SVGElementRareData::instanceUpdatesBlocked): (WebCore::SVGElementRareData::setInstanceUpdatesBlocked): (WebCore::SVGElementRareData::cursorElement): (WebCore::SVGElementRareData::setCursorElement): (WebCore::SVGElementRareData::cursorImageValue): (WebCore::SVGElementRareData::setCursorImageValue):
  • svg/SVGStyledElement.cpp: Move m_instancesUpdatesBlocked into SVGElementRareData. (WebCore::SVGStyledElement::SVGStyledElement): (WebCore::SVGStyledElement::instanceUpdatesBlocked): (WebCore::SVGStyledElement::setInstanceUpdatesBlocked):
  • svg/SVGStyledElement.h:
  • svg/SVGUseElement.cpp: Adapt to element instances changes, now passed as const-reference. (WebCore::dumpInstanceTree):
11:24 Changeset [54008] by aa@chromium.org
  • 13 edits in trunk

2010-01-28 Aaron Boodman <aa@chromium.org>

Reviewed by Darin Adler.

Expand the NotificationPresenter::checkPermission() interface to send the full
URL of the requesting context, as well as a pointer to the document if that
context was a document.

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

  • notifications/Notification.cpp: (WebCore::Notification::Notification):
  • notifications/NotificationCenter.cpp: (WebCore::NotificationCenter::NotificationCenter): (WebCore::NotificationCenter::checkPermission): Pass the full URL and document from the requesting context when calling NotificationPresenter::checkPermission().
  • notifications/NotificationPresenter.h: (WebCore::NotificationPresenter::): Expand interface.

2010-01-27 Aaron Boodman <aa@chromium.org>

Reviewed by Darin Adler.

Send full URL and application id of requesting context to Chromium
when checking notification permissions.

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

  • public/WebDocument.h:
  • src/WebDocument.cpp: (WebKit::WebDocument::applicationID): Implement applicationID() method.
  • public/WebNotificationPresenter.h:
  • src/NotificationPresenterImpl.cpp: (WebKit::NotificationPresenterImpl::checkPermission):
  • src/NotificationPresenterImpl.h: Send applicationID and full URL through to Chromium.

2010-01-27 Aaron Boodman <aa@chromium.org>

Expand NotificationCenter::checkPermission() interface.
It now passes the full URL instead of just the origin.

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

  • WebCoreSupport/WebDesktopNotificationsDelegate.cpp: (WebDesktopNotificationsDelegate::checkPermission):
  • WebCoreSupport/WebDesktopNotificationsDelegate.h:
10:34 Changeset [54007] by jam@chromium.org
  • 2 edits in trunk/LayoutTests/fast/dom/Window

Unreviewed, trivial expectations update.

10:08 QtWebKitBackportingFixes edited by laszlo.1.gombos@nokia.com
(diff)
10:07 Changeset [54006] by simon.fraser@apple.com
  • 6 edits
    4 adds in trunk

2010-01-28 Simon Fraser <simon.fraser@apple.com>

Reviewed by Dan Bernstein.

Video can overlap position:fixed element when scrolling
https://bugs.webkit.org/show_bug.cgi?id=32180

When the scroll position changes, and we're using overlap to decide what gets composited,
then we need to re-evaluate what gets compositing when scrolling in case fixed postion elements
overlap composited elements (e.g. video).

Test: compositing/geometry/video-fixed-scrolling.html

  • page/FrameView.cpp: (WebCore::FrameView::updateCompositingLayers): (WebCore::FrameView::scrollPositionChanged): Annotate the calls to updateCompositingLayers() with the type of change that occurred.
  • rendering/RenderLayer.cpp: (WebCore::RenderLayer::updateCompositingAndLayerListsIfNeeded): Ditto.
  • rendering/RenderLayerCompositor.h:
  • rendering/RenderLayerCompositor.cpp: (WebCore::RenderLayerCompositor::updateCompositingLayers): For scroll changes, check for hierarchy update if m_compositingConsultsOverlap is true, and do a geometry update. For other changes, always check for hierarchy updates.
09:47 Changeset [54005] by hyatt@apple.com
  • 7 edits
    2 adds in trunk

Fix for user stylesheet bugs where they get parsed using the wrong compatibility mode. This was a regression
caused from Safari 3 -> 4 by the Acid3 fixes to create a real DOCTYPE in the DOM.

Reviewed by Darin Adler.

This patch makes style selector creation lazy, so that the selector isn't built unless someone asks for it.

In order to avoid creating the style selector before the compatibility mode has been determined, document
style is now created without having a dependence on an instantiated style selector. The creation of the style
is now done as a static method on CSSStyleSelector called styleForDocument, and a couple of font-related
functions have been made static as well so that they can be used by this method.

m_styleSelector on Document is now an OwnPtr.

Added userscripts/mixed-case-stylesheet.html

  • css/CSSStyleSelector.cpp:

(WebCore::CSSStyleSelector::styleForDocument):
(WebCore::CSSStyleSelector::applyProperty):
(WebCore::CSSStyleSelector::checkForGenericFamilyChange):
(WebCore::CSSStyleSelector::setFontSize):
(WebCore::CSSStyleSelector::getComputedSizeFromSpecifiedSize):
(WebCore::CSSStyleSelector::fontSizeForKeyword):

  • css/CSSStyleSelector.h:
  • dom/Document.cpp:

(WebCore::Document::Document):
(WebCore::Document::~Document):
(WebCore::Document::recalcStyle):
(WebCore::Document::createStyleSelector):
(WebCore::Document::attach):
(WebCore::Document::setVisuallyOrdered):
(WebCore::Document::recalcStyleSelector):

  • dom/Document.h:

(WebCore::Document::styleSelector):
(WebCore::Document::visuallyOrdered):

  • html/HTMLDocument.cpp:

(WebCore::HTMLDocument::determineParseMode):

  • loader/PlaceholderDocument.cpp:

(WebCore::PlaceholderDocument::attach):

09:03 Changeset [54004] by eric@webkit.org
  • 1 edit
    2 adds in trunk/LayoutTests

2010-01-28 Anton Muhin <antonm@google.com>

Reviewed by Alexey Proskuryakov.

Add layout tests to verify assignment to items of NodeList
https://bugs.webkit.org/show_bug.cgi?id=34213

  • fast/dom/NodeList/nodelist-item-assignment-expected.txt: Added.
  • fast/dom/NodeList/nodelist-item-assignment.html: Added.
08:47 Changeset [54003] by eric@webkit.org
  • 9 edits
    2 moves in trunk/WebCore

2010-01-28 Kent Tamura <tkent@chromium.org>

Reviewed by Darin Adler.

Rename ISODateTime.{cpp,h} to DateComponents.{cpp,h}
https://bugs.webkit.org/show_bug.cgi?id=34245

No functional changes.

  • GNUmakefile.am:
  • WebCore.gypi:
  • WebCore.pro:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • html/DateComponents.cpp: Copied from WebCore/html/ISODateTime.cpp. (WebCore::DateComponents::maxWeekNumberInYear): (WebCore::DateComponents::parseYear): (WebCore::DateComponents::addDay): (WebCore::DateComponents::addMinute): (WebCore::DateComponents::parseTimeZone): (WebCore::DateComponents::parseMonth): (WebCore::DateComponents::parseDate): (WebCore::DateComponents::parseWeek): (WebCore::DateComponents::parseTime): (WebCore::DateComponents::parseDateTimeLocal): (WebCore::DateComponents::parseDateTime): (WebCore::DateComponents::setMillisecondsSinceMidnightInternal): (WebCore::DateComponents::setMillisecondsSinceEpochForDateInternal): (WebCore::DateComponents::setMillisecondsSinceEpochForDate): (WebCore::DateComponents::setMillisecondsSinceEpochForDateTime): (WebCore::DateComponents::setMillisecondsSinceEpochForDateTimeLocal): (WebCore::DateComponents::setMillisecondsSinceEpochForMonth): (WebCore::DateComponents::setMillisecondsSinceMidnight): (WebCore::DateComponents::setMillisecondsSinceEpochForWeek): (WebCore::DateComponents::millisecondsSinceEpochForTime): (WebCore::DateComponents::millisecondsSinceEpoch): (WebCore::DateComponents::toStringForTime): (WebCore::DateComponents::toString):
  • html/DateComponents.h: Copied from WebCore/html/ISODateTime.h. (WebCore::DateComponents::DateComponents):
  • html/HTMLInputElement.cpp: (WebCore::HTMLInputElement::parseToDouble): (WebCore::HTMLInputElement::valueAsDate): (WebCore::HTMLInputElement::setValueAsDate): (WebCore::HTMLInputElement::setValueAsNumber): (WebCore::HTMLInputElement::formStringToDateComponents):
  • html/HTMLInputElement.h:
  • html/ISODateTime.cpp: Removed.
  • html/ISODateTime.h: Removed.
  • html/ValidityState.cpp: (WebCore::ValidityState::typeMismatch):
08:44 Changeset [54002] by ossy@webkit.org
  • 5 edits in trunk/WebKitTools

[Qt] Unreviewed, roll out r54000.

  • DumpRenderTree/qt/DumpRenderTreeQt.cpp:

(WebCore::DumpRenderTree::DumpRenderTree):

  • DumpRenderTree/qt/DumpRenderTreeQt.h:
  • DumpRenderTree/qt/LayoutTestControllerQt.cpp:

(LayoutTestController::reset):

  • DumpRenderTree/qt/LayoutTestControllerQt.h:

(LayoutTestController::display):

08:28 Changeset [54001] by eric@webkit.org
  • 9 edits in trunk/WebCore

2010-01-28 Alexander Pavlov <apavlov@chromium.org>

Reviewed by Pavel Feldman.

Use platform-specific monospace fonts in the Web Inspector

Move the OS version detection code into the frontend,
add custom CSS monospace styles for known platforms.
https://bugs.webkit.org/show_bug.cgi?id=34040

  • inspector/InspectorFrontendHost.cpp: (WebCore::InspectorFrontendHost::platform):
  • inspector/front-end/BreakpointsSidebarPane.js: (WebInspector.BreakpointsSidebarPane.prototype._appendBreakpointElement):
  • inspector/front-end/DatabaseQueryView.js: (WebInspector.DatabaseQueryView):
  • inspector/front-end/InspectorBackendStub.js: (.WebInspector.InspectorBackendStub.prototype.releaseWrapperObjectGroup): (.WebInspector.InspectorBackendStub.prototype.setInjectedScriptSource):
  • inspector/front-end/SourceFrame.js: (WebInspector.SourceFrame.prototype._createConditionElement):
  • inspector/front-end/inspector.css:
  • inspector/front-end/inspector.html:
  • inspector/front-end/inspector.js: (WebInspector.URLRegExp.i.get platform): (WebInspector._detectPlatform):
06:41 Changeset [54000] by eric@webkit.org
  • 5 edits in trunk/WebKitTools

2010-01-28 Benjamin Poulain <benjamin.poulain@nokia.com>

Reviewed by Kenneth Rohde Christiansen.

[Qt] Implement the display() method of the layout test controller
https://bugs.webkit.org/show_bug.cgi?id=34258

  • DumpRenderTree/qt/DumpRenderTreeQt.cpp: (WebCore::DumpRenderTree::DumpRenderTree): (WebCore::DumpRenderTree::showPage): (WebCore::DumpRenderTree::hidePage):
  • DumpRenderTree/qt/DumpRenderTreeQt.h:
  • DumpRenderTree/qt/LayoutTestControllerQt.cpp: (LayoutTestController::reset): (LayoutTestController::display):
  • DumpRenderTree/qt/LayoutTestControllerQt.h:
05:39 Changeset [53999] by kenneth@webkit.org
  • 2 edits in trunk/WebKit/qt

Do not set the combobox font on Maemo5 and S60; use the
default instead.

Reviewed by Simon Hausmann.

  • WebCoreSupport/QtFallbackWebPopup.cpp:

(WebCore::QtFallbackWebPopup::populate):

05:22 Changeset [53998] by cjerdonek@webkit.org
  • 3 edits in trunk/WebKitTools

Eliminated the filename parameter from functions in
check-webkit-style's cpp.py where it is no longer used.

Reviewed by Shinichiro Hamaji.

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

  • Scripts/webkitpy/style/processors/cpp.py:
    • Reduced number of occurrences of "filename" variable from approximately 200 to 120.
  • Scripts/webkitpy/style/processors/cpp_unittest.py:
    • Refactored unit tests as necessary to accommodate changes to cpp.py.
    • Fixed bug in CppStyleTestBase.perform_include_what_you_use() where the incorrect file extension was getting passed to cpp_style.check_language().
05:06 Changeset [53997] by hausmann@webkit.org
  • 2 edits in trunk/WebKit/qt

[Qt] Fix for endless print loop when printing web pages

Patch by Trond Kjernåsen <trond@trolltech.com> on 2010-01-28
Reviewed by Simon Hausmann.

  • Api/qwebframe.cpp:

(QWebFrame::print):

04:34 Changeset [53996] by eric@webkit.org
  • 4 edits in trunk/WebCore

2010-01-28 Mikhail Naganov <mnaganov@chromium.org>

Reviewed by Timothy Hatcher.

Simplify views logic in Profiler panel, and fix the problem described in the issue.
Remove image from Welcome view.
Preserve border on status bar-like buttons in Welcome view on window deactivation.

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

  • inspector/front-end/ProfilesPanel.js: (WebInspector.ProfilesPanel.prototype.show): (WebInspector.ProfilesPanel.prototype.reset): (WebInspector.ProfilesPanel.prototype.addProfileHeader): (WebInspector.ProfilesPanel.prototype.showView): (WebInspector.ProfilesPanel.prototype._updateInterface):
  • inspector/front-end/WelcomeView.js: (WebInspector.WelcomeView):
  • inspector/front-end/inspector.css:
03:54 Changeset [53995] by zecke@webkit.org
  • 2 edits in trunk/WebCore

[Qt] Speed up KURL to QUrl conversion

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

The WebCore::String::utf8 method will use the generic WebCore::TextCodec
and then call the encode method on it. In QtWebKit this class is implemented
around the QTextCodec class. Instead of going the generic codec way we treat
the WebCore::String as a QString (no copying) and then use the built-in
QString::toUtf8 to do the conversion.

03:00 Changeset [53994] by benm@google.com
  • 12 edits in trunk

[Android] [Qt] Touch event page co-ordinates are incorrect when touch is received in an iframe.
https://bugs.webkit.org/show_bug.cgi?id=34162

Reviewed by Simon Hausmann.

WebCore:

The pageX/pageY co-ordinates of a touch event should be relative to the containing frame, not the main frame. This change implements that and also updates the existing touch-inside-iframe test to also examine the page co-ordinates.

  • dom/Touch.cpp:

(WebCore::Touch::Touch): Remove dead code.

  • dom/Touch.h: Remove dead code.
  • page/EventHandler.cpp:

(WebCore::EventHandler::handleTouchEvent): Rename framePoint to pagePoint, as it seems a clearer name. Also adjust the pagePoint to be relative to the touch target element's containing frame rather than the main frame.

  • platform/PlatformTouchEvent.h: Rename Android specific constructor parameter to better reflect it's contents.
  • platform/PlatformTouchPoint.h: Ditto.
  • platform/android/PlatformTouchEventAndroid.cpp: Ditto.
  • platform/android/PlatformTouchPointAndroid.cpp: Ditto.

LayoutTests:

Update the existing iframe touch event test to also examine the page co-ordinates of the touch that is received.

  • fast/events/touch/resources/touch-inside-iframe2.html:
  • fast/events/touch/touch-inside-iframe-expected.txt:
  • fast/events/touch/touch-inside-iframe.html:
02:46 Changeset [53993] by eric@webkit.org
  • 2 edits in trunk/WebCore

2010-01-28 Yury Semikhatsky <yurys@chromium.org>

Reviewed by Pavel Feldman.

Simplify isActivation method.

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

Test: WebCore/manual-tests/inspector/debugger-scopes-inspection.html

  • bindings/js/JSInjectedScriptHostCustom.cpp: (WebCore::JSInjectedScriptHost::isActivation):
01:20 Changeset [53992] by xan@webkit.org
  • 2 edits in trunk/LayoutTests

2010-01-28 Xan Lopez <xlopez@igalia.com>

[GTK] Can DnD files to our filechooser widget (<input type="file">)
https://bugs.webkit.org/show_bug.cgi?id=34246

Skip failing test because of this missing feature.

  • platform/gtk/Skipped:
01:10 Changeset [53991] by tkent@chromium.org
  • 9 edits
    1 delete in trunk/LayoutTests

2010-01-28 Kent Tamura <tkent@chromium.org>

Reviewed by Darin Fisher.

Remove tests for dates later than 275760-09-13T00:00Z.
https://bugs.webkit.org/show_bug.cgi?id=34240

Date.UTC() of V8 doesn't support such dates though Date.UTC() of
JavaScriptCore does.

  • fast/forms/input-valueasnumber-date-expected.txt:
  • fast/forms/input-valueasnumber-datetime-expected.txt:
  • fast/forms/input-valueasnumber-datetimelocal-expected.txt:
  • fast/forms/input-valueasnumber-month-expected.txt:
  • fast/forms/script-tests/input-valueasnumber-date.js:
  • fast/forms/script-tests/input-valueasnumber-datetime.js:
  • fast/forms/script-tests/input-valueasnumber-datetimelocal.js:
  • fast/forms/script-tests/input-valueasnumber-month.js:
  • platform/win/fast/forms/input-valueasnumber-datetime-expected.txt: Removed.
00:45 Changeset [53990] by eric@webkit.org
  • 5 edits in trunk/WebKitTools

2010-01-28 Chris Jerdonek <cjerdonek@webkit.org>

Reviewed by Shinichiro Hamaji.

In check-webkit-style, eliminated the dependency of
processors/cpp_unittest.py on checker.py.

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

  • Scripts/webkitpy/style/checker.py:
    • Addressed FIXME by removing STYLE_CATEGORIES data.
    • Added style_categories().
  • Scripts/webkitpy/style/checker_unittest.py:
    • Minor changes.
  • Scripts/webkitpy/style/processors/cpp.py:
    • Added categories attribute to CppProcessor class (data was originally checker.STYLE_CATEGORIES).
  • Scripts/webkitpy/style/processors/cpp_unittest.py:
    • Addressed FIXME by eliminating "import" from checker.py.
00:29 Changeset [53989] by eric@webkit.org
  • 3 edits in trunk/WebKitTools

2010-01-28 Anton Muhin <antonm@chromium.org>

Reviewed by Shinichiro Hamaji.

Improve treatment of conditions and rest of the line for if, else, switch and alikes
https://bugs.webkit.org/show_bug.cgi?id=34173

  • Scripts/webkitpy/style/cpp_style.py:
  • Scripts/webkitpy/style/cpp_style_unittest.py:
00:14 Changeset [53988] by mrowe@apple.com
  • 10 edits
    2 adds in branches/safari-4-branch

Merge r53567.

00:14 Changeset [53987] by mrowe@apple.com
  • 5 edits
    2 adds in branches/safari-4-branch

Merge r53368.

00:14 Changeset [53986] by mrowe@apple.com
  • 30 edits
    1 copy
    7 adds in branches/safari-4-branch

Merge r53512.

00:13 Changeset [53985] by mrowe@apple.com
  • 3 edits
    2 adds in branches/safari-4-branch

Merge r53369.

00:13 Changeset [53984] by mrowe@apple.com
  • 10 edits
    2 adds in branches/safari-4-branch

Merge r53220.

00:13 Changeset [53983] by mrowe@apple.com
  • 3 edits
    2 adds in branches/safari-4-branch

Merge r53163.

00:13 Changeset [53982] by mrowe@apple.com
  • 6 edits
    2 adds in branches/safari-4-branch

Merge r52351.

00:13 Changeset [53981] by mrowe@apple.com
  • 3 edits
    2 adds in branches/safari-4-branch

Merge r52781.

00:13 Changeset [53980] by mrowe@apple.com
  • 14 edits
    2 adds in branches/safari-4-branch

Merge r52773.

00:13 Changeset [53979] by mrowe@apple.com
  • 3 edits
    2 adds in branches/safari-4-branch

Merge r52352.

00:13 Changeset [53978] by mrowe@apple.com
  • 7 edits
    2 adds in branches/safari-4-branch

Merge r52233.

00:13 Changeset [53977] by mrowe@apple.com
  • 4 edits
    2 adds in branches/safari-4-branch

Merge r50196.

00:12 Changeset [53976] by eric@webkit.org
  • 2 edits in trunk/WebKitTools

2010-01-28 Joe Mason <jmason@rim.com>

Reviewed by Adam Barth.

Limit login retries to 5
https://bugs.webkit.org/show_bug.cgi?id=34193

  • Scripts/webkitpy/bugzilla.py:

01/27/10:

23:29 Changeset [53975] by tkent@chromium.org
  • 4 edits
    6 adds in trunk

2010-01-27 Kent Tamura <tkent@chromium.org>

Reviewed by Darin Adler.

rangeOverflow/rangeUnderflow support for type=date
https://bugs.webkit.org/show_bug.cgi?id=34209

  • fast/forms/ValidityState-rangeOverflow-date-expected.txt: Added.
  • fast/forms/ValidityState-rangeOverflow-date.html: Added.
  • fast/forms/ValidityState-rangeUnderflow-date-expected.txt: Added.
  • fast/forms/ValidityState-rangeUnderflow-date.html: Added.
  • fast/forms/script-tests/ValidityState-rangeOverflow-date.js: Added.
  • fast/forms/script-tests/ValidityState-rangeUnderflow-date.js: Added.

2010-01-27 Kent Tamura <tkent@chromium.org>

Reviewed by Darin Adler.

rangeOverflow/rangeUnderflow support for type=date
https://bugs.webkit.org/show_bug.cgi?id=34209

Add DATE type support to rangeUnderflow(), rangeOverflow(),
minimum(), and maximum() of HTMLInputElement.
In order to unify parsing code for value, min, and max strings,
introduce parseToDouble() function and it is called by
valueAsDate() and valueAsNumber() too.

Tests: fast/forms/ValidityState-rangeOverflow-date.html

fast/forms/ValidityState-rangeUnderflow-date.html

  • html/HTMLInputElement.cpp: (WebCore::HTMLInputElement::rangeUnderflow): Support DATE type, and use parseToDouble(). (WebCore::HTMLInputElement::rangeOverflow): ditto. (WebCore::HTMLInputElement::minimum): ditto. (WebCore::HTMLInputElement::maximum): ditto. (WebCore::HTMLInputElement::doubleValueFor): Added. (WebCore::HTMLInputElement::valueAsDate): Use parseToDouble(). (WebCore::HTMLInputElement::valueAsNumber): Use parseToDouble().
  • html/HTMLInputElement.h: Declare parseToDouble().
23:27 Changeset [53974] by darin@chromium.org
  • 2 edits in trunk/WebCore

2010-01-27 Darin Fisher <darin@chromium.org>

Fix chromium build bustage (take 2).

  • bindings/scripts/CodeGeneratorV8.pm: Really treat V8CustomGetter like CustomGetter in all cases.
23:18 Changeset [53973] by ossy@webkit.org
  • 2 edits in trunk/LayoutTests

[Qt] Skip test introduced in r53972 because of missing eventSender.beginDragWithFiles()

  • platform/qt/Skipped: fast/dom/Window/window-postmessage-clone-frames.html skipped.
22:50 Changeset [53972] by darin@chromium.org
  • 2 edits in trunk/WebCore

2010-01-27 Darin Fisher <darin@chromium.org>

Fix chromium build bustage.

  • bindings/scripts/CodeGeneratorV8.pm: Treat V8CustomGetter like CustomGetter in all cases.
22:36 Changeset [53971] by darin@chromium.org
  • 3 edits in trunk/WebKit/chromium

2010-01-27 Darin Fisher <darin@chromium.org>

Reviewed by Pavel Feldman.

Expose HistoryItem::documentSequenceNumber
https://bugs.webkit.org/show_bug.cgi?id=34243

  • public/WebHistoryItem.h:
  • src/WebHistoryItem.cpp: (WebKit::WebHistoryItem::documentSequenceNumber): (WebKit::WebHistoryItem::setDocumentSequenceNumber):
21:26 Changeset [53970] by oliver@apple.com
  • 2 edits in trunk/JavaScriptCore

Windows build fix.

20:47 Changeset [53969] by oliver@apple.com
  • 12 edits
    3 adds in trunk

2010-01-27 Oliver Hunt <oliver@apple.com>

Reviewed by Maciej Stachowiak.

MessageEvent.data should deserialize in the context of the MessageEvent's global object
https://bugs.webkit.org/show_bug.cgi?id=34227

Add logic to allow us to create an Object, Array, or Date instance
so we can create them in the context of a specific global object,
rather than just using the current lexical global object.

  • JavaScriptCore.exp:
  • runtime/DateInstance.cpp: (JSC::DateInstance::DateInstance):
  • runtime/DateInstance.h:
  • runtime/JSGlobalObject.h: (JSC::constructEmptyObject): (JSC::constructEmptyArray):

2010-01-27 Oliver Hunt <oliver@apple.com>

Reviewed by Maciej Stachowiak.

MessageEvent.data should deserialize in the context of the MessageEvent's global object
https://bugs.webkit.org/show_bug.cgi?id=34227

Test that the object returned from postMessage.data is created in the correct context

  • fast/dom/Window/resources/window-postmessage-clone-frames-frame.html: Added.
  • fast/dom/Window/window-postmessage-clone-frames-expected.txt: Added.
  • fast/dom/Window/window-postmessage-clone-frames.html: Added.

2010-01-27 Oliver Hunt <oliver@apple.com>

Reviewed by Maciej Stachowiak.

MessageEvent.data should deserialize in the context of the MessageEvent's global object
https://bugs.webkit.org/show_bug.cgi?id=34227

Make ScriptValue deserialisation support the provision of a specific global
object to use when creating new objects. This then allows us to make
MessageEvent.data and PopStateEvent.state deserialize in the correct
context.

Test: fast/dom/Window/window-postmessage-clone-frames.html

  • bindings/js/JSPopStateEventCustom.cpp: Remove custom implementation of state getter
  • bindings/js/SerializedScriptValue.cpp: (WebCore::DeserializingTreeWalker::DeserializingTreeWalker): (WebCore::DeserializingTreeWalker::createOutputArray): (WebCore::DeserializingTreeWalker::createOutputObject): (WebCore::DeserializingTreeWalker::convertIfTerminal): (WebCore::SerializedScriptValueData::deserialize): (WebCore::SerializedScriptValue::deserialize):
  • bindings/js/SerializedScriptValue.h: (WebCore::SerializedScriptValue::deserialize):
  • bindings/scripts/CodeGeneratorJS.pm: Update bindings generation to pass the correct global object, and to treat "any" as synonymous with SerializedValue.
  • dom/PopStateEvent.idl:
19:24 Changeset [53968] by mrobinson@webkit.org
  • 2 edits in trunk/WebKitTools

Adding myself to the committer list. No review necessary.

  • Scripts/webkitpy/committers.py:
19:22 Changeset [53967] by eric@webkit.org
  • 2 edits in trunk/WebKit/gtk

2010-01-27 Martin Robinson <mrobinson@webkit.org>

Reviewed by Gustavo Noronha Silva.

[GTK] Expose the IM context via the API
https://bugs.webkit.org/show_bug.cgi?id=33327

Expose the GtkIMMultiContext as a property of WebKitWebView. This will
allow embedders to generate the input method context menu entries and
make testing certain IM context behavior possible.

  • webkit/webkitwebview.cpp: (webkit_web_view_get_property): (DNDContentsRequest::webkit_web_view_get_im_context): (DNDContentsRequest::webkit_web_view_class_init):
18:55 Changeset [53966] by eric@webkit.org
  • 3 edits in trunk/WebCore

2010-01-27 Nicholas Young <nicholas.young@nokia.com>

Reviewed by Eric Carlson.

Make media controls fade in/out transition duration configurable on a theme level.
https://bugs.webkit.org/show_bug.cgi?id=34196

No new tests needed (I hope).

  • rendering/RenderMedia.cpp: (WebCore::RenderMedia::RenderMedia): Removed constant from contructor - the initial value is meaningless. (WebCore::RenderMedia::updateControlVisibility): Get fade in/out duration from theme.
  • rendering/RenderTheme.h: (WebCore::RenderTheme::mediaControlsFadeInDuration): New virtual method. (WebCore::RenderTheme::mediaControlsFadeOutDuration): New virtual method.
17:41 Changeset [53965] by eric@webkit.org
  • 7 edits
    1 copy
    2 moves
    1 add in trunk

2010-01-27 John Abd-El-Malek <jam@chromium.org>

Reviewed by Dimitri Glazkov.

Tests that when a page removes an iframe that sleeps in its unload handler and
terminates its JS execution, the outer page's JS continues running.

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

  • fast/dom/Window/slow-unload-handler-expected.txt: Copied from LayoutTests/fast/dom/Window/slow_unload_handler-expected.txt.
  • fast/dom/Window/slow-unload-handler-only-frame-is-stopped.html: Added.
  • fast/dom/Window/slow-unload-handler-only-frame-is-stopped-expected.txt: Added.
  • fast/dom/Window/slow-unload-handler.html: Copied from LayoutTests/fast/dom/Window/slow_unload_handler.html.
  • fast/dom/Window/slow_unload_handler-expected.txt: Removed.
  • fast/dom/Window/slow_unload_handler.html: Removed.
  • platform/gtk/Skipped:
  • platform/mac/Skipped:
  • platform/qt/Skipped:
  • platform/win/Skipped:

2010-01-27 John Abd-El-Malek <jam@chromium.org>

Reviewed by Dimitri Glazkov.

Tests that when a page removes an iframe that sleeps in its unload handler and
terminates its JS execution, the outer page's JS continues running.

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

Tests: fast/dom/Window/slow-unload-handler-only-frame-is-stopped.html

fast/dom/Window/slow-unload-handler.html

  • bindings/v8/DateExtension.cpp: (WebCore::DateExtension::OnSleepDetected):
17:24 Changeset [53964] by eric@webkit.org
  • 5 edits
    3 adds in trunk

2010-01-27 Kent Tamura <tkent@chromium.org>

Reviewed by Darin Adler.

Add valueAsNumber support for type=datetime-local.
https://bugs.webkit.org/show_bug.cgi?id=34200

The test data is equivalent to input-valueasnumber-datetime.html.
The expectation contains two FAIL lines because they check
unimplemented features.

  • fast/forms/input-valueasnumber-datetimelocal-expected.txt: Added.
  • fast/forms/input-valueasnumber-datetimelocal.html: Added.
  • fast/forms/script-tests/input-valueasnumber-datetimelocal.js: Added.

2010-01-27 Kent Tamura <tkent@chromium.org>

Reviewed by Darin Adler.

Add valueAsNumber support for type=datetime-local.
https://bugs.webkit.org/show_bug.cgi?id=34200

Implement necessary methods of ISODateTime, and call them from
HTMLInputElement.

Test: fast/forms/input-valueasnumber-datetimelocal.html

  • html/HTMLInputElement.cpp: (WebCore::HTMLInputElement::valueAsNumber): (WebCore::HTMLInputElement::setValueAsNumber):
  • html/ISODateTime.cpp: (WebCore::ISODateTime::setMillisecondsSinceEpochForDateTimeLocal): Implemented. Just call setMillisecondsSinceEpochForDateTime(). (WebCore::ISODateTime::millisecondsSinceEpochForTime): Accept to be called for m_type=DateTimeLocal. (WebCore::ISODateTime::toString): Add DateTimeLocal support.
  • html/ISODateTime.h: Declare new methods.
17:11 Changeset [53963] by mrowe@apple.com
  • 2 edits
    1 add in branches/safari-4-branch/WebCore

Merge r53867.

17:05 Changeset [53962] by eric@webkit.org
  • 2 edits in trunk/WebKitTools

2010-01-27 George Wright <gwright@rim.com>

Reviewed by Adam Treat.

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

Add missing include for wtf/Platform.h

  • DumpRenderTree/AccessibilityController.h:
16:46 Changeset [53961] by sfalken@apple.com
  • 3 edits in trunk/WebCore

Windows Debug_All build fix.

  • platform/graphics/win/WKCACFLayer.cpp:
  • platform/graphics/win/WKCACFLayerRenderer.cpp:
16:28 Changeset [53960] by aroben@apple.com
  • 3 edits in trunk/WebKit/win

Make it possible to instantiate WebSerializedJSValue using WebKitCreateInstance

Reviewed by Dave Hyatt.

  • ForEachCoClass.h:
  • WebKitClassFactory.cpp:

Added WebSerializedJSValue.

16:23 Changeset [53959] by bweinstein@apple.com
  • 1 edit
    2 adds in trunk/LayoutTests

Rubber-stamped by Adam Roben.

Add platform specific failing Windows results for two recently
added tests. I will comment on their originating bugs saying that
failing results were landed for Windows.

<https://bugs.webkit.org/show_bug.cgi?id=29564>
<https://bugs.webkit.org/show_bug.cgi?id=32696>

  • platform/win/fast/css/button-height-expected.txt: Added.
  • platform/win/fast/forms/input-valueasnumber-datetime-expected.txt: Added.
16:16 Changeset [53958] by eric@webkit.org
  • 3 edits in trunk/WebCore

2010-01-27 Evan Martin <evan@chromium.org>

Reviewed by David Levin.

[chromium] complex text draws newlines as bells
https://bugs.webkit.org/show_bug.cgi?id=34186

Revert r45496 -- once we've got a glyph array, it is too late to normalize
because we could have had multiple codepoints combine into one glyph. The
Uniscribe code it mentions it's duplicating uses the log cluster map to fix
this.

Instead, we just normalize the input text if it contains any non-ascii-space
whitespace.

This fixes fast/text/international/hindi-whitespace, which currently has an
incorrect baseline containing a square box glyph.

  • platform/graphics/chromium/FontLinux.cpp: (WebCore::TextRunWalker::getTextRun): (WebCore::TextRunWalker::getNormalizedTextRun):
  • platform/graphics/chromium/HarfbuzzSkia.cpp: (WebCore::stringToGlyphs):
15:41 Changeset [53957] by ap@apple.com
  • 20 edits in trunk

Reviewed by Darin Adler.

https://bugs.webkit.org/show_bug.cgi?id=34150
WebKit needs a mechanism to catch stale HashMap entries

It is very difficult to catch stale pointers that are HashMap keys - since a pointer's hash
is just its value, it is very unlikely that any observable problem is reproducible.

This extends hash table consistency checks to check that pointers are referencing allocated
memory blocks, and makes it possible to invoke the checks explicitly (it is not feasible
to enable CHECK_HASHTABLE_CONSISTENCY by default, because that affects performance too much).

  • wtf/HashMap.h: (WTF::::checkConsistency): Call through to HashTable implementation. We can add similar calls to HashSet and HashCountedSet, but I haven't seen hard to debug problems with those yet.
  • wtf/HashSet.h: (WTF::::remove): The version of checkTableConsistency that's guarded by CHECK_HASHTABLE_CONSISTENCY is now called internalCheckTableConsistency().
  • wtf/HashTable.h: (WTF::HashTable::internalCheckTableConsistency): (WTF::HashTable::internalCheckTableConsistencyExceptSize): (WTF::HashTable::checkTableConsistencyExceptSize): Expose checkTableConsistency() even if CHECK_HASHTABLE_CONSISTENCY is off. (WTF::::add): Updated for checkTableConsistency renaming. (WTF::::addPassingHashCode): Ditto. (WTF::::removeAndInvalidate): Ditto. (WTF::::remove): Ditto. (WTF::::rehash): Ditto. (WTF::::checkTableConsistency): The assertion for !shouldExpand() was not correct - this function returns true for tables with m_table == 0. (WTF::::checkTableConsistencyExceptSize): Call checkValueConsistency for key. Potentially, we could do the same for values.
  • wtf/HashTraits.h: (WTF::GenericHashTraits::checkValueConsistency): An empty function that can be overridden to add checks. Currently, the only override is for pointer hashes.
  • wtf/RefPtrHashMap.h: (WTF::::remove): Updated for checkTableConsistency renaming.
15:32 Changeset [53956] by eric@webkit.org
  • 7 edits in trunk

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

Reviewed by Kenneth Rohde Christiansen.

[Qt] DRT Provide worker thread ability to track counters
https://bugs.webkit.org/show_bug.cgi?id=34221

Implement workerThreadCount() in LayoutTestController of Qt DRT

Tests:
fast/workers/dedicated-worker-lifecycle.html
fast/workers/shared-worker-frame-lifecycle.html
fast/workers/shared-worker-lifecycle.html
fast/workers/worker-lifecycle.html

  • platform/qt/Skipped:

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

Reviewed by Kenneth Rohde Christiansen.

[Qt] DRT Provide worker thread ability to track counters
https://bugs.webkit.org/show_bug.cgi?id=34221

Implement workerThreadCount() in LayoutTestController of Qt DRT

Tests:
fast/workers/dedicated-worker-lifecycle.html
fast/workers/shared-worker-frame-lifecycle.html
fast/workers/shared-worker-lifecycle.html
fast/workers/worker-lifecycle.html

  • Api/qwebpage.cpp: (qt_drt_workerThreadCount):

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

Reviewed by Kenneth Rohde Christiansen.

[Qt] DRT Provide worker thread ability to track counters
https://bugs.webkit.org/show_bug.cgi?id=34221

Implement workerThreadCount() in LayoutTestController of Qt DRT

Tests:
fast/workers/dedicated-worker-lifecycle.html
fast/workers/shared-worker-frame-lifecycle.html
fast/workers/shared-worker-lifecycle.html
fast/workers/worker-lifecycle.html

  • DumpRenderTree/qt/LayoutTestControllerQt.cpp: (LayoutTestController::workerThreadCount):
  • DumpRenderTree/qt/LayoutTestControllerQt.h:
15:16 Changeset [53955] by eric@webkit.org
  • 4 edits in trunk/WebCore

2010-01-27 Brian Tarricone <brian@kakai.com>

Reviewed by Gustavo Noronha Silva.

[gtk] Add support for windowless NPAPI plugins

Much of this is a translation of how the Qt backend does this,
modified for Gtk.

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

No new tests; there is already a windowless test in the tree.

  • plugins/PluginView.cpp: (WebCore::PluginView::handleEvent): (WebCore::PluginView::PluginView):
  • plugins/PluginView.h:
  • plugins/gtk/PluginViewGtk.cpp: (WebCore::getRootWindow): (WebCore::PluginView::updatePluginWidget): (WebCore::PluginView::paint): (WebCore::PluginView::handleKeyboardEvent): (WebCore::inputEventState): (WebCore::PluginView::initXEvent): (WebCore::setXButtonEventSpecificFields): (WebCore::setXMotionEventSpecificFields): (WebCore::setXCrossingEventSpecificFields): (WebCore::PluginView::handleMouseEvent): (WebCore::PluginView::handleFocusInEvent): (WebCore::PluginView::handleFocusOutEvent): (WebCore::PluginView::setNPWindowRect): (WebCore::PluginView::setNPWindowIfNeeded): (WebCore::PluginView::getValueStatic): (WebCore::PluginView::invalidateRect): (WebCore::getPluginDisplay): (WebCore::plug_removed_cb): (WebCore::getVisualAndColormap): (WebCore::PluginView::platformStart): (WebCore::PluginView::platformDestroy):
14:59 Changeset [53954] by eric@webkit.org
  • 2 edits in trunk/JavaScriptCore

2010-01-27 Anton Muhin <antonm@chromium.org>

Reviewed by Darin Adler.

Remove trailing \ from inline function code
https://bugs.webkit.org/show_bug.cgi?id=34223

  • assembler/ARMv7Assembler.h: (JSC::ARMThumbImmediate::countLeadingZerosPartial):
14:41 Changeset [53953] by eric@webkit.org
  • 2 edits in trunk/WebCore

2010-01-27 Anton Muhin <antonm@chromium.org>

Reviewed by David Levin.

[v8] minor cosmetic fix in V8 bindings codegenerator
https://bugs.webkit.org/show_bug.cgi?id=34224

  • bindings/scripts/CodeGeneratorV8.pm:
14:08 Changeset [53952] by kenneth@webkit.org
  • 2 edits in trunk/WebKitTools

Rubberstamped by Simon Hausmann.

[Qt] QtLauncher refactoring:

Make the LauncherWindow depend less on the view, so that more
code can be shared in near future.

  • QtLauncher/main.cpp:

(LauncherWindow::sendTouchEvent):
(LauncherWindow::loadFinished):
(LauncherWindow::zoomIn):
(LauncherWindow::zoomOut):
(LauncherWindow::resetZoom):
(LauncherWindow::toggleZoomTextOnly):
(LauncherWindow::dumpHtml):
(LauncherWindow::selectElements):
(LauncherWindow::setupUI):

13:36 Changeset [53951] by darin@chromium.org
  • 2 edits in trunk/WebCore

2010-01-27 Darin Fisher <darin@chromium.org>

Fix Chromium build bustage.

  • rendering/RenderBoxModelObject.cpp: Add wtf/CurrentTime.h include.
13:17 Changeset [53950] by darin@chromium.org
  • 17 edits
    2 deletes in trunk

2010-01-26 Darin Fisher <darin@chromium.org>

Reviewed by Brady Eidson and David Levin.

Chains of history items representing same-document navigation need to
always remember that association

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

Replace HistoryItem's Document pointer with a DocumentSequenceNumber.
During session history traversal, if the current HistoryItem and the
target HistoryItem have the same DocumentSequenceNumber, then it means
that the current Document should remain.

NOTE: To support Chromium's serialization of HistoryItems, I generate
DocumentSequenceNumbers that are unique across application launches.
DocumentSequenceNumbers are generated using a counter initialized with
the time of day.

Test: fast/loader/stateobjects/document-destroyed-navigate-back.html

  • dom/Document.cpp: (WebCore::Document::detach):
  • dom/Document.h:
  • history/BackForwardList.cpp: (WebCore::BackForwardList::pushStateItem):
  • history/BackForwardListChromium.cpp: (WebCore::BackForwardList::pushStateItem):
  • history/HistoryItem.cpp: (WebCore::generateDocumentSequenceNumber): (WebCore::HistoryItem::HistoryItem): (WebCore::HistoryItem::~HistoryItem): (WebCore::HistoryItem::setStateObject):
  • history/HistoryItem.h: (WebCore::HistoryItem::setDocumentSequenceNumber): (WebCore::HistoryItem::documentSequenceNumber):
  • loader/FrameLoader.cpp: (WebCore::FrameLoader::navigateWithinDocument): (WebCore::FrameLoader::loadItem):
  • loader/HistoryController.cpp: (WebCore::HistoryController::updateBackForwardListForFragmentScroll): (WebCore::HistoryController::pushState): (WebCore::HistoryController::replaceState):
  • loader/RedirectScheduler.cpp: (WebCore::RedirectScheduler::scheduleHistoryNavigation):
  • page/History.cpp: (WebCore::History::stateObjectAdded):
  • page/Page.cpp: (WebCore::Page::goToItem):
13:14 Changeset [53949] by oliver@apple.com
  • 6 edits in trunk/WebCore

2010-01-27 Oliver Hunt <oliver@apple.com>

Reviewed by Simon Fraser.

Animated scaling of background-image is too slow
https://bugs.webkit.org/show_bug.cgi?id=33808

Implement a version of the RenderImage animated scaling optimisation
for background images. Due to the possibility of arbitrary transforms
being applied to containing elements we explicitly check the current
CTM of the context for scaling or rotation.

  • platform/graphics/GraphicsContext.cpp: (WebCore::GraphicsContext::drawTiledImage):
  • platform/graphics/GraphicsContext.h:
  • platform/graphics/transforms/TransformationMatrix.h: (WebCore::TransformationMatrix::isIdentityOrTranslation):
  • rendering/RenderBoxModelObject.cpp: (WebCore::RenderBoxModelScaleData::RenderBoxModelScaleData): (WebCore::RenderBoxModelScaleData::~RenderBoxModelScaleData): (WebCore::RenderBoxModelScaleData::size): (WebCore::RenderBoxModelScaleData::time): (WebCore::RenderBoxModelScaleData::useLowQualityScale): (WebCore::RenderBoxModelScaleData::hiqhQualityRepaintTimer): (WebCore::RenderBoxModelScaleData::setSize): (WebCore::RenderBoxModelScaleData::setTime): (WebCore::RenderBoxModelScaleData::setUseLowQualityScale): (WebCore::RenderBoxModelScaleObserver::boxModelObjectDestroyed): (WebCore::RenderBoxModelScaleObserver::highQualityRepaintTimerFired): (WebCore::RenderBoxModelScaleObserver::shouldPaintBackgroundAtLowQuality): (WebCore::RenderBoxModelObject::highQualityRepaintTimerFired): (WebCore::RenderBoxModelObject::~RenderBoxModelObject): (WebCore::RenderBoxModelObject::paintFillLayerExtended):
  • rendering/RenderBoxModelObject.h:
12:40 Changeset [53948] by kenneth@webkit.org
  • 3 edits
    2 adds in trunk/WebKitTools

Rubberstamped by Simon Hausmann.

[Qt] QtLauncher refactoring:

Move out code from the MainWindow (renamed to LauncherWindow)
that is not depending on the view, and add it to a new class
called MainWindow.

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

(LauncherWindow::LauncherWindow):
(LauncherWindow::eventFilter):
(LauncherWindow::loadStarted):
(LauncherWindow::loadFinished):
(LauncherWindow::newWindow):
(LauncherWindow::setupUI):
(WebPage::createWindow):
(main):

  • QtLauncher/mainwindow.cpp: Added.

(MainWindow::MainWindow):
(MainWindow::buildUI):
(MainWindow::page):
(MainWindow::setAddressUrl):
(MainWindow::addCompleterEntry):
(MainWindow::load):
(MainWindow::changeLocation):
(MainWindow::openFile):

  • QtLauncher/mainwindow.h: Added.
12:16 Changeset [53947] by eric@webkit.org
  • 6 edits
    1 copy in trunk

2010-01-27 Yael Aharon <yael.aharon@nokia.com>

Reviewed by Laszlo Gombos.

[Qt] Enable websockets support in QtWebKit
https://bugs.webkit.org/show_bug.cgi?id=34180

Remove websockets tests from skiplist.

  • platform/qt/Skipped:

2010-01-27 Yael Aharon <yael.aharon@nokia.com>

Reviewed by Laszlo Gombos.

[Qt] Enable websockets support in QtWebKit
https://bugs.webkit.org/show_bug.cgi?id=34180

Connected between SocketStreamHandle and QTcpSocket/QSslSocket
to enable support for websockets.
Proxy authentication is not yet supported.
SSL certificate errors are ignored while this is under development.

  • WebCore.pro:
  • platform/network/qt/SocketStreamHandle.h:
  • platform/network/qt/SocketStreamHandlePrivate.h: Added.
  • platform/network/qt/SocketStreamHandleQt.cpp: (WebCore::SocketStreamHandlePrivate::SocketStreamHandlePrivate): (WebCore::SocketStreamHandlePrivate::~SocketStreamHandlePrivate): (WebCore::SocketStreamHandlePrivate::socketConnected): (WebCore::SocketStreamHandlePrivate::socketReadyRead): (WebCore::SocketStreamHandlePrivate::send): (WebCore::SocketStreamHandlePrivate::close): (WebCore::SocketStreamHandlePrivate::socketSentdata): (WebCore::SocketStreamHandlePrivate::socketClosed): (WebCore::SocketStreamHandlePrivate::socketError): (WebCore::SocketStreamHandlePrivate::socketClosedCallback): (WebCore::SocketStreamHandlePrivate::socketErrorCallback): (WebCore::SocketStreamHandlePrivate::socketSslErrors): (WebCore::SocketStreamHandle::SocketStreamHandle): (WebCore::SocketStreamHandle::~SocketStreamHandle): (WebCore::SocketStreamHandle::platformSend): (WebCore::SocketStreamHandle::platformClose):
12:00 Changeset [53946] by eric@webkit.org
  • 4 edits
    2 adds in trunk

2010-01-27 Mads Ager <ager@chromium.org>

Reviewed by Dimitri Glazkov.

[V8] Support getting integer-named properties using indexing notation on document object
https://bugs.webkit.org/show_bug.cgi?id=34211

  • fast/dom/HTMLDocument/get-iframe-with-integer-name-expected.txt: Added.
  • fast/dom/HTMLDocument/get-iframe-with-integer-name.html: Added.

2010-01-27 Mads Ager <ager@chromium.org>

Reviewed by Dimitri Glazkov.

[V8] Support getting integer-named properties using indexing notation on document object
https://bugs.webkit.org/show_bug.cgi?id=34211

Add indexed property getter to HTMLDocument to support getting
elements with integer names using indexing notation on document.

Test: fast/dom/HTMLDocument/get-iframe-with-integer-name.html

  • bindings/scripts/CodeGeneratorV8.pm:
  • bindings/v8/custom/V8HTMLDocumentCustom.cpp: (WebCore::V8HTMLDocument::indexedPropertyGetter):
11:30 Changeset [53945] by yurys@chromium.org
  • 4 edits
    1 add in trunk/WebCore

2010-01-27 Yury Semikhatsky <yurys@chromium.org>

Reviewed by Oliver Hunt.

Instead of relying on Object.prototype.toString result use JSObject::isActivationObject
to check if a scope node is a JSActivation. Object.prototype.toString for JSActivation
will call JSActivation::toThisObject whose result depends on the current call stack.

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

Test: WebCore/manual-tests/inspector/debugger-scopes-inspection.html

  • bindings/js/JSInjectedScriptHostCustom.cpp: (WebCore::JSInjectedScriptHost::isActivation):
  • inspector/InjectedScriptHost.idl:
  • inspector/front-end/InjectedScript.js: (injectedScriptConstructor.):
  • manual-tests/inspector/debugger-scopes-inspection.html: Added.
11:18 Changeset [53944] by eric@webkit.org
  • 10 edits in trunk/WebCore

2010-01-27 Anton Muhin <antonm@google.com>

Review by Adam Barth.

Implement Node map in intrusive way for better speed.
https://bugs.webkit.org/show_bug.cgi?id=33957

  • bindings/v8/DOMData.h: (WebCore::DOMData::removeObjectsFromWrapperMap):
  • bindings/v8/DOMDataStore.cpp: (WebCore::DOMDataStore::IntrusiveDOMWrapperMap::get): (WebCore::DOMDataStore::IntrusiveDOMWrapperMap::set): (WebCore::DOMDataStore::IntrusiveDOMWrapperMap::removeIfPresent): (WebCore::DOMDataStore::IntrusiveDOMWrapperMap::contains): (WebCore::DOMDataStore::IntrusiveDOMWrapperMap::visit): (WebCore::DOMDataStore::IntrusiveDOMWrapperMap::clear): (WebCore::DOMDataStore::IntrusiveDOMWrapperMap::allocateTableEntry): (WebCore::DOMDataStore::IntrusiveDOMWrapperMap::freeTableEntry):
  • bindings/v8/DOMDataStore.h: (WebCore::DOMDataStore::IntrusiveDOMWrapperMap::IntrusiveDOMWrapperMap): (WebCore::DOMDataStore::IntrusiveDOMWrapperMap::clearEntry): (WebCore::DOMDataStore::IntrusiveDOMWrapperMap::clearEntries): (WebCore::DOMDataStore::IntrusiveDOMWrapperMap::visitEntries): (WebCore::DOMDataStore::IntrusiveDOMWrapperMap::TableChunk::TableChunk): (WebCore::DOMDataStore::domNodeMap):
  • bindings/v8/StaticDOMDataStore.cpp: (WebCore::StaticDOMDataStore::StaticDOMDataStore):
  • bindings/v8/StaticDOMDataStore.h:
  • bindings/v8/V8DOMMap.cpp: (WebCore::getDOMNodeMap):
  • bindings/v8/V8DOMMap.h: (WebCore::AbstractWeakReferenceMap::AbstractWeakReferenceMap): (WebCore::AbstractWeakReferenceMap::weakReferenceCallback): (WebCore::WeakReferenceMap::WeakReferenceMap): (WebCore::WeakReferenceMap::set): (WebCore::WeakReferenceMap::visit):
  • bindings/v8/V8DOMWrapper.cpp: (WebCore::V8DOMWrapper::convertDocumentToV8Object): (WebCore::getWrapper): (WebCore::V8DOMWrapper::convertNodeToV8Object): (WebCore::V8DOMWrapper::convertNewNodeToV8Object):
  • bindings/v8/V8DOMWrapper.h:
11:15 Changeset [53943] by darin@chromium.org
  • 3 edits in trunk/WebKit/chromium

2010-01-26 Darin Fisher <darin@chromium.org>

Reviewed by David Levin.

Add methods to support running a nested modal loop outside of WebKit.
https://bugs.webkit.org/show_bug.cgi?id=34199

  • public/WebView.h:
  • src/WebViewImpl.cpp: (WebKit::WebView::willEnterModalLoop): Create PageGroupLoadDeferrer (WebKit::WebView::didExitModalLoop): Destroy PageGroupLoadDeferrer
11:01 Changeset [53942] by eric@webkit.org
  • 7 edits in trunk

2010-01-27 Kinuko Yasuda <kinuko@chromium.org>

Reviewed by Eric Seidel.

Remove fast/events/keydown-numpad-keys.html from gtk's Skipped list.
https://bugs.webkit.org/show_bug.cgi?id=28247

  • platform/gtk/Skipped:

2010-01-27 Kinuko Yasuda <kinuko@chromium.org>

Reviewed by Eric Seidel.

Add key event mappings for numpad keys for Gtk and Chromium/Gtk.
http://bugs.webkit.org/show_bug.cgi?id=28247

Test: fast/events/keydown-numpad-keys.html

  • platform/chromium/KeyCodeConversionGtk.cpp: (WebCore::windowsKeyCodeForKeyEvent):
  • platform/gtk/KeyEventGtk.cpp: (WebCore::windowsKeyCodeForKeyEvent):

2010-01-27 Kinuko Yasuda <kinuko@chromium.org>

Reviewed by Eric Seidel.

Correctly handle the KeyLocation argument that has been introduced
recently to test location-dependent key events in EventSender.keyDown.
http://bugs.webkit.org/show_bug.cgi?id=28247

Test: fast/events/keydown-numpad-keys.html

  • DumpRenderTree/gtk/EventSender.cpp: (keyDownCallback):
10:36 Changeset [53941] by pfeldman@chromium.org
  • 20 edits
    1 delete in trunk/WebCore

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

Reviewed by Timothy Hatcher.

Web Inspector: migrate from SourceFrame to SourceFrame2.

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

  • WebCore.gypi:
  • WebCore.vcproj/WebCore.vcproj:
  • bindings/js/JSInspectorFrontendHostCustom.cpp:
  • bindings/v8/custom/V8InspectorFrontendHostCustom.cpp:
  • inspector/InspectorFrontendHost.cpp:
  • inspector/InspectorFrontendHost.h:
  • inspector/InspectorFrontendHost.idl:
  • inspector/front-end/InspectorBackendStub.js:
  • inspector/front-end/InspectorFrontendHostStub.js:
  • inspector/front-end/ResourceView.js: (WebInspector.ResourceView.prototype._innerSelectContentTab):
  • inspector/front-end/ResourcesPanel.js: (WebInspector.getResourceContent):
  • inspector/front-end/ScriptView.js: (WebInspector.ScriptView): (WebInspector.ScriptView.prototype.setupSourceFrameIfNeeded):
  • inspector/front-end/ScriptsPanel.js: (WebInspector.ScriptsPanel.prototype._sidebarResizeDrag):
  • inspector/front-end/SourceFrame.js: (WebInspector.SourceFrame): (WebInspector.SourceFrame.prototype.set executionLine): (WebInspector.SourceFrame.prototype.revealLine): (WebInspector.SourceFrame.prototype.addBreakpoint): (WebInspector.SourceFrame.prototype.removeBreakpoint): (WebInspector.SourceFrame.prototype.addMessage): (WebInspector.SourceFrame.prototype.clearMessages): (WebInspector.SourceFrame.prototype.sizeToFitContentHeight): (WebInspector.SourceFrame.prototype.setContent): (WebInspector.SourceFrame.prototype.findSearchMatches): (WebInspector.SourceFrame.prototype._collectRegexMatches): (WebInspector.SourceFrame.prototype.setSelection): (WebInspector.SourceFrame.prototype._incrementMessageRepeatCount): (WebInspector.SourceFrame.prototype._addExistingMessagesToSource): (WebInspector.SourceFrame.prototype._addMessageToSource): (WebInspector.SourceFrame.prototype._addExistingBreakpointsToSource): (WebInspector.SourceFrame.prototype._addBreakpointToSource): (WebInspector.SourceFrame.prototype._removeBreakpointFromSource): (WebInspector.SourceFrame.prototype._contextMenu.addConditionalBreakpoint): (WebInspector.SourceFrame.prototype._contextMenu): (WebInspector.SourceFrame.prototype._toggleBreakpoint): (WebInspector.SourceFrame.prototype._editBreakpointCondition.committed): (WebInspector.SourceFrame.prototype._editBreakpointCondition.dismissed): (WebInspector.SourceFrame.prototype._editBreakpointCondition): (WebInspector.SourceFrame.prototype._showBreakpointConditionPopup): (WebInspector.SourceFrame.prototype._createConditionElement): (WebInspector.SourceFrame.prototype._keyDown): (WebInspector.SourceFrame.prototype._evalSelectionInCallFrame): (WebInspector.SourceFrame.prototype._breakpointChanged): (WebInspector.SourceFrame.prototype.resize): (WebInspector.BreakpointLineNumberDecorator): (WebInspector.BreakpointLineNumberDecorator.prototype.decorate): (WebInspector.BreakpointLineNumberDecorator.prototype._paintBreakpoint): (WebInspector.BreakpointLineNumberDecorator.prototype._paintProgramCounter): (WebInspector.BreakpointLineNumberDecorator.prototype.mouseDown): (WebInspector.BreakpointLineNumberDecorator.prototype.contextMenu): (WebInspector.ExecutionLineDecorator): (WebInspector.ExecutionLineDecorator.prototype.decorate):
  • inspector/front-end/SourceFrame2.js: Removed.
  • inspector/front-end/SourceView.js: (WebInspector.SourceView): (WebInspector.SourceView.prototype.show): (WebInspector.SourceView.prototype.resize): (WebInspector.SourceView.prototype.setupSourceFrameIfNeeded): (WebInspector.SourceView.prototype._contentLoaded): (WebInspector.SourceView.prototype.performSearch.findSearchMatches): (WebInspector.SourceView.prototype.performSearch): (WebInspector.SourceView.prototype._jumpToSearchResult): (WebInspector.SourceView.prototype._sourceFrameSetupFinished):
  • inspector/front-end/TextEditor.js: (WebInspector.TextEditor.prototype._mouseDown): (WebInspector.TextEditor.prototype._copy.delayCopy): (WebInspector.TextEditor.prototype._copy): (WebInspector.TextEditor.prototype._cut):
  • inspector/front-end/WebKit.qrc:
  • inspector/front-end/inspector.html:
10:19 Changeset [53940] by eric@webkit.org
  • 2 edits in trunk/WebCore

2010-01-27 Dominik Röttsches <dominik.roettsches@access-company.com>

Reviewed by Xan Lopez.

[Gtk] For removing ICU, implement IDN support by means of libidn
https://bugs.webkit.org/show_bug.cgi?id=31470

In order to avoid a new dependency,
IDN support now based on GLib for the GLib unicode backend.

  • platform/KURL.cpp: (WebCore::appendEncodedHostname):
09:55 Changeset [53939] by darin@chromium.org
  • 3 edits
    1 add in trunk/WebKit/chromium

2010-01-27 Darin Fisher <darin@chromium.org>

Reviewed by Dimitri Glazkov.

Adding missing WebMutationEvent.cpp file to fix build bustage.
https://bugs.webkit.org/show_bug.cgi?id=33882

  • public/WebMutationEvent.h: Remove unnecessary constructor.
  • src/WebEvent.cpp: Remove unnecessary include of WebMutationEvent.h.
  • src/WebMutationEvent.cpp: Added.
09:43 Changeset [53938] by kenneth@webkit.org
  • 2 edits in trunk/WebKitTools

Rubberstamped by Simon Hausmann.

[Qt] QtLauncher, coding style fixes.

  • QtLauncher/main.cpp:

(MainWindow::MainWindow):
(MainWindow::sendTouchEvent):
(MainWindow::eventFilter):
(MainWindow::loadURL):
(MainWindow::setupUI):
(WebPage::createPlugin):

09:41 BuildingOnWindows edited by vestbo@webkit.org
(diff)
09:40 BuildingOnWindows edited by vestbo@webkit.org
(diff)
09:02 Changeset [53937] by kenneth@webkit.org
  • 4 edits
    2 copies in trunk/WebKitTools

Rubberstamped by Simon Hausmann.

[Qt] QtLauncher refactoring, separating the webview and
adding a QGraphicsWebView based version.

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

(MainWindow::MainWindow):

  • QtLauncher/webpage.h:
  • QtLauncher/webview.cpp: Added.

(createContextMenu):
(WebViewGraphicsBased::mousePressEvent):
(WebViewTraditional::mousePressEvent):
(WebViewGraphicsBased::contextMenuEvent):
(WebViewTraditional::contextMenuEvent):

  • QtLauncher/webview.h: Added.

(WebViewGraphicsBased::WebViewGraphicsBased):
(WebViewTraditional::WebViewTraditional):

08:14 Changeset [53936] by hausmann@webkit.org
  • 2 edits in trunk/WebKit/qt

[Qt] Update the .def files with exported symbols

Reviewed by Laszlo Gombos.

  • symbian/eabi/QtWebKitu.def: Add two mangled missing new symbols for arm eabi.
08:06 Changeset [53935] by hausmann@webkit.org
  • 4 edits in trunk

.: [Qt] Don't build the tests in packages, only the launcher(s)

Reviewed by Kenneth Rohde Christiansen.

  • WebKit.pro:

WebCore: [Qt] For package builds use the .def files on Symbian

Reviewed by Kenneth Rohde Christiansen.

  • WebCore.pro:
07:15 Changeset [53934] by eric@webkit.org
  • 6 edits
    9 adds in trunk/WebKit/chromium

2010-01-27 Jay Campan <jcampan@google.com>

Reviewed by Darin Fisher.

Adding EventListeners to the chromium API.

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

  • WebKit.gyp:
  • public/WebEvent.h: Added.
  • public/WebEventListener.h: Added.
  • public/WebMutationEvent.h: Added.
  • public/WebNode.h:
  • public/WebString.h: (WebKit::operator==): (WebKit::operator!=):
  • src/EventListenerWrapper.cpp: Added.
  • src/EventListenerWrapper.h: Added.
  • src/WebEvent.cpp: Added.
  • src/WebEventListener.cpp: Added.
  • src/WebEventListenerPrivate.cpp: Added.
  • src/WebEventListenerPrivate.h: Added.
  • src/WebNode.cpp: (WebKit::WebNode::addEventListener): (WebKit::WebNode::removeEventListener):
  • src/WebString.cpp: (WebKit::WebString::equals):
06:58 Changeset [53933] by eric@webkit.org
  • 2 edits in trunk/WebCore

2010-01-27 Alexander Pavlov <apavlov@chromium.org>

Reviewed by Timothy Hatcher.

Avoid watch expressions duplication
https://bugs.webkit.org/show_bug.cgi?id=33996

  • inspector/front-end/WatchExpressionsSidebarPane.js: (WebInspector.WatchExpressionsSidebarPane.prototype._settingsLoaded):
06:47 Changeset [53932] by apavlov@chromium.org
  • 2 edits in trunk/WebKitTools

2010-01-27 Alexander Pavlov <apavlov@chromium.org>

Adding myself as committer. No review necessary.

  • Scripts/webkitpy/committers.py:
06:41 Changeset [53931] by eric@webkit.org
  • 12 edits
    2 adds in trunk/WebCore

2010-01-27 Marcus Bulach <bulach@chromium.org>

Reviewed by Dimitri Glazkov.

Adds EventSource bindings for V8
https://bugs.webkit.org/show_bug.cgi?id=33695

No new tests (existing layout tests for EventSource should pass).

  • Android.v8bindings.mk:
  • WebCore.gypi:
  • bindings/scripts/CodeGeneratorV8.pm:
  • bindings/v8/DOMObjectsInclude.h:
  • bindings/v8/DerivedSourcesAllInOne.cpp:
  • bindings/v8/V8DOMWrapper.cpp:
  • bindings/v8/V8DOMWrapper.h:
  • bindings/v8/V8Index.cpp:
  • bindings/v8/V8Index.h:
  • bindings/v8/WorkerContextExecutionProxy.cpp:
  • bindings/v8/custom/V8CustomBinding.h:
  • bindings/v8/custom/V8EventSourceConstructor.cpp: Added.
  • bindings/v8/custom/V8EventSourceCustom.cpp: Added.
06:25 Changeset [53930] by eric@webkit.org
  • 5 edits in trunk

2010-01-27 Kent Hansen <kent.hansen@nokia.com>

Reviewed by Simon Hausmann.

[Qt] Meta-methods can't be introspected using ES5 API
https://bugs.webkit.org/show_bug.cgi?id=34087

Add getOwnPropertyDescriptor() and getOwnPropertyNames() reimplementations.

Tests are in WebKit/qt/tests/qwebframe

  • bridge/qt/qt_runtime.cpp: (JSC::Bindings::QtRuntimeMetaMethod::getOwnPropertyDescriptor): (JSC::Bindings::QtRuntimeMetaMethod::getOwnPropertyNames): (JSC::Bindings::QtRuntimeConnectionMethod::getOwnPropertyDescriptor): (JSC::Bindings::QtRuntimeConnectionMethod::getOwnPropertyNames):
  • bridge/qt/qt_runtime.h:

2010-01-27 Kent Hansen <kent.hansen@nokia.com>

Reviewed by Simon Hausmann.

[Qt] Meta-methods can't be introspected using ES5 API
https://bugs.webkit.org/show_bug.cgi?id=34087

Test that Object.getOwnPropertyDescriptor and
Object.getOwnPropertyNames work with meta-methods.

  • tests/qwebframe/tst_qwebframe.cpp:
06:21 Changeset [53929] by kenneth@webkit.org
  • 3 edits
    2 adds in trunk/WebKitTools

Rubberstamped by Simon Hausmann

[Qt] QtLauncher refactoring, separating utility methods.

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

(urlFromUserInput):

  • QtLauncher/utils.h: Added.
06:07 Changeset [53928] by eric@webkit.org
  • 2 edits in trunk/JavaScriptCore

2010-01-27 Kwang Yul Seo <skyul@company100.net>

Reviewed by Eric Seidel.

[BREWMP] Port WTF's randomNumber
https://bugs.webkit.org/show_bug.cgi?id=33566

Use GETRAND to generate 4 byte random byte sequence to implement
weakRandomNumber. Create a secure random number generator with
AEECLSID_RANDOM to implement randomNumber.

  • wtf/RandomNumber.cpp: (WTF::weakRandomNumber): (WTF::randomNumber):
05:51 Changeset [53927] by eric@webkit.org
  • 3 edits
    2 adds in trunk

2010-01-27 Tony Chang <tony@chromium.org>

Reviewed by Eric Seidel.

Fix a crash when trying to indent a block element that was contained
in a list. This was happening because enclosingBlock() in
htmlediting.cpp can return the current the same Node when a block
element (like an <hr> or a <table>) is passed in. This causes
the indent command to think that it is not in a list item.

Work around this by checking to see if enclosingBlock returned the
same Node.

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

  • editing/execCommand/indent-block-in-list-expected.txt: Added.
  • editing/execCommand/indent-block-in-list.html: Added.

2010-01-27 Tony Chang <tony@chromium.org>

Reviewed by Eric Seidel.

Fix a crash when trying to indent a block element that was contained
in a list. This was happening because enclosingBlock() in
htmlediting.cpp can return the current the same Node when a block
element (like an <hr> or a <table>) is passed in. This causes
the indent command to think that it is not in a list item.

Work around this by checking to see if enclosingBlock returned the
same Node.

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

Test: editing/execCommand/indent-block-in-list.html

  • editing/IndentOutdentCommand.cpp: (WebCore::IndentOutdentCommand::tryIndentingAsListItem):
05:34 Changeset [53926] by eric@webkit.org
  • 2 edits in trunk/JavaScriptCore

2010-01-27 Kwang Yul Seo <skyul@company100.net>

Reviewed by Eric Seidel.

[BREWMP] Port getCPUTime
https://bugs.webkit.org/show_bug.cgi?id=33572

Use GETUPTIMEMS which returns a continuously and
linearly increasing millisecond timer from the time the device
was powered on. This function is enough to implement getCPUTime.

  • runtime/TimeoutChecker.cpp: (JSC::getCPUTime):
05:01 Changeset [53925] by eric@webkit.org
  • 9 edits
    1 copy
    1 add in trunk/WebKitTools

2010-01-27 Eric Seidel <eric@webkit.org>

Reviewed by Darin Adler.

webkit-patch needs an open-bugs command
https://bugs.webkit.org/show_bug.cgi?id=30793

  • Scripts/test-webkitpy: import OpenBugsTest
  • Scripts/webkit-patch: import OpenBugs
  • Scripts/webkitpy/commands/download_unittest.py: don't import unittest
  • Scripts/webkitpy/commands/early_warning_system_unittest.py: ditto
  • Scripts/webkitpy/commands/queries_unittest.py: ditto
  • Scripts/webkitpy/commands/queues_unittest.py: ditto
  • Scripts/webkitpy/commands/upload_unittest.py: ditto
  • Scripts/webkitpy/mock_bugzillatool.py: log when user.open_url is called.
04:43 Changeset [53924] by jocelyn.turcotte@nokia.com
  • 4 edits in trunk

[Qt] Add the "d" suffix to QtWebKit's dll on Windows.

Reviewed by Tor Arne Vestbø.

  • WebKit.pri:

WebCore:

  • WebCore.pro:
04:41 Changeset [53923] by ossy@webkit.org
  • 1 edit
    1 delete in trunk/LayoutTests

[Qt] Qt port doesn't need platform dependent expected file anymore
for editing/pasteboard/paste-noscript-xhtml.html from r53873.

  • platform/qt/editing/pasteboard/paste-noscript-xhtml-expected.txt: Removed.
04:41 Changeset [53922] by eric@webkit.org
  • 2 edits
    1 add in trunk/JavaScriptCore

2010-01-27 Kwang Yul Seo <skyul@company100.net>

Reviewed by Oliver Hunt.

[BREWMP] Add MarkStack fastMalloc implementation for platforms without VirtualAlloc or mmap.
https://bugs.webkit.org/show_bug.cgi?id=33582

Use fastMalloc and fastFree to implement MarkStack::allocateStack and
MarkStack::releaseStack for platforms without page level allocation.

  • runtime/MarkStack.h: (JSC::MarkStack::MarkStackArray::shrinkAllocation):
  • runtime/MarkStackNone.cpp: Added. (JSC::MarkStack::initializePagesize): (JSC::MarkStack::allocateStack): (JSC::MarkStack::releaseStack):
04:17 Changeset [53921] by eric@webkit.org
  • 2 edits in trunk/JavaScriptCore

2010-01-27 Kwang Yul Seo <skyul@company100.net>

Reviewed by Eric Seidel.

[BREWMP] Don't use time function
https://bugs.webkit.org/show_bug.cgi?id=33577

Calling time(0) in BREW devices causes a crash because time
is not properly ported in most devices. Cast currentTime() to
time_t to get the same result as time(0).

  • wtf/DateMath.cpp: (WTF::calculateUTCOffset):
04:10 Changeset [53920] by ossy@webkit.org
  • 2 edits in trunk/LayoutTests

[Qt] fast/backgrounds/animated-svg-as-background.html make fast/backgrounds/svg-as-background-5.html crash.
https://bugs.webkit.org/show_bug.cgi?id=34202

  • platform/qt/Skipped: fast/backgrounds/animated-svg-as-background.html enabled because bug rolled out by r53912.
03:56 Changeset [53919] by kenneth@webkit.org
  • 4 edits in trunk/WebKitTools

Rubberstamped by Simon Hausmann

[Qt] Refactor the code in the QtLauncher dealing with HTTP proxy.

  • QtLauncher/main.cpp:

(MainWindow::MainWindow):

  • QtLauncher/webpage.cpp:

(WebPage::WebPage):
(WebPage::applyProxy):

  • QtLauncher/webpage.h:
03:51 Changeset [53918] by jocelyn.turcotte@nokia.com
  • 2 edits in trunk

[Qt] Build fix for windows when QTDIR contains release libraries.

Unreviewed build fix

  • WebKit.pri: Use the <name>.lib syntax for linking instead of qmake's -l<name> emulation
03:43 Changeset [53917] by eric@webkit.org
  • 3 edits
    6 adds in trunk

2010-01-27 Matt Perry <mpcomplete@chromium.org>

Reviewed by Eric Seidel.

Fix a bug where dispatchDocumentElementAvailable was fired for fragment parsing on XML documents.
https://bugs.webkit.org/show_bug.cgi?id=33920

  • userscripts/resources: Added.
  • userscripts/resources/blank.xhtml: Added.
  • userscripts/script-not-run-for-fragments-expected.txt: Added.
  • userscripts/script-not-run-for-fragments.html: Added.
  • userscripts/script-run-at-start-expected.txt: Added.
  • userscripts/script-run-at-start.html: Added.

2010-01-27 Matt Perry <mpcomplete@chromium.org>

Reviewed by Eric Seidel.

Fix a bug where dispatchDocumentElementAvailable was fired for fragment parsing on XML documents.
https://bugs.webkit.org/show_bug.cgi?id=33920

Tests: userscripts/script-not-run-for-fragments.html

userscripts/script-run-at-start.html

  • dom/XMLTokenizerLibxml2.cpp: (WebCore::XMLTokenizer::startElementNs):
03:18 Changeset [53916] by pfeldman@chromium.org
  • 3 edits in trunk/WebCore

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

Reviewed by Timothy Hatcher.

Web Inspector: Improve SourceHTMLTokenizer so that it treats script tag well.

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

  • inspector/front-end/SourceHTMLTokenizer.js: (WebInspector.SourceHTMLTokenizer): (WebInspector.SourceHTMLTokenizer.prototype._isAttribute): (WebInspector.SourceHTMLTokenizer.prototype._isAttributeValue): (WebInspector.SourceHTMLTokenizer.prototype._setAttributeValue): (WebInspector.SourceHTMLTokenizer.prototype._setAttribute): (WebInspector.SourceHTMLTokenizer.prototype._stringToken): (WebInspector.SourceHTMLTokenizer.prototype.nextToken):
  • inspector/front-end/SourceHTMLTokenizer.re2js:
03:16 Changeset [53915] by pfeldman@chromium.org
  • 8 edits
    6 deletes in trunk/WebCore

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

Reviewed by Timothy Hatcher.

Web Inspector: nuke quarantine wrappers.

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

  • Android.jscbindings.mk:
  • GNUmakefile.am:
  • WebCore.gypi:
  • WebCore.pro:
  • WebCore.vcproj/WebCore.vcproj:
  • bindings/js/JSBindingsAllInOne.cpp:
  • bindings/js/JSInspectedObjectWrapper.cpp: Removed.
  • bindings/js/JSInspectedObjectWrapper.h: Removed.
  • bindings/js/JSInspectorCallbackWrapper.cpp: Removed.
  • bindings/js/JSInspectorCallbackWrapper.h: Removed.
  • bindings/js/JSQuarantinedObjectWrapper.cpp: Removed.
  • bindings/js/JSQuarantinedObjectWrapper.h: Removed.
02:41 BuildingOnWindows edited by vestbo@webkit.org
(diff)
01:49 Changeset [53914] by jocelyn.turcotte@nokia.com
  • 2 edits in trunk/WebKitTools

[Qt] Corrects debug build of DumpRenderTree on Windows.

Reviewed by Tor Arne Vestbø.

  • DumpRenderTree/qt/DumpRenderTree.pro:
01:09 Changeset [53913] by ap@apple.com
  • 2 edits in trunk

Fix a typo in ChangeLogs.

01:06 Changeset [53912] by ap@apple.com
  • 21 edits in trunk

Revert r53899 (HashMap<AtomicStringImpl*, Value> key checks) and subsequent bug fixes,
because they make SVG tests crash in release builds.

00:41 Changeset [53911] by ossy@webkit.org
  • 2 edits in trunk/LayoutTests

[Qt] fast/backgrounds/animated-svg-as-background.html make fast/backgrounds/svg-as-background-5.html crash.
https://bugs.webkit.org/show_bug.cgi?id=34202

  • platform/qt/Skipped: fast/backgrounds/animated-svg-as-background.html skipped until fix.
00:36 Changeset [53910] by philn@webkit.org
  • 3 edits in trunk/WebCore

2010-01-25 Philippe Normand <pnormand@igalia.com>

Reviewed by Eric Seidel.

autobuffer value not forwarded media element to MediaPlayer
https://bugs.webkit.org/show_bug.cgi?id=33889

  • html/HTMLMediaElement.cpp: (WebCore::HTMLMediaElement::loadResource): Forward autobuffer value to MediaPlayer instance.
  • platform/graphics/MediaPlayer.cpp: (WebCore::MediaPlayer::load): Forward autobuffer and preservesPitch values to newly created platform media player instance.
00:24 Changeset [53909] by philn@webkit.org
  • 5 edits in trunk

2010-01-25 Philippe Normand <pnormand@igalia.com>

Reviewed by Eric Seidel.

[Gtk] media/video-reverse-play-duration.html fails on and off on Gtk buildbots
https://bugs.webkit.org/show_bug.cgi?id=34086

Cache media duration and fix didEnd() in case of reverse
playback. When EOS was reached but in case of reverse playback the
position is not always 0. So to not confuse the HTMLMediaElement
we synchronize position and duration values.

  • platform/graphics/gtk/MediaPlayerPrivateGStreamer.cpp: (WebCore::MediaPlayerPrivate::MediaPlayerPrivate): (WebCore::MediaPlayerPrivate::duration): (WebCore::MediaPlayerPrivate::updateStates): (WebCore::MediaPlayerPrivate::didEnd): (WebCore::MediaPlayerPrivate::durationChanged):
  • platform/graphics/gtk/MediaPlayerPrivateGStreamer.h:

2010-01-25 Philippe Normand <pnormand@igalia.com>

Reviewed by Eric Seidel.

[Gtk] media/video-reverse-play-duration.html fails on and off on Gtk buildbots
https://bugs.webkit.org/show_bug.cgi?id=34086

  • platform/gtk/Skipped: Unskip fixed test.
00:06 Changeset [53908] by ap@apple.com
  • 2 edits in trunk/WebCore
  • WebCore.xcodeproj/project.pbxproj: Actually land the change to add ContainerNodeAlgorithms.h (it's "svn resolved", not "svn revert"!).

01/26/10:

23:54 Changeset [53907] by dbates@webkit.org
  • 2 edits in trunk/LayoutTests

2010-01-26 Daniel Bates <dbates@webkit.org>

Unreviewed. Made a minor typo when I added the test case fast/css/button-height.html
to the GTK Skipped file. In changeset 53900, I added test "fast/css/button-height.htm"
to the GTK Skipped file instead of "fast/css/button-height.html". See bug #33936
for more details.

  • platform/gtk/Skipped:
23:53 Changeset [53906] by ossy@webkit.org
  • 2 edits in trunk/LayoutTests

[Qt] Failing tests after r53895 skipped.
https://bugs.webkit.org/show_bug.cgi?id=34167

  • platform/qt/Skipped:
    • http/tests/history/redirect-301.pl
    • http/tests/history/redirect-302.pl
    • http/tests/history/redirect-303.pl
    • http/tests/history/redirect-307.pl
23:46 Changeset [53905] by ossy@webkit.org
  • 2 edits in trunk/LayoutTests

Unreviewed trivial fix.

  • platform/qt/editing/pasteboard/paste-noscript-xhtml-expected.txt: updated after r53873.
23:02 Changeset [53904] by ap@apple.com
  • 2 edits in trunk/WebKitLibraries

More Windows build fixing.

  • win/tools/vsprops/common.vsprops: Disable warning C4180 (qualifier applied to function type has no meaning; ignored). This is a known bug - MSVC tries to compile a wrong specialization sometimes - but it's not instantiated, so it's harmless.
22:39 Changeset [53903] by ap@apple.com
  • 2 edits in trunk/JavaScriptCore

More Windows build fixing.

  • wtf/HashTraits.h: _msize takes void*, remove const qualifier from type.
22:22 Changeset [53902] by ap@apple.com
  • 2 edits in trunk/JavaScriptCore

Windows build fix.

  • wtf/HashTraits.h: Include malloc.h for _msize().
22:19 Changeset [53901] by ap@apple.com
  • 2 edits in trunk/JavaScriptCore

Build fix.

  • wtf/HashTable.h: (WTF::HashTable::checkTableConsistencyExceptSize): Remove const from a static (empty) version of this function.
22:07 Changeset [53900] by dbates@webkit.org
  • 3 edits in trunk/LayoutTests

2010-01-26 Daniel Bates <dbates@webkit.org>

Unreviewed. Updated expected failing results for table-percent-height.html
to reflect one pixel difference between the results generated by the Qt bot
and my machine. Added failing test fast/css/button-height.html to GTK Skipped
file as we need to look into this. See bug #33936 for more details.

  • platform/gtk/Skipped: Added failing test fast/css/button-height.html.
  • platform/qt/fast/replaced/table-percent-height-expected.txt: Updated result.
21:57 Changeset [53899] by ap@apple.com
  • 21 edits in trunk

Reviewed by Darin Adler.

https://bugs.webkit.org/show_bug.cgi?id=34150
WebKit needs a mechanism to catch stale HashMap entries

It is very difficult to catch stale pointers that are HashMap keys - since a pointer's hash
is just its value, it is very unlikely that any observable problem is reproducible.

This extends hash table consistency checks to check that pointers are referencing allocated
memory blocks, and makes it possible to invoke the checks explicitly (it is not feasible
to enable CHECK_HASHTABLE_CONSISTENCY by default, because that affects performance too much).

  • wtf/HashMap.h: (WTF::::checkConsistency): Call through to HashTable implementation. We can add similar calls to HashSet and HashCountedSet, but I haven't seen hard to debug problems with those yet.
  • wtf/HashSet.h: (WTF::::remove): The version of checkTableConsistency that's guarded by CHECK_HASHTABLE_CONSISTENCY is now called internalCheckTableConsistency().
  • wtf/HashTable.h: (WTF::HashTable::internalCheckTableConsistency): (WTF::HashTable::internalCheckTableConsistencyExceptSize): (WTF::HashTable::checkTableConsistencyExceptSize): Expose checkTableConsistency() even if CHECK_HASHTABLE_CONSISTENCY is off. (WTF::::add): Updated for checkTableConsistency renaming. (WTF::::addPassingHashCode): Ditto. (WTF::::removeAndInvalidate): Ditto. (WTF::::remove): Ditto. (WTF::::rehash): Ditto. (WTF::::checkTableConsistency): The assertion for !shouldExpand() was not correct - this function returns true for tables with m_table == 0. (WTF::::checkTableConsistencyExceptSize): Call checkValueConsistency for key. Potentially, we could do the same for values.
  • wtf/HashTraits.h: (WTF::GenericHashTraits::checkValueConsistency): An empty function that can be overridden to add checks. Currently, the only override is for pointer hashes.
  • wtf/RefPtrHashMap.h: (WTF::::remove): Updated for checkTableConsistency renaming.
21:54 Changeset [53898] by eric@webkit.org
  • 2 edits
    1 add in trunk/WebKit/chromium

2010-01-26 Yaar Schnitman <yaar@chromium.org>

Reviewed by Darin Fisher.

Add KeyboardTest to WebKit API tests
https://bugs.webkit.org/show_bug.cgi?id=34178

  • WebKit.gyp:
  • tests/KeyboardTest.cpp: Added.
21:29 Changeset [53897] by dbates@webkit.org
  • 5 edits
    3 adds in trunk

2010-01-26 Daniel Bates <dbates@webkit.org>

Reviewed by Tor Arne Vestbø.

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

[Qt] Fixes an issue where the height of <button>- and
<input type="button">- elements are fixed to the height of the
button label font plus padding. That is, the CSS height property
is being ignored.

Instead, we should honor the user-specified height, if appropriate
for the platform and context. Notice, the Mac ports do not honor the
height for <input type="button"> elements unless a border and/or
background is also specified.

Test: fast/css/button-height.html

  • platform/qt/RenderThemeQt.cpp: (WebCore::RenderThemeQt::adjustButtonStyle):

2010-01-26 Daniel Bates <dbates@webkit.org>

Reviewed by Tor Arne Vestbø.

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

Tests that the user-specified height for <button>- and <input type="button">-
elements are honored, if appropriate for the platform and context.

  • fast/css/button-height-expected.txt: Added.
  • fast/css/button-height.html: Added.
  • fast/replaced/table-percent-height-expected.txt: Added notice about failing tests in Windows ports.
  • fast/replaced/table-percent-height.html: Ditto.
  • platform/qt/fast/replaced/table-percent-height-expected.txt: Added.
21:26 Changeset [53896] by dbates@webkit.org
  • 4 edits in trunk/WebKitTools

2010-01-26 Daniel Bates <dbates@webkit.org>

Reviewed by Adam Barth.

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

Moves the try/catch for OSError exceptions in Executive.run_command
to Credentials.read_credentials() so that the unit test
webkitpy.scm_unittest.SCMClassTests.test_error_handlers can
assert that Executive.run_command throws an OSError exception.

  • Scripts/webkitpy/credentials.py:
  • Scripts/webkitpy/executive.py: Moved try/catch for OSError to method Credentials.read_credentials().
  • Scripts/webkitpy/executive_unittest.py: Removed tests that no longer apply: test_run_command_with_bad_command_check_return_code and test_run_command_with_bad_command_check_calls_error_handler. Added new test to assert that run_command throws OSError exceptions.
21:21 Changeset [53895] by eric@webkit.org
  • 5 edits in trunk

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

Reviewed by Kenneth Rohde Christiansen.

[Qt] DRT WebHistory support
https://bugs.webkit.org/show_bug.cgi?id=34167

  • platform/qt/Skipped:

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

Reviewed by Kenneth Rohde Christiansen.

[Qt] DRT WebHistory support
https://bugs.webkit.org/show_bug.cgi?id=34167

  • DumpRenderTree/qt/LayoutTestControllerQt.cpp: (LayoutTestController::reset): (LayoutTestController::webHistoryItemCount): (LayoutTestController::keepWebHistory):
  • DumpRenderTree/qt/LayoutTestControllerQt.h:
21:04 Changeset [53894] by eric@webkit.org
  • 2 edits in trunk/WebCore

2010-01-26 Alexander Pavlov <apavlov@chromium.org>

Reviewed by Timothy Hatcher.

Enable the Audits panel via a shortcut (Cmd/Ctrl+Alt+A)
https://bugs.webkit.org/show_bug.cgi?id=34158

  • inspector/front-end/inspector.js: (WebInspector.documentKeyDown):
20:45 Changeset [53893] by tkent@chromium.org
  • 5 edits
    24 adds in trunk

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

Reviewed by Darin Adler.

HTMLInputElement::valueAsNumber support except type=datetime-local.
https://bugs.webkit.org/show_bug.cgi?id=32696

input-valueasnumber-datetime-expected.txt and
input-valueasnumber-time-expected.txt have FAIL lines because they
test unimplemented features.

  • fast/forms/input-valueasnumber-date-expected.txt: Added.
  • fast/forms/input-valueasnumber-date.html: Added.
  • fast/forms/input-valueasnumber-datetime-expected.txt: Added.
  • fast/forms/input-valueasnumber-datetime.html: Added.
  • fast/forms/input-valueasnumber-month-expected.txt: Added.
  • fast/forms/input-valueasnumber-month.html: Added.
  • fast/forms/input-valueasnumber-number-expected.txt: Added.
  • fast/forms/input-valueasnumber-number.html: Added.
  • fast/forms/input-valueasnumber-range-expected.txt: Added.
  • fast/forms/input-valueasnumber-range.html: Added.
  • fast/forms/input-valueasnumber-time-expected.txt: Added.
  • fast/forms/input-valueasnumber-time.html: Added.
  • fast/forms/input-valueasnumber-unsupported-expected.txt: Added.
  • fast/forms/input-valueasnumber-unsupported.html: Added.
  • fast/forms/input-valueasnumber-week-expected.txt: Added.
  • fast/forms/input-valueasnumber-week.html: Added.
  • fast/forms/script-tests/input-valueasnumber-date.js: Added.
  • fast/forms/script-tests/input-valueasnumber-datetime.js: Added.
  • fast/forms/script-tests/input-valueasnumber-month.js: Added.
  • fast/forms/script-tests/input-valueasnumber-number.js: Added.
  • fast/forms/script-tests/input-valueasnumber-range.js: Added.
  • fast/forms/script-tests/input-valueasnumber-time.js: Added.
  • fast/forms/script-tests/input-valueasnumber-unsupported.js: Added.
  • fast/forms/script-tests/input-valueasnumber-week.js: Added.

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

Reviewed by Darin Adler.

HTMLInputElement::valueAsNumber support except type=datetime-local.
https://bugs.webkit.org/show_bug.cgi?id=32696

Tests: fast/forms/input-valueasnumber-date.html

fast/forms/input-valueasnumber-datetime.html
fast/forms/input-valueasnumber-month.html
fast/forms/input-valueasnumber-number.html
fast/forms/input-valueasnumber-range.html
fast/forms/input-valueasnumber-time.html
fast/forms/input-valueasnumber-unsupported.html
fast/forms/input-valueasnumber-week.html

  • html/HTMLInputElement.cpp: (WebCore::HTMLInputElement::valueAsNumber): According to the specification, we should return NaN for invalid values. (WebCore::HTMLInputElement::setValueAsNumber): According to the specification, throws NOT_SUPPORTED_ERR for Infinitiy or NaN, and throws INVALID_STATE_ERR for unsupported types. (WebCore::HTMLInputElement::formStringToDouble): According to the step 14 of Real Number in the specification, we should round -0 to 0.
  • html/HTMLInputElement.h: Declare valueAsNumber() and setValueAsNumber().
  • html/HTMLInputElement.idl: Add valueAsNumber.
20:44 WebKit Team edited by laszlo.1.gombos@nokia.com
add my nick (diff)
20:41 Changeset [53892] by hyatt@apple.com
  • 2 edits in trunk/WebCore

Make sure there is a lock on serialize in SerializedScriptValueData, since type errors can be created when
exceptions are thrown.

Reviewed by Oliver Hunt.

  • bindings/js/SerializedScriptValue.cpp:

(WebCore::SerializedScriptValueData::serialize):

20:20 Changeset [53891] by eric@webkit.org
  • 2 edits in trunk/JavaScriptCore

2010-01-26 Lyon Chen <liachen@rim.com>

Reviewed by Maciej Stachowiak.

Opcode.h use const void* for Opcode cause error #1211 for RVCT compiler
https://bugs.webkit.org/show_bug.cgi?id=33902

  • bytecode/Opcode.h:
20:17 Changeset [53890] by tkent@chromium.org
  • 7 edits in trunk/WebCore

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

Reviewed by Eric Seidel.

Add Date type support for Objective-C binding.
https://bugs.webkit.org/show_bug.cgi?id=32810

The Date type in IDLs is mapped to NSTimeInterval in Objective-C.

  • bindings/objc/DOMInternal.h: Add kit() and core() for NSTimeInterval.
  • bindings/scripts/CodeGenerator.pm: Make Date a primitive type.
  • bindings/scripts/CodeGeneratorJS.pm: Move some code for the CodeGeenrator.pm change.
  • bindings/scripts/CodeGeneratorObjC.pm:
  • bindings/scripts/CodeGeneratorV8.pm: Move some code for the CodeGeenrator.pm change.
  • html/HTMLInputElement.idl: Remove Objective-C exclusion for valueAsDate.
20:09 Changeset [53889] by tkent@chromium.org
  • 4 edits in trunk

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

Reviewed by Eric Seidel.

[Win] Add modifiers parameter support to Windows DumpRenderTree
https://bugs.webkit.org/show_bug.cgi?id=34068

Remove listbox-deselect-scroll.html and listbox-selectio-2.html
from Skipped. We improved their portability and Windows DRT now
havs modifiers parameter of eventSender.mouseDown() and
eventSender.mouseUp().

  • platform/win/Skipped:

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

Reviewed by Eric Seidel.

[Win] Add modifiers parameter support to Windows DumpRenderTree
https://bugs.webkit.org/show_bug.cgi?id=34068

Add support for functional name modifiers; "addSelectionKey" and
"rangeSelectionKey", and modifiers parameter to eventSender.mouseDown()
and eventSender.mouseUp().
This change is similar to r53498 for Mac.

  • DumpRenderTree/win/EventSender.cpp: (buildModifierFlags): New function to set MK_CONTROL or MK_SHIFT to WPARAM. (mouseDownCallback): Call buidlModifiersFlags(). (mouseUpCallback): ditto. (keyDownCallback): Add support for "addSelectionkey" and "rangeSelectionKey".
19:58 Changeset [53888] by cjerdonek@webkit.org
  • 1 edit in trunk/WebKitTools/Scripts/webkitpy/autoinstall.py

No review: fixing broken autoinstall.py

19:04 Changeset [53887] by cjerdonek@webkit.org
  • 2 edits in trunk/WebKitTools

2010-01-26 Chris Jerdonek <cjerdonek@webkit.org>

Reviewed by Eric Seidel.

The Python autoinstall cache directory now only gets created
in the directory containing autoinstall.py.

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

  • Scripts/webkitpy/autoinstall.py:
    • Also added a README file to the cache directory saying where it came from.
18:56 Changeset [53886] by dimich@chromium.org
  • 2 edits in trunk/WebCore

Not reviewed, attempt to fix Tiger build.

  • rendering/RenderTreeAsText.cpp:

(WebCore::write): Use UNUSED_PARAM macros to prevent compiler warning on targets without accelerated compositing.

18:54 Changeset [53885] by rolandsteiner@chromium.org
  • 2 edits in trunk/WebCore

Incorrect boolean expression in isMailBlockquote() (WebCore/htmlediting.cpp)
https://bugs.webkit.org/show_bug.cgi?id=34156

Reviewed by Darin Adler.

No new tests (minor code change).

  • editing/htmlediting.cpp:

(WebCore::isMailBlockquote):

18:25 Changeset [53884] by pkasting@chromium.org
  • 5 edits in trunk/WebCore

Handle broken images more correctly in the open-source image decoders.
https://bugs.webkit.org/show_bug.cgi?id=33747

Reviewed by David Levin.

No tests since Safari doesn't use these decoders and there's already a
broken-image test in the tree.

  • platform/graphics/ImageSource.cpp:

(WebCore::ImageSource::createFrameAtIndex): No need to check isSizeAvailable() since size() is now always safe.

  • platform/image-decoders/ImageDecoder.h:

(WebCore::ImageDecoder::size): Don't assert that the bitmap is valid; we don't need a complete successful decode to be able to give info about the size.

  • platform/image-decoders/qt/RGBA32BufferQt.cpp:

(WebCore::RGBA32Buffer::setSize): Don't set the frame to complete on failure, since it's not complete, but empty, and callers can better handle empty frames.

  • platform/image-decoders/skia/ImageDecoderSkia.cpp:

(WebCore::RGBA32Buffer::setSize): Don't set the frame to complete on failure, since it's not complete, but empty, and callers can better handle empty frames.

18:23 Changeset [53883] by zimmermann@webkit.org
  • 3 edits in trunk/WebCore

2010-01-26 Nikolas Zimmermann <nzimmermann@rim.com>

Not reviewed. Synchronize Qt/Gtk build systems with Mac/Win, should fix Qt compilation. Gtk results still pending.

  • GNUmakefile.am:
  • WebCore.pro:
18:14 Changeset [53882] by cjerdonek@webkit.org
  • 4 edits
    4 moves
    2 adds in trunk/WebKitTools

2010-01-26 Chris Jerdonek <cjerdonek@webkit.org>

Reviewed by Eric Seidel.

Moved the check-webkit-style processors into a new
webkitpy/style/processors directory.

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

  • Scripts/webkitpy/style/checker.py:
  • Scripts/webkitpy/style/checker_unittest.py:
  • Scripts/webkitpy/style/cpp_style.py: Removed.
  • Scripts/webkitpy/style/cpp_style_unittest.py: Removed.
  • Scripts/webkitpy/style/processors: Added.
  • Scripts/webkitpy/style/processors/init.py: Added.
  • Scripts/webkitpy/style/processors/cpp.py: Copied from WebKitTools/Scripts/webkitpy/style/cpp_style.py.
  • Scripts/webkitpy/style/processors/cpp_unittest.py: Copied from WebKitTools/Scripts/webkitpy/style/cpp_style_unittest.py.
  • Scripts/webkitpy/style/processors/text.py: Copied from WebKitTools/Scripts/webkitpy/style/text_style.py.
  • Scripts/webkitpy/style/processors/text_unittest.py: Copied from WebKitTools/Scripts/webkitpy/style/text_style_unittest.py.
  • Scripts/webkitpy/style/text_style.py: Removed.
  • Scripts/webkitpy/style/text_style_unittest.py: Removed.
  • Scripts/webkitpy/style/unittests.py:
18:02 Changeset [53881] by zimmermann@webkit.org
  • 2 edits in trunk/WebCore

2010-01-26 Nikolas Zimmermann <nzimmermann@rim.com>

Not reviewed. Attempt to fix V8 - change SetterMethod function signatures, just like it has been done for JSSVGPODTypeWrapper.

  • bindings/v8/V8SVGPODTypeWrapper.h:
17:53 Changeset [53880] by zimmermann@webkit.org
  • 2 edits in trunk/WebCore

2010-01-26 Nikolas Zimmermann <nzimmermann@rim.com>

Not reviewd. Revert Base.xcconfig changes, was not meant to be committed.

  • Configurations/Base.xcconfig:
17:51 Changeset [53879] by zimmermann@webkit.org
  • 142 edits
    3 adds
    3 deletes in trunk/WebCore

2010-01-26 Nikolas Zimmermann <nzimmermann@rim.com>

Reviewed by Oliver Hunt.

SVG consumes way too much memory to store animated properties in the DOM
https://bugs.webkit.org/show_bug.cgi?id=34188

Shrink WebCore library size to 75% (measured in debug builds) and reduce SVG memory usage to <25%.
Adding Oliver testcase as manual-tests/svg-node-count-vs-scroll.xhtml, which creates an arbitary number of
rects (50.000 <rect> elements added to the DOM by default) - memory usage down to 111M from 503M (RPRVT).
The SVG DOM side is almost fine, still some optimizations possible, that will be implemented soon - next
target is the SVG render tree, there are plenty of possibilities to reduce memory usage there.

Redesign the way we store animated properties in the individual SVG*Element files. Short story: In order
to support SVG DOM through the bindings (e.g. JS) we need to associate SVGAnimatedProperty objects with
the SVGElement object that created it - we used to store this pointer directly in the SVGAnimatedProperty.
This means, every SVGAnimatedProperty stored in a SVGRectElement, stored a pointer to the SVGRectElement, resulting
in excessive memory usage. This is now properly implemented, without wasting tons of memory. Unfortunately this
requires touching all SVG*Element files.

Detailed list of changes:

  • Remove template bloat by not specializing class templates for each attributeName/tagName combination. Instead SVGAnimatedProperty is a non-specialized template class now, only depending on the AnimatedType (ie. SVGLength for SVGAnimatedLength) -> This heavily reduces the generated code, thus shrinking WebCore size. I hope we can build windows again without SVGAllInOne.cpp
  • Remove "exportStrings" / "exportString" handling from make_names.pl - SVG defined string literals for each QualifiedName, in order to use template specialization based on "const char*" parameters. All that bloat is gone, so there's no need for it anymore.
  • Redesign SVGAnimatedProperty so it does not need any back-pointers to the SVGElement that created it
  • Don't actually store the XML DOM attribute name associated with a SVG DOM property in SVGAnimatedProperty, move the associatedAttributeName() function inside the macro declaration and just return the passed macro parameter DOMAttribute there, storing is inefficient and useless.
  • Remove SynchronizablePropertyController, which was living in SVGElement as member variable, keeping a HashMap<AttributeName, SVGAnimatedProperty>. It was needed before to lookup a SVGAnimatedProperty for a XML DOM attribute, in order to synchronize SVG <-> XML dom properties/attributes. Instead just add a "synchronizeProperty(const QualifiedName&)" method to all SVG*Element classes. As each SVG*Element class knows about its animated properties it can just ask them to synchronize themselves - no need for any dynamic lookups anymore.
  • Remove SynchronizableTypeWrapper which added more complexity for the sake of SVG <-> XML DOM synchronization, all replaced by synchronizeProperty.
  • Pass around any POD objects as const references, instead of copying them for no reason.
  • Clean up SVGAnimatedProperty, splitting up into SVGAnimatedProperty/PropertySynchronizer/PropertyTraits.
  • Remove baseValue/setBaseValue code from SVGDocumentExtensions, not needed anymore.

... and tons of changes to all SVG*Element classes, adapting to the new way of handling animated properties.

  • GNUmakefile.am: Remove Synchronizable* from build, add new SVGAnimatedPropertySynchronizer/Traits files
  • WebCore.gypi: Ditto.
  • WebCore.vcproj/WebCore.vcproj: Ditto.
  • WebCore.xcodeproj/project.pbxproj: Ditto.
  • bindings/js/JSSVGPODTypeWrapper.h: Change synchronization callback signatures, as we pass around const-references now.
  • dom/Element.cpp: Let updateAnimatedSVGAttribute take a QualifiedName instead of pure Strings. (WebCore::Element::getAttribute): (WebCore::Element::hasAttributes):
  • dom/Element.h: (WebCore::Element::updateAnimatedSVGAttribute): (WebCore::Element::attributes):
  • dom/make_names.pl: Remove SVG specific "exportString" / "exportStrings" functionality, see above.
  • html/HTMLAttributeNames.in: Remove "exportString" tag from "className" attribute.
  • manual-tests/svg-node-count-vs-scroll.xhtml: Added. Can be used to verify memory consumption with a lot of DOM objects.
  • mathml/mathattrs.in: MathML doesn't need "exportStrings" - remove it.
  • mathml/mathtags.in: Ditto.
  • svg/SVGAElement.cpp: (WebCore::SVGAElement::SVGAElement): (WebCore::SVGAElement::synchronizeProperty):
  • svg/SVGAElement.h:
  • svg/SVGAllInOne.cpp: Remove SynchronizablePropertyController.cpp
  • svg/SVGAltGlyphElement.cpp: (WebCore::SVGAltGlyphElement::SVGAltGlyphElement): (WebCore::SVGAltGlyphElement::synchronizeProperty):
  • svg/SVGAltGlyphElement.h:
  • svg/SVGAnimateTransformElement.cpp: (WebCore::SVGAnimateTransformElement::applyResultsToTarget):
  • svg/SVGAnimatedProperty.h: Rewritten, see above for details. (WebCore::SVGAnimatedPropertyTearOff::create): (WebCore::SVGAnimatedPropertyTearOff::setBaseVal): (WebCore::SVGAnimatedPropertyTearOff::setAnimVal): (WebCore::SVGAnimatedPropertyTearOff::baseVal): (WebCore::SVGAnimatedPropertyTearOff::animVal): (WebCore::SVGAnimatedPropertyTearOff::associatedAttributeName): (WebCore::SVGAnimatedPropertyTearOff::SVGAnimatedPropertyTearOff): (WebCore::SVGAnimatedPropertyTearOff::~SVGAnimatedPropertyTearOff): (WebCore::SVGAnimatedProperty::~SVGAnimatedProperty): (WebCore::SVGAnimatedProperty::SVGAnimatedProperty): (WebCore::SVGAnimatedProperty::value): (WebCore::SVGAnimatedProperty::baseValue): (WebCore::SVGAnimatedProperty::setValue): (WebCore::SVGAnimatedProperty::setBaseValue): (WebCore::SVGAnimatedProperty::shouldSynchronize): (WebCore::SVGAnimatedProperty::setShouldSynchronize):
  • svg/SVGAnimatedPropertySynchronizer.h: Added. (WebCore::):
  • svg/SVGAnimatedPropertyTraits.h: Added. (WebCore::):
  • svg/SVGAnimatedTemplate.h: Move SVGAnimatedPropertyTraits into its own file. (WebCore::SVGAnimatedTemplate::forgetWrapper): (WebCore::lookupOrCreateWrapper):
  • svg/SVGAnimationElement.cpp: (WebCore::SVGAnimationElement::SVGAnimationElement): (WebCore::SVGAnimationElement::synchronizeProperty):
  • svg/SVGAnimationElement.h:
  • svg/SVGCircleElement.cpp: (WebCore::SVGCircleElement::SVGCircleElement): (WebCore::SVGCircleElement::synchronizeProperty):
  • svg/SVGCircleElement.h:
  • svg/SVGClipPathElement.cpp: (WebCore::SVGClipPathElement::SVGClipPathElement): (WebCore::SVGClipPathElement::synchronizeProperty):
  • svg/SVGClipPathElement.h:
  • svg/SVGComponentTransferFunctionElement.cpp: (WebCore::SVGComponentTransferFunctionElement::SVGComponentTransferFunctionElement): (WebCore::SVGComponentTransferFunctionElement::parseMappedAttribute): (WebCore::SVGComponentTransferFunctionElement::synchronizeProperty):
  • svg/SVGComponentTransferFunctionElement.h:
  • svg/SVGCursorElement.cpp: (WebCore::SVGCursorElement::SVGCursorElement): (WebCore::SVGCursorElement::synchronizeProperty):
  • svg/SVGCursorElement.h:
  • svg/SVGDefsElement.cpp: (WebCore::SVGDefsElement::SVGDefsElement): (WebCore::SVGDefsElement::synchronizeProperty):
  • svg/SVGDefsElement.h:
  • svg/SVGDocumentExtensions.h:
  • svg/SVGElement.cpp: Adapt to synchronization changes: use synchronizeProperty() call, instead of SynchronizablePropertyController. (WebCore::SVGElement::updateAnimatedSVGAttribute):
  • svg/SVGElement.h: Don't store SynchronizablePropertyController anymore, it's gone. (WebCore::SVGElement::synchronizeProperty): (WebCore::SVGElement::setSynchronizedSVGAttributes):
  • svg/SVGEllipseElement.cpp: (WebCore::SVGEllipseElement::SVGEllipseElement): (WebCore::SVGEllipseElement::synchronizeProperty):
  • svg/SVGEllipseElement.h:
  • svg/SVGExternalResourcesRequired.cpp:
  • svg/SVGExternalResourcesRequired.h:
  • svg/SVGFEBlendElement.cpp: (WebCore::SVGFEBlendElement::SVGFEBlendElement): (WebCore::SVGFEBlendElement::synchronizeProperty):
  • svg/SVGFEBlendElement.h:
  • svg/SVGFEColorMatrixElement.cpp: (WebCore::SVGFEColorMatrixElement::SVGFEColorMatrixElement): (WebCore::SVGFEColorMatrixElement::synchronizeProperty):
  • svg/SVGFEColorMatrixElement.h:
  • svg/SVGFEComponentTransferElement.cpp: (WebCore::SVGFEComponentTransferElement::SVGFEComponentTransferElement): (WebCore::SVGFEComponentTransferElement::synchronizeProperty):
  • svg/SVGFEComponentTransferElement.h:
  • svg/SVGFECompositeElement.cpp: (WebCore::SVGFECompositeElement::SVGFECompositeElement): (WebCore::SVGFECompositeElement::parseMappedAttribute): (WebCore::SVGFECompositeElement::synchronizeProperty):
  • svg/SVGFECompositeElement.h:
  • svg/SVGFEDiffuseLightingElement.cpp: (WebCore::SVGFEDiffuseLightingElement::SVGFEDiffuseLightingElement): (WebCore::SVGFEDiffuseLightingElement::synchronizeProperty):
  • svg/SVGFEDiffuseLightingElement.h:
  • svg/SVGFEDisplacementMapElement.cpp: (WebCore::SVGFEDisplacementMapElement::SVGFEDisplacementMapElement): (WebCore::SVGFEDisplacementMapElement::synchronizeProperty):
  • svg/SVGFEDisplacementMapElement.h:
  • svg/SVGFEGaussianBlurElement.cpp: (WebCore::SVGFEGaussianBlurElement::SVGFEGaussianBlurElement): (WebCore::SVGFEGaussianBlurElement::synchronizeProperty):
  • svg/SVGFEGaussianBlurElement.h:
  • svg/SVGFEImageElement.cpp: (WebCore::SVGFEImageElement::SVGFEImageElement): (WebCore::SVGFEImageElement::synchronizeProperty):
  • svg/SVGFEImageElement.h:
  • svg/SVGFELightElement.cpp: (WebCore::SVGFELightElement::SVGFELightElement): (WebCore::SVGFELightElement::synchronizeProperty):
  • svg/SVGFELightElement.h:
  • svg/SVGFEMergeNodeElement.cpp: (WebCore::SVGFEMergeNodeElement::SVGFEMergeNodeElement): (WebCore::SVGFEMergeNodeElement::synchronizeProperty):
  • svg/SVGFEMergeNodeElement.h:
  • svg/SVGFEMorphologyElement.cpp: (WebCore::SVGFEMorphologyElement::SVGFEMorphologyElement): (WebCore::SVGFEMorphologyElement::synchronizeProperty):
  • svg/SVGFEMorphologyElement.h:
  • svg/SVGFEOffsetElement.cpp: (WebCore::SVGFEOffsetElement::SVGFEOffsetElement): (WebCore::SVGFEOffsetElement::synchronizeProperty):
  • svg/SVGFEOffsetElement.h:
  • svg/SVGFESpecularLightingElement.cpp: (WebCore::SVGFESpecularLightingElement::SVGFESpecularLightingElement): (WebCore::SVGFESpecularLightingElement::synchronizeProperty):
  • svg/SVGFESpecularLightingElement.h:
  • svg/SVGFETileElement.cpp: (WebCore::SVGFETileElement::SVGFETileElement): (WebCore::SVGFETileElement::synchronizeProperty):
  • svg/SVGFETileElement.h:
  • svg/SVGFETurbulenceElement.cpp: (WebCore::SVGFETurbulenceElement::SVGFETurbulenceElement): (WebCore::SVGFETurbulenceElement::synchronizeProperty):
  • svg/SVGFETurbulenceElement.h:
  • svg/SVGFilterElement.cpp: (WebCore::SVGFilterElement::SVGFilterElement): (WebCore::SVGFilterElement::synchronizeProperty):
  • svg/SVGFilterElement.h:
  • svg/SVGFilterPrimitiveStandardAttributes.cpp: (WebCore::SVGFilterPrimitiveStandardAttributes::SVGFilterPrimitiveStandardAttributes): (WebCore::SVGFilterPrimitiveStandardAttributes::synchronizeProperty):
  • svg/SVGFilterPrimitiveStandardAttributes.h:
  • svg/SVGFitToViewBox.cpp:
  • svg/SVGFitToViewBox.h:
  • svg/SVGFontElement.cpp: (WebCore::SVGFontElement::SVGFontElement): (WebCore::SVGFontElement::synchronizeProperty):
  • svg/SVGFontElement.h:
  • svg/SVGForeignObjectElement.cpp: (WebCore::SVGForeignObjectElement::SVGForeignObjectElement): (WebCore::SVGForeignObjectElement::synchronizeProperty):
  • svg/SVGForeignObjectElement.h:
  • svg/SVGGElement.cpp: (WebCore::SVGGElement::SVGGElement): (WebCore::SVGGElement::synchronizeProperty):
  • svg/SVGGElement.h:
  • svg/SVGGradientElement.cpp: (WebCore::SVGGradientElement::SVGGradientElement): (WebCore::SVGGradientElement::synchronizeProperty):
  • svg/SVGGradientElement.h:
  • svg/SVGImageElement.cpp: (WebCore::SVGImageElement::SVGImageElement): (WebCore::SVGImageElement::synchronizeProperty):
  • svg/SVGImageElement.h:
  • svg/SVGLineElement.cpp: (WebCore::SVGLineElement::SVGLineElement): (WebCore::SVGLineElement::synchronizeProperty):
  • svg/SVGLineElement.h:
  • svg/SVGLinearGradientElement.cpp: (WebCore::SVGLinearGradientElement::SVGLinearGradientElement): (WebCore::SVGLinearGradientElement::synchronizeProperty):
  • svg/SVGLinearGradientElement.h:
  • svg/SVGList.h: Adapt to const-reference changes. (WebCore::SVGPODListItem::setValue):
  • svg/SVGMPathElement.cpp: (WebCore::SVGMPathElement::SVGMPathElement): (WebCore::SVGMPathElement::synchronizeProperty):
  • svg/SVGMPathElement.h:
  • svg/SVGMarkerElement.cpp: (WebCore::SVGMarkerElement::SVGMarkerElement): (WebCore::SVGMarkerElement::synchronizeProperty):
  • svg/SVGMarkerElement.h:
  • svg/SVGMaskElement.cpp: (WebCore::SVGMaskElement::SVGMaskElement): (WebCore::SVGMaskElement::synchronizeProperty):
  • svg/SVGMaskElement.h:
  • svg/SVGPathElement.cpp: (WebCore::SVGPathElement::SVGPathElement): (WebCore::SVGPathElement::synchronizeProperty):
  • svg/SVGPathElement.h:
  • svg/SVGPatternElement.cpp: (WebCore::SVGPatternElement::SVGPatternElement): (WebCore::SVGPatternElement::synchronizeProperty):
  • svg/SVGPatternElement.h:
  • svg/SVGPolyElement.cpp: (WebCore::SVGPolyElement::SVGPolyElement): (WebCore::SVGPolyElement::svgAttributeChanged): (WebCore::SVGPolyElement::synchronizeProperty):
  • svg/SVGPolyElement.h:
  • svg/SVGRadialGradientElement.cpp: (WebCore::SVGRadialGradientElement::SVGRadialGradientElement): (WebCore::SVGRadialGradientElement::synchronizeProperty):
  • svg/SVGRadialGradientElement.h:
  • svg/SVGRectElement.cpp: (WebCore::SVGRectElement::SVGRectElement): (WebCore::SVGRectElement::synchronizeProperty):
  • svg/SVGRectElement.h:
  • svg/SVGSVGElement.cpp: (WebCore::SVGSVGElement::SVGSVGElement): (WebCore::SVGSVGElement::synchronizeProperty):
  • svg/SVGSVGElement.h:
  • svg/SVGScriptElement.cpp: (WebCore::SVGScriptElement::SVGScriptElement): (WebCore::SVGScriptElement::synchronizeProperty):
  • svg/SVGScriptElement.h:
  • svg/SVGStopElement.cpp: (WebCore::SVGStopElement::SVGStopElement): (WebCore::SVGStopElement::synchronizeProperty):
  • svg/SVGStopElement.h:
  • svg/SVGStyledElement.cpp: (WebCore::SVGStyledElement::SVGStyledElement): (WebCore::SVGStyledElement::parseMappedAttribute): (WebCore::SVGStyledElement::synchronizeProperty):
  • svg/SVGStyledElement.h:
  • svg/SVGStyledTransformableElement.cpp: (WebCore::SVGStyledTransformableElement::SVGStyledTransformableElement): (WebCore::SVGStyledTransformableElement::synchronizeProperty):
  • svg/SVGStyledTransformableElement.h:
  • svg/SVGSwitchElement.cpp: (WebCore::SVGSwitchElement::SVGSwitchElement): (WebCore::SVGSwitchElement::synchronizeProperty):
  • svg/SVGSwitchElement.h:
  • svg/SVGSymbolElement.cpp: (WebCore::SVGSymbolElement::SVGSymbolElement): (WebCore::SVGSymbolElement::synchronizeProperty):
  • svg/SVGSymbolElement.h:
  • svg/SVGTRefElement.cpp: (WebCore::SVGTRefElement::SVGTRefElement): (WebCore::SVGTRefElement::synchronizeProperty):
  • svg/SVGTRefElement.h:
  • svg/SVGTextContentElement.cpp: (WebCore::SVGTextContentElement::SVGTextContentElement): (WebCore::SVGTextContentElement::synchronizeProperty):
  • svg/SVGTextContentElement.h:
  • svg/SVGTextElement.cpp: (WebCore::SVGTextElement::SVGTextElement): (WebCore::SVGTextElement::synchronizeProperty):
  • svg/SVGTextElement.h:
  • svg/SVGTextPathElement.cpp: (WebCore::SVGTextPathElement::SVGTextPathElement): (WebCore::SVGTextPathElement::synchronizeProperty):
  • svg/SVGTextPathElement.h:
  • svg/SVGTextPositioningElement.cpp: (WebCore::SVGTextPositioningElement::SVGTextPositioningElement): (WebCore::SVGTextPositioningElement::synchronizeProperty):
  • svg/SVGTextPositioningElement.h:
  • svg/SVGURIReference.cpp:
  • svg/SVGURIReference.h:
  • svg/SVGUseElement.cpp: (WebCore::SVGUseElement::SVGUseElement): (WebCore::SVGUseElement::synchronizeProperty):
  • svg/SVGUseElement.h:
  • svg/SVGViewElement.cpp: (WebCore::SVGViewElement::SVGViewElement): (WebCore::SVGViewElement::synchronizeProperty):
  • svg/SVGViewElement.h:
  • svg/SVGViewSpec.cpp: (WebCore::SVGViewSpec::SVGViewSpec):
  • svg/SVGViewSpec.h: (WebCore::SVGViewSpec::contextElement):
  • svg/SynchronizablePropertyController.cpp: Removed.
  • svg/SynchronizablePropertyController.h: Removed.
  • svg/SynchronizableTypeWrapper.h: Removed.
  • svg/svgattrs.in: Remove "exportStrings" tag.
  • svg/svgtags.in: Ditto.
  • svg/xlinkattrs.in:
17:35 Changeset [53878] by adele@apple.com
  • 5 edits
    2 adds in trunk

WebCore: Fix for <rdar://problem/7169464> REGRESSION (r47444): PLT is 1% slower due to implementation of :valid and :invalid CSS selectors
https://bugs.webkit.org/show_bug.cgi?id=34029

Reviewed by Darin Adler.

If we never hit the valid or invalid selectors for a particular document,
then we'll skip the validity checks when deciding about style sharing.

  • css/CSSStyleSelector.cpp:

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

  • dom/Document.cpp: (WebCore::Document::Document):
  • dom/Document.h:

(WebCore::Document::considerValidity):
(WebCore::Document::setConsiderValidity):

LayoutTests: Test for <rdar://problem/7169464> REGRESSION (r47444): PLT is 1% slower due to implementation of :valid and :invalid CSS selectors
https://bugs.webkit.org/show_bug.cgi?id=34029

Reviewed by Darin Adler.

  • fast/css/pseudo-valid-dynamic-expected.txt: Added.
  • fast/css/pseudo-valid-dynamic.html: Added.
17:31 Changeset [53877] by jorlow@chromium.org
  • 6 edits in trunk/WebCore

Revert 53797 as it seems to be causing crashes.
https://bugs.webkit.org/show_bug.cgi?id=34153

Patch by Jeremy Orlow <jorlow@chromium.org> on 2010-01-26
Reviewed by Simon Fraser.

  • page/FrameView.cpp:

(WebCore::FrameView::useSlowRepaints):
(WebCore::FrameView::useSlowRepaintsIfNotOverlapped):

  • page/FrameView.h:
  • platform/ScrollView.cpp:

(WebCore::ScrollView::scrollContents):

  • platform/ScrollView.h:
  • rendering/RenderObject.cpp:

(WebCore::RenderObject::styleWillChange):
(WebCore::RenderObject::destroy):

17:20 Changeset [53876] by dimich@chromium.org
  • 2 edits in trunk/WebCore

DumpRenderTree silently crashes at the end of the run because geolocationControllerClient pointer is 0.
https://bugs.webkit.org/show_bug.cgi?id=34191

Reviewed by Steve Falkenburg.

  • page/GeolocationController.cpp:

(WebCore::GeolocationController::~GeolocationController): Add check for m_client being 0.
(WebCore::GeolocationController::addObserver): Ditto.
(WebCore::GeolocationController::removeObserver): Ditto.
(WebCore::GeolocationController::lastPosition): Ditto.

17:09 Changeset [53875] by ukai@chromium.org
  • 2 edits in trunk/LayoutTests

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

Reviewed by Alexey Proskuryakov.

Fix url used in websocket-event-target.html
https://bugs.webkit.org/show_bug.cgi?id=34066

  • websocket/tests/script-tests/websocket-event-target.js:
17:08 Changeset [53874] by simon.fraser@apple.com
  • 8 edits in trunk/WebCore

2010-01-26 Simon Fraser <simon.fraser@apple.com>

Reviewed by Adele Peterson.

Enhance externalRepresentation() to make it more useful for layer debugging
https://bugs.webkit.org/show_bug.cgi?id=34143

Enhance externalRepresentation() to output additional information about layers when certain flags are passed.
The new flags allow you to show all layers (not just those that would paint), to the structure of the z-order
and normal flow lists, and to show which layers are composited.

  • WebCore.base.exp: The signature of externalRepresentation() changed.
  • platform/text/TextStream.h: Add operator<<(void*)
  • platform/text/TextStream.cpp: Implement operator<<(void*)
  • rendering/RenderLayer.h:
  • rendering/RenderLayer.cpp: (showLayerTree): New method outside the WebCore namespace, for ease of calling from gdb.
  • rendering/RenderTreeAsText.h: New behavior flags for externalRepresentation().
  • rendering/RenderTreeAsText.cpp: (WebCore::write): Output compositing information if requested. Also clean up some -1/1 magic numbers with an enum. (WebCore::writeLayers): If requested, show layer nesting via the z-order and normal flow lists. (WebCore::externalRepresentation): New arguments
17:00 Changeset [53873] by enrica@apple.com
  • 3 edits in trunk/LayoutTests

Fixing for failing test on qt.
https://bugs.webkit.org/show_bug.cgi?id=34148

Reviewed by Jon Honeycutt.

Changed url in anchor tag to avoid the additional trailing slash.

  • editing/pasteboard/paste-noscript-xhtml-expected.txt:
  • editing/resources/htmlcontent.html:
16:36 Changeset [53872] by mitz@apple.com
  • 2 edits in trunk/WebCore

Fixed an off-by-one error.

Reviewed by Simon Fraser.

  • rendering/break_lines.cpp:

(WebCore::shouldBreakAfter):

16:22 Changeset [53871] by dimich@chromium.org
  • 6 edits
    5 adds in trunk

Avoid reloading iframe on re-parenting between documents.
https://bugs.webkit.org/show_bug.cgi?id=32848

Reviewed by David Levin.

WebCore:

Achieved by setting a flag on iframe element when it is a target of document.adoptNode(node) operation.
The flag prevents unload/load cycle and is reset once element is attached to a new document.
If iframe is adopted but not actually inserted into the tree, it gets unloaded by async timer
once JS yields, to avoid having active content in non-attached iframe.

Test: fast/frames/iframe-reparenting.html

  • dom/Document.cpp:

(WebCore::Document::adoptNode): If the adopted node is iframe, set a remainsAliveOnRemovalFromTree on it.

  • html/HTMLFrameElementBase.h:
  • html/HTMLFrameElementBase.cpp:

(WebCore::HTMLFrameElementBase::HTMLFrameElementBase):
(WebCore::HTMLFrameElementBase::attach): Skip actual loading of the frame if it has remainsAliveOnRemovalFromTree flag. Reset the flag.
(WebCore::HTMLFrameElementBase::willRemove): Skip unloading the frame if it has remainsAliveOnRemovalFromTree flag set.
(WebCore::HTMLFrameElementBase::setRemainsAliveOnRemovalFromTree): Set the flag, start the async timer to check if the frame was actually attached.
(WebCore::HTMLFrameElementBase::checkAttachedTimerFired):

  • html/HTMLFrameOwnerElement.h:

(WebCore::HTMLFrameOwnerElement::willRemove): Move from private to protected, since it is conditionally called in HTMLFrameElementBase::willRemove now.

LayoutTests:

  • fast/frames/iframe-reparenting-expected.txt: Added.
  • fast/frames/iframe-reparenting.html: Added.
  • fast/frames/resources/iframe-reparenting-frame1.html: Added.
  • fast/frames/resources/iframe-reparenting-frame2.html: Added.
  • fast/frames/resources/iframe-reparenting-iframe-content.html: Added.
16:09 Changeset [53870] by kevino@webkit.org
  • 2 edits in trunk/WebKitTools

[wx] Build fix, make sure stub function returns a value.

14:59 Changeset [53869] by aroben@apple.com
  • 15 edits in trunk

2010-01-26 Adam Roben <aroben@apple.com>

No review, rolling out r53861.
http://trac.webkit.org/changeset/53861
https://bugs.webkit.org/show_bug.cgi?id=33224

Caused 2 regression tests to fail.

  • fast/loader/stateobjects/document-destroyed-navigate-back-expected.txt:
  • fast/loader/stateobjects/document-destroyed-navigate-back.html:

2010-01-26 Adam Roben <aroben@apple.com>

No review, rolling out r53861.
http://trac.webkit.org/changeset/53861
https://bugs.webkit.org/show_bug.cgi?id=33224

Caused 2 regression tests to fail.

  • dom/Document.cpp: (WebCore::Document::detach): (WebCore::Document::registerHistoryItem): (WebCore::Document::unregisterHistoryItem):
  • dom/Document.h:
  • history/BackForwardList.cpp: (WebCore::BackForwardList::pushStateItem):
  • history/BackForwardListChromium.cpp: (WebCore::BackForwardList::pushStateItem):
  • history/HistoryItem.cpp: (WebCore::HistoryItem::HistoryItem): (WebCore::HistoryItem::~HistoryItem): (WebCore::HistoryItem::setStateObject): (WebCore::HistoryItem::setDocument): (WebCore::HistoryItem::documentDetached):
  • history/HistoryItem.h: (WebCore::HistoryItem::document):
  • loader/FrameLoader.cpp: (WebCore::FrameLoader::navigateWithinDocument): (WebCore::FrameLoader::loadItem):
  • loader/HistoryController.cpp: (WebCore::HistoryController::updateBackForwardListForFragmentScroll): (WebCore::HistoryController::pushState): (WebCore::HistoryController::replaceState):
  • loader/RedirectScheduler.cpp: (WebCore::RedirectScheduler::scheduleHistoryNavigation):
  • page/History.cpp: (WebCore::History::stateObjectAdded):
  • page/Page.cpp: (WebCore::Page::goToItem):
14:48 Changeset [53868] by mitz@apple.com
  • 3 edits
    4 adds in trunk

<rdar://problem/7576663> Crash caused by anonymous list item
https://bugs.webkit.org/show_bug.cgi?id=34183

Reviewed by Beth Dakin.

WebCore:

Test: fast/lists/anonymous-items.html

enclosingList() and previousListItem() were DOM-based, but in order to work with anonymous
list items, they need to work with rthe render tree.

  • rendering/RenderListItem.cpp:

(WebCore::isList): Factored out.
(WebCore::enclosingList): Added this variant that takes a RenderObject.
(WebCore::previousListItem): Changed to travers the render tree.
(WebCore::RenderListItem::calcValue): Use the RenderObject version of enclosingList()
(WebCore::RenderListItem::setExplicitValue): Added an assertion.
(WebCore::RenderListItem::clearExplicitValue): Ditto.

LayoutTests:

  • fast/lists/anonymous-items.html: Added.
  • platform/mac/fast/lists/anonymous-items-expected.checksum: Added.
  • platform/mac/fast/lists/anonymous-items-expected.png: Added.
  • platform/mac/fast/lists/anonymous-items-expected.txt: Added.
14:45 Changeset [53867] by bweinstein@apple.com
  • 2 edits
    1 add in trunk/WebCore

Crash in WebKit!WebCore::RenderMenuList::itemStyle
https://bugs.webkit.org/show_bug.cgi?id=34182
<rdar://7087757>

Reviewed by Jon Honeycutt.

Added bounds checks in RenderMenuList to make sure we are
not making an out of bounds check in a vector once an option
element has been deleted. If we are out of bounds, we fall back to
a default value and return early, and in the case of itemStyle, we use a
previous option's style, if it is available.

  • manual-tests/select-delete-item.html: Added.
  • rendering/RenderMenuList.cpp:

(WebCore::RenderMenuList::itemText): If out of bounds check, return early.
(WebCore::RenderMenuList::itemToolTip): Ditto.
(WebCore::RenderMenuList::itemIsEnabled): Ditto.
(WebCore::RenderMenuList::itemStyle): If out of bounds check, try using the 0th index

option style, then fall back to the select's style if that option doesn't exist.

(WebCore::RenderMenuList::itemBackgroundColor): If out of bounds check, return early.
(WebCore::RenderMenuList::itemIsSeparator): Ditto.
(WebCore::RenderMenuList::itemIsLabel): Ditto.
(WebCore::RenderMenuList::itemIsSelected): Ditto.

14:20 Changeset [53866] by barraclough@apple.com
  • 1 edit in trunk/WebCore/ChangeLog

Whoops, fixerate ChangeLog.

14:00 Changeset [53865] by barraclough@apple.com
  • 7 edits in trunk/WebCore

<<<<<<< .mine

13:14 Changeset [53864] by sfalken@apple.com
  • 14 edits in trunk

2010-01-26 Steve Falkenburg <sfalken@apple.com>

Reviewed by Oliver Hunt.

Windows build references non-existent include paths
https://bugs.webkit.org/show_bug.cgi?id=34175

2010-01-26 Steve Falkenburg <sfalken@apple.com>

Reviewed by Oliver Hunt.

Windows build references non-existent include paths
https://bugs.webkit.org/show_bug.cgi?id=34175

  • WebKit.vcproj/WebKit.vcproj:

2010-01-26 Steve Falkenburg <sfalken@apple.com>

Reviewed by Oliver Hunt.

Windows build references non-existent include paths
https://bugs.webkit.org/show_bug.cgi?id=34175

  • WebCore.vcproj/QTMovieWin.vcproj:
  • WebCore.vcproj/WebCoreCommon.vsprops:

2010-01-26 Steve Falkenburg <sfalken@apple.com>

Reviewed by Oliver Hunt.

Windows build references non-existent include paths
https://bugs.webkit.org/show_bug.cgi?id=34175

  • DumpRenderTree/win/DumpRenderTree.vcproj:
  • DumpRenderTree/win/ImageDiff.vcproj:
  • DumpRenderTree/win/TestNetscapePlugin/TestNetscapePlugin.vcproj:
12:20 Changeset [53863] by cfleizach@apple.com
  • 3 edits in trunk/LayoutTests

Another attempt to get this test to pass for GTK.

Add ability for image maps to be focused via tabbing
https://bugs.webkit.org/show_bug.cgi?id=17513

  • fast/events/tab-imagemap-expected.txt:
  • fast/events/tab-imagemap.html:
12:02 Changeset [53862] by kenneth@webkit.org
  • 3 edits
    2 adds in trunk/WebKitTools

[Qt] Separating out the UrlLoader from the QtLauncher in it's
own implementation and header file.

Reviewed by Ariya Hidayat.

  • QtLauncher/main.cpp:

(main):

  • QtLauncher/urlloader.cpp: Added.

(UrlLoader::UrlLoader):
(UrlLoader::loadNext):
(UrlLoader::init):
(UrlLoader::getUrl):

  • QtLauncher/urlloader.h: Added.
11:30 Changeset [53861] by darin@chromium.org
  • 15 edits in trunk

2010-01-25 Darin Fisher <darin@chromium.org>

Reviewed by Brady Eidson.

Chains of history items representing same-document navigation need to
always remember that association

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

Replace HistoryItem's Document pointer with a DocumentSequenceNumber.
During session history traversal, if the current HistoryItem and the
target HistoryItem have the same DocumentSequenceNumber, then it means
that the current Document should remain.

NOTE: To support Chromium's serialization of HistoryItems, I generate
DocumentSequenceNumbers that are unique across application launches.
DocumentSequenceNumbers are generated using a counter initialized with
the time of day.

Test: fast/loader/stateobjects/document-destroyed-navigate-back.html

  • dom/Document.cpp: (WebCore::Document::detach):
  • dom/Document.h:
  • history/BackForwardList.cpp: (WebCore::BackForwardList::pushStateItem):
  • history/BackForwardListChromium.cpp: (WebCore::BackForwardList::pushStateItem):
  • history/HistoryItem.cpp: (WebCore::generateDocumentSequenceNumber): (WebCore::HistoryItem::HistoryItem): (WebCore::HistoryItem::~HistoryItem): (WebCore::HistoryItem::setStateObject):
  • history/HistoryItem.h: (WebCore::HistoryItem::setDocumentSequenceNumber): (WebCore::HistoryItem::documentSequenceNumber):
  • loader/FrameLoader.cpp: (WebCore::FrameLoader::navigateWithinDocument): (WebCore::FrameLoader::loadItem):
  • loader/HistoryController.cpp: (WebCore::HistoryController::updateBackForwardListForFragmentScroll): (WebCore::HistoryController::pushState): (WebCore::HistoryController::replaceState):
  • loader/RedirectScheduler.cpp: (WebCore::RedirectScheduler::scheduleHistoryNavigation):
  • page/History.cpp: (WebCore::History::stateObjectAdded):
  • page/Page.cpp: (WebCore::Page::goToItem):
11:22 Changeset [53860] by oliver@apple.com
  • 2 edits in trunk/JavaScriptCore

2010-01-26 Oliver Hunt <oliver@apple.com>

Reviewed by Geoffrey Garen.

Using JavaScriptCore API with a webkit vended context can result in slow script dialog
https://bugs.webkit.org/show_bug.cgi?id=34172

Make the APIShim correctly increment and decrement the timeout
entry counter.

  • API/APIShims.h: (JSC::APIEntryShimWithoutLock::APIEntryShimWithoutLock): (JSC::APIEntryShimWithoutLock::~APIEntryShimWithoutLock): (JSC::APICallbackShim::APICallbackShim): (JSC::APICallbackShim::~APICallbackShim):
11:00 Changeset [53859] by cfleizach@apple.com
  • 3 edits in trunk/LayoutTests

One more attempt to get GTK version to pass.

Add ability for image maps to be focused via tabbing
https://bugs.webkit.org/show_bug.cgi?id=17513

  • fast/events/tab-imagemap-expected.txt:
  • fast/events/tab-imagemap.html:
10:51 Changeset [53858] by cfleizach@apple.com
  • 2 edits in trunk/LayoutTests

Unreviewed.

Attempt to make this test pass on GTK.

Add ability for image maps to be focused via tabbing
https://bugs.webkit.org/show_bug.cgi?id=17513

  • fast/events/tab-imagemap.html:
10:06 Changeset [53857] by cfleizach@apple.com
  • 25 edits
    2 adds in trunk

Add ability for image maps to be focused via tabbing
https://bugs.webkit.org/show_bug.cgi?id=17513

Reviewed by Darin Adler.

WebCore:

Test: fast/events/tab-imagemap.html

  • accessibility/AXObjectCache.cpp:

(WebCore::AXObjectCache::focusedImageMapUIElement):
(WebCore::AXObjectCache::focusedUIElementForPage):

  • accessibility/AXObjectCache.h:
  • accessibility/AccessibilityImageMapLink.h:

(WebCore::AccessibilityImageMapLink::areaElement):
(WebCore::AccessibilityImageMapLink::mapElement):
(WebCore::AccessibilityImageMapLink::isImageMapLink):

  • accessibility/AccessibilityObject.h:

(WebCore::AccessibilityObject::isImageMapLink):

  • accessibility/AccessibilityRenderObject.cpp:

(WebCore::AccessibilityRenderObject::accessibilityParentForImageMap):

  • html/HTMLAreaElement.cpp:

(WebCore::HTMLAreaElement::getPath):
(WebCore::HTMLAreaElement::getRect):
(WebCore::HTMLAreaElement::imageElement):
(WebCore::HTMLAreaElement::isKeyboardFocusable):
(WebCore::HTMLAreaElement::isFocusable):
(WebCore::HTMLAreaElement::dispatchBlurEvent):
(WebCore::HTMLAreaElement::updateFocusAppearance):
(WebCore::HTMLAreaElement::supportsFocus):

  • html/HTMLAreaElement.h:
  • html/HTMLMapElement.cpp:

(WebCore::HTMLMapElement::imageElement):

  • html/HTMLMapElement.h:
  • platform/graphics/GraphicsContext.h:
  • platform/graphics/cairo/GraphicsContextCairo.cpp:

(WebCore::GraphicsContext::drawFocusRing):

  • platform/graphics/haiku/GraphicsContextHaiku.cpp:

(WebCore::GraphicsContext::drawFocusRing):

  • platform/graphics/mac/GraphicsContextMac.mm:

(WebCore::drawFocusRingToContext):
(WebCore::GraphicsContext::drawFocusRing):

  • platform/graphics/qt/GraphicsContextQt.cpp:

(WebCore::GraphicsContext::drawFocusRing):

  • platform/graphics/skia/GraphicsContextSkia.cpp:

(WebCore::GraphicsContext::drawFocusRing):

  • platform/graphics/win/GraphicsContextCGWin.cpp:

(WebCore::GraphicsContext::drawFocusRing):

  • platform/graphics/wince/GraphicsContextWince.cpp:

(WebCore::GraphicsContext::drawFocusRing):

  • platform/graphics/wx/GraphicsContextWx.cpp:

(WebCore::GraphicsContext::drawFocusRing):

  • rendering/RenderImage.cpp:

(WebCore::RenderImage::paint):
(WebCore::RenderImage::paintFocusRings):
(WebCore::RenderImage::imageMap):

  • rendering/RenderImage.h:
  • rendering/RenderReplaced.h:

LayoutTests:

  • fast/events/resources/tabindex-focus-blur-all.js:

(test):
(testProgrammaticFocus):

  • fast/events/tab-imagemap-expected.txt: Added.
  • fast/events/tab-imagemap.html: Added.
  • fast/events/tabindex-focus-blur-all-expected.txt:
08:39 Changeset [53856] by pfeldman@chromium.org
  • 6 edits in trunk/WebCore

2010-01-26 Pavel Feldman <pfeldman@chromium.org>

Reviewed by Timothy Hatcher.

Web Inspector: add support for breakpoints, messages and search into the SourceFrame2.

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

08:33 Changeset [53855] by pfeldman@chromium.org
  • 11 edits in trunk/WebCore

2010-01-26 Pavel Feldman <pfeldman@chromium.org>

Reviewed by Timothy Hather.

Web Inspector: add methods for getting resource content from within frontend.

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

  • inspector/InspectorBackend.cpp: (WebCore::InspectorBackend::getResourceContent):
  • inspector/InspectorBackend.h:
  • inspector/InspectorBackend.idl:
  • inspector/InspectorFrontend.cpp: (WebCore::InspectorFrontend::didGetResourceContent):
  • inspector/InspectorFrontend.h:
  • inspector/InspectorFrontendHost.cpp:
  • inspector/InspectorFrontendHost.h:
  • inspector/InspectorFrontendHost.idl:
  • inspector/front-end/InspectorBackendStub.js: (.WebInspector.InspectorBackendStub.prototype.getResourceContent):
  • inspector/front-end/InspectorFrontendHostStub.js:
07:18 QtWebKitBackportingFixes edited by hausmann@webkit.org
(diff)
07:09 Changeset [53854] by hausmann@webkit.org
  • 2 edits in trunk/JavaScriptCore

[Qt] Fix compilation of QtScript with non-gcc compilers

Variable length stack arrays are a gcc extension. Use QVarLengthArray
as a more portable solution that still tries to allocate on the stack
first.

  • qt/api/qscriptvalue_p.h:

(QScriptValuePrivate::call):

06:13 Changeset [53853] by hausmann@webkit.org
  • 2 edits in trunk/JavaScriptCore

[Qt] Fix the build on platforms without JIT support.

Reviewed by Tor Arne Vestbø.

The JIT support should be determined at compile-time via wtf/Platform.h

  • qt/api/QtScript.pro:
06:02 Changeset [53852] by pfeldman@chromium.org
  • 2 edits in trunk/WebCore

2010-01-26 Pavel Feldman <pfeldman@chromium.org>

Not reviewed: added null check into the timeline agent getter.

  • inspector/InspectorTimelineAgent.h: (WebCore::InspectorTimelineAgent::retrieve):
05:39 Changeset [53851] by eric@webkit.org
  • 2 edits in trunk/WebKit/qt

2010-01-26 Simon Hausmann <simon.hausmann@nokia.com>

Reviewed by Kenneth Rohde Christiansen.

[Qt] Show comboboxes on Maemo 5
https://bugs.webkit.org/show_bug.cgi?id=34088

Don't try to show the combobox by simulating a mouse event from QCursor::pos() to
get the combobox position right. The position on Maemo 5 is independent from the mouse
and there's no QCursor::pos().

  • WebCoreSupport/QtFallbackWebPopup.cpp: (WebCore::QtFallbackWebPopup::show):
05:00 Changeset [53850] by eric@webkit.org
  • 5 edits
    21 adds in trunk

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

Reviewed by Simon Hausmann.

First steps of the QtScript API.

Two new classes were created; QScriptEngine and QScriptValue.
The first should encapsulate a javascript context and the second a script
value.

This API is still in development, so it isn't compiled by default.
To trigger compilation, pass --qmakearg="CONFIG+=build-qtscript" to
build-webkit.

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

  • WebKit.pro:

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

Reviewed by Simon Hausmann.

First steps of the QtScript API.

Two new classes were created; QScriptEngine and QScriptValue.
The first should encapsulate a javascript context and the second a script
value.

This API is still in development, so it isn't compiled by default.
To trigger compilation, pass --qmakearg="CONFIG+=build-qtscript" to
build-webkit.

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

  • qt/api/QtScript.pro: Added.
  • qt/api/qscriptconverter_p.h: Added. (QScriptConverter::toString):
  • qt/api/qscriptengine.cpp: Added. (QScriptEngine::QScriptEngine): (QScriptEngine::~QScriptEngine): (QScriptEngine::evaluate): (QScriptEngine::collectGarbage):
  • qt/api/qscriptengine.h: Added.
  • qt/api/qscriptengine_p.cpp: Added. (QScriptEnginePrivate::QScriptEnginePrivate): (QScriptEnginePrivate::~QScriptEnginePrivate): (QScriptEnginePrivate::evaluate):
  • qt/api/qscriptengine_p.h: Added. (QScriptEnginePrivate::get): (QScriptEnginePrivate::collectGarbage): (QScriptEnginePrivate::makeJSValue): (QScriptEnginePrivate::context):
  • qt/api/qscriptvalue.cpp: Added. (QScriptValue::QScriptValue): (QScriptValue::~QScriptValue): (QScriptValue::isValid): (QScriptValue::isBool): (QScriptValue::isBoolean): (QScriptValue::isNumber): (QScriptValue::isNull): (QScriptValue::isString): (QScriptValue::isUndefined): (QScriptValue::isError): (QScriptValue::isObject): (QScriptValue::isFunction): (QScriptValue::toString): (QScriptValue::toNumber): (QScriptValue::toBool): (QScriptValue::toBoolean): (QScriptValue::toInteger): (QScriptValue::toInt32): (QScriptValue::toUInt32): (QScriptValue::toUInt16): (QScriptValue::call): (QScriptValue::engine): (QScriptValue::operator=): (QScriptValue::equals): (QScriptValue::strictlyEquals):
  • qt/api/qscriptvalue.h: Added. (QScriptValue::):
  • qt/api/qscriptvalue_p.h: Added. (QScriptValuePrivate::): (QScriptValuePrivate::get): (QScriptValuePrivate::QScriptValuePrivate): (QScriptValuePrivate::isValid): (QScriptValuePrivate::isBool): (QScriptValuePrivate::isNumber): (QScriptValuePrivate::isNull): (QScriptValuePrivate::isString): (QScriptValuePrivate::isUndefined): (QScriptValuePrivate::isError): (QScriptValuePrivate::isObject): (QScriptValuePrivate::isFunction): (QScriptValuePrivate::toString): (QScriptValuePrivate::toNumber): (QScriptValuePrivate::toBool): (QScriptValuePrivate::toInteger): (QScriptValuePrivate::toInt32): (QScriptValuePrivate::toUInt32): (QScriptValuePrivate::toUInt16): (QScriptValuePrivate::equals): (QScriptValuePrivate::strictlyEquals): (QScriptValuePrivate::assignEngine): (QScriptValuePrivate::call): (QScriptValuePrivate::engine): (QScriptValuePrivate::context): (QScriptValuePrivate::value): (QScriptValuePrivate::object): (QScriptValuePrivate::inherits): (QScriptValuePrivate::isJSBased): (QScriptValuePrivate::isNumberBased): (QScriptValuePrivate::isStringBased):
  • qt/api/qtscriptglobal.h: Added.
  • qt/tests/qscriptengine/qscriptengine.pro: Added.
  • qt/tests/qscriptengine/tst_qscriptengine.cpp: Added. (tst_QScriptEngine::tst_QScriptEngine): (tst_QScriptEngine::~tst_QScriptEngine): (tst_QScriptEngine::init): (tst_QScriptEngine::cleanup): (tst_QScriptEngine::collectGarbage): (tst_QScriptEngine::evaluate):
  • qt/tests/qscriptvalue/qscriptvalue.pro: Added.
  • qt/tests/qscriptvalue/tst_qscriptvalue.cpp: Added. (tst_QScriptValue::tst_QScriptValue): (tst_QScriptValue::~tst_QScriptValue): (tst_QScriptValue::init): (tst_QScriptValue::cleanup): (tst_QScriptValue::ctor): (tst_QScriptValue::toString_data): (tst_QScriptValue::toString): (tst_QScriptValue::copyConstructor_data): (tst_QScriptValue::copyConstructor): (tst_QScriptValue::assignOperator_data): (tst_QScriptValue::assignOperator): (tst_QScriptValue::dataSharing): (tst_QScriptValue::constructors_data): (tst_QScriptValue::constructors): (tst_QScriptValue::call):
  • qt/tests/tests.pri: Added.
  • qt/tests/tests.pro: Added.

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

Reviewed by Simon Hausmann.

First steps of the QtScript API.

Two new classes were created; QScriptEngine and QScriptValue.
The first should encapsulate a javascript context and the second a script
value.

This API is still in development, so it isn't compiled by default.
To trigger compilation, pass --qmakearg="CONFIG+=build-qtscript" to
build-webkit.

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

  • docs/qtwebkit.qdocconf:
04:05 Changeset [53849] by eric@webkit.org
  • 9 edits
    1 copy
    1 add in trunk/WebCore

2010-01-26 Steve Block <steveblock@google.com>

Reviewed by Adam Barth.

Moves JSC-specific classes from bridge/jni/JNIBridge to bridge/jni/jsc/JNIBridgeJSC
https://bugs.webkit.org/show_bug.cgi?id=33958

This allows bridge/jni/JNIBridge to be used with both JSC and V8.
A later change will add the V8 equivalent of these JSC-specific classes.

No new tests, refactoring only.

  • Android.jscbindings.mk: Modified. Added JNIBridgeJSC.cpp
  • GNUmakefile.am: Modified. Added JNIBridgeJSC.h
  • WebCore.xcodeproj/project.pbxproj: Modified. Added JNIBridgeJSC.[cpp|h]
  • bridge/jni/JNIBridge.cpp: Modified. Moved JavaField and JavaArray to JNIBridgeJSC.cpp (appendClassName): Modfied. Guard calls to JSLock (JavaMethod::signature): Modfied. Guard calls to JSLock
  • bridge/jni/JNIBridge.h: Modified. Moved JavaField and JavaArray to JNIBridgeJSC.h
  • bridge/jni/jsc/JNIBridgeJSC.cpp: Copied from WebCore/bridge/jni/JNIBridge.cpp.
  • bridge/jni/jsc/JNIBridgeJSC.h: Copied from WebCore/bridge/jni/JNIBridge.h.
  • bridge/jni/jsc/JNIUtilityPrivate.cpp: Modified. Include JNIBridgeJSC.h
  • bridge/jni/jsc/JavaClassJSC.h: Modified. Include JNIBridgeJSC.h
  • bridge/jni/jsc/JavaInstanceJSC.cpp: Modified. Include JNIBridgeJSC.h
03:06 Changeset [53848] by jocelyn.turcotte@nokia.com
  • 3 edits in trunk/WebCore

Unreviewed build fix.

[Qt] Fix Windows build when sqlite is not available.

  • WebCore.pri:
  • WebCore.pro:
02:44 QtWebKitBackportingFixes edited by hausmann@webkit.org
(diff)
02:40 Changeset [53847] by hausmann@webkit.org
  • 4 edits
    1 delete in trunk

WebCore: [Qt] JavaScript prompt is currently broken.
https://bugs.webkit.org/show_bug.cgi?id=30914

Patch by Holger Hans Peter Freyther <zecke@selfish.org> on 2010-01-26
Reviewed by Simon Hausmann.

Remove the manual test case in favor of an automated
test case in WebKit/qt/tests/qwebpage.

  • manual-tests/qt/java-script-prompt.html: Removed.

WebKit/qt: [Qt] JavaScript prompt is currently broken
https://bugs.webkit.org/show_bug.cgi?id=30914

Patch by Holger Hans Peter Freyther <zecke@selfish.org> on 2010-01-26
Reviewed by Simon Hausmann.

In r52152 a patch was landed to convert a null QString
to an empty WebCore::String in case the prompt was accepted
but the default implementation returned the null QString.

The patch tried to avoid assign to result twice and
was not checking the QString if it is null but the default
value. This lead to always returning an empty string on
successful prompts. Fix it by checking the variable 'x'
for isNull.

The manual test case used didn't cover the case of non
empty input, replace it with an automatic test case that
should cover all cases.

  • WebCoreSupport/ChromeClientQt.cpp:

(WebCore::ChromeClientQt::runJavaScriptPrompt): Fix the bug.

  • tests/qwebpage/tst_qwebpage.cpp: Add automatic test case

(JSPromptPage::JSPromptPage):
(JSPromptPage::javaScriptPrompt):
(tst_QWebPage::testJSPrompt):

02:18 Changeset [53846] by pfeldman@chromium.org
  • 6 edits
    2 moves
    4 adds in trunk/WebCore

2010-01-26 Pavel Feldman <pfeldman@chromium.org>

Reviewed by Timothy Hatcher.

Web Inspector: Implement HTML and CSS highlighters.

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

  • WebCore.gypi:
  • WebCore.vcproj/WebCore.vcproj:
  • inspector/front-end/JavaScriptTokenizer.js: Removed.
  • inspector/front-end/JavaScriptTokenizer.re2js: Removed.
  • inspector/front-end/SourceCSSTokenizer.js: Added. (WebInspector.SourceCSSTokenizer): (WebInspector.SourceCSSTokenizer.prototype._stringToken): (WebInspector.SourceCSSTokenizer.prototype._isPropertyValue): (WebInspector.SourceCSSTokenizer.prototype.nextToken):
  • inspector/front-end/SourceCSSTokenizer.re2js: Added.
  • inspector/front-end/SourceHTMLTokenizer.js: Added. (WebInspector.SourceHTMLTokenizer): (WebInspector.SourceHTMLTokenizer.prototype._stringToken): (WebInspector.SourceHTMLTokenizer.prototype.nextToken):
  • inspector/front-end/SourceHTMLTokenizer.re2js: Added.
  • inspector/front-end/SourceJavaScriptTokenizer.js: Added. (WebInspector.SourceJavaScriptTokenizer): (WebInspector.SourceJavaScriptTokenizer.prototype.nextToken):
  • inspector/front-end/SourceJavaScriptTokenizer.re2js: Added.
  • inspector/front-end/TextEditorHighlighter.js: (WebInspector.TextEditorHighlighter): (WebInspector.TextEditorHighlighter.prototype._highlightLines): (WebInspector.TextEditorHighlighter.Tokenizer): (WebInspector.TextEditorHighlighter.Tokenizer.prototype.set line): (WebInspector.TextEditorHighlighter.Tokenizer.prototype.set condition): (WebInspector.TextEditorHighlighter.Tokenizer.prototype.get condition): (WebInspector.TextEditorHighlighter.Tokenizer.prototype.hasCondition): (WebInspector.TextEditorHighlighter.Tokenizer.prototype.getLexCondition): (WebInspector.TextEditorHighlighter.Tokenizer.prototype.setLexCondition): (WebInspector.TextEditorHighlighter.Tokenizer.prototype._charAt):
  • inspector/front-end/WebKit.qrc:
  • inspector/front-end/inspector.html:
02:04 Changeset [53845] by hausmann@webkit.org
  • 3 edits
    1 add in trunk

REGRESSION(r53835): Fix editing/pasteboard/paste-noscript-xhtml.xhtml
https://bugs.webkit.org/show_bug.cgi?id=34157

Reviewed by Holger Freyther.

WebCore:

Pass the FragmentScriptingPermission correctly through to the DOM
and disallow scripting elements in parseEndElement(), similar to
the libxml tokenizer change in r53835.

  • dom/XMLTokenizerQt.cpp:

(WebCore::handleElementNamespaces):
(WebCore::handleElementAttributes):
(WebCore::XMLTokenizer::parseStartElement):
(WebCore::XMLTokenizer::parseEndElement):

LayoutTests:

Add Qt specific result for this test that differs from the cross-platform
result in only one character: In htmlcontent.html the href attribute value
is http://www.cnn.com, which somehow becomes http://www.cnn.com/ in the cross
platform result. With the Qt xml parser that attribute is somehow preserved
and so our result does not have the trailing slash.

  • platform/qt/editing/pasteboard/paste-noscript-xhtml-expected.txt: Added.
01:42 Changeset [53844] by tkent@chromium.org
  • 2 edits
    2 adds
    5 deletes in trunk/LayoutTests

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

Reviewed by Shinichiro Hamaji.

Convert textarea-rows-cols.html to dumpAsText()
https://bugs.webkit.org/show_bug.cgi?id=34074

  • fast/forms/script-tests/textarea-rows-cols.js: Added.
  • fast/forms/textarea-rows-cols-expected.txt: Added.
  • fast/forms/textarea-rows-cols.html:
  • platform/mac/fast/forms/textarea-rows-cols-expected.checksum: Removed.
  • platform/mac/fast/forms/textarea-rows-cols-expected.png: Removed.
  • platform/mac/fast/forms/textarea-rows-cols-expected.txt: Removed.
  • platform/qt/fast/forms/textarea-rows-cols-expected.txt: Removed.
  • platform/win/fast/forms/textarea-rows-cols-expected.txt: Removed.
01:40 Changeset [53843] by eric@webkit.org
  • 2 edits in trunk/WebCore

2010-01-26 Garret Kelly <gdk@chromium.org>

Reviewed by Eric Seidel.

Add missing declaration for the feMorphology SVG element.
https://bugs.webkit.org/show_bug.cgi?id=34151

  • bindings/v8/V8DOMWrapper.cpp:
00:30 Changeset [53842] by zoltan@webkit.org
  • 2 edits in trunk/WebCore

Unreviewed build fix.

Patch by Andras Becsi <abecsi@inf.u-szeged.hu> on 2010-01-26
[Qt] Build fix after API changes in r53835.

No new tests needed.

  • dom/XMLTokenizerQt.cpp:

(WebCore::XMLTokenizer::XMLTokenizer):
(WebCore::parseXMLDocumentFragment):

01/25/10:

23:36 Changeset [53841] by cjerdonek@webkit.org
  • 7 edits in trunk/WebKitTools

2010-01-25 Chris Jerdonek <cjerdonek@webkit.org>

Reviewed by Shinichiro Hamaji.

Refactored check-webkit-style by removing the file path
parameter from the style error handler functions.

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

  • Scripts/webkitpy/style/checker.py:
    • Added _default_style_error_handler() to StyleChecker class.
    • Moved handle_style_error() to inside _default_style_error_handler().
  • Scripts/webkitpy/style/checker_unittest.py:
    • Removed file path from calls to error handler.
  • Scripts/webkitpy/style/cpp_style.py:
    • Removed file path from calls to error handler.
  • Scripts/webkitpy/style/cpp_style_unittest.py:
    • Removed file path from calls to error handler.
  • Scripts/webkitpy/style/text_style.py:
    • Removed file path from calls to error handler.
  • Scripts/webkitpy/style/text_style_unittest.py:
    • Removed file path from calls to error handler.
22:23 Changeset [53840] by jorlow@chromium.org
  • 17 edits
    1 copy
    21 adds
    23 deletes in trunk

2010-01-22 Jeremy Orlow <jorlow@chromium.org>

Reviewed by Darin Adler.

Make storage events match the spec.
https://bugs.webkit.org/show_bug.cgi?id=30546

Update the storageEvent algorithm to match the change in WebCore.

  • src/StorageAreaProxy.cpp: (WebCore::StorageAreaProxy::storageEvent):

2010-01-22 Jeremy Orlow <jorlow@chromium.org>

Reviewed by Darin Adler.

Make storage events match the spec.
https://bugs.webkit.org/show_bug.cgi?id=30546

This meat of the patch I just posted is very simple. It's just making events
asynchronous, not posting them to the frame that generated them, passing a null
for the key when issuing a clear storage event, and making the events
non-cancelable/non-bubbleable...all of which are clearly stated in the spec.

The asynchronous and not posting to the frame that generated them forced me to
re-write all the layout tests that dealt with storage events. There's a lot of
code there, but I tried to be fairly careful to ensure that test coverage did
not shrink in any area.

Tests: storage/domstorage/events/basic-body-attribute.html

storage/domstorage/events/basic-setattribute.html
storage/domstorage/events/basic.html
storage/domstorage/events/case-sensitive.html
storage/domstorage/events/documentURI.html

  • dom/Document.cpp: (WebCore::Document::Document): (WebCore::Document::enqueueStorageEvent): (WebCore::Document::storageEventTimerFired):
  • dom/Document.h:
  • storage/StorageEvent.cpp: (WebCore::StorageEvent::StorageEvent):
  • storage/StorageEvent.idl:
  • storage/StorageEventDispatcher.cpp: (WebCore::StorageEventDispatcher::dispatch):

2010-01-22 Jeremy Orlow <jorlow@chromium.org>

Reviewed by Darin Adler.

Make storage events match the spec.
https://bugs.webkit.org/show_bug.cgi?id=30546

This meat of the patch I just posted is very simple. It's just making events
asynchronous, not posting them to the frame that generated them, passing a null
for the key when issuing a clear storage event, and making the events
non-cancelable/non-bubbleable...all of which are clearly stated in the spec.

The asynchronous and not posting to the frame that generated them forced me to
re-write all the layout tests that dealt with storage events. There's a lot of
code there, but I tried to be fairly careful to ensure that test coverage did
not shrink in any area.

  • storage/domstorage/complex-values-expected.txt:
  • storage/domstorage/documentURI-expected.txt: Removed.
  • storage/domstorage/documentURI.html: Removed.
  • storage/domstorage/events: Added.
  • storage/domstorage/events/basic-body-attribute-expected.txt: Added.
  • storage/domstorage/events/basic-body-attribute.html: Added.
  • storage/domstorage/events/basic-expected.txt: Added.
  • storage/domstorage/events/basic-setattribute-expected.txt: Added.
  • storage/domstorage/events/basic-setattribute.html: Added.
  • storage/domstorage/events/basic.html: Added.
  • storage/domstorage/events/case-sensitive-expected.txt: Added.
  • storage/domstorage/events/case-sensitive.html: Added.
  • storage/domstorage/events/documentURI-expected.txt: Added.
  • storage/domstorage/events/documentURI.html: Added.
  • storage/domstorage/events/resources: Added.
  • storage/domstorage/events/resources/body-event-handler.html: Added.
  • storage/domstorage/events/resources/eventTestHarness.js: Added.
  • storage/domstorage/events/resources/setattribute-event-handler.html: Added.
  • storage/domstorage/events/script-tests: Added.
  • storage/domstorage/events/script-tests/TEMPLATE.html: Copied from LayoutTests/storage/domstorage/script-tests/TEMPLATE.html.
  • storage/domstorage/events/script-tests/basic-body-attribute.js: Added.
  • storage/domstorage/events/script-tests/basic-setattribute.js: Added.
  • storage/domstorage/events/script-tests/basic.js: Added.
  • storage/domstorage/events/script-tests/case-sensitive.js: Added.
  • storage/domstorage/events/script-tests/documentURI.js: Added.
  • storage/domstorage/localstorage/iframe-events-expected.txt: Removed.
  • storage/domstorage/localstorage/iframe-events.html: Removed.
  • storage/domstorage/localstorage/index-get-and-set-expected.txt:
  • storage/domstorage/localstorage/index-get-and-set.html:
  • storage/domstorage/localstorage/onstorage-attribute-markup-expected.txt: Removed.
  • storage/domstorage/localstorage/onstorage-attribute-markup.html: Removed.
  • storage/domstorage/localstorage/onstorage-attribute-setattribute-expected.txt: Removed.
  • storage/domstorage/localstorage/onstorage-attribute-setattribute.html: Removed.
  • storage/domstorage/localstorage/onstorage-attribute-setwindow-expected.txt: Removed.
  • storage/domstorage/localstorage/onstorage-attribute-setwindow.html: Removed.
  • storage/domstorage/localstorage/simple-events-expected.txt: Removed.
  • storage/domstorage/localstorage/simple-events.html: Removed.
  • storage/domstorage/script-tests/complex-values.js:
  • storage/domstorage/script-tests/documentURI.js: Removed.
  • storage/domstorage/sessionstorage/iframe-events-expected.txt: Removed.
  • storage/domstorage/sessionstorage/iframe-events.html: Removed.
  • storage/domstorage/sessionstorage/index-get-and-set-expected.txt:
  • storage/domstorage/sessionstorage/index-get-and-set.html:
  • storage/domstorage/sessionstorage/onstorage-attribute-markup-expected.txt: Removed.
  • storage/domstorage/sessionstorage/onstorage-attribute-markup.html: Removed.
  • storage/domstorage/sessionstorage/onstorage-attribute-setattribute-expected.txt: Removed.
  • storage/domstorage/sessionstorage/onstorage-attribute-setattribute.html: Removed.
  • storage/domstorage/sessionstorage/onstorage-attribute-setwindow-expected.txt: Removed.
  • storage/domstorage/sessionstorage/onstorage-attribute-setwindow.html: Removed.
  • storage/domstorage/sessionstorage/simple-events-expected.txt: Removed.
  • storage/domstorage/sessionstorage/simple-events.html: Removed.
  • storage/domstorage/window-attributes-exist-expected.txt:
  • storage/domstorage/window-attributes-exist.html:
22:00 Changeset [53839] by mitz@apple.com
  • 6 edits
    3 adds in trunk

<rdar://problem/7573493> Error with line break inside ?&raquo; pair of characters.
https://bugs.webkit.org/show_bug.cgi?id=17475

Reviewed by Darin Adler.

WebCore:

Test: fast/text/line-break-after-question-mark.html

Instead of unconditionally allowing lines to break after a question mark, which was intended
to mimic Internet Explorer, apply the Unicode line breaking behavior after a question mark,
with one exception in the ASCII range to match IE, namely not allowing a line break between
a question mark and a vertical line.

  • rendering/break_lines.cpp:

(WebCore::shouldBreakAfter): Added a next character parameter. Changed to consult a table
for the question mark case in order to keep it fast and not require a text break iterator in
the ASCII case.
(WebCore::nextBreakablePosition): Pass the next character to shouldBreakAfter.

LayoutTests:

  • fast/text/line-break-after-question-mark-expected.txt: Added.
  • fast/text/line-break-after-question-mark.html: Added.
  • fast/text/script-tests/line-break-after-question-mark.js: Added.

():

  • platform/mac/tables/mozilla/bugs/bug6674-expected.checksum: Updated.
  • platform/mac/tables/mozilla/bugs/bug6674-expected.png: Updated.
  • platform/mac/tables/mozilla/bugs/bug6674-expected.txt: Updated.
21:57 Changeset [53838] by pkasting@chromium.org
  • 92 edits in trunk

WebCore: Mac scrollbar thumbs were drawn with the wrong size.
https://bugs.webkit.org/show_bug.cgi?id=34049

Reviewed by Dan Bernstein.

  • platform/chromium/ScrollbarThemeChromiumMac.mm:

(WebCore::ScrollbarThemeChromiumMac::paint):

  • platform/mac/ScrollbarThemeMac.mm:

(WebCore::ScrollbarThemeMac::paint):

LayoutTests: Rebaseline pixel tests affected by changes in scrollbar thumb size.
https://bugs.webkit.org/show_bug.cgi?id=34049

Reviewed by Dan Bernstein.

  • platform/mac/compositing/overflow/overflow-scroll-expected.checksum:
  • platform/mac/compositing/overflow/overflow-scroll-expected.png:
  • platform/mac/fast/backgrounds/repeat/negative-offset-repeat-transformed-expected.checksum:
  • platform/mac/fast/backgrounds/repeat/negative-offset-repeat-transformed-expected.png:
  • platform/mac/fast/body-propagation/overflow/002-expected.checksum:
  • platform/mac/fast/body-propagation/overflow/002-expected.png:
  • platform/mac/fast/body-propagation/overflow/002-xhtml-expected.checksum:
  • platform/mac/fast/body-propagation/overflow/002-xhtml-expected.png:
  • platform/mac/fast/body-propagation/overflow/003-declarative-expected.checksum:
  • platform/mac/fast/body-propagation/overflow/003-declarative-expected.png:
  • platform/mac/fast/body-propagation/overflow/003-expected.checksum:
  • platform/mac/fast/body-propagation/overflow/003-expected.png:
  • platform/mac/fast/body-propagation/overflow/003-xhtml-expected.checksum:
  • platform/mac/fast/body-propagation/overflow/003-xhtml-expected.png:
  • platform/mac/fast/body-propagation/overflow/004-declarative-expected.checksum:
  • platform/mac/fast/body-propagation/overflow/004-declarative-expected.png:
  • platform/mac/fast/body-propagation/overflow/004-expected.checksum:
  • platform/mac/fast/body-propagation/overflow/004-expected.png:
  • platform/mac/fast/body-propagation/overflow/004-xhtml-expected.checksum:
  • platform/mac/fast/body-propagation/overflow/004-xhtml-expected.png:
  • platform/mac/fast/body-propagation/overflow/007-declarative-expected.checksum:
  • platform/mac/fast/body-propagation/overflow/007-declarative-expected.png:
  • platform/mac/fast/body-propagation/overflow/007-expected.checksum:
  • platform/mac/fast/body-propagation/overflow/007-expected.png:
  • platform/mac/fast/body-propagation/overflow/007-xhtml-expected.checksum:
  • platform/mac/fast/body-propagation/overflow/007-xhtml-expected.png:
  • platform/mac/fast/clip/014-expected.checksum:
  • platform/mac/fast/clip/014-expected.png:
  • platform/mac/fast/forms/basic-textareas-expected.checksum:
  • platform/mac/fast/forms/basic-textareas-expected.png:
  • platform/mac/fast/forms/form-element-geometry-expected.checksum:
  • platform/mac/fast/forms/form-element-geometry-expected.png:
  • platform/mac/fast/forms/listbox-hit-test-zoomed-expected.checksum:
  • platform/mac/fast/forms/listbox-hit-test-zoomed-expected.png:
  • platform/mac/fast/forms/textarea-scroll-height-expected.checksum:
  • platform/mac/fast/forms/textarea-scroll-height-expected.png:
  • platform/mac/fast/inline-block/tricky-baseline-expected.checksum:
  • platform/mac/fast/inline-block/tricky-baseline-expected.png:
  • platform/mac/fast/layers/scroll-rect-to-visible-expected.checksum:
  • platform/mac/fast/layers/scroll-rect-to-visible-expected.png:
  • platform/mac/fast/overflow/002-expected.checksum:
  • platform/mac/fast/overflow/002-expected.png:
  • platform/mac/fast/overflow/003-expected.checksum:
  • platform/mac/fast/overflow/003-expected.png:
  • platform/mac/fast/overflow/005-expected.checksum:
  • platform/mac/fast/overflow/005-expected.png:
  • platform/mac/fast/overflow/007-expected.checksum:
  • platform/mac/fast/overflow/007-expected.png:
  • platform/mac/fast/overflow/008-expected.checksum:
  • platform/mac/fast/overflow/008-expected.png:
  • platform/mac/fast/overflow/childFocusRingClip-expected.checksum:
  • platform/mac/fast/overflow/childFocusRingClip-expected.png:
  • platform/mac/fast/overflow/float-in-relpositioned-expected.checksum:
  • platform/mac/fast/overflow/float-in-relpositioned-expected.png:
  • platform/mac/fast/overflow/image-selection-highlight-expected.checksum:
  • platform/mac/fast/overflow/image-selection-highlight-expected.png:
  • platform/mac/fast/overflow/overflow-auto-position-absolute-expected.checksum:
  • platform/mac/fast/overflow/overflow-auto-position-absolute-expected.png:
  • platform/mac/fast/overflow/overflow-rtl-expected.checksum:
  • platform/mac/fast/overflow/overflow-rtl-expected.png:
  • platform/mac/fast/overflow/overflow-stacking-expected.checksum:
  • platform/mac/fast/overflow/overflow-stacking-expected.png:
  • platform/mac/fast/overflow/overflow-text-hit-testing-expected.checksum:
  • platform/mac/fast/overflow/overflow-text-hit-testing-expected.png:
  • platform/mac/fast/overflow/overflow-with-local-background-attachment-expected.checksum:
  • platform/mac/fast/overflow/overflow-with-local-background-attachment-expected.png:
  • platform/mac/fast/overflow/overflow-x-y-expected.checksum:
  • platform/mac/fast/overflow/overflow-x-y-expected.png:
  • platform/mac/fast/overflow/scroll-nested-positioned-layer-in-overflow-expected.checksum:
  • platform/mac/fast/overflow/scroll-nested-positioned-layer-in-overflow-expected.png:
  • platform/mac/fast/overflow/scrollbar-position-update-expected.checksum:
  • platform/mac/fast/overflow/scrollbar-position-update-expected.png:
  • platform/mac/fast/overflow/table-overflow-float-expected.checksum:
  • platform/mac/fast/overflow/table-overflow-float-expected.png:
  • platform/mac/fast/repaint/block-selection-gap-stale-cache-2-expected.checksum:
  • platform/mac/fast/repaint/block-selection-gap-stale-cache-2-expected.png:
  • platform/mac/fast/repaint/layout-state-only-positioned-expected.checksum:
  • platform/mac/fast/repaint/layout-state-only-positioned-expected.png:
  • platform/mac/fast/repaint/selection-gap-overflow-scroll-expected.checksum:
  • platform/mac/fast/repaint/selection-gap-overflow-scroll-expected.png:
  • platform/mac/fast/table/edge-offsets-expected.checksum:
  • platform/mac/fast/table/edge-offsets-expected.png:
  • platform/mac/scrollbars/key-window-not-first-responder-expected.checksum:
  • platform/mac/scrollbars/key-window-not-first-responder-expected.png:
  • platform/mac/svg/custom/invisible-text-after-scrolling-expected.checksum:
  • platform/mac/svg/custom/invisible-text-after-scrolling-expected.png:
  • platform/mac/tables/mozilla/bugs/bug149275-1-expected.checksum:
  • platform/mac/tables/mozilla/bugs/bug149275-1-expected.png:
21:38 WebKit Team edited by jorlow@chromium.org
(diff)
21:15 Changeset [53837] by jorlow@chromium.org
  • 2 edits in trunk/WebKitTools

Adding myself as reviewer. No review necessary.

Patch by Jeremy Orlow <jorlow@chromium.org> on 2010-01-25

  • Scripts/webkitpy/committers.py:
18:17 Changeset [53836] by andersca@apple.com
  • 8 edits in trunk/WebKit/mac

Featureless build fixes.

  • Misc/WebCache.mm:

(+[WebCache empty]):

  • WebCoreSupport/WebApplicationCache.mm:
  • WebCoreSupport/WebChromeClient.h:
  • WebView/WebDataSource.mm:

(-[WebDataSource _transferApplicationCache:]):

  • WebView/WebFrame.mm:

(-[WebFrame _cacheabilityDictionary]):

  • WebView/WebView.mm:

(WebKitInitializeApplicationCachePathIfNecessary):
(-[WebView _close]):

  • WebView/WebViewData.mm:

(-[WebViewPrivate dealloc]):
(-[WebViewPrivate finalize]):

18:07 Changeset [53835] by enrica@apple.com
  • 7 edits
    3 adds in trunk

WebCore: Cleanup unwanted tags after pasting.
https://bugs.webkit.org/show_bug.cgi?id=34148

Reviewed by Darin Adler.

Test: editing/pasteboard/paste-noscript-xhtml.xhtml

  • dom/Element.cpp:

(WebCore::Element::setAttributeNS):

  • dom/Element.h:
  • dom/XMLTokenizer.h:
  • dom/XMLTokenizerLibxml2.cpp:

(WebCore::XMLTokenizer::XMLTokenizer):
(WebCore::handleElementNamespaces):
(WebCore::handleElementAttributes):
(WebCore::XMLTokenizer::startElementNs):
(WebCore::XMLTokenizer::endElementNs):
(WebCore::parseXMLDocumentFragment):

  • html/HTMLElement.cpp:

(WebCore::HTMLElement::createContextualFragment):

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

Reviewed by Darin Adler.

  • editing/pasteboard/paste-noscript-xhtml-expected.txt: Added.
  • editing/pasteboard/paste-noscript-xhtml.xhtml: Added.
  • editing/resources/htmlcontent.html: Added.
17:24 Changeset [53834] by sfalken@apple.com
  • 9 edits in trunk

2010-01-25 Steve Falkenburg <sfalken@apple.com>

Reviewed by Simon Fraser.

A WebGeolocationControllerClient is leaked for every WebView
https://bugs.webkit.org/show_bug.cgi?id=34145

  • WebCoreSupport/WebGeolocationControllerClient.cpp: (WebGeolocationControllerClient::geolocationDestroyed): Added.
  • WebCoreSupport/WebGeolocationControllerClient.h:

2010-01-25 Steve Falkenburg <sfalken@apple.com>

Reviewed by Simon Fraser.

A WebGeolocationControllerClient is leaked for every WebView
https://bugs.webkit.org/show_bug.cgi?id=34145

  • WebCoreSupport/WebGeolocationControllerClient.h:
  • WebCoreSupport/WebGeolocationControllerClient.mm: (WebGeolocationControllerClient::geolocationDestroyed):

2010-01-25 Steve Falkenburg <sfalken@apple.com>

Reviewed by Simon Fraser.

A WebGeolocationControllerClient is leaked for every WebView
https://bugs.webkit.org/show_bug.cgi?id=34145

  • page/GeolocationController.cpp: (WebCore::GeolocationController::~GeolocationController):
  • page/GeolocationControllerClient.h:
17:07 Changeset [53833] by mrowe@apple.com
  • 3 edits
    2 adds in branches/safari-4-branch

Merge r52527.

17:03 Changeset [53832] by mrowe@apple.com
  • 1 edit in branches/safari-4-branch/WebCore/plugins/PluginView.cpp

Merge r53581.

17:02 Changeset [53831] by mrowe@apple.com
  • 1 edit in branches/safari-4-branch/WebCore/ChangeLog

Merge r53581.

16:44 Changeset [53830] by mrowe@apple.com
  • 5 edits
    1 add in branches/safari-4-branch

Merge r46792.

16:36 Changeset [53829] by mrowe@apple.com
  • 3 edits
    2 adds in branches/safari-4-branch

Merge r50698.

16:33 Changeset [53828] by mrowe@apple.com
  • 3 edits
    2 adds in branches/safari-4-branch

Merge r52073.

16:28 Changeset [53827] by mrowe@apple.com
  • 3 edits
    2 adds in branches/safari-4-branch

Merge r48810.

16:09 Changeset [53826] by mrowe@apple.com
  • 3 edits
    2 adds in branches/safari-4-branch

Merge r51877.

16:06 Changeset [53825] by mrowe@apple.com
  • 5 edits in branches/safari-4-branch/WebKit/mac

Merge r53520.

16:00 Changeset [53824] by mrowe@apple.com
  • 4 edits in branches/safari-4-branch/WebKit

Merge r53195.

15:55 Changeset [53823] by mrowe@apple.com
  • 3 edits
    2 adds in branches/safari-4-branch

Merge r51727.

15:53 Changeset [53822] by mrowe@apple.com
  • 2 edits in branches/safari-4-branch/JavaScriptCore

Merge r53089.

15:53 Changeset [53821] by mrowe@apple.com
  • 2 edits in branches/safari-4-branch/WebCore

Merge r51881.

15:52 Changeset [53820] by eric@webkit.org
  • 2 edits in trunk/LayoutTests

2010-01-25 Kent Tamura <tkent@chromium.org>

Reviewed by Darin Adler.

Improve portability of select-item-background-clip.html
https://bugs.webkit.org/show_bug.cgi?id=34072

  • fast/forms/select-item-background-clip.html: Calculate the pixel height at runtime.
15:49 Changeset [53819] by mrowe@apple.com
  • 2 edits in branches/safari-4-branch/WebKit/mac

Merge r51616.

15:47 Changeset [53818] by mrowe@apple.com
  • 2 edits
    1 add in branches/safari-4-branch/WebCore

Merge r51205.

15:26 Changeset [53817] by mrowe@apple.com
  • 13 edits in branches/safari-4-branch

Merge r52749.

15:20 Changeset [53816] by dglazkov@chromium.org
  • 7 edits in trunk/LayoutTests

2010-01-25 Dimitri Glazkov <dglazkov@chromium.org>

Unreviewed, trivial baseline updates.

-webkit-gradient slows down scrolling when page has horizontal scrollbar
https://bugs.webkit.org/show_bug.cgi?id=19650

Update pixel baselines. They changed slightly after http://trac.webkit.org/changeset/53318/
landed.

  • platform/mac/fast/gradients/border-image-gradient-expected.checksum:
  • platform/mac/fast/gradients/border-image-gradient-expected.png:
  • platform/mac/fast/gradients/border-image-gradient-sides-and-corners-expected.checksum:
  • platform/mac/fast/gradients/border-image-gradient-sides-and-corners-expected.png:
  • platform/mac/fast/gradients/simple-gradients-expected.checksum:
  • platform/mac/fast/gradients/simple-gradients-expected.png:
15:12 Changeset [53815] by dimich@chromium.org
  • 2 edits in trunk/JavaScriptCore

Fix Chromium Linux tests: the pthread functions on Linux produce segfault if they receive 0 thread handle.
After r53714, we can have 0 thread handles passed to pthread_join and pthread_detach if corresponding threads
were already terminated and their threadMap entries cleared.
Add a 0 check.

Reviewed by David Levin.

  • wtf/ThreadingPthreads.cpp:

(WTF::waitForThreadCompletion):
(WTF::detachThread):

15:07 Changeset [53814] by cmarrin@apple.com
  • 2 edits in trunk/WebCore

Fixed bug in positioning of scroll layer within the
parent root layer.
https://bugs.webkit.org/show_bug.cgi?id=33847

13:55 WebKit Team edited by michelangelo@webkit.org
(diff)
13:47 WebKit Team edited by michelangelo@webkit.org
(diff)
12:54 WebKit Team edited by bfulgham@webkit.org
Added some (unsubstantiated) claims about what I know. (diff)
12:37 Changeset [53813] by ap@apple.com
  • 2 edits in trunk/WebCore

Rubber-stamped by Geoffrey Garen.

https://bugs.webkit.org/show_bug.cgi?id=34076
An image remains accessible via form.property syntax after being removed from document.

Fix crashing regression tests (tables/mozilla/bugs/bug4527.html et al.)

  • html/HTMLImageElement.cpp: (WebCore::HTMLImageElement::insertedIntoTree): Remove incorrect assertions added in the previous patch - it's mot true that m_for is always a parent; table parsing can reparent the image element, but m_form still needs to be set.
12:07 Changeset [53812] by ap@apple.com
  • 4 edits
    4 adds in trunk

Reviewed by Geoffrey Garen.

https://bugs.webkit.org/show_bug.cgi?id=34076
An image remains accessible via form.property syntax after being removed from document.

Tests: fast/forms/removed-image-as-property.html

fast/forms/reparented-image-as-property.html

  • html/HTMLImageElement.cpp: (WebCore::HTMLImageElement::~HTMLImageElement): This is called during GC - not a good time to make observable changes to DOM. (WebCore::HTMLImageElement::insertedIntoTree): This is the right place to do any work that depends on connectedness to some ancestor. We still allow for m_form to be set via constructor, which happens during parsing. (WebCore::HTMLImageElement::removedFromTree): Ditto.
  • html/HTMLImageElement.h: Added removedFromTree/insertedIntoTree, moved removedFromDocument and insertedIntoDocument to private section, as they shouldn't be called directly.
12:03 Changeset [53811] by eric@webkit.org
  • 3 edits in trunk/WebCore

2010-01-25 Daniel Bates <dbates@webkit.org>

Reviewed by Adam Barth.

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

Fixes misspelled words in comments.

  • accessibility/AccessibilityListBox.cpp: (WebCore::AccessibilityListBox::addChildren): (WebCore::AccessibilityListBox::doAccessibilityHitTest):
  • accessibility/mac/AccessibilityObjectWrapper.mm: (-[AccessibilityObjectWrapper detach]): (AXAttributeStringSetElement):
  • bindings/js/JSDOMBinding.cpp: (WebCore::completeURL):
  • bindings/js/JSDOMWindowBase.cpp:
  • bindings/js/JSDOMWindowBase.h:
  • bindings/js/JSDOMWindowCustom.cpp: (WebCore::JSDOMWindow::getOwnPropertySlot):
  • bindings/js/JSSVGPODTypeWrapper.h:
  • bindings/js/ScriptController.cpp: (WebCore::ScriptController::evaluateInWorld):
  • bridge/NP_jsobject.cpp: (_NPN_SetException):
  • bridge/jni/jsc/JavaInstanceJSC.cpp: (JavaInstance::invokeMethod): (JObjectWrapper::JObjectWrapper):
  • bridge/objc/objc_instance.mm: (ObjcInstance::setValueOfUndefinedField): (ObjcInstance::getValueOfUndefinedField):
  • css/CSSFontSelector.cpp: (WebCore::CSSFontSelector::addFontFaceRule):
  • css/CSSStyleSelector.cpp: (WebCore::CSSStyleSelector::CSSStyleSelector): (WebCore::CSSStyleSelector::sortMatchedRules): (WebCore::CSSStyleSelector::applyDeclarations): (WebCore::CSSStyleSelector::applyProperty):
  • dom/Document.cpp: (WebCore::Document::updateLayoutIgnorePendingStylesheets): (WebCore::Document::detach):
  • dom/Document.h:
  • dom/Element.cpp: (WebCore::Element::updateFocusAppearance):
  • dom/Element.h:
  • dom/MessagePortChannel.h:
  • dom/MouseRelatedEvent.cpp: (WebCore::MouseRelatedEvent::receivedTarget):
  • dom/Node.cpp: (WebCore::Node::isFocusable): (WebCore::Node::shadowAncestorNode):
  • dom/Node.h:
  • dom/Position.cpp: (WebCore::Position::upstream):
  • dom/Position.h:
  • dom/Range.cpp: (WebCore::Range::processContents):
  • dom/StyledElement.cpp: (WebCore::StyledElement::addCSSColor):
  • editing/ApplyStyleCommand.cpp: (WebCore::StyleChange::reconcileTextDecorationProperties): (WebCore::StyleChange::extractTextStyles): (WebCore::prepareEditingStyleToApplyAt):
  • editing/DeleteButtonController.cpp: (WebCore::isDeletableElement):
  • editing/DeleteSelectionCommand.cpp: (WebCore::DeleteSelectionCommand::initializeStartEnd):
  • editing/Editor.cpp: (WebCore::Editor::learnSpelling):
  • inspector/front-end/ConsoleView.js: (WebInspector.ConsoleMessage.prototype._format):
  • loader/Cache.cpp: (WebCore::Cache::pruneLiveResources): (WebCore::Cache::pruneDeadResources):
  • loader/Cache.h:
  • loader/CachedResource.cpp: (WebCore::CachedResource::makePurgeable):
  • loader/CachedResource.h:
  • loader/CachedResourceHandle.h:
  • loader/DocumentLoader.h:
  • loader/FTPDirectoryParser.cpp: (WebCore::parseOneFTPLine):
  • loader/FrameLoader.cpp: (WebCore::FrameLoader::didExplicitOpen):
  • loader/ResourceLoader.cpp: (WebCore::ResourceLoader::didCancel):
  • loader/WorkerThreadableLoader.h:
  • loader/appcache/ApplicationCacheStorage.cpp: (WebCore::ApplicationCacheStorage::spaceNeeded):
  • notifications/NotificationPresenter.h: (WebCore::NotificationPresenter::):
  • page/animation/AnimationController.cpp: (WebCore::AnimationController::updateAnimations):
  • platform/LinkHash.cpp: (WebCore::cleanPath):
  • platform/graphics/cg/GraphicsContextCG.cpp: (WebCore::calculateDrawingMode):
  • platform/graphics/cg/PatternCG.cpp: (WebCore::Pattern::createPlatformPattern):
  • platform/graphics/mac/ComplexTextController.cpp: (WebCore::ComplexTextController::offsetForPosition):
  • platform/graphics/mac/GraphicsLayerCA.h:
  • platform/graphics/mac/GraphicsLayerCA.mm: (WebCore::GraphicsLayerCA::ensureStructuralLayer):
  • platform/graphics/mac/MediaPlayerPrivateQTKit.mm: (WebCore::MediaPlayerPrivate::doSeek): (WebCore::MediaPlayerPrivate::setClosedCaptionsVisible): (WebCore::MediaPlayerPrivate::hasAvailableVideoFrame):
  • platform/mac/PasteboardMac.mm: (WebCore::Pasteboard::writeSelection): (WebCore::Pasteboard::writePlainText): (WebCore::Pasteboard::writeURL):
  • platform/text/TextCodecICU.cpp: (WebCore::TextCodecICU::registerExtendedEncodingNames):
  • platform/text/TextEncodingDetectorICU.cpp: (WebCore::detectTextEncoding):
  • rendering/AutoTableLayout.cpp: (WebCore::AutoTableLayout::calcEffectiveWidth): (WebCore::AutoTableLayout::layout):
  • rendering/RenderBox.cpp: (WebCore::RenderBox::calcAbsoluteHorizontalValues): (WebCore::RenderBox::calcAbsoluteVerticalValues): (WebCore::RenderBox::calcAbsoluteHorizontalReplaced): (WebCore::RenderBox::calcAbsoluteVerticalReplaced):
  • rendering/RenderLayer.cpp: (WebCore::RenderLayer::scrollByRecursively): (WebCore::RenderLayer::paintLayer): (WebCore::RenderLayer::hitTestLayer):
  • rendering/RenderObject.cpp: (WebCore::RenderObject::destroy):
  • rendering/RenderTextControl.cpp: (WebCore::RenderTextControl::setInnerTextValue):
  • rendering/RenderTheme.cpp: (WebCore::RenderTheme::stateChanged):
  • rendering/SVGRenderSupport.cpp: (WebCore::SVGRenderBase::mapLocalToContainer):
  • rendering/SVGRootInlineBox.cpp: (WebCore::SVGRootInlineBox::buildTextChunks):
  • rendering/TransformState.cpp: (WebCore::TransformState::flattenWithTransform):
  • wml/WMLInputElement.cpp: (WebCore::WMLInputElement::dispatchBlurEvent): (WebCore::WMLInputElement::suggestedValue):
11:34 WebKit Team edited by darin@chromium.org
(diff)
11:32 WebKit Team edited by darin@chromium.org
(diff)
11:26 Changeset [53810] by yurys@chromium.org
  • 2 edits in trunk/LayoutTests

2010-01-25 Yury Semikhatsky <yurys@chromium.org>

Unreviewed. Add new inspector test added in r53807 to skip
list on qt platform.

  • platform/qt/Skipped:
11:20 Changeset [53809] by dbates@webkit.org
  • 62 edits in trunk/WebCore

2010-01-25 Daniel Bates <dbates@webkit.org>

Reviewed by Adam Barth.

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

Fixes misspelled words in comments.

  • accessibility/AccessibilityListBox.cpp: (WebCore::AccessibilityListBox::addChildren): (WebCore::AccessibilityListBox::doAccessibilityHitTest):
  • accessibility/mac/AccessibilityObjectWrapper.mm: (-[AccessibilityObjectWrapper detach]): (AXAttributeStringSetElement):
  • bindings/js/JSDOMBinding.cpp: (WebCore::completeURL):
  • bindings/js/JSDOMWindowBase.cpp:
  • bindings/js/JSDOMWindowBase.h:
  • bindings/js/JSDOMWindowCustom.cpp: (WebCore::JSDOMWindow::getOwnPropertySlot):
  • bindings/js/JSSVGPODTypeWrapper.h:
  • bindings/js/ScriptController.cpp: (WebCore::ScriptController::evaluateInWorld):
  • bridge/NP_jsobject.cpp: (_NPN_SetException):
  • bridge/jni/jsc/JavaInstanceJSC.cpp: (JavaInstance::invokeMethod): (JObjectWrapper::JObjectWrapper):
  • bridge/objc/objc_instance.mm: (ObjcInstance::setValueOfUndefinedField): (ObjcInstance::getValueOfUndefinedField):
  • css/CSSFontSelector.cpp: (WebCore::CSSFontSelector::addFontFaceRule):
  • css/CSSStyleSelector.cpp: (WebCore::CSSStyleSelector::CSSStyleSelector): (WebCore::CSSStyleSelector::sortMatchedRules): (WebCore::CSSStyleSelector::applyDeclarations): (WebCore::CSSStyleSelector::applyProperty):
  • dom/Document.cpp: (WebCore::Document::updateLayoutIgnorePendingStylesheets): (WebCore::Document::detach):
  • dom/Document.h:
  • dom/Element.cpp: (WebCore::Element::updateFocusAppearance):
  • dom/Element.h:
  • dom/MessagePortChannel.h:
  • dom/MouseRelatedEvent.cpp: (WebCore::MouseRelatedEvent::receivedTarget):
  • dom/Node.cpp: (WebCore::Node::isFocusable): (WebCore::Node::shadowAncestorNode):
  • dom/Node.h:
  • dom/Position.cpp: (WebCore::Position::upstream):
  • dom/Position.h:
  • dom/Range.cpp: (WebCore::Range::processContents):
  • dom/StyledElement.cpp: (WebCore::StyledElement::addCSSColor):
  • editing/ApplyStyleCommand.cpp: (WebCore::StyleChange::reconcileTextDecorationProperties): (WebCore::StyleChange::extractTextStyles): (WebCore::prepareEditingStyleToApplyAt):
  • editing/DeleteButtonController.cpp: (WebCore::isDeletableElement):
  • editing/DeleteSelectionCommand.cpp: (WebCore::DeleteSelectionCommand::initializeStartEnd):
  • editing/Editor.cpp: (WebCore::Editor::learnSpelling):
  • inspector/front-end/ConsoleView.js: (WebInspector.ConsoleMessage.prototype._format):
  • loader/Cache.cpp: (WebCore::Cache::pruneLiveResources): (WebCore::Cache::pruneDeadResources):
  • loader/Cache.h:
  • loader/CachedResource.cpp: (WebCore::CachedResource::makePurgeable):
  • loader/CachedResource.h:
  • loader/CachedResourceHandle.h:
  • loader/DocumentLoader.h:
  • loader/FTPDirectoryParser.cpp: (WebCore::parseOneFTPLine):
  • loader/FrameLoader.cpp: (WebCore::FrameLoader::didExplicitOpen):
  • loader/ResourceLoader.cpp: (WebCore::ResourceLoader::didCancel):
  • loader/WorkerThreadableLoader.h:
  • loader/appcache/ApplicationCacheStorage.cpp: (WebCore::ApplicationCacheStorage::spaceNeeded):
  • notifications/NotificationPresenter.h: (WebCore::NotificationPresenter::):
  • page/animation/AnimationController.cpp: (WebCore::AnimationController::updateAnimations):
  • platform/LinkHash.cpp: (WebCore::cleanPath):
  • platform/graphics/cg/GraphicsContextCG.cpp: (WebCore::calculateDrawingMode):
  • platform/graphics/cg/PatternCG.cpp: (WebCore::Pattern::createPlatformPattern):
  • platform/graphics/mac/ComplexTextController.cpp: (WebCore::ComplexTextController::offsetForPosition):
  • platform/graphics/mac/GraphicsLayerCA.h:
  • platform/graphics/mac/GraphicsLayerCA.mm: (WebCore::GraphicsLayerCA::ensureStructuralLayer):
  • platform/graphics/mac/MediaPlayerPrivateQTKit.mm: (WebCore::MediaPlayerPrivate::doSeek): (WebCore::MediaPlayerPrivate::setClosedCaptionsVisible): (WebCore::MediaPlayerPrivate::hasAvailableVideoFrame):
  • platform/mac/PasteboardMac.mm: (WebCore::Pasteboard::writeSelection): (WebCore::Pasteboard::writePlainText): (WebCore::Pasteboard::writeURL):
  • platform/text/TextCodecICU.cpp: (WebCore::TextCodecICU::registerExtendedEncodingNames):
  • platform/text/TextEncodingDetectorICU.cpp: (WebCore::detectTextEncoding):
  • rendering/AutoTableLayout.cpp: (WebCore::AutoTableLayout::calcEffectiveWidth): (WebCore::AutoTableLayout::layout):
  • rendering/RenderBox.cpp: (WebCore::RenderBox::calcAbsoluteHorizontalValues): (WebCore::RenderBox::calcAbsoluteVerticalValues): (WebCore::RenderBox::calcAbsoluteHorizontalReplaced): (WebCore::RenderBox::calcAbsoluteVerticalReplaced):
  • rendering/RenderLayer.cpp: (WebCore::RenderLayer::scrollByRecursively): (WebCore::RenderLayer::paintLayer): (WebCore::RenderLayer::hitTestLayer):
  • rendering/RenderObject.cpp: (WebCore::RenderObject::destroy):
  • rendering/RenderTextControl.cpp: (WebCore::RenderTextControl::setInnerTextValue):
  • rendering/RenderTheme.cpp: (WebCore::RenderTheme::stateChanged):
  • rendering/SVGRenderSupport.cpp: (WebCore::SVGRenderBase::mapLocalToContainer):
  • rendering/SVGRootInlineBox.cpp: (WebCore::SVGRootInlineBox::buildTextChunks):
  • rendering/TransformState.cpp: (WebCore::TransformState::flattenWithTransform):
  • wml/WMLInputElement.cpp: (WebCore::WMLInputElement::dispatchBlurEvent): (WebCore::WMLInputElement::suggestedValue):
10:58 Changeset [53808] by ggaren@apple.com
  • 2 edits in trunk/WebCore

Suggested by Darin Adler.

Fixed a spelling error, tightened up some grammar.

  • bindings/js/JSDOMBinding.cpp:

(WebCore::forgetDOMObject):
(WebCore::forgetDOMNode):

10:57 Changeset [53807] by eric@webkit.org
  • 3 edits
    2 adds in trunk

2010-01-25 Yury Semikhatsky <yurys@chromium.org>

Reviewed by Timothy Hatcher.

Test that Web Inspector doesn't change methods declared by the inspected
page.
https://bugs.webkit.org/show_bug.cgi?id=34089

  • inspector/inspected-objects-not-overriden-expected.txt: Added.
  • inspector/inspected-objects-not-overriden.html: Added.

2010-01-25 Yury Semikhatsky <yurys@chromium.org>

Reviewed by Timothy Hatcher.

Instead of adding inspector methods to the inspected global objects
use private implementation visible from inspector code only.
https://bugs.webkit.org/show_bug.cgi?id=34089

Test: inspector/inspected-objects-not-overriden.html

  • inspector/front-end/InjectedScript.js: (injectedScriptConstructor):
08:37 QtWebKitBackportingFixes edited by benjamin.poulain@nokia.com
(diff)
08:06 Changeset [53806] by pfeldman@chromium.org
  • 5 edits in trunk/WebCore

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

Reviewed by Timothy Hatcher.

Web Inspector: don't show headers/content tabs in scripts view.

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

  • inspector/front-end/ResourceView.js: (WebInspector.ResourceView): (WebInspector.ResourceView.prototype.set headersVisible): (WebInspector.ResourceView.prototype._selectContentTab): (WebInspector.ResourceView.prototype._innerSelectContentTab):
  • inspector/front-end/ResourcesPanel.js: (WebInspector.ResourcesPanel.prototype.show): (WebInspector.ResourcesPanel.prototype.recreateViewForResourceIfNeeded): (WebInspector.ResourcesPanel.prototype.showResource):
  • inspector/front-end/ScriptsPanel.js: (WebInspector.ScriptsPanel.prototype.show): (WebInspector.ScriptsPanel.prototype._showScriptOrResource):
  • inspector/front-end/inspector.css:
07:37 Changeset [53805] by philn@webkit.org
  • 2 edits in trunk/LayoutTests

2010-01-25 Philippe Normand <pnormand@igalia.com>

Rubber-stamped by Xan Lopez.

[Gtk] media/video-reverse-play-duration.html fails on and off on Gtk buildbots
https://bugs.webkit.org/show_bug.cgi?id=34086

  • platform/gtk/Skipped: Skipping the test, requires investigation on the failure.
07:24 Changeset [53804] by eric@webkit.org
  • 2 edits in trunk/WebCore

2010-01-25 Simon Hausmann <hausmann@webkit.org>

Reviewed by Kenneth Rohde Christiansen.

[Qt] Reduce the size of the library when compiling for Maemo5
https://bugs.webkit.org/show_bug.cgi?id=34050

Exclude the Web Inspector's qrc file in the Maemo5 build, just
like we do for the Symbian build. Saves about ~900k.

  • WebCore.pro:
07:08 Changeset [53803] by eric@webkit.org
  • 5 edits in trunk/WebCore

2010-01-25 Simon Hausmann <hausmann@webkit.org>

Reviewed by Kenneth Rohde Christiansen.

[Qt] In the StyledPainter determine the style from the Render and Scrollbar theme instead of from the paint device
https://bugs.webkit.org/show_bug.cgi?id=34054

Getting the style from the painter's paint device is a hack that breaks when
the paint device's style is different than the style that is used when
calculating the metries earlier when there is no painter available.

This change moves us closer to always using the same style.

  • platform/qt/RenderThemeQt.cpp: (WebCore::StylePainter::StylePainter): (WebCore::StylePainter::init): (WebCore::RenderThemeQt::paintButton): (WebCore::RenderThemeQt::paintTextField): (WebCore::RenderThemeQt::paintMenuList): (WebCore::RenderThemeQt::paintMenuListButton): (WebCore::RenderThemeQt::paintSliderTrack): (WebCore::RenderThemeQt::paintMediaMuteButton): (WebCore::RenderThemeQt::paintMediaPlayButton): (WebCore::RenderThemeQt::paintMediaSliderTrack): (WebCore::RenderThemeQt::paintMediaSliderThumb):
  • platform/qt/RenderThemeQt.h:
  • platform/qt/ScrollbarThemeQt.cpp: (WebCore::ScrollbarThemeQt::paint): (WebCore::ScrollbarThemeQt::hitTest): (WebCore::ScrollbarThemeQt::shouldCenterOnThumb): (WebCore::ScrollbarThemeQt::scrollbarThickness): (WebCore::ScrollbarThemeQt::thumbLength): (WebCore::ScrollbarThemeQt::trackPosition): (WebCore::ScrollbarThemeQt::trackLength): (WebCore::ScrollbarThemeQt::paintScrollCorner): (WebCore::ScrollbarThemeQt::style):
  • platform/qt/ScrollbarThemeQt.h:
06:42 Changeset [53802] by eric@webkit.org
  • 6 edits in trunk

2010-01-25 Simon Hausmann <hausmann@webkit.org>

Reviewed by Kenneth Rohde Christiansen.

[Qt] In RenderThemeQt determine the QStyle from the page client instead of the page's view
https://bugs.webkit.org/show_bug.cgi?id=34053

  • platform/qt/QWebPageClient.h: Add function to return the style.
  • platform/qt/RenderThemeQt.cpp: (WebCore::RenderThemeQt::qStyle): Determine the style from the page client instead of the page's view.

2010-01-25 Simon Hausmann <hausmann@webkit.org>

Reviewed by Kenneth Rohde Christiansen.

[Qt] In RenderThemeQt determine the QStyle from the page client instead of the page's view
https://bugs.webkit.org/show_bug.cgi?id=34053

  • Api/qgraphicswebview.cpp: (QGraphicsWebViewPrivate::style): Implement QWebPageClient::style() and return the graphics widget's style.
  • Api/qwebpage.cpp: (QWebPageWidgetClient::style): Implement QWebPageClient::style() and return the widget's style.
06:20 Changeset [53801] by hausmann@webkit.org
  • 3 edits in trunk/WebCore

[Qt] Clean up code to determine palette from page client in RenderThemeQt
https://bugs.webkit.org/show_bug.cgi?id=34052

Patch by Simon Hausmann <hausmann@webkit.org> on 2010-01-25
Reviewed by Kenneth Christiansen.

Use the RenderTheme's associated page instead of going through the
render tree and the document.

  • platform/qt/RenderThemeQt.cpp:

(WebCore::RenderThemeQt::setPaletteFromPageClientIfExists):
(WebCore::RenderThemeQt::initializeCommonQStyleOptions):

  • platform/qt/RenderThemeQt.h:
05:56 QtWebKitBackportingFixes edited by hausmann@webkit.org
(diff)
05:53 QtWebKitBackportingFixes edited by hausmann@webkit.org
(diff)
05:18 Changeset [53800] by hausmann@webkit.org
  • 2 edits in trunk

[Qt] Fix the build on Maemo5.

Reviewed by Laszlo Gombos.

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

  • WebKit.pri: Disable the use of uitools, just like it's done for Symbian.
05:14 Changeset [53799] by eric@webkit.org
  • 3 edits
    1 add in trunk

2010-01-25 Janne Koskinen <janne.p.koskinen@digia.com>

Reviewed by Simon Hausmann.

[Qt] Phone backup support for QtWebkit for Symbian devices.
https://bugs.webkit.org/show_bug.cgi?id=34077

  • WebCore.pro:

2010-01-25 Janne Koskinen <janne.p.koskinen@digia.com>

Reviewed by Simon Hausmann.

[Qt] Phone backup support for QtWebkit for Symbian devices.
https://bugs.webkit.org/show_bug.cgi?id=34077

  • symbian/backup_registration.xml: Added.
05:13 Changeset [53798] by yurys@chromium.org
  • 2 edits
    3 copies
    2 adds in trunk/LayoutTests

2010-01-25 Yury Semikhatsky <yurys@chromium.org>

Reviewed by Pavel Feldman.

Update inspector test expectations. Line number and source
name are present only in Chromium when console message is
logged before document load is completed.

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

  • platform/chromium/inspector/console-log-before-inspector-open-expected.txt: Added.
  • platform/gtk/inspector/console-log-before-inspector-open-expected.txt: Copied from LayoutTests/inspector/console-log-before-inspector-open-expected.txt.
  • platform/mac/inspector/console-log-before-inspector-open-expected.txt: Copied from LayoutTests/inspector/console-log-before-inspector-open-expected.txt.
  • platform/win/inspector/console-log-before-inspector-open-expected.txt: Renamed from LayoutTests/inspector/console-log-before-inspector-open-expected.txt.
04:57 QtWebKitBackportingFixes edited by koshuin@gmail.com
(diff)
04:37 Changeset [53797] by eric@webkit.org
  • 6 edits in trunk/WebCore

2010-01-25 Benjamin Poulain <benjamin.poulain@nokia.com>

Reviewed by Antti Koivisto.

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

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

  • page/FrameView.cpp: (WebCore::FrameView::useSlowRepaints): (WebCore::FrameView::useSlowRepaintsIfNotOverlapped): (WebCore::FrameView::registerFixedPositionedObject): (WebCore::FrameView::unregisterFixedPositionedObject): (WebCore::FrameView::scrollContentsFastPath):
  • page/FrameView.h:
  • platform/ScrollView.cpp: (WebCore::ScrollView::scrollContents): (WebCore::ScrollView::scrollContentsFastPath):
  • platform/ScrollView.h:
  • rendering/RenderObject.cpp: (WebCore::RenderObject::styleWillChange): (WebCore::RenderObject::destroy):
04:20 WebKit Team edited by cjerdonek@webkit.org
Fixed mangled special character and added missing quote. (diff)
04:16 WebKit Team edited by cjerdonek@webkit.org
Added introduction and key. (diff)
03:43 WebKit Team edited by cjerdonek@webkit.org
Restored page outline. (diff)
03:18 Changeset [53796] by eric@webkit.org
  • 2 edits in trunk/WebKitTools

2010-01-25 Chris Jerdonek <cjerdonek@webkit.org>

Reviewed by Adam Barth.

Improved prepare-ChangeLog so that it preserves the relative
indentation of a git commit message.

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

  • Scripts/prepare-ChangeLog:
    • Also adjusted the script so that it does not add white space characters to empty lines.
02:58 Changeset [53795] by pfeldman@chromium.org
  • 9 edits
    1 add in trunk/WebCore

2010-01-24 Pavel Feldman <pfeldman@chromium.org>

Reviewed by Timothy Hatcher.

Web Inspector: Add support for addMessage/clearMessages to SourceFrame2.

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

  • WebCore.gypi:
  • WebCore.vcproj/WebCore.vcproj:
  • inspector/front-end/SourceFrame2.js: (WebInspector.SourceFrame2): (WebInspector.SourceFrame2.prototype.addMessage): (WebInspector.SourceFrame2.prototype.clearMessages): (WebInspector.SourceFrame2.prototype._incrementMessageRepeatCount): (WebInspector.SourceFrame2.prototype._addExistingMessagesToSource): (WebInspector.SourceFrame2.prototype._addMessageToSource): (WebInspector.SourceFrame2.prototype.resize):
  • inspector/front-end/TextEditor.js: (WebInspector.TextEditor): (WebInspector.TextEditor.prototype.setDivDecoration): (WebInspector.TextEditor.prototype._lineHeight): (WebInspector.TextEditor.prototype._highlightChanged): (WebInspector.TextEditor.prototype.packAndRepaintAll): (WebInspector.TextEditor.prototype._updateSize): (WebInspector.TextEditor.prototype._repaintAll): (WebInspector.TextEditor.prototype._paint): (WebInspector.TextEditor.prototype._paintLinesContinuation): (WebInspector.TextEditor.prototype._mouseOut): (WebInspector.TextEditor.prototype._updateDivDecorations): (WebInspector.TextEditor.prototype._positionDivDecoration): (WebInspector.TextEditor.prototype._paintSelection): (WebInspector.TextEditor.prototype._replaceSelectionWith):
  • inspector/front-end/TextEditorHighlighter.js: (WebInspector.TextEditorHighlighter.prototype.updateHighlight):
  • inspector/front-end/WebKit.qrc:
  • inspector/front-end/inspector.css:
  • inspector/front-end/inspector.html:
  • inspector/front-end/textEditor.css: Added.
02:37 Changeset [53794] by pfeldman@chromium.org
  • 9 edits in trunk/WebCore

2010-01-24 Pavel Feldman <pfeldman@chromium.org>

Reviewed by Timothy Hatcher.

Web Inspector: Replace split pane with tabbed pane in resource
contents view.

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

01:43 QtWebKitBackportingFixes edited by hausmann@webkit.org
(diff)
00:45 WebKit Team edited by hausmann@webkit.org
(diff)
00:02 WebKit Team edited by krit@webkit.org
(diff)
Note: See TracTimeline for information about the timeline view.