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

Timeline



Jun 7, 2010:

11:36 PM Changeset in webkit [60823] by tony@chromium.org
  • 2 edits in trunk/WebKit/chromium

2010-06-07 Tony Chang <​tony@chromium.org>

Reviewed by Kent Tamura.

[chromium] roll chromium DEPS
​https://bugs.webkit.org/show_bug.cgi?id=40277

  • DEPS:
11:26 PM Changeset in webkit [60822] by morrita@google.com
  • 17 edits
    16 adds in trunk

2010-06-07 MORITA Hajime <​morrita@google.com>

Reviewed by Kent Tamura.

[Mac] <meter> elements should be rendered as level indicators.
​https://bugs.webkit.org/show_bug.cgi?id=40217

  • fast/dom/HTMLMeterElement/meter-appearances-capacity.html: Added.
  • fast/dom/HTMLMeterElement/meter-appearances-rating-relevancy.html: Added.
  • fast/dom/HTMLMeterElement/meter-optimums.html: Added.
  • platform/chromium/test_expectations.txt:
  • platform/gtk/Skipped:
  • platform/mac/Skipped: removed <meter> related tests.
  • platform/mac/fast/dom/HTMLMeterElement/meter-appearances-capacity-expected.checksum: Added.
  • platform/mac/fast/dom/HTMLMeterElement/meter-appearances-capacity-expected.png: Added.
  • platform/mac/fast/dom/HTMLMeterElement/meter-appearances-capacity-expected.txt: Added.
  • platform/mac/fast/dom/HTMLMeterElement/meter-appearances-rating-relevancy-expected.checksum: Added.
  • platform/mac/fast/dom/HTMLMeterElement/meter-appearances-rating-relevancy-expected.png: Added.
  • platform/mac/fast/dom/HTMLMeterElement/meter-appearances-rating-relevancy-expected.txt: Added.
  • platform/mac/fast/dom/HTMLMeterElement/meter-element-expected.checksum: Added.
  • platform/mac/fast/dom/HTMLMeterElement/meter-element-expected.png: Added.
  • platform/mac/fast/dom/HTMLMeterElement/meter-element-expected.txt: Copied from LayoutTests/platform/qt/fast/dom/HTMLMeterElement/meter-element-expected.txt.
  • platform/mac/fast/dom/HTMLMeterElement/meter-optimums-expected.checksum: Added.
  • platform/mac/fast/dom/HTMLMeterElement/meter-optimums-expected.png: Added.
  • platform/mac/fast/dom/HTMLMeterElement/meter-optimums-expected.txt: Added.
  • platform/win/Skipped:

2010-06-07 MORITA Hajime <​morrita@google.com>

Reviewed by Kent Tamura.

[Mac] <meter> elements should be rendered as level indicators.
​https://bugs.webkit.org/show_bug.cgi?id=40217

Implemented RenderThemeMac::paintMeter() using NSLevelIndicatorCell.

This change also added new -webkit-appearance values to select the
style of level indicators:

  • relevancy-level-indicator,
  • continuous-capacity-level-indicator,
  • discrete-capacity-level-indicator, and
  • rating-level-indicator.

The size of elements are adjusted based on the bounds of the indicator
to make room for painting whole indicator cells.

Tests: fast/dom/HTMLMeterElement/meter-appearances-capacity.html

fast/dom/HTMLMeterElement/meter-appearances-rating-relevancy.html
fast/dom/HTMLMeterElement/meter-optimums.html

  • css/CSSPrimitiveValueMappings.h: (WebCore::CSSPrimitiveValue::CSSPrimitiveValue):
  • css/CSSValueKeywords.in:
  • html/HTMLMeterElement.cpp: (WebCore::HTMLMeterElement::gaugeRegion):
  • html/HTMLMeterElement.h: (WebCore::HTMLMeterElement::):
  • platform/ThemeTypes.h: (WebCore::):
  • rendering/RenderMeter.cpp: (WebCore::RenderMeter::calcWidth): (WebCore::RenderMeter::calcHeight):
  • rendering/RenderMeter.h:
  • rendering/RenderTheme.cpp: (WebCore::RenderTheme::adjustStyle): (WebCore::RenderTheme::paint): (WebCore::RenderTheme::paintBorderOnly): (WebCore::RenderTheme::paintDecorations): (WebCore::RenderTheme::meterSizeForBounds):
  • rendering/RenderTheme.h:
  • rendering/RenderThemeMac.h:
  • rendering/RenderThemeMac.mm: (WebCore::RenderThemeMac::meterSizeForBounds): (WebCore::RenderThemeMac::paintMeter): (WebCore::RenderThemeMac::levelIndicatorStyleFor): (WebCore::RenderThemeMac::levelIndicatorFor):
10:23 PM Changeset in webkit [60821] by morrita@google.com
  • 4 edits in trunk

2010-06-06 MORITA Hajime <​morrita@google.com>

Unreviewd, follow up to r60820

​https://bugs.webkit.org/show_bug.cgi?id=40219
[Mac] ENABLE_METER_TAG should be enabled

Added ENABLE_METER_TAG.

  • Configurations/FeatureDefines.xcconfig:

2010-06-06 MORITA Hajime <​morrita@google.com>

Unreviewd, follow up to r60820

​https://bugs.webkit.org/show_bug.cgi?id=40219
[Mac] ENABLE_METER_TAG should be enabled

  • Configurations/FeatureDefines.xcconfig:
10:02 PM Changeset in webkit [60820] by morrita@google.com
  • 5 edits in trunk

2010-06-07 MORITA Hajime <​morrita@google.com>

Reviewed by Kent Tamura.

​https://bugs.webkit.org/show_bug.cgi?id=40219
[Mac] ENABLE_METER_TAG should be enabled

Added ENABLE_METER_TAG.

  • Configurations/FeatureDefines.xcconfig:

2010-06-07 MORITA Hajime <​morrita@google.com>

Reviewed by Kent Tamura.

[Mac] ENABLE_METER_TAG should be enabled.
​https://bugs.webkit.org/show_bug.cgi?id=40219

  • Configurations/FeatureDefines.xcconfig:
  • WebCore.xcodeproj/project.pbxproj:
9:09 PM Changeset in webkit [60819] by tonikitoo@webkit.org
  • 28 edits
    2 deletes in trunk

Revert "Refactor platform dependent editing behavior code out of Settings"

This reverts commit r60816: Crashed all bots.

9:08 PM Changeset in webkit [60818] by tonikitoo@webkit.org
  • 2 edits in trunk/WebCore

Revert "2010-06-07 Antonio Gomes <​tonikitoo@webkit.org>"

This reverts r60817: Unsuccessful attept to fix the bots.

9:00 PM Changeset in webkit [60817] by tonikitoo@webkit.org
  • 2 edits in trunk/WebCore

2010-06-07 Antonio Gomes <​tonikitoo@webkit.org>

Unreviewed attempt to fix crashy bots (Mac,Win,Qt,Gtk) after r60816.

  • editing/Editor.cpp: (WebCore::Editor::behavior):
8:08 PM Changeset in webkit [60816] by tonikitoo@webkit.org
  • 28 edits
    2 adds in trunk

Refactor platform dependent editing behavior code out of Settings
​https://bugs.webkit.org/show_bug.cgi?id=39854

Reviewed by Ojan Vafai and Darin Adler.
Patch by Antonio Gomes <​tonikitoo@webkit.org>

WebCore:

As per discussion in bug 36627, the following was suggested by Darin Adler: "before the addition of any
new EditingBehavior, I think we need to add functions to translate the overall behavior policy into specific
behaviors rather than directly saying 'behavior == Mac' or 'behavior == Windows'. Helper functions that
expression the different rules in plain language.".

Patch is a first step of an incremental approach to address this request. It does not yet replace all
direct use of Editing{Mac,Windows}Behaviors, but adds infra for that and replaces some of them.

No behavior change, so no new tests.

  • GNUmakefile.am:
  • WebCore.pro:
  • WebCore/WebCore.gypi
  • WebCore/WebCore.vcproj/WebCore.vcproj
  • WebCore/WebCore.xcodeproj/project.pbxproj
  • page/Settings.cpp:

(WebCore::Settings::Settings):

  • page/Settings.h: Moved out EditingBehavior enum to WebCore/editing/EditingBehaviorTypes.h,

and renamed to EditingBehaviorTypes.
(WebCore::Settings::setEditingBehaviorType): Renamed from setEditingBehavior.
(WebCore::Settings::editingBehaviorType): Renamed from editingBehavior.

  • editing/EditingBehavior.h: Added.

(WebCore::EditingBehavior::EditingBehavior): Class to work as a central point for
editing behavior that might have to be handled differently in the different platforms/ports.
Class should be not used or instantiated directly, but rather implicitly constructed
through the Editor class.
(WebCore::EditingBehavior::shouldMoveCaretToHorizontalBoundaryWhenPastTopOrBottom):
(WebCore::EditingBehavior::shouldConsiderSelectionAsDirectional):
(WebCore::EditingBehavior::shouldCenterAlignWhenSelectionIsRevealed):

  • editing/EditingBehaviorTypes.h: Added.

(WebCore::):

  • editing/Editor.cpp:

(WebCore::Editor::behavior): Getter for EditingBehavior class.

  • editing/Editor.h:
  • editing/EditorCommand.cpp:

(WebCore::executeToggleStyle):
(WebCore::SelectionController::setIsDirectional):
(WebCore::SelectionController::positionForPlatform):
(WebCore::SelectionController::modify):

  • page/EventHandler.cpp:

(WebCore::EventHandler::handleMousePressEventSingleClick):

  • rendering/RenderBlock.cpp:

(WebCore::RenderBlock::positionForPointWithInlineChildren):

  • Call sites will look like:

if (frame->editor()->behavior().shouldThisIfThat()))

do something

WebKit:

EditingBehavior enum was renamed to EditingBehaviorTypes and moved out from Settings.h to
EditingBehaviorTypes.h . Call sites in WebKit/ adjusted accordingly.

WebKit/chromium:

  • src/WebSettingsImpl.cpp:

(WebKit::WebSettingsImpl::setEditingBehavior):

WebKit/gtk:

  • webkit/webkitprivate.cpp:

(WebKit::core):

  • webkit/webkitprivate.h:
  • webkit/webkitwebview.cpp:

(webkit_web_view_update_settings):
(webkit_web_view_settings_notify):

WebKit/mac:

  • WebView/WebFrame.mm:

(core):

  • WebView/WebFrameInternal.h:
  • WebView/WebView.mm:

(-[WebView _preferencesChangedNotification:]):

WebKit/qt:

  • WebCoreSupport/DumpRenderTreeSupportQt.cpp:

(DumpRenderTreeSupportQt::setEditingBehavior):

WebKit/win:

  • WebView.cpp:

(WebView::notifyPreferencesChanged):

7:33 PM Changeset in webkit [60815] by eric@webkit.org
  • 2 edits in trunk/WebCore

2010-06-07 Eric Seidel <​eric@webkit.org>

Unreviewed. Just commit an edit XCode keeps making automatically.

It appears someone hand-edited the file, XCode no-like.

  • WebCore.xcodeproj/project.pbxproj:
7:25 PM Changeset in webkit [60814] by eric@webkit.org
  • 2 edits in trunk/WebCore

2010-06-07 Eric Seidel <​eric@webkit.org>

Reviewed by Adam Barth.

HTML5 Parser fails script-tests which use document.write
​https://bugs.webkit.org/show_bug.cgi?id=40276

We were leaving the TreeBuilder paused when executing the scripts
resulting in document.write() calls being ignored.

I don't see a good way to ASSERT this correct behavior, since
the HTML5ScriptRunner and the HTML5TreeBuilder do not know about
each other. We should never have document.write() called while
the HTML5TreeBuilder is paused, however both document.write and
the network call the same HTML5Tokenizer::write method. It's OK
to be paused when the network writes, but during document.write
(or any other script execution).

Tested by all of fast/url and numerous other tests.

  • html/HTML5Tokenizer.cpp: (WebCore::HTML5Tokenizer::notifyFinished):
    • Unpause the treebuilder before executing scripts.

(WebCore::HTML5Tokenizer::executeScriptsWaitingForStylesheets):

  • Unpause the treebuilder before executing scripts.
5:57 PM Changeset in webkit [60813] by eric@webkit.org
  • 9 edits in trunk/WebCore

2010-06-07 Eric Seidel <​eric@webkit.org>

Reviewed by Adam Barth.

HTML5 Parser hits ASSERT in fast/events/stop-load-in-unload-handler-using-document-write.html
​https://bugs.webkit.org/show_bug.cgi?id=40268

Teach HTML5Tokenizer about m_parserStopped.

While tracking down how m_parserStopped was used, I found
several useless implementations of stopParsing() which I removed.

I also found a comment in HTMLTokenizer which still talks about the
"part" (an old class now split into Frame, Page, FrameLoader, etc.)
and an nearly empty FrameLoader::tokenizerProcessedData which I just
inlined into its one caller.

Tests:

fast/events/stop-load-in-unload-handler-using-document-write.html
http/tests/security/xssAuditor/full-block-base-href.html

  • html/HTML5Tokenizer.cpp: (WebCore::HTML5Tokenizer::pumpLexerIfPossible):
    • New method used instead of direct calls to pumpLexer for when callers would like to pump the lexer if possible, but don't want to check if the parser is stopped, etc.

(WebCore::HTML5Tokenizer::pumpLexer):

  • Added an ASSERT that the parser is not stopped.
  • Stop pumping if the parser is stopped.

(WebCore::HTML5Tokenizer::write):

  • Ignore the write if the parser is already stopped.

(WebCore::HTML5Tokenizer::end):
(WebCore::HTML5Tokenizer::attemptToEnd):

  • Move comment back to finish() which was incorrectly moved here.

(WebCore::HTML5Tokenizer::endIfDelayed):
(WebCore::HTML5Tokenizer::finish):

  • Move comment back here from attemptToEnd()

(WebCore::HTML5Tokenizer::resumeParsingAfterScriptExecution):

  • Remove a bogus ASSERT. The lexer will stop when it needs more data, which may not necessarily mean m_source.isEmpty().
  • html/HTML5Tokenizer.h:
    • Add pumpLexerIfPossible().
  • html/HTMLTokenizer.cpp: (WebCore::HTMLTokenizer::stopParsing):
    • Add a FIXME about if we really should call checkCompleted() here.
    • Inline FrameLoader::tokenizerProcessedData() since this was the only caller.
  • loader/FrameLoader.cpp:
    • Remove tokenizerProcessedData()
  • loader/FrameLoader.h:
    • Remove tokenizerProcessedData()
  • loader/MediaDocument.cpp:
    • Remove unneeded Tokenizer::stopParsing() override.
  • loader/PluginDocument.cpp:
    • Remove unneeded Tokenizer::stopParsing() override.
  • loader/SinkDocument.cpp:
    • Remove unneeded Tokenizer::stopParsing() override.
4:42 PM Changeset in webkit [60812] by crogers@google.com
  • 2 edits in branches/audio/WebCore/audio

Better support for HRIR composite generation

4:42 PM Changeset in webkit [60811] by crogers@google.com
  • 1 edit in branches/audio/WebCore/audio/mac/AudioFileWriterMac.cpp

Fix audio file writer code to properly close the file

4:42 PM Changeset in webkit [60810] by crogers@google.com
  • 241 adds in branches/audio/WebCore/audio/AudioSpatialization/IRC_Composite

Add composite HRIR impulse responses

4:41 PM Changeset in webkit [60809] by crogers@google.com
  • 1 add in branches/audio/WebCore/audio/AudioFileWriter.h

Still need AudioFileWriter.h header

4:41 PM Changeset in webkit [60808] by crogers@google.com
  • 1 edit in branches/audio/WebCore/audio/mac/FFTFrameMac.cpp

Fix order of initialization in FFTFrame copy constructor

4:41 PM Changeset in webkit [60807] by crogers@google.com
  • 1 move
    1 add
    1 delete in branches/audio/WebCore/audio

Move AudioFileWriter to mac/AudioFileWriterMac

  • for testing purposes only
  • simplify implementation
4:25 PM Changeset in webkit [60806] by eric@webkit.org
  • 2 edits in trunk/WebKit/qt

2010-06-07 Anders Bakken <​agbakken@gmail.com>

Reviewed by David Levin.

[Qt] NotificationPresenterClientQt.cpp has coding-style errors
​https://bugs.webkit.org/show_bug.cgi?id=40256

  • WebCoreSupport/NotificationPresenterClientQt.cpp: (WebCore::NotificationPresenterClientQt::show):
3:45 PM Changeset in webkit [60805] by Martin Robinson
  • 5 edits
    5 adds in trunk

2010-06-07 Martin Robinson <​mrobinson@igalia.com>

Reviewed by Oliver Hunt.

[GTK] HTML5 input type=range is not drawn
​https://bugs.webkit.org/show_bug.cgi?id=39561

Uncomment and add baselines for slider tests which are now passing. Group
remaining failing slider tests by their type of failure.

  • platform/gtk/Skipped:
  • platform/gtk/fast/forms/range-thumb-height-percentage-expected.txt: Added.
  • platform/gtk/fast/forms/slider-thumb-shared-style-expected.txt: Added.
  • platform/gtk/fast/forms/slider-thumb-stylability-expected.txt: Added.
  • platform/gtk/fast/forms/thumbslider-crash-expected.txt: Added.
  • platform/gtk/fast/forms/thumbslider-no-parent-slider-expected.txt: Added.

2010-06-07 Adam Barth <​abarth@webkit.org>

Reviewed by Eric Seidel.

Fix XFrameOptions and xssAuditor crashes in HTML5 parser
​https://bugs.webkit.org/show_bug.cgi?id=40265

We're not supposed to call end() while the tokenizer's write() method.
This causes a bunch of LayoutTests to crash. In particular, this patch
fixes crashes in the following tests:

Tests:

  • http/tests/security/XFrameOptions/x-frame-options-deny-meta-tag-in-body.html
  • http/tests/security/XFrameOptions/x-frame-options-deny-meta-tag-parent-same-origin-deny.html
  • http/tests/security/XFrameOptions/x-frame-options-deny-meta-tag.html
  • http/tests/security/xssAuditor/full-block-base-href.html
  • http/tests/security/xssAuditor/full-block-get-from-iframe.html
  • http/tests/security/xssAuditor/full-block-iframe-javascript-url.html
  • http/tests/security/xssAuditor/full-block-link-onclick.html
  • http/tests/security/xssAuditor/full-block-post-from-iframe.html
  • http/tests/security/xssAuditor/full-block-script-tag.html
  • http/tests/security/xssAuditor/xss-protection-parsing-01.html
  • html/HTML5Tokenizer.cpp: (WebCore::): (WebCore::HTML5Tokenizer::HTML5Tokenizer): (WebCore::HTML5Tokenizer::write): (WebCore::HTML5Tokenizer::attemptToEnd): (WebCore::HTML5Tokenizer::endIfDelayed): (WebCore::HTML5Tokenizer::finish): (WebCore::HTML5Tokenizer::resumeParsingAfterScriptExecution):
  • html/HTML5Tokenizer.h: (WebCore::HTML5Tokenizer::inWrite):
3:32 PM Changeset in webkit [60804] by Martin Robinson
  • 2 edits in trunk/LayoutTests

2010-06-07 Martin Robinson <​mrobinson@igalia.com>

Unreviewed.

[GTK] r60803 broke media/controls-drag-timebar.html
​https://bugs.webkit.org/show_bug.cgi?id=40269

  • platform/gtk/Skipped: Skip test which is now failing.
3:13 PM Changeset in webkit [60803] by Martin Robinson
  • 2 edits in trunk/WebKitTools

2010-06-07 Martin Robinson <​mrobinson@igalia.com>

Reviewed by Oliver Hunt.

[GTK] Mouse movement should not trigger a replay of saved events in the event sender
​https://bugs.webkit.org/show_bug.cgi?id=40267

When a mouse button is down, automatically queue mouse motion events and
do not play them back until the mouse button is released. This matches the
behavior of other ports' EventSenders.

  • DumpRenderTree/gtk/EventSender.cpp: (mouseMoveToCallback): Don't automatically replaySavedEvents for mouse motion. (sendOrQueueEvent): Add a shouldReplaySavedEvents argument defaulting to true.
2:53 PM Changeset in webkit [60802] by abarth@webkit.org
  • 3 edits in trunk/WebCore

2010-06-07 Adam Barth <​abarth@webkit.org>

Reviewed by Eric Seidel.

Fix XFrameOptions and xssAuditor crashes in HTML5 parser
​https://bugs.webkit.org/show_bug.cgi?id=40265

We're not supposed to call end() while the tokenizer's write() method.
This causes a bunch of LayoutTests to crash. In particular, this patch
fixes crashes in the following tests:

Tests:

  • http/tests/security/XFrameOptions/x-frame-options-deny-meta-tag-in-body.html
  • http/tests/security/XFrameOptions/x-frame-options-deny-meta-tag-parent-same-origin-deny.html
  • http/tests/security/XFrameOptions/x-frame-options-deny-meta-tag.html
  • http/tests/security/xssAuditor/full-block-base-href.html
  • http/tests/security/xssAuditor/full-block-get-from-iframe.html
  • http/tests/security/xssAuditor/full-block-iframe-javascript-url.html
  • http/tests/security/xssAuditor/full-block-link-onclick.html
  • http/tests/security/xssAuditor/full-block-post-from-iframe.html
  • http/tests/security/xssAuditor/full-block-script-tag.html
  • http/tests/security/xssAuditor/xss-protection-parsing-01.html
  • html/HTML5Tokenizer.cpp: (WebCore::): (WebCore::HTML5Tokenizer::HTML5Tokenizer): (WebCore::HTML5Tokenizer::write): (WebCore::HTML5Tokenizer::attemptToEnd): (WebCore::HTML5Tokenizer::endIfDelayed): (WebCore::HTML5Tokenizer::finish): (WebCore::HTML5Tokenizer::resumeParsingAfterScriptExecution):
  • html/HTML5Tokenizer.h: (WebCore::HTML5Tokenizer::inWrite):
2:40 PM Changeset in webkit [60801] by eric@webkit.org
  • 2 edits in trunk/WebCore

2010-06-07 Eric Seidel <​eric@webkit.org>

Reviewed by Adam Barth.

HTML5 parser hits ASSERT in http/tests/misc/window-dot-stop.html
​https://bugs.webkit.org/show_bug.cgi?id=40264

  • html/HTML5Tokenizer.cpp: (WebCore::HTML5Tokenizer::executeScript):
2:13 PM Changeset in webkit [60800] by kinuko@chromium.org
  • 2 edits in trunk/WebCore

2010-06-07 Kinuko Yasuda <​kinuko@chromium.org>

Unreviewed.

Avoid "warning: no previous prototype for" in Snow Leopard compilation.

  • platform/BlobItem.cpp: Added static to getFileSnapshotModificationTime().
1:55 PM Changeset in webkit [60799] by kinuko@chromium.org
  • 24 edits
    2 adds in trunk

2010-06-07 Kinuko Yasuda <​kinuko@chromium.org>

Reviewed by Jian Li.

Refactor FormData and Blob for better support of Blobs synthesized by BlobBuilder.
​https://bugs.webkit.org/show_bug.cgi?id=39083

Replace FormDataList::Item list with BlobItemList to get it compiled
with the refactoring in FormDataList.

  • src/WebSearchableFormData.cpp: (WebCore::HasSuitableTextElement):

2010-06-07 Kinuko Yasuda <​kinuko@chromium.org>

Reviewed by Jian Li.

Refactor FormData and Blob for better support of Blobs synthesized by BlobBuilder.
​https://bugs.webkit.org/show_bug.cgi?id=39083

  • Introduces a new class BlobItem as a basic component of Blob and FormDataList.
  • File would become a special type of Blob that contains only one FileBlobItem.
  • Fix the dependency violation in FormData so that the files under platform/ do not include any html/ files.

The patch doesn't support the latest File API changes (e.g. type,
url and slice's type parameters) and should not change any of its
existing behaviors.

The existing tests should be able to be used for regression.

  • CMakeLists.txt:
  • GNUmakefile.am:
  • WebCore.gypi:
  • WebCore.pro:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • html/Blob.cpp: (WebCore::Blob::Blob): (WebCore::Blob::size): (WebCore::Blob::path): (WebCore::Blob::append): (WebCore::Blob::slice):
  • html/Blob.h: (WebCore::Blob::create): (WebCore::Blob::type): (WebCore::Blob::items): (WebCore::Blob::Blob):
  • html/File.cpp: (WebCore::File::File): (WebCore::File::name):
  • html/File.h: (WebCore::File::fileName):
  • html/FileReader.cpp: (WebCore::FileReader::readAsBinaryString): (WebCore::FileReader::readAsText):
  • html/FileStream.cpp: (WebCore::FileStream::openForRead):
  • html/FormDataList.cpp: (WebCore::FormDataList::appendString): Moved the line ending fix logic to StringBlobItem::convertToCString. (WebCore::FormDataList::appendBlob):
  • html/FormDataList.h: (WebCore::FormDataList::items):
  • html/HTMLFormElement.cpp: (WebCore::HTMLFormElement::createFormData):
  • html/HTMLProgressElement.cpp:
  • platform/BlobItem.cpp: Added.
  • platform/BlobItem.h: Added.
  • platform/network/FormData.cpp: (WebCore::FormDataElement::FormDataElement): (WebCore::FormData::create): (WebCore::FormData::createMultiPart): (WebCore::FormData::deepCopy): (WebCore::FormData::appendData): (WebCore::FormData::appendString): (WebCore::FormData::appendFile): (WebCore::FormData::appendFileRange): (WebCore::FormData::appendItems): (WebCore::FormData::appendItem): (WebCore::FormData::appendKeyValuePairItems):
  • platform/network/FormData.h: (WebCore::operator!=):
  • platform/network/mac/FormDataStreamMac.mm: (WebCore::closeCurrentStream): (WebCore::advanceCurrentStream): (WebCore::formCreate): (WebCore::formRead): (WebCore::setHTTPBody):
  • xml/XMLHttpRequest.cpp: (WebCore::XMLHttpRequest::send):
1:14 PM Changeset in webkit [60798] by eric@webkit.org
  • 5 edits in trunk/WebCore

2010-06-07 Vangelis Kokkevis <​vangelis@chromium.org>

Reviewed by Dimitri Glazkov.

[chromium] Decoupled transform matrix update of composited layers from
drawing and interposed a sorting step so that the layers are now composited
back to front to get transparency working correctly. Also added missing code
for the preserves3D CSS property.

  • platform/graphics/chromium/GraphicsLayerChromium.cpp: (WebCore::GraphicsLayerChromium::updateLayerPreserves3D):
  • platform/graphics/chromium/LayerChromium.h: (WebCore::LayerChromium::setDrawTransform): (WebCore::LayerChromium::drawTransform): (WebCore::LayerChromium::setDrawOpacity): (WebCore::LayerChromium::drawOpacity): (WebCore::LayerChromium::preserves3D):
  • platform/graphics/chromium/LayerRendererChromium.cpp: (WebCore::compareLayerZ): (WebCore::LayerRendererChromium::drawLayers): (WebCore::LayerRendererChromium::updateLayersRecursive): (WebCore::LayerRendererChromium::drawLayer):
  • platform/graphics/chromium/LayerRendererChromium.h:
1:01 PM Changeset in webkit [60797] by eric@webkit.org
  • 2 edits in trunk/WebCore

2010-06-07 Anton Muhin <​antonm@chromium.org>

Reviewed by Nate Chapin.

[Chromium] convert a key to AtomicString in V8NodeList::namedPropertyGetter
​https://bugs.webkit.org/show_bug.cgi?id=40238

Comparison to "length" below could be performed faster in this (major) case
as it would only require direct pointer comparison (cf. to full text
comparison if key is converted to WebCore::String.)

  • bindings/v8/custom/V8NodeListCustom.cpp: (WebCore::V8NodeList::namedPropertyGetter):
12:27 PM Changeset in webkit [60796] by eric@webkit.org
  • 2 edits in trunk/WebCore

2010-06-07 Ariya Hidayat <​ariya.hidayat@codeaurora.org>

Reviewed by Kenneth Rohde Christiansen.

[Qt] Fix GraphicsContext::clipOut to take the transformation matrix into account.

  • platform/graphics/qt/GraphicsContextQt.cpp: (WebCore::GraphicsContext::clipOut):
12:22 PM Changeset in webkit [60795] by kov@webkit.org
  • 2 edits in trunk/LayoutTests

2010-06-07 Gustavo Noronha Silva <​gustavo.noronha@collabora.co.uk>

Unreviewed.

[gtk] web fonts not loaded properly in scribd html5 reader
​https://bugs.webkit.org/show_bug.cgi?id=38758

Also reset the expected result for the dynamic test for
backslash/yen sign.

  • platform/gtk/fast/text/backslash-to-yen-sign-dynamic-expected.txt:
12:08 PM Changeset in webkit [60794] by kov@webkit.org
  • 2 adds in trunk/WebCore/platform/graphics/cairo

Adding files I forgot last commit.

12:03 PM Changeset in webkit [60793] by kov@webkit.org
  • 5 edits in trunk

WebCore

2010-06-07 Gustavo Noronha Silva <​gustavo.noronha@collabora.co.uk>

Reviewed by Xan Lopez.

[gtk] web fonts not loaded properly in scribd html5 reader
​https://bugs.webkit.org/show_bug.cgi?id=38758

Avoid always giving a positive reply in createFontPlatformData. We
do not want every non-existing family name to be considered a hit,
or weird usage of local sources in @font-face rules will make
WebKitGTK+ never consider the online fonts.

  • GNUmakefile.am:
  • platform/graphics/cairo/FontCacheCairo.cpp: (WebCore::isWellKnownFontName): (WebCore::FontCache::createFontPlatformData):
  • platform/graphics/cairo/GOwnPtrCairo.cpp: Added. (WTF::FcPattern): (WTF::FcObjectSet): (WTF::FcFontSet):
  • platform/graphics/cairo/GOwnPtrCairo.h: Added.

LayoutTest

2010-06-07 Gustavo Noronha Silva <​gustavo.noronha@collabora.co.uk>

Reviewed by Xan Lopez.

[gtk] web fonts not loaded properly in scribd html5 reader
​https://bugs.webkit.org/show_bug.cgi?id=38758

Reset expected result for test that uses fonts we do not really
support, and are not "web fonts".

  • platform/gtk/fast/text/backslash-to-yen-sign-expected.txt:
11:54 AM Changeset in webkit [60792] by crogers@google.com
  • 1 edit in branches/audio/WebCore/audio/HTMLXAudioElement.cpp

More windows porting work - get rid of usleep() - courtesy Ian Lewis

11:51 AM Changeset in webkit [60791] by abarth@webkit.org
  • 2 edits in trunk/WebCore

2010-06-07 Adam Barth <​abarth@webkit.org>

Reviewed by Eric Seidel.

"Fix" fast/dom/stripNullFromTextNodes.html in HTML5 parser
​https://bugs.webkit.org/show_bug.cgi?id=40200

The proper handling of null characters is to replace them with U+FFFD.
This patch "fixes" fast/dom/stripNullFromTextNodes.html in the sense
that it makes it give the proper output w.r.t. HTML5. However, that
doesn't match the behavior of the old parser, which just stripped the
nulls.

  • html/HTML5Lexer.h: (WebCore::HTML5Lexer::InputStreamPreprocessor::peek):
11:34 AM Changeset in webkit [60790] by abarth@webkit.org
  • 3 edits in trunk/WebCore

2010-06-07 Adam Barth <​abarth@webkit.org>

Reviewed by Eric Seidel.

HTML5 parser should normalize line endings
​https://bugs.webkit.org/show_bug.cgi?id=40199

When reading characters from the source, we need to normalize line
endings, which means treating \r as \n and folding \r\n sequences into
a single \n. Doing this requires a slightly tighter pattern for
advancing the source (to make sure we update our state machine
correctly).

Fixes fast/css/first-child-pseudo-class.html and hundreds of other
LayoutTests in the HTML5 parser.

  • html/HTML5Lexer.cpp: (WebCore::HTML5Lexer::nextToken):
  • html/HTML5Lexer.h: (WebCore::HTML5Lexer::InputStreamPreprocessor::LineEndingNormalizer):
    • Introduced InputStreamPreprocessor to keep track of our line endings state machine. This class will grow to eventually do all the input stream preprocessing required by the HTML5 spec.

(WebCore::HTML5Lexer::InputStreamPreprocessor::nextInputCharacter):

  • We need to be careful not to read *source more than once because we need to know whether we've transformed the character. Keeping the next input character in this state machine requires some careful coding, but seems advantageous.

(WebCore::HTML5Lexer::InputStreamPreprocessor::peek):
(WebCore::HTML5Lexer::InputStreamPreprocessor::advance):

10:53 AM Changeset in webkit [60789] by andreip@google.com
  • 4 edits in trunk/WebCore

2010-06-07 Andrei Popescu <​andreip@google.com>

Reviewed by Jeremy Orlow.

run-bindings-tests results broken by Changeset 60776
​https://bugs.webkit.org/show_bug.cgi?id=40230

The change to CodeGeneratorJS.pm in
​http://trac.webkit.org/changeset/60776/trunk/WebCore/bindings/scripts/CodeGeneratorJS.pm
added the possibility to use [CallWith=ScriptExecutionContext] in idl files.

Before the above change was submitted, CodeGeneratorJS.pm was modified so that native
functions return EncodedJSValue instead of JSValues. This was done in
​http://trac.webkit.org/changeset/60631/trunk/WebCore/bindings/scripts/CodeGeneratorJS.pm

This CL updates the CodeGeneratorJS.pm to correctly return an EncodedJSValue in the case
where the ScriptExecutionContext pointer is 0 at the time when the native function is invoked.

No new tests, just fixing run-bindings-tests.

  • bindings/scripts/CodeGeneratorJS.pm:
  • bindings/scripts/test/JS/JSTestObj.cpp: (WebCore::jsTestObjPrototypeFunctionWithScriptExecutionContext):
  • bindings/scripts/test/JS/JSTestObj.h:
10:50 AM Changeset in webkit [60788] by eric@webkit.org
  • 1 edit
    1 add in trunk/WebCore

2010-06-07 Kwang Yul Seo <​skyul@company100.net>

Reviewed by Eric Seidel.

[BREWMP] Add dummy SearchPopupMenu implementation
​https://bugs.webkit.org/show_bug.cgi?id=40224

Add dummy implementation of SearchPopupMenu.

  • platform/brew/SearchPopupMenuBrew.cpp: Added. (WebCore::SearchPopupMenu::saveRecentSearches): (WebCore::SearchPopupMenu::loadRecentSearches): (WebCore::SearchPopupMenu::SearchPopupMenu): (WebCore::SearchPopupMenu::enabled):
10:34 AM Changeset in webkit [60787] by eric@webkit.org
  • 3 edits in trunk/JavaScriptCore

2010-06-07 Kwang Yul Seo <​skyul@company100.net>

Reviewed by Eric Seidel.

[BREWMP] Add more types to OwnPtr
​https://bugs.webkit.org/show_bug.cgi?id=39667

Add ISSL and ISocket to the list of OwnPtr-ed type.

  • wtf/OwnPtrCommon.h:
  • wtf/brew/OwnPtrBrew.cpp: (WTF::deleteOwnedPtr):
8:26 AM Changeset in webkit [60786] by jchaffraix@webkit.org
  • 9 edits
    22 deletes in trunk

Unreviewed, reverting my HTML comment parsing change as it
breaks Gtk and Qt.

8:02 AM Changeset in webkit [60785] by Martin Robinson
  • 10 edits in trunk

2010-06-07 Martin Robinson <​mrobinson@igalia.com>

Reviewed by Xan Lopez.

[GTK] gtk_widget_get_window should replace widget->window
​https://bugs.webkit.org/show_bug.cgi?id=40180

Replace all uses of widget->window with gtk_widget_get_window. For older
GTK+ versions, #define gtk_widget_get_window in GtkVersioning.h.

No tests necessary as functionality has not changed.

  • platform/gtk/GtkPluginWidget.cpp: (WebCore::GtkPluginWidget::invalidateRect): Replace widget->window use.
  • platform/gtk/GtkVersioning.h: Add gtk_widget_get_window for old GTK+ versions.
  • platform/gtk/PlatformScreenGtk.cpp: (WebCore::getVisual): Replace widget->window use. (WebCore::screenRect): Ditto.
  • platform/gtk/PopupMenuGtk.cpp: (WebCore::PopupMenu::show): Ditto.
  • platform/gtk/WidgetGtk.cpp: (WebCore::gdkDrawable): Ditto. (WebCore::Widget::setCursor): Ditto.

2010-06-07 Martin Robinson <​mrobinson@igalia.com>

Reviewed by Xan Lopez.

[GTK] gtk_widget_get_window should replace widget->window
​https://bugs.webkit.org/show_bug.cgi?id=40180

Replace uses of widget->window with gtk_widget_get_window.

  • WebCoreSupport/ChromeClientGtk.cpp: (WebKit::ChromeClient::invalidateContentsAndWindow): Replace widget->window uses. (WebKit::ChromeClient::scroll): Ditto. (WebKit::widgetScreenPosition): Ditto.
  • WebCoreSupport/DragClientGtk.cpp: Move gtk_widget_get_window define to GtkVersioning.h.
  • webkit/webkitwebview.cpp: (webkit_web_view_popup_menu_handler): Replace widget->window uses. (webkit_web_view_button_press_event): Ditto.
7:47 AM Changeset in webkit [60784] by Nikolas Zimmermann
  • 21 edits
    23 adds in trunk

2010-06-07 Nikolas Zimmermann <​nzimmermann@rim.com>

Reviewed by Sam Weinig.

Add CPP bindings generator
​https://bugs.webkit.org/show_bug.cgi?id=38279

Add CPP bindings generator used in Olympia platform.
We can succesfully generate cpp bindings for all IDLs except:

  • DOMWindow.idl / EventTarget.idl (filtered out in the IDL list in DerivedSources.make for non-jsc/v8 generators anyways, so no problem at all)
  • inspector/ (not needed for us at all)
  • svg/ (needs custom handling, related to animated attributes)

All generated files are free of style issues: "Total errors found: 0 in 510 files"
Not integrated in any build system so far, though I've successfully tested on Mac and inside Olympia, of course.

  • bindings/cpp: Added.
  • bindings/cpp/WebDOMCString.cpp: Added. (WebDOMCString::reset): (WebDOMCString::assign): (WebDOMCString::length): (WebDOMCString::data): (WebDOMCString::utf16): (WebDOMCString::fromUTF16): (WebDOMCString::WebDOMCString): (WebDOMCString::operator=): (WebDOMCString::operator WTF::CString):
  • bindings/cpp/WebDOMCString.h: Added. (WebDOMCString::~WebDOMCString): (WebDOMCString::WebDOMCString): (WebDOMCString::operator=): (WebDOMCString::isEmpty): (WebDOMCString::isNull):
  • bindings/cpp/WebDOMEventListenerCustom.cpp: Added. (WebDOMEventListener::handleEvent): (toWebKit):
  • bindings/cpp/WebDOMEventTarget.cpp: Added. (WebDOMEventTarget::WebDOMEventTargetPrivate::WebDOMEventTargetPrivate): (WebDOMEventTarget::WebDOMEventTarget): (WebDOMEventTarget::~WebDOMEventTarget): (WebDOMEventTarget::impl): (toWebCore): (toWebKit):
  • bindings/cpp/WebDOMEventTarget.h: Added.
  • bindings/cpp/WebDOMHTMLDocumentCustom.cpp: Added. (documentWrite): (WebDOMHTMLDocument::write): (WebDOMHTMLDocument::writeln):
  • bindings/cpp/WebDOMNodeCustom.cpp: Added. (WebDOMNode::insertBefore): (WebDOMNode::replaceChild): (WebDOMNode::removeChild): (WebDOMNode::appendChild): (WebDOMNode::addEventListener): (WebDOMNode::removeEventListener):
  • bindings/cpp/WebDOMObject.h: Added. (WebDOMObject::WebDOMObject): (WebDOMObject::~WebDOMObject):
  • bindings/cpp/WebDOMString.cpp: Added. (WebDOMString::reset): (WebDOMString::assign): (WebDOMString::length): (WebDOMString::data): (WebDOMString::utf8): (WebDOMString::fromUTF8): (WebDOMString::WebDOMString): (WebDOMString::operator=): (WebDOMString::operator WebCore::String): (WebDOMString::operator WebCore::AtomicString): (WebDOMString::equals):
  • bindings/cpp/WebDOMString.h: Added. (WebDOMString::~WebDOMString): (WebDOMString::WebDOMString): (WebDOMString::operator=): (WebDOMString::isEmpty): (WebDOMString::isNull):
  • bindings/cpp/WebExceptionHandler.cpp: Added. (globalExceptionHandler): (webInstallExceptionHandler): (webRaiseDOMException):
  • bindings/cpp/WebExceptionHandler.h: Added. (webDOMRaiseError):
  • bindings/cpp/WebNativeEventListener.cpp: Added. (WebNativeEventListener::WebNativeEventListener): (WebNativeEventListener::~WebNativeEventListener): (WebNativeEventListener::handleEvent): (WebNativeEventListener::reportError): (WebNativeEventListener::operator==):
  • bindings/cpp/WebNativeEventListener.h: Added. (WebNativeEventListener::create): (WebNativeEventListener::cast):
  • bindings/scripts/CodeGeneratorCPP.pm: Added.
  • bindings/scripts/test/CPP: Added.
  • bindings/scripts/test/CPP/WebDOMTestCallback.cpp: Added. (WebDOMTestCallback::WebDOMTestCallbackPrivate::WebDOMTestCallbackPrivate): (WebDOMTestCallback::WebDOMTestCallback): (WebDOMTestCallback::impl): (WebDOMTestCallback::~WebDOMTestCallback): (WebDOMTestCallback::callbackWithClass1Param): (WebDOMTestCallback::callbackWithClass2Param): (WebDOMTestCallback::callbackWithNonBoolReturnType): (toWebCore): (toWebKit):
  • bindings/scripts/test/CPP/WebDOMTestCallback.h: Added.
  • bindings/scripts/test/CPP/WebDOMTestInterface.cpp: Added. (WebDOMTestInterface::WebDOMTestInterfacePrivate::WebDOMTestInterfacePrivate): (WebDOMTestInterface::WebDOMTestInterface): (WebDOMTestInterface::impl): (WebDOMTestInterface::~WebDOMTestInterface): (toWebCore): (toWebKit):
  • bindings/scripts/test/CPP/WebDOMTestInterface.h: Added.
  • bindings/scripts/test/CPP/WebDOMTestObj.cpp: Added. (WebDOMTestObj::WebDOMTestObjPrivate::WebDOMTestObjPrivate): (WebDOMTestObj::WebDOMTestObj): (WebDOMTestObj::impl): (WebDOMTestObj::~WebDOMTestObj): (WebDOMTestObj::readOnlyIntAttr): (WebDOMTestObj::readOnlyStringAttr): (WebDOMTestObj::readOnlyTestObjAttr): (WebDOMTestObj::intAttr): (WebDOMTestObj::setIntAttr): (WebDOMTestObj::longLongAttr): (WebDOMTestObj::setLongLongAttr): (WebDOMTestObj::unsignedLongLongAttr): (WebDOMTestObj::setUnsignedLongLongAttr): (WebDOMTestObj::stringAttr): (WebDOMTestObj::setStringAttr): (WebDOMTestObj::testObjAttr): (WebDOMTestObj::setTestObjAttr): (WebDOMTestObj::attrWithException): (WebDOMTestObj::setAttrWithException): (WebDOMTestObj::attrWithSetterException): (WebDOMTestObj::setAttrWithSetterException): (WebDOMTestObj::attrWithGetterException): (WebDOMTestObj::setAttrWithGetterException): (WebDOMTestObj::scriptStringAttr): (WebDOMTestObj::voidMethod): (WebDOMTestObj::voidMethodWithArgs): (WebDOMTestObj::intMethod): (WebDOMTestObj::intMethodWithArgs): (WebDOMTestObj::objMethod): (WebDOMTestObj::objMethodWithArgs): (WebDOMTestObj::methodThatRequiresAllArgs): (WebDOMTestObj::methodThatRequiresAllArgsAndThrows): (WebDOMTestObj::serializedValue): (WebDOMTestObj::methodWithException): (WebDOMTestObj::addEventListener): (WebDOMTestObj::removeEventListener): (WebDOMTestObj::withDynamicFrame): (WebDOMTestObj::withDynamicFrameAndArg): (WebDOMTestObj::withDynamicFrameAndOptionalArg): (WebDOMTestObj::withScriptStateVoid): (WebDOMTestObj::withScriptStateObj): (WebDOMTestObj::withScriptStateVoidException): (WebDOMTestObj::withScriptStateObjException): (WebDOMTestObj::methodWithOptionalArg): (WebDOMTestObj::methodWithNonOptionalArgAndOptionalArg): (WebDOMTestObj::methodWithNonOptionalArgAndTwoOptionalArgs): (toWebCore): (toWebKit):
  • bindings/scripts/test/CPP/WebDOMTestObj.h: Added.
  • css/StyleSheet.idl: Export helper method isCSSStyleSheet() for cpp bindings only.
  • dom/CustomEvent.idl: Disabled for cpp bindings, needs arbitary input parameter handling support ('DOMObject' type).
  • dom/Document.idl: Ignore getCSSCanvasContext() for cpp bindings and export helper method isHTMLDocument() (cpp bindings only).
  • dom/Event.idl: Export isMutationEvent()/isMouseEvent()/isUIEvent() helpers for cpp bindings only.
  • dom/EventListener.h: Add CPPEventListenerType.
  • dom/Node.idl: addEventListener/removeEventListener need [Custom] flag for cpp bindings.
  • dom/PopStateEvent.idl: Disabled just like CustomEvent, for the same reason ("any" argument type handling).
  • html/HTMLCanvasElement.idl: Disable getContext() method for cpp bindings, as it's done for objc bindings.
  • html/HTMLDocument.idl: Add [NoCPPCustom] marker for open(), as we're just autogenerating it.
  • html/HTMLInputElement.idl: Disable valueAsDate() method for cpp bindings, as we're missing native 'Date' type support.
  • html/TextMetrics.h: Add missing PassRefPtr.h include.
  • html/canvas/CanvasRenderingContext2D.idl: Disable methods (setLineCap, etc..) for cpp bindings whose names clash with property setters.
  • page/AbstractView.idl: Rename 'Media' type to 'StyleMedia' - this should have been done before.
  • page/Location.idl: Disable several methods which would need a custom implementation for the cpp bindings.
  • page/WebKitPoint.h: Add missing PassRefPtr.h include.
  • storage/SQLResultSet.idl: Correct type from 'long' to 'long long', for cpp bindings only though as it wouldn't build on Chromium/V8.
  • workers/DedicatedWorkerContext.idl: Disable postMessage() for now when using cpp bindings, would need custom code for us.
  • workers/WorkerContext.idl: Hide constructors from cpp bindings.

2010-06-07 Nikolas Zimmermann <​nzimmermann@rim.com>

Reviewed by Sam Weinig.

Add CPP bindings generator
​https://bugs.webkit.org/show_bug.cgi?id=38279

Integrate CPP bindings in run-bindings-tests.

  • Scripts/run-bindings-tests:
7:43 AM Changeset in webkit [60783] by eric@webkit.org
  • 1 edit
    1 add in trunk/WebKitSite

2010-06-07 Jeremy Moskovich <​jeremy@chromium.org>

Reviewed by Darin Adler.

Add documentation on the various assertion macros in WebKit.

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

  • coding/assertion-guidelines.html: Added.
7:43 AM Changeset in webkit [60782] by robert@webkit.org
  • 2 edits in trunk/LayoutTests

2010-06-07 Robert Hogan <​robert@webkit.org>

Reviewed by Kenneth Rohde Christiansen.

[Qt] Put some structure on the Qt Skipped file

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

  • platform/qt/Skipped:
7:41 AM Changeset in webkit [60781] by jchaffraix@webkit.org
  • 9 edits
    22 adds in trunk

WebCore: Space should be allowed between -- and > in comment end
​https://bugs.webkit.org/show_bug.cgi?id=21945

Reviewed by Alexey Proskuryakov.

Implemented the HTML comment parsing algorithm so that we match HTML5 and
FF when parsing comments. Missing from this patch is
the parser errors, which will be added in a follow up patch.

Added tests cases for broken comments.

Tests: fast/parser/broken-comment-1.html

fast/parser/broken-comment-2.html
fast/parser/broken-comment-3.html
fast/parser/broken-comment-4.html
fast/parser/broken-comment-5.html
fast/parser/broken-comment-6.html
fast/parser/broken-comment-in-head-1.html
fast/parser/broken-comment-in-head-2.html
fast/parser/broken-comment-in-head-3.html
fast/parser/broken-comment-in-head-4.html
fast/parser/broken-comment-in-head-5.html

  • html/HTMLTokenizer.cpp:

(WebCore::HTMLTokenizer::parseComment): Now we use a state machine
that matches the HTML5 specification.
(WebCore::HTMLTokenizer::emitCommentToken): This emits the comment token
and keep some of the original behavior as parseComment is called in wrong
context (inside a <title> for example). Added a more explicit comment about
this.
(WebCore::HTMLTokenizer::parseTag): Remove the handling of <!--> in quirks mode.
HTML5 specifies that we should accept this in strict mode too.

  • html/HTMLTokenizer.h:

(WebCore::HTMLTokenizer::): Added a new enum for the comment parsing state.

LayoutTests: Tests for:
Space should be allowed between -- and > in comment end
​https://bugs.webkit.org/show_bug.cgi?id=21945

Reviewed by Alexey Proskuryakov.

  • fast/css/word-space-extra.html: Tweaked the comment as -- > is now

a valid comment end.

  • fast/parser/broken-comment-1-expected.txt: Added.
  • fast/parser/broken-comment-1.html: Added.
  • fast/parser/broken-comment-2-expected.txt: Added.
  • fast/parser/broken-comment-2.html: Added.
  • fast/parser/broken-comment-3-expected.txt: Added.
  • fast/parser/broken-comment-3.html: Added.
  • fast/parser/broken-comment-4-expected.txt: Added.
  • fast/parser/broken-comment-4.html: Added.
  • fast/parser/broken-comment-5-expected.txt: Added.
  • fast/parser/broken-comment-5.html: Added.
  • fast/parser/broken-comment-6-expected.txt: Added.
  • fast/parser/broken-comment-6.html: Added.

Those test checks that broken comments in body are correctly parsed.

  • fast/parser/broken-comment-in-head-1-expected.txt: Added.
  • fast/parser/broken-comment-in-head-1.html: Added.
  • fast/parser/broken-comment-in-head-2-expected.txt: Added.
  • fast/parser/broken-comment-in-head-2.html: Added.
  • fast/parser/broken-comment-in-head-3-expected.txt: Added.
  • fast/parser/broken-comment-in-head-3.html: Added.
  • fast/parser/broken-comment-in-head-4-expected.txt: Added.
  • fast/parser/broken-comment-in-head-4.html: Added.
  • fast/parser/broken-comment-in-head-5-expected.txt: Added.
  • fast/parser/broken-comment-in-head-5.html: Added.

Those test checks that broken comments in head are correctly parsed.

  • fast/parser/broken-comments-vs-parsing-mode.html: Needed to wrap the broken comment

in a title as we match FF here and would not recover from it.

  • fast/parser/comments-expected.txt:
  • fast/parser/comments.html:

Added more tests for comment parsing. Made sure that we close empty comments
<!--> and <!---> as is specified in HTML5.

  • html5lib/runner-expected.txt: Removed the test that we now pass.
7:29 AM Changeset in webkit [60780] by eric@webkit.org
  • 2 edits in trunk/WebCore

2010-06-07 Mahesh Kulakrni <​mahesh.kulkarni@nokia.com>

Reviewed by Simon Hausmann.

[QT] QT_BEARER flag is not enabled on S60 properly
​https://bugs.webkit.org/show_bug.cgi?id=39357

enable QT_BEARER for all platform based on qtmobility +
bearer module availability or for qt 4.7+

  • WebCore.pri:
6:59 AM Changeset in webkit [60779] by andreip@google.com
  • 1 edit in trunk/WebKitTools/Scripts/webkitpy/common/config/committers.py

Unreviewed, add Andrei Popescu to committers list.

6:56 AM Changeset in webkit [60778] by tonikitoo@webkit.org
  • 2 edits in trunk/WebKit/qt

Expose the editing behavior setting in DRT to test all editing code paths ​https://bugs.webkit.org/show_bug.cgi?id=39680

Patch by Antonio Gomes <​tonikitoo@webkit.org>, Laszlo Gombos <Laszlo Gombos> on 2010-06-06
Reviewed by Kenneth Christiansen, Eric Seidel.

Make setEditingBehavior() a no-operation if the editingBehavior
argument is not recognized to avoid using an uninitialized variable.

  • WebCoreSupport/DumpRenderTreeSupportQt.cpp:

(DumpRenderTreeSupportQt::setEditingBehavior):

6:40 AM WebKit Team edited by andreip@google.com
Added myself to list of Committers. (diff)
5:32 AM Changeset in webkit [60777] by eric@webkit.org
  • 2 edits in trunk/WebCore

2010-06-07 Leon Clarke <​leonclarke@google.com>

Reviewed by Jeremy Orlow.

Fix indexeddb idls in android makefile
​https://bugs.webkit.org/show_bug.cgi?id=40169

No new tests. Android-specific build fix.

  • Android.derived.v8bindings.mk:
5:01 AM Changeset in webkit [60776] by andreip@google.com
  • 48 edits
    6 copies
    1 move in trunk

2010-06-07 Andrei Popescu <​andreip@google.com>

Reviewed by Jeremy Orlow.

[indexedDB] It is impossible to create object stores
​https://bugs.webkit.org/show_bug.cgi?id=40054

No new tests. Layout tests will be added separately.

  • Android.mk:
  • CMakeLists.txt:
  • GNUmakefile.am:
  • WebCore.gypi:
  • WebCore.pro:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • bindings/scripts/CodeGeneratorJS.pm:
  • bindings/scripts/CodeGeneratorV8.pm:
  • bindings/scripts/test/GObject/WebKitDOMTestObj.cpp: (webkit_dom_test_obj_with_script_execution_context):
  • bindings/scripts/test/GObject/WebKitDOMTestObj.h:
  • bindings/scripts/test/JS/JSTestObj.cpp: (WebCore::): (WebCore::jsTestObjPrototypeFunctionWithScriptExecutionContext):
  • bindings/scripts/test/JS/JSTestObj.h:
  • bindings/scripts/test/ObjC/DOMTestObj.h:
  • bindings/scripts/test/ObjC/DOMTestObj.mm: (-[DOMTestObj withScriptExecutionContext]):
  • bindings/scripts/test/TestObj.idl:
  • bindings/scripts/test/V8/V8TestObj.cpp: (WebCore::TestObjInternal::withScriptExecutionContextCallback): (WebCore::):
  • page/DOMWindow.cpp: (WebCore::DOMWindow::clear): (WebCore::DOMWindow::indexedDB):
  • storage/IDBCallbacks.h:
  • storage/IDBDatabase.h:
  • storage/IDBDatabaseImpl.cpp: (WebCore::IDBDatabaseImpl::createObjectStore):
  • storage/IDBDatabaseImpl.h:
  • storage/IDBDatabaseRequest.cpp: (WebCore::IDBDatabaseRequest::IDBDatabaseRequest): (WebCore::IDBDatabaseRequest::createObjectStore):
  • storage/IDBDatabaseRequest.h:
  • storage/IDBDatabaseRequest.idl:
  • storage/IDBObjectStore.cpp: Removed.
  • storage/IDBObjectStore.h: (WebCore::IDBObjectStore::~IDBObjectStore):
  • storage/IDBObjectStoreImpl.cpp: Added. (WebCore::IDBObjectStoreImpl::~IDBObjectStoreImpl): (WebCore::IDBObjectStoreImpl::IDBObjectStoreImpl): (WebCore::IDBObjectStoreImpl::indexNames): (WebCore::IDBObjectStoreImpl::createIndex): (WebCore::IDBObjectStoreImpl::index): (WebCore::IDBObjectStoreImpl::removeIndex):
  • storage/IDBObjectStoreImpl.h: Added. (WebCore::IDBObjectStoreImpl::create): (WebCore::IDBObjectStoreImpl::name): (WebCore::IDBObjectStoreImpl::keyPath):
  • storage/IDBObjectStoreRequest.cpp: (WebCore::IDBObjectStoreRequest::IDBObjectStoreRequest):
  • storage/IDBObjectStoreRequest.h: (WebCore::IDBObjectStoreRequest::create):
  • storage/IDBRequest.cpp: (WebCore::IDBRequest::onSuccess):
  • storage/IDBRequest.h:
  • storage/IndexedDatabaseRequest.cpp: (WebCore::IndexedDatabaseRequest::IndexedDatabaseRequest): (WebCore::IndexedDatabaseRequest::open):
  • storage/IndexedDatabaseRequest.h: (WebCore::IndexedDatabaseRequest::create):
  • storage/IndexedDatabaseRequest.idl:

2010-06-07 Andrei Popescu <​andreip@google.com>

Reviewed by Jeremy Orlow.

[indexedDB] It is impossible to create object stores
​https://bugs.webkit.org/show_bug.cgi?id=40054

  • WebKit.gyp:
  • public/WebIDBCallbacks.h: (WebKit::WebIDBCallbacks::onSuccess):
  • public/WebIDBDatabase.h: (WebKit::WebIDBDatabase::createObjectStore):
  • public/WebIDBObjectStore.h: Added. (WebKit::WebIDBObjectStore::~WebIDBObjectStore): (WebKit::WebIDBObjectStore::name): (WebKit::WebIDBObjectStore::keyPath):
  • src/IDBCallbacksProxy.cpp: (WebCore::IDBCallbacksProxy::onSuccess):
  • src/IDBCallbacksProxy.h:
  • src/IDBDatabaseProxy.cpp: (WebCore::IDBDatabaseProxy::createObjectStore):
  • src/IDBDatabaseProxy.h:
  • src/IDBObjectStoreProxy.cpp: Added. (WebCore::IDBObjectStoreProxy::create): (WebCore::IDBObjectStoreProxy::IDBObjectStoreProxy): (WebCore::IDBObjectStoreProxy::~IDBObjectStoreProxy): (WebCore::IDBObjectStoreProxy::name): (WebCore::IDBObjectStoreProxy::keyPath): (WebCore::IDBObjectStoreProxy::indexNames): (WebCore::IDBObjectStoreProxy::createIndex): (WebCore::IDBObjectStoreProxy::index): (WebCore::IDBObjectStoreProxy::removeIndex):
  • src/IDBObjectStoreProxy.h: Added.
  • src/IndexedDatabaseProxy.cpp: (WebCore::IndexedDatabaseProxy::open):
  • src/WebIDBCallbacksImpl.cpp: (WebCore::WebIDBCallbacksImpl::onSuccess):
  • src/WebIDBCallbacksImpl.h:
  • src/WebIDBDatabaseImpl.cpp: (WebKit::WebIDBDatabaseImpl::createObjectStore):
  • src/WebIDBDatabaseImpl.h:
  • src/WebIDBObjectStoreImpl.cpp: Added. (WebKit::WebIDBObjectStoreImpl::WebIDBObjectStoreImpl): (WebKit::WebIDBObjectStoreImpl::~WebIDBObjectStoreImpl): (WebKit::WebIDBObjectStoreImpl::name): (WebKit::WebIDBObjectStoreImpl::keyPath):
  • src/WebIDBObjectStoreImpl.h: Added.
  • src/WebIndexedDatabaseImpl.cpp: (WebKit::WebIndexedDatabaseImpl::open):
4:57 AM QtWebKitTodo edited by Simon Hausmann
(diff)
4:48 AM Changeset in webkit [60775] by Nikolas Zimmermann
  • 4 edits in trunk/WebCore

2010-06-07 Nikolas Zimmermann <​nzimmermann@rim.com>

Not reviewed. Updated run-bindings-tests results.

  • bindings/scripts/test/JS/JSTestInterface.cpp: (WebCore::JSTestInterfaceConstructor::constructTestInterface):
  • bindings/scripts/test/JS/JSTestObj.cpp: (WebCore::jsTestObjPrototypeFunctionVoidMethod): (WebCore::jsTestObjPrototypeFunctionVoidMethodWithArgs): (WebCore::jsTestObjPrototypeFunctionIntMethod): (WebCore::jsTestObjPrototypeFunctionIntMethodWithArgs): (WebCore::jsTestObjPrototypeFunctionObjMethod): (WebCore::jsTestObjPrototypeFunctionObjMethodWithArgs): (WebCore::jsTestObjPrototypeFunctionMethodThatRequiresAllArgs): (WebCore::jsTestObjPrototypeFunctionMethodThatRequiresAllArgsAndThrows): (WebCore::jsTestObjPrototypeFunctionSerializedValue): (WebCore::jsTestObjPrototypeFunctionMethodWithException): (WebCore::jsTestObjPrototypeFunctionCustomMethod): (WebCore::jsTestObjPrototypeFunctionCustomMethodWithArgs): (WebCore::jsTestObjPrototypeFunctionCustomArgsAndException): (WebCore::jsTestObjPrototypeFunctionAddEventListener): (WebCore::jsTestObjPrototypeFunctionRemoveEventListener): (WebCore::jsTestObjPrototypeFunctionWithDynamicFrame): (WebCore::jsTestObjPrototypeFunctionWithDynamicFrameAndArg): (WebCore::jsTestObjPrototypeFunctionWithDynamicFrameAndOptionalArg): (WebCore::jsTestObjPrototypeFunctionWithDynamicFrameAndUserGesture): (WebCore::jsTestObjPrototypeFunctionWithDynamicFrameAndUserGestureASAD): (WebCore::jsTestObjPrototypeFunctionWithScriptStateVoid): (WebCore::jsTestObjPrototypeFunctionWithScriptStateObj): (WebCore::jsTestObjPrototypeFunctionWithScriptStateVoidException): (WebCore::jsTestObjPrototypeFunctionWithScriptStateObjException): (WebCore::jsTestObjPrototypeFunctionMethodWithOptionalArg): (WebCore::jsTestObjPrototypeFunctionMethodWithNonOptionalArgAndOptionalArg): (WebCore::jsTestObjPrototypeFunctionMethodWithNonOptionalArgAndTwoOptionalArgs): (WebCore::jsTestObjPrototypeFunctionOverloadedMethod1): (WebCore::jsTestObjPrototypeFunctionOverloadedMethod2): (WebCore::jsTestObjPrototypeFunctionOverloadedMethod3): (WebCore::jsTestObjPrototypeFunctionOverloadedMethod4): (WebCore::jsTestObjPrototypeFunctionOverloadedMethod):
  • bindings/scripts/test/JS/JSTestObj.h:
4:45 AM QtWebKitTodo edited by Simon Hausmann
(diff)
4:40 AM QtWebKitTodo edited by Simon Hausmann
(diff)
4:33 AM Changeset in webkit [60774] by eric@webkit.org
  • 3 edits in trunk/WebCore

2010-06-07 Pavel Podivilov <​podivilov@chromium.org>

Reviewed by Pavel Feldman.

Web Inspector: Implement JSON parsing for InspectorValue.
​https://bugs.webkit.org/show_bug.cgi?id=40064

  • inspector/InspectorValues.cpp: (WebCore::): (WebCore::InspectorValue::asBool): (WebCore::InspectorValue::asNumber): (WebCore::InspectorValue::asString): (WebCore::InspectorValue::asObject): (WebCore::InspectorValue::asArray): (WebCore::InspectorValue::readJSON): (WebCore::InspectorValue::writeJSON): (WebCore::InspectorBasicValue::asBool): (WebCore::InspectorBasicValue::asNumber): (WebCore::InspectorBasicValue::writeJSON): (WebCore::InspectorString::asString): (WebCore::InspectorObject::asObject): (WebCore::InspectorObject::getBool): (WebCore::InspectorObject::getNumber): (WebCore::InspectorObject::getString): (WebCore::InspectorObject::getObject): (WebCore::InspectorObject::getArray): (WebCore::InspectorObject::get): (WebCore::InspectorArray::asArray):
  • inspector/InspectorValues.h: (WebCore::InspectorObject::begin): (WebCore::InspectorObject::end):
4:24 AM Changeset in webkit [60773] by Simon Hausmann
  • 2 edits in trunk/JavaScriptCore

[Qt] Crash when compiling on Snow Leopard and running on Leopard
​https://bugs.webkit.org/show_bug.cgi?id=31403

Patch by Benjamin Poulain <​benjamin.poulain@nokia.com> on 2010-06-07
Reviewed by Simon Hausmann.

Disable the use of pthread_setname_np and other symbols
when targetting Leopard.

Use the defines TARGETING_XX instead of BUILDING_ON_XX
for features that cannot be used before Snow Leopard.

  • wtf/Platform.h:
3:36 AM Changeset in webkit [60772] by jocelyn.turcotte@nokia.com
  • 2 edits in trunk/WebCore

[Qt] Fix text selection drawing.
​https://bugs.webkit.org/show_bug.cgi?id=40221

Reviewed by Simon Hausmann.

The regression was introduced in r60169.

  • platform/graphics/qt/FontQt.cpp:

(WebCore::drawTextCommon):

2:57 AM Changeset in webkit [60771] by yurys@chromium.org
  • 18 edits in trunk

2010-06-07 Yury Semikhatsky <​yurys@chromium.org>

Reviewed by Pavel Feldman.

Web Inspector: should be possible to distinguish extension scripts from main world scripts
​https://bugs.webkit.org/show_bug.cgi?id=40220

  • bindings/js/ScriptDebugServer.cpp: remove global listeners set which is not used anymore. (WebCore::ScriptDebugServer::dispatchDidParseSource): pass script wrold type to the listeners. (WebCore::currentWorldType): (WebCore::ScriptDebugServer::sourceParsed):
  • bindings/js/ScriptDebugServer.h:
  • bindings/v8/ScriptDebugServer.cpp: (WebCore::ScriptDebugServer::dispatchDidParseSource):
  • inspector/InspectorController.cpp: (WebCore::InspectorController::didParseSource):
  • inspector/InspectorController.h:
  • inspector/InspectorFrontend.cpp: (WebCore::InspectorFrontend::parsedScriptSource):
  • inspector/InspectorFrontend.h:
  • inspector/ScriptDebugListener.h: pass type of the isolated world where the script was compiled to didParseSource. (WebCore::):
  • inspector/front-end/InjectedScript.js: (injectedScriptConstructor):
  • inspector/front-end/Script.js: (WebInspector.Script):
  • inspector/front-end/ScriptsPanel.js: (WebInspector.ScriptsPanel.prototype.addScript): (WebInspector.ScriptsPanel.prototype._addScriptToFilesMenu): use different style to highlight content scripts.
  • inspector/front-end/inspector.css: (#scripts-files option.extension-script):
  • inspector/front-end/inspector.js: (WebInspector.parsedScriptSource):

2010-06-07 Yury Semikhatsky <​yurys@chromium.org>

Reviewed by Pavel Feldman.

Web Inspector: should be possible to distinguish extension scripts from main world scripts
​https://bugs.webkit.org/show_bug.cgi?id=40220

  • src/js/DebuggerAgent.js: (devtools.DebuggerAgent.prototype.initUI): (devtools.DebuggerAgent.prototype.addScriptInfo_): (devtools.DebuggerAgent.prototype.formatCallFrame_): (devtools.ScriptInfo.prototype.worldType):
  • src/js/DebuggerScript.js: (debuggerScriptConstructor.DebuggerScript.getScripts): use only context id without context type when filtering scripts so that extension scripts are also included. (debuggerScriptConstructor.DebuggerScript._formatScript):
  • src/js/devTools.css:
1:58 AM Changeset in webkit [60770] by loki@webkit.org
  • 2 edits in trunk/JavaScriptCore
  • jit/JITOpcodes32_64.cpp:

(JSC::JIT::privateCompileCTINativeCall):

Reviewed by NOBODY (JSVALUE32_64 build fix).

Jun 6, 2010:

9:53 PM Changeset in webkit [60769] by staikos@webkit.org
  • 1 edit in trunk/WebCore/platform/qt/WidgetQt.cpp

Unreviewed: Since 2006 I learned out to spell my name.

8:29 PM Changeset in webkit [60768] by morrita@google.com
  • 2 edits in trunk/WebCore

2010-06-06 MORITA Hajime <​morrita@google.com>

Unreviewd, Chromium windows build fix.

  • rendering/RenderThemeChromiumWin.cpp: (WebCore::RenderThemeChromiumWin::paintProgressBar):
8:07 PM Changeset in webkit [60767] by morrita@google.com
  • 7 edits
    3 adds in trunk

2010-06-06 MORITA Hajime <​morrita@google.com>

Reviewed by Kent Tamura.

ASSERTION FAILED with -webkit-appearance:progress-bar for non-progress bar elements
​https://bugs.webkit.org/show_bug.cgi?id=40158

  • fast/css/invalid-appearance-progress-bar-meter-expected.txt: Added.
  • fast/css/invalid-appearance-progress-bar-meter.html: Added.
  • fast/css/script-tests/invalid-appearance-progress-bar-meter.js: Added.

2010-06-06 MORITA Hajime <​morrita@google.com>

Reviewed by Kent Tamura.

ASSERTION FAILED with -webkit-appearance:progress-bar for non <progress> elements
​https://bugs.webkit.org/show_bug.cgi?id=40158

paintProgressBar() and paintMeter() assumed given RenderObject is
RenderProgress or RenderMeter respectively, but arbitrary elements
can have -webkit-appearance: progress-bar and such elements
violates that assumption. So this change added type check before
downcasting the RenderObject.

  • platform/qt/RenderThemeQt.cpp: (WebCore::RenderThemeQt::paintProgressBar):
  • rendering/RenderTheme.cpp: (WebCore::RenderTheme::paintMeter):
  • rendering/RenderThemeChromiumSkia.cpp: (WebCore::RenderThemeChromiumSkia::paintProgressBar):
  • rendering/RenderThemeChromiumWin.cpp: (WebCore::RenderThemeChromiumWin::paintProgressBar):
  • rendering/RenderThemeMac.mm: (WebCore::RenderThemeMac::paintProgressBar):
5:07 PM Changeset in webkit [60766] by barraclough@apple.com
  • 2 edits in trunk/JavaScriptCore

Reviewed by NOBODY (windows build fix pt 2).

4:54 PM Changeset in webkit [60765] by barraclough@apple.com
  • 3 edits in trunk/WebCore
  • bridge/qt/qt_runtime.cpp:

(JSC::Bindings::findMethodIndex):
(JSC::Bindings::QtRuntimeMetaMethod::call):
(JSC::Bindings::QtRuntimeConnectionMethod::call):

Reviewed by NOBODY (Qt build fix pt 2).

4:50 PM Changeset in webkit [60764] by barraclough@apple.com
  • 2 edits in trunk/WebCore
  • bridge/qt/qt_runtime.cpp:

(JSC::Bindings::findMethodIndex):
(JSC::Bindings::QtRuntimeMetaMethod::call):
(JSC::Bindings::QtRuntimeConnectionMethod::call):

Reviewed by NOBODY (Qt build fix).

4:46 PM Changeset in webkit [60763] by barraclough@apple.com
  • 3 edits in trunk/JavaScriptCore

Reviewed by NOBODY (windows build fix pt 1).

4:34 PM Changeset in webkit [60762] by barraclough@apple.com
  • 90 edits in trunk

Bug 40214 - Clean up error construction / throwing in JSC.

Reviewed by Sam Weinig.

The one egregious insanity here is that creating an error requires
a VM-entry-esqe-host call (the string argument is wrapped as a JS
object & pushed on the RegisterFile, then unwrapped back to a
UString). Changing this also means you only require a global
object, not an ExecState, to create an error.

The methods to create error objects are also parameterized
requiring a switch on the type, which can be made cleaner and
faster by moving to a separate method per error type. Code to add
divot information to error had been duplicated, and is coalesced
back into a single function.

Convenience methods added to create & throw type & syntax error
with a default error message, since this is a common case.

Also, errors are currently thrown either using
"throwError(exec, error)" or "exec->setException(error)" - unify
on the former, since this is more commonly used. Add
"throwVMError(exec, error)" equivalents, as a convenience for
cases where the result was being wrapped in "JSValue::encode(...)".

JavaScriptCore:

  • API/JSCallbackConstructor.cpp:

(JSC::constructJSCallback):

  • API/JSCallbackFunction.cpp:

(JSC::JSCallbackFunction::call):

  • API/JSCallbackObjectFunctions.h:

(JSC::::getOwnPropertySlot):
(JSC::::put):
(JSC::::deleteProperty):
(JSC::::construct):
(JSC::::hasInstance):
(JSC::::call):
(JSC::::toNumber):
(JSC::::toString):
(JSC::::staticValueGetter):
(JSC::::staticFunctionGetter):
(JSC::::callbackGetter):

  • API/JSObjectRef.cpp:

(JSObjectMakeError):

(JSC::BytecodeGenerator::emitNewError):
(JSC::BytecodeGenerator::emitThrowExpressionTooDeepException):

  • bytecompiler/BytecodeGenerator.h:
  • bytecompiler/NodesCodegen.cpp:

(JSC::ThrowableExpressionData::emitThrowError):
(JSC::RegExpNode::emitBytecode):
(JSC::PostfixErrorNode::emitBytecode):
(JSC::PrefixErrorNode::emitBytecode):
(JSC::AssignErrorNode::emitBytecode):
(JSC::ForInNode::emitBytecode):
(JSC::ContinueNode::emitBytecode):
(JSC::BreakNode::emitBytecode):
(JSC::ReturnNode::emitBytecode):
(JSC::LabelNode::emitBytecode):

  • interpreter/CallFrame.h:
  • interpreter/Interpreter.cpp:

(JSC::Interpreter::throwException):
(JSC::Interpreter::privateExecute):

  • jit/JITStubs.cpp:

(JSC::DEFINE_STUB_FUNCTION):

  • jsc.cpp:

(functionRun):
(functionLoad):
(functionCheckSyntax):

  • parser/Nodes.h:
  • runtime/ArrayConstructor.cpp:

(JSC::constructArrayWithSizeQuirk):

  • runtime/ArrayPrototype.cpp:

(JSC::arrayProtoFuncToString):
(JSC::arrayProtoFuncToLocaleString):
(JSC::arrayProtoFuncJoin):
(JSC::arrayProtoFuncFilter):
(JSC::arrayProtoFuncMap):
(JSC::arrayProtoFuncEvery):
(JSC::arrayProtoFuncForEach):
(JSC::arrayProtoFuncSome):
(JSC::arrayProtoFuncReduce):
(JSC::arrayProtoFuncReduceRight):

  • runtime/BooleanPrototype.cpp:

(JSC::booleanProtoFuncToString):
(JSC::booleanProtoFuncValueOf):

  • runtime/DatePrototype.cpp:

(JSC::dateProtoFuncToString):
(JSC::dateProtoFuncToUTCString):
(JSC::dateProtoFuncToISOString):
(JSC::dateProtoFuncToDateString):
(JSC::dateProtoFuncToTimeString):
(JSC::dateProtoFuncToLocaleString):
(JSC::dateProtoFuncToLocaleDateString):
(JSC::dateProtoFuncToLocaleTimeString):
(JSC::dateProtoFuncGetTime):
(JSC::dateProtoFuncGetFullYear):
(JSC::dateProtoFuncGetUTCFullYear):
(JSC::dateProtoFuncToGMTString):
(JSC::dateProtoFuncGetMonth):
(JSC::dateProtoFuncGetUTCMonth):
(JSC::dateProtoFuncGetDate):
(JSC::dateProtoFuncGetUTCDate):
(JSC::dateProtoFuncGetDay):
(JSC::dateProtoFuncGetUTCDay):
(JSC::dateProtoFuncGetHours):
(JSC::dateProtoFuncGetUTCHours):
(JSC::dateProtoFuncGetMinutes):
(JSC::dateProtoFuncGetUTCMinutes):
(JSC::dateProtoFuncGetSeconds):
(JSC::dateProtoFuncGetUTCSeconds):
(JSC::dateProtoFuncGetMilliSeconds):
(JSC::dateProtoFuncGetUTCMilliseconds):
(JSC::dateProtoFuncGetTimezoneOffset):
(JSC::dateProtoFuncSetTime):
(JSC::setNewValueFromTimeArgs):
(JSC::setNewValueFromDateArgs):
(JSC::dateProtoFuncSetMilliSeconds):
(JSC::dateProtoFuncSetUTCMilliseconds):
(JSC::dateProtoFuncSetSeconds):
(JSC::dateProtoFuncSetUTCSeconds):
(JSC::dateProtoFuncSetMinutes):
(JSC::dateProtoFuncSetUTCMinutes):
(JSC::dateProtoFuncSetHours):
(JSC::dateProtoFuncSetUTCHours):
(JSC::dateProtoFuncSetDate):
(JSC::dateProtoFuncSetUTCDate):
(JSC::dateProtoFuncSetMonth):
(JSC::dateProtoFuncSetUTCMonth):
(JSC::dateProtoFuncSetFullYear):
(JSC::dateProtoFuncSetUTCFullYear):
(JSC::dateProtoFuncSetYear):
(JSC::dateProtoFuncGetYear):
(JSC::dateProtoFuncToJSON):

  • runtime/Error.cpp:

(JSC::createError):
(JSC::createEvalError):
(JSC::createRangeError):
(JSC::createReferenceError):
(JSC::createSyntaxError):
(JSC::createTypeError):
(JSC::createURIError):
(JSC::addErrorSourceInfo):
(JSC::addErrorDivotInfo):
(JSC::addErrorInfo):
(JSC::hasErrorInfo):
(JSC::throwError):
(JSC::throwTypeError):
(JSC::throwSyntaxError):

  • runtime/Error.h:

(JSC::throwVMError):
(JSC::throwVMTypeError):

  • runtime/ErrorConstructor.cpp:

(JSC::constructWithErrorConstructor):
(JSC::callErrorConstructor):

  • runtime/ErrorConstructor.h:
  • runtime/ErrorInstance.cpp:

(JSC::ErrorInstance::ErrorInstance):
(JSC::ErrorInstance::create):

  • runtime/ErrorInstance.h:
  • runtime/ErrorPrototype.cpp:

(JSC::ErrorPrototype::ErrorPrototype):

  • runtime/ExceptionHelpers.cpp:

(JSC::createStackOverflowError):
(JSC::createUndefinedVariableError):
(JSC::createInvalidParamError):
(JSC::createNotAConstructorError):
(JSC::createNotAFunctionError):
(JSC::createNotAnObjectError):
(JSC::throwOutOfMemoryError):

  • runtime/ExceptionHelpers.h:
  • runtime/Executable.cpp:

(JSC::EvalExecutable::compile):
(JSC::ProgramExecutable::checkSyntax):
(JSC::ProgramExecutable::compile):

  • runtime/FunctionConstructor.cpp:

(JSC::constructFunction):

  • runtime/FunctionPrototype.cpp:

(JSC::functionProtoFuncToString):
(JSC::functionProtoFuncApply):
(JSC::functionProtoFuncCall):

  • runtime/Identifier.cpp:

(JSC::Identifier::from):

  • runtime/Identifier.h:
  • runtime/JSArray.cpp:

(JSC::JSArray::put):

  • runtime/JSFunction.cpp:

(JSC::callHostFunctionAsConstructor):

  • runtime/JSGlobalObjectFunctions.cpp:

(JSC::encode):
(JSC::decode):
(JSC::globalFuncEval):

  • runtime/JSONObject.cpp:

(JSC::Stringifier::appendStringifiedValue):
(JSC::Walker::walk):
(JSC::JSONProtoFuncParse):
(JSC::JSONProtoFuncStringify):

  • runtime/JSObject.cpp:

(JSC::throwSetterError):
(JSC::JSObject::put):
(JSC::JSObject::putWithAttributes):
(JSC::JSObject::defaultValue):
(JSC::JSObject::hasInstance):
(JSC::JSObject::defineOwnProperty):

  • runtime/JSObject.h:
  • runtime/JSValue.cpp:

(JSC::JSValue::toObjectSlowCase):
(JSC::JSValue::synthesizeObject):
(JSC::JSValue::synthesizePrototype):

  • runtime/NativeErrorConstructor.cpp:

(JSC::constructWithNativeErrorConstructor):
(JSC::callNativeErrorConstructor):

  • runtime/NativeErrorConstructor.h:
  • runtime/NumberPrototype.cpp:

(JSC::numberProtoFuncToString):
(JSC::numberProtoFuncToLocaleString):
(JSC::numberProtoFuncValueOf):
(JSC::numberProtoFuncToFixed):
(JSC::numberProtoFuncToExponential):
(JSC::numberProtoFuncToPrecision):

  • runtime/ObjectConstructor.cpp:

(JSC::objectConstructorGetPrototypeOf):
(JSC::objectConstructorGetOwnPropertyDescriptor):
(JSC::objectConstructorGetOwnPropertyNames):
(JSC::objectConstructorKeys):
(JSC::toPropertyDescriptor):
(JSC::objectConstructorDefineProperty):
(JSC::objectConstructorDefineProperties):
(JSC::objectConstructorCreate):

  • runtime/ObjectPrototype.cpp:

(JSC::objectProtoFuncDefineGetter):
(JSC::objectProtoFuncDefineSetter):

  • runtime/RegExpConstructor.cpp:

(JSC::constructRegExp):

  • runtime/RegExpObject.cpp:

(JSC::RegExpObject::match):

  • runtime/RegExpPrototype.cpp:

(JSC::regExpProtoFuncTest):
(JSC::regExpProtoFuncExec):
(JSC::regExpProtoFuncCompile):
(JSC::regExpProtoFuncToString):

  • runtime/StringPrototype.cpp:

(JSC::stringProtoFuncToString):

WebCore:

  • WebCore.xcodeproj/project.pbxproj:
  • bindings/js/JSArrayBufferConstructor.h:

(WebCore::construct):

  • bindings/js/JSArrayBufferViewHelper.h:

(WebCore::setWebGLArrayHelper):

  • bindings/js/JSAudioConstructor.cpp:

(WebCore::constructAudio):

  • bindings/js/JSCanvasRenderingContext2DCustom.cpp:

(WebCore::JSCanvasRenderingContext2D::setFillColor):
(WebCore::JSCanvasRenderingContext2D::setStrokeColor):
(WebCore::JSCanvasRenderingContext2D::drawImage):
(WebCore::JSCanvasRenderingContext2D::drawImageFromRect):
(WebCore::JSCanvasRenderingContext2D::setShadow):
(WebCore::JSCanvasRenderingContext2D::createPattern):
(WebCore::JSCanvasRenderingContext2D::fillText):
(WebCore::JSCanvasRenderingContext2D::strokeText):

  • bindings/js/JSClipboardCustom.cpp:

(WebCore::JSClipboard::clearData):
(WebCore::JSClipboard::getData):
(WebCore::JSClipboard::setDragImage):

  • bindings/js/JSDOMBinding.cpp:

(WebCore::setDOMException):
(WebCore::toJSSequence):

  • bindings/js/JSDOMWrapper.cpp:

(WebCore::DOMObject::defineOwnProperty):

  • bindings/js/JSDesktopNotificationsCustom.cpp:

(WebCore::JSNotificationCenter::requestPermission):

  • bindings/js/JSEventSourceConstructor.cpp:

(WebCore::constructEventSource):

  • bindings/js/JSHTMLDocumentCustom.cpp:

(WebCore::JSHTMLDocument::open):

  • bindings/js/JSHTMLInputElementCustom.cpp:

(WebCore::JSHTMLInputElement::selectionStart):
(WebCore::JSHTMLInputElement::setSelectionStart):
(WebCore::JSHTMLInputElement::selectionEnd):
(WebCore::JSHTMLInputElement::setSelectionEnd):
(WebCore::JSHTMLInputElement::setSelectionRange):

  • bindings/js/JSImageConstructor.cpp:

(WebCore::constructImage):

  • bindings/js/JSJavaScriptCallFrameCustom.cpp:

(WebCore::JSJavaScriptCallFrame::evaluate):

  • bindings/js/JSMessageChannelConstructor.cpp:

(WebCore::JSMessageChannelConstructor::construct):

  • bindings/js/JSMessagePortCustom.cpp:

(WebCore::fillMessagePortArray):

  • bindings/js/JSOptionConstructor.cpp:

(WebCore::constructHTMLOptionElement):

  • bindings/js/JSSVGMatrixCustom.cpp:

(WebCore::JSSVGMatrix::multiply):

  • bindings/js/JSSharedWorkerConstructor.cpp:

(WebCore::constructSharedWorker):

  • bindings/js/JSWebGLRenderingContextCustom.cpp:

(WebCore::JSWebGLRenderingContext::bufferData):
(WebCore::JSWebGLRenderingContext::bufferSubData):
(WebCore::getObjectParameter):
(WebCore::JSWebGLRenderingContext::getFramebufferAttachmentParameter):
(WebCore::JSWebGLRenderingContext::getParameter):
(WebCore::JSWebGLRenderingContext::getProgramParameter):
(WebCore::JSWebGLRenderingContext::getShaderParameter):
(WebCore::JSWebGLRenderingContext::getUniform):
(WebCore::JSWebGLRenderingContext::texImage2D):
(WebCore::JSWebGLRenderingContext::texSubImage2D):
(WebCore::dataFunctionf):
(WebCore::dataFunctioni):
(WebCore::dataFunctionMatrix):

  • bindings/js/JSWebSocketConstructor.cpp:

(WebCore::constructWebSocket):

  • bindings/js/JSWebSocketCustom.cpp:

(WebCore::JSWebSocket::send):

  • bindings/js/JSWorkerConstructor.cpp:

(WebCore::constructWorker):

  • bindings/js/JSXMLHttpRequestConstructor.cpp:

(WebCore::constructXMLHttpRequest):

  • bindings/js/JSXMLHttpRequestCustom.cpp:

(WebCore::JSXMLHttpRequest::open):

  • bindings/js/SerializedScriptValue.cpp:

(WebCore::BaseWalker::throwStackOverflow):
(WebCore::BaseWalker::throwInterruptedException):
(WebCore::SerializingTreeWalker::startArray):
(WebCore::SerializingTreeWalker::startObject):

  • bindings/js/WorkerScriptController.cpp:

(WebCore::WorkerScriptController::setException):

  • bindings/scripts/CodeGeneratorJS.pm:
  • bridge/c/c_instance.cpp:

(JSC::Bindings::CInstance::moveGlobalExceptionToExecState):
(JSC::Bindings::CInstance::invokeMethod):
(JSC::Bindings::CInstance::invokeDefaultMethod):
(JSC::Bindings::CInstance::invokeConstruct):

  • bridge/jni/jsc/JNIBridgeJSC.cpp:

(JavaField::dispatchValueFromInstance):
(JavaField::dispatchSetValueToInstance):

  • bridge/jni/jsc/JavaInstanceJSC.cpp:

(JavaInstance::invokeMethod):

  • bridge/objc/objc_instance.mm:

(ObjcInstance::moveGlobalExceptionToExecState):
(ObjcInstance::invokeMethod):

  • bridge/objc/objc_runtime.mm:

(JSC::Bindings::ObjcField::valueFromInstance):
(JSC::Bindings::ObjcField::setValueToInstance):
(JSC::Bindings::ObjcArray::setValueAt):
(JSC::Bindings::ObjcArray::valueAt):
(JSC::Bindings::callObjCFallbackObject):

  • bridge/objc/objc_utility.h:
  • bridge/objc/objc_utility.mm:

(JSC::Bindings::throwError):

  • bridge/runtime_array.cpp:

(JSC::RuntimeArray::put):

  • bridge/runtime_method.cpp:

(JSC::callRuntimeMethod):

  • bridge/runtime_object.cpp:

(JSC::Bindings::RuntimeObject::throwInvalidAccessError):

WebKit/mac:

  • Plugins/Hosted/NetscapePluginInstanceProxy.mm:

(WebKit::NetscapePluginInstanceProxy::moveGlobalExceptionToExecState):

  • Plugins/Hosted/ProxyInstance.mm:

(WebKit::ProxyInstance::invokeMethod):

5:26 AM Changeset in webkit [60761] by krit@webkit.org
  • 18 edits
    3 adds in trunk

2010-06-06 Dirk Schulze <​krit@webkit.org>

Reviewed by Nikolas Zimmermann.

hit testing does not respect clip paths
​https://bugs.webkit.org/show_bug.cgi?id=15162

Test: svg/dynamic-updates/SVGClipPath-influences-hitTesting.html


Added a check, if a float point is not only in the shape/object boundaries
but also is not in the clipped away area of a clipPath.

  • rendering/HitTestRequest.h: (WebCore::HitTestRequest::): (WebCore::HitTestRequest::svgClipContent):
  • rendering/RenderPath.cpp: (WebCore::RenderPath::fillContains): (WebCore::RenderPath::nodeAtFloatPoint):
  • rendering/RenderPath.h:
  • rendering/RenderSVGContainer.cpp: (WebCore::RenderSVGContainer::nodeAtFloatPoint):
  • rendering/RenderSVGImage.cpp: (WebCore::RenderSVGImage::nodeAtFloatPoint):
  • rendering/RenderSVGResourceClipper.cpp: (WebCore::RenderSVGResourceClipper::hitTestClipContent):
  • rendering/RenderSVGResourceClipper.h:
  • rendering/RenderSVGText.cpp: (WebCore::RenderSVGText::nodeAtFloatPoint):
  • rendering/SVGRenderSupport.cpp: (WebCore::pointInClippingArea):
  • rendering/SVGRenderSupport.h:

2010-06-06 Dirk Schulze <​krit@webkit.org>

Reviewed by Nikolas Zimmermann.

hit testing does not respect clip paths
​https://bugs.webkit.org/show_bug.cgi?id=15162


Two tests were incorrect, because of the assumption, that clipPath doesn't
influence hit testing. They needed an update.
The new test checks, that clipped areas don't throw a hit.

  • svg/dynamic-updates/SVGClipPath-influences-hitTesting-expected.txt: Added.
  • svg/dynamic-updates/SVGClipPath-influences-hitTesting.html: Added.
  • svg/dynamic-updates/SVGClipPathElement-dom-clipPathUnits-attr-expected.txt:
  • svg/dynamic-updates/SVGClipPathElement-svgdom-clipPathUnits-prop-expected.txt:
  • svg/dynamic-updates/script-tests/SVGClipPath-influences-hitTesting.js: Added. (executeBackgroundTest): (executeTest):
  • svg/dynamic-updates/script-tests/SVGClipPathElement-dom-clipPathUnits-attr.js: (shouldBeEqualToString.executeTest):
  • svg/dynamic-updates/script-tests/SVGClipPathElement-svgdom-clipPathUnits-prop.js: (executeTest):
4:35 AM Changeset in webkit [60760] by krit@webkit.org
  • 7 edits in trunk/WebCore

2010-06-06 Dirk Schulze <​krit@webkit.org>

Reviewed by Nikolas Zimmermann.

SVG Pattern/Gradient clean-up
​https://bugs.webkit.org/show_bug.cgi?id=40205

Transformations of SVG Patterns and Gradients can be bundeled.
This calculation also needs just to be calculated once. It's not a great speed-up for most platforms,
but a clean-up and preparation for new features like vectorEffects and others.
Now, that we don't recalucluate the gradient and its transformations, it was necessary to add a way
to transform alreday existing gradients on Cairo. This is done in the same way like Skia transforms
gradients after they were created.


This patch doesn't change functionality, so no new tests added.

  • platform/graphics/Gradient.cpp:
  • platform/graphics/cairo/GradientCairo.cpp: (WebCore::Gradient::setPlatformGradientSpaceTransform):
  • rendering/RenderSVGResourceGradient.cpp: (WebCore::RenderSVGResourceGradient::applyResource):
  • rendering/RenderSVGResourceGradient.h:
  • rendering/RenderSVGResourcePattern.cpp: (WebCore::RenderSVGResourcePattern::applyResource): (WebCore::RenderSVGResourcePattern::createTileImage): (WebCore::RenderSVGResourcePattern::buildPattern):
  • rendering/RenderSVGResourcePattern.h:
1:04 AM Changeset in webkit [60759] by tkent@chromium.org
  • 1 edit
    4 adds in trunk/LayoutTests

Unreviewed, test expectation fix.

Add image results of tests added by r60752 for Chromium.

  • platform/chromium-linux/svg/custom/fill-opacity-hsl-expected.checksum: Added.
  • platform/chromium-linux/svg/custom/fill-opacity-hsl-expected.png: Added.
  • platform/chromium-linux/svg/custom/fill-opacity-rgba-expected.checksum: Added.
  • platform/chromium-linux/svg/custom/fill-opacity-rgba-expected.png: Added.
12:40 AM Changeset in webkit [60758] by tkent@chromium.org
  • 1 edit
    4 adds in trunk/LayoutTests

Unreviewed, test expectation fix.

Add image results of tests added by r60752 for Chromium.

  • platform/chromium-win/svg/custom/fill-opacity-hsl-expected.checksum: Added.
  • platform/chromium-win/svg/custom/fill-opacity-hsl-expected.png: Added.
  • platform/chromium-win/svg/custom/fill-opacity-rgba-expected.checksum: Added.
  • platform/chromium-win/svg/custom/fill-opacity-rgba-expected.png: Added.
Note: See TracTimeline for information about the timeline view.