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

Timeline



Jun 18, 2012:

11:52 PM Changeset in webkit [120682] by Csaba Osztrogonác
  • 4 edits in trunk/LayoutTests

[Qt] Unreviewed gardening, skip failing tests.

  • platform/qt-4.8/Skipped:
  • platform/qt-5.0/Skipped:
  • platform/qt/Skipped:
11:35 PM Changeset in webkit [120681] by rniwa@webkit.org
  • 87 edits in trunk/LayoutTests

Use testRunner instead of layoutTestController in fast/animation, autoresize, backgrounds, block, body-propagation, borders, and box-* tests
https://bugs.webkit.org/show_bug.cgi?id=89422

Reviewed by Kent Tamura.

  • fast/animation/request-animation-frame-detach-element.html:
  • fast/animation/request-animation-frame-detach-element2.html:
  • fast/animation/request-animation-frame-disabled.html:
  • fast/animation/request-animation-frame-during-modal.html:
  • fast/animation/request-animation-frame-iframe.html:
  • fast/animation/script-tests/request-animation-frame-cancel.js:
  • fast/animation/script-tests/request-animation-frame-cancel2.js:
  • fast/animation/script-tests/request-animation-frame-disabled.js:
  • fast/animation/script-tests/request-animation-frame-timestamps-advance.js:
  • fast/animation/script-tests/request-animation-frame-timestamps.js:
  • fast/animation/script-tests/request-animation-frame-within-callback.js:
  • fast/animation/script-tests/request-animation-frame.js:
  • fast/autoresize/autoresize-with-iframe.html:
  • fast/autoresize/basic.html:
  • fast/autoresize/turn-off-autoresize.html:
  • fast/backgrounds/animated-gif-as-background.html:
  • fast/backgrounds/animated-svg-as-mask.html:
  • fast/backgrounds/background-fast-path-simple-color.html:
  • fast/backgrounds/background-fill-zero-area-crash.html:
  • fast/backgrounds/background-repeat-computed-style.html:
  • fast/backgrounds/multiple-backgrounds-assert.html:
  • fast/backgrounds/root-background-propagation.html:
  • fast/backgrounds/root-background-propagation2.html:
  • fast/backgrounds/size/contain-and-cover-zoomed.html:
  • fast/backgrounds/solid-color-context-restore.html:
  • fast/block/:
  • fast/borders/border-image-reset-by-border-shorthand.html:
  • fast/borders/border-image-slice-omission.html:
  • fast/borders/border-mixed-alpha.html:
  • fast/borders/border-radius-complex-inner.html:
  • fast/borders/border-radius-parsing.html:
  • fast/borders/negative-border-width.html:
  • fast/borders/outline-alpha-block.html:
  • fast/borders/outline-alpha-inline.html:
  • fast/box-shadow/box-shadow-clipped-slices.html:
  • fast/box-shadow/box-shadow-currentcolor.html:
  • fast/box-shadow/no-blur-multiple-offsets.html:
  • fast/box-shadow/resources/box-shadow-overflow-scroll.js:
  • fast/box-shadow/spread-multiple-inset.html:
  • fast/box-shadow/spread-multiple-normal.html:
11:33 PM Changeset in webkit [120680] by aestes@apple.com
  • 2 edits in trunk/Source/WebCore

[Mac] Editor::fontAttributesForSelectionStart() leaks a NSShadow
https://bugs.webkit.org/show_bug.cgi?id=89424

Reviewed by Dan Bernstein.

  • editing/mac/EditorMac.mm:

(WebCore::Editor::fontAttributesForSelectionStart): Store the NSShadow
in a RetainPtr so that it's released before going out of scope.

11:20 PM Changeset in webkit [120679] by tkent@chromium.org
  • 21 edits in trunk/Source/WebCore

Refactoring: Introduce a class to represent restoring state of a form control
https://bugs.webkit.org/show_bug.cgi?id=89412

Reviewed by Hajime Morita.

HTMLFormControlElementWithState::saveFormControlState had two return
values; One is a boolean to represent whether we should save the state
or not, and another is form control value. This patch introduces
FormControlState class representing these values.

We're going to extend FormControlState class so that it can contain
multiple strings to fix a FileInputType issue and it have another type
to fix Bug 89409.

No new tests. Just a refactoring.

  • html/FormController.cpp:

(WebCore::FormController::formElementsState):
Gets a state string from a FormControlState instance.
(WebCore::FormController::takeStateForFormElement):
Creates a FormControlState instance with a state string.

  • html/FormController.h:

(WebCore::FormControlState): Added.

  • html/BaseCheckableInputType.cpp:

(WebCore::BaseCheckableInputType::saveFormControlState):
Use FormControlState class.
(WebCore::BaseCheckableInputType::restoreFormControlState): ditto.

  • html/BaseCheckableInputType.h:

(BaseCheckableInputType): ditto.

  • html/FileInputType.cpp:

(WebCore::FileInputType::saveFormControlState): ditto.
(WebCore::FileInputType::restoreFormControlState): ditto.

  • html/FileInputType.h:

(FileInputType): ditto.

  • html/HTMLFormControlElementWithState.cpp:

(WebCore::HTMLFormControlElementWithState::saveFormControlState): ditto.
(WebCore::HTMLFormControlElementWithState::finishParsingChildren): ditto.

  • html/HTMLFormControlElementWithState.h:

(HTMLFormControlElementWithState): ditto.
(WebCore::HTMLFormControlElementWithState::restoreFormControlState): ditto.

  • html/HTMLInputElement.cpp:

(WebCore::HTMLInputElement::saveFormControlState): ditto.
(WebCore::HTMLInputElement::restoreFormControlState): ditto.

  • html/HTMLInputElement.h:

(HTMLInputElement): ditto.

  • html/HTMLSelectElement.cpp:

(WebCore::HTMLSelectElement::saveFormControlState): ditto.
(WebCore::HTMLSelectElement::restoreFormControlState): ditto.

  • html/HTMLSelectElement.h:
  • html/HTMLTextAreaElement.cpp:

(WebCore::HTMLTextAreaElement::saveFormControlState): ditto.
(WebCore::HTMLTextAreaElement::restoreFormControlState): ditto.

  • html/HTMLTextAreaElement.h:
  • html/HiddenInputType.cpp:

(WebCore::HiddenInputType::saveFormControlState): ditto.
(WebCore::HiddenInputType::restoreFormControlState): ditto.

  • html/HiddenInputType.h:

(HiddenInputType): ditto.

  • html/InputType.cpp:

(WebCore::InputType::saveFormControlState): ditto.
(WebCore::InputType::restoreFormControlState): ditto.

  • html/InputType.h:

(InputType): ditto.

  • html/PasswordInputType.cpp:

(WebCore::PasswordInputType::saveFormControlState): ditto.
(WebCore::PasswordInputType::restoreFormControlState): ditto.

  • html/PasswordInputType.h:

(PasswordInputType): ditto.

11:16 PM Changeset in webkit [120678] by zandobersek@gmail.com
  • 5 edits in trunk/LayoutTests

Unreviewed GTK gardening, updating baselines after rollouts in r120627.

  • platform/gtk/fast/hidpi/clip-text-in-hidpi-expected.txt:
  • platform/gtk/fast/hidpi/image-set-background-dynamic-expected.txt:
  • platform/gtk/fast/hidpi/image-set-border-image-dynamic-expected.txt:
  • platform/gtk/fast/hidpi/image-set-in-content-dynamic-expected.txt:
11:09 PM Changeset in webkit [120677] by Csaba Osztrogonác
  • 4 edits
    2 adds in trunk/LayoutTests

[Qt] Unreviewed gardening, update expected files.

  • platform/qt/Skipped: Skip a new failing test to paint the bot green.
  • platform/qt/fast/css/getComputedStyle/computed-style-without-renderer-expected.txt: Updated after r120609.
  • platform/qt/fast/multicol/progression-reverse-expected.png: Added after r120609.
  • platform/qt/fast/multicol/progression-reverse-expected.txt: Added after r120609.
  • platform/qt/svg/css/getComputedStyle-basic-expected.txt: Updated after r120609.
11:07 PM Changeset in webkit [120676] by commit-queue@webkit.org
  • 5 edits
    3 adds in trunk

Source/JavaScriptCore: Changed JSC to always record line number information so that error.stack
and window.onerror() can report proper line numbers.
https://bugs.webkit.org/show_bug.cgi?id=89410

Patch by Mark Lam <mark.lam@apple.com> on 2012-06-18
Reviewed by Geoffrey Garen.

  • bytecode/CodeBlock.cpp:

(JSC::CodeBlock::CodeBlock):
(JSC::CodeBlock::lineNumberForBytecodeOffset):
(JSC::CodeBlock::shrinkToFit): m_lineInfo is now available unconditionally.

  • bytecode/CodeBlock.h:

(JSC::CodeBlock::addLineInfo):
(JSC::CodeBlock::hasLineInfo): Unused. Now removed.
(JSC::CodeBlock::needsCallReturnIndices):
(CodeBlock):
(RareData): Hoisted m_lineInfo out of m_rareData. m_lineInfo is now
filled in unconditionally.

  • bytecompiler/BytecodeGenerator.h:

(JSC::BytecodeGenerator::addLineInfo):

LayoutTests: Ensure that error.stack and window.onerror always have the appropriate
line numbers of the exception being thrown.
https://bugs.webkit.org/show_bug.cgi?id=89410

Patch by Mark Lam <mark.lam@apple.com> on 2012-06-18
Reviewed by Geoffrey Garen.

  • fast/js/exception-line-number-expected.txt: Added.
  • fast/js/exception-line-number.html: Added.
  • fast/js/script-tests/exception-line-number.js: Added.

(foo):
(window.onerror):

10:47 PM Changeset in webkit [120675] by Joone Hur
  • 4 edits in trunk

[EFL] Fontconfig can't be linked properly
https://bugs.webkit.org/show_bug.cgi?id=89418

Unreviewed build fix.

Make Fontconfig be linked correctly.

Source/WebCore:

No new tests. This is only a build fix.

  • PlatformEfl.cmake: add FONTCONFIG_LIBRARIES to WebCore_LIBRARIES.

Tools:

  • MiniBrowser/efl/CMakeLists.txt: add FONTCONFIG_LIBRARIES to MiniBrowser_LIBRARIES.
10:47 PM Changeset in webkit [120674] by kinuko@chromium.org
  • 2 edits in trunk/LayoutTests

Unreviewed test fix (one more) for r120667: skip '.svn' directory from the test results.

  • fast/forms/file/input-file-entries.html:
10:34 PM Changeset in webkit [120673] by Martin Robinson
  • 2 edits in trunk/Source/WebCore

[Cairo] Fix memory leaks in GLContextGLX.cpp
https://bugs.webkit.org/show_bug.cgi?id=89346

Patch by Sudarsana Nagineni <sudarsana.nagineni@linux.intel.com> on 2012-06-18
Reviewed by Martin Robinson.

Fix memory leaks found in GLContextGLX.cpp.

  • platform/graphics/glx/GLContextGLX.cpp:

(WebCore::GLContextGLX::cleanupActiveContextsAtExit): Removing element reduces
size of the vector, so the loop termination condition was failing in the last
iteration, when the vector contains more than one element. This patch fixes the
issue by iterating the vector backwards, so the termination condition doesn't
fail until the size() reaches 0.
(WebCore::GLContextGLX::createPbufferContext): Delete GLXPbuffer
returned by glXCreatePbuffer().
(WebCore::GLContextGLX::createPixmapContext): Delete XVisualInfo
returned by glXChooseVisual().

10:19 PM Changeset in webkit [120672] by mikelawther@chromium.org
  • 1 edit
    1 add in trunk/LayoutTests

Unreviewed gardening. Add missing expectation file.

  • fast/text/title-crash-expected.txt: Added.
10:09 PM Changeset in webkit [120671] by Joone Hur
  • 2 edits in trunk/Tools

Unreviewed. Updating email for committers.py script.

  • Scripts/webkitpy/common/config/committers.py:
10:08 PM Changeset in webkit [120670] by kinuko@chromium.org
  • 2 edits in trunk/LayoutTests

Unreviewed test fix for r120667: skip '.svn' directory from the test results.

  • fast/forms/file/input-file-entries.html:
9:56 PM Changeset in webkit [120669] by Simon Fraser
  • 2 edits in trunk/Tools

Fix a typo that caused TestFailures to have a syntax error.

Sadly TestFailures is broken for another reason now too:
https://bugs.webkit.org/show_bug.cgi?id=89419

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

(ViewController.prototype._domForBuildName):

9:34 PM Changeset in webkit [120668] by commit-queue@webkit.org
  • 16 edits
    2 copies
    3 adds in trunk

Source/WebCore: Expose WEBGL_depth_texture extension to WebGL
https://bugs.webkit.org/show_bug.cgi?id=87029

Patch by Gregg Tavares <gman@google.com> on 2012-06-18
Reviewed by Kenneth Russell.

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

  • CMakeLists.txt:
  • DerivedSources.make:
  • GNUmakefile.list.am:
  • Target.pri:
  • WebCore.gypi:
  • bindings/js/JSWebGLRenderingContextCustom.cpp:

(WebCore::toJS):

  • bindings/v8/custom/V8WebGLRenderingContextCustom.cpp:

(WebCore::toV8Object):

  • html/canvas/WebGLDepthTexture.cpp: Copied from Source/WebCore/html/canvas/WebGLExtension.h.

(WebCore):
(WebCore::WebGLDepthTexture::WebGLDepthTexture):
(WebCore::WebGLDepthTexture::~WebGLDepthTexture):
(WebCore::WebGLDepthTexture::getName):
(WebCore::WebGLDepthTexture::create):
(WebCore::WebGLDepthTexture::supported):

  • html/canvas/WebGLDepthTexture.h: Copied from Source/WebCore/html/canvas/WebGLExtension.h.

(WebCore):
(WebGLDepthTexture):

  • html/canvas/WebGLDepthTexture.idl: Copied from Source/WebCore/html/canvas/WebGLExtension.h.
  • html/canvas/WebGLExtension.h:
  • html/canvas/WebGLFramebuffer.cpp:

(WebCore::WebGLFramebuffer::initializeAttachments):

  • html/canvas/WebGLRenderingContext.cpp:

(WebCore):
(WebCore::WebGLRenderingContext::validateSettableTexFormat):
(WebCore::WebGLRenderingContext::copyTexImage2D):
(WebCore::WebGLRenderingContext::copyTexSubImage2D):
(WebCore::WebGLRenderingContext::generateMipmap):
(WebCore::WebGLRenderingContext::getExtension):
(WebCore::WebGLRenderingContext::getSupportedExtensions):
(WebCore::WebGLRenderingContext::texImage2DBase):
(WebCore::WebGLRenderingContext::texImage2D):
(WebCore::WebGLRenderingContext::texSubImage2DBase):
(WebCore::WebGLRenderingContext::texSubImage2D):
(WebCore::WebGLRenderingContext::validateTexFuncFormatAndType):
(WebCore::WebGLRenderingContext::validateTexFuncParameters):
(WebCore::WebGLRenderingContext::validateTexFuncData):

  • html/canvas/WebGLRenderingContext.h:

(WebCore):
(WebGLRenderingContext):

  • platform/graphics/GraphicsContext3D.cpp:

(WebCore::GraphicsContext3D::getClearBitsByAttachmentType):
(WebCore):
(WebCore::GraphicsContext3D::getClearBitsByFormat):

  • platform/graphics/GraphicsContext3D.h:

LayoutTests: Expose WEBGL_depth_texture extension to WebGL
https://bugs.webkit.org/show_bug.cgi?id=87029

Patch by Gregg Tavares <gman@google.com> on 2012-06-18
Reviewed by Kenneth Russell.

  • fast/canvas/webgl/resources/webgl-test-utils.js:

(WebGLTestUtils):

  • fast/canvas/webgl/webgl-depth-texture-expected.txt: Added.
  • fast/canvas/webgl/webgl-depth-texture.html: Added.
9:13 PM Changeset in webkit [120667] by kinuko@chromium.org
  • 21 edits
    8 adds in trunk

Support File/DirectoryEntry access for <input type=file> if FileSystem API is enabled
https://bugs.webkit.org/show_bug.cgi?id=88293

Reviewed by Tony Chang.

Source/WebCore:

Proposal email on whatwg: http://lists.whatwg.org/pipermail/whatwg-whatwg.org/2011-November/033814.html
Proposal draft: http://wiki.whatwg.org/wiki/DragAndDropEntries

Test: fast/forms/file/input-file-entries.html

  • Modules/filesystem/HTMLInputElementFileSystem.cpp: Added.
  • Modules/filesystem/HTMLInputElementFileSystem.h: Added.
  • Modules/filesystem/HTMLInputElementFileSystem.idl: Added.
  • WebCore.gypi:
  • html/FileInputType.cpp:

(WebCore::FileInputType::receiveDroppedFiles): Retrieves dropped file
system Id if the given DragData has it.
(WebCore::FileInputType::droppedFileSystemId): Added. This returns
dropped file system Id retrieved in receiveDroppedFiles, or an empty
string if the platform does not support isolated file system for
dropped files.

  • html/FileInputType.h:
  • html/HTMLInputElement.cpp:

(WebCore::HTMLInputElement::droppedFileSystemId): Added.

  • html/HTMLInputElement.h:
  • html/InputType.cpp:

(WebCore::InputType::droppedFileSystemId): Added.

  • html/InputType.h:
  • platform/DragData.cpp:
  • platform/DragData.h:
  • platform/chromium/DragDataChromium.cpp:

(WebCore::DragData::droppedFileSystemId): Added.

LayoutTests:

  • fast/forms/file/input-file-entries-expected.txt: Added.
  • fast/forms/file/input-file-entries.html: Added.
  • fast/forms/resources/dir1/UTF8-2.txt: Added.
  • fast/forms/resources/dir1/UTF8.txt: Added.
  • fast/forms/resources/dir2/green.jpg: Added.
  • platform/efl/Skipped:
  • platform/efl/TestExpectations:
  • platform/gtk/TestExpectations:
  • platform/mac/Skipped:
  • platform/qt/Skipped:
  • platform/win/Skipped:
  • platform/wincairo/Skipped:
  • platform/wk2/Skipped:
9:02 PM Changeset in webkit [120666] by shinyak@chromium.org
  • 4 edits
    2 adds in trunk

[Crash][Editing] VisibleSelection::adjustSelectionToAvoidCrossingEditingBoundaries() crashes in some Shadow DOM case.
https://bugs.webkit.org/show_bug.cgi?id=88500

Reviewed by Ryosuke Niwa.

Source/WebCore:

When selecting from a child of shadow host to an element in Shadow DOM, the start position of
Selection comes after the end position of Selection in some cases. This caused a crash.

For example, let's think '<div id="host">A</div>D' and host has a Shadow DOM whose innerHTML
is 'B<content></content>C'. When selecting from just after A (p1 = (PositionIsOffsetInAnchor,
#text, 1)) to just before B (p2 = (PositionIsOffsetInAnchor, shadowRoot, 0)), since
comparePosition always judge as p2 is before p1, m_start and m_end in VisibleSelection
will be p2 and p1 respectively.

However, the anchorNodes of p1 and p2 have the different tree scope, we have to adjust p2 to
the same tree scope as p1. Since p2 is m_start, m_start was adjusted to positionAfterNode(div).
This breaks the assumption that m_start is always before (or equal to) m_end.

This patch keeps the assumption. When we cannot move m_start to positionAfterNode(div),
we move m_start to positionBeforeNode(div). Note that we cannot always move m_start to
positionBeforeNode(div), because when selecting just after D (p3 = (PositionIsOffsetInAnchor,
D's parent, 2), we don't want to select A.

In this case, m_start should be adjusted to positionAfterNode(div).

Test: editing/shadow/breaking-editing-boundaries.html

  • editing/VisibleSelection.cpp:

(WebCore::adjustPositionForEnd):
(WebCore):
(WebCore::adjustPositionForStart):
(WebCore::VisibleSelection::adjustSelectionToAvoidCrossingShadowBoundaries):

  • editing/VisibleSelection.h:

(VisibleSelection):

LayoutTests:

  • editing/shadow/breaking-editing-boundaries-expected.txt: Added.
  • editing/shadow/breaking-editing-boundaries.html: Added.
8:54 PM Changeset in webkit [120665] by morrita@google.com
  • 2 edits in trunk/LayoutTests

Unreviewed test marking. Marked border-radius-clip-002.htm as a failure.

  • platform/chromium/TestExpectations:
8:34 PM Changeset in webkit [120664] by aestes@apple.com
  • 2 edits in trunk/Source/JavaScriptCore

Fix r120663, which didn't land the change that was reviewed.

8:32 PM Changeset in webkit [120663] by aestes@apple.com
  • 2 edits in trunk/Source/JavaScriptCore

[JSC] In JSGlobalData.cpp, enableAssembler() sometimes leaks two CF objects
https://bugs.webkit.org/show_bug.cgi?id=89415

Reviewed by Sam Weinig.

In the case where canUseJIT was a non-NULL CFBooleanRef,
enableAssembler() would leak both canUseJITKey and canUseJIT by
returning before calling CFRelease. Fix this by using RetainPtr.

  • runtime/JSGlobalData.cpp:

(JSC::enableAssembler):

8:20 PM Changeset in webkit [120662] by mikelawther@chromium.org
  • 3 edits
    1 add in trunk

Crash when setting title dynamically
https://bugs.webkit.org/show_bug.cgi?id=88083

Reviewed by Dan Bernstein.

Source/WebCore:

Test: fast/text/title-crash.html

The crashing code takes a rare branch in StyleResolver::styleForElement() where
m_parentStyle is set to point to m_style. Consequently, while applying properties
to m_style we end up mutating m_parentStyle.

In this situation, we clone style() and point m_parentStyle at the clone. The
clone is destroyed at the end of StyleResolver::styleForElement().

  • css/StyleResolver.cpp:

(WebCore::StyleResolver::collectMatchingRulesForList):

LayoutTests:

  • fast/text/title-crash.html: Added.
8:04 PM Changeset in webkit [120661] by enne@google.com
  • 3 edits
    3 adds in trunk

RenderLayerBacking sometimes doesn't attach scrollbars to graphics layer tree
https://bugs.webkit.org/show_bug.cgi?id=89402

Reviewed by James Robinson.

Source/WebCore:

RenderLayerBacking only attaches scrollbar layers to the graphics
layer tree if there's a clip layer. This is wrong, because these
scrollbars become orphans and other code expects them to handle
repainting the scrollbars (which now become invisible). Fix by always
attaching scrollbar layers.

Test: compositing/overflow/image-load-overflow-scrollbars.html

  • rendering/RenderLayerBacking.cpp:

(WebCore::RenderLayerBacking::updateInternalHierarchy):

LayoutTests:

Add a test where an image load creates overflow.

  • compositing/overflow/image-load-overflow-scrollbars-expected.png: Added.
  • compositing/overflow/image-load-overflow-scrollbars-expected.txt: Added.
  • compositing/overflow/image-load-overflow-scrollbars.html: Added.
7:54 PM Changeset in webkit [120660] by fischman@chromium.org
  • 2 edits in trunk/LayoutTests

Make serve-video.php serve 206 responses
https://bugs.webkit.org/show_bug.cgi?id=89252

Reviewed by Adam Barth.

  • http/tests/media/resources/serve-video.php:
7:51 PM Changeset in webkit [120659] by abarth@webkit.org
  • 2 edits in trunk/Source/WebKit/chromium

[Chromium] GesturePinchBegin and friends are gesture events
https://bugs.webkit.org/show_bug.cgi?id=89398

Reviewed by James Robinson.

  • public/WebInputEvent.h:

(WebKit::WebInputEvent::isGestureEventType):

7:25 PM Changeset in webkit [120658] by morrita@google.com
  • 2 edits in trunk/LayoutTests

Unreviewed test marking. Marked border-radius-clip-002.htm as a failure.

  • platform/chromium/TestExpectations:
7:06 PM Changeset in webkit [120657] by Simon Fraser
  • 4 edits in trunk/Tools

Provide bug links for suspicious commits
https://bugs.webkit.org/show_bug.cgi?id=89408

Reviewed by Adam Barth.

Add links to bugs for the suspicious commits if we have bug data.

  • BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/scripts/Bugzilla.js:
  • BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/scripts/ui/notifications.js:
  • BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/styles/notifications.css:

(ol.notifications>li ul.causes>li>div.description>span>span.bugID::before):
(ol.notifications>li ul.causes>li>div.description>span>span.bugID>a):

6:54 PM Changeset in webkit [120656] by jchaffraix@webkit.org
  • 4 edits
    6 copies in branches/chromium/1132

Merge 120257 - Padding and borders can cause integer overflow in block layouts

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

< rdar://problem/11328762>
Reviewed by Tony Chang.
Source/WebCore:
Tests: fast/block/block-size-integer-overflow.html
fast/flexbox/box-size-integer-overflow.html
fast/table/table-size-integer-overflow.html

  • rendering/AutoTableLayout.cpp: Decreased max int.

(WebCore::AutoTableLayout::computePreferredLogicalWidths):

  • rendering/FixedTableLayout.cpp: Use shared constant.

(WebCore::FixedTableLayout::computePreferredLogicalWidths):

  • rendering/RenderBlock.cpp: Removed unused constant.
  • rendering/TableLayout.h: Add shared constant.

(TableLayout):
LayoutTests:

  • fast/block/block-size-integer-overflow-expected.txt: Added.
  • fast/block/block-size-integer-overflow.html: Added.
  • fast/flexbox/box-size-integer-overflow-expected.txt: Added.
  • fast/flexbox/box-size-integer-overflow.html: Added.
  • fast/table/table-size-integer-overflow-expected.txt: Added.
  • fast/table/table-size-integer-overflow.html: Added.

TBR=tc@chromium.org
Review URL: https://chromiumcodereview.appspot.com/10583010

6:38 PM Changeset in webkit [120655] by commit-queue@webkit.org
  • 10 edits in trunk

[Chromium] DeviceOrientation Cleanup
https://bugs.webkit.org/show_bug.cgi?id=89354

Patch by Amy Ousterhout <aousterh@chromium.org> on 2012-06-18
Reviewed by Kent Tamura.

Source/WebCore:

No new tests because there is no intended change in behavior.

  • platform/mock/DeviceOrientationClientMock.h: added OVERRIDE specifier for virtual functions

(DeviceOrientationClientMock):

Source/WebKit/chromium:

  • public/WebDeviceOrientation.h: added const-qualifiers to getters, simplified an overly-complicated parameter to two functions

(WebKit::WebDeviceOrientation::isNull):
(WebKit::WebDeviceOrientation::canProvideAlpha):
(WebKit::WebDeviceOrientation::alpha):
(WebKit::WebDeviceOrientation::canProvideBeta):
(WebKit::WebDeviceOrientation::beta):
(WebKit::WebDeviceOrientation::canProvideGamma):
(WebKit::WebDeviceOrientation::gamma):
(WebKit::WebDeviceOrientation::canProvideAbsolute):
(WebKit::WebDeviceOrientation::absolute):
(WebDeviceOrientation):

  • public/WebDeviceOrientationClient.h: added WEBKIT_ASSERT_NOT_REACHED to virtual functions

(WebKit::WebDeviceOrientationClient::setController):
(WebKit::WebDeviceOrientationClient::startUpdating):
(WebKit::WebDeviceOrientationClient::stopUpdating):
(WebKit::WebDeviceOrientationClient::lastOrientation):

  • public/WebDeviceOrientationClientMock.h: added OVERRIDE specifier for virtual functions

(WebDeviceOrientationClientMock):

  • src/DeviceOrientationClientProxy.h: designated functions as 'virtual' and added OVERRIDE specifier

(DeviceOrientationClientProxy):

  • src/WebDeviceOrientation.cpp: simplified overly-complicated parameter to two functions

(WebKit::WebDeviceOrientation::WebDeviceOrientation):
(WebKit::WebDeviceOrientation::operator=):

Tools:

  • DumpRenderTree/chromium/WebViewHost.h: added OVERRIDE specifier for virtual function

(WebViewHost):

6:32 PM Changeset in webkit [120654] by commit-queue@webkit.org
  • 4 edits in trunk

[EFL] media/video-seek-past-end-playing.html is flaky
https://bugs.webkit.org/show_bug.cgi?id=89301

Patch by Christophe Dumez <Christophe Dumez> on 2012-06-18
Reviewed by Philippe Normand.

Source/WebCore:

Switch to double for media duration and playback position
calculations to avoid precision problems.
This patch also removes the workaround for the precision
problem in didEnd() which is no longer needed.

Test: media/video-seek-past-end-playing.html

  • platform/graphics/gstreamer/MediaPlayerPrivateGStreamer.cpp:

(WebCore::MediaPlayerPrivateGStreamer::playbackPosition):
(WebCore::MediaPlayerPrivateGStreamer::duration):

LayoutTests:

Unskip media/video-seek-past-end-playing.html now that the
precision problem in MediaPlayerPrivateGStreamer is fixed.

  • platform/efl/TestExpectations:
6:30 PM Changeset in webkit [120653] by scherkus@chromium.org
  • 117 edits
    16 moves
    1 add
    6 deletes in trunk/LayoutTests

2012-06-18 Andrew Scherkus <scherkus@chromium.org>

[Chromium] Unreviewed. Rebaseline media layout tests due to updated controls.

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

  • platform/chromium-linux-x86/media/video-empty-source-expected.txt: Removed.
  • platform/chromium-linux/fast/layers/video-layer-expected.png:
  • platform/chromium-linux/fullscreen/full-screen-stacking-context-expected.png:
  • platform/chromium-linux/http/tests/media/video-buffered-range-contains-currentTime-expected.png:
  • platform/chromium-linux/media/audio-controls-rendering-expected.png:
  • platform/chromium-linux/media/audio-repaint-expected.png:
  • platform/chromium-linux/media/controls-after-reload-expected.png:
  • platform/chromium-linux/media/controls-layout-direction-expected.png:
  • platform/chromium-linux/media/controls-strict-expected.png:
  • platform/chromium-linux/media/controls-styling-expected.png:
  • platform/chromium-linux/media/controls-without-preload-expected.png:
  • platform/chromium-linux/media/media-controls-clone-expected.png:
  • platform/chromium-linux/media/media-document-audio-repaint-expected.png:
  • platform/chromium-linux/media/video-controls-rendering-expected.png:
  • platform/chromium-linux/media/video-display-toggle-expected.png:
  • platform/chromium-linux/media/video-empty-source-expected.png:
  • platform/chromium-linux/media/video-empty-source-expected.txt: Removed.
  • platform/chromium-linux/media/video-no-audio-expected.png:
  • platform/chromium-linux/media/video-playing-and-pause-expected.png:
  • platform/chromium-linux/media/video-zoom-controls-expected.png:
  • platform/chromium-mac-leopard/fast/layers/video-layer-expected.png:
  • platform/chromium-mac-leopard/http/tests/media/video-buffered-range-contains-currentTime-expected.png:
  • platform/chromium-mac-leopard/media/audio-controls-rendering-expected.png:
  • platform/chromium-mac-leopard/media/audio-repaint-expected.png:
  • platform/chromium-mac-leopard/media/controls-after-reload-expected.png:
  • platform/chromium-mac-leopard/media/controls-strict-expected.png:
  • platform/chromium-mac-leopard/media/controls-styling-expected.png:
  • platform/chromium-mac-leopard/media/controls-without-preload-expected.png:
  • platform/chromium-mac-leopard/media/media-document-audio-repaint-expected.png:
  • platform/chromium-mac-leopard/media/video-controls-rendering-expected.png:
  • platform/chromium-mac-leopard/media/video-display-toggle-expected.png:
  • platform/chromium-mac-leopard/media/video-empty-source-expected.png:
  • platform/chromium-mac-leopard/media/video-empty-source-expected.txt: Removed.
  • platform/chromium-mac-leopard/media/video-no-audio-expected.png:
  • platform/chromium-mac-leopard/media/video-playing-and-pause-expected.png:
  • platform/chromium-mac-snowleopard/fast/layers/video-layer-expected.png:
  • platform/chromium-mac-snowleopard/fullscreen/full-screen-stacking-context-expected.png:
  • platform/chromium-mac-snowleopard/http/tests/media/video-buffered-range-contains-currentTime-expected.png:
  • platform/chromium-mac-snowleopard/media/audio-controls-rendering-expected.png:
  • platform/chromium-mac-snowleopard/media/audio-repaint-expected.png:
  • platform/chromium-mac-snowleopard/media/controls-after-reload-expected.png:
  • platform/chromium-mac-snowleopard/media/controls-layout-direction-expected.png:
  • platform/chromium-mac-snowleopard/media/controls-strict-expected.png:
  • platform/chromium-mac-snowleopard/media/controls-styling-expected.png:
  • platform/chromium-mac-snowleopard/media/controls-without-preload-expected.png:
  • platform/chromium-mac-snowleopard/media/media-controls-clone-expected.png:
  • platform/chromium-mac-snowleopard/media/media-document-audio-repaint-expected.png:
  • platform/chromium-mac-snowleopard/media/video-controls-rendering-expected.png:
  • platform/chromium-mac-snowleopard/media/video-display-toggle-expected.png:
  • platform/chromium-mac-snowleopard/media/video-empty-source-expected.png:
  • platform/chromium-mac-snowleopard/media/video-empty-source-expected.txt: Removed.
  • platform/chromium-mac-snowleopard/media/video-no-audio-expected.png:
  • platform/chromium-mac-snowleopard/media/video-playing-and-pause-expected.png:
  • platform/chromium-mac/fast/layers/video-layer-expected.png:
  • platform/chromium-mac/fast/layers/video-layer-expected.txt:
  • platform/chromium-mac/fullscreen/full-screen-stacking-context-expected.png:
  • platform/chromium-mac/http/tests/media/video-buffered-range-contains-currentTime-expected.png:
  • platform/chromium-mac/media/audio-controls-rendering-expected.png:
  • platform/chromium-mac/media/audio-controls-rendering-expected.txt:
  • platform/chromium-mac/media/audio-repaint-expected.png:
  • platform/chromium-mac/media/audio-repaint-expected.txt:
  • platform/chromium-mac/media/controls-after-reload-expected.png:
  • platform/chromium-mac/media/controls-after-reload-expected.txt:
  • platform/chromium-mac/media/controls-layout-direction-expected.png:
  • platform/chromium-mac/media/controls-strict-expected.png:
  • platform/chromium-mac/media/controls-strict-expected.txt:
  • platform/chromium-mac/media/controls-styling-expected.png:
  • platform/chromium-mac/media/controls-styling-expected.txt:
  • platform/chromium-mac/media/controls-without-preload-expected.png:
  • platform/chromium-mac/media/controls-without-preload-expected.txt:
  • platform/chromium-mac/media/media-controls-clone-expected.png:
  • platform/chromium-mac/media/media-controls-clone-expected.txt:
  • platform/chromium-mac/media/media-document-audio-repaint-expected.png:
  • platform/chromium-mac/media/media-document-audio-repaint-expected.txt:
  • platform/chromium-mac/media/video-controls-rendering-expected.png:
  • platform/chromium-mac/media/video-controls-rendering-expected.txt:
  • platform/chromium-mac/media/video-display-toggle-expected.png:
  • platform/chromium-mac/media/video-display-toggle-expected.txt:
  • platform/chromium-mac/media/video-empty-source-expected.png:
  • platform/chromium-mac/media/video-empty-source-expected.txt:
  • platform/chromium-mac/media/video-no-audio-expected.png:
  • platform/chromium-mac/media/video-no-audio-expected.txt:
  • platform/chromium-mac/media/video-playing-and-pause-expected.png:
  • platform/chromium-mac/media/video-playing-and-pause-expected.txt:
  • platform/chromium-mac/media/video-zoom-controls-expected.png:
  • platform/chromium-mac/media/video-zoom-controls-expected.txt:
  • platform/chromium-win-vista/media/video-empty-source-expected.txt: Removed.
  • platform/chromium-win-xp/media/video-empty-source-expected.txt: Removed.
  • platform/chromium-win/fast/layers/video-layer-expected.png:
  • platform/chromium-win/fast/layers/video-layer-expected.txt:
  • platform/chromium-win/fullscreen/full-screen-stacking-context-expected.png:
  • platform/chromium-win/http/tests/media/video-buffered-range-contains-currentTime-expected.png:
  • platform/chromium-win/media/audio-controls-rendering-expected.png:
  • platform/chromium-win/media/audio-controls-rendering-expected.txt:
  • platform/chromium-win/media/audio-repaint-expected.png:
  • platform/chromium-win/media/audio-repaint-expected.txt:
  • platform/chromium-win/media/controls-after-reload-expected.png:
  • platform/chromium-win/media/controls-after-reload-expected.txt:
  • platform/chromium-win/media/controls-layout-direction-expected.png:
  • platform/chromium-win/media/controls-strict-expected.png:
  • platform/chromium-win/media/controls-strict-expected.txt:
  • platform/chromium-win/media/controls-styling-expected.png:
  • platform/chromium-win/media/controls-styling-expected.txt:
  • platform/chromium-win/media/controls-without-preload-expected.png:
  • platform/chromium-win/media/controls-without-preload-expected.txt:
  • platform/chromium-win/media/media-controls-clone-expected.png:
  • platform/chromium-win/media/media-controls-clone-expected.txt:
  • platform/chromium-win/media/media-document-audio-repaint-expected.png:
  • platform/chromium-win/media/media-document-audio-repaint-expected.txt:
  • platform/chromium-win/media/video-controls-rendering-expected.png:
  • platform/chromium-win/media/video-controls-rendering-expected.txt:
  • platform/chromium-win/media/video-display-toggle-expected.png:
  • platform/chromium-win/media/video-display-toggle-expected.txt:
  • platform/chromium-win/media/video-empty-source-expected.png:
  • platform/chromium-win/media/video-empty-source-expected.txt:
  • platform/chromium-win/media/video-no-audio-expected.png:
  • platform/chromium-win/media/video-no-audio-expected.txt:
  • platform/chromium-win/media/video-playing-and-pause-expected.png:
  • platform/chromium-win/media/video-playing-and-pause-expected.txt:
  • platform/chromium-win/media/video-zoom-controls-expected.png:
  • platform/chromium-win/media/video-zoom-controls-expected.txt:
  • platform/chromium/TestExpectations:
  • platform/win-xp/fast/layers/video-layer-expected.txt: Renamed from LayoutTests/platform/win/fast/layers/video-layer-expected.txt.
  • platform/win-xp/media/audio-controls-rendering-expected.txt: Renamed from LayoutTests/platform/win/media/audio-controls-rendering-expected.txt.
  • platform/win-xp/media/audio-repaint-expected.txt: Renamed from LayoutTests/platform/win/media/audio-repaint-expected.txt.
  • platform/win-xp/media/controls-after-reload-expected.txt: Renamed from LayoutTests/platform/win/media/controls-after-reload-expected.txt.
  • platform/win-xp/media/controls-strict-expected.txt: Renamed from LayoutTests/platform/win/media/controls-strict-expected.txt.
  • platform/win-xp/media/controls-styling-expected.txt: Renamed from LayoutTests/platform/win/media/controls-styling-expected.txt.
  • platform/win-xp/media/controls-without-preload-expected.txt: Renamed from LayoutTests/platform/win/media/controls-without-preload-expected.txt.
  • platform/win-xp/media/media-controls-clone-expected.txt: Renamed from LayoutTests/platform/win/media/media-controls-clone-expected.txt.
  • platform/win-xp/media/media-document-audio-repaint-expected.txt: Renamed from LayoutTests/platform/win/media/media-document-audio-repaint-expected.txt.
  • platform/win-xp/media/video-controls-rendering-expected.txt: Renamed from LayoutTests/platform/win/media/video-controls-rendering-expected.txt.
  • platform/win-xp/media/video-display-toggle-expected.txt: Renamed from LayoutTests/platform/win/media/video-display-toggle-expected.txt.
  • platform/win-xp/media/video-empty-source-expected.txt: Renamed from LayoutTests/platform/win/media/video-empty-source-expected.txt.
  • platform/win-xp/media/video-no-audio-expected.txt: Renamed from LayoutTests/platform/win/media/video-no-audio-expected.txt.
  • platform/win-xp/media/video-playing-and-pause-expected.txt: Renamed from LayoutTests/platform/win/media/video-playing-and-pause-expected.txt.
  • platform/win-xp/media/video-zoom-controls-expected.png: Renamed from LayoutTests/platform/win/media/video-zoom-controls-expected.png.
  • platform/win-xp/media/video-zoom-controls-expected.txt: Renamed from LayoutTests/platform/win/media/video-zoom-controls-expected.txt.
6:29 PM Changeset in webkit [120652] by fischman@chromium.org
  • 2 edits in trunk/Source/WebKit/chromium

Unreviewed. Rolled DEPS.

  • DEPS:
6:21 PM Changeset in webkit [120651] by atwilson@chromium.org
  • 2 edits in trunk/LayoutTests

Unreviewed chromium expectations update to mark tests as timing out due to r120627.

  • platform/chromium/TestExpectations:
6:12 PM Changeset in webkit [120650] by commit-queue@webkit.org
  • 2 edits in trunk/Tools

[chromium] Stop passing deprecated 'direct' parameter to webkit_support::CreateGraphicsContext3D
https://bugs.webkit.org/show_bug.cgi?id=89254

Patch by James Robinson <jamesr@chromium.org> on 2012-06-18
Reviewed by Adrienne Walker.

This parameter doesn't mean anything since this codepath is only used for onscreen contexts and is deprecated
upstream.

  • DumpRenderTree/chromium/WebViewHost.cpp:

(WebViewHost::createGraphicsContext3D):

6:09 PM Changeset in webkit [120649] by Simon Fraser
  • 2 edits in trunk/Tools

Minor fix suggested by Adam Barth.

Use $(this._what).empty(); rather than removing children one at a time.
For some reason removeAllChildren() is undefined on this node.

  • BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/scripts/ui/notifications.js:
6:03 PM Changeset in webkit [120648] by Simon Fraser
  • 7 edits in trunk/Tools

Show TOT revision in garden-o-matic, and make the revisions into links
https://bugs.webkit.org/show_bug.cgi?id=89396

Reviewed by Adam Barth.

The "latest revision" notification now reads "Latest revision processed by every bot: NN (trunk is at NN)"
where the revisions are links to trac.

To avoid setting innerHTML, I made base.createLinkNode() and used it in a bunch of places, which had
the knock-on effect of flipping the target and href attributes for anchors in some test output.

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

(ViewController.prototype._displayTesters.updateList.list):
(ViewController.prototype._displayTesters.updateList):
(ViewController.prototype._domForRegressionRange.trac.commitDataForRevisionRange):
(ViewController.prototype._domForRegressionRange):
(ViewController.prototype._domForAuxiliaryUIElements):
(ViewController.prototype._domForBuildName):
(ViewController.prototype):
(ViewController.prototype.):

  • BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/scripts/base.js:
  • BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/scripts/garden-o-matic.js:
  • BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/scripts/model.js:
  • BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/scripts/ui/notifications.js:
  • BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/scripts/ui/notifications_unittests.js:
6:02 PM Changeset in webkit [120647] by atwilson@chromium.org
  • 1 edit
    1 move
    1 add in trunk/LayoutTests

Unreviewed chromium expectations.

  • platform/chromium-mac/compositing/fixed-position-scroll-offset-history-restore-expected.png: Added.
  • platform/chromium-win/compositing/fixed-position-scroll-offset-history-restore-expected.png: Renamed from LayoutTests/platform/chromium-linux/compositing/fixed-position-scroll-offset-history-restore-expected.png.
5:58 PM Changeset in webkit [120646] by dpranke@chromium.org
  • 2 edits in trunk/Tools

REGRESSION(r100558): NRWT should work without SVN or GIT
https://bugs.webkit.org/show_bug.cgi?id=76630

Patch by Csaba Osztrogonác <Csaba Osztrogonác> on 2012-06-18
Reviewed by Dirk Pranke.

NRWT is not supposed to have a requirement that we have an
actual SVN or Git checkout, and we were unnecessarily
initializing the checkout. This patch removes that line :).

  • Scripts/webkitpy/layout_tests/run_webkit_tests.py:

(main):

5:50 PM Changeset in webkit [120645] by commit-queue@webkit.org
  • 21 edits
    8 deletes in trunk

Unreviewed, rolling out r120593.
http://trac.webkit.org/changeset/120593
https://bugs.webkit.org/show_bug.cgi?id=89404

Caused failures in fast/forms/file/input-file-entries.html
(Requested by atwilson on #webkit).

Patch by Sheriff Bot <webkit.review.bot@gmail.com> on 2012-06-18

Source/WebCore:

  • Modules/filesystem/DOMFileSystem.cpp:

(WebCore::DOMFileSystem::createIsolatedFileSystem):

  • Modules/filesystem/HTMLInputElementFileSystem.cpp: Removed.
  • Modules/filesystem/HTMLInputElementFileSystem.h: Removed.
  • Modules/filesystem/HTMLInputElementFileSystem.idl: Removed.
  • WebCore.gypi:
  • html/FileInputType.cpp:

(WebCore::FileInputType::receiveDroppedFiles):

  • html/FileInputType.h:

(FileInputType):

  • html/HTMLInputElement.cpp:
  • html/HTMLInputElement.h:

(HTMLInputElement):

  • html/InputType.cpp:
  • html/InputType.h:

(InputType):

  • platform/DragData.cpp:

(WebCore::DragData::modifierKeyState):
(WebCore):

  • platform/DragData.h:

(DragData):

  • platform/chromium/DragDataChromium.cpp:

LayoutTests:

  • fast/forms/file/input-file-entries-expected.txt: Removed.
  • fast/forms/file/input-file-entries.html: Removed.
  • fast/forms/resources/dir1/UTF8-2.txt: Removed.
  • fast/forms/resources/dir1/UTF8.txt: Removed.
  • fast/forms/resources/dir2/green.jpg: Removed.
  • platform/efl/Skipped:
  • platform/efl/TestExpectations:
  • platform/gtk/TestExpectations:
  • platform/mac/Skipped:
  • platform/qt/Skipped:
  • platform/win/Skipped:
  • platform/wincairo/Skipped:
  • platform/wk2/Skipped:
4:54 PM Changeset in webkit [120644] by jamesr@google.com
  • 6 edits
    2 adds in trunk/Source

[chromium] Use WebImageLayer in GraphicsLayerChromium for image layers
https://bugs.webkit.org/show_bug.cgi?id=89150

Reviewed by Adrienne Walker.

Source/Platform:

Adds a WebImageLayer type to represent layers backed by (guess what) Images.

  • Platform.gypi:
  • chromium/public/WebImageLayer.h: Added.

(WebCore):
(WebKit):
(WebImageLayer):
(WebKit::WebImageLayer::WebImageLayer):
(WebKit::WebImageLayer::~WebImageLayer):

Source/WebCore:

Uses WebImageLayer instead of ImageLayerChromium for GraphicsLayerChromium::setContentsToImage.

  • platform/graphics/chromium/GraphicsLayerChromium.cpp:

(WebCore::GraphicsLayerChromium::setContentsToImage):

Source/WebKit/chromium:

Implementation of WebImageLayer. Note: This could (should?) live in WebCore/platform/chromium/support/, but it's
here to be next to the other Web*Layer.cpp implementation files.

  • WebKit.gyp:
  • src/WebImageLayer.cpp: Added.

(WebKit):
(WebKit::WebImageLayer::create):
(WebKit::WebImageLayer::WebImageLayer):
(WebKit::WebImageLayer::setBitmap):

4:41 PM Changeset in webkit [120643] by jamesr@google.com
  • 11 edits in trunk/Source

[chromium] GraphicsLayerChromium should use WebContentLayer directly
https://bugs.webkit.org/show_bug.cgi?id=89145

Reviewed by Adrienne Walker.

Source/Platform:

Add an opaque rect parameter to WebContentLayerClient::paintContents() for a client to (optionally) provide
information about opaque portions of the painted region for culling optimizations.

  • chromium/public/WebContentLayerClient.h:

(WebContentLayerClient):

Source/WebCore:

Ports GraphicsLayerChromium (or more specifically OpaqueRectTrackingContentLayerDelegate) to implement
WebContentLayerClient instead of ContentLayerDelegate. In addition to the mechanical changes, this requires
eagerly setting drawsContent on GraphicsLayerChromium::m_layer since WebContentLayerImpl's default value is
different from LayerChromium's. Covered by all of compositing/

  • platform/graphics/chromium/GraphicsLayerChromium.cpp:

(WebCore::GraphicsLayerChromium::GraphicsLayerChromium):

  • platform/graphics/chromium/OpaqueRectTrackingContentLayerDelegate.cpp:

(WebCore::OpaqueRectTrackingContentLayerDelegate::paintContents):

  • platform/graphics/chromium/OpaqueRectTrackingContentLayerDelegate.h:

(OpaqueRectTrackingContentLayerDelegate):

Source/WebKit/chromium:

  • src/WebContentLayerImpl.cpp:

(WebKit::WebContentLayerImpl::paintContents):

  • tests/CCOcclusionTrackerTest.cpp:

(WebKitTests::CCOcclusionTrackerTestMainThreadTypes::createLayer):

  • tests/OpaqueRectTrackingContentLayerDelegateTest.cpp:

(WebCore):
(WebCore::OpaqueRectTrackingContentLayerDelegateTest::OpaqueRectTrackingContentLayerDelegateTest):
(WebCore::OpaqueRectTrackingContentLayerDelegateTest::canvasRect):
(WebCore::TEST_F):

  • tests/WebLayerTest.cpp:
4:35 PM Changeset in webkit [120642] by enrica@apple.com
  • 2 edits in trunk/LayoutTests

Skipping dictation tests on WK2.

Unreviewed.

  • platform/mac-wk2/Skipped:
4:30 PM Changeset in webkit [120641] by commit-queue@webkit.org
  • 13 edits in trunk

Add from-image to css3-images image-resolution
https://bugs.webkit.org/show_bug.cgi?id=85451

Patch by David Barr <davidbarr@chromium.org> on 2012-06-18
Reviewed by Tony Chang.

Source/WebCore:

The css3-images module is at candidate recommendation.
http://www.w3.org/TR/2012/CR-css3-images-20120417/#the-image-resolution

No new tests; extended fast/css/image-resolution/image-resolution.html

  • css/CSSParser.cpp: Accept from-image identifier in image-resolution property.

(WebCore::CSSParser::parseImageResolution): Map CSSValueFromImage to identifier value from cssValuePool.

  • css/CSSValueKeywords.in: Add from-image.
  • css/StyleBuilder.cpp: Extend ApplyPropertyImageResolution to apply RenderStyle::imageResolutionSource.

(WebCore::ApplyPropertyImageResolution::applyInheritValue): Apply RenderStyle::imageResolutionSource.
(WebCore::ApplyPropertyImageResolution::applyInitialValue): Apply RenderStyle::imageResolutionSource.
(WebCore::ApplyPropertyImageResolution::applyValue): Map CSSValueFromImage to ImageResolutionFromImage.

  • rendering/RenderImage.cpp: Extend conditions for recalculation of intrinsic size.

(WebCore::RenderImage::styleDidChange): Update intrinsic size if RenderStyle::imageResolutionSource() has changed.

  • rendering/style/RenderStyle.cpp: Include StyleRareInheritedData::m_imageResolutionSource in style diff.

(WebCore::RenderStyle::diff): Map change in StyleRareInheritedData::m_imageResolutionSource to StyleDifferenceLayout.

  • rendering/style/RenderStyle.h: Add RenderStyle::imageResolutionSource, RenderStyle::setImageResolutionSource, RenderStyle::initialImageResolutionSource.
  • rendering/style/RenderStyleConstants.h: Add enum ImageResolutionSource.
  • rendering/style/StyleRareInheritedData.cpp: Add StyleRareInheritedData::m_imageResolutionSource.

(WebCore::StyleRareInheritedData::StyleRareInheritedData): Add m_imageResolutionSource to default and copy constructor.
(WebCore::StyleRareInheritedData::operator==): Include m_imageResolutionSource in comparison.

  • rendering/style/StyleRareInheritedData.h: Add StyleRareInheritedData::m_imageResolutionSource.

(StyleRareInheritedData): Add 1-bit field StyleRareInheritedData::m_imageResolutionSource.

LayoutTests:

  • fast/css/image-resolution/image-resolution-expected.txt:
  • fast/css/image-resolution/image-resolution.html:
4:16 PM Changeset in webkit [120640] by jamesr@google.com
  • 4 edits in trunk/Source/WebCore

[chromium] REGRESSION(r120346) ContentLayerPainter should clear rect to be painted
https://bugs.webkit.org/show_bug.cgi?id=89376

Reviewed by Adrienne Walker.

In r120346, a refactor moved a clearRect()/clip() call from ContentLayerPainter - which all
ContentLayerChromiums use - out to OpaqueRectTrackingContentLayerDelegate - which all GraphicsLayerChromium
backed layers use, but which callers using WebContentLayer directly do not use. This meant that the canvas
wasn't cleared/clipped as WebContentLayerClient.h claimed it would be. This moves the clear and the clip
into CanvasLayerTextureUpdater which all paths use, including layers that don't use the ContentLayerPainter.

  • platform/graphics/chromium/ContentLayerChromium.cpp:

(WebCore::ContentLayerPainter::paint):

  • platform/graphics/chromium/OpaqueRectTrackingContentLayerDelegate.cpp:

(WebCore::OpaqueRectTrackingContentLayerDelegate::paintContents):

4:14 PM Changeset in webkit [120639] by igor.o@sisa.samsung.com
  • 12 edits
    8 deletes in trunk

Unreviewed, rolling out r120119.
http://trac.webkit.org/changeset/120119

Multiple regressions on ClusterFuzz

  • page/animation/AnimationBase.cpp:

(WebCore::AnimationBase::updateStateMachine):

  • page/animation/AnimationController.cpp:

(WebCore::AnimationControllerPrivate::updateAnimations):
(WebCore::AnimationControllerPrivate::pauseAnimationAtTime):
(WebCore::AnimationControllerPrivate::pauseTransitionAtTime):
(WebCore::AnimationController::cancelAnimations):
(WebCore::AnimationController::updateAnimations):

  • page/animation/ImplicitAnimation.cpp:

(WebCore::ImplicitAnimation::pauseAnimation):
(WebCore::ImplicitAnimation::sendTransitionEvent):

  • page/animation/KeyframeAnimation.cpp:

(WebCore::KeyframeAnimation::KeyframeAnimation):
(WebCore::KeyframeAnimation::pauseAnimation):
(WebCore::KeyframeAnimation::endAnimation):
(WebCore::KeyframeAnimation::sendAnimationEvent):

  • rendering/RenderBlock.cpp:

(WebCore::RenderBlock::updateFirstLetterStyle):
(WebCore::RenderBlock::createFirstLetterRenderer):

  • rendering/RenderInline.cpp:

(WebCore::RenderInline::clippedOverflowRectForRepaint):

  • rendering/RenderObject.cpp:
  • rendering/RenderObject.h:

(RenderObject):

LayoutTests:

  • animations/first-letter-animation-expected.txt: Removed.
  • animations/first-letter-animation.html: Removed.
  • animations/first-letter-play-state-expected.txt: Removed.
  • animations/first-letter-play-state.html: Removed.
  • platform/chromium/TestExpectations:
  • platform/efl/TestExpectations:
  • transitions/first-letter-color-transition-expected.txt: Removed.
  • transitions/first-letter-color-transition.html: Removed.
  • transitions/first-letter-transition-expected.txt: Removed.
    • transitions/first-letter-transition.html: Removed.
4:09 PM Changeset in webkit [120638] by rniwa@webkit.org
  • 3 edits
    2 adds in trunk

REGRESSION(r118414): some pages on concursolutions.com doesn't show up
https://bugs.webkit.org/show_bug.cgi?id=89399

Reviewed by Adam Barth.

Source/WebCore:

The bug was caused by insertAdjacentHTML's not passing context element to createFragmentForInnerOuterHTML.

Test: fast/html/adjacent-html-context-element.html

  • html/HTMLElement.cpp:

(WebCore::HTMLElement::insertAdjacentHTML):

LayoutTests:

Add a regression test.

  • fast/html/adjacent-html-context-element-expected.txt: Added.
  • fast/html/adjacent-html-context-element.html: Added.
4:03 PM Changeset in webkit [120637] by isherman@chromium.org
  • 2 edits in trunk/Source/WebKit/chromium

[Chromium] Ensure layout has happened before calling into hasNonEmptyBoundingBox()
Also update existing code to use updateLayoutIgnorePendingStylesheets()
https://bugs.webkit.org/show_bug.cgi?id=89258

Reviewed by James Robinson.

  • src/WebNode.cpp:

(WebKit::WebNode::isFocusable): Use updateLayoutIgnorePendingStylesheets()
(WebKit::WebNode::hasNonEmptyBoundingBox): Force layout before checking the bounding box

3:37 PM Changeset in webkit [120636] by zmo@google.com
  • 5 edits in trunk/Source/WebCore

Fix framebuffer-object-attachment.html failures
https://bugs.webkit.org/show_bug.cgi?id=89387

Reviewed by Kenneth Russell.

  • html/canvas/WebGLFramebuffer.cpp: fix detachment behavior with depth/stencil/depth_stencil conflicts

(WebCore::WebGLFramebuffer::setAttachmentForBoundFramebuffer):
(WebCore::WebGLFramebuffer::removeAttachmentFromBoundFramebuffer):

  • html/canvas/WebGLFramebuffer.h:

(WebGLFramebuffer):

  • html/canvas/WebGLRenderingContext.cpp:

(WebCore):
(WebCore::WebGLRenderingContext::framebufferRenderbuffer): move logic to WebGLFramebuffer
(WebCore::WebGLRenderingContext::framebufferTexture2D): Ditto.
(WebCore::WebGLRenderingContext::getParameter): Correct the wrong assumption that it's always checking the drawingbuffer's DEPTH_BITS/STENCIL_BITS

  • html/canvas/WebGLRenderingContext.h:

(WebGLRenderingContext):

3:17 PM Changeset in webkit [120635] by atwilson@chromium.org
  • 1 edit
    1 add in trunk/LayoutTests

Unreviewed expectations update for chromium after r120602.

  • platform/chromium-mac-snowleopard/ietestcenter/css3/bordersbackgrounds/border-radius-clip-002-expected.png: Added.
3:17 PM Changeset in webkit [120634] by commit-queue@webkit.org
  • 8 edits in trunk/Source

[chromium] Ensure that skipping frames during an accelerated animation doesn't cause starvation
https://bugs.webkit.org/show_bug.cgi?id=87766

Patch by Ian Vollick <vollick@chromium.org> on 2012-06-18
Reviewed by James Robinson.

Source/WebCore:

Unit tests:

CCLayerTreeHostTestCheckerboardDoesNotStarveDraws.runMultiThread
CCSchedulerStateMachineTest.TestFailedDrawsWillEventuallyForceADrawAfterTheNextCommit

  • platform/graphics/chromium/cc/CCSchedulerStateMachine.cpp:

(WebCore::CCSchedulerStateMachine::CCSchedulerStateMachine):
(WebCore::CCSchedulerStateMachine::updateState):
(WebCore::CCSchedulerStateMachine::didDrawIfPossibleCompleted):
(WebCore::CCSchedulerStateMachine::setMaximumNumberOfFailedDrawsBeforeDrawIsForced):
(WebCore):

  • platform/graphics/chromium/cc/CCSchedulerStateMachine.h:

(CCSchedulerStateMachine):

Source/WebKit/chromium:

  • tests/CCLayerTreeHostTest.cpp:

(WTF):
(CCLayerTreeHostTestCheckerboardDoesNotStarveDraws):
(WTF::CCLayerTreeHostTestCheckerboardDoesNotStarveDraws::CCLayerTreeHostTestCheckerboardDoesNotStarveDraws):
(WTF::CCLayerTreeHostTestCheckerboardDoesNotStarveDraws::beginTest):
(WTF::CCLayerTreeHostTestCheckerboardDoesNotStarveDraws::afterTest):
(WTF::CCLayerTreeHostTestCheckerboardDoesNotStarveDraws::animateLayers):
(WTF::CCLayerTreeHostTestCheckerboardDoesNotStarveDraws::drawLayersOnCCThread):
(WTF::CCLayerTreeHostTestCheckerboardDoesNotStarveDraws::prepareToDrawOnCCThread):
(WTF::TEST_F):

  • tests/CCSchedulerStateMachineTest.cpp:

(WebCore::TEST):
(WebCore):

  • tests/CCThreadedTest.cpp:

(WebKitTests::MockLayerTreeHostImpl::prepareToDraw):

  • tests/CCThreadedTest.h:

(WebKitTests::TestHooks::prepareToDrawOnCCThread):

3:16 PM Changeset in webkit [120633] by zandobersek@gmail.com
  • 2 edits in trunk/Tools

[garden-o-matic] Add support for the GTK builders
https://bugs.webkit.org/show_bug.cgi?id=89360

Reviewed by Adam Barth.

Add the 'gtk' platform, essentially a copy of the 'apple' platform since
both get their data from build.webkit.org.

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

(.):

3:08 PM Changeset in webkit [120632] by rwlbuis@webkit.org
  • 2 edits in trunk

[BlackBerry] Enable LEGACY_VIEWPORT_ADAPTION
https://bugs.webkit.org/show_bug.cgi?id=89381

Reviewed by Antonio Gomes.

PR 163598

Enable LEGACY_VIEWPORT_ADAPTION.

  • Source/cmake/OptionsBlackBerry.cmake:
3:08 PM Changeset in webkit [120631] by thakis@chromium.org
  • 6 edits in branches/chromium/1132

Merge 120629 - [chromium/mac] Unbreak smooth scrolling.
https://bugs.webkit.org/show_bug.cgi?id=89327

Reviewed by Dimitri Glazkov.

Broken by Sam in r115589 / r115591.

Source/WebCore:

  • platform/mac/ScrollAnimatorMac.mm:

(WebCore::scrollAnimationEnabledForSystem):

Tools:

  • DumpRenderTree/mac/DumpRenderTree.mm:

(resetDefaultsToConsistentValues):

  • TestWebKitAPI/mac/InjectedBundleControllerMac.mm:

(TestWebKitAPI::InjectedBundleController::platformInitialize):

  • WebKitTestRunner/InjectedBundle/mac/InjectedBundleMac.mm:

(WTR::InjectedBundle::platformInitialize):

TBR=thakis@chromium.org
Review URL: https://chromiumcodereview.appspot.com/10572004

3:07 PM Changeset in webkit [120630] by abarth@webkit.org
  • 12 edits
    4 adds
    2 deletes in trunk/LayoutTests

Update baselines after http://trac.webkit.org/changeset/120609.

  • platform/chromium-linux-x86/svg/css/getComputedStyle-basic-expected.txt:
  • platform/chromium-linux/fast/multicol/progression-reverse-expected.png: Added.
  • platform/chromium-linux/svg/css/getComputedStyle-basic-expected.txt:
  • platform/chromium-mac-leopard/fast/css/getComputedStyle: Removed.
  • platform/chromium-mac-leopard/fast/css/getComputedStyle/computed-style-expected.txt: Removed.
  • platform/chromium-mac-leopard/svg/css/getComputedStyle-basic-expected.txt:
  • platform/chromium-mac-snowleopard/fast/multicol/progression-reverse-expected.png: Added.
  • platform/chromium-mac-snowleopard/svg/css/getComputedStyle-basic-expected.txt:
  • platform/chromium-mac/fast/css/getComputedStyle/computed-style-expected.txt: Removed.
  • platform/chromium-mac/fast/css/getComputedStyle/computed-style-without-renderer-expected.txt:
  • platform/chromium-mac/fast/multicol/progression-reverse-expected.png: Added.
  • platform/chromium-mac/svg/css/getComputedStyle-basic-expected.txt:
  • platform/chromium-win-vista/svg/css/getComputedStyle-basic-expected.txt:
  • platform/chromium-win-xp/svg/css/getComputedStyle-basic-expected.txt:
  • platform/chromium-win/fast/css/getComputedStyle/computed-style-expected.txt:
  • platform/chromium-win/fast/css/getComputedStyle/computed-style-without-renderer-expected.txt:
  • platform/chromium-win/fast/multicol/progression-reverse-expected.png: Added.
  • platform/chromium-win/svg/css/getComputedStyle-basic-expected.txt:
2:48 PM Changeset in webkit [120629] by thakis@chromium.org
  • 6 edits in trunk

[chromium/mac] Unbreak smooth scrolling.
https://bugs.webkit.org/show_bug.cgi?id=89327

Reviewed by Dimitri Glazkov.

Broken by Sam in r115589 / r115591.

Source/WebCore:

  • platform/mac/ScrollAnimatorMac.mm:

(WebCore::scrollAnimationEnabledForSystem):

Tools:

  • DumpRenderTree/mac/DumpRenderTree.mm:

(resetDefaultsToConsistentValues):

  • TestWebKitAPI/mac/InjectedBundleControllerMac.mm:

(TestWebKitAPI::InjectedBundleController::platformInitialize):

  • WebKitTestRunner/InjectedBundle/mac/InjectedBundleMac.mm:

(WTR::InjectedBundle::platformInitialize):

2:43 PM Changeset in webkit [120628] by enrica@apple.com
  • 2 edits in trunk/LayoutTests

Updating test results to include flakey cookies tests.

Unreviewed.

  • platform/mac/TestExpectations:
2:38 PM Changeset in webkit [120627] by commit-queue@webkit.org
  • 41 edits in trunk

Unreviewed, rolling out r120541, r120547, and r120548.
http://trac.webkit.org/changeset/120541
http://trac.webkit.org/changeset/120547
http://trac.webkit.org/changeset/120548
https://bugs.webkit.org/show_bug.cgi?id=89383

This approach doesn't work for WebKit2 (Requested by abarth on
#webkit).

Patch by Sheriff Bot <webkit.review.bot@gmail.com> on 2012-06-18

Source/WebCore:

  • testing/InternalSettings.cpp:

(WebCore::InternalSettings::InternalSettings):
(WebCore::InternalSettings::restoreTo):

  • testing/InternalSettings.h:

(InternalSettings):

Tools:

  • DumpRenderTree/LayoutTestController.cpp:

(setBackingScaleFactorCallback):
(LayoutTestController::staticFunctions):

  • DumpRenderTree/LayoutTestController.h:

(LayoutTestController):

  • DumpRenderTree/blackberry/LayoutTestControllerBlackBerry.cpp:

(LayoutTestController::setBackingScaleFactor):

  • DumpRenderTree/efl/LayoutTestControllerEfl.cpp:

(LayoutTestController::setBackingScaleFactor):

  • DumpRenderTree/gtk/LayoutTestControllerGtk.cpp:

(LayoutTestController::setBackingScaleFactor):

  • DumpRenderTree/mac/LayoutTestControllerMac.mm:

(LayoutTestController::setBackingScaleFactor):

  • DumpRenderTree/win/LayoutTestControllerWin.cpp:

(LayoutTestController::setBackingScaleFactor):

  • DumpRenderTree/wx/LayoutTestControllerWx.cpp:

(LayoutTestController::setBackingScaleFactor):

  • WebKitTestRunner/InjectedBundle/Bindings/LayoutTestController.idl:
  • WebKitTestRunner/InjectedBundle/InjectedBundle.cpp:

(WTR::InjectedBundle::didReceiveMessage):
(WTR::InjectedBundle::postSetBackingScaleFactor):
(WTR):

  • WebKitTestRunner/InjectedBundle/InjectedBundle.h:

(InjectedBundle):

  • WebKitTestRunner/InjectedBundle/LayoutTestController.cpp:

(WTR::LayoutTestController::setBackingScaleFactor):
(WTR):
(WTR::LayoutTestController::callSetBackingScaleFactorCallback):

  • WebKitTestRunner/InjectedBundle/LayoutTestController.h:

(LayoutTestController):

  • WebKitTestRunner/TestInvocation.cpp:

(WTR::TestInvocation::didReceiveMessageFromInjectedBundle):

LayoutTests:

  • compositing/tiled-layers-hidpi.html:
  • fast/canvas/2d.backingStorePixelRatio.html:
  • fast/canvas/2d.imageDataHD.html:
  • fast/hidpi/broken-image-icon-hidpi.html:
  • fast/hidpi/broken-image-with-size-hidpi.html:
  • fast/hidpi/clip-text-in-hidpi.html:
  • fast/hidpi/focus-rings.html:
  • fast/hidpi/image-set-as-background.html:
  • fast/hidpi/image-set-background-dynamic.html:
  • fast/hidpi/image-set-background-repeat-without-size.html:
  • fast/hidpi/image-set-background-repeat.html:
  • fast/hidpi/image-set-border-image-comparison.html:
  • fast/hidpi/image-set-border-image-dynamic.html:
  • fast/hidpi/image-set-border-image-simple.html:
  • fast/hidpi/image-set-in-content-dynamic.html:
  • fast/hidpi/image-set-out-of-order.html:
  • fast/hidpi/image-set-simple.html:
  • fast/hidpi/image-set-without-specified-width.html:
  • fast/hidpi/resize-corner-hidpi.html:
  • fast/hidpi/video-controls-in-hidpi.html:
  • platform/mac-wk2/plugins/contents-scale-factor.html:
  • svg/as-image/image-respects-deviceScaleFactor.html:
2:34 PM Changeset in webkit [120626] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WebCore

[BlackBerry] NetworkJob::startNewJobWithRequest() should check the frame pointer
https://bugs.webkit.org/show_bug.cgi?id=89369

Patch by Yong Li <yoli@rim.com> on 2012-06-18
Reviewed by Antonio Gomes.

NetworkJob::startNewJobWithRequest() should stop proceeding if the frame is null,
which means it is a PingLoader job and should just finish now.

  • platform/network/blackberry/NetworkJob.cpp:

(WebCore::NetworkJob::startNewJobWithRequest):

2:08 PM Changeset in webkit [120625] by abarth@webkit.org
  • 2 edits in trunk/LayoutTests

Fix the expected result for this test. This got landed with the wrong expected.txt file.

  • http/tests/security/contentSecurityPolicy/source-list-parsing-malformed-meta-expected.txt:
1:40 PM Changeset in webkit [120624] by mitz@apple.com
  • 3 edits
    2 adds in trunk

Ideographic comma and full-stops are mishandled in linebreak
https://bugs.webkit.org/show_bug.cgi?id=87041

Reviewed by Alexey Proskuryakov.

Source/WebCore:

Test: fast/text/line-breaks-after-ideographic-comma-or-full-stop-2.html

Specifically, line breaks were always allowed after an ideographic comma or full stop, even
before a closing bracket.

  • rendering/break_lines.cpp:

(WebCore::shouldBreakAfter): Removed code that unconditionally allowed line breaks after
ideographic commas and full stops, which was added for <http://webkit.org/b/17411> to work
around an issue in Unicode 5.0. Current line break iterator implementations are based on
newer versions of Unicode, which do not have that issue, so we can use them instead of the
hardcoded approximate rule.

LayoutTests:

  • fast/text/line-breaks-after-ideographic-comma-or-full-stop-2-expected.txt: Added.
  • fast/text/line-breaks-after-ideographic-comma-or-full-stop-2.html: Added.
1:35 PM Changeset in webkit [120623] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WebCore

[TextureMapper] Rename shader variables to match the naming scheme of the newer filter shaders
https://bugs.webkit.org/show_bug.cgi?id=89324

Patch by Martin Robinson <mrobinson@igalia.com> on 2012-06-18
Reviewed by Noam Rosenthal.

Rename shader variables to align them with those in the new filter shaders.

No new tests. This is just a simple refactor.

  • platform/graphics/texmap/TextureMapperShaderManager.cpp:

Rename shader variables so that uniforms are prefixed by "u_", attributes by "a_",
and texture samplers by "s_".

1:23 PM Changeset in webkit [120622] by tonikitoo@webkit.org
  • 3 edits in trunk/Source/WebKit/blackberry

[BlackBerry] Overscroll can get reset while interacting with a page, due to style recalculations and scroll position clamping
https://bugs.webkit.org/show_bug.cgi?id=89371
PR #166982

Reviewed by Adam Treat.
Patch by Antonio Gomes <agomes@rim.com>

While scrolling, the WebKit thread gets intercalated WebPage::setScrollPosition
calls dispatched from the UI thread. These calls are wrapped with
ScrollableArea::setConstrainsScrollingToContentEdge(false|true) calls, which
can clamp the scroll position (removes overscroll) if set to 'true'.

The following situation can happen: user is scrolling, and WebKit
thread is not in the middle of a WebPage::setScrollPosition call
(so setConstrainsScrollingToContentEdge is 'true'), and a relayout happens.
In this scenario, the scroll position will get clamped to 0,0, removing any
possible overscrolling.

See the stack trace below, paying special attention to frame 16:

Patch fixes this issue by tying ScrollableArea::setConstrainsScrollingToContentEdge
to BackingStore::m_isZoomingOrScrolling, when we are sure clamping should not happen.

  • Api/BackingStore.cpp:

(BlackBerry::WebKit::BackingStorePrivate::setScrollingOrZooming):

  • Api/WebPage.cpp:

(BlackBerry::WebKit::WebPage::setScrollPosition):

1:17 PM Changeset in webkit [120621] by loislo@chromium.org
  • 4 edits in trunk/Source/WebCore

Web Inspector: Implement native memory bar diagram
https://bugs.webkit.org/show_bug.cgi?id=89106

Patch by Alexei Filippov <alexeif@chromium.org> on 2012-06-18
Reviewed by Pavel Feldman.

  • inspector/front-end/NativeMemorySnapshotView.js:

(WebInspector.NativeMemoryBarChart):
(WebInspector.NativeMemoryBarChart.prototype._updateStats):
(WebInspector.NativeMemoryBarChart.prototype.willHide):
(WebInspector.NativeMemoryBarChart.prototype.wasShown):
(WebInspector.NativeMemoryBarChart.prototype._updateView):

  • inspector/front-end/ProfileLauncherView.js:

(WebInspector.ProfileLauncherView):

  • inspector/front-end/nativeMemoryProfiler.css:

(.memory-bar-chart-name):
(.memory-bar-chart-bar):
(.memory-bar-chart-size):
(.memory-bar-chart-total):

1:06 PM Changeset in webkit [120620] by dpranke@chromium.org
  • 2 edits in trunk/Tools

REGRESSION(r120546): It made 3 webkitpy tests fail
https://bugs.webkit.org/show_bug.cgi?id=89332

Patch by Zan Dobersek <zandobersek@gmail.com> on 2012-06-18
Reviewed by Adam Barth.

Update the BaselineOptimizer unittest after r120546.

  • Scripts/webkitpy/common/checkout/baselineoptimizer_unittest.py:

(BaselineOptimizerTest.test_no_add_mac_future):
(BaselineOptimizerTest.test_mac_future):
(BaselineOptimizerTest.test_complex_shadowing):

12:59 PM Changeset in webkit [120619] by dpranke@chromium.org
  • 5 edits in trunk/Tools

new-run-webkit-tests output gets confused when logging
https://bugs.webkit.org/show_bug.cgi?id=63793

Reviewed by Adam Barth.

Ship all of the log messages generated in the worker back
to the manager; this allows the messages to be properly
serialized to stderr and for the meter to flush its output correctly.

Note however that this will likely result in messages to be
logged out of order between workers; I'm not sure that there's
an easy way to fix this short of buffering a potentially
unbounded amount of data. It might be better to just log through
stderr unless we're in 'metering' mode, but it's also worth
noting that we already get messages out of order through stderr
on Windows.

  • Scripts/webkitpy/layout_tests/controllers/manager.py:

(Manager.handle_done):
(Manager.handle_finished_test):
(Manager._log_messages):

  • Scripts/webkitpy/layout_tests/controllers/worker.py:

(Worker.init):
(Worker._set_up_logging):
(Worker.run):
(Worker._run_test):
(Worker.cleanup):
(Worker.run_single_test):
(_WorkerLogHandler):
(_WorkerLogHandler.init):
(_WorkerLogHandler.emit):

  • Scripts/webkitpy/layout_tests/views/metered_stream.py:

(MeteredStream.write):
(MeteredStream.writeln):

  • Scripts/webkitpy/layout_tests/views/printing.py:

(Printer.writeln):

12:53 PM Changeset in webkit [120618] by loislo@chromium.org
  • 2 edits
    1 add in trunk/PerformanceTests

Web Inspector: native memory snapshot performance and coverage test implementation.
https://bugs.webkit.org/show_bug.cgi?id=89363

We have to track the performance of MemoryAgent.getProcessMemoryDistribution.
Also this test will work as a burn down chart for Unknown memory metric.

Reviewed by Pavel Feldman.

  • inspector/native-memory-snapshot.html: Added.
  • inspector/performance-test.js:

(initialize_TimeTracker.InspectorTest.runPerformanceTest.Timer):
(initialize_TimeTracker.InspectorTest.runPerformanceTest.Timer.prototype.reportSize):
(initialize_TimeTracker.InspectorTest.runPerformanceTest.Timer.prototype._dump):
(initialize_TimeTracker.InspectorTest.runPerformanceTest):

12:51 PM Changeset in webkit [120617] by commit-queue@webkit.org
  • 3 edits
    2 adds in trunk

Crash in CSPSource::parseSource
https://bugs.webkit.org/show_bug.cgi?id=89353

Patch by Mike West <mkwst@chromium.org> on 2012-06-18
Reviewed by Adam Barth.

Source/WebCore:

Test: http/tests/security/contentSecurityPolicy/source-list-parsing-malformed-meta.html

  • page/ContentSecurityPolicy.cpp:

(WebCore::CSPSourceList::parseSource):

Checking that we're not past the end of the string before deref.

LayoutTests:

  • http/tests/security/contentSecurityPolicy/source-list-parsing-malformed-meta.html: Added.
12:30 PM Changeset in webkit [120616] by robert@webkit.org
  • 3 edits
    20 adds in trunk

CSS 2.1 failure: table-height-algorithm-012 fails
https://bugs.webkit.org/show_bug.cgi?id=85652

Reviewed by Julien Chaffraix.

Source/WebCore:

Test: css2.1/20110323/table-height-algorithm-012.htm

fast/css/vertical-align-baseline-rowspan-001.htm
fast/css/vertical-align-baseline-rowspan-002.htm
fast/css/vertical-align-baseline-rowspan-003.htm
fast/css/vertical-align-baseline-rowspan-004.htm
fast/css/vertical-align-baseline-rowspan-005.htm
fast/css/vertical-align-baseline-rowspan-006.htm
fast/css/vertical-align-baseline-rowspan-007.htm
fast/css/vertical-align-baseline-rowspan-008.htm
fast/css/vertical-align-baseline-rowspan-009.htm

A spanning cell with Vertical-align set to 'baseline' should align the cell's content baseline
(which is the bottom of the first line of text or in-flow content) with the baseline of the first of the rows it spans.

  • rendering/RenderTableSection.cpp:

(WebCore::RenderTableSection::calcRowLogicalHeight): Only calculate the baseline for the first row in a rowspan. This

ensures that any height added to the rowspan from the baseline is only added to the first row in the span rather than
to each of them.

(WebCore::RenderTableSection::layoutRows): Rather than get the baseline for each row in a rowspan (with the result that

only the baseline for the last row actually sticks), calculate it for the first row in the rowspan only. This function is
inefficient when handling rowspans so this doesn't fix the fact that redundant calculations are performed on each subsequent
row in the span - that should be done separately.

LayoutTests:

  • css2.1/20110323/table-height-algorithm-012-expected.html: Added.
  • css2.1/20110323/table-height-algorithm-012.htm: Added.
  • fast/css/vertical-align-baseline-rowspan-001-expected.html: Added.
  • fast/css/vertical-align-baseline-rowspan-001.htm: Added.
  • fast/css/vertical-align-baseline-rowspan-002-expected.html: Added.
  • fast/css/vertical-align-baseline-rowspan-002.htm: Added.
  • fast/css/vertical-align-baseline-rowspan-003-expected.html: Added.
  • fast/css/vertical-align-baseline-rowspan-003.htm: Added.

These are a few variations on table-height-algorithm-012.htm

  • fast/css/vertical-align-baseline-rowspan-004-expected.html: Added.
  • fast/css/vertical-align-baseline-rowspan-004.htm: Added.
  • fast/css/vertical-align-baseline-rowspan-005-expected.html: Added.
  • fast/css/vertical-align-baseline-rowspan-005.htm: Added.
  • fast/css/vertical-align-baseline-rowspan-006-expected.html: Added.
  • fast/css/vertical-align-baseline-rowspan-006.htm: Added.

These three test the calculation of the baseline RenderTableSection::calcRowLogicalHeight,
where we now always use the first row in the span. Without this the 'Filler Text' in the middle
cell aligns beyond the top of the table.

  • fast/css/vertical-align-baseline-rowspan-007-expected.html: Added.
  • fast/css/vertical-align-baseline-rowspan-007.htm: Added.
  • fast/css/vertical-align-baseline-rowspan-008-expected.html: Added.
  • fast/css/vertical-align-baseline-rowspan-008.htm: Added.
  • fast/css/vertical-align-baseline-rowspan-009-expected.html: Added.
  • fast/css/vertical-align-baseline-rowspan-009.htm: Added.

These make sure the alignment used in the first row of the span is correct when things like
the height of the cell and the height of the span are specified.

12:16 PM Changeset in webkit [120615] by hayato@chromium.org
  • 5 edits in trunk

Event dispatcher should use InsertionPoint::hasDistribution instead of InsertinPoint::isActive in re-targeting.
https://bugs.webkit.org/show_bug.cgi?id=89180

Reviewed by Dimitri Glazkov.

Source/WebCore:

If an event is fired on a fallback element within a insertion
point which does not select any nodes, the result of event's
re-targeting will be wrong since it uses
InsertionPoint::isActive() to judge an existence of a lower boundary.
That must use InsertionPoint::hasDistribution().

Test: fast/dom/shadow/shadow-dom-event-dispatching.html

  • dom/EventDispatcher.cpp:

(WebCore::EventDispatcher::ensureEventAncestors):

LayoutTests:

  • fast/dom/shadow/shadow-dom-event-dispatching-expected.txt:
  • fast/dom/shadow/shadow-dom-event-dispatching.html:
12:02 PM Changeset in webkit [120614] by zandobersek@gmail.com
  • 4 edits in trunk/Tools

[garden-o-matic] Builder names without underscores cause incorrect BuildSelector behavior
https://bugs.webkit.org/show_bug.cgi?id=89362

Reviewed by Simon Fraser.

Replace white spaces, braces and dots in the builder name with underscores. This
way the JQuery tabs will be properly matched with the according container.

  • BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/scripts/ui/results.js:
11:52 AM Changeset in webkit [120613] by abarth@webkit.org
  • 9 edits in trunk

Source/WebCore: [chromium] Add iccjpeg and qcms to chromium port

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

Reviewed by Adam Barth.

Covered by existing layout tests which will be rebaselined.

  • WebCore.gyp/WebCore.gyp: Add qcms to the build.
  • platform/image-decoders/ImageDecoder.h: (WebCore::ImageDecoder::qcmsOutputDeviceProfile): Return an sRGB profile. On OSX, return the default RGB profile. Add FIXME to use the user's monitor profile and verify that profile for other platforms.
  • platform/image-decoders/jpeg/JPEGImageDecoder.cpp: (turboSwizzled): For libjpeg-turbo, JCS_EXT_BGRA and JCS_EXT_RGBA are the two known output color spaces for which the decoder uses a data swizzle. (colorSpaceHasAlpha): JPEG's have no alpha in the output color space. For libjpeg-turbo, alpha may be present in the swizzled output color space.

(WebCore::JPEGImageReader::JPEGImageReader):
(WebCore::JPEGImageReader::close):
(WebCore::JPEGImageReader::decode): For QCMSLIB, create the color transform
to use during decoding, and ensure we switch to inputing RGBA data to qcms
even if the desired output data is BGRA: outputScanlines() sends BGRA data
to the frame buffer following color correction if needed.
(JPEGImageReader):
(WebCore::JPEGImageReader::colorTransform): qcms color transform getter.
(WebCore::JPEGImageReader::createColorTransform): Create color transform.
Release the existing transform (if any) and assign to the color transform
created from the color profile data.
(WebCore::JPEGImageDecoder::outputScanlines): Minor style fix. Apply color
transform to each decoded image row.

  • platform/image-decoders/png/PNGImageDecoder.cpp: (WebCore::PNGImageReader::PNGImageReader): (WebCore::PNGImageReader::close): (WebCore::PNGImageReader::currentBufferSize): Move this adjacent to other setters and getters. (WebCore::PNGImageReader::decodingSizeOnly): Ditto. (WebCore::PNGImageReader::setHasAlpha): Ditto. (WebCore::PNGImageReader::hasAlpha): Ditto. (WebCore::PNGImageReader::interlaceBuffer): Ditto. (WebCore::PNGImageReader::createRowBuffer): Creates a temporary row buffer, used when a color transform is applied to the decoded image pixels. (WebCore::PNGImageReader::rowBuffer): Return the temporary row buffer. (WebCore::PNGImageReader::colorTransform): qcms color transform getter. (WebCore::PNGImageReader::createColorTransform): Create color transform. Release the existing transform (if any) and assign to the color transform created from the color profile data. (WebCore::PNGImageDecoder::headerAvailable): For QCMSLIB, create the color transform to use for decoding. Clear m_colorProfile (not used anymore). (WebCore::PNGImageDecoder::rowAvailable): Create temporary row buffer if a color transform is needed for decoding. Apply color transform to each decoded image row.
  • platform/image-decoders/skia/ImageDecoderSkia.cpp: (WebCore::ImageFrame::setColorProfile): Old method of colorProfiles is no longer used. Add a FIXME to remove the old implementation. (WebCore::ImageFrame::setStatus): Remove old color correction code.

Patch by Tony Payne <tpayne@chromium.org> on 2012-06-18

Source/WTF: [chromium] Add iccjpeg and qcms to chromium port.
https://bugs.webkit.org/show_bug.cgi?id=81974

Patch by Tony Payne <tpayne@chromium.org> on 2012-06-18

  • wtf/Platform.h: Add Chromium USE defines for ICCJPEG and QCMSLIB

to each Chromium platform, excluding Android.

Reviewed by Adam Barth.

11:45 AM Changeset in webkit [120612] by zandobersek@gmail.com
  • 17 edits in trunk/LayoutTests

Unreviewed GTK gardening, updating baselines after r120596 and r120609.

  • platform/gtk/fast/css/getComputedStyle/computed-style-expected.txt:
  • platform/gtk/fast/hidpi/video-controls-in-hidpi-expected.txt:
  • platform/gtk/fast/layers/video-layer-expected.txt:
  • platform/gtk/media/audio-controls-rendering-expected.txt:
  • platform/gtk/media/audio-repaint-expected.txt:
  • platform/gtk/media/controls-strict-expected.txt:
  • platform/gtk/media/controls-styling-expected.txt:
  • platform/gtk/media/controls-without-preload-expected.txt:
  • platform/gtk/media/media-controls-clone-expected.txt:
  • platform/gtk/media/media-document-audio-repaint-expected.txt:
  • platform/gtk/media/video-controls-rendering-expected.txt:
  • platform/gtk/media/video-empty-source-expected.txt:
  • platform/gtk/media/video-no-audio-expected.txt:
  • platform/gtk/media/video-volume-slider-expected.txt:
  • platform/gtk/media/video-zoom-controls-expected.txt:
  • platform/gtk/svg/css/getComputedStyle-basic-expected.txt:
11:45 AM Changeset in webkit [120611] by dpranke@chromium.org
  • 4 edits in trunk/Tools

nrwt: metered output doesn't handle C cleanly
https://bugs.webkit.org/show_bug.cgi?id=89249

Reviewed by Tony Chang.

If you ctrl-c a running nrwt (w/o --verbose), then all but
two characters of the last update are erased, and then we print
"interrupted, exiting" as an update, which itself then gets
erased. Fix this so that we flush the meter (making the last
update persistent, so that we print the C for a visual clue),
print the interrupt message, and move on ...

  • Scripts/webkitpy/layout_tests/controllers/manager.py:

(Manager._run_tests):

  • Scripts/webkitpy/layout_tests/views/metered_stream.py:

(MeteredStream.flush):

  • Scripts/webkitpy/layout_tests/views/printing.py:

(Printer._write):
(Printer):
(Printer.flush):

11:37 AM Changeset in webkit [120610] by Csaba Osztrogonác
  • 2 edits in trunk/LayoutTests

[Qt] Unreviewed gardening, skip one more failing test to paint.

  • platform/qt/Skipped:
11:05 AM Changeset in webkit [120609] by mitz@apple.com
  • 20 edits
    3 adds in trunk

Column progression direction cannot be set independently of writing direction
https://bugs.webkit.org/show_bug.cgi?id=89315

Source/WebCore:

Added a column-progression property, which can have the values 'normal' and 'reverse'. When
set to 'reverse', the first column is positioned at the end side of the block, and
subsequent columns go from the in the start direction (in the inline column axis case; in
the block column axis case, the first column coincides with the block and subsequent columns
go before the block).

Reviewed by Darin Adler.

Test: fast/multicol/progression-reverse.html

  • css/CSSComputedStyleDeclaration.cpp:

(WebCore::CSSComputedStyleDeclaration::getPropertyCSSValue): Added handling of the new
property.

  • css/CSSParser.cpp:

(WebCore::CSSParser::parseValue): Added validity check for the new property. The valid
values are the identifiers 'normal' and 'reverse'.

  • css/CSSPrimitiveValueMappings.h:

(WebCore::CSSPrimitiveValue::CSSPrimitiveValue): Added this mapping from ColumnProgression
enum values to the identifiers 'normal and 'reverse'.
(WebCore::CSSPrimitiveValue::operator ColumnProgression): Added this mapping from the
identifiers 'normal' and 'reverse' to ColumnProgression enum values.

  • css/CSSProperty.cpp:

(WebCore::CSSProperty::isInheritedProperty): Added the new property to the set of
non-inherited properties.

  • css/CSSPropertyNames.in: Added -webkit-column-progression.
  • css/StyleBuilder.cpp:

(WebCore::StyleBuilder::StyleBuilder): Added a handler for the new property.

  • rendering/ColumnInfo.h:

(WebCore::ColumnInfo::ColumnInfo): Added an initializer for the m_progressionIsReversed
member variable.
(WebCore::ColumnInfo::progressionIsReversed): Added this accessor.
(WebCore::ColumnInfo::setProgressionIsReversed): Ditto.
(ColumnInfo): Added a m_progressionIsReversed member variable.

  • rendering/RenderBlock.cpp:

(WebCore::RenderBlock::paintColumnRules): Changed to advance in the reverse direction if
needed.
(WebCore::RenderBlock::setDesiredColumnCountAndWidth): Added call to
ColumnInfo::setProgressionIsReversed(), based on the new property.
(WebCore::RenderBlock::columnRectAt): Changed to take the progression direction into
account.

  • rendering/style/RenderStyle.h: Added accessors and initial value for column progression.
  • rendering/style/RenderStyleConstants.h: Added ColumnProgression enum.
  • rendering/style/StyleMultiColData.cpp:

(WebCore::StyleMultiColData::StyleMultiColData): Added initializer for the m_progression
member variable.
(WebCore::StyleMultiColData::operator==): Added equality check for m_progression.

  • rendering/style/StyleMultiColData.h:

(StyleMultiColData): Added m_progression member variable.

LayoutTests:

Reviewed by Darin Adler.

  • fast/css/getComputedStyle/computed-style-expected.txt:
  • fast/css/getComputedStyle/computed-style-without-renderer-expected.txt:
  • fast/multicol/progression-reverse-expected.png: Added.
  • fast/multicol/progression-reverse-expected.txt: Added.
  • fast/multicol/progression-reverse.html: Added.
  • platform/mac/fast/css/getComputedStyle/computed-style-expected.txt:
  • platform/mac/fast/css/getComputedStyle/computed-style-without-renderer-expected.txt:
  • platform/mac/svg/css/getComputedStyle-basic-expected.txt:
  • svg/css/getComputedStyle-basic-expected.txt:
10:51 AM JavaScriptCore edited by Thomas Guy Cornyn
Typo, 3rd paragraph - "JavaScriptCode" (diff)
10:23 AM Changeset in webkit [120608] by commit-queue@webkit.org
  • 4 edits in trunk/Source/WebCore

[TextureMapper] Replace the source transform with a flip uniform
https://bugs.webkit.org/show_bug.cgi?id=89317

Patch by Martin Robinson <mrobinson@igalia.com> on 2012-06-18
Reviewed by Noam Rosenthal.

No new tests. This should not change any observable behavior.

  • platform/graphics/texmap/TextureMapperGL.cpp: Instead of building and passing

the m4src matrix to the shader, just pass a single uniform that says whether or
not the texture should be flipped.

  • platform/graphics/texmap/TextureMapperShaderManager.cpp: Change the shader programs

to deal with a flip variable rather than a source matrix.
(WebCore::TextureMapperShaderProgram::TextureMapperShaderProgram): Instead of getting the location
of the source matrix, get the location of the flip uniform.
(WebCore::TextureMapperShaderProgramSimple::TextureMapperShaderProgramSimple): Ditto.
(WebCore::TextureMapperShaderProgramRectSimple::TextureMapperShaderProgramRectSimple): Ditto.
(WebCore::TextureMapperShaderProgramOpacityAndMask::TextureMapperShaderProgramOpacityAndMask): Ditto.
(WebCore::TextureMapperShaderProgramRectOpacityAndMask::TextureMapperShaderProgramRectOpacityAndMask): Ditto.

  • platform/graphics/texmap/TextureMapperShaderManager.h:

(WebCore::TextureMapperShaderProgram::flipLocation): Added this getter.
(TextureMapperShaderProgram): Rename the source matrix location member to reflect above changes.

10:17 AM Changeset in webkit [120607] by zandobersek@gmail.com
  • 7 edits in trunk

Unreviewed, rolling out r120598.
http://trac.webkit.org/changeset/120598
https://bugs.webkit.org/show_bug.cgi?id=89359

Patch causes crashes on EFL and GTK release builds (Requested
by zdobersek on #webkit).

Patch by Sheriff Bot <webkit.review.bot@gmail.com> on 2012-06-18

Source/WebCore:

  • platform/graphics/Pattern.h:

(Pattern):

  • platform/graphics/cairo/PlatformContextCairo.cpp:

(WebCore::PlatformContextCairo::prepareForFilling):

  • platform/graphics/cairo/PlatformContextCairo.h:

LayoutTests:

  • platform/efl/Skipped:
  • platform/gtk/TestExpectations:
10:15 AM Changeset in webkit [120606] by commit-queue@webkit.org
  • 18 edits in trunk/Source/WebCore

Unreviewed, rolling out r120592.
http://trac.webkit.org/changeset/120592
https://bugs.webkit.org/show_bug.cgi?id=89357

Broke v8 bindings on windows. (Requested by atwilson on
#webkit).

Patch by Sheriff Bot <webkit.review.bot@gmail.com> on 2012-06-18

  • bindings/js/JSJavaScriptCallFrameCustom.cpp:
  • bindings/js/ScriptDebugServer.cpp:
  • bindings/js/ScriptDebugServer.h:

(ScriptDebugServer):

  • bindings/v8/DebuggerScript.js:

(.):

  • bindings/v8/JavaScriptCallFrame.cpp:
  • bindings/v8/JavaScriptCallFrame.h:

(JavaScriptCallFrame):

  • bindings/v8/ScriptDebugServer.cpp:
  • bindings/v8/ScriptDebugServer.h:
  • bindings/v8/ScriptFunctionCall.cpp:

(WebCore::ScriptFunctionCall::call):

  • bindings/v8/custom/V8JavaScriptCallFrameCustom.cpp:
  • inspector/InjectedScript.cpp:
  • inspector/InjectedScript.h:

(InjectedScript):

  • inspector/InjectedScriptSource.js:
  • inspector/Inspector.json:
  • inspector/InspectorDebuggerAgent.cpp:

(WebCore::InspectorDebuggerAgent::setScriptSource):

  • inspector/InspectorDebuggerAgent.h:

(InspectorDebuggerAgent):

  • inspector/JavaScriptCallFrame.idl:
10:11 AM Changeset in webkit [120605] by jchaffraix@webkit.org
  • 2 edits in trunk/Source/WebCore

RenderTableSection wastes vector capacity
https://bugs.webkit.org/show_bug.cgi?id=86561

Reviewed by Antti Koivisto.

Memory optimization, covered by existing tests.

The issue is that addChild calls Vector::grow() which can over-allocate our row grid (especially
for small tables as we allocate at least 16 entries). As we don't know if addChild is called
in a loop (e.g. during parsing), we postpone the shrinkToFit call at layout time.

(WebCore::RenderTableSection::layout):
Added a shrinkToFit call with some explanation as to why it's good trade-off to do it now.

10:04 AM Changeset in webkit [120604] by commit-queue@webkit.org
  • 18 edits in trunk/LayoutTests

[EFL] Rebaseline media tests after r120596
https://bugs.webkit.org/show_bug.cgi?id=89352

Unreviewed EFL gardening. Rebaseline a few media tests
due to r120596.

Patch by Christophe Dumez <Christophe Dumez> on 2012-06-18

  • platform/efl/fast/layers/video-layer-expected.txt:
  • platform/efl/media/audio-controls-rendering-expected.txt:
  • platform/efl/media/audio-repaint-expected.png:
  • platform/efl/media/audio-repaint-expected.txt:
  • platform/efl/media/controls-after-reload-expected.txt:
  • platform/efl/media/controls-strict-expected.txt:
  • platform/efl/media/controls-styling-expected.txt:
  • platform/efl/media/controls-without-preload-expected.txt:
  • platform/efl/media/media-controls-clone-expected.png:
  • platform/efl/media/media-controls-clone-expected.txt:
  • platform/efl/media/media-document-audio-repaint-expected.txt:
  • platform/efl/media/video-controls-rendering-expected.png:
  • platform/efl/media/video-controls-rendering-expected.txt:
  • platform/efl/media/video-display-toggle-expected.txt:
  • platform/efl/media/video-empty-source-expected.txt:
  • platform/efl/media/video-zoom-controls-expected.png:
  • platform/efl/media/video-zoom-controls-expected.txt:
9:55 AM Changeset in webkit [120603] by wangxianzhu@chromium.org
  • 6 edits
    1 move
    1 delete in trunk

[Chromium] Change back "linuxish" to "linux" and include WebFontRendering.cpp on Android
https://bugs.webkit.org/show_bug.cgi?id=89228

Reviewed by Tony Chang.

Source/WebKit/chromium:

  • WebKit.gyp:
  • public/linux/WebFontRendering.h:

(WebKit):
(WebFontRendering):

  • public/linuxish/WebFontRendering.h: Removed.
  • src/linux/WebFontRendering.cpp: Renamed from Source/WebKit/chromium/src/linuxish/WebFontRendering.cpp.

(WebKit):
(WebKit::WebFontRendering::setHinting):
(WebKit::WebFontRendering::setAutoHint):
(WebKit::WebFontRendering::setUseBitmaps):
(WebKit::WebFontRendering::setAntiAlias):
(WebKit::WebFontRendering::setSubpixelRendering):
(WebKit::WebFontRendering::setSubpixelPositioning):
(WebKit::WebFontRendering::setLCDOrder):
(WebKit::WebFontRendering::setLCDOrientation):

Tools:

  • DumpRenderTree/chromium/LayoutTestController.cpp:
  • DumpRenderTree/chromium/TestShellAndroid.cpp:
9:40 AM Changeset in webkit [120602] by commit-queue@webkit.org
  • 5 edits
    3 deletes in trunk

webkit fails IETC border-radius-clip-002
https://bugs.webkit.org/show_bug.cgi?id=83205

Source/WebCore:

Added addRoundedRectClip instead of just clip when clipToBorderRadius
is true and bgLayer->clip() == ContentFillBox.

Patch by Takashi Sakamoto <tasak@google.com> on 2012-06-18
Reviewed by Simon Fraser.

This change is covered by the existing test: ietestcenter/css3/bordersbackgrounds/border-radius-clip-002.
However expected images should be updated. I updated chromium-linux
and chromium-mac's expected images and removed chromium-win,
chromium-mac-leopard and chromium-mac-snowleopard's expected images.

  • rendering/RenderBoxModelObject.cpp:

(WebCore::RenderBoxModelObject::paintFillLayerExtended):
As the code for "clipping to content boxes as necessary" doesn't
consider border-radius or not, added the code for checking
clipToBorderRadius or not, and if so, modified to use
getBackgroundRoundedRect for obtaining rounded rect and to use
addRoundedRectClip by using the obtained rounded rect.

LayoutTests:

Patch by Takashi Sakamoto <tasak@google.com> on 2012-06-18
Reviewed by Simon Fraser.

  • platform/chromium-linux/ietestcenter/css3/bordersbackgrounds/border-radius-clip-002-expected.png:
  • platform/chromium-mac/ietestcenter/css3/bordersbackgrounds/border-radius-clip-002-expected.png:
  • platform/chromium-mac-leopard/ietestcenter/css3/bordersbackgrounds/border-radius-clip-002-expected.png: Removed.
  • platform/chromium-mac-snowleopard/ietestcenter/css3/bordersbackgrounds/border-radius-clip-002-expected.png: Removed.
  • platform/chromium-win/ietestcenter/css3/bordersbackgrounds/border-radius-clip-002-expected.png: Removed.
9:33 AM Changeset in webkit [120601] by commit-queue@webkit.org
  • 3 edits
    4 adds in trunk

Source/WebCore: Allow updateFixedElementsAfterScrolling() to run during layout.
https://bugs.webkit.org/show_bug.cgi?id=84965

This is needed because HistoryController restores the previous scroll
position during layout (because we don't want an ugly flash of wrongly-
scrolled content).

Patch by Iain Merrick <husky@google.com> on 2012-06-18
Reviewed by Simon Fraser.

Test: compositing/fixed-position-scroll-offset-history-restore.html

  • page/FrameView.cpp:

(WebCore::FrameView::updateFixedElementsAfterScrolling):

LayoutTests: Allow updateFixedElementsAfterScrolling() to run during layout.
https://bugs.webkit.org/show_bug.cgi?id=84965

This is needed because HistoryController restores the previous scroll
position during layout (because we don't want an ugly flash of wrongly-
scrolled content).

Patch by Iain Merrick <husky@google.com> on 2012-06-18
Reviewed by Simon Fraser.

  • compositing/fixed-position-scroll-offset-history-restore-expected.txt: Added.
  • compositing/fixed-position-scroll-offset-history-restore.html: Added.
  • compositing/resources/fixed-position-scroll-offset-history-restore-2.html: Added.
  • platform/chromium-linux/compositing/fixed-position-scroll-offset-history-restore-expected.png: Added.
9:04 AM Changeset in webkit [120600] by apavlov@chromium.org
  • 3 edits
    3 adds in trunk

CSSOM wrappers are not built for rules in the per-document and page group user stylesheets
https://bugs.webkit.org/show_bug.cgi?id=89221

Reviewed by Antti Koivisto.

Source/WebCore:

The issue manifested itself by not displaying injected page group stylesheet data in the Styles pane.

Test: inspector/styles/inject-stylesheet.html

  • css/StyleResolver.cpp:

(WebCore::StyleResolver::collectMatchingRulesForList):

LayoutTests:

  • inspector/styles/inject-stylesheet-expected.txt: Added.
  • inspector/styles/inject-stylesheet.html: Added.
  • inspector/styles/resources/inject-stylesheet-iframe-data.html: Added.
8:52 AM Changeset in webkit [120599] by Csaba Osztrogonác
  • 2 edits in trunk/LayoutTests

[Qt] Unreviewed gardening, skip one more failing test. (Less test == less problem. :) )

  • platform/qt/Skipped:
8:41 AM Changeset in webkit [120598] by commit-queue@webkit.org
  • 7 edits in trunk

[cairo] Fix LayoutTests/fast/canvas/patternfill-repeat.html
https://bugs.webkit.org/show_bug.cgi?id=53085

Patch by Dominik Röttsches <dominik.rottsches@intel.com> on 2012-06-18
Reviewed by Martin Robinson.

Source/WebCore:

Clipping previously unlimited vertical and horizontal pattern repeats with
a clip rectangle similar to the Qt Graphics Context.

No new tests, this patch fixes
canvas/philip/tests/2d.pattern.paint.repeat* tests.

  • platform/graphics/Pattern.h:

(WebCore::Pattern::getPatternSpaceTransform): Adding constant getter method to be able to map to pattern space externally, needed by PlatformContextCairo::clipForPatternFilling.

  • platform/graphics/cairo/PlatformContextCairo.cpp:

(WebCore::PlatformContextCairo::prepareForFilling):
(WebCore::PlatformContextCairo::clipForPatternFilling): Clip pattern repeats if needed.

  • platform/graphics/cairo/PlatformContextCairo.h:

LayoutTests:

canvas/philip/tests/2d.pattern.paint.repeat* tests now passing.

  • platform/efl/Skipped:
  • platform/gtk/TestExpectations:
8:06 AM Changeset in webkit [120597] by rjkroege@chromium.org
  • 2 edits in trunk/Source/WebCore

Touch events with default actions should be handled.
https://bugs.webkit.org/show_bug.cgi?id=89304.

Reviewed by Kent Tamura.

Covered by existing tests and tests in blocked bug
https://bugs.webkit.org/show_bug.cgi?id=88807

  • page/EventHandler.cpp:

(WebCore::EventHandler::handleTouchEvent): Return true that the touch event
was handled for either preventDefault or defaultHandled.

8:04 AM Changeset in webkit [120596] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WebCore

REGRESSION introduced in bug 88623: new shadowPseudoId elements need to get default styling.
https://bugs.webkit.org/show_bug.cgi?id=89280

Patch by Silvia Pfeiffer <silviapf@chromium.org> on 2012-06-18
Reviewed by Eric Carlson.

No new tests; existing tests found this bug.

  • css/html.css:

(input[type="range"]::-webkit-slider-container, input[type="range"]::-webkit-media-slider-container):
Add -webkit-media-slider-container to the default styling.
(input[type="range"]::-webkit-slider-thumb, input[type="range"]::-webkit-media-slider-thumb):
Add -webkit-media-slider-thumb to the default styling.

7:45 AM Changeset in webkit [120595] by zandobersek@gmail.com
  • 2 edits in trunk/LayoutTests

Unreviewed GTK gardening, remove duplicates from the GTK WK2
Skipped list that are already present in the generic WK2 Skipped
list or in GTK's TestExpectations.

  • platform/gtk-wk2/Skipped:
7:27 AM Changeset in webkit [120594] by abecsi@webkit.org
  • 3 edits
    2 adds in trunk/Source/WebKit/qt

[Qt] Update QML plugins to Qt 5 style
https://bugs.webkit.org/show_bug.cgi?id=89027

Reviewed by Csaba Osztrogonác.

Since support for Qt 4 style plugins was removed from Qt 5
we need to update our plugins to the new style to be able
to build with a more recent Qt 5 hash.
Add empty plugin description json files for now.

  • declarative/experimental/plugin.cpp:
  • declarative/experimental/plugin.json: Added.
  • declarative/plugin.cpp:

(WebKitQmlPlugin):

  • declarative/plugin.json: Added.
7:10 AM Changeset in webkit [120593] by kinuko@chromium.org
  • 21 edits
    10 adds in trunk

Support File/DirectoryEntry access for <input type=file> if FileSystem API is enabled
https://bugs.webkit.org/show_bug.cgi?id=88293

Reviewed by Tony Chang.

Source/WebCore:

Proposal email on whatwg: http://lists.whatwg.org/pipermail/whatwg-whatwg.org/2011-November/033814.html
Proposal draft: http://wiki.whatwg.org/wiki/DragAndDropEntries

Test: fast/forms/file/input-file-entries.html

  • Modules/filesystem/HTMLInputElementFileSystem.cpp: Added.
  • Modules/filesystem/HTMLInputElementFileSystem.h: Added.
  • Modules/filesystem/HTMLInputElementFileSystem.idl: Added.
  • WebCore.gypi:
  • html/FileInputType.cpp:

(WebCore::FileInputType::receiveDroppedFiles): Retrieves dropped file
system Id if the given DragData has it.
(WebCore::FileInputType::droppedFileSystemId): Added. This returns
dropped file system Id retrieved in receiveDroppedFiles, or an empty
string if the platform does not support isolated file system for
dropped files.

  • html/FileInputType.h:
  • html/HTMLInputElement.cpp:

(WebCore::HTMLInputElement::droppedFileSystemId): Added.

  • html/HTMLInputElement.h:
  • html/InputType.cpp:

(WebCore::InputType::droppedFileSystemId): Added.

  • html/InputType.h:
  • platform/DragData.cpp:
  • platform/DragData.h:
  • platform/chromium/DragDataChromium.cpp:

(WebCore::DragData::droppedFileSystemId): Added.

LayoutTests:

  • fast/forms/file/input-file-entries-expected.txt: Added.
  • fast/forms/file/input-file-entries.html: Added.
  • fast/forms/resources/dir1/UTF8-2.txt: Added.
  • fast/forms/resources/dir1/UTF8.txt: Added.
  • fast/forms/resources/dir2/green.jpg: Added.
  • platform/efl/Skipped:
  • platform/efl/TestExpectations:
  • platform/gtk/TestExpectations:
  • platform/mac/Skipped:
  • platform/qt/Skipped:
  • platform/win/Skipped:
  • platform/wincairo/Skipped:
  • platform/wk2/Skipped:
6:54 AM Changeset in webkit [120592] by commit-queue@webkit.org
  • 18 edits in trunk/Source/WebCore

Web Inspector: Support 'Restart frame' in protocol and backend
https://bugs.webkit.org/show_bug.cgi?id=89147

Patch by Peter Rybin <peter.rybin@gmail.com> on 2012-06-18
Reviewed by Yury Semikhatsky.

Inspector.json got new command in Debugger domain.
A call path from InspectorDebuggerAgent to V8 Debug.LiveEdit.RestartFrame API
is built.
JSC implementation is a stub with fix me (40300).

  • bindings/js/JSJavaScriptCallFrameCustom.cpp:

(WebCore::JSJavaScriptCallFrame::restart):
(WebCore):

  • bindings/js/JavaScriptCallFrame.h:

(JavaScriptCallFrame):

  • bindings/js/ScriptDebugServer.cpp:

(WebCore):
(WebCore::ScriptDebugServer::updateCallStack):

  • bindings/js/ScriptDebugServer.h:

(ScriptDebugServer):

  • bindings/v8/DebuggerScript.js:

(.):

  • bindings/v8/JavaScriptCallFrame.cpp:

(WebCore::JavaScriptCallFrame::restart):
(WebCore):

  • bindings/v8/JavaScriptCallFrame.h:

(JavaScriptCallFrame):

  • bindings/v8/ScriptDebugServer.cpp:

(WebCore):
(WebCore::ScriptDebugServer::updateCallStack):

  • bindings/v8/ScriptDebugServer.h:
  • bindings/v8/ScriptFunctionCall.cpp:

(WebCore::ScriptFunctionCall::call):

  • bindings/v8/custom/V8JavaScriptCallFrameCustom.cpp:

(WebCore::V8JavaScriptCallFrame::restartCallback):
(WebCore):

  • inspector/InjectedScript.cpp:

(WebCore::InjectedScript::restartFrame):
(WebCore):

  • inspector/InjectedScript.h:

(InjectedScript):

  • inspector/InjectedScriptSource.js:

(.):

  • inspector/Inspector.json:
  • inspector/InspectorDebuggerAgent.cpp:

(WebCore::InspectorDebuggerAgent::restartFrame):

  • inspector/InspectorDebuggerAgent.h:

(InspectorDebuggerAgent):

  • inspector/JavaScriptCallFrame.idl:
6:46 AM Changeset in webkit [120591] by commit-queue@webkit.org
  • 10 edits in trunk

Unreviewed, rolling out r120485 and r120582.
http://trac.webkit.org/changeset/120485
http://trac.webkit.org/changeset/120582
https://bugs.webkit.org/show_bug.cgi?id=89345

chromeos tree breakage (Requested by morrita on #webkit).

Patch by Sheriff Bot <webkit.review.bot@gmail.com> on 2012-06-18

Source/WebCore:

  • WebCore.gyp/WebCore.gyp:
  • platform/image-decoders/ImageDecoder.h:

(ImageFrame):
(ImageDecoder):

  • platform/image-decoders/jpeg/JPEGImageDecoder.cpp:

(turboSwizzled):
(WebCore::JPEGImageReader::JPEGImageReader):
(WebCore::JPEGImageReader::close):
(WebCore::JPEGImageReader::decode):
(JPEGImageReader):
(WebCore::JPEGImageDecoder::outputScanlines):

  • platform/image-decoders/png/PNGImageDecoder.cpp:

(WebCore::PNGImageReader::PNGImageReader):
(WebCore::PNGImageReader::close):
(WebCore::PNGImageReader::currentBufferSize):
(PNGImageReader):
(WebCore::PNGImageReader::decodingSizeOnly):
(WebCore::PNGImageReader::interlaceBuffer):
(WebCore::PNGImageReader::hasAlpha):
(WebCore::PNGImageReader::setHasAlpha):
(WebCore::PNGImageDecoder::headerAvailable):
(WebCore::PNGImageDecoder::rowAvailable):

  • platform/image-decoders/skia/ImageDecoderSkia.cpp:

(WebCore):
(WebCore::resolveColorSpace):
(WebCore::createColorSpace):
(WebCore::ImageFrame::setColorProfile):
(WebCore::ImageFrame::setStatus):

Source/WTF:

  • wtf/Platform.h:

LayoutTests:

  • platform/chromium/TestExpectations:
5:53 AM Changeset in webkit [120590] by Csaba Osztrogonác
  • 2 edits in trunk/LayoutTests

[Qt] Gardening after r120311. It made inspector/extensions/extensions-panel.html flakey.
https://bugs.webkit.org/show_bug.cgi?id=89333

Patch by János Badics <János Badics> on 2012-06-18
Reviewed by Csaba Osztrogonác.

  • platform/qt/Skipped:
4:49 AM Companies and Organizations that have contributed to WebKit edited by dominik.rottsches@intel.com
(diff)
4:40 AM Changeset in webkit [120589] by loislo@chromium.org
  • 6 edits in trunk/Source/WebCore

Web Inspector: Add heap profiler owned memory to the pie chart
https://bugs.webkit.org/show_bug.cgi?id=89203

Heap profiler may consume considerable amount of memory
which is currently falls into Unknown category.
Give it a separate sector on the pie chart.

Patch by Alexei Filippov <alexeif@chromium.org> on 2012-06-18
Reviewed by Yury Semikhatsky.

  • bindings/js/ScriptProfiler.h:

(WebCore::ScriptProfiler::profilerSnapshotsSize):

  • bindings/v8/ScriptProfiler.cpp:

(WebCore::ScriptProfiler::profilerSnapshotsSize):
(WebCore):

  • bindings/v8/ScriptProfiler.h:

(ScriptProfiler):

  • inspector/InspectorMemoryAgent.cpp:

(MemoryBlockName):
(WebCore):
(WebCore::inspectorData):
(WebCore::InspectorMemoryAgent::getProcessMemoryDistribution):

  • inspector/front-end/NativeMemorySnapshotView.js:

(WebInspector.MemoryBlockViewProperties._initialize):

4:13 AM Changeset in webkit [120588] by mario@webkit.org
  • 7 edits in trunk

[GTK] Get rid of DumpRenderTreeSupportGtk::{in|de}crementAccessibilityValue
https://bugs.webkit.org/show_bug.cgi?id=89226

Reviewed by Martin Robinson.

Source/WebCore:

Implement webkitAccessibleValueGetMinimumIncrement() properly,
doing something similar to what is done in AccessibilityObject's
increment() and decrement() functions.

Also, added missing bits to webkitAccessibleValueSetCurrentValue
to make sure we check the new value against range limits.

  • accessibility/gtk/WebKitAccessibleInterfaceValue.cpp:

(webkitAccessibleValueGetCurrentValue):
(webkitAccessibleValueGetMaximumValue):
(webkitAccessibleValueGetMinimumValue):
(webkitAccessibleValueSetCurrentValue):
(webkitAccessibleValueGetMinimumIncrement):

Source/WebKit/gtk:

Remove unnecesary functions incrementAccessibilityValue() and
decrementAccessibilityValue() from DumpRenderTreeSupportGtk.

  • WebCoreSupport/DumpRenderTreeSupportGtk.cpp:
  • WebCoreSupport/DumpRenderTreeSupportGtk.h:

(DumpRenderTreeSupportGtk):

Tools:

Implement increment() and decrement() functions in term of the AtkValue interface,
instead of using DumpRenderTreeSupportGtk helper class.

  • DumpRenderTree/gtk/AccessibilityUIElementGtk.cpp:

(AccessibilityUIElement::intValue):
(AccessibilityUIElement::minValue):
(AccessibilityUIElement::maxValue):
(alterCurrentValue):
(AccessibilityUIElement::increment):
(AccessibilityUIElement::decrement):

4:11 AM Changeset in webkit [120587] by kinuko@chromium.org
  • 3 edits
    6 deletes in trunk/LayoutTests

fast/files/xhr-response-blob should use other file/blob type than javascript
https://bugs.webkit.org/show_bug.cgi?id=89338

Reviewed by Kentaro Hara.

Changing the test to use the most basic text/plain type.
Also removing platform-specific expectations.

  • fast/files/script-tests/xhr-response-blob.js: Updated.
  • fast/files/xhr-response-blob-expected.txt: Updated.
  • platform/chromium-linux/fast/files/xhr-response-blob-expected.txt: Removed.
  • platform/chromium-mac-snowleopard/fast/files/xhr-response-blob-expected.txt: Removed.
  • platform/chromium-mac/fast/files/xhr-response-blob-expected.txt: Removed.
  • platform/chromium/fast/files/xhr-response-blob-expected.txt: Removed.
  • platform/qt/fast/files/xhr-response-blob-expected.txt: Removed.
4:09 AM Changeset in webkit [120586] by hbono@chromium.org
  • 3 edits in trunk/Source/WebKit/chromium

[chromium] Spellchecker should show suggestions only when right-clicking a misspelled word.
https://bugs.webkit.org/show_bug.cgi?id=89331

Reviewed by Hajime Morita.

When a selection includes two or more misspelled words, it is not so easy to
select one word from them and to show its suggestions. To avoid this problem,
this change shows suggestions only when the selection is collapsed. For this
case, we can use Range::setStart and setEnd to convert a DocumentMarker to a
Range.

  • src/ContextMenuClientImpl.cpp:

(WebKit::ContextMenuClientImpl::getCustomMenuFromDefaultItems): Added a check that the selection is collapsed.

  • src/WebFrameImpl.cpp:

(WebKit::WebFrameImpl::replaceMisspelledRange): Added a check that the selection is collapsed. Also, use Range::setStart() and Range::setEnd() to get the marker range to emulate the behavior of DocumentMarker::markersInRange().

3:56 AM Changeset in webkit [120585] by kinuko@chromium.org
  • 2 edits in trunk/Source

[chromium] Cleanup: remove PlatformSupport::createAsyncFileSystem as it's no longer used
https://bugs.webkit.org/show_bug.cgi?id=89083

Reviewed by Kentaro Hara.

Source/WebCore:

No new tests as this has no functional changes.

  • platform/chromium/PlatformSupport.h:

(PlatformSupport):

Source/WebKit/chromium:

  • src/PlatformSupport.cpp:
3:43 AM Changeset in webkit [120584] by Csaba Osztrogonác
  • 2 edits
    1 add in trunk/LayoutTests

Unreviewed gardening after r120574.

  • fast/files/xhr-response-blob-expected.txt:
  • platform/qt/fast/files/xhr-response-blob-expected.txt: Added.
3:30 AM Changeset in webkit [120583] by commit-queue@webkit.org
  • 9 edits in trunk

Fix syntax of FileReader::readAsDataURL result
https://bugs.webkit.org/show_bug.cgi?id=89330

When readAsDataURL generates data URL without media type, it used to omit semicolon before
"base64" keyword. But this semicolor is required in this case, according to RFC2397.

Patch by Taiju Tsuiki <tzik@chromium.org> on 2012-06-18
Reviewed by Kent Tamura.

Source/WebCore:

Test: fast/files/read-blob-async.html

fast/files/workers/worker-read-blob-async.html:
fast/files/workers/worker-read-blob-sync.html:

  • fileapi/FileReaderLoader.cpp:

(WebCore::FileReaderLoader::convertToDataURL):

LayoutTests:

  • fast/files/read-blob-async-expected.txt:
  • fast/files/workers/worker-read-blob-async-expected.txt:
  • fast/files/workers/worker-read-blob-sync-expected.txt:
  • platform/chromium/fast/files/read-blob-async-expected.txt:
  • platform/chromium/fast/files/workers/worker-read-blob-async-expected.txt:
  • platform/chromium/fast/files/workers/worker-read-blob-sync-expected.txt:
3:24 AM Changeset in webkit [120582] by morrita@google.com
  • 2 edits in trunk/Source/WebCore

Unreviewed chormium crash fix. This should be revisited.

  • platform/image-decoders/ImageDecoder.h:

(WebCore::ImageDecoder::qcmsOutputDeviceProfile):

3:03 AM Changeset in webkit [120581] by commit-queue@webkit.org
  • 4 edits
    1 add in trunk

[EFL] fast/files/xhr-response-blob.html is crashing
https://bugs.webkit.org/show_bug.cgi?id=89334

Patch by Christophe Dumez <Christophe Dumez> on 2012-06-18
Reviewed by Kentaro Hara.

Source/WebCore:

Call wrap() instead of createWrapper() in JSBlob's toJS()
to avoid hitting the !getCachedWrapper ASSERT in
createWrapper() because we fail to use the cached wrapper.

Test: fast/files/xhr-response-blob.html

  • bindings/js/JSBlobCustom.cpp:

(WebCore::toJS):

LayoutTests:

Unskip fast/files/xhr-response-blob.html now that it
no longer crashes.

  • platform/efl/TestExpectations:
  • platform/efl/fast/files/xhr-response-blob-expected.txt: Added.
2:31 AM Changeset in webkit [120580] by hans@chromium.org
  • 4 edits in trunk/LayoutTests

[Chromium] fast/events/constructors/speech-recognition-event-constructor.html is failing
https://bugs.webkit.org/show_bug.cgi?id=88856

Reviewed by Adam Barth.

This fixes and re-enables the layout test that I forgot to update in r120053.

  • fast/events/constructors/speech-recognition-event-constructor-expected.txt:
  • fast/events/constructors/speech-recognition-event-constructor.html:
  • platform/chromium/TestExpectations:
2:07 AM Changeset in webkit [120579] by loislo@chromium.org
  • 2 edits in trunk/Source/WebCore

Web Inspector: rename Unknown to Other in Native heap snapshot.
https://bugs.webkit.org/show_bug.cgi?id=89306

Cosmetic change.

Reviewed by Yury Semikhatsky.

  • inspector/front-end/NativeMemorySnapshotView.js:

(WebInspector.NativeMemoryProfileType.prototype.buttonClicked.didReceiveMemorySnapshot):
(WebInspector.NativeMemoryProfileType.prototype.buttonClicked):
(WebInspector.MemoryBlockViewProperties._initialize):

1:24 AM Changeset in webkit [120578] by commit-queue@webkit.org
  • 2 edits in trunk/LayoutTests

[EFL] skip fast/files/xhr-response-blob.html
https://bugs.webkit.org/show_bug.cgi?id=89335

Unreviewed EFL gardening. Skip fast/files/xhr-response-blob.html
because it is crashing.

Patch by Christophe Dumez <Christophe Dumez> on 2012-06-18

  • platform/efl/TestExpectations:
1:12 AM Changeset in webkit [120577] by sergio@webkit.org
  • 2 edits in trunk/Source/WebCore

TextureMapperLayer: invisible layers do not let their children to be painted
https://bugs.webkit.org/show_bug.cgi?id=88786

Reviewed by Noam Rosenthal.

Do not early discard recursively painting children layers if the
current layer is not visible.

Changes already covered by compositing tests. This fixes both
compositing/backface-visibility/backface-visibility-3d.html and
compositing/backface-visibility/backface-visibility-non3d.html.

  • platform/graphics/texmap/TextureMapperLayer.cpp:

(WebCore::TextureMapperLayer::computeTransformsRecursive):
(WebCore::TextureMapperLayer::paintSelf):
(WebCore::TextureMapperLayer::isVisible):

12:25 AM Changeset in webkit [120576] by commit-queue@webkit.org
  • 4 edits
    9 adds in trunk/LayoutTests

[EFL] Rebaseline needed after r117339
https://bugs.webkit.org/show_bug.cgi?id=89328

Unreviewed EFL gardening. Rebaseline a few tests after r117339.

Patch by Christophe Dumez <Christophe Dumez> on 2012-06-18

  • platform/efl/TestExpectations:
  • platform/efl/editing/pasteboard/innerText-inline-table-expected.png: Added.
  • platform/efl/editing/pasteboard/innerText-inline-table-expected.txt: Added.
  • platform/efl/editing/selection/iframe-expected.png: Added.
  • platform/efl/editing/selection/iframe-expected.txt: Added.
  • platform/efl/editing/selection/inline-table-expected.png: Added.
  • platform/efl/editing/selection/inline-table-expected.txt: Added.
  • platform/efl/editing/selection/table-caret-3-expected.png: Added.
  • platform/efl/editing/selection/table-caret-3-expected.txt: Added.
  • platform/efl/tables/mozilla/bugs/bug2479-2-expected.png:
  • platform/efl/tables/mozilla/bugs/bug2479-2-expected.txt:
12:12 AM Changeset in webkit [120575] by commit-queue@webkit.org
  • 22 edits
    4 adds in trunk/LayoutTests

[EFL] Rebaseline needed after r120477
https://bugs.webkit.org/show_bug.cgi?id=89325

Unreviewed EFL gardening. Rebaseline a few tests after r120477.

Patch by Christophe Dumez <Christophe Dumez> on 2012-06-18

  • platform/efl/TestExpectations:
  • platform/efl/editing/deleting/delete-3800834-fix-expected.png: Added.
  • platform/efl/editing/deleting/delete-3800834-fix-expected.txt: Added.
  • platform/efl/fast/dynamic/011-expected.png:
  • platform/efl/fast/dynamic/011-expected.txt:
  • platform/efl/fast/forms/formmove3-expected.png:
  • platform/efl/fast/forms/formmove3-expected.txt:
  • platform/efl/fast/forms/preserveFormDuringResidualStyle-expected.png:
  • platform/efl/fast/forms/preserveFormDuringResidualStyle-expected.txt:
  • platform/efl/fast/invalid/001-expected.png:
  • platform/efl/fast/invalid/001-expected.txt:
  • platform/efl/fast/invalid/003-expected.png:
  • platform/efl/fast/invalid/003-expected.txt:
  • platform/efl/fast/invalid/004-expected.png:
  • platform/efl/fast/invalid/004-expected.txt:
  • platform/efl/fast/invalid/007-expected.png:
  • platform/efl/fast/invalid/007-expected.txt:
  • platform/efl/fast/invalid/019-expected.png:
  • platform/efl/fast/invalid/019-expected.txt:
  • platform/efl/fast/multicol/span/clone-anonymous-block-non-inline-child-crash-expected.txt: Added.
  • platform/efl/tables/mozilla/bugs/bug647-expected.png:
  • platform/efl/tables/mozilla/bugs/bug647-expected.txt:
  • platform/efl/tables/mozilla/other/wa_table_tr_align-expected.png:
  • platform/efl/tables/mozilla/other/wa_table_tr_align-expected.txt:
12:06 AM Changeset in webkit [120574] by kinuko@chromium.org
  • 19 edits in trunk

XHR_RESPONSE_BLOB is not enabled on some platforms
https://bugs.webkit.org/show_bug.cgi?id=88859

Reviewed by Adam Barth.

Source/WebCore:

Removing XHR_RESPONSE_BLOB since the current implementation has
no platform-specific part.

Test: http/tests/xmlhttprequest/blob-response-size.html

  • bindings/generic/RuntimeEnabledFeatures.cpp:

(WebCore):

  • bindings/generic/RuntimeEnabledFeatures.h:

(RuntimeEnabledFeatures):

  • bindings/js/JSXMLHttpRequestCustom.cpp:

(WebCore::JSXMLHttpRequest::visitChildren):
(WebCore::JSXMLHttpRequest::response):

  • bindings/v8/custom/V8XMLHttpRequestCustom.cpp:

(WebCore::V8XMLHttpRequest::responseAccessorGetter):

  • platform/chromium/PlatformSupport.h:

(PlatformSupport):

  • xml/XMLHttpRequest.cpp:

(WebCore::XMLHttpRequest::responseBlob):
(WebCore::XMLHttpRequest::setResponseType):
(WebCore::XMLHttpRequest::clearResponseBuffers):
(WebCore::XMLHttpRequest::didReceiveData):

  • xml/XMLHttpRequest.h:

(XMLHttpRequest):

  • xml/XMLHttpRequest.idl:

Source/WebKit/chromium:

Removing XHR_RESPONSE_BLOB since the current implementation has
no platform-specific part.

  • src/PlatformSupport.cpp:
  • src/WebRuntimeFeatures.cpp:

(WebKit::WebRuntimeFeatures::enableXHRResponseBlob):
(WebKit::WebRuntimeFeatures::isXHRResponseBlobEnabled):

LayoutTests:

Removing xhr.Blob related test from Skipped files.

  • platform/efl/Skipped:
  • platform/gtk/TestExpectations:
  • platform/mac/Skipped:
  • platform/qt/Skipped:
  • platform/win/Skipped:
  • platform/wincairo/Skipped:

Jun 17, 2012:

11:43 PM Changeset in webkit [120573] by commit-queue@webkit.org
  • 2 edits
    1 add in trunk/LayoutTests

[EFL] Add EFL baseline for fast/parser/nested-fragment-parser-crash.html
https://bugs.webkit.org/show_bug.cgi?id=89329

Unreviewed EFL gardening. Use the same expectation for
fast/parser/nested-fragment-parser-crash.html as mac.
The test is passing since it does not crash.

Patch by Christophe Dumez <Christophe Dumez> on 2012-06-17

  • platform/efl/TestExpectations:
  • platform/efl/fast/parser/nested-fragment-parser-crash-expected.txt: Added.
11:36 PM Changeset in webkit [120572] by commit-queue@webkit.org
  • 4 edits in trunk/Source/WebCore

[TextureMapper] Reduce boilerplate code in the TextureMapperShaderManager
https://bugs.webkit.org/show_bug.cgi?id=89293

Patch by Martin Robinson <mrobinson@igalia.com> on 2012-06-17
Reviewed by Noam Rosenthal.

Refactor TextureMapperShaderManager to reduce boilerplate code, by:

  1. Making fragment and shader source arguments to the superclass constructor.
  2. Removing virtual methods (which should not be called in constructors) with constructor implementations. This exchanges a small amount of code duplication for a lot less boilerplate.
  3. Standardizing on the fooLocation name instead of fooVariable.
  4. Removing the ::prepare() series of methods for non-filter shaders. It's betters to have all the uniform and attribute binding in the same place.

No new tests. This is just a refactor and shouldn't change any functionality.

  • platform/graphics/texmap/TextureMapperGL.cpp:

Handle binding the mask texture and opacity at both of the previous ::prepare callsites. Also
rename "shaderInfo" to "program".

  • platform/graphics/texmap/TextureMapperShaderManager.cpp: Perform the refactoring described above.
  • platform/graphics/texmap/TextureMapperShaderManager.h: Ditto.
11:32 PM Changeset in webkit [120571] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WebCore

Fix Framebuffer restrictions
https://bugs.webkit.org/show_bug.cgi?id=89320

Patch by Gregg Tavares <gman@google.com> on 2012-06-17
Reviewed by Kenneth Russell.

No new tests, just fixing old ones

  • html/canvas/WebGLFramebuffer.cpp:

(WebCore::WebGLFramebuffer::checkStatus):

10:34 PM UsingGitWithWebKit edited by yosin@chromium.org
Add "Copying file aka svn copy" section (diff)
9:57 PM Changeset in webkit [120570] by morrita@google.com
  • 2 edits in trunk/LayoutTests

Unreviewed, marking some tests as failures..

  • platform/chromium/TestExpectations:
9:57 PM Changeset in webkit [120569] by yosin@chromium.org
  • 20 edits in trunk/Source/WebCore

[Forms] Move search field related code to RenderSearchField from RenderTextControlSingleLine
https://bugs.webkit.org/show_bug.cgi?id=88980

Reviewed by Kent Tamura.

This patch moves search field related methods in RenderTextControlSingleLine
to new class RenderSearchField and changes related classes to use
RenderSearchField.

No new tests. This patch doesn't change behavior.

  • CMakeLists.txt: Added new file RenderSearchField.cpp
  • GNUmakefile.list.am: Added new file RenderSearchField.cpp and RenderSearchField.h
  • Target.pri: ditto
  • WebCore.gypi: ditto
  • WebCore.vcproj/WebCore.vcproj: ditto
  • WebCore.xcodeproj/project.pbxproj: ditto
  • html/HTMLInputElement.cpp:

(WebCore::HTMLInputElement::subtreeHasChanged): Call RenderSearchField::updateCancelButtonVisibility
(WebCore::HTMLInputElement::addSearchResult): Changed to call InputType::addSearchResult.

  • html/InputType.cpp:

(WebCore::InputType::addSearchResult): Added.

  • html/InputType.h:
  • html/SearchInputType.cpp:

(WebCore::SearchInputType::addSearchResult): Moved from HTMLInputElement.
(WebCore::SearchInputType::createRenderer): Added to create RenderSearchField.
(WebCore::SearchInputType::searchEventsShouldBeDispatched): Moved from HTMLInputElement.
(WebCore::SearchInputType::subtreeHasChanged) Moved search related code from HTMLInputElement.

  • html/SearchInputType.h:
  • html/shadow/TextControlInnerElements.cpp:

(WebCore::SearchFieldResultsButtonElement::defaultEventHandler): Use RenderSearchField instead of RenderTextControlSingleLine.

  • loader/FormSubmission.cpp:

(WebCore::FormSubmission::create): Got rid isSearchField() guard for addSearchResult().

  • rendering/RenderSearchField.cpp:

(WebCore::RenderSearchField::RenderSearchField): Renamed to RenderSearchField.
(WebCore::RenderSearchField::~RenderSearchField): ditto
(WebCore::RenderSearchField::resultsButtonElement): ditto
(WebCore::RenderSearchField::cancelButtonElement): ditto
(WebCore::RenderSearchField::addSearchResult): ditto
(WebCore::RenderSearchField::showPopup): ditto
(WebCore::RenderSearchField::hidePopup): ditto
(WebCore::RenderSearchField::computeControlHeight): Added.
(WebCore::RenderSearchField::updateFromElement): Moved search field related code from RenderTextControlSingleLine.
(WebCore::RenderSearchField::updateCancelButtonVisibility): Renamed to RenderSearchField.
(WebCore::RenderSearchField::visibilityForCancelButton): ditto
(WebCore::RenderSearchField::autosaveName): ditto
(WebCore::RenderSearchField::valueChanged): ditto
(WebCore::RenderSearchField::itemText): ditto
(WebCore::RenderSearchField::itemLabel): ditto
(WebCore::RenderSearchField::itemIcon): ditto
(WebCore::RenderSearchField::itemIsEnabled): ditto
(WebCore::RenderSearchField::itemStyle): ditto
(WebCore::RenderSearchField::menuStyle): ditto
(WebCore::RenderSearchField::clientInsetLeft): ditto
(WebCore::RenderSearchField::clientInsetRight): ditto
(WebCore::RenderSearchField::clientPaddingLeft): ditto
(WebCore::RenderSearchField::clientPaddingRight): ditto
(WebCore::RenderSearchField::listSize): ditto
(WebCore::RenderSearchField::selectedIndex): ditto
(WebCore::RenderSearchField::popupDidHide): ditto
(WebCore::RenderSearchField::itemIsSeparator): ditto
(WebCore::RenderSearchField::itemIsLabel): ditto
(WebCore::RenderSearchField::itemIsSelected): ditto
(WebCore::RenderSearchField::setTextFromItem): ditto
(WebCore::RenderSearchField::fontSelector): ditto
(WebCore::RenderSearchField::hostWindow): ditto
(WebCore::RenderSearchField::createScrollbar): ditto
(WebCore::RenderSearchField::computeHeightLimit): Added.
(WebCore::RenderSearchField::centerContainerIfNeeded): Added.

  • rendering/RenderSearchField.h:

(WebCore::toRenderSearchField): Added.

  • rendering/RenderTextControlSingleLine.cpp:

(WebCore::RenderTextControlSingleLine::RenderTextControlSingleLine): Remove search field related code.
(WebCore::RenderTextControlSingleLine::~RenderTextControlSingleLine): ditto.
(WebCore::RenderTextControlSingleLine::computeHeightLimit): Added.
(WebCore::RenderTextControlSingleLine::layout): Use new methods for removing isSearchField.
(WebCore::RenderTextControlSingleLine::computeControlHeight): Added.
(WebCore::RenderTextControlSingleLine::updateFromElement): Remove search field related code.

  • rendering/RenderTextControlSingleLine.h:

(RenderTextControlSingleLine):
(WebCore::RenderTextControlSingleLine::centerContainerIfNeeded): Added.
(WebCore::RenderTextControlSingleLine::containerElement): Exposed for RenderSearchField.
(WebCore::RenderTextControlSingleLine::innerBlockElement): Exposed for RenderSearchField.

  • rendering/RenderingAllInOne.cpp: Added RenderSearchField.cpp
9:35 PM Changeset in webkit [120568] by ggaren@apple.com
  • 6 edits in trunk/Source/JavaScriptCore

GC copy phase spends needless cycles zero-filling blocks
https://bugs.webkit.org/show_bug.cgi?id=89128

Reviewed by Gavin Barraclough.

We only need to zero-fill when we're allocating memory that might not
get fully initialized before GC.

  • heap/CopiedBlock.h:

(JSC::CopiedBlock::createNoZeroFill):
(JSC::CopiedBlock::create): Added a way to create without zero-filling.
This is our optimization.

(JSC::CopiedBlock::zeroFillToEnd):
(JSC::CopiedBlock::CopiedBlock): Split zero-filling out from creation,
so we can sometimes create without zero-filling.

  • heap/CopiedSpace.cpp:

(JSC::CopiedSpace::init):
(JSC::CopiedSpace::tryAllocateSlowCase):
(JSC::CopiedSpace::doneCopying): Renamed addNewBlock to allocateBlock()
to clarify that the new block is always newly-allocated.

(JSC::CopiedSpace::doneFillingBlock): Make sure to zero-fill to the end
of a block that might be used in the future for allocation. (Most of the
time, this is a no-op, since we've already filled the block completely.)

(JSC::CopiedSpace::getFreshBlock): Removed this function because the
abstraction of "allocation must succeed" is no longer useful.

  • heap/CopiedSpace.h: Updated declarations to match.
  • heap/CopiedSpaceInlineMethods.h:

(JSC::CopiedSpace::allocateBlockForCopyingPhase): New function, which
knows that it can skip zero-filling.

Added tighter scoping to our lock, to improve parallelism.

(JSC::CopiedSpace::allocateBlock): Folded getFreshBlock functionality
into this function, for simplicity.

  • heap/MarkStack.cpp:

(JSC::SlotVisitor::startCopying):
(JSC::SlotVisitor::allocateNewSpace): Use our new zero-fill-free helper
function for great good.

9:18 PM Changeset in webkit [120567] by yosin@chromium.org
  • 1 edit
    2 copies in trunk/Source/WebCore

[Forms] Copy RenderTextControlSingleLine.{cpp,h} to RenderSearchFiled.{cpp,h}
https://bugs.webkit.org/show_bug.cgi?id=89155

This patch copies RenderTextControlSingleLine.cpp and .h into RenderSearchField.cpp
and .h with just "cp" code. New files aren't compiled until bug 88980.

No new tests. This patch doesn't change behavior.

  • rendering/RenderSearchField.cpp: Added. (WebCore): (WebCore::RenderTextControlInnerBlock::positionForPoint): (WebCore::RenderTextControlSingleLine::RenderTextControlSingleLine): (WebCore::RenderTextControlSingleLine::~RenderTextControlSingleLine): (WebCore::RenderTextControlSingleLine::containerElement): (WebCore::RenderTextControlSingleLine::innerBlockElement): (WebCore::RenderTextControlSingleLine::innerSpinButtonElement): (WebCore::RenderTextControlSingleLine::resultsButtonElement): (WebCore::RenderTextControlSingleLine::cancelButtonElement): (WebCore::RenderTextControlSingleLine::textBaseStyle): (WebCore::RenderTextControlSingleLine::addSearchResult): (WebCore::RenderTextControlSingleLine::showPopup): (WebCore::RenderTextControlSingleLine::hidePopup): (WebCore::RenderTextControlSingleLine::paint): (WebCore::RenderTextControlSingleLine::layout): (WebCore::RenderTextControlSingleLine::nodeAtPoint): (WebCore::RenderTextControlSingleLine::styleDidChange): (WebCore::RenderTextControlSingleLine::capsLockStateMayHaveChanged): (WebCore::RenderTextControlSingleLine::hasControlClip): (WebCore::RenderTextControlSingleLine::controlClipRect): (WebCore::RenderTextControlSingleLine::getAvgCharWidth): (WebCore::RenderTextControlSingleLine::preferredContentWidth): (WebCore::RenderTextControlSingleLine::computeControlHeight): (WebCore::RenderTextControlSingleLine::updateFromElement): (WebCore::RenderTextControlSingleLine::createInnerTextStyle): (WebCore::RenderTextControlSingleLine::createInnerBlockStyle): (WebCore::RenderTextControlSingleLine::updateCancelButtonVisibility): (WebCore::RenderTextControlSingleLine::visibilityForCancelButton): (WebCore::RenderTextControlSingleLine::textShouldBeTruncated): (WebCore::RenderTextControlSingleLine::autosaveName): (WebCore::RenderTextControlSingleLine::valueChanged): (WebCore::RenderTextControlSingleLine::itemText): (WebCore::RenderTextControlSingleLine::itemLabel): (WebCore::RenderTextControlSingleLine::itemIcon): (WebCore::RenderTextControlSingleLine::itemIsEnabled): (WebCore::RenderTextControlSingleLine::itemStyle): (WebCore::RenderTextControlSingleLine::menuStyle): (WebCore::RenderTextControlSingleLine::clientInsetLeft): (WebCore::RenderTextControlSingleLine::clientInsetRight): (WebCore::RenderTextControlSingleLine::clientPaddingLeft): (WebCore::RenderTextControlSingleLine::clientPaddingRight): (WebCore::RenderTextControlSingleLine::listSize): (WebCore::RenderTextControlSingleLine::selectedIndex): (WebCore::RenderTextControlSingleLine::popupDidHide): (WebCore::RenderTextControlSingleLine::itemIsSeparator): (WebCore::RenderTextControlSingleLine::itemIsLabel): (WebCore::RenderTextControlSingleLine::itemIsSelected): (WebCore::RenderTextControlSingleLine::setTextFromItem): (WebCore::RenderTextControlSingleLine::fontSelector): (WebCore::RenderTextControlSingleLine::hostWindow): (WebCore::RenderTextControlSingleLine::autoscroll): (WebCore::RenderTextControlSingleLine::scrollWidth): (WebCore::RenderTextControlSingleLine::scrollHeight): (WebCore::RenderTextControlSingleLine::scrollLeft): (WebCore::RenderTextControlSingleLine::scrollTop): (WebCore::RenderTextControlSingleLine::setScrollLeft): (WebCore::RenderTextControlSingleLine::setScrollTop): (WebCore::RenderTextControlSingleLine::scroll): (WebCore::RenderTextControlSingleLine::logicalScroll): (WebCore::RenderTextControlSingleLine::createScrollbar): (WebCore::RenderTextControlSingleLine::inputElement):
  • rendering/RenderSearchField.h: Added. (WebCore): (RenderTextControlSingleLine): (WebCore::RenderTextControlSingleLine::popupIsVisible): (WebCore::RenderTextControlSingleLine::isTextField): (WebCore::toRenderTextControlSingleLine): (RenderTextControlInnerBlock): (WebCore::RenderTextControlInnerBlock::RenderTextControlInnerBlock): (WebCore::RenderTextControlInnerBlock::hasLineIfEmpty):
9:11 PM Changeset in webkit [120566] by yosin@chromium.org
  • 1 edit
    2 deletes in trunk/Source/WebCore

Unreviewed, rolling out r120565.
http://trac.webkit.org/changeset/120565
https://bugs.webkit.org/show_bug.cgi?id=89155

Bad commit message

  • rendering/RenderSearchField.cpp: Removed.
  • rendering/RenderSearchField.h: Removed.
9:08 PM Changeset in webkit [120565] by yosin@chromium.org
  • 1 edit
    2 copies in trunk/Source/WebCore

Try 1

7:43 PM Changeset in webkit [120564] by morrita@google.com
  • 4 edits in trunk/Source

Unreviewed build fix.

Source/WebCore:

  • platform/graphics/chromium/cc/CCLayerTreeHostImpl.cpp:

(WebCore::CCLayerTreeHostImpl::removePassesWithCachedTextures):

Source/WebKit/chromium:

  • tests/CCLayerTreeHostImplTest.cpp:
7:42 PM Changeset in webkit [120563] by Philippe Normand
  • 5 edits in trunk/Source/WebCore

[GStreamer] optimize ::naturalSize()
https://bugs.webkit.org/show_bug.cgi?id=89314

Reviewed by Martin Robinson.

Keep track of the video-sink sinkpad in the MediaPlayerPrivate and
adapt the webkitGstGetPadCaps API accordingly. The benefit of this
change is to reduce the number of calls to
gst_element_get_static_pad() and the corresponding gst_pad_unref().

  • platform/graphics/gstreamer/GStreamerVersioning.cpp:

(webkitGstGetPadCaps):

  • platform/graphics/gstreamer/GStreamerVersioning.h:
  • platform/graphics/gstreamer/MediaPlayerPrivateGStreamer.cpp:

(WebCore::MediaPlayerPrivateGStreamer::naturalSize):

  • platform/graphics/gstreamer/MediaPlayerPrivateGStreamer.h:

(MediaPlayerPrivateGStreamer):

7:35 PM Changeset in webkit [120562] by Philippe Normand
  • 2 edits in trunk

Unreviewed, GTK 64-bit build fix after r120551.

  • Source/autotools/symbols.filter:
7:28 PM Changeset in webkit [120561] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WebCore

Don't mark main frame layer opaque if the frame view is transparent
https://bugs.webkit.org/show_bug.cgi?id=88234

Patch by Arvid Nilsson <anilsson@rim.com> on 2012-06-17
Reviewed by Simon Fraser.

  • rendering/RenderLayerBacking.cpp:

(WebCore::RenderLayerBacking::createPrimaryGraphicsLayer):

7:22 PM Changeset in webkit [120560] by jchaffraix@webkit.org
  • 1 edit
    2 adds in trunk/LayoutTests

colspan in table-layout:fixed does not add up widths properly
https://bugs.webkit.org/show_bug.cgi?id=20213

Reviewed by Abhishek Arya.

The bug has been fixed (likely by bug 14858) so landing the test case as fixed table layout with cell spanning several
<col>'s with fixed widths is not properly covered.

  • fast/table/fixed-table-layout/fixed-layout-column-colspan-wrong-size-expected.txt: Added.
  • fast/table/fixed-table-layout/fixed-layout-column-colspan-wrong-size.html: Added.
7:12 PM Changeset in webkit [120559] by pdr@google.com
  • 3 edits
    2 adds in trunk

Prevent crash in SVGDocumentExtensions::removeAllElementReferencesForTarget.
https://bugs.webkit.org/show_bug.cgi?id=88144

Reviewed by Abhishek Arya.

Source/WebCore:

When iterating over referencing elements to rebuild after a reference change in
SVGDocumentExtensions::removeAllElementReferencesForTarget, we can
modify the underlying toBeNotified vector, invalidating it. This change checks
that a vector element is valid before rebuilding, preventing a crash.

Some definitions from SVGDocumentExtensions that may put this patch in context:

An example of a "referenced elements" is a <path>.
An example of a "referencing element" is a <textPath href='some_path_id'>.
m_elementDependencies is a map from referenced elements (e.g., paths) to
a set of referencing elements (e.g., textPaths).

The check that the vector element is valid relies on checking if the referencing
element is in m_elementDependencies. This check is allowed because in the
destructor of SVGTextPathElement (and SVGFeImageElement),
removeAllTargetReferencesForElement() is called, removing the referencing element
from m_elementDependencies.

Simply checking if the referencing element is anywhere in m_elementDependencies
is enough to show it is valid, but that requires iterating over all referenced
elements to see if the given referencing element is present. This change
only checks if the textPath is still in the elements referencing the
path being removed, and only removes the referenced element from
m_elementDependencies after forcing the referencing elements to be rebuilt.

Test: svg/text/textpath-reference-crash.html

  • svg/SVGDocumentExtensions.cpp:

(WebCore::SVGDocumentExtensions::removeAllElementReferencesForTarget):

LayoutTests:

  • svg/text/textpath-reference-crash-expected.txt: Added.
  • svg/text/textpath-reference-crash.html: Added.
7:09 PM Changeset in webkit [120558] by yosin@chromium.org
  • 1 edit
    2 deletes in trunk/Source/WebCore

Unreviewed, rolling out r120390.
http://trac.webkit.org/changeset/120390
https://bugs.webkit.org/show_bug.cgi?id=89155

Failed to copy merge history to RenderSerachField.{cpp,h}

  • rendering/RenderSearchField.cpp: Removed.
  • rendering/RenderSearchField.h: Removed.
7:06 PM Changeset in webkit [120557] by yosin@chromium.org
  • 20 edits in trunk/Source/WebCore

Unreviewed, rolling out r120432.
http://trac.webkit.org/changeset/120432
https://bugs.webkit.org/show_bug.cgi?id=88980

Failed to copy merge history to RenderSerachField.{cpp,h}

  • CMakeLists.txt:
  • GNUmakefile.list.am:
  • Target.pri:
  • WebCore.gypi:
  • WebCore.vcproj/WebCore.vcproj:
  • WebCore.xcodeproj/project.pbxproj:
  • html/HTMLInputElement.cpp:

(WebCore::HTMLInputElement::subtreeHasChanged):
(WebCore::HTMLInputElement::searchEventsShouldBeDispatched):
(WebCore):
(WebCore::HTMLInputElement::addSearchResult):

  • html/HTMLInputElement.h:

(HTMLInputElement):

  • html/InputType.cpp:
  • html/InputType.h:

(InputType):

  • html/SearchInputType.cpp:
  • html/SearchInputType.h:

(SearchInputType):

  • html/shadow/TextControlInnerElements.cpp:

(WebCore::SearchFieldResultsButtonElement::defaultEventHandler):

  • loader/FormSubmission.cpp:

(WebCore::FormSubmission::create):

  • rendering/RenderSearchField.cpp:

(WebCore::RenderTextControlInnerBlock::positionForPoint):
(WebCore):
(WebCore::RenderTextControlSingleLine::RenderTextControlSingleLine):
(WebCore::RenderTextControlSingleLine::~RenderTextControlSingleLine):
(WebCore::RenderTextControlSingleLine::containerElement):
(WebCore::RenderTextControlSingleLine::innerBlockElement):
(WebCore::RenderTextControlSingleLine::innerSpinButtonElement):
(WebCore::RenderTextControlSingleLine::resultsButtonElement):
(WebCore::RenderTextControlSingleLine::cancelButtonElement):
(WebCore::RenderTextControlSingleLine::textBaseStyle):
(WebCore::RenderTextControlSingleLine::addSearchResult):
(WebCore::RenderTextControlSingleLine::showPopup):
(WebCore::RenderTextControlSingleLine::hidePopup):
(WebCore::RenderTextControlSingleLine::paint):
(WebCore::RenderTextControlSingleLine::layout):
(WebCore::RenderTextControlSingleLine::nodeAtPoint):
(WebCore::RenderTextControlSingleLine::styleDidChange):
(WebCore::RenderTextControlSingleLine::capsLockStateMayHaveChanged):
(WebCore::RenderTextControlSingleLine::hasControlClip):
(WebCore::RenderTextControlSingleLine::controlClipRect):
(WebCore::RenderTextControlSingleLine::getAvgCharWidth):
(WebCore::RenderTextControlSingleLine::preferredContentWidth):
(WebCore::RenderTextControlSingleLine::computeControlHeight):
(WebCore::RenderTextControlSingleLine::updateFromElement):
(WebCore::RenderTextControlSingleLine::createInnerTextStyle):
(WebCore::RenderTextControlSingleLine::createInnerBlockStyle):
(WebCore::RenderTextControlSingleLine::updateCancelButtonVisibility):
(WebCore::RenderTextControlSingleLine::visibilityForCancelButton):
(WebCore::RenderTextControlSingleLine::textShouldBeTruncated):
(WebCore::RenderTextControlSingleLine::autosaveName):
(WebCore::RenderTextControlSingleLine::valueChanged):
(WebCore::RenderTextControlSingleLine::itemText):
(WebCore::RenderTextControlSingleLine::itemLabel):
(WebCore::RenderTextControlSingleLine::itemIcon):
(WebCore::RenderTextControlSingleLine::itemIsEnabled):
(WebCore::RenderTextControlSingleLine::itemStyle):
(WebCore::RenderTextControlSingleLine::menuStyle):
(WebCore::RenderTextControlSingleLine::clientInsetLeft):
(WebCore::RenderTextControlSingleLine::clientInsetRight):
(WebCore::RenderTextControlSingleLine::clientPaddingLeft):
(WebCore::RenderTextControlSingleLine::clientPaddingRight):
(WebCore::RenderTextControlSingleLine::listSize):
(WebCore::RenderTextControlSingleLine::selectedIndex):
(WebCore::RenderTextControlSingleLine::popupDidHide):
(WebCore::RenderTextControlSingleLine::itemIsSeparator):
(WebCore::RenderTextControlSingleLine::itemIsLabel):
(WebCore::RenderTextControlSingleLine::itemIsSelected):
(WebCore::RenderTextControlSingleLine::setTextFromItem):
(WebCore::RenderTextControlSingleLine::fontSelector):
(WebCore::RenderTextControlSingleLine::hostWindow):
(WebCore::RenderTextControlSingleLine::autoscroll):
(WebCore::RenderTextControlSingleLine::scrollWidth):
(WebCore::RenderTextControlSingleLine::scrollHeight):
(WebCore::RenderTextControlSingleLine::scrollLeft):
(WebCore::RenderTextControlSingleLine::scrollTop):
(WebCore::RenderTextControlSingleLine::setScrollLeft):
(WebCore::RenderTextControlSingleLine::setScrollTop):
(WebCore::RenderTextControlSingleLine::scroll):
(WebCore::RenderTextControlSingleLine::logicalScroll):
(WebCore::RenderTextControlSingleLine::createScrollbar):
(WebCore::RenderTextControlSingleLine::inputElement):

  • rendering/RenderSearchField.h:

(RenderTextControlSingleLine):
(WebCore::RenderTextControlSingleLine::isTextField):
(WebCore::toRenderTextControlSingleLine):
(WebCore):
(RenderTextControlInnerBlock):
(WebCore::RenderTextControlInnerBlock::RenderTextControlInnerBlock):
(WebCore::RenderTextControlInnerBlock::hasLineIfEmpty):

  • rendering/RenderTextControlSingleLine.cpp:

(WebCore::RenderTextControlSingleLine::RenderTextControlSingleLine):
(WebCore::RenderTextControlSingleLine::~RenderTextControlSingleLine):
(WebCore):
(WebCore::RenderTextControlSingleLine::containerElement):
(WebCore::RenderTextControlSingleLine::innerBlockElement):
(WebCore::RenderTextControlSingleLine::resultsButtonElement):
(WebCore::RenderTextControlSingleLine::cancelButtonElement):
(WebCore::RenderTextControlSingleLine::addSearchResult):
(WebCore::RenderTextControlSingleLine::showPopup):
(WebCore::RenderTextControlSingleLine::hidePopup):
(WebCore::RenderTextControlSingleLine::layout):
(WebCore::RenderTextControlSingleLine::computeControlHeight):
(WebCore::RenderTextControlSingleLine::updateFromElement):
(WebCore::RenderTextControlSingleLine::updateCancelButtonVisibility):
(WebCore::RenderTextControlSingleLine::visibilityForCancelButton):
(WebCore::RenderTextControlSingleLine::autosaveName):
(WebCore::RenderTextControlSingleLine::valueChanged):
(WebCore::RenderTextControlSingleLine::itemText):
(WebCore::RenderTextControlSingleLine::itemLabel):
(WebCore::RenderTextControlSingleLine::itemIcon):
(WebCore::RenderTextControlSingleLine::itemIsEnabled):
(WebCore::RenderTextControlSingleLine::itemStyle):
(WebCore::RenderTextControlSingleLine::menuStyle):
(WebCore::RenderTextControlSingleLine::clientInsetLeft):
(WebCore::RenderTextControlSingleLine::clientInsetRight):
(WebCore::RenderTextControlSingleLine::clientPaddingLeft):
(WebCore::RenderTextControlSingleLine::clientPaddingRight):
(WebCore::RenderTextControlSingleLine::listSize):
(WebCore::RenderTextControlSingleLine::selectedIndex):
(WebCore::RenderTextControlSingleLine::popupDidHide):
(WebCore::RenderTextControlSingleLine::itemIsSeparator):
(WebCore::RenderTextControlSingleLine::itemIsLabel):
(WebCore::RenderTextControlSingleLine::itemIsSelected):
(WebCore::RenderTextControlSingleLine::setTextFromItem):
(WebCore::RenderTextControlSingleLine::fontSelector):
(WebCore::RenderTextControlSingleLine::hostWindow):
(WebCore::RenderTextControlSingleLine::createScrollbar):

  • rendering/RenderTextControlSingleLine.h:

(WebCore):
(RenderTextControlSingleLine):
(WebCore::RenderTextControlSingleLine::popupIsVisible):

  • rendering/RenderingAllInOne.cpp:
6:59 PM Changeset in webkit [120556] by fpizlo@apple.com
  • 2 edits in trunk/Source/JavaScriptCore

DFG should attempt to use structure watchpoints for all inlined get_by_id's and put_by_id's
https://bugs.webkit.org/show_bug.cgi?id=89316

Reviewed by Oliver Hunt.

  • dfg/DFGByteCodeParser.cpp:

(JSC::DFG::ByteCodeParser::addStructureTransitionCheck):
(ByteCodeParser):
(JSC::DFG::ByteCodeParser::handleGetById):
(JSC::DFG::ByteCodeParser::parseBlock):

5:20 PM Changeset in webkit [120555] by Simon Fraser
  • 7 edits in trunk/Tools

garden-o-matic 'Results' panel is broken for the Apple platform
https://bugs.webkit.org/show_bug.cgi?id=89310

Reviewed by Adam Barth.

For platforms that don't use accumulated build directories on the server,
carry along buildLocation data with buildInfo, so that we know where to
look for the test results files for a given test.

Renamed historicalResultsSummaryURLs to historicalResultsLocations because
it now returns an array of objects with buildNumber, revision and url info.

Fixed results.fetchResultsURLs() to use this data to find results.

Fixed the results display to not scroll when you click on a test to see the results.
Made the h3s look less ugly

Don't try to show the flakiness dashboard for the Apple platform.

  • BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/scripts/checkout.js:
  • BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/scripts/config.js:
  • BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/scripts/results.js:
  • BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/scripts/ui.js:
  • BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/scripts/ui/results.js:
  • BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/styles/results.css:

(.results-view .top-panel):
(.results-view h3):

4:06 PM Changeset in webkit [120554] by Darin Adler
  • 2 edits in trunk/Source/WebCore

REGRESSION (r111041): Missing element type check in RenderThemeMac::paintMediaFullscreenButton
https://bugs.webkit.org/show_bug.cgi?id=89270

Reviewed by Oliver Hunt.

  • rendering/RenderThemeMac.mm:

(WebCore::RenderThemeMac::paintMediaFullscreenButton): Use the proper idiom for getting
a media control element's type.

2:19 PM Changeset in webkit [120553] by commit-queue@webkit.org
  • 13 edits in trunk/Source

[Chromium] Compositor should avoid drawing quads when cached textures are available and contents unchanged
https://bugs.webkit.org/show_bug.cgi?id=88482

Patch by Zeev Lieber <zlieber@chromium.org> on 2012-06-17
Reviewed by Adrienne Walker.

Source/WebCore:

Post-processing CCRenderPassList after it's been generated and
removing all render surface quads for which there are
cached textures, and whose content didn't change. Added a new
flag to CCLayerImpl to differentiate surface property change and
layer property change.

The changes are covered by new unit tests that check that
the removal algorithm functions propertly. No rendering
behaviour change, so no new layout tests.

  • platform/graphics/chromium/cc/CCDamageTracker.cpp:

(WebCore::layerNeedsToRedrawOntoItsTargetSurface):
(WebCore):
(WebCore::CCDamageTracker::extendDamageForLayer):
(WebCore::CCDamageTracker::extendDamageForRenderSurface):

  • platform/graphics/chromium/cc/CCLayerImpl.cpp:

(WebCore::CCLayerImpl::CCLayerImpl):
(WebCore::CCLayerImpl::layerSurfacePropertyChanged):
(WebCore):
(WebCore::CCLayerImpl::resetAllChangeTrackingForSubtree):
(WebCore::CCLayerImpl::setOpacity):
(WebCore::CCLayerImpl::setTransform):

  • platform/graphics/chromium/cc/CCLayerImpl.h:

(CCLayerImpl):

  • platform/graphics/chromium/cc/CCLayerTreeHostImpl.cpp:

(WebCore::CCLayerTreeHostImpl::calculateRenderPasses):
(WebCore::CCLayerTreeHostImpl::removeRenderPassesRecursive):
(WebCore):
(WebCore::CCLayerTreeHostImpl::removePassesWithCachedTextures):
(WebCore::CCLayerTreeHostImpl::prepareToDraw):

  • platform/graphics/chromium/cc/CCLayerTreeHostImpl.h:

(FrameData):
(CCLayerTreeHostImpl):

  • platform/graphics/chromium/cc/CCRenderPass.h:

(CCRenderPass):

  • platform/graphics/chromium/cc/CCRenderSurface.cpp:

(WebCore::CCRenderSurface::prepareContentsTexture):
(WebCore::CCRenderSurface::hasCachedContentsTexture):
(WebCore):
(WebCore::CCRenderSurface::contentsChanged):

  • platform/graphics/chromium/cc/CCRenderSurface.h:

(CCRenderSurface):

Source/WebKit/chromium:

Added tests to verify quad removal algorithm. Added infrastructure
code to generate custom render passes out of a script encoded as C
string.

  • tests/CCDamageTrackerTest.cpp:

(WebKitTests::TEST_F):

  • tests/CCLayerImplTest.cpp:

(WebCore):
(WebCore::TEST):

  • tests/CCLayerTreeHostImplTest.cpp:
2:06 PM Changeset in webkit [120552] by Patrick Gansterer
  • 2 edits in trunk/Source/WTF

[WIN] Simplify implementation of currentTime()
https://bugs.webkit.org/show_bug.cgi?id=83156

Reviewed by Brent Fulgham.

Use GetSystemTimeAsFileTime() instead of ftime().
This avoids an unneeded call to the MS CRT and
brings the Win/WinCE code closer together.

  • wtf/CurrentTime.cpp:

(WTF::lowResUTCTime):
(WTF::currentTime):

11:36 AM Changeset in webkit [120551] by Philippe Normand
  • 2 edits in trunk

Unreviewed, GTK build fix after r120397.

  • Source/autotools/symbols.filter:
4:15 AM Changeset in webkit [120550] by zandobersek@gmail.com
  • 5 edits in trunk/LayoutTests

Unreviewed GTK gardening, updating baselines after r120541.

  • platform/gtk/fast/hidpi/clip-text-in-hidpi-expected.txt:
  • platform/gtk/fast/hidpi/image-set-background-dynamic-expected.txt:
  • platform/gtk/fast/hidpi/image-set-border-image-dynamic-expected.txt:
  • platform/gtk/fast/hidpi/image-set-in-content-dynamic-expected.txt:
3:16 AM Changeset in webkit [120549] by commit-queue@webkit.org
  • 2 edits in trunk/LayoutTests

[EFL] Skip media/video-seek-past-end-playing.html
https://bugs.webkit.org/show_bug.cgi?id=89302

Unreviewed EFL gardening. Skip flaky test.

Patch by Christophe Dumez <Christophe Dumez> on 2012-06-17

  • platform/efl/TestExpectations:
1:34 AM Changeset in webkit [120548] by abarth@webkit.org
  • 3 edits in trunk/Source/WebCore

Attempt to fix a large number of tests I broke with http://trac.webkit.org/changeset/120547

We need to reset the device scale facter after each test.

  • testing/InternalSettings.cpp:

(WebCore::InternalSettings::InternalSettings):
(WebCore::InternalSettings::restoreTo):

  • testing/InternalSettings.h:

(InternalSettings):

12:01 AM Changeset in webkit [120547] by abarth@webkit.org
  • 5 edits in trunk/LayoutTests

Update a few more tests to use internals.settings.setDeviceScaleFactor.

  • compositing/tiled-layers-hidpi.html:
  • fast/canvas/2d.backingStorePixelRatio.html:
  • fast/canvas/2d.imageDataHD.html:
  • platform/mac-wk2/plugins/contents-scale-factor.html:
Note: See TracTimeline for information about the timeline view.