Timeline
Jan 30, 2012:
- 11:50 PM Changeset in webkit [106339] by
-
- 2 edits in trunk/LayoutTests
[Qt] Unreviewed gardening.
- platform/qt/Skipped: Skip failing tests.
- 11:45 PM Changeset in webkit [106338] by
-
- 2 edits in trunk/Source/WebCore
Web Inspector: [BlackBerry] Add BlackBerry UA Strings to the Inspector's UA switcher
https://bugs.webkit.org/show_bug.cgi?id=77343
Patch by Konrad Piascik <kpiascik@rim.com> on 2012-01-30
Reviewed by Yury Semikhatsky.
- inspector/front-end/SettingsScreen.js:
(WebInspector.SettingsScreen.prototype._createUserAgentSelectRowElement.get const):
- 11:37 PM Changeset in webkit [106337] by
-
- 10 edits2 adds in trunk
Web Inspector: should be possible to step through all event listeners when event listener breakpoint is hit
https://bugs.webkit.org/show_bug.cgi?id=77331
Source/WebCore:
Inspector instrumentation is called before and after each event handler invokation.
In case inspector front-end is closed it is no-op, otherwise it may stop execution
on an event listener breakpoint.
Reviewed by Pavel Feldman.
Test: inspector/debugger/step-through-event-listeners.html
- dom/EventTarget.cpp:
(WebCore::EventTarget::fireEventListeners):
- inspector/InspectorInstrumentation.cpp:
(WebCore::InspectorInstrumentation::willDispatchEventImpl):
(WebCore::InspectorInstrumentation::willHandleEventImpl):
(WebCore):
(WebCore::InspectorInstrumentation::didHandleEventImpl):
(WebCore::InspectorInstrumentation::didDispatchEventImpl):
(WebCore::InspectorInstrumentation::willDispatchEventOnWindowImpl):
(WebCore::InspectorInstrumentation::didDispatchEventOnWindowImpl):
- inspector/InspectorInstrumentation.h:
(InspectorInstrumentation):
(WebCore::InspectorInstrumentation::willHandleEvent):
(WebCore):
(WebCore::InspectorInstrumentation::didHandleEvent):
LayoutTests:
Test that debugger will stop in each event listener when pausing on an event listener
breakpoint.
Reviewed by Pavel Feldman.
- inspector/debugger/step-through-event-listeners-expected.txt: Added.
- inspector/debugger/step-through-event-listeners.html: Added.
- platform/gtk/Skipped:
- platform/mac/Skipped:
- platform/qt/Skipped:
- platform/win/Skipped:
- platform/wincairo/Skipped:
- 10:36 PM Changeset in webkit [106336] by
-
- 5 edits in trunk
Attach light children after removing a shadow root.
https://bugs.webkit.org/show_bug.cgi?id=74267
Reviewed by Ryosuke Niwa.
Source/WebCore:
Tests: fast/dom/shadow/dynamically-created-shadow-root-expected.html
fast/dom/shadow/dynamically-created-shadow-root.html:
- dom/Element.cpp:
(WebCore::Element::removeShadowRoot):
LayoutTests:
- fast/dom/shadow/dynamically-created-shadow-root-expected.html:
- fast/dom/shadow/dynamically-created-shadow-root.html:
- 10:23 PM Changeset in webkit [106335] by
-
- 22 edits in trunk/Source
Unreviewed, rolling out r106324.
http://trac.webkit.org/changeset/106324
https://bugs.webkit.org/show_bug.cgi?id=77406
Broke CCLayerTreeHostTestLayerOcclusion.runMultiThread and
runSingleThread (Requested by yuzo1 on #webkit).
Patch by Sheriff Bot <webkit.review.bot@gmail.com> on 2012-01-30
Source/WebCore:
- platform/graphics/FloatRect.cpp:
- platform/graphics/FloatRect.h:
(WebCore):
- platform/graphics/chromium/Canvas2DLayerChromium.cpp:
(WebCore::Canvas2DLayerChromium::paintContentsIfDirty):
- platform/graphics/chromium/Canvas2DLayerChromium.h:
(WebCore):
(Canvas2DLayerChromium):
- platform/graphics/chromium/ContentLayerChromium.cpp:
(WebCore::ContentLayerChromium::paintContentsIfDirty):
- platform/graphics/chromium/ContentLayerChromium.h:
(WebCore):
(ContentLayerChromium):
- platform/graphics/chromium/ImageLayerChromium.cpp:
(WebCore::ImageLayerChromium::paintContentsIfDirty):
- platform/graphics/chromium/ImageLayerChromium.h:
(WebCore):
(ImageLayerChromium):
- platform/graphics/chromium/LayerChromium.cpp:
- platform/graphics/chromium/LayerChromium.h:
(WebCore):
(WebCore::LayerChromium::paintContentsIfDirty):
(LayerChromium):
- platform/graphics/chromium/RenderSurfaceChromium.h:
- platform/graphics/chromium/TiledLayerChromium.cpp:
- platform/graphics/chromium/TiledLayerChromium.h:
(WebCore):
- platform/graphics/chromium/cc/CCLayerTreeHost.cpp:
(WebCore::CCLayerTreeHost::paintContentsIfDirty):
(WebCore::CCLayerTreeHost::paintMaskAndReplicaForRenderSurface):
(WebCore::CCLayerTreeHost::paintLayerContents):
- platform/graphics/chromium/cc/CCLayerTreeHost.h:
(WebCore):
- platform/graphics/chromium/cc/CCQuadCuller.cpp:
(WebCore::enclosedIntRect):
(WebCore):
Source/WebKit/chromium:
- tests/CCLayerTreeHostCommonTest.cpp:
(WebCore):
- tests/CCLayerTreeHostTest.cpp:
(WTF::ContentLayerChromiumWithUpdateTracking::paintContentsIfDirty):
- tests/Canvas2DLayerChromiumTest.cpp:
(WebCore::Canvas2DLayerChromiumTest::fullLifecycleTest):
- tests/TiledLayerChromiumTest.cpp:
(FakeLayerTextureUpdater):
(WTF::FakeTiledLayerChromium::paintContentsIfDirty):
(WTF::FakeLayerTextureUpdater::prepareToUpdate):
- 10:20 PM Changeset in webkit [106334] by
-
- 3 edits in trunk
Build fix after r106321.
- Websites/webkit-perf.appspot.com/report_handler.py:
- Websites/webkit-perf.appspot.com/manifest_handler.py:
(ManifestHandler.get):
- 9:51 PM Changeset in webkit [106333] by
-
- 2 edits in trunk/Tools
[wx] Unreviewed. Update download URLs to new domain.
- 9:47 PM Changeset in webkit [106332] by
-
- 3 edits2 adds in trunk
Fix ASSERT fail within AudioBus::processWithGainFrom()
https://bugs.webkit.org/show_bug.cgi?id=76685
Patch by Raymond Liu <raymond.liu@intel.com> on 2012-01-30
Reviewed by Daniel Bates.
Source/WebCore:
Test: webaudio/audionode-connect-order.html
- webaudio/AudioBasicProcessorNode.cpp:
(WebCore::AudioBasicProcessorNode::AudioBasicProcessorNode):
LayoutTests:
- webaudio/audionode-connect-order-expected.txt: Added.
- webaudio/audionode-connect-order.html: Added.
- 9:04 PM Changeset in webkit [106331] by
-
- 6 edits10 adds in trunk
<style scoped>: Implement scoped stylesheets and basic application
https://bugs.webkit.org/show_bug.cgi?id=73190
Source/WebCore:
Implementing support for <style scoped>:
Add a vector stack to CSSStyleSelector that keeps track of encountered scoping elements. This is
used for O(1) access to all relevant style sheets for a given element.
Adapt matching of author style sheets to also consult appropriate scoped sheets.
Finally, prevent style sharing from crossing into/out of scoped style boundaries.
Reviewed by Antti Koivisto.
Tests: fast/css/style-scoped/style-scoped-attach.html
fast/css/style-scoped/style-scoped-basic.html
fast/css/style-scoped/style-scoped-detach.html
fast/css/style-scoped/style-scoped-remove-scoped.html
fast/css/style-scoped/style-scoped-set-scoped.html
- css/CSSStyleSelector.cpp:
(RuleSet):
(WebCore::CSSStyleSelector::CSSStyleSelector): add code for scoped style sheets
(WebCore::CSSStyleSelector::collectFeatures): ditto
(WebCore):
(WebCore::CSSStyleSelector::determineScopingElement): determine whether an author sheet is scoped (and to which scope), or global
(WebCore::CSSStyleSelector::scopedRuleSetForElement): returns the RuleSet for the <style scoped> contained by the passed-in element (if any), or 0
(WebCore::CSSStyleSelector::appendAuthorStylesheets): add code for scoped style sheets
(WebCore::CSSStyleSelector::setupScopingElementStack): determine scoping element ancestors of the given element
(WebCore::CSSStyleSelector::pushParent): simplify and refactor SelectorChecker::pushParent, as code in CSStyleSelector needs partial access
(WebCore::CSSStyleSelector::popParent): ditto
(WebCore::CSSStyleSelector::sortAndTransferMatchedRules): helper function
(WebCore::CSSStyleSelector::matchAuthorRules): use AuthorRuleSetIterator to iterate over all relevant RuleSets
(WebCore::CSSStyleSelector::matchRules): adapt for scoped style rules
(WebCore::CSSStyleSelector::matchAllRules): ditto
(WebCore::CSSStyleSelector::locateCousinList): prevent style sharing across scope boundaries
(WebCore::CSSStyleSelector::canShareStyleWithElement): ditto
(WebCore::CSSStyleSelector::locateSharedStyle): ditto
(WebCore::CSSStyleSelector::pseudoStyleForElement): changed call to matchAuthorRules
(WebCore::CSSStyleSelector::styleForPage): add comment
(WebCore::CSSStyleSelector::checkRegionStyle): add global scope parameter
(WebCore::CSSStyleSelector::pseudoStyleRulesForElement): changed call to matchAuthorRules
(WebCore::RuleSet::addRulesFromSheet): adapt for scoped style rules
- css/CSSStyleSelector.h:
(CSSStyleSelector):
(WebCore::CSSStyleSelector::ScopeStackFrame::ScopeStackFrame): struct holding an Element pointer and a RuleSet pointer, to be used in a Vector
(WebCore::CSSStyleSelector::scopingElementStackIsConsistent): returns if the last seen parent matches the passed-in element
- css/SelectorChecker.cpp:
(WebCore::SelectorChecker::setupParentStack): set up the parent stack (refactoring)
(WebCore::SelectorChecker::pushParent): simplify and refactor
- css/SelectorChecker.h:
(WebCore::SelectorChecker::popParent): ditto
(WebCore::SelectorChecker::parentStackIsEmpty): ditto
(WebCore::SelectorChecker::parentStackIsConsistent): make parameter const
LayoutTests:
Test that <style scoped> affects styles and is updated correctly:
-) basic test
-) test setting/un-setting the 'scoped' attribute
-) test inserting/removing a whole <style scoped> element
Reviewed by Antti Koivisto.
- fast/css/style-scoped/style-scoped-attach-expected.txt: Added.
- fast/css/style-scoped/style-scoped-attach.html: Added.
- fast/css/style-scoped/style-scoped-basic-expected.txt: Added.
- fast/css/style-scoped/style-scoped-basic.html: Added.
- fast/css/style-scoped/style-scoped-detach-expected.txt: Added.
- fast/css/style-scoped/style-scoped-detach.html: Added.
- fast/css/style-scoped/style-scoped-remove-scoped-expected.txt: Added.
- fast/css/style-scoped/style-scoped-remove-scoped.html: Added.
- fast/css/style-scoped/style-scoped-set-scoped-expected.txt: Added.
- fast/css/style-scoped/style-scoped-set-scoped.html: Added.
- 9:03 PM Changeset in webkit [106330] by
-
- 5 edits in trunk/Source/WebKit/efl
[EFL] Define the names of view smart class
https://bugs.webkit.org/show_bug.cgi?id=76370
Patch by Eunsol Park <eunsol47.park@samsung.com> on 2012-01-30
Reviewed by Andreas Kling.
Strings of ewk_tiled_view's name were defined differently,
so the definitions were modified to make users not confused.
- ewk/ewk_private.h:
- ewk/ewk_view.cpp:
- ewk/ewk_view_single.cpp:
(_ewk_view_single_smart_class_new):
- ewk/ewk_view_tiled.cpp:
(ewk_view_tiled_smart_set):
(_ewk_view_tiled_smart_class_new):
- 8:59 PM Changeset in webkit [106329] by
-
- 3 edits2 adds in tags/Safari-535.18.5
Merge r106305.
- 8:52 PM Changeset in webkit [106328] by
-
- 5 edits in tags/Safari-535.18.5/Source
Versioning.
- 8:42 PM Changeset in webkit [106327] by
-
- 1 copy in tags/Safari-535.18.5
New tag.
- 8:39 PM Changeset in webkit [106326] by
-
- 4 edits in trunk
[wx] Update build dirs and includes after recent trunk changes.
- 8:37 PM Changeset in webkit [106325] by
-
- 2 edits in trunk/LayoutTests
[Chromium] Unreviewed test expectation change
plugins/crash-restoring-plugin-page-from-page-cache.html fails or timeouts.
- platform/chromium/test_expectations.txt:
- 8:03 PM Changeset in webkit [106324] by
-
- 22 edits in trunk/Source
[chromium] Compute occlusion during paint loop
https://bugs.webkit.org/show_bug.cgi?id=76858
Patch by Dana Jansens <danakj@chromium.org> on 2012-01-30
Reviewed by James Robinson.
Source/WebCore:
New unit tests in TiledLayerChromiumTest.cpp, CCLayerTreeHostCommonTest.cpp, CCLayerTreeHostTest.cpp
- platform/graphics/FloatRect.cpp:
(WebCore::enclosedIntRect):
(WebCore):
- platform/graphics/FloatRect.h:
(WebCore):
- platform/graphics/chromium/Canvas2DLayerChromium.cpp:
(WebCore::Canvas2DLayerChromium::paintContentsIfDirty):
- platform/graphics/chromium/Canvas2DLayerChromium.h:
(Canvas2DLayerChromium):
- platform/graphics/chromium/ContentLayerChromium.cpp:
(WebCore::ContentLayerChromium::paintContentsIfDirty):
- platform/graphics/chromium/ContentLayerChromium.h:
(ContentLayerChromium):
- platform/graphics/chromium/ImageLayerChromium.cpp:
(WebCore::ImageLayerChromium::paintContentsIfDirty):
- platform/graphics/chromium/ImageLayerChromium.h:
(ImageLayerChromium):
- platform/graphics/chromium/LayerChromium.cpp:
(WebCore::LayerChromium::contentToScreenSpaceTransform):
(WebCore):
(WebCore::LayerChromium::addSelfToOccludedScreenSpace):
(WebCore::LayerChromium::isPaintedAxisAlignedInScreen):
- platform/graphics/chromium/LayerChromium.h:
(WebCore):
(WebCore::LayerChromium::paintContentsIfDirty):
(LayerChromium):
- platform/graphics/chromium/RenderSurfaceChromium.h:
(RenderSurfaceChromium):
- platform/graphics/chromium/TiledLayerChromium.cpp:
(WebCore::TiledLayerChromium::addSelfToOccludedScreenSpace):
- platform/graphics/chromium/TiledLayerChromium.h:
(WebCore):
():
- platform/graphics/chromium/cc/CCLayerTreeHost.cpp:
(WebCore::CCLayerTreeHost::paintContentsIfDirty):
(WebCore::CCLayerTreeHost::paintMaskAndReplicaForRenderSurface):
(RenderSurfaceRegion):
(WebCore):
(WebCore::pushTargetRenderSurfaceRegion):
(WebCore::popAndPushTargetRenderSurfaceRegion):
(WebCore::CCLayerTreeHost::paintLayerContents):
- platform/graphics/chromium/cc/CCLayerTreeHost.h:
(WebCore):
():
- platform/graphics/chromium/cc/CCQuadCuller.cpp:
Source/WebKit/chromium:
- tests/CCLayerTreeHostCommonTest.cpp:
(WebCore::TEST):
(WebCore):
- tests/CCLayerTreeHostTest.cpp:
(WTF::ContentLayerChromiumWithUpdateTracking::paintContentsIfDirty):
(WTF):
(TestLayerChromium):
(WTF::TestLayerChromium::create):
(WTF::TestLayerChromium::paintContentsIfDirty):
(WTF::TestLayerChromium::drawsContent):
(WTF::TestLayerChromium::occludedScreenSpace):
(WTF::TestLayerChromium::clearOccludedScreenSpace):
(WTF::TestLayerChromium::TestLayerChromium):
(WTF::setLayerPropertiesForTesting):
(CCLayerTreeHostTestLayerOcclusion):
(WTF::CCLayerTreeHostTestLayerOcclusion::CCLayerTreeHostTestLayerOcclusion):
(WTF::CCLayerTreeHostTestLayerOcclusion::beginTest):
(WTF::CCLayerTreeHostTestLayerOcclusion::afterTest):
- tests/Canvas2DLayerChromiumTest.cpp:
(WebCore::Canvas2DLayerChromiumTest::fullLifecycleTest):
- tests/TiledLayerChromiumTest.cpp:
(WTF::FakeLayerTextureUpdater::setOpaquePaintRect):
(FakeLayerTextureUpdater):
(WTF::FakeTiledLayerChromium::paintContentsIfDirty):
(WTF::FakeLayerTextureUpdater::prepareToUpdate):
(WTF::TEST):
(WTF):
- 7:53 PM Changeset in webkit [106323] by
-
- 14 edits2 adds in trunk/Source/WebCore
Add ElementAttributeData class to replace internal uses of NamedNodeMap
https://bugs.webkit.org/show_bug.cgi?id=77233
Reviewed by Andreas Kling.
Move part of non-DOM functionality of NamedNodeMap into a separate class. This is
the first step toward the goal of separating NamedNodeMap from internal attribute
storage, as described in https://bugs.webkit.org/show_bug.cgi?id=75069.
The internal attribute storage is exposed as attributeData() in Element, and when
necessary (because it has no back pointer to Element) via methods in Element.
No new tests. Except from setClass() change this is just moving the code, no new
feature was added.
- CMakeLists.txt:
- GNUmakefile.list.am:
- Target.pri:
- WebCore.gypi:
- WebCore.xcodeproj/project.pbxproj:
- dom/DOMAllInOne.cpp:
- dom/Element.cpp:
(WebCore::Element::idAttributeChanged):
- dom/Element.h:
(WebCore::Element::attributeData):
(WebCore::Element::ensureAttributeData):
(Element):
(WebCore::Element::idForStyleResolution):
- dom/ElementAttributeData.cpp: Added.
(WebCore):
(WebCore::ElementAttributeData::setClass): the only caller of this function
already deal with the case when the element has no class, so don't do it here.
- dom/ElementAttributeData.h: Added.
(WebCore):
(ElementAttributeData):
(WebCore::ElementAttributeData::clearClass):
(WebCore::ElementAttributeData::classNames):
(WebCore::ElementAttributeData::idForStyleResolution):
(WebCore::ElementAttributeData::setIdForStyleResolution):
(WebCore::ElementAttributeData::ElementAttributeData):
- dom/NamedNodeMap.cpp:
(WebCore::NamedNodeMap::clearAttributes):
- dom/NamedNodeMap.h:
(WebCore::NamedNodeMap::attributeData):
(NamedNodeMap):
- dom/StyledElement.cpp:
(WebCore::StyledElement::classAttributeChanged):
- dom/StyledElement.h:
(WebCore::StyledElement::classNames):
- html/ClassList.cpp:
(WebCore::ClassList::classNames):
- 7:28 PM Changeset in webkit [106322] by
-
- 2 edits in releases/WebKitGTK/webkit-1.6
Small build fixes.
- configure.ac: Correct the version number to be 1.6.2 and fix
a bad merge of gtkdoc patches.
- 7:14 PM Changeset in webkit [106321] by
-
- 8 edits in trunk
Python code in webkit-perf.appspot.com uses camelCase
https://bugs.webkit.org/show_bug.cgi?id=77392
Reviewed by Adam Barth.
Use PEP8 style naming conventions instead of camelCase.
- Websites/webkit-perf.appspot.com/create_handler.py:
(CreateHandler.post):
(CreateHandler._create_builder):
(CreateHandler._create_branch):
(CreateHandler._create_platform):
- Websites/webkit-perf.appspot.com/dashboard_handler.py:
(DashboardHandler.get):
- Websites/webkit-perf.appspot.com/manifest_handler.py:
(ManifestHandler.get):
- Websites/webkit-perf.appspot.com/merge_tests_handler.py:
(MergeTestsHandler.post):
- Websites/webkit-perf.appspot.com/models.py:
(create_in_transaction_with_numeric_id_holder):
(delete_model_with_numeric_id_holder):
(modelFromNumericId):
(Builder.authenticate):
(Builder.hashed_password):
(Test.cache_key):
- Websites/webkit-perf.appspot.com/report_handler.py:
(ReportHandler.post):
(ReportHandler._model_by_key_name_in_body_or_error):
(ReportHandler._integer_in_body):
(ReportHandler._timestamp_in_body):
(ReportHandler.bypass_authentication):
(ReportHandler._results_are_valid):
(ReportHandler._results_are_valid._is_float_convertible):
(ReportHandler._create_build_if_possible):
(ReportHandler._create_build_if_possible.execute):
(ReportHandler._add_test_if_needed):
(ReportHandler._add_test_if_needed.execute):
(ReportHandler):
(AdminReportHandler.bypass_authentication):
- Websites/webkit-perf.appspot.com/runs_handler.py:
(RunsHandler.get):
- 7:10 PM Changeset in webkit [106320] by
-
- 12 edits in trunk/Source
Add Plumming to get graphics error messages to JS Console
https://bugs.webkit.org/show_bug.cgi?id=77238
Source/WebCore:
Patch by Gregg Tavares <Gregg Tavares> on 2012-01-30
Reviewed by Kenneth Russell.
No new tests. No change in behavior.
- html/canvas/WebGLRenderingContext.cpp:
(WebGLRenderingContextErrorMessageCallback):
(WebCore::WebGLRenderingContextErrorMessageCallback::WebGLRenderingContextErrorMessageCallback):
(WebCore::WebGLRenderingContextErrorMessageCallback::onErrorMessage):
(WebCore::WebGLRenderingContextErrorMessageCallback::~WebGLRenderingContextErrorMessageCallback):
(WebCore):
(WebCore::WebGLRenderingContext::create):
(WebCore::WebGLRenderingContext::initializeNewContext):
(WebCore::WebGLRenderingContext::~WebGLRenderingContext):
- html/canvas/WebGLRenderingContext.h:
(WebGLRenderingContext):
- platform/graphics/GraphicsContext3D.h:
(ErrorMessageCallback):
(WebCore::GraphicsContext3D::ErrorMessageCallback::~ErrorMessageCallback):
(GraphicsContext3D):
- platform/graphics/cairo/GraphicsContext3DCairo.cpp:
(WebCore::GraphicsContext3D::setErrorMessageCallback):
(WebCore):
- platform/graphics/efl/GraphicsContext3DEfl.cpp:
(WebCore::GraphicsContext3D::setErrorMessageCallback):
(WebCore):
- platform/graphics/qt/GraphicsContext3DQt.cpp:
(WebCore::GraphicsContext3D::setErrorMessageCallback):
(WebCore):
Source/WebKit/chromium:
Patch by Gregg Tavares <gman@google.com> on 2012-01-30
Reviewed by Kenneth Russell.
- public/platform/WebGraphicsContext3D.h:
(WebGraphicsErrorMessageCallback):
(WebKit::WebGraphicsContext3D::WebGraphicsErrorMessageCallback::~WebGraphicsErrorMessageCallback):
(WebGraphicsContext3D):
(WebKit::WebGraphicsContext3D::setErrorMessageCallback):
- src/GraphicsContext3DChromium.cpp:
(WebCore::GraphicsContext3D::~GraphicsContext3D):
(WebCore):
(GraphicsErrorMessageCallbackAdapter):
(WebCore::GraphicsErrorMessageCallbackAdapter::~GraphicsErrorMessageCallbackAdapter):
(WebCore::GraphicsErrorMessageCallbackAdapter::GraphicsErrorMessageCallbackAdapter):
(WebCore::GraphicsErrorMessageCallbackAdapter::onErrorMessage):
(WebCore::GraphicsErrorMessageCallbackAdapter::create):
(WebCore::GraphicsContext3DPrivate::setErrorMessageCallback):
- src/GraphicsContext3DPrivate.h:
(WebCore):
():
- 6:58 PM Changeset in webkit [106319] by
-
- 4 edits in trunk/Source/WebCore
Simplify RenderLayerCompositor::frameViewDidScroll
https://bugs.webkit.org/show_bug.cgi?id=77398
Reviewed by Sam Weinig.
Remove the scrollPosition parameter from RenderLayerCompositor::frameViewDidScroll; we can just get it
from the associated FrameView object.
- page/FrameView.cpp:
(WebCore::FrameView::scrollPositionChanged):
- rendering/RenderLayerCompositor.cpp:
(WebCore::RenderLayerCompositor::frameViewDidChangeSize):
Call frameViewDidScroll() here instead of setting the scroll layer position explicitly.
(WebCore::RenderLayerCompositor::frameViewDidScroll):
(WebCore::RenderLayerCompositor::ensureRootLayer):
- rendering/RenderLayerCompositor.h:
(RenderLayerCompositor):
- 6:55 PM Changeset in webkit [106318] by
-
- 3 edits2 adds in trunk
Don't select the next selectable index when deselecting an option in select elements with size set to a value greater than one.
https://bugs.webkit.org/show_bug.cgi?id=76389
Patch by Pablo Flouret <pablof@motorola.com> on 2012-01-30
Reviewed by Kent Tamura.
This behavior matches the rest of the browsers.
Source/WebCore:
Test: fast/forms/select/option-selecting.html
- html/HTMLSelectElement.cpp:
(WebCore::HTMLSelectElement::optionSelectionStateChanged):
LayoutTests:
- fast/forms/select/option-selecting-expected.txt: Added.
- fast/forms/select/option-selecting.html: Added.
- 6:43 PM Changeset in webkit [106317] by
-
- 2 edits in trunk/Source/WebCore
Scrollbars don't show when scrolling on the scrolling thread
https://bugs.webkit.org/show_bug.cgi?id=77396
<rdar://problem/10710736>
Reviewed by Sam Weinig.
Use ScrollAnimator::scrollToOffsetWithoutAnimation when updating the frame view scroll offset,
since that function will end up invalidating the scrollbars correctly.
- page/ScrollingCoordinator.cpp:
(WebCore::ScrollingCoordinator::didUpdateMainFrameScrollPosition):
- 6:43 PM Changeset in webkit [106316] by
-
- 2 edits in trunk/LayoutTests
Layout Test plugins/crash-restoring-plugin-page-from-page-cache.html is slow on Chromium
https://bugs.webkit.org/show_bug.cgi?id=77397
Unreviewed gardening. Marking plugins/crash-restoring-plugin-page-from-page-cache.html slow
on Chromium.
- platform/chromium/test_expectations.txt:
- 6:40 PM Changeset in webkit [106315] by
-
- 4 edits in trunk/Tools
REGRESSION(r105797): prepare-ChangeLog for a .cpp file can
output an empty method name (i.e. "()")
https://bugs.webkit.org/show_bug.cgi?id=77336
Reviewed by Darin Adler.
r105797 tried to detect a change outside methods, but it causes a bug that
prepare-ChangeLog can output an empty method name, like this:
- foo/bar/baz.cpp: (method1): (): (method2):
This is because the cpp parser in prepare-ChangeLog cannot distinguish
'{' as the beginning of a method with '{' as the beginning of an array definition
at the top level.
int a[] = { 1, 2, 3 }; This '{' is the beginning of an array definition.
void func() { This '{' is the beginning of a method.
...;
}
This patch fixes prepare-ChangeLog so that it skips an array definition at the top level.
- Scripts/prepare-ChangeLog:
(get_function_line_ranges_for_cpp): Modified as described above.
(generateFunctionLists): As a hack, modified so that prepare-ChangeLog does not output
an empty method name. Ideally this should not happen but may happen, since the
parsers are not perfect.
- Scripts/webkitperl/prepare-ChangeLog_unittest/resources/cpp_unittests.cpp: Added test cases.
(NameSpace7):
(NameSpace8):
(Class109):
- Scripts/webkitperl/prepare-ChangeLog_unittest/resources/cpp_unittests-expected.txt:
- 6:22 PM Changeset in webkit [106314] by
-
- 4 edits in trunk/Source/JavaScriptCore
get_by_val_arguments is broken in the interpreter
https://bugs.webkit.org/show_bug.cgi?id=77389
Reviewed by Gavin Barraclough.
When get_by_val had wad a value profile added, the same slot was not added to
get_by_val_arguments. This broke the interpreter as the interpreter falls
back on its regular get_by_val implementation.
No tests are added as the interpreter is fairly broken in its
current state (multiple tests fail due to this bug).
- bytecode/CodeBlock.cpp:
(JSC::CodeBlock::dump):
- bytecode/Opcode.h:
(JSC):
():
- bytecompiler/BytecodeGenerator.cpp:
(JSC::BytecodeGenerator::emitGetArgumentByVal):
- 6:20 PM Changeset in webkit [106313] by
-
- 2 edits in trunk/Source/WebKit/chromium
Unreviewed, rolling out r106259.
http://trac.webkit.org/changeset/106259
https://bugs.webkit.org/show_bug.cgi?id=77395
Breaks shared builders as they also build webkittests with
-Wexit-time-destructors. (Requested by leviw on #webkit).
Patch by Sheriff Bot <webkit.review.bot@gmail.com> on 2012-01-30
- WebKitUnitTests.gyp:
- 6:11 PM Changeset in webkit [106312] by
-
- 2 edits in trunk/PerformanceTests
Skip inspector tests since they have been timing out.
- Skipped:
- 6:04 PM Changeset in webkit [106311] by
-
- 5 edits in trunk/Source/WebCore
Show repaint counters in individual tiles
https://bugs.webkit.org/show_bug.cgi?id=77390
<rdar://problem/10767967>
Reviewed by Darin Adler.
- platform/graphics/ca/mac/TileCache.h:
- platform/graphics/ca/mac/TileCache.mm:
(WebCore::TileCache::setNeedsDisplayInRect):
Make sure to invalidate the repaint counter rect if necessary.
(WebCore::TileCache::drawLayer):
Draw the repaint counter.
(WebCore::TileCache::showRepaintCounter):
New function that determines whether we should show repaint counters for the given tile cache.
- platform/graphics/ca/mac/WebTileLayer.h:
- platform/graphics/ca/mac/WebTileLayer.mm:
(-[WebTileLayer incrementRepaintCount]):
Add method for getting the repaint count.
- 5:57 PM Changeset in webkit [106310] by
-
- 4 edits in trunk
<rdar://problem/10778045> REGRESSION (r91935): text-combine fails
https://bugs.webkit.org/show_bug.cgi?id=77373
Reviewed by Darin Adler.
Source/WebCore:
Removed tests that were failing because of this bug from the Lion skipped
list.
- platform/graphics/mac/GlyphPageTreeNodeMac.cpp:
(WebCore::GlyphPage::fill): Changed to use CTFontGetGlyphsForCharacters,
rather than wkGetVerticalGlyphsForCharacters, for non-fullwidth fonts.
LayoutTests:
Removed tests that were failing because of this bug from the skipped list.
- platform/mac-lion/Skipped:
- 5:36 PM Changeset in webkit [106309] by
-
- 1 edit2 moves in trunk/LayoutTests
Fix typo in test name from my last commit.
Spotted and rubber-stamped by Darin Adler.
- plugins/crash-restoring-plugin-page-from-page-cache-expected.txt: Copied from plugins/crash-restoring-pluging-page-from-page-cache-expected.txt.
- plugins/crash-restoring-plugin-page-from-page-cache.html: Copied from plugins/crash-restoring-pluging-page-from-page-cache.html.
- plugins/crash-restoring-pluging-page-from-page-cache-expected.txt: Removed.
- plugins/crash-restoring-pluging-page-from-page-cache.html: Removed.
- 5:36 PM Changeset in webkit [106308] by
-
- 9 edits in trunk
Enable memcache on webkit-perf.appspot.com
https://bugs.webkit.org/show_bug.cgi?id=77378
Reviewed by Adam Barth.
Eanble memcache on dashboard, manifest, and runs handlers. Clear appropriate caches when new runs are reported,
or new models are created. We flush all caches when tests are merged since it's hard to figure out dependencies
in that case. Luckily, we merge tests only occassionally and manually (or hope so) so this shouldn't be an issue.
- Websites/webkit-perf.appspot.com/create_handler.py:
(CreateHandler.post):
- Websites/webkit-perf.appspot.com/dashboard_handler.py:
(DashboardHandler.get):
- Websites/webkit-perf.appspot.com/manifest_handler.py:
(ManifestHandler.get):
- Websites/webkit-perf.appspot.com/merge_tests_handler.py:
(MergeTestsHandler.post):
- Websites/webkit-perf.appspot.com/models.py:
(createInTransactionWithNumericIdHolder):
(Test):
(Test.cacheKey):
- Websites/webkit-perf.appspot.com/report_handler.py:
(ReportHandler.post):
(ReportHandler._addTestIfNeeded):
- Websites/webkit-perf.appspot.com/runs_handler.py:
(RunsHandler.get):
- 5:25 PM Changeset in webkit [106307] by
-
- 4 edits in trunk/Source/WebCore
BlackBerry - Support Proxy-Authenticate headers when a proxy is configured
https://bugs.webkit.org/show_bug.cgi?id=77361
Though we have a proxy configured, we might not have the auth
credentials it requires. Support Proxy-Authenticate for that case.
Patch by Christopher Hutten-Czapski <chutten@rim.com> on 2012-01-30
Reviewed by George Staikos.
- platform/network/blackberry/NetworkJob.cpp:
(WebCore::NetworkJob::handleNotifyHeaderReceived):
(WebCore::NetworkJob::handleAuthHeader):
(WebCore::NetworkJob::sendRequestWithCredentials):
- platform/network/blackberry/NetworkJob.h:
- platform/network/blackberry/NetworkManager.cpp:
(WebCore::NetworkManager::startJob):
- 5:23 PM Changeset in webkit [106306] by
-
- 3 edits in trunk/Source/WebCore
[chromium] Remove unnecessary retry logic in LayerRendererChromium initialization for accelerated painting
https://bugs.webkit.org/show_bug.cgi?id=77247
Reviewed by Kenneth Russell.
The accelerate painting setting is done differently from other capability-dependent settings for no good reason,
requiring that we retry initialization with different settings. For all other settings we set the capabilities
bit to true if the setting is requested and if the required capabilities exist on the underlying context.
- platform/graphics/chromium/LayerRendererChromium.cpp:
(WebCore::LayerRendererChromium::create):
(WebCore::LayerRendererChromium::initialize):
- platform/graphics/chromium/cc/CCLayerTreeHostImpl.cpp:
(WebCore::CCLayerTreeHostImpl::initializeLayerRenderer):
- 5:10 PM Changeset in webkit [106305] by
-
- 3 edits2 adds in trunk
<rdar://problem/10707072>
Crashes in WebProcess at WebCore::Node::rendererIsNeeded
Reviewed by Darin Adler.
Source/WebCore:
In specific circumstances a plugin element can be without a render style at the point in time where
the page navigated and enters the page cache.
When this is the cash, the element should not enter into the "custom style for renderer" mode and should
instead use the default render style machinery.
Test: plugins/crash-restoring-pluging-page-from-page-cache.html
- html/HTMLPlugInImageElement.cpp:
(WebCore::HTMLPlugInImageElement::documentWillSuspendForPageCache): Only setHasCustomStyleForRenderer and
forceRecalc if there actually is a custom style to be used.
(WebCore::HTMLPlugInImageElement::documentDidResumeFromPageCache): Only clearHasCustomStyleForRenderer if there
actually was a custom style to be cleared.
(WebCore::HTMLPlugInImageElement::customStyleForRenderer): This should only be called if there actually is a
custom style to be used. Otherwise the element would have to fallback to the "normal" RenderStyle which might
not exist.
LayoutTests:
- plugins/crash-restoring-pluging-page-from-page-cache-expected.txt: Added.
- plugins/crash-restoring-pluging-page-from-page-cache.html: Added.
- 5:07 PM Changeset in webkit [106304] by
-
- 7 edits in trunk/Source/WebCore
Show debug borders for individual tile cache tiles
https://bugs.webkit.org/show_bug.cgi?id=77388
Reviewed by Sam Weinig.
- platform/graphics/GraphicsLayer.cpp:
(WebCore::GraphicsLayer::updateDebugIndicators):
Give tile cache tiles a thin dark blue border.
- platform/graphics/ca/mac/TileCache.h:
(WebCore::TileCache::tileDebugBorderWidth):
(WebCore::TileCache::tileDebugBorderColor):
Add getters and member variables for the tile debug border width and color.
- platform/graphics/ca/mac/TileCache.mm:
(WebCore::TileCache::TileCache):
Initialize m_tileDebugBorderWidth.
(WebCore::TileCache::setTileDebugBorderWidth):
Update the border width of each tile layer.
(WebCore::TileCache::setTileDebugBorderColor):
Update the border color of each tile layer.
(WebCore::TileCache::createTileLayer):
Set the border color and border width.
- platform/graphics/ca/mac/WebTileCacheLayer.h:
- platform/graphics/ca/mac/WebTileCacheLayer.mm:
(-[WebTileCacheLayer borderColor]):
(-[WebTileCacheLayer setBorderColor:]):
(-[WebTileCacheLayer borderWidth]):
(-[WebTileCacheLayer setBorderWidth:]):
Call through to the TileCache.
- platform/graphics/mac/WebLayer.mm:
(drawLayerContents):
Don't draw the repaint counter for tile cache layers, each tile will maintain its own repaint counter.
- 5:07 PM Changeset in webkit [106303] by
-
- 2 edits in branches/subpixellayout/Source/WebCore/css
Change computed style calculations to round/truncate values at query as opposed to computation time to avoid loosing precision when adjusting values for zoom.
- 5:04 PM Changeset in webkit [106302] by
-
- 6 edits in trunk/Tools
run-webkit-tests calls out to webkit-build-directory twice
https://bugs.webkit.org/show_bug.cgi?id=77248
Reviewed by Dirk Pranke.
This reduces to calls to webkit-build-directory to one call and
saves 500ms for run-webkit-tests over a single file.
- Scripts/webkit-build-directory:
If called with no arguments, print out both the top-level directory and the
configuration directory.
- Scripts/webkit-build-directory:
- Scripts/webkitpy/layout_tests/port/config.py:
(Config.build_directory):
When called with no arguments, get and cache both the top-level and configuration directories.
- Scripts/webkitpy/layout_tests/port/config_standalone.py:
(main):
- Scripts/webkitpy/layout_tests/port/config_unittest.py:
(ConfigTest.assert_configuration):
(ConfigTest.test_build_directory.mock_webkit_build_directory):
(ConfigTest.test_build_directory):
(ConfigTest.test_default_configurationnotfound):
- Scripts/webkitpy/layout_tests/port/webkit.py:
(WebKitPort._build_path):
Cache the root directory in the options object so that other users
of the options object don't need to call out to webkit-build-directory.
- 4:35 PM Changeset in webkit [106301] by
-
- 5 edits in trunk
single-file input elements should refuse multi-file drags
https://bugs.webkit.org/show_bug.cgi?id=25913
Patch by Rakesh KN <rakesh.kn@motorola.com> on 2012-01-30
Reviewed by Eric Seidel.
Refuse the multiple file drags onto a single file input element.
Source/WebCore:
No new tests: Covered by existing tests, updated the expected results.
- page/DragController.cpp:
(WebCore::DragController::tryDocumentDrag):
Setting the dragSession.numberOfItemsToBeAccepted to 0 so that drag operation is none if the
file input element under mouse is single input type and number of files dragged onto that
input element are more than 1.
LayoutTests:
- editing/pasteboard/file-input-files-access-expected.txt:
Expected result changed.
- editing/pasteboard/script-tests/file-input-files-access.js:
(runTest):
Removed the 'FIXME' debug and comment as it is not valid after this change.
- 4:29 PM Changeset in webkit [106300] by
-
- 18 edits1 move3 deletes in trunk
[chromium] Use region reported painted opaque for draw culling
https://bugs.webkit.org/show_bug.cgi?id=76015
Patch by Dana Jansens <danakj@chromium.org> on 2012-01-30
Reviewed by James Robinson.
Source/WebCore:
New unit tests in CCTiledLayerImplTest.cpp, CCQuadCullerTest.cpp, CCLayerTreeHostImplTest.cpp
- platform/graphics/chromium/LayerChromium.cpp:
(WebCore::LayerChromium::setOpaque):
- platform/graphics/chromium/TiledLayerChromium.cpp:
(UpdatableTile):
(WebCore::TiledLayerChromium::pushPropertiesTo):
(WebCore::TiledLayerChromium::prepareToUpdateTiles):
- platform/graphics/chromium/cc/CCDrawQuad.cpp:
(WebCore::CCDrawQuad::opaqueRect):
(WebCore):
- platform/graphics/chromium/cc/CCDrawQuad.h:
(CCDrawQuad):
(WebCore::CCDrawQuad::needsBlending):
- platform/graphics/chromium/cc/CCQuadCuller.cpp:
(WebCore::CCQuadCuller::cullOccludedQuads):
- platform/graphics/chromium/cc/CCSolidColorDrawQuad.cpp:
(WebCore::CCSolidColorDrawQuad::CCSolidColorDrawQuad):
- platform/graphics/chromium/cc/CCTileDrawQuad.cpp:
(WebCore::CCTileDrawQuad::create):
(WebCore::CCTileDrawQuad::CCTileDrawQuad):
- platform/graphics/chromium/cc/CCTileDrawQuad.h:
(CCTileDrawQuad):
- platform/graphics/chromium/cc/CCTiledLayerImpl.cpp:
(DrawableTile):
(WebCore::DrawableTile::opaqueRect):
(WebCore::DrawableTile::setOpaqueRect):
(WebCore::CCTiledLayerImpl::appendQuads):
(WebCore::CCTiledLayerImpl::pushTileProperties):
- platform/graphics/chromium/cc/CCTiledLayerImpl.h:
(CCTiledLayerImpl):
Source/WebKit/chromium:
- tests/CCLayerTreeHostImplTest.cpp:
(WebKit::BlendStateCheckLayer::appendQuads):
(WebKit::BlendStateCheckLayer::setOpaqueColor):
(BlendStateCheckLayer):
(WebKit::BlendStateCheckLayer::BlendStateCheckLayer):
(WebKit::TEST_F):
- tests/CCQuadCullerTest.cpp:
(WebCore::MakeTileQuad):
(WebCore::setQuads):
(WebCore::TEST):
(WebCore):
- tests/CCTiledLayerImplTest.cpp:
(CCLayerTestCommon::createLayer):
(CCLayerTestCommon::TEST):
(CCLayerTestCommon::getQuads):
(CCLayerTestCommon::coverageVisibleRectOnTileBoundaries):
(CCLayerTestCommon::coverageVisibleRectIntersectsTiles):
(CCLayerTestCommon::coverageVisibleRectIntersectsBounds):
(CCLayerTestCommon):
LayoutTests:
- platform/chromium-linux/compositing/iframes/iframe-in-composited-layer-expected.png:
- platform/chromium-mac-leopard/fast/repaint/block-selection-gap-in-composited-layer-expected.png: Removed.
- platform/chromium-mac-snowleopard/fast/repaint/block-selection-gap-in-composited-layer-expected.png: Removed.
- platform/chromium-win/fast/repaint/block-selection-gap-in-composited-layer-expected.png: Removed.
- platform/chromium/fast/repaint/block-selection-gap-in-composited-layer-expected.png: Renamed from LayoutTests/platform/chromium-linux/fast/repaint/block-selection-gap-in-composited-layer-expected.png.
- platform/chromium/test_expectations.txt:
- 4:22 PM Changeset in webkit [106299] by
-
- 2 edits in trunk/LayoutTests
[Chromium] Unreviewed test expectation change.
Remove passing media/media-blocked-by-beforeload.html from test_expectations.txt
after r103509
Patch by Shadi Khalek <shadi@chromium.org> on 2012-01-30
- platform/chromium/test_expectations.txt:
- 4:18 PM Changeset in webkit [106298] by
-
- 3 edits2 adds in trunk
Crash in previousLinePosition when moving into a root inline box without leaves
https://bugs.webkit.org/show_bug.cgi?id=76812
Reviewed by Enrica Casucci.
Source/WebCore:
The crash was caused by us assuming that every root inline box has at least one leaf,
which isn't true when we create inline boxes for an empty text run with margin, border, etc...
Test: editing/selection/move-into-empty-root-inline-box.html
- editing/visible_units.cpp:
(WebCore::previousLinePosition):
(WebCore::nextLinePosition):
LayoutTests:
Add a regression test for the crash. Unfortunately, we can only test previousLinePosition.
- editing/selection/move-into-empty-root-inline-box-expected.txt: Added.
- editing/selection/move-into-empty-root-inline-box.html: Added.
- 3:59 PM Changeset in webkit [106297] by
-
- 5 edits in trunk
Unexpected syntax error
https://bugs.webkit.org/show_bug.cgi?id=77340
Reviewed by Gavin Barraclough.
Source/JavaScriptCore:
Function calls and new expressions have the same semantics for
assignment, so should simply share their lhs handling.
- parser/Parser.cpp:
(JSC::::parseMemberExpression):
LayoutTests:
Add new tests for correct parsing of new expressions
- fast/js/parser-syntax-check-expected.txt:
- fast/js/script-tests/parser-syntax-check.js:
(runTest):
(invalid):
- 3:56 PM Changeset in webkit [106296] by
-
- 2 edits in trunk/Source/JavaScriptCore
Unreviewed ARMv7 build fix.
- tools/CodeProfiling.cpp:
(JSC):
(JSC::setProfileTimer):
(JSC::CodeProfiling::begin):
(JSC::CodeProfiling::end):
- 3:35 PM Changeset in webkit [106295] by
-
- 2 edits in trunk/Source/JavaScriptCore
Using OS(WIN) or OS(MAC) should cause a build error.
https://bugs.webkit.org/show_bug.cgi?id=77162
Reviewed by Darin Adler.
- wtf/Platform.h: Expand them into something that will cause a compile error.
- 3:16 PM Changeset in webkit [106294] by
-
- 2 edits in trunk/LayoutTests
css3/flexbox/cross-axis-scrollbar.html ref test fails on Windows Chromium bots
https://bugs.webkit.org/show_bug.cgi?id=77309
Reviewed by Dirk Pranke.
It looks like the scrollbars on windows overflow by a pixel, so the
edge of the scrollbar is being covered up by the trailing green box.
Use z-order to force the scrollbar on top.
This is kind of speculative. I want to see if the test passes on the
bots before removing the line in test_expectations.txt.
- css3/flexbox/cross-axis-scrollbar-expected.html:
- 3:16 PM Changeset in webkit [106293] by
-
- 4 edits in trunk/Tools
Parsing test_expecations.txt + Skipped lists takes too long
https://bugs.webkit.org/show_bug.cgi?id=77059
Reviewed by Dirk Pranke.
This saves ~100ms on the Apple Mac port.
-memoize a bunch of path methods.
-Avoid doing multiple disk accesses per line.
-Parse the skipped list directly instead of turning it into a test_expecations.txt
formatting string and parsing that.
- Scripts/webkitpy/layout_tests/models/test_expectations.py:
(TestExpectationParser):
(TestExpectationParser.expectation_for_skipped_test):
(TestExpectationParser._parse_line):
(TestExpectationParser._collect_matching_tests):
(TestExpectations.init):
(TestExpectations._add_skipped_tests):
- Scripts/webkitpy/layout_tests/models/test_expectations_unittest.py:
(test_add_skipped_tests):
(test_add_skipped_tests_duplicate):
- Scripts/webkitpy/layout_tests/port/base.py:
(Port):
(Port.test_isfile):
(Port.normalize_test_name):
(Port.layout_tests_dir):
(Port.abspath_for_test):
- Scripts/webkitpy/layout_tests/run_webkit_tests.py:
- 3:06 PM Changeset in webkit [106292] by
-
- 6 edits4 adds in trunk
webkit-perf.appspot.com should have an ability to merge tests
https://bugs.webkit.org/show_bug.cgi?id=77250
Reviewed by Adam Barth.
Added admin/merge-tests. This page allows administrators to easily merge test objects.
Also add the forgotten title.png and rename api/create to admin/create since we don't allow
non-admins to create new models anyway.
- Websites/webkit-perf.appspot.com/app.yaml:
- Websites/webkit-perf.appspot.com/css: Added.
- Websites/webkit-perf.appspot.com/css/title.png: Added.
- Websites/webkit-perf.appspot.com/main.py:
- Websites/webkit-perf.appspot.com/merge_tests.yaml: Added.
- Websites/webkit-perf.appspot.com/merge_tests_handler.py: Added.
(MergeTestHandler):
(MergeTestHandler.get):
(MergeTestHandler.post):
- Websites/webkit-perf.appspot.com/models.py:
(deleteModelWithNumericIdHolder):
- Websites/webkit-perf.appspot.com/static/create-models.html:
- 2:54 PM Changeset in webkit [106291] by
-
- 2 edits in trunk/Source/WebCore
!m_insideRegionPaint assertion in RenderRegion.cpp is invalid
https://bugs.webkit.org/show_bug.cgi?id=77372
Reviewed by David Hyatt.
Removing the !m_insideRegionPaint assertion that's in three functions in
RenderRegion. It's triggering in numerous layout tests and isn't valid.
No new tests as this just removes assertions.
- rendering/RenderRegion.cpp:
(WebCore::RenderRegion::setRenderBoxRegionInfo):
(WebCore::RenderRegion::takeRenderBoxRegionInfo):
(WebCore::RenderRegion::removeRenderBoxRegionInfo):
- 2:48 PM Changeset in webkit [106290] by
-
- 2 edits in trunk/Source/JavaScriptCore
[BlackBerry] OS(QNX) also has TM_GMTOFF, TM_ZONE, and TIMEGM
https://bugs.webkit.org/show_bug.cgi?id=77360
Patch by Yong Li <yoli@rim.com> on 2012-01-30
Reviewed by Rob Buis.
Turn on HAVE(TM_GMTOFF), HAVE(TM_ZONE), and HAVE(TIMEGM)
for OS(QNX).
- wtf/Platform.h:
- 2:10 PM Changeset in webkit [106289] by
-
- 9 edits in trunk/Source
[chromium] Always pre-reserve scrollbar and scroll corner textures
https://bugs.webkit.org/show_bug.cgi?id=77251
Reviewed by James Robinson.
Source/WebCore:
Add a flag to LayerChromium that says that a layer's textures should
always be reserved. Prior to painting layers, find all layers marked
as such and reserve their textures. This will prevent texture memory
limits from being hit before the root layer's scrollbars are reserved
and painted.
- platform/graphics/chromium/LayerChromium.cpp:
(WebCore::LayerChromium::LayerChromium):
- platform/graphics/chromium/LayerChromium.h:
(LayerChromium):
(WebCore::LayerChromium::reserveTextures):
(WebCore::LayerChromium::setAlwaysReserveTextures):
(WebCore::LayerChromium::alwaysReserveTextures):
- platform/graphics/chromium/TiledLayerChromium.cpp:
(WebCore::TiledLayerChromium::reserveTextures):
- platform/graphics/chromium/TiledLayerChromium.h:
():
- platform/graphics/chromium/cc/CCLayerTreeHost.cpp:
(WebCore::CCLayerTreeHost::updateLayers):
(WebCore::CCLayerTreeHost::reserveTextures):
(WebCore):
- platform/graphics/chromium/cc/CCLayerTreeHost.h:
():
Source/WebKit/chromium:
- src/NonCompositedContentHost.cpp:
(WebKit::reserveScrollbarLayers):
(WebKit):
(WebKit::NonCompositedContentHost::setViewport):
- 2:07 PM Changeset in webkit [106288] by
-
- 2 edits in trunk/Source/JavaScriptCore
Speculative Windows build fix.
- assembler/MacroAssemblerCodeRef.h:
(FunctionPtr):
- 2:01 PM Changeset in webkit [106287] by
-
- 2 edits in trunk/Source/WebKit2
[Qt] Disconnect the LayerTreeHost from layers when they get removed from their parent.
https://bugs.webkit.org/show_bug.cgi?id=77362
Reviewed by Noam Rosenthal.
This fixes crashes seen in layout tests under certain timing circumstances
that appeared after r106109 was landed.
- WebProcess/WebCoreSupport/WebGraphicsLayer.cpp:
(WebCore::WebGraphicsLayer::replaceChild):
(WebCore::WebGraphicsLayer::removeFromParent):
- 1:44 PM Changeset in webkit [106286] by
-
- 2 edits in trunk/Source/WebCore
Speculative 32-bit build-fix.
- WebCore.exp.in:
- 1:22 PM Changeset in webkit [106285] by
-
- 2 edits in trunk/LayoutTests
Unreviewed gardening. Marking fast/js/dfg-int32array-overflow-values.html as slow
for Chromium Debug bots.
- platform/chromium/test_expectations.txt:
- 1:21 PM Changeset in webkit [106284] by
-
- 2 edits in trunk/Source/WebKit/chromium
Roll Chromium DEPS from 119623 to 119700.
- DEPS:
- 1:19 PM Changeset in webkit [106283] by
-
- 3 edits in trunk/LayoutTests
fast/css/fontsize-unit-rems-crash.html should use dumpAsText
https://bugs.webkit.org/show_bug.cgi?id=77351
Reviewed by Simon Fraser.
Switching the above test to only use text results as the html element is display:none.
- fast/css/fontsize-unit-rems-crash-expected.txt:
- fast/css/fontsize-unit-rems-crash.html:
- 1:15 PM Changeset in webkit [106282] by
-
- 2 edits in trunk/Tools
Unreviewed. Add myself to CSS, GStreamer, Qt related watchlists.
- Scripts/webkitpy/common/config/watchlist:
- 1:11 PM Changeset in webkit [106281] by
-
- 14 edits in trunk
[CSSRegions]Add support for background-color in region styling
https://bugs.webkit.org/show_bug.cgi?id=71488
Reviewed by David Hyatt.
Source/WebCore:
Based on work by Alexandru Chiculita (Alexandru Chiculita).
Previous patches for region styling were touching RenderObject::style() method. After several attempts to avoid regressions
(including caching of RenderObject::style() pointer in most used methods), we decided to attempt a different approach:
Step1: before each region is repainted, we compute the style for each box that falls into the region
Step2: before paint, we store the box original style
Step3: paint the region contents using the style in region
Step4: after paint is finished, we restore the box original style (and store the box style in region for future region paint)
Tests for region styling are also enabled with this patch.
- WebCore.exp.in:
- rendering/RenderFlowThread.cpp:
(WebCore::RenderFlowThread::clearRenderBoxCustomStyle):
(WebCore::RenderFlowThread::setRegionRangeForBox):
- rendering/RenderFlowThread.h:
():
- rendering/RenderLayer.cpp:
(WebCore::RenderLayer::paint):
(WebCore::RenderLayer::hitTest):
- rendering/RenderObject.cpp:
- rendering/RenderObject.h:
(WebCore::RenderObject::style):
(RenderObject):
- rendering/RenderObjectChildList.cpp:
(WebCore::RenderObjectChildList::removeChildNode):
- rendering/RenderRegion.cpp:
(WebCore::RenderRegion::RenderRegion):
(WebCore::RenderRegion::setRegionBoxesRegionStyle):
(WebCore):
(WebCore::RenderRegion::restoreRegionBoxesOriginalStyle):
(WebCore::RenderRegion::paintReplaced):
(WebCore::RenderRegion::setRenderBoxRegionInfo):
(WebCore::RenderRegion::takeRenderBoxRegionInfo):
(WebCore::RenderRegion::removeRenderBoxRegionInfo):
(WebCore::RenderRegion::renderBoxRegionStyle):
(WebCore::RenderRegion::computeStyleInRegion):
(WebCore::RenderRegion::clearBoxStyleInRegion):
- rendering/RenderRegion.h:
(RenderRegion):
- rendering/RenderView.cpp:
(WebCore::RenderView::RenderView):
- rendering/RenderView.h:
(WebCore):
(RenderView):
LayoutTests:
Enable the region styling tests again.
- platform/mac-snowleopard/Skipped:
- 1:08 PM Changeset in webkit [106280] by
-
- 2 edits1 add in trunk/Source/WebCore
WebCore build exceeds address space on 32-bit Windows builders (again).
https://bugs.webkit.org/show_bug.cgi?id=77357
Reviewed by Adam Roben.
Add an Inspector All-In-One file, but only use it in Release and Production builds.
This differs from our other All-In-One files, but it is a better approach because it makes
debugging possible in the Debug configuration (the symbols will be in the correct .obj file
for the original .cpp files).
- WebCore.vcproj/WebCore.vcproj:
Also, let VS have its way with the vcproj file.
- inspector/InspectorAllInOne.cpp: Added.
- 1:07 PM Changeset in webkit [106279] by
-
- 2 edits4 adds in trunk/LayoutTests
IndexedDB: Add tests for structured clone data
https://bugs.webkit.org/show_bug.cgi?id=74897
Validate that the types listed in the HTML5 "structured clone algorithm"
round trip properly through Indexed DB storage. Types with incomplete
support in the Chromium port are excluded (http://crbug.com/108012).
Landing disabled since cr-linux is failing test for unknown reasons.
Reviewed by Tony Chang.
- platform/chromium/test_expectations.txt:
- storage/indexeddb/resources/test-data.html: Added.
- storage/indexeddb/resources/test-data.txt: Added.
- storage/indexeddb/structured-clone-expected.txt: Added.
- storage/indexeddb/structured-clone.html: Added.
- 12:59 PM Changeset in webkit [106278] by
-
- 3 edits in trunk/Source/WebCore
Limit periodic flushing inside ImageBufferCG to just Lion
https://bugs.webkit.org/show_bug.cgi?id=77353
<rdar://problem/10328309>
Reviewed by Chris Marrin.
- platform/graphics/cg/ImageBufferCG.cpp:
- platform/graphics/cg/ImageBufferDataCG.h:
- 12:57 PM Changeset in webkit [106277] by
-
- 2 edits in trunk/Source/JavaScriptCore
https://bugs.webkit.org/show_bug.cgi?id=77163
MacroAssemblerCodeRef.h uses OS(WIN) instead of OS(WINDOWS)
Rubber stamped by Geoff Garen
- assembler/MacroAssemblerCodeRef.h:
- 12:38 PM Changeset in webkit [106276] by
-
- 5 edits in trunk/Source/JavaScriptCore
Unreviewed build fix for interpreter builds.
- bytecode/CodeBlock.cpp:
(JSC::CodeBlock::CodeBlock):
- bytecode/CodeBlock.h:
(CodeBlock):
- interpreter/Interpreter.cpp:
(JSC::Interpreter::privateExecute):
- tools/CodeProfile.cpp:
(JSC::CodeProfile::sample):
- 12:25 PM Changeset in webkit [106275] by
-
- 3 edits5 adds in trunk
Memory leak caused by PeerConnection add a NULL media stream
https://bugs.webkit.org/show_bug.cgi?id=76150
Source/WebCore:
It was my missunderstanding that the IDL keyword [StrictTypeChecking] also protects against
null or undefined arguments, it doesn't. Added checks for null pointers.
Reviewed by Adam Barth.
Test: fast/mediastream/peerconnection-addstream.html
- mediastream/PeerConnection.cpp:
(WebCore::PeerConnection::addStream):
(WebCore::PeerConnection::removeStream):
LayoutTests:
Added a basic LayoutTest for null/undefined arguments to PeerConnection::addStream().
Reviewed by Adam Barth.
- fast/mediastream/peerconnection-addstream-expected.txt: Added.
- fast/mediastream/peerconnection-addstream.html: Added.
- fast/mediastream/script-tests/peerconnection-addstream.js: Added.
- 12:01 PM Changeset in webkit [106274] by
-
- 7 edits in trunk/Source
[Windows] Optionally invert colors when drawing to a WebView's backing store.
https://bugs.webkit.org/show_bug.cgi?id=77168
Reviewed by Sam Weinig.
Source/WebCore:
- css/CSSPrimitiveValueMappings.h: Assert that CompositeDifference is
not converted to a CSS value. Exposing a new compositing operation to
CSS is outside the scope of this patch.
(WebCore::CSSPrimitiveValue::CSSPrimitiveValue):
- platform/graphics/GraphicsTypes.h: Add CompositeDifference as a
CompositeOperator. Also, remove an outdated comment.
- platform/graphics/cg/GraphicsContextCG.cpp:
(WebCore::GraphicsContext::setPlatformCompositeOperation): Map
CompositeDifference to kCGBlendModeDifference.
Source/WebKit/win:
- WebView.cpp:
(WebView::WebView): Initialize m_shouldInvertColors to false.
(WebView::paintIntoBackingStore): If m_shouldInvertColors is true, draw
an opaque white quad using the CompositeDifference blend mode. This
blend operation instructs CoreGraphics to take the difference between
the source pixel (white) and the background pixel, resulting in an
inverted pixel.
- WebView.h: Define m_shouldInvertColors.
- 11:50 AM Changeset in webkit [106273] by
-
- 2 edits in trunk/Source/WebCore
Limit the shadow offset CG hack to just SL and Lion
https://bugs.webkit.org/show_bug.cgi?id=77348
<rdar://problem/10158016>
Reviewed by Chris Marrin.
No new tests, current tests cover this.
- platform/graphics/cg/GraphicsContextCG.cpp:
(WebCore::GraphicsContext::setPlatformShadow):
- 11:47 AM Changeset in webkit [106272] by
-
- 2 edits in releases/WebKitGTK/webkit-1.6
Update NEWS and libtool version in preparation for the 1.6.2 release
- 11:47 AM Changeset in webkit [106271] by
-
- 18 edits in trunk/Source
https://bugs.webkit.org/show_bug.cgi?id=77263
PlatformScreenMac should not rely on NSWindow for important bits of data
Reviewed by Geoff Garen.
../WebCore:
The main problem is that we cannot rely on the NSWindow for information about
the deviceScaleFactor because we cannot access an NSWindow from within
WebCore for WebKit2 windows. Instead, we can fetch it from
WebCore::deviceScaleFactor(), but we need a Frame to call that. So
screenAvailableRect and screenRect both now take a FrameView* instead of a
Widget*. All existing call sites actually sent a FrameView in anyway, so this
is not a big change, but it does require touching a lot of platforms.
- WebCore.exp.in:
- platform/PlatformScreen.h:
(WebCore):
- platform/blackberry/PlatformScreenBlackBerry.cpp:
(WebCore::screenAvailableRect):
(WebCore::screenRect):
- platform/chromium/PlatformScreenChromium.cpp:
(WebCore::screenRect):
(WebCore::screenAvailableRect):
- platform/chromium/PlatformSupport.h:
(WebCore):
(PlatformSupport):
():
- platform/efl/PlatformScreenEfl.cpp:
(WebCore::screenRect):
(WebCore::screenAvailableRect):
- platform/gtk/PlatformScreenGtk.cpp:
(WebCore::screenRect):
(WebCore::screenAvailableRect):
- platform/qt/PlatformScreenQt.cpp:
(WebCore::screenRect):
(WebCore::screenAvailableRect):
- platform/win/PlatformScreenWin.cpp:
(WebCore::screenRect):
(WebCore::screenAvailableRect):
- platform/wx/ScreenWx.cpp:
(WebCore::screenRect):
(WebCore::screenAvailableRect):
The Mac-only functions toUserSpace() and toDeviceSpace() were also updated to
take a parameter for the deviceScaleFactor.
- platform/mac/PlatformScreenMac.mm:
(WebCore::screenRect):
(WebCore::screenAvailableRect):
(WebCore::toUserSpace):
(WebCore::toDeviceSpace):
../WebKit/chromium:
These two functions now take a FrameView instead of a Widget.
- src/PlatformSupport.cpp:
(WebCore::PlatformSupport::screenRect):
(WebCore::PlatformSupport::screenAvailableRect):
../WebKit/mac:
toUserSpace() and toDeviceSpace() now take a parameter for the
deviceScaleFactor.
- WebCoreSupport/WebChromeClient.mm:
(windowScaleFactor):
(WebChromeClient::setWindowRect):
(WebChromeClient::windowRect):
../WebKit2:
toUserSpace() and toDeviceSpace() now take a parameter for the
deviceScaleFactor.
- UIProcess/API/mac/WKView.mm:
(-[WKView _convertToDeviceSpace:]):
(-[WKView _convertToUserSpace:]):
- 11:38 AM WebKitGTK/1.6.x edited by
- (diff)
- 11:37 AM WebKitGTK/1.6.x edited by
- (diff)
- 11:37 AM Changeset in webkit [106270] by
-
- 2 edits in releases/WebKitGTK/webkit-1.6/Source/WebCore
Merging r102958
- 11:35 AM Changeset in webkit [106269] by
-
- 7 edits in releases/WebKitGTK/webkit-1.6/Source
Merging r100725
- 11:33 AM Changeset in webkit [106268] by
-
- 2 edits in releases/WebKitGTK/webkit-1.6
Merging r98699
- 11:32 AM Changeset in webkit [106267] by
-
- 2 edits in releases/WebKitGTK/webkit-1.6/Source/WebKit/gtk
Mergigg r97979
- 11:28 AM Changeset in webkit [106266] by
-
- 2 edits in trunk/LayoutTests
Enable web intents layout tests for Chromium build.
https://bugs.webkit.org/show_bug.cgi?id=77347
Patch by Greg Billock <gbillock@google.com> on 2012-01-30
Reviewed by Ryosuke Niwa.
- platform/chromium/test_expectations.txt:
- 11:28 AM Changeset in webkit [106265] by
-
- 3 edits in releases/WebKitGTK/webkit-1.6/Source/WebKit/gtk
Merging r97913
- 11:26 AM Changeset in webkit [106264] by
-
- 4 edits in trunk
Unreviewed build fix following bug#76855
Source/JavaScriptCore:
- JavaScriptCore.exp:
Tools:
- TestWebKitAPI/Tests/WTF/MetaAllocator.cpp:
():
(TestWebKitAPI::TEST_F):
- 11:23 AM Changeset in webkit [106263] by
-
- 7 edits1 add in releases/WebKitGTK/webkit-1.6
Merging r97901
- 11:16 AM Changeset in webkit [106262] by
-
- 9 edits1 delete in releases/WebKitGTK/webkit-1.6
Merging r97269
- 11:13 AM Changeset in webkit [106261] by
-
- 5 edits2 adds in releases/WebKitGTK/webkit-1.6
Merging r96378
- 11:09 AM Changeset in webkit [106260] by
-
- 2 edits in trunk/Source/JavaScriptCore
CaseFoldingHash::hash() doesn't handle 8 bit strings directly
https://bugs.webkit.org/show_bug.cgi?id=76652
Reviewed by Andreas Kling.
- wtf/text/StringHash.h:
(WTF::CaseFoldingHash::hash): Added 8 bit string code path.
- 11:06 AM Changeset in webkit [106259] by
-
- 2 edits in trunk/Source/WebKit/chromium
[chromium] enable -Wexit-time-destructors for webkit unit tests
https://bugs.webkit.org/show_bug.cgi?id=77300
Patch by Ami Fischman <fischman@chromium.org> on 2012-01-30
Reviewed by Tony Chang.
- WebKitUnitTests.gyp: Enable clang check for exit time destructors.
- 10:52 AM Changeset in webkit [106258] by
-
- 7 edits in trunk/Source/WebCore
Kill CSSMutableStyleDeclarationConstIterator
https://bugs.webkit.org/show_bug.cgi?id=77342
Reviewed by Sam Weinig.
CSSMutableStyleDeclaration is an array and should be iterated using an index. This simplifies the code.
- css/CSSMutableStyleDeclaration.cpp:
(WebCore::CSSMutableStyleDeclaration::copyPropertiesFrom):
(WebCore::CSSMutableStyleDeclaration::removeProperty):
(WebCore::CSSMutableStyleDeclaration::setProperty):
(WebCore::CSSMutableStyleDeclaration::setPropertyInternal):
(WebCore::CSSMutableStyleDeclaration::parseDeclaration):
(WebCore::CSSMutableStyleDeclaration::addParsedProperties):
(WebCore::CSSMutableStyleDeclaration::addParsedProperty):
(WebCore::CSSMutableStyleDeclaration::merge):
(WebCore::CSSMutableStyleDeclaration::removePropertiesInSet):
- css/CSSMutableStyleDeclaration.h:
(WebCore):
(WebCore::CSSMutableStyleDeclaration::createForSVGFontFaceElement):
(CSSMutableStyleDeclaration):
- css/CSSStyleDeclaration.cpp:
(WebCore::CSSStyleDeclaration::CSSStyleDeclaration):
- css/CSSStyleDeclaration.h:
(CSSStyleDeclaration):
():
- css/CSSStyleSelector.cpp:
(WebCore::CSSStyleSelector::styleForKeyframe):
(WebCore::CSSStyleSelector::applyDeclaration):
- editing/EditingStyle.cpp:
- 10:51 AM Changeset in webkit [106257] by
-
- 4 edits in trunk/Source/JavaScriptCore
stringProtoFuncReplace converts 8 bit strings to 16 bit during replacement
https://bugs.webkit.org/show_bug.cgi?id=76651
Reviewed by Geoffrey Garen.
Made local function substituteBackreferencesSlow a template function
based on character width. Cleaned up getCharacters() in both UString
and StringImpl. Changed getCharacters<UChar> to up convert an 8 bit
string to 16 bits if necessary.
- runtime/StringPrototype.cpp:
(JSC::substituteBackreferencesSlow):
(JSC::substituteBackreferences):
- runtime/UString.h:
(JSC::LChar):
(JSC::UChar):
- wtf/text/StringImpl.h:
(WTF::UChar):
- 10:50 AM WebKitGTK/1.6.x edited by
- (diff)
- 10:36 AM Changeset in webkit [106256] by
-
- 5 edits in trunk/Source/WebKit/chromium
[chromium] MediaStream API: Fix a few memory leaks
https://bugs.webkit.org/show_bug.cgi?id=77334
Reviewed by Darin Fisher.
It seems that either the behaviour of WebPrivatePtr has changed or I misunderstood
something, but I fixed a memory leak in WebMediaStreamDescriptor and safeguarded
WebMediaStreamSource.
- public/platform/WebMediaStreamDescriptor.h:
(WebKit::WebMediaStreamDescriptor::WebMediaStreamDescriptor):
- public/platform/WebMediaStreamSource.h:
(WebKit::WebMediaStreamSource::WebMediaStreamSource):
(WebKit::WebMediaStreamSource::operator=):
(WebMediaStreamSource):
- src/WebMediaStreamDescriptor.cpp:
(WebKit::WebMediaStreamDescriptor::assign):
- src/WebMediaStreamSource.cpp:
(WebKit::WebMediaStreamSource::assign):
(WebKit):
- 10:28 AM Changeset in webkit [106255] by
-
- 15 edits in trunk/Source/JavaScriptCore
Clean up putDirect
https://bugs.webkit.org/show_bug.cgi?id=76232
Reviewed by Sam Weinig.
Part 3 - merge op_put_getter & op_put_setter.
Putting these separately is inefficient (and makes future optimiation,
e.g. making GetterSetter immutable) harder. Change to emit a single
op_put_getter_setter bytecode op. Ultimately we should probably be
able to merge this with put direct, to create a common op to initialize
object literal properties.
- bytecode/CodeBlock.cpp:
(JSC::CodeBlock::dump):
- bytecode/Opcode.h:
(JSC):
():
- bytecompiler/BytecodeGenerator.cpp:
(JSC::BytecodeGenerator::emitPutGetterSetter):
- bytecompiler/BytecodeGenerator.h:
(BytecodeGenerator):
- bytecompiler/NodesCodegen.cpp:
(JSC::PropertyListNode::emitBytecode):
- interpreter/Interpreter.cpp:
(JSC::Interpreter::privateExecute):
- jit/JIT.cpp:
(JSC::JIT::privateCompileMainPass):
- jit/JIT.h:
(JIT):
- jit/JITPropertyAccess.cpp:
(JSC::JIT::emit_op_put_getter_setter):
- jit/JITPropertyAccess32_64.cpp:
(JSC::JIT::emit_op_put_getter_setter):
- jit/JITStubs.cpp:
(JSC::DEFINE_STUB_FUNCTION):
- jit/JITStubs.h:
():
- runtime/JSObject.cpp:
(JSC::JSObject::putDirectVirtual):
(JSC::JSObject::putDirectAccessor):
(JSC):
(JSC::putDescriptor):
(JSC::JSObject::defineOwnProperty):
- runtime/JSObject.h:
():
(JSC::JSObject::putDirectInternal):
(JSC::JSObject::putDirect):
(JSC::JSObject::putDirectWithoutTransition):
- 10:22 AM Changeset in webkit [106254] by
-
- 4 edits in trunk/Source
Dromaeo tests call parseSimpleLengthValue() on 8 bit strings
https://bugs.webkit.org/show_bug.cgi?id=76649
Reviewed by Geoffrey Garen.
Source/JavaScriptCore:
- JavaScriptCore.exp: Added export for charactersToDouble.
Source/WebCore:
No functionality change, therefore no new tests.
Added 8 bit patch for parseSimpleLengthValue().
- css/CSSParser.cpp:
(WebCore::parseSimpleLengthValue):
- 10:10 AM Changeset in webkit [106253] by
-
- 4 edits in trunk/Source
WebCore decodeEscapeSequences unnecessarily converts 8 bit strings to 16 bit when decoding.
https://bugs.webkit.org/show_bug.cgi?id=76648
Reviewed by Geoffrey Garen.
Source/JavaScriptCore:
Added a new overloaded append member that takes a String& argument, an offest
and a length to do direct sub string appending to a StringBuilder.
- wtf/text/StringBuilder.h:
(WTF::StringBuilder::append):
Source/WebCore:
Using new overloaded append(String&, offset, length) member to build result string.
The new member properly handles 8/16 bit-ness of strings.
Functionality not changed, therefore no new tests.
- platform/text/DecodeEscapeSequences.h:
(WebCore::decodeEscapeSequences):
- 9:53 AM Changeset in webkit [106252] by
-
- 2 edits in trunk/Source/WebCore
Not reviewed: follow up to r105625, use proper event categoty in inspector frontend.
- inspector/front-end/BreakpointsSidebarPane.js:
(WebInspector.EventListenerBreakpointsSidebarPane):
- 9:31 AM Changeset in webkit [106251] by
-
- 3 edits2 adds in trunk
REGRESSION (r82580): Reproducible crash in CSSPrimitiveValue::computeLengthDouble
https://bugs.webkit.org/show_bug.cgi?id=61989
Patch by Parag Radke <nrqv63@motorola.com> on 2012-01-30
Reviewed by Simon Fraser.
Source/WebCore:
According to css3 specs when font-size is specified in 'rems' for an element implies the font-size
of the root element. In this case as HTML element has a property 'display:none' and hence renderer
is NULL causes this crash.
Test: fast/css/fontsize-unit-rems-crash.html
- css/CSSPrimitiveValue.cpp:
(WebCore::CSSPrimitiveValue::computeLengthDouble):
Added a null check for the root element's RenderStyle as it can be null in case of html has a property
hidden or display:none.
LayoutTests:
Added a test case to check rems unit (css3) with html property display:none.
- fast/css/fontsize-unit-rems-crash-expected.txt: Added.
- fast/css/fontsize-unit-rems-crash.html: Added.
- 9:27 AM Changeset in webkit [106250] by
-
- 19 edits in trunk
[Qt] WKTR: Use a software rendering pipiline when running tests.
https://bugs.webkit.org/show_bug.cgi?id=76708
Reviewed by Kenneth Rohde Christiansen.
Source/WebCore:
- platform/graphics/qt/TextureMapperQt.cpp: Allow setting the context to null.
(WebCore::TextureMapperQt::setGraphicsContext):
- platform/graphics/qt/TextureMapperQt.h:
(WebCore::TextureMapperQt::initialize):
Source/WebKit2:
Animation layout tests require the graphics layers tree to be updated
to pass. WebkitTestRunner doesn't show its wrapping QQuickView, which
prevents the rendering pipeline to run and then in turn blocks the web
process from processing further graphics layer updates.
This allows the tests to use a TextureMapperQt to empty the LayerTreeHost's
message queue and render the layers in software on an offscreen buffer.
- UIProcess/API/qt/qquickwebpage.cpp:
(QQuickWebPagePrivate::paint):
- UIProcess/API/qt/qquickwebview.cpp:
(QQuickWebViewPrivate::QQuickWebViewPrivate):
(QQuickWebViewPrivate::setNeedsDisplay):
(QQuickWebViewExperimental::setRenderToOffscreenBuffer):
(QQuickWebViewExperimental::renderToOffscreenBuffer):
- UIProcess/API/qt/qquickwebview_p.h:
- UIProcess/API/qt/qquickwebview_p_p.h:
(QQuickWebViewPrivate::setRenderToOffscreenBuffer):
(QQuickWebViewPrivate::renderToOffscreenBuffer):
- UIProcess/qt/LayerTreeHostProxyQt.cpp:
(WebKit::LayerTreeHostProxy::paintToGraphicsContext):
Tools:
- WebKitTestRunner/qt/PlatformWebViewQt.cpp:
(WTR::WrapperWindow::WrapperWindow):
(WTR::PlatformWebView::PlatformWebView): Use software rendering of layers since the wrapping QQuickView isn't shown.
LayoutTests:
- platform/qt-wk2/Skipped:
- 9:19 AM Changeset in webkit [106249] by
-
- 2 edits in trunk/Source/WebKit2
[Qt][WK2] Render layers do not get flushed when the scroll animation finishes
https://bugs.webkit.org/show_bug.cgi?id=77338
Reviewed by Noam Rosenthal.
Schedule a layer flush when setting the visible content rect and scale.
Fixed with the help of Jocelyn Turcotte <jocelyn.turcotte@nokia.com>
- WebProcess/WebPage/qt/LayerTreeHostQt.cpp:
(WebKit::LayerTreeHostQt::setVisibleContentRectAndScale):
- 8:58 AM Changeset in webkit [106248] by
-
- 8 edits in trunk/Source/WebCore
Remove all references to NSPasteboard objects from the Pasteboard
class interface on Mac.
https://bugs.webkit.org/show_bug.cgi?id=77261
This is cleanup work needed as first step in the direction of
removing access to NSPasteboard from the WebProcess.
This way all access to the NSPasteboard object are internal to
the class.
Removed static methods taking NSPasteboard as paramenter and changed
the constructor of the class to take the pasteboard name instead of
the NSPasteboard object.
Reviewed by Alexey Proskuryakov.
No new tests. There is no change in behavior.
- editing/Editor.cpp: Removed ununsed private method writeSelectionToPasteboard.
(WebCore):
- editing/Editor.h: Ditto.
(Editor):
():
- editing/mac/EditorMac.mm:
(WebCore::Editor::writeSelectionToPasteboard):
(WebCore::Editor::readSelectionFromPasteboard):
- platform/Pasteboard.h:
(Pasteboard):
- platform/mac/ClipboardMac.mm:
(WebCore::ClipboardMac::writeRange):
(WebCore::ClipboardMac::writePlainText):
(WebCore::ClipboardMac::writeURL):
- platform/mac/DragDataMac.mm:
(WebCore::DragData::asPlainText):
(WebCore::DragData::asURL):
(WebCore::DragData::asFragment):
- platform/mac/PasteboardMac.mm:
(WebCore::Pasteboard::generalPasteboard):
(WebCore::Pasteboard::Pasteboard):
(WebCore::Pasteboard::writeSelectionForTypes): Added.
(WebCore::Pasteboard::writePlainText):
(WebCore::Pasteboard::writeSelection):
(WebCore::Pasteboard::writeURLForTypes): Added.
(WebCore::Pasteboard::writeURL):
(WebCore::Pasteboard::writeImage):
- 8:46 AM Changeset in webkit [106247] by
-
- 32 edits in trunk/Source
Reduce non-CSSOM API of CSSStyleDeclaration
https://bugs.webkit.org/show_bug.cgi?id=77299
Reviewed by Andreas Kling.
../WebCore:
CSSStyleDeclaration should expose the CSSOM API only. Subclasses should expose the internal API only.
This will move us closer to being able to split the CSSOM API from the internal implementation.
- Make CSSStyleDeclaration CSSOM functions virtual, internal functions non-virtual.
- Move implementations to subclasses (CSSComputedStyleDeclaration, CSSMutableStyleDeclaration).
- Make CSSOM functions in the subclasses private (making it harder to invoke them internally).
- Switch a bunch of places to use internal API instead of CSSOM.
- bindings/v8/custom/V8CSSStyleDeclarationCustom.cpp:
(WebCore::V8CSSStyleDeclaration::namedPropertyGetter):
(WebCore::V8CSSStyleDeclaration::namedPropertySetter):
Switch to *Internal versions of the CSSOM functions.
- css/CSSComputedStyleDeclaration.cpp:
(WebCore):
(WebCore::CSSComputedStyleDeclaration::length):
(WebCore::CSSComputedStyleDeclaration::cssPropertyMatches):
(WebCore::CSSComputedStyleDeclaration::copyPropertiesInSet):
Move copyPropertiesInSet to subclasses, devirtualize.
(WebCore::CSSComputedStyleDeclaration::getPropertyCSSValue):
(WebCore::CSSComputedStyleDeclaration::getPropertyValue):
(WebCore::CSSComputedStyleDeclaration::getPropertyPriority):
(WebCore::CSSComputedStyleDeclaration::getPropertyShorthand):
(WebCore::CSSComputedStyleDeclaration::isPropertyImplicit):
(WebCore::CSSComputedStyleDeclaration::setProperty):
(WebCore::CSSComputedStyleDeclaration::removeProperty):
- css/CSSComputedStyleDeclaration.h:
(CSSComputedStyleDeclaration):
- css/CSSFontFaceRule.cpp:
(WebCore::CSSFontFaceRule::cssText):
- css/CSSMutableStyleDeclaration.cpp:
(WebCore::CSSMutableStyleDeclaration::length):
(WebCore::CSSMutableStyleDeclaration::asText):
(WebCore::CSSMutableStyleDeclaration::cssText):
(WebCore):
(WebCore::CSSMutableStyleDeclaration::getPropertyCSSValue):
(WebCore::CSSMutableStyleDeclaration::getPropertyValue):
(WebCore::CSSMutableStyleDeclaration::getPropertyPriority):
(WebCore::CSSMutableStyleDeclaration::getPropertyShorthand):
(WebCore::CSSMutableStyleDeclaration::isPropertyImplicit):
(WebCore::CSSMutableStyleDeclaration::setProperty):
(WebCore::CSSMutableStyleDeclaration::removeProperty):
(WebCore::CSSMutableStyleDeclaration::copyPropertiesInSet):
Move copyPropertiesInSet to subclasses, devirtualize.
(WebCore::CSSMutableStyleDeclaration::cssPropertyMatches):
(WebCore::CSSMutableStyleDeclaration::removeEquivalentProperties):
Move diff() to CSSMutableStyleDeclaration, rename to removeEquivalentProperties, switch to mutate
this object instead of the argument object.
- css/CSSMutableStyleDeclaration.h:
(CSSMutableStyleDeclaration):
(WebCore::CSSMutableStyleDeclaration::propertyCount):
(WebCore::CSSMutableStyleDeclaration::isEmpty):
(WebCore::CSSMutableStyleDeclaration::propertyAt):
Expose properties and property count internally (iterator should be removed in favor of these).
- css/CSSStyleDeclaration.cpp:
(WebCore):
- css/CSSStyleDeclaration.h:
(CSSStyleDeclaration):
(WebCore::CSSStyleDeclaration::getPropertyCSSValueInternal):
(WebCore::CSSStyleDeclaration::getPropertyValueInternal):
(WebCore::CSSStyleDeclaration::setPropertyInternal):
Add *Internal versions of some CSSOM APIs to support some editing and bindings uses.
These take propertyIDs instead of strings names.
- css/CSSStyleRule.cpp:
(WebCore::CSSStyleRule::cssText):
- css/CSSStyleSelector.cpp:
(WebCore::leftToRightDeclaration):
(WebCore::rightToLeftDeclaration):
(WebCore::CSSStyleSelector::collectMatchingRulesForList):
(WebCore::CSSStyleSelector::matchPageRulesForList):
- css/WebKitCSSKeyframeRule.cpp:
(WebCore::WebKitCSSKeyframeRule::cssText):
- dom/StyledElement.cpp:
(WebCore::StyledElement::updateStyleAttribute):
- editing/ApplyStyleCommand.cpp:
(WebCore::ApplyStyleCommand::applyRelativeFontStyleChange):
(WebCore::ApplyStyleCommand::removeEmbeddingUpToEnclosingBlock):
(WebCore::ApplyStyleCommand::applyInlineStyleToNodeRange):
(WebCore::ApplyStyleCommand::applyInlineStyleToPushDown):
(WebCore::ApplyStyleCommand::addBlockStyle):
(WebCore::ApplyStyleCommand::addInlineStyleIfNeeded):
- editing/EditingStyle.cpp:
():
(WebCore::copyEditingProperties):
(WebCore):
(WebCore::propertyCSSValue):
(WebCore::getRGBAFontColor):
(WebCore::EditingStyle::overrideWithStyle):
(WebCore::EditingStyle::removeStyleAddedByNode):
(WebCore::EditingStyle::removeStyleConflictingWithStyleOfNode):
(WebCore::EditingStyle::triStateOfStyle):
(WebCore::EditingStyle::styleIsPresentInComputedStyleOfNode):
(WebCore::EditingStyle::prepareToApplyAt):
(WebCore::removePropertiesInStyle):
(WebCore::EditingStyle::removeStyleFromRulesAndContext):
(WebCore::EditingStyle::removePropertiesInElementDefaultStyle):
(WebCore::StyleChange::StyleChange):
(WebCore::fontWeightIsBold):
(WebCore::getPropertiesNotIn):
(WebCore::getIdentifierValue):
(WebCore::hasTransparentBackgroundColor):
- editing/EditingStyle.h:
Adapt to changes.
(WebCore):
- editing/ReplaceSelectionCommand.cpp:
(WebCore::ReplaceSelectionCommand::removeRedundantStylesAndKeepStyleSpanInline):
(WebCore::handleStyleSpansBeforeInsertion):
(WebCore::ReplaceSelectionCommand::handleStyleSpans):
- editing/mac/EditorMac.mm:
(WebCore::styleForSelectionStart):
- editing/markup.cpp:
(WebCore::StyledMarkupAccumulator::appendStyleNodeOpenTag):
(WebCore::StyledMarkupAccumulator::appendElement):
- html/ImageDocument.cpp:
(WebCore::ImageDocument::resizeImageToFit):
(WebCore::ImageDocument::restoreImageSize):
(WebCore::ImageDocument::windowSizeChanged):
- html/canvas/CanvasRenderingContext2D.cpp:
(WebCore::CanvasRenderingContext2D::setFont):
- html/canvas/CanvasStyle.cpp:
(WebCore::currentColor):
- inspector/InspectorStyleSheet.cpp:
(WebCore::InspectorStyleSheet::revalidateStyle):
- page/PageSerializer.cpp:
(WebCore::PageSerializer::retrieveResourcesForCSSDeclaration):
- rendering/RenderTreeAsText.cpp:
(WebCore::isEmptyOrUnstyledAppleStyleSpan):
- svg/SVGStyledElement.cpp:
(WebCore::SVGStyledElement::getPresentationAttribute):
../WebKit/qt:
- Api/qwebelement.cpp:
(QWebElement::styleProperty):
(QWebElement::setStyleProperty):
- WebCoreSupport/DumpRenderTreeSupportQt.cpp:
(DumpRenderTreeSupportQt::computedStyleIncludingVisitedInfo):
- 8:40 AM Changeset in webkit [106246] by
-
- 3 edits in trunk/Source/WebCore
Web Inspector: protocol validator should fail if one of response fields turns optional
https://bugs.webkit.org/show_bug.cgi?id=76452
Reviewed by Yury Semikhatsky.
- inspector/Inspector-0.1.json:
- inspector/Inspector.json:
- inspector/generate-inspector-protocol-version:
(compare_commands):
(compare_events):
(compare_params_list):
(compare_types):
(self_test):
- 8:17 AM Changeset in webkit [106245] by
-
- 3 edits in trunk/Source/WebCore
Web Inspector: keyboard navigation through comparison view in heap profiler should update retainers view
https://bugs.webkit.org/show_bug.cgi?id=77326
Keyboard navigation in the detailed heap snapshot view now updates retainers view.
Reviewed by Pavel Feldman.
- inspector/front-end/DataGrid.js: Added SelectedNode/DeselectedNode events to DataGrid.
(WebInspector.DataGridNode.prototype.select):
(WebInspector.DataGridNode.prototype.deselect):
- inspector/front-end/DetailedHeapshotView.js:
(WebInspector.DetailedHeapshotView.prototype._selectionChanged):
(WebInspector.DetailedHeapshotView.prototype._setRetainmentDataGridSource):
- 8:05 AM BuildingQtOnLinux edited by
- Remove --no-webkit2 flag comment, this is not important. (diff)
- 7:49 AM Changeset in webkit [106244] by
-
- 2 edits in trunk/Source/WebCore
2012-01-30 Yury Semikhatsky <yurys@chromium.org>
Unreviewed. Fix inspector front-end compilation.
- inspector/front-end/RemoteObject.js: (WebInspector.RemoteObject): (WebInspector.RemoteObject.fromPayload):
- 7:24 AM Changeset in webkit [106243] by
-
- 3 edits in trunk/Source/WebKit/chromium
[chromium] remove exit time destructors from TextureManagerTest
https://bugs.webkit.org/show_bug.cgi?id=77333
Reviewed by Tony Gentilcore.
Non-trivial, global objects require an at-exit time destructor which add
unnecessary complexity and delays to shutdown.
- tests/FakeCCLayerTreeHostClient.h: added newline at EOF
- tests/TextureManagerTest.cpp:
(WTF::FakeTextureAllocator::~FakeTextureAllocator):
(TextureManagerTest):
(WTF::TextureManagerTest::TextureManagerTest):
(WTF::TextureManagerTest::~TextureManagerTest):
(WTF::TextureManagerTest::texturesMemorySize):
(WTF::TextureManagerTest::createTextureManager):
(WTF::TextureManagerTest::requestTexture):
(WTF::TEST_F):
- 7:19 AM Changeset in webkit [106242] by
-
- 2 edits in trunk/Source/WebCore
Node::parentOrHostElement(): Node::shadowHost() already returns an Element*
https://bugs.webkit.org/show_bug.cgi?id=77332
Reviewed by Kent Tamura.
No new tests. (simple refactoring)
- dom/Node.cpp:
(WebCore::Node::parentOrHostElement):
- 7:05 AM Changeset in webkit [106241] by
-
- 1 edit3 adds in trunk
Add manual tests for tap highlighting
Rubberstamped by Simon Hausmann.
- ManualTests/qt/tap-highlighting-colors.html: Added.
- ManualTests/qt/tap-highlighting-images.html: Added.
- ManualTests/qt/tap-highlighting-inlines.html: Added.
- 6:25 AM Changeset in webkit [106240] by
-
- 6 edits in trunk/Source
Unreviewed. Fix make distcheck.
Source/WebCore:
- GNUmakefile.am: Add idl files in Source/WebCore/html/shadow/ to
EXTRA_DIST.
- GNUmakefile.list.am: Add missing files.
Source/WebKit2:
- GNUmakefile.am: Add missing files.
- WebProcess/WebPage/TapHighlightController.cpp: Add #if
ENABLE(TOUCH_EVENTS).
- 6:18 AM BuildingQtOnLinux edited by
- Add a link to qt5 build script. (diff)
- 5:34 AM Changeset in webkit [106239] by
-
- 2 edits in trunk/LayoutTests
[Qt][WK2] http/tests/history/replacestate-post-to-get-2.html fails with timeout
https://bugs.webkit.org/show_bug.cgi?id=76784
- platform/qt-wk2/Skipped: Unskip now passing test.
- 5:28 AM BuildingQt5OnHarmattan edited by
- (diff)
- 5:13 AM SettingUpDevelopmentEnvironmentForN9 edited by
- (diff)
- 5:10 AM Changeset in webkit [106238] by
-
- 5 edits2 adds in trunk
Web Inspector: TabbedEditorContainer should save open tabs.
https://bugs.webkit.org/show_bug.cgi?id=76912
Reviewed by Pavel Feldman.
Source/WebCore:
Test: inspector/tabbed-editors-history.html
- inspector/front-end/ScriptsPanel.js:
(WebInspector.ScriptsPanel.prototype._uiSourceCodeAdded):
(WebInspector.ScriptsPanel.prototype._reset):
(WebInspector.ScriptsPanel.prototype._showFile):
(WebInspector.ScriptsPanel.prototype._updateExecutionLine):
(WebInspector.ScriptsPanel.prototype._editorSelected):
(WebInspector.EditorContainer.prototype.uiSourceCodeAdded):
(WebInspector.ScriptsPanel.SingleFileEditorContainer.prototype.showFile):
(WebInspector.ScriptsPanel.SingleFileEditorContainer.prototype.reset):
- inspector/front-end/TabbedEditorContainer.js:
(WebInspector.TabbedEditorContainer):
(WebInspector.TabbedEditorContainer.prototype.showFile):
(WebInspector.TabbedEditorContainer.prototype._editorClosedByUserAction):
(WebInspector.TabbedEditorContainer.prototype._editorSelectedByUserAction):
(WebInspector.TabbedEditorContainer.prototype._updateHistory.tabIdToURL):
(WebInspector.TabbedEditorContainer.prototype._updateHistory):
(WebInspector.TabbedEditorContainer.prototype._appendFileTab):
(WebInspector.TabbedEditorContainer.prototype._tabClosed):
(WebInspector.TabbedEditorContainer.prototype._tabSelected):
(WebInspector.TabbedEditorContainer.prototype.reset):
(WebInspector.TabbedEditorContainer.History):
(WebInspector.TabbedEditorContainer.History.prototype.index):
(WebInspector.TabbedEditorContainer.History.prototype.update):
(WebInspector.TabbedEditorContainer.History.prototype.remove):
(WebInspector.TabbedEditorContainer.History.prototype.save):
- inspector/front-end/TabbedPane.js:
(WebInspector.TabbedPane.prototype.appendTab):
(WebInspector.TabbedPane.prototype.closeTab):
(WebInspector.TabbedPane.prototype._innerCloseTab):
(WebInspector.TabbedPane.prototype.closeAllTabs):
(WebInspector.TabbedPane.prototype.lastOpenedTabIds):
(WebInspector.TabbedPane.prototype._tabsSelectChanged):
LayoutTests:
- inspector/tabbed-editors-history-expected.txt: Added.
- inspector/tabbed-editors-history.html: Added.
- 5:04 AM Changeset in webkit [106237] by
-
- 5 edits in trunk
Web Inspector: [Styles] Unable to paste and subsequently edit multiple properties in the Styles pane
https://bugs.webkit.org/show_bug.cgi?id=77209
Reviewed by Pavel Feldman.
Source/WebCore:
Check if the property value contains a ";" before kicking the freeflow text update.
- inspector/front-end/StylesSidebarPane.js:
(WebInspector.StylePropertyTreeElement.prototype):
LayoutTests:
- inspector/styles/styles-add-blank-property-expected.txt:
- inspector/styles/styles-add-blank-property.html:
- 5:02 AM Changeset in webkit [106236] by
-
- 3 edits in trunk/Source/WebCore
Web Inspector: Change Open Resource and Go To function shortcuts in scripts panel.
https://bugs.webkit.org/show_bug.cgi?id=77321
Reviewed by Pavel Feldman.
- inspector/front-end/FilteredItemSelectionDialog.js:
(WebInspector.JavaScriptOutlineDialog.createShortcut):
(WebInspector.OpenResourceDialog.createShortcut):
- inspector/front-end/ScriptsPanel.js:
- 5:00 AM Changeset in webkit [106235] by
-
- 2 edits in trunk/Source/WebCore
Web Inspector: hide function popover in heap snapshot view before navigating to the function definition
https://bugs.webkit.org/show_bug.cgi?id=77330
Reviewed by Pavel Feldman.
- inspector/front-end/DetailedHeapshotView.js:
(WebInspector.DetailedHeapshotView.prototype.willHide):
- 4:50 AM Changeset in webkit [106234] by
-
- 2 edits in trunk/Source/WebCore
[BlackBerry] Credential backing store implementation
https://bugs.webkit.org/show_bug.cgi?id=76761
Patch by Jonathan Dong <Jonathan Dong> on 2012-01-30
Reviewed by Antonio Gomes.
Implemented credential backing store database and related
operations in class CredentialBackingStore.
- platform/network/blackberry/CredentialBackingStore.cpp:
(WebCore::CredentialBackingStore::~CredentialBackingStore):
(WebCore::CredentialBackingStore::open):
(WebCore::CredentialBackingStore::close):
(WebCore::CredentialBackingStore::addLogin):
(WebCore::CredentialBackingStore::updateLogin):
(WebCore::CredentialBackingStore::hasLogin):
(WebCore::CredentialBackingStore::getLogin):
(WebCore::CredentialBackingStore::removeLogin):
(WebCore::CredentialBackingStore::clear):
- 4:47 AM Changeset in webkit [106233] by
-
- 2 edits in trunk/Tools
[Qt] Store build config immediately instead of waiting for a successfull build
Otherwise we will not pick up changes to the config if the build is aborted,
for example due to build errors or if the user pressed Ctrl+Z. We now write
the new config before starting a build, but still defer to write the SVN
revision once the build completes successfully.
Reviewed by Ossy.
- 4:36 AM Changeset in webkit [106232] by
-
- 3 edits1 add in trunk
.: Manual test of number of resize events emitted during page generation.
https://bugs.webkit.org/show_bug.cgi?id=77212
Is needed to test if too many resize events are send when using fixed
layout, and needs to be a manual test because the test framework does
not currently support testing fixed layout.
Patch by Allan Sandfeld Jensen <allan.jensen@nokia.com> on 2012-01-30
Reviewed by Kenneth Rohde Christiansen.
- ManualTests/resize-events.html: Added.
Source/WebCore: Only send resize events when layout size changes.
https://bugs.webkit.org/show_bug.cgi?id=77212
When using fixed layout the widget size is the size of content, therefore
resize checks must check against layoutsize and not widget size.
Patch by Allan Sandfeld Jensen <allan.jensen@nokia.com> on 2012-01-30
Reviewed by Kenneth Rohde Christiansen.
Needs to be manual tests because the test framework does not currently
support testing fixed layout.
Tests: ManualTests/resize-events.html
- page/FrameView.cpp:
(WebCore::FrameView::layout):
(WebCore::FrameView::performPostLayoutTasks):
- 4:10 AM Changeset in webkit [106231] by
-
- 4 edits1 add in trunk/Tools
[Qt] Add "New Window" button to the Qt MiniBrowser
https://bugs.webkit.org/show_bug.cgi?id=77281
Patch by Alexander Færøy <ahf@0x90.dk> on 2012-01-30
Reviewed by Tor Arne Vestbø.
- MiniBrowser/qt/BrowserWindow.cpp:
(BrowserWindow::BrowserWindow):
- MiniBrowser/qt/MiniBrowser.qrc:
- MiniBrowser/qt/icons/plus.png: Added.
- MiniBrowser/qt/qml/BrowserWindow.qml:
- 4:08 AM WebKitEFLLayoutTest edited by
- Add section on debugging (diff)
- 4:06 AM WebKitEFLLayoutTest edited by
- Improve formatting (diff)
- 4:06 AM Changeset in webkit [106230] by
-
- 2 edits in trunk/Source/WebCore
Web Inspector: TimelinePanel does not respect InspectorFrontendHost.canSaveAs
https://bugs.webkit.org/show_bug.cgi?id=77301
Reviewed by Vsevolod Vlasov.
- inspector/front-end/TimelinePanel.js:
(WebInspector.TimelinePanel.prototype._registerShortcuts):
- 3:39 AM Changeset in webkit [106229] by
-
- 2 edits in trunk/Source/WebCore
2012-01-30 Nikolas Zimmermann <nzimmermann@rim.com>
Not reviewed. Fix Clang build for real after r106218.
- inspector/CodeGeneratorInspector.py: (Generator.go): Add virtual destructor to the ABCs.
- 3:32 AM Changeset in webkit [106228] by
-
- 2 edits in trunk/Source/WebCore
No need to keep anonymous RenderBlock in DETAILS.
https://bugs.webkit.org/show_bug.cgi?id=77322
Patch by Shinya Kawanaka <shinyak@google.com> on 2012-01-30
Reviewed by Hajime Morita.
Anonymous RenderBlock of DETAILS element was not squashed when detaching them.
However, it should be removed.
No new tests. Should be covered by existing tests.
- rendering/RenderBlock.cpp:
(WebCore::canMergeContiguousAnonymousBlocks):
- 3:20 AM SettingUpDevelopmentEnvironmentForN9 edited by
- (diff)
- 3:17 AM Changeset in webkit [106227] by
-
- 2 edits in trunk/Source/WebCore
Custom written CSS lexer
https://bugs.webkit.org/show_bug.cgi?id=70107
Rubber Stamped by Csaba Osztrogonác.
Do not advance pointer at the end of input, just
keep returning with END_TOKEN.
- css/CSSParser.cpp:
(WebCore::CSSParser::lex):
- 3:10 AM SettingUpDevelopmentEnvironmentForN9 edited by
- (diff)
- 2:48 AM Changeset in webkit [106226] by
-
- 2 edits in trunk/LayoutTests
[Chromium] Unreviewed test expectation change.
fast/js/dfg-int32array.html is slow on WIN DEBUG and MAC DEBUG.
- platform/chromium/test_expectations.txt:
- 2:40 AM Changeset in webkit [106225] by
-
- 9 edits2 adds in trunk
The query selector for HTMLContentElement should follow the shadow dom spec.
https://bugs.webkit.org/show_bug.cgi?id=75946
Patch by Shinya Kawanaka <shinyak@google.com> on 2012-01-30
Reviewed by Hajime Morita.
Source/WebCore:
Checks the query selector of HTMLContentElement is valid.
If not valid, the selector won't match anything, and shows fallback element.
Test: fast/dom/shadow/content-selector-query.html
- html/shadow/ContentSelectorQuery.cpp:
(WebCore::ContentSelectorQuery::ContentSelectorQuery):
(WebCore::ContentSelectorQuery::isValidSelector):
(WebCore::ContentSelectorQuery::matches):
When a select query is not valid, any element won't be matched.
(WebCore::validateSubSelector):
(WebCore::validateSelector):
(WebCore::ContentSelectorQuery::validateSelectorList):
Validate selectors.
- html/shadow/ContentSelectorQuery.h:
- html/shadow/HTMLContentElement.cpp:
(WebCore::HTMLContentElement::isSelectValid):
Returns true if select attribute is valid.
- html/shadow/HTMLContentElement.h:
- testing/Internals.cpp:
(WebCore::Internals::isValidContentSelect):
- testing/Internals.h:
- testing/Internals.idl:
LayoutTests:
- fast/dom/shadow/content-selector-query-expected.txt: Added.
- fast/dom/shadow/content-selector-query.html: Added.
- 2:39 AM Changeset in webkit [106224] by
-
- 7 edits in trunk/Tools
Unreviewed rolling out r106222.
- Scripts/webkitpy/layout_tests/controllers/single_test_runner.py:
(SingleTestRunner.init):
- Scripts/webkitpy/layout_tests/controllers/worker.py:
(Worker.handle_test_list):
- Scripts/webkitpy/layout_tests/models/test_input.py:
(TestInput.init):
- Scripts/webkitpy/layout_tests/run_webkit_tests.py:
(_set_up_derived_options):
(parse_args):
- Scripts/webkitpy/layout_tests/run_webkit_tests_integrationtest.py:
(MainTest.test_repeat_each):
- WebKitTestRunner/TestInvocation.cpp:
(WTR::TestInvocation::setIsPixelTest):
- 2:10 AM Changeset in webkit [106223] by
-
- 17 edits5 adds2 deletes in trunk
Unreviewed, rolling out r106219.
http://trac.webkit.org/changeset/106219
https://bugs.webkit.org/show_bug.cgi?id=77083
This broke Chromium's test_shell.
Source/WebCore:
- GNUmakefile.list.am:
- Target.pri:
- WebCore.gypi:
- WebCore.vcproj/WebCore.vcproj:
- WebCore.xcodeproj/project.pbxproj:
- platform/mock/SpeechInputClientMock.cpp: Added.
(WebCore):
(WebCore::SpeechInputClientMock::SpeechInputClientMock):
(WebCore::SpeechInputClientMock::setListener):
(WebCore::SpeechInputClientMock::startRecognition):
(WebCore::SpeechInputClientMock::stopRecording):
(WebCore::SpeechInputClientMock::cancelRecognition):
(WebCore::SpeechInputClientMock::addRecognitionResult):
(WebCore::SpeechInputClientMock::clearResults):
(WebCore::SpeechInputClientMock::timerFired):
- platform/mock/SpeechInputClientMock.h: Added.
(WebCore):
(SpeechInputClientMock):
Source/WebKit/chromium:
- WebKit.gyp:
- public/WebSpeechInputControllerMock.h: Added.
(WebKit):
(WebSpeechInputControllerMock):
(WebKit::WebSpeechInputControllerMock::~WebSpeechInputControllerMock):
- public/WebSpeechInputResult.h:
(WebSpeechInputResult):
- src/WebSpeechInputControllerMockImpl.cpp: Added.
(WebKit):
(WebKit::WebSpeechInputControllerMock::create):
(WebKit::WebSpeechInputControllerMockImpl::WebSpeechInputControllerMockImpl):
(WebKit::WebSpeechInputControllerMockImpl::~WebSpeechInputControllerMockImpl):
(WebKit::WebSpeechInputControllerMockImpl::addMockRecognitionResult):
(WebKit::WebSpeechInputControllerMockImpl::clearResults):
(WebKit::WebSpeechInputControllerMockImpl::didCompleteRecording):
(WebKit::WebSpeechInputControllerMockImpl::didCompleteRecognition):
(WebKit::WebSpeechInputControllerMockImpl::setRecognitionResult):
(WebKit::WebSpeechInputControllerMockImpl::startRecognition):
(WebKit::WebSpeechInputControllerMockImpl::cancelRecognition):
(WebKit::WebSpeechInputControllerMockImpl::stopRecording):
- src/WebSpeechInputControllerMockImpl.h: Added.
(WebCore):
(WebKit):
(WebSpeechInputControllerMockImpl):
- src/WebSpeechInputResult.cpp:
Tools:
- DumpRenderTree/DumpRenderTree.gypi:
- DumpRenderTree/chromium/LayoutTestController.cpp:
(LayoutTestController::addMockSpeechInputResult):
- DumpRenderTree/chromium/LayoutTestController.h:
(WebKit):
(LayoutTestController):
():
- DumpRenderTree/chromium/MockWebSpeechInputController.cpp: Removed.
- DumpRenderTree/chromium/MockWebSpeechInputController.h: Removed.
- DumpRenderTree/chromium/TestShell.cpp:
- DumpRenderTree/chromium/WebViewHost.cpp:
(WebViewHost::speechInputController):
- DumpRenderTree/chromium/WebViewHost.h:
(WebKit):
(WebViewHost::speechInputControllerMock):
(WebViewHost):
():
- 2:06 AM Changeset in webkit [106222] by
-
- 6 edits in trunk/Tools
2012-01-30 Balazs Kelemen <kbalazs@webkit.org>
[WK2] add flag to only check pixel results if png files exist
https://bugs.webkit.org/show_bug.cgi?id=70484
Reviewed by Dirk Pranke.
- Scripts/webkitpy/layout_tests/controllers/single_test_runner.py: (SingleTestRunner.init): (SingleTestRunner._should_fetch_expected_checksum):
- Scripts/webkitpy/layout_tests/controllers/worker.py: (Worker.handle_test_list):
- Scripts/webkitpy/layout_tests/models/test_input.py: (TestInput.init):
- Scripts/webkitpy/layout_tests/run_webkit_tests.py: (_set_up_derived_options): (parse_args):
- Scripts/webkitpy/layout_tests/run_webkit_tests_integrationtest.py: (MainTest.test_skip_pixel_test_if_no_baseline_option):
- WebKitTestRunner/TestInvocation.cpp: (WTR::TestInvocation::setIsPixelTest):
- 1:42 AM Changeset in webkit [106221] by
-
- 2 edits in trunk/Source/WebCore
2012-01-30 Yury Semikhatsky <yurys@chromium.org>
Unreviewed. Clang build fix after r106218
- inspector/CodeGeneratorInspector.py:
- 1:40 AM Changeset in webkit [106220] by
-
- 3 edits2 moves1 add in trunk/Source/WebCore
Make GraphicsContext3DPrivate of GTK port shareable.
https://bugs.webkit.org/show_bug.cgi?id=77296
Patch by ChangSeok Oh <ChangSeok Oh> on 2012-01-30
Reviewed by Eric Seidel.
GraphicsContext3DPrivate.cpp/h of GTK port look shareable with another port
using glx backend for WebGL. For example, EFL port.
Moved GraphicsContext3DPrivate.cpp and its header into Source/WebCore/platform/graphics/glx.
No new tests required, because of no new feature.
- GNUmakefile.am:
- GNUmakefile.list.am:
- platform/graphics/glx/GraphicsContext3DPrivate.cpp: Renamed from Source/WebCore/platform/graphics/gtk/GraphicsContext3DPrivate.cpp.
(sharedDisplay):
(WebCore):
(WebCore::activeGraphicsContexts):
(WebCore::GraphicsContext3DPrivate::addActiveGraphicsContext):
(WebCore::GraphicsContext3DPrivate::removeActiveGraphicsContext):
(WebCore::GraphicsContext3DPrivate::cleanupActiveContextsAtExit):
(WebCore::GraphicsContext3DPrivate::create):
(WebCore::GraphicsContext3DPrivate::createPbufferContext):
(WebCore::GraphicsContext3DPrivate::createPixmapContext):
(WebCore::GraphicsContext3DPrivate::GraphicsContext3DPrivate):
(WebCore::GraphicsContext3DPrivate::~GraphicsContext3DPrivate):
(WebCore::GraphicsContext3DPrivate::makeContextCurrent):
- platform/graphics/glx/GraphicsContext3DPrivate.h: Renamed from Source/WebCore/platform/graphics/gtk/GraphicsContext3DPrivate.h.
(WebCore):
(GraphicsContext3DPrivate):
- 1:31 AM Changeset in webkit [106219] by
-
- 17 edits2 adds5 deletes in trunk
Speech Input: move MockSpeechInputClient into Chromium DumpRenderTree implementation
https://bugs.webkit.org/show_bug.cgi?id=77083
Reviewed by Darin Fisher.
Source/WebCore:
Remove SpeechInputClientMock. The mock is moving to the DumpRenderTree
implementation.
No new tests, just refactoring.
- GNUmakefile.list.am:
- Target.pri:
- WebCore.gypi:
- WebCore.vcproj/WebCore.vcproj:
- WebCore.xcodeproj/project.pbxproj:
- platform/mock/SpeechInputClientMock.cpp: Removed.
- platform/mock/SpeechInputClientMock.h: Removed.
Source/WebKit/chromium:
Remove the WebSpeechInputControllerMock interface and proxy
implementation. The mock is moving to the DumpRenderTree
implementation instead, which removes the need to expose this
interface in the WebKit API.
Also add a proper copy constructor for WebSpeechInputResult.
The default one doesn't do a proper copy.
- WebKit.gyp:
- public/WebSpeechInputControllerMock.h: Removed.
- public/WebSpeechInputResult.h:
(WebKit::WebSpeechInputResult::WebSpeechInputResult):
(WebSpeechInputResult):
- src/WebSpeechInputControllerMockImpl.cpp: Removed.
- src/WebSpeechInputControllerMockImpl.h: Removed.
- src/WebSpeechInputResult.cpp:
(WebKit::WebSpeechInputResult::set):
(WebKit):
Tools:
Add MockWebSpeechInputController which provides a mock implementation
of the WebSpeechInputController interface, and use that in
DumpRenderTree.
- DumpRenderTree/DumpRenderTree.gypi:
- DumpRenderTree/chromium/LayoutTestController.cpp:
(LayoutTestController::addMockSpeechInputResult):
- DumpRenderTree/chromium/LayoutTestController.h:
():
- DumpRenderTree/chromium/MockWebSpeechInputController.cpp: Added.
(MockWebSpeechInputController::create):
(MockWebSpeechInputController::addMockRecognitionResult):
(MockWebSpeechInputController::clearResults):
(MockWebSpeechInputController::startRecognition):
(MockWebSpeechInputController::cancelRecognition):
(MockWebSpeechInputController::stopRecording):
(MockWebSpeechInputController::MockWebSpeechInputController):
(MockWebSpeechInputController::speechTaskFired):
(MockWebSpeechInputController::SpeechTask::SpeechTask):
(MockWebSpeechInputController::SpeechTask::stop):
(MockWebSpeechInputController::SpeechTask::runIfValid):
- DumpRenderTree/chromium/MockWebSpeechInputController.h: Added.
(WebKit):
(MockWebSpeechInputController):
(MockWebSpeechInputController::taskList):
(SpeechTask):
- DumpRenderTree/chromium/TestShell.cpp:
- DumpRenderTree/chromium/WebViewHost.cpp:
(WebViewHost::speechInputController):
- DumpRenderTree/chromium/WebViewHost.h:
(WebKit):
(WebViewHost::speechInputControllerMock):
(WebViewHost):
():
- 12:57 AM Changeset in webkit [106218] by
-
- 4 edits in trunk/Source/WebCore
Web Inspector: CodeGeneratorInspector.py: clean InspectorBackendDispatcher.h
https://bugs.webkit.org/show_bug.cgi?id=77062
Patch by Peter Rybin <peter.rybin@gmail.com> on 2012-01-30
Reviewed by Yury Semikhatsky.
Code generator is changed. Now it generates InspectorBackendDispatcher
as an abstract class so that implementation details could be fully
hidden in .cpp file in a separate 'impl' class. This should make .h
file more clear.
Also new formal interfaces to domain agents are generated. This is an
improvement over the current case when interfaces to agents are
definen implicitly at calling sites.
- inspector/CodeGeneratorInspector.py:
(RawTypes):
(RawTypes.OutputPassModel):
(RawTypes.OutputPassModel.ByPointer):
(RawTypes.OutputPassModel.ByPointer.get_argument_prefix):
(RawTypes.OutputPassModel.ByPointer.get_parameter_type_suffix):
(RawTypes.OutputPassModel.ByReference):
(RawTypes.OutputPassModel.ByReference.get_argument_prefix):
(RawTypes.OutputPassModel.ByReference.get_parameter_type_suffix):
(RawTypes.BaseType.is_event_param_check_optional):
(RawTypes.String):
(RawTypes.String.get_output_pass_model):
(RawTypes.String.is_heavy_value):
(RawTypes.Int):
(RawTypes.Int.get_output_pass_model):
(RawTypes.Int.is_heavy_value):
(RawTypes.Number):
(RawTypes.Number.get_output_pass_model):
(RawTypes.Number.is_heavy_value):
(RawTypes.Bool.get_c_param_type):
(RawTypes.Bool):
(RawTypes.Bool.get_output_pass_model):
(RawTypes.Bool.is_heavy_value):
(RawTypes.Object):
(RawTypes.Object.get_output_pass_model):
(RawTypes.Object.is_heavy_value):
(RawTypes.Any.get_c_initializer):
(RawTypes.Any):
(RawTypes.Any.get_output_pass_model):
(RawTypes.Any.is_heavy_value):
(RawTypes.Array):
(RawTypes.Array.get_output_pass_model):
(RawTypes.Array.is_heavy_value):
(InspectorBackendDispatcherImpl):
(void):
(Generator):
(Generator.go):
(Generator.process_command):
- inspector/InspectorController.cpp:
(WebCore::InspectorController::connectFrontend):
- inspector/WorkerInspectorController.cpp:
(WebCore::WorkerInspectorController::connectFrontend):
- 12:56 AM SettingUpDevelopmentEnvironmentForN9 edited by
- (diff)
Jan 29, 2012:
- 11:30 PM Changeset in webkit [106217] by
-
- 16 edits2 deletes in trunk
Custom written CSS lexer
https://bugs.webkit.org/show_bug.cgi?id=70107
Reviewed by Antti Koivisto and Oliver Hunt.
.:
Remove tokenizer.cpp from intermediate sources.
- wscript:
Source/JavaScriptCore:
Add new helper functions for the custom written CSS lexer.
- wtf/ASCIICType.h:
(WTF::toASCIILowerUnchecked):
(WTF):
(WTF::isASCIIAlphaCaselessEqual):
Source/WebCore:
This patch replaces the flex based CSS lexer to a
new, custom written one. The new code is more
than 2 times faster according to oprofile and CPU
cycle counters.
The code structure is quite straightforward: it choose
the possible token group based on the first character
and employ utility functions to parse the longer than
one character long ones. Most of the utilities are inline
to make the lexer fast.
All build systems updated. Including removing the flex support.
Existing tests cover this feature.
- CMakeLists.txt:
- DerivedSources.make:
- DerivedSources.pri:
- GNUmakefile.am:
- GNUmakefile.list.am:
- WebCore.gyp/WebCore.gyp:
- WebCore.gyp/scripts/action_maketokenizer.py: Removed.
- WebCore.gypi:
- WebCore.vcproj/WebCore.vcproj:
- WebCore.xcodeproj/project.pbxproj:
- css/CSSParser.cpp:
(WebCore::CSSParser::CSSParser):
(WebCore::CSSParser::setupParser):
(WebCore::parseSimpleLengthValue):
(WebCore::mightBeRGBA):
(WebCore::mightBeRGB):
():
(WebCore::isCSSLetter):
(WebCore):
(WebCore::isCSSEscape):
(WebCore::isURILetter):
(WebCore::isIdentifierStartAfterDash):
(WebCore::isEqualToCSSIdentifier):
(WebCore::checkAndSkipEscape):
(WebCore::skipWhiteSpace):
(WebCore::CSSParser::isIdentifierStart):
(WebCore::CSSParser::checkAndSkipString):
(WebCore::CSSParser::parseEscape):
(WebCore::CSSParser::parseIdentifier):
(WebCore::CSSParser::parseString):
(WebCore::CSSParser::parseURI):
(WebCore::CSSParser::parseUnicodeRange):
(WebCore::CSSParser::parseNthChild):
(WebCore::CSSParser::parseNthChildExtra):
(WebCore::CSSParser::detectFunctionTypeToken):
(WebCore::CSSParser::detectMediaQueryToken):
(WebCore::CSSParser::detectNumberToken):
(WebCore::CSSParser::detectDashToken):
(WebCore::CSSParser::detectAtToken):
(WebCore::CSSParser::lex):
(WebCore::CSSParser::markSelectorListStart):
(WebCore::CSSParser::markSelectorListEnd):
(WebCore::CSSParser::markRuleBodyStart):
(WebCore::CSSParser::markRuleBodyEnd):
(WebCore::CSSParser::markPropertyStart):
(WebCore::CSSParser::markPropertyEnd):
- css/CSSParser.h:
(WebCore::CSSParser::token):
(CSSParser):
():
- css/tokenizer.flex: Removed.
- 11:22 PM Changeset in webkit [106216] by
-
- 3 edits1 add in trunk/Source/WebKit/chromium
[Chromium] REGRESSION(r87067): WebFrame::setFindEndstateFocusAndSelection()
doesn't set the selection for <input> and <textarea>
https://bugs.webkit.org/show_bug.cgi?id=77186
Reviewed by Hajime Morita.
When the find-in-page box is closed, WebFrame::stopFinding(false) is
called. It calls setFindEndstateFocusAndSelection(). Before r87067,
m_activeMatch was stored as the normal selection in <input> or
<textarea> by accident. r87067 stopped this accidental behavior.
However the behavior of pre-r87067 is useful and we should support it
for contentEditable elements too.
- src/WebFrameImpl.cpp:
(WebKit::WebFrameImpl::setFindEndstateFocusAndSelection):
- Add special handing for <input> and <textarea> to find a focusable parent.
- Sets the active match as the selection even if a focusable parent is found.
- tests/WebFrameTest.cpp:
(WebKit::TEST_F): Add tests for find() and stopFinding().
- tests/data/find.html: Added.
- 11:07 PM Changeset in webkit [106215] by
-
- 2 edits in trunk/LayoutTests
[Chromium] Unreviewed test expectation change.
inspector/extensions/extensions-{events,sidebar}.html time out for WIN DBG.
- platform/chromium/test_expectations.txt:
- 10:55 PM Changeset in webkit [106214] by
-
- 3 edits in trunk/Source/WebKit/chromium
[chromium/mac] Fix two-finger scrolling
https://bugs.webkit.org/show_bug.cgi?id=77316
http://trac.webkit.org/changeset/106021/trunk changed the phase enums
to match AppKit. This changes WebMouseWheelEvent accordingly.
No tests, because DRT doesn't support synthetic scroll events with
event phases.
Patch by Nico Weber <nicolasweber@gmx.de> on 2012-01-29
Reviewed by Anders Carlsson.
- public/WebInputEvent.h:
- src/mac/WebInputEventFactory.mm:
- 9:50 PM Changeset in webkit [106213] by
-
- 2 edits in trunk/LayoutTests
[Chromium] Unreviewed test expectation change.
svg/W3C-SVG-1.1/masking-intro-01-f.svg does not crash after r105978.
- platform/chromium/test_expectations.txt:
- 8:08 PM Changeset in webkit [106212] by
-
- 2 edits in trunk/LayoutTests
[Chromium] Unreviewed test expectation change.
fast/events/touch/touch-gesture-scroll.html is flaky on MAC and WIN.
- platform/chromium/test_expectations.txt:
- 7:59 PM Changeset in webkit [106211] by
-
- 3 edits in trunk/PerformanceTests
DOM/DOMDivWalk.html result is unreliable
https://bugs.webkit.org/show_bug.cgi?id=77313
Reviewed by Adam Birth.
Trigger garbage collection manually between test runs to avoid GC
to occur in the middle of runs.
- DOM/resources/dom-perf.js:
(BenchmarkSuite.prototype.RunSingle):
- resources/runner.js:
(gc.else.gcRec):
(gc):
(runLoop):
- 7:59 PM Changeset in webkit [106210] by
-
- 2 edits in trunk/LayoutTests
[Chromium] Unreviewed test expectation change.
Layout Test fast/events/touch/send-oncancel-event.html is flaky on MAC and WIN.
- platform/chromium/test_expectations.txt:
- 7:49 PM Changeset in webkit [106209] by
-
- 2 edits in trunk/Source/WebCore
Prepare WebCore.gyp for ffmpeg source transition.
https://bugs.webkit.org/show_bug.cgi?id=77254
We're migrating our ffmpeg repo from a set of patches living on top of
a tarball to an actual git fork of upstream. The paths have changed
slightly.
In order to not break the current build, we'll keep both paths around
until the transition is complete. Afterward the 'patched-ffmpeg' path
will be removed.
Patch by Dale Curtis <dalecurtis@chromium.org> on 2012-01-29
Reviewed by Tony Chang.
No new tests. GYP change, if it doesn't work, nothing will compile.
- WebCore.gyp/WebCore.gyp:
- 7:47 PM Changeset in webkit [106208] by
-
- 8 edits in trunk
Add a ShadowRoot constructor as 'WebKitShadowRootConstructor', enabled by SHADOW_DOM flag.
https://bugs.webkit.org/show_bug.cgi?id=76354
Reviewed by Hajime Morita.
We use vendor-prefixed name, 'WebKitShadowRoot', instead of 'ShadowRoot'
since this is a feature under development.
Source/WebCore:
- dom/ShadowRoot.cpp:
(WebCore::ShadowRoot::create):
(WebCore):
- dom/ShadowRoot.h:
(WebCore):
(ShadowRoot):
- dom/ShadowRoot.idl:
- page/DOMWindow.idl:
LayoutTests:
- fast/dom/shadow/shadow-root-js-api.html:
- 6:39 PM Changeset in webkit [106207] by
-
- 2 edits in trunk/Source/JavaScriptCore
REGRESSION (r105576-r105582): Web Inspector Crash in JSC::JSValue::toString(JSC::ExecState*) const
https://bugs.webkit.org/show_bug.cgi?id=77146
<rdar://problem/10770586>
Reviewed by Oliver Hunt.
The old JIT expects that the result of the last operation is in the lastResultRegister. The DFG JIT is
designed to correctly track the lastResultRegister by looking at SetLocal nodes. However, when the DFG
JIT inlines a code block, it forgets that the inlined code block's result would have been placed in the
lastResultRegister. Hence if we OSR exit on the first node following the end of an inlined code block
that had a return value, and that first node uses the return value, the old JIT will get massively
confused. This patch takes a surgical approach: instead of making the DFG smarter, it makes the old
JIT slightly dumber.
- jit/JITCall.cpp:
(JSC::JIT::emit_op_call_put_result):
- 6:23 PM Changeset in webkit [106206] by
-
- 2 edits in trunk/LayoutTests
[Chromium] Unreviewed test expectation change.
fast/js/navigator-language.html is flaky.
- platform/chromium/test_expectations.txt:
- 6:08 PM Changeset in webkit [106205] by
-
- 2 edits1 delete in trunk/LayoutTests
Unreviewed gardening. css3/flexbox/cross-axis-scrollbar.html is failing on Chromium Windows.
Removing an incorrect pixel expectation (it's a ref test) and marking it as failing on
Chromium Windows.
- platform/chromium-win/css3/flexbox/cross-axis-scrollbar-expected.png: Removed.
- platform/chromium/test_expectations.txt:
- 5:41 PM Changeset in webkit [106204] by
-
- 2 edits in trunk/Source/JavaScriptCore
Build fix for Mac non-x64 platforms.
- tools/CodeProfiling.cpp:
(JSC):
- 4:21 PM SettingUpDevelopmentEnvironmentForN9 edited by
- (diff)
- 3:59 PM Changeset in webkit [106203] by
-
- 2 edits in trunk/Source/WebCore
[chromium] Use decoding swizzle only on libjpeg-turbo 1.1.90+
https://bugs.webkit.org/show_bug.cgi?id=74286
Reviewed by Kenneth Russell.
No new tests. Covered by many existing tests: in particular
fast/images/*, fast/canvas/*,
tables/mozilla/bugs/bug29314.html
tables/mozilla/bugs/bug13169.html
tables/mozilla/bugs/bug10565.html
tables/mozilla/bugs/bug11026.html
fast/repaint/backgroundSizeRepaint.html
fast/repaint/block-layout-inline-children-replaced.html
fast/repaint/clipped-relative.html
fast/repaint/selected-replaced.html
tables/mozilla/bugs/bug12908-1.html
- platform/image-decoders/jpeg/JPEGImageDecoder.cpp:
(rgbOutputColorSpace): Swizzle decode iff libjpeg-turbo is r732 or above.
- 3:49 PM Changeset in webkit [106202] by
-
- 4 edits in trunk/Source/WebKit2
Unreviewed, rolling out r106022.
http://trac.webkit.org/changeset/106022
https://bugs.webkit.org/show_bug.cgi?id=77305
It broke Qt-WK2 API tests (Requested by Ossy_weekend on
#webkit).
Patch by Sheriff Bot <webkit.review.bot@gmail.com> on 2012-01-29
- UIProcess/API/qt/qquickwebview.cpp:
(QQuickWebViewPrivate::setUseTraditionalDesktopBehaviour):
(QQuickWebView::geometryChanged):
- UIProcess/API/qt/qquickwebview_p.h:
- UIProcess/API/qt/qwebviewportinfo.cpp:
(QWebViewportInfo::currentScale):
(QWebViewportInfo::devicePixelRatio):
(QWebViewportInfo::initialScale):
(QWebViewportInfo::minimumScale):
(QWebViewportInfo::maximumScale):
(QWebViewportInfo::isScalable):
(QWebViewportInfo::layoutSize):
- 3:47 PM SettingUpDevelopmentEnvironmentForN9 edited by
- (diff)
- 3:41 PM Changeset in webkit [106201] by
-
- 5 edits in trunk/Source/WebCore
Unreviewed, rolling out r105999.
http://trac.webkit.org/changeset/105999
https://bugs.webkit.org/show_bug.cgi?id=77304
Validating that this caused a performance regression in page
load tests. (Requested by leviw on #webkit).
Patch by Sheriff Bot <webkit.review.bot@gmail.com> on 2012-01-29
- dom/NamedNodeMap.cpp:
- dom/NamedNodeMap.h:
(NamedNodeMap):
- dom/StyledElement.cpp:
(WebCore::StyledElement::updateStyleAttribute):
(WebCore::StyledElement::createInlineStyleDecl):
(WebCore):
(WebCore::StyledElement::destroyInlineStyleDecl):
(WebCore::StyledElement::ensureInlineStyleDecl):
(WebCore::StyledElement::style):
(WebCore::StyledElement::addSubresourceAttributeURLs):
- dom/StyledElement.h:
(WebCore::StyledElement::inlineStyleDecl):
(StyledElement):
- 3:36 PM Changeset in webkit [106200] by
-
- 2 edits in trunk/LayoutTests
Reserved Word tests fail on Chromium after r106198
https://bugs.webkit.org/show_bug.cgi?id=77303
Unreviewed gardening. disabling new reserved word tests on Chromium after r106198.
- platform/chromium/test_expectations.txt:
- 3:23 PM Changeset in webkit [106199] by
-
- 19 edits in trunk
Unreviewed, rolling out r106109.
http://trac.webkit.org/changeset/106109
https://bugs.webkit.org/show_bug.cgi?id=77302
It made tests crash (Requested by Ossy_weekend on #webkit).
Patch by Sheriff Bot <webkit.review.bot@gmail.com> on 2012-01-29
Source/WebCore:
- platform/graphics/qt/TextureMapperQt.cpp:
(WebCore::TextureMapperQt::setGraphicsContext):
- platform/graphics/qt/TextureMapperQt.h:
(WebCore::TextureMapperQt::initialize):
Source/WebKit2:
- UIProcess/API/qt/qquickwebpage.cpp:
- UIProcess/API/qt/qquickwebpage_p_p.h:
(QQuickWebPagePrivate):
- UIProcess/API/qt/qquickwebview.cpp:
(QQuickWebViewPrivate::QQuickWebViewPrivate):
- UIProcess/API/qt/qquickwebview_p.h:
- UIProcess/API/qt/qquickwebview_p_p.h:
(QQuickWebViewPrivate):
- UIProcess/DrawingAreaProxy.h:
- UIProcess/DrawingAreaProxyImpl.cpp:
- UIProcess/DrawingAreaProxyImpl.h:
(DrawingAreaProxyImpl):
- UIProcess/LayerTreeHostProxy.h:
(LayerTreeHostProxy):
- UIProcess/qt/LayerTreeHostProxyQt.cpp:
(WebKit::LayerTreeHostProxy::paintToCurrentGLContext):
- UIProcess/qt/QtPageClient.cpp:
(QtPageClient::setViewNeedsDisplay):
Tools:
- WebKitTestRunner/qt/PlatformWebViewQt.cpp:
(WTR::WrapperWindow::WrapperWindow):
(WTR::PlatformWebView::PlatformWebView):
LayoutTests:
- platform/qt-wk2/Skipped:
- 12:23 AM Changeset in webkit [106198] by
-
- 9 edits in trunk
Reserve 'let'
https://bugs.webkit.org/show_bug.cgi?id=77293
Rubber stamped by Oliver Hunt.
'let' may become a keyword in ES6. We're going to try experimentally reserving it,
to see if this breaks the web.
Source/JavaScriptCore:
- parser/Keywords.table:
LayoutTests:
- fast/js/keywords-and-reserved_words-expected.txt:
- fast/js/reserved-words-as-property-expected.txt:
- fast/js/reserved-words-strict-expected.txt:
- fast/js/script-tests/keywords-and-reserved_words.js:
- fast/js/script-tests/reserved-words-as-property.js:
- fast/js/script-tests/reserved-words-strict.js: