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

Timeline



May 25, 2011:

11:58 PM Changeset in webkit [87364] by barraclough@apple.com
  • 3 edits in trunk/Source/JavaScriptCore

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

Reviewed by Sam Weinig.

Move the silent spill/fill methods in the DFG JIT to the JITCodeGenerator
so that they are available to the SpeculativeJIT.

  • dfg/DFGJITCodeGenerator.h:

(JSC::DFG::JITCodeGenerator::silentSpillGPR):
(JSC::DFG::JITCodeGenerator::silentSpillFPR):
(JSC::DFG::JITCodeGenerator::silentFillGPR):
(JSC::DFG::JITCodeGenerator::silentFillFPR):
(JSC::DFG::JITCodeGenerator::silentSpillAllRegisters):
(JSC::DFG::JITCodeGenerator::silentFillAllRegisters):

  • dfg/DFGNonSpeculativeJIT.h:
11:48 PM Changeset in webkit [87363] by jer.noble@apple.com
  • 7 edits in trunk

2011-05-25 Jer Noble <​jer.noble@apple.com>

Reviewed by Dan Bernstein.

30 second rewind button dysfunctional viewing trailers fullscreen.
​https://bugs.webkit.org/show_bug.cgi?id=61505

Add support for painting MediaReturnToRealtimeButton, as well as hide and show
the correct default video control buttons when entering full-screen mode.

  • css/fullscreenQuickTime.css: (video:-webkit-full-screen::-webkit-media-controls-return-to-realtime-button):
  • html/shadow/MediaControlRootElement.cpp: (WebCore::MediaControlRootElement::enteredFullscreen): Show the "Back 30s" and

"Return to Realtime" buttons for Live Streams in full-screen mode.

(WebCore::MediaControlRootElement::exitedFullscreen): Reset the styles of

all the buttons affected by enteredFullscreen().

  • rendering/RenderMediaControls.cpp: (WebCore::RenderMediaControls::paintMediaControlsPart): Add support for

MediaReturnToRealtimeButton.

2011-05-25 Jer Noble <​jer.noble@apple.com>

Reviewed by Dan Bernstein.

30 second rewind button dysfunctional viewing trailers fullscreen.
​https://bugs.webkit.org/show_bug.cgi?id=61505

Updated WKSI Libraries with new images for Backward buttons in default
video controls.

  • libWebKitSystemInterfaceLeopard.a:
  • libWebKitSystemInterfaceSnowLeopard.a:
11:46 PM Changeset in webkit [87362] by commit-queue@webkit.org
  • 4 edits in trunk/Source/WebCore

2011-05-25 Luke Macpherson <​macpherson@chromium.org>

Reviewed by Dimitri Glazkov.

Implement various CSS font properties in CSSStyleApplyProperty.
​https://bugs.webkit.org/show_bug.cgi?id=60603

No new tests - refactoring only.

  • css/CSSStyleApplyProperty.cpp: (WebCore::ApplyPropertyFont): Added templated class to handle font properties (WebCore::ApplyPropertyFontWeight): Added class to handle font weight calculation (WebCore::CSSStyleApplyProperty::CSSStyleApplyProperty): Instantiate ApplyPropertyFont/ApplyPropertyFontWeight for appropriate properties.
  • css/CSSStyleSelector.cpp: (WebCore::CSSStyleSelector::applyProperty): Remove properties that are now implemented in CSSStyleApplyProperty
  • css/CSSStyleSelector.h: (WebCore::CSSStyleSelector::fontDescription): Added getter for m_style->fontDescription() (WebCore::CSSStyleSelector::parentFontDescription): Added getter for m_parentStyle->fontDescription() (WebCore::CSSStyleSelector::setFontDescription): Added setter for m_style->setFontDescription() that automatically sets m_fontDirty
11:20 PM Changeset in webkit [87361] by simonjam@chromium.org
  • 2 edits in trunk/Source/WebCore

2011-05-25 James Simonsen <​simonjam@chromium.org>

Reviewed by Adam Barth.

Add site-specific hack for zipcar.com with old versions of requirejs.
​https://bugs.webkit.org/show_bug.cgi?id=61321

Old versions of requirejs (< 0.15.0) try to load scripts in parallel but execute them in
order. This used to work in webkit by setting a bogus script type (script/cache), then
changing the type to a valid one when they wanted to execute it. This hack translates the
behavior into the new API (by disabling forceAsync).

  • html/HTMLScriptElement.cpp: (WebCore::needsOldRequirejsQuirk): Added. (WebCore::HTMLScriptElement::insertedIntoDocument): If hack is needed, set a proper script type so script loads. If script isn't async, disable forceAsync so script executes in order.
11:15 PM Changeset in webkit [87360] by commit-queue@webkit.org
  • 14 edits in trunk/Source/WebCore

2011-05-25 Andreas Kling <​kling@webkit.org>

Reviewed by David Levin.

Rebaseline run-bindings-tests.
​https://bugs.webkit.org/show_bug.cgi?id=61469

  • bindings/scripts/test/JS/:
  • bindings/scripts/test/V8/:
10:32 PM Changeset in webkit [87359] by barraclough@apple.com
  • 12 edits in trunk/Source/JavaScriptCore

An attempt to revive Windows bots.

Patch by Ryosuke Niwa <​rniwa@webkit.org> on 2011-05-25

  • runtime/RegExp.cpp:
  • runtime/RegExp.h:
10:19 PM Changeset in webkit [87358] by rniwa@webkit.org
  • 3 edits in trunk/Source/JavaScriptCore

2011-05-25 Ryosuke Niwa <​rniwa@webkit.org>

An attempt to revive Windows bots.

  • runtime/RegExp.cpp:
  • runtime/RegExp.h:
9:16 PM Changeset in webkit [87357] by commit-queue@webkit.org
  • 2 edits in trunk/LayoutTests

2011-05-25 Igor Oliveira <​igor.oliveira@openbossa.org>

Reviewed by Antonio Gomes.

[Qt] REGRESSION(r87298) Skip drag-not-loaded-image.html failing test
​https://bugs.webkit.org/show_bug.cgi?id=61465

Skip drag-not-loaded-image.html layout test from Qt. see bug 31332

  • platform/qt/Skipped:
9:12 PM Changeset in webkit [87356] by barraclough@apple.com
  • 9 edits in trunk/Source/JavaScriptCore

Bug 61501 - Unify AbstractMacroAssembler::differenceBetween methods.

Reviewed by Sam Weinig.

  • assembler/AbstractMacroAssembler.h:

(JSC::AbstractMacroAssembler::Call::Call):
(JSC::AbstractMacroAssembler::Call::fromTailJump):
(JSC::AbstractMacroAssembler::Jump::Jump):
(JSC::AbstractMacroAssembler::Jump::link):
(JSC::AbstractMacroAssembler::Jump::linkTo):
(JSC::AbstractMacroAssembler::Jump::isSet):
(JSC::AbstractMacroAssembler::differenceBetween):
(JSC::AbstractMacroAssembler::linkJump):
(JSC::AbstractMacroAssembler::getLinkerCallReturnOffset):

  • assembler/LinkBuffer.h:

(JSC::LinkBuffer::link):
(JSC::LinkBuffer::locationOf):
(JSC::LinkBuffer::locationOfNearCall):
(JSC::LinkBuffer::returnAddressOffset):

  • assembler/MacroAssemblerARM.h:

(JSC::MacroAssemblerARM::linkCall):

  • assembler/MacroAssemblerARMv7.h:

(JSC::MacroAssemblerARMv7::linkCall):

  • assembler/MacroAssemblerMIPS.h:

(JSC::MacroAssemblerMIPS::linkCall):

  • assembler/MacroAssemblerSH4.cpp:

(JSC::MacroAssemblerSH4::linkCall):

  • assembler/MacroAssemblerX86.h:

(JSC::MacroAssemblerX86::linkCall):

  • assembler/MacroAssemblerX86_64.h:

(JSC::MacroAssemblerX86_64::linkCall):

9:05 PM Changeset in webkit [87355] by barraclough@apple.com
  • 4 edits in trunk/Source/JavaScriptCore

​https://bugs.webkit.org/show_bug.cgi?id=61500
Add JSObject::offsetOfPropertyStorage

Reviewed by Sam Weinig.

  • jit/JITPropertyAccess.cpp:

(JSC::JIT::compileGetDirectOffset):
(JSC::JIT::compileGetByIdHotPath):
(JSC::JIT::emit_op_put_by_id):
(JSC::JIT::compilePutDirectOffset):

  • jit/JITPropertyAccess32_64.cpp:

(JSC::JIT::compileGetByIdHotPath):
(JSC::JIT::emit_op_put_by_id):
(JSC::JIT::compilePutDirectOffset):
(JSC::JIT::compileGetDirectOffset):

  • runtime/JSObject.h:

(JSC::JSObject::offsetOfPropertyStorage):

8:50 PM Changeset in webkit [87354] by commit-queue@webkit.org
  • 26 edits in trunk/LayoutTests

2011-05-25 Steve Lacey <​sjl@chromium.org>

Reviewed by Eric Carlson.

Rebaseline media tests after moving media controls to flexbox layout.

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

  • platform/chromium-mac/fast/layers/video-layer-expected.txt:
  • platform/chromium-mac/media/audio-controls-rendering-expected.txt:
  • platform/chromium-mac/media/audio-repaint-expected.txt:
  • platform/chromium-mac/media/controls-after-reload-expected.txt:
  • platform/chromium-mac/media/controls-strict-expected.txt:
  • platform/chromium-mac/media/controls-styling-expected.txt:
  • platform/chromium-mac/media/controls-without-preload-expected.txt:
  • platform/chromium-mac/media/video-controls-rendering-expected.txt:
  • platform/chromium-mac/media/video-display-toggle-expected.txt:
  • platform/chromium-mac/media/video-empty-source-expected.txt:
  • platform/chromium-mac/media/video-no-audio-expected.txt:
  • platform/chromium-mac/media/video-volume-slider-expected.txt:
  • platform/chromium-win/fast/layers/video-layer-expected.txt:
  • platform/chromium-win/media/audio-controls-rendering-expected.txt:
  • platform/chromium-win/media/audio-repaint-expected.txt:
  • platform/chromium-win/media/controls-after-reload-expected.txt:
  • platform/chromium-win/media/controls-strict-expected.txt:
  • platform/chromium-win/media/controls-styling-expected.txt:
  • platform/chromium-win/media/controls-without-preload-expected.txt:
  • platform/chromium-win/media/video-controls-rendering-expected.txt:
  • platform/chromium-win/media/video-display-toggle-expected.txt:
  • platform/chromium-win/media/video-empty-source-expected.txt:
  • platform/chromium-win/media/video-no-audio-expected.txt:
  • platform/chromium-win/media/video-volume-slider-expected.txt:
  • platform/chromium/test_expectations.txt:
8:32 PM Changeset in webkit [87353] by commit-queue@webkit.org
  • 5 edits in trunk

2011-05-25 Gregg Tavares <Gregg Tavares>

Reviewed by Kenneth Russell.

Add 'noExtensions' option to Web/GraphicsContext3D attributes.
​https://bugs.webkit.org/show_bug.cgi?id=61460

  • Source/WebCore/html/canvas/WebGLRenderingContext.cpp:
  • Source/WebCore/platform/graphics/GraphicsContext3D.h:
  • Source/WebKit/chromium/public/WebGraphicsContext3D.h:
  • Source/WebKit/chromium/src/GraphicsContext3DChromium.cpp:
8:13 PM Changeset in webkit [87352] by leviw@chromium.org
  • 5 edits in trunk/Source/WebCore

2011-05-25 Levi Weintraub <​leviw@chromium.org>

Reviewed by Eric Seidel.

Switch itemBoundingBoxRect to use IntPoint
​https://bugs.webkit.org/show_bug.cgi?id=60789

Changing itemBoundingBoxRect to use an IntPoint representing the offset
to be added tot he box rect instead of a pair of ints.

No new tests since this is just refactoring.

  • accessibility/AccessibilityListBox.cpp: (WebCore::AccessibilityListBox::elementAccessibilityHitTest):
  • accessibility/AccessibilityListBoxOption.cpp: (WebCore::AccessibilityListBoxOption::elementRect):
  • rendering/RenderListBox.cpp: (WebCore::RenderListBox::itemBoundingBoxRect): (WebCore::RenderListBox::addFocusRingRects): (WebCore::RenderListBox::paintItemForeground): (WebCore::RenderListBox::paintItemBackground): (WebCore::RenderListBox::nodeAtPoint):
  • rendering/RenderListBox.h:
7:31 PM Changeset in webkit [87351] by morrita@google.com
  • 16 edits
    6 adds in trunk

2011-05-24 MORITA Hajime <​morrita@google.com>

RenderText with empty text is not created inside ShadowContentElement
​https://bugs.webkit.org/show_bug.cgi?id=61111

  • Added tests that capture the probelem.
  • Updated expectations that were wrong.
  • fast/html/details-replace-summary-child.html: Added.
  • fast/html/details-replace-text.html: Added.
  • platform/gtk/fast/html/details-no-summary4-expected.txt:
  • platform/gtk/fast/html/details-open-javascript-expected.txt:
  • platform/gtk/fast/html/details-open4-expected.txt:
  • platform/mac/fast/html/details-add-details-child-2-expected.txt:
  • platform/mac/fast/html/details-no-summary4-expected.txt:
  • platform/mac/fast/html/details-open-javascript-expected.txt:
  • platform/mac/fast/html/details-open4-expected.txt:
  • platform/mac/fast/html/details-replace-summary-child-expected.png: Added.
  • platform/mac/fast/html/details-replace-summary-child-expected.txt: Added.
  • platform/mac/fast/html/details-replace-text-expected.png: Added.
  • platform/mac/fast/html/details-replace-text-expected.txt: Added.
  • platform/qt/fast/html/details-no-summary4-expected.txt:
  • platform/qt/fast/html/details-open-javascript-expected.txt:
  • platform/qt/fast/html/details-open4-expected.txt:

2011-05-24 MORITA Hajime <​morrita@google.com>

Reviewed by Dimitri Glazkov.

RenderText with empty text is not created inside ShadowContentElement
​https://bugs.webkit.org/show_bug.cgi?id=61111

  • Changed Text::rendererIsNeeded() to be shadow-aware.
  • previousRenderer(), nextRenderer() and parentRenderer() on NodeRenderingContext is now safe even if it's used with an attached node, which was originally used only during attach(). We need this change because these APIs are called inside CharacterData::updateRenderer() callstack.

Tests: fast/html/details-replace-summary-child.html

fast/html/details-replace-text.html

  • dom/NodeRenderingContext.cpp: (WebCore::NodeRenderingContext::NodeRenderingContext): (WebCore::NodeRenderingContext::nextRenderer): (WebCore::NodeRenderingContext::previousRenderer): (WebCore::NodeRenderingContext::parentRenderer): (WebCore::NodeRenderingContext::shouldCreateRenderer):
  • dom/NodeRenderingContext.h: (WebCore::NodeRenderingContext::parentNodeForRenderingAndStyle):
  • dom/Text.cpp: (WebCore::Text::rendererIsNeeded):
6:58 PM Changeset in webkit [87350] by adamk@chromium.org
  • 2 edits in trunk/LayoutTests

2011-05-25 Adam Klein <​adamk@chromium.org>

Unreviewed. Add Win/CRASH expectation for new crashy plugin test,
reported as ​http://crbug.com/83994.

  • platform/chromium/test_expectations.txt:
6:50 PM Changeset in webkit [87349] by adamk@chromium.org
  • 2 edits in trunk/LayoutTests

2011-05-25 Adam Klein <​adamk@chromium.org>

Unreviewed. Temporarily ignore MHTML test failures on Windows
while we dig into it in ​http://crbug.com/83833.

  • platform/chromium/test_expectations.txt:
6:48 PM Changeset in webkit [87348] by qi.2.zhang@nokia.com
  • 13 edits in trunk

2011-05-25 Qi Zhang <​qi.2.zhang@nokia.com>

Reviewed by Simon Hausmann.

WebKitTestRunner needs layoutTestController.setPrivateBrowsingEnabled
​https://bugs.webkit.org/show_bug.cgi?id=42697

  • platform/mac-wk2/Skipped:
  • platform/qt-wk2/Skipped:

2011-05-25 Qi Zhang <​qi.2.zhang@nokia.com>

Reviewed by Simon Hausmann.

WebKitTestRunner needs layoutTestController.setPrivateBrowsingEnabled
​https://bugs.webkit.org/show_bug.cgi?id=42697

Implemented layoutTestController.setPrivateBrowsingEnabled for WebKitTestRunner.

  • WebProcess/InjectedBundle/API/c/WKBundle.cpp: (WKBundleSetPrivateBrowsingEnabled):
  • WebProcess/InjectedBundle/API/c/WKBundlePrivate.h:
  • WebProcess/InjectedBundle/InjectedBundle.cpp: (WebKit::InjectedBundle::setPrivateBrowsingEnabled):
  • WebProcess/InjectedBundle/InjectedBundle.h:

2011-05-25 Qi Zhang <​qi.2.zhang@nokia.com>

Reviewed by Simon Hausmann.

WebKitTestRunner needs layoutTestController.setPrivateBrowsingEnabled
​https://bugs.webkit.org/show_bug.cgi?id=42697

Implemented layoutTestController.setPrivateBrowsingEnabled for WebKitTestRunner.

  • WebKitTestRunner/InjectedBundle/Bindings/LayoutTestController.idl:
  • WebKitTestRunner/InjectedBundle/InjectedBundle.cpp: (WTR::InjectedBundle::beginTesting):
  • WebKitTestRunner/InjectedBundle/LayoutTestController.cpp: (WTR::LayoutTestController::setPrivateBrowsingEnabled):
  • WebKitTestRunner/InjectedBundle/LayoutTestController.h:
6:30 PM Changeset in webkit [87347] by commit-queue@webkit.org
  • 2 edits
    2 moves in trunk/Source/WebCore

2011-05-25 Sailesh Agrawal <​sail@chromium.org>

Reviewed by Tony Chang.

Rename ScrollbarOverlayUtilitiesMac to ScrollbarOverlayUtilitiesChromiumMac
​https://bugs.webkit.org/show_bug.cgi?id=61401

ScrollbarOverlayUtilitiesMac.mm was being excluded from the WebCore project. Fix was to make sure the file name ended with ChromiumMac.

  • WebCore.gypi:
  • platform/chromium/ScrollbarOverlayUtilitiesChromiumMac.h: Copied from Source/WebCore/platform/chromium/ScrollbarOverlayUtilitiesMac.h.
  • platform/chromium/ScrollbarOverlayUtilitiesChromiumMac.mm: Copied from Source/WebCore/platform/chromium/ScrollbarOverlayUtilitiesMac.mm.
  • platform/chromium/ScrollbarOverlayUtilitiesMac.h: Removed.
  • platform/chromium/ScrollbarOverlayUtilitiesMac.mm: Removed.
6:12 PM Changeset in webkit [87346] by oliver@apple.com
  • 24 edits in trunk/Source

2011-05-25 Oliver Hunt <​oliver@apple.com>

Reviewed by Geoffrey Garen.

Make RegExp GC allocated
​https://bugs.webkit.org/show_bug.cgi?id=61490

Make RegExp GC allocated. Basically mechanical change to replace
most use of [Pass]RefPtr<RegExp> with RegExp* or WriteBarrier<RegExp>
where actual ownership happens.

Made the RegExpCache use Strong<> references currently to avoid any
changes in behaviour.

  • JavaScriptCore.exp:
  • bytecode/CodeBlock.cpp: (JSC::CodeBlock::visitAggregate):
  • bytecode/CodeBlock.h: (JSC::CodeBlock::addRegExp):
  • bytecompiler/BytecodeGenerator.cpp: (JSC::BytecodeGenerator::addRegExp): (JSC::BytecodeGenerator::emitNewRegExp):
  • bytecompiler/BytecodeGenerator.h:
  • runtime/JSCell.h:
  • runtime/JSGlobalData.cpp: (JSC::JSGlobalData::JSGlobalData): (JSC::JSGlobalData::clearBuiltinStructures): (JSC::JSGlobalData::addRegExpToTrace):
  • runtime/JSGlobalData.h:
  • runtime/JSGlobalObject.cpp: (JSC::JSGlobalObject::reset):
  • runtime/RegExp.cpp: (JSC::RegExp::RegExp): (JSC::RegExp::create): (JSC::RegExp::invalidateCode):
  • runtime/RegExp.h: (JSC::RegExp::createStructure):
  • runtime/RegExpCache.cpp: (JSC::RegExpCache::lookupOrCreate): (JSC::RegExpCache::create):
  • runtime/RegExpCache.h:
  • runtime/RegExpConstructor.cpp: (JSC::constructRegExp):
  • runtime/RegExpObject.cpp: (JSC::RegExpObject::RegExpObject): (JSC::RegExpObject::visitChildren):
  • runtime/RegExpObject.h: (JSC::RegExpObject::setRegExp): (JSC::RegExpObject::RegExpObjectData::RegExpObjectData):
  • runtime/RegExpPrototype.cpp: (JSC::RegExpPrototype::RegExpPrototype): (JSC::regExpProtoFuncCompile):
  • runtime/RegExpPrototype.h:
  • runtime/StringPrototype.cpp: (JSC::stringProtoFuncMatch): (JSC::stringProtoFuncSearch):

2011-05-25 James Robinson <​jamesr@chromium.org>

Reviewed by Geoffrey Garen

CachedResource overhead size calculation ignores the actual size of the URL
​https://bugs.webkit.org/show_bug.cgi?id=61481

CachedResource::overheadSize is used to determine the size of an entry in the memory cache to know when to evict
it. When the resource is a large data: URL, for example representing image or audio data, the URL size itself
can be significant.

This patch uses an estimate of actual number of bytes used by the URL that is valid for ASCII urls and close for
other types of strings instead of a fixed number.

  • loader/cache/CachedResource.cpp: (WebCore::CachedResource::overheadSize):
6:08 PM Changeset in webkit [87345] by oliver@apple.com
  • 26 edits in trunk/Source

Revert last change, something weird happened when I went to land.

6:07 PM Changeset in webkit [87344] by jamesr@google.com
  • 2 edits in trunk/Source/WebCore

2011-05-25 James Robinson <​jamesr@chromium.org>

Reviewed by Geoffrey Garen

CachedResource overhead size calculation ignores the actual size of the URL
​https://bugs.webkit.org/show_bug.cgi?id=61481

CachedResource::overheadSize is used to determine the size of an entry in the memory cache to know when to evict
it. When the resource is a large data: URL, for example representing image or audio data, the URL size itself
can be significant.

This patch uses an estimate of actual number of bytes used by the URL that is valid for ASCII urls and close for
other types of strings instead of a fixed number.

  • loader/cache/CachedResource.cpp: (WebCore::CachedResource::overheadSize):
6:01 PM Changeset in webkit [87343] by oliver@apple.com
  • 26 edits in trunk/Source

2011-05-25 Oliver Hunt <​oliver@apple.com>

Reviewed by Geoffrey Garen.

Make RegExp GC allocated
​https://bugs.webkit.org/show_bug.cgi?id=61490

Make RegExp GC allocated. Basically mechanical change to replace
most use of [Pass]RefPtr<RegExp> with RegExp* or WriteBarrier<RegExp>
where actual ownership happens.

Made the RegExpCache use Strong<> references currently to avoid any
changes in behaviour.

  • JavaScriptCore.exp:
  • bytecode/CodeBlock.cpp: (JSC::CodeBlock::visitAggregate):
  • bytecode/CodeBlock.h: (JSC::CodeBlock::addRegExp):
  • bytecompiler/BytecodeGenerator.cpp: (JSC::BytecodeGenerator::addRegExp): (JSC::BytecodeGenerator::emitNewRegExp):
  • bytecompiler/BytecodeGenerator.h:
  • runtime/JSCell.h:
  • runtime/JSGlobalData.cpp: (JSC::JSGlobalData::JSGlobalData): (JSC::JSGlobalData::clearBuiltinStructures): (JSC::JSGlobalData::addRegExpToTrace):
  • runtime/JSGlobalData.h:
  • runtime/JSGlobalObject.cpp: (JSC::JSGlobalObject::reset):
  • runtime/RegExp.cpp: (JSC::RegExp::RegExp): (JSC::RegExp::create): (JSC::RegExp::invalidateCode):
  • runtime/RegExp.h: (JSC::RegExp::createStructure):
  • runtime/RegExpCache.cpp: (JSC::RegExpCache::lookupOrCreate): (JSC::RegExpCache::create):
  • runtime/RegExpCache.h:
  • runtime/RegExpConstructor.cpp: (JSC::constructRegExp):
  • runtime/RegExpObject.cpp: (JSC::RegExpObject::RegExpObject): (JSC::RegExpObject::visitChildren):
  • runtime/RegExpObject.h: (JSC::RegExpObject::setRegExp): (JSC::RegExpObject::RegExpObjectData::RegExpObjectData):
  • runtime/RegExpPrototype.cpp: (JSC::RegExpPrototype::RegExpPrototype): (JSC::regExpProtoFuncCompile):
  • runtime/RegExpPrototype.h:
  • runtime/StringPrototype.cpp: (JSC::stringProtoFuncMatch): (JSC::stringProtoFuncSearch):

2011-05-25 Oliver Hunt <​oliver@apple.com>

Reviewed by Geoffrey Garen.

Make RegExp GC allocated
​https://bugs.webkit.org/show_bug.cgi?id=61490

RegExp is GC'd so we don't need the RefPtr shenanigans anymore.

  • bindings/js/SerializedScriptValue.cpp: (WebCore::CloneDeserializer::readTerminal):
5:59 PM Changeset in webkit [87342] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WebKit/chromium

2011-05-25 Shishir Agrawal <​shishir@chromium.org>

Reviewed by Dimitri Glazkov.

Fix crash in Chromium memory test.
​https://bugs.webkit.org/show_bug.cgi?id=61451

  • src/WebViewImpl.cpp: (WebKit::WebViewImpl::WebViewImpl):
5:52 PM Changeset in webkit [87341] by adamk@chromium.org
  • 2 edits in trunk/LayoutTests

2011-05-25 Adam Klein <​adamk@chromium.org>

Unreviewed. Fix to previous update, mark drag-not-loaded-image crash as DEBUG-only.

  • platform/chromium/test_expectations.txt:
5:44 PM Changeset in webkit [87340] by jer.noble@apple.com
  • 2 edits in trunk/LayoutTests

Unreviewed; Skip failing tests after r87328.

  • platform/mac/Skipped:
5:27 PM Changeset in webkit [87339] by adamk@chromium.org
  • 2 edits in trunk/LayoutTests

2011-05-25 Adam Klein <​adamk@chromium.org>

Unreviewed. Test expectations update.

  • platform/chromium/test_expectations.txt:
5:24 PM Changeset in webkit [87338] by adamk@chromium.org
  • 6 edits in trunk/Source/WebKit/chromium

2011-05-25 Sheriff Bot <​webkit.review.bot@gmail.com>

Unreviewed, rolling out r87333.
​http://trac.webkit.org/changeset/87333
​https://bugs.webkit.org/show_bug.cgi?id=61488

Breaks Chromium build due to pure virtual methods. (Requested
by aklein on #webkit).

  • public/WebWidget.h:
  • src/WebPopupMenuImpl.cpp:
  • src/WebPopupMenuImpl.h:
  • src/WebViewImpl.cpp:
  • src/WebViewImpl.h:
5:04 PM Changeset in webkit [87337] by jam@chromium.org
  • 1 edit in trunk/Source/WebKit/chromium/src/WebViewImpl.cpp

Revert 79703 since it broke pdf scrolling (​http://crbug.com/81878) - 2011-02-15 Brett Wilson <​brettw@chromium.org>

Reviewed by Darin Fisher.

Recover the mouse capture state when we recieve a mouse event where there is no longer a button down.
​https://bugs.webkit.org/show_bug.cgi?id=54503

  • src/WebViewImpl.cpp: (WebKit::WebViewImpl::handleInputEvent):

TBR=​brettw@chromium.org

4:54 PM Changeset in webkit [87336] by Martin Robinson
  • 7 edits in trunk/Source/WebCore

2011-05-25 Martin Robinson <​mrobinson@igalia.com>

Reviewed by Dirk Schulze.

[Cairo] Move the global alpha property from GraphicsContext to PlatformContextCairo
​https://bugs.webkit.org/show_bug.cgi?id=60185

Remove Cairo #ifdefs from GraphicsContext.h by pushing the global alpha
state into PlatformContextCairo. Since Cairo is the only platform that needs
to store this, this is the proper place for it. Change the image mask stack into
a more generic state stack that can keep track of the multiple bits of platform
specific state and properly handle save/restore pairs.

No new tests. This is just a code refactor.

  • platform/graphics/GraphicsContext.h: Remove Cairo #ifdefs. (WebCore::GraphicsContextState::GraphicsContextState): Ditto.
  • platform/graphics/cairo/ContextShadowCairo.cpp: Access global alpha from PlatformContextCairo now. (WebCore::ContextShadow::drawRectShadow):
  • platform/graphics/cairo/FontCairo.cpp: Ditto. (WebCore::Font::drawGlyphs):
  • platform/graphics/cairo/GraphicsContextCairo.cpp: (WebCore::setPlatformFill): Ditto. (WebCore::setPlatformStroke): Ditto. (WebCore::GraphicsContext::setAlpha): Ditto.
  • platform/graphics/cairo/PlatformContextCairo.cpp: Moved ImageMaskInformation class from the header file, since it can be private now. Abstract the image mask state and the global alpha in a generic State class. (WebCore::ImageMaskInformation::update): (WebCore::ImageMaskInformation::isValid): (WebCore::ImageMaskInformation::maskSurface): (WebCore::ImageMaskInformation::maskRect): (WebCore::PlatformContextCairo::State::State): Added. (WebCore::PlatformContextCairo::PlatformContextCairo): Intialize the state class when this class is constructed. (WebCore::PlatformContextCairo::restore): Now pop the last state off the state stack. (WebCore::PlatformContextCairo::~PlatformContextCairo): Added this so that we can forward declare the State class in the header. (WebCore::PlatformContextCairo::save): Push a new state onto the stack. (WebCore::PlatformContextCairo::pushImageMask): Operate on the state stack now. (WebCore::PlatformContextCairo::globalAlpha): Added. (WebCore::PlatformContextCairo::setGlobalAlpha): Added. (WebCore::PlatformContextCairo::drawSurfaceToContext: Call globalAlpha().
  • platform/graphics/cairo/PlatformContextCairo.h: Changed the image mask stack into a more generic state stack, much like PlatformContextChromium.
4:41 PM Changeset in webkit [87335] by mrowe@apple.com
  • 7 edits in trunk/Source

<rdar://problem/9504058> Need some way to query user gesture state from injected bundle

Reviewed by Sam Weinig.

Source/WebCore:

  • WebCore.exp.in: Add an export that WebKit2 needs.

Source/WebKit2:

Add WKBundleIsProcessingUserGesture.

  • WebProcess/InjectedBundle/API/c/WKBundle.cpp:

(WKBundleIsPageBoxVisible):
(WKBundleIsProcessingUserGesture):

  • WebProcess/InjectedBundle/API/c/WKBundlePrivate.h:
  • WebProcess/InjectedBundle/InjectedBundle.cpp:

(WebKit::InjectedBundle::isProcessingUserGesture):

  • WebProcess/InjectedBundle/InjectedBundle.h:
4:39 PM Changeset in webkit [87334] by adamk@chromium.org
  • 3 edits
    7 adds
    1 delete in trunk/LayoutTests

2011-05-25 Adam Klein <​adamk@chromium.org>

Unreviewed. Rebaseline svg tests after r87310.

  • platform/chromium-linux-x86/svg/W3C-SVG-1.1-SE/filters-felem-01-b-expected.png: Added.
  • platform/chromium-linux-x86/svg/W3C-SVG-1.1-SE/filters-felem-01-b-expected.txt: Added.
  • platform/chromium-linux-x86/svg/W3C-SVG-1.1/filters-felem-01-b-expected.png: Added.
  • platform/chromium-linux/svg/W3C-SVG-1.1-SE/filters-felem-01-b-expected.png: Added.
  • platform/chromium-linux/svg/W3C-SVG-1.1-SE/filters-felem-01-b-expected.txt: Added.
  • platform/chromium-linux/svg/W3C-SVG-1.1/filters-felem-01-b-expected.png:
  • platform/chromium-mac/svg/W3C-SVG-1.1/filters-felem-01-b-expected.png: Removed.
  • platform/chromium-win/svg/W3C-SVG-1.1-SE/filters-felem-01-b-expected.png: Added.
  • platform/chromium-win/svg/W3C-SVG-1.1-SE/filters-felem-01-b-expected.txt: Added.
  • platform/chromium-win/svg/W3C-SVG-1.1/filters-felem-01-b-expected.png:
4:30 PM Changeset in webkit [87333] by commit-queue@webkit.org
  • 6 edits in trunk/Source/WebKit/chromium

2011-05-25 Sailesh Agrawal <​sail@chromium.org>

Reviewed by Darin Fisher.

Expose willStartLiveResize and willEndLiveResize in WebWidget
​https://bugs.webkit.org/show_bug.cgi?id=60518

  • public/WebWidget.h:
  • src/WebPopupMenuImpl.cpp: (WebKit::WebPopupMenuImpl::willStartLiveResize): (WebKit::WebPopupMenuImpl::willEndLiveResize):
  • src/WebPopupMenuImpl.h:
  • src/WebViewImpl.cpp: (WebKit::WebViewImpl::willStartLiveResize): (WebKit::WebViewImpl::willEndLiveResize):
  • src/WebViewImpl.h:
4:09 PM Changeset in webkit [87332] by adamk@chromium.org
  • 2 edits in trunk/LayoutTests

2011-05-25 Adam Klein <​adamk@chromium.org>

Unreviewed. Removing test_expectations for non-existent tests.

  • platform/chromium/test_expectations.txt:
4:08 PM Changeset in webkit [87331] by jer.noble@apple.com
  • 2 edits in trunk/Source/WebCore

Ureviewed; Build fix for Leopard and Snow Leopard.

Move _wkQTMovieResolvedURL outside of a #if check for Leopard and SL.

  • WebCore.exp.in:
4:05 PM Changeset in webkit [87330] by msaboff@apple.com
  • 4 edits in trunk/Source/WebCore

2011-05-25 Michael Saboff <​msaboff@apple.com>

Reviewed by Sam Weinig.

Cleanup of commented items from ​https://bugs.webkit.org/show_bug.cgi?id=61222
​https://bugs.webkit.org/show_bug.cgi?id=61478

Cleanup of further comments after patch was landed. Changes include
using /2 instead of >>1, adding a blank line after class include,
making method names start with lower case and adding clarifying
comments.

No new tests as the changes are stylistic and not functional.

  • loader/cache/MemoryCache.cpp: (WebCore::MemoryCache::pruneLiveResourcesToPercentage): (WebCore::MemoryCache::pruneDeadResourcesToPercentage):
  • loader/cache/MemoryCache.h: (WebCore::MemoryCache::pruneToPercentage):
  • platform/mac/MemoryPressureHandlerMac.mm: (WebCore::MemoryPressureHandler::respondToMemoryPressure):
4:00 PM Changeset in webkit [87329] by Stephanie Lewis
  • 11 edits in trunk/Source

Source/WebCore: ​https://bugs.webkit.org/show_bug.cgi?id=61407
part of <rdar://problem/8675177>
Move Quicklook quirk down into WebCore so it can be caculated when the user
starts a reload. Restructure the conditions so it only called when absolutely
necessary.

Reviewed by Brady Eidson.

No change in functionality so no new tests.

  • loader/FrameLoader.cpp:

(WebCore::FrameLoader::subresourceCachePolicy):

  • page/Settings.cpp:

(WebCore::Settings::Settings):

  • page/Settings.h:
  • platform/network/cf/ResourceRequest.h:
  • platform/network/mac/ResourceRequestMac.mm:

(WebCore::initQuickLookResourceCachingQuirks):
(WebCore::ResourceRequest::useQuickLookResourceCachingQuirks):

Source/WebKit/mac: ​https://bugs.webkit.org/show_bug.cgi?id=61407
part of <rdar://problem/8675177>
Remove Quicklook pref. Moving it enirely down into WebCore so it can
be calculated only on reloads.

Reviewed by Brady Eidson.

  • WebView/WebPreferenceKeysPrivate.h:
  • WebView/WebPreferences.mm:

(+[WebPreferences initialize]):

  • WebView/WebPreferencesPrivate.h:
  • WebView/WebView.mm:

(-[WebView _preferencesChanged:]):

3:53 PM Changeset in webkit [87328] by jer.noble@apple.com
  • 16 edits in trunk

2011-05-24 Jer Noble <​jer.noble@apple.com>

Reviewed by Darin Adler.

Video fails to play on Vimeo
​https://bugs.webkit.org/show_bug.cgi?id=61403

No new tests; Covered by media/video-canvas-source.html.

Vimeo redirects their assets from player.vimeo.com to av.vimeo.com, which is rejected
by AVFoundation and QTKit due to our setting a ForbidCrossSiteReference option when
creating an AVAsset or QTMovie. Instead, we should just reject local->remote and
remote->local and make our answer to hasSingleSecurityOrigin dynamic.

When checking whether a given request has a single security origin, use a
SecurityOrigin to check the host, port, and scheme.

  • WebCore.exp.in:
  • platform/graphics/avfoundation/MediaPlayerPrivateAVFoundation.h: (WebCore::MediaPlayerPrivateAVFoundation::assetURL): Added.
  • platform/graphics/avfoundation/MediaPlayerPrivateAVFoundationObjC.h:
  • platform/graphics/avfoundation/MediaPlayerPrivateAVFoundationObjC.mm: (WebCore::MediaPlayerPrivateAVFoundationObjC::createAVAssetForURL): Exchange ForbidCrossSiteReference

for ForbidRemoteReferenceToLocal and ForbidLocalReferenceToRemote

(WebCore::MediaPlayerPrivateAVFoundationObjC::hasSingleSecurityOrigin): Check to see that the

requested and resolved URLs have the same host and port.

  • platform/graphics/mac/MediaPlayerPrivateQTKit.mm: (WebCore::MediaPlayerPrivateQTKit::commonMovieAttributes): Exchange NoCrossSiteAttribute for

NoRemoteToLocalSiteAttribute and NoLocalToRemoteSiteAttribute.

(WebCore::MediaPlayerPrivateQTKit::hasSingleSecurityOrigin): Check to see that the

requested and resolved URLs have the same host and port.

  • platform/mac/WebCoreSystemInterface.h: Added wkAVAssetResolvedURL.
  • platform/mac/WebCoreSystemInterface.mm: Ditto.

2011-05-24 Jer Noble <​jer.noble@apple.com>

Reviewed by Darin Adler.

Video fails to play on Vimeo
​https://bugs.webkit.org/show_bug.cgi?id=61403

  • WebCoreSupport/WebSystemInterface.mm: (InitWebCoreSystemInterface): Added support for wkAVAssetResolvedURL and

wkQTMovieResolvedURL.

2011-05-24 Jer Noble <​jer.noble@apple.com>

Reviewed by Darin Adler.

Video fails to play on Vimeo
​https://bugs.webkit.org/show_bug.cgi?id=61403

  • WebProcess/WebCoreSupport/mac/WebSystemInterface.mm: (InitWebCoreSystemInterface): Added support for wkAVAssetResolvedURL and

wkQTMovieResolvedURL.

2011-05-25 Jer Noble <​jer.noble@apple.com>

Reviewed by Darin Adler.

Video fails to play on Vimeo
​https://bugs.webkit.org/show_bug.cgi?id=61403

Added functions to retrieve the resolved URL for media types supported on
mac.

  • WebKitSystemInterface.h:
  • WebKitSystemInterface.m: (WKAVAssetResolvedURL): Added. (WKQTMovieResolvedURL): Added.
3:49 PM Changeset in webkit [87327] by oliver@apple.com
  • 6 edits in trunk/Source/JavaScriptCore

2011-05-25 Oliver Hunt <​oliver@apple.com>

Reviewed by Geoffrey Garen.

Generate regexp code lazily
​https://bugs.webkit.org/show_bug.cgi?id=61476

RegExp construction now simply validates the RegExp, it does
not perform actual codegen.

  • runtime/RegExp.cpp: (JSC::RegExp::RegExp): (JSC::RegExp::recompile): (JSC::RegExp::compile): (JSC::RegExp::match):
  • runtime/RegExp.h: (JSC::RegExp::recompileIfNecessary):
  • runtime/RegExpConstructor.h: (JSC::RegExpConstructor::performMatch):
  • runtime/RegExpObject.cpp: (JSC::RegExpObject::match):
  • runtime/StringPrototype.cpp: (JSC::stringProtoFuncReplace): (JSC::stringProtoFuncMatch): (JSC::stringProtoFuncSearch): (JSC::stringProtoFuncSplit):
3:46 PM Changeset in webkit [87326] by jer.noble@apple.com
  • 2 edits in trunk/LayoutTests

Unreviewed; Update Skipped list for failing tests.

A new full-screen test requires <video> full screen support, which does not
exist on mac-leopard. Follow-up to r87322.

  • platform/mac-leopard/Skipped:
3:03 PM Changeset in webkit [87325] by adamk@chromium.org
  • 1 edit
    4 deletes in trunk/LayoutTests

2011-05-25 Adam Klein <​adamk@chromium.org>

Unreviewed. Test expectations update, followup to r87291.

The aforementioned change cleared out these files, when the intent
seems to have been removal to remove them.

  • platform/chromium-mac/ietestcenter/Javascript/15.5.4.20-1-1-expected.txt: Removed.
  • platform/chromium-mac/ietestcenter/Javascript/15.5.4.20-1-2-expected.txt: Removed.
  • platform/chromium-win/ietestcenter/Javascript/15.5.4.20-1-1-expected.txt: Removed.
  • platform/chromium-win/ietestcenter/Javascript/15.5.4.20-1-2-expected.txt: Removed.
2:56 PM Changeset in webkit [87324] by jhoneycutt@apple.com
  • 6 edits
    3 adds in trunk

REGRESSION (WebKit2): Crash in Flash on USA Today photo gallery
​https://bugs.webkit.org/show_bug.cgi?id=61428
<rdar://problem/9457006>

Reviewed by Adam Roben.

Source/WebKit2:

The crash occurs when Flash posts a message to a window that it
creates, and in processing the message, it calls NPN_Evaluate to
evaluate JavaScript that removes the plug-in from the page. Flash then
crashes when we return to Flash code.

  • Platform/WorkItem.h:

(DerefWorkItem::DerefWorkItem):
Initialize m_ptr.
(DerefWorkItem::execute):
Deref the object.
(WorkItem::createDeref):
Create and return a DerefWorkItem.

  • WebProcess/Plugins/PluginView.cpp:

(WebKit::PluginView::unprotectPluginFromDestruction):
If the PluginView has only one reference left, deref it asynchronously.

Tools:

The crash occurs when Flash posts a message to a window that it
creates, and in processing the message, it calls NPN_Evaluate to
evaluate JavaScript that removes the plug-in from the page. Flash then
crashes when we return to Flash code.

This test emulates that behavior.

  • DumpRenderTree/TestNetscapePlugIn/win/CallJSThatDestroysPlugin.cpp: Added.

(CallJSThatDestroysPlugin::CallJSThatDestroysPlugin):
Initialize member vars.
(CallJSThatDestroysPlugin::~CallJSThatDestroysPlugin):
Remove our custom property from the message window, and destroy it.
(CallJSThatDestroysPlugin::NPP_Destroy):
Set m_isDestroyed, log that the plug-in was destroyed, and notify the
layout test controller that we're done.
(wndProc):
Get the PluginTest object, and call its runTest() function.
(CallJSThatDestroysPlugin::NPP_New):
Setup the test: register a class for the message-only window, create
it, and post a message to it to run the test.
(CallJSThatDestroysPlugin::runTest):
Execute JS that removes the plug-in from the page, and if we're not
destroyed, log a success message.

  • DumpRenderTree/TestNetscapePlugIn/win/TestNetscapePlugin.vcproj:

Add new test to project.

LayoutTests:

  • platform/win/plugins/call-javascript-that-destroys-plugin-expected.txt: Added.
  • platform/win/plugins/call-javascript-that-destroys-plugin.html: Added.
2:47 PM Changeset in webkit [87323] by scherkus@chromium.org
  • 3 edits
    2 adds in trunk

2011-05-25 Andrew Scherkus <​scherkus@chromium.org>

Reviewed by Eric Carlson.

Fix media element regression where ended event stopped firing after changing the src attribute.

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

  • media/media-ended-expected.txt: Added.
  • media/media-ended.html: Added.

2011-05-25 Andrew Scherkus <​scherkus@chromium.org>

Reviewed by Eric Carlson.

Fix media element regression where ended event stopped firing after changing the src attribute.

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

Test: media/media-ended.html

  • html/HTMLMediaElement.cpp: (WebCore::HTMLMediaElement::prepareForLoad):
2:45 PM QtWebKitBuildBots edited by Csaba Osztrogonác
Update Qt on Windows bots to 4.7.3 (diff)
2:38 PM Changeset in webkit [87322] by jer.noble@apple.com
  • 7 edits
    3 adds in trunk

2011-05-25 Jer Noble <​jer.noble@apple.com>

Reviewed by Darin Adler.

REGRESSION: Fullscreen button on embedded Vimeo videos does nothing
​https://bugs.webkit.org/show_bug.cgi?id=61461

  • fullscreen/full-screen-iframe-legacy-expected.txt: Added.
  • fullscreen/full-screen-iframe-legacy.html: Added.
  • fullscreen/resources/legacy.html: Added.

2011-05-25 Jer Noble <​jer.noble@apple.com>

Reviewed by Darin Adler.

REGRESSION: Fullscreen button on embedded Vimeo videos does nothing
​https://bugs.webkit.org/show_bug.cgi?id=61461

Tests: fullscreen/full-screen-iframe-legacy.html

Allow calls from the legacy full-screen API to bypass the iframe
"webkitallowfullscreen" requirement by adding a parameter to
Document::webkitRequestFullScreenForElement specifying the strictness
of that check. Specify this new parameter everywhere that function is
called, including in the default controls' full-screen button handler.

  • dom/Document.cpp: (WebCore::Document::webkitRequestFullScreenForElement):
  • dom/Document.h:
  • dom/Element.cpp: (WebCore::Element::requestFullScreen): Renamed from webkitRequestFullScreen.
  • html/HTMLMediaElement.cpp: (WebCore::HTMLMediaElement::enterFullscreen):
  • html/shadow/MediaControlElements.cpp: (WebCore::MediaControlFullscreenButtonElement::defaultEventHandler):
2:33 PM Changeset in webkit [87321] by tony@chromium.org
  • 3 edits in trunk/Tools

2011-05-25 Tony Chang <​tony@chromium.org>

Reviewed by Adam Barth.

webkitpy's NetworkTransaction shouldn't use mechanize
​https://bugs.webkit.org/show_bug.cgi?id=61450

mechanize.HTTPError is just an alias for urllib2.HTTPError, so
use urllib2 directly. Re-add NetworkTransaction to
test_results_uploader.py (reverting r87124).

  • Scripts/webkitpy/common/net/networktransaction.py:
  • Scripts/webkitpy/layout_tests/layout_package/test_results_uploader.py:
2:31 PM Changeset in webkit [87320] by caio.oliveira@openbossa.org
  • 3 edits in trunk/LayoutTests

[Qt] Missing update for expected results in LayoutTests after r87315
​https://bugs.webkit.org/show_bug.cgi?id=61467

Reviewed by Andreas Kling.

We now expose qt_sender as a non-enumarated property in the
global object. Fix our expectations in relevant tests.

  • platform/qt/fast/dom/Window/window-properties-expected.txt:
  • platform/qt/fast/dom/Window/window-property-descriptors-expected.txt:
2:29 PM Changeset in webkit [87319] by commit-queue@webkit.org
  • 4 edits
    14 adds in trunk

2011-05-25 Kulanthaivel Palanichamy <​kulanthaivel@codeaurora.org>

Reviewed by David Hyatt.

Selector matching doesn't update when DOM changes ("[data-a=x] #x")
​https://bugs.webkit.org/show_bug.cgi?id=60752

Added test cases for all the attribute selector types (CSS2.1 & CSS3).

  • fast/css/attribute-selector-begin-dynamic-no-elementstyle-expected.txt: Added.
  • fast/css/attribute-selector-begin-dynamic-no-elementstyle.html: Added.
  • fast/css/attribute-selector-contain-dynamic-no-elementstyle-expected.txt: Added.
  • fast/css/attribute-selector-contain-dynamic-no-elementstyle.html: Added.
  • fast/css/attribute-selector-end-dynamic-no-elementstyle-expected.txt: Added.
  • fast/css/attribute-selector-end-dynamic-no-elementstyle.html: Added.
  • fast/css/attribute-selector-exact-dynamic-no-elementstyle-expected.txt: Added.
  • fast/css/attribute-selector-exact-dynamic-no-elementstyle.html: Added.
  • fast/css/attribute-selector-hyphen-dynamic-no-elementstyle-expected.txt: Added.
  • fast/css/attribute-selector-hyphen-dynamic-no-elementstyle.html: Added.
  • fast/css/attribute-selector-list-dynamic-no-elementstyle-expected.txt: Added.
  • fast/css/attribute-selector-list-dynamic-no-elementstyle.html: Added.
  • fast/css/attribute-selector-set-dynamic-no-elementstyle-expected.txt: Added.
  • fast/css/attribute-selector-set-dynamic-no-elementstyle.html: Added.

2011-05-25 Kulanthaivel Palanichamy <​kulanthaivel@codeaurora.org>

Reviewed by David Hyatt.

Selector matching doesn't update when DOM changes ("[data-a=x] #x")
​https://bugs.webkit.org/show_bug.cgi?id=60752

Currently CSSStyleSelector maintains a HashSet of attributes (m_selectorAttrs)
which are used in CSS attribute selectors to determine the need for style
recalculation whenever element attributes are manipulated in DOM.
In certain conditions (element with no style, element is styled and attribute
is not a mapped attribute, attribute is of type 'type' or read-only)
even when attribute selector matches for an element, the attribute is not
added to m_selectorAttrs. This results in missing style recalculations
when a DOM element attribute is changed and is not found in m_selectorAttrs.

Removing the above said conditions in
CSSStyleSelector::SelectorChecker::checkOneSelector() for registering
attributes in m_selectorAttrs will solve this issue. But this particular
function is called numerous times which triggers adding duplicate attributes
again and again.

This patch follows the approach taken for collecting ids in selectors, where
all the attributes in selectors are added to a HashSet at the time of adding
style rules to CSSStyleSelector from StyleSheets and when
CSSStyleSelector::hasSelectorForAttribute() is called, the attribute is
simply looked up in this pre-populated hash set.

Test: fast/css/attribute-selector-dynamic-no-elementstyle.html

  • css/CSSStyleSelector.cpp: (WebCore::CSSStyleSelector::SelectorChecker::checkSelector): (WebCore::CSSStyleSelector::checkSelector): (WebCore::CSSStyleSelector::SelectorChecker::checkOneSelector): (WebCore::collectFeaturesFromSelector): (WebCore::CSSStyleSelector::applyProperty): (WebCore::CSSStyleSelector::hasSelectorForAttribute):
  • css/CSSStyleSelector.h:
2:23 PM Changeset in webkit [87318] by adamk@chromium.org
  • 2 edits in trunk/LayoutTests

2011-05-25 Adam Klein <​adamk@chromium.org>

Unreviewed. Update chromium expectations after r87307.

  • platform/chromium/test_expectations.txt: suppress, WK61470.
1:32 PM Changeset in webkit [87317] by rniwa@webkit.org
  • 7 edits in trunk/Source/WebCore

2011-05-25 Ryosuke Niwa <​rniwa@webkit.org>

Reviewed by James Robinson.

CSSStyleSelector should provide a way to obtain rules from non-author stylesheets
​https://bugs.webkit.org/show_bug.cgi?id=61454

Replaced bool and enum arguments of styleRulesForElement and pseudoStyleRulesForElement
by one enum, and provided a way to obtain rules from UA/user stylesheets.

  • css/CSSRule.h: Removed CSSRuleFilter.
  • css/CSSStyleSelector.cpp: (WebCore::CSSStyleSelector::styleRulesForElement): See above. (WebCore::CSSStyleSelector::pseudoStyleRulesForElement): Ditto.
  • css/CSSStyleSelector.h: Moved CSSRuleFilter here.
  • editing/markup.cpp: (WebCore::styleFromMatchedRulesForElement): Calls styleRulesForElement.
  • inspector/InspectorCSSAgent.cpp: (WebCore::InspectorCSSAgent::getStylesForNode): Ditto.
  • page/DOMWindow.cpp: (WebCore::DOMWindow::getMatchedCSSRules): Ditto.
12:37 PM Changeset in webkit [87316] by ggaren@apple.com
  • 6 edits in trunk/Source/JavaScriptCore

2011-05-24 Geoffrey Garen <​ggaren@apple.com>

Reviewed by Geoffrey Garen.

Removed MarkSetProperties because it was unused
​https://bugs.webkit.org/show_bug.cgi?id=61418

  • heap/MarkStack.h: (JSC::MarkSet::MarkSet): (JSC::MarkStack::append):
  • runtime/JSActivation.cpp: (JSC::JSActivation::visitChildren):
  • runtime/JSArray.h: (JSC::JSArray::visitChildrenDirect):
  • runtime/JSPropertyNameIterator.cpp: (JSC::JSPropertyNameIterator::visitChildren):
  • runtime/WriteBarrier.h: (JSC::MarkStack::appendValues):
12:28 PM Changeset in webkit [87315] by caio.oliveira@openbossa.org
  • 8 edits in trunk/Source

2011-05-25 Caio Marcelo de Oliveira Filho <​caio.oliveira@openbossa.org>

Reviewed by Andreas Kling.

[Qt] JSC bridge: implement qt_sender without using Scope Chain
​https://bugs.webkit.org/show_bug.cgi?id=61343

Create a stack to keep track of the sender objects. This is simpler than
the similar mechanism in QObject (C++ API), that keeps a stack per-object.

Since we do not support multiple threads, one static stack will be enough for
handling the behavior.

This behavior is covered by the tst_QWebFrame::connectAndDisconnect() auto test.

  • bridge/qt/qt_instance.cpp: (JSC::Bindings::QtInstance::qtSenderStack): We have one static stack of QObject*. The top of the stack contains the last object that emitted signal that called a JavaScript function.
  • bridge/qt/qt_instance.h: (JSC::Bindings::QtInstance::QtSenderStack::top): (JSC::Bindings::QtInstance::QtSenderStack::push): (JSC::Bindings::QtInstance::QtSenderStack::pop): Minimal functionality to manipulate the sender stack.
  • bridge/qt/qt_runtime.cpp: (JSC::Bindings::QtConnectionObject::execute): Remove the previous code that modified the scope chain. Push the sender object to the stack before calling the JavaScript function (the "slot" in Qt-speak) and pop it afterwards.

2011-05-25 Caio Marcelo de Oliveira Filho <​caio.oliveira@openbossa.org>

Reviewed by Andreas Kling.

[Qt] JSC bridge: implement qt_sender without using Scope Chain
​https://bugs.webkit.org/show_bug.cgi?id=61343

Create a 'qt_sender' property in the global object, that returns the top of
the qtSenderStack. This is an alternative implementation for the feature of
providing a way for a function (acting as a Qt 'slot') discover which object
emitted the signal that caused it to be executed.

This reduces the coupling of the Qt bridge and JSC internal implementation. The
patch tries to use as much JSC public API as possible.

This behavior is covered by the tst_QWebFrame::connectAndDisconnect() auto test.

  • WebCoreSupport/FrameLoaderClientQt.cpp: (WebCore::FrameLoaderClientQt::dispatchDidClearWindowObjectInWorld): Instead of emitting the QWebPage::javaScriptWindowObjectCleared() directly, calls a QWebPagePrivate function to do it.
  • Api/qwebframe_p.h:
  • Api/qwebframe.cpp: (QWebFramePrivate::didClearedWindowObject): Before emitting the signal mentioned, adds the 'qt_sender' to the fresh global object.

(qtSenderCallback):
Returns the JSObjectRef corresponding to the top of qtSenderStack.

(QWebFramePrivate::addQtSenderToGlobalObject):
Create a property with a qtSenderCallback as getter function in the global object.

12:25 PM Changeset in webkit [87314] by kevino@webkit.org
  • 2 edits in trunk/Tools

[Chromium] Unreviewed build fix, do not define WTF/JS_EXPORT_PRIVATE to JS_EXPORTDATA
until the port begins using those export macros.

12:23 PM Changeset in webkit [87313] by commit-queue@webkit.org
  • 20 edits
    2 deletes in trunk

2011-05-25 Sheriff Bot <​webkit.review.bot@gmail.com>

Unreviewed, rolling out r87257.
​http://trac.webkit.org/changeset/87257
​https://bugs.webkit.org/show_bug.cgi?id=61457

This broke layout tests, see bug 61431. (Requested by
hwennborg on #webkit).

  • storage/indexeddb/migrate-basics-expected.txt: Removed.
  • storage/indexeddb/migrate-basics.html: Removed.

2011-05-25 Sheriff Bot <​webkit.review.bot@gmail.com>

Unreviewed, rolling out r87257.
​http://trac.webkit.org/changeset/87257
​https://bugs.webkit.org/show_bug.cgi?id=61457

This broke layout tests, see bug 61431. (Requested by
hwennborg on #webkit).

  • storage/IDBBackingStore.h:
  • storage/IDBFactoryBackendImpl.cpp: (WebCore::IDBFactoryBackendImpl::IDBFactoryBackendImpl): (WebCore::IDBFactoryBackendImpl::open):
  • storage/IDBFactoryBackendImpl.h:
  • storage/IDBFactoryBackendInterface.h:
  • storage/IDBLevelDBBackingStore.cpp:
  • storage/IDBLevelDBBackingStore.h:
  • storage/IDBSQLiteBackingStore.cpp:
  • storage/IDBSQLiteBackingStore.h:

2011-05-25 Sheriff Bot <​webkit.review.bot@gmail.com>

Unreviewed, rolling out r87257.
​http://trac.webkit.org/changeset/87257
​https://bugs.webkit.org/show_bug.cgi?id=61457

This broke layout tests, see bug 61431. (Requested by
hwennborg on #webkit).

  • public/WebIDBFactory.h:
  • src/AssertMatchingEnums.cpp:
  • src/IDBFactoryBackendProxy.cpp:
  • src/IDBFactoryBackendProxy.h:
  • src/WebIDBFactoryImpl.cpp: (WebKit::WebIDBFactoryImpl::WebIDBFactoryImpl): (WebKit::WebIDBFactoryImpl::open):
  • src/WebIDBFactoryImpl.h:

2011-05-25 Sheriff Bot <​webkit.review.bot@gmail.com>

Unreviewed, rolling out r87257.
​http://trac.webkit.org/changeset/87257
​https://bugs.webkit.org/show_bug.cgi?id=61457

This broke layout tests, see bug 61431. (Requested by
hwennborg on #webkit).

  • DumpRenderTree/chromium/LayoutTestController.cpp: (LayoutTestController::LayoutTestController): (LayoutTestController::clearAllDatabases):
  • DumpRenderTree/chromium/LayoutTestController.h:
12:22 PM Changeset in webkit [87312] by alexis.menard@openbossa.org
  • 23 edits
    1 copy
    3 adds in trunk/Source

2011-05-25 Alexis Menard <​alexis.menard@openbossa.org>

Reviewed by Eric Carlson.

[Qt] Enable usage of QuickTime mediaplayer for the Qt port on Mac.
​https://bugs.webkit.org/show_bug.cgi?id=61279

Enable the QuickTime backend for the Qt port on Mac. The patch adds the bits in WebCore
to enable the QTKit backend of the Mac port. It can be enabled by passing DEFINES+=USE_QTKIT=1
when calling build-webkit.

  • WebCore.pri:
  • WebCore.pro:
  • features.pri:
  • platform/KURL.h:
  • platform/SharedBuffer.h:
  • platform/cf/KURLCFNet.cpp:
  • platform/cf/SharedBufferCF.cpp:
  • platform/graphics/FloatSize.h:
  • platform/graphics/IntRect.h:
  • platform/graphics/IntSize.h:
  • platform/graphics/MediaPlayer.cpp:
  • platform/graphics/cg/FloatSizeCG.cpp:
  • platform/graphics/cg/IntRectCG.cpp:
  • platform/graphics/mac/MediaPlayerPrivateQTKit.h:
  • platform/graphics/mac/MediaPlayerPrivateQTKit.mm: (WebCore::MediaPlayerPrivateQTKit::createQTMovieView): (WebCore::MediaPlayerPrivateQTKit::createQTVideoRenderer): (WebCore::MediaPlayerPrivateQTKit::createQTMovieLayer): (WebCore::MediaPlayerPrivateQTKit::preferredRenderingMode): (WebCore::MediaPlayerPrivateQTKit::paint): (-[WebCoreMovieObserver layerHostChanged:]):
  • platform/mac/KURLMac.mm:
  • platform/mac/SharedBufferMac.mm: (+[WebCoreSharedBufferData initialize]):
  • platform/mac/WebCoreObjCExtras.mm:
  • platform/qt/WebCoreSystemInterface.h: Added.
  • platform/qt/WebCoreSystemInterface.mm: Added.

2011-05-25 Alexis Menard <​alexis.menard@openbossa.org>

Reviewed by Eric Carlson.

[Qt] Enable usage of QuickTime mediaplayer for the Qt port on Mac.
​https://bugs.webkit.org/show_bug.cgi?id=61279

Enable the usage of QuickTime backend for the Qt port. It can be enabled by
passing DEFINES+=USE_QTKIT=1 when calling build-webkit.

  • Api/qwebpage.cpp: (QWebPagePrivate::QWebPagePrivate):
  • QtWebKit.pro:
  • WebCoreSupport/FullScreenVideoQt.cpp: (WebCore::FullScreenVideoQt::enterFullScreenForNode): (WebCore::FullScreenVideoQt::requiresFullScreenForVideoPlayback): (WebCore::FullScreenVideoQt::isValid):
  • WebCoreSupport/WebSystemInterface.h: Added.
  • WebCoreSupport/WebSystemInterface.mm: Added. (InitWebCoreSystemInterface):
12:20 PM Changeset in webkit [87311] by qi.2.zhang@nokia.com
  • 9 edits
    2 adds in trunk

2011-05-25 Qi Zhang <​qi.2.zhang@nokia.com>

Reviewed by Andreas Kling.

[Qt] fast/css/disabled-author-styles.html failed
​https://bugs.webkit.org/show_bug.cgi?id=61438

Unskip test case and provide expect result files.

  • platform/qt/Skipped:
  • platform/qt/fast/css/disabled-author-styles-expected.png: Added.
  • platform/qt/fast/css/disabled-author-styles-expected.txt: Added.

2011-05-25 Qi Zhang <​qi.2.zhang@nokia.com>

Reviewed by Andreas Kling.

[Qt] fast/css/disabled-author-styles.html failed
​https://bugs.webkit.org/show_bug.cgi?id=61438

Provided API setAuthorAndUserStylesEnabled for DumpRenderTree.

  • WebCoreSupport/DumpRenderTreeSupportQt.cpp: (DumpRenderTreeSupportQt::setAuthorAndUserStylesEnabled):
  • WebCoreSupport/DumpRenderTreeSupportQt.h:

2011-05-25 Qi Zhang <​qi.2.zhang@nokia.com>

Reviewed by Andreas Kling.

[Qt] fast/css/disabled-author-styles.html failed
​https://bugs.webkit.org/show_bug.cgi?id=61438

Implemented API setAuthorAndUserStylesEnabled in LayoutTestControllerQt.

  • DumpRenderTree/qt/DumpRenderTreeQt.cpp: (WebCore::WebPage::resetSettings):
  • DumpRenderTree/qt/LayoutTestControllerQt.cpp: (LayoutTestController::setAuthorAndUserStylesEnabled):
  • DumpRenderTree/qt/LayoutTestControllerQt.h:
12:01 PM Changeset in webkit [87310] by rwlbuis@webkit.org
  • 4 edits
    3 adds in trunk

2011-05-25 Rob Buis <​rbuis@rim.com>

Reviewed by Eric Seidel.

WebKit does not handle missing filter elements correctly
​https://bugs.webkit.org/show_bug.cgi?id=12569

Updated improved pixel test result and add new SVG1.1 SE test and results.

  • platform/mac/svg/W3C-SVG-1.1-SE/filters-felem-01-b-expected.png: Added.
  • platform/mac/svg/W3C-SVG-1.1-SE/filters-felem-01-b-expected.txt: Added.
  • platform/mac/svg/W3C-SVG-1.1/filters-felem-01-b-expected.png:
  • svg/W3C-SVG-1.1-SE/filters-felem-01-b.svg: Added.

2011-05-25 Rob Buis <​rbuis@rim.com>

Reviewed by Eric Seidel.
New test rubber-stamped by Dirk Schulze.

WebKit does not handle missing filter elements correctly
​https://bugs.webkit.org/show_bug.cgi?id=12569

Do not render elements that reference non-existant filters.

Test: svg/W3C-SVG-1.1-SE/filters-felem-01-b.svg

  • rendering/svg/SVGRenderSupport.cpp: (WebCore::SVGRenderSupport::prepareToRenderSVGContent):
11:51 AM Changeset in webkit [87309] by Nate Chapin
  • 10 edits in trunk

2011-05-25 Nate Chapin <Nate Chapin>

Reviewed by Adam Barth.

Put view-source documents in a unique origin and always allow them to run scripts. This ensures tools like
XMLViewer work even when scripts are disabled, while still providing the protections expected when scripts are disabled.
​https://bugs.webkit.org/show_bug.cgi?id=59113

  • bindings/ScriptControllerBase.cpp: (WebCore::ScriptController::canExecuteScripts): Check whether the document is viewing source, and allow scripts in that case.
  • css/CSSStyleSelector.cpp: (WebCore::CSSStyleSelector::matchUARules):
  • dom/Document.cpp: (WebCore::Document::Document): (WebCore::Document::setIsViewSource): Renamed from setUsesViewSourceStyles(), set's a unique security origin if we are viewing source.
  • dom/Document.h: (WebCore::Document::isViewSource): Renamed from usesViewSourceStyles().
  • html/HTMLViewSourceDocument.cpp: (WebCore::HTMLViewSourceDocument::HTMLViewSourceDocument):
  • loader/FrameLoader.cpp: (WebCore::FrameLoader::receivedFirstData):
  • xml/XMLTreeViewer.cpp: (WebCore::XMLTreeViewer::transformDocumentToTreeView):
11:48 AM Changeset in webkit [87308] by oliver@apple.com
  • 3 edits in trunk/Source/JavaScriptCore

2011-05-25 Oliver Hunt <​oliver@apple.com>

Reviewed by Geoffrey Garen.

Make allocations with guard pages ensure that the allocation succeeded
​https://bugs.webkit.org/show_bug.cgi?id=61453

Add null checks, and make PageBlock's operator bool() use
the realbase, rather than the start of usable memory.

  • wtf/OSAllocatorPosix.cpp: (WTF::OSAllocator::reserveAndCommit):
  • wtf/PageBlock.h: (WTF::PageBlock::operator bool): (WTF::PageBlock::PageBlock):
11:45 AM Changeset in webkit [87307] by jchaffraix@webkit.org
  • 11 edits in trunk

2011-05-25 Julien Chaffraix <​jchaffraix@codeaurora.org>

Reviewed by James Robinson.

​http://philip.html5.org/tests/canvas/suite/tests/2d.composite.uncovered.fill.source-in.html fails
​https://bugs.webkit.org/show_bug.cgi?id=39027

And:

  • canvas/philip/tests/2d.composite.uncovered.pattern.source-in-expected.txt:
  • canvas/philip/tests/2d.composite.uncovered.pattern.source-out-expected.txt: Those 2 tests need a trailing line that was added as part of this change.
  • fast/canvas/canvas-composite-alpha.html: Updated the test result to match the specification. The changes are fairly simple:
    • Clear up the destination rectangle as it should be always transparent now (source is transparent outside the source rectangle).
    • Clear the composition rectangle if source or destination is transparent.
    • Fixed the test when run in the browser.

Validated the changes against FF and Opera (IE 9 cannot render this test).

  • platform/chromium/test_expectations.txt:
  • platform/gtk/Skipped:
  • platform/mac/Skipped:
  • platform/qt/Skipped: Re-enable 4 tests.

2011-05-25 Julien Chaffraix <​jchaffraix@codeaurora.org>

Reviewed by James Robinson.

​http://philip.html5.org/tests/canvas/suite/tests/2d.composite.uncovered.fill.source-in.html fails
​https://bugs.webkit.org/show_bug.cgi?id=39027

And:

Those 4 tests have the same root cause for failing. Instead of splitting them in 2 fixes, I squashed
them together as the fix is common.

Based on an older work by Ariya Hidayat.

  • html/canvas/CanvasRenderingContext2D.cpp: (WebCore::CanvasRenderingContext2D::fill): (WebCore::CanvasRenderingContext2D::fillRect): Added call displayTransparencyEverywhere for those 2 cases if we have to.

(WebCore::CanvasRenderingContext2D::shouldDisplayTransparencyEverywhere): This method returns true if
the spec says to display transparency everywhere. We only do so for source-in and source-out for now.
(WebCore::CanvasRenderingContext2D::displayTransparencyEverywhere): This method implements the spec
(which matches other browsers' behavior) and "display transparency elsewhere".

  • html/canvas/CanvasRenderingContext2D.h: Added the two previous methods.
10:58 AM Changeset in webkit [87306] by robert@webkit.org
  • 6 edits in trunk/LayoutTests

2011-05-20 Robert Hogan <​robert@webkit.org>

Reviewed by Geoffrey Garen.

[Qt/Gtk/Chromium] Fix plugins/embed-attributes-setting.html

Use the test netscape plugin instead of quicktime.
Allows the test to pass on Qt/Gtk/Chromium as well as Mac.

​https://bugs.webkit.org/show_bug.cgi?id=61182
​http://code.google.com/p/chromium/issues/detail?id=15797

  • platform/chromium/test_expectations.txt:
  • platform/gtk/Skipped:
  • platform/qt/Skipped:
  • plugins/embed-attributes-setting-expected.txt:
  • plugins/embed-attributes-setting.html:
10:22 AM Changeset in webkit [87305] by mitz@apple.com
  • 2 edits in trunk/Source/WebKit2

<rdar://problem/9496795> REGRESSION (r86851): Plug-ins disappear when scrolled with non-1 page scale
​https://bugs.webkit.org/show_bug.cgi?id=61448

Reviewed by Anders Carlsson.

  • WebProcess/Plugins/PluginView.cpp:

(WebKit::PluginView::clipRectInWindowCoordinates): Correct for page scale
after intersecting the two rectangles.

10:10 AM Changeset in webkit [87304] by kevino@webkit.org
  • 2 edits in trunk/Source/JavaScriptCore

Reviewed by Eric Seidel.

Add JS_EXPORT_PRIVATE macro for exported methods in bytecompiler headers.

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

10:06 AM Changeset in webkit [87303] by leviw@chromium.org
  • 22 edits in trunk/Source/WebCore

2011-05-25 Levi Weintraub <​leviw@chromium.org>

Reviewed by Eric Seidel.

Add IntPoint versions of IntRect::move and IntPoint::move
​https://bugs.webkit.org/show_bug.cgi?id=61390

Adding move methods to IntRect and IntPoint that take an IntPoint
that's acting as an offset. Since we also apply negative offsets,
also adding a single parameter operator- for IntPoint.

No new tests since this is refactoring.

  • page/FrameView.cpp: (WebCore::FrameView::invalidateScrollbarRect): (WebCore::FrameView::convertFromRenderer): (WebCore::FrameView::convertToRenderer):
  • platform/ScrollView.cpp: (WebCore::ScrollView::wheelEvent):
  • platform/ScrollView.h: (WebCore::ScrollView::convertChildToSelf): (WebCore::ScrollView::convertSelfToChild):
  • platform/ScrollbarThemeComposite.cpp: (WebCore::ScrollbarThemeComposite::invalidatePart):
  • platform/graphics/IntPoint.h: (WebCore::IntPoint::move): (WebCore::operator-):
  • platform/graphics/IntRect.h: (WebCore::IntRect::move):
  • platform/graphics/filters/FilterEffect.cpp: (WebCore::FilterEffect::requestedRegionOfInputImageData):
  • platform/graphics/gpu/Texture.cpp: (WebCore::Texture::updateSubRect):
  • platform/mac/WidgetMac.mm: (WebCore::Widget::paint):
  • rendering/RenderBlock.cpp: (WebCore::RenderBlock::logicalRectToPhysicalRect): (WebCore::RenderBlock::selectionGaps):
  • rendering/RenderBox.cpp: (WebCore::RenderBox::offsetFromContainer): (WebCore::RenderBox::computeRectForRepaint): (WebCore::RenderBox::localCaretRect):
  • rendering/RenderDetailsMarker.cpp: (WebCore::RenderDetailsMarker::paint):
  • rendering/RenderInline.cpp: (WebCore::RenderInline::culledInlineVisualOverflowBoundingBox):
  • rendering/RenderInputSpeech.cpp: (WebCore::RenderInputSpeech::paintInputFieldSpeechButton):
  • rendering/RenderLayerBacking.cpp: (WebCore::paintScrollbar): (WebCore::RenderLayerBacking::paintContents):
  • rendering/RenderLayerCompositor.cpp: (WebCore::paintScrollbar): (WebCore::RenderLayerCompositor::paintContents):
  • rendering/RenderListItem.cpp: (WebCore::RenderListItem::positionListMarker):
  • rendering/RenderListMarker.cpp: (WebCore::RenderListMarker::paint):
  • rendering/RenderTableCell.cpp: (WebCore::RenderTableCell::computeRectForRepaint):
  • rendering/RenderTextControlSingleLine.cpp: (WebCore::RenderTextControlSingleLine::nodeAtPoint):
  • rendering/RenderView.cpp: (WebCore::RenderView::repaintViewRectangle):
10:04 AM Changeset in webkit [87302] by leviw@chromium.org
  • 22 edits in trunk/Source/WebCore

2011-05-25 Levi Weintraub <​leviw@chromium.org>

Reviewed by Eric Seidel.

Switch addFocusRingRects to use IntPoint
​https://bugs.webkit.org/show_bug.cgi?id=60783

Switching addFocusRingRects to use an IntPoint representing the offset being
applied instead of a pair of ints.

No new tests as this is only refactoring.

  • platform/graphics/FloatSize.h: (WebCore::flooredIntPoint):
  • rendering/RenderBlock.cpp: (WebCore::RenderBlock::addFocusRingRects):
  • rendering/RenderBlock.h:
  • rendering/RenderBox.cpp: (WebCore::RenderBox::addFocusRingRects):
  • rendering/RenderBox.h:
  • rendering/RenderInline.cpp: (WebCore::RenderInline::addFocusRingRects):
  • rendering/RenderInline.h:
  • rendering/RenderListBox.cpp: (WebCore::RenderListBox::addFocusRingRects):
  • rendering/RenderListBox.h:
  • rendering/RenderObject.cpp: (WebCore::RenderObject::paintFocusRing): (WebCore::RenderObject::absoluteFocusRingQuads):
  • rendering/RenderObject.h: (WebCore::RenderObject::addFocusRingRects):
  • rendering/RenderTextControl.cpp: (WebCore::RenderTextControl::addFocusRingRects):
  • rendering/RenderTextControl.h:
  • rendering/RenderTextControlSingleLine.cpp: (WebCore::RenderTextControlSingleLine::addFocusRingRects):
  • rendering/RenderTextControlSingleLine.h:
  • rendering/svg/RenderSVGContainer.cpp: (WebCore::RenderSVGContainer::addFocusRingRects):
  • rendering/svg/RenderSVGContainer.h:
  • rendering/svg/RenderSVGImage.cpp: (WebCore::RenderSVGImage::addFocusRingRects):
  • rendering/svg/RenderSVGImage.h:
  • rendering/svg/RenderSVGPath.cpp: (WebCore::RenderSVGPath::addFocusRingRects):
  • rendering/svg/RenderSVGPath.h:
9:50 AM Changeset in webkit [87301] by commit-queue@webkit.org
  • 6 edits in trunk/Source/WebCore

2011-05-25 Sheriff Bot <​webkit.review.bot@gmail.com>

Unreviewed, rolling out r87229.
​http://trac.webkit.org/changeset/87229
​https://bugs.webkit.org/show_bug.cgi?id=61445

Lots of tests crashing in
CFNetwork!URLResponse::createFilenameFromResponseHeaders on
Windows XP (Requested by aroben on #webkit).

  • platform/network/ResourceResponseBase.cpp: (WebCore::ResourceResponseBase::adopt): (WebCore::ResourceResponseBase::suggestedFilename): (WebCore::ResourceResponseBase::setSuggestedFilename): (WebCore::ResourceResponseBase::httpStatusText): (WebCore::ResourceResponseBase::setHTTPStatusText): (WebCore::ResourceResponseBase::httpHeaderField): (WebCore::ResourceResponseBase::setHTTPHeaderField): (WebCore::ResourceResponseBase::httpHeaderFields): (WebCore::ResourceResponseBase::isAttachment): (WebCore::ResourceResponseBase::setLastModifiedDate): (WebCore::ResourceResponseBase::lastModifiedDate): (WebCore::ResourceResponseBase::wasCached): (WebCore::ResourceResponseBase::connectionReused): (WebCore::ResourceResponseBase::setConnectionReused): (WebCore::ResourceResponseBase::connectionID): (WebCore::ResourceResponseBase::setConnectionID): (WebCore::ResourceResponseBase::resourceLoadTiming): (WebCore::ResourceResponseBase::setResourceLoadTiming): (WebCore::ResourceResponseBase::resourceLoadInfo): (WebCore::ResourceResponseBase::setResourceLoadInfo):
  • platform/network/ResourceResponseBase.h:
  • platform/network/cf/ResourceResponse.h: (WebCore::ResourceResponse::ResourceResponse):
  • platform/network/cf/ResourceResponseCFNet.cpp: (WebCore::ResourceResponse::platformLazyInit):
  • platform/network/mac/ResourceResponseMac.mm: (WebCore::ResourceResponse::platformLazyInit):
9:48 AM Changeset in webkit [87300] by commit-queue@webkit.org
  • 2 edits in trunk/LayoutTests

2011-05-25 Leon Scroggins <​scroggo@google.com>

Reviewed by Steve Block.

Update chromium expectations for more canvas fixes.
​https://bugs.webkit.org/show_bug.cgi?id=61433

  • platform/chromium/test_expectations.txt:
9:38 AM Changeset in webkit [87299] by Adam Roben
  • 4 edits in trunk/Tools

Identify, rather than skip, builds where run-webkit-tests exited early due to too many failures

Fixes <​http://webkit.org/b/61441> TestFailures page should show when run-webkit-tests
started exiting early due to too many crashes, timeouts, or failures

Reviewed by David Kilzer.

  • BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/Builder.js:

(Builder.prototype.getNumberOfFailingTests): Rather than returning -1 ("unknown") when
run-webkit-tests exits early due to too many failures, pass that information in another
argument to the callback.
(Builder.prototype.startFetchingBuildHistory): Updated the documentation comment to reflect
the new structure of the object passed to the callback.
(Builder.prototype._getFailingTests): Updated to expect a tooManyFailures boolean from
getNumberOfFailingTests and to pass that along to our own callbacks.
(Builder.prototype._incorporateBuildHistory): Updated to expect a tooManyFailures boolean
from _getFailingTests and to store that value in the history object.

  • BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/TestFailures.css:

(.info): Added.

  • BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/ViewController.js:

(ViewController.prototype._displayBuilder): Updated for change in structure to the history
object and to add a note when run-webkit-tests exited early due to too many failures.
(ViewController.prototype._displayTesters): Renamed testersAndFailureCounts to
latestBuildInfos since each entry now contains more than just the tester and failure count.
Now displays a message for testers where the latest build exited early due to too many
failures. Updated to expect a tooManyFailures boolean from getNumberOfFailingTests and to
store that value in latestBuildInfos.

9:33 AM Changeset in webkit [87298] by commit-queue@webkit.org
  • 3 edits
    2 adds in trunk

2011-05-25 Igor Oliveira <​igor.oliveira@openbossa.org>

Reviewed by Antonio Gomes.

[Qt] QtWebKit crashes when dragging not loaded images
​https://bugs.webkit.org/show_bug.cgi?id=61314

Checks if WebKit crashes when dragging an image not yet loaded.

  • http/tests/misc/drag-not-loaded-image-expected.txt: Added.
  • http/tests/misc/drag-not-loaded-image.html: Added.

2011-05-25 Igor Oliveira <​igor.oliveira@openbossa.org>

Reviewed by Antonio Gomes.

[Qt] QtWebKit crashes when dragging not loaded images
​https://bugs.webkit.org/show_bug.cgi?id=61314

Checks if nativeImageForCurrentFrame is a valid pointer.

Test: http/tests/misc/drag-not-loaded-image.html

  • platform/qt/DragImageQt.cpp: (WebCore::createDragImageFromImage):
9:27 AM Changeset in webkit [87297] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WebCore

2011-05-25 Ryuan Choi <​ryuan.choi@samsung.com>

Reviewed by Xan Lopez.

[GTK] combobox and scrollbar are rendered small size at gtk+2.22.0
​https://bugs.webkit.org/show_bug.cgi?id=61436

Fix arguments of gdk_drawable_get_size.

No new tests, no behavior change.

  • platform/gtk/GtkVersioning.c: (getGdkDrawableSize):
8:02 AM QtWebKitRelease22 edited by Ademar Reis
(diff)
7:58 AM Changeset in webkit [87296] by caseq@chromium.org
  • 2 edits in trunk/LayoutTests

2011-05-25 Andrey Kosyakov <​caseq@chromium.org>

Unreviewed. Skipped 3 new inspector tests that time out on gtk.

  • platform/gtk/Skipped:
7:50 AM QtWebKitRelease22 edited by Ademar Reis
(diff)
7:46 AM QtWebKitRelease22 edited by Ademar Reis
(diff)
7:20 AM Changeset in webkit [87295] by loki@webkit.org
  • 2 edits in trunk/LayoutTests

2011-05-25 Gabor Loki <​loki@webkit.org>

Rubber-stamped by Csaba Osztrogonác.

[Qt] Skip failed tests after r87290 and r87292 on ARM

  • platform/qt-arm/Skipped:
7:02 AM Changeset in webkit [87294] by apavlov@chromium.org
  • 5 edits in trunk/Source/WebCore

2011-05-23 Alexander Pavlov <​apavlov@chromium.org>

Reviewed by Yury Semikhatsky.

Web Inspector: extreme lagging when searching for "img[src*="
​https://bugs.webkit.org/show_bug.cgi?id=61282

No new tests, as this is a refactoring: ElementsTreeElement DOM is manipulated directly
when built and search-highlighted, rather than through innerHTML.

  • inspector/front-end/DOMSyntaxHighlighter.js: (WebInspector.DOMSyntaxHighlighter): (WebInspector.DOMSyntaxHighlighter.prototype.createSpan):
  • inspector/front-end/ElementsPanel.js: (WebInspector.ElementsPanel.prototype._hideSearchHighlights):
  • inspector/front-end/ElementsTreeOutline.js: (WebInspector.ElementsTreeElement.prototype.highlightSearchResults): (WebInspector.ElementsTreeElement.prototype.hideSearchHighlights): (WebInspector.ElementsTreeElement.prototype._updateSearchHighlight.updateEntryShow): (WebInspector.ElementsTreeElement.prototype._updateSearchHighlight.updateEntryHide): (WebInspector.ElementsTreeElement.prototype._updateSearchHighlight): (WebInspector.ElementsTreeElement.prototype._addNewAttribute): (WebInspector.ElementsTreeElement.prototype.updateTitle): (WebInspector.ElementsTreeElement.prototype._buildAttributeDOM): ():
  • inspector/front-end/utilities.js: ():
5:34 AM Changeset in webkit [87293] by commit-queue@webkit.org
  • 15 edits
    9 adds in trunk/Source/WebCore

2011-05-25 Leandro Gracia Gil <​leandrogracia@chromium.org>

Reviewed by Tony Gentilcore.

Introduce HTML5 track list objects.
​https://bugs.webkit.org/show_bug.cgi?id=60184

Introduce the TrackList, MultipleTrackList and ExclusiveTrackList objects
for their use in the MediaStream API and the HTML Media Element.
Current spec: ​http://www.whatwg.org/specs/web-apps/current-work/multipage/the-iframe-element.html#tracklist

The Media Stream API will test this feature in the bug 56587.
More tests related to the HTML Media Element may be added later.

  • CMakeLists.txt:
  • CodeGenerators.pri:
  • DerivedSources.cpp:
  • DerivedSources.make:
  • GNUmakefile.list.am:
  • WebCore.gypi:
  • WebCore.pro:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • bindings/js/JSEventTarget.cpp: (WebCore::toJS):
  • bindings/v8/V8DOMWrapper.cpp: (WebCore::V8DOMWrapper::convertEventTargetToV8Object):
  • dom/DOMAllInOne.cpp:
  • dom/EventTarget.cpp: (WebCore::EventTarget::toTrackList): (WebCore::EventTarget::toMultipleTrackList): (WebCore::EventTarget::toExclusiveTrackList):
  • dom/EventTarget.h:
  • dom/ExclusiveTrackList.cpp: Added. (WebCore::ExclusiveTrackList::create): (WebCore::ExclusiveTrackList::ExclusiveTrackList): (WebCore::ExclusiveTrackList::~ExclusiveTrackList): (WebCore::ExclusiveTrackList::clear): (WebCore::ExclusiveTrackList::select): (WebCore::ExclusiveTrackList::toExclusiveTrackList):
  • dom/ExclusiveTrackList.h: Added. (WebCore::ExclusiveTrackList::selectedIndex):
  • dom/ExclusiveTrackList.idl: Added.
  • dom/MultipleTrackList.cpp: Added. (WebCore::MultipleTrackList::create): (WebCore::MultipleTrackList::MultipleTrackList): (WebCore::MultipleTrackList::~MultipleTrackList): (WebCore::MultipleTrackList::clear): (WebCore::MultipleTrackList::isEnabled): (WebCore::MultipleTrackList::enable): (WebCore::MultipleTrackList::disable): (WebCore::MultipleTrackList::toMultipleTrackList):
  • dom/MultipleTrackList.h: Added.
  • dom/MultipleTrackList.idl: Added.
  • dom/TrackList.cpp: Added. (WebCore::Track::create): (WebCore::Track::Track): (WebCore::TrackList::create): (WebCore::TrackList::TrackList): (WebCore::TrackList::~TrackList): (WebCore::TrackList::length): (WebCore::TrackList::checkIndex): (WebCore::TrackList::getID): (WebCore::TrackList::getKind): (WebCore::TrackList::getLabel): (WebCore::TrackList::getLanguage): (WebCore::TrackList::clear): (WebCore::TrackList::postChangeEvent): (WebCore::TrackList::dispatchChangeEvent): (WebCore::TrackList::toTrackList): (WebCore::TrackList::scriptExecutionContext): (WebCore::TrackList::eventTargetData): (WebCore::TrackList::ensureEventTargetData):
  • dom/TrackList.h: Added. (WebCore::Track::id): (WebCore::Track::kind): (WebCore::Track::label): (WebCore::Track::language): (WebCore::TrackList::DispatchTask::create): (WebCore::TrackList::DispatchTask::performTask): (WebCore::TrackList::DispatchTask::DispatchTask): (WebCore::TrackList::refEventTarget): (WebCore::TrackList::derefEventTarget):
  • dom/TrackList.idl: Added.
5:06 AM QtWebKitRelease22 edited by Ademar Reis
(diff)
4:55 AM Changeset in webkit [87292] by keishi@webkit.org
  • 8 edits
    2 adds in trunk/Source/WebCore

2011-05-25 Keishi Hattori <​keishi@webkit.org>

Reviewed by Kent Tamura.

Move ElementWithPseudoId to separate file
​https://bugs.webkit.org/show_bug.cgi?id=61430

No new tests because no behavior changes.

  • CMakeLists.txt: Added ElementWithPseudoId.cpp
  • GNUmakefile.list.am: Added ElementWithPseudoId.{h,cpp}
  • WebCore.gypi: Added ElementWithPseudoId.{h,cpp}
  • WebCore.pro: Added ElementWithPseudoId.{h,cpp}
  • WebCore.vcproj/WebCore.vcproj: Added ElementWithPseudoId.{h,cpp}
  • WebCore.xcodeproj/project.pbxproj: Added ElementWithPseudoId.{h,cpp}
  • html/ValidationMessage.cpp: Removed ElementWithPseudoId definition.
  • html/shadow/ElementWithPseudoId.cpp: Added. (WebCore::ElementWithPseudoId::shadowPseudoId):
  • html/shadow/ElementWithPseudoId.h: Added. (WebCore::ElementWithPseudoId::create): (WebCore::ElementWithPseudoId::ElementWithPseudoId):
4:43 AM Changeset in webkit [87291] by antonm@chromium.org
  • 5 edits
    4 adds in trunk/LayoutTests

2011-05-25 Anton Muhin <​antonm@chromium.org>

Unreviewed.

Update test expectations after 3.3.6.1 v8 roll.

  • platform/chromium-mac/ietestcenter/Javascript/15.5.4.20-1-1-expected.txt:
  • platform/chromium-mac/ietestcenter/Javascript/15.5.4.20-1-2-expected.txt:
  • platform/chromium-win/ietestcenter/Javascript/15.5.4.20-1-1-expected.txt:
  • platform/chromium-win/ietestcenter/Javascript/15.5.4.20-1-2-expected.txt:
  • platform/chromium/ietestcenter: Added.
  • platform/chromium/ietestcenter/Javascript: Added.
  • platform/chromium/ietestcenter/Javascript/15.4.4.14-5-28-expected.txt: Added.
  • platform/chromium/ietestcenter/Javascript/15.4.4.15-5-28-expected.txt: Added.
4:39 AM Changeset in webkit [87290] by alexis.menard@openbossa.org
  • 2 edits in trunk/Source/WebCore

2011-05-25 Alexis Menard <​alexis.menard@openbossa.org>

Reviewed by Antonio Gomes.

[Qt] When QWebView has a different style set on it then the scrollbars are not rendered correctly
​https://bugs.webkit.org/show_bug.cgi?id=34635

Make sure the scrollbar is properly rendered on Mac with another style than the Mac style. The code had
two paths, one for Mac and one for any other styles. The problem is that on Mac you can still run the
application with -style plastique for example, therefore the code used for other styles should also be
used whenever the current style is not the Mac style.

  • platform/qt/ScrollbarThemeQt.cpp: (WebCore::ScrollbarThemeQt::paint):
4:21 AM Changeset in webkit [87289] by ukai@chromium.org
  • 2 edits in trunk/LayoutTests

Unreviewed.

Update chromium test expectations.

  • platform/chromium/test_expectations.txt: svn/dynamic-updates failing
4:19 AM Changeset in webkit [87288] by loki@webkit.org
  • 2 edits in trunk/LayoutTests

2011-05-25 Gabor Loki <​loki@webkit.org>

Rubber-stamped by Csaba Osztrogonác.

[Qt] Skip a flakey timed out test on ARM

  • platform/qt-arm/Skipped:
4:08 AM Changeset in webkit [87287] by ukai@chromium.org
  • 2 edits in trunk/LayoutTests

Unreviewed.

Update chromium test expectations.

  • platform/chromium/test_expectations.txt: mhtml tests pass
4:00 AM Changeset in webkit [87286] by ukai@chromium.org
  • 2 edits in trunk/LayoutTests

Unreviewed.

Update chromium test expectations.

  • platform/chromium/test_expectations.txt:
3:41 AM Changeset in webkit [87285] by ukai@chromium.org
  • 2 edits in trunk/LayoutTests

Unreviewed.

Update chromium test expectations.

  • platform/chromium/test_expectations.txt:
3:34 AM Changeset in webkit [87284] by commit-queue@webkit.org
  • 1 edit
    36 adds in trunk/LayoutTests

2011-05-25 Felician Marton <​marton.felician.zoltan@stud.u-szeged.hu>

Reviewed by Csaba Osztrogonác.

Added dynamic tests for missing properties (and attributes) in TextElement. The corresponding Mac pixel test results also added.
​https://bugs.webkit.org/show_bug.cgi?id=59677

  • platform/mac/svg/dynamic-updates/SVGTextElement-dom-dx-attr-expected.png: Added.
  • platform/mac/svg/dynamic-updates/SVGTextElement-dom-dy-attr-expected.png: Added.
  • platform/mac/svg/dynamic-updates/SVGTextElement-dom-rotate-attr-expected.png: Added.
  • platform/mac/svg/dynamic-updates/SVGTextElement-dom-x-attr-expected.png: Added.
  • platform/mac/svg/dynamic-updates/SVGTextElement-dom-y-attr-expected.png: Added.
  • platform/mac/svg/dynamic-updates/SVGTextElement-svgdom-dx-prop-expected.png: Added.
  • platform/mac/svg/dynamic-updates/SVGTextElement-svgdom-dy-prop-expected.png: Added.
  • platform/mac/svg/dynamic-updates/SVGTextElement-svgdom-x-prop-expected.png: Added.
  • platform/mac/svg/dynamic-updates/SVGTextElement-svgdom-y-prop-expected.png: Added.
  • svg/dynamic-updates/SVGTextElement-dom-dx-attr-expected.txt: Added.
  • svg/dynamic-updates/SVGTextElement-dom-dx-attr.html: Added.
  • svg/dynamic-updates/SVGTextElement-dom-dy-attr-expected.txt: Added.
  • svg/dynamic-updates/SVGTextElement-dom-dy-attr.html: Added.
  • svg/dynamic-updates/SVGTextElement-dom-rotate-attr-expected.txt: Added.
  • svg/dynamic-updates/SVGTextElement-dom-rotate-attr.html: Added.
  • svg/dynamic-updates/SVGTextElement-dom-x-attr-expected.txt: Added.
  • svg/dynamic-updates/SVGTextElement-dom-x-attr.html: Added.
  • svg/dynamic-updates/SVGTextElement-dom-y-attr-expected.txt: Added.
  • svg/dynamic-updates/SVGTextElement-dom-y-attr.html: Added.
  • svg/dynamic-updates/SVGTextElement-svgdom-dx-prop-expected.txt: Added.
  • svg/dynamic-updates/SVGTextElement-svgdom-dx-prop.html: Added.
  • svg/dynamic-updates/SVGTextElement-svgdom-dy-prop-expected.txt: Added.
  • svg/dynamic-updates/SVGTextElement-svgdom-dy-prop.html: Added.
  • svg/dynamic-updates/SVGTextElement-svgdom-x-prop-expected.txt: Added.
  • svg/dynamic-updates/SVGTextElement-svgdom-x-prop.html: Added.
  • svg/dynamic-updates/SVGTextElement-svgdom-y-prop-expected.txt: Added.
  • svg/dynamic-updates/SVGTextElement-svgdom-y-prop.html: Added.
  • svg/dynamic-updates/script-tests/SVGTextElement-dom-dx-attr.js: Added. (executeTest):
  • svg/dynamic-updates/script-tests/SVGTextElement-dom-dy-attr.js: Added. (executeTest):
  • svg/dynamic-updates/script-tests/SVGTextElement-dom-rotate-attr.js: Added. (executeTest):
  • svg/dynamic-updates/script-tests/SVGTextElement-dom-x-attr.js: Added. (executeTest):
  • svg/dynamic-updates/script-tests/SVGTextElement-dom-y-attr.js: Added. (executeTest):
  • svg/dynamic-updates/script-tests/SVGTextElement-svgdom-dx-prop.js: Added. (executeTest):
  • svg/dynamic-updates/script-tests/SVGTextElement-svgdom-dy-prop.js: Added. (executeTest):
  • svg/dynamic-updates/script-tests/SVGTextElement-svgdom-x-prop.js: Added. (executeTest):
  • svg/dynamic-updates/script-tests/SVGTextElement-svgdom-y-prop.js: Added. (executeTest):
3:12 AM Changeset in webkit [87283] by yutak@chromium.org
  • 2 edits in trunk/LayoutTests

2011-05-25 Yuta Kitamura <​yutak@chromium.org>

Reviewed by Kent Tamura.

[Chromium] Clean up WebSocket test expectations
​https://bugs.webkit.org/show_bug.cgi?id=61429

  • platform/chromium/test_expectations.txt: Remove two test expectations for tests that are no longer failing.
2:52 AM Changeset in webkit [87282] by yutak@chromium.org
  • 4 edits in trunk

2011-05-25 Yuta Kitamura <​yutak@chromium.org>

Reviewed by Kent Tamura.

WebSocket: Use fail() when WebSocketChannel has failed
​https://bugs.webkit.org/show_bug.cgi?id=61353

  • http/tests/websocket/tests/frame-length-overflow-expected.txt: Added a new console message.

2011-05-25 Yuta Kitamura <​yutak@chromium.org>

Reviewed by Kent Tamura.

WebSocket: Use fail() when WebSocketChannel has failed
​https://bugs.webkit.org/show_bug.cgi?id=61353

An existing error message has been modified, but it is impossible
to test this message in LayoutTests because it is only shown when
memory allocation has failed, which is hard to reproduce reliably.

One new message has been added. It is covered by an existing test
http/tests/websocket/tests/frame-length-overflow.html.

There is no other change in behavior. No new tests are added.

  • websockets/WebSocketChannel.cpp: (WebCore::WebSocketChannel::fail): Do not close if we know the socket stream is already closed. This does not change the behavior, because SocketStreamBase does nothing if it is already closed. (WebCore::WebSocketChannel::didOpen): (WebCore::WebSocketChannel::didReceiveData): We need to set m_shouldDiscardReceivedData to true before calling fail(), so I moved the error message from appendToBuffer() to here. The error message was rephrased in order to improve readability. (WebCore::WebSocketChannel::appendToBuffer): Unnested the code. (WebCore::WebSocketChannel::processBuffer):
2:35 AM Changeset in webkit [87281] by apavlov@chromium.org
  • 2 edits in trunk/Source/WebCore

2011-05-16 Alexander Pavlov <​apavlov@chromium.org>

Reviewed by David Levin.

CSSParser: m_implicitShorthand should probably be RAII
​https://bugs.webkit.org/show_bug.cgi?id=51586

It was impossible to entirely eliminate the m_implicitShorthand manual changes
due to the parseFill*() method stateful call chains.

No new tests, as this is a refactoring.

  • css/CSSParser.cpp: (ImplicitScope::ImplicitScope): (ImplicitScope::~ImplicitScope): (WebCore::CSSParser::parseShorthand): (WebCore::CSSParser::parse4Values): (WebCore::CSSParser::parseBorderRadius):
2:03 AM Changeset in webkit [87280] by podivilov@chromium.org
  • 6 edits in trunk

2011-05-24 Pavel Podivilov <​podivilov@chromium.org>

Reviewed by Yury Semikhatsky.

Web Inspector: inspected page crashes when call stack is empty after live edit.
​https://bugs.webkit.org/show_bug.cgi?id=61364

  • inspector/debugger/live-edit-expected.txt:
  • inspector/debugger/live-edit.html:
  • platform/chromium/test_expectations.txt:

2011-05-24 Pavel Podivilov <​podivilov@chromium.org>

Reviewed by Yury Semikhatsky.

Web Inspector: inspected page crashes when call stack is empty after live edit.
​https://bugs.webkit.org/show_bug.cgi?id=61364

  • bindings/v8/ScriptDebugServer.cpp: (WebCore::ScriptDebugServer::currentCallFrame):
12:48 AM Changeset in webkit [87279] by ager@chromium.org
  • 2 edits in trunk/LayoutTests

2011-05-25 Mads Ager <​ager@chromium.org>

Reviewed by Adam Barth.

Update chromium expectations after removal of execScript from V8
​https://bugs.webkit.org/show_bug.cgi?id=61267

  • platform/chromium/fast/dom/prototype-inheritance-expected.txt:
12:45 AM Changeset in webkit [87278] by yurys@chromium.org
  • 3 edits
    3 adds in trunk

2011-05-23 Yury Semikhatsky <​yurys@chromium.org>

Reviewed by Adam Barth.

Chromium DevTools: Tab crashes with "Aw, snap!" on entering "(new Image())." in console
​https://bugs.webkit.org/show_bug.cgi?id=61194

Test: inspector/console/console-eval-syntax-error.html

  • bindings/v8/custom/V8InjectedScriptHostCustom.cpp: (WebCore::V8InjectedScriptHost::evaluateCallback): return immediately in case of syntax error

2011-05-23 Yury Semikhatsky <​yurys@chromium.org>

Reviewed by Adam Barth.

Chromium DevTools: Tab crashes with "Aw, snap!" on entering "(new Image())." in console
​https://bugs.webkit.org/show_bug.cgi?id=61194

  • inspector/console/console-eval-syntax-error-expected.txt: Added.
  • inspector/console/console-eval-syntax-error.html: Added.
  • platform/chromium/inspector/console/console-eval-syntax-error-expected.txt: Added.
12:30 AM Component Model edited by dominicc@chromium.org
Updated the status of legacy shadow DOM post r87014 (diff)
12:17 AM Changeset in webkit [87277] by yuzo@google.com
  • 10 edits in trunk/Source/WebCore

2011-05-25 Yuzo Fujishima <​yuzo@google.com>

Reviewed by Kent Tamura.

Fix for Bug 61352 - Refactor RenderView::{enable,disable}LayoutState call sites to use RIIA
​https://bugs.webkit.org/show_bug.cgi?id=61352

No new tests because no behavior changes.

  • html/shadow/MediaControlElements.cpp: (WebCore::RenderMediaVolumeSliderContainer::layout):
  • page/FrameView.cpp: (WebCore::FrameView::layout):
  • rendering/RenderBlock.cpp: (WebCore::RenderBlock::repaintOverhangingFloats): (WebCore::RenderBlock::updateFirstLetter):
  • rendering/RenderLayer.cpp: (WebCore::RenderLayer::updateLayerPositions): (WebCore::RenderLayer::updateScrollInfoAfterLayout):
  • rendering/RenderListBox.cpp: (WebCore::RenderListBox::layout):
  • rendering/RenderListItem.cpp: (WebCore::RenderListItem::updateMarkerLocation):
  • rendering/RenderMedia.cpp: (WebCore::RenderMedia::layout):
  • rendering/RenderView.h: (WebCore::RenderView::disableLayoutState): (WebCore::RenderView::enableLayoutState): (WebCore::LayoutStateDisabler::LayoutStateDisabler): (WebCore::LayoutStateDisabler::~LayoutStateDisabler):
  • rendering/svg/RenderSVGRoot.cpp: (WebCore::RenderSVGRoot::layout):

May 24, 2011:

11:42 PM Changeset in webkit [87276] by Csaba Osztrogonác
  • 2 edits in trunk/Source/WebCore

[Qt] Unreviewed typo fix after r87228.

  • WebCore.pro:
11:37 PM Changeset in webkit [87275] by timothy@apple.com
  • 3 edits in trunk/Source/WebKit2

Make the Web Inspector bring the right window to the front when docked.

rdar://problem/9464685
​https://webkit.org/b/61420

Reviewed by Brian Weinstein.

  • UIProcess/mac/WebInspectorProxyMac.mm:

(WebKit::WebInspectorProxy::platformBringToFront): Order front the parent window of the Inspector's WKView.

  • UIProcess/win/WebInspectorProxyWin.cpp:

(WebKit::WebInspectorProxy::platformBringToFront): Ditto.

11:23 PM QtWebKitBuildBots edited by Csaba Osztrogonác
Update Qt on minimal bot (diff)
11:03 PM Changeset in webkit [87274] by tkent@chromium.org
  • 33 edits
    3 adds
    3 deletes in trunk

2011-05-24 Keishi Hattori <​keishi@webkit.org>

Reviewed by Kent Tamura.

Disable textfield implementation of <input type=color>. Add INPUT_COLOR feature flag. Add input color sanitizer.
​https://bugs.webkit.org/show_bug.cgi?id=61273

  • configure.ac: Added INPUT_COLOR feature flag.

2011-05-24 Keishi Hattori <​keishi@webkit.org>

Reviewed by Kent Tamura.

Disable textfield implementation of <input type=color>. Add INPUT_COLOR feature flag. Add input color sanitizer.
​https://bugs.webkit.org/show_bug.cgi?id=61273

  • fast/forms/ValidityState-typeMismatch-color-expected.txt: Removed.
  • fast/forms/ValidityState-typeMismatch-color.html: Removed.
  • fast/forms/color/input-value-sanitization-color-expected.txt: Added.
  • fast/forms/color/input-value-sanitization-color.html: Added. Tests sanitization algorithm for input type=color.
  • fast/forms/input-type-change3-expected.txt:
  • fast/forms/input-widths-expected.txt:
  • fast/forms/input-widths.html: Removed type=color because it is no loger a text input type.
  • fast/forms/script-tests/ValidityState-patternMismatch-unsupported.js: Changed to range input type.
  • fast/forms/script-tests/ValidityState-typeMismatch-color.js: Removed.
  • fast/forms/script-tests/input-type-change3.js:
  • platform/gtk/Skipped: Skip fast/forms/color.
  • platform/mac/Skipped: Skip fast/forms/color.
  • platform/qt/Skipped: Skip fast/forms/color.
  • platform/win/Skipped: Skip fast/forms/color.

2011-05-24 Keishi Hattori <​keishi@webkit.org>

Reviewed by Kent Tamura.

Disable textfield implementation of <input type=color>. Add INPUT_COLOR feature flag. Add input color sanitizer.
​https://bugs.webkit.org/show_bug.cgi?id=61273

  • Configurations/FeatureDefines.xcconfig: Added COLOR_INPUT feature flag.

2011-05-24 Keishi Hattori <​keishi@webkit.org>

Reviewed by Kent Tamura.

Disable textfield implementation of <input type=color>. Add INPUT_COLOR feature flag. Add input color sanitizer.
​https://bugs.webkit.org/show_bug.cgi?id=61273

Test: fast/forms/color/input-value-sanitization-color.html

  • Configurations/FeatureDefines.xcconfig: Added COLOR_INPUT feature flag.
  • GNUmakefile.am: Added COLOR_INPUT feature flag.
  • features.pri: Added COLOR_INPUT feature flag.
  • html/ColorInputType.cpp: Wrapped with COLOR_INPUT feature flag. (WebCore::ColorInputType::isColorControl): Added. (WebCore::ColorInputType::fallbackValue): Added. Fallback value defined in spec. (WebCore::ColorInputType::sanitizeValue): Added. Sanitize value string as defined in spec.
  • html/ColorInputType.h: Wrapped with COLOR_INPUT feature flag. (WebCore::ColorInputType::ColorInputType): Changed to inherit BaseButtonInputType.
  • html/InputType.cpp: (WebCore::InputType::isColorControl): Added.
  • html/InputType.h:

2011-05-24 Keishi Hattori <​keishi@webkit.org>

Reviewed by Kent Tamura.

Disable textfield implementation of <input type=color>. Add INPUT_COLOR feature flag. Add input color sanitizer.
​https://bugs.webkit.org/show_bug.cgi?id=61273

  • features.gypi: Added INPUT_COLOR feature flag.

2011-05-24 Keishi Hattori <​keishi@webkit.org>

Reviewed by Kent Tamura.

Disable input color. Add INPUT_COLOR feature flag. Implement input color sanitizer.
​https://bugs.webkit.org/show_bug.cgi?id=61273

  • Configurations/FeatureDefines.xcconfig: Added INPUT_COLOR feature flag.

2011-05-24 Keishi Hattori <​keishi@webkit.org>

Reviewed by Kent Tamura.

Disable textfield implementation of <input type=color>. Add INPUT_COLOR feature flag. Add input color sanitizer.
​https://bugs.webkit.org/show_bug.cgi?id=61273

  • Configurations/FeatureDefines.xcconfig: Added INPUT_COLOR feature flag.

2011-05-24 Keishi Hattori <​keishi@webkit.org>

Reviewed by Kent Tamura.

Disable textfield implementation of <input type=color>. Add INPUT_COLOR feature flag. Add input color sanitizer.
​https://bugs.webkit.org/show_bug.cgi?id=61273

  • Scripts/build-webkit: Added INPUT_COLOR feature flag.

2011-05-24 Keishi Hattori <​keishi@webkit.org>

Reviewed by Kent Tamura.

Disable input color. Add INPUT_COLOR feature flag. Implement input color sanitizer.
​https://bugs.webkit.org/show_bug.cgi?id=61273

  • win/tools/vsprops/FeatureDefines.vsprops: Added INPUT_COLOR feature flag.
  • win/tools/vsprops/FeatureDefinesCairo.vsprops: Added INPUT_COLOR feature flag.
10:56 PM Changeset in webkit [87273] by Csaba Osztrogonác
  • 2 edits in trunk/LayoutTests

[Qt][WK2] Skip new failing test.

  • platform/qt-wk2/Skipped:
9:54 PM Changeset in webkit [87272] by mjs@apple.com
  • 2 edits in trunk/Source/WebKit2

2011-05-24 Maciej Stachowiak <​mjs@apple.com>

Reviewed by Dan Bernstein.

Update the WebProcess sandbox exceptions to silence warnings for a file that is accessed uselessly
​https://bugs.webkit.org/show_bug.cgi?id=61349
<rdar://problem/9476255>

  • WebProcess/com.apple.WebProcess.sb:
9:43 PM Changeset in webkit [87271] by sfalken@apple.com
  • 279 edits in branches/safari-534-branch/LayoutTests

2011-05-24 Steve Falkenburg <​sfalken@apple.com>

Rubber stamped by Maciej Stachowiak.

Update test results after ​http://trac.webkit.org/changeset/87211
which disabled several features.

9:38 PM Changeset in webkit [87270] by commit-queue@webkit.org
  • 3 edits
    2 adds
    16 deletes in trunk/LayoutTests

2011-05-24 Annie Sullivan <​sullivan@chromium.org>

Reviewed by Ryosuke Niwa.

Convert LayoutTests/editing/selection/4895428-1 and -4 to dump-as-markup
​https://bugs.webkit.org/show_bug.cgi?id=61402

Converts the two html files to dump-as-markup and adds doctype, html, and body tags.

  • editing/selection/4895428-1-expected.txt: Added.
  • editing/selection/4895428-1.html: Converted to dump-as-markup
  • editing/selection/4895428-4-expected.txt: Added.
  • editing/selection/4895428-4.html: Converted to dump-as-markup
  • platform/chromium-linux/editing/selection/4895428-1-expected.png: Removed.
  • platform/chromium-linux/editing/selection/4895428-4-expected.png: Removed.
  • platform/chromium-win/editing/selection/4895428-1-expected.png: Removed.
  • platform/chromium-win/editing/selection/4895428-1-expected.txt: Removed.
  • platform/chromium-win/editing/selection/4895428-4-expected.png: Removed.
  • platform/chromium-win/editing/selection/4895428-4-expected.txt: Removed.
  • platform/gtk/editing/selection/4895428-1-expected.png: Removed.
  • platform/gtk/editing/selection/4895428-1-expected.txt: Removed.
  • platform/gtk/editing/selection/4895428-4-expected.png: Removed.
  • platform/gtk/editing/selection/4895428-4-expected.txt: Removed.
  • platform/mac-leopard/editing/selection/4895428-1-expected.png: Removed.
  • platform/mac-leopard/editing/selection/4895428-4-expected.png: Removed.
  • platform/mac/editing/selection/4895428-1-expected.png: Removed.
  • platform/mac/editing/selection/4895428-1-expected.txt: Removed.
  • platform/mac/editing/selection/4895428-4-expected.png: Removed.
  • platform/mac/editing/selection/4895428-4-expected.txt: Removed.
9:38 PM Changeset in webkit [87269] by kevino@webkit.org
  • 2 edits in trunk/Source/JavaScriptCore

Reviewed by Eric Seidel.

Add export macros to WTFString.h.

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

9:35 PM Changeset in webkit [87268] by commit-queue@webkit.org
  • 4 edits in trunk

2011-05-24 Steve Lacey <​sjl@chromium.org>

Reviewed by Eric Carlson.

Move chromium media controls timeline back to flexible rather than absolute
layout.

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

  • platform/chromium/test_expectations.txt: Added expected failures from this change.

2011-05-24 Steve Lacey <​sjl@chromium.org>

Reviewed by Eric Carlson.

Move chromium media controls timeline back to flexible rather than absolute
layout.

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

  • css/mediaControlsChromium.css: (audio::-webkit-media-controls-current-time-display, video::-webkit-media-controls-current-time-display): (audio::-webkit-media-controls-timeline, video::-webkit-media-controls-timeline):
9:26 PM Changeset in webkit [87267] by tkent@chromium.org
  • 4 edits
    1 add
    21 deletes in trunk/LayoutTests

2011-05-24 Kent Tamura <​tkent@chromium.org>

Reviewed by Hajime Morita.

Convert fast/forms/input-type-change.html to a dumpAsText() test
​https://bugs.webkit.org/show_bug.cgi?id=61029

Merge input-type-change2 to input-type-change.html, and convert it to
a dumpAsText() test.

  • fast/forms/input-type-change-expected.txt:
  • fast/forms/input-type-change.html:
  • fast/forms/input-type-change2.html: Removed.
  • platform/chromium-linux/fast/forms/input-type-change-expected.png: Removed.
  • platform/chromium-linux/fast/forms/input-type-change2-expected.png: Removed.
  • platform/chromium-mac-leopard/fast/forms/input-type-change-expected.png: Removed.
  • platform/chromium-mac/fast/forms/input-type-change-expected.png: Removed.
  • platform/chromium-win/fast/forms/input-type-change-expected.png: Removed.
  • platform/chromium-win/fast/forms/input-type-change-expected.txt: Removed.
  • platform/chromium-win/fast/forms/input-type-change2-expected.png: Removed.
  • platform/chromium-win/fast/forms/input-type-change2-expected.txt: Removed.
  • platform/gtk/fast/forms/input-type-change-expected.png: Removed.
  • platform/gtk/fast/forms/input-type-change-expected.txt: Removed.
  • platform/gtk/fast/forms/input-type-change2-expected.png: Removed.
  • platform/gtk/fast/forms/input-type-change2-expected.txt: Removed.
  • platform/mac-leopard/fast/forms/input-type-change-expected.png: Removed.
  • platform/mac-leopard/fast/forms/input-type-change2-expected.png: Removed.
  • platform/mac/fast/forms/input-type-change-expected.png: Removed.
  • platform/mac/fast/forms/input-type-change-expected.txt:
  • platform/mac/fast/forms/input-type-change-expected.txt: Removed.
  • platform/mac/fast/forms/input-type-change2-expected.png: Removed.
  • platform/mac/fast/forms/input-type-change2-expected.txt: Removed.
  • platform/qt-arm/Skipped: Remove input-type-change2.html.
  • platform/qt-mac/Skipped: ditto.
  • platform/qt/fast/forms/input-type-change-expected.txt: Removed.
  • platform/qt/fast/forms/input-type-change2-expected.txt: Removed.
9:20 PM Changeset in webkit [87266] by ukai@chromium.org
  • 2 edits in trunk/Source/WebCore

Unreviewed. Fix clang error.

error: initialization of pointer of type 'WebCore::MHTMLArchive *' to
NULL from a constant boolean expression [-Werror,-Wbool-conversio

  • loader/archive/mhtml/MHTMLArchive.cpp:

(WebCore::MHTMLArchive::create):

9:17 PM Changeset in webkit [87265] by commit-queue@webkit.org
  • 2 edits in trunk/LayoutTests

2011-05-24 Julien Chaffraix <​jchaffraix@codeaurora.org>

Reviewed by Hajime Morita.

<legend> element is dropped on the floor
​https://bugs.webkit.org/show_bug.cgi?id=18691

  • html5lib/resources/webkit02.dat: Added a passing test for <legend> parsing.
9:14 PM Changeset in webkit [87264] by commit-queue@webkit.org
  • 2 edits
    1 add
    9 deletes in trunk/LayoutTests

2011-05-24 Annie Sullivan <​sullivan@chromium.org>

Reviewed by Tony Chang.

editing/deleting/5032066 should be a dump-as-markup test
​https://bugs.webkit.org/show_bug.cgi?id=61327

Converts 5032066 to dump-as-markup and adds doctype, html, and body tags.

  • editing/deleting/5032066-expected.txt: Added.
  • editing/deleting/5032066.html:
  • platform/chromium-linux/editing/deleting/5032066-expected.png: Removed.
  • platform/chromium-win/editing/deleting/5032066-expected.png: Removed.
  • platform/chromium-win/editing/deleting/5032066-expected.txt: Removed.
  • platform/gtk/editing/deleting/5032066-expected.txt: Removed.
  • platform/mac-leopard/editing/deleting/5032066-expected.png: Removed.
  • platform/mac/editing/deleting/5032066-expected.png: Removed.
  • platform/mac/editing/deleting/5032066-expected.txt: Removed.
  • platform/qt/editing/deleting/5032066-expected.png: Removed.
  • platform/qt/editing/deleting/5032066-expected.txt: Removed.
9:04 PM Changeset in webkit [87263] by commit-queue@webkit.org
  • 2 edits
    1 add
    9 deletes in trunk/LayoutTests

2011-05-24 Annie Sullivan <​sullivan@chromium.org>

Reviewed by Ryosuke Niwa.

Convert LayoutTests/editing/deleting/5115601.html to dump-as-markup.
​https://bugs.webkit.org/show_bug.cgi?id=61330

Converted to dump-as-markup and added a little html cleanup.

  • editing/deleting/5115601-expected.txt: Added.
  • editing/deleting/5115601.html: Convert to dump-as-markup.
  • platform/chromium-linux/editing/deleting/5115601-expected.png: Removed.
  • platform/chromium-win/editing/deleting/5115601-expected.png: Removed.
  • platform/chromium-win/editing/deleting/5115601-expected.txt: Removed.
  • platform/gtk/editing/deleting/5115601-expected.txt: Removed.
  • platform/mac-leopard/editing/deleting/5115601-expected.png: Removed.
  • platform/mac/editing/deleting/5115601-expected.png: Removed.
  • platform/mac/editing/deleting/5115601-expected.txt: Removed.
  • platform/qt/editing/deleting/5115601-expected.png: Removed.
  • platform/qt/editing/deleting/5115601-expected.txt: Removed.
8:50 PM Changeset in webkit [87262] by commit-queue@webkit.org
  • 2 edits
    1 add
    9 deletes in trunk/LayoutTests

2011-05-24 Annie Sullivan <​sullivan@chromium.org>

Reviewed by Ryosuke Niwa.

Convert LayoutTests/editing/deleting/5206311-2.html to dump-as-markup
​https://bugs.webkit.org/show_bug.cgi?id=61332

Used dump-as-markup to improve readability of results.
Also improved HTML and cleaned up descriptions.

  • editing/deleting/5206311-2-expected.txt: Added.
  • editing/deleting/5206311-2.html: Convert to dump-as-markup.
  • platform/chromium-linux/editing/deleting/5206311-2-expected.png: Removed.
  • platform/chromium-win/editing/deleting/5206311-2-expected.png: Removed.
  • platform/chromium-win/editing/deleting/5206311-2-expected.txt: Removed.
  • platform/gtk/editing/deleting/5206311-2-expected.txt: Removed.
  • platform/mac-leopard/editing/deleting/5206311-2-expected.png: Removed.
  • platform/mac/editing/deleting/5206311-2-expected.png: Removed.
  • platform/mac/editing/deleting/5206311-2-expected.txt: Removed.
  • platform/qt/editing/deleting/5206311-2-expected.png: Removed.
  • platform/qt/editing/deleting/5206311-2-expected.txt: Removed.
8:39 PM Changeset in webkit [87261] by commit-queue@webkit.org
  • 2 edits
    1 add
    9 deletes in trunk/LayoutTests

2011-05-24 Annie Sullivan <​sullivan@chromium.org>

Reviewed by Ryosuke Niwa.

Convert LayoutTests/editing/deleting/delete-block-table.html to dumpAsText test.
​https://bugs.webkit.org/show_bug.cgi?id=61333

Converts delete-block-table.html to dumpAsText test by changing to use runDumpAsTextEditingTest.

  • editing/deleting/delete-block-table-expected.txt: Added.
  • editing/deleting/delete-block-table.html: Converted to dump-as-markup.
  • platform/chromium-linux/editing/deleting/delete-block-table-expected.png: Removed.
  • platform/chromium-win/editing/deleting/delete-block-table-expected.png: Removed.
  • platform/chromium-win/editing/deleting/delete-block-table-expected.txt: Removed.
  • platform/gtk/editing/deleting/delete-block-table-expected.png: Removed.
  • platform/gtk/editing/deleting/delete-block-table-expected.txt: Removed.
  • platform/mac-leopard/editing/deleting/delete-block-table-expected.png: Removed.
  • platform/mac/editing/deleting/delete-block-table-expected.png: Removed.
  • platform/mac/editing/deleting/delete-block-table-expected.txt: Removed.
  • platform/qt/editing/deleting/delete-block-table-expected.txt: Removed.
8:39 PM Changeset in webkit [87260] by abarth@webkit.org
  • 10 edits in trunk/Source

2011-05-24 Adam Barth <​abarth@webkit.org>

Reviewed by Eric Seidel.

Add FIXME comments about removing FrameLoader::isProcessingUserGesture
​https://bugs.webkit.org/show_bug.cgi?id=61395

All the callers of the API are wrong. They should call the more modern
static member function of ScriptController instead. I'll remove the
FrameLoader function in a future patch.

  • html/HTMLMediaElement.cpp: (WebCore::HTMLMediaElement::processingUserGesture):
  • rendering/RenderFileUploadControl.cpp: (WebCore::RenderFileUploadControl::click):

2011-05-24 Adam Barth <​abarth@webkit.org>

Reviewed by Eric Seidel.

Add FIXME comments about removing FrameLoader::isProcessingUserGesture
​https://bugs.webkit.org/show_bug.cgi?id=61395

  • src/FrameLoaderClientImpl.cpp: (WebKit::FrameLoaderClientImpl::dispatchDidNavigateWithinPage):
  • src/WebFrameImpl.cpp: (WebKit::WebFrameImpl::isProcessingUserGesture):

2011-05-24 Adam Barth <​abarth@webkit.org>

Reviewed by Eric Seidel.

Add FIXME comments about removing FrameLoader::isProcessingUserGesture
​https://bugs.webkit.org/show_bug.cgi?id=61395

  • WebView/WebView.mm: (-[WebView _isProcessingUserGesture]):

2011-05-24 Adam Barth <​abarth@webkit.org>

Reviewed by Eric Seidel.

Add FIXME comments about removing FrameLoader::isProcessingUserGesture
​https://bugs.webkit.org/show_bug.cgi?id=61395

  • WebCoreSupport/FrameLoaderClientQt.cpp: (drtPrintFrameUserGestureStatus):
8:28 PM Changeset in webkit [87259] by commit-queue@webkit.org
  • 2 edits in trunk/LayoutTests

2011-05-24 Justin Novosad <​junov@chromium.org>

Reviewed by Stephen White.

[Chromium] cleanup test expectations for drawImage that are no longer
flaky
​https://bugs.webkit.org/show_bug.cgi?id=61389

  • platform/chromium/test_expectations.txt:
8:03 PM Changeset in webkit [87258] by gyuyoung.kim@samsung.com
  • 2 edits in trunk/Source/WebCore

2011-05-24 Gyuyoung Kim <​gyuyoung.kim@samsung.com>

Unreviewed. Fix style error.

  • platform/efl/WidgetEfl.cpp: (WebCore::WidgetPrivate::WidgetPrivate):
8:00 PM Changeset in webkit [87257] by commit-queue@webkit.org
  • 20 edits
    2 adds in trunk

2011-05-24 Greg Simon <​gregsimon@chromium.org>

Reviewed by Dimitri Glazkov.

Control Indexeddb backends from LayoutTestController
​https://bugs.webkit.org/show_bug.cgi?id=61000

LayoutTest for migration between SQLite and LevelDB
in IndxededDB.

  • storage/indexeddb/migrate-basics-expected.txt: Added.
  • storage/indexeddb/migrate-basics.html: Added.

2011-05-24 Greg Simon <​gregsimon@chromium.org>

Reviewed by Dimitri Glazkov.

Control Indexeddb backends from LayoutTestController
​https://bugs.webkit.org/show_bug.cgi?id=61000

Test: storage/indexeddb/migrate-basics.html

  • storage/IDBBackingStore.h:
  • storage/IDBFactoryBackendImpl.cpp: (WebCore::IDBFactoryBackendImpl::IDBFactoryBackendImpl): (WebCore::IDBFactoryBackendImpl::open): (WebCore::IDBFactoryBackendImpl::setEnableMigration): (WebCore::IDBFactoryBackendImpl::migrate):
  • storage/IDBFactoryBackendImpl.h:
  • storage/IDBFactoryBackendInterface.h:
  • storage/IDBLevelDBBackingStore.cpp: (WebCore::IDBLevelDBBackingStore::backingStoreExists):
  • storage/IDBLevelDBBackingStore.h: (WebCore::IDBLevelDBBackingStore::backingStoreType):
  • storage/IDBSQLiteBackingStore.cpp: (WebCore::IDBSQLiteBackingStore::backingStoreExists):
  • storage/IDBSQLiteBackingStore.h: (WebCore::IDBSQLiteBackingStore::backingStoreType):

2011-05-24 Greg Simon <​gregsimon@chromium.org>

Reviewed by Dimitri Glazkov.

Control Indexeddb backends from LayoutTestController
​https://bugs.webkit.org/show_bug.cgi?id=61000

  • public/WebIDBFactory.h: (WebKit::WebIDBFactory::setEnableMigration):
  • src/AssertMatchingEnums.cpp:
  • src/IDBFactoryBackendProxy.cpp: (WebKit::IDBFactoryBackendProxy::setEnableMigration):
  • src/IDBFactoryBackendProxy.h:
  • src/WebIDBFactoryImpl.cpp: (WebKit::WebIDBFactory::setOverrideBackingStoreType): (WebKit::WebIDBFactory::setTemporaryDatabaseFolder): (WebKit::WebIDBFactoryImpl::WebIDBFactoryImpl): (WebKit::WebIDBFactoryImpl::open): (WebKit::WebIDBFactoryImpl::setEnableMigration):
  • src/WebIDBFactoryImpl.h:

2011-05-24 Greg Simon <​gregsimon@chromium.org>

Reviewed by Dimitri Glazkov.

Control Indexeddb backends from LayoutTestController
​https://bugs.webkit.org/show_bug.cgi?id=61000

  • DumpRenderTree/chromium/LayoutTestController.cpp: (LayoutTestController::LayoutTestController): (LayoutTestController::setOverrideIndexedDBBackingStore): (LayoutTestController::clearAllDatabases):
  • DumpRenderTree/chromium/LayoutTestController.h:
7:54 PM Changeset in webkit [87256] by ukai@chromium.org
  • 2 edits in trunk/LayoutTests

Unreviewed.

Chromium expectations update. (mhtml)

  • platform/chromium/test_expectations.txt:
7:52 PM Changeset in webkit [87255] by mrowe@apple.com
  • 2 edits in trunk/Source/WebKit2

Fix the 32-bit build.

Explicitly use uint32_t when encoding / decoding a type that is a typedef of OSType,
as this is declared as different underlying types in 32- and 64-bit (unsigned long
vs unsigned int).

  • Shared/mac/KeychainAttribute.cpp:

(CoreIPC::encode):
(CoreIPC::decode):

7:35 PM Changeset in webkit [87254] by commit-queue@webkit.org
  • 2 edits in trunk/LayoutTests

2011-05-24 Leon Scroggins <​scroggo@google.com>

Reviewed by Kenneth Russell.

Update expectations for tests drawing a canvas into another canvas (or itself), which have been fixed in gpu.
​https://bugs.webkit.org/show_bug.cgi?id=61362

  • platform/chromium/test_expectations.txt:
7:22 PM Changeset in webkit [87253] by weinig@apple.com
  • 2 edits in trunk/Source/WebKit2

Enable coalesced wheel events
<rdar://problem/9444786>

Reviewed by Maciej Stachowiak.

  • UIProcess/WebPageProxy.cpp:

Flip the switch for coalescing wheel events rather than dropping them.

7:18 PM Changeset in webkit [87252] by Lucas Forschler
  • 5 edits in branches/safari-534-branch/Source

Versioning.

7:16 PM Changeset in webkit [87251] by Lucas Forschler
  • 1 copy in tags/Safari-534.39

New tag.

7:14 PM Changeset in webkit [87250] by Lucas Forschler
  • 9 edits
    8 deletes in branches/safari-534-branch/Source/WebKit2

Rollout 87221.

7:07 PM Changeset in webkit [87249] by rniwa@webkit.org
  • 2 edits in trunk/LayoutTests

2011-05-24 Ryosuke Niwa <​rniwa@webkit.org>

Final attempt to fix a test; if this fails, I'm just going to skip it.

  • fast/events/selectstart-by-double-triple-clicks.html:
6:26 PM Changeset in webkit [87248] by leo.yang@torchmobile.com.cn
  • 4 edits
    2 adds in trunk

2011-05-24 Leo Yang <​leo.yang@torchmobile.com.cn>

Reviewed by Ryosuke Niwa.

[SVG] Assertion failure by dragging text between input fields inside <foreignObject>s
​https://bugs.webkit.org/show_bug.cgi?id=60692

Add NULL check of 'holder' in WebCore::ReplacementFragment::ReplacementFragment()
before call VisibleSelection::selectionFromContentsOfNode() to prevent crashing.

Test: editing/pasteboard/drag-drop-input-in-svg.svg

  • editing/ReplaceSelectionCommand.cpp: (WebCore::ReplacementFragment::ReplacementFragment):

2011-05-24 Leo Yang <​leo.yang@torchmobile.com.cn>

Reviewed by Ryosuke Niwa.

[SVG] Assertion failure by dragging text between input fields inside <foreignObject>s
​https://bugs.webkit.org/show_bug.cgi?id=60692

Test case to verify dragging and dropping text between input fields
inside SVG <foreignObject>s. Test passes if no crash occurs in debug mode.

Drag and drop is not supported in Qt DRT, so skip this case on Qt platform.

  • editing/pasteboard/drag-drop-input-in-svg-expected.txt: Added.
  • editing/pasteboard/drag-drop-input-in-svg.svg: Added.
  • platform/qt/Skipped:
6:05 PM Changeset in webkit [87247] by rniwa@webkit.org
  • 2 edits in trunk/LayoutTests

2011-05-24 Ryosuke Niwa <​rniwa@webkit.org>

Yet another attempt to make Chromium Windows test bots happy.

  • fast/events/selectstart-by-double-triple-clicks.html:
5:59 PM Changeset in webkit [87246] by Lucas Forschler
  • 3 edits
    2 copies in branches/safari-534-branch

Merge r87187.

5:56 PM Changeset in webkit [87245] by Lucas Forschler
  • 9 edits
    2 copies in branches/safari-534-branch

Merge r87179 + fix.

5:48 PM Changeset in webkit [87244] by aestes@apple.com
  • 3 edits in trunk/Source/WebCore

2011-05-24 Andy Estes <​aestes@apple.com>

Reviewed by Geoffrey Garen.

REGRESSION (r70748): WebKit cannot play QuickTime movies on Mac OS X Wiki Server pages
​https://bugs.webkit.org/show_bug.cgi?id=61229

This site-specific hack maintains compatibility with Mac OS X Wiki Server,
which embeds QuickTime movies using an object tag containing QuickTime's
ActiveX classid. Treat this classid as valid only if OS X Server's unique
'generator' meta tag is present. Only apply this quirk if there is no
fallback content, which ensures the quirk will disable itself if Wiki
Server is updated to generate an alternate embed tag as fallback content.

  • html/HTMLObjectElement.cpp: (WebCore::HTMLObjectElement::shouldAllowQuickTimeClassIdQuirk): Return true if site-specific quirks are enabled, the object element has no fallback content, the classid attribute matches QuickTime's classid and the document has a 'generator' meta tag matching Mac OS X Web Services Server's unique generator string. (WebCore::HTMLObjectElement::hasValidClassId): Call shouldAllowQuickTimeClassIdQuirk()
  • html/HTMLObjectElement.h:
5:33 PM Changeset in webkit [87243] by ggaren@apple.com
  • 2 edits in trunk/Source/JavaScriptCore

Maybe fix the Mac build now?

5:29 PM Changeset in webkit [87242] by Lucas Forschler
  • 2 edits in branches/safari-534-branch/Source/WebKit2

Merge r87170.

5:27 PM Changeset in webkit [87241] by Lucas Forschler
  • 9 edits
    8 copies in branches/safari-534-branch/Source/WebKit2

Merge r87221.

5:25 PM Changeset in webkit [87240] by Lucas Forschler
  • 2 edits in branches/safari-534-branch/Source/WebKit2

Merge r87154.

5:24 PM Changeset in webkit [87239] by Nate Chapin
  • 29 edits in trunk/Source/WebCore

2011-05-24 Nate Chapin <Nate Chapin>

Reviewed by Adam Barth.

Change CachedResources to take a ResourceRequest instead of
a url in their constructors and provide a very minimal set of
cases for avoiding reusing a CachedResource. The ResourceRequest
part of this change requires pushing responsibility
for calling Document::completeURL() to the caller, instead of
CachedResourceLoader, since ResourceRequest ASSERTs if it
is constructed with an invalid url.

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

Refactor, no new tests.

  • css/CSSFontSelector.cpp: (WebCore::CSSFontSelector::addFontFaceRule):
  • css/CSSImageValue.cpp: (WebCore::CSSImageValue::cachedImage):
  • css/CSSImportRule.cpp: (WebCore::CSSImportRule::insertedIntoParent):
  • dom/ProcessingInstruction.cpp: (WebCore::ProcessingInstruction::checkStyleSheet):
  • dom/ScriptElement.cpp: (WebCore::ScriptElement::requestScript):
  • html/HTMLLinkElement.cpp: (WebCore::HTMLLinkElement::process):
  • loader/ImageLoader.cpp: (WebCore::ImageLoader::updateFromElement):
  • loader/cache/CachedCSSStyleSheet.cpp: (WebCore::CachedCSSStyleSheet::CachedCSSStyleSheet): (WebCore::CachedCSSStyleSheet::didAddClient): (WebCore::CachedCSSStyleSheet::checkNotify):
  • loader/cache/CachedCSSStyleSheet.h:
  • loader/cache/CachedFont.cpp: (WebCore::CachedFont::CachedFont):
  • loader/cache/CachedFont.h:
  • loader/cache/CachedImage.cpp: (WebCore::CachedImage::CachedImage): (WebCore::CachedImage::checkShouldPaintBrokenImage):
  • loader/cache/CachedImage.h:
  • loader/cache/CachedResource.cpp: (WebCore::CachedResource::CachedResource): (WebCore::reuseRequest): (WebCore::CachedResource::allowReuseOfRequest): (WebCore::CachedResource::removeClient): (WebCore::CachedResource::canUseCacheValidator):
  • loader/cache/CachedResource.h: (WebCore::CachedResource::resourceRequest): (WebCore::CachedResource::url):
  • loader/cache/CachedResourceLoader.cpp:
  • loader/cache/CachedResourceLoader.h:
  • loader/cache/CachedResourceRequest.cpp: (WebCore::CachedResourceRequest::load):
  • loader/cache/CachedScript.cpp: (WebCore::CachedScript::CachedScript):
  • loader/cache/CachedScript.h:
  • loader/cache/CachedXSLStyleSheet.cpp: (WebCore::CachedXSLStyleSheet::CachedXSLStyleSheet): (WebCore::CachedXSLStyleSheet::didAddClient): (WebCore::CachedXSLStyleSheet::checkNotify):
  • loader/cache/CachedXSLStyleSheet.h:
  • svg/SVGFEImageElement.cpp: (WebCore::SVGFEImageElement::requestImageResource):
  • svg/SVGFontFaceUriElement.cpp: (WebCore::SVGFontFaceUriElement::loadFont):
  • xml/XSLImportRule.cpp: (WebCore::XSLImportRule::loadSheet):
5:24 PM Changeset in webkit [87238] by Lucas Forschler
  • 5 edits
    2 copies in branches/safari-534-branch/Source/WebKit2

Merge r87153.

5:23 PM Changeset in webkit [87237] by weinig@apple.com
  • 2 edits in trunk/Source/WebKit2

2011-05-24 Sam Weinig <​sam@webkit.org>

Reviewed by Maciej Stachowiak.

Additional sandbox extension required for quarantine database
<rdar://problem/9397494>
​https://bugs.webkit.org/show_bug.cgi?id=61405

  • WebProcess/com.apple.WebProcess.sb: Add read-data access for ~/Library/Preferences.
5:18 PM Changeset in webkit [87236] by Lucas Forschler
  • 2 edits in branches/safari-534-branch/Source/WebKit2

Merge r87119.

5:18 PM Changeset in webkit [87235] by tkent@chromium.org
  • 9 edits in trunk/Source/WebCore

2011-05-24 Kent Tamura <​tkent@chromium.org>

Reviewed by Dimitri Glazkov.

Use Element::shadowPseudoId() for text field decoration elements
​https://bugs.webkit.org/show_bug.cgi?id=61351

No new tests. Covered by existing tests.

  • css/CSSSelector.cpp: Remove support for the following static pseudo selectors:
    • -webkit-inner-spin-button
    • -webkit-input-speech-button
    • -webkit-outer-spin-button
    • -webkit-search-cancel-button
    • -webkit-search-decoration
    • -webkit-search-results-button
    • -webkit-search-results-decoration

(WebCore::CSSSelector::pseudoId):
(WebCore::nameToPseudoTypeMap):
(WebCore::CSSSelector::extractPseudoType):

  • css/CSSSelector.h: ditto.
  • css/CSSStyleSelector.cpp: (WebCore::CSSStyleSelector::matchRules): Store shadowPseudoId() result in a local variable because SearchFieldResultsButtonElement::shadowPseudoId() is a little expensive.
  • html/shadow/TextControlInnerElements.cpp: Override Element::shadowPseudoId() instead of Element::styleForRenderer(). (WebCore::SearchFieldResultsButtonElement::shadowPseudoId): (WebCore::SearchFieldCancelButtonElement::shadowPseudoId): (WebCore::SpinButtonElement::shadowPseudoId): (WebCore::InputFieldSpeechButtonElement::shadowPseudoId):
  • html/shadow/TextControlInnerElements.h: Update declarations.
  • rendering/RenderTextControlSingleLine.cpp: Remove createFooStyle() functions.
  • rendering/RenderTextControlSingleLine.h: ditto.
  • rendering/style/RenderStyleConstants.h: Remove unused pseudo IDs.
5:16 PM Changeset in webkit [87234] by Lucas Forschler
  • 5 edits in branches/safari-534-branch/Source/JavaScriptCore

Merge r87109.

5:12 PM Changeset in webkit [87233] by ggaren@apple.com
  • 2 edits in trunk/Source/JavaScriptCore

Maybe fix the Mac build?

5:00 PM Changeset in webkit [87232] by Lucas Forschler
  • 16 edits in branches/safari-534-branch/Source

Merge r87102.

4:49 PM Changeset in webkit [87231] by Dimitri Glazkov
  • 2 edits in trunk/Source/WebCore

2011-05-24 Dimitri Glazkov <Dimitri Glazkov>

Sorted WebCore.xcodeproj again.

  • WebCore.xcodeproj/project.pbxproj: Ran sort-XCode-project-file.
4:46 PM Changeset in webkit [87230] by ggaren@apple.com
  • 11 edits
    1 copy in trunk/Source/JavaScriptCore

2011-05-24 Geoffrey Garen <​ggaren@apple.com>

Reviewed by Oliver Hunt.

Split HeapRootVisitor into its own class
​https://bugs.webkit.org/show_bug.cgi?id=61399

  • GNUmakefile.list.am:
  • JavaScriptCore.gypi:
  • JavaScriptCore.vcproj/JavaScriptCore/JavaScriptCore.vcproj:
  • JavaScriptCore.xcodeproj/project.pbxproj:
  • heap/HandleHeap.cpp:
  • heap/HandleStack.cpp:
  • heap/Heap.cpp:
  • heap/HeapRootVisitor.h: Copied from Source/JavaScriptCore/heap/MarkStack.h.
  • heap/MarkStack.h:
  • runtime/ArgList.cpp:
  • runtime/SmallStrings.cpp:
4:46 PM Changeset in webkit [87229] by Stephanie Lewis
  • 6 edits in trunk/Source/WebCore

​https://bugs.webkit.org/show_bug.cgi?id=61345
part of <rdar://problem/8675177>
Don't initialize ResourceResponse.m_suggestedFilename until we want to use it. Initializing it requires reading in and parsing a plist.
Add new initialization state CommonAndUncommonFields to ResourceResponse. This will be for all the header fields.
Move suggestedFilename initialization to AllFields so it is never initialized unless we ask for it.
Add "Content-Type" to common headers since it is looked for by every CSS sheet load.

Reviewed by Geoff Garen.

No new tests because functionality has not changed.

  • platform/network/ResourceResponseBase.cpp:

(WebCore::ResourceResponseBase::adopt):
(WebCore::ResourceResponseBase::suggestedFilename):
(WebCore::ResourceResponseBase::setSuggestedFilename):
(WebCore::ResourceResponseBase::httpStatusText):
(WebCore::ResourceResponseBase::setHTTPStatusText):
(WebCore::ResourceResponseBase::httpHeaderField):
(WebCore::ResourceResponseBase::setHTTPHeaderField):
(WebCore::ResourceResponseBase::httpHeaderFields):
(WebCore::ResourceResponseBase::isAttachment):
(WebCore::ResourceResponseBase::setLastModifiedDate):
(WebCore::ResourceResponseBase::lastModifiedDate):
(WebCore::ResourceResponseBase::wasCached):
(WebCore::ResourceResponseBase::connectionReused):
(WebCore::ResourceResponseBase::setConnectionReused):
(WebCore::ResourceResponseBase::connectionID):
(WebCore::ResourceResponseBase::setConnectionID):
(WebCore::ResourceResponseBase::resourceLoadTiming):
(WebCore::ResourceResponseBase::setResourceLoadTiming):
(WebCore::ResourceResponseBase::resourceLoadInfo):
(WebCore::ResourceResponseBase::setResourceLoadInfo):

  • platform/network/ResourceResponseBase.h:
  • platform/network/cf/ResourceResponse.h:

(WebCore::ResourceResponse::ResourceResponse):

  • platform/network/cf/ResourceResponseCFNet.cpp:

(WebCore::ResourceResponse::platformLazyInit):

  • platform/network/mac/ResourceResponseMac.mm:

(WebCore::ResourceResponse::platformLazyInit):

4:43 PM Changeset in webkit [87228] by msaboff@apple.com
  • 13 edits
    3 adds in trunk/Source

2011-05-24 Michael Saboff <​msaboff@apple.com>

Reviewed by Geoffrey Garen.

Improve handling in WebCore of low memory situations
​https://bugs.webkit.org/show_bug.cgi?id=61222

Added call to enable memory pressure handling.

  • WebView/WebView.mm: (-[WebView _commonInitializationWithFrameName:groupName:usesDocumentViews:]):
  • WebView/WebViewInternal.h:

2011-05-24 Michael Saboff <​msaboff@apple.com>

Reviewed by Geoffrey Garen.

Improve handling in WebCore of low memory situations
​https://bugs.webkit.org/show_bug.cgi?id=61222

Added new class, MemoryPressureHandler, to platform to respond to low
system memory events. Added Mac specific implementation that reduces the
page cache, NSURL cache, WebCore cache by half their current usage
and then garbage collects. This can allow Safari to continue operating
in a low memory situation with minimal paging.
Split up prune operations in MemoryCache to allow for pruning to a
specific actual size to support this change.

No tests added as the functionality of WebKit wasn't changed.

  • GNUmakefile.list.am:
  • WebCore.exp.in:
  • WebCore.gypi:
  • WebCore.pro:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • loader/cache/MemoryCache.cpp: (WebCore::MemoryCache::pruneLiveResources): (WebCore::MemoryCache::forcePruneLiveResources): (WebCore::MemoryCache::pruneLiveResourcesToTarget): (WebCore::MemoryCache::pruneDeadResources): (WebCore::MemoryCache::forcePruneDeadResources): (WebCore::MemoryCache::pruneDeadResourcesToTarget):
  • loader/cache/MemoryCache.h: (WebCore::MemoryCache::forcePrune):
  • platform/MemoryPressureHandler.cpp: Added. (WebCore::memoryPressureHandler): (WebCore::MemoryPressureHandler::MemoryPressureHandler): (WebCore::MemoryPressureHandler::install): (WebCore::MemoryPressureHandler::respondToMemoryPressure):
  • platform/MemoryPressureHandler.h: Added.
  • platform/mac/MemoryPressureHandler.mm: Added. (WebCore::MemoryPressureHandler::install): (WebCore::MemoryPressureHandler::respondToMemoryPressure):

2011-05-24 Michael Saboff <​msaboff@apple.com>

Reviewed by Geoffrey Garen.

Improve handling in WebCore of low memory situations
​https://bugs.webkit.org/show_bug.cgi?id=61222

Added call to enable memory pressure handling for Mac case.

  • WebProcess/WebProcess.cpp: (WebKit::WebProcess::initializeWebProcess):
  • WebProcess/WebProcess.h:
  • WebProcess/mac/WebProcessMac.mm: (WebKit::WebProcess::platformInitializeWebProcess):
4:19 PM Changeset in webkit [87227] by tkent@chromium.org
  • 4 edits in trunk/Source/WebCore

2011-05-24 Kent Tamura <​tkent@chromium.org>

Reviewed by Dimitri Glazkov.

Node::shadowAncestorNode() and shadowTreeRootNode() should be const.
​https://bugs.webkit.org/show_bug.cgi?id=61398

shadowAncestorNode() and shadowTreeRootNode() should be const
though they can return 'this' pointer.

No new tests. This doesn't change any visible behavior.

  • dom/Node.cpp: (WebCore::Node::shadowAncestorNode): Make this const. (WebCore::Node::shadowTreeRootNode): ditto.
  • dom/Node.h: Update declarations.
  • html/shadow/TextControlInnerElements.h: (WebCore::SpinButtonElement::isEnabledFormControl): Need no const_cast<>. (WebCore::SpinButtonElement::isReadOnlyFormControl): ditto.
4:13 PM Changeset in webkit [87226] by Lucas Forschler
  • 1 edit in branches/safari-534-branch/Source/JavaScriptCore/wtf/Platform.h

Merge radar 9489360.

4:03 PM Changeset in webkit [87225] by Lucas Forschler
  • 5 edits in branches/safari-534-branch/Source/WebKit2

Merge r87113.

4:01 PM Changeset in webkit [87224] by Lucas Forschler
  • 2 edits in branches/safari-534-branch/Source/WebKit2

Merge r87093.

3:58 PM Changeset in webkit [87223] by Lucas Forschler
  • 3 edits
    2 copies in branches/safari-534-branch

Merge r87083.

3:53 PM Changeset in webkit [87222] by Lucas Forschler
  • 2 edits in branches/safari-534-branch/Source/WebKit2

Merge r87077.

3:52 PM Changeset in webkit [87221] by beidson@apple.com
  • 9 edits
    8 adds in trunk/Source/WebKit2

Part of <rdar://problem/9434311> and ​https://bugs.webkit.org/show_bug.cgi?id=61334
Should shim a few SecKeychainItem* methods on SnowLeopard

Reviewed by Darin Adler.

Initialize the KeychainItem shim on SnowLeopard:

  • WebProcess/mac/WebProcessMac.mm:

(WebKit::WebProcess::initializeShim):

On SnowLeopard, shim the 4 SecKeychainItem methods:

  • WebProcess/mac/WebProcessShim.h:
  • WebProcess/mac/WebProcessShim.mm:

(WebKit::shimSecKeychainItemCopyContent):
(WebKit::shimSecKeychainItemCreateFromContent):
(WebKit::shimSecKeychainItemModifyContent):
(WebKit::shimSecKeychainItemFreeContent):
(WebKit::WebKitWebProcessKeychainItemShimInitialize):

  • WebProcess/mac/KeychainItemShimMethods.h: Added.
  • WebProcess/mac/KeychainItemShimMethods.mm: Added.

Manage a set of shim-controlled AttributeList data:
(WebKit::shimManagedAttributeLists):
(WebKit::freeAttributeListContents):
(WebKit::allocateAttributeListContents):

Manage a set of shim-controlled KeychainItem data:
(WebKit::shimManagedKeychainItemContents):
(WebKit::allocateKeychainItemContentData):

Free any shim-controlled AttributeList or KeychainItem data:
(WebKit::webFreeAttributeListContentOnMainThread):
(WebKit::webFreeAttributeListContent):
(WebKit::webFreeKeychainItemContentOnMainThread):
(WebKit::webFreeKeychainItemContent):

Marshall these 3 KeychainItem methods to the UIProcess:
(WebKit::webSecKeychainItemCopyContentOnMainThread):
(WebKit::webSecKeychainItemCopyContent):
(WebKit::webSecKeychainItemCreateFromContentOnMainThread):
(WebKit::webSecKeychainItemCreateFromContent):
(WebKit::webSecKeychainItemModifyContentOnMainThread):
(WebKit::webSecKeychainItemModifyContent):
(WebKit::initializeKeychainItemShim):

Container for an AttributeList entry that knows how to encode/decode:

  • Shared/mac/KeychainAttribute.cpp: Added.

(WebKit::KeychainAttribute::KeychainAttribute):
(CoreIPC::encode):
(CoreIPC::decode):

  • Shared/mac/KeychainAttribute.h: Added.

Container for the relevant arguments to the shimmed methods, to send to the UIProcess:

  • Shared/mac/SecKeychainItemRequestData.cpp: Added.

(WebKit::SecKeychainItemRequestData::SecKeychainItemRequestData):
(WebKit::SecKeychainItemRequestData::initializeWithAttributeList):
(WebKit::SecKeychainItemRequestData::~SecKeychainItemRequestData):
(WebKit::SecKeychainItemRequestData::attributeList):
(WebKit::SecKeychainItemRequestData::encode):
(WebKit::SecKeychainItemRequestData::decode):

  • Shared/mac/SecKeychainItemRequestData.h: Added.

(WebKit::SecKeychainItemRequestData::keychainItem):
(WebKit::SecKeychainItemRequestData::itemClass):
(WebKit::SecKeychainItemRequestData::length):
(WebKit::SecKeychainItemRequestData::data):

Container for the possible output from the shimmed methods, to send back to the WebProcess:

  • Shared/mac/SecKeychainItemResponseData.cpp: Added.

(WebKit::SecKeychainItemResponseData::SecKeychainItemResponseData):
(WebKit::SecKeychainItemResponseData::encode):
(WebKit::SecKeychainItemResponseData::decode):

  • Shared/mac/SecKeychainItemResponseData.h: Added.

(WebKit::SecKeychainItemResponseData::itemClass):
(WebKit::SecKeychainItemResponseData::data):
(WebKit::SecKeychainItemResponseData::resultCode):
(WebKit::SecKeychainItemResponseData::attributes):
(WebKit::SecKeychainItemResponseData::keychainItem):

Actually implement the calls to the SecKeychainItem* methods in the UIProcess:

  • UIProcess/WebProcessProxy.h:
  • UIProcess/WebProcessProxy.messages.in:
  • UIProcess/mac/WebProcessProxyMac.mm:

(WebKit::WebProcessProxy::secKeychainItemCopyContent):
(WebKit::WebProcessProxy::secKeychainItemCreateFromContent):
(WebKit::WebProcessProxy::secKeychainItemModifyContent):

  • WebKit2.xcodeproj/project.pbxproj: Project file stuff
  • Configurations/Shim.xcconfig:
3:27 PM Changeset in webkit [87220] by Lucas Forschler
  • 2 edits in branches/safari-534-branch/Source/WebKit2

Merge r87046.

3:23 PM Changeset in webkit [87219] by Lucas Forschler
  • 3 edits
    3 copies in branches/safari-534-branch

Merge r87026.

3:22 PM Changeset in webkit [87218] by rniwa@webkit.org
  • 3 edits
    1 add in trunk/LayoutTests

2011-05-24 Ryosuke Niwa <​rniwa@webkit.org>

Reviewed by Tony Chang.

[chromium] selectstart-by-drag.html and selectstart-by-double-triple-clicks.html
fail on Windows and Linux
​https://bugs.webkit.org/show_bug.cgi?id=61377

selectstart-by-drag.html pass on Chromium Windows and Chromium Linux so don't use
the failing test expectation on Windows port.

In selectstart-by-double-triple-clicks.html, /\s|\s$/ isn't recognized by V8 so
replace it by /[ \t]|[ \t]$/.

  • platform/chromium/fast/events/selectstart-by-drag-expected.txt: Copied from LayoutTests/fast/events/selectstart-by-drag-expected.txt.
  • fast/events/selectstart-by-double-triple-clicks.html:
  • platform/chromium/test_expectations.txt:
3:20 PM Changeset in webkit [87217] by Lucas Forschler
  • 4 edits
    3 copies in branches/safari-534-branch

Merge r87009.

3:17 PM Changeset in webkit [87216] by Lucas Forschler
  • 2 edits in branches/safari-534-branch/Source/WebKit2

Merge r87005.

3:12 PM Changeset in webkit [87215] by simonjam@chromium.org
  • 5 edits in trunk

2011-05-24 James Simonsen <​simonjam@chromium.org>

Reviewed by Eric Seidel.

ASSERT_NOT_REACHED reached with broken ideograph and system fallback
​https://bugs.webkit.org/show_bug.cgi?id=53528

  • platform/chromium/test_expectations.txt: Remove CRASH from fixed test.

2011-05-24 James Simonsen <​simonjam@chromium.org>

Reviewed by Eric Seidel.

ASSERT_NOT_REACHED reached with broken ideograph and system fallback
​https://bugs.webkit.org/show_bug.cgi?id=53528

Test: fast/text/justify-ideograph-vertical.html (on chromium linux)

  • platform/graphics/FontFastPath.cpp: (WebCore::Font::glyphDataForCharacter): Identify and handle broken ideographs as any other variant.
  • platform/graphics/SimpleFontData.h: (WebCore::SimpleFontData::variantFontData): Added BrokenIdeographVariant.
3:10 PM Changeset in webkit [87214] by rniwa@webkit.org
  • 2 edits in trunk/LayoutTests

2011-05-24 Ryosuke Niwa <​rniwa@webkit.org>

Skip fast/events/scroll-in-scaled-page-with-overflow-hidden.html on Mac WebKit2
since WebKitTestRunner doesn't implement eventSender.mouseScrollBy yet (bug 42194).

  • platform/mac-wk2/Skipped:
3:09 PM Changeset in webkit [87213] by abarth@webkit.org
  • 6 edits in trunk/Source

2011-05-24 Jay Civelli <​jcivelli@chromium.org>

Rubberstamped by David Kilzer.

Updated some files that I forgot in my previous MHTML CL.

  • Configurations/FeatureDefines.xcconfig:

2011-05-24 Jay Civelli <​jcivelli@chromium.org>

Rubberstamped by David Kilzer.

Updated some files that I forgot in my previous MHTML CL.

  • Configurations/FeatureDefines.xcconfig:

2011-05-24 Jay Civelli <​jcivelli@chromium.org>

Rubberstamped by David Kilzer.

Updated some files that I forgot in my previous MHTML CL.

  • Configurations/FeatureDefines.xcconfig:
3:06 PM Changeset in webkit [87212] by ap@apple.com
  • 2 edits in trunk/LayoutTests

Skip platform/mac/editing/input/maxlength.html - textInputController is not implemented.

  • platform/mac-wk2/Skipped:
3:04 PM Changeset in webkit [87211] by sfalken@apple.com
  • 10 edits in branches/safari-534-branch

2011-05-24 Steve Falkenburg <​sfalken@apple.com>

Reviewed by Adam Roben.

Disable features on safari-534-branch.
<rdar://problem/9261347>

This change requires a full rebuild.

2:59 PM Changeset in webkit [87210] by robert@webkit.org
  • 5 edits in trunk

2011-05-24 Robert Hogan <​robert@webkit.org>

Reviewed by Ryosuke Niwa.

[Qt] Fix fast/events/selectstart-by-single-click-with-shift.html
​https://bugs.webkit.org/show_bug.cgi?id=61393

  • platform/qt/Skipped: Unskip fast/events/selectstart-by-single-click-with-shift.html

2011-05-24 Robert Hogan <​robert@webkit.org>

Reviewed by Ryosuke Niwa.

[Qt] Fix fast/events/selectstart-by-single-click-with-shift.html
​https://bugs.webkit.org/show_bug.cgi?id=61393

  • DumpRenderTree/qt/EventSenderQt.h:
  • DumpRenderTree/qt/EventSenderQt.cpp: (getModifiers): static function for interpreting modifiers (EventSender::mouseDown): Respect keyboard modifiers (EventSender::keyDown): use new static function (EventSender::sendOrQueueEvent): whitespace fix
2:56 PM Changeset in webkit [87209] by rniwa@webkit.org
  • 2 edits in trunk/LayoutTests

2011-05-24 Ryosuke Niwa <​rniwa@webkit.org>

Skip fast/events/selectstart-by-double-triple-clicks.html and selectstart-by-drag.html
on Mac WebKit 2 per the bug 42194.

  • platform/mac-wk2/Skipped:
2:47 PM Changeset in webkit [87208] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WebCore

2011-05-24 Syed Idris Shah <​syed.idris-shah@nokia.com>

Reviewed by Andreas Kling.

[Qt] fast/canvas/webgl/gl-uniform-arrays.html failing for Qt on Linux
​https://bugs.webkit.org/show_bug.cgi?id=60377

LayoutTests/fast/canvas/webgl/gl-uniform-arrays.html

We do not need assert for the useProgram as program can be null.

  • platform/graphics/qt/GraphicsContext3DQt.cpp: (WebCore::GraphicsContext3D::useProgram): Program can be null. Removing assert.
2:36 PM Changeset in webkit [87207] by rniwa@webkit.org
  • 2 edits in trunk/LayoutTests

2011-05-24 Ryosuke Niwa <​rniwa@webkit.org>

Skip fast/events/scroll-in-scaled-page-with-overflow-hidden.html added by r87187 on GTK
because GTK port doesn't implement eventSender.scalePageBy yet (bug 61396).

  • platform/gtk/Skipped:
2:32 PM Changeset in webkit [87206] by Lucas Forschler
  • 9 edits
    1 copy in branches/safari-534-branch/Source

Merge r86995.

2:29 PM Changeset in webkit [87205] by rniwa@webkit.org
  • 2 edits in trunk/LayoutTests

2011-05-24 Ryosuke Niwa <​rniwa@webkit.org>

Skip fast/events/scroll-in-scaled-page-with-overflow-hidden.html added by r87187 on Windows
because Windows port doesn't implement eventSender.scalePageBy yet (bug 59098).

  • platform/win/Skipped:
2:22 PM Changeset in webkit [87204] by rniwa@webkit.org
  • 3 edits
    2 adds in trunk

2011-05-24 Ryosuke Niwa <​rniwa@webkit.org>

Reviewed by Darin Adler.

Undo gets broken in contenteditable area when a text field's value is set by script
​https://bugs.webkit.org/show_bug.cgi?id=61340

Added a test to ensure WebKit does not clear undo stack when setting the value of input or textarea.

  • editing/undo/undo-after-setting-value-expected.txt: Added.
  • editing/undo/undo-after-setting-value.html: Added.

2011-05-24 Ryosuke Niwa <​rniwa@webkit.org>

Reviewed by Darin Adler.

Undo gets broken in contenteditable area when a text field's value is set by script
​https://bugs.webkit.org/show_bug.cgi?id=61340

The bug was caused by RenderTextControl::setInnerTextValue's clearing undo stack by
calling clearUndoRedoOperations whenever script sets new value to input or textarea.

Fixed the bug by removing the offending call to clearUndoRedoOperations. While this call
was added by r15565 to fix a crash, SimpleEditCommands have since become much more robust
and the test added by r15565 (fast/forms/text-field-setvalue-crash.html) still passes.

Test: editing/undo/undo-after-setting-value.html

  • rendering/RenderTextControl.cpp: (WebCore::RenderTextControl::setInnerTextValue):
2:12 PM Changeset in webkit [87203] by adamk@chromium.org
  • 2 edits in trunk/LayoutTests

2011-05-24 Adam Klein <​adamk@chromium.org>

Unreviewed. Updating Chromium test expectations.

  • platform/chromium/test_expectations.txt:
2:06 PM Changeset in webkit [87202] by Lucas Forschler
  • 10 edits in branches/safari-534-branch/Source

Merge r86992.

2:01 PM Changeset in webkit [87201] by mitz@apple.com
  • 2 edits in trunk/Source/WebCore

Move code to discover if a CTFont has vertical glyphs out of SimpleFontData::platformInit()
​https://bugs.webkit.org/show_bug.cgi?id=61392

Reviewed by Dave Hyatt.

  • platform/graphics/mac/SimpleFontDataMac.mm:

(WebCore::fontHasVerticalGlyphs): Moved code here...
(WebCore::SimpleFontData::platformInit): ...from here.

1:59 PM Changeset in webkit [87200] by Lucas Forschler
  • 6 edits in branches/safari-534-branch

Merge r87022.

1:56 PM Changeset in webkit [87199] by ggaren@apple.com
  • 2 edits
    3 deletes in trunk/Source/JavaScriptCore

Fix the Mac build: Yes, please do remove these files, svn.

1:51 PM Changeset in webkit [87198] by ggaren@apple.com
  • 18 edits in trunk/Source/JavaScriptCore

2011-05-24 Geoffrey Garen <​ggaren@apple.com>

Reviewed by Oliver Hunt.

Let's just have one way to get the system page size, bokay?
​https://bugs.webkit.org/show_bug.cgi?id=61384

  • CMakeListsEfl.txt:
  • CMakeListsWinCE.txt:
  • GNUmakefile.list.am:
  • JavaScriptCore.exp:
  • JavaScriptCore.gypi:
  • JavaScriptCore.pro:
  • JavaScriptCore.vcproj/JavaScriptCore/JavaScriptCore.vcproj: MarkStack[Platform].cpp is gone completely now, since it only existed to provide a duplicate way to access the system page size.
  • heap/MarkStack.cpp: (JSC::MarkStack::reset):
  • heap/MarkStack.h: (JSC::::MarkStackArray): (JSC::::shrinkAllocation): Use WTF::pageSize.
  • heap/MarkStackPosix.cpp:
  • heap/MarkStackSymbian.cpp:
  • heap/MarkStackWin.cpp: Removed now-empty files.
  • jit/ExecutableAllocator.cpp: (JSC::ExecutableAllocator::reprotectRegion):
  • jit/ExecutableAllocator.h: (JSC::ExecutableAllocator::ExecutableAllocator): (JSC::ExecutablePool::ExecutablePool): (JSC::ExecutablePool::poolAllocate):
  • jit/ExecutableAllocatorFixedVMPool.cpp: Use WTF::pageSize.
  • wscript: Removed now-empty files.
  • wtf/PageBlock.cpp: (WTF::systemPageSize): Integrated questionable Symbian page size rule from ExecutableAllocator, because that seems like what the original author should have done.
1:41 PM Changeset in webkit [87197] by kbr@google.com
  • 25 edits
    5 adds in trunk

2011-05-24 Kenneth Russell <​kbr@google.com>

Reviewed by James Robinson.

Implement Float64Array
​https://bugs.webkit.org/show_bug.cgi?id=39144

Implemented Float64Array based on the existing Float32Array sources.
Built and tested in Chromium on Linux and Mac OS X and Safari on Mac OS X.

  • CMakeLists.txt:
  • CodeGenerators.pri:
  • DerivedSources.cpp:
  • DerivedSources.make:
  • GNUmakefile.list.am:
  • UseJSC.cmake:
  • WebCore.gypi:
  • WebCore.pro:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • bindings/js/JSBindingsAllInOne.cpp:
  • bindings/js/JSDOMWindowCustom.cpp: (WebCore::JSDOMWindow::float64Array):
  • bindings/js/JSFloat64ArrayCustom.cpp: Added. (WebCore::JSFloat64Array::indexSetter): (WebCore::toJS): (WebCore::JSFloat64Array::set): (WebCore::JSFloat64ArrayConstructor::constructJSFloat64Array):
  • bindings/v8/custom/V8Float64ArrayCustom.cpp: Added. (WebCore::V8Float64Array::constructorCallback): (WebCore::V8Float64Array::setCallback): (WebCore::toV8):
  • html/canvas/ArrayBufferView.h: (WebCore::ArrayBufferView::isDoubleArray):
  • html/canvas/Float64Array.cpp: Added. (WebCore::Float64Array::create): (WebCore::Float64Array::Float64Array): (WebCore::Float64Array::subarray):
  • html/canvas/Float64Array.h: Added. (WebCore::Float64Array::set): (WebCore::Float64Array::item): (WebCore::Float64Array::isDoubleArray):
  • html/canvas/Float64Array.idl: Added.
  • page/DOMWindow.idl:
  • workers/WorkerContext.idl:

2011-05-24 Kenneth Russell <​kbr@google.com>

Reviewed by James Robinson.

Implement Float64Array
​https://bugs.webkit.org/show_bug.cgi?id=39144

Added unit tests to existing typed array tests in Khronos repository and
synced layout test with them. Updated other affected layout tests.

  • fast/canvas/webgl/array-unit-tests-expected.txt:
  • fast/canvas/webgl/array-unit-tests.html:
  • fast/dom/Window/script-tests/window-property-descriptors.js:
  • fast/dom/Window/window-properties.html:
  • fast/dom/script-tests/constructed-objects-prototypes.js: (constructorPropertiesOnWindow):
  • fast/dom/script-tests/prototype-inheritance-2.js: (constructorNamesForWindow):
  • fast/dom/script-tests/prototype-inheritance.js:
  • fast/js/script-tests/global-constructors.js:
1:23 PM Changeset in webkit [87196] by Simon Fraser
  • 4 edits in trunk/Source/WebCore

2011-05-24 Simon Fraser <Simon Fraser>

Reviewed by Chris Marrin.

Add ability to set compositing layer bounds origin
​https://bugs.webkit.org/show_bug.cgi?id=61381

On Core Animation layers, the origin of the bounds
rectangle affects where layer content is rendered, as well
as the offset of sublayers. Currently we always set the
bounds offset to a zero point, but may need to make use
of the bounds offset in future, so expose it via GraphicsLayer.

  • platform/graphics/GraphicsLayer.h: (WebCore::GraphicsLayer::boundsOrigin): (WebCore::GraphicsLayer::setBoundsOrigin):
  • platform/graphics/ca/GraphicsLayerCA.cpp: (WebCore::GraphicsLayerCA::setSize): (WebCore::GraphicsLayerCA::setBoundsOrigin): (WebCore::GraphicsLayerCA::setAllowTiledLayer): (WebCore::GraphicsLayerCA::commitLayerChangesBeforeSublayers): (WebCore::GraphicsLayerCA::updateBounds): (WebCore::GraphicsLayerCA::ensureStructuralLayer): (WebCore::GraphicsLayerCA::swapFromOrToTiledLayer):
  • platform/graphics/ca/GraphicsLayerCA.h:
1:18 PM Changeset in webkit [87195] by weinig@apple.com
  • 2 edits in trunk/Source/WebKit2

Clean up after Geoff and fix the WebKit2 build.

  • WebProcess/Plugins/Netscape/NPRuntimeObjectMap.h:
1:16 PM Changeset in webkit [87194] by Lucas Forschler
  • 10 edits in branches/safari-534-branch

Rollout 87186. It breaks the branch.

1:07 PM Changeset in webkit [87193] by leandro@webkit.org
  • 2 edits in trunk/Source/WebCore

2011-05-24 Leandro Pereira <​leandro@profusion.mobi>

[EFL] Build fix.

  • CMakeLists.txt: Also build platform/network/ContentTypeParser.cpp, added by r86289.
1:03 PM Changeset in webkit [87192] by leandro@webkit.org
  • 2 edits in trunk/Source/WebCore

2011-05-24 Leandro Pereira <​leandro@profusion.mobi>

[EFL] Build fix.

  • bridge/runtime_root.cpp: Include <heap/Weak.h> instead of <JavaScriptCore/Weak.h>.
12:51 PM Changeset in webkit [87191] by rniwa@webkit.org
  • 2 edits
    3 deletes in trunk/LayoutTests

2011-05-24 Ryosuke Niwa <​rniwa@webkit.org>

Skip the tests added by r87096 and remove the failing test expectations on Qt.
The failures are tracked by the bug 61322.

  • platform/qt/Skipped:
  • platform/qt/fast/events/selectstart-by-double-triple-clicks-expected.txt: Removed.
  • platform/qt/fast/events/selectstart-by-drag-expected.txt: Removed.
  • platform/qt/fast/events/selectstart-by-single-click-with-shift-expected.txt: Removed.
12:39 PM Changeset in webkit [87190] by oliver@apple.com
  • 2 edits in trunk/Source/JavaScriptCore

2011-05-24 Oliver Hunt <​oliver@apple.com>

Reviewed by Gavin Barraclough.

Interpreter crashes with gc validation enabled due to failure to mark initial cache structure
​https://bugs.webkit.org/show_bug.cgi?id=61385

The interpreter uses the structure slot of get_by_id and put_by_id to hold
the initial structure it encountered so that it can identify whether a
given access is stable.

When marking though we only visit the slot when we've decided to cache, and
so this value could die. This was "safe" as the value was only used for a
pointer compare, but it was incorrect. We now just mark the slot like we
should have been doing already.

  • bytecode/CodeBlock.cpp: (JSC::CodeBlock::visitStructures):
12:32 PM Changeset in webkit [87189] by jcivelli@chromium.org
  • 36 edits
    1 copy
    35 adds in trunk

2011-05-24 Jay Civelli <​jcivelli@chromium.org>

Reviewed by Adam Barth.

Adding MHTML reading support.
​https://bugs.webkit.org/show_bug.cgi?id=7168

  • configure.ac:

2011-05-24 Jay Civelli <​jcivelli@chromium.org>

Reviewed by Adam Barth.

Adding MHTML reading support to WebCore.
​https://bugs.webkit.org/show_bug.cgi?id=7168

  • mhtml/frame_0.html_original: Added.
  • mhtml/frame_1.html_original: Added.
  • mhtml/frame_2.html_original: Added.
  • mhtml/frame_4.html_original: Added.
  • mhtml/multi_frames.html_original: Added.
  • mhtml/multi_frames_ie.mht: Added.
  • mhtml/multi_frames_unmht.mht: Added.
  • mhtml/page_with_css_and_js.html_original: Added.
  • mhtml/page_with_css_and_js_ie.mht: Added.
  • mhtml/page_with_css_and_js_unmht.mht: Added.
  • mhtml/page_with_image.html_original: Added.
  • mhtml/page_with_image_ie.mht: Added.
  • mhtml/page_with_image_unmht.mht: Added.
  • mhtml/resources/red_square.png: Added.
  • mhtml/simple_page.html_original: Added.
  • mhtml/simple_page_ie.mht: Added.
  • mhtml/simple_page_unmht.mht: Added.
  • platform/chromium/mhtml/multi_frames_ie-expected.txt: Added.
  • platform/chromium/mhtml/multi_frames_unmht-expected.txt: Added.
  • platform/chromium/mhtml/page_with_css_and_js_ie-expected.txt: Added.
  • platform/chromium/mhtml/page_with_css_and_js_unmht-expected.txt: Added.
  • platform/chromium/mhtml/page_with_image_ie-expected.txt: Added.
  • platform/chromium/mhtml/page_with_image_unmht-expected.txt: Added.
  • platform/chromium/mhtml/simple_page_ie-expected.txt: Added.
  • platform/chromium/mhtml/simple_page_unmht-expected.txt: Added.

2011-05-24 Jay Civelli <​jcivelli@chromium.org>

Reviewed by Adam Barth.

Adding MHTML reading support. That required some basic MIME header parsing.
Modified DocumentLoader to keep a reference to the currently Archive loaded,
so we can have different policies for loading subresources depending on the
archive type.
​https://bugs.webkit.org/show_bug.cgi?id=7168

  • CMakeLists.txt:
  • Configurations/FeatureDefines.xcconfig:
  • GNUmakefile.am:
  • GNUmakefile.list.am:
  • WebCore.gyp/WebCore.gyp:
  • WebCore.gypi:
  • WebCore.pro:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • features.pri:
  • loader/DocumentLoader.cpp: (WebCore::DocumentLoader::commitLoad): (WebCore::DocumentLoader::setupForReplaceByMIMEType): (WebCore::DocumentLoader::popArchiveForSubframe): (WebCore::DocumentLoader::scheduleArchiveLoad):
  • loader/DocumentLoader.h:
  • loader/FrameLoader.cpp: (WebCore::FrameLoader::loadURLIntoChildFrame): (WebCore::FrameLoader::loadArchive): (WebCore::FrameLoader::stopAllLoaders): (WebCore::FrameLoader::finishedLoadingDocument):
  • loader/FrameLoader.h: (WebCore::FrameLoader::archive):
  • loader/MainResourceLoader.cpp: (WebCore::MainResourceLoader::continueAfterContentPolicy):
  • loader/ResourceLoader.cpp: (WebCore::ResourceLoader::start):
  • loader/archive/Archive.cpp: Added.
  • loader/archive/Archive.h: (WebCore::Archive::mainResource):
  • loader/archive/ArchiveFactory.cpp: (WebCore::archiveFactoryCreate): (WebCore::archiveMIMETypes): (WebCore::ArchiveFactory::create): (WebCore::ArchiveFactory::registerKnownArchiveMIMETypes):
  • loader/archive/ArchiveFactory.h:
  • loader/archive/ArchiveResourceCollection.cpp: (WebCore::ArchiveResourceCollection::addAllResources): (WebCore::ArchiveResourceCollection::popSubframeArchive):
  • loader/archive/ArchiveResourceCollection.h:
  • loader/archive/cf/LegacyWebArchive.cpp: (WebCore::LegacyWebArchive::create): (WebCore::LegacyWebArchive::type):
  • loader/archive/cf/LegacyWebArchive.h:
  • loader/archive/mhtml/MHTMLArchive.cpp: Added.
  • loader/archive/mhtml/MHTMLArchive.h: Added.
  • loader/archive/mhtml/MHTMLParser.cpp: Added.
  • loader/archive/mhtml/MHTMLParser.h: Added.
  • platform/MIMETypeRegistry.cpp: (WebCore::initializeSupportedNonImageMimeTypes):
  • platform/mac/PasteboardMac.mm: (WebCore::Pasteboard::documentFragment):
  • platform/network/MIMEHeader.cpp: Added.
  • platform/network/MIMEHeader.h: Added.

2011-05-24 Jay Civelli <​jcivelli@chromium.org>

Reviewed by Adam Barth.

Adding MHTML reading support to WebCore.
​https://bugs.webkit.org/show_bug.cgi?id=7168

  • features.gypi:

2011-05-24 Jay Civelli <​jcivelli@chromium.org>

Reviewed by Adam Barth.

Adding MHTML reading support.
​https://bugs.webkit.org/show_bug.cgi?id=7168

  • Scripts/build-webkit:
  • Scripts/old-run-webkit-tests:
  • Scripts/webkitperl/features.pm:
  • Scripts/webkitpy/layout_tests/port/test_files.py:
  • Scripts/webkitpy/layout_tests/port/webkit.py:
12:24 PM Changeset in webkit [87188] by ggaren@apple.com
  • 2 edits in trunk/Source/WebCore

Try to fix some builds: #include Weak.h for Weak<T>.

  • bridge/runtime_root.h:
12:23 PM Changeset in webkit [87187] by weinig@apple.com
  • 3 edits
    2 adds in trunk

2011-05-24 Sam Weinig <​sam@webkit.org>

Reviewed by Beth Dakin.

Can't scroll scaled page that has overflow:hidden on its root
<rdar://problem/9029189>
​https://bugs.webkit.org/show_bug.cgi?id=61339

Test: fast/events/scroll-in-scaled-page-with-overflow-hidden.html

  • page/FrameView.cpp: (WebCore::FrameView::applyOverflowToViewport): Scrollbars should be enabled for the viewport when scaled, even if overflow:hidden is specified.

2011-05-24 Sam Weinig <​sam@webkit.org>

Reviewed by Beth Dakin.

Can't scroll scaled page that has overflow:hidden on its root
<rdar://problem/9029189>
​https://bugs.webkit.org/show_bug.cgi?id=61339

  • fast/events/scroll-in-scaled-page-with-overflow-hidden-expected.txt: Added.
  • fast/events/scroll-in-scaled-page-with-overflow-hidden.html: Added.
12:22 PM Changeset in webkit [87186] by sfalken@apple.com
  • 10 edits in branches/safari-534-branch

2011-05-24 Steve Falkenburg <​sfalken@apple.com>

Reviewed by Adam Roben.

Disable features on safari-534-branch.
<rdar://problem/9261347>

12:16 PM Changeset in webkit [87185] by Lucas Forschler
  • 2 edits in branches/safari-534-branch/Source/WebKit2

Merge r86985.

12:13 PM Changeset in webkit [87184] by Adam Roben
  • 2 edits in trunk/Source/JavaScriptCore

Windows build fix

  • JavaScriptCore.vcproj/JavaScriptCore/JavaScriptCore.def: Removed now-inline functions.
12:07 PM Changeset in webkit [87183] by ggaren@apple.com
  • 2 edits in trunk/Source/JavaScriptCore

Windows build fix: update the #if OS(WINDOWS) section to match my last patch.

  • heap/MarkStack.h:

(JSC::::shrinkAllocation):

12:03 PM Changeset in webkit [87182] by Adam Roben
  • 2 edits in trunk/Source/WebCore

Leopard build fix

  • bridge/runtime_root.cpp: Added a missing #include.
11:57 AM Changeset in webkit [87181] by jianli@chromium.org
  • 6 edits in trunk

Make start parameter of Blob.slice optional.
​https://bugs.webkit.org/show_bug.cgi?id=59628

Reviewed by Kenneth Russell.

Source/WebCore:

The start parameter should be optional per the latest FILE API
spec: ​http://dev.w3.org/2006/webapi/FileAPI/#dfn-Blob.

  • fileapi/Blob.h:
  • fileapi/Blob.idl: Make start parameter optional.

LayoutTests:

  • fast/files/blob-slice-test-expected.txt:
  • fast/files/blob-slice-test.html:
11:55 AM Changeset in webkit [87180] by ggaren@apple.com
  • 2 edits in trunk/Source/JavaScriptCore

Split out function definitions and class definitions from class
declarations in MarkStack.h, for readability.

Rubber-stamped by Oliver Hunt.

  • heap/MarkStack.h:

(JSC::MarkStack::MarkStack):
(JSC::MarkStack::~MarkStack):
(JSC::MarkStack::addOpaqueRoot):
(JSC::MarkStack::containsOpaqueRoot):
(JSC::MarkStack::opaqueRootCount):
(JSC::MarkSet::MarkSet):
(JSC::MarkStack::allocateStack):
(JSC::MarkStack::releaseStack):
(JSC::MarkStack::pageSize):
(JSC::::MarkStackArray):
(JSC::::~MarkStackArray):
(JSC::::expand):
(JSC::::append):
(JSC::::removeLast):
(JSC::::last):
(JSC::::isEmpty):
(JSC::::size):
(JSC::::shrinkAllocation):

11:51 AM Changeset in webkit [87179] by Adam Roben
  • 9 edits
    2 adds in trunk

Invalidate JS wrappers for NPObjects when they are finalized

This will cause the underlying NPObject to be released at finalization time, rather than at
destruction time (which is unpredictable and could occur after the plugin has been
unloaded).

Test: plugins/npobject-js-wrapper-destroyed-after-plugin-unload.html

Fixes <​http://webkit.org/b/61316> <rdar://problem/9489824> Crash in deallocateNPObject when
reloading yahoo.com webarchive in WebKit2

and

<​http://webkit.org/b/61317> <rdar://problem/9489829> Crash in _NPN_DeallocateObject when
reloading yahoo.com webarchive in WebKit1

Reviewed by Oliver Hunt.

Source/WebCore:

  • bridge/runtime_object.cpp:

(JSC::Bindings::RuntimeObject::~RuntimeObject): Assert that we've already been invalidated.

  • bridge/runtime_root.cpp:

(JSC::Bindings::RootObject::invalidate):
(JSC::Bindings::RootObject::addRuntimeObject):
Updated for m_runtimeObjects type change.

(JSC::Bindings::RootObject::finalize): Added. Invalidates the RuntimeObject and removes it
from the map.

  • bridge/runtime_root.h: Now inherits from WeakHandleOwner.

Changed m_runtimeObjects from a WeakGCMap to a HashMap of JSC::Weak objects so that we will
be notified when the RuntimeObjects are finalized.

Source/WebKit2:

  • WebProcess/Plugins/Netscape/JSNPObject.cpp:

(WebKit::JSNPObject::~JSNPObject): Assert that we've already been invalidated, rather than
trying to perform invalidation now (when the plugin might already be unloaded).

  • WebProcess/Plugins/Netscape/NPRuntimeObjectMap.cpp:

(WebKit::NPRuntimeObjectMap::getOrCreateJSObject):
(WebKit::NPRuntimeObjectMap::invalidate):
Updated for m_jsNPObjects type change.

(WebKit::NPRuntimeObjectMap::finalize): Added. Invalidates the JSNPObject and removes it
from the map.

  • WebProcess/Plugins/Netscape/NPRuntimeObjectMap.h: Now inherits from WeakHandleOwner.

Changed m_jsNPObjects from a WeakGCMap to a HashMap of JSC::Weak objects so that we will be
notified when the JSNPObjects are finalized.

LayoutTests:

Test that we don't crash when a JS wrapper for an NPObject is destroyed after its plugin is unloaded

  • plugins/npobject-js-wrapper-destroyed-after-plugin-unload-expected.txt: Added.
  • plugins/npobject-js-wrapper-destroyed-after-plugin-unload.html: Added.

(startTest): Gets a JS wrapper for an NPObject from the plugin, allocate a bunch of memory
so the JS wrapper will be finalized, then destroy the plugin and wait for a little bit
before calling finishTest.
(finishTest): Force a GC so the JS wrapper will be destroyed. If we didn't crash, we passed!

11:50 AM Changeset in webkit [87178] by senorblanco@chromium.org
  • 26 edits in trunk

2011-05-24 Stephen White <​senorblanco@chromium.org>

Reviewed by Kenneth Russell.

Remove most style checks on Source/ThirdParty/glu, since it doesn't
follow WebKit style.
​https://bugs.webkit.org/show_bug.cgi?id=61365

  • Scripts/webkitpy/style/checker.py:

2011-05-24 Stephen White <​senorblanco@chromium.org>

Reviewed by Kenneth Russell.

Unfork glu sources as much as possible.
​https://bugs.webkit.org/show_bug.cgi?id=61365

Restore the glu sources closer to the original version on oss.sgi.com
by removing the WebKit-specific include paths, restoring the include
order, and renaming the glu* functions via #defines rather than in
the source.

  • glu/README.webkit: Update the README to reflect the changes from trunk.
  • glu/glu.gyp: Add Source/Thirdparty/glu as an include path, so we can remove the hardcoded paths from the source files.
  • glu/gluos.h: Add #defines to rename the glu entry points to internal_*.
  • glu/internal_glu.h:
  • glu/libtess/dict.c:
  • glu/libtess/geom.c:
  • glu/libtess/geom.h:
  • glu/libtess/memalloc.c:
  • glu/libtess/mesh.c:
  • glu/libtess/mesh.h:
  • glu/libtess/normal.c:
  • glu/libtess/normal.h:
  • glu/libtess/priorityq-heap.c:
  • glu/libtess/priorityq-sort.h:
  • glu/libtess/priorityq.c:
  • glu/libtess/priorityq.h:
  • glu/libtess/render.c:
  • glu/libtess/render.h:
  • glu/libtess/sweep.c:
  • glu/libtess/sweep.h:
  • glu/libtess/tess.c:
  • glu/libtess/tess.h:
  • glu/libtess/tessmono.c: Remove WebKit-specific include paths, and internal_ prefixes from glu entry points. Restore include order to original source order.
11:49 AM Changeset in webkit [87177] by oliver@apple.com
  • 6 edits in trunk/Source/JavaScriptCore

2011-05-24 Oliver Hunt <​oliver@apple.com>

Reviewed by Geoffrey Garen.

Avoid creating unnecessary identifiers and strings in the syntax checker
​https://bugs.webkit.org/show_bug.cgi?id=61378

Selectively tell the lexer that there are some places it does not need to
do the real work of creating Identifiers for IDENT and STRING tokens.

Make parseString and parseIdentifier templatized on whether they should
do real work, or merely validate the tokens.

SunSpider --parse-only reports ~5-8% win depending on hardware.

  • parser/ASTBuilder.h: (JSC::ASTBuilder::createDotAccess):
  • parser/JSParser.cpp: (JSC::JSParser::next): (JSC::JSParser::consume): (JSC::JSParser::parseVarDeclarationList): (JSC::JSParser::parseConstDeclarationList): (JSC::JSParser::parseExpression): (JSC::JSParser::parseAssignmentExpression): (JSC::JSParser::parseConditionalExpression): (JSC::JSParser::parseBinaryExpression): (JSC::JSParser::parseProperty): (JSC::JSParser::parseObjectLiteral): (JSC::JSParser::parseArrayLiteral): (JSC::JSParser::parseArguments): (JSC::JSParser::parseMemberExpression):
  • parser/Lexer.cpp: (JSC::Lexer::parseIdentifier): (JSC::Lexer::parseString): (JSC::Lexer::lex):
  • parser/Lexer.h:
  • parser/SyntaxChecker.h: (JSC::SyntaxChecker::createDotAccess): (JSC::SyntaxChecker::createProperty):
11:46 AM Changeset in webkit [87176] by Adam Roben
  • 2 edits in trunk/Tools

Don't allow forcing builds through the build.webkit.org web interface

Someone has been forcing bogus builds.

  • BuildSlaveSupport/build.webkit.org-config/master.cfg:
11:39 AM Changeset in webkit [87175] by qi.2.zhang@nokia.com
  • 4 edits in trunk

2011-05-24 Qi Zhang <​qi.2.zhang@nokia.com>

Reviewed by Adam Roben.

WebKitTestRunner needs layoutTestController.setJavaScriptCanAccessClipboard
​https://bugs.webkit.org/show_bug.cgi?id=42669

Fixes a typo issue for patch 94265.

  • WebProcess/InjectedBundle/API/c/WKBundle.cpp: (WKBundleSetJavaScriptCanAccessClipboard):

2011-05-24 Qi Zhang <​qi.2.zhang@nokia.com>

Reviewed by Adam Roben.

WebKitTestRunner needs layoutTestController.setJavaScriptCanAccessClipboard
​https://bugs.webkit.org/show_bug.cgi?id=42669

Fixes the regression issue caused by patch 94265. The patch didn't intialize the setting before the
layouttest starting, it will make the test cases after editing/execCommand/clipboard-access.html failed.
Because that case changed the setting.

  • WebKitTestRunner/InjectedBundle/InjectedBundle.cpp: (WTR::InjectedBundle::beginTesting):
11:32 AM WebKit Team edited by helder@sencha.com
(diff)
11:24 AM Changeset in webkit [87174] by commit-queue@webkit.org
  • 3 edits in trunk

2011-05-24 Mike Reed <​reed@google.com>

Reviewed by Kenneth Russell.

skia: fix stroking of zero-height rectangles
​https://bugs.webkit.org/show_bug.cgi?id=61284

Tests: canvas/philip/tests/2d.line.miter.lineedge.html

canvas/philip/tests/2d.strokeRect.zero.4.html

  • platform/graphics/skia/GraphicsContextSkia.cpp: (WebCore::GraphicsContext::strokeRect):
11:16 AM Changeset in webkit [87173] by Philippe Normand
  • 7 edits in trunk/Source/WebCore

2011-05-24 Zan Dobersek <​zandobersek@gmail.com> and Philippe Normand <​pnormand@igalia.com>

Reviewed by Kenneth Russell.

Enable FloatArray usage outside WebGL guards
​https://bugs.webkit.org/show_bug.cgi?id=61356

  • webaudio/AudioBuffer.cpp:
  • webaudio/AudioBuffer.idl:
  • webaudio/RealtimeAnalyser.cpp:
  • webaudio/RealtimeAnalyser.h:
  • webaudio/RealtimeAnalyserNode.h:
  • webaudio/RealtimeAnalyserNode.idl:
11:12 AM Changeset in webkit [87172] by adamk@chromium.org
  • 3 edits
    3 deletes in trunk

2011-05-24 Adam Klein <​adamk@chromium.org>

Unreviewed, rolling out r87145.
​http://trac.webkit.org/changeset/87145
​https://bugs.webkit.org/show_bug.cgi?id=61194

Regression in LayoutTests/inspector/debugger/scripts-panel.html on Chromium Win & Linux.

  • inspector/console/console-eval-syntax-error-expected.txt: Removed.
  • inspector/console/console-eval-syntax-error.html: Removed.
  • platform/chromium/inspector/console/console-eval-syntax-error-expected.txt: Removed.

2011-05-24 Adam Klein <​adamk@chromium.org>

Unreviewed, rolling out r87145.
​http://trac.webkit.org/changeset/87145
​https://bugs.webkit.org/show_bug.cgi?id=61194

Regression in LayoutTests/inspector/debugger/scripts-panel.html on Chromium Win & Linux.

  • bindings/v8/custom/V8InjectedScriptHostCustom.cpp: (WebCore::V8InjectedScriptHost::evaluateCallback):
11:04 AM Changeset in webkit [87171] by mdelaney@apple.com
  • 5 edits
    2 adds in trunk

2011-05-24 Matthew Delaney <​mdelaney@apple.com>

Reviewed by Simon Fraser.

Clamp coordinates to integers for canvas create/getImageData routines
​https://bugs.webkit.org/show_bug.cgi?id=61135

  • fast/canvas/canvas-getImageData-largeNonintegralDimensions-expected.txt: Added.
  • fast/canvas/canvas-getImageData-largeNonintegralDimensions.html: Added.

2011-05-24 Matthew Delaney <​mdelaney@apple.com>

Reviewed by Simon Fraser.

Clamp coordinates to integers for canvas create/getImageData routines
​https://bugs.webkit.org/show_bug.cgi?id=61135

Test: fast/canvas/canvas-getImageData-largeNonintegralDimensions.html

  • html/HTMLCanvasElement.cpp: (WebCore::HTMLCanvasElement::convertLogicalToDevice): clamp to ints
  • html/canvas/CanvasRenderingContext2D.cpp: (WebCore::CanvasRenderingContext2D::createImageData): (WebCore::CanvasRenderingContext2D::getImageData):
  • platform/graphics/cg/ImageBufferDataCG.cpp: (WebCore::ImageBufferData::getData):
10:53 AM Changeset in webkit [87170] by weinig@apple.com
  • 2 edits in trunk/Source/WebKit2

2011-05-24 Sam Weinig <​sam@webkit.org>

Reviewed by John Sullivan.

WebKit tells client that the page has become unresponsive while the beforeUnloadConfirm dialog is up
​https://bugs.webkit.org/show_bug.cgi?id=61372

  • UIProcess/WebPageProxy.cpp: (WebKit::WebPageProxy::runBeforeUnloadConfirmPanel): Stop the unresponsiveness timer like we do for other client functions that spin a runloop.
10:45 AM Changeset in webkit [87169] by kevino@webkit.org
  • 2 edits in trunk/Source/WebCore

Reviewed by Kevin Ollivier.

[wx] Make sure x and y adjustments are applied to all operations, and fix the calculations for the height and width checks to take into account x and y offsets.

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

10:43 AM Changeset in webkit [87168] by tony@chromium.org
  • 3 edits
    2 adds in trunk

2011-05-24 Tony Chang <​tony@chromium.org>

Reviewed by James Robinson.

fix render overflow computation for input type=range
​https://bugs.webkit.org/show_bug.cgi?id=61132

  • fast/forms/slider-hit-testing-expected.txt: Added.
  • fast/forms/slider-hit-testing.html: Added.

2011-05-24 Tony Chang <​tony@chromium.org>

Reviewed by James Robinson.

fix render overflow computation for input type=range
​https://bugs.webkit.org/show_bug.cgi?id=61132

Test: fast/forms/slider-hit-testing.html

We need to clear m_overflow otherwise we use the first
size of the slider for hit testing (i.e., changes to the size
cause problems). This only shows up if the thumb isn't contained
in the bounds of the slider. Making the range have a smaller height
than the thumb shows the bug.

  • rendering/RenderSlider.cpp: (WebCore::RenderSlider::layout): Clear the overflow during layout

so we re-compute the overflow based on the current size of the
slider.

10:41 AM Changeset in webkit [87167] by alokp@chromium.org
  • 9 edits in trunk/Source

2011-05-24 Alok Priyadarshi <​alokp@chromium.org>

Reviewed by James Robinson.

Enable skia gpu rendering for content layers
​https://bugs.webkit.org/show_bug.cgi?id=56749

  • src/WebViewImpl.cpp: (WebKit::WebViewImpl::setIsAcceleratedCompositingActive): (WebKit::WebViewImpl::reallocateRenderer):

2011-05-24 Alok Priyadarshi <​alokp@chromium.org>

Reviewed by James Robinson.

Enable skia gpu rendering for content layers
​https://bugs.webkit.org/show_bug.cgi?id=56749

This patch adds gpu accelerated rendering for content and root layers.
Layer textures are updated in two stages:

  1. Paint: The non-accelerated path paints the layer contents into a bitmap.

The accelerated path paints into an SkPicture, which is essentially a display list.

  1. Upload: The non-accelerated path uploads the pixels obtained in the first stage to a texture.

The accelerated path renders the picture directly into the texture.

The accelerated path can be enabled by --enable-accelerated-drawing command-line flag in chromium.
Note that this patch does not accelerate the CG path on mac. We have two options here - use skia or accelrate CG path.
Any bugs should be caught by layout tests.

  • platform/graphics/chromium/ContentLayerChromium.cpp: (WebCore::ContentLayerChromium::createTextureUpdater):
  • platform/graphics/chromium/LayerRendererChromium.cpp: (WebCore::LayerRendererChromium::create): (WebCore::LayerRendererChromium::LayerRendererChromium): (WebCore::LayerRendererChromium::skiaContext): (WebCore::LayerRendererChromium::updateAndDrawLayers): (WebCore::LayerRendererChromium::createRootLayerTextureUpdater):
  • platform/graphics/chromium/LayerRendererChromium.h: (WebCore::LayerRendererChromium::acceleratedDrawing):
  • platform/graphics/chromium/LayerTextureUpdaterCanvas.cpp: (WebCore::LayerTextureUpdaterSkPicture::LayerTextureUpdaterSkPicture): (WebCore::LayerTextureUpdaterSkPicture::~LayerTextureUpdaterSkPicture): (WebCore::LayerTextureUpdaterSkPicture::prepareToUpdate): (WebCore::LayerTextureUpdaterSkPicture::updateTextureRect): (WebCore::LayerTextureUpdaterSkPicture::deleteFrameBuffer): (WebCore::LayerTextureUpdaterSkPicture::createFrameBuffer): (WebCore::LayerTextureUpdaterSkPicture::clearFrameBuffer):
  • platform/graphics/chromium/LayerTextureUpdaterCanvas.h: (WebCore::LayerTextureUpdaterSkPicture::orientation):
  • platform/graphics/chromium/LayerTilerChromium.cpp: (WebCore::LayerTilerChromium::updateRect): (WebCore::LayerTilerChromium::draw):
10:36 AM Changeset in webkit [87166] by podivilov@chromium.org
  • 5 edits in trunk/Source/WebCore

2011-05-24 Pavel Podivilov <​podivilov@chromium.org>

Reviewed by Yury Semikhatsky.

Web Inspector: display keyboard shortcuts in scripts panel sidebar buttons tooltip text.
​https://bugs.webkit.org/show_bug.cgi?id=61358

  • English.lproj/localizedStrings.js:
  • inspector/front-end/CallStackSidebarPane.js: (WebInspector.CallStackSidebarPane.prototype._selectedPlacardByIndex): (WebInspector.CallStackSidebarPane.prototype.registerShortcuts):
  • inspector/front-end/Panel.js: (WebInspector.Panel.prototype.registerShortcut):
  • inspector/front-end/ScriptsPanel.js: (WebInspector.ScriptsPanel): (WebInspector.ScriptsPanel.prototype._createSidebarButtons): (WebInspector.ScriptsPanel.prototype._createSidebarButtonAndRegisterShortcuts):
10:15 AM Changeset in webkit [87165] by kbr@google.com
  • 1 edit
    2 adds
    2 deletes in trunk/LayoutTests

2011-05-24 Kenneth Russell <​kbr@google.com>

Unreviewed. Fixed incorrect test expectations I previously checked in.

  • platform/chromium-gpu-mac/compositing/animation/busy-indicator-actual.png: Removed.
  • platform/chromium-gpu-mac/compositing/animation/busy-indicator-expected.png: Added.
  • platform/chromium-gpu-win/compositing/animation/busy-indicator-actual.png: Removed.
  • platform/chromium-gpu-win/compositing/animation/busy-indicator-expected.png: Added.
10:14 AM Changeset in webkit [87164] by adamk@chromium.org
  • 2 edits in trunk/LayoutTests

2011-05-24 Adam Klein <​adamk@chromium.org>

Unreviewed. Updated Chromium test expectations.

  • platform/chromium/test_expectations.txt: svg failure on Linux, BUGWK61370.
10:12 AM Changeset in webkit [87163] by Philippe Normand
  • 2 edits in trunk/Source/WebCore

2011-05-24 Philippe Normand <​pnormand@igalia.com>

Reviewed by Andres Kling.

JSAudioContextCustom doesn't encode errors
​https://bugs.webkit.org/show_bug.cgi?id=61357

Fix AudioContext return values

  • bindings/js/JSAudioContextCustom.cpp: (WebCore::JSAudioContextConstructor::constructJSAudioContext):
10:07 AM Changeset in webkit [87162] by Csaba Osztrogonác
  • 2 edits in trunk/Tools

2011-05-24 Csaba Osztrogonác <Csaba Osztrogonác>

Reviewed by Kenneth Rohde Christiansen.

[ORWT]Make --skipped=only option work with parameters with LayoutTests/ prefix
​https://bugs.webkit.org/show_bug.cgi?id=61052

  • Scripts/old-run-webkit-tests:
9:43 AM Changeset in webkit [87161] by rwlbuis@webkit.org
  • 3 edits in trunk/Source/WebCore

2011-05-24 Rob Buis <​rbuis@rim.com>

Rubber-stamped by Dirk Schulze.

Remove requiresLayer inlines, already provided by RenderSVGModelObject.

  • rendering/svg/RenderSVGHiddenContainer.h: (WebCore::RenderSVGHiddenContainer::isSVGHiddenContainer):
  • rendering/svg/RenderSVGImage.h:
9:30 AM Changeset in webkit [87160] by andreas.kling@nokia.com
  • 12 edits
    2 deletes in trunk/Source/WebKit2

2011-05-24 Andreas Kling <​kling@webkit.org>

Reviewed by Simon Hausmann.

[WK2] Change TiledDrawingArea to use ShareableBitmap instead of UpdateChunk.
​https://bugs.webkit.org/show_bug.cgi?id=61296

Pass UpdateInfo containing ShareableBitmaps instead of UpdateChunk for tile updates.
Only the bounds rect and bitmap handle in the UpdateInfo are used since none of the
other parameters are needed for TiledDrawingArea.

  • Shared/ShareableBitmap.h:
  • Shared/qt/ShareableBitmapQt.cpp: (WebKit::ShareableBitmap::createQImage): (WebKit::ShareableBitmap::createGraphicsContext): (WebKit::ShareableBitmap::paint):
  • Shared/qt/UpdateChunk.cpp: Removed.
  • Shared/qt/UpdateChunk.h: Removed.
  • UIProcess/TiledDrawingAreaProxy.cpp: (WebKit::TiledDrawingAreaProxy::didReceiveMessage): (WebKit::TiledDrawingAreaProxy::waitUntilUpdatesComplete):
  • UIProcess/TiledDrawingAreaProxy.h:
  • UIProcess/TiledDrawingAreaTile.h:
  • UIProcess/qt/TiledDrawingAreaProxyQt.cpp: (WebKit::TiledDrawingAreaProxy::snapshotTaken):
  • UIProcess/qt/TiledDrawingAreaTileQt.cpp: (WebKit::TiledDrawingAreaTile::incorporateUpdate):
  • WebKit2.pro:
  • WebProcess/WebPage/TiledDrawingArea.cpp: (WebKit::TiledDrawingArea::updateTile): (WebKit::TiledDrawingArea::didReceiveMessage):
  • WebProcess/WebPage/TiledDrawingArea.h:
  • WebProcess/WebPage/qt/TiledDrawingAreaQt.cpp: (WebKit::TiledDrawingArea::paintIntoBitmap):
9:25 AM Changeset in webkit [87159] by adamk@chromium.org
  • 2 edits in trunk/LayoutTests

2011-05-24 Adam Klein <​adamk@chromium.org>

Unreviewed.

Chromium test expectations update.
​https://bugs.webkit.org/show_bug.cgi?id=61169

  • platform/chromium/test_expectations.txt: Marked optimize-contrast tests as IMAGE failure, BUGWK61169.
9:11 AM Changeset in webkit [87158] by Nikolas Zimmermann
  • 2 edits in trunk/Source/WebCore

2011-05-24 Nikolas Zimmermann <​nzimmermann@rim.com>

Not reviewed, build fix.

Remove platform layering violation: TextRun stores RenderObjects for SVG Fonts support
​https://bugs.webkit.org/show_bug.cgi?id=60254

Forgot to rename the destructor.

  • platform/graphics/SimpleFontData.h: (WebCore::SimpleFontData::AdditionalFontData::~AdditionalFontData):
9:06 AM Changeset in webkit [87157] by msaboff@apple.com
  • 2 edits in trunk/Source/JavaScriptCore

2011-05-23 Michael Saboff <​msaboff@apple.com>

Reviewed by Mark Rowe.

Safari often freezes when clicking "Return free memory" in Caches dialog
​https://bugs.webkit.org/show_bug.cgi?id=61325

There are two fixes and improvement in instrumentation code used to find
one of the problems.
Changed ReleaseFreeList() to set the "decommitted" bit when releasing
pages to the system and moving Spans from the normal list to the returned
list.
Added a "not making forward progress" check to TCMalloc_PageHeap::scavenge
to eliminate an infinite loop if we can't meet the pagesToRelease target.
Added a check for the decommitted bit being set properly in
TCMalloc_PageHeap::CheckList.

  • wtf/FastMalloc.cpp: (WTF::TCMalloc_PageHeap::scavenge): (WTF::TCMalloc_PageHeap::Check): (WTF::TCMalloc_PageHeap::CheckList): (WTF::ReleaseFreeList):
8:49 AM Changeset in webkit [87156] by Nikolas Zimmermann
  • 4 edits in trunk/Source/WebCore

2011-05-24 Nikolas Zimmermann <​nzimmermann@rim.com>

Not reviewed, build fix.

Remove platform layering violation: TextRun stores RenderObjects for SVG Fonts support
​https://bugs.webkit.org/show_bug.cgi?id=60254

Rename SimpleFontData::FontData subclass to AdditionalFontData, to avoid clashes with the WebCore::FontData class, which breaks the win build.

  • platform/graphics/SimpleFontData.cpp: (WebCore::SimpleFontData::SimpleFontData):
  • platform/graphics/SimpleFontData.h: (WebCore::SimpleFontData::fontData):
  • svg/SVGFontData.h:
8:44 AM Changeset in webkit [87155] by qi.2.zhang@nokia.com
  • 2 edits in trunk/LayoutTests

2011-05-24 Qi Zhang <​qi.2.zhang@nokia.com>

Reviewed by Andreas Kling.

[Qt] fast/workers/shared-worker-frame-lifecycle.html failed on Qt Bot
​https://bugs.webkit.org/show_bug.cgi?id=34281

Remove the test case from skip list.

  • platform/qt/Skipped:
8:39 AM Changeset in webkit [87154] by beidson@apple.com
  • 2 edits in trunk/Source/WebKit2

Build fix after r87153

  • WebProcess/mac/SecItemShimMethods.mm: Don't compile this code on SnowLeopard.
8:31 AM Changeset in webkit [87153] by beidson@apple.com
  • 5 edits
    2 adds in trunk/Source/WebKit2

Part of <rdar://problem/9434311> and ​https://bugs.webkit.org/show_bug.cgi?id=61334
Should shim a few SecKeychainItem* methods on SnowLeopard

Reviewed by Sam Weinig.

In preparation for different WebProcess shims depending on platform, move the current
shim to its own files.

  • WebKit2.xcodeproj/project.pbxproj:


  • WebProcess/mac/WebProcessMac.mm:

(WebKit::WebProcess::initializeShim): For non-SnowLeopard platforms, call initializeSecItemShim().

  • WebProcess/mac/SecItemShimMethods.h: Added.
  • WebProcess/mac/SecItemShimMethods.mm: Added. Actual webSecItem* methods moved from WebProcessMac.mm.

(WebKit::webSecItemCopyMatchingMainThread):
(WebKit::webSecItemCopyMatching):
(WebKit::webSecItemAddOnMainThread):
(WebKit::webSecItemAdd):
(WebKit::webSecItemUpdateOnMainThread):
(WebKit::webSecItemUpdate):
(WebKit::webSecItemDeleteOnMainThread):
(WebKit::webSecItemDelete):
(WebKit::initializeSecItemShim): Initialize the callbacks via WebKitWebProcessSecItemShimInitialize

  • WebProcess/mac/WebProcessShim.h:
  • WebProcess/mac/WebProcessShim.mm: Wrap all the actual code here in a !BUILDING_ON_SNOW_LEOPARD block

(WebKit::shimSecItemCopyMatching):
(WebKit::shimSecItemAdd):
(WebKit::shimSecItemUpdate):
(WebKit::shimSecItemDelete):
(WebKit::WebKitWebProcessSecItemShimInitialize):

8:27 AM Changeset in webkit [87152] by Nikolas Zimmermann
  • 38 edits
    1 move
    1 add in trunk/Source/WebCore

2011-05-24 Nikolas Zimmermann <​nzimmermann@rim.com>

Reviewed by Antti Koivisto.

Remove platform layering violation: TextRun stores RenderObjects for SVG Fonts support
​https://bugs.webkit.org/show_bug.cgi?id=60254

First part:
Remove a long-standing layering violation in TextRun: it stores RenderObject/RenderSVGResource pointers for SVG Fonts support.
Replace the two Render* pointers with a single RefPtr<AbstractRenderingContext>. AbstractRenderingContext is a helper class,
that can be derived from in order to associate additional data with a TextRun. This effectively reduces the memory consumption of TextRun.

Introduce rendering/TextRunRenderingContext.h, which inherits from TextRun::AbstractRenderingContext and holds additional data.
If the primary font in use is a SVG Font then allocate a TextRunRenderingContext object and store it in the RefPtr<AbstractRenderingContext>
in TextRun. If the primary font is not a SVG Font, don't even allocate the TextRunRenderingContext structure, as we won't need the context data.
SVG Fonts glyph matching only works within a context, so we need access to the RenderObject that's drawing the text.

This is the main preparation patch for the SVG Fonts rewrite, that will allow us to share the simple text code path for SVG Fonts as well,
making all CSS text properties work for HTML text using SVG Fonts, and allows proper integration within the GlyphPage concept. Soon
we can intermix WOFF/SVG & native fonts, within segment font declarations.

Second part:
Remove a long-standing layering violation in SimpleFontData: it stores SVGFontData objects, that are living in svg/.
Use the same concept as above, introduce SimpleFontData::AbstractFontData, and let SVGFontData inherit from AbstractFontData and extent it.
If SVG Fonts are used, CSSFontFaceSource will create a SVGFontData object and pass it as PassOwnPtr<AbstractFontData> to SimpleFontData.

All layering violations are gone now, SVG Fonts are cleanly integrated now.
Doesn't affect any tests yet, refactoring only.

  • CMakeLists.txt:
  • GNUmakefile.list.am:
  • WebCore.gypi:
  • WebCore.pro:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • css/CSSFontFaceSource.cpp: (WebCore::CSSFontFaceSource::getFontData):
  • platform/graphics/Font.cpp: (WebCore::Font::drawText): (WebCore::Font::width): (WebCore::Font::selectionRectForText): (WebCore::Font::offsetForPosition):
  • platform/graphics/Font.h:
  • platform/graphics/SimpleFontData.cpp: (WebCore::SimpleFontData::SimpleFontData): (WebCore::SimpleFontData::~SimpleFontData):
  • platform/graphics/SimpleFontData.h: (WebCore::SimpleFontData::FontData::~FontData): (WebCore::SimpleFontData::fontMetrics): (WebCore::SimpleFontData::setMaxCharWidth): (WebCore::SimpleFontData::setAvgCharWidth): (WebCore::SimpleFontData::setSpaceWidth): (WebCore::SimpleFontData::setSpaceGlyph): (WebCore::SimpleFontData::setZeroWidthSpaceGlyph): (WebCore::SimpleFontData::fontData): (WebCore::SimpleFontData::isSVGFont): (WebCore::SimpleFontData::setMissingGlyphData):
  • platform/graphics/TextRun.h: (WebCore::TextRun::TextRun): (WebCore::TextRun::setAllowTabs): (WebCore::TextRun::setXPos): (WebCore::TextRun::RenderingContext::~RenderingContext): (WebCore::TextRun::renderingContext): (WebCore::TextRun::setRenderingContext):
  • rendering/EllipsisBox.cpp: (WebCore::EllipsisBox::paint): (WebCore::EllipsisBox::selectionRect): (WebCore::EllipsisBox::paintSelection):
  • rendering/InlineTextBox.cpp: (WebCore::InlineTextBox::selectionRect): (WebCore::InlineTextBox::paint): (WebCore::InlineTextBox::paintSelection): (WebCore::InlineTextBox::paintCompositionBackground): (WebCore::InlineTextBox::paintSpellingOrGrammarMarker): (WebCore::InlineTextBox::paintTextMatchMarker): (WebCore::InlineTextBox::computeRectForReplacementMarker): (WebCore::InlineTextBox::offsetForPosition): (WebCore::InlineTextBox::positionForOffset): (WebCore::InlineTextBox::constructTextRun):
  • rendering/InlineTextBox.h:
  • rendering/RenderBlock.cpp: (WebCore::stripTrailingSpace): (WebCore::RenderBlock::constructTextRun):
  • rendering/RenderBlock.h:
  • rendering/RenderBlockLineLayout.cpp: (WebCore::setLogicalWidthForTextRun): (WebCore::textWidth): (WebCore::tryHyphenating): (WebCore::RenderBlock::LineBreaker::nextLineBreak): (WebCore::RenderBlock::checkLinesForTextOverflow):
  • rendering/RenderCombineText.cpp: (WebCore::RenderCombineText::combineText):
  • rendering/RenderFileUploadControl.cpp: (WebCore::RenderFileUploadControl::paintObject): (WebCore::RenderFileUploadControl::computePreferredLogicalWidths):
  • rendering/RenderFlexibleBox.cpp: (WebCore::RenderFlexibleBox::applyLineClamp):
  • rendering/RenderImage.cpp: (WebCore::RenderImage::setImageSizeForAltText): (WebCore::RenderImage::paintReplaced):
  • rendering/RenderListBox.cpp: (WebCore::RenderListBox::updateFromElement):
  • rendering/RenderListMarker.cpp: (WebCore::RenderListMarker::paint): (WebCore::RenderListMarker::computePreferredLogicalWidths): (WebCore::RenderListMarker::getRelativeMarkerRect):
  • rendering/RenderText.cpp: (WebCore::RenderText::widthFromCache): (WebCore::RenderText::trimmedPrefWidths): (WebCore::RenderText::computePreferredLogicalWidths): (WebCore::RenderText::width):
  • rendering/RenderTextControl.cpp: (WebCore::RenderTextControl::getAvgCharWidth):
  • rendering/svg/RenderSVGAllInOne.cpp:
  • rendering/svg/SVGInlineTextBox.cpp: (WebCore::SVGInlineTextBox::offsetForPositionInFragment): (WebCore::SVGInlineTextBox::prepareGraphicsContextForTextPainting): (WebCore::SVGInlineTextBox::restoreGraphicsContextAfterTextPainting): (WebCore::SVGInlineTextBox::constructTextRun): (WebCore::SVGInlineTextBox::paintText):
  • rendering/svg/SVGTextLayoutEngineSpacing.cpp: (WebCore::SVGTextLayoutEngineSpacing::calculateSVGKerning):
  • rendering/svg/SVGTextMetrics.cpp: (WebCore::constructTextRun): (WebCore::SVGTextMetrics::measureCharacterRange):
  • rendering/svg/SVGTextRunRenderingContext.cpp: Copied from svg/SVGFont.cpp. (WebCore::svgFontAndFontFaceElementForFontData): (WebCore::firstParentRendererForNonTextNode): (WebCore::referencingRenderObjectFromRun): (WebCore::activePaintingResourceFromRun): (WebCore::SVGTextRunWalker::walk): (WebCore::floatWidthMissingGlyphCallback): (WebCore::floatWidthOfSubStringUsingSVGFont): (WebCore::SVGTextRunRenderingContext::floatWidthUsingSVGFont): (WebCore::SVGTextRunRenderingContext::drawTextUsingSVGFont): (WebCore::SVGTextRunRenderingContext::selectionRectForTextUsingSVGFont): (WebCore::SVGTextRunRenderingContext::offsetForPositionForTextUsingSVGFont):
  • rendering/svg/SVGTextRunRenderingContext.h: Added. (WebCore::SVGTextRunRenderingContext::create): (WebCore::SVGTextRunRenderingContext::context): (WebCore::SVGTextRunRenderingContext::activePaintingResource): (WebCore::SVGTextRunRenderingContext::setActivePaintingResource): (WebCore::SVGTextRunRenderingContext::SVGTextRunRenderingContext): (WebCore::textRunNeedsRenderingContext):
  • svg/SVGAllInOne.cpp:
  • svg/SVGFont.cpp: Removed.
  • svg/SVGFontData.cpp: (WebCore::SVGFontData::initializeFontData):
  • svg/SVGFontData.h: (WebCore::SVGFontData::create): (WebCore::SVGFontData::isSVGFontData):
  • svg/SVGGlyphElement.cpp: (WebCore::SVGGlyphElement::SVGGlyphElement): (WebCore::SVGGlyphElement::buildGenericGlyphIdentifier): (WebCore::SVGGlyphElement::buildGlyphIdentifier):
  • svg/SVGHKernElement.cpp: (WebCore::SVGHKernElement::SVGHKernElement): (WebCore::SVGHKernElement::buildHorizontalKerningPair):
  • svg/SVGVKernElement.cpp: (WebCore::SVGVKernElement::SVGVKernElement): (WebCore::SVGVKernElement::buildVerticalKerningPair):
  • svg/SVGVKernElement.h:
8:13 AM Changeset in webkit [87151] by commit-queue@webkit.org
  • 5 edits in trunk/Source/WebCore

2011-05-24 Ryuan Choi <​ryuan.choi@samsung.com>

Reviewed by Andreas Kling.

[GTK] Implement GraphicsContextCairo::imageInterpolationQuality().
​https://bugs.webkit.org/show_bug.cgi?id=60827

Implement getter/setter of imageInterpolationQuality and logic to change
interpolation algorithm when drawing image.
Mac and Qt already implemented it.

  • platform/graphics/cairo/GraphicsContextCairo.cpp: (WebCore::GraphicsContext::setImageInterpolationQuality): (WebCore::GraphicsContext::imageInterpolationQuality):
  • platform/graphics/cairo/GraphicsContextPlatformPrivateCairo.h:
  • platform/graphics/cairo/PlatformContextCairo.cpp: (WebCore::PlatformContextCairo::drawSurfaceToContext):
  • platform/graphics/cairo/PlatformContextCairo.h: (WebCore::PlatformContextCairo::setImageInterpolationQuality): Added. (WebCore::PlatformContextCairo::imageInterpolationQuality): Added.
8:11 AM Changeset in webkit [87150] by commit-queue@webkit.org
  • 24 edits
    4 copies
    2 adds in trunk/Source/WebCore

2011-05-24 Leandro Gracia Gil <​leandrogracia@chromium.org>

Reviewed by Tony Gentilcore.

Media Stream API: adding Stream and GeneratedStream classes.
​https://bugs.webkit.org/show_bug.cgi?id=56666

Add the code for the Stream and GeneratedStream classes and integrate it
with the existing stream generation requests.

Tests for the Media Stream API will be provided by the bug 56587.

  • CMakeLists.txt:
  • CodeGenerators.pri:
  • DerivedSources.cpp:
  • DerivedSources.make:
  • GNUmakefile.list.am:
  • WebCore.gypi:
  • WebCore.pro:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • bindings/js/JSEventTarget.cpp: (WebCore::toJS):
  • bindings/v8/V8DOMWrapper.cpp: (WebCore::V8DOMWrapper::convertEventTargetToV8Object):
  • dom/DOMAllInOne.cpp:
  • dom/EventTarget.cpp: (WebCore::EventTarget::toStream): (WebCore::EventTarget::toGeneratedStream):
  • dom/EventTarget.h:
  • dom/GeneratedStream.cpp: Added. (WebCore::GeneratedStream::DispatchUpdateTask::create): (WebCore::GeneratedStream::DispatchUpdateTask::performTask): (WebCore::GeneratedStream::DispatchUpdateTask::DispatchUpdateTask): (WebCore::GeneratedStream::create): (WebCore::GeneratedStream::GeneratedStream): (WebCore::GeneratedStream::~GeneratedStream): (WebCore::GeneratedStream::toGeneratedStream): (WebCore::GeneratedStream::detachEmbedder): (WebCore::GeneratedStream::stop): (WebCore::GeneratedStream::onStop):
  • dom/GeneratedStream.h: Added.
  • dom/GeneratedStream.idl: Added.
  • dom/Stream.cpp: Added. (WebCore::Stream::create): (WebCore::Stream::Stream): (WebCore::Stream::~Stream): (WebCore::Stream::toStream): (WebCore::Stream::streamEnded): (WebCore::Stream::scriptExecutionContext): (WebCore::Stream::eventTargetData): (WebCore::Stream::ensureEventTargetData):
  • dom/Stream.h: Added. (WebCore::Stream::readyState): (WebCore::Stream::label): (WebCore::Stream::refEventTarget): (WebCore::Stream::derefEventTarget):
  • dom/Stream.idl: Added.
  • page/CallbackTask.h: (WebCore::CallbackTask1::Scheduler::~Scheduler):
  • page/MediaStreamClient.h:
  • page/MediaStreamController.cpp: (WebCore::MediaStreamController::stopGeneratedStream): (WebCore::MediaStreamController::streamFailed):
  • page/MediaStreamController.h:
  • page/MediaStreamFrameController.cpp: (WebCore::MediaStreamFrameController::Request::Request): (WebCore::MediaStreamFrameController::GenerateStreamRequest::create): (WebCore::MediaStreamFrameController::GenerateStreamRequest::abort): (WebCore::MediaStreamFrameController::GenerateStreamRequest::GenerateStreamRequest): (WebCore::MediaStreamFrameController::unregister): (WebCore::MediaStreamFrameController::getStreamFromLabel): (WebCore::MediaStreamFrameController::generateStream): (WebCore::MediaStreamFrameController::stopGeneratedStream): (WebCore::MediaStreamFrameController::streamGenerated): (WebCore::MediaStreamFrameController::streamGenerationFailed): (WebCore::MediaStreamFrameController::streamFailed):
  • page/MediaStreamFrameController.h: (WebCore::MediaStreamFrameController::StreamClient::StreamClient): (WebCore::MediaStreamFrameController::StreamClient::isGeneratedStream):
  • page/NavigatorUserMediaError.h: (WebCore::NavigatorUserMediaError::create): (WebCore::NavigatorUserMediaError::NavigatorUserMediaError):
  • page/NavigatorUserMediaSuccessCallback.h:
  • page/NavigatorUserMediaSuccessCallback.idl:
8:10 AM Changeset in webkit [87149] by ddkilzer@apple.com
  • 2 edits in trunk/Source/ThirdParty/ANGLE

Part 2 of 2: <​http://webkit.org/b/60805> Enable -Wnewline-eof on ANGLE

Reviewed by Adam Roben.

  • Configurations/Base.xcconfig:

(GCC_WARN_ABOUT_MISSING_NEWLINE): Added. Set to YES.

8:09 AM Changeset in webkit [87148] by Adam Roben
  • 4 edits in trunk/Tools

Make TestFailures show how many tests are failing on each tester, and omit testers with no failures

Fixes <​http://webkit.org/b/61063> <rdar://problem/9460533> TestFailures page shows testers
that don't have any failing tests, which isn't useful

Reviewed by David Kilzer.

  • BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/Buildbot.js:

(Buildbot.prototype.getTesters): Renamed from getTesterNames. Now returns Builder objects
instead of name strings.
(Buildbot.prototype._buildersForNames): Added. Helper function to convert an array of
builder names into an array of builders.

  • BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/Builder.js:

(Builder.prototype.getMostRecentCompletedBuildNumber): Added. Returns the build number of
the most recently completed build, or -1 if there is no such build.
(Builder.prototype.getNumberOfFailingTests): Added. Returns the number of tests that failed
in the given build, or -1 if the number could not be determined. Some of this code came from
_getFailingTests.
(Builder.prototype._getBuildJSON): Added. Code came from _getFailingTests.
(Builder.prototype._getFailingTests): Changed to use new _getBuildJSON and
getNumberOfFailingTests functions.

  • BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/ViewController.js:

(ViewController.prototype._displayTesters): Get the current number of test failures for each
tester and show it in the list. Omit testers that have no failures at all. We keep the
testers and failure counts in an array and sort it before displaying the current data, as
the order in which data will be fetched is unpredictable.

7:48 AM Changeset in webkit [87147] by Antti Koivisto
  • 4 edits in trunk/Source/WebCore

REGRESSION (r45620): Node list caches never deleted
​https://bugs.webkit.org/show_bug.cgi?id=61268
<rdar://problem/9467379>

Reviewed by Oliver Hunt.

NodeListsNodeData::isEmpty() tests if RefCounted objects have refcount of zero which is impossible.
As a results NodeList caches are never deleted, causing bad performance in DOM mutating operations as
they repeatedly invalidate caches.

  • dom/Node.cpp:

(WebCore::Node::childNodes):

Construct m_childNodeListCaches lazily.

(WebCore::Node::unregisterDynamicNodeList):
(WebCore::Node::notifyLocalNodeListsAttributeChanged):
(WebCore::Node::notifyLocalNodeListsChildrenChanged):
(WebCore::Node::removeNodeListCacheIfPossible):

Add a helper.

(WebCore::NodeListsNodeData::invalidateCaches):

Invalidate m_childNodeListCaches by clearing it if there are no additional clients

(WebCore::NodeListsNodeData::isEmpty):

Test emptiness of various NodeListCaches simply by testing hash emptiness instead of testing for non-zero ref count of items.
m_childNodeListCaches is empty if it is null.

  • dom/Node.h:
  • dom/NodeRareData.h:

(WebCore::NodeListsNodeData::NodeListsNodeData):

Construct m_childNodeListCaches lazily.

7:17 AM Changeset in webkit [87146] by mnaganov@chromium.org
  • 4 edits in trunk/Source

2011-05-24 Mikhail Naganov <​mnaganov@chromium.org>

Reviewed by Yury Semikhatsky.

Web Inspector: Reorganize InspectorBackendDispatch code to eliminate duplication
​https://bugs.webkit.org/show_bug.cgi?id=61300

  • inspector/CodeGeneratorInspector.pm:

2011-05-24 Mikhail Naganov <​mnaganov@chromium.org>

Reviewed by Yury Semikhatsky.

Web Inspector: Reorganize InspectorBackendDispatch code to eliminate duplication
​https://bugs.webkit.org/show_bug.cgi?id=61300

  • src/WebDevToolsAgentImpl.cpp: (WebKit::WebDevToolsAgent::shouldInterruptForMessage):
7:15 AM BuildingGtk edited by steveg20101@hotmail.com
reading is very helpful in our learning (diff)
7:10 AM QtWebKitBuildBots edited by Csaba Osztrogonác
(diff)
6:11 AM QtWebKitBuildBots edited by Laszlo Gombos
(diff)
5:38 AM Changeset in webkit [87145] by yurys@chromium.org
  • 3 edits
    3 adds in trunk

2011-05-23 Yury Semikhatsky <​yurys@chromium.org>

Reviewed by Adam Barth.

Chromium DevTools: Tab crashes with "Aw, snap!" on entering "(new Image())." in console
​https://bugs.webkit.org/show_bug.cgi?id=61194

  • inspector/console/console-eval-syntax-error-expected.txt: Added.
  • inspector/console/console-eval-syntax-error.html: Added.
  • platform/chromium/inspector/console/console-eval-syntax-error-expected.txt: Added.

2011-05-23 Yury Semikhatsky <​yurys@chromium.org>

Reviewed by Adam Barth.

Chromium DevTools: Tab crashes with "Aw, snap!" on entering "(new Image())." in console
​https://bugs.webkit.org/show_bug.cgi?id=61194

Test: inspector/console/console-eval-syntax-error.html

  • bindings/v8/custom/V8InjectedScriptHostCustom.cpp: (WebCore::V8InjectedScriptHost::evaluateCallback): return immediately in case of syntax error
5:16 AM QtWebKitBuildBots edited by Laszlo Gombos
(diff)
4:22 AM Changeset in webkit [87144] by loki@webkit.org
  • 2 edits in trunk/LayoutTests

2011-05-24 Gabor Loki <​loki@webkit.org>

Rubber-stamped by Csaba Osztrogonác.

[Qt] Skip failing new test introduced in r87096

  • platform/qt-arm/Skipped:
4:14 AM QtWebKitBuildBots edited by abecsi@webkit.org
(diff)
3:55 AM QtWebKit edited by Csaba Osztrogonác
Add link to QtWebKit buildbots (diff)
3:54 AM Changeset in webkit [87143] by steveblock@google.com
  • 17 edits
    2 adds
    8 deletes in trunk/Source/WebCore

2011-04-13 Steve Block <​steveblock@google.com>

Reviewed by Eric Seidel.

Geolocation callbacks should make use of new callback generation mechanism
​https://bugs.webkit.org/show_bug.cgi?id=40065

No new tests, refactoring only.

  • Android.derived.jscbindings.mk:
  • Android.derived.v8bindings.mk:
  • Android.jscbindings.mk:
  • Android.v8bindings.mk:
  • CMakeLists.txt:
  • CodeGenerators.pri:
  • DerivedSources.cpp:
  • DerivedSources.make:
  • GNUmakefile.list.am:
  • UseJSC.cmake:
  • WebCore.gypi:
  • WebCore.order:
  • WebCore.pro:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • bindings/js/JSBindingsAllInOne.cpp:
  • bindings/js/JSCustomPositionCallback.cpp: Removed.
  • bindings/js/JSCustomPositionCallback.h: Removed.
  • bindings/js/JSCustomPositionErrorCallback.cpp: Removed.
  • bindings/js/JSCustomPositionErrorCallback.h: Removed.
  • bindings/js/JSGeolocationCustom.cpp: (WebCore::JSGeolocation::getCurrentPosition): (WebCore::JSGeolocation::watchPosition):
  • bindings/v8/custom/V8CustomPositionCallback.cpp: Removed.
  • bindings/v8/custom/V8CustomPositionCallback.h: Removed.
  • bindings/v8/custom/V8CustomPositionErrorCallback.cpp: Removed.
  • bindings/v8/custom/V8CustomPositionErrorCallback.h: Removed.
  • bindings/v8/custom/V8GeolocationCustom.cpp: (WebCore::V8Geolocation::getCurrentPositionCallback): (WebCore::V8Geolocation::watchPositionCallback):
  • page/PositionCallback.h:
  • page/PositionCallback.idl: Added.
  • page/PositionErrorCallback.h:
  • page/PositionErrorCallback.idl: Added.
3:46 AM QtWebKitBuildBots created by Csaba Osztrogonác
Add the list of QtWebKit buildbots
3:04 AM Changeset in webkit [87142] by commit-queue@webkit.org
  • 2 edits in trunk/LayoutTests

2011-05-24 Leon Scroggins <​scroggo@google.com>

Reviewed by James Robinson.

Update expectations for shadow tests which are now passing.
​https://bugs.webkit.org/show_bug.cgi?id=61278

Also remove some expectations for missing tests.

  • platform/chromium/test_expectations.txt:
2:27 AM Changeset in webkit [87141] by Csaba Osztrogonác
  • 2 edits in trunk/LayoutTests

[Qt][Mac] Skip one more failing test.

  • platform/qt-mac/Skipped:
2:24 AM Changeset in webkit [87140] by podivilov@chromium.org
  • 6 edits
    2 adds in trunk/Source/WebCore

2011-05-23 Pavel Podivilov <​podivilov@chromium.org>

Reviewed by Yury Semikhatsky.

Web Inspector: watched expressions buttons do not match inspector styles.
​https://bugs.webkit.org/show_bug.cgi?id=30554

  • WebCore.gypi:
  • inspector/front-end/BreakpointsSidebarPane.js: (WebInspector.XHRBreakpointsSidebarPane):
  • inspector/front-end/Images/deleteIcon.png: Added.
  • inspector/front-end/Images/paneRefreshButtons.png: Added.
  • inspector/front-end/WatchExpressionsSidebarPane.js: (WebInspector.WatchExpressionsSidebarPane.prototype.reset): (WebInspector.WatchExpressionsSidebarPane.prototype._addButtonClicked): (WebInspector.WatchExpressionsSidebarPane.prototype._refreshButtonClicked): (WebInspector.WatchExpressionsSidebarPane.prototype.refreshExpressions): (WebInspector.WatchExpressionsSection): (WebInspector.WatchExpressionsSection.prototype.update.appendResult): (WebInspector.WatchExpressionsSection.prototype.update): (WebInspector.WatchExpressionsSection.prototype.saveExpressions): (WebInspector.WatchExpressionsSection.prototype._mouseMove): (WebInspector.WatchExpressionsSection.prototype._mouseOut): (WebInspector.WatchExpressionsSection.prototype._updateHoveredElement): (WebInspector.WatchExpressionsSection.CompareProperties): (WebInspector.WatchExpressionTreeElement.prototype.update): (WebInspector.WatchExpressionTreeElement.prototype.editingCancelled):
  • inspector/front-end/WebKit.qrc:
  • inspector/front-end/inspector.css: (.properties-tree.watch-expressions): (.properties-tree.watch-expressions li): (.section .properties .delete-button): (.section .properties li.hovered .delete-button): (.pane > .title > .pane-title-button): (.pane > .title > .pane-title-button:hover): (.pane > .title > .pane-title-button:active): (.pane > .title > .pane-title-button.add): (.pane > .title > .pane-title-button.refresh):
2:15 AM Changeset in webkit [87139] by yutak@chromium.org
  • 6 edits in trunk/Source/WebCore

2011-05-24 Yuta Kitamura <​yutak@chromium.org>

Reviewed by Kent Tamura.

WebSocket: Add fail() to WebSocketChannel and its family
​https://bugs.webkit.org/show_bug.cgi?id=61277

This is the first part of refactoring effort described in bug 61115.
Existing error reports will be modified to use fail() in a later patch.

No change in behavior, thus no new tests.

  • websockets/ThreadableWebSocketChannel.h:
  • websockets/WebSocketChannel.cpp: (WebCore::WebSocketChannel::fail):
  • websockets/WebSocketChannel.h:
  • websockets/WorkerThreadableWebSocketChannel.cpp: (WebCore::WorkerThreadableWebSocketChannel::fail): (WebCore::WorkerThreadableWebSocketChannel::Peer::fail): (WebCore::WorkerThreadableWebSocketChannel::mainThreadFail): (WebCore::WorkerThreadableWebSocketChannel::Bridge::fail):
  • websockets/WorkerThreadableWebSocketChannel.h:
2:05 AM Changeset in webkit [87138] by abecsi@webkit.org
  • 2 edits in trunk/Source/WebCore

2011-05-24 Andras Becsi <​abecsi@webkit.org>

Reviewed by Csaba Osztrogonác.

[Qt][V8] REGRESSION (r87063): Fix the build.
​https://bugs.webkit.org/show_bug.cgi?id=61290

No new tests needed.

2:02 AM Changeset in webkit [87137] by commit-queue@webkit.org
  • 3 edits in trunk/Source/WebCore

2011-05-24 Naoki Takano <​takano.naoki@gmail.com>

Reviewed by Kent Tamura.

[Chromium]Add clipping for listBox in popup window to fix wrong location display when the autofill item is really long.
​https://bugs.webkit.org/show_bug.cgi?id=61252
​http://code.google.com/p/chromium/issues/detail?id=83539

Manual test: manual-tests/autofill-popup-location.html.

  • manual-tests/autofill-popup-location.html: With long long text, test the shown location is correct.
  • platform/chromium/PopupMenuChromium.cpp: (WebCore::PopupListBox::setBaseWidth): Clip width with m_maxWindowWidth. (WebCore::PopupListBox::PopupListBox): Add m_maxWindowWidth initialization. (WebCore::PopupContainer::layoutAndCalculateWidgetRect): Call setMaxWidthAndLayout() when the width is clipped with screen size. (WebCore::PopupListBox::setMaxWidthAndLayout): Set m_maxWindowWidth and call layout(). (WebCore::PopupListBox::layout): Compare renewed windowWidth and m_maxWindowWidth, and clip the it.
1:42 AM Changeset in webkit [87136] by commit-queue@webkit.org
  • 4 edits in trunk

2011-05-24 James Robinson <​jamesr@chromium.org>

Reviewed by Kenneth Russell.

[chromium] Properly handle huge initial dimensions for 2d canvas when accelerated 2d canvas is enabled
​https://bugs.webkit.org/show_bug.cgi?id=60077

Update expectation to reflect that the canvas layer isn't composited.

  • platform/chromium-gpu/compositing/tiling/huge-layer-img-expected.txt:

2011-05-24 James Robinson <​jamesr@chromium.org>

Reviewed by Kenneth Russell.

[chromium] Properly handle huge initial dimensions for 2d canvas when accelerated 2d canvas is enabled
​https://bugs.webkit.org/show_bug.cgi?id=60077

Return NULL from DrawingBuffer::create() if the initial dimensions are larger than we can support in hardware so
that we properly use the software canvas path and not enable the compositor for the canvas.

compositing/tiling/huge-layer-img.html triggers this case by trying to create a 20000 pixel high canvas. The
failure was previously masked by other bugs fixed by r85661.

  • platform/graphics/chromium/DrawingBufferChromium.cpp: (WebCore::DrawingBuffer::DrawingBuffer):

Return NULL from DrawingBuffer::create() if resize() fails.

1:10 AM Changeset in webkit [87135] by ukai@chromium.org
  • 3 edits
    2 adds in trunk

2011-05-24 Fumitoshi Ukai <​ukai@chromium.org>

Reviewed by Adam Barth.

close before open WebSocket doesn't fire close event.
​https://bugs.webkit.org/show_bug.cgi?id=57301

  • http/tests/websocket/tests/close-before-open-expected.txt: Added.
  • http/tests/websocket/tests/close-before-open.html: Added.

2011-05-24 Fumitoshi Ukai <​ukai@chromium.org>

Reviewed by Adam Barth.

close before open WebSocket doesn't fire close event.
​https://bugs.webkit.org/show_bug.cgi?id=57301

In platformClose(), m_readStream may be 0 until proxy is resolved.
It should callback didClose() in such case.

Test: http/tests/websocket/tests/close-before-open.html

  • platform/network/cf/SocketStreamHandleCFNet.cpp: (WebCore::SocketStreamHandle::platformClose):
12:42 AM Changeset in webkit [87134] by zoltan@webkit.org
  • 2 edits in trunk/LayoutTests

[Qt][WK2] Paint the bot green for ultimate satisfaction of Ossy.

  • platform/qt-wk2/Skipped:
12:22 AM Changeset in webkit [87133] by Csaba Osztrogonác
  • 2 edits in trunk/LayoutTests

[Qt][Mac] Skip failing tests.

  • platform/qt-mac/Skipped:
12:16 AM Changeset in webkit [87132] by commit-queue@webkit.org
  • 3 edits
    2 adds
    18 deletes in trunk/LayoutTests

2011-05-24 Annie Sullivan <​sullivan@chromium.org>

Reviewed by Ryosuke Niwa.

editing/deleting/5026848-2 and editing/deleting/5026848-3 should be dump-as-markup tests
​https://bugs.webkit.org/show_bug.cgi?id=61323

Convert editing/deleting/5026848-2 and editing/deleting/5026848-3 to dump-as-markup.

  • editing/deleting/5026848-2-expected.txt: Added.
  • editing/deleting/5026848-2.html: Convert to dump-as-markup.
  • editing/deleting/5026848-3-expected.txt: Added.
  • editing/deleting/5026848-3.html: Convert to dump-as-markup.
  • platform/chromium-linux/editing/deleting/5026848-2-expected.png: Removed.
  • platform/chromium-linux/editing/deleting/5026848-3-expected.png: Removed.
  • platform/chromium-win/editing/deleting/5026848-2-expected.png: Removed.
  • platform/chromium-win/editing/deleting/5026848-2-expected.txt: Removed.
  • platform/chromium-win/editing/deleting/5026848-3-expected.png: Removed.
  • platform/chromium-win/editing/deleting/5026848-3-expected.txt: Removed.
  • platform/gtk/editing/deleting/5026848-2-expected.txt: Removed.
  • platform/gtk/editing/deleting/5026848-3-expected.txt: Removed.
  • platform/mac-leopard/editing/deleting/5026848-2-expected.png: Removed.
  • platform/mac-leopard/editing/deleting/5026848-3-expected.png: Removed.
  • platform/mac/editing/deleting/5026848-2-expected.png: Removed.
  • platform/mac/editing/deleting/5026848-2-expected.txt: Removed.
  • platform/mac/editing/deleting/5026848-3-expected.png: Removed.
  • platform/mac/editing/deleting/5026848-3-expected.txt: Removed.
  • platform/qt/editing/deleting/5026848-2-expected.png: Removed.
  • platform/qt/editing/deleting/5026848-2-expected.txt: Removed.
  • platform/qt/editing/deleting/5026848-3-expected.png: Removed.
  • platform/qt/editing/deleting/5026848-3-expected.txt: Removed.
12:04 AM Changeset in webkit [87131] by loki@webkit.org
  • 2 edits in trunk/LayoutTests

2011-05-23 Gabor Loki <​loki@webkit.org>

Rubber-stamped by Csaba Osztrogonác.

[Qt] Skip failing test because of missing anti-aliased fonts on ARM

  • platform/qt-arm/Skipped:
Note: See TracTimeline for information about the timeline view.