Timeline
Jun 5, 2012:
- 11:53 PM Changeset in webkit [119565] by
-
- 3 edits2 adds in trunk
ASSERTION FAILED: ASSERT(!isPercentageIntrinsicSize) in RenderReplaced::computeIntrinsicRatioInformationForRenderBox
https://bugs.webkit.org/show_bug.cgi?id=88197
Patch by Joe Thomas <joethomas@motorola.com> on 2012-06-05
Reviewed by Daniel Bates.
RenderSVGRoot extends RenderReplaced and it overrides the computeIntrinsicRatioInformation()
method of RenderReplaced. RenderSVGRoot::computeIntrinsicRatioInformation
sets isPercentageIntrinsicSize to true while it handles width and height of percentage types.
So this assertion should be fired only if intrinsicRatio is non-zero.
Source/WebCore:
Test: svg/in-html/svg-assert-failure-percentage.html
- rendering/RenderReplaced.cpp:
(WebCore::RenderReplaced::computeIntrinsicRatioInformationForRenderBox):
LayoutTests:
- svg/in-html/svg-assert-failure-percentage-expected.txt: Added.
- svg/in-html/svg-assert-failure-percentage.html: Added.
- 11:45 PM Changeset in webkit [119564] by
-
- 3 edits2 adds in trunk
Changing flexbox justify/alignment doesn't update their positioning
https://bugs.webkit.org/show_bug.cgi?id=88366
Reviewed by Ojan Vafai.
Source/WebCore:
Test: css3/flexbox/style-change.html
- rendering/style/RenderStyle.cpp:
(WebCore::RenderStyle::diff): Since these member variables were moved
out of StyleFlexibleBoxData, we have to compare them in RenderStyle::diff.
LayoutTests:
- css3/flexbox/style-change-expected.txt: Added.
- css3/flexbox/style-change.html: Added.
- 10:51 PM Changeset in webkit [119563] by
-
- 2 edits in trunk/Source/WebCore
JSC implementation of SerializedScriptValue for IndexedDB
https://bugs.webkit.org/show_bug.cgi?id=88048
Reviewed by George Staikos.
No new tests. All the current indexeddb test cases apply when indexeddb works for JSC.
- bindings/js/SerializedScriptValue.cpp:
(WebCore::CloneSerializer::serializeNumber):
(CloneSerializer):
(WebCore::CloneDeserializer::toWireString):
(CloneDeserializer):
(WebCore::SerializedScriptValue::create):
(WebCore::SerializedScriptValue::toWireString):
(WebCore):
(WebCore::SerializedScriptValue::createFromWire):
(WebCore::SerializedScriptValue::numberValue):
(WebCore::SerializedScriptValue::deserialize):
- 10:41 PM Changeset in webkit [119562] by
-
- 3 edits2 adds in trunk
Nested CSS flexbox renders incorrectly
https://bugs.webkit.org/show_bug.cgi?id=87679
Reviewed by Julien Chaffraix.
Source/WebCore:
If the containing block with a perpendicular writing mode has an overrideHeight,
then its logical height should be based off that instead of its RenderStyle.
Test: css3/flexbox/perpendicular-writing-modes-inside-flex-item.html
- rendering/RenderBox.cpp:
(WebCore::RenderBox::perpendicularContainingBlockLogicalHeight):
LayoutTests:
- css3/flexbox/perpendicular-writing-modes-inside-flex-item-expected.txt: Added.
- css3/flexbox/perpendicular-writing-modes-inside-flex-item.html: Added.
- 10:37 PM Changeset in webkit [119561] by
-
- 2 edits in trunk/Source/WebCore
Need special-case toJS(IDBAny) for IDBCursorWithValue
https://bugs.webkit.org/show_bug.cgi?id=87963
Reviewed by George Staikos.
No new tests, this is for indexedDB, which is not working for JSC binding.
When indexedDB working for JSC, all the indexdb test cases should apply.
- bindings/js/JSIDBAnyCustom.cpp:
(WebCore::toJS):
- 10:35 PM Changeset in webkit [119560] by
-
- 2 edits in trunk/Source/WebKit/chromium
Roll Chromium DEPS from r140653 to r140700.
- DEPS:
- 10:32 PM Changeset in webkit [119559] by
-
- 3 edits in trunk/Source/WebKit2
[Qt][WK2] Use Component.onCompleted in API tests for single and multi file upload tests.
https://bugs.webkit.org/show_bug.cgi?id=88247
Patch by Dinu Jacob <dinu.jacob@nokia.com> on 2012-06-05
Reviewed by Kenneth Rohde Christiansen.
With r116936, Component.onCompleted can be used for callback instead of the
1s timer in single and multi file upload tests.
- UIProcess/API/qt/tests/qmltests/WebView/tst_multiFileUpload.qml:
- UIProcess/API/qt/tests/qmltests/WebView/tst_singleFileUpload.qml:
- 10:25 PM Changeset in webkit [119558] by
-
- 2 edits in trunk/Source/JavaScriptCore
Improve Math.round and Math.floor intrinsic
https://bugs.webkit.org/show_bug.cgi?id=88314
Reviewed by Filip Pizlo.
Currently we call a native function from the JIT code to complete the
"round" and "floor" operations. We could inline some fast paths
especially for those positive values on the platforms where floating
point truncation is supported.
This brings 3% gain on Kraken, especially 32% on audio-oscillator,
and slight win on SunSpider, measured on IA32.
- jit/ThunkGenerators.cpp:
(JSC::floorThunkGenerator):
(JSC):
(JSC::roundThunkGenerator):
- 10:19 PM Changeset in webkit [119557] by
-
- 2 edits in trunk
[GTK] show the feature list in alphabetical order
https://bugs.webkit.org/show_bug.cgi?id=88343
Patch by Danilo Cesar Lemes de Paula <danilo.cesar@collabora.co.uk> on 2012-06-05
Reviewed by Gustavo Noronha Silva.
- configure.ac:
- 10:08 PM Changeset in webkit [119556] by
-
- 8 edits in trunk/Source/JavaScriptCore
Remove JSObject::m_inheritorID
https://bugs.webkit.org/show_bug.cgi?id=88378
Reviewed by Geoff Garen.
This is rarely used, and not performance critical (the commonly accessed copy is cached on JSFunction),
and most objects don't need an inheritorID (this value is only used if the object is used as a prototype).
Instead use a private named value in the object's property storage.
- dfg/DFGSpeculativeJIT.h:
(JSC::DFG::SpeculativeJIT::emitAllocateBasicJSObject):
- No need m_inheritorID to initialize!
- jit/JITInlineMethods.h:
(JSC::JIT::emitAllocateBasicJSObject):
- No need m_inheritorID to initialize!
- llint/LowLevelInterpreter.asm:
- No need m_inheritorID to initialize!
- runtime/JSGlobalData.h:
(JSGlobalData):
- Added private name 'm_inheritorIDKey'.
- runtime/JSGlobalThis.cpp:
(JSC::JSGlobalThis::setUnwrappedObject):
- resetInheritorID is now passed a JSGlobalData&.
- runtime/JSObject.cpp:
(JSC::JSObject::visitChildren):
- No m_inheritorID to be marked.
(JSC::JSObject::createInheritorID):
- Store the newly created inheritorID in the property map.
- runtime/JSObject.h:
(JSC::JSObject::resetInheritorID):
- Remove the inheritorID from property storage.
(JSC::JSObject::inheritorID):
- Read the inheritorID from property storage.
- 10:01 PM ScalesAndZooms created by
- 9:58 PM Changeset in webkit [119555] by
-
- 1 edit1 delete in trunk/LayoutTests
Remove erroneous expected image added in r119544.
- platform/chromium-mac/fast/forms/select/optgroup-rendering-expected.txt: Removed.
- 9:44 PM Changeset in webkit [119554] by
-
- 4 edits in trunk/Source
[chromium] Set default memory allocation limit bytes when GL_CHROMIUM_gpu_memory_manager is not supported by graphics context.
https://bugs.webkit.org/show_bug.cgi?id=87996
Patch by Michal Mocny <mmocny@google.com> on 2012-06-05
Reviewed by James Robinson.
Soon we hope to have renderers not manage there own memory allocation limits, and rely exclusively on the
allocation given by GpuMemoryManager. However, when the graphics context does not support the
GL_CHROMIUM_gpu_memory_manager extension, such as the one used by DumpRenderTree, we must default the
memory allocation limit to some non-0 value.
- platform/graphics/chromium/LayerRendererChromium.cpp:
(WebCore::LayerRendererChromium::initialize):
- platform/graphics/chromium/cc/CCSingleThreadProxy.cpp:
(WebCore::CCSingleThreadProxy::postSetContentsMemoryAllocationLimitBytesToMainThreadOnImplThread):
- platform/graphics/chromium/cc/CCSingleThreadProxy.h:
(WebCore):
(DebugScopedSetMainThreadFromEither):
(WebCore::DebugScopedSetMainThreadFromEither::DebugScopedSetMainThreadFromEither):
(WebCore::DebugScopedSetMainThreadFromEither::~DebugScopedSetMainThreadFromEither):
- 8:44 PM Changeset in webkit [119553] by
-
- 5 edits1 delete in trunk/Source/WebKit/chromium
Unreviewed, rolling out r119494.
http://trac.webkit.org/changeset/119494
https://bugs.webkit.org/show_bug.cgi?id=87911
We found similar APIs are already implemented
- public/WebWidget.h:
- src/WebViewImpl.cpp:
- src/WebViewImpl.h:
(WebViewImpl):
- tests/WebViewTest.cpp:
- tests/data/textarea.html: Removed.
- 8:04 PM Changeset in webkit [119552] by
-
- 9 edits in trunk/Source
[Qt] Use GraphicsContext3DOpenGLES.cpp when using OpenGL ES
https://bugs.webkit.org/show_bug.cgi?id=78720
Patch by Roland Takacs <takacs.roland@stud.u-szeged.hu> on 2012-06-05
Reviewed by Noam Rosenthal.
Source/WebCore:
Functions are removed from GraphicsContext3DQt that already implemented
in GraphicsContext3DCommon. Macros that not supported at GLES have been
changed to their appropriate pair that found in GraphicsContext3D
(e.g. GL_FRAMEBUFFER_EXT to GraphicsContext3D::FRAMEBUFFER).
It was needed because OpenGLES does not support these macros.
Extensions3DQt is removed beceause it has never been used.
Some functions have been moved fron GraphicsContext3DCommon to *OpenGL
and *OpenGLES to provide specialization.
The "m_depthStencilBuffer" member is added to Qt port because it uses
"m_depthStencilBuffer" like the other platforms.
- Target.pri:
- platform/graphics/GraphicsContext3D.h:
- platform/graphics/opengl/GraphicsContext3DOpenGL.cpp:
(WebCore::GraphicsContext3D::depthRange):
(WebCore):
(WebCore::GraphicsContext3D::clearDepth):
- platform/graphics/opengl/GraphicsContext3DOpenGLCommon.cpp:
(WebCore::GraphicsContext3D::prepareTexture):
(WebCore::GraphicsContext3D::readRenderingResults):
(WebCore::GraphicsContext3D::reshape):
(WebCore::GraphicsContext3D::copyTexImage2D):
(WebCore::GraphicsContext3D::copyTexSubImage2D):
(WebCore::GraphicsContext3D::readPixels):
- platform/graphics/opengl/GraphicsContext3DOpenGLES.cpp:
(WebCore::GraphicsContext3D::reshapeFBOs):
(WebCore::GraphicsContext3D::depthRange):
(WebCore):
(WebCore::GraphicsContext3D::clearDepth):
- platform/graphics/qt/GraphicsContext3DQt.cpp:
(WebCore::GraphicsContext3D::isGLES2Compliant):
(WebCore::GraphicsContext3DPrivate::blitMultisampleFramebuffer):
(WebCore::GraphicsContext3D::GraphicsContext3D):
(WebCore::GraphicsContext3D::~GraphicsContext3D):
(WebCore):
Source/WTF:
Defined a global Opengl ES macro to check OpenGL ES on every platforms.
- wtf/Platform.h:
- 7:41 PM Changeset in webkit [119551] by
-
- 7 edits3 adds in trunk/LayoutTests
Chromium rebaselines after r115091.
- platform/chromium-mac-leopard/tables/mozilla_expected_failures/collapsing_borders/bug41262-5-expected.png:
- platform/chromium-mac-leopard/tables/mozilla_expected_failures/collapsing_borders/bug41262-5-expected.txt: Added.
- platform/chromium-mac-snowleopard/tables/mozilla_expected_failures/collapsing_borders/bug41262-5-expected.png:
- platform/chromium-mac-snowleopard/tables/mozilla_expected_failures/collapsing_borders/bug41262-5-expected.txt: Added.
- platform/chromium-mac/tables/mozilla_expected_failures/collapsing_borders/bug41262-5-expected.png:
- platform/chromium-mac/tables/mozilla_expected_failures/collapsing_borders/bug41262-5-expected.txt: Added.
- platform/chromium-win/tables/mozilla_expected_failures/collapsing_borders/bug41262-5-expected.png:
- platform/chromium-win/tables/mozilla_expected_failures/collapsing_borders/bug41262-5-expected.txt:
- platform/chromium/TestExpectations:
- 7:13 PM Changeset in webkit [119550] by
-
- 4 edits1 add in trunk/LayoutTests
More Chromium rebaselines after r119528 and r119512.
- platform/chromium-linux/editing/selection/3690703-2-expected.png:
- platform/chromium-linux/editing/selection/3690703-expected.png:
- platform/chromium-linux/editing/selection/3690719-expected.png:
- platform/chromium-linux/fast/sub-pixel/float-containing-block-with-margin-expected.png: Added.
- 6:57 PM Changeset in webkit [119549] by
-
- 2 edits in trunk/Source/WebKit/blackberry
[BlackBerry]Differentiate options and group options by TypeOption and TypeOptionInGroup
https://bugs.webkit.org/show_bug.cgi?id=88342
Patch by Crystal Zhang <haizhang@rim.com> on 2012-06-05
Reviewed by Rob Buis.
RIM PR: 163141
Reviewed internally by Mike Fenton.
We set all options' type to TypeOption, which can't tell if an option belongs to a group,
introduce TypeOptionInGroup to fix it.
- WebKitSupport/InputHandler.cpp:
(BlackBerry::WebKit::InputHandler::openSelectPopup):
- 6:51 PM Changeset in webkit [119548] by
-
- 4 edits in trunk/Source/WebCore
Special layout handler should be done on top frame being printed.
https://bugs.webkit.org/show_bug.cgi?id=88201
Patch by Vitaly Buka <vitalybuka@chromium.org> on 2012-06-05
Reviewed by Brady Eidson.
No new tests. Root case is already covered by tests.
Case described in the issue can be reproduced only by direct call
to Frame::setPrinting of subframe. Probably it's not possible with
layout tests.
- page/Frame.cpp:
(WebCore::Frame::setPrinting):
Use shouldUsePrintingLayout to choose proper version of forceLayout().
(WebCore::Frame::shouldUsePrintingLayout):
Checks if current frame is the top frame being printed.
- rendering/RenderView.cpp:
(WebCore::RenderView::shouldUsePrintingLayout): Forward to Frame.
- 6:40 PM Changeset in webkit [119547] by
-
- 36 edits5 adds in trunk
[BlackBerry] Conditionally enlarge HTML5 video controls in fullscreen mode
https://bugs.webkit.org/show_bug.cgi?id=87551
Patch by Max Feil <mfeil@rim.com> on 2012-06-05
Reviewed by Antonio Gomes.
Source/WebCore:
Changes to make media controls larger in fullscreen mode. The
BlackBerry media controls used to be sized by a combination
of CSS styles specified in mediaControlsBlackBerry.css and
hard coded constants in RenderThemeBlackBerry.cpp. This patch
moves all the dimensional sizes to a common location, i.e.
RenderThemeBlackBerry.cpp. Having all the logic in C++ code allows
more complex decision making for when and how much to enlarge the
media controls. Currently the controls are enlarged by a factor
of 2 if the document is in fullscreen mode, the video element
is the current fullscreen element, and the deviceScaleFactor of
the page is less than 2.0. This avoids overly large controls
on pages which are already at least doubling the size of the
controls via viewport meta tag device-width settings. In other
words, if the CSS pixels are already enlarged compared to device
pixels by a factor of at least 2 in length and width.
In order to accomplish this, I had to add a new Element* parameter
to RenderTheme::adjustSliderThumbSize(). This is needed so
that fullscreen mode can be checked and the deviceScaleFactor
retrieved. I feel that this does not have a big impact on the
code and the other ports, for which I will simply be adding the
additional parameter in their platform code.
Tests:
video-controls-enlarged-fullscreen-meta.html
video-controls-enlarged-fullscreen-nometa.html
- css/mediaControlsBlackBerry.css:
(audio::-webkit-media-controls-panel, video::-webkit-media-controls-panel):
(audio::-webkit-media-controls-play-button, video::-webkit-media-controls-play-button):
(audio::-webkit-media-controls-timeline-container, video::-webkit-media-controls-timeline-container):
(audio::-webkit-media-controls-current-time-display, video::-webkit-media-controls-current-time-display):
(audio::-webkit-media-controls-time-remaining-display, video::-webkit-media-controls-time-remaining-display):
(audio::-webkit-media-controls-timeline, video::-webkit-media-controls-timeline):
(audio::-webkit-media-controls-fullscreen-button, video::-webkit-media-controls-fullscreen-button):
(audio::-webkit-media-controls-mute-button, video::-webkit-media-controls-mute-button):
(audio::-webkit-media-controls-volume-slider-container, video::-webkit-media-controls-volume-slider-container):
(audio::-webkit-media-controls-volume-slider, video::-webkit-media-controls-volume-slider):
- html/shadow/SliderThumbElement.cpp:
(WebCore::RenderSliderThumb::updateAppearance):
- platform/blackberry/RenderThemeBlackBerry.cpp:
(WebCore):
(WebCore::determineRenderSlider):
(WebCore::determineFullScreenMultiplier):
(WebCore::RenderThemeBlackBerry::adjustSliderThumbSize):
(WebCore::RenderThemeBlackBerry::adjustMediaControlStyle):
(WebCore::RenderThemeBlackBerry::adjustSliderTrackStyle):
(WebCore::RenderThemeBlackBerry::paintMediaSliderTrack):
(WebCore::RenderThemeBlackBerry::paintMediaSliderThumb):
- platform/blackberry/RenderThemeBlackBerry.h:
(RenderThemeBlackBerry):
- platform/efl/RenderThemeEfl.cpp:
(WebCore::RenderThemeEfl::adjustSliderThumbSize):
- platform/efl/RenderThemeEfl.h:
(RenderThemeEfl):
- platform/gtk/RenderThemeGtk.h:
- platform/gtk/RenderThemeGtk2.cpp:
(WebCore::RenderThemeGtk::adjustSliderThumbSize):
- platform/gtk/RenderThemeGtk3.cpp:
(WebCore::RenderThemeGtk::adjustSliderThumbSize):
- platform/qt/RenderThemeQt.cpp:
(WebCore::RenderThemeQt::adjustSliderThumbSize):
- platform/qt/RenderThemeQt.h:
(RenderThemeQt):
- platform/qt/RenderThemeQtMobile.cpp:
(WebCore::RenderThemeQtMobile::adjustSliderThumbSize):
- platform/qt/RenderThemeQtMobile.h:
(RenderThemeQtMobile):
- rendering/RenderThemeChromiumLinux.cpp:
(WebCore::RenderThemeChromiumLinux::adjustSliderThumbSize):
- rendering/RenderThemeChromiumLinux.h:
(RenderThemeChromiumLinux):
- rendering/RenderThemeChromiumSkia.cpp:
(WebCore::RenderThemeChromiumSkia::adjustSliderThumbSize):
- rendering/RenderThemeChromiumSkia.h:
(RenderThemeChromiumSkia):
- rendering/RenderThemeChromiumWin.cpp:
(WebCore):
(WebCore::RenderThemeChromiumWin::adjustSliderThumbSize):
- rendering/RenderThemeChromiumWin.h:
(RenderThemeChromiumWin):
- rendering/RenderThemeMac.h:
(RenderThemeMac):
- rendering/RenderThemeSafari.cpp:
(WebCore::RenderThemeSafari::adjustSliderThumbSize):
- rendering/RenderThemeSafari.h:
(RenderThemeSafari):
- rendering/RenderThemeWin.cpp:
(WebCore::RenderThemeWin::adjustSliderThumbSize):
- rendering/RenderThemeWin.h:
(RenderThemeWin):
- rendering/RenderThemeWinCE.cpp:
(WebCore::RenderThemeWinCE::adjustSliderThumbSize):
- rendering/RenderThemeWinCE.h:
(RenderThemeWinCE):
- rendering/RenderTheme.cpp:
(WebCore::RenderTheme::adjustStyle):
(WebCore::RenderTheme::adjustMediaControlStyle):
(WebCore):
(WebCore::RenderTheme::adjustSliderThumbStyle):
(WebCore::RenderTheme::adjustSliderThumbSize):
- rendering/RenderTheme.h:
(RenderTheme):
Source/WebKit/qt:
Add a new parameter to adjustSliderThumbSize() for BlackBerry
port. Fixing compiles of other ports.
- WebCoreSupport/RenderThemeQStyle.cpp:
(WebCore::RenderThemeQStyle::adjustSliderThumbSize):
- WebCoreSupport/RenderThemeQStyle.h:
(RenderThemeQStyle):
LayoutTests:
- media/media-controls.js:
(mediaControlsButtonDimensions):
- platform/blackberry/media/video-controls-enlarged-fullscreen-meta-expected.txt: Added.
- platform/blackberry/media/video-controls-enlarged-fullscreen-meta.html: Added.
- platform/blackberry/media/video-controls-enlarged-fullscreen-nometa-expected.txt: Added.
- platform/blackberry/media/video-controls-enlarged-fullscreen-nometa.html: Added.
- 6:37 PM Changeset in webkit [119546] by
-
- 6 edits in trunk/Source
https://bugs.webkit.org/show_bug.cgi?id=88370
Memory sampler should trigger low memory signal
Reviewed by Geoff Garen.
Source/WebCore:
No new tests. Verify by running stress test which crashes
in a few minutes without the fix.
Fix assumption in block code. We could get in a state where timer_event_source
had already been released before the block ran.
- platform/mac/MemoryPressureHandlerMac.mm:
(WebCore::MemoryPressureHandler::holdOff):
Source/WebKit2:
Send low memory signal when running the memory sampler. We'd
like to test memory that cannot be freed.
- Shared/WebMemorySampler.cpp:
(WebKit::WebMemorySampler::sampleTimerFired):
- Shared/WebMemorySampler.h:
(WebMemorySampler):
- Shared/mac/WebMemorySampler.mac.mm:
(WebKit):
(WebKit::WebMemorySampler::sendMemoryPressureEvent):
- 6:10 PM Changeset in webkit [119545] by
-
- 4 edits in trunk/LayoutTests
Stop generating image results in the tests added by r114172.
Rubber-stamped by Simon Fraser.
- css3/filters/huge-region-composited.html:
- css3/filters/huge-region.html:
- platform/chromium/TestExpectations:
- 6:01 PM Changeset in webkit [119544] by
-
- 2 edits6 adds in trunk/LayoutTests
Chromium rebaselines after r118772.
- platform/chromium-mac-leopard/fast/forms/select/optgroup-rendering-expected.png: Added.
- platform/chromium-mac-snowleopard/fast/forms/select/optgroup-rendering-expected.png: Added.
- platform/chromium-mac/fast/forms/select/optgroup-rendering-expected.png: Added.
- platform/chromium-mac/fast/forms/select/optgroup-rendering-expected.txt: Added.
- platform/chromium-win/fast/forms/select/optgroup-rendering-expected.png: Added.
- platform/chromium-win/fast/forms/select/optgroup-rendering-expected.txt: Added.
- platform/chromium/TestExpectations:
- 5:57 PM Changeset in webkit [119543] by
-
- 2 edits in trunk/Source/WebKit/mac
Build fix.
- WebCoreSupport/WebFrameLoaderClient.mm:
(WebFrameLoaderClient::createPlugin):
The error constant has been renamed.
- 5:48 PM Changeset in webkit [119542] by
-
- 2 edits in trunk/Source/WebKit/mac
Rename a plug-in loading error constant
https://bugs.webkit.org/show_bug.cgi?id=88381
<rdar://problem/11602601>
Reviewed by Sam Weinig.
- Misc/WebKitErrors.h:
- 5:47 PM Changeset in webkit [119541] by
-
- 2 edits in trunk/LayoutTests
Unreviewed. Fixing an incorrect path in lion Skipped file.
- platform/mac-lion/Skipped:
- 5:39 PM Changeset in webkit [119540] by
-
- 24 edits in trunk/Source/WebCore
[Forms] Introduce InputNumber type as an alias of double for replacing it to Decimal
https://bugs.webkit.org/show_bug.cgi?id=88275
Reviewed by Kent Tamura.
This patch introduced InputNumber type as alias of double to be
replaced to Decimal and convertDoubleToInputNumber and
convertToInputNumberToDouble for reducing size of patch when we
introduce Decimal type.
This patch added new vairables to use const variable to reduce
copy operation when introducing Decimal type.
In the implementation, we use InputNumber, will be Decimal, however some methods in IDL
use double. To clarify internal usage of value and external usage, thit patch renamed
following functions:
- parseToDouble => parseNumber
- setValueAsNumber => setValueAsDouble and setValueAsInputNumber
- valueAsNumber => valueAsDouble
This patch is a part of introducing Decimal, https://bugs.webkit.org/show_bug.cgi?id=80009
No new tests. This patch doesn't change behavior.
- html/BaseDateAndTimeInputType.cpp:
(WebCore::BaseDateAndTimeInputType::valueAsDate): Replaced parseToNumber() to valueAsDouble()
(WebCore::BaseDateAndTimeInputType::setValueAsDate): Inserted convertDoubleToInputNumber.
(WebCore::BaseDateAndTimeInputType::valueAsDouble): Use new function parseToDouble().
(WebCore::BaseDateAndTimeInputType::setValueAsInputNumber): Added.
(WebCore::BaseDateAndTimeInputType::defaultValueForStepUp): Changed return type to InputNumber.
(WebCore::BaseDateAndTimeInputType::parseToNumber): This is replacement of parseToDouble().
(WebCore::BaseDateAndTimeInputType::parseToDouble): Added for valueAsDate and valueAsDouble.
(WebCore::BaseDateAndTimeInputType::serialize): Changed parameter type to InputNumber.
(WebCore::BaseDateAndTimeInputType::serializeWithComponents): Replace double to InputNumber for getAllowdStep.
- html/BaseDateAndTimeInputType.h:
(BaseDateAndTimeInputType):
- html/DateInputType.cpp:
(WebCore::DateInputType::createStepRange): Replaced double to InputNumber.
- html/DateInputType.h:
(DateInputType):
- html/DateTimeInputType.cpp:
(WebCore::DateTimeInputType::defaultValueForStepUp): Changed return type to InputNumber.
(WebCore::DateTimeInputType::createStepRange): Replaced double to InputNumber.
- html/DateTimeInputType.h:
(DateTimeInputType):
- html/DateTimeLocalInputType.cpp:
(WebCore::DateTimeLocalInputType::createStepRange): Replaced double to InputNumber.
- html/DateTimeLocalInputType.h:
(DateTimeLocalInputType):
- html/HTMLInputElement.cpp:
(WebCore::HTMLInputElement::getAllowedValueStep): Changed parameter type to InputNumber.
(WebCore::HTMLInputElement::valueAsNumber): Use InputType::valueAsDouble.
(WebCore::HTMLInputElement::setValueAsNumber): Use InputType::valueAsDouble.
- html/HTMLInputElement.h:
(HTMLInputElement):
- html/InputType.cpp:
(WebCore::InputType::valueAsDouble): Replacement of valueAsNumber.
(WebCore::InputType::setValueAsDouble): Replacement of valueAsNumber.
(WebCore::InputType::setValueAsInputNumber): Replacement of valueAsNumber.
(WebCore::InputType::rangeUnderflow): Replaced double to InputNumber.
(WebCore::InputType::rangeOverflow): Replaced double to InputNumber.
(WebCore::InputType::defaultValueForStepUp): Changed return type to InputNumber.
(WebCore::InputType::isInRange): Replaced double to InputNumber.
(WebCore::InputType::isOutOfRange): Replaced double to InputNumber.
(WebCore::InputType::stepMismatch): Replaced double to InputNumber.
(WebCore::InputType::validationMessage): Replaced double to InputNumber.
(WebCore::InputType::parseToNumber): Replacement of parseToDouble.
(WebCore::InputType::parseToNumberOrNaN): Added for reducing number of std::numeric_limits<double>::quiet_NaN().
(WebCore::InputType::parseToNumberWithDecimalPlaces): Replaced double to InputNumber.
(WebCore::InputType::serialize): Changed parameter type to InputNumber.
(WebCore::InputType::applyStep): Replaced double to InputNumber.
(WebCore::InputType::getAllowedValueStep): Changed parameter type to InputNumber.
(WebCore::InputType::stepUpFromRenderer): Replaced double to InputNumber.
- html/InputType.h:
(InputType):
- html/MonthInputType.cpp:
(WebCore::MonthInputType::defaultValueForStepUp): Changed return type to InputNumber.
(WebCore::MonthInputType::createStepRange): Replaced double to InputNumber.
(WebCore::MonthInputType::parseToNumber): Replacement of parseToDouble.
- html/MonthInputType.h:
(MonthInputType):
- html/NumberInputType.cpp:
(WebCore::NumberInputType::valueAsDouble): Replacement of valueAsNumber. Changed to use parseToDoubleForNumberType.
(WebCore::NumberInputType::setValueAsDouble): Replacement of setValueAsNumber. Changed to use serializeForNumberType.
(WebCore::NumberInputType::setValueAsInputNumber): Replacement of setValueAsNumber.
(WebCore::NumberInputType::createStepRange): Replaced double to InputNumber.
(WebCore::NumberInputType::parseToNumber): Replacement of parseToDouble.
(WebCore::NumberInputType::parseToNumberWithDecimalPlaces): Replaced double to InputNumber.
(WebCore::NumberInputType::serialize):
- html/NumberInputType.h:
(NumberInputType):
- html/RangeInputType.cpp:
(WebCore::ensureMaximum): Replaced double to InputNumber.
(WebCore::RangeInputType::valueAsDouble): Replacement of valueAsNumber.
(WebCore::RangeInputType::setValueAsInputNumber): Replacement of setValueAsNumber.
(WebCore::RangeInputType::createStepRange): Replaced double to InputNumber.
(WebCore::RangeInputType::handleKeydownEvent): Replaced double to InputNumber.
(WebCore::RangeInputType::parseToNumber): Replacement of parseToDouble.
(WebCore::RangeInputType::serialize): Changed parameter type to InputNumber.
(WebCore::RangeInputType::sanitizeValue): Replaced double to InputNumber.
- html/RangeInputType.h:
(RangeInputType):
- html/StepRange.cpp:
(WebCore::StepRange::StepRange): Replaced double to InputNumber.
(WebCore::StepRange::acceptableError): Replaced double to InputNumber.
(WebCore::StepRange::alignValueForStep): Replaced double to InputNumber.
(WebCore):
(WebCore::StepRange::clampValue): Replaced double to InputNumber.
(WebCore::StepRange::stepMismatch): Replaced double to InputNumber.
- html/StepRange.h:
(WebCore::convertDoubleToInputNumber): Added.
(WebCore::convertInputNumberToDouble): Added.
(WebCore::StepRange::NumberWithDecimalPlaces::NumberWithDecimalPlaces): Replaced double to InputNumber.
(WebCore::StepRange::StepDescription::defaultValue): Replaced double to InputNumber.
(StepRange):
(WebCore::StepRange::maximum): Replaced double to InputNumber.
(WebCore::StepRange::minimum): Replaced double to InputNumber.
(WebCore::StepRange::step): Replaced double to InputNumber.
(WebCore::StepRange::stepBase): Replaced double to InputNumber.
(WebCore::StepRange::defaultValue): Replaced double to InputNumber.
(WebCore::StepRange::proportionFromValue): Replaced double to InputNumber.
(WebCore::StepRange::valueFromProportion): Replaced double to InputNumber.
- html/TimeInputType.cpp:
(WebCore::TimeInputType::defaultValueForStepUp): Replaced double to InputNumber.
(WebCore::TimeInputType::createStepRange): Replaced double to InputNumber.
- html/TimeInputType.h:
(TimeInputType):
- html/WeekInputType.cpp:
(WebCore::WeekInputType::createStepRange): Replaced double to InputNumber.
- html/WeekInputType.h:
(WeekInputType):
- html/shadow/CalendarPickerElement.cpp:
(WebCore::CalendarPickerElement::writeDocument): Replaced double to InputNumber.
- html/shadow/SliderThumbElement.cpp:
(WebCore::sliderPosition): Replaced double to InputNumber.
(WebCore::RenderSliderThumb::layout): Replaced double to InputNumber.
(WebCore::SliderThumbElement::setPositionFromPoint): Replaced double to InputNumber.
- 5:38 PM Changeset in webkit [119539] by
-
- 2 edits1 add in trunk/LayoutTests
Unreviewed gardening. Chromium windows test expectation updates.
- platform/chromium-win/fast/sub-pixel: Added.
- platform/chromium-win/fast/sub-pixel/float-containing-block-with-margin-expected.png: Added.
- platform/chromium/TestExpectations:
- 5:32 PM Changeset in webkit [119538] by
-
- 2 edits in trunk/Source/WebKit/chromium
Roll Chromium DEPS from r140528 to r140653.
- DEPS:
- 5:31 PM Changeset in webkit [119537] by
-
- 37 edits18 adds in trunk/LayoutTests
Unreviewed gardening. More updated Chromium test expectations following 119528.
- platform/chromium-linux-x86/editing/selection: Added.
- platform/chromium-linux-x86/editing/selection/3690703-2-expected.png: Added.
- platform/chromium-linux-x86/editing/selection/3690703-expected.png: Added.
- platform/chromium-linux-x86/editing/selection/3690719-expected.png: Added.
- platform/chromium-linux-x86/fast/sub-pixel: Added.
- platform/chromium-linux-x86/fast/sub-pixel/selection: Added.
- platform/chromium-linux-x86/fast/sub-pixel/selection/selection-gaps-at-fractional-offsets-expected.png: Added.
- platform/chromium-linux/fast/sub-pixel: Added.
- platform/chromium-linux/fast/sub-pixel/selection: Added.
- platform/chromium-linux/fast/sub-pixel/selection/selection-gaps-at-fractional-offsets-expected.png: Added.
- platform/chromium-mac-leopard/editing/selection/3690703-2-expected.png:
- platform/chromium-mac-leopard/editing/selection/3690703-expected.png:
- platform/chromium-mac-leopard/editing/selection/3690719-expected.png:
- platform/chromium-mac-leopard/editing/selection/selection-button-text-expected.png:
- platform/chromium-mac-leopard/editing/selection/vertical-lr-ltr-extend-line-backward-br-expected.png:
- platform/chromium-mac-leopard/editing/selection/vertical-lr-ltr-extend-line-forward-br-expected.png:
- platform/chromium-mac-leopard/editing/selection/vertical-rl-ltr-extend-line-backward-br-expected.png:
- platform/chromium-mac-leopard/editing/selection/vertical-rl-ltr-extend-line-backward-p-expected.png:
- platform/chromium-mac-leopard/editing/selection/vertical-rl-ltr-extend-line-backward-wrap-expected.png:
- platform/chromium-mac-leopard/editing/selection/vertical-rl-ltr-extend-line-forward-br-expected.png:
- platform/chromium-mac-leopard/editing/selection/vertical-rl-ltr-extend-line-forward-p-expected.png:
- platform/chromium-mac-leopard/editing/selection/vertical-rl-ltr-extend-line-forward-wrap-expected.png:
- platform/chromium-mac-leopard/editing/selection/vertical-rl-rtl-extend-line-backward-br-expected.png:
- platform/chromium-mac-leopard/editing/selection/vertical-rl-rtl-extend-line-backward-p-expected.png:
- platform/chromium-mac-leopard/editing/selection/vertical-rl-rtl-extend-line-forward-br-expected.png:
- platform/chromium-mac-leopard/editing/selection/vertical-rl-rtl-extend-line-forward-p-expected.png:
- platform/chromium-mac/editing/selection/3690703-2-expected.png:
- platform/chromium-mac/editing/selection/3690703-expected.png:
- platform/chromium-mac/editing/selection/3690719-expected.png:
- platform/chromium-mac/editing/selection/selection-button-text-expected.png:
- platform/chromium-mac/editing/selection/vertical-lr-ltr-extend-line-backward-br-expected.png:
- platform/chromium-mac/editing/selection/vertical-lr-ltr-extend-line-forward-br-expected.png:
- platform/chromium-mac/editing/selection/vertical-rl-ltr-extend-line-backward-br-expected.png:
- platform/chromium-mac/editing/selection/vertical-rl-ltr-extend-line-backward-p-expected.png:
- platform/chromium-mac/editing/selection/vertical-rl-ltr-extend-line-backward-wrap-expected.png:
- platform/chromium-mac/editing/selection/vertical-rl-ltr-extend-line-forward-br-expected.png:
- platform/chromium-mac/editing/selection/vertical-rl-ltr-extend-line-forward-p-expected.png:
- platform/chromium-mac/editing/selection/vertical-rl-ltr-extend-line-forward-wrap-expected.png:
- platform/chromium-mac/editing/selection/vertical-rl-rtl-extend-line-backward-br-expected.png:
- platform/chromium-mac/editing/selection/vertical-rl-rtl-extend-line-backward-p-expected.png:
- platform/chromium-mac/editing/selection/vertical-rl-rtl-extend-line-forward-br-expected.png:
- platform/chromium-mac/editing/selection/vertical-rl-rtl-extend-line-forward-p-expected.png:
- platform/chromium-mac/fast/sub-pixel/selection: Added.
- platform/chromium-mac/fast/sub-pixel/selection/selection-gaps-at-fractional-offsets-expected.png: Added.
- platform/chromium-win-vista/fast/sub-pixel: Added.
- platform/chromium-win-vista/fast/sub-pixel/selection: Added.
- platform/chromium-win-vista/fast/sub-pixel/selection/selection-gaps-at-fractional-offsets-expected.png: Added.
- platform/chromium-win/editing/selection/3690703-2-expected.png:
- platform/chromium-win/editing/selection/3690703-expected.png:
- platform/chromium-win/editing/selection/3690719-expected.png:
- platform/chromium-win/fast/sub-pixel: Added.
- platform/chromium-win/fast/sub-pixel/selection: Added.
- platform/chromium-win/fast/sub-pixel/selection/selection-gaps-at-fractional-offsets-expected.png: Added.
- platform/chromium/TestExpectations:
- 5:25 PM Changeset in webkit [119536] by
-
- 4 edits4 adds1 delete in trunk/LayoutTests
Chromium rebaselines for r119523 and r119529.
- platform/chromium-linux/compositing/layer-creation: Removed.
- platform/chromium-mac-snowleopard/fast/sub-pixel/float-containing-block-with-margin-expected.png: Added.
- platform/chromium-mac-snowleopard/fullscreen/full-screen-stacking-context-expected.png:
- platform/chromium-mac/compositing/iframes/scroll-fixed-transformed-element-expected.png: Added.
- platform/chromium-mac/fullscreen/full-screen-stacking-context-expected.png:
- platform/chromium-win/compositing/backing/no-backing-for-perspective-expected.txt:
- platform/chromium-win/compositing/iframes/scroll-fixed-transformed-element-expected.png: Added.
- platform/chromium-win/compositing/layer-creation/no-compositing-for-preserve-3d-expected.txt: Added.
- 5:15 PM Changeset in webkit [119535] by
-
- 2 edits in trunk/Source/WebKit2
<rdar://problem/11575898> and https://bugs.webkit.org/show_bug.cgi?id=88372
REGRESSION(110494): Can no longer drag-and-drop links into FileMaker Pro 12
- Before r110494 we used to write the NSStringPboardType to the pasteboard. After r110494 we write NSURLPboardType instead. That's basically a no-op, but the NSStringPboardType needs to be restored.
- Before r110494 we wrote the link destination URL to the pasteboard. After r110494 we write the image src URL if the link is an image. We need to revert to writing the link URL.
Fortunately these changes can be accomplished with a 2-line fix.
Reviewed by Enrica Casucci.
- UIProcess/API/mac/WKView.mm:
(-[WKView _setPromisedData:WebCore::withFileName:withExtension:withTitle:withURL:withVisibleURL:withArchive:WebCore::forPasteboard:]):
n
- 5:05 PM Changeset in webkit [119534] by
-
- 2 edits in trunk/Source/WebCore
Build fix after r119527.
- dom/ViewportArguments.cpp:
(WebCore::computeViewportAttributes):
- 4:12 PM Changeset in webkit [119533] by
-
- 2 edits in trunk/Source/WebCore
[Cairo] [OpenGL] Enable GL_OES_standard_derivatives
https://bugs.webkit.org/show_bug.cgi?id=87583
Patch by Martin Robinson <mrobinson@igalia.com> on 2012-06-05
Reviewed by Alejandro G. Castro.
No new tests. This is covered by oes-standard-derivatives.html
in the Khronos WebGL conformance tests, which may one day be part
of the WebKit test suite.
Activate the GL_OES_standard_derivatives on Cairo along with
Qt and Mac.
- platform/graphics/opengl/Extensions3DOpenGL.cpp:
(WebCore::Extensions3DOpenGL::ensureEnabled):
(WebCore::Extensions3DOpenGL::isEnabled):
- 3:45 PM Changeset in webkit [119532] by
-
- 2 edits in trunk/Tools
Buildbot fix.
- BuildSlaveSupport/build.webkit.org-config/master.cfg: We lose the type attribute earlier
in the function. Store in a local variable.
(loadBuilderConfig):
- 3:28 PM Changeset in webkit [119531] by
-
- 3 edits in trunk/LayoutTests
Fixing two leopard-specific baselines following r119528.
- platform/chromium-mac-leopard/fast/repaint/japanese-rl-selection-repaint-expected.png:
- platform/chromium-mac-leopard/fast/repaint/japanese-rl-selection-repaint-in-regions-expected.png:
- 3:19 PM Changeset in webkit [119530] by
-
- 2 edits in trunk/LayoutTests
Fix Chromium test expectations.
- platform/chromium/TestExpectations:
- 3:18 PM Changeset in webkit [119529] by
-
- 8 edits1 copy1 add in trunk
Avoid creating compositing layers for preserve-3d without transformed descendants
https://bugs.webkit.org/show_bug.cgi?id=88115
Source/WebCore:
Reviewed by Antti Koivisto.
Avoid creating compositing layers, and therefore using excess backing store,
for elements that have -webkit-transform-style: preserve-3d, but no 3D-transformed
descendants that would be affected by that preserve-3d.
Test: compositing/layer-creation/no-compositing-for-preserve-3d.html
- rendering/RenderLayer.h: Replace the "mustOverlap" flag with a enum
that describes the different reasons for indirect compositing, so that
we can use that information to decide whether to allocate backing store.
(WebCore::RenderLayer::setIndirectCompositingReason):
(WebCore::RenderLayer::indirectCompositingReason):
(WebCore::RenderLayer::mustCompositeForIndirectReasons):
- rendering/RenderLayer.cpp:
(WebCore::RenderLayer::RenderLayer): Initialize m_indirectCompositingReason
- rendering/RenderLayerCompositor.h: New out param for computeCompositingRequirements()
that is uses to indicate that a 3d-transformed descendant has been encountered.
Rename requiresCompositingWhenDescendantsAreCompositing() to requiresCompositingForIndirectReason(),
and return the reason as an out param.
- rendering/RenderLayerCompositor.cpp:
(WebCore::RenderLayerCompositor::updateCompositingLayers): Pass in the saw3DTransform
param to computeCompositingRequirements().
(WebCore::RenderLayerCompositor::computeCompositingRequirements): Return a flag
from computeCompositingRequirements() that is set to true if we've seen descendants
that have 3d transforms. This is later used to decide whether to composite for
perspective or preserve-3d.
Change the "mustOverlapCompositedLayers" code to use the new "indirect compositing"
enum flags.
After enumerating children, call requiresCompositingForIndirectReason() and
record the reason in the layer.
(WebCore::RenderLayerCompositor::needsToBeComposited): Use mustCompositeForIndirectReasons() now.
(WebCore::RenderLayerCompositor::requiresOwnBackingStore): Consult the indirect compositing
reason rather than just looking for the overlap flag.
(WebCore::RenderLayerCompositor::reasonForCompositing): Now that we have more information
about indirect compositing reasons, the logging can be more detailed.
(WebCore::RenderLayerCompositor::requiresCompositingForTransform): This now only looks
for 3d transforms. We now treat perspective and perserve-3d as "indirect" reasons, because
whether they composite depends on descendants having non-affine transforms.
(WebCore::RenderLayerCompositor::requiresCompositingForIndirectReason): Includes the logic
previously in requiresCompositingWhenDescendantsAreCompositing(), and now determines
whether to composite for preserve-3d and perspective, based on whether we have transformed descendants.
LayoutTests:
Reviewed by Antti Koivisto.
- compositing/backing/no-backing-for-perspective-expected.txt:
- compositing/backing/no-backing-for-perspective.html: Change the transform to be non-affine so that
we keep making layers for perspective.
- compositing/layer-creation/no-compositing-for-preserve-3d-expected.txt: Added.
- compositing/layer-creation/no-compositing-for-preserve-3d.html: Copied from LayoutTests/compositing/backing/no-backing-for-perspective.html.
- 2:56 PM Changeset in webkit [119528] by
-
- 37 edits6 adds in trunk
Block selection gaps painted not properly pixel snapped
https://bugs.webkit.org/show_bug.cgi?id=88000
Reviewed by Ryosuke Niwa.
Source/WebCore:
Fixes for selection painting in both inline line heights and block gaps. We were unintentionally
flooring inline and block offsets in RenderBlock, but actually need to intentionally floor the
logical left and right values in logical(Left|Right)SelectionGap to properly align our block
selection gaps. This is to properly mirror the behavior in pixelSnappedLogicalRightOffsetForLine.
We were also unintentionally upcasting LayoutUnits to floats in InlineTextBox's selection painting
routine. Now we're properly rounding.
Test: fast/sub-pixel/selection/selection-gaps-at-fractional-offsets.html
- rendering/InlineTextBox.cpp:
(WebCore::InlineTextBox::paintSelection): Move paint values to integers and do proper rounding.
- rendering/RenderBlock.cpp:
(WebCore::blockDirectionOffset): Fix incorrect flooring in return value.
(WebCore::inlineDirectionOffset): Ditto.
(WebCore): Removing previous attempt to properly mirror inline box selection pixel alignment. This
implementation rounded the final x value instead of properly rounding our block offset then flooring
the offset that corresponds to the behavior in InlineTextBox. It also didn't properly pixel snap
the y offset and height.
(WebCore::RenderBlock::blockSelectionGap): Properly pixel snapping the gap rect before painting.
(WebCore::RenderBlock::logicalLeftSelectionGap): Correctly match InlineTextBox's selection pixel
alignment by flooring the logicalLeft and pixel snapping the subsequent rect. We do this in order
to maintain proper render tree pixel snapping while simultaneously mirroring the flooring in the
inline direction that occurs in InlineTextBox.cpp's alignSelectionRectToDevicePixels method.
(WebCore::RenderBlock::logicalRightSelectionGap): Ditto with the right.
LayoutTests:
One new selection test and updated snowleopard expectations for a handful that improved.
Note: a couple of the RTL results appear worse, but sadly are only returning to our old pre-sub-pixel results.
- fast/sub-pixel/selection: Added.
- fast/sub-pixel/selection/selection-gaps-at-fractional-offsets-expected.txt: Added.
- fast/sub-pixel/selection/selection-gaps-at-fractional-offsets.html: Added.
- platform/chromium-mac-snowleopard/editing/selection/3690703-2-expected.png:
- platform/chromium-mac-snowleopard/editing/selection/3690703-expected.png:
- platform/chromium-mac-snowleopard/editing/selection/3690719-expected.png:
- platform/chromium-mac-snowleopard/editing/selection/selection-button-text-expected.png:
- platform/chromium-mac-snowleopard/editing/selection/vertical-lr-ltr-extend-line-backward-br-expected.png:
- platform/chromium-mac-snowleopard/editing/selection/vertical-lr-ltr-extend-line-forward-br-expected.png:
- platform/chromium-mac-snowleopard/editing/selection/vertical-rl-ltr-extend-line-backward-br-expected.png:
- platform/chromium-mac-snowleopard/editing/selection/vertical-rl-ltr-extend-line-backward-p-expected.png:
- platform/chromium-mac-snowleopard/editing/selection/vertical-rl-ltr-extend-line-backward-wrap-expected.png:
- platform/chromium-mac-snowleopard/editing/selection/vertical-rl-ltr-extend-line-forward-br-expected.png:
- platform/chromium-mac-snowleopard/editing/selection/vertical-rl-ltr-extend-line-forward-p-expected.png:
- platform/chromium-mac-snowleopard/editing/selection/vertical-rl-ltr-extend-line-forward-wrap-expected.png:
- platform/chromium-mac-snowleopard/editing/selection/vertical-rl-rtl-extend-line-backward-br-expected.png:
- platform/chromium-mac-snowleopard/editing/selection/vertical-rl-rtl-extend-line-backward-p-expected.png:
- platform/chromium-mac-snowleopard/editing/selection/vertical-rl-rtl-extend-line-forward-br-expected.png:
- platform/chromium-mac-snowleopard/editing/selection/vertical-rl-rtl-extend-line-forward-p-expected.png:
- platform/chromium-mac-snowleopard/fast/sub-pixel: Added.
- platform/chromium-mac-snowleopard/fast/sub-pixel/selection: Added.
- platform/chromium-mac-snowleopard/fast/sub-pixel/selection/selection-gaps-at-fractional-offsets-expected.png: Added.
- platform/chromium-mac/fast/repaint/japanese-rl-selection-repaint-expected.png:
- platform/chromium-mac/fast/repaint/japanese-rl-selection-repaint-in-regions-expected.png:
- platform/chromium/TestExpectations:
- platform/efl/Skipped:
- platform/gtk-wk2/Skipped:
- platform/mac-lion/Skipped:
- platform/mac-snowleopard/Skipped:
- platform/mac-wk2/Skipped:
- platform/mac/Skipped:
- platform/qt-4.8/Skipped:
- platform/qt/Skipped:
- platform/win-wk2/Skipped:
- platform/win-xp/Skipped:
- platform/win/Skipped:
- platform/wincairo/Skipped:
- platform/wk2/Skipped:
- 2:36 PM Changeset in webkit [119527] by
-
- 10 edits12 deletes in trunk
Remove support for target-densitydpi in the viewport meta tag
https://bugs.webkit.org/show_bug.cgi?id=88047
Reviewed by Kenneth Rohde Christiansen.
Source/WebCore:
The target-densitydpi parameter was originally implemented on Android,
but it does not appear to be widely used and introduces significant
implementation complexity because it forces us to track three scale
factors (defaultDeviceScaleFactor, deviceScaleFactor, and
pageScaleFactor) rather than just two (deviceScaleFactor and
pageScaleFactor).
There don't appear to be many web sites that use target-densitydpi
because it is not supported on iOS. There are also concerns from the
standards community about the design of the feature. It seems our best
course of action is to remove target-densitydpi and address these use
cases via other mechanisms, such as responsive images and device units
in CSS because those approaches are likely to be implemented broadly.
- dom/ViewportArguments.cpp:
(WebCore::computeViewportAttributes):
(WebCore):
(WebCore::setViewportFeature):
(WebCore::viewportErrorMessageTemplate):
(WebCore::viewportErrorMessageLevel):
- dom/ViewportArguments.h:
(WebCore::ViewportArguments::ViewportArguments):
(ViewportArguments):
(WebCore::ViewportArguments::operator==):
Source/WebKit/blackberry:
- Api/WebPage.cpp:
(BlackBerry::WebKit::WebPagePrivate::recomputeVirtualViewportFromViewportArguments):
- Api/WebViewportArguments.cpp:
(BlackBerry::WebKit::WebViewportArguments::targetDensityDpi):
(BlackBerry::WebKit::WebViewportArguments::setTargetDensityDpi):
- Api/WebViewportArguments.h:
Source/WebKit/efl:
- ewk/ewk_view.cpp:
(_ewk_view_priv_new):
LayoutTests:
These tests are no longer needed because they're testing a feature we
no longer support.
- fast/viewport/viewport-133-expected.txt: Removed.
- fast/viewport/viewport-133.html: Removed.
- fast/viewport/viewport-92-expected.txt: Removed.
- fast/viewport/viewport-92.html: Removed.
- fast/viewport/viewport-93-expected.txt: Removed.
- fast/viewport/viewport-93.html: Removed.
- fast/viewport/viewport-94-expected.txt: Removed.
- fast/viewport/viewport-94.html: Removed.
- fast/viewport/viewport-95-expected.txt: Removed.
- fast/viewport/viewport-95.html: Removed.
- fast/viewport/viewport-96-expected.txt: Removed.
- fast/viewport/viewport-96.html: Removed.
- 2:32 PM Changeset in webkit [119526] by
-
- 2 edits in trunk/Source/JavaScriptCore
DFG CFG simplification should not attempt to deref nodes inside of an unreachable subgraph
https://bugs.webkit.org/show_bug.cgi?id=88362
Reviewed by Gavin Barraclough.
- dfg/DFGCFGSimplificationPhase.cpp:
(JSC::DFG::CFGSimplificationPhase::fixPhis):
(JSC::DFG::CFGSimplificationPhase::removePotentiallyDeadPhiReference):
- 2:31 PM Changeset in webkit [119525] by
-
- 6 edits2 deletes in branches/chromium/1132/Source
Revert 119486 - Merge 117978 - IndexedDB: Fire error when there are problems opening a DB
https://bugs.webkit.org/show_bug.cgi?id=85579
Source/WebCore:
We used to either fire success or get into an infinite loop.
Reviewed by Tony Chang.
New unit test in
Source/WebKit/chromium/tests/IDBAbortOnCorruptTest.cpp
- Modules/indexeddb/IDBDatabaseBackendImpl.cpp:
(WebCore::IDBDatabaseBackendImpl::IDBDatabaseBackendImpl):
(WebCore::IDBDatabaseBackendImpl::openInternal):
(WebCore::IDBDatabaseBackendImpl::openConnection):
- Modules/indexeddb/IDBDatabaseBackendImpl.h:
(WebCore::IDBDatabaseBackendImpl::create):
(IDBDatabaseBackendImpl):
- Modules/indexeddb/IDBFactoryBackendImpl.cpp:
(WebCore::IDBFactoryBackendImpl::deleteDatabase):
(WebCore::IDBFactoryBackendImpl::openInternal):
- Modules/indexeddb/IDBFactoryBackendImpl.h:
(IDBFactoryBackendImpl):
- Modules/indexeddb/IDBLevelDBBackingStore.h:
(IDBLevelDBBackingStore):
Source/WebKit/chromium:
Reviewed by Tony Chang.
- WebKit.gypi:
- tests/IDBAbortOnCorruptTest.cpp: Added.
(WebCore):
(MockIDBCallbacks):
(WebCore::MockIDBCallbacks::MockIDBCallbacks):
(WebCore::MockIDBCallbacks::~MockIDBCallbacks):
(WebCore::MockIDBCallbacks::onError):
(WebCore::MockIDBCallbacks::onSuccess):
(WebCore::MockIDBCallbacks::onSuccessWithContinuation):
(WebCore::MockIDBCallbacks::onSuccessWithPrefetch):
(WebCore::MockIDBCallbacks::onBlocked):
(FailingBackingStore):
(WebCore::FailingBackingStore::~FailingBackingStore):
(WebCore::FailingBackingStore::open):
(WebCore::FailingBackingStore::createIDBDatabaseMetaData):
(FailingIDBFactoryBackendImpl):
(WebCore::FailingIDBFactoryBackendImpl::~FailingIDBFactoryBackendImpl):
(WebCore::FailingIDBFactoryBackendImpl::create):
(WebCore::FailingIDBFactoryBackendImpl::removeIDBDatabaseBackend):
(WebCore::FailingIDBFactoryBackendImpl::openBackingStore):
(WebCore::TEST):
- tests/IDBFakeBackingStore.h: Copied from Source/WebCore/Modules/indexeddb/IDBLevelDBBackingStore.h.
(WebCore):
(IDBFakeBackingStore):
TBR=dgrogan@chromium.org
Review URL: https://chromiumcodereview.appspot.com/10536007
TBR=dgrogan@chromium.org
Review URL: https://chromiumcodereview.appspot.com/10534013
- 2:24 PM Changeset in webkit [119524] by
-
- 2 edits in trunk/LayoutTests
Fix Chromium test expectations.
- platform/chromium/TestExpectations:
- 2:16 PM Changeset in webkit [119523] by
-
- 2 edits2 adds in trunk/LayoutTests
Unreviewed chromium mac rebaseline for float-containing-block-with-margin.
- platform/chromium-mac-leopard/fast/sub-pixel: Added.
- platform/chromium-mac-leopard/fast/sub-pixel/float-containing-block-with-margin-expected.png: Added.
- platform/chromium-mac/fast/sub-pixel/float-containing-block-with-margin-expected.png:
- 1:54 PM Changeset in webkit [119522] by
-
- 2 edits in trunk/LayoutTests
Add Chromium test failure expectations per Skia roll:
http://src.chromium.org/viewvc/chrome?view=rev&revision=140580
- platform/chromium/TestExpectations:
- 1:53 PM Changeset in webkit [119521] by
-
- 2 edits in trunk/Tools
Buildbot fix.
- BuildSlaveSupport/build.webkit.org-config/master.cfg:
(loadBuilderConfig):
- 1:47 PM Changeset in webkit [119520] by
-
- 12 edits in trunk/Tools
webkitpy: clean up win-specific path handling and tests
https://bugs.webkit.org/show_bug.cgi?id=88281
Reviewed by Adam Barth.
There was a bunch of inconsistent logic for handling path
conversions for win32 and cygwin paths due to us sometimes
looking at sys.platform and sometimes using mock hosts. This
patch cleans everything up so that we are required to pass
PlatformInfo objects to the path module and stop trying to do
different things when running on cygwin or win32 hosts (except
in the path_unittest module itself).
This may slightly reduce test coverage for the win32 code paths
but will be a lot easier to follow and maintain.
- Scripts/webkitpy/common/system/path.py:
(abspath_to_uri):
(_convert_path):
- Scripts/webkitpy/common/system/path_unittest.py:
(AbspathTest.platforminfo):
(AbspathTest.test_abspath_to_uri_cygwin):
(AbspathTest.test_abspath_to_uri_unixy):
(AbspathTest.test_abspath_to_uri_win):
(AbspathTest.test_abspath_to_uri_escaping_unixy):
(AbspathTest.test_abspath_to_uri_escaping_cygwin):
(AbspathTest.test_stop_cygpath_subprocess):
- Scripts/webkitpy/common/system/platforminfo.py:
(PlatformInfo.init):
(PlatformInfo.is_cygwin):
- Scripts/webkitpy/common/system/platforminfo_mock.py:
(MockPlatformInfo.is_cygwin):
- Scripts/webkitpy/layout_tests/port/base.py:
(Port.to.show_results_html_file):
- Scripts/webkitpy/layout_tests/port/driver.py:
(Driver.test_to_uri):
(Driver.uri_to_test):
- Scripts/webkitpy/layout_tests/port/driver_unittest.py:
(DriverTest.test_test_to_uri):
(DriverTest.test_uri_to_test):
- Scripts/webkitpy/layout_tests/port/mock_drt_unittest.py:
(MockDRTPortTest.make_port):
(MockDRTTest.input_line):
(MockChromiumDRTTest.test_pixeltestfails):
- Scripts/webkitpy/layout_tests/port/test.py:
- Scripts/webkitpy/layout_tests/port/win.py:
(WinPort.show_results_html_file):
- Scripts/webkitpy/layout_tests/run_webkit_tests_integrationtest.py:
(MainTest.test_unexpected_failures):
(MainTest.test_results_directory_absolute):
(MainTest.test_results_directory_default):
(MainTest.test_results_directory_relative):
- 1:41 PM Changeset in webkit [119519] by
-
- 3 edits in trunk/Tools
Workaround buildbot bug when merging build requests.
https://bugs.webkit.org/show_bug.cgi?id=88158
Reviewed by Mark Rowe.
For expediency, we still want testers to pick the latest build to test, but to ensure
that the builders are reporting the correct revision number, we should let the building bots
run as they normally would. That is, the builder picks the oldest revision, and merges
subsequent requests. Because merged requests get appended to the list of changes for a given
build, the last change in that list is also the latest change. When we used pickLatestBuild
for the builders, this was not the case (it would always report the penultimate change).
http://trac.buildbot.net/ticket/2309 is tracking the buildbot bug.
- BuildSlaveSupport/build.webkit.org-config/config.json: Remove the mergeRequests override
for all Apple builders.
- BuildSlaveSupport/build.webkit.org-config/master.cfg: Have only non-building bots pick
the latest build to run.
(loadBuilderConfig):
- 1:38 PM Changeset in webkit [119518] by
-
- 5 edits in trunk/Source/JavaScriptCore
Entry into JSC should CRASH() if the Heap is busy
https://bugs.webkit.org/show_bug.cgi?id=88355
Reviewed by Geoffrey Garen.
Interpreter::execute() returns jsNull() right now if we try to enter it while
the Heap is busy (e.g. with a collection), which is okay, but some code paths
that call Interpreter::execute() allocate objects before checking if the Heap
is busy. Attempting to execute JS code while the Heap is busy should not be
allowed and should be enforced by a release-mode CRASH() to prevent vague,
unhelpful backtraces later on if somebody makes a mistake. Normally, recursively
executing JS code is okay, e.g. for evals, but it should not occur during a
Heap allocation or collection because the Heap is not guaranteed to be in a
consistent state (especially during collections). We are protected from
executing JS on the same Heap concurrently on two separate threads because
they must each take a JSLock first. However, we are not protected from reentrant
execution of JS on the same thread because JSLock allows reentrancy. Therefore,
we should fail early if we detect an entrance into JS code while the Heap is busy.
- heap/Heap.cpp: Changed Heap::collect so that it sets the m_operationInProgress field
at the beginning of collection and then unsets it at the end so that it is set at all
times throughout the duration of a collection rather than sporadically during various
phases. There is no reason to unset during a collection because our collector does
not currently support running additional JS between the phases of a collection.
(JSC::Heap::getConservativeRegisterRoots):
(JSC::Heap::markRoots):
(JSC::Heap::collect):
- interpreter/Interpreter.cpp:
(JSC::Interpreter::execute): Crash if the Heap is busy.
- runtime/Completion.cpp: Crash if the Heap is busy. We do it here before we call
Interpreter::execute() because we do some allocation prior to calling execute() which
could cause Heap corruption if, for example, that allocation caused a collection.
(JSC::evaluate):
- 1:21 PM NewRunWebKitTests edited by
- Removed link to HackingOnNewRunWebKitTests as it onlyhas a link back … (diff)
- 1:06 PM Changeset in webkit [119517] by
-
- 4 edits1 copy in trunk/Source/WebKit2
[Qt] Implement log channel initialization for WebKit2
Reviewed by Alexis Menard.
- 12:59 PM Changeset in webkit [119516] by
-
- 2 edits in trunk/Source/WebKit2
WebLayerTreeRenderer behaves wrongly when no contentsScale is set.
https://bugs.webkit.org/show_bug.cgi?id=88357
Patch by Jocelyn Turcotte <jocelyn.turcotte@nokia.com> on 2012-06-05
Reviewed by Alexis Menard.
The contentsScale needs to get a default value in case no
value is explicitely set yet.
This fixes the blank-until-resize problem when running MiniBrowser
in --desktop mode.
- UIProcess/WebLayerTreeRenderer.cpp:
(WebKit::WebLayerTreeRenderer::WebLayerTreeRenderer):
- 12:52 PM Changeset in webkit [119515] by
-
- 2 edits2 deletes in trunk/LayoutTests
Assertion failure on mac-lion-wk2 in fast/hidpi/device-scale-factor-paint.html
https://bugs.webkit.org/show_bug.cgi?id=88356
Unreviewed, remove incorrect baselines and skip test on Mac.
- platform/mac/Skipped:
- platform/mac/fast/hidpi/device-scale-factor-paint-expected.png: Removed.
- platform/mac/fast/hidpi/device-scale-factor-paint-expected.txt: Removed.
- 12:22 PM Changeset in webkit [119514] by
-
- 6 edits1 copy3 adds in trunk
[V8] Improve variable resolution order on window
https://bugs.webkit.org/show_bug.cgi?id=84247
Reviewed by Ojan Vafai.
This changes the V8 flag to turn on es52_globals and updates the layout tests to reflect the fixed behavior.
Source/WebCore:
Test: fast/dom/Window/window-property-shadowing-onclick.html
- bindings/v8/V8DOMWindowShell.cpp:
(WebCore::V8DOMWindowShell::initContextIfNeeded): Set the flag as we initialize V8.
- bindings/v8/WorkerContextExecutionProxy.cpp:
(WebCore::WorkerContextExecutionProxy::initIsolate): Ditto.
LayoutTests:
- fast/dom/Window/window-property-shadowing-name-expected.txt: The expectation is now reversed.
- fast/dom/Window/window-property-shadowing-name.html: JSC fails so this test says fail. Bug 75575 tracks JSC.
- fast/dom/Window/window-property-shadowing-onclick-expected.txt: Added.
- fast/dom/Window/window-property-shadowing-onclick.html: Added.
- platform/chromium/fast/dom/Window/window-property-shadowing-name-expected.txt: Copied from LayoutTests/fast/dom/Window/window-property-shadowing-name-expected.txt.
- platform/chromium/fast/dom/Window/window-property-shadowing-onclick-expected.txt: Added.
- 12:16 PM Changeset in webkit [119513] by
-
- 2 edits in trunk/Source/WebCore
Delete dead code from RenderBox::computeLogicalWidthInRegion
https://bugs.webkit.org/show_bug.cgi?id=88353
Reviewed by Tony Chang.
No new tests obviously since this is dead code.
- rendering/RenderBox.cpp:
(WebCore::RenderBox::computeLogicalWidthInRegion):
- 12:16 PM Changeset in webkit [119512] by
-
- 3 edits3 adds1 delete in trunk/LayoutTests
Change fast/sub-pixel/float-containing-block-with-margin.html to pixel test
https://bugs.webkit.org/show_bug.cgi?id=88351
Reviewed by Levi Weintraub.
Change fast/sub-pixel/float-containing-block-with-margin.html to pixel
test to allow for platform differences as the default styling makes it
very hard to use a reftests for this.
- fast/sub-pixel/float-containing-block-with-margin-expected.html: Removed.
- fast/sub-pixel/float-containing-block-with-margin-expected.png: Added.
- fast/sub-pixel/float-containing-block-with-margin-expected.txt: Added.
- fast/sub-pixel/float-containing-block-with-margin.html:
- platform/chromium-mac/fast/sub-pixel/float-containing-block-with-margin-expected.png: Added.
- platform/chromium/TestExpectations:
- 12:01 PM Changeset in webkit [119511] by
-
- 1 edit2 adds2 deletes in trunk/LayoutTests
Unreviewed gardening. Fixing improper text expectation following r119503.
- platform/chromium-mac/fast/line-grid/line-align-left-edges-expected.txt: Removed.
- platform/chromium-win/fast/line-grid/line-align-left-edges-expected.txt: Removed.
- platform/chromium/fast/line-grid: Added.
- platform/chromium/fast/line-grid/line-align-left-edges-expected.txt: Added.
- 11:53 AM Changeset in webkit [119510] by
-
- 8 edits2 copies in trunk/Source
[Chromium] Move createLocalStorageNamespace to Platform.h
https://bugs.webkit.org/show_bug.cgi?id=85766
Source/Platform:
Reviewed by James Robinson.
Part of a refactoring series. See tracking bug 82948.
- Platform.gypi:
- chromium/public/Platform.h:
(WebKit):
(Platform):
(WebKit::Platform::createLocalStorageNamespace):
- chromium/public/WebStorageArea.h: Added.
(WebKit):
(WebStorageArea):
(WebKit::WebStorageArea::~WebStorageArea):
(WebKit::WebStorageArea::setItem):
(WebKit::WebStorageArea::removeItem):
(WebKit::WebStorageArea::clear):
- chromium/public/WebStorageNamespace.h: Added.
(WebKit):
(WebStorageNamespace):
(WebKit::WebStorageNamespace::~WebStorageNamespace):
(WebKit::WebStorageNamespace::isSameNamespace):
Source/WebKit/chromium:
Reviewed by James Robinson.
Part of a refactoring series. See tracking bug 82948.
- WebKit.gyp:
- public/WebStorageArea.h:
- public/WebStorageNamespace.h:
- public/platform/WebKitPlatformSupport.h:
(WebKit):
(WebKitPlatformSupport):
- 11:53 AM Changeset in webkit [119509] by
-
- 5 edits4 adds in trunk/LayoutTests
Chromium rebaseline per Chromium r140536.
- platform/chromium-linux-x86/http/tests/media: Added.
- platform/chromium-linux-x86/http/tests/media/video-buffered-range-contains-currentTime-expected.png: Added.
- platform/chromium-linux/http/tests/media/video-buffered-range-contains-currentTime-expected.png:
- platform/chromium-mac-snowleopard/http/tests/media/video-buffered-range-contains-currentTime-expected.png:
- platform/chromium-mac/http/tests/media/video-buffered-range-contains-currentTime-expected.png:
- platform/chromium-win-vista/http/tests/media: Added.
- platform/chromium-win-vista/http/tests/media/video-buffered-range-contains-currentTime-expected.png: Added.
- platform/chromium-win/http/tests/media/video-buffered-range-contains-currentTime-expected.png:
- 11:39 AM Changeset in webkit [119508] by
-
- 10 edits in trunk
New constructor for WebIntent to be used for delivery
https://bugs.webkit.org/show_bug.cgi?id=87143
Patch by Greg Billock <gbillock@google.com> on 2012-06-05
Reviewed by Darin Fisher.
Source/WebCore:
- Modules/intents/Intent.cpp:
(WebCore::Intent::setExtras):
(WebCore):
- Modules/intents/Intent.h:
(Intent):
Source/WebKit/chromium:
When delivering an intent to webkit, the caller needs to be able to
provide the action, type, and data, and also extra data and ports.
- public/WebIntent.h:
(WebIntent):
- src/WebFrameImpl.cpp:
(WebKit::WebFrameImpl::deliverIntent):
- src/WebIntent.cpp:
(WebKit::WebIntent::WebIntent):
(WebKit):
(WebKit::WebIntent::reset):
(WebKit::WebIntent::messagePortChannelsRelease):
- 11:32 AM Changeset in webkit [119507] by
-
- 16 edits2 adds in trunk
Change overrideSizes to be content-box instead of border-box
https://bugs.webkit.org/show_bug.cgi?id=88116
Reviewed by Tony Chang.
Source/WebCore:
Previously, the code used border-box everywhere except for one place
where it used content-box when setting the override size. Conversely,
it subtracted the borderAndPaddingSize everywhere it grabbed the override
size except for one.
Instead, make the override size override the content-box size.
Converging on using the border-box size would be tricky given RenderTableCell's
tricky management of intrinsic padding.
There is no behavior change except in RenderBox::availableLogicalHeightUsing,
which fixes a table test.
- rendering/RenderBox.cpp:
(WebCore::RenderBox::setOverrideLogicalContentHeight):
(WebCore::RenderBox::setOverrideLogicalContentWidth):
(WebCore::RenderBox::overrideLogicalContentWidth):
(WebCore::RenderBox::overrideLogicalContentHeight):
(WebCore::RenderBox::computeLogicalWidthInRegion):
(WebCore::RenderBox::computeLogicalHeight):
(WebCore::RenderBox::computePercentageLogicalHeight):
(WebCore::RenderBox::availableLogicalHeightUsing):
We were incorrectly subtracting the borderAndPaddingLogicalWidth instead
of the borderAndPaddingLogicalHeight.
- rendering/RenderBox.h:
(RenderBox):
- rendering/RenderDeprecatedFlexibleBox.cpp:
(WebCore::RenderDeprecatedFlexibleBox::layoutHorizontalBox):
(WebCore::RenderDeprecatedFlexibleBox::layoutVerticalBox):
(WebCore::RenderDeprecatedFlexibleBox::applyLineClamp):
(WebCore::RenderDeprecatedFlexibleBox::allowedChildFlex):
- rendering/RenderFlexibleBox.cpp:
(WebCore::RenderFlexibleBox::computeAvailableFreeSpace):
(WebCore::RenderFlexibleBox::setLogicalOverrideSize):
(WebCore::RenderFlexibleBox::applyStretchAlignmentToChild):
- rendering/RenderTableCell.cpp:
(WebCore::RenderTableCell::setOverrideHeightFromRowHeight):
LayoutTests:
- fast/table/padding-height-and-override-height-expected.txt:
- fast/table/padding-height-and-override-height.html:
Tests the change in RenderBox::availableLogicalHeightUsing to correctly subtract the
borderAndPaddingLogicalHeight instead of the borderAndPaddingLogicalWidth.
- platform/chromium-linux/tables/mozilla/bugs/bug131020-expected.png:
- platform/chromium-win/tables/mozilla/bugs/bug131020-expected.txt:
The new results are more correct. The table cell had 2px too much padding-top and padding-bottom before.
- 11:28 AM Changeset in webkit [119506] by
-
- 2 edits in trunk/Source/WebCore
REGRESSION (r118197): PluginStrategy should be used even if ENABLE(NETSCAPE_PLUGIN_API) is false
https://bugs.webkit.org/show_bug.cgi?id=88296
Reviewed by Dan Bernstein.
refresh() and initPlugins() should call into PluginStrategy even if
ENABLE(NETSCAPE_PLUGIN_API) is false. Some ports support non-Netscape
plug-ins, so PluginStrategy should be consulted even if Netscape
plug-ins specifically are disabled.
Ports that support platform strategies but not Netscape plug-ins should
handle this inside their PluginStrategy rather than modifying generic
plug-in code.
- plugins/PluginData.cpp:
(WebCore::PluginData::refresh):
(WebCore::PluginData::initPlugins):
- 11:23 AM Changeset in webkit [119505] by
-
- 2 edits in trunk/Tools
[Win] DumpRenderTree should call resetInternalsObject
https://bugs.webkit.org/show_bug.cgi?id=88346
Reviewed by Simon Fraser.
Invoke it on Windows in the same place it is invoked on Mac.
- DumpRenderTree/win/DumpRenderTree.cpp:
(resetWebViewToConsistentStateBeforeTesting):
- 11:15 AM Changeset in webkit [119504] by
-
- 2 edits in trunk/LayoutTests
http/tests/media/remove-while-loading.html times out on mac-lion-wk2
https://bugs.webkit.org/show_bug.cgi?id=88348
Unreviewed skip of this test for greenness.
- platform/mac-wk2/Skipped:
- 11:13 AM Changeset in webkit [119503] by
-
- 43 edits4 adds3 deletes in trunk/LayoutTests
Unreviewed gardening. Updating test expectations after r119456.
- fast/line-grid/line-grid-contains-value-expected.txt: Added.
- platform/chromium-mac-leopard/css2.1/t100801-c544-valgn-03-d-agi-expected.png:
- platform/chromium-mac-leopard/fast/forms/001-expected.png:
- platform/chromium-mac-leopard/fast/inline/inline-box-background-expected.png:
- platform/chromium-mac-leopard/fast/inline/inline-box-background-long-image-expected.png:
- platform/chromium-mac-leopard/fast/inline/inline-box-background-repeat-x-expected.png:
- platform/chromium-mac-leopard/fast/inline/inline-box-background-repeat-y-expected.png:
- platform/chromium-mac-leopard/fast/replaced/width100percent-checkbox-expected.png:
- platform/chromium-mac-leopard/fast/replaced/width100percent-radio-expected.png:
- platform/chromium-mac-leopard/tables/mozilla/bugs/bug1318-expected.png:
- platform/chromium-mac-leopard/tables/mozilla/bugs/bug4527-expected.png:
- platform/chromium-mac-snowleopard/css2.1/t100801-c544-valgn-03-d-agi-expected.png:
- platform/chromium-mac-snowleopard/fast/forms/001-expected.png:
- platform/chromium-mac-snowleopard/fast/inline/inline-box-background-expected.png:
- platform/chromium-mac-snowleopard/fast/inline/inline-box-background-long-image-expected.png:
- platform/chromium-mac-snowleopard/fast/inline/inline-box-background-repeat-x-expected.png:
- platform/chromium-mac-snowleopard/fast/inline/inline-box-background-repeat-y-expected.png:
- platform/chromium-mac-snowleopard/fast/replaced/width100percent-checkbox-expected.png:
- platform/chromium-mac-snowleopard/fast/replaced/width100percent-radio-expected.png:
- platform/chromium-mac-snowleopard/tables/mozilla/bugs/bug1318-expected.png:
- platform/chromium-mac-snowleopard/tables/mozilla/bugs/bug4527-expected.png:
- platform/chromium-mac/css2.1/t100801-c544-valgn-03-d-agi-expected.png:
- platform/chromium-mac/css2.1/t100801-c544-valgn-03-d-agi-expected.txt:
- platform/chromium-mac/fast/forms/001-expected.png:
- platform/chromium-mac/fast/forms/001-expected.txt:
- platform/chromium-mac/fast/inline/inline-box-background-expected.png:
- platform/chromium-mac/fast/inline/inline-box-background-long-image-expected.png:
- platform/chromium-mac/fast/inline/inline-box-background-repeat-x-expected.png:
- platform/chromium-mac/fast/inline/inline-box-background-repeat-y-expected.png:
- platform/chromium-mac/fast/line-grid/line-align-left-edges-expected.png:
- platform/chromium-mac/fast/line-grid/line-align-left-edges-expected.txt: Added.
- platform/chromium-mac/fast/line-grid/line-grid-contains-value-expected.txt: Added.
- platform/chromium-mac/fast/replaced/width100percent-checkbox-expected.png:
- platform/chromium-mac/fast/replaced/width100percent-checkbox-expected.txt:
- platform/chromium-mac/fast/replaced/width100percent-radio-expected.png:
- platform/chromium-mac/fast/replaced/width100percent-radio-expected.txt:
- platform/chromium-mac/tables/mozilla/bugs/bug1318-expected.png:
- platform/chromium-mac/tables/mozilla/bugs/bug1318-expected.txt:
- platform/chromium-mac/tables/mozilla/bugs/bug4527-expected.png:
- platform/chromium-mac/tables/mozilla/bugs/bug4527-expected.txt:
- platform/chromium-win/css2.1/t100801-c544-valgn-03-d-agi-expected.png:
- platform/chromium-win/css2.1/t100801-c544-valgn-03-d-agi-expected.txt:
- platform/chromium-win/fast/line-grid/line-align-left-edges-expected.png:
- platform/chromium-win/fast/line-grid/line-align-left-edges-expected.txt: Added.
- platform/chromium-win/fast/line-grid/line-grid-contains-value-expected.png:
- platform/chromium/TestExpectations:
- platform/chromium/fast/line-grid: Removed.
- platform/chromium/fast/line-grid/line-align-left-edges-expected.txt: Removed.
- platform/efl/fast/line-grid/line-grid-contains-value-expected.txt: Removed.
- platform/gtk/fast/line-grid/line-grid-contains-value-expected.txt: Removed.
- 11:05 AM Changeset in webkit [119502] by
-
- 3 edits2 adds in trunk
Crash when modifying fixed-position elements in a detached frame
https://bugs.webkit.org/show_bug.cgi?id=88288
Patch by Douglas Stockwell <dstockwell@chromium.org> on 2012-06-05
Reviewed by James Robinson.
Source/WebCore:
Test: fast/block/positioning/fixed-position-detached-frame.html
- css/StyleResolver.cpp:
(WebCore::StyleResolver::collectMatchingRulesForList):
LayoutTests:
- fast/block/positioning/fixed-position-detached-frame-expected.txt: Added.
- fast/block/positioning/fixed-position-detached-frame.html: Added.
- 10:33 AM Changeset in webkit [119501] by
-
- 1 move in branches/old/subpixellayout
Moving the sub-pixel layout branch to branches/old since the feature has landed and development on the branch has ceased.
- 10:22 AM Changeset in webkit [119500] by
-
- 2 edits in trunk/Source/WebKit/chromium
Roll Chromium DEPS from r140492 to r140528.
- DEPS:
- 10:12 AM Changeset in webkit [119499] by
-
- 7 edits1 add in trunk/Source/WebCore
Web Inspector: render timeline paint category dark green
https://bugs.webkit.org/show_bug.cgi?id=88333
Reviewed by Pavel Feldman.
- Added a new, dark-green color for timeline events.
- WebCore.gypi: Added timelineBarDarkGreen.png
- inspector/front-end/Images/timelineBarDarkGreen.png: Added.
- inspector/front-end/Images/timelineCheckmarks.png: Added a dark-green checkbox.
- inspector/front-end/Images/timelineDots.png: Added a dark-green dot.
- inspector/front-end/TimelinePresentationModel.js:
(WebInspector.TimelinePresentationModel.categories):
- inspector/front-end/WebKit.qrc: Added timelineBarDarkGreen.png.
- inspector/front-end/timelinePanel.css:
(.timeline-category-statusbar-item.timeline-category-painting .timeline-category-checkbox):
(.timeline-category-painting .timeline-graph-bar):
(.popover .timeline-painting):
(.timeline-category-painting .timeline-tree-icon):
- 10:08 AM Changeset in webkit [119498] by
-
- 8 edits in trunk
Web Inspector: serialize edge counts instead of indexes in heap snapshot
https://bugs.webkit.org/show_bug.cgi?id=88324
The serialized node structure currently holds an index
of its first containment edge in the edges array.
The index can be quite big (up to 7 digits for large snapshots).
The patch changes the serialization format to pass
node containment edge count instead. For most nodes the count
is just a single digit number.
This reduces serialized snapshot size and therefore its transfer time.
Patch by Alexei Filippov <alexeif@chromium.org> on 2012-06-05
Reviewed by Yury Semikhatsky.
PerformanceTests:
- inspector/heap-snapshot-performance-test.js:
Source/WebCore:
- inspector/front-end/HeapSnapshot.js:
(WebInspector.HeapSnapshotNode.prototype._edgeIndexesStart):
(WebInspector.HeapSnapshotNode.prototype._edgeIndexesEnd):
(WebInspector.HeapSnapshotNode.prototype._ordinal):
(WebInspector.HeapSnapshotNodeIterator):
(WebInspector.HeapSnapshot.prototype._init):
(WebInspector.HeapSnapshot.prototype._buildEdgeIndexes):
(WebInspector.HeapSnapshot.prototype._buildRetainers):
(WebInspector.HeapSnapshot.prototype._bfs):
(WebInspector.HeapSnapshot.prototype._buildAggregates):
(WebInspector.HeapSnapshot.prototype._buildPostOrderIndex):
(WebInspector.HeapSnapshot.prototype._buildDominatorTree):
(WebInspector.HeapSnapshot.prototype._markPageOwnedNodes):
(WebInspector.HeapSnapshot.prototype._markQueriableHeapObjects):
LayoutTests:
- inspector/profiler/heap-snapshot-expected.txt:
- inspector/profiler/heap-snapshot-test.js:
(initialize_HeapSnapshotTest.InspectorTest.createHeapSnapshotMockObject):
(initialize_HeapSnapshotTest.InspectorTest.createHeapSnapshotMockRaw):
(initialize_HeapSnapshotTest.InspectorTest.createHeapSnapshotMockWithDOM):
(initialize_HeapSnapshotTest.):
- inspector/profiler/heap-snapshot.html:
- 10:01 AM WebKitGTK/WebKit2Roadmap edited by
- (diff)
- 9:30 AM Changeset in webkit [119497] by
-
- 2 edits in trunk/Source/WebKit2
[Qt] Don't propagate viewport size changes if the viewport is empty.
https://bugs.webkit.org/show_bug.cgi?id=88336
Reviewed by Tor Arne Vestbø.
Qt Quick sequentially set the x, y, width, height of the view. We really want to
start propagating further when the size of the viewport becomes valid, i.e. both
height and width are defined. In any case lower level classes are discarding an empty
size such as WebPage::sendViewportAttributesChanged() so it was pointless to send the size
through the IPC bus for nothing.
- UIProcess/API/qt/qquickwebview.cpp:
(QQuickWebViewLegacyPrivate::updateViewportSize):
- 9:23 AM Changeset in webkit [119496] by
-
- 4 edits in trunk/Source/WebKit/blackberry
[BlackBerry]Web Inspector highlight is slow
https://bugs.webkit.org/show_bug.cgi?id=88331
Patch by Konrad Piascik <kpiascik@rim.com> on 2012-06-05
Reviewed by Rob Buis.
Highlight is slow on large pages because of constant repaint calls.
Change to use the accelerated layer highlight all the time.
- Api/BackingStore.cpp:
(BlackBerry::WebKit::BackingStorePrivate::renderContents):
- WebCoreSupport/InspectorClientBlackBerry.cpp:
(WebCore::InspectorClientBlackBerry::highlight):
(WebCore::InspectorClientBlackBerry::hideHighlight):
- WebCoreSupport/InspectorOverlay.cpp:
(WebCore::InspectorOverlay::clear):
(WebCore::InspectorOverlay::update):
- 9:18 AM Changeset in webkit [119495] by
-
- 3 edits in trunk/Tools
[GTK] Fix two more memory leaks in DRT
https://bugs.webkit.org/show_bug.cgi?id=88255
Patch by Sudarsana Nagineni <sudarsana.nagineni@linux.intel.com> on 2012-06-05
Reviewed by Martin Robinson.
Fixed memory leaks in GTK's DRT code.
- DumpRenderTree/gtk/DumpRenderTree.cpp:
(dumpHistoryItem): webkit_web_history_item_get_target() returns a
g_strdup()'d string. So, use GOwnPtr to manage memory automatically.
- DumpRenderTree/gtk/PixelDumpSupportGtk.cpp:
(createBitmapContextFromWebView): Free the cairo surface.
- 9:04 AM Changeset in webkit [119494] by
-
- 5 edits1 add in trunk/Source/WebKit/chromium
[chromium] There is no way to retrieve composition character rectangle in WebKit/chromium
https://bugs.webkit.org/show_bug.cgi?id=87911
Patch by Seigo Nonaka <nona@chromium.org> on 2012-06-05
Reviewed by Ryosuke Niwa.
Add an API for retreieving each character bounds in composition text.
This API is necessary for implementing IMR_QUERYCHARPOSITION message in Windows.
The message is used by Japanese IME for showing their window at the correct position.
- public/WebWidget.h:
(WebWidget):
(WebKit::WebWidget::compositionCharacterBounds):
- src/WebViewImpl.cpp:
(WebKit::WebViewImpl::compositionCharacterBounds):
(WebKit):
- src/WebViewImpl.h:
(WebViewImpl):
- 8:47 AM Changeset in webkit [119493] by
-
- 5 edits in trunk/Source/WebKit2
[GTK] Add webkit_download_get_received_data_length to WebKit2 GTK+ API
https://bugs.webkit.org/show_bug.cgi?id=88323
Reviewed by Martin Robinson.
- UIProcess/API/gtk/WebKitDownload.cpp:
(webkit_download_get_received_data_length):
- UIProcess/API/gtk/WebKitDownload.h:
- UIProcess/API/gtk/docs/webkit2gtk-sections.txt:
- UIProcess/API/gtk/tests/TestDownloads.cpp:
- 8:37 AM Changeset in webkit [119492] by
-
- 5 edits6 adds in trunk
CSS 2.1 failure: border-conflict-element-021a
https://bugs.webkit.org/show_bug.cgi?id=86885
Patch by Arpita Bahuguna <arpitabahuguna@gmail.com> on 2012-06-05
Reviewed by Julien Chaffraix.
Source/WebCore:
When two adjacent table row groups (thead, tbody, tfoot) have the same border-width
and the same border-style in a 'border-collapse: collapse' table the color of the border
from the top-most table row wins.
Tests: fast/table/border-collapsing/adjacent-row-groups-multi.html
fast/table/border-collapsing/adjacent-row-groups.html
- rendering/RenderTable.cpp:
(WebCore::RenderTable::paintObject):
Currently each row group is sent for paint from top (firstChild) to bottom thereby causing
the borders of the bottom-most row group (which is painted last) to overlap over the
previous row-group. This breaks the precedence for collapsed borders between adjacent cells.
This fix thus reverses the order in which the row-groups/sections are passed for painting.
Additional change has been done to iterate over the RenderTableSections directly.
(WebCore):
(WebCore::RenderTable::bottomSection):
- rendering/RenderTable.h:
New function has been added to obtain the last section of the table.
(RenderTable):
- rendering/RenderTableSection.h:
(RenderTableSection):
RenderTableSection's paint() method has now been made public so as to make it accessible
from RenderTable.
LayoutTests:
- css2.1/20110323/border-conflict-element-021a-expected.html: Added.
- css2.1/20110323/border-conflict-element-021a.htm: Added.
Added ref test under the css2.1 test-suite for border-conflict-element-021a.
- fast/table/border-collapsing/adjacent-row-groups-expected.html: Added.
- fast/table/border-collapsing/adjacent-row-groups.html: Added.
Added ref test for verifying 'border-collapse: collapse' table rendering when two
adjacent table row-groups have the same border-width and the same border-style. In
such a case the color of the border from the top-most table row-group wins.
- fast/table/border-collapsing/adjacent-row-groups-multi.html: Added.
- fast/table/border-collapsing/adjacent-row-groups-multi-expected.html: Added.
Additional ref test added for verifying 'border-collapse: collapse' table rendering
for adjacent table row-groups when more than one table headers and/or footers are specified.
- 8:32 AM Changeset in webkit [119491] by
-
- 2 edits in trunk/LayoutTests
Chromium unreviewed sheriffing: enable blob tests that are passing since r119454.
- platform/chromium/TestExpectations:
fast/filesystem/file-writer-gc-blob.html
fast/filesystem/workers/file-writer-sync-truncate-extend.html
fast/filesystem/workers/file-writer-gc-blob.html
- 8:26 AM Changeset in webkit [119490] by
-
- 2 edits in trunk/Source/WebCore
[Qt][Win] Fix IDL dependency generation
https://bugs.webkit.org/show_bug.cgi?id=88304
Reviewed by Tor Arne Vestbø.
- DerivedSources.pri:
- 8:17 AM Changeset in webkit [119489] by
-
- 8 edits7 deletes in trunk
Unreviewed, rolling out r119467, r119471, and r119476.
http://trac.webkit.org/changeset/119467
http://trac.webkit.org/changeset/119471
http://trac.webkit.org/changeset/119476
https://bugs.webkit.org/show_bug.cgi?id=88332
it broke chromium-mac static initializers test.\ (Requested by
loislo on #webkit).
Patch by Sheriff Bot <webkit.review.bot@gmail.com> on 2012-06-05
Source/WebCore:
- WebCore.gyp/WebCore.gyp:
- WebCore.gypi:
- platform/graphics/FontPlatformData.h:
(FontPlatformData):
- platform/graphics/cocoa/FontPlatformDataCocoa.mm:
(WebCore::FontPlatformData::platformDataInit):
(WebCore::FontPlatformData::platformDataAssign):
- platform/graphics/harfbuzz/ng/HarfBuzzFace.cpp: Removed.
- platform/graphics/harfbuzz/ng/HarfBuzzFace.h: Removed.
- platform/graphics/harfbuzz/ng/HarfBuzzFaceCoreText.cpp: Removed.
- platform/graphics/harfbuzz/ng/HarfBuzzShaper.cpp: Removed.
- platform/graphics/harfbuzz/ng/HarfBuzzShaper.h: Removed.
- platform/graphics/mac/FontComplexTextMac.cpp:
(WebCore::Font::selectionRectForComplexText):
(WebCore::Font::drawComplexText):
(WebCore::Font::floatWidthForComplexText):
(WebCore::Font::offsetForPositionForComplexText):
LayoutTests:
- platform/chromium-mac/css3/font-feature-settings-rendering-expected.png: Removed.
- platform/chromium-mac/css3/font-feature-settings-rendering-expected.txt: Removed.
- platform/chromium/TestExpectations:
- 8:13 AM Changeset in webkit [119488] by
-
- 2 edits in trunk/Tools
[Qt] Building when a top directory is named Source
https://bugs.webkit.org/show_bug.cgi?id=77727
Patch by Allan Sandfeld Jensen <allan.jensen@nokia.com> on 2012-06-05
Reviewed by Tor Arne Vestbø.
Change regex to only replace the last instance of OUT_PWD.
- qmake/mkspecs/features/default_pre.prf:
- 8:01 AM Changeset in webkit [119487] by
-
- 2 edits in trunk/Source/WebCore
Web Inspector: SHIFT key as helper to color adjustment in color picker
https://bugs.webkit.org/show_bug.cgi?id=88319
Reviewed by Pavel Feldman.
With Shift depressed, the dragger in the color picker will snap to the horizontal
or vertical axis while dragging, depending on which axis offset is greater.
- inspector/front-end/Spectrum.js:
(WebInspector.Spectrum.colorDragStart):
(WebInspector.Spectrum.colorDrag):
(WebInspector.Spectrum.draggable.move):
(WebInspector.Spectrum.draggable.start):
- 7:51 AM Changeset in webkit [119486] by
-
- 6 edits2 copies in branches/chromium/1132/Source
Merge 117978 - IndexedDB: Fire error when there are problems opening a DB
https://bugs.webkit.org/show_bug.cgi?id=85579
Source/WebCore:
We used to either fire success or get into an infinite loop.
Reviewed by Tony Chang.
New unit test in
Source/WebKit/chromium/tests/IDBAbortOnCorruptTest.cpp
- Modules/indexeddb/IDBDatabaseBackendImpl.cpp:
(WebCore::IDBDatabaseBackendImpl::IDBDatabaseBackendImpl):
(WebCore::IDBDatabaseBackendImpl::openInternal):
(WebCore::IDBDatabaseBackendImpl::openConnection):
- Modules/indexeddb/IDBDatabaseBackendImpl.h:
(WebCore::IDBDatabaseBackendImpl::create):
(IDBDatabaseBackendImpl):
- Modules/indexeddb/IDBFactoryBackendImpl.cpp:
(WebCore::IDBFactoryBackendImpl::deleteDatabase):
(WebCore::IDBFactoryBackendImpl::openInternal):
- Modules/indexeddb/IDBFactoryBackendImpl.h:
(IDBFactoryBackendImpl):
- Modules/indexeddb/IDBLevelDBBackingStore.h:
(IDBLevelDBBackingStore):
Source/WebKit/chromium:
Reviewed by Tony Chang.
- WebKit.gypi:
- tests/IDBAbortOnCorruptTest.cpp: Added.
(WebCore):
(MockIDBCallbacks):
(WebCore::MockIDBCallbacks::MockIDBCallbacks):
(WebCore::MockIDBCallbacks::~MockIDBCallbacks):
(WebCore::MockIDBCallbacks::onError):
(WebCore::MockIDBCallbacks::onSuccess):
(WebCore::MockIDBCallbacks::onSuccessWithContinuation):
(WebCore::MockIDBCallbacks::onSuccessWithPrefetch):
(WebCore::MockIDBCallbacks::onBlocked):
(FailingBackingStore):
(WebCore::FailingBackingStore::~FailingBackingStore):
(WebCore::FailingBackingStore::open):
(WebCore::FailingBackingStore::createIDBDatabaseMetaData):
(FailingIDBFactoryBackendImpl):
(WebCore::FailingIDBFactoryBackendImpl::~FailingIDBFactoryBackendImpl):
(WebCore::FailingIDBFactoryBackendImpl::create):
(WebCore::FailingIDBFactoryBackendImpl::removeIDBDatabaseBackend):
(WebCore::FailingIDBFactoryBackendImpl::openBackingStore):
(WebCore::TEST):
- tests/IDBFakeBackingStore.h: Copied from Source/WebCore/Modules/indexeddb/IDBLevelDBBackingStore.h.
(WebCore):
(IDBFakeBackingStore):
TBR=dgrogan@chromium.org
Review URL: https://chromiumcodereview.appspot.com/10536007
- 7:50 AM Changeset in webkit [119485] by
-
- 2 edits in trunk/Source/WebCore
Web Inspector: aggregate all events before first frame into a fake frame
https://bugs.webkit.org/show_bug.cgi?id=88229
- in Timeline's frame mode, start aggregating events by frame even before we get first frame marker.
- inspector/front-end/TimelineFrameController.js:
(WebInspector.TimelineFrameController.prototype._addRecord):
(WebInspector.TimelineFrameController.prototype._flushFrame):
(WebInspector.TimelineFrameController.prototype._createFrame):
- 5:27 AM Changeset in webkit [119484] by
-
- 5 edits in trunk/Source/WebCore
JSC:need to implement Dictionary::getWithUndefinedOrNullCheck for IDB
https://bugs.webkit.org/show_bug.cgi?id=88283
Reviewed by Kentaro Hara.
No new tests, idb not working for JSC yet.
- bindings/js/Dictionary.cpp:
(WebCore::Dictionary::getWithUndefinedOrNullCheck):
- bindings/js/Dictionary.h:
(Dictionary):
- bindings/js/JSDictionary.cpp:
(WebCore::JSDictionary::getWithUndefinedOrNullCheck):
(WebCore):
- bindings/js/JSDictionary.h:
(JSDictionary):
- 4:37 AM FeatureFlags edited by
- Add CUSTOM_SCHEME_HANDLER (diff)
- 4:35 AM Changeset in webkit [119483] by
-
- 1 edit2 adds in trunk/LayoutTests
Hit ASSERT when editing attribute value in Element in SVG Document
https://bugs.webkit.org/show_bug.cgi?id=88179
Reviewed by Pavel Feldman.
Test setting attribute on non HTML element.
- inspector/elements/set-attribute-non-html-expected.txt: Added.
- inspector/elements/set-attribute-non-html.svg: Added.
- 4:32 AM Changeset in webkit [119482] by
-
- 30 edits2 adds in trunk
Add 'isProtocolHandlerRegistered' and 'unregisterProtocolHandler'.
https://bugs.webkit.org/show_bug.cgi?id=73176
Patch by Dongwoo Im <dw.im@samsung.com> on 2012-06-05
Reviewed by Adam Barth.
Two more APIs are added in Custom Scheme Handler specification.
http://dev.w3.org/html5/spec/Overview.html#custom-handlers
One is 'isProtocolHandlerRegistered' to query whether the specific URL
is registered or not.
The other is 'unregisterProtocolHandler' to remove the registered URL.
.:
- Source/cmake/WebKitFeatures.cmake: Add a macro 'ENABLE_CUSTOM_SCHEME_HANDLER'.
- Source/cmakeconfig.h.cmake: Add a macro 'ENABLE_CUSTOM_SCHEME_HANDLER'.
Source/JavaScriptCore:
- Configurations/FeatureDefines.xcconfig: Add a macro 'ENABLE_CUSTOM_SCHEME_HANDLER'.
Source/WebCore:
Test: fast/dom/unregister-protocol-handler.html
- Configurations/FeatureDefines.xcconfig: Add a macro 'ENABLE_CUSTOM_SCHEME_HANDLER'.
- loader/EmptyClients.h: Add prototypes.
(EmptyChromeClient):
(WebCore::EmptyChromeClient::isProtocolHandlerRegistered):
(WebCore::EmptyChromeClient::unregisterProtocolHandler):
- page/ChromeClient.h:
(ChromeClient):
- page/NavigatorRegisterProtocolHandler.cpp:
(WebCore):
(WebCore::customHandlersStateString):
(WebCore::NavigatorRegisterProtocolHandler::isProtocolHandlerRegistered): Query if the handler is registered.
(WebCore::NavigatorRegisterProtocolHandler::unregisterProtocolHandler): Unregister the registered handler.
- page/NavigatorRegisterProtocolHandler.h:
(NavigatorRegisterProtocolHandler):
- page/NavigatorRegisterProtocolHandler.idl:
Source/WebKit/chromium:
- features.gypi: Add a macro 'ENABLE_CUSTOM_SCHEME_HANDLER'.
Source/WebKit/efl:
- WebCoreSupport/ChromeClientEfl.cpp:
(WebCore::customHandlerDataCreate):
(WebCore::customHandlerDataDelete):
(WebCore::ChromeClientEfl::registerProtocolHandler):
(WebCore):
(WebCore::ChromeClientEfl::isProtocolHandlerRegistered): Query if the handler is registered.
(WebCore::ChromeClientEfl::unregisterProtocolHandler): Unregister the registered handler.
- WebCoreSupport/ChromeClientEfl.h:
(ChromeClientEfl):
- ewk/ewk_custom_handler.cpp:
(ewk_custom_handler_is_protocol_handler_registered): Query if the handler is registered.
(ewk_custom_handler_unregister_protocol_handler): Unregister the registered handler.
- ewk/ewk_custom_handler_private.h: Adds private function.
- ewk/ewk_view.h: Adds enumeration.
Source/WebKit/mac:
- Configurations/FeatureDefines.xcconfig: Add a macro 'ENABLE_CUSTOM_SCHEME_HANDLER'.
Source/WebKit2:
- Configurations/FeatureDefines.xcconfig: Add a macro 'ENABLE_CUSTOM_SCHEME_HANDLER'.
Tools:
- Scripts/webkitperl/FeatureList.pm: Add a macro 'ENABLE_CUSTOM_SCHEME_HANDLER'.
WebKitLibraries:
- win/tools/vsprops/FeatureDefines.vsprops: Add a macro 'ENABLE_CUSTOM_SCHEME_HANDLER'.
- win/tools/vsprops/FeatureDefinesCairo.vsprops: Add a macro 'ENABLE_CUSTOM_SCHEME_HANDLER'.
LayoutTests:
- fast/dom/unregister-protocol-handler-expected.txt: Added.
- fast/dom/unregister-protocol-handler.html: Added.
- 4:28 AM Changeset in webkit [119481] by
-
- 5 edits in trunk/LayoutTests
https://bugs.webkit.org/show_bug.cgi?id=88029
content-element-api.html and shadow-element.html depend on V8.
Reviewed by Kentaro Hara.
Replaced typeof checking with comparison.
- fast/dom/shadow/content-element-api.html:
- fast/dom/shadow/shadow-element.html:
- 4:13 AM Changeset in webkit [119480] by
-
- 2 edits in trunk/Source/WebKit2
[EFL][WK2] WebKit2-EFL does not link with gold linker
https://bugs.webkit.org/show_bug.cgi?id=88295
Unreviewed. Fix linking when using gold linker.
Patch by Christophe Dumez <Christophe Dumez> on 2012-06-05
- PlatformEfl.cmake:
- 3:57 AM Changeset in webkit [119479] by
-
- 4 edits in trunk
Chromium tests: "Add support for the Blob constructor" [r115582] regressed blob layout tests
https://bugs.webkit.org/show_bug.cgi?id=85174
Reviewed by Kentaro Hara.
Source/WebCore:
This fixes a crash problem which could happen when the constructor is
given an array which contains String-type item(s).
There're still some Text mismatches between v8 results and
JSC results, most of them are v8 not throwing exception
when it is given an object whose toString() method throws
exception. (The issue will be addresse in a separate patch.)
Test: fast/files/blob-constructor.html
- bindings/v8/custom/V8BlobCustom.cpp:
(WebCore::V8Blob::constructorCallback):
LayoutTests:
Changing blob-constructor.html label from CRASH to TEXT.
- platform/chromium/TestExpectations:
- 3:39 AM Changeset in webkit [119478] by
-
- 2 edits in trunk/Source/WebCore
Remove unnecessary constructor of SelectorQueryCache
https://bugs.webkit.org/show_bug.cgi?id=87942
Reviewed by Ryosuke Niwa.
Per the comment from darin@ (https://bugs.webkit.org/show_bug.cgi?id=87942#c27),
this patch removes an inline constructor that has no arguments.
To have the compiler generate the constructor,
this patch also removes WTF_MAKE_NONCOPYABLE().
No tests. No change in behavior.
- dom/SelectorQuery.h:
(SelectorQueryCache):
- 3:25 AM Changeset in webkit [119477] by
-
- 3 edits in trunk/Source/WebCore
Remove unused methods from SelectorQuery.h
https://bugs.webkit.org/show_bug.cgi?id=88289
Reviewed by Antti Koivisto.
This patch removes SelectorDataList::size() and
SelectorDataList::SelectorDataList().
SelectorDataList::size() is not used by anybody.
SelectorDataList::SelectorDataList() can be auto-generated
by a compiler.
No tests. No change in behavior.
- dom/SelectorQuery.cpp:
- dom/SelectorQuery.h:
(SelectorDataList):
- 2:55 AM Changeset in webkit [119476] by
-
- 2 edits in trunk/LayoutTests
[Chromium] Unreviewed test expectations update
- platform/chromium/TestExpectations:
- 2:47 AM WebKitGTK/WebKit2Roadmap edited by
- (diff)
- 2:46 AM Changeset in webkit [119475] by
-
- 10 edits3 adds in trunk
[GTK][WK2] Implement API for Geolocation permission requests in the GTK port
https://bugs.webkit.org/show_bug.cgi?id=83879
Reviewed by Carlos Garcia Campos.
Source/WebKit2:
Added a new kind of permission request for supporting the
Geolocation API in WebKit2GTK+.
New WebKitGeolocationPermissionRequest class, implementing the
WebKitPermissionRequest interface, to enabling client applications
to allow or deny geolocation permission requests.
- GNUmakefile.list.am: Added new files.
- UIProcess/API/gtk/WebKitGeolocationPermissionRequest.cpp: Added.
(_WebKitGeolocationPermissionRequestPrivate):
(webkitGeolocationPermissionRequestAllow):
(webkitGeolocationPermissionRequestDeny):
(webkit_permission_request_interface_init):
(webkit_geolocation_permission_request_init):
(webkitGeolocationPermissionRequestFinalize):
(webkit_geolocation_permission_request_class_init):
(webkitGeolocationPermissionRequestCreate):
- UIProcess/API/gtk/WebKitGeolocationPermissionRequest.h: Added.
(_WebKitGeolocationPermissionRequest):
(_WebKitGeolocationPermissionRequestClass):
- UIProcess/API/gtk/WebKitGeolocationPermissionRequestPrivate.h: Added.
Include WKGeolocationPermissionRequest.h in WebKitPrivate.h, so we
can use it from the new WebKitGeolocationPermissionRequest class.
- UIProcess/API/gtk/WebKitPrivate.h: Added new include.
Implement geolocation permission requests in the UI client.
- UIProcess/API/gtk/WebKitUIClient.cpp:
(decidePolicyForGeolocationPermissionRequest): Create a new
instance of the WebKitGeolocationPermissionRequest class and call
to webkitWebViewMakePermissionRequest().
(attachUIClientToView): Provide an implementation function for
decidePolicyForGeolocationPermissionRequest.
- UIProcess/API/gtk/WebKitWebView.h: Add new element to
WebKitPermissionRequestType enumeration for geolocation requests.
- UIProcess/API/gtk/webkit2.h: Added header file for
WebKitGeolocationPermissionRequest
New unit tests to test allowing and denying permission requests.
- UIProcess/API/gtk/tests/TestWebKitWebView.cpp:
(testWebViewPermissionRequests): New test that loads a simple HTML
with JavaScript code requesting the current location.
(beforeAll): Added new test.
Updated documentation files.
- UIProcess/API/gtk/docs/webkit2gtk-docs.sgml:
- UIProcess/API/gtk/docs/webkit2gtk-sections.txt:
Tools:
Make minibrowser connect to the new 'permission requests' signal
to allow users handle the Geolocation permission requests.
- MiniBrowser/gtk/BrowserWindow.c:
(geolocationRequestDialogCallback): Callback for the dialog asking
the user for allowing or not geolocation permission requests.
(webViewDecidePermissionRequest): Launch a dialog to ask the user
whether to allow or deny geolocation permission requests.
(browserWindowConstructed): Connect to 'permission-request' signal.
- 2:37 AM Changeset in webkit [119474] by
-
- 2 edits in trunk/Source/WebCore
Build fix for r119470.
- loader/EmptyClients.cpp: Add #include "IntentRequest.h".
- 2:34 AM Changeset in webkit [119473] by
-
- 2 edits in trunk/Source/WebCore
Remove unused function: Document::formElements
https://bugs.webkit.org/show_bug.cgi?id=88272
Reviewed by Kentaro Hara.
No new tests. No behavior change.
- dom/Document.h:
(Document): remove formElements(). It was used for radio group
validation, but the current code doesn't use it any more.
- 2:33 AM Changeset in webkit [119472] by
-
- 3 edits in trunk/LayoutTests
Unreviewed, build fix after r119469.
- inspector/styles/lazy-computed-style-expected.txt:
- inspector/styles/styles-computed-trace-expected.txt:
- 2:28 AM Changeset in webkit [119471] by
-
- 2 edits in trunk/Source/WebCore
Unreviewed build fix on Chromium mac
- platform/graphics/cocoa/FontPlatformDataCocoa.mm:
(WebCore::isAATFont):
- 2:08 AM Changeset in webkit [119470] by
-
- 7 edits in trunk/Source
Move some function definitions in EmptyClients.h to EmptyClients.cpp
https://bugs.webkit.org/show_bug.cgi?id=88285
Reviewed by Ryosuke Niwa.
Source/WebCore:
If we have definitions of functions with RefPtr<> in EmptyClients.h like
virtual void runOpenPanel(Frame*, PassRefPtr<FileChooser>) { }
and a source file includes EmptyClient.h, the source file needs to
include FileChooser.h though the source file doesn't use FileChooser
class explicitly because the definition of runOpenPanel() needs to call
FileChooser::deref().
To avoid this, we had better move such functions to EmptyClients.cpp.
No new tests. No behavior changes.
- loader/EmptyClients.cpp: Move such functions from EmptyClients.h to
here, and move EmptyPopupMenu and EmptySearchPopupMenu too because
they're used only by EmptyChromeClient.
(WebCore):
(EmptyPopupMenu):
(WebCore::EmptyPopupMenu::show):
(WebCore::EmptyPopupMenu::hide):
(WebCore::EmptyPopupMenu::updateFromElement):
(WebCore::EmptyPopupMenu::disconnectClient):
(EmptySearchPopupMenu):
(WebCore::EmptySearchPopupMenu::popupMenu):
(WebCore::EmptySearchPopupMenu::saveRecentSearches):
(WebCore::EmptySearchPopupMenu::loadRecentSearches):
(WebCore::EmptySearchPopupMenu::enabled):
(WebCore::EmptyChromeClient::createPopupMenu):
(WebCore::EmptyChromeClient::createSearchPopupMenu):
(WebCore::EmptyChromeClient::createColorChooser):
(WebCore::EmptyChromeClient::runOpenPanel):
(WebCore::EmptyFrameLoaderClient::dispatchDecidePolicyForNewWindowAction):
(WebCore::EmptyFrameLoaderClient::dispatchDecidePolicyForNavigationAction):
(WebCore::EmptyFrameLoaderClient::dispatchWillSendSubmitEvent):
(WebCore::EmptyFrameLoaderClient::dispatchWillSubmitForm):
(WebCore::EmptyFrameLoaderClient::createDocumentLoader):
(WebCore::EmptyFrameLoaderClient::createFrame):
(WebCore::EmptyFrameLoaderClient::createPlugin):
(WebCore::EmptyFrameLoaderClient::createJavaAppletWidget):
(WebCore::EmptyFrameLoaderClient::createMediaPlayerProxyPlugin):
(WebCore::EmptyFrameLoaderClient::createNetworkingContext):
(WebCore::EmptyFrameLoaderClient::dispatchIntent):
(WebCore::EmptyTextCheckerClient::requestCheckingOfString):
(WebCore::EmptyEditorClient::registerUndoStep):
(WebCore::EmptyEditorClient::registerRedoStep):
(WebCore::EmptyContextMenuClient::customizeMenu):
- loader/EmptyClients.h: Move such function definitions to
EmptyClient.cpp, and append OVERRIDE.
(EmptyChromeClient):
(EmptyFrameLoaderClient):
(EmptyTextCheckerClient):
(EmptyEditorClient):
(EmptyContextMenuClient):
- svg/graphics/SVGImage.cpp: Remove unnecessary #includes.
Source/WebKit/chromium:
- src/WebHelperPluginImpl.cpp: Remove unnecessary #includes, and add necessary #includes.
- src/WebPagePopupImpl.cpp: ditto.
- 1:56 AM Changeset in webkit [119469] by
-
- 5 edits in trunk
Web Inspector: "-webkit"-prefixed properties are considered inherited in the Computed Style pane
https://bugs.webkit.org/show_bug.cgi?id=88257
Reviewed by Pavel Feldman.
Source/WebCore:
Canonical names for properties should be used when checking if they are inherited (since only canonical names
are put into the usedProperties set).
- inspector/front-end/StylesSidebarPane.js:
(WebInspector.ComputedStylePropertiesSection.prototype._isPropertyInherited):
LayoutTests:
- inspector/elements/elements-panel-styles-expected.txt:
- inspector/elements/elements-panel-styles.html:
- 1:27 AM Changeset in webkit [119468] by
-
- 2 edits in trunk/Source/WebKit/chromium
Unreviewed. Rolled DEPS.
- DEPS:
- 1:15 AM Changeset in webkit [119467] by
-
- 8 edits8 adds in trunk
[Chromium] Implement font shaping with font-feature-settings on Mac
https://bugs.webkit.org/show_bug.cgi?id=69826
Reviewed by Tony Chang.
Source/WebCore:
Add HarfBuzz-ng text shaper.
Chromium mac port uses it as secondary text shaper to support OpenType features.
HarfBuzz-ng is only used when -webkit-font-feature-settings is specified and
corresponding font is not an AAT font.
No new tests. css3/font-feature-settings-rendering.html should pass on Chromium mac port.
- WebCore.gyp/WebCore.gyp: Added harfbuzz-ng dependencies.
- WebCore.gypi: Added harfbuzz-ng files.
- platform/graphics/FontPlatformData.h:
(FontPlatformData): Added m_harfbuzzFace.
- platform/graphics/cocoa/FontPlatformDataCocoa.mm:
(WebCore::FontPlatformData::platformDataInit): Copy m_harfbuzzFace.
(WebCore::FontPlatformData::platformDataAssign): Ditto.
(WebCore):
(WebCore::isAATFont):
(WebCore::FontPlatformData::harfbuzzFace):
- platform/graphics/harfbuzz/ng/HarfBuzzFace.cpp: Added.
(WebCore):
(WebCore::harfbuzzFaceCache):
(WebCore::HarfBuzzFace::HarfBuzzFace):
(WebCore::HarfBuzzFace::~HarfBuzzFace):
- platform/graphics/harfbuzz/ng/HarfBuzzFace.h: Added.
(WebCore):
(HarfBuzzFace):
(WebCore::HarfBuzzFace::create):
- platform/graphics/harfbuzz/ng/HarfBuzzFaceCoreText.cpp: Added.
(WebCore):
(WebCore::floatToHarfBuzzPosition):
(WebCore::getGlyph):
(WebCore::getGlyphHorizontalAdvance):
(WebCore::getGlyphHorizontalOrigin):
(WebCore::getGlyphExtents):
(WebCore::harfbuzzCoreTextGetFontFuncs):
(WebCore::releaseTableData):
(WebCore::harfbuzzCoreTextGetTable):
(WebCore::HarfBuzzFace::createFace):
(WebCore::HarfBuzzFace::createFont):
(WebCore::HarfBuzzShaper::createGlyphBufferAdvance):
- platform/graphics/harfbuzz/ng/HarfBuzzShaper.cpp: Added.
(WebCore):
(WebCore::harfbuzzPositionToFloat):
(WebCore::HarfBuzzShaper::HarfBuzzRun::HarfBuzzRun):
(WebCore::HarfBuzzShaper::HarfBuzzRun::setGlyphAndPositions):
(WebCore::HarfBuzzShaper::HarfBuzzRun::characterIndexForXPosition):
(WebCore::HarfBuzzShaper::HarfBuzzRun::xPositionForOffset):
(WebCore::HarfBuzzShaper::HarfBuzzShaper):
(WebCore::HarfBuzzShaper::~HarfBuzzShaper):
(WebCore::HarfBuzzShaper::setFontFeatures):
(WebCore::HarfBuzzShaper::shape):
(WebCore::HarfBuzzShaper::setupHarfBuzzRun):
(WebCore::HarfBuzzShaper::shapeHarfBuzzRun):
(WebCore::HarfBuzzShaper::setGlyphPositionsForHarfBuzzRun):
(WebCore::HarfBuzzShaper::offsetForPosition):
(WebCore::HarfBuzzShaper::selectionRect):
- platform/graphics/harfbuzz/ng/HarfBuzzShaper.h: Added.
(WebCore):
(HarfBuzzShaper):
(WebCore::HarfBuzzShaper::totalWidth):
(HarfBuzzRun):
(WebCore::HarfBuzzShaper::HarfBuzzRun::create):
(WebCore::HarfBuzzShaper::HarfBuzzRun::setWidth):
(WebCore::HarfBuzzShaper::HarfBuzzRun::numCharacters):
(WebCore::HarfBuzzShaper::HarfBuzzRun::numGlyphs):
(WebCore::HarfBuzzShaper::HarfBuzzRun::width):
(WebCore::HarfBuzzShaper::HarfBuzzRun::rtl):
- platform/graphics/mac/FontComplexTextMac.cpp:
(WebCore):
(WebCore::preferHarfBuzz): Added.
(WebCore::Font::selectionRectForComplexText): Use HarfBuzzShaper if font-feature-settings exists.
(WebCore::Font::drawComplexText): Ditto.
(WebCore::Font::floatWidthForComplexText): Ditto.
(WebCore::Font::offsetForPositionForComplexText): Ditto.
LayoutTests:
Added the expectation for sss3/font-feature-settings-rendering.html.
- platform/chromium-mac/css3/font-feature-settings-rendering-expected.png: Added.
- platform/chromium-mac/css3/font-feature-settings-rendering-expected.txt: Added.
- platform/chromium/TestExpectations: Removed css3/font-feature-settings-rendering.html.
- 1:12 AM Changeset in webkit [119466] by
-
- 5 edits in trunk
IETC: FileList.item(-1) should return null instead of raising
https://bugs.webkit.org/show_bug.cgi?id=77899
Patch by Li Yin <li.yin@intel.com> on 2012-06-05
Reviewed by Kentaro Hara.
Source/WebCore:
IETC: http://samples.msdn.microsoft.com/ietestcenter/fileapi/filelist.htm
Accroding to http://es5.github.com/#x9.6
When the index is negative, it should not raise exception.
Firefox and IE returns null when there is not indexth File object.
Test: fast/files/file-list-test.html
- fileapi/FileList.idl:
LayoutTests:
IETC: http://samples.msdn.microsoft.com/ietestcenter/fileapi/filelist.htm
Ref: http://es5.github.com/#x9.6
Some index values have been covered in this test, including negative index,
undefined, null, normal index(0~length-1), more than length index.
- fast/files/file-list-test-expected.txt:
- fast/files/file-list-test.html:
- 12:38 AM Changeset in webkit [119465] by
-
- 12 edits2 adds in trunk
EventHandler shouldn't dispatch fake mousemove events when scrolling on devices that don't have a mouse
https://bugs.webkit.org/show_bug.cgi?id=88270
Reviewed by James Robinson.
Source/WebCore:
This patch adds a setting analogous to deviceSupportsTouch to determine
whether the device supports mouse. We then don't dispatch fake mouse
events on devices that don't have a mouse in the first place.
Test: fast/events/touch/scroll-without-mouse-lacks-mousemove-events.html
- page/EventHandler.cpp:
(WebCore::EventHandler::fakeMouseMoveEventTimerFired):
- page/Settings.cpp:
(WebCore::Settings::Settings):
- page/Settings.h:
(WebCore::Settings::setDeviceSupportsMouse):
(WebCore::Settings::deviceSupportsMouse):
(Settings):
- testing/InternalSettings.cpp:
(WebCore::InternalSettings::InternalSettings):
(WebCore::InternalSettings::restoreTo):
(WebCore::InternalSettings::setDeviceSupportsMouse):
(WebCore):
- testing/InternalSettings.h:
(InternalSettings):
- testing/InternalSettings.idl:
Source/WebKit/chromium:
- public/WebSettings.h:
- src/WebSettingsImpl.cpp:
(WebKit::WebSettingsImpl::setDeviceSupportsMouse):
(WebKit):
- src/WebSettingsImpl.h:
(WebSettingsImpl):
LayoutTests:
- fast/events/touch/scroll-without-mouse-lacks-mousemove-events-expected.txt: Added.
- fast/events/touch/scroll-without-mouse-lacks-mousemove-events.html: Added.
Jun 4, 2012:
- 11:40 PM Changeset in webkit [119464] by
-
- 3 edits3 adds in trunk
DFG CFG simplification should correct the variables at the head of the predecessor block
https://bugs.webkit.org/show_bug.cgi?id=88284
Reviewed by Geoffrey Garen.
Source/JavaScriptCore:
- dfg/DFGCFGSimplificationPhase.cpp:
(JSC::DFG::CFGSimplificationPhase::mergeBlocks):
LayoutTests:
- fast/js/dfg-constant-fold-first-local-read-after-block-merge-expected.txt: Added.
- fast/js/dfg-constant-fold-first-local-read-after-block-merge.html: Added.
- fast/js/script-tests/dfg-constant-fold-first-local-read-after-block-merge.js: Added.
(foo):
(bar):
- 11:38 PM Changeset in webkit [119463] by
-
- 4 edits in trunk/Tools
Make a builder group support+expect multiple loads.
https://bugs.webkit.org/show_bug.cgi?id=88260
Change BuilderGroup to allow expecting multiple
loads. This allows merging results for a given set
of builders into one group.
Add a test that verifies that the group count
increments correctly based on successful and failed
list loads.
Patch by Chase Phillips <cmp@google.com> on 2012-06-04
Reviewed by Ojan Vafai.
- TestResultServer/static-dashboards/builders.js:
(BuilderGroup):
(BuilderGroup.prototype.setbuilder):
(BuilderGroup.prototype.append):
(BuilderGroup.prototype.loaded):
(requestBuilderList.xhr.onload):
(requestBuilderList.xhr.onerror):
(onErrorLoadingBuilderList):
(loadBuildersList):
- TestResultServer/static-dashboards/flakiness_dashboard_unittests.js:
(test):
- TestResultServer/static-dashboards/run-unittests.html:
- 11:07 PM Changeset in webkit [119462] by
-
- 2 edits1 delete in trunk/LayoutTests
Unreviewed chromium windows rebaseline for r119456.
- platform/chromium-win-vista/fast/multicol: Removed.
- platform/chromium-win/fast/multicol/float-multicol-expected.png:
- 10:44 PM Changeset in webkit [119461] by
-
- 2 edits2 adds in trunk/LayoutTests
Unreviewed chromium test expectation updates for new tests.
- fast/sub-pixel/inline-block-with-padding-expected.txt:
- platform/chromium-mac/fast/sub-pixel: Added.
- platform/chromium-mac/fast/sub-pixel/inline-block-with-padding-expected.txt: Added.
- 10:33 PM Changeset in webkit [119460] by
-
- 2 edits in trunk/LayoutTests
[Chromium] Fix a TestExpectations format error.
- platform/chromium/TestExpectations:
- 10:20 PM Changeset in webkit [119459] by
-
- 4 edits in trunk/LayoutTests
Unreviewed chromium test and expectation updates for r119456.
- fast/sub-pixel/inline-block-with-padding.html:
- platform/chromium-mac-snowleopard/fast/events/offsetX-offsetY-expected.txt:
- platform/chromium/TestExpectations:
- 10:06 PM LayoutUnit edited by
- Updated status (diff)
- 10:01 PM Changeset in webkit [119458] by
-
- 3 edits in trunk/Source/WebCore
Leaking ClipRects
https://bugs.webkit.org/show_bug.cgi?id=88282
Reviewed by Dan Bernstein.
In r118562 I made the ClipRectsCache use RefPtr<ClipRects>. However, ClipRects
was initialized with m_refCnt=0, not 1 as adoptRef() and friends expect. Also,
there was a manual ref() in RenderLayer::updateClipRects() which this patch removes.
- rendering/RenderLayer.cpp:
(WebCore::RenderLayer::updateClipRects):
- rendering/RenderLayer.h:
(WebCore::ClipRects::ClipRects):
- 9:49 PM Changeset in webkit [119457] by
-
- 2 edits in trunk/Tools
Python test fix attempt for Chromium Windows.
- Scripts/webkitpy/layout_tests/port/driver.py:
(Driver.uri_to_test):
- 9:42 PM Changeset in webkit [119456] by
-
- 31 edits6 adds in trunk
Add missing FractionalLayoutUnit += operator and move LineWidth to use all floats
https://bugs.webkit.org/show_bug.cgi?id=88259
Patch by Emil A Eklund <eae@chromium.org> and Levi Weintraub <leviw@chromium.org> on 2012-06-04
Reviewed by Ryosuke Niwa.
Source/WebCore:
Fix two rounding bugs in LineLayout and RenderBlock that caused inlines
and floats to wrap incorrectly.
Tests: fast/sub-pixel/float-containing-block-with-margin.html
fast/sub-pixel/float-with-right-margin-zoom.html
fast/sub-pixel/inline-block-with-padding.html
- platform/FractionalLayoutUnit.h:
(WebCore::operator+=):
Add missing float version of += operator thus avoiding a silent cast to
int loosing precision.
- rendering/RenderBlockLineLayout.cpp:
(WebCore::LineWidth::computeAvailableWidthFromLeftAndRight):
(WebCore::LineWidth::updateAvailableWidth):
Change LineWidth left/right to floating point and instead of rounding the
left and right edge separately.
- rendering/RenderFlexibleBox.cpp:
(WebCore::RenderFlexibleBox::resolveFlexibleLengths):
Use explicit cast to avoid compiler ambiguity warning.
LayoutTests:
- fast/dom/elementFromPoint-relative-to-viewport.html:
- fast/events/offsetX-offsetY.html:
Remove special casing for subpixel layout as it is no longer needed.
- fast/sub-pixel/float-containing-block-with-margin-expected.html: Added.
- fast/sub-pixel/float-containing-block-with-margin.html: Added.
- fast/sub-pixel/float-with-right-margin-zoom-expected.txt: Added.
- fast/sub-pixel/float-with-right-margin-zoom.html: Added.
- fast/sub-pixel/inline-block-with-padding-expected.txt: Added.
- fast/sub-pixel/inline-block-with-padding.html: Added.
New tests for wrapping logic.
- platform/chromium-linux/css2.1/t100801-c544-valgn-03-d-agi-expected.png:
- platform/chromium-linux/css2.1/t100801-c544-valgn-03-d-agi-expected.txt:
- platform/chromium-linux/fast/line-grid/line-align-left-edges-expected.png:
- platform/chromium-linux/fast/line-grid/line-grid-contains-value-expected.png:
- platform/chromium-linux/fast/multicol/float-multicol-expected.png:
- platform/chromium-win/fast/events/offsetX-offsetY-expected.txt:
- platform/chromium-win/fast/line-grid/line-grid-contains-value-expected.txt:
- platform/chromium-win/fast/multicol/float-multicol-expected.txt:
Update expectations to match pre-subpixel rendering.
- platform/chromium/TestExpectations:
Mark updated tests as failing on mac and windows temporarily.
- platform/chromium/fast/dom/elementFromPoint-relative-to-viewport-expected.txt:
Remove special casing for subpixel layout as it is no longer needed.
- platform/chromium/fast/line-grid/line-align-left-edges-expected.txt:
Update expectations to match pre-subpixel rendering.
- platform/efl/Skipped:
- platform/gtk-wk2/Skipped:
- platform/mac-lion/Skipped:
- platform/mac-snowleopard/Skipped:
- platform/mac-wk2/Skipped:
- platform/mac/Skipped:
- platform/qt-4.8/Skipped:
- platform/qt/Skipped:
- platform/win-wk2/Skipped:
- platform/win-xp/Skipped:
- platform/win/Skipped:
- platform/wincairo/Skipped:
- platform/wk2/Skipped:
Skip new sub-pixel tests on platforms that does not enable sub-pixel layout.
- 9:38 PM Changeset in webkit [119455] by
-
- 14 edits in trunk
CSS property 'order' should be a number rather than an int
https://bugs.webkit.org/show_bug.cgi?id=88111
Reviewed by Ojan Vafai.
Source/WebCore:
This changed in the spec:
http://dev.w3.org/csswg/css3-flexbox/#order
I also fixed the compile with CSS3_FLEXBOX disabled.
No new tests, covered by existing tests.
- css/CSSComputedStyleDeclaration.cpp:
(WebCore):
(WebCore::CSSComputedStyleDeclaration::getPropertyCSSValue):
- css/CSSParser.cpp:
(WebCore::CSSParser::parseValue):
- css/CSSProperty.cpp: Parse as float, remove clamping for HashMap<int>.
(WebCore::CSSProperty::isInheritedProperty):
- css/StyleBuilder.cpp:
(WebCore::StyleBuilder::StyleBuilder):
- css/StyleResolver.cpp:
(WebCore::StyleResolver::collectMatchingRulesForList):
- rendering/RenderFlexibleBox.cpp: Remove traits for HashMap<int>.
(RenderFlexibleBox::OrderIterator):
- rendering/RenderFlexibleBox.h:
- rendering/style/RenderStyle.h:
- rendering/style/StyleRareNonInheritedData.h:
(StyleRareNonInheritedData):
LayoutTests:
- css3/flexbox/css-properties-expected.txt:
- css3/flexbox/css-properties.html: Allow decimal values.
- css3/flexbox/flex-order.html: Fix tests based on float limits.
- 9:29 PM Changeset in webkit [119454] by
-
- 2 edits in trunk/LayoutTests
Syntax error in fast/filesystem/resources/file-writer-gc-blob.js
https://bugs.webkit.org/show_bug.cgi?id=87939
Reviewed by Alexey Proskuryakov.
- fast/filesystem/resources/file-writer-gc-blob.js: Add parentheses around the blob data.
(startWrite):
- 9:11 PM FeatureFlags edited by
- Add LEGACY_VIEWPORT_ADAPTION (diff)
- 8:17 PM Changeset in webkit [119453] by
-
- 9 edits in trunk/Source/JavaScriptCore
Unreviewed.
Rolled out r119364 because it's still causing crashes (when running
v8-earley in release builds of DRT)
This time for sure!
- heap/Heap.cpp:
(JSC::Heap::collect):
- heap/MarkedBlock.cpp:
(JSC::MarkedBlock::sweep):
- heap/MarkedBlock.h:
(JSC::MarkedBlock::resetAllocator):
(JSC):
- heap/MarkedSpace.cpp:
(JSC::ResetAllocator::operator()):
(JSC):
(JSC::MarkedSpace::resetAllocators):
(JSC::MarkedSpace::sweepWeakSets):
- heap/MarkedSpace.h:
(MarkedSpace):
- heap/WeakBlock.cpp:
(JSC::WeakBlock::sweep):
- heap/WeakSet.cpp:
(JSC::WeakSet::sweep):
(JSC::WeakSet::tryFindAllocator):
- heap/WeakSet.h:
(JSC::WeakSet::shrink):
- 8:01 PM Changeset in webkit [119452] by
-
- 8 edits in trunk
Turn LEGACY_VIEWPORT_ADAPTION USE flag into an ENABLE flag.
https://bugs.webkit.org/show_bug.cgi?id=88243
Patch by Hugo Parente Lima <Hugo Parente Lima> on 2012-06-04
Reviewed by Adam Barth.
.:
- Source/cmake/WebKitFeatures.cmake:
Source/WebCore:
No new tests needed, this just rename a compiler flag.
- dom/Document.cpp:
(WebCore::Document::setDocType):
- dom/ViewportArguments.h:
- html/HTMLMetaElement.cpp:
(WebCore::HTMLMetaElement::process):
Tools:
- qmake/mkspecs/features/features.prf:
- 7:13 PM Changeset in webkit [119451] by
-
- 2 edits in trunk/Tools
webkitpy.layout_tests.port.server_process_unittest.TestServerProcess.test_basic is flaky
https://bugs.webkit.org/show_bug.cgi?id=88280
Unreviewed, build fix.
- Scripts/webkitpy/layout_tests/port/server_process_unittest.py:
(TestServerProcess.test_basic):
- 7:05 PM Changeset in webkit [119450] by
-
- 2 edits in trunk/LayoutTests
Skip more tests on Windows to get the bots greener.
See r119449 for more context.
- platform/win/Skipped:
- 6:10 PM Changeset in webkit [119449] by
-
- 2 edits in trunk/LayoutTests
Start skipping all the failing tests on Windows to get the bots greener.
At this point it doesn't make sense to try to split the hundreds of failures out into
separate bugs and rebaseline. It makes more sense to use the Skipped list as a todo list
of things that need to be checked / updated for Windows.
- 6:10 PM Changeset in webkit [119448] by
-
- 1 edit in branches/chromium/1132/Source/WebCore/css/SelectorChecker.cpp
Merge 118891 - REGRESSION(r111497): The "option" element doesn't match CSS pseudo class :enabled
https://bugs.webkit.org/show_bug.cgi?id=87719
Reviewed by Kent Tamura.
Source/WebCore:
This patch added checking of "option" element for CSS pseudo class :enabled as same as
:disabled to selector checker. Before r111497, it was done by using isFormControlElement.
After that revision, HTMLOptionElement was no longer derived from HTMLFormControlElement.
Test: fast/form/select/optgroup-rendering.html
- css/SelectorChecker.cpp:
(WebCore::SelectorChecker::checkOneSelector): Checking element is option element as same
as PseudoDisabled in PseudoEnabled case.
LayoutTests:
- platform/chromium-linux/fast/forms/select/optgroup-rendering-expected.png: Updated. Text color of option element is bule from CSS pseudo class option:enabled.
- platform/chromium/test_expectations.txt: Update comment.
TBR=yosin@chromium.org
Review URL: https://chromiumcodereview.appspot.com/10477022
- 6:07 PM Changeset in webkit [119447] by
-
- 4 edits in trunk/Source
[chromium] Remove redundant setNeedsCommit when prepareToDraw fails
https://bugs.webkit.org/show_bug.cgi?id=88246
Reviewed by James Robinson.
Source/WebCore:
When prepareToDraw fails it explicitly calls setNeedsCommit, but the
scheduler already takes care of this for it. When a frame is not
successfully drawn, the scheduler state machine sets the needsCommit
bit itself.
This behaviour is covered by CCSchedulerStateMachineTest.TestFailedDrawSetsNeedsCommitAndDoesNotDrawAgain
- platform/graphics/chromium/cc/CCLayerTreeHostImpl.cpp:
(WebCore::CCLayerTreeHostImpl::prepareToDraw):
Source/WebKit/chromium:
- tests/CCLayerTreeHostImplTest.cpp:
- 5:51 PM Changeset in webkit [119446] by
-
- 4 edits1 add in trunk/Source
[WebSocket] Send requires super linear time against data size
https://bugs.webkit.org/show_bug.cgi?id=87383
Reviewed by Kent Tamura.
Source/WebCore:
WebSocket send operation requires super linear time to send data
against data size. This is because removing sent data in Vector<char>
always requires data copy.
Introduced wtf/StreamBuffer realizes O(n) buffer operations.
This patch is only for optimization. No new tests are needed.
- platform/network/SocketStreamHandleBase.cpp:
(WebCore::SocketStreamHandleBase::sendPendingData):
- platform/network/SocketStreamHandleBase.h:
(SocketStreamHandleBase):
Source/WTF:
- wtf/StreamBuffer.h: Added.
(WTF):
(StreamBuffer):
(WTF::StreamBuffer::StreamBuffer):
(WTF::StreamBuffer::~StreamBuffer):
(WTF::StreamBuffer::isEmpty):
(WTF::StreamBuffer::append):
(WTF::StreamBuffer::consume):
(WTF::StreamBuffer::size):
(WTF::StreamBuffer::firstBlockData):
(WTF::StreamBuffer::firstBlockSize):
- 5:37 PM Changeset in webkit [119445] by
-
- 3 edits in trunk/Tools
test-webkitpy is hanging under cygwin
https://bugs.webkit.org/show_bug.cgi?id=88269
Reviewed by Ryosuke Niwa.
The 'multiprocessing' module doesn't work correctly under Cygwin yet,
so we need to skip any tests that use it.
- Scripts/webkitpy/common/system/executive.py:
(Executive.run_in_parallel):
- Scripts/webkitpy/common/system/executive_unittest.py:
(ExecutiveTest.test_run_in_parallel):
- 5:28 PM Changeset in webkit [119444] by
-
- 2 edits in trunk/Source/JavaScriptCore
DFG arguments simplification should have rationalized handling of TearOffArguments
https://bugs.webkit.org/show_bug.cgi?id=88206
Reviewed by Geoffrey Garen.
- Accesses to the unmodified arguments register ought to have the same effect on alias/escape analysis of arguments as accesses to the mutable arguments register.
- The existence of TearOffArguments should not get in the way of arguments aliasing.
- TearOffArguments should be eliminated if CreateArguments is eliminated.
- dfg/DFGArgumentsSimplificationPhase.cpp:
(JSC::DFG::ArgumentsSimplificationPhase::run):
(JSC::DFG::ArgumentsSimplificationPhase::observeBadArgumentsUse):
- 5:27 PM Changeset in webkit [119443] by
-
- 4 edits1 add in trunk/Tools
test-webkitpy is failing on win32 after r115054
https://bugs.webkit.org/show_bug.cgi?id=88264
Reviewed by Ryosuke Niwa.
The failing http_server test was improperly suppressed (the
tuple was missing a comma, and so wasn't a tuple at all, just a
string) and so we were skipping *all* tests on win32. The right
way to skip a single test is to disable it inside the test
itself.
- Scripts/webkitpy/layout_tests/servers/http_server_unittest.py:
(TestHttpServer.test_start_cmd):
- Scripts/webkitpy/test/test_finder.py:
(TestFinder._default_names):
- Scripts/webkitpy/test/main.py:
(Tester.run): Add error message if we don't find any tests to run.
- Scripts/webkitpy/test/main_unittest.py:
- 5:21 PM Changeset in webkit [119442] by
-
- 1 edit2 adds in trunk/LayoutTests
Unreviewed new baseline for test added in r119423.
- platform/mac/fast/hidpi/device-scale-factor-paint-expected.png: Added.
- platform/mac/fast/hidpi/device-scale-factor-paint-expected.txt: Added.
- 5:00 PM Changeset in webkit [119441] by
-
- 14 edits in trunk/Source/JavaScriptCore
Remove enabledProfilerReference
https://bugs.webkit.org/show_bug.cgi?id=88258
Reviewed by Michael Saboff.
Make the enabled profiler a member of JSGlobalData, and switch code that accesses it to do so directly
via the JSGlobalData, rather than holding a Profiler reference to it. Do not pass the Profiler
reference to JIT code. This patch does not change the stack layout on entry into JIT code (passing an
unused void* instead), since this is an intrusive change better handled in a separate patch.
- interpreter/Interpreter.cpp:
(JSC::Interpreter::throwException):
(JSC::Interpreter::execute):
(JSC::Interpreter::executeCall):
(JSC::Interpreter::executeConstruct):
(JSC::Interpreter::privateExecute):
- jit/JITCode.h:
(JSC::JITCode::execute):
- Don't pass Profiler to JIT code.
- jit/JITOpcodes.cpp:
(JSC::JIT::emit_op_profile_will_call):
(JSC::JIT::emit_op_profile_did_call):
- jit/JITOpcodes32_64.cpp:
(JSC::JIT::emit_op_profile_will_call):
(JSC::JIT::emit_op_profile_did_call):
- jit/JITStubs.cpp:
(JSC):
(JSC::ctiTrampoline):
(JSC::ctiVMThrowTrampoline):
(JSC::ctiOpThrowNotCaught):
(JSC::JITThunks::JITThunks):
(JSC::DEFINE_STUB_FUNCTION):
- For ARM_THUMB2, rename ENABLE_PROFILER_REFERENCE_OFFSET to FIRST_STACK_ARGUMENT (which is how it is being used).
- For MIPS, remove ENABLE_PROFILER_REFERENCE_OFFSET.
- jit/JITStubs.h:
(JITStackFrame):
(JSC):
- Renamed enabledProfilerReference to unusedX.
- llint/LLIntSlowPaths.cpp:
(JSC::LLInt::LLINT_SLOW_PATH_DECL):
- llint/LowLevelInterpreter.asm:
- profiler/Profiler.cpp:
(JSC):
(JSC::Profiler::startProfiling):
(JSC::Profiler::stopProfiling):
- profiler/Profiler.h:
(Profiler):
- Removed s_sharedEnabledProfilerReference, enabledProfilerReference().
- runtime/JSGlobalData.cpp:
(JSC::JSGlobalData::JSGlobalData):
- runtime/JSGlobalData.h:
(JSC):
(JSC::JSGlobalData::enabledProfiler):
(JSGlobalData):
- Added m_enabledProfiler, enabledProfiler().
- runtime/JSGlobalObject.cpp:
(JSC::JSGlobalObject::~JSGlobalObject):
- 4:45 PM Changeset in webkit [119440] by
-
- 2 edits in trunk/Source/WebCore
Mock scrollbars differ by 1px in DRT vs. WKTR
https://bugs.webkit.org/show_bug.cgi?id=80879
Rubber-stamped by Ryosuke Niwa.
Disable r119431 except for Apple Mac port.
- platform/ScrollbarThemeComposite.cpp:
(WebCore::ScrollbarThemeComposite::thumbLength):
- 4:38 PM Changeset in webkit [119439] by
-
- 2 edits in trunk/Source/WebCore
Document cleanup can get confused if parser still exists
https://bugs.webkit.org/show_bug.cgi?id=88250
Reviewed by Geoffrey Garen.
No new tests; no behavior changes.
- dom/Document.cpp:
(WebCore::Document::removedLastRef): Detach parser earlier
- 4:32 PM Changeset in webkit [119438] by
-
- 3 edits in trunk/Source/JavaScriptCore
get_argument_by_val should be profiled everywhere
https://bugs.webkit.org/show_bug.cgi?id=88205
Reviewed by Geoffrey Garen.
- jit/JITOpcodes32_64.cpp:
(JSC::JIT::emitSlow_op_get_argument_by_val):
- llint/LLIntSlowPaths.cpp:
(JSC::LLInt::LLINT_SLOW_PATH_DECL):
- 4:27 PM Changeset in webkit [119437] by
-
- 6 edits12 adds in trunk
DFG arguments simplification takes unkindly to direct accesses to the arguments register
https://bugs.webkit.org/show_bug.cgi?id=88261
Source/JavaScriptCore:
Reviewed by Geoffrey Garen.
Fixed arguments simplification for direct accesses to the arguments register, which may
arise if CSE had not run. Fixed CSE so that it does run prior to arguments simplification,
by making it a full-fledged member of the fixpoint. Fixed other issues in arguments
simplification, like realizing that it needs to bail if there is a direct assignment to
the arguments register, and failing to turn CreateArguments into PhantomArguments. Also
fixed CSE's handling of store elimination of captured locals in the presence of a
GetMyArgumentByVal (or one of its friends), and fixed CSE to correctly fixup variables at
tail if the Flush it removes is the last operation on a local in a basic block.
- bytecode/CodeBlock.cpp:
(JSC::CodeBlock::dump):
- dfg/DFGArgumentsSimplificationPhase.cpp:
(JSC::DFG::ArgumentsSimplificationPhase::run):
(JSC::DFG::ArgumentsSimplificationPhase::isOKToOptimize):
- dfg/DFGCSEPhase.cpp:
(JSC::DFG::CSEPhase::run):
(JSC::DFG::CSEPhase::setLocalStoreElimination):
(JSC::DFG::CSEPhase::performNodeCSE):
(CSEPhase):
- dfg/DFGDriver.cpp:
(JSC::DFG::compile):
LayoutTests:
Reviewed by Geoffrey Garen.
- fast/js/dfg-arguments-alias-one-block-expected.txt: Added.
- fast/js/dfg-arguments-alias-one-block-osr-exit-expected.txt: Added.
- fast/js/dfg-arguments-alias-one-block-osr-exit.html: Added.
- fast/js/dfg-arguments-alias-one-block-overwrite-arguments-expected.txt: Added.
- fast/js/dfg-arguments-alias-one-block-overwrite-arguments.html: Added.
- fast/js/dfg-arguments-alias-one-block-overwrite-expected.txt: Added.
- fast/js/dfg-arguments-alias-one-block-overwrite.html: Added.
- fast/js/dfg-arguments-alias-one-block.html: Added.
- fast/js/script-tests/dfg-arguments-alias-one-block-osr-exit.js: Added.
(foo):
- fast/js/script-tests/dfg-arguments-alias-one-block-overwrite-arguments.js: Added.
(foo):
- fast/js/script-tests/dfg-arguments-alias-one-block-overwrite.js: Added.
(foo):
- fast/js/script-tests/dfg-arguments-alias-one-block.js: Added.
(foo):
- 4:25 PM Changeset in webkit [119436] by
-
- 4 edits in trunk/Source
[BlackBerry] Split AnimationFrameRateController into its own file
https://bugs.webkit.org/show_bug.cgi?id=88242
Patch by Andrew Lo <anlo@rim.com> on 2012-06-04
Reviewed by Antonio Gomes.
Include new header file to reflect re-organization of platform animation
header files.
Source/WebCore:
- platform/graphics/DisplayRefreshMonitor.h:
Source/WebKit/blackberry:
- Api/WebPageCompositor_p.h:
- 4:19 PM Changeset in webkit [119435] by
-
- 11 edits in trunk/Source
[chromium] Allow CCLayerImpl to find its layer tree host, and use this for CCVideoLayerImpl instead of always-null pointer.
https://bugs.webkit.org/show_bug.cgi?id=88252
Reviewed by James Robinson.
Source/WebCore:
CCVideoLayerImpl has a pointer to its CCLayerTreeHostImpl which is never
set to be non-null currently. Instead of making this a special case, we
move the CCLayerTreeHostImpl* up to the CCLayerImpl base class, and set
the pointer on every layer during commit via the TreeSynchronizer's walk
through the layer tree.
Unit tests: Tested by TreeSynchronizer tests.
- platform/graphics/chromium/TreeSynchronizer.cpp:
(WebCore::TreeSynchronizer::synchronizeTrees):
(WebCore::TreeSynchronizer::synchronizeTreeRecursive):
- platform/graphics/chromium/TreeSynchronizer.h:
(WebCore):
(TreeSynchronizer):
- platform/graphics/chromium/cc/CCLayerImpl.cpp:
(WebCore::CCLayerImpl::CCLayerImpl):
- platform/graphics/chromium/cc/CCLayerImpl.h:
(WebCore):
(WebCore::CCLayerImpl::layerTreeHostImpl):
(WebCore::CCLayerImpl::setLayerTreeHostImpl):
(CCLayerImpl):
- platform/graphics/chromium/cc/CCLayerTreeHost.cpp:
(WebCore::CCLayerTreeHost::finishCommitOnImplThread):
- platform/graphics/chromium/cc/CCVideoLayerImpl.cpp:
(WebCore::CCVideoLayerImpl::CCVideoLayerImpl):
(WebCore::CCVideoLayerImpl::setNeedsRedraw):
- platform/graphics/chromium/cc/CCVideoLayerImpl.h:
Source/WebKit/chromium:
- tests/ScrollbarLayerChromiumTest.cpp:
(WebCore::TEST):
- tests/TreeSynchronizerTest.cpp:
(WebKitTests::expectTreesAreIdentical):
(WebKitTests::TEST):
- 4:03 PM Changeset in webkit [119434] by
-
- 2 edits in trunk/Source/WebKit/chromium
yet another build fix attempt.
- src/WebHelperPluginImpl.cpp:
- 3:52 PM Changeset in webkit [119433] by
-
- 2 edits in trunk/Source/WebKit/chromium
Another Chromium Windows build fix attempt after r119411.
- src/WebHelperPluginImpl.cpp:
- 3:26 PM Changeset in webkit [119432] by
-
- 2 edits in trunk/Source/WebKit/chromium
[chromium] CCDamageTrackerTest.verifyDamageForPerspectiveClippedLayer needs to be cleaner
https://bugs.webkit.org/show_bug.cgi?id=85245
Reviewed by James Robinson.
Updated CCDamageTrackerTest.verifyDamageForPerspectiveClippedLayer
so that the test is clearer and cleaner. The original test was
covering what it needed, but in a confusing and not-so-practical
way. This patch adds comments and performs a tighter test so that
the intent is a bit more clear.
- tests/CCDamageTrackerTest.cpp:
(WebKitTests::TEST_F):
- 3:25 PM Changeset in webkit [119431] by
-
- 2 edits in trunk/Source/WebCore
Mock scrollbars differ by 1px in DRT vs. WKTR
https://bugs.webkit.org/show_bug.cgi?id=80879
Reviewed by Simon Fraser.
Round the length of the scrollbar thumb instead of truncating it.
No new tests, fixes a mismatch between pixel results from DRT and WKTR.
Change is covered by all existing tests with scrollbars run with low tolerance.
- platform/ScrollbarThemeComposite.cpp:
(WebCore::ScrollbarThemeComposite::thumbLength):
- 3:24 PM Changeset in webkit [119430] by
-
- 2 edits in trunk/Source/WebCore
Fix an unused result warning when building with clang.
- platform/cf/BinaryPropertyList.cpp:
(WebCore::BinaryPropertyListWriter::writePropertyList):
- 2:56 PM Changeset in webkit [119429] by
-
- 2 edits in trunk/Source/JavaScriptCore
Fix a struct/class mismatch.
- heap/Handle.h:
(Handle):
- 2:47 PM Changeset in webkit [119428] by
-
- 3 edits2 adds in trunk
Don't crash when re-parenting a removed HTMLMediaElement
https://bugs.webkit.org/show_bug.cgi?id=88058
Reviewed by Eric Carlson.
Source/WebCore:
Test: media/media-reparent.html
- html/HTMLMediaElement.cpp:
(WebCore::HTMLMediaElement::userCancelledLoad): Clear the pending load flags because any pending load has been cancelled.
LayoutTests:
- media/media-reparent-expected.txt: Added.
- media/media-reparent.html: Added.
- 2:41 PM Changeset in webkit [119427] by
-
- 1 edit in branches/chromium/1132/Source/WebCore/rendering/FixedTableLayout.cpp
Merge 118819 - REGRESSION(r111742): box-sizing: border-box doesn't work on fixed table layout
https://bugs.webkit.org/show_bug.cgi?id=87536
Reviewed by Tony Chang.
Source/WebCore:
Tests: fast/table/fixed-table-layout/cell-box-sizing-fixed-table-layout.html
fast/table/fixed-table-layout/column-box-sizing-fixed-table-layout.html
fast/table/fixed-table-layout/column-group-box-sizing-fixed-table-layout.html
fast/table/fixed-table-layout/column-in-column-group-box-sizing-fixed-table-layout.html
The change in r111742 completely ignored border-sizing (following the table's code lead unfortunately).
The issue is that we would count the borders and paddings twice for the border-box case which would lead
to the content-box including them too. From a web-author, this behavior is equivalent to ignoring box-sizing.
- rendering/FixedTableLayout.cpp:
(WebCore::FixedTableLayout::calcWidthArray):
Reworked the function to properly use computeBorderBoxLogicalWidth() for correctness. This matches
what RenderBox and the rendering code does generally. Also refactored the code to avoid the need
for local variables and make it more readable.
LayoutTests:
- fast/table/fixed-table-layout/cell-box-sizing-fixed-table-layout-expected.html: Added.
- fast/table/fixed-table-layout/cell-box-sizing-fixed-table-layout.html: Added.
This test covers the regression.
- fast/table/fixed-table-layout/column-box-sizing-fixed-table-layout-expected.html: Added.
- fast/table/fixed-table-layout/column-box-sizing-fixed-table-layout.html: Added.
- fast/table/fixed-table-layout/column-group-box-sizing-fixed-table-layout-expected.html: Added.
- fast/table/fixed-table-layout/column-group-box-sizing-fixed-table-layout.html: Added.
- fast/table/fixed-table-layout/column-in-column-group-box-sizing-fixed-table-layout-expected.html: Added.
- fast/table/fixed-table-layout/column-in-column-group-box-sizing-fixed-table-layout.html: Added.
Those test ensured I didn't regress and document our current behavior ie that column and column group fixed
width apply to the cell's border-box.
TBR=jchaffraix@webkit.org
Review URL: https://chromiumcodereview.appspot.com/10479019
- 2:35 PM Changeset in webkit [119426] by
-
- 8 edits in trunk/Source/WebCore
Unreviewed, rolling out r119421 and r119425.
http://trac.webkit.org/changeset/119421
http://trac.webkit.org/changeset/119425
https://bugs.webkit.org/show_bug.cgi?id=88256
not reliable, let's have a do-over (Requested by kling on
#webkit).
Patch by Sheriff Bot <webkit.review.bot@gmail.com> on 2012-06-04
- dom/Element.cpp:
(WebCore::Element::detachAttribute):
(WebCore::Element::removeAttribute):
(WebCore::Element::attributes):
(WebCore::Element::setAttributeInternal):
(WebCore::Element::parserSetAttributes):
(WebCore::Element::hasEquivalentAttributes):
(WebCore::Element::createAttributeData):
(WebCore):
(WebCore::Element::setAttributeNode):
(WebCore::Element::removeAttributeNode):
(WebCore::Element::getAttributeNode):
(WebCore::Element::getAttributeNodeNS):
(WebCore::Element::hasAttribute):
(WebCore::Element::hasAttributeNS):
(WebCore::Element::normalizeAttributes):
(WebCore::Element::cloneAttributesFromElement):
- dom/Element.h:
(WebCore::Element::attributeData):
(Element):
(WebCore::Element::ensureAttributeData):
(WebCore::Element::updatedAttributeData):
(WebCore::Element::ensureUpdatedAttributeData):
- dom/ElementAttributeData.cpp:
(WebCore::ElementAttributeData::attrIfExists):
(WebCore::ElementAttributeData::ensureAttr):
(WebCore::ElementAttributeData::setAttr):
(WebCore::ElementAttributeData::removeAttr):
(WebCore::ElementAttributeData::setClass):
(WebCore):
(WebCore::ElementAttributeData::ensureInlineStyle):
(WebCore::ElementAttributeData::ensureMutableInlineStyle):
(WebCore::ElementAttributeData::destroyInlineStyle):
(WebCore::ElementAttributeData::addAttribute):
(WebCore::ElementAttributeData::removeAttribute):
(WebCore::ElementAttributeData::isEquivalent):
(WebCore::ElementAttributeData::detachAttrObjectsFromElement):
(WebCore::ElementAttributeData::getAttributeItemIndexSlowCase):
(WebCore::ElementAttributeData::cloneDataFrom):
(WebCore::ElementAttributeData::clearAttributes):
(WebCore::ElementAttributeData::replaceAttribute):
(WebCore::ElementAttributeData::getAttributeNode):
- dom/ElementAttributeData.h:
(WebCore::ElementAttributeData::create):
(ElementAttributeData):
(WebCore::ElementAttributeData::setIdForStyleResolution):
(WebCore::ElementAttributeData::setAttributeStyle):
(WebCore::ElementAttributeData::length):
(WebCore::ElementAttributeData::isEmpty):
(WebCore::ElementAttributeData::attributeItem):
(WebCore::ElementAttributeData::getAttributeItem):
(WebCore::ElementAttributeData::ElementAttributeData):
(WebCore::ElementAttributeData::attributeVector):
(WebCore::ElementAttributeData::clonedAttributeVector):
(WebCore::ElementAttributeData::getAttributeItemIndex):
- dom/StyledElement.cpp:
(WebCore::StyledElement::style):
(WebCore::StyledElement::classAttributeChanged):
(WebCore::StyledElement::removeInlineStyleProperty):
(WebCore::StyledElement::addSubresourceAttributeURLs):
- html/parser/HTMLConstructionSite.cpp:
(WebCore::HTMLConstructionSite::createHTMLElementFromSavedElement):
- xml/parser/XMLDocumentParserQt.cpp:
(WebCore::XMLDocumentParser::XMLDocumentParser):
- 1:23 PM Changeset in webkit [119425] by
-
- 3 edits in trunk/Source/WebCore
Unreviewed build fixes for cr-android and Qt.
- dom/ElementAttributeData.h:
(WebCore::ElementAttributeData::setClass):
Change ElementAttributeData::setClass() to take an AtomicString instead of a String.
The only call site has an AtomicString anyway.
- xml/parser/XMLDocumentParserQt.cpp:
(WebCore::XMLDocumentParser::XMLDocumentParser): Add missing const.
- 1:14 PM Changeset in webkit [119424] by
-
- 2 edits in trunk/Source/WebKit/chromium
Fix attempt after r119411.
- src/WebHelperPluginImpl.cpp:
- 1:11 PM Changeset in webkit [119423] by
-
- 15 edits3 adds in trunk
.: [chromium] Fix software rendering for device-scale-factor > 1
https://bugs.webkit.org/show_bug.cgi?id=88136
Patch by Sadrul Habib Chowdhury <sadrul@chromium.org> on 2012-06-04
Reviewed by Darin Fisher.
Export WebCore::Page::setDeviceScaleFactor and WebCore::Settings::setDefaultDeviceScaleFactor.
- Source/autotools/symbols.filter:
Source/WebCore: [chromium] Fix software rendering for device-scale-factor > 1
https://bugs.webkit.org/show_bug.cgi?id=88136
Patch by Sadrul Habib Chowdhury <sadrul@chromium.org> on 2012-06-04
Reviewed by Darin Fisher.
Test: fast/hidpi/device-scale-factor-paint.html
- WebCore.exp.in:
- testing/InternalSettings.cpp:
(WebCore::InternalSettings::setDeviceScaleFactor):
(WebCore):
- testing/InternalSettings.h:
(InternalSettings):
- testing/InternalSettings.idl:
Source/WebKit/chromium: [chromium] Fix software rendering for larger device-scale-factor
https://bugs.webkit.org/show_bug.cgi?id=88136
Patch by Sadrul Habib Chowdhury <sadrul@chromium.org> on 2012-06-04
Reviewed by Darin Fisher.
The fix is to apply the device-scale factor on the GraphicsContext.
(and add a WebWidgetClient::deviceScaleFactor method to facilitate that).
- public/WebWidgetClient.h:
(WebWidgetClient):
(WebKit::WebWidgetClient::deviceScaleFactor):
- src/PageWidgetDelegate.cpp:
(WebKit::PageWidgetDelegate::paint):
- src/WebPopupMenuImpl.cpp:
(WebKit::WebPopupMenuImpl::paint):
Source/WebKit2: [chromium] Fix software rendering for device-scale-factor > 1
https://bugs.webkit.org/show_bug.cgi?id=88136
Patch by Sadrul Habib Chowdhury <sadrul@chromium.org> on 2012-06-04
Reviewed by Darin Fisher.
Export WebCore::Page::setDeviceScaleFactor and WebCore::Settings::setDefaultDeviceScaleFactor.
- win/WebKit2.def:
- win/WebKit2CFLite.def:
LayoutTests: [chromium] Fix software rendering for device-scale-factor > 1
https://bugs.webkit.org/show_bug.cgi?id=88136
Patch by Sadrul Habib Chowdhury <sadrul@chromium.org> on 2012-06-04
Reviewed by Darin Fisher.
- fast/hidpi/device-scale-factor-paint.html: Added.
- platform/chromium/fast/hidpi/device-scale-factor-paint-expected.html: Added.
- 1:10 PM Changeset in webkit [119422] by
-
- 2 edits in trunk/Tools
[GTK] Memory leaks in AccessibilityUIElementGtk.cpp
https://bugs.webkit.org/show_bug.cgi?id=88215
Patch by Sudarsana Nagineni <sudarsana.nagineni@linux.intel.com> on 2012-06-04
Reviewed by Martin Robinson.
Fixed memory leaks in AccessibilityUIElement by using GOwnPtr
for newly allocated strings.
- DumpRenderTree/gtk/AccessibilityUIElementGtk.cpp:
(attributeSetToString):
(AccessibilityUIElement::allAttributes):
- 1:00 PM Changeset in webkit [119421] by
-
- 8 edits in trunk/Source/WebCore
Make ElementAttributeData a variable-sized object to reduce memory use.
<http://webkit.org/b/88240>
Reviewed by Antti Koivisto.
Take advantage of the fact that we know at ElementAttributeData construction time how many attributes
it needs to accomodate and allocate exactly as much space as needed instead of using a Vector.
For elements that never have their attribute list mutated (the vast majority), this saves a lot of
memory and removes the indirection to Vector<Attribute>'s heap-allocated storage.
Introduced a mutability flag to ElementAttributeData and sprinkled assertions all over to make sure
that nothing tries to mutate an element with a raw attribute array.
When an Element's attribute(s) are mutated, we reconstruct the ElementAttributeData, this time using
a Vector as backing instead. This is triggered when calling Element::mutableAttributeData().
This reduces memory consumption by 3.2MB when viewing the full HTML5 spec at <http://whatwg.org/c/>.
This is a 35% reduction in DOM attribute memory use.
Furthermore, that page ends up promoting 27% of the elements to mutable attribute storage due to dynamic
adding of "class" attributes. For more static pages, savings are even greater.
- dom/Element.cpp:
(WebCore::Element::detachAttribute):
(WebCore::Element::removeAttribute):
(WebCore::Element::attributes):
(WebCore::Element::setAttributeInternal):
(WebCore::Element::parserSetAttributes):
(WebCore::Element::hasEquivalentAttributes):
(WebCore::Element::setAttributeNode):
(WebCore::Element::removeAttributeNode):
(WebCore::Element::getAttributeNode):
(WebCore::Element::getAttributeNodeNS):
(WebCore::Element::hasAttribute):
(WebCore::Element::hasAttributeNS):
(WebCore::Element::normalizeAttributes):
(WebCore::Element::cloneAttributesFromElement):
(WebCore::Element::createMutableAttributeData):
- dom/Element.h:
(WebCore::Element::attributeData):
(Element):
(WebCore::Element::updatedAttributeData):
(WebCore::Element::ensureAttributeData):
(WebCore::Element::ensureUpdatedAttributeData):
(WebCore::Element::mutableAttributeData):
- dom/ElementAttributeData.cpp:
(WebCore::ElementAttributeData::createImmutable):
(WebCore::ElementAttributeData::ElementAttributeData):
(WebCore::ElementAttributeData::~ElementAttributeData):
(WebCore::ElementAttributeData::attrIfExists):
(WebCore::ElementAttributeData::ensureAttr):
(WebCore::ElementAttributeData::setAttr):
(WebCore::ElementAttributeData::removeAttr):
(WebCore::ElementAttributeData::ensureInlineStyle):
(WebCore::ElementAttributeData::ensureMutableInlineStyle):
(WebCore::ElementAttributeData::destroyInlineStyle):
(WebCore::ElementAttributeData::addAttribute):
(WebCore::ElementAttributeData::removeAttribute):
(WebCore::ElementAttributeData::isEquivalent):
(WebCore::ElementAttributeData::detachAttrObjectsFromElement):
(WebCore::ElementAttributeData::getAttributeItemIndexSlowCase):
(WebCore::ElementAttributeData::cloneDataFrom):
(WebCore::ElementAttributeData::clearAttributes):
(WebCore::ElementAttributeData::replaceAttribute):
(WebCore::ElementAttributeData::getAttributeNode):
- dom/ElementAttributeData.h:
(ElementAttributeData):
(WebCore::ElementAttributeData::setClass):
(WebCore::ElementAttributeData::setIdForStyleResolution):
(WebCore::ElementAttributeData::inlineStyle):
(WebCore::ElementAttributeData::setAttributeStyle):
(WebCore::ElementAttributeData::isMutable):
(WebCore::ElementAttributeData::makeMutable):
(WebCore::ElementAttributeData::length):
(WebCore::ElementAttributeData::isEmpty):
(WebCore::ElementAttributeData::array):
(WebCore::ElementAttributeData::attributeItem):
(WebCore::ElementAttributeData::getAttributeItem):
(WebCore::ElementAttributeData::getAttributeItemIndex):
- dom/StyledElement.cpp:
(WebCore::StyledElement::style):
(WebCore::StyledElement::classAttributeChanged):
(WebCore::StyledElement::removeInlineStyleProperty):
(WebCore::StyledElement::addSubresourceAttributeURLs):
- html/parser/HTMLConstructionSite.cpp:
(WebCore::HTMLConstructionSite::createHTMLElementFromSavedElement):
- xml/parser/XMLDocumentParserQt.cpp:
(WebCore::XMLDocumentParser::XMLDocumentParser):
- 12:56 PM Changeset in webkit [119420] by
-
- 7 edits in trunk/Source
Source/WebCore: [chromium] Combobox options and autofill options should not be scaled for device-scale factor
https://bugs.webkit.org/show_bug.cgi?id=87921
Patch by Sadrul Habib Chowdhury <sadrul@chromium.org> on 2012-06-04
Reviewed by Darin Fisher.
Chrome takes care of scaling the combobox/autofill lists in the browser side. So it is not
necessary to scale here. This change reverts parts of changes in r111539.
Manually tested with different scale factors.
- platform/chromium/PopupListBox.cpp:
(WebCore::PopupListBox::paint):
(WebCore::PopupListBox::paintRow):
(WebCore::PopupListBox::getRowHeight):
- platform/chromium/PopupListBox.h:
(PopupContainerSettings):
- platform/chromium/PopupMenuChromium.cpp:
(WebCore::PopupMenuChromium::show):
- rendering/RenderMenuList.cpp:
(WebCore::RenderMenuList::showPopup):
Source/WebKit/chromium: Combobox options and autofill options should not be scaled for device-scale factor.
https://bugs.webkit.org/show_bug.cgi?id=87921
Patch by Sadrul Habib Chowdhury <sadrul@chromium.org> on 2012-06-04
Reviewed by Darin Fisher.
- src/WebViewImpl.cpp:
(WebKit::WebViewImpl::applyAutofillSuggestions):
- 12:53 PM Changeset in webkit [119419] by
-
- 19 edits2 moves in trunk/Source
[chromium] Make LayerRendererChromium use RenderPasses instead of RenderSurfaces
https://bugs.webkit.org/show_bug.cgi?id=88132
Reviewed by Adrienne Walker.
Source/WebCore:
This patch is partly renaming, partly moving data into CCRenderPass
and CCRenderPassDrawQuad. LayerRendererChromium should accept as input
a set of RenderPasses and this should be sufficient for drawing. It
should have no notion of RenderSurfaces at all.
First, what isn't done: RenderSurfaces still hold the textures
where RenderPasses are drawn to/from, so that these textures can
persist across frames. This will be addressed separately.
Otherwise, this completely removes the concept of RenderSurface from
LayerRenderChromium and the CCDrawQuad, replacing it with RenderPass.
The "renderSurfaceTextureManager" is renamed to the "implTextureManager"
and like-wise for the TextureAllocator, as these are not used
exclusively for render surfaces (passes), and a texture manager for
render passes that does not require a RenderSurface* will be the focus
of future changes.
Covered by existing tests.
- WebCore.gypi:
- platform/graphics/chromium/LayerRendererChromium.cpp:
(WebCore::LayerRendererChromium::LayerRendererChromium):
(WebCore::LayerRendererChromium::setVisible):
(WebCore::LayerRendererChromium::releaseRenderPassTextures):
(WebCore::LayerRendererChromium::viewportChanged):
(WebCore::LayerRendererChromium::clearRenderPass):
(WebCore::LayerRendererChromium::beginDrawingFrame):
(WebCore::LayerRendererChromium::drawRenderPass):
(WebCore::LayerRendererChromium::drawQuad):
(WebCore::LayerRendererChromium::drawBackgroundFilters):
(WebCore::LayerRendererChromium::drawRenderPassQuad):
(WebCore::LayerRendererChromium::copyPlaneToTexture):
(WebCore::LayerRendererChromium::drawHeadsUpDisplay):
(WebCore::LayerRendererChromium::finishDrawingFrame):
(WebCore::LayerRendererChromium::copyTextureToFramebuffer):
(WebCore::LayerRendererChromium::getFramebufferTexture):
(WebCore::LayerRendererChromium::isCurrentRenderPass):
(WebCore::LayerRendererChromium::useRenderPass):
(WebCore::LayerRendererChromium::useManagedTexture):
(WebCore::LayerRendererChromium::bindFramebufferToTexture):
(WebCore::LayerRendererChromium::setScissorToRect):
(WebCore::LayerRendererChromium::setDrawFramebufferRect):
(WebCore::LayerRendererChromium::initializeSharedObjects):
(WebCore::LayerRendererChromium::renderPassProgram):
(WebCore::LayerRendererChromium::renderPassProgramAA):
(WebCore::LayerRendererChromium::renderPassMaskProgram):
(WebCore::LayerRendererChromium::renderPassMaskProgramAA):
(WebCore::LayerRendererChromium::cleanupSharedObjects):
- platform/graphics/chromium/LayerRendererChromium.h:
(WebCore):
(LayerRendererChromium):
- platform/graphics/chromium/cc/CCDrawQuad.cpp:
(WebCore::CCDrawQuad::toRenderPassDrawQuad):
- platform/graphics/chromium/cc/CCDrawQuad.h:
(WebCore):
(CCDrawQuad):
- platform/graphics/chromium/cc/CCHeadsUpDisplay.cpp:
(WebCore::CCHeadsUpDisplay::draw):
- platform/graphics/chromium/cc/CCLayerTreeHostImpl.cpp:
(WebCore::CCLayerTreeHostImpl::calculateRenderPasses):
(WebCore::CCLayerTreeHostImpl::drawLayers):
- platform/graphics/chromium/cc/CCQuadCuller.cpp:
- platform/graphics/chromium/cc/CCRenderPass.cpp:
(WebCore::CCRenderPass::CCRenderPass):
(WebCore::CCRenderPass::appendQuadsForRenderSurfaceLayer):
- platform/graphics/chromium/cc/CCRenderPass.h:
(CCRenderPass):
(WebCore::CCRenderPass::framebufferOutputRect):
- platform/graphics/chromium/cc/CCRenderPassDrawQuad.cpp: Renamed from Source/WebCore/platform/graphics/chromium/cc/CCRenderSurfaceDrawQuad.cpp.
(WebCore):
(WebCore::CCRenderPassDrawQuad::create):
(WebCore::CCRenderPassDrawQuad::CCRenderPassDrawQuad):
- platform/graphics/chromium/cc/CCRenderPassDrawQuad.h: Renamed from Source/WebCore/platform/graphics/chromium/cc/CCRenderSurfaceDrawQuad.h.
(WebCore):
(CCRenderPassDrawQuad):
(WebCore::CCRenderPassDrawQuad::renderPass):
(WebCore::CCRenderPassDrawQuad::isReplica):
(WebCore::CCRenderPassDrawQuad::maskTextureId):
(WebCore::CCRenderPassDrawQuad::filters):
(WebCore::CCRenderPassDrawQuad::backgroundFilters):
- platform/graphics/chromium/cc/CCRenderSurface.cpp:
(WebCore::CCRenderSurface::prepareContentsTexture):
(WebCore::CCRenderSurface::prepareBackgroundTexture):
(WebCore::CCRenderSurface::appendQuads):
- platform/graphics/chromium/cc/CCRenderSurface.h:
(WebCore):
(CCRenderSurface):
- platform/graphics/chromium/cc/CCRenderer.h:
(CCRenderer):
- platform/graphics/chromium/cc/CCScrollbarLayerImpl.cpp:
(WebCore::CCScrollbarLayerImpl::willDraw):
- platform/graphics/chromium/cc/CCVideoLayerImpl.cpp:
(WebCore::CCVideoLayerImpl::reserveTextures):
Source/WebKit/chromium:
- tests/CCLayerTreeHostImplTest.cpp:
- tests/LayerRendererChromiumTest.cpp:
(FakeCCRendererClient::FakeCCRendererClient):
(FakeCCRendererClient::rootRenderPass):
(FakeCCRendererClient):
(TEST_F):
- 12:52 PM Changeset in webkit [119418] by
-
- 2 edits in trunk
[chromium] Issue async events for console.time/timeEnd
https://bugs.webkit.org/show_bug.cgi?id=88003
Patch by Kevin Greer <kgr@chromium.org> on 2012-06-04
Reviewed by Pavel Feldman.
- ../../Source/WebCore/page/Console.cpp:
- 12:36 PM Changeset in webkit [119417] by
-
- 6 edits in trunk/Source/WebKit2
Use ENABLE() and USE() macros in a few more places
https://bugs.webkit.org/show_bug.cgi?id=88245
Reviewed by Anders Carlsson.
- UIProcess/API/C/WKPluginSiteDataManager.cpp: Only include npapi.h if
Netscape plug-in API is enabled.
- UIProcess/mac/CorrectionPanel.h: Wrap with USE(AUTOCORRECTION_PANEL)
- WebProcess/Authentication/AuthenticationManager.cpp: Use the stub
version of tryUsePlatformCertificateInfoForChallenge if
Security.framework isn't available.
- WebProcess/WebProcess.messages.in: Only generate two messages if
Security.framework is available.
- WebProcess/mac/KeychainItemShimMethods.mm: Wrap with
USE(SECURITY_FRAMEWORK).
- 12:30 PM Changeset in webkit [119416] by
-
- 2 edits in trunk/Tools
[GTK] Memory leaks in DumpRenderTree.cpp
https://bugs.webkit.org/show_bug.cgi?id=88213
Patch by Sudarsana Nagineni <sudarsana.nagineni@linux.intel.com> on 2012-06-04
Reviewed by Martin Robinson.
Fixed memory leaks in DumpRenderTree.
- DumpRenderTree/gtk/DumpRenderTree.cpp:
(appendString): Accepts const gchar* as the second argument.
(dumpFramesAsText): Free the string returned by g_strdup_printf().
(dumpHistoryItem): Free the string returned by g_uri_parse_scheme().
- 12:26 PM Changeset in webkit [119415] by
-
- 2 edits in trunk/LayoutTests
[Gtk] http/tests/media/video-buffered.html is failing after r119268
https://bugs.webkit.org/show_bug.cgi?id=88138
Reviewed by Eric Carlson.
- http/tests/media/video-buffered.html:
- 12:22 PM Changeset in webkit [119414] by
-
- 4 edits in trunk/LayoutTests
Speed up video-media-source-play & video-media-source-seek LayoutTests.
https://bugs.webkit.org/show_bug.cgi?id=87986
Patch by Aaron Colwell <acolwell@chromium.org> on 2012-06-04
Reviewed by Eric Carlson.
- http/tests/media/media-source/video-media-source-play.html: Updated test to only play 1 second of media data instead of ~3 seconds.
- http/tests/media/media-source/video-media-source-seek-expected.txt:
- http/tests/media/media-source/video-media-source-seek.html:
- Triggers seek after playing 0.5 seconds instead of 2 seconds.
- Seeks to the last segment instead of a segment that is ~2 seconds from the end.
- Added an webkitSourceAbort() call before appending the segments for the seek. This is to prevent this test from failing when the old v0.3 "flush on seek" behavior is removed from the Chromium code.
- 12:21 PM Changeset in webkit [119413] by
-
- 3 edits in trunk/Source/WebKit2
Add stub WebKit2 implementations of PLUGIN_PROXY_FOR_VIDEO methods declared in FrameLoaderClient
https://bugs.webkit.org/show_bug.cgi?id=88244
Reviewed by Anders Carlsson.
- WebProcess/WebCoreSupport/WebFrameLoaderClient.cpp:
(WebKit::WebFrameLoaderClient::createMediaPlayerProxyPlugin): Add a stub implementation.
(WebKit::WebFrameLoaderClient::hideMediaPlayerProxyPlugin): Ditto.
(WebKit::WebFrameLoaderClient::showMediaPlayerProxyPlugin): Ditto.
- WebProcess/WebCoreSupport/WebFrameLoaderClient.h:
- 12:14 PM Changeset in webkit [119412] by
-
- 2 edits in trunk/Source/WTF
Port RAMSize to WinCE
https://bugs.webkit.org/show_bug.cgi?id=87854
Reviewed by Geoffrey Garen.
WinCE has no GlobalMemoryStatusEx() function. Use GlobalMemoryStatus() instead.
- wtf/RAMSize.cpp:
(WTF::computeRAMSize):
- 12:03 PM Changeset in webkit [119411] by
-
- 14 edits1 copy2 adds in trunk
Enable Chromium media player to instantiate a plugin
https://bugs.webkit.org/show_bug.cgi?id=87399
Patch by David Dorwin <ddorwin@chromium.org> on 2012-06-04
Reviewed by Kent Tamura.
Adds WebHelperPlugin, an off-screen widget that contains an <object> tag.
Source/WebKit/chromium:
- WebKit.gyp:
- public/WebHelperPlugin.h: Copied from Source/WebKit/chromium/public/WebPopupType.h.
(WebKit):
(WebHelperPlugin):
(WebKit::WebHelperPlugin::~WebHelperPlugin):
- public/WebMediaPlayerClient.h:
(WebKit):
- public/WebPopupType.h:
- public/WebView.h:
- public/WebViewClient.h:
(WebKit):
(WebViewClient):
(WebKit::WebViewClient::initializeHelperPluginWebFrame):
- src/WebFrameImpl.cpp:
(WebKit::WebFrameImpl::WebFrameImpl):
(WebKit::WebFrameImpl::initializeAsMainFrame):
- src/WebFrameImpl.h:
(WebFrameImpl):
- src/WebHelperPluginImpl.cpp: Added.
(WebKit):
(WebKit::addString):
(WebKit::writeDocument):
(HelperPluginChromeClient):
(WebKit::HelperPluginChromeClient::HelperPluginChromeClient):
(WebKit::WebHelperPluginImpl::WebHelperPluginImpl):
(WebKit::WebHelperPluginImpl::~WebHelperPluginImpl):
(WebKit::WebHelperPluginImpl::init):
(WebKit::WebHelperPluginImpl::initializeFrame):
(WebKit::WebHelperPluginImpl::initPage):
(WebKit::WebHelperPluginImpl::setCompositorSurfaceReady):
(WebKit::WebHelperPluginImpl::composite):
(WebKit::WebHelperPluginImpl::layout):
(WebKit::WebHelperPluginImpl::setFocus):
(WebKit::WebHelperPluginImpl::close):
(WebKit::WebHelperPluginImpl::closeHelperPlugin):
(WebKit::WebHelperPlugin::create):
- src/WebHelperPluginImpl.h: Copied from Source/WebKit/chromium/public/WebPopupType.h.
(WebCore):
(WebKit):
(WebHelperPluginImpl):
- src/WebMediaPlayerClientImpl.cpp:
(WebKit::WebMediaPlayerClientImpl::~WebMediaPlayerClientImpl):
(WebKit::WebMediaPlayerClientImpl::createHelperPlugin):
(WebKit):
(WebKit::WebMediaPlayerClientImpl::closeHelperPlugin):
- src/WebMediaPlayerClientImpl.h:
(WebKit):
(WebMediaPlayerClientImpl):
- src/WebViewImpl.cpp:
(WebKit::WebViewImpl::initializeMainFrame):
(WebKit::WebViewImpl::initializeHelperPluginFrame):
(WebKit):
(WebKit::WebViewImpl::createHelperPlugin):
- src/WebViewImpl.h:
(WebKit):
(WebViewImpl):
Tools:
- DumpRenderTree/chromium/WebViewHost.cpp:
(WebViewHost::createPopupMenu):
- 11:30 AM Changeset in webkit [119410] by
-
- 3 edits in trunk/Source/WebKit/chromium
Remove obsolete acceptMIMETypes member.
https://bugs.webkit.org/show_bug.cgi?id=88241
Patch by Raymes Khoury <raymes@chromium.org> on 2012-06-04
Reviewed by Darin Fisher.
Obsoleted by https://bugs.webkit.org/show_bug.cgi?id=87271 and http://codereview.chromium.org/10414085/.
- public/WebFileChooserParams.h:
(WebFileChooserParams):
- src/ChromeClientImpl.cpp:
(WebKit::ChromeClientImpl::runOpenPanel):
- 11:06 AM Changeset in webkit [119409] by
-
- 3 edits4 adds in trunk
Crash in multi-column layout.
https://bugs.webkit.org/show_bug.cgi?id=88022
Reviewed by Ojan Vafai.
Source/WebCore:
The patch addresses two problems::
- |this| in RenderBlock::splitBlocks can get destroyed when we
move its children to the clone and later call updateBeforeAfterContent
on the parent. So, we stop accessing its member variables and cache
it in a local.
- Positioned objects were not getting cleared from our grand parents.
This will happen if our immediate children got moved to a clone tree,
however at our parent nothing was moved. So, we make sure to remove
the positioned objects at every level while we are doing the cloning.
Tests: fast/multicol/span/empty-anonymous-block-split-crash.html
fast/multicol/span/positioned-objects-not-removed-crash.html
- rendering/RenderBlock.cpp:
(WebCore::RenderBlock::splitBlocks):
LayoutTests:
- fast/multicol/span/empty-anonymous-block-split-crash-expected.txt: Added.
- fast/multicol/span/empty-anonymous-block-split-crash.html: Added.
- fast/multicol/span/positioned-objects-not-removed-crash-expected.txt: Added.
- fast/multicol/span/positioned-objects-not-removed-crash.html: Added.
- 11:04 AM Changeset in webkit [119408] by
-
- 3 edits in trunk/Source/WebCore
Mac build fails with SVG disabled
https://bugs.webkit.org/show_bug.cgi?id=88165
Reviewed by Dirk Schulze.
Wrap all of SVGCSSPropertyNames in #if ENABLE(SVG),
and mark an unused argument in collectMatchingRulesForList.
- css/SVGCSSPropertyNames.in:
- css/StyleResolver.cpp:
(WebCore::StyleResolver::collectMatchingRulesForList):
- 11:01 AM Changeset in webkit [119407] by
-
- 2 edits in trunk/Source/WebCore
BUILD FIX: Unused variable with ENABLE(CONTEXT_MENUS) off
This broke in r118374.
- bindings/js/JSInspectorFrontendHostCustom.cpp:
(WebCore::JSInspectorFrontendHost::showContextMenu): Add
UNUSED_PARAM(exec) when ENABLE(CONTEXT_MENUS) is off.
- 10:07 AM Changeset in webkit [119406] by
-
- 2 edits10 moves in trunk/LayoutTests
Rename CSS flexbox test files to match new property names
https://bugs.webkit.org/show_bug.cgi?id=88174
Reviewed by Ojan Vafai.
flex-pack became justify-content and flex-line-pack became
align-content. Rename the tests to match the new names.
- css3/flexbox/flex-justify-content-expected.txt: Renamed from LayoutTests/css3/flexbox/flex-pack-expected.txt.
- css3/flexbox/flex-justify-content.html: Renamed from LayoutTests/css3/flexbox/flex-pack.html.
- css3/flexbox/multiline-align-content-expected.txt: Renamed from LayoutTests/css3/flexbox/multiline-line-pack-expected.txt.
- css3/flexbox/multiline-align-content-horizontal-column-expected.txt: Renamed from LayoutTests/css3/flexbox/multiline-line-pack-horizontal-column-expected.txt.
- css3/flexbox/multiline-align-content-horizontal-column.html: Renamed from LayoutTests/css3/flexbox/multiline-line-pack-horizontal-column.html.
- css3/flexbox/multiline-align-content.html: Renamed from LayoutTests/css3/flexbox/multiline-line-pack.html.
- css3/flexbox/multiline-align-self-expected.txt: Renamed from LayoutTests/css3/flexbox/multiline-align-expected.txt.
- css3/flexbox/multiline-align-self.html: Renamed from LayoutTests/css3/flexbox/multiline-align.html.
- css3/flexbox/multiline-justify-content-expected.txt: Renamed from LayoutTests/css3/flexbox/multiline-pack-expected.txt.
- css3/flexbox/multiline-justify-content.html: Renamed from LayoutTests/css3/flexbox/multiline-pack.html.
- platform/chromium/TestExpectations:
- 10:05 AM Changeset in webkit [119405] by
-
- 2 edits in trunk/LayoutTests
Fix a couple flexbox tests to use flex-start and flex-end
https://bugs.webkit.org/show_bug.cgi?id=88170
Reviewed by Ojan Vafai.
This test was still using start/end for align-self, which fallback
to stretch. The test still passed because it is testing that all
values of align-self behave the same and stretch produced the same
behavior as flex-start/flex-end.
- css3/flexbox/cross-axis-scrollbar.html:
- 9:05 AM Changeset in webkit [119404] by
-
- 7 edits2 adds in trunk
[GTK][WK2] Implement geolocation provider for the GTK port
https://bugs.webkit.org/show_bug.cgi?id=83877
Reviewed by Carlos Garcia Campos.
Source/WebKit2:
Add a new client-based geolocation provider for WebKit2GTK+, relying the
new Geoclue-based geolocation provider available in WebCore.
- GNUmakefile.am:
- GNUmakefile.list.am:
- UIProcess/API/gtk/WebKitGeolocationProvider.cpp: Added.
(toGeolocationProvider):
(startUpdatingCallback):
(stopUpdatingCallback):
(WebKitGeolocationProvider::~WebKitGeolocationProvider):
(WebKitGeolocationProvider::create):
(WebKitGeolocationProvider::WebKitGeolocationProvider):
(WebKitGeolocationProvider::startUpdating):
(WebKitGeolocationProvider::stopUpdating):
(WebKitGeolocationProvider::notifyPositionChanged):
(WebKitGeolocationProvider::notifyErrorOccurred):
- UIProcess/API/gtk/WebKitGeolocationProvider.h: Added.
- UIProcess/API/gtk/WebKitPrivate.h:
- UIProcess/API/gtk/WebKitWebContext.cpp:
(_WebKitWebContextPrivate):
(createDefaultWebContext):
Tools:
- gtk/generate-gtkdoc:
(get_webkit2_options): Ignore WebKitGeolocationProvider files.
- 9:02 AM Changeset in webkit [119403] by
-
- 3 edits3 deletes in trunk
Unreviewed, rolling out r119393.
http://trac.webkit.org/changeset/119393
https://bugs.webkit.org/show_bug.cgi?id=88236
Test is incorrect (duplicate content) and is causing Chromium
failures (Requested by schenney on #webkit).
Patch by Sheriff Bot <webkit.review.bot@gmail.com> on 2012-06-04
Source/WebCore:
- html/canvas/CanvasRenderingContext2D.cpp:
(WebCore::CanvasRenderingContext2D::drawImage):
LayoutTests:
- fast/canvas/drawImage-clipped-source-expected.txt: Removed.
- fast/canvas/drawImage-clipped-source.html: Removed.
- fast/canvas/drawImage-clipped-source.js: Removed.
- 8:50 AM Changeset in webkit [119402] by
-
- 4 edits in trunk/LayoutTests
Update test expectations after 119392
https://bugs.webkit.org/show_bug.cgi?id=88230
Unreviewed update of test expectations.
- platform/efl/TestExpectations:
- platform/gtk/TestExpectations:
- platform/qt/TestExpectations:
- 8:42 AM Changeset in webkit [119401] by
-
- 30 edits in trunk/Source
[chromium] Cleanup scissor rect computation/use with damage
https://bugs.webkit.org/show_bug.cgi?id=87167
Patch by Zeev Lieber <zlieber@chromium.org> on 2012-06-04
Reviewed by Adrienne Walker.
Source/WebCore:
Performing scissorRect computation during
calculateRenderPasses. Storing scissorRect in shared quad state
instead of computing it during drawQuad. Added scissorRect fields
into layers and render surfaces.
Covered by existing layout tests. Introduced more unit tests to
test end-to-end drawing using mock graphic context, and added more
test cases to CCLayerTreeHostCommon to verify scissorRect computation.
- platform/graphics/chromium/LayerChromium.h:
(WebCore::LayerChromium::scissorRect):
(WebCore::LayerChromium::setScissorRect):
(LayerChromium):
- platform/graphics/chromium/LayerRendererChromium.cpp:
(WebCore::LayerRendererChromium::clearRenderSurface):
(WebCore::LayerRendererChromium::drawRenderPass):
(WebCore::LayerRendererChromium::drawQuad):
(WebCore::LayerRendererChromium::drawRenderSurfaceQuad):
- platform/graphics/chromium/LayerRendererChromium.h:
(LayerRendererChromium):
- platform/graphics/chromium/RenderSurfaceChromium.cpp:
(WebCore::RenderSurfaceChromium::computeRootScissorRectInCurrentSurface):
(WebCore):
- platform/graphics/chromium/RenderSurfaceChromium.h:
(WebCore::RenderSurfaceChromium::scissorRect):
(WebCore::RenderSurfaceChromium::setScissorRect):
(RenderSurfaceChromium):
- platform/graphics/chromium/cc/CCDrawQuad.h:
(WebCore::CCDrawQuad::scissorRect):
- platform/graphics/chromium/cc/CCLayerImpl.cpp:
(WebCore::CCLayerImpl::createSharedQuadState):
- platform/graphics/chromium/cc/CCLayerImpl.h:
(CCLayerImpl):
(WebCore::CCLayerImpl::scissorRect):
(WebCore::CCLayerImpl::setScissorRect):
- platform/graphics/chromium/cc/CCLayerTreeHost.cpp:
(WebCore::CCLayerTreeHost::updateLayers):
- platform/graphics/chromium/cc/CCLayerTreeHostCommon.cpp:
(WebCore):
(WebCore::calculateLayerScissorRect):
(WebCore::calculateSurfaceScissorRect):
(WebCore::calculateDrawTransformsInternal):
(WebCore::calculateVisibleAndScissorRectsInternal):
(WebCore::CCLayerTreeHostCommon::calculateDrawTransforms):
(WebCore::CCLayerTreeHostCommon::calculateVisibleAndScissorRects):
- platform/graphics/chromium/cc/CCLayerTreeHostCommon.h:
(CCLayerTreeHostCommon):
- platform/graphics/chromium/cc/CCLayerTreeHostImpl.cpp:
(WebCore::CCLayerTreeHostImpl::calculateRenderSurfaceLayerList):
(WebCore::CCLayerTreeHostImpl::calculateRenderPasses):
(WebCore::CCLayerTreeHostImpl::drawLayers):
(WebCore::CCLayerTreeHostImpl::swapBuffers):
- platform/graphics/chromium/cc/CCLayerTreeHostImpl.h:
(CCLayerTreeHostImpl):
- platform/graphics/chromium/cc/CCRenderPass.cpp:
(WebCore::CCRenderPass::appendQuadsForRenderSurfaceLayer):
- platform/graphics/chromium/cc/CCRenderPass.h:
(CCRenderPass):
- platform/graphics/chromium/cc/CCRenderSurface.cpp:
(WebCore::CCRenderSurface::createSharedQuadState):
(WebCore::CCRenderSurface::createReplicaSharedQuadState):
(WebCore):
(WebCore::CCRenderSurface::computeRootScissorRectInCurrentSurface):
(WebCore::CCRenderSurface::appendQuads):
- platform/graphics/chromium/cc/CCRenderSurface.h:
(CCRenderSurface):
(WebCore::CCRenderSurface::setScissorRect):
(WebCore::CCRenderSurface::scissorRect):
- platform/graphics/chromium/cc/CCRenderSurfaceDrawQuad.cpp:
(WebCore::CCRenderSurfaceDrawQuad::create):
(WebCore::CCRenderSurfaceDrawQuad::CCRenderSurfaceDrawQuad):
- platform/graphics/chromium/cc/CCRenderSurfaceDrawQuad.h:
(CCRenderSurfaceDrawQuad):
- platform/graphics/chromium/cc/CCRenderer.h:
(CCRenderer):
- platform/graphics/chromium/cc/CCSharedQuadState.cpp:
(WebCore::CCSharedQuadState::create):
(WebCore::CCSharedQuadState::CCSharedQuadState):
- platform/graphics/chromium/cc/CCSharedQuadState.h:
(CCSharedQuadState):
(WebCore::CCSharedQuadState::scissorRect):
Source/WebKit/chromium:
Added unit tests to CCLayerTreeHostImpl using mock graphic context
to verify end-to-end quad drawing.
Added more test cases to CCLayerTreeHostCommon to verify clip and
scissor rect computations.
- tests/CCDamageTrackerTest.cpp:
(WebKitTests::executeCalculateDrawTransformsAndVisibility):
- tests/CCLayerIteratorTest.cpp:
- tests/CCLayerTreeHostCommonTest.cpp:
- tests/CCLayerTreeHostImplTest.cpp:
- tests/CCOcclusionTrackerTest.cpp:
(WebKitTests::CCOcclusionTrackerTest::calcDrawEtc):
- tests/CCRenderSurfaceTest.cpp:
- 8:30 AM Changeset in webkit [119400] by
-
- 13 edits2 moves in trunk/Source/WebKit2
[Qt][WK2] Rename QtViewportInteractionEngine to QtViewportHandler.
https://bugs.webkit.org/show_bug.cgi?id=88226
Patch by Zalan Bujtas <zbujtas@gmail.com> on 2012-06-04
Reviewed by Kenneth Rohde Christiansen.
- Target.pri:
- UIProcess/API/qt/qquickwebpage_p_p.h:
(WebKit):
- UIProcess/API/qt/qquickwebview.cpp:
(QQuickWebViewFlickablePrivate::~QQuickWebViewFlickablePrivate):
(QQuickWebViewFlickablePrivate::onComponentComplete):
(QQuickWebViewFlickablePrivate::didChangeViewportProperties):
(QQuickWebViewFlickablePrivate::updateViewportSize):
(QQuickWebViewFlickablePrivate::pageDidRequestScroll):
(QQuickWebViewFlickablePrivate::didChangeContentsSize):
- UIProcess/API/qt/qquickwebview_p.h:
(WebKit):
- UIProcess/API/qt/qquickwebview_p_p.h:
(WebKit):
(QQuickWebViewPrivate::viewportHandler):
(QQuickWebViewFlickablePrivate::viewportHandler):
(QQuickWebViewFlickablePrivate):
- UIProcess/API/qt/qwebkittest.cpp:
(QWebKitTest::contentsScale):
(QWebKitTest::devicePixelRatio):
(QWebKitTest::initialScale):
(QWebKitTest::minimumScale):
(QWebKitTest::maximumScale):
(QWebKitTest::isScalable):
(QWebKitTest::layoutSize):
- UIProcess/qt/QtGestureRecognizer.cpp:
(WebKit::QtGestureRecognizer::viewportHandler):
- UIProcess/qt/QtGestureRecognizer.h:
(WebKit):
- UIProcess/qt/QtPanGestureRecognizer.cpp:
(WebKit::QtPanGestureRecognizer::update):
(WebKit::QtPanGestureRecognizer::finish):
(WebKit::QtPanGestureRecognizer::cancel):
- UIProcess/qt/QtPinchGestureRecognizer.cpp:
(WebKit::QtPinchGestureRecognizer::update):
(WebKit::QtPinchGestureRecognizer::finish):
(WebKit::QtPinchGestureRecognizer::cancel):
- UIProcess/qt/QtViewportHandler.cpp: Copied from Source/WebKit2/UIProcess/qt/QtViewportInteractionEngine.cpp.
(WebKit::ViewportUpdateDeferrer::ViewportUpdateDeferrer):
(WebKit::ViewportUpdateDeferrer::~ViewportUpdateDeferrer):
(ViewportUpdateDeferrer):
(WebKit::QtViewportHandler::suspendPageContent):
(WebKit::QtViewportHandler::resumePageContent):
(WebKit::QtViewportHandler::cssScaleFromItem):
(WebKit::QtViewportHandler::itemScaleFromCSS):
(WebKit::QtViewportHandler::itemCoordFromCSS):
(WebKit::QtViewportHandler::itemRectFromCSS):
(WebKit::QtViewportHandler::QtViewportHandler):
(WebKit::QtViewportHandler::~QtViewportHandler):
(WebKit::QtViewportHandler::innerBoundedCSSScale):
(WebKit::QtViewportHandler::outerBoundedCSSScale):
(WebKit::QtViewportHandler::viewportAttributesChanged):
(WebKit::QtViewportHandler::pageContentsSizeChanged):
(WebKit::QtViewportHandler::setPageItemRectVisible):
(WebKit::QtViewportHandler::animatePageItemRectVisible):
(WebKit::QtViewportHandler::flickMoveStarted):
(WebKit::QtViewportHandler::flickMoveEnded):
(WebKit::QtViewportHandler::pageItemPositionChanged):
(WebKit::QtViewportHandler::pageContentPositionRequested):
(WebKit::QtViewportHandler::scaleAnimationStateChanged):
(WebKit::QtViewportHandler::scaleAnimationValueChanged):
(WebKit::QtViewportHandler::touchBegin):
(WebKit::QtViewportHandler::touchEnd):
(WebKit::QtViewportHandler::computePosRangeForPageItemAtScale):
(WebKit::QtViewportHandler::focusEditableArea):
(WebKit::QtViewportHandler::zoomToAreaGestureEnded):
(WebKit::QtViewportHandler::nearestValidBounds):
(WebKit::QtViewportHandler::setCSSScale):
(WebKit::QtViewportHandler::currentCSSScale):
(WebKit::QtViewportHandler::scrollAnimationActive):
(WebKit::QtViewportHandler::panGestureActive):
(WebKit::QtViewportHandler::panGestureStarted):
(WebKit::QtViewportHandler::panGestureRequestUpdate):
(WebKit::QtViewportHandler::panGestureEnded):
(WebKit::QtViewportHandler::panGestureCancelled):
(WebKit::QtViewportHandler::scaleAnimationActive):
(WebKit::QtViewportHandler::cancelScrollAnimation):
(WebKit::QtViewportHandler::interruptScaleAnimation):
(WebKit::QtViewportHandler::pinchGestureActive):
(WebKit::QtViewportHandler::pinchGestureStarted):
(WebKit::QtViewportHandler::pinchGestureRequestUpdate):
(WebKit::QtViewportHandler::pinchGestureEnded):
(WebKit::QtViewportHandler::pinchGestureCancelled):
(WebKit::QtViewportHandler::visibleContentsRect):
(WebKit::QtViewportHandler::informVisibleContentChange):
(WebKit::QtViewportHandler::viewportItemSizeChanged):
(WebKit::QtViewportHandler::scaleContent):
- UIProcess/qt/QtViewportHandler.h: Copied from Source/WebKit2/UIProcess/qt/QtViewportInteractionEngine.h.
(QtViewportHandler):
- UIProcess/qt/QtViewportInteractionEngine.cpp: Removed.
- UIProcess/qt/QtViewportInteractionEngine.h: Removed.
- UIProcess/qt/QtWebPageEventHandler.cpp:
(WebKit::QtWebPageEventHandler::QtWebPageEventHandler):
(WebKit::QtWebPageEventHandler::setViewportHandler):
(WebKit::QtWebPageEventHandler::inputPanelVisibleChanged):
(WebKit::QtWebPageEventHandler::doneWithTouchEvent):
(WebKit::QtWebPageEventHandler::didFindZoomableArea):
- UIProcess/qt/QtWebPageEventHandler.h:
(WebKit):
(QtWebPageEventHandler):
(WebKit::QtWebPageEventHandler::viewportHandler):
- 8:30 AM Changeset in webkit [119399] by
-
- 3 edits in trunk/Source/WebCore
Remove SelectorQueryCacheEntry from SelectorQuery.h
https://bugs.webkit.org/show_bug.cgi?id=88204
Reviewed by Hajime Morita.
As anttik@ pointed out in https://bugs.webkit.org/show_bug.cgi?id=87942#c15,
we can remove SelectorQueryCacheEntry by having SelectorQuery hold CSSSelectorList.
Tests: fast/dom/SelectorAPI/* No change in test results.
- dom/SelectorQuery.cpp:
(WebCore::SelectorQuery::SelectorQuery):
(WebCore::SelectorQueryCache::add):
- dom/SelectorQuery.h:
(SelectorQuery):
(SelectorQueryCache):
- 8:18 AM Changeset in webkit [119398] by
-
- 2 edits in trunk/Source/WebCore
[GTK] Memory leak in PasteboardHelper
https://bugs.webkit.org/show_bug.cgi?id=88216
Patch by Sudarsana Nagineni <sudarsana.nagineni@linux.intel.com> on 2012-06-04
Reviewed by Martin Robinson.
Fixed a memory leak in PasteboardHelper by using adoptRef instead
of just getting a new reference of TargetList.
No new tests because of no behavior changes.
- platform/gtk/PasteboardHelper.cpp:
(WebCore::PasteboardHelper::dropAtomsForContext):
- 8:15 AM Changeset in webkit [119397] by
-
- 3 edits in trunk/Source/WebKit/gtk
[GTK] Memory leak in WebKitWebView
https://bugs.webkit.org/show_bug.cgi?id=88214
Patch by Sudarsana Nagineni <sudarsana.nagineni@linux.intel.com> on 2012-06-04
Reviewed by Martin Robinson.
Fixed a memory leak in WebKitWebView by making
GeolocationClientMock to be owned by the WebView.
- webkit/webkitwebview.cpp:
(webkit_web_view_init):
- webkit/webkitwebviewprivate.h:
- 8:00 AM Changeset in webkit [119396] by
-
- 2 edits in trunk/Source/WebCore
Hit ASSERT when editing attribute value in Element in SVG Document
https://bugs.webkit.org/show_bug.cgi?id=88179
Patch by Rob Buis <rbuis@rim.com> on 2012-06-04
Reviewed by Pavel Feldman.
Make sure the <span> is created as a HTML Element regardless of document in use.
- inspector/InspectorDOMAgent.cpp:
(WebCore):
(WebCore::InspectorDOMAgent::setAttributesAsText):
- 7:57 AM Changeset in webkit [119395] by
-
- 18 edits in trunk/Source/WebCore
[Forms] Change function and variable names independent from "double" type
https://bugs.webkit.org/show_bug.cgi?id=88220
Reviewed by Kent Tamura.
This patch is a part of introducing decimal arithmetic to input type.
This patch renames function and variable names which contain term "double"
to "number" to separate concept and implementation.
No new tests. This patch doesn't change behavior.
- html/BaseDateAndTimeInputType.cpp:
(WebCore::BaseDateAndTimeInputType::valueAsDate): Replace parseToDouble to parseToNumber.
(WebCore::BaseDateAndTimeInputType::valueAsNumber): ditto.
(WebCore::BaseDateAndTimeInputType::parseToNumber): ditto.
- html/BaseDateAndTimeInputType.h:
(BaseDateAndTimeInputType):
- html/DateInputType.cpp:
(WebCore::DateInputType::createStepRange): Replace parseToDouble to parseToNumber.
- html/DateTimeInputType.cpp:
(WebCore::DateTimeInputType::createStepRange): Replace parseToDouble to parseToNumber.
- html/DateTimeLocalInputType.cpp:
(WebCore::DateTimeLocalInputType::createStepRange): Replace parseToDouble to parseToNumber.
- html/InputType.cpp:
(WebCore::InputType::rangeUnderflow): Replace parseToDouble to parseToNumber and doubleValue to numericValue.
(WebCore::InputType::rangeOverflow): ditto.
(WebCore::InputType::isInRange): ditto.
(WebCore::InputType::isOutOfRange): ditto.
(WebCore::InputType::stepMismatch): ditto.
(WebCore::InputType::validationMessage): ditto.
(WebCore::InputType::parseToNumber): ditto.
(WebCore::InputType::parseToDoubleWithDecimalPlaces): ditto.
(WebCore::InputType::stepUpFromRenderer): dttio.
- html/InputType.h:
(InputType):
- html/MonthInputType.cpp:
(WebCore::MonthInputType::createStepRange): Replace parseToDouble to parseToNumber.
(WebCore::MonthInputType::parseToNumber): ditto.
- html/MonthInputType.h:
(MonthInputType):
- html/NumberInputType.cpp:
(WebCore::NumberInputType::valueAsNumber): Replace parseToDouble to parseToNumber.
(WebCore::NumberInputType::createStepRange): ditto.
(WebCore::NumberInputType::parseToNumber): ditto.
- html/NumberInputType.h:
- html/RangeInputType.cpp:
(WebCore::ensureMaximum): Added helper function for createStepRange for avoiding cast in max() and using "const".
(WebCore::RangeInputType::valueAsNumber): Replace parseToDouble to parseToNumber.
(WebCore::RangeInputType::createStepRange): ditto.
(WebCore::RangeInputType::handleKeydownEvent): ditto.
(WebCore::RangeInputType::parseToNumber): ditto.
(WebCore::RangeInputType::sanitizeValue): ditto.
- html/RangeInputType.h:
(RangeInputType):
- html/StepRange.cpp:
(WebCore::StepRange::parseStep): Replace DoubleWithDecimalPlaces to NumberWithDecimalPlaces.
(WebCore::StepRange::stepMismatch): Replace parseToDouble to parseToNumber.
- html/StepRange.h:
(WebCore::StepRange::NumberWithDecimalPlaces): Renamed from DoubleWithDecimalPlaces.
(WebCore::StepRange::NumberWithDecimalPlacesOrMissing): Renamed from DoubleWithDecimalPlacesOrMissing.
- html/TimeInputType.cpp:
(WebCore::TimeInputType::createStepRange): Replace parseToDouble to parseToNumber.
- html/WeekInputType.cpp:
(WebCore::WeekInputType::createStepRange): Replace parseToDouble to parseToNumber.
- 7:55 AM Changeset in webkit [119394] by
-
- 8 edits in trunk/Source
BUILD FIX: FeatureDefines.xcconfig should match across projects
Source/JavaScriptCore:
- Configurations/FeatureDefines.xcconfig:
- Add missing ENABLE_LEGACY_CSS_VENDOR_PREFIXES.
Source/WebCore:
- Configurations/FeatureDefines.xcconfig:
- Add missing ENABLE_CSS_IMAGE_RESOLUTION.
Source/WebKit/mac:
- Configurations/FeatureDefines.xcconfig:
- Fix sort order of ENABLE_CSS3_FLEXBOX.
- Fix typo in ENABLE_FONT_BOOSTING.
- Add missing ENABLE_LEGACY_CSS_VENDOR_PREFIXES.
Source/WebKit2:
- Configurations/FeatureDefines.xcconfig:
- Add missing ENABLE_LEGACY_CSS_VENDOR_PREFIXES.
- 7:43 AM Changeset in webkit [119393] by
-
- 3 edits3 adds in trunk
Canvas 2D - Source rectangles that overlap the source image boundary, not supported by drawImage
https://bugs.webkit.org/show_bug.cgi?id=66574
Patch by Justin Novosad <junov@chromium.org> on 2012-06-04
Reviewed by Stephen White.
Source/WebCore:
Test: fast/canvas/drawImage-clipped-source.html
When the source rectangle argument to drawImage is not entirely
contatained by the domain of the source image, the source rectangle
is clipped, and the destination rectangle is clipped proportionately
Old behavior was to silently fail, which did not comply with the spec.
- html/canvas/CanvasRenderingContext2D.cpp:
(WebCore::normalizeAndClipRects):
(WebCore):
(WebCore::CanvasRenderingContext2D::drawImage):
LayoutTests:
Adding a new layout test that verifies that the source and destination
rectangle arguments to drawImage are properly clipped when the source
rectangle is not entirely contained whithin the domian of the source
image. Tests image to canvas draws and canvas to canvas draws.
- fast/canvas/drawImage-clipped-source-expected.txt: Added.
- fast/canvas/drawImage-clipped-source.html: Added.
- fast/canvas/drawImage-clipped-source.js: Added.
(patternTest.this.testPixel):
(patternTest.this.testRedSquare):
(patternTest.this.testAggregatePattern):
(patternTest):
(drawTestPattern):
(executeTest):
- 6:31 AM Changeset in webkit [119392] by
-
- 7 edits in trunk
[Cairo] Implementation of GraphicsContext does not save/restore ImageInterpolation correctly
https://bugs.webkit.org/show_bug.cgi?id=87985
Patch by Dominik Röttsches <dominik.rottsches@intel.com> on 2012-06-04
Reviewed by Gustavo Noronha Silva.
Source/WebCore:
Moving image interpolation quality setting into state so that it's saved and restored.
Covered by existing test fast/canvas/canvas-imageSmoothingEnabled.html which is now passing.
- platform/graphics/cairo/GraphicsContextPlatformPrivateCairo.h:
(GraphicsContextPlatformPrivate): Removing stale member that was never used.
- platform/graphics/cairo/PlatformContextCairo.cpp:
(WebCore::PlatformContextCairo::State::State):
(PlatformContextCairo::State):
(WebCore::PlatformContextCairo::PlatformContextCairo):
(WebCore::PlatformContextCairo::drawSurfaceToContext):
(WebCore::PlatformContextCairo::setImageInterpolationQuality):
(WebCore):
(WebCore::PlatformContextCairo::imageInterpolationQuality):
- platform/graphics/cairo/PlatformContextCairo.h:
(PlatformContextCairo): Moving implementations for image interpolation quality setter into implementation file.
LayoutTests:
Unskipping fast/canvas/canvas-imageSmoothingEnabled.html now that it passes.
- platform/efl/TestExpectations:
- platform/gtk/TestExpectations:
- 6:23 AM Changeset in webkit [119391] by
-
- 7 edits2 adds in trunk
Fix assert in non-scaling strokes in patterns
https://bugs.webkit.org/show_bug.cgi?id=88198
Source/WebCore:
When calculating the CTM for a rect or path in a pattern we need to use
an SVGPatternElement's local transform. Pattern did not expose the transform
because it does not extend from from SVGStyledLocatable or SVGStyledTransformable.
This patch exposes the pattern's transform in localCoordinateSpaceTransform.
SVGGradientElement does not need this special handling because the children
of a gradient element are restricted to not include shape and structural elements
that would need to calculate a CTM.
Before this patch the added test would assert.
This patch is a prerequisite to a major refactoring of RenderSVGShape &
friends, and the test will be unskipped in a followup patch.
Reviewed by Dirk Schulze.
Tests: svg/stroke/non-scaling-stroke-pattern-expected.svg
svg/stroke/non-scaling-stroke-pattern.svg
- svg/SVGPatternElement.cpp:
(WebCore::SVGPatternElement::localCoordinateSpaceTransform):
(WebCore):
- svg/SVGPatternElement.h:
(SVGPatternElement):
- svg/SVGStyledElement.cpp:
(WebCore::SVGStyledElement::localCoordinateSpaceTransform):
LayoutTests:
Reviewed by Dirk Schulze.
- platform/chromium/TestExpectations:
- platform/mac/TestExpectations:
- svg/stroke/non-scaling-stroke-pattern-expected.svg: Added.
- svg/stroke/non-scaling-stroke-pattern.svg: Added.
- 6:15 AM Changeset in webkit [119390] by
-
- 2 edits in trunk/Source/WebKit2
[Qt] Tap-to-zoom overshoot animation.
https://bugs.webkit.org/show_bug.cgi?id=87108
Patch by Allan Sandfeld Jensen <allan.jensen@nokia.com> on 2012-06-04
Reviewed by Tor Arne Vestbø.
Reverts the change in animation, so the default is more "neutral".
This should be made configurable later.
- UIProcess/qt/QtViewportInteractionEngine.cpp:
(WebKit::QtViewportInteractionEngine::animatePageItemRectVisible):
- 5:38 AM Changeset in webkit [119389] by
-
- 4 edits1 copy1 add in trunk
Web Inspector: speed up _calculateRetainedSizes function
https://bugs.webkit.org/show_bug.cgi?id=87863
Patch by Alexei Filippov <alexeif@chromium.org> on 2012-06-04
Reviewed by Yury Semikhatsky.
PerformanceTests:
- inspector/heap-snapshot-advanced.html: Added.
- inspector/heap-snapshot-performance-test.js: Copied from PerformanceTests/inspector/heap-snapshot.html.
(test.performanceTest.step0):
(test.performanceTest.step1):
(test.performanceTest.step2):
(test.performanceTest.cleanup):
(test.performanceTest.done):
(test.performanceTest):
(test):
- inspector/heap-snapshot.html:
Source/WebCore:
- inspector/front-end/HeapSnapshot.js:
(WebInspector.HeapSnapshot.prototype._calculateRetainedSizes):
- 5:35 AM Changeset in webkit [119388] by
-
- 5 edits in trunk/LayoutTests
Unreviewed. Disable heap profiler test on mac platform.
- platform/mac/Skipped:
- platform/qt/Skipped:
- platform/win/Skipped:
- platform/wincairo/Skipped:
- 5:17 AM Changeset in webkit [119387] by
-
- 2 edits in trunk/Source/WebKit/qt
Unreviewed, rolling out r119384.
http://trac.webkit.org/changeset/119384
https://bugs.webkit.org/show_bug.cgi?id=88227
Need to update some tests, I'll roll in later. (Requested by
Zoltan on #webkit).
Patch by Sheriff Bot <webkit.review.bot@gmail.com> on 2012-06-04
- WebCoreSupport/FrameLoaderClientQt.cpp:
(WebCore::FrameLoaderClientQt::dispatchWillSendRequest):
- 5:02 AM Changeset in webkit [119386] by
-
- 2 edits in trunk/Source/WebCore
Web Inspector: add unknown size to the memory pie-chart legend
https://bugs.webkit.org/show_bug.cgi?id=88081
Patch by Yury Semikhatsky <yurys@chromium.org> on 2012-06-04
Reviewed by Pavel Feldman.
Added "Unknown" part size to the memory pie-chart legend.
- inspector/front-end/NativeMemorySnapshotView.js:
(WebInspector.NativeMemoryProfileType.prototype.buttonClicked.didReceiveMemorySnapshot):
(WebInspector.NativeMemoryProfileType.prototype.buttonClicked):
(WebInspector.MemoryBlockViewProperties._initialize):
(WebInspector.NativeMemoryPieChart.prototype._paint):
- 4:31 AM Changeset in webkit [119385] by
-
- 5 edits in trunk/Source/WebKit2
[Qt][WK2] Refactor the tap gesture recognizer
https://bugs.webkit.org/show_bug.cgi?id=88096
Reviewed by Simon Hausmann.
Refactor the tap gesture recognizer so that its internal state machine is
self contained and independent of input event types.
Also change the internal API for tap highlighting to make the code easier
to understand on the caller side.
This patch makes the tap gesture recognizer more robust, consistent with
other gesture recognizers and fixes some issues related to highlighting
and link activation with respect to native touch events.
- UIProcess/qt/QtTapGestureRecognizer.cpp:
(WebKit::QtTapGestureRecognizer::QtTapGestureRecognizer):
(WebKit::QtTapGestureRecognizer::withinDistance):
(WebKit::QtTapGestureRecognizer::update):
(WebKit::QtTapGestureRecognizer::finish):
(WebKit::QtTapGestureRecognizer::cancel):
(WebKit::QtTapGestureRecognizer::highlightTimeout):
(WebKit::QtTapGestureRecognizer::singleTapTimeout):
(WebKit::QtTapGestureRecognizer::tapAndHoldTimeout):
(WebKit::QtTapGestureRecognizer::reset):
- UIProcess/qt/QtTapGestureRecognizer.h:
(QtTapGestureRecognizer):
- UIProcess/qt/QtWebPageEventHandler.cpp:
(WebKit::QtWebPageEventHandler::QtWebPageEventHandler):
(WebKit::QtWebPageEventHandler::activateTapHighlight):
(WebKit::QtWebPageEventHandler::deactivateTapHighlight):
(WebKit):
(WebKit::QtWebPageEventHandler::handleSingleTapEvent):
(WebKit::QtWebPageEventHandler::handleDoubleTapEvent):
(WebKit::QtWebPageEventHandler::doneWithTouchEvent):
- UIProcess/qt/QtWebPageEventHandler.h:
(QtWebPageEventHandler):
- 4:26 AM Changeset in webkit [119384] by
-
- 2 edits in trunk/Source/WebKit/qt
[Qt] Qt DRT / WTR should be able to load external resources
https://bugs.webkit.org/show_bug.cgi?id=87326
Reviewed by Ryosuke Niwa.
Allow external resources to be loaded when the main frame's URL is also an external resource.
This change is analogous to r118231 for Mac port and to r119153 Chromium port.
- WebCoreSupport/FrameLoaderClientQt.cpp:
(WebCore::blockRequest):
(WebCore):
(WebCore::isLocalhost):
(WebCore::hostIsUsedBySomeTestsToGenerateError):
(WebCore::FrameLoaderClientQt::dispatchWillSendRequest):
- 4:12 AM Changeset in webkit [119383] by
-
- 1 copy in releases/WebKitGTK/webkit-1.9.3
Tagging the WebKitGTK+ 1.9.3 release
- 4:00 AM Changeset in webkit [119382] by
-
- 4 edits in trunk
Unreviewed. Update NEWS and configure.ac for 1.9.3 release
.:
- configure.ac: Bump version number.
Source/WebKit/gtk:
- NEWS: Added release notes for 1.9.3.
- 3:24 AM Changeset in webkit [119381] by
-
- 7 edits in trunk/Source/WebKit
[Qt] Don't use WebKit HAVE() macro in public Qt headers
Reviewed by Kenneth Rohde Christiansen.
- 3:09 AM Changeset in webkit [119380] by
-
- 3 edits2 adds in trunk
Adding explicit console error for duplicated CSP directives.
https://bugs.webkit.org/show_bug.cgi?id=88193
Source/WebCore:
This patch adds an explicit error message for duplicated CSP
directives instead of reusing the "unrecognized directive" error.
Patch by Mike West <mkwst@chromium.org> on 2012-06-04
Reviewed by Adam Barth.
Test: http/tests/security/contentSecurityPolicy/duplicate-directive.html
- page/ContentSecurityPolicy.cpp:
(CSPDirectiveList):
(WebCore::CSPDirectiveList::logDuplicateDirective):
(WebCore):
(WebCore::CSPDirectiveList::parseReportURI):
(WebCore::CSPDirectiveList::setCSPDirective):
(WebCore::CSPDirectiveList::applySandboxPolicy):
(WebCore::CSPDirectiveList::addDirective):
LayoutTests:
Patch by Mike West <mkwst@chromium.org> on 2012-06-04
Reviewed by Adam Barth.
- http/tests/security/contentSecurityPolicy/duplicate-directive-expected.txt: Added.
- http/tests/security/contentSecurityPolicy/duplicate-directive.html: Added.
- 2:48 AM Changeset in webkit [119379] by
-
- 5 edits in trunk
REGRESSION(r117929) [Forms] input type=number thinks 0 is not a number
https://bugs.webkit.org/show_bug.cgi?id=88208
Reviewed by Kent Tamura.
Source/WebCore:
This patch fixes invalid handling of return value of parseToDoubleForNumberType
in NumberInputType::isAcceptableValue to use isfinite() function for checking
whether specified string is valid number or not. Before r117929, return value
of parseToDoubleForNumberType was bool, but after r117929, it is double.
Test: fast/forms/number/input-number-from-renderer.html
- html/NumberInputType.cpp:
(WebCore::NumberInputType::isAcceptableValue): Use isfinite()
LayoutTests:
This patch introduced function checkIt() for code sharing and added
more test cases to input-number-commit-valid-only.htm
- fast/forms/number/input-number-commit-valid-only-expected.txt: Changed for additional tests.
- fast/forms/number/input-number-commit-valid-only.html: Changed.
- 2:32 AM Changeset in webkit [119378] by
-
- 2 edits in trunk/Source/WebCore
Crash in WebCore::RenderView::getRetainedWidgets
https://bugs.webkit.org/show_bug.cgi?id=88217
Reviewed by Kenneth Rohde Christiansen.
There is no known repro, the crash has been seen on the field. It is likely that it happens with frame
flattening enabled only.
Null check root renderer in performPostLayoutTasks. The root can legitimely be null for several reasons and
is mostly null checked everywhere else.
- page/FrameView.cpp:
(WebCore::FrameView::performPostLayoutTasks):
- 2:28 AM Changeset in webkit [119377] by
-
- 4 edits in trunk
[EFL] Reset "WebKitTabToLinksPreferenceKey" in DumpRenderTreeChrome::resetDefaultsToConsistentValues
https://bugs.webkit.org/show_bug.cgi?id=87668
Patch by Mikhail Pozdnyakov <mikhail.pozdnyakov@intel.com> on 2012-06-04
Reviewed by Ryosuke Niwa.
Tools:
Layout test controller "WebKitTabToLinksPreferenceKey" setting should be reset to false
otherwise LayoutTests/fast/html/tab-order.html is failing as it has a link inside the test description which might
be included in focus chain (if "WebKitTabToLinksPreferenceKey" setting is set to true).
- DumpRenderTree/efl/DumpRenderTreeChrome.cpp:
(DumpRenderTreeChrome::resetDefaultsToConsistentValues):
LayoutTests:
- platform/efl/Skipped: Unskipped fast/html/tab-order.html.
- 2:06 AM Changeset in webkit [119376] by
-
- 3 edits in trunk/Source/WebKit/gtk
[GTK] Remove geoclue dependency from WebKit API Layer
https://bugs.webkit.org/show_bug.cgi?id=87801
Reviewed by Martin Robinson.
Make GeolocationClient for WebKitGTK+ use the new Geoclue-based
geolocation provider available in WebCore.
- WebCoreSupport/GeolocationClientGtk.cpp:
(WebKit):
(WebKit::GeolocationClient::GeolocationClient):
(WebKit::GeolocationClient::startUpdating):
(WebKit::GeolocationClient::stopUpdating):
(WebKit::GeolocationClient::setEnableHighAccuracy):
(WebKit::GeolocationClient::notifyPositionChanged):
(WebKit::GeolocationClient::notifyErrorOccurred):
- WebCoreSupport/GeolocationClientGtk.h:
(GeolocationClient):
- 1:52 AM Changeset in webkit [119375] by
-
- 4 edits in trunk/Source/WebCore
Use SelectorQuery in Element::webkitMatchesSelector()
https://bugs.webkit.org/show_bug.cgi?id=88209
Reviewed by Antti Koivisto.
By using SelectorQuery in Element::webkitMatchesSelector(),
we can not only simplify the code but also improve the performance
thanks to the cache mechanism of SelectorQuery which was implemented
in bug 87942.
Tests: fast/dom/SelectorAPI/*. No change in test results.
- dom/Element.cpp:
(WebCore::Element::webkitMatchesSelector):
- dom/SelectorQuery.cpp:
(WebCore::SelectorQuery::matches):
(WebCore):
- dom/SelectorQuery.h:
(SelectorQuery):
- 1:45 AM Changeset in webkit [119374] by
-
- 11 edits2 adds in trunk
[EFL] Refactor ewk_view_context_paint code.
https://bugs.webkit.org/show_bug.cgi?id=85588
Patch by Tomasz Morawski <t.morawski@samsung.com> on 2012-06-04
Reviewed by Hajime Morita.
Source/WebKit:
Added new ewk_paint_context.cpp file to source list.
- PlatformEfl.cmake:
Source/WebKit/efl:
The main purpose of this change is reduce size of ewk_view.h/cpp files and make use of ewk_view_context_paint
object by ewk_view_single and ewk_view_tiled instead of direct operations on cairo.
The code is more objective and looks very simple now.
Remove ewk_view_context_paint code from ewk_view h/cpp file and move it to separate new ewk_context_paint file.
The ewk_view_context_paint has been renamed to ewk_context_paint and it is not binded to view on creation time.
- ewk/ewk_paint_context.cpp: Added.
Functions renamed and moved from ewk_view to ewk_paint_context file.
(ewk_paint_context_new):
(ewk_paint_context_from_image_new): New function.
(ewk_paint_context_from_image_data_new): New function.
(ewk_paint_context_free):
(ewk_paint_context_save):
(ewk_paint_context_restore):
(ewk_paint_context_clip):
(ewk_paint_context_scale):
(ewk_paint_context_translate):
(ewk_paint_context_paint): Added FrameView parameter.
(ewk_paint_context_paint_contents): Added FrameView parameter.
- ewk/ewk_paint_context_private.h: Added.
(_Ewk_Paint_Context):
- ewk/ewk_private.h: Functions ewk_view_paint and ewk_view_paint_contents moved from ewk_view.h.
- ewk/ewk_tiled_backing_store.h: Removed unused include.
- ewk/ewk_view.cpp: Moved all function releated to paint context object to new ewk_paint_context file.
(_Ewk_View_Private_Data):
(ewk_view_paint): Changed function according change in ewk_paint_context_paint function.
(ewk_view_paint_contents): Changed function according change in ewk_paint_context_paint_contents function.
- ewk/ewk_view.h: Removed all paint context functions and ewk_view_paint, ewk_view_paint_contents function from
public API.
- ewk/ewk_view_single.cpp:
(_ewk_view_single_smart_repaints_process):
Refactor of _ewk_view_single_smart_repaints_process function.
Removed goto statements.
Removed variables declaration from function beginning.
- ewk/ewk_view_tiled.cpp:
(_ewk_view_tiled_render_cb): Make use of ewk_paint_context object instead of direction operation on cairo.
Looks simpler and more objective.
Tools:
- DumpRenderTree/efl/CMakeLists.txt: Added include directory.
- 1:04 AM WebKitGTK/WebKit2Roadmap edited by
- (diff)
- 1:03 AM WebKitGTK/WebKit2Roadmap edited by
- (diff)
- 1:02 AM WebKitGTK/WebKit2Roadmap edited by
- (diff)
- 1:01 AM WebKitGTK/WebKit2Roadmap edited by
- (diff)
- 12:00 AM Changeset in webkit [119373] by
-
- 2 edits in trunk/Tools
[EFL] Compilation warning in DumpRenderTreeView.cpp on 64-Bit
https://bugs.webkit.org/show_bug.cgi?id=87869
Patch by Christophe Dumez <Christophe Dumez> on 2012-06-04
Reviewed by Andreas Kling.
Replace %lld by PRId64 macro to print an int64_t value and silent a
compilation warning on 64-Bit.
- DumpRenderTree/efl/DumpRenderTreeView.cpp:
(onExceededApplicationCacheQuota):
