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

Timeline



May 15, 2010:

11:57 PM Changeset in webkit [59567] by eric@webkit.org
  • 3 edits
    3 deletes in trunk

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

Unreviewed, rolling out r59565.
​http://trac.webkit.org/changeset/59565
​https://bugs.webkit.org/show_bug.cgi?id=39173

Broke GTK (Requested by abarth on #webkit).

  • fast/canvas/canvas-toDataURL-case-insensitive-mimetype-expected.txt: Removed.
  • fast/canvas/canvas-toDataURL-case-insensitive-mimetype.html: Removed.
  • fast/canvas/script-tests/canvas-toDataURL-case-insensitive-mimetype.js: Removed.

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

Unreviewed, rolling out r59565.
​http://trac.webkit.org/changeset/59565
​https://bugs.webkit.org/show_bug.cgi?id=39173

Broke GTK (Requested by abarth on #webkit).

  • dom/CanvasSurface.cpp: (WebCore::CanvasSurface::toDataURL):
8:50 PM Changeset in webkit [59566] by dbates@webkit.org
  • 2 edits
    1 add in trunk/WebKitTools

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

Reviewed by Chris Jerdonek.

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

Add function parseSvnPropertyValue to parse single-line and multi-line
property values of an SVN property change.

  • Scripts/VCSUtils.pm:

Added function parseSvnPropertyValue. We will use this as part of
Bug #38885 <​https://bugs.webkit.org/show_bug.cgi?id=38885>.

  • Scripts/webkitperl/VCSUtils_unittest/parseSvnPropertyValue.pl: Added.
8:15 PM Changeset in webkit [59565] by eric@webkit.org
  • 3 edits
    3 adds in trunk

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

Reviewed by Kenneth Rohde Christiansen.

Add a test to verify that Canvas's toDataURL() is case insensitive wrt the mimeType argument.
​https://bugs.webkit.org/show_bug.cgi?id=39153

Spec link:
​http://www.whatwg.org/specs/web-apps/current-work/#dom-canvas-todataurl

  • fast/canvas/canvas-toDataURL-case-insensitive-mimetype-expected.txt: Added.
  • fast/canvas/canvas-toDataURL-case-insensitive-mimetype.html: Added.
  • fast/canvas/script-tests/canvas-toDataURL-case-insensitive-mimetype.js: Added. (tryMimeType):

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

Reviewed by Kenneth Rohde Christiansen.

Canvas's toDataURL() should be case insensitive wrt the mimeType argument.
​https://bugs.webkit.org/show_bug.cgi?id=39153

Spec link:
​http://www.whatwg.org/specs/web-apps/current-work/#dom-canvas-todataurl

Test: fast/canvas/canvas-toDataURL-case-insensitive-mimetype.html

  • dom/CanvasSurface.cpp: (WebCore::CanvasSurface::toDataURL):
8:03 PM Changeset in webkit [59564] by eric@webkit.org
  • 3 edits in trunk/WebCore

2010-05-15 Anders Bakken <​agbakken@gmail.com>

Reviewed by Kenneth Rohde Christiansen.

QObject::objectName() returns a QString and QtField::name()
creates a temporary QByteArray object and returns constData() from it.
This is not safe.

This patch changes the signature of the function to return a
QByteArray instead of const char *.

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

  • bridge/qt/qt_instance.cpp: (JSC::Bindings::QtInstance::getPropertyNames): (JSC::Bindings::QtField::name):
  • bridge/qt/qt_runtime.h:
7:52 PM Changeset in webkit [59563] by eric@webkit.org
  • 2 edits in trunk/WebCore

2010-05-15 Anders Bakken <​agbakken@gmail.com>

Reviewed by Kenneth Rohde Christiansen.

Don't unnecessarily copy data when searching for methods in QtClass.

[Qt] JSValue QtClass::fallbackObject can be optimized
​https://bugs.webkit.org/show_bug.cgi?id=37684

  • bridge/qt/qt_class.cpp: (JSC::Bindings::QtClass::fallbackObject):
7:34 PM Changeset in webkit [59562] by eric@webkit.org
  • 2 edits in trunk/WebCore

2010-05-15 Anders Bakken <​agbakken@gmail.com>

Reviewed by Kenneth Rohde Christiansen.

Identifier::ascii() is not threadsafe and shouldn't be used for
converting to and from QString/QByteArray.

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

  • bridge/qt/qt_class.cpp: (JSC::Bindings::QtClass::fallbackObject): (JSC::Bindings::QtClass::fieldNamed):
7:22 PM Changeset in webkit [59561] by eric@webkit.org
  • 2 edits in trunk/WebCore

2010-05-15 Chang Shu <​chang.shu@nokia.com>

Reviewed by Kenneth Rohde Christiansen.

Based on the spec and Philip Taylor's test suite, createImageData() should throw
an INDEX_SIZE_ERR exception if either the sw or sh arguments are zero.
The link to the spec:
​http://philip.html5.org/tests/canvas/suite/tests/spec.html#testrefs.2d.imageData.getcreate.zero

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

The complete test suite is in the process of checking in.
​https://bugs.webkit.org/show_bug.cgi?id=20553

  • html/canvas/CanvasRenderingContext2D.cpp: (WebCore::CanvasRenderingContext2D::createImageData):
7:08 PM Changeset in webkit [59560] by eric@webkit.org
  • 4 edits in trunk/WebKit/chromium

2010-05-15 Darin Fisher <​darin@chromium.org>

Reviewed by Adam Barth.

[chromium] Expose element accessor on WebPluginContainer
​https://bugs.webkit.org/show_bug.cgi?id=39152

  • public/WebPluginContainer.h:
  • src/WebPluginContainerImpl.cpp: (WebKit::WebPluginContainerImpl::element):
  • src/WebPluginContainerImpl.h:
6:43 PM Changeset in webkit [59559] by eric@webkit.org
  • 4 edits in trunk/WebCore

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

Reviewed by Darin Adler.

XMLHttpRequest.getResponseHeader doesn't need to be custom
​https://bugs.webkit.org/show_bug.cgi?id=39125

I don't see any reason this method needs to be custom. No new tests
because there's no behavior change.

  • bindings/js/JSXMLHttpRequestCustom.cpp:
  • bindings/v8/custom/V8XMLHttpRequestCustom.cpp:
  • xml/XMLHttpRequest.idl:
5:30 PM Changeset in webkit [59558] by abarth@webkit.org
  • 3 edits in trunk/LayoutTests

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

Reviewed by David Hyatt.

Rebaseline client-rects.html on Qt
​https://bugs.webkit.org/show_bug.cgi?id=39169

  • platform/qt/Skipped:
  • platform/qt/fast/multicol/client-rects-expected.txt:
5:19 PM Changeset in webkit [59557] by eric@webkit.org
  • 14 edits in trunk/WebCore

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

Reviewed by Darin Adler.

XMLHttpRequest.setRequestHeader shouldn't be custom
​https://bugs.webkit.org/show_bug.cgi?id=39124

I don't see any reason why this method needs to be custom. No tests
because no behavior change.

  • bindings/js/JSXMLHttpRequestCustom.cpp:
  • bindings/scripts/CodeGeneratorJS.pm:
  • bindings/scripts/CodeGeneratorV8.pm:
  • bindings/scripts/test/GObject/WebKitDOMTestObj.cpp: (webkit_dom_test_obj_method_that_requires_all_args): (webkit_dom_test_obj_method_that_requires_all_args_and_throws):
  • bindings/scripts/test/GObject/WebKitDOMTestObj.h:
  • bindings/scripts/test/JS/JSTestObj.cpp: (WebCore::): (WebCore::jsTestObjPrototypeFunctionMethodThatRequiresAllArgs): (WebCore::jsTestObjPrototypeFunctionMethodThatRequiresAllArgsAndThrows):
  • bindings/scripts/test/JS/JSTestObj.h:
  • bindings/scripts/test/ObjC/DOMTestObj.h:
  • bindings/scripts/test/ObjC/DOMTestObj.mm: (-[DOMTestObj methodThatRequiresAllArgs:objArg:]): (-[DOMTestObj methodThatRequiresAllArgsAndThrows:objArg:]):
  • bindings/scripts/test/TestObj.idl:
  • bindings/scripts/test/V8/V8TestObj.cpp: (WebCore::TestObjInternal::methodThatRequiresAllArgsCallback): (WebCore::TestObjInternal::methodThatRequiresAllArgsAndThrowsCallback): (WebCore::ConfigureV8TestObjTemplate):
  • bindings/v8/custom/V8XMLHttpRequestCustom.cpp:
  • xml/XMLHttpRequest.idl:
3:33 PM Changeset in webkit [59556] by ap@apple.com
  • 3 edits
    2 adds in trunk

Reviewed by Darin Adler.

​https://bugs.webkit.org/show_bug.cgi?id=39048
<rdar://problem/7978384> REGRESSION (r55802): XML errors on Google maps ("Entity 'nbsp' not
defined") with JavaScript disabled

Google Maps goes into mobile mode if JavaScript is disabled for some reason, and sends XHTML
content with XHTML Mobile DOCTYPE. We want to handle it whether XHTMLMP is enabled or not.

Test: fast/doctypes/xhtml-with-xhtmlmp-doctype.xhtml

  • dom/XMLTokenizerLibxml2.cpp: (WebCore::externalSubsetHandler): Restore pre-55802 behavior for builds that don't have XHTMLMP enabled. Given that r55802 negated the condition without breaking XHTMLMP, this line is actually always needed.
3:29 PM Changeset in webkit [59555] by eric@webkit.org
  • 8 edits in trunk

2010-05-15 Jochen Eisinger <​jochen@chromium.org>

Reviewed by Dmitry Titov.

Implement canEstablishDatabase call for workers.
​https://bugs.webkit.org/show_bug.cgi?id=38742

  • public/WebCommonWorkerClient.h:
  • src/DatabaseObserver.cpp: (WebCore::DatabaseObserver::canEstablishDatabase):
  • src/WebWorkerBase.cpp: (WebKit::WebWorkerBase::allowDatabase): (WebKit::WebWorkerBase::allowDatabaseTask): (WebKit::WebWorkerBase::AllowDatabaseMainThreadBridge::AllowDatabaseMainThreadBridge): (WebKit::WebWorkerBase::AllowDatabaseMainThreadBridge::cancel): (WebKit::WebWorkerBase::AllowDatabaseMainThreadBridge::result): (WebKit::WebWorkerBase::AllowDatabaseMainThreadBridge::signalCompleted): (WebKit::WebWorkerBase::AllowDatabaseMainThreadBridge::didComplete):
  • src/WebWorkerBase.h:
  • src/WebWorkerClientImpl.h: (WebKit::WebWorkerClientImpl::allowDatabase):

2010-05-15 Jochen Eisinger <​jochen@chromium.org>

Reviewed by Dmitry Titov.

Add allowDatabase method to TestWebWorker.
​https://bugs.webkit.org/show_bug.cgi?id=38742

  • DumpRenderTree/chromium/TestWebWorker.h: (TestWebWorker::allowDatabase):
3:19 PM Changeset in webkit [59554] by eric@webkit.org
  • 3 edits in trunk/WebCore

2010-05-15 Vangelis Kokkevis <​vangelis@chromium.org>

Reviewed by Darin Fisher.

Fixing LayerChromium hierarchy updates where due to mixed ownership
handling, sublayer updates didn't always properly update parent-to-child
and child-to-parent pointers in sync.
​https://bugs.webkit.org/show_bug.cgi?id=39138

  • platform/graphics/chromium/LayerChromium.cpp: (WebCore::LayerChromium::~LayerChromium): (WebCore::LayerChromium::setNeedsCommit): Typo fix. (WebCore::LayerChromium::insertSublayer): (WebCore::LayerChromium::removeFromSuperlayer): (WebCore::LayerChromium::removeSublayer): (WebCore::LayerChromium::replaceSublayer): (WebCore::LayerChromium::removeAllSublayers): (WebCore::LayerChromium::setSublayers):
  • platform/graphics/chromium/LayerChromium.h: (WebCore::LayerChromium::setSuperlayer):
3:08 PM Changeset in webkit [59553] by eric@webkit.org
  • 3 edits in trunk/WebCore

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

Reviewed by Timothy Hatcher.

Got rid of absolute positioning in favor of flex boxes where possible.
Added shadow for help window.
Changed window style names to more comprehensible.
​https://bugs.webkit.org/show_bug.cgi?id=39120

  • inspector/front-end/HelpScreen.js: (WebInspector.HelpScreen):
  • inspector/front-end/helpScreen.css: (.help-window-outer): (body.attached .help-window-outer): (.help-window-main): (body.attached .help-window-main): (.help-window-caption): (body.attached .help-window-caption): (.help-window-title): (.help-scrollable): (.help-close-button): (body.platform-mac .help-close-button): (body:not(.platform-mac) .help-close-button):
1:29 PM Changeset in webkit [59552] by Simon Hausmann
  • 1 edit
    19 deletes in trunk/WebKitTools

[Qt] Rename QtLauncher to QtTestBrowser
​https://bugs.webkit.org/show_bug.cgi?id=37665

Rubber-stamped by Antti Koivisto.

Forgot to remove the original directory after the rename.

  • QtLauncher: Removed.
1:23 PM Changeset in webkit [59551] by eric@webkit.org
  • 4 edits in trunk/WebCore

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

Unreviewed, rolling out r59549.
​http://trac.webkit.org/changeset/59549
​https://bugs.webkit.org/show_bug.cgi?id=39167

Broke Chromium build (Requested by abarth on #webkit).

  • bindings/scripts/CodeGeneratorV8.pm:
  • storage/IDBRequest.cpp: (WebCore::IDBRequest::suspend): (WebCore::IDBRequest::resume): (WebCore::IDBRequest::timerFired): (WebCore::IDBRequest::onEventCommon):
  • storage/IDBRequest.h:
1:03 PM Changeset in webkit [59550] by eric@webkit.org
  • 5 edits in trunk/WebCore

2010-05-15 Vangelis Kokkevis <​vangelis@chromium.org>

Reviewed by Darin Fisher.

Adding code path for composited layers (LayerChromium) containing only
an Image element.
​https://bugs.webkit.org/show_bug.cgi?id=39142

No new tests, covered under the existing tests for composited layers.

  • platform/graphics/chromium/GraphicsLayerChromium.cpp: (WebCore::GraphicsLayerChromium::setContentsToImage): (WebCore::GraphicsLayerChromium::updateContentsImage):
  • platform/graphics/chromium/GraphicsLayerChromium.h:
  • platform/graphics/chromium/LayerChromium.cpp: (WebCore::LayerChromium::LayerChromium): (WebCore::LayerChromium::setContents):
  • platform/graphics/chromium/LayerChromium.h: (WebCore::LayerChromium::contents):
12:53 PM Changeset in webkit [59549] by eric@webkit.org
  • 4 edits in trunk/WebCore

2010-05-15 Andrei Popescu <​andreip@google.com>

Reviewed by Jeremy Orlow.

IDBRequest is not a proper ActiveDomObject
​https://bugs.webkit.org/show_bug.cgi?id=39001

Add IDBRequest interface to the IsActiveDomType
routine of the CodeGeneratorV8.pm.

Rename IDBRequest::m_stopped to m_suspended and reset
the flag when resume() is called.

No new tests, Indexed Database isn't yet testable.

  • bindings/scripts/CodeGeneratorV8.pm:
  • storage/IDBRequest.cpp: (WebCore::IDBRequest::suspend): (WebCore::IDBRequest::resume): (WebCore::IDBRequest::timerFired): (WebCore::IDBRequest::onEventCommon):
  • storage/IDBRequest.h:
12:41 PM Changeset in webkit [59548] by eric@webkit.org
  • 3 edits in trunk/WebCore

2010-05-15 Andrei Bucur <​abucur@adobe.com>

Reviewed by Gustavo Noronha Silva.

Functions using the cairo_pattern_t object returned by
Pattern::createPlatformPattern sould destroy it after the painting
operation. Failing to do so causes memory leaks.

Test: No test was added because the patch adds no new functionality.
It just completes the normal life cycle of a cairo_pattern_t object.

  • platform/graphics/cairo/FontCairo.cpp: (WebCore::Font::drawGlyphs):
  • platform/graphics/cairo/GraphicsContextCairo.cpp: (WebCore::setPlatformFill): (WebCore::setPlatformStroke):
12:22 PM Changeset in webkit [59547] by oliver@apple.com
  • 7 edits in trunk

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

Reviewed by Maciej Stachowiak.

Incorrect codegen for slowcase of < in 64-bit
​https://bugs.webkit.org/show_bug.cgi?id=39151

Call the correct stud for the slowcases of the < operator.

  • jit/JITArithmetic.cpp: (JSC::JIT::emitSlow_op_jnless): (JSC::JIT::emitSlow_op_jless):

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

Reviewed by Maciej Stachowiak.

Incorrect codegen for slowcase of < in 64-bit
​https://bugs.webkit.org/show_bug.cgi?id=39151

Add yet more tests for the < operator when comparing integer strings.

  • fast/js/comparison-operators-greater-expected.txt:
  • fast/js/comparison-operators-less-expected.txt:
  • fast/js/script-tests/comparison-operators-greater.js:
  • fast/js/script-tests/comparison-operators-less.js:
11:56 AM Changeset in webkit [59546] by eric@webkit.org
  • 2 edits in trunk/WebKitTools

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

Unreviewed, rolling out r59544.
​http://trac.webkit.org/changeset/59544
​https://bugs.webkit.org/show_bug.cgi?id=39165

Cased LayoutTest to start crashing (Requested by abarth on
#webkit).

  • Scripts/old-run-webkit-tests:
11:38 AM Changeset in webkit [59545] by eric@webkit.org
  • 2 edits in trunk/JavaScriptCore

2010-05-15 Leo Yang <​leo.yang@torchmobile.com.cn>

Reviewed by Darin Adler.

Fix bug ​https://bugs.webkit.org/show_bug.cgi?id=38890

  • runtime/DatePrototype.cpp: (JSC::formatLocaleDate):
11:26 AM Changeset in webkit [59544] by eric@webkit.org
  • 2 edits in trunk/WebKitTools

2010-05-15 Mario Sanchez Prada <​msanchez@igalia.com>

Reviewed by Xan Lopez.

Ensure DRT loads GAIL (Gtk+ module), for a11y tests
​https://bugs.webkit.org/show_bug.cgi?id=38648

Add the GTK_MODULES envvar (set to "gail") to the clean
environment when running DRT for the Gtk+ port

  • Scripts/old-run-webkit-tests:
10:34 AM Changeset in webkit [59543] by eric@webkit.org
  • 3 edits in trunk/WebKit/chromium

2010-05-15 Jay Civelli <​jcivelli@chromium.org>

Reviewed by Darin Fisher.

[chromium] Adding a notification on WebViewClient when a new
node gets focused.
​https://bugs.webkit.org/show_bug.cgi?id=38508

  • public/WebViewClient.h: (WebKit::WebViewClient::focusedNodeChanged):
  • src/ChromeClientImpl.cpp: (WebKit::ChromeClientImpl::focusedNodeChanged):
10:01 AM Changeset in webkit [59542] by yael.aharon@nokia.com
  • 5 edits in trunk/LayoutTests

Unreviewed.

Update expected results after 59541

  • platform/qt/fast/dom/Window/window-properties-expected.txt:
  • platform/qt/fast/dom/Window/window-property-descriptors-expected.txt:
  • platform/qt/fast/dom/prototype-inheritance-expected.txt:
  • platform/qt/fast/js/global-constructors-expected.txt:
9:41 AM Changeset in webkit [59541] by yael.aharon@nokia.com
  • 26 edits
    16 adds in trunk

Initial support for HTMLMeterElement
​https://bugs.webkit.org/show_bug.cgi?id=38140

Reviewed by Kent Tamura.

WebCore:

This patch adds support for HTMLMeterElement, including default rendering
of the meter element. That is because some platforms do not have a native gauge.
The feature is turned on only for the Qt platform.

Tests: fast/dom/HTMLMeterElement/meter-element.html

fast/dom/HTMLMeterElement/set-meter-properties.html

  • DerivedSources.cpp:
  • DerivedSources.make:
  • GNUmakefile.am:
  • WebCore.gypi:
  • WebCore.pri:
  • WebCore.pro:
  • WebCore.vcproj/WebCore.vcproj:
  • css/CSSPrimitiveValueMappings.h:

(WebCore::CSSPrimitiveValue::CSSPrimitiveValue):

  • css/CSSValueKeywords.in:
  • css/html.css:

(meter):
(meter::-webkit-meter):

  • html/HTMLAttributeNames.in:
  • html/HTMLElement.cpp:

(WebCore::inlineTagList):

  • html/HTMLElementsAllInOne.cpp:
  • html/HTMLMeterElement.cpp: Added.

(WebCore::HTMLMeterElement::HTMLMeterElement):
(WebCore::HTMLMeterElement::create):
(WebCore::HTMLMeterElement::createRenderer):
(WebCore::HTMLMeterElement::formControlType):
(WebCore::HTMLMeterElement::parseMappedAttribute):
(WebCore::HTMLMeterElement::min):
(WebCore::HTMLMeterElement::setMin):
(WebCore::HTMLMeterElement::max):
(WebCore::HTMLMeterElement::setMax):
(WebCore::HTMLMeterElement::value):
(WebCore::HTMLMeterElement::setValue):
(WebCore::HTMLMeterElement::low):
(WebCore::HTMLMeterElement::setLow):
(WebCore::HTMLMeterElement::high):
(WebCore::HTMLMeterElement::setHigh):
(WebCore::HTMLMeterElement::optimum):
(WebCore::HTMLMeterElement::setOptimum):

  • html/HTMLMeterElement.h: Added.

(WebCore::HTMLMeterElement::recalcWillValidate):

  • html/HTMLMeterElement.idl: Added.
  • html/HTMLTagNames.in:
  • page/DOMWindow.idl:
  • platform/ThemeTypes.h:
  • rendering/RenderMeter.cpp: Added.

(WebCore::RenderMeter::RenderMeter):
(WebCore::RenderMeter::layout):
(WebCore::RenderMeter::updateFromElement):

  • rendering/RenderMeter.h: Added.

(WebCore::RenderMeter::renderName):
(WebCore::RenderMeter::isMeter):
(WebCore::toRenderMeter):

  • rendering/RenderObject.h:

(WebCore::RenderObject::isMeter):

  • rendering/RenderTheme.cpp:

(WebCore::RenderTheme::adjustStyle):
(WebCore::RenderTheme::paint):
(WebCore::RenderTheme::paintBorderOnly):
(WebCore::RenderTheme::paintDecorations):
(WebCore::RenderTheme::adjustMeterStyle):
(WebCore::RenderTheme::paintMeter):

  • rendering/RenderTheme.h:

LayoutTests:

  • fast/dom/HTMLMeterElement: Added.
  • fast/dom/HTMLMeterElement/meter-element.html: Added.
  • fast/dom/HTMLMeterElement/script-tests: Added.
  • fast/dom/HTMLMeterElement/script-tests/TEMPLATE.html: Added.
  • fast/dom/HTMLMeterElement/script-tests/set-meter-properties.js: Added.
  • fast/dom/HTMLMeterElement/set-meter-properties-expected.txt: Added.
  • fast/dom/HTMLMeterElement/set-meter-properties.html: Added.
  • platform/chromium/test_expectations.txt:
  • platform/gtk/Skipped:
  • platform/mac/Skipped:
  • platform/qt/fast/dom/HTMLMeterElement: Added.
  • platform/qt/fast/dom/HTMLMeterElement/meter-element-expected.checksum: Added.
  • platform/qt/fast/dom/HTMLMeterElement/meter-element-expected.png: Added.
  • platform/qt/fast/dom/HTMLMeterElement/meter-element-expected.txt: Added.
  • platform/win/Skipped:
9:36 AM Changeset in webkit [59540] by eric@webkit.org
  • 10 edits
    3 copies
    5 adds in trunk

2010-05-15 Eric Uhrhane <​ericu@chromium.org>

Reviewed by Dmitry Titov.

Add bindings for async DB API in Workers.
​https://bugs.webkit.org/show_bug.cgi?id=34992

  • storage/change-version-handle-reuse.html: Refactored to share code with the worker version of the test.
  • storage/change-version-handle-reuse-expected.html: Removed reference to reloading, which this test doesn't do.
  • storage/change-version-handle-reuse.js: This is the extracted shared core of the test.
  • storage/execute-sql-args.html: Refactored to share code with the worker version of the test.
  • storage/execute-sql-args.js: This is the extracted shared core of the test.

These are the worker versions of the tests; they're tiny shims around shared code.

  • fast/workers/storage/execute-sql-args-worker-expected.txt: Added.
  • fast/workers/storage/execute-sql-args-worker.html: Added.
  • fast/workers/storage/change-version-handle-reuse-worker-expected.txt: Added.
  • fast/workers/storage/change-version-handle-reuse-worker.html: Added. This file gets included from worker database tests; it does all the boilerplate that's needed on the DOM side.
  • fast/workers/storage/resources/database-worker-controller.js: Added. This file is the boilerplate for the worker side of the database tests. It will import and run the actual test on command, and will report back completion and exceptions.
  • fast/workers/storage/resources/database-worker.js: Added.

2010-05-15 Eric Uhrhane <​ericu@chromium.org>

Reviewed by Dmitry Titov.

Add bindings for async DB API in Workers.
​https://bugs.webkit.org/show_bug.cgi?id=34992

Tests: storage/change-version-handle-reuse-worker.html

storage/execute-sql-args-worker.html

  • bindings/js/JSWorkerContextCustom.cpp: Add openDatabase binding. (WebCore::JSWorkerContext::openDatabase):
  • bindings/v8/custom/V8WorkerContextCustom.cpp: Add openDatabase stub; Chromium will need work both in V8 and in the browser process before we can turn this on there. (WebCore::V8WorkerContext::openDatabaseCallback):

Add NoStaticTables flags to all objects now shared with workers.

  • storage/Database.idl:
  • storage/SQLError.idl:
  • storage/SQLResultSet.idl:
  • storage/SQLResultSetRowList.idl:
  • storage/SQLTransaction.idl:
  • workers/WorkerContext.h: Add databaseExceededQuota.
  • workers/WorkerContext.cpp: (WebCore::WorkerContext::databaseExceededQuota): Add stub implementation for testing; you just get 5MB for now. (WebCore::WorkerContext::openDatabase): Remove invalid assertion.

Add the IDL for the call to openDatabase.

  • workers/WorkerContext.idl:
7:00 AM Changeset in webkit [59539] by eric@webkit.org
  • 3 edits in trunk/WebKit/qt

2010-05-15 Charles Wei <​charles.wei@torchmobile.com.cn>

Reviewed by Simon Hausmann.

This patch fixes the build failure of WebKit on Linux for Qt4.5
​https://bugs.webkig.org/show_bug.cgi?id=39108

  • Api/qgraphicswebview.cpp: (QGraphicsWebViewPrivate::syncLayers):
  • WebCoreSupport/QtPlatformPlugin.cpp:
6:49 AM Changeset in webkit [59538] by eric@webkit.org
  • 1 edit
    1 add in trunk/WebKitTools

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

Reviewed by Maciej Stachowiak.

Add script to check for minimum python version and install if missing on Tiger
​https://bugs.webkit.org/show_bug.cgi?id=38886

Per Maciej's request on webkit-dev:
​https://lists.webkit.org/pipermail/webkit-dev/2010-May/012785.html
provide a script which can automatically install Python on Tiger where
the system provided version is too old to be of use.

Note this uses the official Mac Python installer from python.org.
This installs a copy of Python in /Library/Frameworks/Python.framework.
It also makes symlinks from /usr/local/bin to the Python.framework/bin.

I have tested this script on Leopard and it worked fine. I have not
tested it on Tiger as I do not have access to a Tiger machine. In
either case this should provide a great starting point for someone
wishing to upgrade their copy of Python on Tiger.

Future patches can make our scripts depend on a success return from this
script and then they can either skip tests/sections for which python is
insufficient, or they can fail themselves.

  • Scripts/ensure-valid-python: Added.
6:37 AM Changeset in webkit [59537] by eric@webkit.org
  • 6 edits
    36 adds in trunk

2010-05-15 Leandro Pereira <​leandro@profusion.mobi>

Reviewed by Adam Treat.

[EFL] Add build system for the EFL port.
​http://webkit.org/b/37945

  • CMakeLists.txt: Added.
  • cmake/FindCFLite.cmake: Added.
  • cmake/FindCairo.cmake: Added.
  • cmake/FindEFL.cmake: Added.
  • cmake/FindFreetype.cmake: Added.
  • cmake/FindGDK-PixBuf.cmake: Added.
  • cmake/FindGDK.cmake: Added.
  • cmake/FindGIO.cmake: Added.
  • cmake/FindGlib.cmake: Added.
  • cmake/FindICU.cmake: Added.
  • cmake/FindLibSoup2.cmake: Added.
  • cmake/FindLibXlst.cmake: Added.
  • cmake/FindPango.cmake: Added.
  • cmake/FindSqlite.cmake: Added.
  • cmake/LibFindMacros.cmake: Added.
  • cmake/OptionsCommon.cmake: Added.
  • cmake/OptionsEfl.cmake: Added.
  • cmake/WebKitEfl.cmake: Added.
  • cmake/WebKitFS.cmake: Added.
  • cmake/WebKitFeatures.cmake: Added.
  • cmake/WebKitGenerators.cmake: Added.
  • cmake/WebKitHelpers.cmake: Added.
  • cmake/WebKitMacros.cmake: Added.
  • cmakeconfig.h.cmake: Added.

2010-05-15 Leandro Pereira <​leandro@profusion.mobi>

Reviewed by Adam Treat.

[EFL] Add build system for the EFL port.
​http://webkit.org/b/37945

  • CMakeLists.txt: Added.
  • CMakeListsEfl.txt: Added.
  • config.h: Add conditional to look for cmakeconfig.h.
  • jsc/CMakeLists.txt: Added.
  • jsc/CMakeListsEfl.txt: Added.
  • wtf/CMakeLists.txt: Added.
  • wtf/CMakeListsEfl.txt: Added.

2010-05-15 Leandro Pereira <​leandro@profusion.mobi>

Reviewed by Adam Treat.

[EFL] Add build system for the EFL port.
​http://webkit.org/b/37945

  • CMakeLists.txt: Added.
  • CMakeListsEfl.txt: Added.
  • config.h: Add conditional to look for cmakeconfig.h.

2010-05-15 Leandro Pereira <​leandro@profusion.mobi>

Reviewed by Adam Treat.

[EFL] Add build system for the EFL port.
​http://webkit.org/b/37945

  • CMakeLists.txt: Added.
  • CMakeListsEfl.txt: Added.
6:14 AM Changeset in webkit [59536] by eric@webkit.org
  • 4 edits in trunk/WebCore

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

Reviewed by Darin Adler.

Clipboard.setData doesn't need to be custom
​https://bugs.webkit.org/show_bug.cgi?id=39133

The implementation of the custom binding for this function was a bit
different than its declaration in the IDL file. I've updated the IDL
to match the implementation, but we probably should check whether our
behavior is correct with respect to the specification.

No tests because I've maintained our current behavior.

  • bindings/js/JSClipboardCustom.cpp:
  • bindings/v8/custom/V8ClipboardCustom.cpp:
  • dom/Clipboard.idl:
5:09 AM Changeset in webkit [59535] by xan@webkit.org
  • 2 edits in trunk/LayoutTests

2010-05-15 Xan Lopez <​xlopez@igalia.com>

Skip failing test, it depends on one patch still in the commit
queue.

  • platform/gtk/Skipped:
3:42 AM Changeset in webkit [59534] by eric@webkit.org
  • 9 edits
    2 copies
    1 add in trunk/WebKitTools

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

Reviewed by Adam Barth.

Display queue position inside EWS bubbles
​https://bugs.webkit.org/show_bug.cgi?id=38979

This ended up being a lot of clean-up to our status server code.

Added a new WorkItems model, a form with which to fill it,
and taught the Attachment class how to calculate the current queue
postion for an attachment using the data in WorkItems.

I also finally made statusbubble.* and dashboard.* not use copy-paste code.

The Attachment class has this summary() method which is very
controller/view-like and does not belong in a model class.
This patch got rid of all direct uses of summary().

  • QueueStatusServer/handlers/dashboard.py:
    • Build row objects to hand off to the view instead of handing off a summary object and expecting the view to process it directly.
  • QueueStatusServer/handlers/statusbubble.py:
    • Build bubble object and hand them off to the view.
  • QueueStatusServer/handlers/updatestatus.py:
    • Code cleanup. Just move some code into _queue_status_from_request to make the main put() handler easier to read.
  • QueueStatusServer/handlers/updateworkitems.py: Added.
    • Controller to handle storing WorkItems model objects.
  • QueueStatusServer/main.py:
    • Add route for /update-work-items
  • QueueStatusServer/model/attachment.py:
    • Add new methods to replace direct summary() access.
    • Teach attachment how to calculate queue positions from WorkItems data.
  • QueueStatusServer/model/queues.py:
    • Move name_with_underscores here for easier re-use.
  • QueueStatusServer/model/workitems.py: Added.
    • New model for storing what items are currently queue for any bot.
  • QueueStatusServer/templates/dashboard.html:
    • Kill the copy/paste code!
  • QueueStatusServer/templates/statusbubble.html:
    • Kill the copy/paste code!
  • QueueStatusServer/templates/updateworkitems.html: Added.
    • Simple form for updating a queue's current work items.
3:14 AM Changeset in webkit [59533] by eric@webkit.org
  • 7 edits in trunk/WebCore

2010-05-15 Yaar Schnitman <​yaar@chromium.org>

Reviewed by Adam Barth.

V8 overload support ported to JSC. Unforking of overloaded declarations in WebGL & Canvas will be done in subsequent patches.
​https://bugs.webkit.org/show_bug.cgi?id=38920

Note that JSTestCallback.cpp/h include minor irrelevant changes too since somebody forgot to update them after changing the JS generator.

  • bindings/scripts/CodeGenerator.pm:
  • bindings/scripts/CodeGeneratorJS.pm:
  • bindings/scripts/CodeGeneratorV8.pm:
  • bindings/scripts/test/JS/JSTestCallback.cpp: (WebCore::JSTestCallback::JSTestCallback): (WebCore::JSTestCallback::~JSTestCallback):
  • bindings/scripts/test/JS/JSTestCallback.h:
  • bindings/scripts/test/JS/JSTestObj.cpp: (WebCore::): (WebCore::jsTestObjPrototypeFunctionOverloadedMethod1): (WebCore::jsTestObjPrototypeFunctionOverloadedMethod2): (WebCore::jsTestObjPrototypeFunctionOverloadedMethod3): (WebCore::jsTestObjPrototypeFunctionOverloadedMethod4): (WebCore::testObjPrototypeFunctionOverloadedMethod):
  • bindings/scripts/test/JS/JSTestObj.h:
  • bindings/scripts/test/TestObj.idl:
2:55 AM Changeset in webkit [59532] by eric@webkit.org
  • 5 edits
    2 adds in trunk

2010-05-15 Joanmarie Diggs <​joanmarie.diggs@gmail.com>

Reviewed by Xan Lopez.

​https://bugs.webkit.org/show_bug.cgi?id=30500
[Gtk] Find a way for WebKit to "announce" itself so that ATs can readily distinguish it from true Gtk/Gail

The "announcement" is now made in the form of an object attribute
associated with the AtkObject.

  • platform/gtk/accessibility/object-attributes.html: Added.
  • platform/gtk/accessibility/object-attributes-expected.txt: Added.

2010-05-15 Joanmarie Diggs <​joanmarie.diggs@gmail.com>

Reviewed by Xan Lopez.

​https://bugs.webkit.org/show_bug.cgi?id=30500
[Gtk] Find a way for WebKit to "announce" itself so that ATs can readily distinguish it from true Gtk/Gail

The "announcement" is now made in the form of an object attribute
associated with the AtkObject.

  • accessibility/gtk/AccessibilityObjectWrapperAtk.cpp: (webkit_accessible_get_attributes):

2010-05-15 Joanmarie Diggs <​joanmarie.diggs@gmail.com>

Reviewed by Xan Lopez.

​https://bugs.webkit.org/show_bug.cgi?id=30500
[Gtk] Find a way for WebKit to "announce" itself so that ATs can readily distinguish it from true Gtk/Gail

The "announcement" is now made in the form of an object attribute
associated with the AtkObject.

  • DumpRenderTree/gtk/AccessibilityUIElementGtk.cpp: (AccessibilityUIElement::allAttributes): (attributeSetToString):
2:44 AM Changeset in webkit [59531] by eric@webkit.org
  • 4 edits in trunk

2010-05-15 Mario Sanchez Prada <​msanchez@igalia.com>

Reviewed by Xan Lopez.

Accessibility: Implement isSelected in DRT for GTK
​https://bugs.webkit.org/show_bug.cgi?id=31018

Do not skip aria-controls-with-tabs.html anymore

  • platform/gtk/Skipped:

2010-05-15 Mario Sanchez Prada <​msanchez@igalia.com>

Reviewed by Xan Lopez.

Accessibility: Implement isSelected in DRT for GTK
​https://bugs.webkit.org/show_bug.cgi?id=31018

Implement AccessibilityUIElement::isSelected() for Gtk

  • DumpRenderTree/gtk/AccessibilityUIElementGtk.cpp: (AccessibilityUIElement::isSelected):
2:33 AM Changeset in webkit [59530] by eric@webkit.org
  • 4 edits in trunk/LayoutTests

2010-05-15 Yael Aharon <​yael.aharon@nokia.com>

Reviewed by Darin Adler.

Update notifications test results
​https://bugs.webkit.org/show_bug.cgi?id=39126
Add missing line endings.

  • fast/notifications/notifications-replace-expected.txt:
  • fast/notifications/notifications-request-permission-expected.txt:
  • fast/notifications/notifications-rtl-expected.txt:
2:21 AM Changeset in webkit [59529] by eric@webkit.org
  • 3 edits in trunk/WebKit/chromium

2010-05-15 Evan Stade <​estade@chromium.org>

Reviewed by Adam Barth.

[chromium] "Check spelling in this field" context menu item always checked
​https://bugs.webkit.org/show_bug.cgi?id=39018

Initialize all POD members of WebContextMenuData in a default
constructor.

No tests because this code would need some major refactoring to unit
test, and AFAICT it's not testable via layout tests. See note at top
of getCustomMenuFromDefaultItems.

  • public/WebContextMenuData.h: (WebKit::WebContextMenuData::WebContextMenuData):
  • src/ContextMenuClientImpl.cpp: (WebKit::ContextMenuClientImpl::getCustomMenuFromDefaultItems):
2:10 AM Changeset in webkit [59528] by eric@webkit.org
  • 6 edits in trunk

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

Reviewed by Darin Adler.

Show a console message when X-Frame-Options blocks a load
​https://bugs.webkit.org/show_bug.cgi?id=39087

Update expected results to show the console message.

  • http/tests/security/XFrameOptions/x-frame-options-deny-meta-tag-expected.txt:
  • http/tests/security/XFrameOptions/x-frame-options-deny-meta-tag-in-body-expected.txt:
  • http/tests/security/XFrameOptions/x-frame-options-deny-meta-tag-parent-same-origin-deny-expected.txt:

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

Reviewed by Darin Adler.

Show a console message when X-Frame-Options blocks a load
​https://bugs.webkit.org/show_bug.cgi?id=39087

The blank page confuses people when X-Frame-Options blocks a load. The
console message should at least give them a hint as to what's going on.

  • dom/Document.cpp: (WebCore::Document::processHttpEquiv):
1:58 AM Changeset in webkit [59527] by eric@webkit.org
  • 4 edits in trunk/JavaScriptCore

2010-05-15 Chao-ying Fu <​fu@mips.com>

Reviewed by Oliver Hunt.

Update MIPS JIT for unsigned right shift, Math.sqrt, load16
​https://bugs.webkit.org/show_bug.cgi?id=38412

Fixed MIPS build failure.

  • assembler/MIPSAssembler.h: (JSC::MIPSAssembler::srl): (JSC::MIPSAssembler::srlv): (JSC::MIPSAssembler::sqrtd):
  • assembler/MacroAssemblerMIPS.h: (JSC::MacroAssemblerMIPS::urshift32): (JSC::MacroAssemblerMIPS::sqrtDouble): (JSC::MacroAssemblerMIPS::load16): (JSC::MacroAssemblerMIPS::supportsFloatingPointSqrt):
  • jit/JSInterfaceJIT.h:
1:46 AM Changeset in webkit [59526] by eric@webkit.org
  • 11 edits
    1 copy in trunk

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

Reviewed by Geoffrey Garen.

WebInspector: JSC Should provide heap size info for Timeline panel.
​https://bugs.webkit.org/show_bug.cgi?id=38420

  • JavaScriptCore.exp:
  • JavaScriptCore.vcproj/JavaScriptCore/JavaScriptCore.def:
  • runtime/Collector.cpp: (JSC::Heap::size):
  • runtime/Collector.h:

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

Reviewed by Geoffrey Garen.

WebInspector: JSC Should provide heap size info for Timeline panel.
​https://bugs.webkit.org/show_bug.cgi?id=38420

  • GNUmakefile.am:
  • WebCore.pro:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • bindings/js/ScriptGCEvent.cpp: Added. (WebCore::ScriptGCEvent::getHeapSize):
  • bindings/js/ScriptGCEvent.h:
1:25 AM Changeset in webkit [59525] by eric@webkit.org
  • 6 edits in trunk

2010-05-15 Dave Tapuska <​dtapuska@rim.com>

Reviewed by Darin Adler.

Add a new define for we need strict arena alignment. ARMv5 with
double word instructions set this value.

  • wtf/Platform.h:

2010-05-15 Dave Tapuska <​dtapuska@rim.com>

Reviewed by Darin Adler.

Fix alignment faults on ARMv5TE architectures.

In debug mode the RenderAreaDebugHeader was 12 bytes which
caused an alignment failure for RenderObjects because they
would start on a 4 byte boundary. Align the RenderArenaDebugHeader
using the ARENA_ALIGN macro.

In NDEBUG mode the RenderArena objects come from a pool which
has 4 byte alignment. Allow the alignment to be based on the
AllocAlignmentInteger. This define will be activated for ARMv5TE.

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

May 14, 2010:

11:50 PM Changeset in webkit [59524] by darin@chromium.org
  • 18 edits in trunk/WebKit/chromium

2010-05-14 Darin Fisher <​darin@chromium.org>

Reviewed by Dimitri Glazkov.

[chromium] Remove temporary relative include paths
​https://bugs.webkit.org/show_bug.cgi?id=38776

  • public/WebPasswordAutocompleteListener.h:
  • src/FrameLoaderClientImpl.h:
  • src/PlatformMessagePortChannel.h:
  • src/WebAnimationControllerImpl.h:
  • src/WebBindings.cpp:
  • src/WebDataSourceImpl.h:
  • src/WebDevToolsAgentPrivate.h:
  • src/WebFileChooserCompletionImpl.h:
  • src/WebFrameImpl.h:
  • src/WebInputEventConversion.h:
  • src/WebPluginContainerImpl.h:
  • src/WebPluginLoadObserver.h:
  • src/WebPopupMenuImpl.h:
  • src/WebSettingsImpl.h:
  • src/WebViewImpl.h:
  • src/WebWorkerClientImpl.h:
  • src/WrappedResourceRequest.h:
  • src/WrappedResourceResponse.h:
11:32 PM Changeset in webkit [59523] by abarth@webkit.org
  • 4 edits in trunk/WebCore

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

Reviewed by Darin Adler.

XMLHttpRequest.overrideMimeType doesn't need to be custom
​https://bugs.webkit.org/show_bug.cgi?id=39127

I don't see any reason why this method should be custom. No tests
because no behavior change.

  • bindings/js/JSXMLHttpRequestCustom.cpp:
  • bindings/v8/custom/V8XMLHttpRequestCustom.cpp:
  • xml/XMLHttpRequest.idl:
11:28 PM Changeset in webkit [59522] by abarth@webkit.org
  • 2 edits in trunk/WebKitTools

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

Reviewed by Eric Seidel.

scm.py should use self.run instead of run_command
​https://bugs.webkit.org/show_bug.cgi?id=38957

We've wanted to do this for a while, but it's a prerequiste for running
SVN from the cwd instead of the checkout_root.

  • Scripts/webkitpy/common/checkout/scm.py:
11:27 PM Changeset in webkit [59521] by sfalken@apple.com
  • 5 edits in tags/Safari-533.9.3

Versioning.

11:11 PM Changeset in webkit [59520] by sfalken@apple.com
  • 2 edits in tags/Safari-533.9.3/WebCore

Merge r59515.

11:07 PM Changeset in webkit [59519] by sfalken@apple.com
  • 1 copy in tags/Safari-533.9.3

New tag.

11:05 PM Changeset in webkit [59518] by sfalken@apple.com
  • 2 edits in branches/safari-533-branch/WebCore

Merge r59515.

10:32 PM Changeset in webkit [59517] by abarth@webkit.org
  • 2 edits in trunk/LayoutTests

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

Unreviewed.

Skip new test that fails on Qt. I'm sure skipping is the wrong thing
but no one is around to help me figure out what the right thing to do
is and I don't want to leave the bot red all weekend.

  • platform/qt/Skipped:
9:46 PM Changeset in webkit [59516] by adele@apple.com
  • 3 edits
    2 adds in trunk

WebCore: Fix for ​https://bugs.webkit.org/show_bug.cgi?id=39095
<rdar://problem/7984158> REGRESSION (r53085): Infinite recursion in Position::getInlineBoxAndOffset when changing writing direction to right-to-left

Reviewed by Dave Hyatt.

Test: editing/selection/applystyle-to-inline-inside-block.html

  • dom/Position.cpp: (WebCore::Position::getInlineBoxAndOffset): Now that this function correctly goes in and out of editable areas by calling

downstreamIgnoringEditingBoundaries and upstreamIgnoringEditingBoundaries for blocks (added in r58665), we no longer need a special case to
jump out to the parent when we hit an inline (which was added in r53085 when this issue was introduced).

LayoutTests: Test for ​https://bugs.webkit.org/show_bug.cgi?id=39095
<rdar://problem/7984158> REGRESSION (r53085): Infinite recursion in Position::getInlineBoxAndOffset when changing writing direction to right-to-left

Reviewed by Dave Hyatt.

  • editing/selection/applystyle-to-inline-in-block-expected.txt: Added.
  • editing/selection/applystyle-to-inline-in-block.html: Added.
8:49 PM Changeset in webkit [59515] by bweinstein@apple.com
  • 2 edits in trunk/WebCore

Hopefully a last fix for the build. Call -mkdir instead of mkdir, so we don't bail if the mkdir call fails. Additionally,
use the %Env% syntax instead of $(Env), which is what we need if we are referring to Windows environment variables instead
of nmake variables.

Reviewed by Mark Rowe.

  • WebCore.vcproj/WebCore.make:
8:40 PM Changeset in webkit [59514] by Stephanie Lewis
  • 6 edits in trunk

Update order files.

Rubber-stamped by Mark Rowe.

JavaScriptCore:

WebCore:

  • WebCore.order:

WebKit/mac:

  • WebKit.order:
8:17 PM Changeset in webkit [59513] by hyatt@apple.com
  • 16 edits
    2 adds in trunk

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

Reviewed by Simon Fraser.

WebCore:

Make RenderLayers position properly inside multicolumn layout.

(1) Create RenderLayers when column properties are specified. This allows descendant layers to easily
know they are inside a multicolumn layout when positioning.

(2) Patch updateLayerPosition to offset the layer into the first column that contains the layer. Splitting
of a layer across columns still does not work, but the layer will now at least appear in the first column in
which it occurs.

(3) Fix pagination bugs caused by the fact that visual overflow doesn't propagate up the line box tree when layers
are involved. Make sure to use lineTop and lineBottom in this case (it's not perfect, but it gets most cases
working properly). With this change elements like <video> will paginate properly and not get split across pages.

(4) Make column repainting and layer positioning work with RTL columns by fixing adjustForColumns to
account for RTL.

(5) Fix RTL pagination by making sure the column pagination rects are the content width of the whole block and not just
the width of one column.

Added fast/multicol/layers-in-multicol.html

  • rendering/RenderBlock.cpp:

(WebCore::RenderBlock::layoutColumns):
(WebCore::RenderBlock::adjustForColumns):

  • rendering/RenderBox.h:

(WebCore::RenderBox::locationOffset):

  • rendering/RenderBoxModelObject.h:

(WebCore::RenderBoxModelObject::requiresLayer):

  • rendering/RenderLayer.cpp:

(WebCore::RenderLayer::updateLayerPosition):
(WebCore::RenderLayer::convertToLayerCoords):
(WebCore::RenderLayer::shouldBeNormalFlowOnly):

  • rendering/RenderLineBoxList.cpp:

(WebCore::RenderLineBoxList::paint):

  • rendering/RenderView.cpp:

(WebCore::RenderView::RenderView):

  • rendering/RenderView.h:

(WebCore::RenderView::requiresLayer):

  • rendering/style/RenderStyle.h:

(WebCore::InheritedFlags::specifiesColumns):

LayoutTests:

Reset all multicolumn test results now that columns create RenderLayers.

  • fast/multicol/client-rects-expected.txt:
  • fast/multicol/layers-in-multicol.html: Added.
  • fast/multicol/single-line-expected.txt:
  • platform/mac/fast/multicol/column-rules-expected.txt:
  • platform/mac/fast/multicol/columns-shorthand-parsing-expected.txt:
  • platform/mac/fast/multicol/float-avoidance-expected.txt:
  • platform/mac/fast/multicol/float-multicol-expected.txt:


Add a layout test for various types of RenderLayers inside multicol layouts.

  • platform/mac/fast/multicol/layers-in-multicol-expected.txt: Added.
7:54 PM Changeset in webkit [59512] by eric@webkit.org
  • 5 edits in trunk

2010-05-14 Anton Muhin <​antonm@chromium.org>

Reviewed by Darin Fisher.

[Chromium] Consider implementing addOriginAccessWhitelistEntry method
​https://bugs.webkit.org/show_bug.cgi?id=37578

Remove deprecated methods.

  • public/WebSecurityPolicy.h:
  • src/WebSecurityPolicy.cpp:

2010-05-14 Anton Muhin <​antonm@chromium.org>

Reviewed by Darin Fisher.

[Chromium] Consider implementing addOriginAccessWhitelistEntry method
​https://bugs.webkit.org/show_bug.cgi?id=37578

Remove deprecated methods.

  • DumpRenderTree/chromium/LayoutTestController.cpp
  • DumpRenderTree/chromium/LayoutTestController.h
6:57 PM Changeset in webkit [59511] by eric@webkit.org
  • 4 edits in trunk/WebCore

2010-05-14 Leon Clarke <​leonclarke@google.com>

Reviewed by Steve Block.

Refactor signature of tokenizeRelAttribute and the variables it refers to so that new features don't keep changing the signature, and make m_disabledState into an enum.
​https://bugs.webkit.org/show_bug.cgi?id=38900

No new tests. Refactoring only.

  • html/HTMLLinkElement.cpp: (WebCore::HTMLLinkElement::HTMLLinkElement): (WebCore::HTMLLinkElement::setDisabledState): (WebCore::HTMLLinkElement::parseMappedAttribute): (WebCore::HTMLLinkElement::tokenizeRelAttribute): (WebCore::HTMLLinkElement::process): (WebCore::HTMLLinkElement::addSubresourceAttributeURLs):
  • html/HTMLLinkElement.h: (WebCore::HTMLLinkElement::RelAttribute::RelAttribute): (WebCore::HTMLLinkElement::isAlternate): (WebCore::HTMLLinkElement::isDisabled): (WebCore::HTMLLinkElement::isEnabledViaScript): (WebCore::HTMLLinkElement::isIcon): (WebCore::HTMLLinkElement::):
  • html/PreloadScanner.cpp: (WebCore::PreloadScanner::processAttribute):
6:43 PM Changeset in webkit [59510] by eric@webkit.org
  • 2 edits in trunk/WebCore

2010-05-14 Noam Rosenthal <​noam.rosenthal@nokia.com>

Reviewed by Kenneth Rohde Christiansen.

[Qt] GraphicsLayer caches directly composited images
​https://bugs.webkit.org/show_bug.cgi?id=38444

Directly-composited images and solid fills shouldn't be cached, as that cache
is never used (see GraphicsLayerQtImpl::paint). Cache is only relevant for HTML content,
but we were missing that test.
The fix makes sure we only cache HTML content.

No new tests: this is a minor optimization.

  • platform/graphics/qt/GraphicsLayerQt.cpp: (WebCore::GraphicsLayerQtImpl::flushChanges):
6:18 PM Changeset in webkit [59509] by mrowe@apple.com
  • 5 edits in tags/Safari-533.9.2

Versioning.

6:08 PM Changeset in webkit [59508] by mrowe@apple.com
  • 2 edits in tags/Safari-533.9.2/WebCore

Merge r59505.

5:56 PM Changeset in webkit [59507] by mrowe@apple.com
  • 1 copy in tags/Safari-533.9.2

New tag.

5:55 PM Changeset in webkit [59506] by mrowe@apple.com
  • 2 edits in branches/safari-533-branch/WebCore

Merge r59505.

5:45 PM Changeset in webkit [59505] by bweinstein@apple.com
  • 2 edits in trunk/WebCore

Build fix. Had the environment variables wrong for source and destination of the copy operation
that was added.

Reviewed by Mark Rowe.

  • WebCore.vcproj/WebCore.make:
5:29 PM Changeset in webkit [59504] by ap@apple.com
  • 2 edits
    4 moves in trunk/LayoutTests

Inspired by Simon Fraser, rubber-stamped by Geoff Garen.

Disabled Sputnik tests that take over 10 seconds in debug builds on my MacBook Pro.

  • platform/mac-tiger/Skipped: These tests were alrady skipped on Tiger, no need to skip explicitly any more.
  • fast/js/sputnik/Conformance/15_Native_Objects/15.1_The_Global_Object/15.1.3/15.1.3.1_decodeURI/S15.1.3.1_A2.5_T1.html: Removed.
  • fast/js/sputnik/Conformance/15_Native_Objects/15.1_The_Global_Object/15.1.3/15.1.3.1_decodeURI/S15.1.3.1_A2.5_T1.html-disabled: Copied from fast/js/sputnik/Conformance/15_Native_Objects/15.1_The_Global_Object/15.1.3/15.1.3.1_decodeURI/S15.1.3.1_A2.5_T1.html.
  • fast/js/sputnik/Conformance/15_Native_Objects/15.1_The_Global_Object/15.1.3/15.1.3.2_decodeURIComponent/S15.1.3.2_A2.5_T1.html: Removed.
  • fast/js/sputnik/Conformance/15_Native_Objects/15.1_The_Global_Object/15.1.3/15.1.3.2_decodeURIComponent/S15.1.3.2_A2.5_T1.html-disabled: Copied from fast/js/sputnik/Conformance/15_Native_Objects/15.1_The_Global_Object/15.1.3/15.1.3.2_decodeURIComponent/S15.1.3.2_A2.5_T1.html.
  • fast/js/sputnik/Unicode/Unicode_218/S7.6_A3.1.html: Removed.
  • fast/js/sputnik/Unicode/Unicode_218/S7.6_A3.1.html-disabled: Copied from fast/js/sputnik/Unicode/Unicode_218/S7.6_A3.1.html.
  • fast/js/sputnik/Unicode/Unicode_218/S7.6_A3.2.html: Removed.
  • fast/js/sputnik/Unicode/Unicode_218/S7.6_A3.2.html-disabled: Copied from fast/js/sputnik/Unicode/Unicode_218/S7.6_A3.2.html.
5:08 PM Changeset in webkit [59503] by eric@webkit.org
  • 9 edits in trunk/JavaScriptCore

2010-05-14 Jedrzej Nowacki <​jedrzej.nowacki@nokia.com>

Reviewed by Kenneth Rohde Christiansen.

Implementation of QtScript API that provide method to convert a value to an object.

[Qt] QtScript is missing toObject API
​https://bugs.webkit.org/show_bug.cgi?id=36594

  • qt/api/qscriptengine.cpp: (QScriptEngine::toObject):
  • qt/api/qscriptengine.h:
  • qt/api/qscriptvalue.cpp: (QScriptValue::toObject):
  • qt/api/qscriptvalue.h:
  • qt/api/qscriptvalue_p.h: (QScriptValuePrivate::toObject):
  • qt/tests/qscriptengine/tst_qscriptengine.cpp: (tst_QScriptEngine::toObject): (tst_QScriptEngine::toObjectTwoEngines):
  • qt/tests/qscriptvalue/tst_qscriptvalue.cpp: (tst_QScriptValue::toObjectSimple):
  • qt/tests/qscriptvalue/tst_qscriptvalue.h:
4:34 PM Changeset in webkit [59502] by abarth@webkit.org
  • 7 edits in trunk/WebCore

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

Reviewed by Eric Seidel.

Remove custom bindings for HTMLInputElement
​https://bugs.webkit.org/show_bug.cgi?id=38344

Fix a bug in the TestObj.idl file and update expected results. This
was originally part of a larger change reviewed by Eric, but I'm
landing it separately to help me understand the more interesting bits
of the patch clearly.

  • bindings/scripts/test/GObject/WebKitDOMTestObj.cpp: (webkit_dom_test_obj_get_attr_with_setter_exception): (webkit_dom_test_obj_set_attr_with_setter_exception): (webkit_dom_test_obj_set_attr_with_getter_exception): (webkit_dom_test_obj_set_property): (webkit_dom_test_obj_get_property):
  • bindings/scripts/test/GObject/WebKitDOMTestObj.h:
  • bindings/scripts/test/JS/JSTestObj.cpp: (WebCore::jsTestObjAttrWithSetterException): (WebCore::setJSTestObjAttrWithSetterException): (WebCore::setJSTestObjAttrWithGetterException):
  • bindings/scripts/test/ObjC/DOMTestObj.mm: (-[DOMTestObj attrWithSetterException]): (-[DOMTestObj setAttrWithSetterException:]): (-[DOMTestObj setAttrWithGetterException:]):
  • bindings/scripts/test/TestObj.idl:
  • bindings/scripts/test/V8/V8TestObj.cpp: (WebCore::TestObjInternal::attrWithSetterExceptionAttrGetter): (WebCore::TestObjInternal::attrWithSetterExceptionAttrSetter): (WebCore::TestObjInternal::attrWithGetterExceptionAttrSetter):
4:12 PM Changeset in webkit [59501] by jparent@chromium.org
  • 2 edits in trunk/BugsSite

Bugzilla: Only show "Rietveld Review" link when in-rietveld is set to +.

Reviewed by Ojan Vafai.

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

  • template/en/custom/attachment/list.html.tmpl:
3:54 PM Changeset in webkit [59500] by jparent@chromium.org
  • 2 edits in trunk/BugsSite

Bugzilla: Don't send mail if the only change is to the in-rietveld flag.

Reviewed by David Kilzer.

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

  • Bugzilla/BugMail.pm:
3:52 PM EFLWebKit edited by leandro@profusion.mobi
(diff)
3:52 PM EFLWebKit edited by leandro@profusion.mobi
(diff)
3:50 PM EFLWebKit edited by leandro@profusion.mobi
(diff)
3:50 PM EFLWebKit edited by leandro@profusion.mobi
(diff)
3:37 PM Changeset in webkit [59499] by kbr@google.com
  • 30 edits
    64 moves in trunk

2010-05-14 Kenneth Russell <​kbr@google.com>

Reviewed by Darin Adler.

Rename WebGLArray types to TypedArray types
​https://bugs.webkit.org/show_bug.cgi?id=39091

Extended functionality of do-webcore-rename script and used it to
rename the WebGLArray types to the TypedArray naming convention.
The only source files which were touched by hand, and which are
being manually reviewed, are:

WebCore/page/DOMWindow.idl
WebCore/bindings/generic/RuntimeEnabledFeatures.h (script's changes undone)
WebKit/WebCore/bindings/js/JSDOMWindowCustom.cpp
WebKit/WebCore/bindings/v8/custom/V8DOMWindowCustom.cpp

These only needed to be touched to update the aliases between the
WebGLArray and TypedArray names introduced in bug 39036. (It was
not feasible to have do-webcore-rename handle this as it would
introduce circular renamings.) These aliases will be removed in
roughly a month once existing WebGL content has been updated.

No new tests; covered under existing WebGL tests. Updated
constructed-objects-prototypes and prototype-inheritance-2 tests.
Ran all layout tests in Safari and all WebGL tests in Chromium.

  • src/GraphicsContext3D.cpp: (WebCore::GraphicsContext3DInternal::bufferData): (WebCore::GraphicsContext3DInternal::bufferSubData):

2010-05-14 Kenneth Russell <​kbr@google.com>

Reviewed by Darin Adler.

Rename WebGLArray types to TypedArray types
​https://bugs.webkit.org/show_bug.cgi?id=39091

Extended functionality of do-webcore-rename script and used it to
rename the WebGLArray types to the TypedArray naming convention.
The only source files which were touched by hand, and which are
being manually reviewed, are:

WebCore/page/DOMWindow.idl
WebCore/bindings/generic/RuntimeEnabledFeatures.h (script's changes undone)
WebKit/WebCore/bindings/js/JSDOMWindowCustom.cpp
WebKit/WebCore/bindings/v8/custom/V8DOMWindowCustom.cpp

These only needed to be touched to update the aliases between the
WebGLArray and TypedArray names introduced in bug 39036. (It was
not feasible to have do-webcore-rename handle this as it would
introduce circular renamings.) These aliases will be removed in
roughly a month once existing WebGL content has been updated.

No new tests; covered under existing WebGL tests. Updated
constructed-objects-prototypes and prototype-inheritance-2 tests.
Ran all layout tests in Safari and all WebGL tests in Chromium.

  • DerivedSources.make:
  • GNUmakefile.am:
  • WebCore.gypi:
  • WebCore.pri:
  • WebCore.pro:
  • WebCore.xcodeproj/project.pbxproj:
  • bindings/js/JSArrayBufferConstructor.cpp: Copied from WebCore/bindings/js/JSWebGLArrayBufferConstructor.cpp. (WebCore::): (WebCore::JSArrayBufferConstructor::JSArrayBufferConstructor): (WebCore::constructCanvasArrayBuffer): (WebCore::JSArrayBufferConstructor::getConstructData):
  • bindings/js/JSArrayBufferConstructor.h: Copied from WebCore/bindings/js/JSWebGLArrayBufferConstructor.h. (WebCore::construct):
  • bindings/js/JSArrayBufferViewCustom.cpp: Copied from WebCore/bindings/js/JSWebGLArrayCustom.cpp. (WebCore::toJS): (WebCore::JSArrayBufferView::slice):
  • bindings/js/JSArrayBufferViewHelper.h: Copied from WebCore/bindings/js/JSWebGLArrayHelper.h.
  • bindings/js/JSDOMWindowCustom.cpp: (WebCore::JSDOMWindow::arrayBuffer): (WebCore::JSDOMWindow::int8Array): (WebCore::JSDOMWindow::uint8Array): (WebCore::JSDOMWindow::int32Array): (WebCore::JSDOMWindow::uint32Array): (WebCore::JSDOMWindow::int16Array): (WebCore::JSDOMWindow::uint16Array): (WebCore::JSDOMWindow::floatArray): (WebCore::JSDOMWindow::webGLArrayBuffer): (WebCore::JSDOMWindow::webGLByteArray): (WebCore::JSDOMWindow::webGLUnsignedByteArray): (WebCore::JSDOMWindow::webGLIntArray): (WebCore::JSDOMWindow::webGLUnsignedIntArray): (WebCore::JSDOMWindow::webGLShortArray): (WebCore::JSDOMWindow::webGLUnsignedShortArray): (WebCore::JSDOMWindow::webGLFloatArray):
  • bindings/js/JSFloatArrayConstructor.cpp: Copied from WebCore/bindings/js/JSWebGLFloatArrayConstructor.cpp. (WebCore::): (WebCore::JSFloatArrayConstructor::JSFloatArrayConstructor): (WebCore::constructCanvasFloatArray): (WebCore::JSFloatArrayConstructor::getConstructData):
  • bindings/js/JSFloatArrayConstructor.h: Copied from WebCore/bindings/js/JSWebGLFloatArrayConstructor.h.
  • bindings/js/JSFloatArrayCustom.cpp: Copied from WebCore/bindings/js/JSWebGLFloatArrayCustom.cpp. (WebCore::JSFloatArray::indexSetter): (WebCore::toJS): (WebCore::JSFloatArray::set):
  • bindings/js/JSInt16ArrayConstructor.cpp: Copied from WebCore/bindings/js/JSWebGLShortArrayConstructor.cpp. (WebCore::): (WebCore::JSInt16ArrayConstructor::JSInt16ArrayConstructor): (WebCore::constructCanvasShortArray): (WebCore::JSInt16ArrayConstructor::getConstructData):
  • bindings/js/JSInt16ArrayConstructor.h: Copied from WebCore/bindings/js/JSWebGLShortArrayConstructor.h.
  • bindings/js/JSInt16ArrayCustom.cpp: Copied from WebCore/bindings/js/JSWebGLShortArrayCustom.cpp. (WebCore::JSInt16Array::indexSetter): (WebCore::toJS): (WebCore::JSInt16Array::set):
  • bindings/js/JSInt32ArrayConstructor.cpp: Copied from WebCore/bindings/js/JSWebGLIntArrayConstructor.cpp. (WebCore::): (WebCore::JSInt32ArrayConstructor::JSInt32ArrayConstructor): (WebCore::constructCanvasIntArray): (WebCore::JSInt32ArrayConstructor::getConstructData):
  • bindings/js/JSInt32ArrayConstructor.h: Copied from WebCore/bindings/js/JSWebGLIntArrayConstructor.h.
  • bindings/js/JSInt32ArrayCustom.cpp: Copied from WebCore/bindings/js/JSWebGLIntArrayCustom.cpp. (WebCore::JSInt32Array::indexSetter): (WebCore::toJS): (WebCore::JSInt32Array::set):
  • bindings/js/JSInt8ArrayConstructor.cpp: Copied from WebCore/bindings/js/JSWebGLByteArrayConstructor.cpp. (WebCore::): (WebCore::JSInt8ArrayConstructor::JSInt8ArrayConstructor): (WebCore::constructCanvasByteArray): (WebCore::JSInt8ArrayConstructor::getConstructData):
  • bindings/js/JSInt8ArrayConstructor.h: Copied from WebCore/bindings/js/JSWebGLByteArrayConstructor.h.
  • bindings/js/JSInt8ArrayCustom.cpp: Copied from WebCore/bindings/js/JSWebGLByteArrayCustom.cpp. (WebCore::JSInt8Array::indexSetter): (WebCore::toJS): (WebCore::JSInt8Array::set):
  • bindings/js/JSUint16ArrayConstructor.cpp: Copied from WebCore/bindings/js/JSWebGLUnsignedShortArrayConstructor.cpp. (WebCore::): (WebCore::JSUint16ArrayConstructor::JSUint16ArrayConstructor): (WebCore::constructCanvasUnsignedShortArray): (WebCore::JSUint16ArrayConstructor::getConstructData):
  • bindings/js/JSUint16ArrayConstructor.h: Copied from WebCore/bindings/js/JSWebGLUnsignedShortArrayConstructor.h.
  • bindings/js/JSUint16ArrayCustom.cpp: Copied from WebCore/bindings/js/JSWebGLUnsignedShortArrayCustom.cpp. (WebCore::JSUint16Array::indexSetter): (WebCore::toJS): (WebCore::JSUint16Array::set):
  • bindings/js/JSUint32ArrayConstructor.cpp: Copied from WebCore/bindings/js/JSWebGLUnsignedIntArrayConstructor.cpp. (WebCore::): (WebCore::JSUint32ArrayConstructor::JSUint32ArrayConstructor): (WebCore::constructCanvasUnsignedIntArray): (WebCore::JSUint32ArrayConstructor::getConstructData):
  • bindings/js/JSUint32ArrayConstructor.h: Copied from WebCore/bindings/js/JSWebGLUnsignedIntArrayConstructor.h.
  • bindings/js/JSUint32ArrayCustom.cpp: Copied from WebCore/bindings/js/JSWebGLUnsignedIntArrayCustom.cpp. (WebCore::JSUint32Array::indexSetter): (WebCore::toJS): (WebCore::JSUint32Array::set):
  • bindings/js/JSUint8ArrayConstructor.cpp: Copied from WebCore/bindings/js/JSWebGLUnsignedByteArrayConstructor.cpp. (WebCore::): (WebCore::JSUint8ArrayConstructor::JSUint8ArrayConstructor): (WebCore::constructCanvasUnsignedByteArray): (WebCore::JSUint8ArrayConstructor::getConstructData):
  • bindings/js/JSUint8ArrayConstructor.h: Copied from WebCore/bindings/js/JSWebGLUnsignedByteArrayConstructor.h.
  • bindings/js/JSUint8ArrayCustom.cpp: Copied from WebCore/bindings/js/JSWebGLUnsignedByteArrayCustom.cpp. (WebCore::JSUint8Array::indexSetter): (WebCore::toJS): (WebCore::JSUint8Array::set):
  • bindings/js/JSWebGLArrayBufferConstructor.cpp: Removed.
  • bindings/js/JSWebGLArrayBufferConstructor.h: Removed.
  • bindings/js/JSWebGLArrayCustom.cpp: Removed.
  • bindings/js/JSWebGLArrayHelper.h: Removed.
  • bindings/js/JSWebGLByteArrayConstructor.cpp: Removed.
  • bindings/js/JSWebGLByteArrayConstructor.h: Removed.
  • bindings/js/JSWebGLByteArrayCustom.cpp: Removed.
  • bindings/js/JSWebGLFloatArrayConstructor.cpp: Removed.
  • bindings/js/JSWebGLFloatArrayConstructor.h: Removed.
  • bindings/js/JSWebGLFloatArrayCustom.cpp: Removed.
  • bindings/js/JSWebGLIntArrayConstructor.cpp: Removed.
  • bindings/js/JSWebGLIntArrayConstructor.h: Removed.
  • bindings/js/JSWebGLIntArrayCustom.cpp: Removed.
  • bindings/js/JSWebGLRenderingContextCustom.cpp: (WebCore::JSWebGLRenderingContext::bufferData): (WebCore::JSWebGLRenderingContext::bufferSubData): (WebCore::JSWebGLRenderingContext::texImage2D): (WebCore::JSWebGLRenderingContext::texSubImage2D): (WebCore::dataFunctionf): (WebCore::dataFunctioni): (WebCore::dataFunctionMatrix):
  • bindings/js/JSWebGLShortArrayConstructor.cpp: Removed.
  • bindings/js/JSWebGLShortArrayConstructor.h: Removed.
  • bindings/js/JSWebGLShortArrayCustom.cpp: Removed.
  • bindings/js/JSWebGLUnsignedByteArrayConstructor.cpp: Removed.
  • bindings/js/JSWebGLUnsignedByteArrayConstructor.h: Removed.
  • bindings/js/JSWebGLUnsignedByteArrayCustom.cpp: Removed.
  • bindings/js/JSWebGLUnsignedIntArrayConstructor.cpp: Removed.
  • bindings/js/JSWebGLUnsignedIntArrayConstructor.h: Removed.
  • bindings/js/JSWebGLUnsignedIntArrayCustom.cpp: Removed.
  • bindings/js/JSWebGLUnsignedShortArrayConstructor.cpp: Removed.
  • bindings/js/JSWebGLUnsignedShortArrayConstructor.h: Removed.
  • bindings/js/JSWebGLUnsignedShortArrayCustom.cpp: Removed.
  • bindings/v8/V8DOMWrapper.cpp:
  • bindings/v8/custom/V8ArrayBufferCustom.cpp: Copied from WebCore/bindings/v8/custom/V8WebGLArrayBufferCustom.cpp. (WebCore::V8ArrayBuffer::constructorCallback):
  • bindings/v8/custom/V8ArrayBufferViewCustom.cpp: Copied from WebCore/bindings/v8/custom/V8WebGLArrayCustom.cpp. (WebCore::toV8): (WebCore::V8ArrayBufferView::sliceCallback):
  • bindings/v8/custom/V8ArrayBufferViewCustom.h: Copied from WebCore/bindings/v8/custom/V8WebGLArrayCustom.h. (WebCore::constructWebGLArray):
  • bindings/v8/custom/V8DOMWindowCustom.cpp: (WebCore::V8DOMWindow::WebGLArrayBufferAccessorGetter): (WebCore::V8DOMWindow::WebGLByteArrayAccessorGetter): (WebCore::V8DOMWindow::WebGLUnsignedByteArrayAccessorGetter): (WebCore::V8DOMWindow::WebGLShortArrayAccessorGetter): (WebCore::V8DOMWindow::WebGLUnsignedShortArrayAccessorGetter): (WebCore::V8DOMWindow::WebGLIntArrayAccessorGetter): (WebCore::V8DOMWindow::WebGLUnsignedIntArrayAccessorGetter): (WebCore::V8DOMWindow::WebGLFloatArrayAccessorGetter):
  • bindings/v8/custom/V8FloatArrayCustom.cpp: Copied from WebCore/bindings/v8/custom/V8WebGLFloatArrayCustom.cpp. (WebCore::V8FloatArray::constructorCallback): (WebCore::V8FloatArray::setCallback): (WebCore::toV8):
  • bindings/v8/custom/V8Int16ArrayCustom.cpp: Copied from WebCore/bindings/v8/custom/V8WebGLShortArrayCustom.cpp. (WebCore::V8Int16Array::constructorCallback): (WebCore::V8Int16Array::setCallback): (WebCore::toV8):
  • bindings/v8/custom/V8Int32ArrayCustom.cpp: Copied from WebCore/bindings/v8/custom/V8WebGLIntArrayCustom.cpp. (WebCore::V8Int32Array::constructorCallback): (WebCore::V8Int32Array::setCallback): (WebCore::toV8):
  • bindings/v8/custom/V8Int8ArrayCustom.cpp: Copied from WebCore/bindings/v8/custom/V8WebGLByteArrayCustom.cpp. (WebCore::V8Int8Array::constructorCallback): (WebCore::V8Int8Array::setCallback): (WebCore::toV8):
  • bindings/v8/custom/V8Uint16ArrayCustom.cpp: Copied from WebCore/bindings/v8/custom/V8WebGLUnsignedShortArrayCustom.cpp. (WebCore::V8Uint16Array::constructorCallback): (WebCore::V8Uint16Array::setCallback): (WebCore::toV8):
  • bindings/v8/custom/V8Uint32ArrayCustom.cpp: Copied from WebCore/bindings/v8/custom/V8WebGLUnsignedIntArrayCustom.cpp. (WebCore::V8Uint32Array::constructorCallback): (WebCore::V8Uint32Array::setCallback): (WebCore::toV8):
  • bindings/v8/custom/V8Uint8ArrayCustom.cpp: Copied from WebCore/bindings/v8/custom/V8WebGLUnsignedByteArrayCustom.cpp. (WebCore::V8Uint8Array::constructorCallback): (WebCore::V8Uint8Array::setCallback): (WebCore::toV8):
  • bindings/v8/custom/V8WebGLArrayBufferCustom.cpp: Removed.
  • bindings/v8/custom/V8WebGLArrayCustom.cpp: Removed.
  • bindings/v8/custom/V8WebGLArrayCustom.h: Removed.
  • bindings/v8/custom/V8WebGLByteArrayCustom.cpp: Removed.
  • bindings/v8/custom/V8WebGLFloatArrayCustom.cpp: Removed.
  • bindings/v8/custom/V8WebGLIntArrayCustom.cpp: Removed.
  • bindings/v8/custom/V8WebGLRenderingContextCustom.cpp: (WebCore::vertexAttribAndUniformHelperf): (WebCore::uniformHelperi): (WebCore::uniformMatrixHelper):
  • bindings/v8/custom/V8WebGLShortArrayCustom.cpp: Removed.
  • bindings/v8/custom/V8WebGLUnsignedByteArrayCustom.cpp: Removed.
  • bindings/v8/custom/V8WebGLUnsignedIntArrayCustom.cpp: Removed.
  • bindings/v8/custom/V8WebGLUnsignedShortArrayCustom.cpp: Removed.
  • html/canvas/ArrayBuffer.cpp: Copied from WebCore/html/canvas/WebGLArrayBuffer.cpp. (WebCore::ArrayBuffer::create): (WebCore::ArrayBuffer::ArrayBuffer): (WebCore::ArrayBuffer::data): (WebCore::ArrayBuffer::byteLength): (WebCore::ArrayBuffer::~ArrayBuffer): (WebCore::ArrayBuffer::tryAllocate):
  • html/canvas/ArrayBuffer.h: Copied from WebCore/html/canvas/WebGLArrayBuffer.h.
  • html/canvas/ArrayBuffer.idl: Copied from WebCore/html/canvas/WebGLArrayBuffer.idl.
  • html/canvas/ArrayBufferView.cpp: Copied from WebCore/html/canvas/WebGLArray.cpp. (WebCore::ArrayBufferView::ArrayBufferView): (WebCore::ArrayBufferView::~ArrayBufferView): (WebCore::ArrayBufferView::setImpl): (WebCore::ArrayBufferView::calculateOffsetAndLength):
  • html/canvas/ArrayBufferView.h: Copied from WebCore/html/canvas/WebGLArray.h. (WebCore::ArrayBufferView::buffer): (WebCore::ArrayBufferView::verifySubRange): (WebCore::ArrayBufferView::clampOffsetAndNumElements):
  • html/canvas/ArrayBufferView.idl: Copied from WebCore/html/canvas/WebGLArray.idl.
  • html/canvas/FloatArray.cpp: Copied from WebCore/html/canvas/WebGLFloatArray.cpp. (WebCore::FloatArray::create): (WebCore::FloatArray::FloatArray): (WebCore::FloatArray::slice):
  • html/canvas/FloatArray.h: Copied from WebCore/html/canvas/WebGLFloatArray.h. (WebCore::FloatArray::set): (WebCore::FloatArray::item):
  • html/canvas/FloatArray.idl: Copied from WebCore/html/canvas/WebGLFloatArray.idl.
  • html/canvas/Int16Array.cpp: Copied from WebCore/html/canvas/WebGLShortArray.cpp. (WebCore::Int16Array::create): (WebCore::Int16Array::Int16Array): (WebCore::Int16Array::slice):
  • html/canvas/Int16Array.h: Copied from WebCore/html/canvas/WebGLShortArray.h.
  • html/canvas/Int16Array.idl: Copied from WebCore/html/canvas/WebGLShortArray.idl.
  • html/canvas/Int32Array.cpp: Copied from WebCore/html/canvas/WebGLIntArray.cpp. (WebCore::Int32Array::create): (WebCore::Int32Array::Int32Array): (WebCore::Int32Array::slice):
  • html/canvas/Int32Array.h: Copied from WebCore/html/canvas/WebGLIntArray.h.
  • html/canvas/Int32Array.idl: Copied from WebCore/html/canvas/WebGLIntArray.idl.
  • html/canvas/Int8Array.cpp: Copied from WebCore/html/canvas/WebGLByteArray.cpp. (WebCore::Int8Array::create): (WebCore::Int8Array::Int8Array): (WebCore::Int8Array::slice):
  • html/canvas/Int8Array.h: Copied from WebCore/html/canvas/WebGLByteArray.h.
  • html/canvas/Int8Array.idl: Copied from WebCore/html/canvas/WebGLByteArray.idl.
  • html/canvas/IntegralTypedArrayBase.h: Copied from WebCore/html/canvas/WebGLIntegralTypedArrayBase.h. (WebCore::IntegralTypedArrayBase::set): (WebCore::IntegralTypedArrayBase::item): (WebCore::IntegralTypedArrayBase::IntegralTypedArrayBase):
  • html/canvas/TypedArrayBase.h: Copied from WebCore/html/canvas/WebGLTypedArrayBase.h. (WebCore::TypedArrayBase::set): (WebCore::TypedArrayBase::TypedArrayBase): (WebCore::TypedArrayBase::create):
  • html/canvas/Uint16Array.cpp: Copied from WebCore/html/canvas/WebGLUnsignedShortArray.cpp. (WebCore::Uint16Array::create): (WebCore::Uint16Array::Uint16Array): (WebCore::Uint16Array::slice):
  • html/canvas/Uint16Array.h: Copied from WebCore/html/canvas/WebGLUnsignedShortArray.h.
  • html/canvas/Uint16Array.idl: Copied from WebCore/html/canvas/WebGLUnsignedShortArray.idl.
  • html/canvas/Uint32Array.cpp: Copied from WebCore/html/canvas/WebGLUnsignedIntArray.cpp. (WebCore::Uint32Array::create): (WebCore::Uint32Array::Uint32Array): (WebCore::Uint32Array::slice):
  • html/canvas/Uint32Array.h: Copied from WebCore/html/canvas/WebGLUnsignedIntArray.h.
  • html/canvas/Uint32Array.idl: Copied from WebCore/html/canvas/WebGLUnsignedIntArray.idl.
  • html/canvas/Uint8Array.cpp: Copied from WebCore/html/canvas/WebGLUnsignedByteArray.cpp. (WebCore::Uint8Array::create): (WebCore::Uint8Array::Uint8Array): (WebCore::Uint8Array::slice):
  • html/canvas/Uint8Array.h: Copied from WebCore/html/canvas/WebGLUnsignedByteArray.h.
  • html/canvas/Uint8Array.idl: Copied from WebCore/html/canvas/WebGLUnsignedByteArray.idl.
  • html/canvas/WebGLArray.cpp: Removed.
  • html/canvas/WebGLArray.h: Removed.
  • html/canvas/WebGLArray.idl: Removed.
  • html/canvas/WebGLArrayBuffer.cpp: Removed.
  • html/canvas/WebGLArrayBuffer.h: Removed.
  • html/canvas/WebGLArrayBuffer.idl: Removed.
  • html/canvas/WebGLBuffer.cpp: (WebCore::WebGLBuffer::associateBufferData): (WebCore::WebGLBuffer::associateBufferSubData):
  • html/canvas/WebGLBuffer.h: (WebCore::WebGLBuffer::elementArrayBuffer):
  • html/canvas/WebGLByteArray.cpp: Removed.
  • html/canvas/WebGLByteArray.h: Removed.
  • html/canvas/WebGLByteArray.idl: Removed.
  • html/canvas/WebGLFloatArray.cpp: Removed.
  • html/canvas/WebGLFloatArray.h: Removed.
  • html/canvas/WebGLFloatArray.idl: Removed.
  • html/canvas/WebGLGetInfo.cpp: (WebCore::WebGLGetInfo::WebGLGetInfo): (WebCore::WebGLGetInfo::getWebGLFloatArray): (WebCore::WebGLGetInfo::getWebGLIntArray): (WebCore::WebGLGetInfo::getWebGLUnsignedByteArray):
  • html/canvas/WebGLGetInfo.h:
  • html/canvas/WebGLIntArray.cpp: Removed.
  • html/canvas/WebGLIntArray.h: Removed.
  • html/canvas/WebGLIntArray.idl: Removed.
  • html/canvas/WebGLIntegralTypedArrayBase.h: Removed.
  • html/canvas/WebGLRenderingContext.cpp: (WebCore::WebGLRenderingContext::bufferData): (WebCore::WebGLRenderingContext::bufferSubData): (WebCore::WebGLRenderingContext::getUniform): (WebCore::WebGLRenderingContext::getVertexAttrib): (WebCore::WebGLRenderingContext::readPixels): (WebCore::WebGLRenderingContext::texImage2D): (WebCore::WebGLRenderingContext::texSubImage2D): (WebCore::WebGLRenderingContext::uniform1fv): (WebCore::WebGLRenderingContext::uniform1iv): (WebCore::WebGLRenderingContext::uniform2fv): (WebCore::WebGLRenderingContext::uniform2iv): (WebCore::WebGLRenderingContext::uniform3fv): (WebCore::WebGLRenderingContext::uniform3iv): (WebCore::WebGLRenderingContext::uniform4fv): (WebCore::WebGLRenderingContext::uniform4iv): (WebCore::WebGLRenderingContext::uniformMatrix2fv): (WebCore::WebGLRenderingContext::uniformMatrix3fv): (WebCore::WebGLRenderingContext::uniformMatrix4fv): (WebCore::WebGLRenderingContext::vertexAttrib1fv): (WebCore::WebGLRenderingContext::vertexAttrib2fv): (WebCore::WebGLRenderingContext::vertexAttrib3fv): (WebCore::WebGLRenderingContext::vertexAttrib4fv): (WebCore::WebGLRenderingContext::getWebGLFloatArrayParameter): (WebCore::WebGLRenderingContext::getWebGLIntArrayParameter): (WebCore::WebGLRenderingContext::getWebGLUnsignedByteArrayParameter):
  • html/canvas/WebGLRenderingContext.h:
  • html/canvas/WebGLRenderingContext.idl:
  • html/canvas/WebGLShortArray.cpp: Removed.
  • html/canvas/WebGLShortArray.h: Removed.
  • html/canvas/WebGLShortArray.idl: Removed.
  • html/canvas/WebGLTypedArrayBase.h: Removed.
  • html/canvas/WebGLUnsignedByteArray.cpp: Removed.
  • html/canvas/WebGLUnsignedByteArray.h: Removed.
  • html/canvas/WebGLUnsignedByteArray.idl: Removed.
  • html/canvas/WebGLUnsignedIntArray.cpp: Removed.
  • html/canvas/WebGLUnsignedIntArray.h: Removed.
  • html/canvas/WebGLUnsignedIntArray.idl: Removed.
  • html/canvas/WebGLUnsignedShortArray.cpp: Removed.
  • html/canvas/WebGLUnsignedShortArray.h: Removed.
  • html/canvas/WebGLUnsignedShortArray.idl: Removed.
  • page/DOMWindow.idl:
  • platform/graphics/GraphicsContext3D.h:
  • platform/graphics/mac/GraphicsContext3DMac.cpp: (WebCore::GraphicsContext3D::bufferData): (WebCore::GraphicsContext3D::bufferSubData):
  • platform/graphics/qt/GraphicsContext3DQt.cpp: (WebCore::GraphicsContext3D::bufferData): (WebCore::GraphicsContext3D::bufferSubData):

2010-05-14 Kenneth Russell <​kbr@google.com>

Reviewed by Darin Adler.

Rename WebGLArray types to TypedArray types
​https://bugs.webkit.org/show_bug.cgi?id=39091

Extended functionality of do-webcore-rename script and used it to
rename the WebGLArray types to the TypedArray naming convention.
The only source files which were touched by hand, and which are
being manually reviewed, are:

WebCore/page/DOMWindow.idl
WebCore/bindings/generic/RuntimeEnabledFeatures.h (script's changes undone)
WebKit/WebCore/bindings/js/JSDOMWindowCustom.cpp
WebKit/WebCore/bindings/v8/custom/V8DOMWindowCustom.cpp

These only needed to be touched to update the aliases between the
WebGLArray and TypedArray names introduced in bug 39036. (It was
not feasible to have do-webcore-rename handle this as it would
introduce circular renamings.) These aliases will be removed in
roughly a month once existing WebGL content has been updated.

No new tests; covered under existing WebGL tests. Updated
constructed-objects-prototypes and prototype-inheritance-2 tests.
Ran all layout tests in Safari and all WebGL tests in Chromium.

  • fast/dom/script-tests/constructed-objects-prototypes.js: (constructorPropertiesOnWindow):
  • fast/dom/script-tests/prototype-inheritance-2.js: (constructorNamesForWindow):

2010-05-14 Kenneth Russell <​kbr@google.com>

Reviewed by Darin Adler.

Rename WebGLArray types to TypedArray types
​https://bugs.webkit.org/show_bug.cgi?id=39091

Extended functionality of do-webcore-rename script and used it to
rename the WebGLArray types to the TypedArray naming convention.
The only source files which were touched by hand, and which are
being manually reviewed, are:

WebCore/page/DOMWindow.idl
WebCore/bindings/generic/RuntimeEnabledFeatures.h (script's changes undone)
WebKit/WebCore/bindings/js/JSDOMWindowCustom.cpp
WebKit/WebCore/bindings/v8/custom/V8DOMWindowCustom.cpp

These only needed to be touched to update the aliases between the
WebGLArray and TypedArray names introduced in bug 39036. (It was
not feasible to have do-webcore-rename handle this as it would
introduce circular renamings.) These aliases will be removed in
roughly a month once existing WebGL content has been updated.

No new tests; covered under existing WebGL tests. Updated
constructed-objects-prototypes and prototype-inheritance-2 tests.
Ran all layout tests in Safari and all WebGL tests in Chromium.

  • Scripts/do-webcore-rename: Handle the case where some renames are substrings of others. Support renaming files containing custom JS bindings. If isDOMTypeRename is non-zero, expand the regexp which rewrites the file's contents in order to support custom JS bindings.
3:32 PM Changeset in webkit [59498] by sfalken@apple.com
  • 2 edits in trunk/WebCore

<rdar://problem/7985864> Connection properties dictionary should use standard callbacks for keys, values
​https://bugs.webkit.org/show_bug.cgi?id=39132

Reviewed by Sam Weinig.

  • platform/network/cf/ResourceHandleCFNet.cpp:

(WebCore::createConnectionProperties):

3:27 PM Changeset in webkit [59497] by mrowe@apple.com
  • 1 edit in branches/safari-533-branch/WebKit/win/ChangeLog

Merge r59438.

3:27 PM Changeset in webkit [59496] by mrowe@apple.com
  • 5 edits in branches/safari-533-branch/WebKit/win

Merge r59434.

3:14 PM Changeset in webkit [59495] by eric@webkit.org
  • 3 edits
    2 adds in trunk

2010-05-14 Abhishek Arya <​inferno@chromium.org>

Reviewed by David Hyatt.

Tests that large colspan in a fixed table layout does not result in crash.
​https://bugs.webkit.org/show_bug.cgi?id=38261

  • fast/table/fixed-table-layout-large-colspan-crash-expected.txt: Added.
  • fast/table/fixed-table-layout-large-colspan-crash.html: Added.

2010-05-14 Abhishek Arya <​inferno@chromium.org>

Reviewed by David Hyatt.

Move the m_width(Length) and m_columns(RenderTable::ColumnStruct)
vector out-of-bounds check out of the ASSERT into the main code.
​https://bugs.webkit.org/show_bug.cgi?id=38261

Test: fast/table/fixed-table-layout-large-colspan-crash.html

  • rendering/FixedTableLayout.cpp: (WebCore::FixedTableLayout::calcWidthArray):
3:08 PM Changeset in webkit [59494] by mrowe@apple.com
  • 5 edits in tags/Safari-533.9.1

Versioning.

3:04 PM Changeset in webkit [59493] by mrowe@apple.com
  • 2 edits in tags/Safari-533.9.1/WebCore

Merge r59481.

2:54 PM Changeset in webkit [59492] by mrowe@apple.com
  • 1 copy in tags/Safari-533.9.1

New tag.

2:54 PM Changeset in webkit [59491] by mrowe@apple.com
  • 2 edits in branches/safari-533-branch/WebCore

Merge r59481.

2:14 PM Changeset in webkit [59490] by eric@webkit.org
  • 28 edits in trunk

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

Unreviewed, rolling out r59489.
​http://trac.webkit.org/changeset/59489
​https://bugs.webkit.org/show_bug.cgi?id=37815

abarth broke the known universe and then some.

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

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

Unreviewed, rolling out r59489.
​http://trac.webkit.org/changeset/59489
​https://bugs.webkit.org/show_bug.cgi?id=37815

abarth broke the known universe and then some.

  • Android.jscbindings.mk:
  • GNUmakefile.am:
  • WebCore.gypi:
  • WebCore.pro:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • bindings/js/JSBindingsAllInOne.cpp:
  • bindings/js/JSCallbackData.cpp: (WebCore::JSCallbackData::invokeCallback):
  • bindings/js/JSEventListener.cpp: (WebCore::JSEventListener::handleEvent):
  • bindings/js/JSInjectedScriptHostCustom.cpp: (WebCore::InjectedScriptHost::createInjectedScript):
  • bindings/js/ScheduledAction.cpp: (WebCore::ScheduledAction::executeFunctionInContext): (WebCore::ScheduledAction::execute):
  • bindings/js/ScheduledAction.h:
  • bindings/js/ScriptController.cpp: (WebCore::ScriptController::evaluateInWorld):
  • bindings/js/ScriptController.h:
  • bindings/js/ScriptFunctionCall.cpp: (WebCore::ScriptFunctionCall::call):
  • bindings/objc/ObjCEventListener.mm: (WebCore::ObjCEventListener::handleEvent):
  • bindings/objc/WebScriptObject.mm: (-[WebScriptObject callWebScriptMethod:withArguments:]): (-[WebScriptObject evaluateWebScript:]):
  • bindings/scripts/CodeGeneratorObjC.pm:
  • bindings/scripts/test/ObjC/DOMTestCallback.mm: (-[DOMTestCallback callbackWithClass1Param:]): (-[DOMTestCallback callbackWithClass2Param:strArg:]): (-[DOMTestCallback callbackWithNonBoolReturnType:]): (-[DOMTestCallback customCallback:class6Param:]):
  • bindings/scripts/test/ObjC/DOMTestInterface.mm:
  • bindings/scripts/test/ObjC/DOMTestObj.mm: (-[DOMTestObj readOnlyIntAttr]): (-[DOMTestObj readOnlyStringAttr]): (-[DOMTestObj readOnlyTestObjAttr]): (-[DOMTestObj intAttr]): (-[DOMTestObj setIntAttr:]): (-[DOMTestObj longLongAttr]): (-[DOMTestObj setLongLongAttr:]): (-[DOMTestObj unsignedLongLongAttr]): (-[DOMTestObj setUnsignedLongLongAttr:]): (-[DOMTestObj stringAttr]): (-[DOMTestObj setStringAttr:]): (-[DOMTestObj testObjAttr]): (-[DOMTestObj setTestObjAttr:]): (-[DOMTestObj attrWithException]): (-[DOMTestObj setAttrWithException:]): (-[DOMTestObj attrWithSetterException]): (-[DOMTestObj setAttrWithSetterException:]): (-[DOMTestObj attrWithGetterException]): (-[DOMTestObj setAttrWithGetterException:]): (-[DOMTestObj customAttr]): (-[DOMTestObj setCustomAttr:]): (-[DOMTestObj scriptStringAttr]): (-[DOMTestObj voidMethod]): (-[DOMTestObj voidMethodWithArgs:strArg:objArg:]): (-[DOMTestObj intMethod]): (-[DOMTestObj intMethodWithArgs:strArg:objArg:]): (-[DOMTestObj objMethod]): (-[DOMTestObj objMethodWithArgs:strArg:objArg:]): (-[DOMTestObj serializedValue:]): (-[DOMTestObj methodWithException]): (-[DOMTestObj customMethod]): (-[DOMTestObj customMethodWithArgs:strArg:objArg:]): (-[DOMTestObj customArgsAndException:]): (-[DOMTestObj addEventListener:listener:useCapture:]): (-[DOMTestObj removeEventListener:listener:useCapture:]): (-[DOMTestObj withDynamicFrame]): (-[DOMTestObj withDynamicFrameAndArg:]): (-[DOMTestObj withDynamicFrameAndOptionalArg:optionalArg:]): (-[DOMTestObj withDynamicFrameAndUserGesture:]): (-[DOMTestObj withDynamicFrameAndUserGestureASAD:optionalArg:]): (-[DOMTestObj withScriptStateVoid]): (-[DOMTestObj withScriptStateObj]): (-[DOMTestObj withScriptStateVoidException]): (-[DOMTestObj withScriptStateObjException]): (-[DOMTestObj methodWithOptionalArg:]): (-[DOMTestObj methodWithNonOptionalArgAndOptionalArg:opt:]): (-[DOMTestObj methodWithNonOptionalArgAndTwoOptionalArgs:opt1:opt2:]):
  • bindings/v8/ScriptController.cpp:
  • bindings/v8/ScriptController.h:
  • html/HTMLFrameElementBase.cpp: (WebCore::HTMLFrameElementBase::isURLAllowed):
1:52 PM Changeset in webkit [59489] by abarth@webkit.org
  • 28 edits in trunk

2010-05-14 Justin Schuh <​jschuh@chromium.org>

Reviewed by Adam Barth.

Moving frame.src checks out of the bindings
​https://bugs.webkit.org/show_bug.cgi?id=37815

Moved JavaScript frame.src checks out of bindings and into
HTMLFrameElementBase. Added main thread state stack to JavaScriptCore
so ExecState is available inside core DOM. Updated affected bindings
(except for GObject, which will need to be updated to avoid origin
failures inside native code).

  • Android.jscbindings.mk:
  • GNUmakefile.am:
  • WebCore.gypi:
  • WebCore.pro:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • bindings/js/JSBindingsAllInOne.cpp:
  • bindings/js/JSCallbackData.cpp: (WebCore::JSCallbackData::invokeCallback):
  • bindings/js/JSEventListener.cpp: (WebCore::JSEventListener::handleEvent):
  • bindings/js/JSInjectedScriptHostCustom.cpp: (WebCore::InjectedScriptHost::createInjectedScript):
  • bindings/js/ScheduledAction.cpp: (WebCore::ScheduledAction::executeFunctionInContext): (WebCore::ScheduledAction::execute):
  • bindings/js/ScheduledAction.h:
  • bindings/js/ScriptController.cpp: (WebCore::ScriptController::evaluateInWorld): (WebCore::ScriptController::canAccessFromCurrentOrigin):
  • bindings/js/ScriptController.h:
  • bindings/js/ScriptFunctionCall.cpp: (WebCore::ScriptFunctionCall::call):
  • bindings/objc/ObjCEventListener.mm: (WebCore::ObjCEventListener::handleEvent):
  • bindings/objc/WebScriptObject.mm: (-[WebScriptObject callWebScriptMethod:withArguments:]): (-[WebScriptObject evaluateWebScript:]):
  • bindings/scripts/CodeGeneratorObjC.pm:
  • bindings/scripts/test/ObjC/DOMTestCallback.mm: (-[DOMTestCallback callbackWithClass1Param:]): (-[DOMTestCallback callbackWithClass2Param:strArg:]): (-[DOMTestCallback callbackWithNonBoolReturnType:]): (-[DOMTestCallback customCallback:class6Param:]):
  • bindings/scripts/test/ObjC/DOMTestInterface.mm:
  • bindings/scripts/test/ObjC/DOMTestObj.mm: (-[DOMTestObj readOnlyIntAttr]): (-[DOMTestObj readOnlyStringAttr]): (-[DOMTestObj readOnlyTestObjAttr]): (-[DOMTestObj intAttr]): (-[DOMTestObj setIntAttr:]): (-[DOMTestObj longLongAttr]): (-[DOMTestObj setLongLongAttr:]): (-[DOMTestObj unsignedLongLongAttr]): (-[DOMTestObj setUnsignedLongLongAttr:]): (-[DOMTestObj stringAttr]): (-[DOMTestObj setStringAttr:]): (-[DOMTestObj testObjAttr]): (-[DOMTestObj setTestObjAttr:]): (-[DOMTestObj attrWithException]): (-[DOMTestObj setAttrWithException:]): (-[DOMTestObj attrWithSetterException]): (-[DOMTestObj setAttrWithSetterException:]): (-[DOMTestObj attrWithGetterException]): (-[DOMTestObj setAttrWithGetterException:]): (-[DOMTestObj customAttr]): (-[DOMTestObj setCustomAttr:]): (-[DOMTestObj scriptStringAttr]): (-[DOMTestObj voidMethod]): (-[DOMTestObj voidMethodWithArgs:strArg:objArg:]): (-[DOMTestObj intMethod]): (-[DOMTestObj intMethodWithArgs:strArg:objArg:]): (-[DOMTestObj objMethod]): (-[DOMTestObj objMethodWithArgs:strArg:objArg:]): (-[DOMTestObj serializedValue:]): (-[DOMTestObj methodWithException]): (-[DOMTestObj customMethod]): (-[DOMTestObj customMethodWithArgs:strArg:objArg:]): (-[DOMTestObj customArgsAndException:]): (-[DOMTestObj addEventListener:listener:useCapture:]): (-[DOMTestObj removeEventListener:listener:useCapture:]): (-[DOMTestObj withDynamicFrame]): (-[DOMTestObj withDynamicFrameAndArg:]): (-[DOMTestObj withDynamicFrameAndOptionalArg:optionalArg:]): (-[DOMTestObj withDynamicFrameAndUserGesture:]): (-[DOMTestObj withDynamicFrameAndUserGestureASAD:optionalArg:]): (-[DOMTestObj withScriptStateVoid]): (-[DOMTestObj withScriptStateObj]): (-[DOMTestObj withScriptStateVoidException]): (-[DOMTestObj withScriptStateObjException]): (-[DOMTestObj methodWithOptionalArg:]): (-[DOMTestObj methodWithNonOptionalArgAndOptionalArg:opt:]): (-[DOMTestObj methodWithNonOptionalArgAndTwoOptionalArgs:opt1:opt2:]):
  • bindings/v8/ScriptController.cpp: (WebCore::ScriptController::canAccessFromCurrentOrigin):
  • bindings/v8/ScriptController.h:
  • html/HTMLFrameElementBase.cpp: (WebCore::HTMLFrameElementBase::isURLAllowed):

2010-05-14 Justin Schuh <​jschuh@chromium.org>

Reviewed by Adam Barth.

Moving frame.src checks out of the bindings
​https://bugs.webkit.org/show_bug.cgi?id=37815

  • http/tests/security/xss-DENIED-iframe-src-alias-expected.txt:
  • http/tests/security/xss-DENIED-iframe-src-alias.html:
1:49 PM Changeset in webkit [59488] by eric@webkit.org
  • 2 edits in trunk/WebKitSite

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

Reviewed by Maciej Stachowiak.

Update tools.html to mention that Tiger users require a newer Python
​https://bugs.webkit.org/show_bug.cgi?id=38822

Also updated the instructions that only Tiger users need
to install a subversion client. Newer OSes already have
subversion included in the install.

  • building/tools.html:
1:36 PM Changeset in webkit [59487] by dimich@chromium.org
  • 2 edits in trunk/WebCore

Adding resized image caching for Skia.
​https://bugs.webkit.org/show_bug.cgi?id=39085

Patch by Sergey Ulanov <​sergeyu@chromium.org> on 2010-05-14
Reviewed by Dmitry Titov and Brett Wilson.

Tests: HTML5 video on YouTube.

  • platform/graphics/skia/ImageSkia.cpp:

(WebCore::Image::drawPattern): Use bitmap->resizedBitmap() to cache resized image when possible.

1:32 PM Changeset in webkit [59486] by ap@apple.com
  • 3 edits
    2 adds in trunk

Reviewed by Brady Eidson.

​https://bugs.webkit.org/show_bug.cgi?id=39008
<rdar://problem/7976142> REGRESSION (r58950): Webkit crashes on clicking back button when in Hotmail

Test: fast/parser/tokenizer-close-during-document-write.html

  • dom/Document.cpp: (WebCore::Document::write): Check that the tokenizer is still around after calling write(). It can become null (as it happens in regression test), and I don't see any guarantee that it would never be replaced with a different one (but I can't make a test for that).
1:31 PM Changeset in webkit [59485] by jianli@chromium.org
  • 2 edits in trunk/WebCore

Fix the code indentation inside a namespace to be consistent with the
style guide.

Rubber-stamped by Dmitry Titov.

  • workers/GenericWorkerTask.h:
1:24 PM Changeset in webkit [59484] by jianli@chromium.org
  • 2 edits in trunk/WebCore

Remove the ENABLE(WORKERS) guard from GenericWorkerTask.h and rename
all occurrences of GenericWorkerTask* to GenericTask* in this file.
​https://bugs.webkit.org/show_bug.cgi?id=39129

Reviewed by Dmitry Titov.

I will have another patch to rename the file and move it to the right
place.

  • workers/GenericWorkerTask.h:

(WebCore::):
(WebCore::GenericTask1::create):
(WebCore::GenericTask1::GenericTask1):
(WebCore::GenericTask2::create):
(WebCore::GenericTask2::GenericTask2):
(WebCore::GenericTask3::create):
(WebCore::GenericTask3::GenericTask3):
(WebCore::GenericTask4::create):
(WebCore::GenericTask4::GenericTask4):
(WebCore::GenericTask5::create):
(WebCore::GenericTask5::GenericTask5):
(WebCore::GenericTask6::create):
(WebCore::GenericTask6::GenericTask6):
(WebCore::GenericTask7::create):
(WebCore::GenericTask7::GenericTask7):
(WebCore::GenericTask8::create):
(WebCore::GenericTask8::GenericTask8):
(WebCore::createCallbackTask):

1:16 PM Changeset in webkit [59483] by eric@webkit.org
  • 3 edits in trunk/WebCore

2010-05-14 Evan Martin <​evan@chromium.org>

Reviewed by David Levin.

[chromium] don't call fontconfig twice in complex text path
​https://bugs.webkit.org/show_bug.cgi?id=38701

Previously getFontDataForCharacters would call fontconfig once to
resolve a font name, then pass that name to getCachedFontPlatformData
to load the font. This would be two trips through fontconfig (including
IPCs). With this change, we completely load the font in the first pass.

This patch is actually the same code as in r58341, which was reverted
because of a performance problem; a performance improvement has now
been committed, and with that change in place this change improves
the Chromium intl2 page cycler performance by another 5%.

See also ​https://bugs.webkit.org/show_bug.cgi?id=37904 .

Test: fast/text/international/bold-bengali.html
(Was checked in as part of the first attempt at this change, continues
to pass after this change.)

  • platform/chromium/ChromiumBridge.h:
  • platform/graphics/chromium/FontCacheLinux.cpp: (WebCore::FontCache::getFontDataForCharacters):
12:49 PM Changeset in webkit [59482] by eric@webkit.org
  • 2 edits in trunk/WebCore

2010-05-14 Young Han Lee <​joybro@company100.net>

Reviewed by Darin Adler.

Don't include RetainPtr.h unless PLATFORM(CF) is set.
The RetainPtr is only used for CF platform in this class.
​https://bugs.webkit.org/show_bug.cgi?id=39043

No new tests, there is no new functionality.

  • platform/network/curl/ResourceHandleManager.cpp:
12:21 PM Changeset in webkit [59481] by bweinstein@apple.com
  • 2 edits in trunk/WebCore

Build fix, we need to copy over the files in $(WebKitOutputDir)/include/JavaScriptCore/private before
WebCore builds. This is because we are building a couple WTF files in WebCore and JavaScriptCore.

Reviewed by Adam Roben.

  • WebCore.vcproj/WebCore.make:
12:18 PM Changeset in webkit [59480] by eric@webkit.org
  • 3 edits in trunk/WebKitTools

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

Reviewed by Adam Barth.

webkit-patch rollout throws exception if bug is already open
​https://bugs.webkit.org/show_bug.cgi?id=38803

This was caused by someone incorrectly wrapping the code. :p
I'm going to have to start demanding unit tests for wrapping changes...

I also fixed the code to be able to reopen bugs which were never confirmed.
Before it regressed, the code would just log in that case. Now it actually
will reopen the bug, but there is a FIXME about how the logic is a bit backwards.

  • Scripts/webkitpy/common/net/bugzilla.py:
  • Scripts/webkitpy/common/net/bugzilla_unittest.py:
    • Yay testing!
11:59 AM Changeset in webkit [59479] by eric@webkit.org
  • 3 edits in trunk/LayoutTests

2010-05-14 Zhenyao Mo <​zmo@google.com>

Reviewed by Eric Seidel.

Temporarily remove a few test cases that failed on leopard bot.
​https://bugs.webkit.org/show_bug.cgi?id=33805

  • fast/canvas/webgl/texture-npot-expected.txt: Removed failed test cases.
  • fast/canvas/webgl/texture-npot.html: Ditto.
11:49 AM Changeset in webkit [59478] by yurys@chromium.org
  • 2 edits in trunk/LayoutTests

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

Unreviewed. Chromium: update test expectations.

  • platform/chromium-linux/fast/loader/recursive-before-unload-crash-expected.txt:
9:55 AM Changeset in webkit [59477] by hamaji@chromium.org
  • 2 edits in trunk/WebCore

2010-05-14 Shinichiro Hamaji <​hamaji@chromium.org>

Unreviewed. Attempt to fix chromium's memory bot error.

More :visited link optimizations to reduce memory usage
​https://bugs.webkit.org/show_bug.cgi?id=39084

The original change (r59386) adds check for m_element in initElement.
As don't initialize m_element in the constructor of CSSStyleSelector,
valgrind complains.

  • css/CSSStyleSelector.cpp: (WebCore::CSSStyleSelector::CSSStyleSelector):
9:33 AM Changeset in webkit [59476] by jorlow@chromium.org
  • 2 edits
    2 deletes in trunk/LayoutTests

2010-05-14 Jeremy Orlow <​jorlow@chromium.org>

Reviewed by Dimitri Glazkov.

Instead of rebaslining prototype-inheritance for Chromium, add indexedDB to its exceptions
​https://bugs.webkit.org/show_bug.cgi?id=39122

The test has a way to name properties that shouldn't be tested (used when they're not enabled
by most ports). Use this feature rather than creating custom baselines.

  • fast/dom/script-tests/prototype-inheritance.js:
  • platform/chromium-mac/fast/dom/prototype-inheritance-expected.txt: Removed.
  • platform/chromium-win/fast/dom/prototype-inheritance-expected.txt: Removed.
9:03 AM Changeset in webkit [59475] by jorlow@chromium.org
  • 1 edit
    2 adds in trunk/LayoutTests

2010-05-14 Jeremy Orlow <​jorlow@chromium.org>

Reviewed by Nate Chapin.

[Chromium] Rebaseline prototype-inheritance
​https://bugs.webkit.org/show_bug.cgi?id=39116

IndexedDB is now turned on for Chromium's test shell. I'm not sure
if this is the correct behavior or not (can someone confirm which it is?)
but we should get this test running again so we can catch real regressions.
And, before taking this out from behind a flag, we should be sure to fix it.

  • platform/chromium-mac/fast/dom/prototype-inheritance-expected.txt: Added.
  • platform/chromium-win/fast/dom/prototype-inheritance-expected.txt: Added.
8:59 AM QtWebKitBugs edited by Simon Hausmann
(diff)
8:24 AM Changeset in webkit [59474] by Simon Hausmann
  • 5 edits
    3 copies
    2 deletes in trunk

.: [Qt] Rename QtLauncher to QtTestBrowser
​https://bugs.webkit.org/show_bug.cgi?id=37665

Rubber-stamped by Antti Koivisto.

  • WebKit.pro:

WebKitTools: [Qt] Rename QtLauncher to QtTestBrowser

Rubber-stamped by Antti Koivisto.

  • QtTestBrowser: Copied from WebKitTools/QtLauncher.
  • QtTestBrowser/QtLauncher.pro: Removed.
  • QtTestBrowser/QtLauncher.qrc: Removed.
  • QtTestBrowser/QtTestBrowser.pro: Copied from WebKitTools/QtLauncher/QtLauncher.pro.
  • QtTestBrowser/QtTestBrowser.qrc: Copied from WebKitTools/QtLauncher/QtLauncher.qrc.
  • Scripts/run-launcher:
  • Scripts/webkitdirs.pm:
8:12 AM QtWebKitTriageRoster edited by jesus@webkit.org
(diff)
8:11 AM QtWebKitBugs edited by jesus@webkit.org
(diff)
7:47 AM Changeset in webkit [59473] by eric@webkit.org
  • 4 edits in trunk/WebCore

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

Reviewed by Darin Adler.

CSSParser::parseColor() shouldn't alter 'color' unless passed a valid color string.
​https://bugs.webkit.org/show_bug.cgi?id=39031

  • css/CSSParser.cpp: (WebCore::CSSParser::parseColor):
  • editing/ApplyStyleCommand.cpp: (WebCore::StyleChange::extractTextStyles): Don't depend on old behavior.
  • html/canvas/CanvasRenderingContext2D.cpp: (WebCore::CanvasRenderingContext2D::setShadow): Remove dead code.
7:16 AM Changeset in webkit [59472] by eric@webkit.org
  • 2 edits in trunk/WebCore

2010-05-14 Jens Alfke <​snej@chromium.org>

Reviewed by John Sullivan.

Don't clear yellow autofill highlight from text fields until the user edits them.
​https://bugs.webkit.org/show_bug.cgi?id=39032
No new tests (Couldn't find any autofill tests. The flag doesn't seem to be exposed to JS.)

  • html/HTMLInputElement.cpp: (WebCore::HTMLInputElement::handleFocusEvent): Don't clear the autofilled flag. (WebCore::HTMLInputElement::setValueFromRenderer): Clear the autofilled flag.
5:57 AM Changeset in webkit [59471] by Csaba Osztrogonác
  • 2 edits in trunk/LayoutTests

Unreviewed, unskip passing tests.

[Qt] animations/dynamic-stylesheet-loading.html fails with accelerated compositing
​https://bugs.webkit.org/show_bug.cgi?id=37381

  • platform/qt/Skipped:
    • animations/simultaneous-start-left.html unskipped.
    • animations/dynamic-stylesheet-loading.html unskipped.
5:52 AM Changeset in webkit [59470] by eric@webkit.org
  • 6 edits
    2 adds in trunk

2010-05-14 Zhenyao Mo <​zmo@google.com>

Reviewed by Dimitri Glazkov.

Must implement OpenGL ES 2.0 semantics for NPOT textures
​https://bugs.webkit.org/show_bug.cgi?id=33805

  • fast/canvas/webgl/texture-npot-expected.txt: Added.
  • fast/canvas/webgl/texture-npot.html: Added.

2010-05-14 Zhenyao Mo <​zmo@google.com>

Reviewed by Dimitri Glazkov.

Must implement OpenGL ES 2.0 semantics for NPOT textures
​https://bugs.webkit.org/show_bug.cgi?id=33805

Test: fast/canvas/webgl/texture-npot.html

  • html/canvas/WebGLRenderingContext.cpp: (WebCore::WebGLRenderingContext::WebGLRenderingContext): Create black 1x1 textures. (WebCore::WebGLRenderingContext::bindTexture): Set texture dimensionality. (WebCore::WebGLRenderingContext::copyTexImage2D): Error check, set texture size. (WebCore::WebGLRenderingContext::drawArrays): Bind to black textures if needed. (WebCore::WebGLRenderingContext::drawElements): Bind to black textures if needed. (WebCore::WebGLRenderingContext::generateMipmap): Error check. (WebCore::WebGLRenderingContext::texImage2DBase): Error check, set texture size. (WebCore::WebGLRenderingContext::texParameterf): Set texture parameter. (WebCore::WebGLRenderingContext::texParameteri): Ditto. (WebCore::WebGLRenderingContext::handleNPOTTextures): Bind to black textures if needed before draw, and bind back to original textures after draw.
  • html/canvas/WebGLRenderingContext.h: Add black 1x1 textures.
  • html/canvas/WebGLTexture.cpp: Add support to track NPOT textures. (WebCore::WebGLTexture::createBlackTexture1x1): Create a black 1x1 texture. (WebCore::WebGLTexture::WebGLTexture): Initialize parameters and states. (WebCore::WebGLTexture::setDimensionality): Set dimensionality. (WebCore::WebGLTexture::setParameteri): Set parameter. (WebCore::WebGLTexture::setParameterf): Ditto. (WebCore::WebGLTexture::setSize): Set level 0 texture size. (WebCore::WebGLTexture::isNPOT): Whether the texture is NPOT. (WebCore::WebGLTexture::updateNPOTStates): Calculate NPOT and NeedToUseBlackTexture states.
  • html/canvas/WebGLTexture.h: Add support to track NPOT textures. (WebCore::WebGLTexture::isNPOT): (WebCore::WebGLTexture::needToUseBlackTexture):
5:39 AM Changeset in webkit [59469] by eric@webkit.org
  • 2 edits in trunk/JavaScriptCore

2010-05-14 Chao-ying Fu <​fu@mips.com>

Reviewed by Oliver Hunt.

String Indexing Failure on JSVALUE32 targets
​https://bugs.webkit.org/show_bug.cgi?id=39034

Remove zero-extend/shift-right code on regT1, because we already have
it in emit_op_get_by_val().

  • jit/JITPropertyAccess.cpp: (JSC::JIT::stringGetByValStubGenerator):
5:36 AM Changeset in webkit [59468] by pfeldman@chromium.org
  • 17 edits
    3 adds in trunk/WebCore

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

Reviewed by Pavel Feldman.

Web Inspector: add help on keyboard shortcuts (take 2).

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

  • English.lproj/localizedStrings.js:
  • WebCore.gypi:
  • WebCore.vcproj/WebCore.vcproj:
  • inspector/front-end/CallStackSidebarPane.js: (WebInspector.CallStackSidebarPane): (WebInspector.CallStackSidebarPane.prototype._placardSelected): (WebInspector.CallStackSidebarPane.prototype.registerShortcuts):
  • inspector/front-end/ConsoleView.js: (WebInspector.ConsoleView): (WebInspector.ConsoleView.prototype._registerShortcuts):
  • inspector/front-end/ElementsPanel.js: (WebInspector.ElementsPanel): (WebInspector.ElementsPanel.prototype._registerShortcuts):
  • inspector/front-end/ElementsTreeOutline.js: (WebInspector.ElementsTreeOutline.prototype._keyDown):
  • inspector/front-end/HelpScreen.js: Added.
  • inspector/front-end/KeyboardShortcut.js:
  • inspector/front-end/ScriptsPanel.js:
5:32 AM QtWebKitTriageRoster edited by Csaba Osztrogonác
(diff)
5:28 AM Changeset in webkit [59467] by eric@webkit.org
  • 2 edits in trunk/WebKit/chromium

2010-05-14 Jochen Eisinger <​jochen@chromium.org>

Reviewed by Eric Seidel.

Make ImageDiff depend on webkit (again).
​https://bugs.webkit.org/show_bug.cgi?id=39000

  • WebKit.gyp:
5:16 AM Changeset in webkit [59466] by eric@webkit.org
  • 4 edits in trunk/WebKit/chromium

2010-05-14 James Simonsen <​simonjam+webkit@google.com>

Reviewed by Darin Fisher.

[chromium] Expose volume through WebMediaPlayerClient
​https://bugs.webkit.org/show_bug.cgi?id=38947

  • public/WebMediaPlayerClient.h:
  • src/WebMediaPlayerClientImpl.cpp: (WebKit::WebMediaPlayerClientImpl::volume):
  • src/WebMediaPlayerClientImpl.h:
5:03 AM QtWebKitJournal edited by Simon Hausmann
(diff)
4:55 AM Changeset in webkit [59465] by eric@webkit.org
  • 10 edits in trunk

2010-05-14 Tony Gentilcore <​tonyg@chromium.org>

Reviewed by Darin Fisher.

[chromium] Update chromium port to send/receive cached metadata
​https://bugs.webkit.org/show_bug.cgi?id=38665

No new tests because no new functionality.

  • platform/chromium/ChromiumBridge.h:
  • platform/network/chromium/ResourceResponse.h: (WebCore::ResourceResponse::ResourceResponse): (WebCore::ResourceResponse::responseTime): (WebCore::ResourceResponse::setResponseTime):

2010-05-14 Tony Gentilcore <​tonyg@chromium.org>

Reviewed by Darin Fisher.

[chromium] Update chromium port to send/receive cached metadata
​https://bugs.webkit.org/show_bug.cgi?id=38665

  • public/WebKitClient.h: (WebKit::WebKitClient::cacheMetadata):
  • public/WebURLLoaderClient.h: (WebKit::WebURLLoaderClient::didReceiveCachedMetadata):
  • public/WebURLResponse.h:
  • src/ChromiumBridge.cpp: (WebCore::ChromiumBridge::cacheMetadata):
  • src/ResourceHandle.cpp: (WebCore::ResourceHandleInternal::didReceiveCachedMetadata): (WebCore::ResourceHandle::cacheMetadata):
  • src/WebURLResponse.cpp: (WebKit::WebURLResponse::responseTime): (WebKit::WebURLResponse::setResponseTime):
4:32 AM Changeset in webkit [59464] by eric@webkit.org
  • 3 edits in trunk/WebKitTools

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

Reviewed by Eric Seidel.

Stop CCing webkit-bot-watchers
​https://bugs.webkit.org/show_bug.cgi?id=39020

webkit-bot-watchers is somewhat of a failed experiment. No one
subscribed to the list (not even me). Removing it from the code
because wms says it bounces email sometimes.

  • Scripts/webkitpy/tool/commands/queues.py:
  • Scripts/webkitpy/tool/commands/sheriffbot_unittest.py:
4:24 AM Changeset in webkit [59463] by jocelyn.turcotte@nokia.com
  • 2 edits in trunk/JavaScriptCore

Fix a VS2010 assert in std::copy
​https://bugs.webkit.org/show_bug.cgi?id=38630

Reviewed by Alexey Proskuryakov.

The assert complains that the output iterator is null.

  • wtf/Vector.h:

(WTF::::operator):

4:21 AM Changeset in webkit [59462] by eric@webkit.org
  • 5 edits in trunk/WebCore

2010-05-14 Chris Evans <​cevans@chromium.org>

Reviewed by Adam Barth.

Chromium-specific fix to prevent zombie infobars.

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

  • bindings/v8/ScriptController.cpp (WebCore::ScriptController::processingUserGesture): Require a more definite "no" signal.
  • loader/RedirectScheduler.cpp: (WebCore::ScheduledFormSubmission::fire): Use a very strong yes / no signal; no need for "possible" here.
  • dom/UserGestureIndicator.h:
  • dom/UserGestureIndicator.cpp: Adapt to tri-state: definitely yes; definitely no; not sure.
3:55 AM Changeset in webkit [59461] by eric@webkit.org
  • 3 edits in trunk/WebKitTools

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

Reviewed by Adam Barth.

REGRESSION: webkit-patch commit-message throws exception
​https://bugs.webkit.org/show_bug.cgi?id=38997

  • Scripts/webkitpy/tool/commands/upload.py:
    • Fix to respect and pass the --squash and --git-commit options.
  • Scripts/webkitpy/tool/commands/upload_unittest.py:
    • Fix the unit test to use the central MockCheckout instead of its own custom Mock which didn't require enough parameters.
2:58 AM Changeset in webkit [59460] by Simon Hausmann
  • 3 edits in trunk/WebCore

[Qt] JavaScript unable to invoke methods declared in QML
​https://bugs.webkit.org/show_bug.cgi?id=38949

Patch by Aaron Kennedy <​tffeeb@gmail.com> on 2010-05-14
Reviewed by Simon Hausmann.

JavaScript code executed by webkit cannot call into QML declared
methods, as it does not check for dynamic meta objects.

  • bridge/qt/qt_instance.cpp:

(JSC::Bindings::QtInstance::stringValue): Use QMetaObject::metacall.

  • bridge/qt/qt_runtime.cpp:

(JSC::Bindings::QtRuntimeMetaMethod::call): Ditto.

2:50 AM Changeset in webkit [59459] by eric@webkit.org
  • 2 edits in trunk/WebCore

2010-05-14 Ben Murdoch <​benm@google.com>

Reviewed by Darin Fisher.

Touch events are disabled by default when using V8.
​https://bugs.webkit.org/show_bug.cgi?id=38990

Touch events are protected by a compile time guard, so it's safe to
assume that the runtime flag should be enabled by default when this
guard is present.

No new tests, build fix for Android.

  • bindings/generic/RuntimeEnabledFeatures.cpp: enable touch events by default.
2:29 AM Changeset in webkit [59458] by mrowe@apple.com
  • 10 edits in branches/safari-533-branch

<rdar://problem/7677994> Disable WebGL

JavaScriptCore:

  • Configurations/FeatureDefines.xcconfig:

WebCore:

  • Configurations/FeatureDefines.xcconfig:

WebKit/mac:

  • Configurations/FeatureDefines.xcconfig:

WebKit2:

  • Configurations/FeatureDefines.xcconfig:

WebKitTools:

  • Scripts/build-webkit:
2:23 AM Changeset in webkit [59457] by jorlow@chromium.org
  • 1 edit
    2 adds in trunk/LayoutTests

2010-05-14 Hans Wennborg <​hans@chromium.org>

Reviewed by Jeremy Orlow.

[Chromium] Rebaseline storage/domstorage/quota.html
​https://bugs.webkit.org/show_bug.cgi?id=38996

Chromium enforces quota on session storage and thus expects to "fail"
the first part of this test.

  • platform/chromium-mac/storage/domstorage/quota-expected.txt: Added.
  • platform/chromium-win/storage/domstorage/quota-expected.txt: Added.
2:14 AM Changeset in webkit [59456] by Simon Hausmann
  • 15 edits in trunk/WebKit/qt

[Qt] Merge overhaul of the QtWebKit API documentation

Patch by Kent Hansen <​kent.hansen@nokia.com>, Jocelyn Turcotte <​jocelyn.turcotte@nokia.com>, Tor Arne Vestbø <​tor.arne.vestbo@nokia.com>, Henry Haverinen <Henry Haverinen>, Jedrzej Nowacki <​jedrzej.nowacki@nokia.com>, Andreas Kling <​andreas.kling@nokia.com> on 2010-05-14
Reviewed by Simon Hausmann.

Numerous improvements in wording, qdoc warning fixes and
clarifications, done in a team work effort.

No functional changes.

  • Api/qwebdatabase.cpp:
  • Api/qwebelement.cpp:
  • Api/qwebframe.cpp:
  • Api/qwebhistoryinterface.cpp:
  • Api/qwebinspector.cpp:
  • Api/qwebkitversion.cpp:
  • Api/qwebpage.cpp:
  • Api/qwebpluginfactory.cpp:
  • Api/qwebsecurityorigin.cpp:
  • Api/qwebsettings.cpp:
  • Api/qwebsettings.h:
  • docs/qtwebkit.qdoc:
  • docs/webkitsnippets/qtwebkit_qwebinspector_snippet.cpp:

(wrapInFunction):

  • docs/webkitsnippets/webelement/main.cpp:

(findButtonAndClick):
(autocomplete1):
(autocomplete2):
(main):

2:04 AM Changeset in webkit [59455] by Simon Hausmann
  • 2 edits in trunk/WebKit/qt

Documentation: Fix overview grouping.

Patch by Martin Smith <​msmith@trolltech.com> on 2010-05-14
Reviewed by Simon Hausmann.

  • docs/qtwebkit.qdoc:
1:49 AM Changeset in webkit [59454] by yurys@chromium.org
  • 7 edits
    2 adds in trunk/LayoutTests

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

Unreviewed. Chromium: update test expectations.

  • platform/chromium-linux/editing/deleting/5408255-expected.checksum:
  • platform/chromium-linux/fast/loader/recursive-before-unload-crash-expected.txt: Added.
  • platform/chromium-mac/editing/deleting/5408255-expected.checksum: Added.
  • platform/chromium-win/editing/deleting/5408255-expected.checksum:
  • platform/chromium-win/editing/deleting/5408255-expected.txt:
  • platform/chromium-win/editing/deleting/delete-select-all-001-expected.txt:
  • platform/chromium-win/editing/selection/4895428-1-expected.txt:
  • platform/chromium-win/editing/selection/4895428-4-expected.txt:
1:44 AM Changeset in webkit [59453] by eric@webkit.org
  • 2 edits in trunk/WebKit

2010-05-14 Lucas De Marchi <​lucas.demarchi@profusion.mobi>

Reviewed by David Levin.

[EF] Include missing header.
​http://webkit.org/b/38905

  • efl/ewk/ewk_main.cpp: Include PageGroup.h
1:43 AM Changeset in webkit [59452] by mrowe@apple.com
  • 2 edits in branches/safari-533-branch/WebKitTools

Merge r59177.

1:32 AM Changeset in webkit [59451] by eric@webkit.org
  • 2 edits in trunk/WebKit/qt

2010-05-14 Benjamin Poulain <​benjamin.poulain@nokia.com>

Reviewed by Laszlo Gombos.

[QT] Update the Symbian version for the user agent
​https://bugs.webkit.org/show_bug.cgi?id=38389

Update the user agent for Symbian2 to Symbian4

  • Api/qwebpage.cpp: (QWebPage::userAgentForUrl):
1:23 AM Changeset in webkit [59450] by mrowe@apple.com
  • 4 edits in branches/safari-533-branch/WebCore

Disable the Audits panel and Workers sidebar in the Web Inspector.

Patch by Timothy Hatcher <​timothy@apple.com> on 2010-05-13
<rdar://problem/7947035>

Reviewed by Ada Chan.

  • inspector/front-end/ScriptsPanel.js:

(WebInspector.ScriptsPanel): Don't make a workers sidebar pane.
(WebInspector.ScriptsPanel.prototype.reset): Null check workers sidebar pane.

  • inspector/front-end/Settings.js: Set auditsPanelEnabled to false.
  • inspector/front-end/WorkersSidebarPane.js:

(WebInspector.didCreateWorker): Null check workers sidebar pane.
(WebInspector.didDestroyWorker): Ditto.

1:21 AM Changeset in webkit [59449] by eric@webkit.org
  • 2 edits in trunk/WebKit/qt

2010-05-14 Dawit Alemayehu <​adawit@kde.org>

Reviewed by Simon Hausmann.

Implemented createJavaAppletWidget to activate Java applet support.

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

  • WebCoreSupport/FrameLoaderClientQt.cpp: (WebCore::FrameLoaderClientQt::createJavaAppletWidget):
1:09 AM Changeset in webkit [59448] by eric@webkit.org
  • 2 edits in trunk/WebCore

2010-05-14 George Wright <​gwright@rim.com>

Reviewed by George Staikos.

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

Update JSDataGridColumnListCustom.cpp to call identifierToAtomicString()
on the property name.

  • bindings/js/JSDataGridColumnListCustom.cpp: (WebCore::JSDataGridColumnList::canGetItemsForName): (WebCore::JSDataGridColumnList::nameGetter):
12:43 AM Changeset in webkit [59447] by eric@webkit.org
  • 3 edits
    3 adds in trunk

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

Reviewed by Darin Adler.

Add test to check that setting various CanvasRenderingContext2D properties
to invalid values is ignored.

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

  • fast/canvas/canvas-invalid-values-expected.txt: Added.
  • fast/canvas/canvas-invalid-values.html: Added.
  • fast/canvas/script-tests/canvas-invalid-values.js: Added. (trySettingMiterLimit): (trySettingLineWidth): (trySettingShadowBlur): (trySettingShadowOffsetX): (trySettingShadowOffsetY):

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

Reviewed by Darin Adler.

Ignore invalid values for various CanvasRenderingContext2D properties
(lineWidth, miterLimit, shadowOffsetX, shadowOffsetY and shadowBlur)

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

Test: fast/canvas/canvas-invalid-values.html

  • html/canvas/CanvasRenderingContext2D.cpp: (WebCore::CanvasRenderingContext2D::setLineWidth): (WebCore::CanvasRenderingContext2D::setMiterLimit): (WebCore::CanvasRenderingContext2D::setShadowOffsetX): (WebCore::CanvasRenderingContext2D::setShadowOffsetY): (WebCore::CanvasRenderingContext2D::setShadowBlur):
12:08 AM Changeset in webkit [59446] by tkent@chromium.org
  • 1 edit in trunk/LayoutTests/platform/chromium/test_expectations.txt

Unreviewed, test expectation update.

deleted-host-in-resource-load-delegate-callback.html sometimes crashes on Windows Release.

Note: See TracTimeline for information about the timeline view.