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

Timeline



Oct 17, 2010:

11:51 PM Changeset in webkit [69942] by commit-queue@webkit.org
  • 11 edits
    6 adds in trunk

2010-10-17 Pascal Massimino <​pascal.massimino@gmail.com>

Reviewed by Adam Barth.

Add support for WebP image decoding in ImageDecoder
using library libwebp-decode library available from
​http://www.webmproject.org/code/

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

Tests: fast/images/webp-image-decoding.html

  • CMakeLists.txt:

add platform/image-decoders/webp files and directory

  • GNUmakefile.am:

add platform/image-decoders/webp directory

  • WebCore.gyp/WebCore.gyp:

add platform/image-decoders/webp directory

  • WebCore.gypi:

add platform/image-decoders/webp directory

  • features.gypi:

add WTF_USE_WEBP=1

  • platform/image-decoders/ImageDecoder.cpp: (WebCore::ImageDecoder::create):

instantiate webp decoder for file starting with 'RIFF????WEBP'

  • platform/image-decoders/webp: Added.
  • platform/image-decoders/webp/WEBPImageDecoder.cpp: Added. (WebCore::WEBPImageDecoder::WEBPImageDecoder): (WebCore::WEBPImageDecoder::~WEBPImageDecoder): (WebCore::WEBPImageDecoder::isSizeAvailable): (WebCore::WEBPImageDecoder::frameBufferAtIndex): (WebCore::WEBPImageDecoder::decode):

sub-class handling decoding of webp images

  • platform/image-decoders/webp/WEBPImageDecoder.h: Added. (WebCore::WEBPImageDecoder::filenameExtension): (WebCore::WEBPImageDecoder::supportsAlpha):

header for webp-decoder sub-class

2010-10-17 Pascal Massimino <​pascal.massimino@gmail.com>

Reviewed by Adam Barth.


Add support for decoding WebP image
​https://bugs.webkit.org/show_bug.cgi?id=47512


  • DEPS:
    • Add dependency to libwebp.
10:42 PM Changeset in webkit [69941] by oliver@apple.com
  • 5 edits in trunk

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

Reviewed by Sam Weinig.

Strict mode: arguments is not valid as the base expression for pre- or post-fix expressions
​https://bugs.webkit.org/show_bug.cgi?id=47791

Simple fix, check for arguments in addition to eval.

  • parser/JSParser.cpp: (JSC::JSParser::parseUnaryExpression):

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

Reviewed by Sam Weinig.

Strict mode: arguments is not valid as the base expression for pre- or post-fix expressions
​https://bugs.webkit.org/show_bug.cgi?id=47791

Add arguments tests, and make pre-/post-fix expression tests cover another case I was
needlessly worried about.

  • fast/js/basic-strict-mode-expected.txt:
  • fast/js/script-tests/basic-strict-mode.js:
9:08 PM Changeset in webkit [69940] by oliver@apple.com
  • 20 edits in trunk

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

Reviewed by Sam Weinig.

Strict mode: Assignment that would create a global should be a late ReferenceError, not a syntax failure
​https://bugs.webkit.org/show_bug.cgi?id=47788

Fixing this required a couple of changes:

  • resolve_base now has a flag to indicate whether it is being used for a put in strict mode. this allows us to throw an exception when we're doing a completely generic resolve for assignment, and that assignment would create a new global.
  • There is a new opcode 'op_ensure_property_exists' that is used to determine whether the property being assigned to already exists on the global object. This currently has no caching, but such caching could be added relatively trivially. It is only used in the case where we know that a property will be placed on the global object, and we cannot verify that the property already exists.

In the jit we plant a call to cti_op_resolve_base_strict_put in the effected case rather
than making op_resolve_base have an additional runtime branch.

There's also a new helper function to create the exception for the invalid assignment.

  • bytecode/CodeBlock.cpp: (JSC::CodeBlock::dump):
  • bytecode/Opcode.h:
  • bytecompiler/BytecodeGenerator.cpp: (JSC::BytecodeGenerator::emitResolveBase): (JSC::BytecodeGenerator::emitResolveBaseForPut):
  • bytecompiler/BytecodeGenerator.h:
  • bytecompiler/NodesCodegen.cpp: (JSC::AssignResolveNode::emitBytecode): (JSC::ForInNode::emitBytecode):
  • interpreter/Interpreter.cpp: (JSC::Interpreter::resolveBase): (JSC::Interpreter::privateExecute):
  • jit/JIT.cpp: (JSC::JIT::privateCompileMainPass):
  • jit/JIT.h:
  • jit/JITOpcodes.cpp: (JSC::JIT::emit_op_resolve_base): (JSC::JIT::emit_op_ensure_property_exists):
  • jit/JITOpcodes32_64.cpp: (JSC::JIT::emit_op_resolve_base): (JSC::JIT::emit_op_ensure_property_exists):
  • jit/JITStubs.cpp: (JSC::DEFINE_STUB_FUNCTION):
  • jit/JITStubs.h:
  • parser/JSParser.cpp: (JSC::JSParser::parseProgram):
  • runtime/ExceptionHelpers.cpp: (JSC::createErrorForInvalidGlobalAssignment):
  • runtime/ExceptionHelpers.h:
  • runtime/Operations.h: (JSC::resolveBase):

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

Reviewed by Sam Weinig.

Strict mode: Assignment that would create a global should be a late ReferenceError, not a syntax failure
​https://bugs.webkit.org/show_bug.cgi?id=47788

Update test to check for the correct behaviour.

  • fast/js/basic-strict-mode-expected.txt:
  • fast/js/script-tests/basic-strict-mode.js:
9:02 PM Changeset in webkit [69939] by tonikitoo@webkit.org
  • 3 edits in trunk/LayoutTests

2010-10-17 Antonio Gomes <​agomes@rim.com>

Reviewed by Simon Fraser.

Enrich /fast/dom/nodesFromRect-basic.html test
​https://bugs.webkit.org/show_bug.cgi?id=47766

The patch makes fast/dom/nodesFromRect-basic.html test more precise:
instead of calling nodesFromRect always with equal vertical (up and down) and
horizontal (right and left) padding values, it are now tests it by passing
different padding values on the various directions.

It is now possible due to the work on bug 46336: Make Document::nodesFromRect more flexible.

  • fast/dom/nodesFromRect-basic-expected.txt:
  • fast/dom/nodesFromRect-basic.html:
7:59 PM Changeset in webkit [69938] by abarth@webkit.org
  • 9 edits in trunk/WebCore

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

Reviewed by Dimitri Glazkov.

Remove explicit passing of user gesture in NavigationScheduler
​https://bugs.webkit.org/show_bug.cgi?id=47786

As requested by Mr. Glazkov, remove more explicit passing of the user
gesture state. We now use a static to pass this information around.

  • bindings/generic/BindingFrame.h: (WebCore::::navigateIfAllowed):
  • bindings/js/JSDOMWindowCustom.cpp: (WebCore::JSDOMWindow::setLocation):
  • bindings/js/JSLocationCustom.cpp: (WebCore::JSLocation::reload):
  • bindings/v8/custom/V8LocationCustom.cpp: (WebCore::V8Location::reloadCallback):
  • inspector/InspectorController.cpp: (WebCore::InspectorController::setResourceTrackingEnabled): (WebCore::InspectorController::reloadPage):
  • loader/NavigationScheduler.cpp: (WebCore::ScheduledNavigation::ScheduledNavigation): (WebCore::ScheduledURLNavigation::ScheduledURLNavigation): (WebCore::ScheduledRedirect::ScheduledRedirect): (WebCore::ScheduledLocationChange::ScheduledLocationChange): (WebCore::ScheduledRefresh::ScheduledRefresh): (WebCore::ScheduledHistoryNavigation::ScheduledHistoryNavigation): (WebCore::ScheduledFormSubmission::ScheduledFormSubmission): (WebCore::NavigationScheduler::scheduleRedirect): (WebCore::NavigationScheduler::mustLockBackForwardList): (WebCore::NavigationScheduler::scheduleLocationChange): (WebCore::NavigationScheduler::scheduleFormSubmission): (WebCore::NavigationScheduler::scheduleRefresh): (WebCore::NavigationScheduler::scheduleHistoryNavigation):
  • loader/NavigationScheduler.h:
  • loader/SubframeLoader.cpp: (WebCore::SubframeLoader::loadOrRedirectSubframe):
6:22 PM Changeset in webkit [69937] by jchaffraix@webkit.org
  • 5 edits
    2 adds in trunk

WebCore: 'transparent' should be a valid color
​https://bugs.webkit.org/show_bug.cgi?id=45482

Reviewed by Andreas Kling.

Test: svg/animations/svg-animate-color-transparent.svg

Based on previous work by Jan Erik Hanssen <​jhanssen@codeaurora.org>

  • platform/ColorData.gperf: Added 'transparent' to the list of colors.

Added the alpha to all the colors as it was needed to support the new color.

  • platform/graphics/Color.cpp:

(WebCore::Color::Color):
(WebCore::Color::setNamedColor): Avoid setting the alpha as now it is not
always 0xFF.

LayoutTests: Test for: 'transparent' should be a valid color
​https://bugs.webkit.org/show_bug.cgi?id=45482

Reviewed by Andreas Kling.

Test that animation works with 'transparent' as they check the validity of the
color.

  • svg/animations/svg-animate-color-transparent-expected.txt: Added.
  • svg/animations/svg-animate-color-transparent.svg: Added.
5:47 PM Changeset in webkit [69936] by jschuh@chromium.org
  • 3 edits
    2 adds in trunk

2010-10-17 Justin Schuh <​jschuh@chromium.org>

Reviewed by Nikolas Zimmermann.

Duplicate use element children in shadow tree.
​https://bugs.webkit.org/show_bug.cgi?id=47561

Test: svg/custom/use-nested-children.svg

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

2010-10-17 Justin Schuh <​jschuh@chromium.org>

Reviewed by Nikolas Zimmermann.

Duplicate use element children in shadow tree.
​https://bugs.webkit.org/show_bug.cgi?id=47561

  • svg/custom/use-nested-children-expected.txt: Added.
  • svg/custom/use-nested-children.svg: Added.
5:04 PM Changeset in webkit [69935] by commit-queue@webkit.org
  • 2 edits in trunk/WebCore

2010-10-17 Hyung Song <​beergun@company100.net>

Reviewed by David Levin.

Add !SINGLE_THREADED guard.
​https://bugs.webkit.org/show_bug.cgi?id=47608

For SINGLE_THREADED ports LockingMutex.tryLock() returns false.
This will prevent interrupt() from falling into infinite loop.

  • platform/sql/SQLiteDatabase.cpp: (WebCore::SQLiteDatabase::interrupt):
3:05 PM Changeset in webkit [69934] by Simon Fraser
  • 1 edit
    1 delete in trunk/LayoutTests

2010-10-17 Simon Fraser <Simon Fraser>

Remove Windows-specific result, which is not longer needed after r69928.

  • platform/win/fast/css/large-number-round-trip-expected.txt: Removed.
2:55 PM Changeset in webkit [69933] by jorlow@chromium.org
  • 32 edits in branches/chromium/552

The names of IndexedDB-specific attributes of DOMWindow should be prefixed with 'webkit'.
​https://bugs.webkit.org/show_bug.cgi?id=47508

BUG=59508

Patch by Andrei Popescu <​andreip@google.com> on 2010-10-11
Reviewed by Jeremy Orlow.

LayoutTests:

  • storage/indexeddb/basics-expected.txt:
  • storage/indexeddb/basics.html:
  • storage/indexeddb/constants-expected.txt:
  • storage/indexeddb/constants.html:
  • storage/indexeddb/database-basics-expected.txt:
  • storage/indexeddb/database-basics.html:
  • storage/indexeddb/database-quota-expected.txt:
  • storage/indexeddb/database-quota.html:
  • storage/indexeddb/index-basics-expected.txt:
  • storage/indexeddb/index-basics.html:
  • storage/indexeddb/index-cursor-expected.txt:
  • storage/indexeddb/index-cursor.html:
  • storage/indexeddb/keyrange-expected.txt:
  • storage/indexeddb/keyrange.html:
  • storage/indexeddb/objectstore-basics-expected.txt:
  • storage/indexeddb/objectstore-basics.html:
  • storage/indexeddb/objectstore-cursor-expected.txt:
  • storage/indexeddb/objectstore-cursor.html:
  • storage/indexeddb/objectstore-removeobjectstore-expected.txt:
  • storage/indexeddb/objectstore-removeobjectstore.html:
  • storage/indexeddb/open-cursor-expected.txt:
  • storage/indexeddb/open-cursor.html:
  • storage/indexeddb/transaction-basics-expected.txt:
  • storage/indexeddb/transaction-basics.html:

WebCore:

  • bindings/generic/RuntimeEnabledFeatures.h:

(WebCore::RuntimeEnabledFeatures::setWebkitIndexedDBEnabled):
(WebCore::RuntimeEnabledFeatures::webkitIndexedDBEnabled):
(WebCore::RuntimeEnabledFeatures::webkitIDBCursorEnabled):
(WebCore::RuntimeEnabledFeatures::webkitIDBDatabaseEnabled):
(WebCore::RuntimeEnabledFeatures::webkitIDBDatabaseErrorEnabled):
(WebCore::RuntimeEnabledFeatures::webkitIDBDatabaseExceptionEnabled):
(WebCore::RuntimeEnabledFeatures::webkitIDBErrorEventEnabled):
(WebCore::RuntimeEnabledFeatures::webkitIDBEventEnabled):
(WebCore::RuntimeEnabledFeatures::webkitIDBFactoryEnabled):
(WebCore::RuntimeEnabledFeatures::webkitIDBIndexEnabled):
(WebCore::RuntimeEnabledFeatures::webkitIDBKeyRangeEnabled):
(WebCore::RuntimeEnabledFeatures::webkitIDBObjectStoreEnabled):
(WebCore::RuntimeEnabledFeatures::webkitIDBRequestEnabled):
(WebCore::RuntimeEnabledFeatures::webkitIDBSuccessEventEnabled):
(WebCore::RuntimeEnabledFeatures::webkitIDBTransactionEnabled):

  • page/DOMWindow.cpp:

(WebCore::DOMWindow::webkitIndexedDB):

  • page/DOMWindow.h:
  • page/DOMWindow.idl:

WebKit/chromium:

  • src/WebRuntimeFeatures.cpp:

(WebKit::WebRuntimeFeatures::enableIndexedDatabase):
(WebKit::WebRuntimeFeatures::isIndexedDatabaseEnabled):
Merge 69540 - 2010-10-11 Andrei Popescu <​andreip@google.com>

Reviewed by Jeremy Orlow.

The names of IndexedDB-specific attributes of DOMWindow should be prefixed with 'webkit'.
​https://bugs.webkit.org/show_bug.cgi?id=47508

  • storage/indexeddb/basics-expected.txt:
  • storage/indexeddb/basics.html:
  • storage/indexeddb/constants-expected.txt:
  • storage/indexeddb/constants.html:
  • storage/indexeddb/database-basics-expected.txt:
  • storage/indexeddb/database-basics.html:
  • storage/indexeddb/database-quota-expected.txt:
  • storage/indexeddb/database-quota.html:
  • storage/indexeddb/index-basics-expected.txt:
  • storage/indexeddb/index-basics.html:
  • storage/indexeddb/index-cursor-expected.txt:
  • storage/indexeddb/index-cursor.html:
  • storage/indexeddb/keyrange-expected.txt:
  • storage/indexeddb/keyrange.html:
  • storage/indexeddb/objectstore-basics-expected.txt:
  • storage/indexeddb/objectstore-basics.html:
  • storage/indexeddb/objectstore-cursor-expected.txt:
  • storage/indexeddb/objectstore-cursor.html:
  • storage/indexeddb/objectstore-removeobjectstore-expected.txt:
  • storage/indexeddb/objectstore-removeobjectstore.html:
  • storage/indexeddb/open-cursor-expected.txt:
  • storage/indexeddb/open-cursor.html:
  • storage/indexeddb/transaction-basics-expected.txt:
  • storage/indexeddb/transaction-basics.html:

2010-10-11 Andrei Popescu <​andreip@google.com>

Reviewed by Jeremy Orlow.

The names of IndexedDB-specific attributes of DOMWindow should be prefixed with 'webkit'.
​https://bugs.webkit.org/show_bug.cgi?id=47508

  • bindings/generic/RuntimeEnabledFeatures.h: (WebCore::RuntimeEnabledFeatures::setWebkitIndexedDBEnabled): (WebCore::RuntimeEnabledFeatures::webkitIndexedDBEnabled): (WebCore::RuntimeEnabledFeatures::webkitIDBCursorEnabled): (WebCore::RuntimeEnabledFeatures::webkitIDBDatabaseEnabled): (WebCore::RuntimeEnabledFeatures::webkitIDBDatabaseErrorEnabled): (WebCore::RuntimeEnabledFeatures::webkitIDBDatabaseExceptionEnabled): (WebCore::RuntimeEnabledFeatures::webkitIDBErrorEventEnabled): (WebCore::RuntimeEnabledFeatures::webkitIDBEventEnabled): (WebCore::RuntimeEnabledFeatures::webkitIDBFactoryEnabled): (WebCore::RuntimeEnabledFeatures::webkitIDBIndexEnabled): (WebCore::RuntimeEnabledFeatures::webkitIDBKeyRangeEnabled): (WebCore::RuntimeEnabledFeatures::webkitIDBObjectStoreEnabled): (WebCore::RuntimeEnabledFeatures::webkitIDBRequestEnabled): (WebCore::RuntimeEnabledFeatures::webkitIDBSuccessEventEnabled): (WebCore::RuntimeEnabledFeatures::webkitIDBTransactionEnabled):
  • page/DOMWindow.cpp: (WebCore::DOMWindow::webkitIndexedDB):
  • page/DOMWindow.h:
  • page/DOMWindow.idl:

2010-10-11 Andrei Popescu <​andreip@google.com>

Reviewed by Jeremy Orlow.

The names of IndexedDB-specific attributes of DOMWindow should be prefixed with 'webkit'.
​https://bugs.webkit.org/show_bug.cgi?id=47508

  • src/WebRuntimeFeatures.cpp: (WebKit::WebRuntimeFeatures::enableIndexedDatabase): (WebKit::WebRuntimeFeatures::isIndexedDatabaseEnabled):

TBR=​andreip@google.com

2:48 PM Changeset in webkit [69932] by jorlow@chromium.org
  • 3 edits in branches/chromium/552/LayoutTests

storage/indexeddb/database-quota.html expects the wrong database version string
​https://bugs.webkit.org/show_bug.cgi?id=47533

BUG=59506

Patch by Andrei Popescu <​andreip@google.com> on 2010-10-11
Reviewed by Jeremy Orlow.

  • storage/indexeddb/database-quota-expected.txt:
  • storage/indexeddb/database-quota.html:

Merge 69612 - 2010-10-11 Andrei Popescu <​andreip@google.com>

Reviewed by Jeremy Orlow.

storage/indexeddb/database-quota.html expects the wrong database version string
​https://bugs.webkit.org/show_bug.cgi?id=47533

  • storage/indexeddb/database-quota-expected.txt:
  • storage/indexeddb/database-quota.html:

TBR=​andreip@google.com

2:46 PM Changeset in webkit [69931] by jorlow@chromium.org
  • 2 edits in branches/chromium/552/WebKit/chromium

[Chromium] Cleanup WebIDBFactory::open once the Chromium plumbing has landed.
​https://bugs.webkit.org/show_bug.cgi?id=47531

BUG=59506

Patch by Andrei Popescu <​andreip@google.com> on 2010-10-11
Reviewed by Jeremy Orlow.

Remove the temporary open() method since the appropriate Chromium plumbing was
added in ​http://codereview.chromium.org/3729003/

  • public/WebIDBFactory.h:

(WebKit::WebIDBFactory::open):
Merge 69670 - 2010-10-11 Andrei Popescu <​andreip@google.com>

Reviewed by Jeremy Orlow.

[Chromium] Cleanup WebIDBFactory::open once the Chromium plumbing has landed.
​https://bugs.webkit.org/show_bug.cgi?id=47531

Remove the temporary open() method since the appropriate Chromium plumbing was
added in ​http://codereview.chromium.org/3729003/

  • public/WebIDBFactory.h: (WebKit::WebIDBFactory::open):

TBR=​andreip@google.com

2:44 PM Changeset in webkit [69930] by Martin Robinson
  • 11 edits in trunk/LayoutTests

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

Rebaseline MathML tests after r69926.

  • platform/gtk/mathml/presentation/attributes-expected.txt:
  • platform/gtk/mathml/presentation/fenced-mi-expected.txt:
  • platform/gtk/mathml/presentation/fractions-expected.txt:
  • platform/gtk/mathml/presentation/fractions-vertical-alignment-expected.txt:
  • platform/gtk/mathml/presentation/mo-expected.txt:
  • platform/gtk/mathml/presentation/row-alignment-expected.txt:
  • platform/gtk/mathml/presentation/row-expected.txt:
  • platform/gtk/mathml/presentation/style-expected.txt:
  • platform/gtk/mathml/presentation/tokenElements-expected.txt:
  • platform/gtk/mathml/presentation/underover-expected.txt:
1:40 PM Changeset in webkit [69929] by Simon Fraser
  • 2 edits in trunk/JavaScriptCore

2010-10-17 Simon Fraser <Simon Fraser>

First part of fix for Windows build failure. Will wait for the
next set of link errors to determine the mangled forms for dtoaRoundSF
and dtoaRoundDP.

1:05 PM Changeset in webkit [69928] by Simon Fraser
  • 12 edits
    2 adds
    1 delete in trunk

2010-10-17 Simon Fraser <Simon Fraser>

Reviewed by Nikolas Zimmermann.

Very large and small numbers fail to round-trip through CSS
​https://bugs.webkit.org/show_bug.cgi?id=20674

Use the new DecimalNumber functionality to convert CSS numberic values to strings
using decimal notation, rather than %g (which can output scientific notation).

Test: fast/css/round-trip-values.html

  • css/CSSPrimitiveValue.cpp: (WebCore::formatNumber): (WebCore::CSSPrimitiveValue::cssText):
12:46 PM Changeset in webkit [69927] by commit-queue@webkit.org
  • 3 edits
    2 adds in trunk

2010-10-17 Cosmin Truta <​ctruta@chromium.org>

Reviewed by Nikolas Zimmermann.

Crash while processing ill-formed SVG with cycles
​https://bugs.webkit.org/show_bug.cgi?id=47498

Checked the behavior of the attributes fill, stroke, clip and mask
when used with URLs that point to wrong elements.

  • svg/custom/invalid-paint-shape-mask.svg: Added.
  • svg/custom/invalid-paint-shape-mask-expected.svg: Added.

2010-10-17 Cosmin Truta <​ctruta@chromium.org>

Reviewed by Nikolas Zimmermann.

Crash while processing ill-formed SVG with cycles
​https://bugs.webkit.org/show_bug.cgi?id=47498

Test: svg/custom/invalid-paint-shape-mask.svg

  • rendering/SVGResources.cpp: (paintingResourceFromSVGPaint): Ensured that the painting resource is either a pattern resource or a gradient resource. (SVGResources::setClipper): (SVGResources::setFilter): (SVGResources::setMarkerStart): (SVGResources::setMarkerMid): (SVGResources::setMarkerEnd): (SVGResources::setMasker): (SVGResources::setFill): (SVGResources::setStroke): Added ASSERT statements for previously-unchecked preconditions.
11:44 AM Changeset in webkit [69926] by commit-queue@webkit.org
  • 26 edits in trunk

2010-10-17 Alex Milowski <​alex@milowski.com>

Reviewed by Kenneth Rohde Christiansen.

Since whitespace and baseline alignment have changed for operators,
many tests needed to be re-baselined.

  • platform/mac/mathml/presentation/attributes-expected.checksum:
  • platform/mac/mathml/presentation/attributes-expected.png:
  • platform/mac/mathml/presentation/attributes-expected.txt:
  • platform/mac/mathml/presentation/fenced-expected.checksum:
  • platform/mac/mathml/presentation/fenced-expected.png:
  • platform/mac/mathml/presentation/fenced-expected.txt:
  • platform/mac/mathml/presentation/fractions-vertical-alignment-expected.checksum:
  • platform/mac/mathml/presentation/fractions-vertical-alignment-expected.png:
  • platform/mac/mathml/presentation/fractions-vertical-alignment-expected.txt:
  • platform/mac/mathml/presentation/mo-expected.checksum:
  • platform/mac/mathml/presentation/mo-expected.png:
  • platform/mac/mathml/presentation/mo-expected.txt:
  • platform/mac/mathml/presentation/roots-expected.checksum:
  • platform/mac/mathml/presentation/roots-expected.png:
  • platform/mac/mathml/presentation/roots-expected.txt:
  • platform/mac/mathml/presentation/row-alignment-expected.checksum:
  • platform/mac/mathml/presentation/row-alignment-expected.png:
  • platform/mac/mathml/presentation/row-alignment-expected.txt:
  • platform/mac/mathml/presentation/row-expected.checksum:
  • platform/mac/mathml/presentation/row-expected.png:
  • platform/mac/mathml/presentation/row-expected.txt:

2010-10-17 Alex Milowski <​alex@milowski.com>

Reviewed by Kenneth Rohde Christiansen.

Fixed extra whitespace above operators when they aren't stretched
yet the stretch height is large. Also, the baseline alignment
was changed so that rows and operators interact appropriately now
that operators don't have the extra white space.

  • mathml/RenderMathMLOperator.cpp:
  • mathml/RenderMathMLRow.cpp:
11:19 AM Changeset in webkit [69925] by commit-queue@webkit.org
  • 4 edits
    3 adds in trunk

2010-10-17 Viatcheslav Ostapenko <​ostapenko.viatcheslav@nokia.com>

Reviewed by Antonio Gomes.

[Qt] Layout tests for fix focusing of control elements on mouse click.
​https://bugs.webkit.org/show_bug.cgi?id=40641

  • fast/events/click-focus-control-expected.txt: Added.
  • fast/events/click-focus-control.html: Added.
  • platform/mac/fast/events/click-focus-control-expected.txt: Added.
  • platform/win/Skipped:

2010-10-17 Viatcheslav Ostapenko <​ostapenko.viatcheslav@nokia.com>

Reviewed by Antonio Gomes.

[Qt] Fix focusing of control elements on mouse click.
Brings GTK fix to Qt platform.
​https://bugs.webkit.org/show_bug.cgi?id=40641

Test: fast/events/click-focus-control.html

  • html/HTMLFormControlElement.cpp: (WebCore::HTMLFormControlElement::isMouseFocusable):
9:02 AM Changeset in webkit [69924] by abarth@webkit.org
  • 14 edits in trunk

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

Reviewed by Dimitri Glazkov.

FrameLoader doesn't need an explicit userGesture parameter
​https://bugs.webkit.org/show_bug.cgi?id=47777

We used to pass around the user gesture state explicitly. Now we use
static state. Explicitly passing the user gesture state to FrameLoader
in these methods is pretty out-dated.

  • bindings/ScriptControllerBase.cpp: (WebCore::ScriptController::executeIfJavaScriptURL):
  • bindings/generic/BindingDOMWindow.h: (WebCore::::createWindow): (WebCore::::open):
  • bindings/js/JSDOMWindowCustom.cpp: (WebCore::createWindow): (WebCore::JSDOMWindow::open):
  • bindings/js/JSDocumentCustom.cpp: (WebCore::JSDocument::setLocation):
  • bindings/js/ScriptController.h:
  • bindings/v8/ScriptController.h:
  • html/HTMLAnchorElement.cpp: (WebCore::handleLinkClick):
  • inspector/InspectorController.cpp: (WebCore::InspectorController::openInInspectedWindow):
  • loader/FrameLoader.cpp: (WebCore::FrameLoader::changeLocation): (WebCore::FrameLoader::urlSelected): (WebCore::FrameLoader::submitForm):
  • loader/FrameLoader.h:
  • loader/NavigationScheduler.cpp: (WebCore::ScheduledURLNavigation::fire): (WebCore::ScheduledRefresh::fire): (WebCore::ScheduledHistoryNavigation::fire): (WebCore::NavigationScheduler::scheduleLocationChange):

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

Reviewed by Dimitri Glazkov.

FrameLoader doesn't need an explicit userGesture parameter
​https://bugs.webkit.org/show_bug.cgi?id=47777

Update for the new API.

  • WebCoreSupport/WebContextMenuClient.cpp: (WebContextMenuClient::searchWithGoogle):
7:31 AM Changeset in webkit [69923] by commit-queue@webkit.org
  • 2 edits in trunk/WebCore

2010-10-17 Sergey A. Sukiyazov <​sergey.sukiyazov@gmail.com>

Reviewed by Andreas Kling

[Qt] Hovering the mouse over links produce a trail of underlined links (X11 paint engine)
​https://bugs.webkit.org/show_bug.cgi?id=42248

The problem will appear because coordinates of points may increase by 0.05f (if line width is odd) inside
method GraphicsContext::adjustLineToPixelBoundaries(...) and become outside of text bounding rect htere,
then the new point coordinates will be passed to Qt graphics engine.

The solution decreases Y cordinates of points inside drawLineForText(...) method only if Qt graphics engine
is X11. The Y coordinates will be increase by 0.5f inside method adjustLineToPixelBoundaries(...), which
called from drawLine(...), and then inside Qt painting engine will be rounded to next greater integer value.

NOTE: This changes will affect only Qt X11 verision and if only X11 Painting Engine will be used.

No new tests.

  • platform/graphics/qt/GraphicsContextQt.cpp: (WebCore::GraphicsContext::drawLineForText):
3:53 AM Changeset in webkit [69922] by rwlbuis@webkit.org
  • 10 edits in trunk

2010-10-17 Rob Buis <​rwlbuis@gmail.com>

Reviewed by Dirk Schulze.

Text under nested link dropped
​https://bugs.webkit.org/show_bug.cgi?id=46460

Allow links inside of tspan and textPath elements.

  • svg/SVGTSpanElement.cpp: (WebCore::SVGTSpanElement::childShouldCreateRenderer):
  • svg/SVGTextPathElement.cpp: (WebCore::SVGTextPathElement::childShouldCreateRenderer):

Oct 16, 2010:

9:47 PM Changeset in webkit [69921] by tonikitoo@webkit.org
  • 4 edits
    1 add in trunk/LayoutTests/platform/gtk/fast/frames/flattening

2010-10-16 Antonio Gomes <​agomes@rim.com>

Complementary to r69919.

Unreviewed expectation update for frame flattening tests from the Gtk+ bots.

Basically 3px difference between the results generated on my machine and the ones
from the Gtk+ bots.

  • platform/gtk/fast/frames/flattening/frameset-flattening-advanced-expected.txt: Added.
  • platform/gtk/fast/frames/flattening/frameset-flattening-grid-expected.txt:
  • platform/gtk/fast/frames/flattening/frameset-flattening-simple-expected.txt:
  • platform/gtk/fast/frames/flattening/frameset-flattening-subframe-resize-expected.txt:
  • platform/gtk/fast/frames/flattening/frameset-flattening-subframesets-expected.txt:
9:43 PM Changeset in webkit [69920] by commit-queue@webkit.org
  • 3 edits in trunk/JavaScriptCore

2010-10-16 Kyusun Kim <​maniagoon@company100.net>

Reviewed by Alexey Proskuryakov.

Add using declarations for currentTimeMS() and parseDateFromNullTerminatedCharacters()
​https://bugs.webkit.org/show_bug.cgi?id=47758

  • wtf/CurrentTime.h:
  • wtf/DateMath.h:
8:51 PM Changeset in webkit [69919] by tonikitoo@webkit.org
  • 8 edits
    11 adds in trunk

[Gtk]: DRT does not support frame flattening testing
​https://bugs.webkit.org/show_bug.cgi?id=38650

Reviewed by Martin Robinson and Xan Lopez.
Patch by Antonio Gomes <​agomes@rim.com>

WebKit/gtk:

Implement enable-frame-flattening API for Gtk+.

  • webkit/webkitwebsettings.cpp:

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

  • webkit/webkitwebview.cpp:

(webkit_web_view_update_settings):
(webkit_web_view_settings_notify):

WebKitTools:

Implement DRT's support for toggling on/off frame flattening
support.

  • DumpRenderTree/gtk/DumpRenderTree.cpp:

(resetDefaultsToConsistentValues):

  • DumpRenderTree/gtk/LayoutTestControllerGtk.cpp:

(LayoutTestController::setFrameFlatteningEnabled):

LayoutTests:

Unskipping passing frame flattening tests and adding expected results
for them.

  • platform/gtk/Skipped:
6:54 PM Changeset in webkit [69918] by dbates@webkit.org
  • 10 edits
    25 adds in trunk/LayoutTests

2010-10-16 Alex Milowski <​alex@milowski.com>

Reviewed by Daniel Bates.

New MathML Baseline for GTK Port
​https://bugs.webkit.org/show_bug.cgi?id=47774

New baseline for MathML tests for GTK port.

  • platform/gtk/mathml/presentation/attributes-expected.checksum: Added.
  • platform/gtk/mathml/presentation/attributes-expected.png: Added.
  • platform/gtk/mathml/presentation/attributes-expected.txt:
  • platform/gtk/mathml/presentation/fenced-mi-expected.checksum: Added.
  • platform/gtk/mathml/presentation/fenced-mi-expected.png: Added.
  • platform/gtk/mathml/presentation/fenced-mi-expected.txt: Added.
  • platform/gtk/mathml/presentation/fractions-expected.checksum: Added.
  • platform/gtk/mathml/presentation/fractions-expected.png: Added.
  • platform/gtk/mathml/presentation/fractions-expected.txt:
  • platform/gtk/mathml/presentation/fractions-vertical-alignment-expected.checksum: Added.
  • platform/gtk/mathml/presentation/fractions-vertical-alignment-expected.png: Added.
  • platform/gtk/mathml/presentation/fractions-vertical-alignment-expected.txt:
  • platform/gtk/mathml/presentation/mo-expected.checksum: Added.
  • platform/gtk/mathml/presentation/mo-expected.png: Added.
  • platform/gtk/mathml/presentation/mo-expected.txt:
  • platform/gtk/mathml/presentation/over-expected.checksum: Added.
  • platform/gtk/mathml/presentation/over-expected.png: Added.
  • platform/gtk/mathml/presentation/row-alignment-expected.checksum: Added.
  • platform/gtk/mathml/presentation/row-alignment-expected.png: Added.
  • platform/gtk/mathml/presentation/row-alignment-expected.txt:
  • platform/gtk/mathml/presentation/row-expected.checksum: Added.
  • platform/gtk/mathml/presentation/row-expected.png: Added.
  • platform/gtk/mathml/presentation/row-expected.txt:
  • platform/gtk/mathml/presentation/style-expected.checksum: Added.
  • platform/gtk/mathml/presentation/style-expected.png: Added.
  • platform/gtk/mathml/presentation/style-expected.txt:
  • platform/gtk/mathml/presentation/tokenElements-expected.checksum: Added.
  • platform/gtk/mathml/presentation/tokenElements-expected.png: Added.
  • platform/gtk/mathml/presentation/tokenElements-expected.txt:
  • platform/gtk/mathml/presentation/under-expected.checksum: Added.
  • platform/gtk/mathml/presentation/under-expected.png: Added.
  • platform/gtk/mathml/presentation/underover-expected.checksum: Added.
  • platform/gtk/mathml/presentation/underover-expected.png: Added.
  • platform/gtk/mathml/presentation/underover-expected.txt:
2:04 PM Changeset in webkit [69917] by kseo@webkit.org
  • 3 edits
    2 adds in trunk/WebCore

[BREWMP] Port TextCodec
​https://bugs.webkit.org/show_bug.cgi?id=45797

Reviewed by Kent Tamura.

Implement TextCodec with Brew MP's ICharsetConv.
Only UTF-8 is supported for now.

  • platform/text/TextEncoding.cpp:

(WebCore::TextEncoding::encode):

  • platform/text/TextEncodingRegistry.cpp:

(WebCore::buildBaseTextCodecMaps):

  • platform/text/brew/TextCodecBrew.cpp: Added.

(WebCore::newTextCodecBrew):
(WebCore::TextCodecBrew::registerBaseEncodingNames):
(WebCore::TextCodecBrew::registerBaseCodecs):
(WebCore::TextCodecBrew::registerExtendedEncodingNames):
(WebCore::TextCodecBrew::registerExtendedCodecs):
(WebCore::TextCodecBrew::TextCodecBrew):
(WebCore::TextCodecBrew::~TextCodecBrew):
(WebCore::TextCodecBrew::decode):
(WebCore::TextCodecBrew::encode):

  • platform/text/brew/TextCodecBrew.h: Added.
1:07 PM Changeset in webkit [69916] by Patrick Gansterer
  • 2 edits in trunk/WebCore

2010-10-16 Patrick Gansterer <Patrick Gansterer>

Unreviewed.

Build fix for !ENABLE(INSPECTOR) after r69844.

  • inspector/InspectorState.h: Added ENABLE(INSPECTOR) guard.
1:00 PM Changeset in webkit [69915] by Patrick Gansterer
  • 2 edits in trunk/WebCore

2010-10-16 Patrick Gansterer <Patrick Gansterer>

Unreviewed, build fix.

WinCE build fix for r69798.

  • platform/text/wince/TextCodecWinCE.cpp: Added missing include.
9:48 AM Changeset in webkit [69914] by inferno@chromium.org
  • 3 edits
    2 adds in trunk

2010-10-16 Johnny Ding <​jnd@chromium.org>

Reviewed by Adam Barth.

Requires a user gesture when opening file choose dialog.
​https://bugs.webkit.org/show_bug.cgi?id=47593

Test: fast/forms/input-file-not-open-without-gesture.html

  • rendering/RenderFileUploadControl.cpp: (WebCore::RenderFileUploadControl::click):

2010-10-16 Johnny Ding <​jnd@chromium.org>

Reviewed by Adam Barth.

Requires a user gesture when opening file choose dialog.
​https://bugs.webkit.org/show_bug.cgi?id=47593

  • fast/forms/input-file-not-open-without-gesture-expected.txt: Added.
  • fast/forms/input-file-not-open-without-gesture.html: Added.
8:37 AM Changeset in webkit [69913] by commit-queue@webkit.org
  • 40 edits
    4 adds in trunk

2010-10-16 Alex Milowski <​alex@milowski.com>

Reviewed by Kenneth Rohde Christiansen.

The tests were re-baselined as operator padding was adjusted.

  • mathml/presentation/fenced-mi.xhtml: Added.

A test for mi vertical alignment in mfenced.

  • platform/mac/mathml/presentation/attributes-expected.checksum:
  • platform/mac/mathml/presentation/attributes-expected.png:
  • platform/mac/mathml/presentation/attributes-expected.txt:
  • platform/mac/mathml/presentation/fenced-expected.checksum:
  • platform/mac/mathml/presentation/fenced-expected.png:
  • platform/mac/mathml/presentation/fenced-expected.txt:
  • platform/mac/mathml/presentation/fenced-mi-expected.checksum: Added.
  • platform/mac/mathml/presentation/fenced-mi-expected.png: Added.
  • platform/mac/mathml/presentation/fenced-mi-expected.txt: Added.
  • platform/mac/mathml/presentation/fractions-expected.checksum:
  • platform/mac/mathml/presentation/fractions-expected.png:
  • platform/mac/mathml/presentation/fractions-expected.txt:
  • platform/mac/mathml/presentation/fractions-vertical-alignment-expected.checksum:
  • platform/mac/mathml/presentation/fractions-vertical-alignment-expected.png:
  • platform/mac/mathml/presentation/fractions-vertical-alignment-expected.txt:
  • platform/mac/mathml/presentation/mo-expected.checksum:
  • platform/mac/mathml/presentation/mo-expected.png:
  • platform/mac/mathml/presentation/mo-expected.txt:
  • platform/mac/mathml/presentation/roots-expected.checksum:
  • platform/mac/mathml/presentation/roots-expected.png:
  • platform/mac/mathml/presentation/roots-expected.txt:
  • platform/mac/mathml/presentation/row-alignment-expected.checksum:
  • platform/mac/mathml/presentation/row-alignment-expected.png:
  • platform/mac/mathml/presentation/row-alignment-expected.txt:
  • platform/mac/mathml/presentation/row-expected.checksum:
  • platform/mac/mathml/presentation/row-expected.png:
  • platform/mac/mathml/presentation/row-expected.txt:
  • platform/mac/mathml/presentation/style-expected.checksum:
  • platform/mac/mathml/presentation/style-expected.png:
  • platform/mac/mathml/presentation/style-expected.txt:
  • platform/mac/mathml/presentation/subsup-expected.checksum:
  • platform/mac/mathml/presentation/subsup-expected.png:
  • platform/mac/mathml/presentation/subsup-expected.txt:
  • platform/mac/mathml/presentation/tokenElements-expected.checksum:
  • platform/mac/mathml/presentation/tokenElements-expected.png:
  • platform/mac/mathml/presentation/tokenElements-expected.txt:
  • platform/mac/mathml/presentation/underover-expected.checksum:
  • platform/mac/mathml/presentation/underover-expected.png:
  • platform/mac/mathml/presentation/underover-expected.txt:

2010-10-16 Alex Milowski <​alex@milowski.com>

Reviewed by Kenneth Rohde Christiansen.

Changed the vertical alignment within mfenced to baseline to
match the rest of the containers (e.g. mrow) and adjusted the
operator spacing to 0.1em.

Test: mathml/presentation/fenced-mi.xhtml

  • css/mathml.css:
  • mathml/RenderMathMLFenced.cpp:
3:45 AM Changeset in webkit [69912] by Patrick Gansterer
  • 8 edits
    3 moves in trunk

2010-10-16 Patrick Gansterer <Patrick Gansterer>

Reviewed by Adam Barth.

Rename StringHasherFunctions.h to StringHasher.h
​https://bugs.webkit.org/show_bug.cgi?id=47200

Now StringHasherFunctions.h only contains the StringHasher class, so rename it to the correct name.

  • GNUmakefile.am:
  • JavaScriptCore.gypi:
  • JavaScriptCore.xcodeproj/project.pbxproj:
  • wtf/StringHashFunctions.h: Removed.
  • wtf/StringHasher.h: Copied from JavaScriptCore/wtf/StringHashFunctions.h.
  • wtf/text/StringHash.h:
  • wtf/text/StringImpl.h:

2010-10-16 Patrick Gansterer <Patrick Gansterer>

Reviewed by Adam Barth.

Rename StringHasherFunctions.h to StringHasher.h
​https://bugs.webkit.org/show_bug.cgi?id=47200

  • ForwardingHeaders/wtf/StringHashFunctions.h: Removed.
  • ForwardingHeaders/wtf/StringHasher.h: Copied from JavaScriptGlue/ForwardingHeaders/wtf/StringHashFunctions.h.

2010-10-16 Patrick Gansterer <Patrick Gansterer>

Reviewed by Adam Barth.

Rename StringHasherFunctions.h to StringHasher.h
​https://bugs.webkit.org/show_bug.cgi?id=47200

  • ForwardingHeaders/wtf/StringHashFunctions.h: Removed.
  • ForwardingHeaders/wtf/StringHasher.h: Copied from WebCore/ForwardingHeaders/wtf/StringHashFunctions.h.
Note: See TracTimeline for information about the timeline view.