Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(5)

Unified Diff: Source/WebCore/ChangeLog

Side-by-side diff isn't available for this file because of its large size.
Issue 7608014: Merge 92672 - Web Inspector: Resources panel does not show main resource cookies. (Closed) Base URL: http://svn.webkit.org/repository/webkit/branches/chromium/835/
Patch Set: Created 9 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
Download patch
Index: Source/WebCore/ChangeLog
===================================================================
--- Source/WebCore/ChangeLog (revision 92737)
+++ Source/WebCore/ChangeLog (working copy)
@@ -1,3 +1,13516 @@
+2011-08-09 Vsevolod Vlasov <vsevik@chromium.org>
+
+ Web Inspector: Resources panel does not show main resource cookies.
+ https://bugs.webkit.org/show_bug.cgi?id=65770
+
+ Reviewed by Pavel Feldman.
+
+ Test: http/tests/inspector/resource-main-cookies.php
+
+ * inspector/InspectorPageAgent.cpp:
+ (WebCore::allResourcesURLsForFrame):
+ (WebCore::InspectorPageAgent::getCookies):
+ (WebCore::InspectorPageAgent::deleteCookie):
+
+2011-08-09 Sheriff Bot <webkit.review.bot@gmail.com>
+
+ Unreviewed, rolling out r92670.
+ http://trac.webkit.org/changeset/92670
+ https://bugs.webkit.org/show_bug.cgi?id=65905
+
+ It broke 3 inspector tests (Requested by Ossy on #webkit).
+
+ * inspector/InjectedScript.cpp:
+ (WebCore::InjectedScript::getProperties):
+ * inspector/InjectedScript.h:
+ * inspector/InjectedScriptSource.js:
+ (.):
+ * inspector/Inspector.json:
+ * inspector/InspectorRuntimeAgent.cpp:
+ (WebCore::InspectorRuntimeAgent::getProperties):
+ * inspector/InspectorRuntimeAgent.h:
+ * inspector/front-end/ObjectPropertiesSection.js:
+ (WebInspector.ObjectPropertyTreeElement.prototype.ondblclick):
+ (WebInspector.ObjectPropertyTreeElement.prototype.update):
+ * inspector/front-end/RemoteObject.js:
+ (WebInspector.RemoteObject.prototype.getOwnProperties):
+ (WebInspector.RemoteObject.prototype.getAllProperties):
+ (WebInspector.RemoteObject.prototype._getProperties.remoteObjectBinder):
+ (WebInspector.RemoteObject.prototype._getProperties):
+ (WebInspector.RemoteObjectProperty):
+ * inspector/front-end/inspector.css:
+ (.section .properties .value.dimmed):
+
+2011-08-09 Pavel Feldman <pfeldman@google.com>
+
+ Web Inspector: there should be a way to tell what properties are non-enumerable when expanding objects.
+ https://bugs.webkit.org/show_bug.cgi?id=65518
+
+ Reviewed by Yury Semikhatsky.
+
+ * inspector/InjectedScript.cpp:
+ (WebCore::InjectedScript::getProperties):
+ * inspector/InjectedScript.h:
+ * inspector/InjectedScriptSource.js:
+ (.):
+ * inspector/Inspector.json:
+ * inspector/InspectorRuntimeAgent.cpp:
+ (WebCore::InspectorRuntimeAgent::getProperties):
+ * inspector/InspectorRuntimeAgent.h:
+ * inspector/front-end/ObjectPropertiesSection.js:
+ (WebInspector.ObjectPropertyTreeElement.prototype.ondblclick):
+ (WebInspector.ObjectPropertyTreeElement.prototype.update):
+ * inspector/front-end/RemoteObject.js:
+ (WebInspector.RemoteObject.prototype.getOwnProperties):
+ (WebInspector.RemoteObject.prototype.getAllProperties):
+ (WebInspector.RemoteObjectProperty):
+ * inspector/front-end/inspector.css:
+ (.section .properties .dimmed):
+
+2011-08-08 Steve Block <steveblock@google.com>
+
+ Removes V8's JavaInstanceJobject, JavaClassJobject and JavaFieldJobject.
+ https://bugs.webkit.org/show_bug.cgi?id=57859
+
+ Reviewed by Alexey Proskuryakov.
+
+ This effectively reverts the changes made in Bugs 55383, 57533 and
+ 55766. These classes were only used by Android. In the future, Android
+ will use the Chromium WebKit API to implement the Java bridge.
+
+ Towards this goal, this patch ...
+ - Simplfies the JavaInstance, JavaClass and JavaField interfaces for V8.
+ These interfaces will be implemented in Chromium's WebKit layer.
+ - Adds a JavaMethod interface for V8
+ - Removes the jvalue <-> JavaValue conversion functions for V8, which are
+ now superfluous
+ - Removes the calls to JavaInstance::begin()/end() for V8, which are now
+ superfluous
+
+ Will be covered by existing tests once the Chromium WebKit layer is complete.
+
+ * WebCore.gypi:
+ * bridge/jni/v8/JNIUtilityPrivate.cpp:
+ * bridge/jni/v8/JNIUtilityPrivate.h:
+ * bridge/jni/v8/JavaClassJobjectV8.cpp: Removed.
+ * bridge/jni/v8/JavaFieldJobjectV8.cpp: Removed.
+ * bridge/jni/v8/JavaFieldJobjectV8.h: Removed.
+ * bridge/jni/v8/JavaFieldV8.h:
+ * bridge/jni/v8/JavaInstanceJobjectV8.cpp: Removed.
+ * bridge/jni/v8/JavaInstanceJobjectV8.h: Removed.
+ * bridge/jni/v8/JavaInstanceV8.h:
+ * bridge/jni/v8/JavaMethodV8.h:
+ (JSC::Bindings::JavaMethod::~JavaMethod):
+ * bridge/jni/v8/JavaNPObjectV8.cpp:
+ (JSC::Bindings::JavaNPObjectHasMethod):
+ (JSC::Bindings::JavaNPObjectInvoke):
+ (JSC::Bindings::JavaNPObjectHasProperty):
+ (JSC::Bindings::JavaNPObjectGetProperty):
+
+2011-08-08 Chris Rogers <crogers@google.com>
+
+ Fix thread-safety of AudioNode deletion
+ https://bugs.webkit.org/show_bug.cgi?id=65888
+
+ Reviewed by Kenneth Russell
+
+ No new tests - JavaScript API is not affected.
+
+ * webaudio/AudioContext.cpp:
+ (WebCore::AudioContext::AudioContext):
+ (WebCore::AudioContext::constructCommon):
+ (WebCore::AudioContext::~AudioContext):
+ (WebCore::AudioContext::uninitialize):
+ (WebCore::AudioContext::handlePostRenderTasks):
+ (WebCore::AudioContext::scheduleNodeDeletion):
+ (WebCore::AudioContext::deleteMarkedNodesDispatch):
+ (WebCore::AudioContext::deleteMarkedNodes):
+ * webaudio/AudioContext.h:
+
+2011-08-08 Chris Marrin <cmarrin@apple.com>
+
+ Fix build breakage caused by http://trac.webkit.org/changeset/92651
+
+ Unreviewed.
+
+ * WebCore.exp.in:
+
+2011-08-08 Chris Marrin <cmarrin@apple.com>
+
+ Logic to compute visible display rect in GraphicsLayerCA::syncCompositingState
+ https://bugs.webkit.org/show_bug.cgi?id=65708
+
+ Add logic to syncCompositingState to compute the visible rect for each
+ layer. This can be used to synchronously render the visible tiles of a
+ TiledLayer and avoid the flashing that often occurs when tiles are rendered
+ asynchronously. A new synchronouslyDisplayTilesInRect is also added to do
+ the actual rendering, but the call is not currently being made.
+
+ Reviewed by Simon Fraser.
+
+ * page/FrameView.cpp:
+ (WebCore::FrameView::syncCompositingStateForThisFrame):
+ * platform/graphics/GraphicsLayer.h:
+ (WebCore::GraphicsLayer::syncCompositingState):
+ * platform/graphics/ca/GraphicsLayerCA.cpp:
+ (WebCore::GraphicsLayerCA::syncCompositingState):
+ (WebCore::GraphicsLayerCA::recursiveCommitChanges):
+ (WebCore::GraphicsLayerCA::platformCALayerPaintContents):
+ (WebCore::GraphicsLayerCA::updateSublayerList):
+ (WebCore::GraphicsLayerCA::swapFromOrToTiledLayer):
+ * platform/graphics/ca/GraphicsLayerCA.h:
+ * platform/graphics/ca/PlatformCALayer.h:
+ * platform/graphics/ca/mac/PlatformCALayerMac.mm:
+ (PlatformCALayer::synchronouslyDisplayTilesInRect):
+ * platform/graphics/transforms/TransformState.cpp:
+ (WebCore::TransformState::operator=):
+ * platform/graphics/transforms/TransformState.h:
+ (WebCore::TransformState::TransformState):
+ * rendering/RenderLayerCompositor.cpp:
+ (WebCore::RenderLayerCompositor::flushPendingLayerChanges):
+
+2011-08-08 Sheriff Bot <webkit.review.bot@gmail.com>
+
+ Unreviewed, rolling out r92619.
+ http://trac.webkit.org/changeset/92619
+ https://bugs.webkit.org/show_bug.cgi?id=65881
+
+ This patch breaks chromium canary (Requested by jianli on
+ #webkit).
+
+ * bindings/v8/V8Binding.h:
+ * bindings/v8/V8GCController.cpp:
+ (WebCore::globalHandleMap):
+ (WebCore::enumerateGlobalHandles):
+ (WebCore::V8GCController::registerGlobalHandle):
+ (WebCore::V8GCController::unregisterGlobalHandle):
+ * bindings/v8/V8HiddenPropertyName.cpp:
+ (WebCore::V8HiddenPropertyName::createString):
+ * bindings/v8/V8HiddenPropertyName.h:
+ * bindings/v8/V8LazyEventListener.cpp:
+ (WebCore::V8LazyEventListener::prepareListenerObject):
+ * bindings/v8/V8NPObject.cpp:
+ * bindings/v8/V8Proxy.cpp:
+ (WebCore::V8Proxy::checkNewLegal):
+ * bindings/v8/V8Utilities.h:
+ (WebCore::AllowAllocation::AllowAllocation):
+ (WebCore::AllowAllocation::~AllowAllocation):
+ (WebCore::SafeAllocation::newInstance):
+ * bindings/v8/WorkerContextExecutionProxy.cpp:
+ (WebCore::WorkerContextExecutionProxy::WorkerContextExecutionProxy):
+ (WebCore::WorkerContextExecutionProxy::initV8):
+ * bindings/v8/WorkerContextExecutionProxy.h:
+ * bindings/v8/WorkerScriptController.cpp:
+ (WebCore::WorkerScriptController::scheduleExecutionTermination):
+
+2011-08-08 Scott Byer <scottbyer@chromium.org>
+
+ Scroll animator changes to nail the framerate
+ https://bugs.webkit.org/show_bug.cgi?id=65645
+
+ Reviewed by James Robinson.
+
+ Partial test in ScrollAnimatorNoneTest::Enabled.
+
+ * platform/ScrollAnimatorNone.cpp:
+ (WebCore::ScrollAnimatorNone::PerAxisData::PerAxisData):
+ (WebCore::ScrollAnimatorNone::PerAxisData::updateDataFromParameters):
+ (WebCore::ScrollAnimatorNone::PerAxisData::animateScroll):
+ (WebCore::ScrollAnimatorNone::ScrollAnimatorNone):
+ (WebCore::ScrollAnimatorNone::~ScrollAnimatorNone):
+ (WebCore::ScrollAnimatorNone::scroll):
+ (WebCore::ScrollAnimatorNone::scrollToOffsetWithoutAnimation):
+ (WebCore::ScrollAnimatorNone::animationTimerFired):
+ (WebCore::ScrollAnimatorNone::stopAnimationTimerIfNeeded):
+ * platform/ScrollAnimatorNone.h:
+
+2011-08-08 Emil A Eklund <eae@chromium.org>
+
+ Switch legacy flexbox to to new layout types
+ https://bugs.webkit.org/show_bug.cgi?id=65340
+
+ Reviewed by Eric Seidel.
+
+ No new tests, no new functionality.
+
+ * rendering/RenderDeprecatedFlexibleBox.cpp:
+ (WebCore::RenderDeprecatedFlexibleBox::calcHorizontalPrefWidths):
+ (WebCore::RenderDeprecatedFlexibleBox::calcVerticalPrefWidths):
+ (WebCore::RenderDeprecatedFlexibleBox::computePreferredLogicalWidths):
+ (WebCore::RenderDeprecatedFlexibleBox::layoutBlock):
+ (WebCore::RenderDeprecatedFlexibleBox::layoutHorizontalBox):
+ (WebCore::RenderDeprecatedFlexibleBox::layoutVerticalBox):
+ (WebCore::RenderDeprecatedFlexibleBox::placeChild):
+ (WebCore::RenderDeprecatedFlexibleBox::allowedChildFlex):
+ * rendering/RenderDeprecatedFlexibleBox.h:
+
+2011-08-08 Emil A Eklund <eae@chromium.org>
+
+ Switch RenderStyle to to new layout types
+ https://bugs.webkit.org/show_bug.cgi?id=65208
+
+ Reviewed by Eric Seidel.
+
+ No new tests, no new functionality.
+
+ * rendering/RenderBox.cpp:
+ (WebCore::RenderBox::addShadowOverflow):
+ * rendering/style/RenderStyle.cpp:
+ (WebCore::RenderStyle::applyTransform):
+ (WebCore::calcRadiiFor):
+ (WebCore::RenderStyle::getRoundedBorderFor):
+ (WebCore::RenderStyle::getRoundedInnerBorderFor):
+ * rendering/style/RenderStyle.h:
+ * rendering/style/ShadowData.cpp:
+ (WebCore::ShadowData::adjustRectForShadow):
+
+2011-08-08 Cris Neckar <cdn@chromium.org>
+
+ Remove counter nodes from the tree and fix-up children when they are removed from the counter map.
+ https://bugs.webkit.org/show_bug.cgi?id=65346
+
+ Reviewed by Adam Barth.
+
+ Covered by existing CSS counter tests.
+
+ * rendering/CounterNode.cpp:
+ (WebCore::CounterNode::~CounterNode):
+
+2011-08-08 Tony Chang <tony@chromium.org>
+
+ implement basic horizontal flexing
+ https://bugs.webkit.org/show_bug.cgi?id=65045
+
+ Reviewed by David Hyatt.
+
+ Test: css3/flexbox/001.html
+
+ * css/CSSPrimitiveValueMappings.h:
+ (WebCore::CSSPrimitiveValue::CSSPrimitiveValue):
+ * rendering/RenderBox.cpp:
+ (WebCore::RenderBox::computeLogicalWidth):
+ * rendering/RenderFlexibleBox.cpp:
+ (WebCore::RenderFlexibleBox::FlexibleBoxIterator::FlexibleBoxIterator): flexitem iterator
+ (WebCore::RenderFlexibleBox::FlexibleBoxIterator::first):
+ (WebCore::RenderFlexibleBox::FlexibleBoxIterator::next):
+ (WebCore::RenderFlexibleBox::FlexibleBoxIterator::reset):
+ (WebCore::RenderFlexibleBox::layoutBlock):
+ (WebCore::preferredFlexItemContentWidth): Returns the intrinsic size of a flex item's content.
+ (WebCore::RenderFlexibleBox::layoutHorizontalBlock): Runs the flex algorithm
+ (WebCore::preferredSizeForMarginsAndPadding):
+ (WebCore::RenderFlexibleBox::computePreferredSize): Calculate the preferred size of the
+ flex items.
+ * rendering/RenderFlexibleBox.h:
+ * rendering/RenderObject.cpp:
+ (WebCore::RenderObject::createObject):
+ * rendering/style/RenderStyleConstants.h:
+
+2011-08-08 David Grogan <dgrogan@chromium.org>
+
+ Add detail to ASSERT message in IDBTransaction::enqueueEvent
+ https://bugs.webkit.org/show_bug.cgi?id=65735
+
+ I've hit this a few times but haven't been able to pin it down. The
+ extra detail could be helpful.
+
+ Reviewed by Nate Chapin.
+
+ No new tests, just an error message.
+
+ * storage/IDBTransaction.cpp:
+ (WebCore::IDBTransaction::enqueueEvent):
+
+2011-08-08 Sheriff Bot <webkit.review.bot@gmail.com>
+
+ Unreviewed, rolling out r92610.
+ http://trac.webkit.org/changeset/92610
+ https://bugs.webkit.org/show_bug.cgi?id=65868
+
+ Caused assertion failures when running fast/multicol tests
+ (Requested by andersca on #webkit).
+
+ * WebCore.xcodeproj/project.pbxproj:
+ * platform/PODIntervalTree.h:
+ (WebCore::PODIntervalTree::allOverlaps):
+ (WebCore::PODIntervalTree::searchForOverlapsFrom):
+ * platform/PODRedBlackTree.h:
+ (WebCore::PODRedBlackTree::add):
+ (WebCore::PODRedBlackTree::remove):
+ (WebCore::PODRedBlackTree::contains):
+ (WebCore::PODRedBlackTree::visitInorder):
+ (WebCore::PODRedBlackTree::size):
+ (WebCore::PODRedBlackTree::checkInvariants):
+ (WebCore::PODRedBlackTree::dump):
+ * rendering/RenderBlock.cpp:
+ (WebCore::RenderBlock::styleDidChange):
+ (WebCore::RenderBlock::addOverflowFromFloats):
+ (WebCore::RenderBlock::repaintOverhangingFloats):
+ (WebCore::RenderBlock::paintFloats):
+ (WebCore::RenderBlock::selectionGaps):
+ (WebCore::RenderBlock::insertFloatingObject):
+ (WebCore::RenderBlock::removeFloatingObject):
+ (WebCore::RenderBlock::removeFloatingObjectsBelow):
+ (WebCore::RenderBlock::positionNewFloats):
+ (WebCore::RenderBlock::logicalLeftOffsetForLine):
+ (WebCore::RenderBlock::logicalRightOffsetForLine):
+ (WebCore::RenderBlock::nextFloatLogicalBottomBelow):
+ (WebCore::RenderBlock::lowestFloatLogicalBottom):
+ (WebCore::RenderBlock::addPositionedFloats):
+ (WebCore::RenderBlock::clearFloats):
+ (WebCore::RenderBlock::addOverhangingFloats):
+ (WebCore::RenderBlock::hasOverhangingFloat):
+ (WebCore::RenderBlock::addIntrudingFloats):
+ (WebCore::RenderBlock::markSiblingsWithFloatsForLayout):
+ (WebCore::RenderBlock::hitTestFloats):
+ (WebCore::RenderBlock::adjustForBorderFit):
+ (WebCore::RenderBlock::FloatingObjects::clear):
+ * rendering/RenderBlock.h:
+ (WebCore::RenderBlock::FloatingObject::FloatingObject):
+ (WebCore::RenderBlock::FloatingObject::setX):
+ (WebCore::RenderBlock::FloatingObject::setY):
+ (WebCore::RenderBlock::FloatingObject::setWidth):
+ (WebCore::RenderBlock::FloatingObject::setHeight):
+ (WebCore::RenderBlock::FloatingObject::setFrameRect):
+ (WebCore::RenderBlock::FloatingObjects::FloatingObjects):
+ (WebCore::RenderBlock::FloatingObjects::set):
+ * rendering/RenderBlockLineLayout.cpp:
+ (WebCore::RenderBlock::layoutRunsAndFloatsInRange):
+ (WebCore::RenderBlock::linkToEndLineIfNeeded):
+ (WebCore::RenderBlock::matchedEndLine):
+ (WebCore::RenderBlock::positionNewFloatOnLine):
+
+2011-08-08 Emil A Eklund <eae@chromium.org>
+
+ Rename absoluteQuadsForRange and InlineTextBox::selectionRect to local*
+ https://bugs.webkit.org/show_bug.cgi?id=65722
+
+ Reviewed by Simon Fraser.
+
+ No new tests, no new functionality.
+
+ * rendering/InlineTextBox.cpp:
+ (WebCore::InlineTextBox::localSelectionRect):
+ Rename InlineTextBox::selectionRect to localSelectionRect to reflect that
+ it, unlike RenderObject::selectionRect returns a rect in the local
+ coordinate space.
+
+ * rendering/InlineTextBox.h:
+ * rendering/RenderText.cpp:
+ (WebCore::localQuadForTextBox):
+ (WebCore::RenderText::absoluteRectsForRange):
+ (WebCore::RenderText::absoluteQuadsForRange):
+ (WebCore::RenderText::selectionRectForRepaint):
+ Rename absoluteQuadForTextBox to localQuadForTextBox to reflect that it
+ returns a quad in the local coordinate space.
+
+ * rendering/svg/RenderSVGInlineText.cpp:
+ (WebCore::RenderSVGInlineText::localCaretRect):
+ * rendering/svg/SVGInlineTextBox.cpp:
+ (WebCore::SVGInlineTextBox::localSelectionRect):
+ * rendering/svg/SVGInlineTextBox.h:
+
+2011-08-08 Jochen Eisinger <jochen@chromium.org>
+
+ Use a raw pointer to the security origin in the ctor of FrameLoadRequest
+ https://bugs.webkit.org/show_bug.cgi?id=65852
+
+ Reviewed by Alexey Proskuryakov.
+
+ * loader/FrameLoadRequest.h:
+ (WebCore::FrameLoadRequest::FrameLoadRequest):
+ * loader/FrameLoader.cpp:
+ (WebCore::FrameLoader::changeLocation):
+ * loader/FrameLoader.h:
+ * loader/NavigationScheduler.cpp:
+ (WebCore::ScheduledURLNavigation::ScheduledURLNavigation):
+ (WebCore::ScheduledURLNavigation::fire):
+ (WebCore::ScheduledRedirect::ScheduledRedirect):
+ (WebCore::ScheduledLocationChange::ScheduledLocationChange):
+ (WebCore::ScheduledRefresh::ScheduledRefresh):
+ (WebCore::NavigationScheduler::scheduleLocationChange):
+ * loader/NavigationScheduler.h:
+
+2011-08-08 Ryosuke Niwa <rniwa@webkit.org>
+
+ Remove redundant inline styles from the pasted contents more aggressively
+ https://bugs.webkit.org/show_bug.cgi?id=65833
+
+ Reviewed by Tony Chang.
+
+ Make removeRedundantStylesAndKeepStyleSpanInline more aggressively remove redundant styles by realizing that
+ when an editing inheritable property in an inline style declaration of an element can be safely removed
+ if it is present either in style rules or in its ancestor and not overridden by style rules or default style
+ of the element.
+
+ Test: editing/pasteboard/paste-with-redundant-style.html
+
+ * css/CSSStyleSelector.cpp:
+ (WebCore::CSSStyleSelector::pseudoStyleRulesForElement): Do not match author style sheets if AuthorCSSRules
+ is not included in the rules to include. This is used in EditingStyle::removePropertiesInElementDefaultStyle.
+ * editing/ApplyStyleCommand.cpp:
+ (WebCore::isStyleSpanOrSpanWithOnlyStyleAttribute): Added; returns true if the element is a style span or
+ span possibly with a style attribute.
+ * editing/ApplyStyleCommand.h:
+ * editing/EditingStyle.cpp:
+ (WebCore::styleFromMatchedRulesForElement): Takes rulesToInclude.
+ (WebCore::EditingStyle::mergeStyleFromRules): Calls styleFromMatchedRulesForElement with AuthorCSSRules
+ | CrossOriginCSSRules to keep the original behavior.
+ (WebCore::EditingStyle::removeStyleFromRulesAndContext): Renamed from removeStyleFromRules; removes styles that
+ are present in context and not overridden by matched rules.
+ (WebCore::EditingStyle::removePropertiesInElementDefaultStyle): Added.
+ * editing/EditingStyle.h:
+ * editing/ReplaceSelectionCommand.cpp:
+ (WebCore::ReplaceSelectionCommand::removeRedundantStylesAndKeepStyleSpanInline): See the description above.
+
+2011-08-08 Dmitry Lomov <dslomov@google.com>
+
+ https://bugs.webkit.org/show_bug.cgi?id=65778
+ [WebWorkers][chromium] Make statics thread-safe and make sure V8 API accesses correct isolates.
+
+ Reviewed by David Levin.
+
+ Covered by existing tests.
+
+ * bindings/v8/V8Binding.h:
+ (WebCore::V8BindingPerIsolateData::lazyEventListenerToStringTemplate):
+ (WebCore::V8BindingPerIsolateData::hiddenPropertyName):
+ (WebCore::V8BindingPerIsolateData::globalHandleMap):
+ (WebCore::AllowAllocation::AllowAllocation): Moving to V8Binding.h from V8Utilities.h to resolve header dependency.
+ (WebCore::AllowAllocation::~AllowAllocation):
+ (WebCore::AllowAllocation::current):
+ (WebCore::SafeAllocation::newInstance):
+ * bindings/v8/V8GCController.cpp:
+ (WebCore::currentGlobalHandleMap):
+ (WebCore::enumerateGlobalHandles):
+ (WebCore::V8GCController::registerGlobalHandle):
+ (WebCore::V8GCController::unregisterGlobalHandle):
+ * bindings/v8/V8HiddenPropertyName.cpp:
+ (WebCore::V8HiddenPropertyName::createString):
+ * bindings/v8/V8HiddenPropertyName.h:
+ * bindings/v8/V8LazyEventListener.cpp:
+ (WebCore::V8LazyEventListener::prepareListenerObject):
+ * bindings/v8/V8NPObject.cpp:
+ * bindings/v8/V8Proxy.cpp:
+ (WebCore::V8Proxy::checkNewLegal):
+ * bindings/v8/V8Utilities.h:
+ * bindings/v8/WorkerContextExecutionProxy.cpp:
+ (WebCore::WorkerContextExecutionProxy::WorkerContextExecutionProxy):
+ (WebCore::WorkerContextExecutionProxy::initIsolate):
+ * bindings/v8/WorkerContextExecutionProxy.h:
+ * bindings/v8/WorkerScriptController.cpp:
+ (WebCore::WorkerScriptController::scheduleExecutionTermination):
+
+2011-08-08 Sheriff Bot <webkit.review.bot@gmail.com>
+
+ Unreviewed, rolling out r92607.
+ http://trac.webkit.org/changeset/92607
+ https://bugs.webkit.org/show_bug.cgi?id=65865
+
+ This patch breaks Chromium Mac Canary (Requested by jianli on
+ #webkit).
+
+ * platform/PlatformWheelEvent.h:
+ (WebCore::PlatformWheelEvent::PlatformWheelEvent):
+ * platform/chromium/ScrollAnimatorChromiumMac.mm:
+ (WebCore::ScrollAnimatorChromiumMac::handleGestureEvent):
+
+2011-08-08 Andrew Wason <rectalogic@rectalogic.com>
+
+ [Qt] Implement WebGL antialiasing (part 3)
+ https://bugs.webkit.org/show_bug.cgi?id=64879
+
+ Reviewed by Noam Rosenthal.
+
+ Existing WebGL layout tests cover this.
+
+ Implement WebGL antialiasing for Qt desktop.
+ Existing code in GraphicsContext3DOpenGL.cpp manages the
+ multisample FBO. GraphicsContext3DQt.cpp creates it and blits
+ it to the regular FBO when painting.
+ Requires adoption of Extensions3DOpenGL to be fully functional.
+
+ * platform/graphics/qt/GraphicsContext3DQt.cpp:
+ (WebCore::GraphicsContext3DInternal::GraphicsContext3DInternal):
+ (WebCore::GraphicsContext3DInternal::paintToTextureMapper):
+ (WebCore::GraphicsContext3DInternal::paint):
+ (WebCore::GraphicsContext3DInternal::multisampleResolve):
+ (WebCore::GraphicsContext3D::GraphicsContext3D):
+ (WebCore::GraphicsContext3D::~GraphicsContext3D):
+
+2011-08-08 Anders Carlsson <andersca@apple.com>
+
+ Fix build.
+
+ * platform/PODRedBlackTree.h:
+ (WebCore::PODRedBlackTree::Counter::visit):
+
+2011-08-08 Alexandru Chiculita <achicu@adobe.com>
+
+ Optimize floating elements lookup
+ https://bugs.webkit.org/show_bug.cgi?id=65668
+
+ Added an interval tree in the FloatingObjects structure. Also added new mechanisms to make
+ sure the tree is updated correctly when a float is repositioned.
+
+ Changed the PODIntervalTree to support giving a search adapter that can be implemented by the
+ client. I'm not adding a different bug for that because PODIntervalTree is not used anywhere else
+ and would be hard to test that the change is not breaking anything.
+
+ Reviewed by David Hyatt.
+
+ No new tests, just a refactor on the floating objects data structure.
+
+ * WebCore.xcodeproj/project.pbxproj:
+ * platform/PODIntervalTree.h:
+ (WebCore::PODIntervalSearchAdapter::PODIntervalSearchAdapter):
+ (WebCore::PODIntervalSearchAdapter::lowValue):
+ (WebCore::PODIntervalSearchAdapter::highValue):
+ (WebCore::PODIntervalSearchAdapter::collectIfNeeded):
+ (WebCore::PODIntervalTree::PODIntervalTree):
+ (WebCore::PODIntervalTree::allOverlaps):
+ (WebCore::PODIntervalTree::allOverlapsWithAdapter):
+ (WebCore::PODIntervalTree::searchForOverlapsFrom):
+ * platform/PODRedBlackTree.h:
+ (WebCore::PODRedBlackTree::PODRedBlackTree):
+ (WebCore::PODRedBlackTree::clear):
+ (WebCore::PODRedBlackTree::isInitialized):
+ (WebCore::PODRedBlackTree::initIfNeeded):
+ (WebCore::PODRedBlackTree::add):
+ (WebCore::PODRedBlackTree::remove):
+ (WebCore::PODRedBlackTree::contains):
+ (WebCore::PODRedBlackTree::visitInorder):
+ (WebCore::PODRedBlackTree::size):
+ (WebCore::PODRedBlackTree::checkInvariants):
+ (WebCore::PODRedBlackTree::dump):
+ * rendering/RenderBlock.cpp:
+ (WebCore::RenderBlock::styleDidChange):
+ (WebCore::RenderBlock::addOverflowFromFloats):
+ (WebCore::RenderBlock::repaintOverhangingFloats):
+ (WebCore::RenderBlock::paintFloats):
+ (WebCore::RenderBlock::selectionGaps):
+ (WebCore::RenderBlock::insertFloatingObject):
+ (WebCore::RenderBlock::removeFloatingObject):
+ (WebCore::RenderBlock::removeFloatingObjectsBelow):
+ (WebCore::RenderBlock::positionNewFloats):
+ (WebCore::::collectIfNeeded):
+ (WebCore::RenderBlock::logicalLeftOffsetForLine):
+ (WebCore::RenderBlock::logicalRightOffsetForLine):
+ (WebCore::RenderBlock::nextFloatLogicalBottomBelow):
+ (WebCore::RenderBlock::lowestFloatLogicalBottom):
+ (WebCore::RenderBlock::addPositionedFloats):
+ (WebCore::RenderBlock::clearFloats):
+ (WebCore::RenderBlock::addOverhangingFloats):
+ (WebCore::RenderBlock::hasOverhangingFloat):
+ (WebCore::RenderBlock::addIntrudingFloats):
+ (WebCore::RenderBlock::markSiblingsWithFloatsForLayout):
+ (WebCore::RenderBlock::hitTestFloats):
+ (WebCore::RenderBlock::adjustForBorderFit):
+ (WebCore::RenderBlock::FloatingObjects::clear):
+ (WebCore::RenderBlock::FloatingObjects::intervalForFloatingObject):
+ (WebCore::RenderBlock::FloatingObjects::addPlacedObject):
+ (WebCore::RenderBlock::FloatingObjects::removePlacedObject):
+ (WebCore::RenderBlock::FloatingObjects::add):
+ (WebCore::RenderBlock::FloatingObjects::remove):
+ (WebCore::RenderBlock::FloatingObjects::computePlacedFloatsTree):
+ (WebCore::::string):
+ * rendering/RenderBlock.h:
+ (WebCore::RenderBlock::FloatingObject::FloatingObject):
+ (WebCore::RenderBlock::FloatingObject::setX):
+ (WebCore::RenderBlock::FloatingObject::setY):
+ (WebCore::RenderBlock::FloatingObject::setWidth):
+ (WebCore::RenderBlock::FloatingObject::setHeight):
+ (WebCore::RenderBlock::FloatingObject::setFrameRect):
+ (WebCore::RenderBlock::FloatingObject::isInPlacedTree):
+ (WebCore::RenderBlock::FloatingObject::setIsInPlacedTree):
+ (WebCore::RenderBlock::FloatIntervalSearchAdapter::FloatIntervalSearchAdapter):
+ (WebCore::RenderBlock::FloatIntervalSearchAdapter::lowValue):
+ (WebCore::RenderBlock::FloatIntervalSearchAdapter::highValue):
+ (WebCore::RenderBlock::FloatingObjects::FloatingObjects):
+ (WebCore::RenderBlock::FloatingObjects::setHorizontalWritingMode):
+ (WebCore::RenderBlock::FloatingObjects::set):
+ (WebCore::RenderBlock::FloatingObjects::placedFloatsTree):
+ (WebCore::RenderBlock::FloatingObjects::computePlacedFloatsTreeIfNeeded):
+ * rendering/RenderBlockLineLayout.cpp:
+ (WebCore::RenderBlock::layoutRunsAndFloatsInRange):
+ (WebCore::RenderBlock::linkToEndLineIfNeeded):
+ (WebCore::RenderBlock::matchedEndLine):
+ (WebCore::RenderBlock::positionNewFloatOnLine):
+
+2011-08-08 Alexei Svitkine <asvitkine@chromium.org>
+
+ [Chromium] Enable rubber banding when scrolling.
+
+ https://bugs.webkit.org/show_bug.cgi?id=65707
+
+ Reviewed by Dimitri Glazkov.
+
+ No new tests since this is just syncing changes to chromium platform.
+
+ * platform/PlatformWheelEvent.h:
+ (WebCore::PlatformWheelEvent::PlatformWheelEvent):
+ * platform/chromium/ScrollAnimatorChromiumMac.mm:
+ (WebCore::ScrollAnimatorChromiumMac::handleGestureEvent):
+
+2011-08-08 Dan Bernstein <mitz@apple.com>
+
+ Build fix.
+
+ * html/NumberInputType.cpp:
+ (WebCore::NumberInputType::sizeShouldIncludeDecoration): Fixed an obvious typo.
+
+2011-08-08 Andras Becsi <abecsi@webkit.org>
+
+ [Qt] Clean up project file after r92492.
+
+ Unreviewed gardening.
+
+ No new tests needed.
+
+ * WebCore.pro: remove nonexistent PluginHalterClient.h, PluginHalter.h, HaltablePlugin.h.
+
+2011-08-08 Pavel Podivilov <podivilov@chromium.org>
+
+ Web Inspector: refactor links creation.
+ https://bugs.webkit.org/show_bug.cgi?id=65612
+
+ Reviewed by Pavel Feldman.
+
+ No new tests - it is refactoring, modified code covered by script-formatter.html test.
+
+ * inspector/front-end/ConsoleView.js:
+ (WebInspector.ConsoleMessage.prototype._formatMessage):
+ (WebInspector.ConsoleMessage.prototype._linkifyLocation):
+ (WebInspector.ConsoleMessage.prototype._linkifyCallFrame):
+ (WebInspector.ConsoleMessage.prototype._populateStackTraceTreeElement):
+ * inspector/front-end/DebuggerPresentationModel.js:
+ (WebInspector.DebuggerPresentationModel):
+ (WebInspector.DebuggerPresentationModel.prototype.addSourceMappingListener):
+ (WebInspector.DebuggerPresentationModel.prototype.removeSourceMappingListener):
+ (WebInspector.DebuggerPresentationModel.prototype.linkifyLocation.updateAnchor.didGetLocation):
+ (WebInspector.DebuggerPresentationModel.prototype.linkifyLocation.updateAnchor):
+ (WebInspector.DebuggerPresentationModel.prototype.linkifyLocation):
+ (WebInspector.DebuggerPresentationModel.prototype.setFormatSourceFiles):
+ (WebInspector.DebuggerPresentationModel.prototype._sourceFileForScript):
+ (WebInspector.DebuggerPresentationModel.prototype._debuggerReset):
+ * inspector/front-end/EventListenersSidebarPane.js:
+ * inspector/front-end/NetworkPanel.js:
+ (WebInspector.NetworkDataGridNode.prototype._refreshInitiatorCell):
+ * inspector/front-end/Panel.js:
+ * inspector/front-end/ProfileDataGridTree.js:
+ (WebInspector.ProfileDataGridNode.prototype.createCell):
+ * inspector/front-end/ScriptsPanel.js:
+ (WebInspector.ScriptsPanel.prototype._folderAndDisplayNameForScriptURL):
+ * inspector/front-end/StylesSidebarPane.js:
+ (WebInspector.StylePropertiesSection.linkifyUncopyable):
+ (WebInspector.StylePropertiesSection):
+ * inspector/front-end/TimelinePanel.js:
+ (WebInspector.TimelinePanel.FormattedRecord.prototype._getRecordDetails):
+ (WebInspector.TimelinePanel.FormattedRecord.prototype._linkifyLocation):
+ (WebInspector.TimelinePanel.FormattedRecord.prototype._linkifyCallFrame):
+ (WebInspector.TimelinePanel.PopupContentHelper.prototype._appendLinkRow):
+ (WebInspector.TimelinePanel.PopupContentHelper.prototype._appendStackTrace):
+ * inspector/front-end/inspector.js:
+ (WebInspector._showSettingsScreen):
+ (WebInspector.formatLinkText):
+ (WebInspector.linkifyResourceAsNode):
+
+2011-08-08 Andrew Wason <rectalogic@rectalogic.com>
+
+ [Qt] Implement WebGL antialiasing (part 3)
+ https://bugs.webkit.org/show_bug.cgi?id=64879
+
+ Reviewed by Noam Rosenthal.
+
+ Existing WebGL layout tests.
+
+ Adopt Extensions3DOpenGL for Qt desktop as a prerequisite
+ for implementing WebGL antialiasing.
+ Extensions3DQt is still used for ES2.
+
+ * WebCore.pri:
+ * WebCore.pro:
+ * platform/graphics/GraphicsContext3D.h:
+ * platform/graphics/opengl/Extensions3DOpenGL.cpp:
+ (WebCore::Extensions3DOpenGL::createVertexArrayOES):
+ (WebCore::Extensions3DOpenGL::deleteVertexArrayOES):
+ (WebCore::Extensions3DOpenGL::isVertexArrayOES):
+ (WebCore::Extensions3DOpenGL::bindVertexArrayOES):
+ * platform/graphics/opengl/GraphicsContext3DOpenGL.cpp:
+ (WebCore::GraphicsContext3D::getExtensions):
+ * platform/graphics/qt/GraphicsContext3DQt.cpp:
+ (WebCore::GraphicsContext3D::layerComposited):
+
+2011-08-08 Benjamin Poulain <benjamin@webkit.org>
+
+ Build fix after r92589, defaultSize is not used.
+
+ Reviewed by Andreas Kling.
+
+ * html/InputType.cpp:
+ (WebCore::InputType::sizeShouldIncludeDecoration):
+
+2011-08-08 Keishi Hattori <keishi@webkit.org>
+
+ Implement <input type=color> UI behavior WebCore part
+ https://bugs.webkit.org/show_bug.cgi?id=62619
+
+ Reviewed by Kent Tamura.
+
+ * WebCore.exp.in: Added __ZN7WebCore12ColorChooser7chooserEv and
+ __ZNK7WebCore12ColorChooser13colorSelectedERKNS_5ColorE
+ * WebCore.xcodeproj/project.pbxproj: Added ColorChooser.{h,cpp}
+ * html/ColorInputType.cpp:
+ (WebCore::ColorInputType::~ColorInputType): Added. Close color chooser. ex. when type attribute changes.
+ (WebCore::ColorInputType::valueAsColor): Added.
+ (WebCore::ColorInputType::setValueAsColor): Added.
+ (WebCore::ColorInputType::valueChanged): Update selected color in color chooser too.
+ (WebCore::ColorInputType::handleClickEvent): Opens color chooser.
+ (WebCore::ColorInputType::handleDOMActivateEvent): Called when element.click(). Open color chooser only
+ when it was initiated by a user interaction.
+ (WebCore::ColorInputType::detach): Close color chooser. Called when input element or its ancestors have "display:none"
+ or is removed from DOM.
+ (WebCore::ColorInputType::colorSelected): Callback from color chooser.
+ (WebCore::ColorInputType::isColorInputType): Returns true.
+ (WebCore::ColorInputType::closeColorChooserIfCurrentClient): Close color chooser if this input type is the current client of ColorChooser.
+ * html/ColorInputType.h:
+ * html/FileInputType.cpp:
+ * html/FileInputType.h:
+ * html/HTMLInputElement.cpp:
+ (WebCore::HTMLInputElement::detach): Calls InputType::detach
+ * html/InputType.cpp:
+ (WebCore::InputType::chrome): Added. Used in FileInputType and ColorInputType.
+ (WebCore::InputType::detach): ColorInputType overrides this.
+ * html/InputType.h:
+ * loader/EmptyClients.h:
+ (WebCore::EmptyChromeClient::openColorChooser):
+ (WebCore::EmptyChromeClient::closeColorChooser):
+ (WebCore::EmptyChromeClient::setSelectedColorInColorChooser):
+ * loader/FrameLoader.cpp:
+ (WebCore::FrameLoader::transitionToCommitted): Close color chooser when navigating away from the page. We do this here
+ because ColorInputType::detach() is called seconds after the page transition so it is too slow.
+ * page/Chrome.cpp:
+ (WebCore::Chrome::openColorChooser): Added.
+ (WebCore::Chrome::closeColorChooser): Added.
+ (WebCore::Chrome::setSelectedColorInColorChooser): Added.
+ * page/Chrome.h:
+ * page/ChromeClient.h:
+ * platform/ColorChooser.cpp: Added.
+ (WebCore::ColorChooserClient::~ColorChooserClient): Disconnects itself from the ColorChooser.
+ (WebCore::ColorChooser::chooser): Get shared instance of ColorChooser.
+ (WebCore::ColorChooser::connectClient): Connects a ColorChooserClient that receives the colorSelected callbacks.
+ (WebCore::ColorChooser::disconnectClient): Disconnects the connectClient.
+ (WebCore::ColorChooser::colorSelected): Called from color chooser listener.
+ * platform/ColorChooser.h: Added.
+ (WebCore::ColorChooserClient::isColorInputType): Added.
+ (WebCore::ColorChooser::client): Added. Returns the current connected client.
+ (WebCore::ColorChooser::ColorChooser): Added.
+
+2011-08-08 Shinya Kawanaka <shinyak@google.com>
+
+ The input[type=number] element should be as wide as necessary to show the widest possible value.
+ https://bugs.webkit.org/show_bug.cgi?id=60673
+
+ Reviewed by Kent Tamura.
+
+ The size of input[type=number] is calculated from min/max/step attributes to show the widest possible value.
+ If min or max attribute is absent, the default size is used.
+ Also, if its css width is not auto, the width is used with priority.
+
+ If min/max/attribute is set dynamically, the size of the input[type=number] will be recalculated.
+
+ Test: fast/forms/input-number-size.html
+
+ * html/HTMLInputElement.cpp:
+ (WebCore::HTMLInputElement::parseMappedAttribute):
+ Added stepAttributeChanged handler.
+ (WebCore::HTMLInputElement::sizeShouldIncludeDecoration):
+ Returns true if a renderer should include decoration (e.g. inner spinbox).
+ Also returns the preferred size of the input.
+ * html/HTMLInputElement.h:
+ * html/InputType.cpp:
+ (WebCore::InputType::sizeShouldIncludeDecoration):
+ Same as WebCore::HTMLInputElement::sizeShouldIncludeDecoration.
+ (WebCore::InputType::stepAttributeChanged):
+ Will be called When step attribute is changed.
+ Sets a flag to recalculate layout.
+ * html/InputType.h:
+ * html/NumberInputType.cpp:
+ (WebCore::lengthBeforeDecimalPoint):
+ Calculates the width before the decimal point.
+ (WebCore::NumberInputType::sizeShouldIncludeDecoration):
+ Same as WebCore::HTMLInputElement::sizeShouldIncludeDecoration.
+ However, this will calculate the preferred size from min/max/step attribute.
+ (WebCore::NumberInputType::minOrMaxAttributeChanged):
+ Sets a flag to recalculate layout.
+ (WebCore::NumberInputType::stepAttributeChanged): ditto.
+ * html/NumberInputType.h:
+ * rendering/RenderTextControlSingleLine.cpp:
+ (WebCore::RenderTextControlSingleLine::preferredContentWidth):
+ Uses preferredSize instead of size.
+ Also, adds innerSpinButtonElement size to width if sizeShouldIncludeDecoration returns true.
+
+2011-08-08 Kenichi Ishibashi <bashi@chromium.org>
+
+ Should not use C-style cast in CSSParser.cpp
+ https://bugs.webkit.org/show_bug.cgi?id=65807
+
+ Remove C-style cast.
+
+ Reviewed by Kent Tamura.
+
+ No new tests because no behavior change.
+
+ * css/CSSParser.cpp:
+ (WebCore::CSSParser::parseAnimationTimingFunction): Replaced a cast with clampToInteger().
+ (WebCore::CSSParser::parseFontFeatureTag): Ditto.
+ (WebCore::CSSParser::createKeyframeRule): Replaced a cast with C++-style cast.
+
+2011-08-05 Pavel Feldman <pfeldman@chromium.org>
+
+ Web Inspector: implement dock/undock in WebKit2 without getting into WebCore.
+ https://bugs.webkit.org/show_bug.cgi?id=65763
+
+ InspectorFrontendClient.h can't have synchronous canAttachWindow() returning value
+ since in the multiprocess environment, you can't immediately get back to it from the host.
+
+ r92384 introduced an unhealthy WebKit -> WebCore -> WebKit canAttach query where embedder
+ was asking itself a question. This loop was fixed.
+
+ Reviewed by Yury Semikhatsky.
+
+ * WebCore.exp.in:
+ * inspector/InspectorController.cpp:
+ * inspector/InspectorController.h:
+ * inspector/InspectorFrontendClient.h:
+ * inspector/InspectorFrontendClientLocal.h:
+ * inspector/front-end/inspector.js:
+ (windowLoaded):
+
+2011-07-28 Kent Tamura <tkent@chromium.org>
+
+ [Chromium] Make form validation bubble fit with Chrome UI style
+ https://bugs.webkit.org/show_bug.cgi?id=65359
+
+ Reviewed by Hajime Morita.
+
+ Change the appearance of validation message bubble for
+ Chromium. Basically it doesn't change for other ports.
+
+ - Introduce a new element to put an icon, and enclose it and the
+ message text by a flexible box.
+ - Repesent the message text by two elements. One for the heading,
+ another for the remaining.
+
+ - Introduce themeChromium.css
+
+ * WebCore.gyp/WebCore.gyp: Add themeChromium.css.
+ * css/html.css:
+ (::-webkit-validation-bubble-message): Make this a flexible box container.
+ (::-webkit-validation-bubble-text-block): Takes flexibility.
+ (::-webkit-validation-bubble-heading):
+ Make the content bold because we removed <b></b>.
+ * css/themeChromium.css: Added.
+ (::-webkit-validation-bubble):
+ (::-webkit-validation-bubble-message):
+ (::-webkit-validation-bubble-arrow):
+ (::-webkit-validation-bubble-arrow-clipper):
+ (::-webkit-validation-bubble-icon):
+ (::-webkit-validation-bubble-heading):
+ (::-webkit-validation-bubble-body):
+ * html/ValidationMessage.cpp:
+ (WebCore::ValidationMessage::setMessageDOMAndStartTimer):
+ Sets the text into m_messageHeading and m_messageBody, instead of m_bubbleMessage.
+ Use ASSERT_NO_EXCEPTION.
+ (WebCore::ValidationMessage::buildBubbleTree):
+ Build the new structure, and use ASSERT_NO_EXCEPTION.
+ (WebCore::ValidationMessage::deleteBubbleTree):
+ Clear m_messageHeading and m_messageBody.
+ * html/ValidationMessage.h:
+ * rendering/RenderThemeChromiumMac.h: Add extraDefaultStyleSheet().
+ * rendering/RenderThemeChromiumMac.mm:
+ (WebCore::RenderThemeChromiumMac::extraDefaultStyleSheet): Append themeChromium.css.
+ * rendering/RenderThemeChromiumSkia.cpp:
+ (WebCore::RenderThemeChromiumSkia::extraDefaultStyleSheet): Append themeChromium.css.
+
+2011-08-07 Keishi Hattori <keishi@webkit.org>
+
+ Sort WebCore.xcodeproj
+ Accomplished using sort-Xcode-project-file.
+
+ * WebCore.xcodeproj/project.pbxproj:
+
+2011-08-07 James Kozianski <koz@chromium.org>
+
+ Make the fullscreen JS API enabled at runtime.
+ https://bugs.webkit.org/show_bug.cgi?id=65501
+
+ Reviewed by Adam Barth.
+
+ * bindings/generic/RuntimeEnabledFeatures.cpp:
+ * bindings/generic/RuntimeEnabledFeatures.h:
+ (WebCore::RuntimeEnabledFeatures::webkitFullScreenAPIEnabled):
+ (WebCore::RuntimeEnabledFeatures::setWebkitFullScreenAPIEnabled):
+ (WebCore::RuntimeEnabledFeatures::webkitRequestFullScreenEnabled):
+ (WebCore::RuntimeEnabledFeatures::webkitIsFullScreenEnabled):
+ (WebCore::RuntimeEnabledFeatures::webkitFullScreenKeyboardInputAllowedEnabled):
+ (WebCore::RuntimeEnabledFeatures::webkitCurrentFullScreenElementEnabled):
+ (WebCore::RuntimeEnabledFeatures::webkitCancelFullScreenEnabled):
+ * dom/Document.idl:
+ * dom/Element.idl:
+
+2011-08-07 Martin Robinson <mrobinson@igalia.com>
+
+ Distribution fix for GTK+.
+
+ * GNUmakefile.am: Add some files to the distribution list.
+ * GNUmakefile.list.am: Strip removed files from the source list.
+
+2011-08-07 Jochen Eisinger <jochen@chromium.org>
+
+ Move FrameLoadRequest to loader/
+ https://bugs.webkit.org/show_bug.cgi?id=65817
+
+ Reviewed by Alexey Proskuryakov.
+
+ * GNUmakefile.list.am:
+ * WebCore.gypi:
+ * WebCore.vcproj/WebCore.vcproj:
+ * WebCore.xcodeproj/project.pbxproj:
+ * loader/FrameLoadRequest.h:
+ (WebCore::FrameLoadRequest::FrameLoadRequest):
+ (WebCore::FrameLoadRequest::isEmpty):
+ (WebCore::FrameLoadRequest::requester):
+ (WebCore::FrameLoadRequest::resourceRequest):
+ (WebCore::FrameLoadRequest::frameName):
+ (WebCore::FrameLoadRequest::setFrameName):
+
+2011-08-07 Sam White <samuel.white@rochester.edu>
+
+ Add the ability to search the AccessibilityObject cache
+ https://bugs.webkit.org/show_bug.cgi?id=64994
+
+ New AccessibilityObject cache search functionality and API for the mac
+ platform to expose it. At this point the AccessibilityObject cache can
+ be searched using one of the supported search keys that have been defined
+ in AccessibilityObject.h, or by passing search text.
+
+ Reviewed by Chris Fleizach.
+
+ Test: platform/mac/accessibility/search-predicate.html
+
+ * accessibility/AccessibilityObject.cpp:
+ (WebCore::AccessibilityObject::isAccessibilityObjectSearchMatch):
+ (WebCore::AccessibilityObject::isAccessibilityTextSearchMatch):
+ (WebCore::AccessibilityObject::accessibilityObjectContainsText):
+ (WebCore::AccessibilityObject::accessibleObjectsWithAccessibilitySearchPredicate):
+ * accessibility/AccessibilityObject.h:
+ * accessibility/mac/AccessibilityObjectWrapper.mm:
+ (createAccessibilitySearchKeyMap):
+ (accessibilitySearchKeyForString):
+ (-[AccessibilityObjectWrapper accessibilityParameterizedAttributeNames]):
+ (-[AccessibilityObjectWrapper accessibilityAttributeValue:forParameter:]):
+
+2011-08-06 Mark Rowe <mrowe@apple.com>
+
+ Fix a memory leak found via code inspection.
+
+ Rubber-stamped by Dan Bernstein.
+
+ * platform/mac/PasteboardMac.mm:
+ (WebCore::Pasteboard::writeSelection): Adopt the newly-allocated WebHTMLConverter
+ instance so that it does not leak.
+
+2011-08-05 Darin Adler <darin@apple.com>
+
+ Fix Timer heap implementation to work with more libraries (other versions of STL)
+ https://bugs.webkit.org/show_bug.cgi?id=65782
+
+ Reviewed by Anders Carlsson.
+
+ No behavior change, so no tests needed. Existing tests pass.
+
+ * platform/Timer.cpp: Added TimerHeapPointer and TimerHeapReference class
+ alongside the TimerHeapIterator class. Also added a swap function. Also
+ added a TimerHeapLessThanFunction class.
+ (WebCore::TimerBase::heapDecreaseKey): Pass pointers in to the TimerHeapIterator
+ since that's how the class works now. Pass a TimerHeapLessThanFunction object
+ instead of letting the library use the < operator directly.
+ (WebCore::TimerBase::heapPopMin): Ditto.
+
+ * platform/Timer.h: Updated for above changes.
+
+2011-08-06 Aron Rosenberg <arosenberg@logitech.com>
+
+ Reviewed by Benjamin Poulain.
+
+ [Qt] Fix build with Intel compiler on Windows
+ https://bugs.webkit.org/show_bug.cgi?id=65088
+
+ Intel compiler needs .lib suffixes instead of .a
+
+ * WebCore.pri:
+
+2011-08-06 Dan Bernstein <mitz@apple.com>
+
+ Move the shared LineBreakIteratorPool from ThreadGlobalData into its own ThreadSpecific
+ https://bugs.webkit.org/show_bug.cgi?id=65809
+
+ Reviewed by Darin Adler and Sam Weinig.
+
+ * platform/ThreadGlobalData.cpp:
+ (WebCore::ThreadGlobalData::destroy): Removed code to clear the m_lineBreakIteratorPool member
+ variable, which was removed.
+ * platform/ThreadGlobalData.h: Removed lineBreakIteratorPool() and associated member variable.
+ * platform/text/LineBreakIteratorPoolICU.h:
+ (WebCore::LineBreakIteratorPool::sharedPool): Changed to return a thread-specific pool.
+
+2011-08-06 Joseph Pecoraro <joepeck@webkit.org>
+
+ Potential Leaks - RetainPtr<> over retaining Create'd objects
+ https://bugs.webkit.org/show_bug.cgi?id=65806
+
+ Reviewed by Darin Adler.
+
+ Fix a possible leak by adopting an allocation instead of retaining it.
+
+ * platform/network/mac/NetworkStateNotifierMac.cpp:
+ (WebCore::NetworkStateNotifier::NetworkStateNotifier):
+ * plugins/mac/PluginPackageMac.cpp:
+ (WebCore::stringListFromResourceId):
+ (WebCore::PluginPackage::fetchInfo):
+
+2011-08-06 Rafael Brandao <rafael.lobo@openbossa.org>
+
+ REGRESSION (r91540): Favicons are not loaded
+ https://bugs.webkit.org/show_bug.cgi?id=65692
+
+ Reviewed by Darin Adler.
+
+ The policy that evaluates whether a page can have icon or not should not
+ need to be reimplemented each time a new IconDatabaseBase is derived,
+ so it was moved to WebCore's scope.
+
+ * loader/icon/IconController.cpp:
+ (WebCore::IconController::startLoader):
+ * loader/icon/IconDatabase.cpp:
+ * loader/icon/IconDatabase.h:
+ * loader/icon/IconDatabaseBase.cpp:
+ (WebCore::documentCanHaveIcon):
+ * loader/icon/IconDatabaseBase.h:
+
+2011-08-06 Kenichi Ishibashi <bashi@chromium.org>
+
+ Should use C++-style cast in CSSParser.cpp
+ https://bugs.webkit.org/show_bug.cgi?id=65807
+
+ Replaces C-style cast with C++style cast.
+
+ Reviewed by Shinichiro Hamaji.
+
+ No new tests because no behavior change.
+
+ * css/CSSParser.cpp:
+ (WebCore::CSSParser::parseAnimationTimingFunction):
+ (WebCore::CSSParser::parseFont):
+ (WebCore::CSSParser::parseColorFromValue):
+ (WebCore::CSSParser::parseFontFeatureTag):
+
+2011-08-06 Nikolas Zimmermann <nzimmermann@rim.com>
+
+ REGRESSION(87526): ASSERT(!needsLayout()) followed by graphical glitches on google charts (svg loaded in iframe)
+ https://bugs.webkit.org/show_bug.cgi?id=64974
+
+ svg/zoom/page/zoom-svg-through-object-with-*.xhtml are flaky
+ https://bugs.webkit.org/show_bug.cgi?id=63186
+
+ Reviewed by Zoltan Herczeg.
+
+ Fix host <-> embedded document size negotiation race. The currently implemented solution relied on a specific
+ order of paint/layout calls, which is broken. Consider rendering a document containing an object/iframe/embed
+ referencing an external SVG file. When FrameView::layout() is called (associated with the outermost RenderView
+ of the host document), it lays out the whole document, and afterwards performPostLayoutTasks() is called.
+ This method then asks the Frames contentRenderer to update its widget positions. This triggers a call
+ of the embedded documents FrameView::layout() method, which now lays out the embedded SVG documents tree
+ through RenderSVGRoot::layout.
+
+ And here's the bug: The size of the object/iframe replaced element, which hosts the embedded document
+ may depend on the intrinsic size of the SVG. We tried to mark the embedded documents _ownerRenderer_ (the RenderPart)
+ as "needs layout and pref width recalc" right from RenderSVGRoot::layout() and hoped that this would cause the whole
+ document to be laid out again, now that the size of the embedded SVG document is known.
+
+ As soon as the SVG document is laid out, the host document will be painted (flush deferred repaints) and an assertion
+ ASSERTS(!needsLayout()) will be fired, as we modified the setNeedsLayout() state after the host document layout finished,
+ right before painting.
+
+ A proper fix is to only keep track in RenderSVGRoot whether it needs to negotiate the size with the host document, but
+ not modify the layout state of the host document in any way. Let FrameView handle the size negotiation right in
+ FrameView::layout().
+
+ Consider following document:
+ <body><iframe src="some.svg"></body>
+
+ After initial loading & parsing of the document, a FrameView exists for the main frame, and a sub-FrameView
+ for the <iframe>. The external SVG document, may not be loaded yet at this point. That means when RenderIFrame::layout()
+ tries to figure out its size (computeLogicalWidth/Height) - the RenderSVGRoot renderer of the external document hasn't
+ been created yet (as the external document hasn't received data yet) - so it falls back to eg. 300x150 CSS default size
+ (in the simplest case, where width/height are both auto).
+
+ Suppose the external document now finished loading, the RenderSVGRoot is created and a global relayout is triggered
+ starting from the main FrameView. As we already laid out the document once, needsLayout() is false for the main FrameView,
+ so _only_ the child frame view that contains the RenderSVGRoot is now laid out, for the first time.
+
+ After layout is done, the SVG document is fully laid out, though the RenderPart which embedded the SVG does NOT notice
+ the SVG has been laid out, so it still carries the default 300x150 size (and needsLayout=false!).
+
+ The fix is to retrigger layout of the parent frame view by marking the owner renderer of the frame view as "needs layout
+ and pref widths recalc" and immediatiely performing a synchronous update of the layout. It's important to do it sync,
+ as scripts depend on the result of the size negotiation (covered extensively with the new tests in svg/as-object).
+
+ Reenable svg/zoom/page/zoom-svg-through-object* tests to see whether the flakiness is gone.
+
+ Tests: svg/as-object/deep-nested-embedded-svg-size-changes-no-layout-triggers-1.html
+ svg/as-object/deep-nested-embedded-svg-size-changes-no-layout-triggers-2.html
+ svg/as-object/embedded-svg-immediate-offsetWidth-query.html
+ svg/as-object/embedded-svg-size-changes-no-layout-triggers.html
+ svg/as-object/embedded-svg-size-changes.html
+ svg/as-object/nested-embedded-svg-size-changes-no-layout-triggers-1.html
+ svg/as-object/nested-embedded-svg-size-changes-no-layout-triggers-2.html
+ svg/as-object/nested-embedded-svg-size-changes.html
+ svg/zoom/page/zoom-svg-through-object-with-absolute-size-2.xhtml
+ svg/zoom/page/zoom-svg-through-object-with-absolute-size.xhtml
+ svg/zoom/page/zoom-svg-through-object-with-auto-size.html
+ svg/zoom/page/zoom-svg-through-object-with-huge-size.xhtml
+ svg/zoom/page/zoom-svg-through-object-with-override-size.html
+ svg/zoom/page/zoom-svg-through-object-with-percentage-size.xhtml
+
+ * page/FrameView.cpp:
+ (WebCore::FrameView::layout): Call forceLayoutParentViewIfNeeded() after layout finished.
+ (WebCore::FrameView::forceLayoutParentViewIfNeeded): Added helper method.
+ (WebCore::FrameView::embeddedContentBox): Moved from RenderPart to a more central place.
+ * page/FrameView.h:
+ * rendering/RenderPart.cpp:
+ (WebCore::RenderPart::embeddedContentBox): Moved to FrameView.
+ * rendering/svg/RenderSVGRoot.cpp: Rename m_didNegotiateSize to m_needsSizeNegotiationWithHostDocument.
+ (WebCore::RenderSVGRoot::RenderSVGRoot):
+ (WebCore::RenderSVGRoot::layout): Only figure out if we need neggotiation, don't actually do anything, let FrameView handle this.
+ * rendering/svg/RenderSVGRoot.h: Remove incorrect negotiateSizeWithHostDocumentIfNeeded() logic.
+ (WebCore::RenderSVGRoot::needsSizeNegotiationWithHostDocument): Added getter for m_needsSizeNegotiationWithHostDocument.
+ (WebCore::RenderSVGRoot::scheduledSizeNegotiationWithHostDocument): Added safe way to clear m_needsSizeNegotiationWithHostDocument (asserts if it was false before).
+
+2011-08-05 Noel Gordon <noel.gordon@gmail.com>
+
+ [Chromium] Remove HaltablePlugin references from gyp project files
+ https://bugs.webkit.org/show_bug.cgi?id=65808
+
+ HaltablePlugin and related classes were removed in r92492. Remove any
+ reference to the deleted files from the gyp project files.
+
+ Reviewed by Adam Barth.
+
+ * WebCore.gypi: remove HaltablePlugin.h reference.
+
+2011-08-05 Mark Rowe <mrowe@apple.com>
+
+ Attempt to fix the build after r92538.
+
+ Like all of the recent "track"-related work the new files in r92538 appear to have
+ been added to completely random places in the Xcode project. However, this time
+ the file paths were marked as being relative to their containing group resulting in
+ Xcode being unable to find the files on disk.
+
+ I've attempted to clean up all of the "track"-related mess in the Xcode project.
+ The location of the files in the project now reflect their locations on disk, and
+ are all correctly marked as being relative to their containing group as is our
+ convention.
+
+ * WebCore.xcodeproj/project.pbxproj:
+
+2011-08-05 Tom Hudson <tomhudson@google.com>
+
+ https://bugs.webkit.org/show_bug.cgi?id=64613
+ Use supported framebuffer renderbuffer mode; chromium command buffer
+ allows DEPTH and STENCIL but not DEPTH_STENCIL.
+
+ Reviewed by James Robinson.
+
+ No new tests because was caught by extant tests, albeit only when
+ accelerated drawing and forced compositing were both turned on.
+
+ * platform/graphics/chromium/LayerTextureUpdaterCanvas.cpp:
+ (WebCore::LayerTextureUpdaterSkPicture::createFrameBuffer):
+
+2011-08-05 Anna Cavender <annacc@chromium.org>
+
+ Adding a WebVTTParser for <track>.
+ https://bugs.webkit.org/show_bug.cgi?id=62882
+
+ Reviewed by Eric Carlson.
+
+ Feature is hidden behind VIDEO_TRACK feature define, which is turned off, but
+ new tests have been added in media/track/ (and are skipped).
+
+ * CMakeLists.txt:
+ * GNUmakefile.list.am:
+ * WebCore.gypi:
+ * WebCore.pro:
+ * WebCore.xcodeproj/project.pbxproj:
+ * html/TextTrackCue.cpp:
+ * html/TextTrackCue.h:
+ * platform/track/CueParser.cpp:
+ * platform/track/CueParser.h:
+ * platform/track/CueParserPrivate.h:
+ * platform/track/WebVTTParser.cpp: Added.
+ * platform/track/WebVTTParser.h: Added.
+
+2011-08-05 Sheriff Bot <webkit.review.bot@gmail.com>
+
+ Unreviewed, rolling out r92330.
+ http://trac.webkit.org/changeset/92330
+ https://bugs.webkit.org/show_bug.cgi?id=65804
+
+ caused various regressions in paste (Requested by rniwa on
+ #webkit).
+
+ * editing/ReplaceSelectionCommand.cpp:
+ (WebCore::isInlineNodeWithStyle):
+ (WebCore::ReplaceSelectionCommand::doApply):
+ * editing/markup.cpp:
+ (WebCore::ancestorToRetainStructureAndAppearance):
+ * editing/markup.h:
+
+2011-08-05 Kent Tamura <tkent@chromium.org>
+
+ Unreviewed, rolling out r92477.
+ http://trac.webkit.org/changeset/92477
+ https://bugs.webkit.org/show_bug.cgi?id=62619
+
+ Layering violation. We should not use WebCore/dom/ in
+ WebCore/platform/.
+
+ * WebCore.exp.in:
+ * WebCore.xcodeproj/project.pbxproj:
+ * html/ColorInputType.cpp:
+ (WebCore::ColorInputType::valueChanged):
+ * html/ColorInputType.h:
+ * html/FileInputType.cpp:
+ (WebCore::FileInputType::chrome):
+ * html/FileInputType.h:
+ * html/HTMLInputElement.cpp:
+ (WebCore::HTMLInputElement::detach):
+ * html/InputType.cpp:
+ * html/InputType.h:
+ * loader/EmptyClients.h:
+ * loader/FrameLoader.cpp:
+ (WebCore::FrameLoader::transitionToCommitted):
+ * page/Chrome.cpp:
+ * page/Chrome.h:
+ * page/ChromeClient.h:
+ * platform/ColorChooser.cpp: Removed.
+ * platform/ColorChooser.h: Removed.
+
+2011-08-05 Darin Adler <darin@apple.com>
+
+ Try to fix Qt Mac build.
+
+ * plugins/mac/PluginViewMac.mm: Removed PluginHalter functions.
+
+2011-08-05 Joseph Pecoraro <joepeck@webkit.org>
+
+ Leak in CFNetwork Loader RetainPtr<> should Adopt a Copy allocation
+ https://bugs.webkit.org/show_bug.cgi?id=65789
+
+ Reviewed by David Kilzer.
+
+ Fix a leak by adopting an allocation instead of retaining it.
+
+ * platform/network/cf/FormDataStreamCFNet.cpp:
+ (WebCore::httpBodyFromRequest):
+
+2011-08-05 Ben Wells <benwells@chromium.org>
+
+ Introduced fast path for border rendering when all visible sides are solid, same rgba color but not all visible
+ https://bugs.webkit.org/show_bug.cgi?id=65762
+
+ Reviewed by Simon Fraser.
+
+ No intended change in behaviour, no new tests.
+
+ * rendering/RenderBoxModelObject.cpp:
+ (WebCore::calculateSideRect):
+ (WebCore::RenderBoxModelObject::paintBorderSides):
+ (WebCore::RenderBoxModelObject::paintBorder):
+
+2011-08-05 Kenichi Ishibashi <bashi@chromium.org>
+
+ Unreviewed build fix on 32-bit Mac.
+
+ * css/CSSParser.cpp:
+ (WebCore::CSSParser::parseFontFeatureTag): Added a cast to int.
+
+2011-08-05 Ryosuke Niwa <rniwa@webkit.org>
+
+ Upwards cursor movement incorrect when previous block ends with <br>
+ https://bugs.webkit.org/show_bug.cgi?id=33247
+
+ Reviewed by Tony Chang.
+
+ The bug was caused by previousLinePosition's trying to obtain the root line box at the position
+ after the previous line's br. This obviously fails because the the position after br is considered
+ as a part of the next line.
+
+ Fixed the bug by obtaining root inline boxes using position at the minimum caret offset as supposed
+ to maximum caret offset. The code was initially introduced by r32508 to fix arrow key movement in RTL text
+ but the test added by the revision continues to pass after this change. Furthermore, this change makes
+ new code consistent with nextLinePosition.
+
+ Also reverted the change added by r55613 because it is no longer needed.
+
+ Tests: editing/execCommand/move-selection-back-line-rtl.html
+ editing/execCommand/move-selection-back-line-strict.html
+
+ * editing/visible_units.cpp:
+ (WebCore::previousLinePosition):
+
+2011-08-05 James Robinson <jamesr@chromium.org>
+
+ [chromium] Accelerated canvas breaks when moving canvases or resources between Pages
+ https://bugs.webkit.org/show_bug.cgi?id=65402
+
+ Reviewed by Stephen White.
+
+ Use one shared GraphicsContext3D for the whole process instead of one per Page as canvases can move between
+ pages and directly draw into contexts in different pages. Also switches DrawingBufferChromium over to use a
+ directly shared the color attachment instead of copying it to a separate texture and removes the now-unnecessary
+ DrawingBuffer::didReset() call and WillPublishCallback mechanism.
+
+ * page/Page.cpp:
+ (WebCore::Page::sharedGraphicsContext3D):
+ * page/Page.h:
+ * platform/graphics/chromium/Canvas2DLayerChromium.cpp:
+ (WebCore::Canvas2DLayerChromium::~Canvas2DLayerChromium):
+ (WebCore::Canvas2DLayerChromium::updateCompositorResources):
+ (WebCore::Canvas2DLayerChromium::textureId):
+ (WebCore::Canvas2DLayerChromium::setDrawingBuffer):
+ * platform/graphics/chromium/CanvasLayerChromium.cpp:
+ (WebCore::CanvasLayerChromium::CanvasLayerChromium):
+ * platform/graphics/chromium/CanvasLayerChromium.h:
+ * platform/graphics/chromium/DrawingBufferChromium.cpp:
+ (WebCore::DrawingBuffer::DrawingBuffer):
+ (WebCore::DrawingBuffer::publishToPlatformLayer):
+ * platform/graphics/chromium/Extensions3DChromium.h:
+ * platform/graphics/chromium/WebGLLayerChromium.cpp:
+ (WebCore::WebGLLayerChromium::WebGLLayerChromium):
+ * platform/graphics/chromium/WebGLLayerChromium.h:
+
+2011-08-05 Anders Carlsson <andersca@apple.com>
+
+ Fix Chromium build.
+
+ * WebCore.gypi:
+
+2011-08-05 Anders Carlsson <andersca@apple.com>
+
+ Remove PluginHalter
+ https://bugs.webkit.org/show_bug.cgi?id=65729
+
+ Reviewed by Darin Adler.
+
+ Remove plug-in halter and associated classes.
+
+ * CMakeLists.txt:
+ * GNUmakefile.list.am:
+ * WebCore.exp.in:
+ * WebCore.pro:
+ * WebCore.vcproj/WebCore.vcproj:
+ * WebCore.xcodeproj/project.pbxproj:
+ * loader/EmptyClients.h:
+ * page/HaltablePlugin.h: Removed.
+ * page/Page.cpp:
+ (WebCore::Page::Page):
+ * page/Page.h:
+ * page/PluginHalter.cpp: Removed.
+ * page/PluginHalter.h: Removed.
+ * page/PluginHalterClient.h: Removed.
+ * page/Settings.cpp:
+ (WebCore::Settings::Settings):
+ * page/Settings.h:
+ * plugins/PluginView.cpp:
+ (WebCore::PluginView::start):
+ (WebCore::PluginView::stop):
+ (WebCore::PluginView::PluginView):
+ * plugins/PluginView.h:
+ * plugins/PluginViewNone.cpp:
+ * plugins/gtk/PluginViewGtk.cpp:
+ * plugins/qt/PluginViewQt.cpp:
+ * plugins/win/PluginViewWin.cpp:
+ * rendering/RenderWidget.cpp:
+ (WebCore::RenderWidget::paint):
+ * rendering/RenderWidget.h:
+
+2011-08-05 Jochen Eisinger <jochen@chromium.org>
+
+ Introduce a new ResourceRequest::TargetType for XHRs
+ https://bugs.webkit.org/show_bug.cgi?id=65544
+
+ Reviewed by Darin Fisher.
+
+ * platform/network/chromium/ResourceRequest.h:
+ * xml/XMLHttpRequest.cpp:
+ (WebCore::XMLHttpRequest::createRequest):
+
+2011-08-05 Yael Aharon <yael.aharon@nokia.com>
+
+ dir=auto needs to work on value of input and textarea elements
+ https://bugs.webkit.org/show_bug.cgi?id=65428
+
+ Reviewed by Darin Adler.
+
+ Changed the directionality algorithm to evaluate the value of input elements and textarea elements when
+ they have the attribute dir-auto Set.
+ Also skip these elements when evaluating the directionality of their ancestors.
+
+ HTMLTextAreaElement::childrenChanged is not called when a user types into the textarea, so call
+ calculateAndAdjustDirectionality() explicitly from HTMLTextAreaElement::subtreeHasChanged().
+
+ HTMLInputElement::childrenChanged is not called when a user types into the textarea, so call
+ calculateAndAdjustDirectionality() explicitly from HTMLTextAreaElement::subtreeHasChanged().
+
+ Tests: fast/dom/HTMLElement/attr-dir-auto-changed-text-form-control.html
+ fast/dom/HTMLElement/attr-dir-auto-text-form-control-child.html
+ fast/dom/HTMLElement/attr-dir-auto-text-form-control.html
+
+ * html/HTMLElement.cpp:
+ (WebCore::HTMLElement::directionality):
+ * html/HTMLElement.h:
+ * html/HTMLTextAreaElement.cpp:
+ (WebCore::HTMLTextAreaElement::subtreeHasChanged):
+
+2011-08-05 Keishi Hattori <keishi@webkit.org>
+
+ Implement <input type=color> UI behavior WebCore part
+ https://bugs.webkit.org/show_bug.cgi?id=62619
+
+ Reviewed by Kent Tamura.
+
+ * WebCore.exp.in: Added __ZN7WebCore12ColorChooser7chooserEv and
+ __ZNK7WebCore12ColorChooser13colorSelectedERKNS_5ColorE
+ * WebCore.xcodeproj/project.pbxproj: Added ColorChooser.{h,cpp}
+ * html/ColorInputType.cpp:
+ (WebCore::ColorInputType::~ColorInputType): Added. Close color chooser. ex. when type attribute changes.
+ (WebCore::ColorInputType::valueAsColor): Added.
+ (WebCore::ColorInputType::setValueAsColor): Added.
+ (WebCore::ColorInputType::valueChanged): Update selected color in color chooser too.
+ (WebCore::ColorInputType::handleClickEvent): Opens color chooser.
+ (WebCore::ColorInputType::handleDOMActivateEvent): Called when element.click(). Open color chooser only
+ when it was initiated by a user interaction.
+ (WebCore::ColorInputType::detach): Close color chooser. Called when input element or its ancestors have "display:none"
+ or is removed from DOM.
+ (WebCore::ColorInputType::colorSelected): Callback from color chooser.
+ (WebCore::ColorInputType::closeColorChooserIfClientIsInDocument): Close color chooser if element is in
+ document. Called when the page navigates away.
+ (WebCore::ColorInputType::closeColorChooserIfCurrentClient): Close color chooser if this input type is the current client of ColorChooser.
+ * html/ColorInputType.h:
+ * html/FileInputType.cpp:
+ * html/FileInputType.h:
+ * html/HTMLInputElement.cpp:
+ (WebCore::HTMLInputElement::detach): Calls InputType::detach
+ * html/InputType.cpp:
+ (WebCore::InputType::chrome): Added. Used in FileInputType and ColorInputType.
+ (WebCore::InputType::detach): ColorInputType overrides this.
+ * html/InputType.h:
+ * loader/EmptyClients.h:
+ (WebCore::EmptyChromeClient::openColorChooser): Added.
+ (WebCore::EmptyChromeClient::closeColorChooser): Added.
+ (WebCore::EmptyChromeClient::setSelectedColorInColorChooser): Added.
+ * loader/FrameLoader.cpp:
+ (WebCore::FrameLoader::transitionToCommitted): Close color chooser when
+ navigating away from the page.
+ * page/Chrome.cpp:
+ (WebCore::Chrome::openColorChooser): Added. Opens the color chooser.
+ (WebCore::Chrome::closeColorChooser): Added. Tries to close the color chooser. Might not close if the
+ listener of the color chooser is another part of the browser or another render process.
+ (WebCore::Chrome::setSelectedColorInColorChooser): Added. Sets the selected color in the color chooser.
+ Again, might not be executed if the listener of the color chooser is another part of the browser or another render process.
+ * page/Chrome.h:
+ * page/ChromeClient.h:
+ * platform/ColorChooser.cpp: Added.
+ (WebCore::ColorChooserClient::~ColorChooserClient): Disconnects itself from the ColorChooser.
+ (WebCore::ColorChooser::chooser): Get shared instance of ColorChooser.
+ (WebCore::ColorChooser::chooser): Get shared instance of ColorChooser.
+ (WebCore::ColorChooser::connectClient): Connects a ColorChooserClient that receives the colorSelected callbacks.
+ (WebCore::ColorChooser::disconnectClient): Disconnects the connectClient.
+ (WebCore::ColorChooser::closeColorChooserIfClientIsInDocument): Close the color chooser if the client is
+ inside the document.
+ (WebCore::ColorChooser::colorSelected): Called from color chooser listener.
+ * platform/ColorChooser.h: Added.
+ (WebCore::ColorChooser::client): Added. Returns the current connected client.
+ (WebCore::ColorChooser::ColorChooser): Added.
+
+2011-08-05 Yury Semikhatsky <yurys@chromium.org>
+
+ Web Inspector: check that detaching frame has been attached before removing it from the console selector
+ https://bugs.webkit.org/show_bug.cgi?id=65686
+
+ Reviewed by Pavel Feldman.
+
+ * inspector/front-end/JavaScriptContextManager.js:
+ (WebInspector.JavaScriptContextManager.prototype._frameDetached):
+
+2011-08-05 Keishi Hattori <keishi@webkit.org>
+
+ Sort WebCore.xcodeproj
+ Accomplished using sort-Xcode-project-file.
+
+ * WebCore.xcodeproj/project.pbxproj:
+
+2011-08-05 Yury Semikhatsky <yurys@chromium.org>
+
+ Web Inspector: constrain maximum depth for returnByValue objects
+ https://bugs.webkit.org/show_bug.cgi?id=65761
+
+ Set maximum depth to 20 for objects returned by value as a result of evaluations.
+
+ Reviewed by Pavel Feldman.
+
+ * bindings/js/ScriptValue.cpp:
+ (WebCore::jsToInspectorValue):
+ (WebCore::ScriptValue::toInspectorValue):
+ * bindings/v8/ScriptValue.cpp:
+ (WebCore::v8ToInspectorValue):
+ (WebCore::ScriptValue::toInspectorValue):
+ * inspector/InjectedScript.cpp:
+ (WebCore::InjectedScript::makeCall):
+ * inspector/InspectorValues.h:
+
+2011-08-05 Mark Pilgrim <pilgrim@chromium.org>
+
+ Remove LegacyDefaultOptionalArguments flag from appcache IDL files
+ https://bugs.webkit.org/show_bug.cgi?id=65752
+
+ Reviewed by Adam Barth.
+
+ No new tests, all existing tests pass.
+
+ * loader/appcache/DOMApplicationCache.idl:
+
+2011-08-04 Hans Wennborg <hans@chromium.org>
+
+ IndexedDB: Stop using free-lists for database/object store/index ids.
+ https://bugs.webkit.org/show_bug.cgi?id=65678
+
+ Reviewed by Tony Chang.
+
+ Don't use free-lists for database/object store/index ids,
+ just assign increasing numbers.
+
+ It turns out that deleting an object store and creating a new one with
+ the same id would cause the delete markers from the old object store to
+ slow down lookups into the new one. Therefore we should generate
+ a new id every time. Running out of ids (64 bits for databases and
+ object stores, 32 bits for indices) is not realistic.
+
+ Also make functions that generate new ids report errors, and make the
+ callers of those functions check the return values.
+
+ We must still delete free-lists when deleting an object store, and we
+ must keep the code for encoding/decoding/comparison of free-list keys
+ since users might have them in their databases.
+
+ This is just a performance optimization, so no new tests.
+
+ * storage/IDBLevelDBBackingStore.cpp:
+ (WebCore::getNewDatabaseId):
+ (WebCore::IDBLevelDBBackingStore::setIDBDatabaseMetaData):
+ (WebCore::getNewObjectStoreId):
+ (WebCore::IDBLevelDBBackingStore::createObjectStore):
+ (WebCore::IDBLevelDBBackingStore::deleteObjectStore):
+ (WebCore::getNewIndexId):
+ (WebCore::IDBLevelDBBackingStore::createIndex):
+ (WebCore::IDBLevelDBBackingStore::deleteIndex):
+
+2011-08-05 Roland Steiner <rolandsteiner@chromium.org>
+
+ Unreviewed: change an instance of isImportRule() that was overlooked in commit 92448.
+
+ No new tests. (No functional change)
+
+ * xml/XSLImportRule.h:
+ (WebCore::XSLImportRule::isImportRule):
+
+2011-08-05 Mark Pilgrim <pilgrim@chromium.org>
+
+ Remove LegacyDefaultOptionalArguments flag from remaining SVG IDL files
+ https://bugs.webkit.org/show_bug.cgi?id=65751
+
+ Reviewed by Adam Barth.
+
+ No new tests, all existing tests pass.
+
+ * svg/SVGElementInstance.idl:
+
+2011-08-05 Pavel Feldman <pfeldman@google.com>
+
+ Web Inspector: do not delay scroll event handling - scroll is already async.
+ https://bugs.webkit.org/show_bug.cgi?id=65693
+
+ Reviewed by Yury Semikhatsky.
+
+ * inspector/front-end/TextViewer.js:
+ (WebInspector.TextViewer.prototype._syncScroll):
+
+2011-08-05 Mark Pilgrim <pilgrim@chromium.org>
+
+ Remove LegacyDefaultOptionalArguments flag from web audio API
+ https://bugs.webkit.org/show_bug.cgi?id=65750
+
+ Reviewed by Adam Barth.
+
+ No new tests, all existing tests pass.
+
+ * webaudio/AudioBufferSourceNode.idl:
+
+2011-08-05 Mark Pilgrim <pilgrim@chromium.org>
+
+ Remove LegacyDefaultOptionalArguments flag from Web Workers
+ https://bugs.webkit.org/show_bug.cgi?id=65746
+
+ Reviewed by Adam Barth.
+
+ No new tests, all existing tests pass.
+
+ * workers/AbstractWorker.idl:
+ * workers/DedicatedWorkerContext.idl:
+ * workers/Worker.idl:
+ * workers/WorkerContext.idl:
+
+2011-08-05 Kenichi Ishibashi <bashi@chromium.org>
+
+ Parsing CSS3 font-feature-settings property
+ https://bugs.webkit.org/show_bug.cgi?id=63618
+
+ Introduces CSS3 font-feature-settings property as -webkit-font-feature-settings. This change only contains parsing part. Parsed information are stored in FontDescription class.
+
+ Reviewed by Shinichiro Hamaji.
+
+ Test: css3/font-feature-settings-parsing.html
+
+ * CMakeLists.txt: Added FontFeatureValue.{h,cpp} and FontFeatureSettings.{h,cpp}.
+ * GNUmakefile.list.am: Ditto.
+ * WebCore.gypi: Ditto.
+ * WebCore.pro: Ditto.
+ * WebCore.vcproj/WebCore.vcproj: Ditto.
+ * WebCore.xcodeproj/project.pbxproj: Ditto.
+ * css/CSSComputedStyleDeclaration.cpp:
+ (WebCore::CSSComputedStyleDeclaration::getPropertyCSSValue): Added CSSPropertyWebkitFontFeatureSettings.
+ * css/CSSParser.cpp:
+ (WebCore::CSSParser::parseValue): Added parsing code for -webkit-font-feature-settings.
+ (WebCore::CSSParser::parseFontFeatureTag): Added.
+ (WebCore::CSSParser::parseFontFeatureSettings): Added.
+ * css/CSSParser.h: Added parseFontFeatureSettings().
+ * css/CSSPropertyNames.in: Added -webkit-font-feature-settings.
+ * css/CSSStyleSelector.cpp:
+ (WebCore::CSSStyleSelector::applyDeclarations): Modified a compile assert to follow adding -webkit-font-feature-settings.
+ (WebCore::CSSStyleSelector::applyProperty): Added a case clause for CSSPropertyWebkitFontFeatureSettings.
+ * css/CSSValue.h:
+ (WebCore::CSSValue::isFontFeatureValue): Added.
+ * css/CSSValueKeywords.in: Added 'on' and 'off' keywords.
+ * css/FontFeatureValue.cpp: Added.
+ (WebCore::FontFeatureValue::FontFeatureValue):
+ (WebCore::FontFeatureValue::value):
+ (WebCore::FontFeatureValue::cssText):
+ * css/FontFeatureValue.h: Added.
+ (WebCore::FontFeatureValue::create):
+ (WebCore::FontFeatureValue::tag):
+ (WebCore::FontFeatureValue::isFontFeatureValue):
+ * platform/graphics/FontDescription.cpp:
+ (WebCore::FontDescription::makeNormalFeatureSettings): Added.
+ * platform/graphics/FontDescription.h:
+ (WebCore::FontDescription::FontDescription): Added m_featureSettings member variable.
+ (WebCore::FontDescription::featureSettings): Added.
+ (WebCore::FontDescription::setFeatureSettings): Added.
+ (WebCore::FontDescription::operator==): Modified to take into account m_featureSettings
+ * platform/graphics/FontFeatureSettings.cpp: Added.
+ (WebCore::FontFeature::FontFeature):
+ (WebCore::FontFeature::operator=):
+ (WebCore::FontFeature::operator==):
+ (WebCore::FontFeatureSettings::FontFeatureSettings):
+ * platform/graphics/FontFeatureSettings.h: Added.
+ (WebCore::FontFeature::tag):
+ (WebCore::FontFeature::value):
+ (WebCore::FontFeatureSettings::create):
+ (WebCore::FontFeatureSettings::append):
+ (WebCore::FontFeatureSettings::size):
+ (WebCore::FontFeatureSettings::operator[]):
+ (WebCore::FontFeatureSettings::at):
+
+2011-08-05 Mark Pilgrim <pilgrim@chromium.org>
+
+ Remove LegacyDefaultOptionalArguments flag from websockets
+ https://bugs.webkit.org/show_bug.cgi?id=65749
+
+ Reviewed by Adam Barth.
+
+ No new tests, all existing tests pass.
+
+ * websockets/WebSocket.idl:
+
+2011-08-05 Adam Barth <abarth@webkit.org>
+
+ Attempt to heal media/video-can-play-type.html and media/video-src-change.html.
+
+ * html/HTMLMediaElement.idl:
+
+2011-08-05 Mark Pilgrim <pilgrim@chromium.org>
+
+ Remove LegacyDefaultOptionalArguments flag from notifications IDL files
+ https://bugs.webkit.org/show_bug.cgi?id=65747
+
+ Reviewed by Adam Barth.
+
+ No new tests, all existing tests pass.
+
+ * notifications/Notification.idl:
+ * notifications/NotificationCenter.idl:
+
+2011-08-05 Sheriff Bot <webkit.review.bot@gmail.com>
+
+ Unreviewed, rolling out r92439.
+ http://trac.webkit.org/changeset/92439
+ https://bugs.webkit.org/show_bug.cgi?id=65753
+
+ Caused 9 tests to fail on Qt (Requested by abarth on #webkit).
+
+ * dom/Document.cpp:
+ (WebCore::Document::implicitOpen):
+ * loader/FrameLoader.cpp:
+ (WebCore::FrameLoader::transitionToCommitted):
+ (WebCore::FrameLoader::detachChildren):
+
+2011-08-04 Ryosuke Niwa <rniwa@webkit.org>
+
+ Use RenderedPosition instead of getInlineBoxAndOffset in Editor and AccessibilityObject
+ https://bugs.webkit.org/show_bug.cgi?id=65647
+
+ Reviewed by Hajime Morita.
+
+ Encapsulated the use of getInlineBoxAndOffset in Editor.cpp and AccessibilityObject.cpp by RenderedPosition.
+
+ Also added rendererFromPosition that returns the renderer of deprecatedNode but using a proper Position interface.
+
+ * accessibility/AccessibilityObject.cpp:
+ (WebCore::updateAXLineStartForVisiblePosition):
+ * editing/Editor.cpp:
+ (WebCore::Editor::firstRectForRange):
+ * editing/RenderedPosition.cpp:
+ (WebCore::rendererFromPosition):
+ (WebCore::RenderedPosition::RenderedPosition):
+ (WebCore::RenderedPosition::absoluteRect):
+ * editing/RenderedPosition.h:
+ (WebCore::RenderedPosition::absoluteRect):
+
+2011-08-04 Hayato Ito <hayato@chromium.org>
+
+ Make ScopedEventQueue enqueue an EventDispatchMediator, instead of an Event.
+ https://bugs.webkit.org/show_bug.cgi?id=65613
+
+ Reviewed by Dimitri Glazkov.
+
+ No changes to functionality so no new tests.
+
+ * dom/Event.h:
+ * dom/EventDispatcher.cpp:
+ (WebCore::EventDispatcher::dispatchScopedEvent):
+ * dom/EventDispatcher.h:
+ * dom/Node.cpp:
+ (WebCore::Node::dispatchScopedEvent):
+ (WebCore::Node::dispatchScopedEventDispatchMediator):
+ * dom/Node.h:
+ * dom/ScopedEventQueue.cpp:
+ (WebCore::ScopedEventQueue::~ScopedEventQueue):
+ (WebCore::ScopedEventQueue::enqueueEventDispatchMediator):
+ (WebCore::ScopedEventQueue::dispatchAllEvents):
+ (WebCore::ScopedEventQueue::dispatchEvent):
+ * dom/ScopedEventQueue.h:
+
+2011-08-04 James Robinson <jamesr@chromium.org>
+
+ Unreviewed build fix. gcc 4.5 can't figure out that the 'data' variables are always initialized in these functions.
+
+ * bindings/v8/custom/V8WebGLRenderingContextCustom.cpp:
+ (WebCore::jsArrayToFloatArray):
+ (WebCore::jsArrayToIntArray):
+
+2011-08-05 Roland Steiner <rolandsteiner@chromium.org>
+
+ Improve const-correctness in CSS code: type information methods
+ https://bugs.webkit.org/show_bug.cgi?id=65506
+
+ Reviewed by Simon Fraser.
+
+ Make "is...()" functions (e.g., "isStyleRule()") const.
+ Also, change CSSRule's type() function to return CSSRuleType.
+
+ No new tests. (no change in semantics)
+
+ * css/CSSCharsetRule.h:
+ (WebCore::CSSCharsetRule::isCharsetRule):
+ (WebCore::CSSCharsetRule::type):
+ * css/CSSFontFaceRule.h:
+ (WebCore::CSSFontFaceRule::isFontFaceRule):
+ (WebCore::CSSFontFaceRule::type):
+ * css/CSSImportRule.h:
+ (WebCore::CSSImportRule::isImportRule):
+ (WebCore::CSSImportRule::type):
+ * css/CSSMediaRule.h:
+ (WebCore::CSSMediaRule::isMediaRule):
+ (WebCore::CSSMediaRule::type):
+ * css/CSSPageRule.h:
+ (WebCore::CSSPageRule::isPageRule):
+ (WebCore::CSSPageRule::type):
+ * css/CSSRule.h:
+ (WebCore::CSSRule::isRule):
+ * css/CSSStyleRule.h:
+ (WebCore::CSSStyleRule::isStyleRule):
+ (WebCore::CSSStyleRule::type):
+ * css/CSSUnknownRule.h:
+ (WebCore::CSSUnknownRule::type):
+ * css/StyleBase.h:
+ (WebCore::StyleBase::isCharsetRule):
+ (WebCore::StyleBase::isFontFaceRule):
+ (WebCore::StyleBase::isImportRule):
+ (WebCore::StyleBase::isKeyframeRule):
+ (WebCore::StyleBase::isKeyframesRule):
+ (WebCore::StyleBase::isMediaRule):
+ (WebCore::StyleBase::isPageRule):
+ (WebCore::StyleBase::isRule):
+ (WebCore::StyleBase::isStyleRule):
+ * css/WebKitCSSKeyframeRule.h:
+ (WebCore::WebKitCSSKeyframeRule::isKeyframeRule):
+ (WebCore::WebKitCSSKeyframeRule::type):
+ * css/WebKitCSSKeyframesRule.h:
+ (WebCore::WebKitCSSKeyframesRule::isKeyframesRule):
+ (WebCore::WebKitCSSKeyframesRule::type):
+
+2011-08-04 Mark Pilgrim <pilgrim@chromium.org>
+
+ Remove LegacyDefaultOptionalArguments flag from plugin IDL files
+ https://bugs.webkit.org/show_bug.cgi?id=65745
+
+ Reviewed by Adam Barth.
+
+ No new tests, all existing tests pass.
+
+ * plugins/DOMMimeTypeArray.idl:
+ * plugins/DOMPlugin.idl:
+ * plugins/DOMPluginArray.idl:
+
+2011-08-04 Mark Pilgrim <pilgrim@chromium.org>
+
+ Remove LegacyDefaultOptionalArguments flag from XML/XSL IDL files
+ https://bugs.webkit.org/show_bug.cgi?id=65740
+
+ Reviewed by Adam Barth.
+
+ No new tests, all existing tests pass.
+
+ * xml/DOMParser.idl:
+ * xml/XMLSerializer.idl:
+ * xml/XPathEvaluator.idl:
+ * xml/XPathExpression.idl:
+ * xml/XPathNSResolver.idl:
+ * xml/XPathResult.idl:
+ * xml/XSLTProcessor.idl:
+
+2011-08-04 Mark Pilgrim <pilgrim@chromium.org>
+
+ Remove LegacyDefaultOptionalArguments flag from canvas IDL files
+ https://bugs.webkit.org/show_bug.cgi?id=65737
+
+ Reviewed by Adam Barth.
+
+ No new tests, all existing tests pass.
+
+ * html/canvas/CanvasGradient.idl:
+ * html/canvas/Float32Array.idl:
+ * html/canvas/Float64Array.idl:
+ * html/canvas/Int16Array.idl:
+ * html/canvas/Int32Array.idl:
+ * html/canvas/Int8Array.idl:
+ * html/canvas/OESVertexArrayObject.idl:
+ * html/canvas/Uint16Array.idl:
+ * html/canvas/Uint32Array.idl:
+ * html/canvas/Uint8Array.idl:
+
+2011-08-04 MORITA Hajime <morrita@google.com>
+
+ Reviewed by Ryosuke Niwa.
+
+ DocumentMarker: Type specific details should be separately held by other object.
+ https://bugs.webkit.org/show_bug.cgi?id=59855
+
+ - Introduced DocumentMarkerDetails abstract class and two its subclasses
+ DocumentMarkerDescription and DocumentMarkerActiveMatch,
+
+ - Replacing DocumentMarker::m_description and
+ DocumentMarker::m_activeMatch with these new details classes.
+
+ No new tests. No behavior change.
+
+ * CMakeLists.txt:
+ * GNUmakefile.list.am:
+ * WebCore.gypi:
+ * WebCore.pro:
+ * WebCore.vcproj/WebCore.vcproj:
+ * WebCore.xcodeproj/project.pbxproj:
+ * dom/DOMAllInOne.cpp:
+ * dom/DocumentMarker.cpp: Added.
+ (WebCore::emptyDescription):
+ (WebCore::DocumentMarkerDetails::DocumentMarkerDetails):
+ (WebCore::DocumentMarkerDetails::~DocumentMarkerDetails):
+ (WebCore::DocumentMarkerDescription::description):
+ (WebCore::DocumentMarkerDescription::isDescription):
+ (WebCore::DocumentMarkerDescription::DocumentMarkerDescription):
+ (WebCore::DocumentMarkerDescription::createUnlessEmpty):
+ (WebCore::DocumentMarkerDescription::compatibleTypes):
+ (WebCore::DocumentMarkerTextMatch::activeMatch):
+ (WebCore::DocumentMarkerTextMatch::isTextMatch):
+ (WebCore::DocumentMarkerTextMatch::DocumentMarkerTextMatch):
+ (WebCore::DocumentMarkerTextMatch::instanceFor):
+ (WebCore::DocumentMarkerTextMatch::compatibleTypes):
+ (WebCore::DocumentMarker::DocumentMarker):
+ (WebCore::DocumentMarker::shiftOffsets):
+ (WebCore::DocumentMarker::setActiveMatch):
+ (WebCore::DocumentMarker::description):
+ (WebCore::DocumentMarker::activeMatch):
+ * dom/DocumentMarker.h:
+ (WebCore::DocumentMarker::details):
+ (WebCore::DocumentMarker::clearDetails):
+ (WebCore::DocumentMarkerDetails::isDescription):
+ (WebCore::DocumentMarkerDetails::isTextMatch):
+ (WebCore::DocumentMarkerDetails::isAllowedFor):
+ * dom/DocumentMarkerController.cpp:
+ (WebCore::DocumentMarkerController::clearDescriptionOnMarkersIntersectingRange):
+ * editing/CompositeEditCommand.cpp:
+ (WebCore::CompositeEditCommand::replaceTextInNodePreservingMarkers):
+
+2011-08-04 Scott Graham <scottmg@chromium.org>
+
+ Bad interaction between document destruction and unload events
+ https://bugs.webkit.org/show_bug.cgi?id=64741
+
+ Reviewed by Adam Barth.
+
+ Three different errors triggered by this test case. The case to
+ consider is a subdocument with an onunload on an element, that
+ destroys the parent document during the onunload. One fix was a
+ lifetime issue fixed by a protecting RefPtr, and another was an
+ additional cancel of event triggers. The main fix was that during the
+ transition to commited state, the documentLoader is being replaced by
+ the provisionalDocumentLoader. But, because during firing events in
+ the subdocument the parent is destroyed, that subevent caused the
+ provisionalDocumentLoader to be detached from its frame. By marking
+ the page as being in committed state before the parent documentLoader
+ is set, this is avoided.
+
+ Test: loader/document-destruction-within-unload.html
+
+ * dom/Document.cpp:
+ (WebCore::Document::implicitOpen):
+ * loader/FrameLoader.cpp:
+ (WebCore::FrameLoader::transitionToCommitted):
+ (WebCore::FrameLoader::detachChildren):
+
+2011-08-04 Simon Fraser <simon.fraser@apple.com>
+
+ Add code to determine whether a Range in inside fixed position content
+ https://bugs.webkit.org/show_bug.cgi?id=65720
+
+ Reviewed by Sam Weinig.
+
+ Propagate the 'wasFixed' flag out from various methods that
+ are used when computing the collection of rects or quads
+ for a range. Use the flag in Range to computer whether
+ the Range is entirely within, partially within, or outside
+ of fixed-position content.
+
+ * WebCore.exp.in:
+ * accessibility/AccessibilityRenderObject.cpp:
+ (WebCore::AccessibilityRenderObject::boundingBoxRect):
+ * dom/Range.cpp:
+ (WebCore::Range::textRects):
+ (WebCore::Range::textQuads):
+ * dom/Range.h:
+ * rendering/RenderBlock.cpp:
+ (WebCore::RenderBlock::absoluteQuads):
+ * rendering/RenderBlock.h:
+ * rendering/RenderBox.cpp:
+ (WebCore::RenderBox::absoluteQuads):
+ * rendering/RenderBox.h:
+ * rendering/RenderInline.cpp:
+ (WebCore::RenderInline::absoluteQuads):
+ * rendering/RenderInline.h:
+ * rendering/RenderObject.h:
+ (WebCore::RenderObject::absoluteQuads):
+ * rendering/RenderText.cpp:
+ (WebCore::RenderText::absoluteRectsForRange):
+ (WebCore::RenderText::absoluteQuads):
+ (WebCore::RenderText::absoluteQuadsForRange):
+ * rendering/RenderText.h:
+ * rendering/RenderView.cpp:
+ (WebCore::RenderView::absoluteQuads):
+ * rendering/RenderView.h:
+ * rendering/svg/RenderSVGHiddenContainer.cpp:
+ (WebCore::RenderSVGHiddenContainer::absoluteQuads):
+ * rendering/svg/RenderSVGHiddenContainer.h:
+ * rendering/svg/RenderSVGInline.cpp:
+ (WebCore::RenderSVGInline::absoluteQuads):
+ * rendering/svg/RenderSVGInline.h:
+ * rendering/svg/RenderSVGModelObject.cpp:
+ (WebCore::RenderSVGModelObject::absoluteQuads):
+ * rendering/svg/RenderSVGModelObject.h:
+ * rendering/svg/RenderSVGText.cpp:
+ (WebCore::RenderSVGText::absoluteQuads):
+ * rendering/svg/RenderSVGText.h:
+
+2011-08-04 Mark Pilgrim <pilgrim@chromium.org>
+
+ Remove LegacyDefaultOptionalArguments flag from Document.idl
+ https://bugs.webkit.org/show_bug.cgi?id=65717
+
+ Reviewed by Adam Barth.
+
+ No new tests, all existing tests pass.
+
+ * dom/Document.idl:
+
+2011-08-04 Simon Fraser <simon.fraser@apple.com>
+
+ Fix release build failure with last commit.
+
+ * rendering/RenderView.cpp:
+ (WebCore::RenderView::mapLocalToContainer):
+
+2011-08-04 Simon Fraser <simon.fraser@apple.com>
+
+ Add code to determine whether a Range in inside fixed position content
+ https://bugs.webkit.org/show_bug.cgi?id=65720
+
+ Reviewed by Dan Bernstein.
+
+ Add an out param for various coordinate mapping functions
+ that tells us whether the point or rectangle being
+ mapped is inside fixed-position content.
+
+ This brings back some of the changes from r89754.
+
+ No behavior changes, so no tests.
+
+ * rendering/RenderBox.cpp:
+ (WebCore::RenderBox::mapLocalToContainer):
+ * rendering/RenderBox.h:
+ * rendering/RenderInline.cpp:
+ (WebCore::RenderInline::mapLocalToContainer):
+ * rendering/RenderInline.h:
+ * rendering/RenderObject.cpp:
+ (WebCore::RenderObject::mapLocalToContainer):
+ (WebCore::RenderObject::localToContainerQuad):
+ * rendering/RenderObject.h:
+ (WebCore::RenderObject::localToAbsoluteQuad):
+ * rendering/RenderView.cpp:
+ (WebCore::RenderView::mapLocalToContainer):
+ * rendering/RenderView.h:
+ * rendering/svg/RenderSVGForeignObject.cpp:
+ (WebCore::RenderSVGForeignObject::mapLocalToContainer):
+ * rendering/svg/RenderSVGForeignObject.h:
+ * rendering/svg/RenderSVGInline.cpp:
+ (WebCore::RenderSVGInline::mapLocalToContainer):
+ * rendering/svg/RenderSVGInline.h:
+ * rendering/svg/RenderSVGModelObject.cpp:
+ (WebCore::RenderSVGModelObject::mapLocalToContainer):
+ * rendering/svg/RenderSVGModelObject.h:
+ * rendering/svg/RenderSVGRoot.cpp:
+ (WebCore::RenderSVGRoot::mapLocalToContainer):
+ * rendering/svg/RenderSVGRoot.h:
+ * rendering/svg/RenderSVGText.cpp:
+ (WebCore::RenderSVGText::mapLocalToContainer):
+ * rendering/svg/RenderSVGText.h:
+ * rendering/svg/SVGRenderSupport.cpp:
+ (WebCore::SVGRenderSupport::mapLocalToContainer):
+ * rendering/svg/SVGRenderSupport.h:
+
+2011-08-04 Mark Pilgrim <pilgrim@chromium.org>
+
+ Remove LegacyDefaultOptionalArguments flag from DOM-related files except Document.idl
+ https://bugs.webkit.org/show_bug.cgi?id=65715
+
+ Reviewed by Adam Barth.
+
+ No new tests, all existing tests pass.
+
+ * dom/CharacterData.idl:
+ * dom/ClientRectList.idl:
+ * dom/Clipboard.idl:
+ * dom/DOMImplementation.idl:
+ * dom/DOMStringList.idl:
+ * dom/DataTransferItem.idl:
+ * dom/DataTransferItems.idl:
+ * dom/DocumentFragment.idl:
+ * dom/Element.idl:
+ * dom/EventListener.idl:
+ * dom/EventTarget.idl:
+ * dom/MediaStream.idl:
+ * dom/MediaStreamList.idl:
+ * dom/MediaStreamTrackList.idl:
+ * dom/MessagePort.idl:
+ * dom/NamedNodeMap.idl:
+ * dom/Node.idl:
+ * dom/NodeFilter.idl:
+ * dom/NodeList.idl:
+ * dom/Range.idl:
+ * dom/RequestAnimationFrameCallback.idl:
+ * dom/StringCallback.idl:
+ * dom/Text.idl:
+ * dom/TouchList.idl:
+
+2011-08-04 Mark Rowe <mrowe@apple.com>
+
+ Future-proof Xcode configuration settings.
+
+ * Configurations/Base.xcconfig:
+ * Configurations/DebugRelease.xcconfig:
+ * Configurations/Version.xcconfig:
+ * Configurations/WebCore.xcconfig:
+
+2011-08-04 Van Lam <vanlam@google.com>
+
+ A few purely stylistic modifications to visible_units.cpp
+ https://bugs.webkit.org/show_bug.cgi?id=65723
+
+ Reviewed by Ryosuke Niwa.
+
+ Renamed greatestValueUnder to greatestOffsetUnder, positionIsInsideBox
+ to positionIsInBoxButNotOnBoundary (to avoid confusion with
+ positionIsInBox, which is just a getInlineBoxAndOffset check).
+ Removed use of invalidOffset as an error value in greatestOffsetUnder
+ and smallestOffsetAbove since semantically it should only be used to
+ check if it makes sense to compare offsets in a single box.
+
+ * editing/visible_units.cpp:
+ (WebCore::greatestOffsetUnder):
+ (WebCore::smallestOffsetAbove):
+ (WebCore::positionIsInBoxButNotOnBoundary):
+ (WebCore::leftWordPositionAcrossBoundary):
+ (WebCore::rightWordPositionAcrossBoundary):
+
+2011-08-04 James Robinson <jamesr@chromium.org>
+
+ [chromium] Implement a global resource limit for DrawingBuffer to limit the amount of GPU memory used by 2d canvas backing stores
+ https://bugs.webkit.org/show_bug.cgi?id=65655
+
+ Reviewed by Kenneth Russell.
+
+ * platform/graphics/gpu/DrawingBuffer.cpp:
+ (WebCore::DrawingBuffer::setResourceLimit):
+ (WebCore::DrawingBuffer::clear):
+ (WebCore::DrawingBuffer::reset):
+ * platform/graphics/gpu/DrawingBuffer.h:
+
+2011-08-04 Kenichi Ishibashi <bashi@chromium.org>
+
+ [Chromium] Reduce memory consumption of HarfbuzzFace
+ https://bugs.webkit.org/show_bug.cgi?id=65688
+
+ Adds a cache for HB_FaceRec to eliminate multiple allocation for the same font.
+
+ Reviewed by Tony Chang.
+
+ No new tests since there is no behavior change.
+
+ * platform/graphics/chromium/HarfbuzzSkia.cpp:
+ (WebCore::getCachedHarfbuzzFace): Added.
+ (WebCore::releaseCachedHarfbuzzFace): Ditto.
+ (WebCore::HarfbuzzFace::HarfbuzzFace): Calls getCachedHarfbuzzFace() instead of allocating HB_FaceRec.
+ (WebCore::HarfbuzzFace::~HarfbuzzFace): Calls releaseCachedHarfbuzzFace() to release the onership of the cache.
+ * platform/graphics/chromium/HarfbuzzSkia.h:
+
+2011-08-04 Sheriff Bot <webkit.review.bot@gmail.com>
+
+ Unreviewed, rolling out r92415.
+ http://trac.webkit.org/changeset/92415
+ https://bugs.webkit.org/show_bug.cgi?id=65728
+
+ Causes many DEBUG crashes (Requested by abarth on #webkit).
+
+ * platform/network/chromium/ResourceRequest.h:
+ * xml/XMLHttpRequest.cpp:
+ (WebCore::XMLHttpRequest::createRequest):
+
+2011-08-04 Tim Horton <timothy_horton@apple.com>
+
+ Reviewed by Nikolas Zimmerman.
+
+ Errors encountered within SVG documents should be reported to the console
+ https://bugs.webkit.org/show_bug.cgi?id=62599
+ <rdar://problem/9727074>
+
+ Make use of SVGLength::construct when parsing Length attributes; we will
+ now propagate errors which occur while parsing SVGLength attributes to the
+ Web Inspector console.
+
+ * svg/SVGCircleElement.cpp:
+ (WebCore::SVGCircleElement::parseMappedAttribute):
+ * svg/SVGCursorElement.cpp:
+ (WebCore::SVGCursorElement::parseMappedAttribute):
+ * svg/SVGEllipseElement.cpp:
+ (WebCore::SVGEllipseElement::parseMappedAttribute):
+ * svg/SVGFilterElement.cpp:
+ (WebCore::SVGFilterElement::parseMappedAttribute):
+ * svg/SVGFilterPrimitiveStandardAttributes.cpp:
+ (WebCore::SVGFilterPrimitiveStandardAttributes::parseMappedAttribute):
+ * svg/SVGForeignObjectElement.cpp:
+ (WebCore::SVGForeignObjectElement::parseMappedAttribute):
+ * svg/SVGImageElement.cpp:
+ (WebCore::SVGImageElement::parseMappedAttribute):
+ * svg/SVGLineElement.cpp:
+ (WebCore::SVGLineElement::parseMappedAttribute):
+ * svg/SVGLinearGradientElement.cpp:
+ (WebCore::SVGLinearGradientElement::parseMappedAttribute):
+ * svg/SVGMarkerElement.cpp:
+ (WebCore::SVGMarkerElement::parseMappedAttribute):
+ * svg/SVGMaskElement.cpp:
+ (WebCore::SVGMaskElement::parseMappedAttribute):
+ * svg/SVGPatternElement.cpp:
+ (WebCore::SVGPatternElement::parseMappedAttribute):
+ * svg/SVGRadialGradientElement.cpp:
+ (WebCore::SVGRadialGradientElement::parseMappedAttribute):
+ * svg/SVGRectElement.cpp:
+ (WebCore::SVGRectElement::parseMappedAttribute):
+ * svg/SVGSVGElement.cpp:
+ (WebCore::SVGSVGElement::parseMappedAttribute):
+ * svg/SVGTextContentElement.cpp:
+ (WebCore::SVGTextContentElement::parseMappedAttribute):
+ * svg/SVGTextPathElement.cpp:
+ (WebCore::SVGTextPathElement::parseMappedAttribute):
+ * svg/SVGUseElement.cpp:
+ (WebCore::SVGUseElement::parseMappedAttribute):
+
+2011-08-04 Sheriff Bot <webkit.review.bot@gmail.com>
+
+ Unreviewed, rolling out r92419.
+ http://trac.webkit.org/changeset/92419
+ https://bugs.webkit.org/show_bug.cgi?id=65726
+
+ "committed partially to branch" (Requested by thorton on
+ #webkit).
+
+ * svg/SVGCircleElement.cpp:
+ (WebCore::SVGCircleElement::parseMappedAttribute):
+ * svg/SVGCursorElement.cpp:
+ (WebCore::SVGCursorElement::parseMappedAttribute):
+ * svg/SVGEllipseElement.cpp:
+ (WebCore::SVGEllipseElement::parseMappedAttribute):
+ * svg/SVGFilterElement.cpp:
+ (WebCore::SVGFilterElement::parseMappedAttribute):
+ * svg/SVGFilterPrimitiveStandardAttributes.cpp:
+ (WebCore::SVGFilterPrimitiveStandardAttributes::parseMappedAttribute):
+ * svg/SVGForeignObjectElement.cpp:
+ (WebCore::SVGForeignObjectElement::parseMappedAttribute):
+ * svg/SVGImageElement.cpp:
+ (WebCore::SVGImageElement::parseMappedAttribute):
+ * svg/SVGLineElement.cpp:
+ (WebCore::SVGLineElement::parseMappedAttribute):
+ * svg/SVGLinearGradientElement.cpp:
+ (WebCore::SVGLinearGradientElement::parseMappedAttribute):
+ * svg/SVGMarkerElement.cpp:
+ (WebCore::SVGMarkerElement::parseMappedAttribute):
+ * svg/SVGMaskElement.cpp:
+ (WebCore::SVGMaskElement::parseMappedAttribute):
+ * svg/SVGPatternElement.cpp:
+ (WebCore::SVGPatternElement::parseMappedAttribute):
+ * svg/SVGRadialGradientElement.cpp:
+ (WebCore::SVGRadialGradientElement::parseMappedAttribute):
+ * svg/SVGRectElement.cpp:
+ (WebCore::SVGRectElement::parseMappedAttribute):
+ * svg/SVGSVGElement.cpp:
+ (WebCore::SVGSVGElement::parseMappedAttribute):
+ * svg/SVGTextContentElement.cpp:
+ (WebCore::SVGTextContentElement::parseMappedAttribute):
+ * svg/SVGTextPathElement.cpp:
+ (WebCore::SVGTextPathElement::parseMappedAttribute):
+ * svg/SVGUseElement.cpp:
+ (WebCore::SVGUseElement::parseMappedAttribute):
+
+2011-08-04 Fady Samuel <fsamuel@chromium.org>
+
+ Background Does Not Scale Correctly with Page
+ https://bugs.webkit.org/show_bug.cgi?id=65690
+
+ Reviewed by Simon Fraser.
+
+ Test: fast/repaint/background-scaling.html
+
+ * rendering/RenderBox.cpp:
+ (WebCore::RenderBox::paintRootBoxFillLayers):
+ * rendering/RenderView.cpp:
+ (WebCore::RenderView::unscaledDocumentRect):
+ (WebCore::RenderView::documentRect):
+ * rendering/RenderView.h:
+
+2011-08-04 Jochen Eisinger <jochen@chromium.org>
+
+ Introduce a new ResourceRequest::TargetType for XHRs
+ https://bugs.webkit.org/show_bug.cgi?id=65544
+
+ Reviewed by Darin Fisher.
+
+ * platform/network/chromium/ResourceRequest.h:
+ * xml/XMLHttpRequest.cpp:
+ (WebCore::XMLHttpRequest::createRequest):
+
+2011-08-04 Sergey Glazunov <serg.glazunov@gmail.com>
+
+ Fix integer overflow in custom bindings for WebGLRenderingContext
+ https://bugs.webkit.org/show_bug.cgi?id=65646
+
+ Reviewed by Kenneth Russell.
+
+ Test: fast/canvas/webgl/uniform-array-length-overflow.html
+
+ * bindings/js/JSWebGLRenderingContextCustom.cpp:
+ (WebCore::toVector): Don't crash if allocation fails.
+ * bindings/v8/custom/V8WebGLRenderingContextCustom.cpp:
+ (WebCore::jsArrayToFloatArray):
+ (WebCore::jsArrayToIntArray):
+
+2011-08-04 Pratik Solanki <psolanki@apple.com>
+
+ Implement CERTIFICATE_CREDENTIALS_SUPPORTED for CFNetwork
+ https://bugs.webkit.org/show_bug.cgi?id=65704
+ <rdar://problem/9877734>
+
+ Reviewed by Oliver Hunt.
+
+ Implement support for client side certificate authentication in the CFNetwork based loader.
+
+ * platform/network/cf/AuthenticationCF.cpp:
+ (WebCore::createCF):
+ (WebCore::core): Also fix leak of password string.
+
+2011-08-04 Chris Rogers <crogers@google.com>
+
+ Make sure that AudioArray is 16-byte aligned
+ https://bugs.webkit.org/show_bug.cgi?id=65651
+
+ Reviewed by Kenneth Russell.
+
+ No new tests - this does not change JavaScript API.
+
+ * platform/audio/AudioArray.h:
+ (WebCore::AudioArray::AudioArray):
+ (WebCore::AudioArray::~AudioArray):
+ (WebCore::AudioArray::allocate):
+ (WebCore::AudioArray::data):
+ (WebCore::AudioArray::size):
+ (WebCore::AudioArray::at):
+ (WebCore::AudioArray::operator[]):
+ (WebCore::AudioArray::alignedAddress):
+ * platform/audio/Biquad.cpp:
+ (WebCore::Biquad::Biquad):
+ * platform/audio/ReverbConvolverStage.cpp:
+ (WebCore::ReverbConvolverStage::ReverbConvolverStage):
+ * webaudio/DelayDSPKernel.cpp:
+ (WebCore::DelayDSPKernel::DelayDSPKernel):
+ * webaudio/RealtimeAnalyser.cpp:
+ (WebCore::RealtimeAnalyser::setFftSize):
+
+2011-08-04 Brady Eidson <beidson@apple.com>
+
+ <rdar://problem/9882581>, <rdar://problem/9868015>, and https://bugs.webkit.org/show_bug.cgi?id=65712
+ REGRESSION (91931) - Two LocalStorage threads started, thread unsafe operations can cause crash or other problems later.
+
+ Reviewed by Darin Adler.
+
+ * storage/StorageTracker.cpp:
+ (WebCore::StorageTracker::internalInitialize): Set the "needs initialization" flag before doing a
+ whole bunch of stuff that might need to check it.
+
+2011-08-04 Jeff Miller <jeffm@apple.com>
+
+ Adopt AVCF media back end on Windows
+ https://bugs.webkit.org/show_bug.cgi?id=65400
+ <rdar://problem/9894105>
+
+ First cut at implementation in MediaPlayerPrivateAVFoundationCF.cpp/.h, based on
+ work by Eric Carlson. Note that use of AVFoundation is determined at runtime, and
+ defaults to off.
+
+ Reviewed by Darin Adler.
+
+ No new tests, uses existing media tests.
+
+ * config.h: Turn on WTF_USE_AVFOUNDATION on Windows if AVFoundationCF is available.
+
+ * platform/graphics/MediaPlayer.cpp:
+ (WebCore::installedMediaEngines): Register MediaPlayerPrivateAVFoundationCF on Windows.
+ * platform/graphics/MediaPlayer.h: Add support for an AVCFPlayer.
+
+ * platform/graphics/avfoundation/MediaPlayerPrivateAVFoundation.cpp:
+ (WebCore::MediaPlayerPrivateAVFoundation::setPreload): Workaround an AVCF limitation that prevents an AVCFPlayer from being created without an AVCFItem.
+ (WebCore::MediaPlayerPrivateAVFoundation::dispatchNotification): Added support for ContentsNeedsDisplay notification.
+
+ * platform/graphics/avfoundation/MediaPlayerPrivateAVFoundation.h:
+ (WebCore::MediaPlayerPrivateAVFoundation::contentsNeedsDisplay): Added support for ContentsNeedsDisplay notification.
+
+ * platform/graphics/avfoundation/MediaPlayerPrivateAVFoundationCF.cpp: Added implementation.
+ (WebCore::AVFWrapper::caVideoLayer):
+ (WebCore::AVFWrapper::videoLayerWrapper):
+ (WebCore::AVFWrapper::videoLayer):
+ (WebCore::AVFWrapper::avPlayer):
+ (WebCore::AVFWrapper::avAsset):
+ (WebCore::AVFWrapper::avPlayerItem):
+ (WebCore::AVFWrapper::timeObserver):
+ (WebCore::AVFWrapper::imageGenerator):
+ (WebCore::AVFWrapper::dispatchQueue):
+ (WebCore::LayerClient::LayerClient):
+ (WebCore::LayerClient::~LayerClient):
+ (WebCore::LayerClient::platformCALayerRespondsToLayoutChanges):
+ (WebCore::LayerClient::platformCALayerAnimationStarted):
+ (WebCore::LayerClient::platformCALayerContentsOrientation):
+ (WebCore::LayerClient::platformCALayerPaintContents):
+ (WebCore::LayerClient::platformCALayerShowDebugBorders):
+ (WebCore::LayerClient::platformCALayerShowRepaintCounter):
+ (WebCore::LayerClient::platformCALayerIncrementRepaintCount):
+ (WebCore::LayerClient::platformCALayerContentsOpaque):
+ (WebCore::LayerClient::platformCALayerDrawsContent):
+ (WebCore::LayerClient::platformCALayerLayerDidDisplay):
+ (WebCore::boolString):
+ (WebCore::metadataKeyNames):
+ (WebCore::CMTimeRangeStartKey):
+ (WebCore::CMTimeRangeDurationKey):
+ (WebCore::CACFContextNeedsFlushNotification):
+ (WebCore::videoLayer):
+ (WebCore::avPlayer):
+ (WebCore::avAsset):
+ (WebCore::avPlayerItem):
+ (WebCore::imageGenerator):
+ (WebCore::MediaPlayerPrivateAVFoundationCF::create):
+ (WebCore::MediaPlayerPrivateAVFoundationCF::registerMediaEngine):
+ (WebCore::MediaPlayerPrivateAVFoundationCF::MediaPlayerPrivateAVFoundationCF):
+ (WebCore::MediaPlayerPrivateAVFoundationCF::~MediaPlayerPrivateAVFoundationCF):
+ (WebCore::MediaPlayerPrivateAVFoundationCF::cancelLoad):
+ (WebCore::MediaPlayerPrivateAVFoundationCF::hasLayerRenderer):
+ (WebCore::MediaPlayerPrivateAVFoundationCF::hasContextRenderer):
+ (WebCore::MediaPlayerPrivateAVFoundationCF::createContextVideoRenderer):
+ (WebCore::MediaPlayerPrivateAVFoundationCF::destroyContextVideoRenderer):
+ (WebCore::MediaPlayerPrivateAVFoundationCF::createVideoLayer):
+ (WebCore::MediaPlayerPrivateAVFoundationCF::destroyVideoLayer):
+ (WebCore::MediaPlayerPrivateAVFoundationCF::hasAvailableVideoFrame):
+ (WebCore::MediaPlayerPrivateAVFoundationCF::createAVPlayer):
+ (WebCore::MediaPlayerPrivateAVFoundationCF::createAVPlayerItem):
+ (WebCore::MediaPlayerPrivateAVFoundationCF::createAVAssetForURL):
+ (WebCore::MediaPlayerPrivateAVFoundationCF::checkPlayability):
+ (WebCore::MediaPlayerPrivateAVFoundationCF::beginLoadingMetadata):
+ (WebCore::MediaPlayerPrivateAVFoundationCF::playerItemStatus):
+ (WebCore::MediaPlayerPrivateAVFoundationCF::platformMedia):
+ (WebCore::MediaPlayerPrivateAVFoundationCF::platformLayer):
+ (WebCore::MediaPlayerPrivateAVFoundationCF::platformSetVisible):
+ (WebCore::MediaPlayerPrivateAVFoundationCF::platformPlay):
+ (WebCore::MediaPlayerPrivateAVFoundationCF::platformPause):
+ (WebCore::MediaPlayerPrivateAVFoundationCF::updateRate):
+ (WebCore::MediaPlayerPrivateAVFoundationCF::platformDuration):
+ (WebCore::MediaPlayerPrivateAVFoundationCF::currentTime):
+ (WebCore::MediaPlayerPrivateAVFoundationCF::seekToTime):
+ (WebCore::MediaPlayerPrivateAVFoundationCF::setVolume):
+ (WebCore::MediaPlayerPrivateAVFoundationCF::setClosedCaptionsVisible):
+ (WebCore::MediaPlayerPrivateAVFoundationCF::rate):
+ (WebCore::timeRangeIsValidAndNotEmpty):
+ (WebCore::MediaPlayerPrivateAVFoundationCF::platformBufferedTimeRanges):
+ (WebCore::MediaPlayerPrivateAVFoundationCF::platformMaxTimeSeekable):
+ (WebCore::MediaPlayerPrivateAVFoundationCF::platformMaxTimeLoaded):
+ (WebCore::MediaPlayerPrivateAVFoundationCF::totalBytes):
+ (WebCore::MediaPlayerPrivateAVFoundationCF::assetStatus):
+ (WebCore::MediaPlayerPrivateAVFoundationCF::paintCurrentFrameInContext):
+ (WebCore::MediaPlayerPrivateAVFoundationCF::paint):
+ (WebCore::mimeTypeCache):
+ (WebCore::MediaPlayerPrivateAVFoundationCF::getSupportedTypes):
+ (WebCore::MediaPlayerPrivateAVFoundationCF::supportsType):
+ (WebCore::MediaPlayerPrivateAVFoundationCF::isAvailable):
+ (WebCore::MediaPlayerPrivateAVFoundationCF::mediaTimeForTimeValue):
+ (WebCore::MediaPlayerPrivateAVFoundationCF::tracksChanged):
+ (WebCore::MediaPlayerPrivateAVFoundationCF::sizeChanged):
+ (WebCore::MediaPlayerPrivateAVFoundationCF::contentsNeedsDisplay):
+ (WebCore::AVFWrapper::AVFWrapper):
+ (WebCore::AVFWrapper::~AVFWrapper):
+ (WebCore::AVFWrapper::scheduleDisconnectAndDelete):
+ (WebCore::AVFWrapper::disconnectAndDeleteAVFWrapper):
+ (WebCore::AVFWrapper::deleteAVFWrapper):
+ (WebCore::AVFWrapper::createAssetForURL):
+ (WebCore::AVFWrapper::createPlayer):
+ (WebCore::AVFWrapper::createPlayerItem):
+ (WebCore::AVFWrapper::periodicTimeObserverCallback):
+ (WebCore::AVFWrapper::notificationCallback):
+ (WebCore::AVFWrapper::loadPlayableCompletionCallback):
+ (WebCore::AVFWrapper::checkPlayability):
+ (WebCore::AVFWrapper::loadMetadataCompletionCallback):
+ (WebCore::AVFWrapper::beginLoadingMetadata):
+ (WebCore::AVFWrapper::seekCompletedCallback):
+ (WebCore::AVFWrapper::seekToTime):
+ (WebCore::AVFWrapper::setAsset):
+ (WebCore::AVFWrapper::platformLayer):
+ (WebCore::AVFWrapper::createAVCFVideoLayer):
+ (WebCore::AVFWrapper::destroyVideoLayer):
+ (WebCore::AVFWrapper::setVideoLayerNeedsCommit):
+ (WebCore::AVFWrapper::setVideoLayerHidden):
+ (WebCore::AVFWrapper::createImageGenerator):
+ (WebCore::AVFWrapper::destroyImageGenerator):
+ (WebCore::AVFWrapper::createImageForTimeInRect):
+ (WebCore::LayerClient::platformCALayerLayoutSublayersOfLayer):
+
+ * platform/graphics/avfoundation/MediaPlayerPrivateAVFoundationCF.h: Added implementation.
+
+2011-08-04 Jochen Eisinger <jochen@chromium.org>
+
+ Get rid of ResourceRequestBase::m_targetType. Move it to chromium's ResourceRequest.
+ https://bugs.webkit.org/show_bug.cgi?id=48483
+
+ Reviewed by Alexey Proskuryakov.
+
+ * loader/PingLoader.cpp:
+ (WebCore::PingLoader::loadImage):
+ (WebCore::PingLoader::sendPing):
+ (WebCore::PingLoader::reportContentSecurityPolicyViolation):
+ * loader/cache/CachedResourceRequest.cpp:
+ (WebCore::CachedResourceRequest::load):
+ * platform/graphics/gstreamer/WebKitWebSourceGStreamer.cpp:
+ (webKitWebSrcStart):
+ * platform/network/ResourceRequestBase.cpp:
+ (WebCore::ResourceRequestBase::adopt):
+ (WebCore::ResourceRequestBase::copyData):
+ * platform/network/ResourceRequestBase.h:
+ (WebCore::ResourceRequestBase::ResourceRequestBase):
+ * platform/network/chromium/ResourceRequest.cpp:
+ (WebCore::ResourceRequest::doPlatformCopyData):
+ (WebCore::ResourceRequest::doPlatformAdopt):
+ * platform/network/chromium/ResourceRequest.h:
+ (WebCore::ResourceRequest::ResourceRequest):
+ (WebCore::ResourceRequest::targetType):
+ (WebCore::ResourceRequest::setTargetType):
+ * workers/DefaultSharedWorkerRepository.cpp:
+ (WebCore::SharedWorkerScriptLoader::load):
+ * workers/Worker.cpp:
+ (WebCore::Worker::create):
+ * workers/WorkerContext.cpp:
+ (WebCore::WorkerContext::importScripts):
+ * workers/WorkerScriptLoader.cpp:
+ (WebCore::WorkerScriptLoader::WorkerScriptLoader):
+ (WebCore::WorkerScriptLoader::createResourceRequest):
+ * workers/WorkerScriptLoader.h:
+ (WebCore::WorkerScriptLoader::create):
+ (WebCore::WorkerScriptLoader::setTargetType):
+
+2011-08-04 Jeff Miller <jeffm@apple.com>
+
+ Fix WinCE build after r92308.
+
+ * config.h: Don't try to include WebCoreHeaderDetection.h for WinCE.
+
+2011-08-04 Darin Fisher <darin@chromium.org>
+
+ webkitRequestAnimationFrame's element argument needs to be marked optional
+ https://bugs.webkit.org/show_bug.cgi?id=65698
+
+ Reviewed by James Robinson.
+
+ * page/DOMWindow.idl:
+
+2011-08-04 Adam Roben <aroben@apple.com>
+
+ Delete some unused code from platform/graphics/win
+
+ MediaPlayerPrivateQuickTimeWin has been unused since r72117, and the GraphicsLayer-related
+ classes have been unused since r75262.
+
+ Fixes <http://webkit.org/b/65689> platform/graphics/win has a bunch of unused code
+
+ Reviewed by Eric Carlson.
+
+ * WebCore.gypi:
+ * WebCore.vcproj/WebCore.vcproj:
+ Removed now-deleted files.
+
+ * platform/graphics/win/GraphicsLayerCACF.cpp: Removed.
+ * platform/graphics/win/GraphicsLayerCACF.h: Removed.
+ * platform/graphics/win/MediaPlayerPrivateQuickTimeWin.cpp: Removed.
+ * platform/graphics/win/MediaPlayerPrivateQuickTimeWin.h: Removed.
+ * platform/graphics/win/WebLayer.cpp: Removed.
+ * platform/graphics/win/WebLayer.h: Removed.
+ * platform/graphics/win/WebTiledLayer.cpp: Removed.
+ * platform/graphics/win/WebTiledLayer.h: Removed.
+
+2011-08-03 Adam Roben <aroben@apple.com>
+
+ Detect and handle overflow in PlatformCALayerWinInternal::constrainedSize
+
+ Google Maps sometimes requests very large (i.e., 2^50 pixels or greater) layers when
+ zooming. PlatformCALayerWinInternal has code to limit tiled layers to 2^27 pixels, but it
+ was not correctly handling overflow. In some cases, this would lead to creating a tiled
+ layer with 0 tiles, which was the cause of this crash.
+
+ Fixes <http://webkit.org/b/65637> <rdar://problem/9784849> Crash beneath
+ PlatformCALayerWinInternal::updateTiles when zooming on Google Maps
+
+ Reviewed by Sam Weinig.
+
+ Test: compositing/tiling/crash-huge-layer.html
+
+ * platform/graphics/ca/win/PlatformCALayerWinInternal.cpp:
+ (PlatformCALayerWinInternal::constrainedSize): Check for overflow before seeing if the
+ number of required tiles is larger than the maximum number of allowed tiles.
+ (PlatformCALayerWinInternal::updateTiles): Added an assertion to catch cases where we have a
+ non-empty tiled layer that contains 0 tiles, which would cause the crash in this bug report.
+
+2011-08-04 Stephen White <senorblanco@chromium.org>
+
+ Set graphics context current before canvas.toDataURL().
+ https://bugs.webkit.org/show_bug.cgi?id=65700
+
+ Reviewed by James Robinson.
+
+ No new tests, unfortunately. The test infrastucture doesn't seem
+ to be conducive to writing multiple-context tests.
+
+ * platform/graphics/skia/ImageBufferSkia.cpp:
+ (WebCore::ImageBuffer::toDataURL):
+
+2011-08-01 Brian Weinstein <bweinstein@apple.com>
+
+ WebKit2: Web Inspector always starts in undocked mode
+ https://bugs.webkit.org/show_bug.cgi?id=65493
+ <rdar://problem/9353114>
+
+ Reviewed by Adam Roben.
+
+ Expose some methods on InspectorFrontendClient through the InspectorController so WebKit2
+ can request the inspector be docked, or ask whether or not it can be docked.
+
+ * WebCore.exp.in: Export needed methods.
+ * inspector/InspectorController.cpp:
+ (WebCore::InspectorController::requestAttachWindow): Call through to InspectorFrontendClient.
+ (WebCore::InspectorController::canAttachWindow): Ditto.
+ * inspector/InspectorController.h:
+ * inspector/InspectorFrontendClient.h:
+ * inspector/InspectorFrontendClientLocal.h:
+
+2011-08-04 Luke Macpherson <macpherson@chromium.org>
+
+ Support cast between CSSPrimitiveValue and EborderFit, use in CSSStyleSelector.
+ https://bugs.webkit.org/show_bug.cgi?id=65665
+
+ Reviewed by Simon Fraser.
+
+ No new tests / refactoring only.
+
+ * css/CSSPrimitiveValueMappings.h:
+ (WebCore::CSSPrimitiveValue::CSSPrimitiveValue):
+ Support cast from EBorderFit.
+ (WebCore::CSSPrimitiveValue::operator EBorderFit):
+ Support cast to EBorderFit.
+ * css/CSSStyleSelector.cpp:
+ (WebCore::CSSStyleSelector::applyProperty):
+ Use new cast to allow use of appropriate macro.
+
+2011-08-04 Pavel Feldman <pfeldman@google.com>
+
+ Web Inspector: rename sendResultByValue to returnByValue in Runtime agent.
+ https://bugs.webkit.org/show_bug.cgi?id=65687
+
+ Reviewed by Yury Semikhatsky.
+
+ * inspector/InjectedScript.cpp:
+ (WebCore::InjectedScript::evaluate):
+ (WebCore::InjectedScript::callFunctionOn):
+ * inspector/InjectedScript.h:
+ * inspector/InjectedScriptSource.js:
+ * inspector/Inspector.json:
+ * inspector/InspectorRuntimeAgent.cpp:
+ (WebCore::InspectorRuntimeAgent::evaluate):
+ (WebCore::InspectorRuntimeAgent::callFunctionOn):
+ * inspector/InspectorRuntimeAgent.h:
+
+2011-08-04 Matt Falkenhagen <falken@chromium.org>
+
+ Use -webkit-locale for font selection.
+ https://bugs.webkit.org/show_bug.cgi?id=10874
+
+ Reviewed by Dan Bernstein.
+
+ Infer a script from -webkit-locale to use to choose a font for generic
+ font families and the default unstyled font. The font is retrieved
+ from per-script font settings (see bug 20797). Since these settings
+ have not changed there should be no visible effect yet.
+
+ * CMakeLists.txt:
+ * GNUmakefile.list.am:
+ * WebCore.gyp/WebCore.gyp:
+ * WebCore.gypi:
+ * WebCore.pro:
+ * WebCore.vcproj/WebCore.vcproj:
+ * WebCore.xcodeproj/project.pbxproj:
+ * css/CSSFontSelector.cpp: Use script when getting font from Settings
+ (WebCore::fontDataForGenericFamily):
+ (WebCore::CSSFontSelector::getFontData):
+ * css/CSSPropertyNames.in: Increase priority of -webkit-locale since it affects font
+ * css/CSSStyleSelector.cpp:
+ (WebCore::CSSStyleSelector::styleForDocument):
+ (WebCore::CSSStyleSelector::applyDeclarations):
+ (WebCore::CSSStyleSelector::applyProperty): Set script in font based on -webkit-locale
+ * page/Settings.cpp:
+ (WebCore::getGenericFontFamilyForScript): Fallback to USCRIPT_COMMON
+ * platform/graphics/FontDescription.h: Add m_script
+ (WebCore::FontDescription::FontDescription):
+ (WebCore::FontDescription::script):
+ (WebCore::FontDescription::setScript):
+ (WebCore::FontDescription::operator==):
+ * platform/text/LocaleToScriptMapping.h: Added.
+ * platform/text/LocaleToScriptMappingDefault.cpp: Added.
+ (WebCore::localeToScriptCodeForFontSelection):
+ (WebCore::LocaleScript::if):
+ (WebCore::LocaleScript::while):
+ * platform/text/LocaleToScriptMappingICU.cpp: Added.
+ (WebCore::scriptCodeForFontSelection):
+ (WebCore::localeToScriptCodeForFontSelection):
+
+2011-08-04 Vsevolod Vlasov <vsevik@chromium.org>
+
+ Web Inspector: Pretty print JSONP in network panel preview tab.
+ https://bugs.webkit.org/show_bug.cgi?id=65559
+
+ Reviewed by Pavel Feldman.
+
+ Test: http/tests/inspector/network/network-preview-json.html
+
+ * inspector/front-end/ResourceJSONView.js:
+ (WebInspector.ResourceJSONView.parseJSON.WebInspector.ResourceJSONView.parseJSONP):
+ (WebInspector.ResourceJSONView.parseJSON.WebInspector.ResourceJSONView.prototype._initialize):
+ (WebInspector.ResourceJSONView.parseJSON.WebInspector.ParsedJSON):
+ * inspector/front-end/ResourcePreviewView.js:
+ (WebInspector.ResourcePreviewView.prototype._createPreviewView):
+
+2011-08-04 Pavel Feldman <pfeldman@google.com>
+
+ Web Inspector: replace isRegex with urlRegex in setBreakpointByUrl
+ https://bugs.webkit.org/show_bug.cgi?id=65684
+
+ Reviewed by Yury Semikhatsky.
+
+ * inspector/Inspector.json:
+ * inspector/InspectorDebuggerAgent.cpp:
+ (WebCore::InspectorDebuggerAgent::setBreakpointByUrl):
+ * inspector/InspectorDebuggerAgent.h:
+ * inspector/front-end/DebuggerModel.js:
+ (WebInspector.DebuggerModel.prototype.setBreakpoint):
+
+2011-08-03 Philippe Normand <pnormand@igalia.com>
+
+ [GTK] fullscreen/video-controls-override.html fails
+ https://bugs.webkit.org/show_bug.cgi?id=65618
+
+ Reviewed by Martin Robinson.
+
+ Implement extraFullScreenStyleSheet() and provide the QuickTime
+ stylesheet for now, later we might want our own stylesheet.
+
+ * GNUmakefile.am:
+ * platform/gtk/RenderThemeGtk.cpp:
+ (WebCore::RenderThemeGtk::extraFullScreenStyleSheet):
+ * platform/gtk/RenderThemeGtk.h:
+
+2011-08-04 Pavel Feldman <pfeldman@google.com>
+
+ Web Inspector: Rename sourceId -> scriptId in the protocol and the source code.
+ https://bugs.webkit.org/show_bug.cgi?id=65682
+
+ Reviewed by Yury Semikhatsky.
+
+ * inspector/InjectedScriptSource.js:
+ ():
+ * inspector/Inspector.json:
+ * inspector/InspectorConsoleAgent.cpp:
+ (WebCore::InspectorConsoleAgent::addMessageToConsole):
+ * inspector/InspectorConsoleAgent.h:
+ * inspector/InspectorConsoleInstrumentation.h:
+ (WebCore::InspectorInstrumentation::addMessageToConsole):
+ * inspector/InspectorDOMAgent.cpp:
+ (WebCore::InspectorDOMAgent::buildObjectForEventListener):
+ * inspector/InspectorDebuggerAgent.cpp:
+ (WebCore::parseLocation):
+ (WebCore::InspectorDebuggerAgent::setBreakpoint):
+ (WebCore::InspectorDebuggerAgent::continueToLocation):
+ (WebCore::InspectorDebuggerAgent::resolveBreakpoint):
+ (WebCore::InspectorDebuggerAgent::setScriptSource):
+ (WebCore::InspectorDebuggerAgent::getScriptSource):
+ (WebCore::InspectorDebuggerAgent::didParseSource):
+ * inspector/InspectorDebuggerAgent.h:
+ * inspector/InspectorInstrumentation.cpp:
+ (WebCore::InspectorInstrumentation::addMessageToConsoleImpl):
+ * inspector/InspectorInstrumentation.h:
+ * inspector/ScriptDebugListener.h:
+ * inspector/front-end/DebuggerModel.js:
+ (WebInspector.DebuggerModel.prototype.scriptForSourceID):
+ (WebInspector.DebuggerModel.prototype.queryScripts):
+ (WebInspector.DebuggerModel.prototype.setScriptSource):
+ (WebInspector.DebuggerModel.prototype._didEditScriptSource):
+ (WebInspector.DebuggerModel.prototype._parsedScriptSource):
+ (WebInspector.DebuggerDispatcher.prototype.scriptParsed):
+ (WebInspector.DebuggerDispatcher.prototype.breakpointResolved):
+ * inspector/front-end/DebuggerPresentationModel.js:
+ (WebInspector.DebuggerPresentationModel.prototype.registerAnchor):
+ (WebInspector.DebuggerPresentationModel.prototype._updateAnchor):
+ (WebInspector.DebuggerPresentationModel.prototype.setScriptSource.didReceiveSource):
+ (WebInspector.DebuggerPresentationModel.prototype.setScriptSource):
+ (WebInspector.DebuggerPresentationModel.prototype._setBreakpointInDebugger.didGetScriptLocation):
+ (WebInspector.DebuggerPresentationModel.prototype._setBreakpointInDebugger):
+ (WebInspector.DebuggerPresentationModel.prototype._breakpointAdded):
+ (WebInspector.DebuggerPresentationModel.prototype.set _debuggerPaused):
+ (WebInspector.DebuggerPresentationModel.prototype._sourceFileForScript):
+ (WebInspector.DebuggerPresentationModel.prototype._scriptForSourceFileId):
+ (WebInspector.DebuggerPresentationModel.prototype._createSourceFileId):
+ (WebInspector.PresenationCallFrame):
+ (WebInspector.PresenationCallFrame.prototype.sourceLine):
+ * inspector/front-end/EventListenersSidebarPane.js:
+ (.):
+ ():
+ * inspector/front-end/Script.js:
+ (WebInspector.Script):
+ (WebInspector.Script.prototype.requestSource):
+ (WebInspector.Script.prototype.editSource):
+ * inspector/front-end/SourceFile.js:
+ (WebInspector.RawSourceCode.prototype.uiLocationToRawLocation):
+ (WebInspector.RawSourceCode.prototype.forceLoadContent):
+ (WebInspector.RawSourceCode.prototype._concatenateScriptsContent):
+
+2011-08-04 Tommy Widenflycht <tommyw@google.com>
+
+ MediaStream API: Rename StreamEvent to MediaStreamEvent
+ https://bugs.webkit.org/show_bug.cgi?id=65669
+
+ Reviewed by Tony Gentilcore.
+
+ * CMakeLists.txt:
+ * CodeGenerators.pri:
+ * DerivedSources.cpp:
+ * DerivedSources.make:
+ * GNUmakefile.list.am:
+ * WebCore.gypi:
+ * WebCore.pro:
+ * WebCore.vcproj/WebCore.vcproj:
+ * WebCore.xcodeproj/project.pbxproj:
+ * bindings/js/JSEventCustom.cpp:
+ (WebCore::toJS):
+ * bindings/v8/custom/V8EventCustom.cpp:
+ (WebCore::toV8):
+ * dom/DOMAllInOne.cpp:
+ * dom/Event.cpp:
+ (WebCore::Event::isMediaStreamEvent):
+ * dom/Event.h:
+ * p2p/MediaStreamEvent.cpp: Renamed from Source/WebCore/dom/StreamEvent.cpp.
+ (WebCore::MediaStreamEvent::create):
+ (WebCore::MediaStreamEvent::MediaStreamEvent):
+ (WebCore::MediaStreamEvent::~MediaStreamEvent):
+ (WebCore::MediaStreamEvent::initMediaStreamEvent):
+ (WebCore::MediaStreamEvent::stream):
+ * p2p/MediaStreamEvent.h: Renamed from Source/WebCore/dom/StreamEvent.h.
+ (WebCore::MediaStreamEvent::isMediaStreamEvent):
+ * p2p/MediaStreamEvent.idl: Renamed from Source/WebCore/dom/StreamEvent.idl.
+ * p2p/PeerConnection.cpp:
+ (WebCore::PeerConnection::dispatchStreamEvent):
+
+2011-08-02 Hans Wennborg <hans@chromium.org>
+
+ IndexedDB: Fix index data invalidation bugs.
+ https://bugs.webkit.org/show_bug.cgi?id=65547
+
+ Reviewed by Tony Chang.
+
+ The function that checks whether a key exists in an index failed
+ to check whether that key was still valid or not.
+
+ Deleting a record from an object store must also delete its exists
+ entry, thus invalidating index keys pointing to that record.
+
+ Test: storage/indexeddb/index-unique.html
+
+ * storage/IDBLevelDBBackingStore.cpp:
+ (WebCore::IDBLevelDBBackingStore::deleteObjectStoreRecord):
+ (WebCore::findKeyInIndex):
+ (WebCore::IDBLevelDBBackingStore::getPrimaryKeyViaIndex):
+ (WebCore::IDBLevelDBBackingStore::keyExistsInIndex):
+ (WebCore::CursorOptions::IndexKeyCursorImpl::loadCurrentRow):
+
+2011-08-04 Alexandru Chiculita <achicu@adobe.com>
+
+ Move PODIntervalTree to Source/WebCore/platform
+ https://bugs.webkit.org/show_bug.cgi?id=65667
+
+ It seems the files were not part of any project, so no project files were modified.
+
+ Reviewed by Adam Barth.
+
+ No new tests, just moving files from one folder to the other.
+
+ * platform/PODArena.h: Renamed from Source/WebCore/platform/graphics/gpu/PODArena.h.
+ (WebCore::PODArena::Allocator::~Allocator):
+ (WebCore::PODArena::FastMallocAllocator::create):
+ (WebCore::PODArena::FastMallocAllocator::allocate):
+ (WebCore::PODArena::FastMallocAllocator::free):
+ (WebCore::PODArena::FastMallocAllocator::FastMallocAllocator):
+ (WebCore::PODArena::create):
+ (WebCore::PODArena::allocateObject):
+ (WebCore::PODArena::~PODArena):
+ (WebCore::PODArena::PODArena):
+ (WebCore::PODArena::minAlignment):
+ (WebCore::PODArena::allocateBase):
+ (WebCore::PODArena::roundUp):
+ (WebCore::PODArena::Chunk::Chunk):
+ (WebCore::PODArena::Chunk::~Chunk):
+ (WebCore::PODArena::Chunk::allocate):
+ * platform/PODInterval.h: Renamed from Source/WebCore/platform/graphics/gpu/PODInterval.h.
+ (WebCore::PODInterval::PODInterval):
+ (WebCore::PODInterval::low):
+ (WebCore::PODInterval::high):
+ (WebCore::PODInterval::data):
+ (WebCore::PODInterval::overlaps):
+ (WebCore::PODInterval::operator<):
+ (WebCore::PODInterval::operator==):
+ (WebCore::PODInterval::maxHigh):
+ (WebCore::PODInterval::setMaxHigh):
+ (WebCore::PODInterval::toString):
+ * platform/PODIntervalTree.h: Renamed from Source/WebCore/platform/graphics/gpu/PODIntervalTree.h.
+ (WebCore::PODIntervalTree::PODIntervalTree):
+ (WebCore::PODIntervalTree::allOverlaps):
+ (WebCore::PODIntervalTree::createInterval):
+ (WebCore::PODIntervalTree::checkInvariants):
+ (WebCore::PODIntervalTree::init):
+ (WebCore::PODIntervalTree::searchForOverlapsFrom):
+ (WebCore::PODIntervalTree::updateNode):
+ (WebCore::PODIntervalTree::checkInvariantsFromNode):
+ * platform/PODRedBlackTree.h: Renamed from Source/WebCore/platform/graphics/gpu/PODRedBlackTree.h.
+ (WebCore::PODRedBlackTree::Visitor::~Visitor):
+ (WebCore::PODRedBlackTree::PODRedBlackTree):
+ (WebCore::PODRedBlackTree::~PODRedBlackTree):
+ (WebCore::PODRedBlackTree::add):
+ (WebCore::PODRedBlackTree::remove):
+ (WebCore::PODRedBlackTree::contains):
+ (WebCore::PODRedBlackTree::visitInorder):
+ (WebCore::PODRedBlackTree::size):
+ (WebCore::PODRedBlackTree::setNeedsFullOrderingComparisons):
+ (WebCore::PODRedBlackTree::checkInvariants):
+ (WebCore::PODRedBlackTree::dump):
+ (WebCore::PODRedBlackTree::setVerboseDebugging):
+ (WebCore::PODRedBlackTree::Node::Node):
+ (WebCore::PODRedBlackTree::Node::~Node):
+ (WebCore::PODRedBlackTree::Node::color):
+ (WebCore::PODRedBlackTree::Node::setColor):
+ (WebCore::PODRedBlackTree::Node::data):
+ (WebCore::PODRedBlackTree::Node::copyFrom):
+ (WebCore::PODRedBlackTree::Node::left):
+ (WebCore::PODRedBlackTree::Node::setLeft):
+ (WebCore::PODRedBlackTree::Node::right):
+ (WebCore::PODRedBlackTree::Node::setRight):
+ (WebCore::PODRedBlackTree::Node::parent):
+ (WebCore::PODRedBlackTree::Node::setParent):
+ (WebCore::PODRedBlackTree::root):
+ (WebCore::PODRedBlackTree::updateNode):
+ (WebCore::PODRedBlackTree::treeSearch):
+ (WebCore::PODRedBlackTree::treeSearchNormal):
+ (WebCore::PODRedBlackTree::treeSearchFullComparisons):
+ (WebCore::PODRedBlackTree::treeInsert):
+ (WebCore::PODRedBlackTree::treeSuccessor):
+ (WebCore::PODRedBlackTree::treeMinimum):
+ (WebCore::PODRedBlackTree::propagateUpdates):
+ (WebCore::PODRedBlackTree::leftRotate):
+ (WebCore::PODRedBlackTree::rightRotate):
+ (WebCore::PODRedBlackTree::insertNode):
+ (WebCore::PODRedBlackTree::deleteFixup):
+ (WebCore::PODRedBlackTree::deleteNode):
+ (WebCore::PODRedBlackTree::visitInorderImpl):
+ (WebCore::PODRedBlackTree::Counter::Counter):
+ (WebCore::PODRedBlackTree::Counter::visit):
+ (WebCore::PODRedBlackTree::Counter::count):
+ (WebCore::PODRedBlackTree::checkInvariantsFromNode):
+ (WebCore::PODRedBlackTree::logIfVerbose):
+ (WebCore::PODRedBlackTree::dumpFromNode):
+
+2011-08-03 Alice Boxhall <aboxhall@chromium.org>
+
+ An element with role=textbox should have settable AXValue unless read-only
+ https://bugs.webkit.org/show_bug.cgi?id=65664
+
+ Reviewed by Chris Fleizach.
+
+ * accessibility/AccessibilityRenderObject.cpp:
+ (WebCore::AccessibilityRenderObject::canSetValueAttribute):
+ Return true for non-native text field with aria-readonly not set.
+
+2011-08-03 Luke Macpherson <macpherson@chromium.org>
+
+ Clean up value clamping in CSSStyleSelector.
+ https://bugs.webkit.org/show_bug.cgi?id=65588
+
+ Reviewed by Darin Adler.
+
+ No new tests / trivial code cleanup only.
+
+ * css/CSSStyleSelector.cpp:
+ (WebCore::CSSStyleSelector::applyProperty):
+ Replace (unsigned int)primitiveValue->getDoubleValue() with primtiveValue->getValue<unsigned>() for correct clamping.
+ Replace clampToInteger(primitiveValue->getDoubleValue()) with primitiveValue->getIntValue().
+
+2011-08-03 Kent Tamura <tkent@chromium.org>
+
+ Fix incorrect checks for HTMLMediaElement
+ https://bugs.webkit.org/show_bug.cgi?id=65590
+
+ Reviewed by Dimitri Glazkov.
+
+ <video> and <audio> can be an HTMLElement instance instead of
+ HTMLMediaElement if MediaPlayer::isAvailable() returns false or
+ the media feature is disabled at runtime.
+
+ * html/HTMLSourceElement.cpp:
+ (WebCore::HTMLSourceElement::insertedIntoTree):
+ (WebCore::HTMLSourceElement::willRemove):
+ * html/HTMLTrackElement.cpp:
+ (WebCore::HTMLTrackElement::insertedIntoTree):
+ (WebCore::HTMLTrackElement::willRemove):
+ * html/shadow/MediaControlElements.cpp:
+ (WebCore::toParentMediaElement):
+ * page/FrameView.cpp:
+ (WebCore::FrameView::updateWidget):
+ * platform/efl/RenderThemeEfl.cpp:
+ (WebCore::RenderThemeEfl::paintMediaMuteButton):
+ * platform/gtk/RenderThemeGtk.cpp:
+ (WebCore::getMediaElementFromRenderObject):
+ * rendering/RenderThemeMac.mm:
+ (WebCore::RenderThemeMac::paintMediaSliderTrack):
+ * rendering/RenderThemeWinCE.cpp:
+ (WebCore::mediaElementParent):
+
+2011-08-03 Sailesh Agrawal <sail@chromium.org>
+
+ Chromium Mac: Make ScrollbarOverlayUtilitiesChromiumMac.h c++ compatible
+ https://bugs.webkit.org/show_bug.cgi?id=65659
+
+ Reviewed by James Robinson.
+
+ ScrollbarOverlayUtilitiesChromiumMac.h couldn't be included from pure c++ code because it used NSInteger. I fixed this by checking for __OBJC__ before using NSInteger.
+
+ Currently no pure c++ files are including ScrollbarOverlayUtilitiesChromiumMac.h but this will change in future patches from jam@chromium.org.
+
+ * platform/chromium/ScrollbarOverlayUtilitiesChromiumMac.h:
+ * platform/chromium/ScrollbarOverlayUtilitiesChromiumMac.mm:
+ (wkScrollbarPainterControllerStyle):
+ (wkSetScrollbarPainterControllerStyle):
+
+2011-08-03 Luke Macpherson <macpherson@chromium.org>
+
+ Support cast between CSSPrimitiveValue and EBoxSizing, use in CSSStyleSelector.
+ https://bugs.webkit.org/show_bug.cgi?id=65657
+
+ Reviewed by Simon Fraser.
+
+ No new tests / refactoring only.
+
+ * css/CSSPrimitiveValueMappings.h:
+ (WebCore::CSSPrimitiveValue::CSSPrimitiveValue):
+ Implement cast from EBoxSizing.
+ (WebCore::CSSPrimitiveValue::operator EBoxSizing):
+ Implement cast to EBoxSizing.
+ * css/CSSStyleSelector.cpp:
+ (WebCore::CSSStyleSelector::applyProperty):
+ Use appropriate macro to simplify code using cast.
+
+2011-08-03 Ian Henderson <ianh@apple.com>
+
+ Draw rectangular box shadow for elements with border-radius if no corners are visible
+ https://bugs.webkit.org/show_bug.cgi?id=65006
+
+ Reviewed by Simon Fraser.
+
+ Change "unroundClippedCorners" to "allCornersClippedOut" and use it to
+ determine when the corners' influence affects the blurred shadow in
+ the current drawing region.
+
+ Test: fast/box-shadow/box-shadow-clipped-slices.html
+
+ * rendering/InlineFlowBox.cpp:
+ (WebCore::InlineFlowBox::paintBoxShadow): Pass PaintInfo through to
+ RenderBoxModelObject::paintBoxShadow.
+ (WebCore::InlineFlowBox::paintBoxDecorations): Ditto.
+ * rendering/InlineFlowBox.h:
+ * rendering/RenderBox.cpp:
+ (WebCore::RenderBox::paintBoxDecorations): Ditto.
+ * rendering/RenderBoxModelObject.cpp:
+ (WebCore::allCornersClippedOut): Determines whether the bounding rects
+ for any of the given rounded rect's corners intersects the clip rect.
+ (WebCore::RenderBoxModelObject::paintBorder): Instead of "unrounding"
+ any corner outside the clip rect, unround all corners at once, and
+ only if they are all outside the clip rect.
+ (WebCore::RenderBoxModelObject::paintBoxShadow): Determine whether any
+ corner will influence shadow drawing using allCornersClippedOut on the
+ influenceRect, whose corners extend past the fillRect's corners by the
+ blur radius and inside the fillRect's corners by the blur radius.
+ * rendering/RenderBoxModelObject.h:
+ * rendering/RenderFieldset.cpp:
+ (WebCore::RenderFieldset::paintBoxDecorations): Pass PaintInfo into
+ RenderBoxModelObject::paintBoxShadow.
+ * rendering/RenderTable.cpp:
+ (WebCore::RenderTable::paintBoxDecorations): Ditto.
+ * rendering/RenderTableCell.cpp:
+ (WebCore::RenderTableCell::paintBoxDecorations): Ditto.
+
+2011-08-03 No'am Rosenthal <noam.rosenthal@nokia.com>
+
+ [Qt][Texmap][REGRESSION] http://webkit.org/blog-files/transform-style.html doesn't show composited content
+ https://bugs.webkit.org/show_bug.cgi?id=65629
+
+ Reviewed by Benjamin Poulain.
+
+ Some non-ES2 initialization was wrongfully #ifdefed in CPU(X86) and thus compiled-out.
+ When put it in the correct #ifdef, composited layers which require an intermediate buffer
+ work again.
+
+ No new tests. Existing opacity tests in LayoutTests/compositing test this.
+
+ * platform/graphics/opengl/TextureMapperGL.cpp:
+ (WebCore::BitmapTextureGL::bind):
+
+2011-08-03 Mark Rowe <mrowe@apple.com>
+
+ Bring some order to FeatureDefines.xcconfig to make it easier to follow.
+
+ Reviewed by Sam Weinig.
+
+ * Configurations/FeatureDefines.xcconfig:
+
+2011-08-03 Mark Rowe <mrowe@apple.com>
+
+ Clean up FeatureDefines.xcconfig to remove some unnecessary conditional settings
+
+ Reviewed by Dave Kilzer.
+
+ * Configurations/FeatureDefines.xcconfig:
+
+2011-08-02 Adrienne Walker <enne@google.com>
+
+ [chromium] Fix issue where switching composited tabs evicts root textures
+ https://bugs.webkit.org/show_bug.cgi?id=65562
+
+ Reviewed by James Robinson.
+
+ LayerTilerChromium::protectTileTextures was hard-coding texture sizes
+ and texture formats. Because this didn't always match textures' actual
+ sizes and formats, they weren't getting protected during eviction.
+
+ * platform/graphics/chromium/LayerTilerChromium.cpp:
+ (WebCore::LayerTilerChromium::protectTileTextures):
+
+2011-08-03 Ryosuke Niwa <rniwa@webkit.org>
+
+ select-all, copy, paste of specialAncestorElements (e.g. pre, h1, etc) nests the element inside itself
+ https://bugs.webkit.org/show_bug.cgi?id=26483
+
+ Reviewed by Enrica Casucci.
+
+ The bug was caused by WebKit serializing pre, h1, etc... to retain structure and appearance when copying
+ rich content and pasting did not remove such nodes wrapping the copied contents.
+
+ Fixed the bug by extending r81887 and r83322 to remove those elements from where WebKit pastes into.
+
+ Test: editing/pasteboard/copy-paste-text-in-h1.html
+
+ * editing/ReplaceSelectionCommand.cpp:
+ (WebCore::nodeHasAttributesToPreserve): Extracted from isInlineNodeWithStyle.
+ (WebCore::isInlineNodeWithStyle): Calls nodeHasAttributesToPreserve.
+ (WebCore::ReplaceSelectionCommand::doApply): Calls ancestorToRetainStructureAndAppearance.
+ Remove nodes copied by ancestorToRetainStructureAndAppearance at insertionPos before pasting the fragment.
+ * editing/markup.cpp:
+ (WebCore::ancestorToRetainStructureAndAppearance): Takes ShouldIncludeParagraphSeparators.
+ * editing/markup.h:
+
+2011-08-03 Mark Pilgrim <pilgrim@chromium.org>
+
+ Remove LegacyDefaultOptionalArguments flag from Console.idl
+ https://bugs.webkit.org/show_bug.cgi?id=65569
+
+ Reviewed by Adam Barth.
+
+ No new tests, all existing tests pass.
+
+ * page/Console.idl:
+
+2011-08-03 Mark Pilgrim <pilgrim@chromium.org>
+
+ Remove LegacyDefaultOptionalArguments flag from HTML DOM IDL files
+ https://bugs.webkit.org/show_bug.cgi?id=65338
+
+ Reviewed by Adam Barth.
+
+ * html/DOMFormData.idl:
+ * html/DOMTokenList.idl:
+ * html/DOMURL.idl:
+ * html/HTMLAllCollection.idl:
+ * html/HTMLAnchorElement.idl:
+ * html/HTMLAudioElement.idl:
+ * html/HTMLButtonElement.idl:
+ * html/HTMLCanvasElement.idl:
+ * html/HTMLCollection.idl:
+ * html/HTMLDivElement.idl:
+ * html/HTMLDocument.idl:
+ * html/HTMLElement.idl:
+ * html/HTMLFieldSetElement.idl:
+ * html/HTMLInputElement.idl:
+ * html/HTMLKeygenElement.idl:
+ * html/HTMLMediaElement.idl:
+ * html/HTMLObjectElement.idl:
+ * html/HTMLOptionsCollection.idl:
+ * html/HTMLOutputElement.idl:
+ * html/HTMLSelectElement.idl:
+ * html/HTMLTableElement.idl:
+ * html/HTMLTableRowElement.idl:
+ * html/HTMLTableSectionElement.idl:
+ * html/HTMLTextAreaElement.idl:
+ * html/HTMLVideoElement.idl:
+ * html/TimeRanges.idl:
+
+2011-08-03 Jeffrey Pfau <jpfau@apple.com>
+
+ Make atomic XML token
+ https://bugs.webkit.org/show_bug.cgi?id=65639
+
+ Reviewed by Adam Barth.
+
+ Create a shared AtomicMarkupTokenBase that is shared by AtomicHTMLToken and the new AtomicXMLToken
+
+ * html/parser/HTMLToken.h:
+ (WebCore::HTMLToken::setForceQuirks):
+ (WebCore::AtomicHTMLToken::AtomicHTMLToken):
+ (WebCore::AtomicHTMLToken::forceQuirks):
+ * html/parser/HTMLTokenizer.cpp:
+ (WebCore::::nameForAttribute):
+ (WebCore::::usesName):
+ (WebCore::::usesAttributes):
+ * xml/parser/MarkupTokenBase.h:
+ (WebCore::AtomicMarkupTokenBase::AtomicMarkupTokenBase):
+ (WebCore::AtomicMarkupTokenBase::type):
+ (WebCore::AtomicMarkupTokenBase::name):
+ (WebCore::AtomicMarkupTokenBase::setName):
+ (WebCore::AtomicMarkupTokenBase::selfClosing):
+ (WebCore::AtomicMarkupTokenBase::getAttributeItem):
+ (WebCore::AtomicMarkupTokenBase::attributes):
+ (WebCore::AtomicMarkupTokenBase::takeAtributes):
+ (WebCore::AtomicMarkupTokenBase::characters):
+ (WebCore::AtomicMarkupTokenBase::comment):
+ (WebCore::AtomicMarkupTokenBase::publicIdentifier):
+ (WebCore::AtomicMarkupTokenBase::systemIdentifier):
+ (WebCore::::initializeAttributes):
+ * xml/parser/XMLToken.h:
+ (WebCore::AtomicXMLToken::AtomicXMLToken):
+ (WebCore::AtomicXMLToken::prefix):
+ (WebCore::AtomicXMLToken::target):
+ (WebCore::AtomicXMLToken::data):
+ (WebCore::AtomicXMLToken::xmlVersion):
+ (WebCore::AtomicXMLToken::xmlStandalone):
+ * xml/parser/XMLTokenizer.cpp:
+ (WebCore::::nameForAttribute):
+ (WebCore::::usesName):
+ (WebCore::::usesAttributes):
+
+2011-08-03 Mark Pilgrim <pilgrim@chromium.org>
+
+ Remove LegacyDefaultOptionalArguments flag from geolocation IDL files
+ https://bugs.webkit.org/show_bug.cgi?id=65355
+
+ Reviewed by Adam Barth.
+
+ No new tests, all existing tests pass.
+
+ * page/Geolocation.idl:
+ * page/PositionCallback.idl:
+ * page/PositionErrorCallback.idl:
+
+2011-08-03 Mark Pilgrim <pilgrim@chromium.org>
+
+ Remove LegacyDefaultOptionalArguments flag from DOMSelection.idl
+ https://bugs.webkit.org/show_bug.cgi?id=65566
+
+ Reviewed by Adam Barth.
+
+ No new tests, all existing tests pass.
+
+ * page/DOMSelection.idl:
+
+2011-08-03 Mark Pilgrim <pilgrim@chromium.org>
+
+ Remove LegacyDefaultOptionalArguments flag from Location.idl
+ https://bugs.webkit.org/show_bug.cgi?id=65565
+
+ Reviewed by Darin Adler.
+
+ No new tests, all existing tests pass.
+
+ * page/Location.idl:
+
+2011-08-03 Sailesh Agrawal <sail@chromium.org>
+
+ Chromium Mac: Fix position of search tickmarks
+ https://bugs.webkit.org/show_bug.cgi?id=65575
+
+ This change adjusts the position of search tickmarks so that they are correctly centered.
+
+ Also, insetting the tickmark rect by 5 pixels caused tickmarks on the overlay scrollbar to look very small. I changed this to inset by 4 pixels instead. The tickmark is the same width as the overlay scrollbar which looks much better.
+
+ Reviewed by James Robinson.
+
+ * platform/chromium/ScrollbarThemeChromiumMac.mm:
+ (WebCore::ScrollbarThemeChromiumMac::paint):
+
+2011-08-03 Sailesh Agrawal <sail@chromium.org>
+
+ Chromium Mac: Make sure scrollbars flash when web page loads
+ https://bugs.webkit.org/show_bug.cgi?id=65586
+
+ On slow web pages scrollbars wouldn't flash when the page was done loading. The problem was that we were flashing the scrollbar 0.1 second after the load operation had started. If the page was slow to load then we might not have scrollbars to flash. To work around this I added an extra check to make sure that we had indeed finished loading the page before flashing the scrollbars.
+
+ Reviewed by James Robinson.
+
+ * platform/chromium/ScrollAnimatorChromiumMac.mm:
+ (WebCore::ScrollAnimatorChromiumMac::initialScrollbarPaintTimerFired):
+
+2011-08-03 Mark Pilgrim <pilgrim@chromium.org>
+
+ Remove LegacyDefaultOptionalArguments flag from navigator IDL files
+ https://bugs.webkit.org/show_bug.cgi?id=65370
+
+ Reviewed by Adam Barth.
+
+ * page/Navigator.idl:
+ * page/NavigatorUserMediaErrorCallback.idl:
+ * page/NavigatorUserMediaSuccessCallback.idl:
+
+2011-08-03 Anders Carlsson <andersca@apple.com>
+
+ webarchive/loading/missing-data.html is failing on Mac
+ https://bugs.webkit.org/show_bug.cgi?id=63507
+
+ Reviewed by Adam Barth.
+
+ When loading a main resource from a web archive, make sure that the working URL is actually
+ the URL of the resource, and not the URL of the web archive itself.
+
+ * loader/FrameLoader.cpp:
+ (WebCore::FrameLoader::receivedFirstData):
+
+2011-08-03 Mark Pilgrim <pilgrim@chromium.org>
+
+ Remove LegacyDefaultOptionalArguments flag from DOMWindow.idl
+ https://bugs.webkit.org/show_bug.cgi?id=65353
+
+ Reviewed by Adam Barth.
+
+ No new tests, all existing tests pass.
+
+ * bindings/scripts/CodeGeneratorJS.pm:
+ (GenerateImplementation):
+ * bindings/scripts/CodeGeneratorV8.pm:
+ (GenerateFunctionCallback):
+ * page/DOMWindow.idl:
+
+2011-08-03 Jeff Miller <jeffm@apple.com>
+
+ Fix Chromimum and Qt builds after r92308.
+
+ * config.h: Don't try to include WebCoreHeaderDetection.h for Chromimum and Qt.
+
+2011-08-03 Jeff Miller <jeffm@apple.com>
+
+ Generate WebCoreHeaderDetection.h on Windows to define HAVE_AVCF
+ https://bugs.webkit.org/show_bug.cgi?id=65634
+
+ AVFoundation support on Windows will (eventually) be predicated on the
+ availability of AVFoundationCF headers and libraries in the WebKit Support
+ Libraries, so we test for the existence of one of the header files (AVCFBase.h)
+ to determine whether to AVCF is available.
+
+ For now, even if AVFoundationCF is available, leave WTF_USE_AVFOUNDATION off, since
+ it's not ready to be turned on yet (which is covered by http://webkit.org/b/65400).
+
+ Ideally, we would generate WebCoreHeaderDetection.h on all platforms so we could
+ include it unconditionally in WebCore's config.h, but unfortunately the Mac-only file
+ ExportFileGenerator.cpp depends on WTF_USE_AVFOUNDATION being set correctly, and since this
+ .cpp file is also generated it doesn't have access to WebCoreHeaderDetection.h.
+
+ Reviewed by Adam Roben.
+
+ No change in functionality, so new tests.
+
+ * DerivedSources.make: Add rule to generate WebCoreHeaderDetection.h to set HAVE_AVCF on Windows.
+ * config.h: Include WebCoreHeaderDetection.h on Windows, check for HAVE(AVCF) but always leave WTF_USE_AVFOUNDATION off on Windows for now.
+
+2011-08-03 Tommy Widenflycht <tommyw@google.com>
+
+ MediaStream API: Implement PeerConnection and SignalingCallback
+ https://bugs.webkit.org/show_bug.cgi?id=65101
+
+ Reviewed by Adam Barth.
+
+ LayoutTests will be provided by a follow-up CL, see <https://bugs.webkit.org/show_bug.cgi?id=56587>.
+
+ * CMakeLists.txt:
+ * CodeGenerators.pri:
+ * DerivedSources.cpp:
+ * DerivedSources.make:
+ * GNUmakefile.am:
+ * GNUmakefile.list.am:
+ * UseJSC.cmake:
+ * WebCore.gyp/WebCore.gyp:
+ * WebCore.gypi:
+ * WebCore.pro:
+ * WebCore.vcproj/WebCore.vcproj:
+ * WebCore.xcodeproj/project.pbxproj:
+ * bindings/generic/RuntimeEnabledFeatures.h:
+ (WebCore::RuntimeEnabledFeatures::peerConnectionEnabled):
+ * bindings/js/JSBindingsAllInOne.cpp:
+ * bindings/js/JSPeerConnectionCustom.cpp: Copied from Source/WebCore/page/MediaStreamClient.h.
+ (WebCore::JSPeerConnectionConstructor::constructJSPeerConnection):
+ * bindings/v8/V8DOMWrapper.cpp:
+ (WebCore::V8DOMWrapper::convertEventTargetToV8Object):
+ * bindings/v8/custom/V8PeerConnectionCustom.cpp: Added.
+ (WebCore::V8PeerConnection::constructorCallback):
+ * dom/DispatchTask.h: Added.
+ (WebCore::SimpleDispatchTask::create):
+ (WebCore::SimpleDispatchTask::performTask):
+ (WebCore::SimpleDispatchTask::SimpleDispatchTask):
+ (WebCore::DispatchTask::create):
+ (WebCore::DispatchTask::performTask):
+ (WebCore::DispatchTask::DispatchTask):
+ * dom/EventNames.h:
+ * dom/EventTarget.cpp:
+ (WebCore::EventTarget::toPeerConnection):
+ * dom/EventTarget.h:
+ * p2p/PeerConnection.cpp: Added.
+ (WebCore::PeerConnection::create):
+ (WebCore::PeerConnection::PeerConnection):
+ (WebCore::PeerConnection::init):
+ (WebCore::PeerConnection::~PeerConnection):
+ (WebCore::PeerConnection::localStreams):
+ (WebCore::PeerConnection::remoteStreams):
+ (WebCore::PeerConnection::processSignalingMessage):
+ (WebCore::PeerConnection::send):
+ (WebCore::PeerConnection::addStream):
+ (WebCore::PeerConnection::removeStream):
+ (WebCore::PeerConnection::close):
+ (WebCore::PeerConnection::onNegotiationStarted):
+ (WebCore::PeerConnection::onNegotiationDone):
+ (WebCore::PeerConnection::streamAdded):
+ (WebCore::PeerConnection::streamRemoved):
+ (WebCore::PeerConnection::onMessage):
+ (WebCore::PeerConnection::onSignalingMessage):
+ (WebCore::PeerConnection::toPeerConnection):
+ (WebCore::PeerConnection::scriptExecutionContext):
+ (WebCore::PeerConnection::eventTargetData):
+ (WebCore::PeerConnection::ensureEventTargetData):
+ (WebCore::PeerConnection::postMessageEvent):
+ (WebCore::PeerConnection::postSignalingEvent):
+ (WebCore::PeerConnection::postStreamAddedEvent):
+ (WebCore::PeerConnection::postStreamRemovedEvent):
+ (WebCore::PeerConnection::postSimpleEvent):
+ (WebCore::PeerConnection::postStartNegotiationTask):
+ (WebCore::PeerConnection::dispatchMessageEvent):
+ (WebCore::PeerConnection::dispatchSignalingEvent):
+ (WebCore::PeerConnection::dispatchStreamEvent):
+ (WebCore::PeerConnection::dispatchSimpleEvent):
+ (WebCore::PeerConnection::dispatchStartNegotiationTask):
+ * p2p/PeerConnection.h: Added.
+ (WebCore::PeerConnection::readyState):
+ (WebCore::PeerConnection::refEventTarget):
+ (WebCore::PeerConnection::derefEventTarget):
+ * p2p/PeerConnection.idl: Added.
+ * p2p/SignalingCallback.h: Copied from Source/WebCore/page/MediaStreamClient.h.
+ (WebCore::SignalingCallback::~SignalingCallback):
+ * p2p/SignalingCallback.idl: Added.
+ * page/DOMWindow.idl:
+ * page/MediaStreamClient.h:
+ * page/MediaStreamController.cpp:
+ (WebCore::MediaStreamController::MediaStreamController):
+ (WebCore::MediaStreamController::unregisterFrameController):
+ (WebCore::MediaStreamController::onSignalingMessage):
+ (WebCore::MediaStreamController::frameToPagePeerConnectionId):
+ (WebCore::MediaStreamController::processSignalingMessage):
+ (WebCore::MediaStreamController::message):
+ (WebCore::MediaStreamController::onMessage):
+ (WebCore::MediaStreamController::addStream):
+ (WebCore::MediaStreamController::onAddStream):
+ (WebCore::MediaStreamController::removeStream):
+ (WebCore::MediaStreamController::onRemoveStream):
+ (WebCore::MediaStreamController::newPeerConnection):
+ (WebCore::MediaStreamController::closePeerConnection):
+ (WebCore::MediaStreamController::startNegotiation):
+ (WebCore::MediaStreamController::onNegotiationStarted):
+ (WebCore::MediaStreamController::onNegotiationDone):
+ * page/MediaStreamController.h:
+ * page/MediaStreamFrameController.cpp:
+ (WebCore::MediaStreamFrameController::Request::isSignalingRequest):
+ (WebCore::MediaStreamFrameController::createPeerConnection):
+ (WebCore::MediaStreamFrameController::newPeerConnection):
+ (WebCore::MediaStreamFrameController::closePeerConnection):
+ (WebCore::MediaStreamFrameController::startNegotiation):
+ (WebCore::MediaStreamFrameController::processSignalingMessage):
+ (WebCore::MediaStreamFrameController::message):
+ (WebCore::MediaStreamFrameController::addStream):
+ (WebCore::MediaStreamFrameController::removeStream):
+ (WebCore::MediaStreamFrameController::onMessage):
+ (WebCore::MediaStreamFrameController::onAddStream):
+ (WebCore::MediaStreamFrameController::onRemoveStream):
+ (WebCore::MediaStreamFrameController::onSignalingMessage):
+ (WebCore::MediaStreamFrameController::onNegotiationStarted):
+ (WebCore::MediaStreamFrameController::onNegotiationDone):
+ * page/MediaStreamFrameController.h:
+ (WebCore::MediaStreamFrameController::ClientBase::isPeerConnection):
+ (WebCore::MediaStreamFrameController::PeerConnectionClient::PeerConnectionClient):
+ (WebCore::MediaStreamFrameController::PeerConnectionClient::~PeerConnectionClient):
+ (WebCore::MediaStreamFrameController::PeerConnectionClient::isPeerConnection):
+
+2011-08-03 Joseph Pecoraro <joepeck@webkit.org>
+
+ Web Inspector: Remove InspectorFrontendHost load/saveSessionSetting
+ https://bugs.webkit.org/show_bug.cgi?id=65582
+
+ Reviewed by Pavel Feldman.
+
+ Remove stale code. WebInspector.settings uses localStorage and
+ handles the "debuggerEnabled" state. So remove the only remaining
+ usage of load/saveSessionSetting.
+
+ * inspector/InspectorFrontendClient.h:
+ * inspector/InspectorFrontendHost.cpp:
+ * inspector/InspectorFrontendHost.h:
+ * inspector/InspectorFrontendHost.idl:
+ * inspector/front-end/InspectorFrontendHostStub.js:
+ * inspector/front-end/ScriptsPanel.js:
+ (WebInspector.ScriptsPanel.prototype._debuggerWasDisabled):
+
+2011-08-03 Adam Barth <abarth@webkit.org>
+
+ Crash in DocumentWriter::endIfNotLoadingMainResource
+ https://bugs.webkit.org/show_bug.cgi?id=65581
+
+ Reviewed by Nate Chapin.
+
+ This function is poorly designed because isLoadingMainResource is a
+ poor proxy for determing whether to flush/finish the parser. Really,
+ we should change how loads complete to match the model in HTML5, but
+ that's pretty tricky. In the meantime, this null check fixes the
+ crash.
+
+ I'm sure there's another bug on file about this crash, but we've never
+ been able to reproduce it. Thanks to Berend-Jan Wever for the test
+ case!
+
+ Test: fast/loader/reload-zero-byte-plugin.html
+
+ * loader/DocumentWriter.cpp:
+ (WebCore::DocumentWriter::endIfNotLoadingMainResource):
+
+2011-08-03 Anders Carlsson <andersca@apple.com>
+
+ WebCore images backing CG patterns should be released on the main thread
+ https://bugs.webkit.org/show_bug.cgi?id=65630
+
+ Reviewed by Sam Weinig.
+
+ * platform/graphics/cg/PatternCG.cpp:
+ (WebCore::patternReleaseOnMainThreadCallback):
+ Deref the image.
+
+ (WebCore::patternReleaseCallback):
+ Call callOnMainThread.
+
+2011-08-03 Luke Macpherson <macpherson@chromium.org>
+
+ Use appropriate macro for CSSPropertyImageRendering.
+ https://bugs.webkit.org/show_bug.cgi?id=65594
+
+ Reviewed by Simon Fraser.
+
+ No new tests / trivial code cleanup only.
+
+ * css/CSSStyleSelector.cpp:
+ (WebCore::CSSStyleSelector::applyProperty):
+ Use HANDLE_INHERIT_AND_INITIAL_AND_PRIMITIVE macro.
+
+2011-08-03 Dan Bernstein <mitz@apple.com>
+
+ FontCache::getFontData() does not set the familyIndex to cAllFamiliesScanned if the last font is a custom font
+ https://bugs.webkit.org/show_bug.cgi?id=65597
+
+ Reviewed by Simon Fraser.
+
+ No test added because behavior is unchanged, but this saves a call to getFontData().
+
+ * platform/graphics/FontCache.cpp:
+ (WebCore::FontCache::getFontData): Eliminated early return inside the loop, allowing execution
+ to continue and familyIndex to be set.
+
+2011-08-03 Ryosuke Niwa <rniwa@webkit.org>
+
+ Add RenderedPosition
+ https://bugs.webkit.org/show_bug.cgi?id=65221
+
+ Reviewed by Hajime Morita.
+
+ Add the skeleton for RenderedPosition and use it in rootBoxForLine.
+
+ * CMakeLists.txt:
+ * GNUmakefile.list.am:
+ * WebCore.gypi:
+ * WebCore.pro:
+ * WebCore.vcproj/WebCore.vcproj:
+ * WebCore.xcodeproj/project.pbxproj:
+ * editing/EditingAllInOne.cpp:
+ * editing/RenderedPosition.cpp: Added.
+ (WebCore::RenderedPosition::RenderedPosition):
+ * editing/RenderedPosition.h: Added.
+ (WebCore::RenderedPosition::isNull):
+ (WebCore::RenderedPosition::rootBox):
+ (WebCore::RenderedPosition::RenderedPosition):
+ * editing/visible_units.cpp:
+ (WebCore::startPositionForLine):
+ (WebCore::endPositionForLine):
+ (WebCore::logicalStartPositionForLine):
+ (WebCore::logicalEndPositionForLine):
+
+2011-08-03 Roland Steiner <rolandsteiner@chromium.org>
+
+ [V8] V8CSSRuleCustom.cpp: not all enum values are listed in switch statement
+ https://bugs.webkit.org/show_bug.cgi?id=65598
+
+ Reviewed by Dimitri Glazkov.
+
+ Added UNKNOWN_RULE to the switch statement.
+
+ No new tests. (No semantic change)
+
+ * bindings/v8/custom/V8CSSRuleCustom.cpp:
+ (WebCore::toV8):
+
+2011-08-03 Pavel Feldman <pfeldman@google.com>
+
+ Web Inspector: [Timeline] Hover on paint events and see rect outline around that area.
+ https://bugs.webkit.org/show_bug.cgi?id=65510
+
+ Reviewed by Yury Semikhatsky.
+
+ * WebCore.exp.in:
+ * WebCore.order:
+ * inspector/DOMNodeHighlighter.cpp:
+ (WebCore::DOMNodeHighlighter::drawRectHighlight):
+ * inspector/DOMNodeHighlighter.h:
+ * inspector/Inspector.json:
+ * inspector/InspectorController.cpp:
+ (WebCore::InspectorController::drawHighlight):
+ * inspector/InspectorController.h:
+ * inspector/InspectorDOMAgent.cpp:
+ (WebCore::InspectorDOMAgent::highlightRect):
+ (WebCore::InspectorDOMAgent::hideHighlight):
+ (WebCore::InspectorDOMAgent::drawHighlight):
+ * inspector/InspectorDOMAgent.h:
+ * inspector/front-end/ResourcesPanel.js:
+ (WebInspector.FrameTreeElement.prototype.onselect):
+ (WebInspector.FrameTreeElement.prototype.set hovered):
+ * inspector/front-end/TimelinePanel.js:
+ (WebInspector.TimelinePanel):
+ (WebInspector.TimelinePanel.prototype._mouseOut):
+ (WebInspector.TimelinePanel.prototype._mouseMove):
+ (WebInspector.TimelinePanel.prototype._highlightRect):
+ (WebInspector.TimelinePanel.prototype._hideRectHighlight):
+ * inspector/front-end/inspector.js:
+ (WebInspector.highlightDOMNode):
+
+2011-08-03 Pavel Feldman <pfeldman@google.com>
+
+ Web Inspector: make text in popover selectable + make popover re-appear faster.
+ https://bugs.webkit.org/show_bug.cgi?id=65603
+
+ Reviewed by Yury Semikhatsky.
+
+ * inspector/front-end/Popover.js:
+ (WebInspector.PopoverHelper.prototype.setTimeout):
+ (WebInspector.PopoverHelper.prototype._mouseMove):
+ (WebInspector.PopoverHelper.prototype._handleMouseAction):
+ * inspector/front-end/popover.css:
+ (.popover .content):
+
+2011-08-03 Vsevolod Vlasov <vsevik@chromium.org>
+
+ Web Inspector: Console scroll position should be saved when switching panels.
+ https://bugs.webkit.org/show_bug.cgi?id=65607
+
+ Reviewed by Pavel Feldman.
+
+ * inspector/front-end/ConsoleView.js:
+ (WebInspector.ConsoleView.prototype._formatAsArrayEntry):
+ (WebInspector.ConsoleView.prototype.get scrollLeft):
+ (WebInspector.ConsoleView.prototype.set scrollLeft):
+ (WebInspector.ConsoleView.prototype.get scrollTop):
+ (WebInspector.ConsoleView.prototype.set scrollTop):
+ * inspector/front-end/Drawer.js:
+ (WebInspector.Drawer.prototype.show):
+ (WebInspector.Drawer.prototype._endStatusBarDragging):
+ (WebInspector.Drawer.prototype.get scrollLeft):
+ (WebInspector.Drawer.prototype.set scrollLeft):
+ (WebInspector.Drawer.prototype.get scrollTop):
+ (WebInspector.Drawer.prototype.set scrollTop):
+
+2011-08-03 Philippe Normand <pnormand@igalia.com>
+
+ libsoup-CRITICAL **: soup_message_io_pause: assertion `io != NULL' failed
+ https://bugs.webkit.org/show_bug.cgi?id=64263
+
+ Reviewed by Martin Robinson.
+
+ Avoid any operation on not yet started messages and completed messages.
+
+ * platform/network/ResourceHandleInternal.h:
+ (WebCore::ResourceHandleInternal::ResourceHandleInternal):
+ * platform/network/soup/ResourceHandleSoup.cpp:
+ (WebCore::finishedCallback):
+ (WebCore::startHTTPRequest):
+ (WebCore::ResourceHandle::platformSetDefersLoading):
+
+2011-08-03 Jeremy Moskovich <jeremy@chromium.org>
+
+ [Chromium] Fix OOP font loading to work on 10.6.6 and above.
+ https://bugs.webkit.org/show_bug.cgi?id=65543
+
+ In 10.6.6 the function used to get the unique ID for an NSFont in the
+ renderer was changed so it fails in the sandbox (it now tries to access
+ the on-disk font file). In order to work around this, we get the font
+ ID from the browser process.
+
+ To speed things up, we introduce 2 levels of caching in WebKit. A font
+ name cache where we can perform a quick lookup without the need for the
+ font id and a font id cache which we can only lookup in after getting
+ the unique ID from the browser process.
+
+ Reviewed by Kenneth Russell.
+
+ No new tests since this is not readily testable.
+
+ * platform/chromium/PlatformBridge.h:
+ * platform/graphics/chromium/CrossProcessFontLoading.h:
+ * platform/graphics/chromium/CrossProcessFontLoading.mm:
+ (WebCore::MemoryActivatedFont::create):
+ (WebCore::MemoryActivatedFont::MemoryActivatedFont):
+ (WebCore::MemoryActivatedFont::~MemoryActivatedFont):
+
+2011-08-03 Hayato Ito <hayato@chromium.org>
+
+ Make EventDispatchMediator RefCounted.
+ https://bugs.webkit.org/show_bug.cgi?id=65529
+
+ Reviewed by Hajime Morita.
+
+ Make EventDispatchMediator RefCounted so that it can be enqueued into a
+ ScopedEventQueue and called later.
+ A necessary change for ScopedEventQueue should be addressed in another patch.
+
+ This is a minor update of r92256, which failed on chromium win builds.
+
+ No changes to functionality so no new tests.
+
+ * dom/Event.cpp:
+ (WebCore::EventDispatchMediator::create):
+ * dom/Event.h:
+ * dom/EventDispatcher.cpp:
+ (WebCore::EventDispatcher::dispatchEvent):
+ * dom/EventDispatcher.h:
+ * dom/KeyboardEvent.cpp:
+ (WebCore::KeyboardEventDispatchMediator::create):
+ * dom/KeyboardEvent.h:
+ * dom/MouseEvent.cpp:
+ (WebCore::MouseEventDispatchMediator::create):
+ * dom/MouseEvent.h:
+ * dom/Node.cpp:
+ (WebCore::Node::dispatchEvent):
+ (WebCore::Node::dispatchKeyEvent):
+ (WebCore::Node::dispatchMouseEvent):
+ (WebCore::Node::dispatchWheelEvent):
+ * dom/WheelEvent.cpp:
+ (WebCore::WheelEventDispatchMediator::create):
+ * dom/WheelEvent.h:
+
+2011-08-03 Pavel Feldman <pfeldman@chromium.org>
+
+ Web Inspector: remove Node parameter from the InspectorClient::highlight
+ https://bugs.webkit.org/show_bug.cgi?id=65549
+
+ Reviewed by Yury Semikhatsky.
+
+ * inspector/InspectorClient.h:
+ * inspector/InspectorDOMAgent.cpp:
+ (WebCore::InspectorDOMAgent::highlight):
+ * loader/EmptyClients.h:
+ (WebCore::EmptyInspectorClient::highlight):
+
+2011-08-03 Antti Koivisto <antti@apple.com>
+
+ https://bugs.webkit.org/show_bug.cgi?id=65599
+ Image loads should trigger didFirstVisuallyNonEmptyLayout more easily
+
+ Reviewed by Dan Bernstein.
+
+ Tweak the heuristic pixel count so logo-sized images trigger didFirstVisuallyNonEmptyLayout.
+
+ * page/FrameView.h:
+ (WebCore::FrameView::incrementVisuallyNonEmptyPixelCount):
+
+2011-08-03 Takashi Toyoshima <toyoshim@chromium.org>
+
+ WebSocket: Could not handle zero length text frame.
+ https://bugs.webkit.org/show_bug.cgi?id=65592
+
+ Reviewed by Kent Tamura.
+
+ Test: http/tests/websocket/tests/hybi/zero-length-text.html
+
+ * websockets/WebSocketChannel.cpp:
+ (WebCore::WebSocketChannel::processFrame):
+
+2011-08-02 Pratik Solanki <psolanki@apple.com>
+
+ Crash in ResourceResponse::platformCompare() with USE(CFNETWORK)
+ https://bugs.webkit.org/show_bug.cgi?id=65584
+
+ Reviewed by David Kilzer.
+
+ * platform/network/cf/ResourceResponseCFNet.cpp:
+ (WebCore::ResourceResponse::platformCompare): Don't call CFEqual with 0 as an argument.
+
+2011-08-02 Takashi Toyoshima <toyoshim@chromium.org>
+
+ WebSocket: Should be closed by receiving invalid continuation frame.
+
+ https://bugs.webkit.org/show_bug.cgi?id=65527
+
+ Reviewed by Kent Tamura.
+
+ Test: http/tests/websocket/tests/hybi/invalid-continuation.html
+
+ * websockets/WebSocketChannel.cpp:
+ (WebCore::WebSocketChannel::processFrame):
+
+2011-08-02 Sheriff Bot <webkit.review.bot@gmail.com>
+
+ Unreviewed, rolling out r92256.
+ http://trac.webkit.org/changeset/92256
+ https://bugs.webkit.org/show_bug.cgi?id=65593
+
+ Causing tons of crashes on the chromium win bots (Requested by
+ jamesr on #webkit).
+
+ * dom/Event.cpp:
+ * dom/Event.h:
+ * dom/EventDispatcher.cpp:
+ (WebCore::EventDispatcher::dispatchEvent):
+ * dom/EventDispatcher.h:
+ * dom/KeyboardEvent.cpp:
+ * dom/KeyboardEvent.h:
+ * dom/MouseEvent.cpp:
+ * dom/MouseEvent.h:
+ * dom/Node.cpp:
+ (WebCore::Node::dispatchEvent):
+ (WebCore::Node::dispatchKeyEvent):
+ (WebCore::Node::dispatchMouseEvent):
+ (WebCore::Node::dispatchWheelEvent):
+ * dom/WheelEvent.cpp:
+ * dom/WheelEvent.h:
+
+2011-08-02 Hayato Ito <hayato@chromium.org>
+
+ Make EventDispatchMediator RefCounted.
+ https://bugs.webkit.org/show_bug.cgi?id=65529
+
+ Reviewed by Dimitri Glazkov.
+
+ Make EventDispatchMediator RefCounted so that it can be enqueued into a
+ ScopedEventQueue and called later.
+ A necessary change for ScopedEventQueue should be addressed in another patch.
+
+ No changes to functionality so no new tests.
+
+ * dom/Event.cpp:
+ (WebCore::EventDispatchMediator::create):
+ * dom/Event.h:
+ * dom/EventDispatcher.cpp:
+ (WebCore::EventDispatcher::dispatchEvent):
+ * dom/EventDispatcher.h:
+ * dom/KeyboardEvent.cpp:
+ (WebCore::KeyboardEventDispatchMediator::create):
+ * dom/KeyboardEvent.h:
+ * dom/MouseEvent.cpp:
+ (WebCore::MouseEventDispatchMediator::create):
+ * dom/MouseEvent.h:
+ * dom/Node.cpp:
+ (WebCore::Node::dispatchEvent):
+ (WebCore::Node::dispatchKeyEvent):
+ (WebCore::Node::dispatchMouseEvent):
+ (WebCore::Node::dispatchWheelEvent):
+ * dom/WheelEvent.cpp:
+ (WebCore::WheelEventDispatchMediator::create):
+ * dom/WheelEvent.h:
+
+2011-08-02 David Reveman <reveman@chromium.org>
+
+ [Chromium] Use edge-distance method for layer anti-aliasing.
+ https://bugs.webkit.org/show_bug.cgi?id=64942
+
+ Reviewed by James Robinson.
+
+ Use an edge distance based method instead of the current linear
+ filtering method for producing anti-aliased edges on 3D transformed
+ layers. This removes the outer border handling from TilingData as
+ it's no longer needed for anti-aliasing.
+
+ Tests: platform/chromium/compositing/tiny-layer-rotated.html (existing)
+ platform/chromium/compositing/huge-layer-rotated.html (existing)
+ compositing/images/content-image-change.html (existing)
+ TilingDataTest in webkit_unit_tests
+
+ * platform/graphics/chromium/LayerRendererChromium.cpp:
+ (WebCore::screenMatrix):
+ (WebCore::LayerRendererChromium::drawLayers):
+ (WebCore::LayerRendererChromium::setDrawViewportRect):
+ (WebCore::LayerRendererChromium::tilerProgramAA):
+ (WebCore::LayerRendererChromium::tilerProgramSwizzleAA):
+ * platform/graphics/chromium/LayerRendererChromium.h:
+ (WebCore::LayerRendererChromium::windowMatrix):
+ * platform/graphics/chromium/LayerTilerChromium.cpp:
+ (WebCore::LayerTilerChromium::setTileSize):
+ (WebCore::LayerTilerChromium::tileLayerRect):
+ (WebCore::LayerTilerChromium::invalidateRect):
+ (WebCore::LayerTilerChromium::prepareToUpdate):
+ (WebCore::LayerTilerChromium::updateRect):
+ (WebCore::LayerTilerChromium::draw):
+ (WebCore::LayerTilerChromium::growLayerToContain):
+ (WebCore::LayerTilerChromium::drawTiles):
+ * platform/graphics/chromium/LayerTilerChromium.h:
+ * platform/graphics/chromium/ShaderChromium.cpp:
+ (WebCore::VertexShaderTile::VertexShaderTile):
+ (WebCore::VertexShaderTile::init):
+ (WebCore::VertexShaderTile::getShaderString):
+ (WebCore::FragmentTexAlphaAABinding::FragmentTexAlphaAABinding):
+ (WebCore::FragmentTexAlphaAABinding::init):
+ (WebCore::FragmentShaderRGBATexAlphaAA::getShaderString):
+ (WebCore::FragmentShaderRGBATexSwizzleAlphaAA::getShaderString):
+ * platform/graphics/chromium/ShaderChromium.h:
+ (WebCore::VertexShaderTile::vertexTexTransformLocation):
+ (WebCore::FragmentTexAlphaAABinding::alphaLocation):
+ (WebCore::FragmentTexAlphaAABinding::samplerLocation):
+ (WebCore::FragmentTexAlphaAABinding::fragmentTexTransformLocation):
+ (WebCore::FragmentTexAlphaAABinding::edgeLocation):
+ * platform/graphics/chromium/TiledLayerChromium.cpp:
+ (WebCore::TiledLayerChromium::TiledLayerChromium):
+ (WebCore::TiledLayerChromium::updateTileSizeAndTilingOption):
+ (WebCore::TiledLayerChromium::createTilerIfNeeded):
+ (WebCore::TiledLayerChromium::setIsMask):
+ * platform/graphics/chromium/TiledLayerChromium.h:
+ * platform/graphics/gpu/TilingData.cpp:
+ (WebCore::computeNumTiles):
+ (WebCore::TilingData::tileXIndexFromSrcCoord):
+ (WebCore::TilingData::tileYIndexFromSrcCoord):
+ (WebCore::TilingData::tileSizeX):
+ (WebCore::TilingData::tileSizeY):
+ (WebCore::TilingData::intersectDrawQuad):
+ (WebCore::TilingData::textureOffset):
+ * platform/graphics/gpu/TilingData.h:
+
+2011-08-01 David Levin <levin@chromium.org>
+
+ Add asserts to RefCounted to make sure ref/deref happens on the right thread.
+ https://bugs.webkit.org/show_bug.cgi?id=31639
+
+ Reviewed by Dmitry Titov.
+
+ No new functionality exposed so no new tests. (The change is basically adding
+ more testing.)
+
+ * ForwardingHeaders/wtf/ThreadRestrictionVerifier.h: Added.
+ * loader/icon/IconDatabase.cpp:
+ (WebCore::IconDatabase::defaultIcon): Set the mutex which does the guarding of the variable.
+ (WebCore::IconDatabase::setIconDataForIconURL): Ditto.
+ (WebCore::IconDatabase::getOrCreateIconRecord): Ditto.
+ (WebCore::IconDatabase::readFromDatabase): Ditto.
+
+2011-08-02 Luke Macpherson <macpherson@chromium.org>
+
+ Clean up value clampling in CSSStyleSelector and CSSPrimitiveValue.
+ https://bugs.webkit.org/show_bug.cgi?id=65441
+
+ Reviewed by Simon Fraser.
+
+ No new tests / refactoring only.
+
+ * css/CSSPrimitiveValue.h:
+ (WebCore::CSSPrimitiveValue::getFloatValue):
+ Implement in terms of the new templated getValue().
+ (WebCore::CSSPrimitiveValue::getIntValue):
+ Implement in terms of the new templated getValue().
+ (WebCore::CSSPrimitiveValue::getValue):
+ Templated getValue that works for all numeric types.
+
+ * css/CSSStyleSelector.cpp:
+ (WebCore::CSSStyleSelector::applyProperty):
+ Use getValue<short> instead of rolling-your-own clamp to short.
+
+2011-08-02 Julien Chaffraix <jchaffraix@webkit.org>
+
+ RenderObject::computeRectForRepaint and clippedOverflowRectForRepaint should be const
+ https://bugs.webkit.org/show_bug.cgi?id=65494
+
+ Reviewed by Darin Adler.
+
+ No new tests, const-ness change only.
+
+ * rendering/RenderObject.h:
+ (WebCore::RenderObject::absoluteClippedOverflowRect):
+ (WebCore::RenderObject::computeAbsoluteRepaintRect):
+ Made those 2 const as they call a const method now.
+
+ * rendering/RenderInline.cpp:
+ (WebCore::RenderInline::clippedOverflowRectForRepaint): Use a const RenderObject* for iterating as
+ this is const.
+
+ * rendering/RenderBlock.cpp:
+ (WebCore::RenderBlock::rectWithOutlineForRepaint):
+ * rendering/RenderBlock.h:
+ * rendering/RenderBox.cpp:
+ (WebCore::RenderBox::clippedOverflowRectForRepaint):
+ (WebCore::RenderBox::computeRectForRepaint):
+ * rendering/RenderBox.h:
+ * rendering/RenderInline.cpp:
+ (WebCore::RenderInline::clippedOverflowRectForRepaint):
+ (WebCore::RenderInline::rectWithOutlineForRepaint):
+ (WebCore::RenderInline::computeRectForRepaint):
+ * rendering/RenderInline.h:
+ * rendering/RenderObject.cpp:
+ (WebCore::RenderObject::rectWithOutlineForRepaint):
+ (WebCore::RenderObject::clippedOverflowRectForRepaint):
+ (WebCore::RenderObject::computeRectForRepaint):
+ * rendering/RenderReplaced.cpp:
+ (WebCore::RenderReplaced::clippedOverflowRectForRepaint):
+ * rendering/RenderReplaced.h:
+ * rendering/RenderTableCell.cpp:
+ (WebCore::RenderTableCell::clippedOverflowRectForRepaint):
+ (WebCore::RenderTableCell::computeRectForRepaint):
+ * rendering/RenderTableCell.h:
+ * rendering/RenderTableCol.cpp:
+ (WebCore::RenderTableCol::clippedOverflowRectForRepaint):
+ * rendering/RenderTableCol.h:
+ * rendering/RenderTableRow.cpp:
+ (WebCore::RenderTableRow::clippedOverflowRectForRepaint):
+ * rendering/RenderTableRow.h:
+ * rendering/RenderText.cpp:
+ (WebCore::RenderText::clippedOverflowRectForRepaint):
+ * rendering/RenderText.h:
+ * rendering/RenderView.cpp:
+ (WebCore::RenderView::computeRectForRepaint):
+ * rendering/RenderView.h:
+ * rendering/svg/RenderSVGForeignObject.cpp:
+ (WebCore::RenderSVGForeignObject::clippedOverflowRectForRepaint):
+ (WebCore::RenderSVGForeignObject::computeRectForRepaint):
+ * rendering/svg/RenderSVGForeignObject.h:
+ * rendering/svg/RenderSVGGradientStop.h:
+ (WebCore::RenderSVGGradientStop::clippedOverflowRectForRepaint):
+ * rendering/svg/RenderSVGHiddenContainer.h:
+ (WebCore::RenderSVGHiddenContainer::clippedOverflowRectForRepaint):
+ * rendering/svg/RenderSVGInline.cpp:
+ (WebCore::RenderSVGInline::clippedOverflowRectForRepaint):
+ (WebCore::RenderSVGInline::computeRectForRepaint):
+ * rendering/svg/RenderSVGInline.h:
+ * rendering/svg/RenderSVGModelObject.cpp:
+ (WebCore::RenderSVGModelObject::clippedOverflowRectForRepaint):
+ (WebCore::RenderSVGModelObject::computeRectForRepaint):
+ * rendering/svg/RenderSVGModelObject.h:
+ * rendering/svg/RenderSVGRoot.cpp:
+ (WebCore::RenderSVGRoot::clippedOverflowRectForRepaint):
+ (WebCore::RenderSVGRoot::computeRectForRepaint):
+ * rendering/svg/RenderSVGRoot.h:
+ * rendering/svg/RenderSVGText.cpp:
+ (WebCore::RenderSVGText::clippedOverflowRectForRepaint):
+ (WebCore::RenderSVGText::computeRectForRepaint):
+ * rendering/svg/RenderSVGText.h:
+ * rendering/svg/SVGRenderSupport.cpp:
+ (WebCore::SVGRenderSupport::clippedOverflowRectForRepaint):
+ (WebCore::SVGRenderSupport::computeRectForRepaint):
+ * rendering/svg/SVGRenderSupport.h:
+ Updated the following method's signature to use a const object.
+
+2011-08-02 Jeffrey Pfau <jpfau@apple.com>
+
+ Add namespace prefix support in XML tokenizer
+ https://bugs.webkit.org/show_bug.cgi?id=65578
+
+ Reviewed by Adam Barth.
+
+ This also changes markup debug printing to print to stderr instead of stdout
+
+ * xml/parser/MarkupTokenBase.h:
+ (WebCore::MarkupTokenBase::printString):
+ * xml/parser/XMLToken.h:
+ (WebCore::XMLToken::XMLDeclarationData::XMLDeclarationData):
+ (WebCore::XMLToken::clear):
+ (WebCore::XMLToken::appendToName):
+ (WebCore::XMLToken::name):
+ (WebCore::XMLToken::target):
+ (WebCore::XMLToken::data):
+ (WebCore::XMLToken::prefix):
+ (WebCore::XMLToken::xmlDeclarationData):
+ (WebCore::XMLToken::endPrefix):
+ (WebCore::XMLToken::hasPrefix):
+ (WebCore::XMLToken::endAttributePrefix):
+ (WebCore::XMLToken::attributeHasPrefix):
+ (WebCore::XMLToken::printAttrs):
+ (WebCore::XMLToken::print):
+ * xml/parser/XMLTokenizer.cpp:
+ (WebCore::XMLTokenizer::nextToken):
+
+2011-08-02 Zhenyao Mo <zmo@google.com>
+
+ Unreviewed, rolling out r92238.
+ http://trac.webkit.org/changeset/92238
+ https://bugs.webkit.org/show_bug.cgi?id=65555
+
+ fail fast/events/scrollbar-double-click.html on Mac
+
+ * platform/chromium/ScrollbarOverlayUtilitiesChromiumMac.h:
+ * platform/chromium/ScrollbarOverlayUtilitiesChromiumMac.mm:
+ * platform/chromium/ScrollbarThemeChromiumMac.h:
+ * platform/chromium/ScrollbarThemeChromiumMac.mm:
+ (WebCore::ScrollbarThemeChromiumMac::registerScrollbar):
+ (WebCore::ScrollbarThemeChromiumMac::setNewPainterForScrollbar):
+ (WebCore::toScrollbarPainterKnobStyle):
+ (WebCore::ScrollbarThemeChromiumMac::paint):
+
+2011-08-02 Mark Pilgrim <pilgrim@chromium.org>
+
+ Remove LegacyDefaultOptionalArguments flag from History.idl
+ https://bugs.webkit.org/show_bug.cgi?id=65572
+
+ Reviewed by Adam Barth.
+
+ No new tests, all existing tests pass.
+
+ * page/History.idl:
+
+2011-08-02 Mark Pilgrim <pilgrim@chromium.org>
+
+ Remove LegacyDefaultOptionalArguments flag from WebKitAnimationList.idl
+ https://bugs.webkit.org/show_bug.cgi?id=65571
+
+ Reviewed by Adam Barth.
+
+ No new tests, all existing tests pass.
+
+ * page/WebKitAnimationList.idl:
+
+2011-08-02 James Robinson <jamesr@chromium.org>
+
+ [chromium] Decouple layer visibility calculations from render surfaces / computed draw transforms
+ https://bugs.webkit.org/show_bug.cgi?id=58840
+
+ Reviewed by Kenneth Russell.
+
+ This decouples the layer visibility related calculations from the LayerChromium and CCLayerImpl trees. This is
+ done by making LayerRendererChromium's "calculate the world" function (renamed
+ calculateDrawTransformsAndVisibility()) a templated, free function that can operate on either LayerChromium +
+ RenderSurfaceChromium pairs or on CCLayerImpl + CCRenderSurface pairs and invoking it twice, once before
+ painting and once before drawing. Before painting, these calculations are used to determine which layers are
+ potentially visible and should be painted, which depends on the calculated opacity, render surface bounds, and
+ draw transforms. Before drawing, these calculations are used to figure out everything needed to draw.
+
+ RenderSurfaceChromium is now used just for visibility calculations when painting. CCRenderSurfaceChromium is
+ responsible for actually drawing the render surfaces.
+
+ This does mean that in the current implementation the second calculations are redundant, but in the future this
+ won't be the case since the pre-draw time calculations might be influenced by animations or gestures. I've
+ added some traces to try to monitor if this math is a significant amount of the per-frame computation. Note
+ that layer sorting only happens on the CCLayerImpl tree and the layer sort algorithm is the most expensive part
+ of the calculations currently.
+
+ This removes all of the weak back/forward pointers between LayerChromium and CCLayerImpls, the trees are now
+ truly decoupled.
+
+ Covered by compositing/ tests.
+
+ * WebCore.gypi:
+ * platform/graphics/chromium/CanvasLayerChromium.cpp:
+ (WebCore::CanvasLayerChromium::createCCLayerImpl):
+ * platform/graphics/chromium/LayerChromium.cpp:
+ (WebCore::LayerChromium::LayerChromium):
+ (WebCore::LayerChromium::~LayerChromium):
+ (WebCore::LayerChromium::createCCLayerImpl):
+ (WebCore::LayerChromium::createRenderSurface):
+ (WebCore::LayerChromium::descendantsDrawsContent):
+ * platform/graphics/chromium/LayerChromium.h:
+ (WebCore::LayerChromium::setUsesLayerScissor):
+ (WebCore::LayerChromium::usesLayerScissor):
+ (WebCore::LayerChromium::clearRenderSurface):
+ (WebCore::LayerChromium::renderSurface):
+ (WebCore::LayerChromium::drawOpacity):
+ (WebCore::LayerChromium::setDrawOpacity):
+ (WebCore::LayerChromium::scissorRect):
+ (WebCore::LayerChromium::setScissorRect):
+ (WebCore::LayerChromium::targetRenderSurface):
+ (WebCore::LayerChromium::setTargetRenderSurface):
+ (WebCore::LayerChromium::drawTransform):
+ (WebCore::LayerChromium::setDrawTransform):
+ (WebCore::LayerChromium::drawableContentRect):
+ (WebCore::LayerChromium::setDrawableContentRect):
+ * platform/graphics/chromium/LayerRendererChromium.cpp:
+ (WebCore::LayerRendererChromium::updateLayers):
+ (WebCore::LayerRendererChromium::drawLayers):
+ (WebCore::LayerRendererChromium::paintLayerContents):
+ (WebCore::LayerRendererChromium::drawLayersInternal):
+ (WebCore::LayerRendererChromium::updateCompositorResources):
+ (WebCore::LayerRendererChromium::setCompositeOffscreen):
+ (WebCore::LayerRendererChromium::getOffscreenLayerTexture):
+ (WebCore::LayerRendererChromium::useRenderSurface):
+ (WebCore::LayerRendererChromium::drawLayer):
+ (WebCore::LayerRendererChromium::initializeSharedObjects):
+ (WebCore::LayerRendererChromium::renderSurfaceProgram):
+ (WebCore::LayerRendererChromium::renderSurfaceMaskProgram):
+ (WebCore::LayerRendererChromium::dumpRenderSurfaces):
+ * platform/graphics/chromium/LayerRendererChromium.h:
+ * platform/graphics/chromium/PluginLayerChromium.cpp:
+ (WebCore::PluginLayerChromium::createCCLayerImpl):
+ * platform/graphics/chromium/RenderSurfaceChromium.cpp:
+ (WebCore::RenderSurfaceChromium::RenderSurfaceChromium):
+ (WebCore::RenderSurfaceChromium::~RenderSurfaceChromium):
+ * platform/graphics/chromium/RenderSurfaceChromium.h:
+ (WebCore::RenderSurfaceChromium::clearLayerList):
+ (WebCore::RenderSurfaceChromium::layerList):
+ (WebCore::RenderSurfaceChromium::setMaskLayer):
+ * platform/graphics/chromium/TiledLayerChromium.cpp:
+ (WebCore::TiledLayerChromium::createCCLayerImpl):
+ (WebCore::TiledLayerChromium::tilingTransform):
+ * platform/graphics/chromium/TreeSynchronizer.cpp:
+ (WebCore::TreeSynchronizer::synchronizeTreeRecursive):
+ * platform/graphics/chromium/VideoLayerChromium.cpp:
+ (WebCore::VideoLayerChromium::createCCLayerImpl):
+ * platform/graphics/chromium/cc/CCCanvasLayerImpl.cpp:
+ (WebCore::CCCanvasLayerImpl::CCCanvasLayerImpl):
+ * platform/graphics/chromium/cc/CCCanvasLayerImpl.h:
+ (WebCore::CCCanvasLayerImpl::create):
+ * platform/graphics/chromium/cc/CCLayerImpl.cpp:
+ (WebCore::CCLayerImpl::CCLayerImpl):
+ (WebCore::CCLayerImpl::~CCLayerImpl):
+ (WebCore::CCLayerImpl::createRenderSurface):
+ (WebCore::CCLayerImpl::bindContentsTexture):
+ * platform/graphics/chromium/cc/CCLayerImpl.h:
+ (WebCore::CCLayerImpl::create):
+ (WebCore::CCLayerImpl::renderSurface):
+ (WebCore::CCLayerImpl::targetRenderSurface):
+ (WebCore::CCLayerImpl::setTargetRenderSurface):
+ * platform/graphics/chromium/cc/CCLayerSorter.cpp:
+ (WebCore::CCLayerSorter::createGraphNodes):
+ * platform/graphics/chromium/cc/CCPluginLayerImpl.cpp:
+ (WebCore::CCPluginLayerImpl::CCPluginLayerImpl):
+ * platform/graphics/chromium/cc/CCPluginLayerImpl.h:
+ (WebCore::CCPluginLayerImpl::create):
+ * platform/graphics/chromium/cc/CCRenderSurface.cpp: Copied from Source/WebCore/platform/graphics/chromium/RenderSurfaceChromium.cpp.
+ (WebCore::CCRenderSurface::CCRenderSurface):
+ (WebCore::CCRenderSurface::~CCRenderSurface):
+ (WebCore::CCRenderSurface::cleanupResources):
+ (WebCore::CCRenderSurface::layerRenderer):
+ (WebCore::CCRenderSurface::drawableContentRect):
+ (WebCore::CCRenderSurface::prepareContentsTexture):
+ (WebCore::CCRenderSurface::releaseContentsTexture):
+ (WebCore::CCRenderSurface::drawSurface):
+ (WebCore::CCRenderSurface::draw):
+ (WebCore::CCRenderSurface::name):
+ (WebCore::writeIndent):
+ (WebCore::CCRenderSurface::dumpSurface):
+ (WebCore::CCRenderSurface::owningLayerId):
+ * platform/graphics/chromium/cc/CCRenderSurface.h: Copied from Source/WebCore/platform/graphics/chromium/RenderSurfaceChromium.h.
+ (WebCore::CCRenderSurface::contentRectCenter):
+ (WebCore::CCRenderSurface::drawOpacity):
+ (WebCore::CCRenderSurface::setDrawOpacity):
+ (WebCore::CCRenderSurface::setDrawTransform):
+ (WebCore::CCRenderSurface::drawTransform):
+ (WebCore::CCRenderSurface::setReplicaDrawTransform):
+ (WebCore::CCRenderSurface::replicaDrawTransform):
+ (WebCore::CCRenderSurface::setOriginTransform):
+ (WebCore::CCRenderSurface::originTransform):
+ (WebCore::CCRenderSurface::setScissorRect):
+ (WebCore::CCRenderSurface::scissorRect):
+ (WebCore::CCRenderSurface::setContentRect):
+ (WebCore::CCRenderSurface::contentRect):
+ (WebCore::CCRenderSurface::setSkipsDraw):
+ (WebCore::CCRenderSurface::skipsDraw):
+ (WebCore::CCRenderSurface::clearLayerList):
+ (WebCore::CCRenderSurface::layerList):
+ (WebCore::CCRenderSurface::setMaskLayer):
+ (WebCore::CCRenderSurface::contentsTexture):
+ * platform/graphics/chromium/cc/CCTiledLayerImpl.cpp:
+ (WebCore::CCTiledLayerImpl::CCTiledLayerImpl):
+ * platform/graphics/chromium/cc/CCTiledLayerImpl.h:
+ (WebCore::CCTiledLayerImpl::create):
+ * platform/graphics/chromium/cc/CCVideoLayerImpl.cpp:
+ (WebCore::CCVideoLayerImpl::CCVideoLayerImpl):
+ * platform/graphics/chromium/cc/CCVideoLayerImpl.h:
+ (WebCore::CCVideoLayerImpl::create):
+
+2011-08-02 Luke Macpherson <macpherson@chromium.org>
+
+ Implement CSSPropertyWebkitTextCombine, CSSPropertyWebkitTextEmphasisPosition and CSSPropertyWebkitTextEmphasisStyle in CSSStyleApplyProperty.
+ https://bugs.webkit.org/show_bug.cgi?id=65517
+
+ Reviewed by Dimitri Glazkov.
+
+ No new tests. Just refactoring and moving code around.
+
+ * css/CSSStyleApplyProperty.cpp:
+ Added class ApplyPropertyTextEmphasisStyle to handle the special TextEmphasisStyle logic.
+ (WebCore::ApplyPropertyTextEmphasisStyle::applyInheritValue):
+ (WebCore::ApplyPropertyTextEmphasisStyle::applyInitialValue):
+ (WebCore::ApplyPropertyTextEmphasisStyle::applyValue):
+ (WebCore::CSSStyleApplyProperty::CSSStyleApplyProperty):
+ Added initializations for CSSPropertyWebkitTextCombine, CSSPropertyWebkitTextEmphasisPosition, CSSPropertyWebkitTextEmphasisStyle.
+ * css/CSSStyleSelector.cpp:
+ (WebCore::CSSStyleSelector::applyProperty):
+ Remove existing implementations.
+
+2011-08-02 Mark Pilgrim <pilgrim@chromium.org>
+
+ Remove LegacyDefaultOptionalArguments flag from SpeechInputResultList.idl
+ https://bugs.webkit.org/show_bug.cgi?id=65570
+
+ Reviewed by Adam Barth.
+
+ No new tests, all existing tests pass
+
+ * page/SpeechInputResultList.idl:
+
+2011-08-02 Mark Pilgrim <pilgrim@chromium.org>
+
+ Remove LegacyDefaultOptionalArguments flag from EventSource.idl
+ https://bugs.webkit.org/show_bug.cgi?id=65568
+
+ Reviewed by Adam Barth.
+
+ No new tests, all existing tests pass.
+
+ * page/EventSource.idl:
+
+2011-08-02 Gavin Peters <gavinp@chromium.org>
+
+ fail earlier to track down null CachedScript execution
+ https://bugs.webkit.org/show_bug.cgi?id=65563
+
+ Over in http://code.google.com/p/chromium/issues/detail?id=75604 I can't reproduce the problem. Careful reading of the code hasn't led me
+ to an obvious cause either. This patch should cause failure earlier, and lead to better stacks. I'll watch Chrome Canary's crash
+ uploads carefully, and remove this CRASH() (and fix the underlying problem) once I understand it.
+
+ Reviewed by Alexey Proskuryakov.
+
+ No new tests, as my goal here is to get telemetry on a bug that I cannot reproduce.
+
+ * dom/ScriptRunner.cpp:
+ (WebCore::ScriptRunner::queueScriptForExecution):
+
+2011-08-02 Sailesh Agrawal <sail@chromium.org>
+
+ Chromium: Update forked ScrollbarThemeChromiumMac.mm
+ https://bugs.webkit.org/show_bug.cgi?id=65555
+
+ Merge recent changes to ScrollbarThemeMac.mm to ScrollbarThemeChromiumMac.mm.
+
+ Reviewed by James Robinson.
+
+ * platform/chromium/ScrollbarOverlayUtilitiesChromiumMac.h:
+ * platform/chromium/ScrollbarOverlayUtilitiesChromiumMac.mm:
+ (wkScrollbarPainterSetEnabled):
+ * platform/chromium/ScrollbarThemeChromiumMac.h:
+ (WebCore::ScrollbarThemeChromiumMac::maxOverlapBetweenPages):
+ * platform/chromium/ScrollbarThemeChromiumMac.mm:
+ (WebCore::ScrollbarThemeChromiumMac::registerScrollbar):
+ (WebCore::ScrollbarThemeChromiumMac::setNewPainterForScrollbar):
+ (WebCore::toScrollbarPainterKnobStyle):
+ (WebCore::ScrollbarThemeChromiumMac::updateScrollbarOverlayStyle):
+ (WebCore::ScrollbarThemeChromiumMac::updateEnabledState):
+ (WebCore::ScrollbarThemeChromiumMac::paint):
+
+2011-08-02 Mark Pilgrim <pilgrim@chromium.org>
+
+ Remove LegacyDefaultOptionalArguments flag from SVG IDL files
+ https://bugs.webkit.org/show_bug.cgi?id=65337
+
+ Reviewed by Nikolas Zimmermann.
+
+ No new tests, all existing tests pass.
+
+ * svg/ElementTimeControl.idl:
+ * svg/SVGAnimationElement.idl:
+ * svg/SVGDocument.idl:
+ * svg/SVGElementInstance.idl:
+ * svg/SVGElementInstanceList.idl:
+ * svg/SVGException.idl:
+ * svg/SVGFEDropShadowElement.idl:
+ * svg/SVGFEGaussianBlurElement.idl:
+ * svg/SVGFEMorphologyElement.idl:
+ * svg/SVGFilterElement.idl:
+ * svg/SVGLocatable.idl:
+ * svg/SVGMarkerElement.idl:
+ * svg/SVGPathElement.idl:
+ * svg/SVGSVGElement.idl:
+ * svg/SVGStylable.idl:
+ * svg/SVGStyleElement.idl:
+ * svg/SVGTests.idl:
+ * svg/SVGTextContentElement.idl:
+
+2011-08-02 Mark Pilgrim <pilgrim@chromium.org>
+
+ Remove LegacyDefaultOptionalArguments flag from CSS IDL files
+ https://bugs.webkit.org/show_bug.cgi?id=65341
+
+ Reviewed by Adam Barth.
+
+ No new tests. All existing tests pass.
+
+ * css/CSSMediaRule.idl:
+ * css/CSSPrimitiveValue.idl:
+ * css/CSSRuleList.idl:
+ * css/CSSStyleDeclaration.idl:
+ * css/CSSStyleSheet.idl:
+ * css/CSSValueList.idl:
+ * css/MediaList.idl:
+ * css/MediaQueryList.idl:
+ * css/MediaQueryListListener.idl:
+ * css/StyleMedia.idl:
+ * css/StyleSheetList.idl:
+ * css/WebKitCSSKeyframesRule.idl:
+ * css/WebKitCSSMatrix.idl:
+
+2011-08-02 Robert Kroeger <rjkroege@chromium.org>
+
+ [chromium] Layering violations in gesture recognizer
+ https://bugs.webkit.org/show_bug.cgi?id=65044
+
+ Reviewed by Adam Barth.
+
+ Divided the gesture recognizer up to correct a layering
+ violation by moving gesture implementation from it to
+ EventHandler::handleGestureEvent so that the gesture recognizer
+ could simply be an engine for generating gesture events from
+ touch events.
+
+ * dom/WheelEvent.cpp:
+ (WebCore::WheelEvent::WheelEvent):
+ (WebCore::WheelEvent::initWheelEvent):
+ * dom/WheelEvent.h:
+ Added symbolic contstant for tick multiplier.
+ * page/EventHandler.cpp:
+ (WebCore::EventHandler::EventHandler):
+ Removed construction of gesture recognizer.
+ (WebCore::EventHandler::handleGestureEvent):
+ Extended for new gesture events added in this patch.
+ (WebCore::EventHandler::handleTouchEvent):
+ Removed call into gesture recognizer.
+ * page/EventHandler.h:
+ * page/FrameView.cpp:
+ (WebCore::FrameView::reset):
+ Removed code to reset gesture recognizer.
+ * platform/PlatformGestureEvent.h:
+ (WebCore::PlatformGestureEvent::PlatformGestureEvent):
+ (WebCore::PlatformGestureEvent::deltaX):
+ (WebCore::PlatformGestureEvent::deltaY):
+ (WebCore::PlatformGestureEvent::shiftKey):
+ (WebCore::PlatformGestureEvent::ctrlKey):
+ (WebCore::PlatformGestureEvent::altKey):
+ (WebCore::PlatformGestureEvent::metaKey):
+ Added additional gesture types and parameters.
+ * platform/PlatformGestureRecognizer.h:
+ * platform/chromium/FramelessScrollView.h:
+ Added support for displatching gesture and touch events.
+ * platform/chromium/GestureRecognizerChromium.cpp:
+ (WebCore::InnerGestureRecognizer::InnerGestureRecognizer):
+ (WebCore::InnerGestureRecognizer::reset):
+ (WebCore::InnerGestureRecognizer::constructClickGestureEvent):
+ (WebCore::InnerGestureRecognizer::processTouchEventForGestures):
+ Removed all depedencies on EventHandler. Instead, create gesture
+ events from touch event stream.
+ (WebCore::InnerGestureRecognizer::constructScrollGesture):
+ (WebCore::touchDown):
+ (WebCore::noGesture):
+ (WebCore::click):
+ (WebCore::isClickOrScroll):
+ (WebCore::inScroll):
+ (WebCore::GestureRecognizerChromium::processTouchEventForGestures):
+ (WebCore::GestureRecognizerChromium::reset):
+ * platform/chromium/GestureRecognizerChromium.h:
+ * platform/chromium/PopupMenuChromium.cpp:
+ Added support to handle touch events and gesture events.
+ (WebCore::PopupContainer::handleTouchEvent):
+ (WebCore::PopupContainer::handleGestureEvent):
+ (WebCore::PopupListBox::handleTouchEvent):
+ (WebCore::PopupListBox::handleGestureEvent):
+ * platform/chromium/PopupMenuChromium.h:
+ * platform/mac/ScrollAnimatorMac.mm:
+ (WebCore::ScrollAnimatorMac::handleGestureEvent):
+ Ignores the newly added gesture event types.
+
+2011-08-01 Michael Saboff <msaboff@apple.com>
+
+ Virtual copying of FastMalloc allocated memory causes madvise MADV_FREE_REUSABLE errors
+ https://bugs.webkit.org/show_bug.cgi?id=65502
+
+ Reviewed by Anders Carlsson.
+
+ Change the vm_copy in PurgeableBuffer::create to be a memcpy. The
+ vm_copy causes the process to have additional references to the same
+ memory region. These additional reference caused madvise(MADV_FREE_REUSABLE)
+ to fail when it encountered such pages.
+
+ No tests added this is a resource defect and not a functional issue.
+
+ * platform/mac/PurgeableBufferMac.cpp:
+ (WebCore::PurgeableBuffer::create):
+
+2011-08-02 Stephen White <senorblanco@chromium.org>
+
+ [Chromium] Remove an unnecessary readback during accelerated
+ canvas-to-canvas draws.
+ https://bugs.webkit.org/show_bug.cgi?id=65560
+
+ Reviewed by James Robinson.
+
+ No new tests. (Printing has few tests, sadly.)
+
+ * platform/graphics/skia/ImageBufferSkia.cpp:
+ (WebCore::ImageBuffer::draw):
+ Revert most of the changes to this file from
+ http://trac.webkit.org/changeset/91870, except for the call which makes
+ the source context current, so that the readPixels() still works.
+
+
+2011-08-02 Filip Pizlo <fpizlo@apple.com>
+
+ JSC GC is far too conservative about growing the heap size, particularly
+ on desktop platforms
+ https://bugs.webkit.org/show_bug.cgi?id=65438
+
+ Reviewed by Oliver Hunt.
+
+ No change in behavior, thus no new tests.
+
+ Pass the LargeHeap hint to JSGlobalData when creating the JSC runtime
+ instance corresponding to non-worker JS code.
+
+ * bindings/js/JSDOMWindowBase.cpp:
+ (WebCore::JSDOMWindowBase::commonJSGlobalData):
+
+2011-08-02 Van Lam <vanlam@google.com>
+
+ --webkit-visual-word: ctrl-arrow is not able to reach the boundary of line
+ https://bugs.webkit.org/show_bug.cgi?id=61346
+
+ Reviewed by Ryosuke Niwa.
+
+ When collecintg word boundaries in a box, explicitly check if the box
+ is at the end of line (so it should contain the end-of-line position)
+ and collect the end-of-line position.
+
+ * editing/visible_units.cpp:
+ (WebCore::isBoxVisuallyLastInLine):
+ (WebCore::appendPositionAtLogicalEndOfLine):
+ (WebCore::collectWordBreaksInBoxInsideBlockWithSameDirectionality):
+ (WebCore::collectWordBreaksInBoxInsideBlockWithDifferntDirectionality):
+
+2011-08-02 Vsevolod Vlasov <vsevik@chromium.org>
+
+ Web Inspector: Should save scroll positions for network panel Response and Preview tabs.
+ https://bugs.webkit.org/show_bug.cgi?id=65557
+
+ Reviewed by Pavel Feldman.
+
+ * inspector/front-end/NetworkItemView.js:
+ (WebInspector.ResourceContentView.prototype.get innerView):
+ (WebInspector.ResourceContentView.prototype.set innerView):
+ (WebInspector.ResourceContentView.prototype.hide):
+ * inspector/front-end/ResourcePreviewView.js:
+ (WebInspector.ResourcePreviewView.prototype.contentLoaded):
+ * inspector/front-end/ResourceResponseView.js:
+ (WebInspector.ResourceResponseView.prototype.contentLoaded):
+
+2011-08-02 Vsevolod Vlasov <vsevik@chromium.org>
+
+ Web Inspector: Remove initator column for JSC, fix network resource initiator tests.
+ https://bugs.webkit.org/show_bug.cgi?id=65534
+
+ Reviewed by Pavel Feldman.
+
+ * inspector/front-end/NetworkPanel.js:
+ (WebInspector.NetworkLogView.prototype._createTable):
+ (WebInspector.NetworkLogView.prototype.switchToDetailedView):
+ (WebInspector.NetworkLogView.prototype.switchToBriefView):
+ (WebInspector.NetworkLogView.prototype._toggleGridMode):
+ (WebInspector.NetworkLogView.prototype._toggleViewingResourceMode):
+ (WebInspector.NetworkDataGridNode.prototype.createCells):
+ (WebInspector.NetworkDataGridNode.prototype.refreshResource):
+ * inspector/front-end/Settings.js:
+
+2011-08-02 Vsevolod Vlasov <vsevik@chromium.org>
+
+ Web Inspector: SourceFrame should store saved scroll positions in View.
+ https://bugs.webkit.org/show_bug.cgi?id=65472
+
+ Reviewed by Pavel Feldman.
+
+ * inspector/front-end/ResourcesPanel.js:
+ (WebInspector.FrameResourceTreeElement.prototype._recreateSourceView):
+ * inspector/front-end/ScriptsPanel.js:
+ (WebInspector.ScriptsPanel.prototype._sourceFileChanged):
+ * inspector/front-end/SourceFrame.js:
+ (WebInspector.SourceFrame.prototype.show):
+ (WebInspector.SourceFrame.prototype.hide):
+ (WebInspector.SourceFrame.prototype.get scrollLeft):
+ (WebInspector.SourceFrame.prototype.set scrollLeft):
+ (WebInspector.SourceFrame.prototype.get scrollTop):
+ (WebInspector.SourceFrame.prototype.set scrollTop):
+ * inspector/front-end/View.js:
+ (WebInspector.View.prototype._innerShow):
+ (WebInspector.View.prototype._innerHide):
+ (WebInspector.View.prototype.detach):
+ (WebInspector.View.prototype.storeScrollPositions):
+ (WebInspector.View.prototype.inheritScrollPositionsFromView):
+ (WebInspector.View.prototype.restoreScrollPositions):
+
+2011-08-02 Jeff Miller <jeffm@apple.com>
+
+ Move WTF_USE_AVFOUNDATION from JavaScriptCore/wtf/platform.h to WebCore/config.h
+ https://bugs.webkit.org/show_bug.cgi?id=65552
+
+ Since this is a WebCore feature, there's no need to define it in JavaScriptCore/wtf/platform.h.
+
+ Reviewed by Adam Roben.
+
+ No change in functionality, so new tests.
+
+ * config.h: Added WTF_USE_AVFOUNDATION.
+
+2011-08-02 Alexandru Chiculita <achicu@adobe.com>
+
+ Split RenderBlock::layoutRunsAndFloats into multiple methods
+ https://bugs.webkit.org/show_bug.cgi?id=65376
+
+ Extracted layoutRunsAndFloatsInRange, linkToEndLineIfNeeded and repaintDirtyFloats methods and moved
+ most of the locals to LineLayoutState.
+
+ Reviewed by David Hyatt.
+
+ No new tests needed because it is just refactoring existing code.
+
+ * rendering/RenderBlock.h:
+ * rendering/RenderBlockLineLayout.cpp:
+ (WebCore::LineLayoutState::LineLayoutState):
+ (WebCore::LineLayoutState::endLineMatched):
+ (WebCore::LineLayoutState::setEndLineMatched):
+ (WebCore::LineLayoutState::checkForFloatsFromLastLine):
+ (WebCore::LineLayoutState::setCheckForFloatsFromLastLine):
+ (WebCore::LineLayoutState::lineInfo):
+ (WebCore::LineLayoutState::endLineLogicalTop):
+ (WebCore::LineLayoutState::setEndLineLogicalTop):
+ (WebCore::LineLayoutState::endLine):
+ (WebCore::LineLayoutState::setEndLine):
+ (WebCore::LineLayoutState::lastFloat):
+ (WebCore::LineLayoutState::setLastFloat):
+ (WebCore::LineLayoutState::floats):
+ (WebCore::LineLayoutState::floatIndex):
+ (WebCore::LineLayoutState::setFloatIndex):
+ (WebCore::RenderBlock::layoutRunsAndFloats):
+ (WebCore::RenderBlock::layoutRunsAndFloatsInRange):
+ (WebCore::RenderBlock::linkToEndLineIfNeeded):
+ (WebCore::RenderBlock::repaintDirtyFloats):
+ (WebCore::RenderBlock::layoutInlineChildren):
+ (WebCore::RenderBlock::determineStartPosition):
+ (WebCore::RenderBlock::determineEndPosition):
+ (WebCore::RenderBlock::matchedEndLine):
+
+2011-08-02 Nat Duca <nduca@chromium.org>
+
+ [chromium] Remove resizeOnscreenContent from LRC. It can be derived.
+ https://bugs.webkit.org/show_bug.cgi?id=65519
+
+ Reviewed by James Robinson.
+
+ * platform/graphics/chromium/LayerRendererChromium.cpp:
+ (WebCore::LayerRendererChromium::setViewport):
+ * platform/graphics/chromium/LayerRendererChromium.h:
+
+2011-08-02 Nico Weber <thakis@chromium.org>
+
+ [chromium] Fix unresolved symbols error when linking to libwebkit.dylib
+ https://bugs.webkit.org/show_bug.cgi?id=65498
+
+ Reviewed by Dimitri Glazkov.
+
+ * WebCore.gyp/mac/adjust_visibility.sh:
+
+2011-08-02 Mihnea Ovidenie <mihnea@adobe.com>
+
+ hover then un-hover makes state change
+ https://bugs.webkit.org/show_bug.cgi?id=56401
+
+ Reviewed by David Hyatt.
+
+ When a 'before' pseudo-element is re-added, we should check whether the insertion point is an anonymous
+ block with inline children. If it is, then we should change the insertion point to the first child of the
+ anonymous block, otherwise the 'before' pseudo-element ends up in a different block. We choose the insertion
+ point to be the first child only if the anonymous block has children, otherwise the before element ends up
+ in a wrong block.
+
+ Tests: fast/dynamic/hover-before-position-after-style-change.html
+ fast/dynamic/hover-before-position-after-style-change2.html
+
+ * rendering/RenderObjectChildList.cpp:
+ (WebCore::RenderObjectChildList::updateBeforeAfterContent):
+
+2011-08-02 Andreas Kling <kling@webkit.org>
+
+ [Qt] Simplify Path::translate() implementation.
+ https://bugs.webkit.org/show_bug.cgi?id=65540
+
+ Reviewed by Benjamin Poulain.
+
+ * platform/graphics/qt/PathQt.cpp:
+ (WebCore::Path::translate):
+
+2011-07-30 Pavel Podivilov <podivilov@chromium.org>
+
+ Web Inspector: merge FormattedSourceFile into SourceFile.
+ https://bugs.webkit.org/show_bug.cgi?id=65411
+
+ Reviewed by Pavel Feldman.
+
+ Prepare for splitting SourceFile into RawSourceCode and UISourceCode:
+ * merge FormattedSourceFile into SourceFile
+ * rename SourceFile to RawSourceCode
+ * hide formatter worker lazy creation in ScriptFormatter
+
+ * inspector/front-end/DebuggerPresentationModel.js:
+ (WebInspector.DebuggerPresentationModel):
+ (WebInspector.DebuggerPresentationModel.prototype._scriptLocationToUILocation):
+ (WebInspector.DebuggerPresentationModel.prototype._uiLocationToScriptLocation):
+ (WebInspector.DebuggerPresentationModel.prototype._addScript):
+ * inspector/front-end/ScriptFormatter.js:
+ (WebInspector.ScriptFormatter):
+ (WebInspector.ScriptFormatter.prototype._didFormatContent):
+ (WebInspector.ScriptFormatter.prototype.get _worker):
+ * inspector/front-end/SourceFile.js:
+ (WebInspector.RawSourceCode):
+ (WebInspector.RawSourceCode.prototype.setFormatted):
+ (WebInspector.RawSourceCode.prototype.createSourceMappingIfNeeded.didRequestContent):
+ (WebInspector.RawSourceCode.prototype.createSourceMappingIfNeeded):
+ (WebInspector.RawSourceCode.prototype._didRequestContent.didFormatContent):
+ (WebInspector.RawSourceCode.prototype._didRequestContent):
+ (WebInspector.RawSourceCode.prototype._invokeRequestContentCallbacks):
+
+2011-08-02 Mihnea Ovidenie <mihnea@adobe.com>
+
+ [CSSRegions]Fix build after changeset 92106
+ https://bugs.webkit.org/show_bug.cgi?id=65539
+
+ Reviewed by Adam Roben.
+
+ No new tests since it is just a build fix.
+
+ * css/CSSStyleSelector.cpp:
+ (WebCore::CSSStyleSelector::applyProperty):
+
+2011-08-01 Vsevolod Vlasov <vsevik@chromium.org>
+
+ Web Inspector: [REGRESSION] Console evaluations leading to resource loading cause errors.
+ https://bugs.webkit.org/show_bug.cgi?id=65466
+
+ Reviewed by Pavel Feldman.
+
+ Test: http/tests/inspector/network/network-initiator-from-console.html
+
+ * inspector/front-end/NetworkPanel.js:
+ (WebInspector.NetworkDataGridNode.prototype._refreshInitiatorCell):
+
+2011-08-02 Alexander Pavlov <apavlov@chromium.org>
+
+ Web Inspector: img[width=90] or align attributes shouldnt be shown above element.style block (as they have lower specificity)
+ https://bugs.webkit.org/show_bug.cgi?id=65514
+
+ Reviewed by Pavel Feldman.
+
+ * inspector/front-end/StylesSidebarPane.js:
+
+2011-08-02 Gyuyoung Kim <gyuyoung.kim@samsung.com>
+
+ [EFL] Add fullscreen button to media control UI.
+ https://bugs.webkit.org/show_bug.cgi?id=64428
+
+ Implement paintMediaFullscreenButton and emit fullscreen signal.
+
+ Reviewed by Antonio Gomes.
+
+ * platform/efl/RenderThemeEfl.cpp:
+ (WebCore::RenderThemeEfl::edjeGroupFromFormType):
+ (WebCore::RenderThemeEfl::emitMediaButtonSignal):
+ (WebCore::RenderThemeEfl::paintMediaFullscreenButton):
+ * platform/efl/RenderThemeEfl.h:
+
+2011-08-01 Andrey Kosyakov <caseq@chromium.org>
+
+ Web Inspector: [Extensions API] webInspector.inspectedPage.reload() is broken
+ https://bugs.webkit.org/show_bug.cgi?id=65476
+
+ - fixed webInspector.inspectedWindow.reload(), brought tests back.
+ - removed redundant setting of user agent in FrameLoader::loadResourceSynchronously()
+
+ Reviewed by Yury Semikhatsky.
+
+ Test: http/tests/inspector/extensions-useragent.html
+
+ * inspector/front-end/ExtensionServer.js:
+ (WebInspector.ExtensionServer.prototype._onReload):
+ * loader/FrameLoader.cpp:
+ (WebCore::FrameLoader::loadResourceSynchronously):
+
+2011-08-02 Pavel Feldman <pfeldman@chromium.org>
+
+ Not reviewed: fixing inspector tests.
+
+ * inspector/front-end/ScriptsPanel.js:
+ (WebInspector.ScriptsPanel.prototype._addOptionToFilesSelect):
+
+2011-08-02 Pavel Feldman <pfeldman@google.com>
+
+ Not reviewed: one line inspector style change.
+
+ * inspector/front-end/networkLogView.css:
+ (.network-error-row, .network-error-row .network-cell-subtitle):
+
+2011-08-02 Amruth Raj <amruthraj@motorola.com>
+
+ Custom cursors cause the WebProcess to crash
+ https://bugs.webkit.org/show_bug.cgi?id=64802
+
+ Reviewed by Martin Robinson.
+
+ * WebCore.exp.in:
+
+2011-08-02 Pavel Feldman <pfeldman@google.com>
+
+ Web Inspector: _normalizeEventTypes (via console's monitorEvents) doesnt handle all events like 'search' and other atypical ones.
+ https://bugs.webkit.org/show_bug.cgi?id=65468
+
+ Reviewed by Yury Semikhatsky.
+
+ * inspector/InjectedScriptSource.js:
+
+2011-08-02 Pavel Feldman <pfeldman@google.com>
+
+ Web Inspector: 404/errors should get a RED visual treatment so easier to identify.
+ https://bugs.webkit.org/show_bug.cgi?id=65471
+
+ Reviewed by Yury Semikhatsky.
+
+ * inspector/front-end/NetworkPanel.js:
+ (WebInspector.NetworkDataGridNode.prototype._refreshStatusCell):
+ * inspector/front-end/networkLogView.css:
+ (.network-error-cell):
+
+2011-08-02 Pavel Feldman <pfeldman@google.com>
+
+ Web Inspector: arrow is a pain to click in console. Drive-by selection reset fix.
+ https://bugs.webkit.org/show_bug.cgi?id=65467
+
+ Reviewed by Yury Semikhatsky.
+
+ * inspector/front-end/ConsoleView.js:
+ (WebInspector.ConsoleView.prototype._messagesClicked):
+ * inspector/front-end/ObjectPropertiesSection.js:
+ (WebInspector.ObjectPropertyTreeElement):
+ * inspector/front-end/inspector.css:
+ (.properties-tree li):
+ (.properties-tree li.parent::before):
+ * inspector/front-end/treeoutline.js:
+ (TreeElement.treeElementToggled):
+
+2011-08-01 Pavel Feldman <pfeldman@google.com>
+
+ Web Inspector: button + "New style rule"
+ https://bugs.webkit.org/show_bug.cgi?id=65409
+
+ Reviewed by Yury Semikhatsky.
+
+ Test: inspector/styles/styles-add-new-rule.html
+
+ * inspector/front-end/StylesSidebarPane.js:
+ (WebInspector.StylesSidebarPane):
+ (WebInspector.BlankStylePropertiesSection.prototype.editingSelectorCommitted):
+
+2011-08-01 Luke Macpherson <macpherson@chromium.org>
+
+ Don't set m_fontDirty when setting zoom unless zoom has actually changed
+ https://bugs.webkit.org/show_bug.cgi?id=65092
+
+ Reviewed by Darin Adler.
+
+ No new tests as no functionality changed - this is an optimization that
+ should be logically equivalent to the current code.
+
+ The intent here is to avoid setting m_fontDirty unless the fornt information is actually dirty.
+
+ * css/CSSStyleSelector.cpp:
+ (WebCore::CSSStyleSelector::styleForDocument):
+ (WebCore::CSSStyleSelector::applyProperty):
+ Use setZoom and setEffectiveZoom wrapper functions.
+ * css/CSSStyleSelector.h:
+ (WebCore::CSSStyleSelector::setZoom):
+ Wrapper for m_style->setZoom() that automatically updates m_fontDirty.
+ (WebCore::CSSStyleSelector::setEffectiveZoom):
+ Wrapper for m_style->setEffectiveZoom that automatically updates m_fontDirty.
+ * page/animation/AnimationBase.cpp:
+ (WebCore::AnimationBase::ensurePropertyMap):
+ Use
+ * rendering/style/RenderStyle.h:
+ (WebCore::InheritedFlags::setZoom):
+ Return true only if underlying values change.
+ (WebCore::InheritedFlags::setZoomWithoutReturnValue):
+ Return void to match function pointer type where required.
+ (WebCore::InheritedFlags::setEffectiveZoom):
+ Return true only if underlying values change.
+
+2011-08-01 Jean-luc Brouillet <jeanluc@chromium.org>
+
+ Removing old source files in gyp files that slow build
+ https://bugs.webkit.org/show_bug.cgi?id=65503
+
+ Reviewed by Adam Barth.
+
+ A number of stale files are listed in the gyp files. These slow the
+ build on Visual Studio 2010. Removing them.
+
+ * WebCore.gypi:
+
+2011-08-01 Bradley Nelson <bradnelson@chromium.org>
+
+ Shard webcore_remaining for msvs official builds
+
+ https://bugs.webkit.org/show_bug.cgi?id=65475
+
+ Reviewed by Darin Fisher.
+
+ webcore_remaining has consistently exceeded Visual Studio 2008's
+ capacity to link when link time code generation is turned on (full
+ release builds). This has been dealt with the past by carving off
+ larger and larger pieces of webcore_remaining (ie webcore_html,
+ webcore_renderer etc).
+ A new option has been added to gyp to automate this sharding:
+ 'msvs_shard': SHARD_COUNT,
+ Targets marked in this way will be built in several pieces
+ (target_0, target_1...).
+ Since the resulting project is less comprehensible in the IDE (and
+ since the linking limitation only afflicts official builds), this
+ option is only needed for those builds.
+ This patch shards webcore_remaining into 10 parts for official builds.
+
+ * WebCore.gyp/WebCore.gyp:
+
+2011-08-01 Alice Boxhall <aboxhall@chromium.org>
+
+ WAI-ARIA textbox role should be AXTextArea for aria-multiline=true, AXTextField for aria-multiline=false
+ https://bugs.webkit.org/show_bug.cgi?id=65451
+
+ Reviewed by Chris Fleizach.
+
+ * accessibility/AccessibilityRenderObject.cpp:
+ (WebCore::AccessibilityRenderObject::determineAriaRoleAttribute):
+
+2011-08-01 srikumar bonda <srikumar.b@gmail.com>
+
+ Use threadsafeCopy() API when using Database Directory path variable
+ https://bugs.webkit.org/show_bug.cgi?id=65436
+
+ Reviewed by David Levin.
+
+ Use threadsafeCopy() API while using m_databaseDirectoryPath string variable
+ when the databaseGuard lock hasn't been taken in DatabaseTracker class.
+ This is just a safety fix.
+
+ No changes to functionality so no new tests.
+
+ * storage/DatabaseTracker.cpp:
+ (WebCore::DatabaseTracker::trackerDatabasePath):
+ (WebCore::DatabaseTracker::deleteOrigin):
+
+2011-08-01 Michael Nordman <michaeln@google.com>
+
+ [Chromium] WebSQLDatabase version handling is broken in multi-process browsers.
+ https://bugs.webkit.org/show_bug.cgi?id=65486
+
+ The WebCore::AbstractDatabase class maintains a global in-memory map of
+ the version numbers associated with open database files, but that map is
+ not reliable in a multi-process system like Chrome. So instead of relying
+ on the cached values in that map, we read the value from the database (and
+ update the cached value) where possible. There are two edge cases where that's
+ not possible because the scriptable interface requires synchronous access
+ to the version: the .version attribute getter and the .openDatabase() method.
+ In those cases, we have no choice but to use the potentially stale cached value.
+
+ Reviewed by Darin Fisher.
+
+ No new tests. Existing layout tests cover the version handling functionality.
+
+ * storage/AbstractDatabase.cpp:
+ (WebCore::AbstractDatabase::version):
+ (WebCore::AbstractDatabase::performOpenAndVerify):
+ (WebCore::AbstractDatabase::getVersionFromDatabase):
+ (WebCore::AbstractDatabase::setVersionInDatabase):
+ (WebCore::AbstractDatabase::setExpectedVersion):
+ (WebCore::AbstractDatabase::getCachedVersion):
+ (WebCore::AbstractDatabase::setCachedVersion):
+ (WebCore::AbstractDatabase::getActualVersionForTransaction):
+ * storage/AbstractDatabase.h:
+ (WebCore::AbstractDatabase::expectedVersion):
+ * storage/ChangeVersionWrapper.cpp:
+ (WebCore::ChangeVersionWrapper::handleCommitFailedAfterPostflight):
+ * storage/ChangeVersionWrapper.h:
+ * storage/Database.cpp:
+ (WebCore::Database::openDatabase):
+ * storage/DatabaseSync.cpp:
+ (WebCore::DatabaseSync::openDatabaseSync):
+ (WebCore::DatabaseSync::changeVersion):
+ * storage/SQLTransaction.cpp:
+ (WebCore::SQLTransaction::SQLTransaction):
+ (WebCore::SQLTransaction::executeSQL):
+ (WebCore::SQLTransaction::openTransactionAndPreflight):
+ (WebCore::SQLTransaction::runCurrentStatement):
+ (WebCore::SQLTransaction::postflightAndCommit):
+ * storage/SQLTransaction.h:
+ * storage/SQLTransactionSync.cpp:
+ (WebCore::SQLTransactionSync::SQLTransactionSync):
+ (WebCore::SQLTransactionSync::executeSQL):
+ (WebCore::SQLTransactionSync::begin):
+ * storage/SQLTransactionSync.h:
+
+2011-08-01 Tim Horton <timothy_horton@apple.com>
+
+ Severe shadow repaint issues with SVGText elements
+ https://bugs.webkit.org/show_bug.cgi?id=63648
+ <rdar://problem/7632269>
+
+ Reviewed by Darin Adler.
+
+ Make SVG elements which inherit from RenderSVGBlock take text-shadow
+ overflow into account when determining whether or not they should draw
+
+ Test: svg/custom/repaint-shadow.svg
+
+ * rendering/RenderBox.h:
+ (WebCore::RenderBox::visualOverflowRect):
+ * rendering/svg/RenderSVGBlock.cpp:
+ (WebCore::RenderSVGBlock::visualOverflowRect):
+ * rendering/svg/RenderSVGBlock.h:
+
+2011-08-01 Zhenyao Mo <zmo@google.com>
+
+ Unreviewed, rolling out r92022.
+ http://trac.webkit.org/changeset/92022
+ https://bugs.webkit.org/show_bug.cgi?id=65203
+
+ try to see if this is causing a bunch of browser tests failues
+ on win bots - will recommit if not
+
+ * platform/graphics/chromium/FontChromiumWin.cpp:
+ (WebCore::TransparencyAwareFontPainter::TransparencyAwareFontPainter::TransparencyAwareFontPainter):
+ (WebCore::TransparencyAwareFontPainter::TransparencyAwareFontPainter::init):
+ (WebCore::TransparencyAwareFontPainter::TransparencyAwareFontPainter::initializeForGDI):
+ (WebCore::TransparencyAwareFontPainter::TransparencyAwareFontPainter::~TransparencyAwareFontPainter):
+ (WebCore::TransparencyAwareFontPainter::TransparencyAwareGlyphPainter::TransparencyAwareGlyphPainter):
+ (WebCore::TransparencyAwareFontPainter::TransparencyAwareGlyphPainter::~TransparencyAwareGlyphPainter):
+ (WebCore::TransparencyAwareFontPainter::TransparencyAwareGlyphPainter::drawGlyphs):
+ (WebCore::TransparencyAwareFontPainter::TransparencyAwareUniscribePainter::hdc):
+ (WebCore::TransparencyAwareFontPainter::TransparencyAwareUniscribePainter::TransparencyAwareUniscribePainter):
+ (WebCore::TransparencyAwareFontPainter::TransparencyAwareUniscribePainter::~TransparencyAwareUniscribePainter):
+ (WebCore::drawGlyphsWin):
+ (WebCore::Font::drawComplexText):
+ * platform/graphics/chromium/UniscribeHelper.cpp:
+ (WebCore::UniscribeHelper::draw):
+ * platform/graphics/skia/PlatformContextSkia.cpp:
+ (WebCore::PlatformContextSkia::isNativeFontRenderingAllowed):
+ * platform/graphics/skia/PlatformContextSkia.h:
+ * platform/graphics/skia/SkiaFontWin.cpp:
+ (WebCore::windowsCanHandleDrawTextShadow):
+ (WebCore::windowsCanHandleTextDrawing):
+ (WebCore::windowsCanHandleTextDrawingWithoutShadow):
+ * platform/graphics/skia/SkiaFontWin.h:
+
+2011-08-01 Scott Graham <scottmg@chromium.org>
+
+ REGRESSION (r39725?): Resources removed from document can not be freed
+ until the document is deleted
+ https://bugs.webkit.org/show_bug.cgi?id=61006
+
+ Reviewed by Antti Koivisto.
+
+ Upon completing a load start a Timer to iterate through
+ CachedResourceLoader's m_documentResources map to check for any items
+ that have only one reference (thus being the reference in the map
+ itself). The map should really be weak, but because the
+ CachedResourceHandle achieves bookkeeping work in addition to
+ reference counting, this is a simpler and more localized way to free
+ the used memory while maintaining the other behaviour (when
+ CachedResource is used as proxy).
+
+ With this patch the testcase at
+ https://bugs.webkit.org/attachment.cgi?id=93850 should no longer
+ consume 400MB of ram on load. Test added for crash discovered in
+ previous revision, but no tests for memory usage.
+
+ Test: http/tests/inspector/network/disabled-cache-crash.html
+
+ * WebCore.exp.in:
+ * loader/cache/CachedResource.h:
+ (WebCore::CachedResource::hasOneHandle):
+ * loader/cache/CachedResourceLoader.cpp:
+ (WebCore::CachedResourceLoader::CachedResourceLoader):
+ (WebCore::CachedResourceLoader::loadDone):
+ (WebCore::CachedResourceLoader::garbageCollectDocumentResourcesTimerFired):
+ * loader/cache/CachedResourceLoader.h:
+ * testing/Internals.cpp:
+ (WebCore::Internals::disableMemoryCache):
+ * testing/Internals.h:
+ * testing/Internals.idl:
+
+2011-08-01 Jochen Eisinger <jochen@chromium.org>
+
+ Never override the policy URL on form submissions.
+ https://bugs.webkit.org/show_bug.cgi?id=61809
+
+ Reviewed by Adam Barth.
+
+ Tests: http/tests/security/cookies/third-party-cookie-blocking-main-frame.html
+ http/tests/security/cookies/third-party-cookie-blocking-user-action.html
+ http/tests/security/cookies/third-party-cookie-blocking.html
+
+ * loader/FrameLoader.cpp:
+ (WebCore::FrameLoader::loadURL):
+ (WebCore::FrameLoader::addExtraFieldsToSubresourceRequest):
+ (WebCore::FrameLoader::addExtraFieldsToMainResourceRequest):
+ (WebCore::FrameLoader::addExtraFieldsToRequest):
+ (WebCore::FrameLoader::loadPostRequest):
+ (WebCore::FrameLoader::loadDifferentDocumentItem):
+ * loader/FrameLoader.h:
+
+
+2011-08-01 Ryosuke Niwa <rniwa@webkit.org>
+
+ Update comment added in r92139 per Darin's suggestion.
+
+ * css/CSSStyleSelector.cpp:
+ (WebCore::CSSStyleSelector::styleForElement):
+
+2011-08-01 Ryosuke Niwa <rniwa@webkit.org>
+
+ Search field in designMode causes a crash
+ https://bugs.webkit.org/show_bug.cgi?id=65362
+
+ Reviewed by Dimitri Glazkov.
+
+ The crashed was caused by editing code inadvertently deleting search and cancel buttons in the design mode.
+ Fixed the bug by avoid inheriting user-modify property from the shadow host.
+
+ Test: editing/input/search-field-crash-in-designmode.html
+
+ * css/CSSStyleSelector.cpp:
+ (WebCore::isAtShadowBoundary): Moved.
+ (WebCore::CSSStyleSelector::styleForElement): Overrides -webkit-user-modify by the initial value after m_style
+ inherited the values from m_parentStyle but before applying matched rules.
+ * dom/Node.cpp:
+ (WebCore::Node::rendererIsEditable): Ignore page's editability inside a shadow DOM.
+
+2011-08-01 Marco Peereboom <marco@peereboom.us>
+
+ [Soup] Set default max-conns to a more sane default value and fix comment
+ https://bugs.webkit.org/show_bug.cgi?id=65335
+
+ Reviewed by Martin Robinson.
+
+ Rigged the xxxterm web browser and validated that the default values were set properly.
+
+ * platform/network/soup/ResourceHandleSoup.cpp:
+ (WebCore::ResourceHandle::defaultSession):
+
+2011-07-28 Abhishek Arya <inferno@chromium.org>
+
+ Regression(82144): Crash in TrailingObjects::updateMidpointsForTrailingBoxes
+ https://bugs.webkit.org/show_bug.cgi?id=65137
+
+ Fix the looping condition to prevent trailingSpaceMidpoint from becoming negative.
+
+ Reviewed by Dave Hyatt.
+
+ Test: fast/block/update-midpoints-for-trailing-boxes-crash.html
+
+ * rendering/RenderBlockLineLayout.cpp:
+ (WebCore::TrailingObjects::updateMidpointsForTrailingBoxes):
+
+2011-08-01 Benjamin Poulain <benjamin@webkit.org>
+
+ [Qt] OpenGLShims does not build on ARM
+ https://bugs.webkit.org/show_bug.cgi?id=65465
+
+ Reviewed by Noam Rosenthal.
+
+ Fix the build with OpenGL ES 2:
+ -lookupOpenGLFunctionAddress() was defined but not used for OpenGL ES.
+ -glBlitFramebuffer() and glRenderbufferStorageMultisample() are not part of the specification.
+ When those are available as platform extension, the extension has been added.
+ -GLchar is not defined on some platform. The patch adds the same typedef as the official definition
+ to avoid conflicts.
+
+ * platform/graphics/cairo/OpenGLShims.cpp:
+ (lookupOpenGLFunctionAddress):
+ (WebCore::initializeOpenGLShims):
+ * platform/graphics/cairo/OpenGLShims.h:
+
+2011-08-01 Yury Semikhatsky <yurys@chromium.org>
+
+ Unreviewed. Trivial fix for inspector/debugger/debugger-completions-on-call-frame.html after r92122.
+
+ * inspector/front-end/ConsoleView.js:
+
+2011-08-01 Hayato Ito <hayato@chromium.org>
+
+ Add support for getting an element in shadow root by its id into a window.internals object.
+ https://bugs.webkit.org/show_bug.cgi?id=64587
+
+ Reviewed by Hajime Morita.
+
+ Test: fast/dom/shadow/get-element-by-id-in-shadow-root.html
+
+ * WebCore.exp.in:
+ * testing/Internals.cpp:
+ (WebCore::Internals::getElementByIdInShadowRoot):
+ * testing/Internals.h:
+ * testing/Internals.idl:
+
+2011-07-15 Hayato Ito <hayato@chromium.org>
+
+ Add support for retrieving an element in TreeScope by id to window.internals object.
+ https://bugs.webkit.org/show_bug.cgi?id=64587
+
+ Reviewed by Hajime Morita.
+
+ Test: fast/dom/shadow/get-element-by-id-in-shadow.html
+
+ * WebCore.exp.in:
+ * testing/Internals.cpp:
+ (WebCore::Internals::getElementByIdInTreeScope):
+ * testing/Internals.h:
+ * testing/Internals.idl:
+
+2011-08-01 Yury Semikhatsky <yurys@chromium.org>
+
+ Web Inspector: typing an expression in an iframe leads to multiple "Unsafe JavaScript attempt to access frame..." errors
+ https://bugs.webkit.org/show_bug.cgi?id=65457
+
+ Console completions are now done using evaluation which returns a JSON object with all property names rather than a remote
+ object. Also Runtime.evaluate and Runtime.callFunctionOn commands were extended with an optional parameter that allows to
+ get result as JSON value.
+
+ Reviewed by Pavel Feldman.
+
+ Test: http/tests/inspector/console-cd-completions.html
+
+ * inspector/InjectedScript.cpp:
+ (WebCore::InjectedScript::evaluate):
+ (WebCore::InjectedScript::callFunctionOn):
+ * inspector/InjectedScript.h:
+ * inspector/InjectedScriptSource.js:
+ (.):
+ ():
+ * inspector/Inspector.json:
+ * inspector/InspectorRuntimeAgent.cpp:
+ (WebCore::asBool):
+ (WebCore::InspectorRuntimeAgent::evaluate):
+ (WebCore::InspectorRuntimeAgent::callFunctionOn):
+ * inspector/InspectorRuntimeAgent.h:
+ * inspector/front-end/ConsoleView.js:
+ (WebInspector.ConsoleView.prototype._completions.evaluated.getCompletions):
+ (WebInspector.ConsoleView.prototype._completions.evaluated):
+ (WebInspector.ConsoleView.prototype._completions.receivedPropertySet):
+ (WebInspector.ConsoleView.prototype._completions):
+ (WebInspector.ConsoleView.prototype.evalInInspectedWindow):
+ * inspector/front-end/RemoteObject.js:
+ (WebInspector.RemoteObject):
+ (WebInspector.RemoteObject.prototype.callFunction):
+ (WebInspector.RemoteObject.prototype.callFunctionJSON):
+ * inspector/front-end/WatchExpressionsSidebarPane.js:
+ (WebInspector.WatchExpressionsSection.prototype.update):
+
+2011-08-01 Mihnea Ovidenie <mihnea@adobe.com>
+
+ BORDER attribute with the object tag, using percentage values not working.
+ https://bugs.webkit.org/show_bug.cgi?id=65176
+
+ Reviewed by Hajime Morita.
+
+ When border presentational attribute for object element has % in it, it should be parsed using HTML parser rules.
+
+ Test: fast/borders/border-width-percent.html
+
+ * html/HTMLElement.cpp:
+ (WebCore::HTMLElement::parseBorderWidthAttribute):
+ * html/HTMLElement.h:
+ * html/HTMLImageElement.cpp:
+ * html/HTMLObjectElement.cpp:
+ (WebCore::HTMLObjectElement::parseMappedAttribute):
+
+2011-08-01 Yuta Kitamura <yutak@chromium.org>
+
+ WebSocket: Receive URL and subprotocol in WebSocketChannel::connect()
+ https://bugs.webkit.org/show_bug.cgi?id=65367
+
+ Reviewed by Kent Tamura.
+
+ To implement multiple subprotocols support (bug 65247), WebSocket::connect() will need to validate
+ the value of subprotocols after constructing WebSocketChannel, because the result depends on which
+ WebSocket protocol is used, which is obtained from WebSocketChannel::useHixie76Protocol(). This
+ means the subprotocol value will not be available at the time of WebSocketChannel construction.
+
+ This change moves URL and subprotocol arguments in WebSocketChannel constructor to
+ WebSocketChannel::connect(), which allows WebSocket::connect() function to check the subprotocol
+ value before the actual connection is established.
+
+ Relocating URL argument is technically not necessary, but seemed legitimate in terms of functional
+ correspondence between WebSocket and WebSocketChannel (constructor versus connect()).
+
+ No change in behavior, thus no new tests.
+
+ * websockets/ThreadableWebSocketChannel.cpp:
+ Remove "url" and "protocol" arguments from constructor and add them in connect().
+ (WebCore::ThreadableWebSocketChannel::create):
+ * websockets/ThreadableWebSocketChannel.h:
+ * websockets/WebSocket.cpp:
+ (WebCore::WebSocket::connect):
+ * websockets/WebSocketChannel.cpp: Same as ThreadableWebSocketChannel.
+ (WebCore::WebSocketChannel::WebSocketChannel):
+ (WebCore::WebSocketChannel::connect):
+ InspectorInstrumentation::didCreateWebSocket() call was moved to connect() because it needs URL.
+ This does not change behavior, because connect() is guaranteed to be called immediately after
+ WebSocketChannel is constructed.
+ * websockets/WebSocketChannel.h:
+ (WebCore::WebSocketChannel::create):
+ * websockets/WorkerThreadableWebSocketChannel.cpp: Same as ThreadableWebSocketChannel.
+ (WebCore::WorkerThreadableWebSocketChannel::WorkerThreadableWebSocketChannel):
+ (WebCore::WorkerThreadableWebSocketChannel::connect):
+ (WebCore::WorkerThreadableWebSocketChannel::Peer::Peer):
+ (WebCore::WorkerThreadableWebSocketChannel::Peer::connect):
+ (WebCore::WorkerThreadableWebSocketChannel::Bridge::mainThreadCreateWebSocketChannel):
+ (WebCore::WorkerThreadableWebSocketChannel::Bridge::Bridge):
+ (WebCore::WorkerThreadableWebSocketChannel::mainThreadConnect):
+ (WebCore::WorkerThreadableWebSocketChannel::Bridge::connect):
+ * websockets/WorkerThreadableWebSocketChannel.h:
+ (WebCore::WorkerThreadableWebSocketChannel::create):
+ (WebCore::WorkerThreadableWebSocketChannel::Peer::create):
+ (WebCore::WorkerThreadableWebSocketChannel::Bridge::create):
+
+2011-08-01 Luke Macpherson <macpherson@chromium.org>
+
+ Add iterator to CSSValueList
+ https://bugs.webkit.org/show_bug.cgi?id=65297
+
+ Reviewed by Darin Adler.
+
+ No new tests / refactoring only.
+
+ * css/CSSPrimitiveValue.h:
+ (WebCore::CSSPrimitiveValue::isLength):
+ Add shorthand to determine if this primitive value is a length.
+ * css/CSSStyleSelector.cpp:
+ Use CSSValueListIterator throughout.
+ (WebCore::CSSStyleSelector::applyProperty):
+ (WebCore::CSSStyleSelector::applyPageSizeProperty):
+ (WebCore::CSSStyleSelector::createTransformOperations):
+ * css/CSSValueList.cpp:
+ (WebCore::CSSValueList::copy):
+ Use itemWithoutBoundsCheck() instead of item().
+ * css/CSSValueList.h:
+ Add CSSValueListIterator and CSSValueListInspector class definitions.
+ (WebCore::CSSValueList::item)
+ Provide inline definition of item.
+ (WebCore::CSSValueListIterator::CSSValueListIterator):
+ (WebCore::CSSValueListIterator::hasMore):
+ Return true if there are more values to consume, including the current value.
+ (WebCore::CSSValueListIterator::value):
+ Return the value at the current position.
+ (WebCore::CSSValueListIterator::next):
+ Move the iterator forward to the next item.
+ (WebCore::CSSValueListIterator::index):
+ Return the current position in the list.
+ (WebCore::CSSValueListInspector::item):
+ Return the item at a given index.
+ (WebCore::CSSValueListInspector::first):
+ Return the first item in the list.
+ (WebCore::CSSValueListInspector::second):
+ Return the second item in the list.
+ (WebCore::CSSValueListInspector::length):
+ Return the size of the underlying list.
+
+2011-08-01 Pavel Feldman <pfeldman@google.com>
+
+ Web Inspector: group scripts by folder in the scripts selector.
+ https://bugs.webkit.org/show_bug.cgi?id=65321
+
+ Reviewed by Yury Semikhatsky.
+
+ Test: inspector/debugger/scripts-sorting.html
+
+ * inspector/front-end/ScriptsPanel.js:
+ (WebInspector.ScriptsPanel.prototype._sourceFileAdded.get if):
+ (WebInspector.ScriptsPanel.prototype._addOptionToFilesSelect.insertOrdered.optionCompare):
+ (WebInspector.ScriptsPanel.prototype._addOptionToFilesSelect.insertOrdered):
+ (WebInspector.ScriptsPanel.prototype._addOptionToFilesSelect):
+ (WebInspector.ScriptsPanel.prototype._folderAndDisplayNameForScriptURL):
+ (WebInspector.ScriptsPanel.prototype.reset):
+ (WebInspector.SourceFrameDelegateForScriptsPanel.prototype.suggestedFileName):
+
+2011-07-28 Pavel Feldman <pfeldman@google.com>
+
+ Web Inspector: [protocol] methods where all parameters are optional should not require empty params object.
+ https://bugs.webkit.org/show_bug.cgi?id=65331
+
+ Reviewed by Yury Semikhatsky.
+
+ * inspector/CodeGeneratorInspector.pm:
+ (generateBackendFunction):
+ (generateArgumentGetters):
+
+2011-08-01 Adam Barth <abarth@webkit.org>
+
+ application/json MIME type shouldn't cause warnings in <script> tags
+ https://bugs.webkit.org/show_bug.cgi?id=65448
+
+ Reviewed by Pavel Feldman.
+
+ This patch stops the web inspector from logging a ton of spam warnings
+ when a web page uses a JSONP service (which points script tags at
+ application/json resources).
+
+ * inspector/front-end/inspector.js:
+ (WebInspector.reset):
+
+2011-08-01 Yuta Kitamura <yutak@chromium.org>
+
+ WebSocket: Rename SocketStreamHandleClient member functions
+ https://bugs.webkit.org/show_bug.cgi?id=65374
+
+ Reviewed by Kent Tamura.
+
+ Rename SocketStreamHandleClient member functions to avoid conflict of the names of
+ FileReaderLoaderClient functions.
+
+ No change in behavior, thus no new tests are added.
+
+ * platform/network/SocketStreamHandleClient.h:
+ (WebCore::SocketStreamHandleClient::didOpenSocketStream):
+ (WebCore::SocketStreamHandleClient::didCloseSocketStream):
+ (WebCore::SocketStreamHandleClient::didReceiveSocketStreamData):
+ (WebCore::SocketStreamHandleClient::didFailSocketStream):
+ * platform/network/brew/SocketStreamHandleBrew.cpp:
+ (WebCore::SocketStreamHandlePrivate::socketConnected):
+ (WebCore::SocketStreamHandlePrivate::socketReadyRead):
+ (WebCore::SocketStreamHandlePrivate::socketClosed):
+ (WebCore::SocketStreamHandlePrivate::socketError):
+ * platform/network/cf/SocketStreamHandleCFNet.cpp:
+ (WebCore::SocketStreamHandle::addCONNECTCredentials):
+ (WebCore::SocketStreamHandle::readStreamCallback):
+ (WebCore::SocketStreamHandle::writeStreamCallback):
+ (WebCore::SocketStreamHandle::reportErrorToClient):
+ (WebCore::SocketStreamHandle::platformClose):
+ * platform/network/qt/SocketStreamHandleQt.cpp:
+ (WebCore::SocketStreamHandlePrivate::socketConnected):
+ (WebCore::SocketStreamHandlePrivate::socketReadyRead):
+ (WebCore::SocketStreamHandlePrivate::socketClosedCallback):
+ (WebCore::SocketStreamHandlePrivate::socketErrorCallback):
+ * platform/network/soup/SocketStreamHandleSoup.cpp:
+ (WebCore::SocketStreamHandle::connected):
+ (WebCore::SocketStreamHandle::readBytes):
+ (WebCore::SocketStreamHandle::platformSend):
+ (WebCore::SocketStreamHandle::platformClose):
+ * websockets/WebSocketChannel.cpp:
+ (WebCore::WebSocketChannel::didOpenSocketStream):
+ (WebCore::WebSocketChannel::didCloseSocketStream):
+ (WebCore::WebSocketChannel::didReceiveSocketStreamData):
+ (WebCore::WebSocketChannel::didFailSocketStream):
+ (WebCore::WebSocketChannel::resumeTimerFired):
+ * websockets/WebSocketChannel.h:
+
+2011-07-31 Sheriff Bot <webkit.review.bot@gmail.com>
+
+ Unreviewed, rolling out r92094.
+ http://trac.webkit.org/changeset/92094
+ https://bugs.webkit.org/show_bug.cgi?id=65443
+
+ Triggered many asserts (Requested by abarth on #webkit).
+
+ * dom/EventDispatcher.cpp:
+ (WebCore::EventDispatcher::dispatchEvent):
+
+2011-07-31 Dominic Cooney <dominicc@chromium.org>
+
+ Event target set to null post event dispatch
+ https://bugs.webkit.org/show_bug.cgi?id=63934
+
+ Reviewed by Dimitri Glazkov.
+
+ Test: fast/events/event-target-reset.html
+
+ * dom/EventDispatcher.cpp:
+ (WebCore::EventDispatcher::dispatchEvent): Reset target to outermost
+ shadow host.
+
+2011-07-31 Sam Weinig <sam@webkit.org>
+
+ REGRESSION: getBoundingClientRect() method of Range incorrectly returns null for collapsed Range
+ https://bugs.webkit.org/show_bug.cgi?id=65324
+
+ Reviewed by Ryosuke Niwa.
+
+ Update to more closely match the spec.
+
+ * dom/Range.cpp:
+ (WebCore::Range::getClientRects):
+ Return an empty ClientRectList instead of null when the range is empty.
+
+ (WebCore::Range::getBoundingClientRect):
+ Always return a ClientRect, even for an empty bounding rect.
+
+2011-07-29 Ryosuke Niwa <rniwa@webkit.org>
+
+ [HTML5] Implement the selectionDirection property on input and textarea
+ https://bugs.webkit.org/show_bug.cgi?id=60403
+
+ Reviewed by Darin Adler.
+
+ Implements selectionDirection property as specified in HTML5 section 4.10.20:
+ http://www.whatwg.org/specs/web-apps/current-work/#textFieldSelection
+
+ selectionDirection is a property on input and textarea elements, which, on getting returns either "none",
+ "forward", or "backward" corresponding to the current selection direction, and on setting, modifies
+ the direction of the current selection. When selection doesn't exist or direction is unknown, "none"
+ is returned on getting. Setting the property value to neither of above values will result in setting "none".
+
+ This patch also adds the optional third argument, direction, to setSelectionRange.
+
+ Test: fast/forms/selection-direction.html
+ fast/forms/selection-wrongtype.html
+
+ * bindings/js/JSHTMLInputElementCustom.cpp:
+ (WebCore::JSHTMLInputElement::selectionDirection): Added.
+ (WebCore::JSHTMLInputElement::setSelectionDirection): Added.
+ (WebCore::JSHTMLInputElement::setSelectionRange): Processes the third argument.
+ * bindings/v8/custom/V8HTMLInputElementCustom.cpp:
+ (WebCore::V8HTMLInputElement::selectionDirectionAccessorGetter): Added.
+ (WebCore::V8HTMLInputElement::selectionDirectionAccessorSetter): Added.
+ (WebCore::V8HTMLInputElement::setSelectionRangeCallback): Processes the third argument.
+ * html/HTMLTextAreaElement.cpp:
+ (WebCore::HTMLTextAreaElement::updateFocusAppearance): Calls hasCachedSelection.
+ * html/HTMLInputElement.cpp:
+ (WebCore::HTMLInputElement::updateFocusAppearance): Ditto.
+ (WebCore::HTMLInputElement::setValue): Calls cacheSelection with SelectionHasNoDirection.
+ * html/HTMLTextAreaElement.idl: Added selectionDirection and the third argument to setSelectionRange.
+ * html/HTMLInputElement.idl: Ditto.
+ * html/HTMLTextFormControlElement.cpp:
+ (WebCore::HTMLTextFormControlElement::HTMLTextFormControlElement): Initializes
+ m_cachedSelectionDirection.
+ (WebCore::HTMLTextFormControlElement::setSelectionStart): Calls setSelectionRange.
+ (WebCore::HTMLTextFormControlElement::setSelectionEnd): Ditto.
+ (WebCore::HTMLTextFormControlElement::setSelectionDirection): Added.
+ (WebCore::HTMLTextFormControlElement::select): Calls setSelectionRange with SelectionHasNoDirection.
+ (WebCore::HTMLTextFormControlElement::setSelectionRange): Added the version that takes direction
+ string as the third argument, which calls the version that takes two ints and TextFieldSelectionDirection.
+ (WebCore::HTMLTextFormControlElement::selectionStart): Calls hasCachedSelection.
+ (WebCore::HTMLTextFormControlElement::selectionEnd): Ditto.
+ (WebCore::directionString): Added.
+ (WebCore::HTMLTextFormControlElement::selectionDirection): Added.
+ (WebCore::HTMLTextFormControlElement::computeSelectionDirection): Added.
+ (WebCore::HTMLTextFormControlElement::selection): Calls hasCachedSelection.
+ (WebCore::HTMLTextFormControlElement::restoreCachedSelection): Calls setSelectionRange.
+ (WebCore::HTMLTextFormControlElement::selectionChanged): Calls cacheSelection.
+ * html/HTMLTextFormControlElement.h:
+ (WebCore::HTMLTextFormControlElement::cacheSelection): Takes TextFieldSelectionDirection.
+ (WebCore::HTMLTextFormControlElement::hasCachedSelection): Extracted from hasCachedSelectionStart;
+ cacheSelection is the only function that modifies m_cacheSelectionStart and m_cacheSelectionEnd
+ and none of its callers ever set one of them be -1 so they are always both -1 or both non-negative.
+ Thus, having two separate functions for m_cacheSelectionStart and m_cacheSelectionEnd was redundant.
+
+2011-07-31 Daniel Bates <dbates@webkit.org>
+
+ [GTK] Remove unused variable coreChild in WebCore::getChildForTable()
+ https://bugs.webkit.org/show_bug.cgi?id=65427
+
+ This variable was unused when it was initially added in <http://trac.webkit.org/changeset/89660> (bug #62718)
+ and it is unused today. We should remove it. This will also resolve a compiler warning.
+
+ Reviewed by Xan Lopez.
+
+ * accessibility/gtk/AccessibilityObjectWrapperAtk.cpp:
+ (getChildForTable):
+
+2011-07-30 Daniel Bates <dbates@webkit.org>
+
+ Fix the WinCairo Debug build after <http://trac.webkit.org/changeset/91914>
+ (https://bugs.webkit.org/show_bug.cgi?id=64744)
+
+ * WebCore.vcproj/WebCore.vcproj: Remove ContextShadowCairo.cpp from the build since this
+ file was deleted in <http://trac.webkit.org/changeset/91914>.
+
+2011-07-30 Daniel Bates <dbates@webkit.org>
+
+ Fix the Chromium Linux Release Flexbot build after <http://trac.webkit.org/changeset/92004>
+ (https://bugs.webkit.org/show_bug.cgi?id=65342)
+
+ * rendering/RenderFlexibleBox.cpp:
+ (WebCore::RenderFlexibleBox::layoutBlock): Update the definition of RenderFlexibleBox::layoutBlock()
+ to be consistent with its declaration in RenderFlexibleBox.h; Add argument layoutPass of type BlockLayoutPass.
+
+2011-07-30 Adam Barth <abarth@webkit.org>
+
+ Update run-bindings-tests results after
+ http://trac.webkit.org/changeset/92068
+
+ * bindings/scripts/test/JS/JSTestCallback.cpp:
+
+2011-07-30 Patrick Gansterer <paroga@webkit.org>
+
+ Remove inclusion of MainThread.h from Threading.h
+ https://bugs.webkit.org/show_bug.cgi?id=65081
+
+ Reviewed by Darin Adler.
+
+ Add missing include statements for MainThread.
+
+ * bindings/js/DOMWrapperWorld.cpp:
+ * bindings/js/JSDOMWindowBase.cpp:
+ * bindings/scripts/CodeGeneratorJS.pm:
+ (GenerateCallbackImplementation):
+ * bindings/v8/DOMDataStore.cpp:
+ * bindings/v8/DOMWrapperWorld.cpp:
+ * bindings/v8/V8Binding.cpp:
+ * bindings/v8/V8DOMMap.cpp:
+ * bindings/v8/V8DOMWrapper.h:
+ * dom/EventTarget.cpp:
+ * fileapi/LocalFileSystem.cpp:
+ * loader/CrossOriginPreflightResultCache.cpp:
+ * loader/ResourceLoadScheduler.cpp:
+ * loader/WorkerThreadableLoader.cpp:
+ * loader/appcache/ApplicationCacheGroup.cpp:
+ * page/DOMWindow.cpp:
+ * page/History.cpp:
+ * page/SecurityOrigin.cpp:
+ * page/mac/EventHandlerMac.mm:
+ * platform/MIMETypeRegistry.cpp:
+ * platform/ThreadGlobalData.cpp:
+ * platform/ThreadTimers.cpp:
+ * platform/TreeShared.h:
+ * platform/audio/HRTFDatabaseLoader.cpp:
+ * platform/graphics/Font.cpp:
+ * platform/graphics/FontFallbackList.h:
+ * platform/graphics/FontFastPath.cpp:
+ * platform/graphics/Image.cpp:
+ * platform/graphics/avfoundation/MediaPlayerPrivateAVFoundation.cpp:
+ * platform/graphics/ca/win/WKCACFViewLayerTreeHost.cpp:
+ * platform/graphics/cg/ImageBufferCG.cpp:
+ * platform/graphics/chromium/LayerRendererChromium.cpp:
+ * platform/graphics/chromium/cc/CCLayerTreeHostImplProxy.cpp:
+ * platform/mac/LocalizedStringsMac.mm:
+ * platform/mac/SharedBufferMac.mm:
+ * platform/network/BlobResourceHandle.cpp:
+ * platform/network/cf/DNSCFNet.cpp:
+ * platform/network/cf/LoaderRunLoopCF.cpp:
+ * platform/text/LocalizedNumberICU.cpp:
+ * platform/text/TextEncodingRegistry.cpp:
+ * platform/text/mac/LocalizedNumberMac.mm:
+ * platform/win/LocalizedStringsWin.cpp:
+ * storage/LocalStorageThread.cpp:
+ * storage/StorageAreaImpl.cpp:
+ * storage/StorageAreaSync.cpp:
+ * storage/StorageNamespaceImpl.cpp:
+ * storage/StorageSyncManager.cpp:
+ * webaudio/AsyncAudioDecoder.cpp:
+ * webaudio/AudioBufferSourceNode.cpp:
+ * webaudio/AudioContext.h
+ * webaudio/AudioContext.cpp:
+ * webaudio/AudioNode.cpp:
+ * webaudio/ConvolverNode.cpp:
+ * webaudio/OfflineAudioDestinationNode.cpp:
+ * webaudio/RealtimeAnalyser.cpp:
+ * webaudio/WaveShaperNode.cpp:
+ * websockets/WorkerThreadableWebSocketChannel.cpp:
+ * workers/WorkerMessagingProxy.cpp:
+
+2011-07-30 Enrica Casucci <enrica@apple.com>
+
+ REGRESSION: culledInlineAbsoluteRects needs to translate a 0,0 point to absolute
+ coordinates so that the top left position is accurate.
+ https://bugs.webkit.org/show_bug.cgi?id=65419
+ <rdar://problem/9408862>
+
+ This patch fixes a regression introduced in r83075.
+ Make sure that culledInlineAbsoluteRects still does
+ a translation of a 0,0 point to absolute coordinates so that the top left position is
+ accurate.
+ This code path could not be tested via JavaScript. The patch
+ extends the Internals object to be able to test this case.
+
+ Reviewed by Dan Bernstein.
+
+ Test: fast/inline/skipped-whitespace-boundingBox.html
+
+ * WebCore.exp.in: Added some exports.
+ * rendering/RenderInline.cpp:
+ (WebCore::RenderInline::culledInlineAbsoluteRects):
+ * testing/Internals.cpp:
+ (WebCore::Internals::boundingBox):
+ * testing/Internals.h:
+ * testing/Internals.idl:
+
+2011-07-26 Pavel Podivilov <podivilov@chromium.org>
+
+ Web Inspector: move location conversion functions to SourceFile.
+ https://bugs.webkit.org/show_bug.cgi?id=65185
+
+ Reviewed by Pavel Feldman.
+
+ Test: inspector/debugger/source-file.html
+
+ * inspector/front-end/DebuggerPresentationModel.js:
+ (WebInspector.DebuggerPresentationModel.prototype._scriptLocationToUILocation.didCreateSourceMapping):
+ (WebInspector.DebuggerPresentationModel.prototype._scriptLocationToUILocation):
+ (WebInspector.DebuggerPresentationModel.prototype._uiLocationToScriptLocation.didCreateSourceMapping):
+ (WebInspector.DebuggerPresentationModel.prototype._uiLocationToScriptLocation):
+ * inspector/front-end/SourceFile.js:
+ (WebInspector.SourceFile.prototype.rawLocationToUILocation):
+ (WebInspector.SourceFile.prototype.uiLocationToRawLocation):
+ (WebInspector.SourceFile.prototype._scriptForRawLocation):
+ (WebInspector.SourceFile.prototype.createSourceMappingIfNeeded):
+ (WebInspector.FormattedSourceFile.prototype.createSourceMappingIfNeeded):
+ (WebInspector.FormattedSourceFile.prototype._didRequestContent):
+
+2011-07-29 Rob Buis <rbuis@rim.com>
+
+ URL references are completely broken in SVG
+ https://bugs.webkit.org/show_bug.cgi?id=63283
+
+ Reviewed by Nikolas Zimmermann.
+
+ Change SVGURIElement::getTarget to be more strict about iri resolving and make it serve as a central
+ place to resolve a iri to a SVGElement. The iri can be same-document or external, for now we only handle
+ the former, the latter will be handled in bug 65344.
+ Accept as same-document if the iri combined with the base uri equals the document url (ignoring
+ the fragment identifier). For convenience a method is added to lookup the element, if found.
+
+ Tests: svg/custom/external-paintserver-reference.svg
+ svg/custom/linking-base-external-reference.xhtml
+ svg/custom/uri-reference-handling.svg
+
+ * css/CSSCursorImageValue.cpp:
+ (WebCore::resourceReferencedByCursorElement):
+ (WebCore::CSSCursorImageValue::~CSSCursorImageValue):
+ (WebCore::CSSCursorImageValue::updateIfSVGCursorIsUsed):
+ * css/CSSFontFaceSource.cpp:
+ (WebCore::CSSFontFaceSource::getFontData):
+ * css/CSSFontSelector.h:
+ (WebCore::CSSFontSelector::document):
+ * css/SVGCSSStyleSelector.cpp:
+ (WebCore::CSSStyleSelector::applySVGProperty):
+ * rendering/svg/RenderSVGTextPath.cpp:
+ (WebCore::RenderSVGTextPath::layoutPath):
+ * rendering/svg/SVGResources.cpp:
+ (WebCore::targetReferenceFromResource):
+ (WebCore::paintingResourceFromSVGPaint):
+ * svg/SVGAltGlyphElement.cpp:
+ (WebCore::SVGAltGlyphElement::hasValidGlyphElements):
+ * svg/SVGFEImageElement.cpp:
+ (WebCore::SVGFEImageElement::requestImageResource):
+ (WebCore::SVGFEImageElement::build):
+ * svg/SVGGlyphRefElement.cpp:
+ (WebCore::SVGGlyphRefElement::hasValidGlyphElement):
+ * svg/SVGLinearGradientElement.cpp:
+ (WebCore::SVGLinearGradientElement::collectGradientAttributes):
+ * svg/SVGMPathElement.cpp:
+ (WebCore::SVGMPathElement::pathElement):
+ * svg/SVGPaint.cpp:
+ (WebCore::SVGPaint::matchesTargetURI):
+ * svg/SVGPatternElement.cpp:
+ (WebCore::SVGPatternElement::collectPatternAttributes):
+ * svg/SVGRadialGradientElement.cpp:
+ (WebCore::SVGRadialGradientElement::collectGradientAttributes):
+ * svg/SVGTRefElement.cpp:
+ (WebCore::SVGTRefElement::updateReferencedText):
+ (WebCore::SVGTRefElement::svgAttributeChanged):
+ (WebCore::SVGTRefElement::buildPendingResource):
+ * svg/SVGTextPathElement.cpp:
+ (WebCore::SVGTextPathElement::insertedIntoDocument):
+ * svg/SVGURIReference.cpp:
+ (WebCore::SVGURIReference::fragmentIdentifierFromIRIString):
+ (WebCore::SVGURIReference::targetElementFromIRIString):
+ * svg/SVGURIReference.h:
+ * svg/SVGUseElement.cpp:
+ (WebCore::SVGUseElement::buildPendingResource):
+ (WebCore::SVGUseElement::hasCycleUseReferencing):
+ (WebCore::SVGUseElement::expandUseElementsInShadowTree):
+ * svg/animation/SVGSMILElement.cpp:
+ (WebCore::SVGSMILElement::targetElement):
+
+2011-07-29 James Simonsen <simonjam@chromium.org>
+
+ Remove zipcar.com site specific hack added in r87361
+ https://bugs.webkit.org/show_bug.cgi?id=65405
+
+ Reviewed by Adam Barth.
+
+ * html/HTMLScriptElement.cpp:
+ (WebCore::HTMLScriptElement::insertedIntoDocument):
+
+2011-07-29 James Robinson <jamesr@chromium.org>
+
+ [chromium] Reorder functions in LayerRendererChromium.cpp to make bug 58840 easier to review
+ https://bugs.webkit.org/show_bug.cgi?id=65354
+
+ Reviewed by Kenneth Russell.
+
+ This moves a few function definitions in LayerRendererChromium around to make the patch on bug 58840 easier to
+ review. In that patch, updatePropertiesAndRenderSurfaces() becomes a templated free function instead of a
+ member function, so it has to be near the top of the .cpp.
+
+ * platform/graphics/chromium/LayerRendererChromium.cpp:
+ (WebCore::calculateVisibleRect):
+ (WebCore::isScaleOrTranslation):
+ (WebCore::LayerRendererChromium::updatePropertiesAndRenderSurfaces):
+ (WebCore::LayerRendererChromium::updateLayers):
+
+2011-07-29 Jeff Miller <jeffm@apple.com>
+
+ Work-in-progress on Adopt AVCF media back end on Windows
+ https://bugs.webkit.org/show_bug.cgi?id=65400
+ <rdar://problem/9083559>
+
+ Prepare WebCore.vcproj to support AVCF media back end on Windows.
+
+ Reviewed by Darin Adler.
+
+ No changes to functionality so no new tests.
+
+ * WebCore.vcproj/WebCore.vcproj: Added MediaPlayerPrivateAVFoundation and MediaPlayerPrivateAVFoundationCF sources.
+ * WebCore.vcproj/WebCoreMediaQT.vsprops: Added additional include directories and dleay load DLLs for AVCF.
+ * platform/graphics/avfoundation/MediaPlayerPrivateAVFoundationCF.cpp: Added, just a stub for now.
+ * platform/graphics/avfoundation/MediaPlayerPrivateAVFoundationCF.h: Added, just a stub for now.
+
+2011-07-29 Jeff Miller <jeffm@apple.com>
+
+ Work-in-progress on Adopt AVCF media back end on Windows
+ https://bugs.webkit.org/show_bug.cgi?id=65400
+ <rdar://problem/9083559>
+
+ Make MediaPlayerPrivateAVFoundation::setDelayCallbacks() const and make the ivars
+ it references mutable so other const member functions can call it.
+
+ Reviewed by Darin Adler.
+
+ No changes to functionality so no new tests.
+
+ * platform/graphics/avfoundation/MediaPlayerPrivateAVFoundation.cpp:
+ (WebCore::MediaPlayerPrivateAVFoundation::setDelayCallbacks):
+ * platform/graphics/avfoundation/MediaPlayerPrivateAVFoundation.h:
+
+2011-07-29 Zhenyao Mo <zmo@google.com>
+
+ Unreviewed, rolling out r92011.
+ http://trac.webkit.org/changeset/92011
+ https://bugs.webkit.org/show_bug.cgi?id=65044
+
+ break chromium win build
+
+ * dom/WheelEvent.cpp:
+ (WebCore::WheelEvent::WheelEvent):
+ (WebCore::WheelEvent::initWheelEvent):
+ * dom/WheelEvent.h:
+ * page/EventHandler.cpp:
+ (WebCore::EventHandler::EventHandler):
+ (WebCore::EventHandler::handleGestureEvent):
+ (WebCore::EventHandler::handleTouchEvent):
+ (WebCore::EventHandler::resetGestureRecognizer):
+ * page/EventHandler.h:
+ * page/FrameView.cpp:
+ (WebCore::FrameView::reset):
+ * platform/PlatformGestureEvent.h:
+ * platform/PlatformGestureRecognizer.h:
+ * platform/ScrollAnimatorWin.cpp:
+ * platform/chromium/FramelessScrollView.h:
+ * platform/chromium/GestureRecognizerChromium.cpp:
+ (WebCore::InnerGestureRecognizer::InnerGestureRecognizer):
+ (WebCore::InnerGestureRecognizer::reset):
+ (WebCore::InnerGestureRecognizer::dispatchSyntheticClick):
+ (WebCore::InnerGestureRecognizer::processTouchEventForGesture):
+ (WebCore::InnerGestureRecognizer::scrollViaTouchMotion):
+ (WebCore::touchDown):
+ (WebCore::noGesture):
+ (WebCore::click):
+ (WebCore::isClickOrScroll):
+ (WebCore::inScroll):
+ * platform/chromium/GestureRecognizerChromium.h:
+ (WebCore::GestureRecognizerChromium::reset):
+ (WebCore::GestureRecognizerChromium::processTouchEventForGesture):
+ * platform/chromium/PopupMenuChromium.cpp:
+ * platform/chromium/PopupMenuChromium.h:
+ * platform/mac/ScrollAnimatorMac.mm:
+ (WebCore::ScrollAnimatorMac::handleGestureEvent):
+
+2011-07-29 Adrienne Walker <enne@google.com>
+
+ [chromium] Only force slow scrolling for iframes under --force-compositing-mode
+ https://bugs.webkit.org/show_bug.cgi?id=65394
+
+ Reviewed by James Robinson.
+
+ The previous code introduced by r91591 made composited frames as well
+ as non-composited subframes go down the slow scrolling path. Adjust
+ the check to use ownerElement to test for subframe and use
+ hasCompositedContent() which is a more correct test for compositing
+ than isEnclosedInCompositingLayer.
+
+ * page/FrameView.cpp:
+ (WebCore::FrameView::useSlowRepaints):
+
+2011-07-29 Emil A Eklund <eae@chromium.org>
+
+ -webkit-marquee with anonymous node causes segmentation fault in Node::document
+ https://bugs.webkit.org/show_bug.cgi?id=64693
+
+ Reviewed by Simon Fraser.
+
+ Test: fast/css/webkit-marquee-anonymous-node-crash.html
+
+ * rendering/RenderLayer.cpp:
+ (WebCore::RenderLayer::scrollTo):
+ Add null check as renderer()->node() is null for anonymous nodes.
+
+2011-07-29 Mike Reed <reed@google.com>
+
+ [skia] never draw with GDI, so that all text can be gpu-accelerated
+ https://bugs.webkit.org/show_bug.cgi?id=65203
+
+ Reviewed by Kenneth Russell.
+
+ * platform/graphics/chromium/FontChromiumWin.cpp:
+ (WebCore::TransparencyAwareFontPainter::TransparencyAwareFontPainter::TransparencyAwareFontPainter):
+ (WebCore::TransparencyAwareFontPainter::TransparencyAwareGlyphPainter::TransparencyAwareGlyphPainter):
+ (WebCore::TransparencyAwareFontPainter::TransparencyAwareGlyphPainter::drawGlyphs):
+ (WebCore::TransparencyAwareFontPainter::TransparencyAwareUniscribePainter::TransparencyAwareUniscribePainter):
+ (WebCore::drawGlyphsWin):
+ (WebCore::Font::drawComplexText):
+ * platform/graphics/chromium/UniscribeHelper.cpp:
+ (WebCore::UniscribeHelper::draw):
+ * platform/graphics/skia/PlatformContextSkia.cpp:
+ * platform/graphics/skia/PlatformContextSkia.h:
+ * platform/graphics/skia/SkiaFontWin.cpp:
+ * platform/graphics/skia/SkiaFontWin.h:
+
+2011-07-29 Sam Weinig <sam@webkit.org>
+
+ Move Region code from WebKit2 to WebCore
+ https://bugs.webkit.org/show_bug.cgi?id=65392
+
+ Reviewed by David Hyatt.
+
+ * CMakeLists.txt:
+ * GNUmakefile.list.am:
+ * WebCore.gypi:
+ * WebCore.pro:
+ * WebCore.vcproj/WebCore.vcproj:
+ * WebCore.xcodeproj/project.pbxproj:
+ Update project files.
+
+ * WebCore.exp.in:
+ Update exports.
+
+ * platform/graphics/Region.cpp: Copied from Source/WebKit2/Platform/Region.cpp.
+ * platform/graphics/Region.h: Copied from Source/WebKit2/Platform/Region.h.
+ Add files.
+
+2011-07-29 Robert Kroeger <rjkroege@chromium.org>
+
+ [chromium] Layering violations in gesture recognizer
+ https://bugs.webkit.org/show_bug.cgi?id=65044
+
+ Reviewed by Adam Barth.
+
+ Divided the gesture recognizer up to correct a layering
+ violation by moving gesture implementation from it to
+ EventHandler::handleGestureEvent so that the gesture recognizer
+ could simply be an engine for generating gesture events from
+ touch events.
+
+ * dom/WheelEvent.cpp:
+ (WebCore::WheelEvent::WheelEvent):
+ (WebCore::WheelEvent::initWheelEvent):
+ * dom/WheelEvent.h:
+ Added symbolic contstant for tick multiplier.
+ * page/EventHandler.cpp:
+ (WebCore::EventHandler::EventHandler):
+ Removed construction of gesture recognizer.
+ (WebCore::EventHandler::handleGestureEvent):
+ Extended for new gesture events added in this patch.
+ (WebCore::EventHandler::handleTouchEvent):
+ Removed call into gesture recognizer.
+ * page/EventHandler.h:
+ * page/FrameView.cpp:
+ (WebCore::FrameView::reset):
+ Removed code to reset gesture recognizer.
+ * platform/PlatformGestureEvent.h:
+ (WebCore::PlatformGestureEvent::PlatformGestureEvent):
+ (WebCore::PlatformGestureEvent::deltaX):
+ (WebCore::PlatformGestureEvent::deltaY):
+ (WebCore::PlatformGestureEvent::shiftKey):
+ (WebCore::PlatformGestureEvent::ctrlKey):
+ (WebCore::PlatformGestureEvent::altKey):
+ (WebCore::PlatformGestureEvent::metaKey):
+ Added additional gesture types and parameters.
+ * platform/PlatformGestureRecognizer.h:
+ * platform/ScrollAnimatorWin.cpp:
+ (WebCore::ScrollAnimatorWin::handleGestureEvent):
+ Added missing method needed to fix the build on windows.
+ * platform/chromium/FramelessScrollView.h:
+ Added support for displatching gesture and touch events.
+ * platform/chromium/GestureRecognizerChromium.cpp:
+ (WebCore::InnerGestureRecognizer::InnerGestureRecognizer):
+ (WebCore::InnerGestureRecognizer::reset):
+ (WebCore::InnerGestureRecognizer::constructClickGestureEvent):
+ (WebCore::InnerGestureRecognizer::processTouchEventForGestures):
+ Removed all depedencies on EventHandler. Instead, create gesture
+ events from touch event stream.
+ (WebCore::InnerGestureRecognizer::constructScrollGesture):
+ (WebCore::touchDown):
+ (WebCore::noGesture):
+ (WebCore::click):
+ (WebCore::isClickOrScroll):
+ (WebCore::inScroll):
+ (WebCore::GestureRecognizerChromium::processTouchEventForGestures):
+ (WebCore::GestureRecognizerChromium::reset):
+ * platform/chromium/GestureRecognizerChromium.h:
+ * platform/chromium/PopupMenuChromium.cpp:
+ Added support to handle touch events and gesture events.
+ (WebCore::PopupContainer::handleTouchEvent):
+ (WebCore::PopupContainer::handleGestureEvent):
+ (WebCore::PopupListBox::handleTouchEvent):
+ (WebCore::PopupListBox::handleGestureEvent):
+ * platform/chromium/PopupMenuChromium.h:
+ * platform/mac/ScrollAnimatorMac.mm:
+ (WebCore::ScrollAnimatorMac::handleGestureEvent):
+ Ignores the newly added gesture event types.
+
+2011-07-29 Eric Carlson <eric.carlson@apple.com>
+
+ Load delegate not consulted before loading some poster images
+ https://bugs.webkit.org/show_bug.cgi?id=65270
+
+ Reviewed by Joseph Pecoraro.
+
+ * html/HTMLVideoElement.cpp:
+ (WebCore::HTMLVideoElement::setDisplayMode): Don't ask a load delegate if it is OK to
+ load an empty poster url.
+
+2011-07-29 Jeff Miller <jeffm@apple.com>
+
+ Re-sort one more file in the WebCore project with Visual Studio.
+
+ * WebCore.vcproj/WebCore.vcproj:
+
+2011-07-29 Jeff Miller <jeffm@apple.com>
+
+ Re-sort the WebCore project with Visual Studio after recent changes that I assume were done by manually editing the XML.
+
+ * WebCore.vcproj/WebCore.vcproj:
+
+2011-07-29 Dan Bernstein <mitz@apple.com>
+
+ Added the regional indicator symbols to the set of codepoints that force use of the complex text code path.
+
+ Fixes <rdar://problem/9864578> Regional indicator symbols do not combine into national flags
+ https://bugs.webkit.org/show_bug.cgi?id=65380
+
+ Reviewed by Anders Carlsson.
+
+ Test: fast/text/regional-indicator-symobls.html
+
+ * platform/graphics/Font.cpp:
+ (WebCore::Font::codePath): Added handling of surrogate pairs, which returns Complex for characters in
+ the range U+1F1E6..U+1F1FF.
+
+2011-07-28 David Hyatt <hyatt@apple.com>
+
+ https://bugs.webkit.org/show_bug.cgi?id=65342
+
+ Add support for positioned floats being placed into a RenderBlock's FloatingObject list and for
+ correct propagation of those floating objects into descendants.
+
+ Reviewed by Sam Weinig.
+
+ * rendering/RenderBlock.cpp:
+ (WebCore::RenderBlock::layoutBlock):
+ The positioned float layout algorithm is 2-pass. You lay out without the positioned objects, and then
+ once they have been placed, you lay out again and use that placement to determine the float positions.
+ layoutBlock now has an extra parameter to indicate which layout pass you are in.
+
+ (WebCore::RenderBlock::addOverflowFromFloats):
+ Don't add overflow from positioned floats ever, since the positioning overflow function takes care of that
+ already.
+
+ (WebCore::RenderBlock::layoutBlockChild):
+ Include positioned floats in the lowestFloatLogicalBottom we look at so that overhang is computed properly.
+
+ (WebCore::RenderBlock::simplifiedLayout):
+ If layoutPositionedObjects indicates that a positioned float moved, then we can't do simplified layout. Bail
+ instead and do a full layout.
+
+ (WebCore::RenderBlock::positionedFloatsNeedRelayout):
+ Helper for layoutBlock in order to bypass the 2-pass model if we can detect that none of the positioned objects
+ will actually need to lay out again. This allows incremental layout to not always fault back to 2-pass as normal
+ flow content streams in.
+
+ (WebCore::RenderBlock::layoutPositionedObjects):
+ Modified to return a boolean indicating whether a positioned float got a layout. If so, we will lay out again
+ once our positioned objects are placed.
+
+ (WebCore::RenderBlock::insertFloatingObject):
+ insertFloatingObject can now handle positioned objects being inserted.
+
+ (WebCore::RenderBlock::positionNewFloats):
+ positionNewFloats doesn't attempt to position this new type of float. It only handles left/right floats.
+
+ (WebCore::RenderBlock::addPositionedFloats):
+ Helper to add in the floating objects for positioned floats. This is used instead of positionNewFloats and
+ just walks the positioned objects instead.
+
+ (WebCore::RenderBlock::clearFloats):
+ clearFloats now handles adding in positioned floats if the layout pass says they should be included.
+
+ (WebCore::RenderBlock::marginLogicalLeftForChild):
+ (WebCore::RenderBlock::marginLogicalRightForChild):
+ New helpers used by addPositionedFloats.
+
+ (WebCore::RenderBlock::FloatingObjects::clear):
+ (WebCore::RenderBlock::FloatingObjects::increaseObjectsCount):
+ (WebCore::RenderBlock::FloatingObjects::decreaseObjectsCount):
+ Make sure the object count tracking handles positioned floats as well as left/right floats.
+
+ * rendering/RenderBlock.h:
+ (WebCore::RenderBlock::logicalLeftForChild):
+ New helper used by addPositionedFloats.
+
+ (WebCore::RenderBlock::FloatingObject::FloatingObject):
+ Change the constructor to take the RenderStyle enum. This makes the construction simpler.
+
+ (WebCore::RenderBlock::hasPositionedFloats):
+ A bit tracks whether or not a block has positioned floats in its positioned objects list.
+
+ (WebCore::RenderBlock::hasOverhangingFloats):
+ Modified to include positioned floats.
+
+ (WebCore::RenderBlock::lowestFloatLogicalBottom):
+ This function excludes positioned floats by default, so that places that - for example - expand to
+ encompass their lowestFloatLogicalBottom won't accidentally expand to encompass positioned objects.
+
+ (WebCore::RenderBlock::lowestFloatLogicalBottomIncludingPositionedFloats):
+ A new function that includes positioned floats for callers that care.
+
+ (WebCore::RenderBlock::FloatingObjects::FloatingObjects):
+ (WebCore::RenderBlock::FloatingObjects::hasPositionedObjects):
+ The object count tracking for positioned floats within a block's FloatingObjects list.
+
+ * rendering/RenderBox.cpp:
+ (WebCore::RenderBox::updateBoxModelInfoFromStyle):
+ Make sure isFloating is set even if isPositioned is also set for the positioned float case.
+
+ * rendering/RenderDeprecatedFlexibleBox.cpp:
+ (WebCore::RenderDeprecatedFlexibleBox::layoutBlock):
+ Modified to do the extra layout pass if needed.
+
+2011-07-29 Scott Byer <scottbyer@chromium.org>
+
+ Scroll animation refinements.
+ https://bugs.webkit.org/show_bug.cgi?id=65154
+
+ Reviewed by Adam Barth.
+
+ Fix scroll animation bugs when scrolling to the ends of a document. Smooth over slow event repeat rates
+ better. Fix tests to allow for unit testing of scrolling up and not just down.
+
+ Additional unit tests added to ScrollAnimatorNoneTest:
+ ScrollQuadraticSmoothed, ScrollLotsQuadraticSmoothed, ScrollDownToBumper, ScrollUpToBumper
+
+ * platform/ScrollAnimatorNone.cpp:
+ (WebCore::ScrollAnimatorNone::Parameters::Parameters):
+ (WebCore::ScrollAnimatorNone::PerAxisData::updateDataFromParameters):
+ (WebCore::ScrollAnimatorNone::scroll):
+ * platform/ScrollAnimatorNone.h:
+
+2011-07-29 Samuel White <samuel_white@apple.com>
+
+ Add the ability to search the AccessibilityObject cache
+ https://bugs.webkit.org/show_bug.cgi?id=64994
+
+ To support searching the AccessibilityObject cache, we first need to
+ implement a minimal set of functions that will allow AccessibilityObjects
+ to be identified when searching using common search criteria. The additional
+ functions below complement the existing identification functionality already
+ available and together provide a basic working set to build search on top of.
+ Additionally, the blockquoteLevel function has been moved into the AccessibilityObject
+ class to make it available to all platforms.
+
+ Reviewed by Chris Fleizach.
+
+ New tests will be included in the following patch that will also implement
+ basic search functionality.
+
+ * accessibility/AccessibilityObject.cpp:
+ (WebCore::AccessibilityObject::isBlockquote):
+ (WebCore::AccessibilityObject::isLandmark):
+ (WebCore::AccessibilityObject::hasMisspelling):
+ (WebCore::AccessibilityObject::blockquoteLevel):
+ * accessibility/AccessibilityObject.h:
+ (WebCore::AccessibilityObject::isUnvisited):
+ (WebCore::AccessibilityObject::hasBoldFont):
+ (WebCore::AccessibilityObject::hasItalicFont):
+ (WebCore::AccessibilityObject::hasPlainText):
+ (WebCore::AccessibilityObject::hasSameFont):
+ (WebCore::AccessibilityObject::hasSameFontColor):
+ (WebCore::AccessibilityObject::hasSameStyle):
+ (WebCore::AccessibilityObject::hasStaticText):
+ (WebCore::AccessibilityObject::hasUnderline):
+ (WebCore::AccessibilityObject::tableLevel):
+ * accessibility/AccessibilityRenderObject.cpp:
+ (WebCore::AccessibilityRenderObject::isUnvisited):
+ (WebCore::AccessibilityRenderObject::hasBoldFont):
+ (WebCore::AccessibilityRenderObject::hasItalicFont):
+ (WebCore::AccessibilityRenderObject::hasPlainText):
+ (WebCore::AccessibilityRenderObject::hasSameFont):
+ (WebCore::AccessibilityRenderObject::hasSameFontColor):
+ (WebCore::AccessibilityRenderObject::hasSameStyle):
+ (WebCore::AccessibilityRenderObject::hasUnderline):
+ * accessibility/AccessibilityRenderObject.h:
+ * accessibility/AccessibilityTable.cpp:
+ (WebCore::AccessibilityTable::tableLevel):
+ * accessibility/AccessibilityTable.h:
+ * accessibility/mac/AccessibilityObjectWrapper.mm:
+ (AXAttributeStringSetBlockquoteLevel):
+ (-[AccessibilityObjectWrapper accessibilityAttributeValue:]):
+
+2011-07-29 Zeng Huiqing <huiqing.zeng@intel.com>
+
+ Avoid calling animation timer updates while the page is loading
+ https://bugs.webkit.org/show_bug.cgi?id=64851
+
+ Reviewed by Simon Fraser.
+
+ There is no need to update the animation timer for a RenderObject that has not yet been inserted into the render tree, or that doesn't have animations. Our tests show that this improves page loading by up to 3x on pages with a large number of transitionable objects.
+
+ * page/animation/AnimationController.cpp:
+ (Web