Timeline
Feb 14, 2015:
- 11:52 PM Changeset in webkit [180125] by
-
- 2 edits in trunk/Source/WebCore
More internal build fixing.
DDDFACache.h is not an actual private header, so we were using open source
redeclaration of DDDFACacheRef with an internal function prototype from DDDFAScanner.h.
- platform/cocoa/TelephoneNumberDetectorCocoa.cpp:
(WebCore::TelephoneNumberDetector::phoneNumbersScanner):
- 11:16 PM Changeset in webkit [180124] by
-
- 2 edits in trunk/Source/WebKit2
Build fix after <http://trac.webkit.org/changeset/180122>
- Platform/mac/StringUtilities.mm:
- 10:11 PM Changeset in webkit [180123] by
-
- 4 edits22 adds in trunk
Add the initial matching implementation for attribute selectors with case-insensitive value
https://bugs.webkit.org/show_bug.cgi?id=141615
Reviewed by Andreas Kling.
Source/WebCore:
Tests: fast/css/case-insensitive-attribute-selector-specificity.html
fast/css/case-insensitive-attribute-selector-styling-html-1.html
fast/css/case-insensitive-attribute-selector-styling-html-2.html
fast/css/case-insensitive-attribute-selector-styling-html-3.html
fast/css/case-insensitive-attribute-selector-styling-xhtml-1.xhtml
fast/css/case-insensitive-attribute-selector-styling-xhtml-2.xhtml
fast/css/case-insensitive-attribute-selector-styling-xhtml-3.xhtml
fast/selectors/case-insensitive-attribute-bascis.html
fast/selectors/case-insensitive-attribute-matching-style-attribute.html
fast/selectors/case-insensitive-attribute-style-update.html
fast/selectors/case-insensitive-attribute-with-case-sensitive-name.html
- css/SelectorChecker.cpp:
(WebCore::SelectorChecker::checkOne):
We already had case-insensitive value matching due to some legacy HTML
behaviors where some attribute values would be matched case-insensitively
depending on the attribute name.
For this patch, I just re-use the same mechanism. I used branches to try
to convey the idea that matching is case sensitive by default unless
the selector has a flag or we are in the weird HTML exception.
- cssjit/SelectorCompiler.cpp:
(WebCore::SelectorCompiler::constructFragmentsInternal):
Disable that case in the CSS JIT for now, I'll implement it later.
LayoutTests:
This covers the basics. I will add some more cases as I do the follow up patches.
I avoided the problem of non-ASCII characters, this will need its own follow up
patch that fixes all attribute matching.
- fast/css/case-insensitive-attribute-selector-specificity-expected.html: Added.
- fast/css/case-insensitive-attribute-selector-specificity.html: Added.
- fast/css/case-insensitive-attribute-selector-styling-html-1-expected.html: Added.
- fast/css/case-insensitive-attribute-selector-styling-html-1.html: Added.
- fast/css/case-insensitive-attribute-selector-styling-html-2-expected.html: Added.
- fast/css/case-insensitive-attribute-selector-styling-html-2.html: Added.
- fast/css/case-insensitive-attribute-selector-styling-html-3-expected.html: Added.
- fast/css/case-insensitive-attribute-selector-styling-html-3.html: Added.
- fast/css/case-insensitive-attribute-selector-styling-xhtml-1-expected.html: Added.
- fast/css/case-insensitive-attribute-selector-styling-xhtml-1.xhtml: Added.
- fast/css/case-insensitive-attribute-selector-styling-xhtml-2-expected.html: Added.
- fast/css/case-insensitive-attribute-selector-styling-xhtml-2.xhtml: Added.
- fast/css/case-insensitive-attribute-selector-styling-xhtml-3-expected.html: Added.
- fast/css/case-insensitive-attribute-selector-styling-xhtml-3.xhtml: Added.
- fast/selectors/case-insensitive-attribute-bascis-expected.txt: Added.
- fast/selectors/case-insensitive-attribute-bascis.html: Added.
- fast/selectors/case-insensitive-attribute-matching-style-attribute-expected.txt: Added.
- fast/selectors/case-insensitive-attribute-matching-style-attribute.html: Added.
- fast/selectors/case-insensitive-attribute-style-update-expected.txt: Added.
- fast/selectors/case-insensitive-attribute-style-update.html: Added.
- fast/selectors/case-insensitive-attribute-with-case-sensitive-name-expected.txt: Added.
- fast/selectors/case-insensitive-attribute-with-case-sensitive-name.html: Added.
- 9:31 PM Changeset in webkit [180122] by
-
- 2 edits in trunk/Source/WebCore
Declare soft-linked functions with extern "C" linkage
More work towards the Maverick Debug build fix:
REGRESSION (r180082): WebCore Debug builds fail on Mavericks due to weak export symbols
<http://webkit.org/b/141607>
- platform/mac/SoftLinking.h: Try declaring soft-linked
functions with extern "C" linkage to see if they stop appearing
as weak externals. This has the added benefit that if the
function signature changes, we'll get a build failure.
- 9:14 PM Changeset in webkit [180121] by
-
- 3 edits in trunk/Source/WebCore
[Mac] RetainPtr member cleanup, possible leaks
https://bugs.webkit.org/show_bug.cgi?id=141616
Patch by Joseph Pecoraro <Joseph Pecoraro> on 2015-02-14
Reviewed by Andreas Kling.
- platform/mac/PlatformSpeechSynthesizerMac.mm:
(-[WebSpeechSynthesisWrapper speakUtterance:]):
Adopt the allocated object to avoid a possible leak.
- platform/mac/ScrollAnimatorMac.mm:
(WebCore::ScrollAnimatorMac::ScrollAnimatorMac):
Adopt to avoid unnecessary retain/autorelease.
- 9:08 PM Changeset in webkit [180120] by
-
- 3 edits in trunk/Websites/perf.webkit.org
Build URL on new perf dashboard doesn't resolve $builderName
https://bugs.webkit.org/show_bug.cgi?id=141583
Reviewed by Darin Adler.
Support $builderName in the build URL template.
- public/js/helper-classes.js:
(TestBuild.buildUrl): Replaced $builderName with the builder name.
- public/v2/manifest.js:
(App.Metric.fullName): Fixed the typo. We need &ni, not &in.
(App.BuilderurlFromBuildNumber): Replaced $builderName with the builder name.
- 8:54 PM Changeset in webkit [180119] by
-
- 5 edits in trunk/Source
[Mac] Possible DDActionContext leaks, RetainPtr cleanup
https://bugs.webkit.org/show_bug.cgi?id=141613
Patch by Joseph Pecoraro <Joseph Pecoraro> on 2015-02-14
Reviewed by Tim Horton.
Source/WebKit/mac:
- WebCoreSupport/WebEditorClient.mm:
(WebEditorClient::WebEditorClient):
Avoid unnecessary retain/autorelease by adopting.
- WebView/WebActionMenuController.mm:
(-[WebActionMenuController _defaultMenuItemsForDataDetectableLink]):
Adopt the allocated object.
Source/WebKit2:
- UIProcess/mac/WKActionMenuController.mm:
(-[WKActionMenuController _defaultMenuItemsForDataDetectableLink]):
Adopt the allocated object.
- 8:44 PM Changeset in webkit [180118] by
-
- 3 edits in trunk/Source/WebCore
IncompleteMetaDataCue leak seen on leaks bot
https://bugs.webkit.org/show_bug.cgi?id=141611
Patch by Joseph Pecoraro <Joseph Pecoraro> on 2015-02-14
Reviewed by Eric Carlson.
- platform/graphics/avfoundation/InbandMetadataTextTrackPrivateAVF.h:
(WebCore::IncompleteMetaDataCue::IncompleteMetaDataCue): Deleted.
(WebCore::IncompleteMetaDataCue::~IncompleteMetaDataCue): Deleted.
(WebCore::IncompleteMetaDataCue::cueData): Deleted.
(WebCore::IncompleteMetaDataCue::startTime): Deleted.
Convert to a struct and hold a list of structs instead of pointers.
- platform/graphics/avfoundation/InbandMetadataTextTrackPrivateAVF.cpp:
(WebCore::InbandMetadataTextTrackPrivateAVF::addDataCue):
(WebCore::InbandMetadataTextTrackPrivateAVF::updatePendingCueEndTimes):
(WebCore::InbandMetadataTextTrackPrivateAVF::flushPartialCues):
Update use of IncompleteMetaDataCue and modernize related code.
- 6:42 PM Changeset in webkit [180117] by
-
- 3 edits in trunk/Source/WebInspectorUI
Web Inspector: Remove specificity approximation
https://bugs.webkit.org/show_bug.cgi?id=141600
Reviewed by Timothy Hatcher.
- Localizations/en.lproj/localizedStrings.js:
- UserInterface/Views/CSSStyleDeclarationSection.js:
(WebInspector.CSSStyleDeclarationSection.prototype.refresh.appendSelector):
- 3:25 PM Changeset in webkit [180116] by
-
- 100 edits in trunk/Source
Web Inspector: remove some unnecessary Inspector prefixes from class names in Inspector namespace
https://bugs.webkit.org/show_bug.cgi?id=141372
Reviewed by Joseph Pecoraro.
Source/JavaScriptCore:
- inspector/ConsoleMessage.cpp:
(Inspector::ConsoleMessage::addToFrontend):
(Inspector::ConsoleMessage::updateRepeatCountInConsole):
- inspector/ConsoleMessage.h:
- inspector/InspectorAgentBase.h:
- inspector/InspectorAgentRegistry.cpp:
(Inspector::AgentRegistry::AgentRegistry):
(Inspector::AgentRegistry::append):
(Inspector::AgentRegistry::appendExtraAgent):
(Inspector::AgentRegistry::didCreateFrontendAndBackend):
(Inspector::AgentRegistry::willDestroyFrontendAndBackend):
(Inspector::AgentRegistry::discardAgents):
(Inspector::InspectorAgentRegistry::InspectorAgentRegistry): Deleted.
(Inspector::InspectorAgentRegistry::append): Deleted.
(Inspector::InspectorAgentRegistry::appendExtraAgent): Deleted.
(Inspector::InspectorAgentRegistry::didCreateFrontendAndBackend): Deleted.
(Inspector::InspectorAgentRegistry::willDestroyFrontendAndBackend): Deleted.
(Inspector::InspectorAgentRegistry::discardAgents): Deleted.
- inspector/InspectorAgentRegistry.h:
- inspector/InspectorBackendDispatcher.cpp:
(Inspector::BackendDispatcher::CallbackBase::CallbackBase):
(Inspector::BackendDispatcher::CallbackBase::isActive):
(Inspector::BackendDispatcher::CallbackBase::sendFailure):
(Inspector::BackendDispatcher::CallbackBase::sendIfActive):
(Inspector::BackendDispatcher::create):
(Inspector::BackendDispatcher::registerDispatcherForDomain):
(Inspector::BackendDispatcher::dispatch):
(Inspector::BackendDispatcher::sendResponse):
(Inspector::BackendDispatcher::reportProtocolError):
(Inspector::BackendDispatcher::getInteger):
(Inspector::BackendDispatcher::getDouble):
(Inspector::BackendDispatcher::getString):
(Inspector::BackendDispatcher::getBoolean):
(Inspector::BackendDispatcher::getObject):
(Inspector::BackendDispatcher::getArray):
(Inspector::BackendDispatcher::getValue):
(Inspector::InspectorBackendDispatcher::CallbackBase::CallbackBase): Deleted.
(Inspector::InspectorBackendDispatcher::CallbackBase::isActive): Deleted.
(Inspector::InspectorBackendDispatcher::CallbackBase::sendFailure): Deleted.
(Inspector::InspectorBackendDispatcher::CallbackBase::sendIfActive): Deleted.
(Inspector::InspectorBackendDispatcher::create): Deleted.
(Inspector::InspectorBackendDispatcher::registerDispatcherForDomain): Deleted.
(Inspector::InspectorBackendDispatcher::dispatch): Deleted.
(Inspector::InspectorBackendDispatcher::sendResponse): Deleted.
(Inspector::InspectorBackendDispatcher::reportProtocolError): Deleted.
(Inspector::InspectorBackendDispatcher::getInteger): Deleted.
(Inspector::InspectorBackendDispatcher::getDouble): Deleted.
(Inspector::InspectorBackendDispatcher::getString): Deleted.
(Inspector::InspectorBackendDispatcher::getBoolean): Deleted.
(Inspector::InspectorBackendDispatcher::getObject): Deleted.
(Inspector::InspectorBackendDispatcher::getArray): Deleted.
(Inspector::InspectorBackendDispatcher::getValue): Deleted.
- inspector/InspectorBackendDispatcher.h:
(Inspector::SupplementalBackendDispatcher::SupplementalBackendDispatcher):
(Inspector::SupplementalBackendDispatcher::~SupplementalBackendDispatcher):
(Inspector::InspectorSupplementalBackendDispatcher::InspectorSupplementalBackendDispatcher): Deleted.
(Inspector::InspectorSupplementalBackendDispatcher::~InspectorSupplementalBackendDispatcher): Deleted.
- inspector/InspectorFrontendChannel.h:
(Inspector::FrontendChannel::~FrontendChannel):
(Inspector::InspectorFrontendChannel::~InspectorFrontendChannel): Deleted.
- inspector/JSGlobalObjectInspectorController.cpp:
(Inspector::JSGlobalObjectInspectorController::JSGlobalObjectInspectorController):
(Inspector::JSGlobalObjectInspectorController::globalObjectDestroyed):
(Inspector::JSGlobalObjectInspectorController::connectFrontend):
(Inspector::JSGlobalObjectInspectorController::disconnectFrontend):
(Inspector::JSGlobalObjectInspectorController::dispatchMessageFromFrontend):
(Inspector::JSGlobalObjectInspectorController::appendExtraAgent):
- inspector/JSGlobalObjectInspectorController.h:
- inspector/agents/InspectorAgent.cpp:
(Inspector::InspectorAgent::didCreateFrontendAndBackend):
(Inspector::InspectorAgent::willDestroyFrontendAndBackend):
- inspector/agents/InspectorAgent.h:
- inspector/agents/InspectorConsoleAgent.cpp:
(Inspector::InspectorConsoleAgent::didCreateFrontendAndBackend):
(Inspector::InspectorConsoleAgent::willDestroyFrontendAndBackend):
- inspector/agents/InspectorConsoleAgent.h:
- inspector/agents/InspectorDebuggerAgent.cpp:
(Inspector::InspectorDebuggerAgent::didCreateFrontendAndBackend):
(Inspector::InspectorDebuggerAgent::willDestroyFrontendAndBackend):
(Inspector::InspectorDebuggerAgent::handleConsoleAssert):
(Inspector::InspectorDebuggerAgent::schedulePauseOnNextStatement):
(Inspector::InspectorDebuggerAgent::pause):
(Inspector::InspectorDebuggerAgent::scriptExecutionBlockedByCSP):
(Inspector::InspectorDebuggerAgent::didPause):
(Inspector::InspectorDebuggerAgent::breakProgram):
(Inspector::InspectorDebuggerAgent::clearBreakDetails):
- inspector/agents/InspectorDebuggerAgent.h:
- inspector/agents/InspectorRuntimeAgent.cpp:
(Inspector::InspectorRuntimeAgent::willDestroyFrontendAndBackend):
- inspector/agents/InspectorRuntimeAgent.h:
- inspector/agents/JSGlobalObjectRuntimeAgent.cpp:
(Inspector::JSGlobalObjectRuntimeAgent::didCreateFrontendAndBackend):
(Inspector::JSGlobalObjectRuntimeAgent::willDestroyFrontendAndBackend):
- inspector/agents/JSGlobalObjectRuntimeAgent.h:
- inspector/augmentable/AlternateDispatchableAgent.h:
- inspector/augmentable/AugmentableInspectorController.h:
- inspector/remote/RemoteInspectorDebuggable.h:
- inspector/remote/RemoteInspectorDebuggableConnection.h:
- inspector/scripts/codegen/cpp_generator.py:
(CppGenerator.cpp_type_for_formal_out_parameter):
(CppGenerator.cpp_type_for_stack_out_parameter):
- inspector/scripts/codegen/cpp_generator_templates.py:
(AlternateBackendDispatcher):
(Alternate):
(void):
(AlternateInspectorBackendDispatcher): Deleted.
(AlternateInspector): Deleted.
- inspector/scripts/codegen/generate_cpp_backend_dispatcher_header.py:
(CppBackendDispatcherHeaderGenerator._generate_alternate_handler_forward_declarations_for_domains.Alternate):
(CppBackendDispatcherHeaderGenerator._generate_dispatcher_declaration_for_command):
(CppBackendDispatcherHeaderGenerator._generate_alternate_handler_forward_declarations_for_domains.AlternateInspector): Deleted.
- inspector/scripts/codegen/generate_cpp_backend_dispatcher_implementation.py:
(CppBackendDispatcherImplementationGenerator._generate_handler_class_destructor_for_domain):
(CppBackendDispatcherImplementationGenerator._generate_large_dispatcher_switch_implementation_for_domain):
(CppBackendDispatcherImplementationGenerator._generate_dispatcher_implementation_for_command):
- inspector/scripts/codegen/generate_cpp_frontend_dispatcher_implementation.py:
(CppFrontendDispatcherImplementationGenerator._generate_dispatcher_implementation_for_event):
- inspector/scripts/codegen/generate_objc_frontend_dispatcher_implementation.py:
(ObjCFrontendDispatcherImplementationGenerator._generate_event):
- inspector/scripts/tests/expected/commands-with-async-attribute.json-result:
- inspector/scripts/tests/expected/commands-with-optional-call-return-parameters.json-result:
- inspector/scripts/tests/expected/domains-with-varying-command-sizes.json-result:
- inspector/scripts/tests/expected/enum-values.json-result:
- inspector/scripts/tests/expected/events-with-optional-parameters.json-result:
- inspector/scripts/tests/expected/generate-domains-with-feature-guards.json-result:
- inspector/scripts/tests/expected/same-type-id-different-domain.json-result:
- inspector/scripts/tests/expected/shadowed-optional-type-setters.json-result:
- inspector/scripts/tests/expected/type-declaration-aliased-primitive-type.json-result:
- inspector/scripts/tests/expected/type-declaration-array-type.json-result:
- inspector/scripts/tests/expected/type-declaration-enum-type.json-result:
- inspector/scripts/tests/expected/type-declaration-object-type.json-result:
- inspector/scripts/tests/expected/type-requiring-runtime-casts.json-result:
- runtime/JSGlobalObjectDebuggable.cpp:
(JSC::JSGlobalObjectDebuggable::connect):
(JSC::JSGlobalObjectDebuggable::disconnect):
- runtime/JSGlobalObjectDebuggable.h:
Source/WebCore:
- WebCore.exp.in:
- inspector/InspectorApplicationCacheAgent.cpp:
(WebCore::InspectorApplicationCacheAgent::didCreateFrontendAndBackend):
(WebCore::InspectorApplicationCacheAgent::willDestroyFrontendAndBackend):
- inspector/InspectorApplicationCacheAgent.h:
- inspector/InspectorCSSAgent.cpp:
(WebCore::InspectorCSSAgent::didCreateFrontendAndBackend):
(WebCore::InspectorCSSAgent::willDestroyFrontendAndBackend):
- inspector/InspectorCSSAgent.h:
- inspector/InspectorController.cpp:
(WebCore::InspectorController::InspectorController):
(WebCore::InspectorController::inspectedPageDestroyed):
(WebCore::InspectorController::connectFrontend):
(WebCore::InspectorController::disconnectFrontend):
(WebCore::InspectorController::show):
(WebCore::InspectorController::close):
(WebCore::InspectorController::dispatchMessageFromFrontend):
- inspector/InspectorController.h:
- inspector/InspectorDOMAgent.cpp:
(WebCore::InspectorDOMAgent::didCreateFrontendAndBackend):
(WebCore::InspectorDOMAgent::willDestroyFrontendAndBackend):
- inspector/InspectorDOMAgent.h:
- inspector/InspectorDOMDebuggerAgent.cpp:
(WebCore::InspectorDOMDebuggerAgent::didCreateFrontendAndBackend):
(WebCore::InspectorDOMDebuggerAgent::willDestroyFrontendAndBackend):
(WebCore::InspectorDOMDebuggerAgent::didInvalidateStyleAttr):
(WebCore::InspectorDOMDebuggerAgent::willInsertDOMNode):
(WebCore::InspectorDOMDebuggerAgent::willRemoveDOMNode):
(WebCore::InspectorDOMDebuggerAgent::willModifyDOMAttr):
(WebCore::InspectorDOMDebuggerAgent::pauseOnNativeEventIfNeeded):
(WebCore::InspectorDOMDebuggerAgent::willSendXMLHttpRequest):
- inspector/InspectorDOMDebuggerAgent.h:
- inspector/InspectorDOMStorageAgent.cpp:
(WebCore::InspectorDOMStorageAgent::didCreateFrontendAndBackend):
(WebCore::InspectorDOMStorageAgent::willDestroyFrontendAndBackend):
- inspector/InspectorDOMStorageAgent.h:
- inspector/InspectorDatabaseAgent.cpp:
(WebCore::InspectorDatabaseAgent::didCreateFrontendAndBackend):
(WebCore::InspectorDatabaseAgent::willDestroyFrontendAndBackend):
- inspector/InspectorDatabaseAgent.h:
- inspector/InspectorDatabaseResource.cpp:
(WebCore::InspectorDatabaseResource::bind):
- inspector/InspectorDatabaseResource.h:
- inspector/InspectorForwarding.h:
- inspector/InspectorIndexedDBAgent.cpp:
(WebCore::InspectorIndexedDBAgent::didCreateFrontendAndBackend):
(WebCore::InspectorIndexedDBAgent::willDestroyFrontendAndBackend):
- inspector/InspectorIndexedDBAgent.h:
- inspector/InspectorLayerTreeAgent.cpp:
(WebCore::InspectorLayerTreeAgent::didCreateFrontendAndBackend):
(WebCore::InspectorLayerTreeAgent::willDestroyFrontendAndBackend):
- inspector/InspectorLayerTreeAgent.h:
- inspector/InspectorPageAgent.cpp:
(WebCore::InspectorPageAgent::didCreateFrontendAndBackend):
(WebCore::InspectorPageAgent::willDestroyFrontendAndBackend):
(WebCore::InspectorPageAgent::getScriptExecutionStatus):
- inspector/InspectorPageAgent.h:
- inspector/InspectorReplayAgent.cpp:
(WebCore::InspectorReplayAgent::didCreateFrontendAndBackend):
(WebCore::InspectorReplayAgent::willDestroyFrontendAndBackend):
- inspector/InspectorReplayAgent.h:
- inspector/InspectorResourceAgent.cpp:
(WebCore::InspectorResourceAgent::didCreateFrontendAndBackend):
(WebCore::InspectorResourceAgent::willDestroyFrontendAndBackend):
- inspector/InspectorResourceAgent.h:
- inspector/InspectorStyleSheet.cpp:
(WebCore::InspectorStyle::styleWithProperties):
- inspector/InspectorTimelineAgent.cpp:
(WebCore::InspectorTimelineAgent::didCreateFrontendAndBackend):
(WebCore::InspectorTimelineAgent::willDestroyFrontendAndBackend):
- inspector/InspectorTimelineAgent.h:
- inspector/InspectorWorkerAgent.cpp:
(WebCore::InspectorWorkerAgent::WorkerFrontendChannel::WorkerFrontendChannel):
(WebCore::InspectorWorkerAgent::didCreateFrontendAndBackend):
(WebCore::InspectorWorkerAgent::willDestroyFrontendAndBackend):
- inspector/InspectorWorkerAgent.h:
- inspector/PageRuntimeAgent.cpp:
(WebCore::PageRuntimeAgent::didCreateFrontendAndBackend):
(WebCore::PageRuntimeAgent::willDestroyFrontendAndBackend):
- inspector/PageRuntimeAgent.h:
- inspector/WorkerInspectorController.cpp:
(WebCore::WorkerInspectorController::~WorkerInspectorController):
(WebCore::WorkerInspectorController::connectFrontend):
(WebCore::WorkerInspectorController::disconnectFrontend):
- inspector/WorkerInspectorController.h:
- inspector/WorkerRuntimeAgent.cpp:
(WebCore::WorkerRuntimeAgent::didCreateFrontendAndBackend):
(WebCore::WorkerRuntimeAgent::willDestroyFrontendAndBackend):
- inspector/WorkerRuntimeAgent.h:
- loader/EmptyClients.h:
- page/PageDebuggable.cpp:
(WebCore::PageDebuggable::connect):
(WebCore::PageDebuggable::disconnect):
- page/PageDebuggable.h:
- testing/Internals.cpp:
(WebCore::Internals::closeDummyInspectorFrontend):
- workers/WorkerMessagingProxy.cpp:
(WebCore::WorkerMessagingProxy::disconnectFromInspector):
Source/WebKit:
- WebKit.vcxproj/WebKitExportGenerator/WebKitExports.def.in:
Source/WebKit/mac:
- WebCoreSupport/WebInspectorClient.mm:
(-[WebInspectorWindowController destroyInspectorView:]):
Source/WebKit/win:
- WebCoreSupport/WebInspectorClient.cpp:
(WebInspectorFrontendClient::destroyInspectorView):
Source/WebKit2:
- WebProcess/WebPage/WebInspector.cpp:
(WebKit::WebInspector::remoteFrontendDisconnected):
- 2:56 PM Changeset in webkit [180115] by
-
- 5 edits in trunk/Source/WebKit2
REGRESSION: Page opens with enlarged font after visiting PDF, navigating back,
then doing a process swap
https://bugs.webkit.org/show_bug.cgi?id=141584
-and corresponding-
rdar://problem/18167729
Reviewed by Tim Horton.
This patch keeps the plugin zoom/scale factors separate from page zoom/scale
factors in the UI process since they are used for slightly different purposes for
plugins (i.e., PDFs) than they are for normal pages. Keeping track of the right
factor for the right type of document will ensure that we don’t use the wrong one.
- UIProcess/WebPageProxy.cpp:
(WebKit::WebPageProxy::WebPageProxy):
(WebKit::WebPageProxy::pageZoomFactor):
(WebKit::WebPageProxy::pageScaleFactor):
(WebKit::WebPageProxy::pluginScaleFactorDidChange):
(WebKit::WebPageProxy::pluginZoomFactorDidChange):
(WebKit::WebPageProxy::didCommitLoadForFrame):
(WebKit::WebPageProxy::pageZoomFactorDidChange): Deleted.
- UIProcess/WebPageProxy.h:
(WebKit::WebPageProxy::pageZoomFactor): Deleted.
(WebKit::WebPageProxy::pageScaleFactor): Deleted.
- UIProcess/WebPageProxy.messages.in:
- WebProcess/Plugins/PluginView.cpp:
(WebKit::PluginView::setPageScaleFactor):
- 1:52 PM Changeset in webkit [180114] by
-
- 11 edits in trunk/Source
REGRESSION (r180082): WebCore Debug builds fail on Mavericks due to weak export symbols
<http://webkit.org/b/141607>
Work towards fixing the Mavericks Debug build.
Source/JavaScriptCore:
- inspector/ScriptDebugServer.h:
(Inspector::ScriptDebugServer::Task): Export class.
- inspector/agents/InspectorDebuggerAgent.h:
(Inspector::InspectorDebuggerAgent::Listener): Export class.
- runtime/JSGlobalObject.h:
(JSC::JSGlobalObject::setConsoleClient): Do not mark inline
method for export.
Source/WebCore:
- dom/Document.h:
(WebCore::Document::setAnnotatedRegionsDirty):
- dom/Event.h:
(WebCore::Event::create):
- dom/Node.h:
(WebCore::Node::hasEditableStyle):
- dom/Position.h:
(WebCore::Position::Position):
- editing/FrameSelection.h:
(WebCore::DragCaretController::clear):
- loader/ResourceLoader.h:
(WebCore::ResourceLoader::originalRequest):
- Do not mark inline methods for export.
- 12:41 PM Changeset in webkit [180113] by
-
- 4 edits in trunk
Web Inspector: Symbol RemoteObject should not send sub-type
https://bugs.webkit.org/show_bug.cgi?id=141604
Patch by Joseph Pecoraro <Joseph Pecoraro> on 2015-02-14
Reviewed by Brian Burg.
Source/JavaScriptCore:
- inspector/InjectedScriptSource.js:
LayoutTests:
- inspector/model/remote-object-expected.txt:
- 11:34 AM Changeset in webkit [180112] by
-
- 2 edits in trunk/Source/WebKit2
REGRESSION(r179573): Web Inspector toolbar is completely messed up on Mavericks
https://bugs.webkit.org/show_bug.cgi?id=141555
Reviewed by Joseph Pecoraro.
The old WKView-based code called _setDrawsBackground:NO on the WKView for Mavericks.
Simulate this old behavior by calling WebPageProxy::setDrawsBackground(false).
WKWebView's setOpaque:BOOL gives a flat background color instead of the intended
gradient, so we can't use that method.
- UIProcess/mac/WebInspectorProxyMac.mm:
(WebKit::WebInspectorProxy::platformCreateInspectorPage):
- 10:59 AM Changeset in webkit [180111] by
-
- 2 edits in trunk/LayoutTests
After updating tests to use kerning, ligatures, and printer fonts, some tests fail
https://bugs.webkit.org/show_bug.cgi?id=139968
Re-adding the test to expectations, it fails flakily (perhaps it's order dependent).
- platform/mac/TestExpectations:
- 10:16 AM Changeset in webkit [180110] by
-
- 12 edits6 adds in trunk
rel="noreferrer" should make window.opener null
https://bugs.webkit.org/show_bug.cgi?id=141579
Reviewed by Darin Adler.
Source/WebCore:
Tests: http/tests/navigation/target-blank-opener-post.html
http/tests/navigation/target-blank-opener.html
We used to avoid passing window.opener policy by temporarily storing it in a FrameLoader
member variable. This works for some clients - ones that invoke delegate callbacks
synchronously - but not in the general case.
So, changed to passing the policy explicitly.
- WebCore.exp.in:
- loader/FrameLoader.cpp:
(WebCore::FrameLoader::FrameLoader):
(WebCore::FrameLoader::urlSelected):
(WebCore::FrameLoader::loadURLIntoChildFrame):
(WebCore::FrameLoader::loadFrameRequest):
(WebCore::FrameLoader::loadURL):
(WebCore::FrameLoader::load):
(WebCore::FrameLoader::loadPostRequest):
(WebCore::FrameLoader::continueLoadAfterNewWindowPolicy):
- loader/FrameLoader.h:
(WebCore::FrameLoader::suppressOpenerInNewFrame): Deleted.
- loader/FrameLoaderTypes.h:
- loader/NavigationScheduler.cpp:
- page/ContextMenuController.cpp:
(WebCore::openNewWindow):
(WebCore::ContextMenuController::contextMenuItemSelected):
Source/WebKit/ios:
- WebView/WebPDFViewPlaceholder.mm:
(-[WebPDFViewPlaceholder simulateClickOnLinkToURL:]): Updated for a new WebCore
function signature. There is no rel="noreferrer" in PDF, so we can just always allow.
Source/WebKit/mac:
- WebView/WebPDFView.mm:
(-[WebPDFView PDFViewWillClickOnLink:withURL:]): Updated for a new WebCore
function signature. There is no rel="noreferrer" in PDF, so we can just always allow.
LayoutTests:
Unfortunately, these tests are not quite real, because they pass even without the
fix. There reason is that delegates respond synchronously in WKTR and DRT.
But if there is any large refactoring, there is a non-zero chance that the tests
will catch future mistakes.
- http/tests/navigation/resources/target-blank-opener-post-window.php: Added.
- http/tests/navigation/resources/target-blank-opener-window.php: Added.
- http/tests/navigation/target-blank-opener-expected.txt: Added.
- http/tests/navigation/target-blank-opener-post-expected.txt: Added.
- http/tests/navigation/target-blank-opener-post.html: Added.
- http/tests/navigation/target-blank-opener.html: Added.
- 9:16 AM Changeset in webkit [180109] by
-
- 2 edits in trunk/Source/WebCore
REGRESSION (r180082): WebCore build on Mountain Lion fails due to weak export for i386
<rdar://problem/19838127>
- Configurations/WebCore.unexp: Add missing symbol.
- 8:05 AM Changeset in webkit [180108] by
-
- 2 edits in trunk/LayoutTests
Re-ordering expectations.
Unreviewed.
Patch by Myles C. Maxfield <mmaxfield@apple.com> on 2015-02-14
- platform/mac/TestExpectations:
- 8:01 AM Changeset in webkit [180107] by
-
- 58 edits in trunk/LayoutTests
[Mac] Cleanup after r177774
https://bugs.webkit.org/show_bug.cgi?id=141590
Patch by Myles C. Maxfield <mmaxfield@apple.com> on 2015-02-14
Reviewed by Zalan Bujtas.
Test expected results simply need to be updated.
- platform/mac/TestExpectations:
- platform/mac/accessibility/content-editable-as-textarea-expected.txt:
- platform/mac/accessibility/press-targets-center-point-expected.txt: Added.
- platform/mac/accessibility/table-attributes-expected.txt:
- platform/mac/accessibility/table-cell-spans-expected.txt:
- platform/mac/accessibility/table-detection-expected.txt:
- platform/mac/accessibility/table-one-cell-expected.txt:
- platform/mac/accessibility/table-sections-expected.txt:
- platform/mac/accessibility/table-with-rules-expected.txt:
- platform/mac/compositing/geometry/limit-layer-bounds-overflow-root-expected.txt: Added.
- platform/mac/compositing/layer-creation/fixed-position-out-of-view-scaled-expected.txt: Added.
- platform/mac/compositing/layer-creation/fixed-position-out-of-view-scaled-scroll-expected.txt: Added.
- platform/mac/compositing/overflow/clipping-behaviour-change-is-not-propagated-to-descendants-expected.txt:
- platform/mac/compositing/overflow/clipping-behaviour-change-is-not-propagated-to-descendants2-expected.txt:
- platform/mac/css1/box_properties/acid_test-expected.txt:
- platform/mac/css2.1/t09-c5526c-display-00-e-expected.txt:
- platform/mac/editing/selection/5057506-2-expected.txt:
- platform/mac/editing/selection/5057506-expected.txt:
- platform/mac/editing/selection/5825350-2-expected.txt:
- platform/mac/editing/selection/mixed-editability-10-expected.txt:
- platform/mac/fast/block/basic/011-expected.txt:
- platform/mac/fast/block/basic/minheight-expected.txt:
- platform/mac/fast/css-grid-layout/grid-item-order-in-content-sized-columns-resolution-expected.txt: Added.
- platform/mac/fast/css/002-expected.txt:
- platform/mac/fast/forms/basic-textareas-expected.txt:
- platform/mac/fast/frames/frame-scrolling-attribute-expected.txt:
- platform/mac/fast/invalid/008-expected.txt:
- platform/mac/fast/overflow/003-expected.txt:
- platform/mac/fast/repaint/fixed-move-after-keyboard-scroll-expected.txt:
- platform/mac/fast/repaint/line-flow-with-floats-9-expected.txt:
- platform/mac/fast/repaint/selection-rl-expected.txt:
- platform/mac/fast/repaint/selection-ruby-rl-expected.txt: Added.
- platform/mac/fast/sub-pixel/inline-block-with-padding-expected.txt: Added.
- platform/mac/fast/table/col-and-colgroup-offsets-expected.txt: Added.
- platform/mac/fast/text/atsui-pointtooffset-calls-cg-expected.txt:
- platform/mac/fast/text/basic/generic-family-reset-expected.txt:
- platform/mac/fast/text/international/vertical-text-metrics-test-expected.txt:
- platform/mac/http/tests/navigation/javascriptlink-frames-expected.txt:
- platform/mac/js/dom/dom-static-property-for-in-iteration-expected.txt: Added.
- platform/mac/media/track/track-cue-rendering-snap-to-lines-not-set-expected.txt: Added.
- platform/mac/platform/mac/accessibility/aria-columnrowheaders-expected.txt: Copied from LayoutTests/platform/mac/accessibility/table-cell-spans-expected.txt.
- platform/mac/platform/mac/accessibility/bounds-for-range-expected.txt: Added.
- platform/mac/platform/mac/accessibility/document-links-expected.txt: Added.
- platform/mac/platform/mac/accessibility/internal-link-anchors-expected.txt: Added.
- platform/mac/scrollbars/overflow-scrollbar-combinations-expected.txt:
- platform/mac/svg/W3C-SVG-1.1/masking-path-04-b-expected.txt:
- platform/mac/svg/batik/text/textEffect2-expected.txt:
- platform/mac/svg/batik/text/textEffect3-expected.txt:
- platform/mac/svg/batik/text/textProperties-expected.txt:
- platform/mac/svg/custom/svg-fonts-in-html-expected.txt:
- platform/mac/svg/custom/text-filter-expected.txt:
- platform/mac/svg/filters/feColorMatrix-values-expected.txt: Added.
- platform/mac/svg/filters/filter-on-filter-for-text-expected.txt:
- platform/mac/svg/filters/filter-on-tspan-expected.txt:
- platform/mac/svg/filters/sourceAlpha-expected.txt:
- platform/mac/svg/repaint/text-mask-update-expected.txt: Added.
- platform/mac/tables/mozilla/bugs/bug131020-expected.txt:
- 12:56 AM Changeset in webkit [180106] by
-
- 2 edits in trunk/Tools
[buildbot] Enable run-api-tests on release Mac bots
https://bugs.webkit.org/show_bug.cgi?id=141518
Reviewed by Darin Adler.
- BuildSlaveSupport/build.webkit.org-config/master.cfg:
(RunUnitTests):
(BuildFactory.init):
(TestFactory.init):
(BuildAndTestFactory.init):
(RunUnitTests.start): Deleted.
(unitTestsSupported): Deleted.
Feb 13, 2015:
- 10:37 PM Changeset in webkit [180105] by
-
- 2 edits in trunk/LayoutTests
http/tests/xmlhttprequest/event-listener-gc.html is flaky
https://bugs.webkit.org/show_bug.cgi?id=33342
Reviewed by Darin Adler.
Instead of using a zero delay timer, which can be legitimately delayed for dozens
of milliseconds, perform the collection in a second load event handler. GC code
has changed so much that I cannot confirm that the test still verifies the fix,
however we collect after returning for the function and unwinding the stack, so it should.
Also, clear the JS variable for a good measure.
- http/tests/xmlhttprequest/event-listener-gc.html:
- 10:20 PM Changeset in webkit [180104] by
-
- 3 edits in trunk/LayoutTests
Update fast/canvas/canvas-measureText-ideographicSpace.html after r177774
https://bugs.webkit.org/show_bug.cgi?id=141595
Reviewed by Zalan Bujtas.
- fast/canvas/canvas-measureText-ideographicSpace.html:
- platform/mac/TestExpectations:
- 9:16 PM Changeset in webkit [180103] by
-
- 2 edits in trunk/LayoutTests
Unreviewed, EFL gardening. Mark font & text tests to failure.
- platform/efl/TestExpectations:
- 9:16 PM Changeset in webkit [180102] by
-
- 3 edits in trunk/Source/JavaScriptCore
Attempt to fix 32bits build after r180098
Patch by Benjamin Poulain <bpoulain@apple.com> on 2015-02-13
- jit/JITOperations.cpp:
- jit/JITOperations.h:
I copied the attribute from the MathObject version of that function when I moved
it over. DFG has no version of a function call taking those attributes.
- 8:59 PM Changeset in webkit [180101] by
-
- 3 edits in trunk/Source/JavaScriptCore
JSContext Inspector: Do not stash console messages for non-debuggable JSContext
https://bugs.webkit.org/show_bug.cgi?id=141589
Patch by Joseph Pecoraro <Joseph Pecoraro> on 2015-02-13
Reviewed by Timothy Hatcher.
Consider developer extras disabled for JSContext inspection if the
RemoteInspector server is not enabled (typically a non-debuggable
process rejected by webinspectord) or if remote debugging on the
JSContext was explicitly disabled via SPI.
When developer extras are disabled, console message will not be stashed.
- inspector/JSGlobalObjectInspectorController.cpp:
(Inspector::JSGlobalObjectInspectorController::developerExtrasEnabled):
- inspector/JSGlobalObjectInspectorController.h:
- 8:53 PM Changeset in webkit [180100] by
-
- 2 edits in trunk/LayoutTests
Unreviewed, EFL gardening. 3 inspector tests are marked to failure.
- platform/efl/TestExpectations:
- 8:23 PM Changeset in webkit [180099] by
-
- 3 edits in branches/safari-600.1.4.15-branch/Source/WebCore
Merge r179768. rdar://problem/19733844.
- 8:20 PM Changeset in webkit [180098] by
-
- 25 edits4 adds in trunk/Source/JavaScriptCore
Add a DFG node for the Pow Intrinsics
https://bugs.webkit.org/show_bug.cgi?id=141540
Patch by Benjamin Poulain <bpoulain@apple.com> on 2015-02-13
Reviewed by Filip Pizlo.
Add a DFG Node for PowIntrinsic. This patch covers the basic cases
need to avoid massive regression. I will iterate over the node to cover
the missing types.
With this patch I get the following progressions on benchmarks:
-LongSpider's math-partial-sums: +5%.
-Kraken's imaging-darkroom: +17%
-AsmBench's cray.c: +6.6%
-CompressionBench: +2.2% globally.
- dfg/DFGAbstractInterpreterInlines.h:
(JSC::DFG::AbstractInterpreter<AbstractStateType>::executeEffects):
Cover a couple of trivial cases:
-If the exponent is zero, the result is always one, regardless of the base.
-If both arguments are constants, compute the result at compile time.
- dfg/DFGByteCodeParser.cpp:
(JSC::DFG::ByteCodeParser::handleIntrinsic):
- dfg/DFGClobberize.h:
(JSC::DFG::clobberize):
- dfg/DFGDoesGC.cpp:
(JSC::DFG::doesGC):
- dfg/DFGFixupPhase.cpp:
(JSC::DFG::FixupPhase::fixupNode):
We only support 2 basic cases at this time:
-Math.pow(double, int)
-Math.pow(double, double).
I'll cover Math.pow(int, int) in a follow up.
- dfg/DFGNode.h:
(JSC::DFG::Node::convertToArithSqrt):
(JSC::DFG::Node::arithNodeFlags):
- dfg/DFGNodeType.h:
- dfg/DFGPredictionPropagationPhase.cpp:
(JSC::DFG::PredictionPropagationPhase::propagate):
(JSC::DFG::PredictionPropagationPhase::doDoubleVoting):
- dfg/DFGSafeToExecute.h:
(JSC::DFG::safeToExecute):
- dfg/DFGSpeculativeJIT.cpp:
(JSC::DFG::compileArithPowIntegerFastPath):
(JSC::DFG::SpeculativeJIT::compileArithPow):
- dfg/DFGSpeculativeJIT.h:
- dfg/DFGSpeculativeJIT32_64.cpp:
(JSC::DFG::SpeculativeJIT::compile):
- dfg/DFGSpeculativeJIT64.cpp:
(JSC::DFG::SpeculativeJIT::compile):
- dfg/DFGStrengthReductionPhase.cpp:
(JSC::DFG::StrengthReductionPhase::handleNode):
- dfg/DFGValidate.cpp:
(JSC::DFG::Validate::validate):
- ftl/FTLCapabilities.cpp:
(JSC::FTL::canCompile):
- ftl/FTLIntrinsicRepository.h:
- ftl/FTLLowerDFGToLLVM.cpp:
(JSC::FTL::LowerDFGToLLVM::compileNode):
(JSC::FTL::LowerDFGToLLVM::compileArithPow):
- ftl/FTLOutput.h:
(JSC::FTL::Output::doublePow):
(JSC::FTL::Output::doublePowi):
- jit/JITOperations.cpp:
- jit/JITOperations.h:
- runtime/MathObject.cpp:
(JSC::mathProtoFuncPow):
(JSC::isDenormal): Deleted.
(JSC::isEdgeCase): Deleted.
(JSC::mathPow): Deleted.
- tests/stress/math-pow-basics.js: Added.
- tests/stress/math-pow-integer-exponent-fastpath.js: Added.
- tests/stress/math-pow-nan-behaviors.js: Added.
- tests/stress/math-pow-with-constants.js: Added.
Start some basic testing of Math.pow().
Due to the various transform, the value change when the code tiers up,
I covered this by checking for approximate values.
- 8:16 PM Changeset in webkit [180097] by
-
- 12 edits8 copies in branches/safari-600.1.4.15-branch
Merged r174823. rdar://problem/19733844
- 8:00 PM Changeset in webkit [180096] by
-
- 20 edits5 copies in branches/safari-600.1.4.15-branch
Merged r174402. rdar://problem/19733844
- 7:55 PM Changeset in webkit [180095] by
-
- 7 edits2 adds in branches/safari-600.5-branch
Source/WebCore: Merge r173806. <rdar://problem/19830405>
2014-09-22 Mihnea Ovidenie <mihnea@adobe.com>
[CSS Regions] Assertion failure and null dereference crash when using animations and regions
https://bugs.webkit.org/show_bug.cgi?id=136918
Reviewed by Andrei Bucur.
In some situations, for instance when an image has an attached animation, the style change caused by the animation
triggers a geometry update for the backing store associated with the image's layer. This may occur before
the layout for the image has finished.
Moreover, if the image in such situation - having a composited layer - is displayed in a region,
sicne the layout did not finish yet, the mappings between the layers of the elements collected in the named flow
and the regions associated with the named flow are not updated and cannot be used.
Therefore in those situations, we have to bail out early and use these mappings only after the layout has finished.
This patch also changes RenderLayerBacking method updateAfterDescendents -> updateAfterDescendants.
Test: fast/regions/animated-image-in-region.html
- rendering/RenderFlowThread.cpp: (WebCore::RenderFlowThread::cachedRegionForCompositedLayer):
- rendering/RenderLayer.cpp: (WebCore::RenderLayer::calculateClipRects):
- rendering/RenderLayerBacking.cpp: (WebCore::RenderLayerBacking::updateAfterDescendants):
- rendering/RenderLayerBacking.h:
- rendering/RenderLayerCompositor.cpp: (WebCore::RenderLayerCompositor::rebuildCompositingLayerTree): (WebCore::RenderLayerCompositor::updateLayerTreeGeometry): (WebCore::RenderLayerCompositor::updateCompositingDescendantGeometry):
LayoutTests: Merge r173806. <rdar://problem/19830405>
2014-09-22 Mihnea Ovidenie <mihnea@adobe.com>
[CSS Regions] Assertion failure and null dereference crash when using animations and regions
https://bugs.webkit.org/show_bug.cgi?id=136918
Reviewed by Andrei Bucur.
- fast/regions/animated-image-in-region-expected.txt: Added.
- fast/regions/animated-image-in-region.html: Added.
- 5:25 PM Changeset in webkit [180094] by
-
- 8 edits in trunk/Source/WebKit2
Crash closing a tab when a context or popup menu is open
https://bugs.webkit.org/show_bug.cgi?id=141582
rdar://problem/17700475
Reviewed by Anders Carlsson.
If a context menu or a popup menu is open when a tab is programmatically closed,
then we'd crash because both the WebContextMenuProxyMac/WebPopupMenuProxyMac
and the WebPageProxy would be deleted while still in use, via messages
handled via the nested event tracking runloop.
Fix by protecting those things while showing the popup. Also programmatically
dismiss the popup when closing the WebPageProxy.
- UIProcess/WebContextMenuProxy.h:
(WebKit::WebContextMenuProxy::cancelTracking):
- UIProcess/WebPageProxy.cpp:
(WebKit::WebPageProxy::close):
(WebKit::WebPageProxy::showPopupMenu): Clean up some EFL-related confusion that we don't need.
Retaining |this| will also retain m_activePopupMenu.
(WebKit::WebPageProxy::hidePopupMenu):
(WebKit::WebPageProxy::showContextMenu):
(WebKit::WebPageProxy::resetState):
- UIProcess/WebPopupMenuProxy.h:
(WebKit::WebPopupMenuProxy::cancelTracking):
- UIProcess/mac/WebContextMenuProxyMac.h:
- UIProcess/mac/WebContextMenuProxyMac.mm:
(WebKit::WebContextMenuProxyMac::showContextMenu):
(WebKit::WebContextMenuProxyMac::cancelTracking):
- UIProcess/mac/WebPopupMenuProxyMac.h: For popups, we need to remember if we were
canceled to avoid trying to send events after closing.
- UIProcess/mac/WebPopupMenuProxyMac.mm:
(WebKit::WebPopupMenuProxyMac::WebPopupMenuProxyMac):
(WebKit::WebPopupMenuProxyMac::showPopupMenu):
(WebKit::WebPopupMenuProxyMac::cancelTracking):
- 5:20 PM Changeset in webkit [180093] by
-
- 3 edits in trunk/Source/WTF
[EFL] Enable bmalloc for EFL port
https://bugs.webkit.org/show_bug.cgi?id=141459
Reviewed by Csaba Osztrogonác.
- wtf/CMakeLists.txt: Remove TCSystemAlloc.cpp/h.
- wtf/FastMalloc.cpp: Enable BMALLOC for EFL port.
- 5:09 PM Changeset in webkit [180092] by
-
- 2 edits in trunk/Websites/perf.webkit.org
Unreviewed build fix after r179591.
- public/api/commits.php:
- 4:59 PM Changeset in webkit [180091] by
-
- 2 edits in trunk/Websites/perf.webkit.org
The status of a A/B testing request always eventually becomes "Failed"
https://bugs.webkit.org/show_bug.cgi?id=141523
Reviewed by Andreas Kling.
The bug was caused by /api/build-requests always setting the status of a build request to 'failed' when
'failedIfNotCompleted' was sent by the buildbot sync'er.
Fixed the bug by only setting the status to 'failed' if it wasn't set to 'completed'.
- public/api/build-requests.php:
(main):
- 4:37 PM Changeset in webkit [180090] by
-
- 3 edits2 adds in trunk
RenderListItem resets its marker's style on style change even if the diff is StyleDifferenceEqual
https://bugs.webkit.org/show_bug.cgi?id=141572
<rdar://problem/19759818>
Reviewed by Simon Fraser.
Source/WebCore:
Do not unnecessarily update the marker's style in RenderListItem::styleDidChange()
if the RenderStyle diff is StyleDifferenceEqual. Doing so can cause unnecessary
layouts.
This was causing high cpu usage on
http://www.nokogiri.org/tutorials/searching_a_xml_html_document.html because there
is an animation that changes the style every second (but the diff is equal) and the
RenderListItem keeps resetting its marker's margin, which triggers a layout and the
RenderListMarker re-computes its margin during layout and updates its style again.
With this change, CPU usage on the page goes from 150% to 20%. There is only a style
recalc happening every second, no more relayouts and repaints.
Test: fast/repaint/list-item-equal-style-change-no-repaint.html
- rendering/RenderListItem.cpp:
(WebCore::RenderListItem::styleDidChange):
LayoutTests:
Add a layout test to make sure that an animation causing a style change
with StyleDifferenceEqual diff does not cause a repaint.
- fast/repaint/list-item-equal-style-change-no-repaint-expected.txt: Added.
- fast/repaint/list-item-equal-style-change-no-repaint.html: Added.
- 4:15 PM Changeset in webkit [180089] by
-
- 3 edits in trunk/Source/WebKit2
Fix the iOS build.
- UIProcess/ios/ViewGestureControllerIOS.mm:
(WebKit::ViewGestureController::removeSwipeSnapshot):
- UIProcess/mac/ViewGestureController.h:
This code is never used on iOS.
- 4:06 PM Changeset in webkit [180088] by
-
- 14 edits in trunk/Source
Make WebCore::IOSurface have single ownership instead of refcounting
https://bugs.webkit.org/show_bug.cgi?id=141578
Reviewed by Anders Carlsson.
In the interests of making it harder to misuse and to make it more like
ImageBuffer, our IOSurface wrapper should have single ownership.
This will also allow future changes which depend on temporarily
consuming an IOSurface to more easily enforce correct usage.
- WebCore.exp.in:
- platform/graphics/cg/IOSurfacePool.cpp:
(WebCore::IOSurfacePool::takeSurface):
(WebCore::IOSurfacePool::addSurface):
(WebCore::IOSurfacePool::insertSurfaceIntoPool):
(WebCore::IOSurfacePool::tryEvictInUseSurface):
(WebCore::IOSurfacePool::tryEvictOldestCachedSurface):
(WebCore::IOSurfacePool::collectInUseSurfaces):
- platform/graphics/cg/IOSurfacePool.h:
- platform/graphics/cocoa/IOSurface.h:
- platform/graphics/cocoa/IOSurface.mm:
(IOSurface::create):
(IOSurface::createFromSendRight):
(IOSurface::createFromSurface):
(IOSurface::createFromImage):
Make IOSurface::create()s return a unique_ptr, and adjust everywhere.
- Shared/mac/RemoteLayerBackingStore.h:
- Shared/mac/RemoteLayerBackingStore.mm:
(WebKit::RemoteLayerBackingStore::Buffer::discard):
- UIProcess/API/mac/WKView.mm:
(-[WKView _takeViewSnapshot]):
- UIProcess/mac/ViewGestureController.h:
- UIProcess/mac/ViewGestureControllerMac.mm:
(WebKit::ViewGestureController::beginSwipeGesture):
(WebKit::ViewGestureController::removeSwipeSnapshot):
- UIProcess/mac/ViewSnapshotStore.h:
- UIProcess/mac/ViewSnapshotStore.mm:
(WebKit::ViewSnapshot::create):
(WebKit::ViewSnapshot::ViewSnapshot):
(WebKit::ViewSnapshot::hasImage):
Adjust to the single-ownership model.
- 3:44 PM Changeset in webkit [180087] by
-
- 6 edits in trunk
Additional emoji group support.
https://bugs.webkit.org/show_bug.cgi?id=141539
rdar://problem/19727527
Reviewed by Sam Weinig.
Source/WebCore:
Adding some new emoji ligatures.
Updated existing test to include the new sequences.
- platform/text/TextBreakIterator.cpp:
(WebCore::cursorMovementIterator):
- rendering/RenderText.cpp:
(WebCore::isEmojiGroupCandidate):
LayoutTests:
Updating test to reflect the new emoji ligatures supported.
- editing/deleting/delete-emoji-expected.txt:
- editing/deleting/delete-emoji.html:
- 3:27 PM Changeset in webkit [180086] by
-
- 3 edits in trunk/Source/WebKit2
[iOS] Unreviewed build fix after r180076.
- UIProcess/WebPageProxy.cpp:
(WebKit::WebPageProxy::creationParameters): We do not
want to do this on iOS.
- UIProcess/WebPageProxy.h: No need to expose function
on iOS (only Mac).
- 3:08 PM Changeset in webkit [180085] by
-
- 6 edits2 adds in trunk/Source/JavaScriptCore
ArithSqrt should not be conditional on supportsFloatingPointSqrt
https://bugs.webkit.org/show_bug.cgi?id=141546
Patch by Benjamin Poulain <bpoulain@apple.com> on 2015-02-13
Reviewed by Geoffrey Garen and Filip Pizlo.
Just fallback to the function call in the DFG codegen.
- dfg/DFGByteCodeParser.cpp:
(JSC::DFG::ByteCodeParser::handleIntrinsic):
- dfg/DFGSpeculativeJIT.cpp:
(JSC::DFG::SpeculativeJIT::compileArithSqrt):
- dfg/DFGSpeculativeJIT.h:
- dfg/DFGSpeculativeJIT32_64.cpp:
(JSC::DFG::SpeculativeJIT::compile):
- dfg/DFGSpeculativeJIT64.cpp:
(JSC::DFG::SpeculativeJIT::compile):
- tests/stress/math-sqrt-basics.js: Added.
Basic coverage.
- tests/stress/math-sqrt-basics-disable-architecture-specific-optimizations.js: Added.
Same tests but forcing the function call.
- 2:57 PM Changeset in webkit [180084] by
-
- 7 edits4 copies2 deletes in branches/safari-600.1.4.15-branch
Merged r179933. rdar://problem/19812665
- 2:46 PM Changeset in webkit [180083] by
-
- 2 edits in trunk/Source/JavaScriptCore
REGRESSION(r180060) New js/regress-141098 test crashes when LLInt is disabled.
https://bugs.webkit.org/show_bug.cgi?id=141577
Reviewed by Benjamin Poulain.
Changed the prologue of the baseline JIT to check for stack space for all
types of code blocks. Previously, it was only checking Function. Now
it checks Program and Eval as well.
- jit/JIT.cpp:
(JSC::JIT::privateCompile):
- 2:40 PM Changeset in webkit [180082] by
-
- 2 edits1 add in trunk/Source/WebCore
Really stop using export files.
https://bugs.webkit.org/show_bug.cgi?id=141521
Reviewed by Mark Rowe.
- Configurations/WebCore.xcconfig:
Don't use exported symbols files.
- Configurations/WebCore.unexp:
Added list of symbols not to export (needed by Xcode 5.0.2).
- 2:18 PM Changeset in webkit [180081] by
-
- 2 edits in trunk/Source/WebCore
Simplify ASSERT in lastRubyRun().
https://bugs.webkit.org/show_bug.cgi?id=141574
Reviewed by Daniel Bates.
r180064 made some of the conditions in the ASSERT redundant.
Covered by existing tests.
- rendering/RenderRuby.cpp:
(WebCore::lastRubyRun):
- 2:17 PM Changeset in webkit [180080] by
-
- 5 edits in branches/safari-600.1.4.15-branch/Source
Versioning.
- 2:15 PM Changeset in webkit [180079] by
-
- 1 copy in tags/Safari-600.1.4.15.5
New tag.
- 2:11 PM Changeset in webkit [180078] by
-
- 2 edits in trunk/LayoutTests
[Win] Document some debug asserts in Accessibility (Windows) tests.
- platform/win/TestExpectations:
- 2:00 PM Changeset in webkit [180077] by
-
- 5 edits2 copies in branches/safari-600.1.4.15-branch
Merged r179772. rdar://problem/19719078
- 1:56 PM Changeset in webkit [180076] by
-
- 10 edits in trunk/Source
[Mac, iOS] Adjust pagination behavior for Mail.app printing use
https://bugs.webkit.org/show_bug.cgi?id=141569
<rdar://problem/14912763>
Reviewed by Anders Carlsson.
Source/WebCore:
- page/Settings.in: Add new pagination setting flag.
- rendering/RenderBlockFlow.cpp:
(WebCore::messageContainerName): Added.
(WebCore::needsPaginationQuirk): Added.
(WebCore::RenderBlockFlow::adjustLinePositionForPagination): Don't move the message content
div to a new page when using this special printing mode.
Source/WebKit2:
- Shared/WebPageCreationParameters.cpp:
(WebKit::WebPageCreationParameters::encode): Add new flag.
(WebKit::WebPageCreationParameters::decode): Ditto.
- Shared/WebPageCreationParameters.h: Ditto.
- UIProcess/WebPageProxy.cpp:
(WebKit::WebPageProxy::creationParameters): Set new page construction flag.
- UIProcess/WebPageProxy.h:
- UIProcess/mac/WebPageProxyMac.mm:
(WebKit::WebPageProxy::paginationQuirkEnabled): Ask for new pagination flag
when running as a Mail client.
- WebProcess/WebPage/WebPage.cpp:
(WebKit::WebPage::WebPage): Set up pagination based on desired settings.
- 1:54 PM Changeset in webkit [180075] by
-
- 3 edits in trunk/Source/JavaScriptCore
Generate incq instead of addq when the immediate value is one
https://bugs.webkit.org/show_bug.cgi?id=141548
Patch by Benjamin Poulain <bpoulain@apple.com> on 2015-02-13
Reviewed by Gavin Barraclough.
JSC emits "addq #1 (rXX)" *a lot*.
This patch replace that by incq, which is one byte shorter
and is the adviced form.
Sunspider: +0.47%
Octane: +0.28%
Kraken: +0.44%
AsmBench, CompressionBench: neutral.
- assembler/MacroAssemblerX86_64.h:
(JSC::MacroAssemblerX86_64::add64):
- assembler/X86Assembler.h:
(JSC::X86Assembler::incq_m):
- 1:48 PM Changeset in webkit [180074] by
-
- 4 edits in trunk/Source/JavaScriptCore
Little clean up of Bytecode Generator's Label
https://bugs.webkit.org/show_bug.cgi?id=141557
Reviewed by Michael Saboff.
- bytecompiler/BytecodeGenerator.h:
- bytecompiler/BytecodeGenerator.cpp:
Label was a friend of BytecodeGenerator in order to access
m_instructions. There is no need for that, BytecodeGenerator
has a public getter.
- bytecompiler/Label.h:
(JSC::Label::Label):
(JSC::Label::setLocation):
(JSC::BytecodeGenerator::newLabel):
Make it explicit that the generator must exist.
- 1:35 PM Changeset in webkit [180073] by
-
- 5 edits in branches/safari-600.4-branch/Source
Versioning.
- 1:33 PM Changeset in webkit [180072] by
-
- 1 copy in tags/Safari-600.4.9
New tag.
- 1:25 PM Changeset in webkit [180071] by
-
- 3 edits in trunk/Source/WebCore
[iOS] DumpRenderTree.app fails to link due to undefined classes
<rdar://problem/19831228>
Following r179945, DumpRenderTree.app fails to link due to undefined symbols:
Undefined symbols for architecture x86_64:
"_OBJC_CLASS_$_WebEvent", referenced from:
objc-class-ref in libDumpRenderTree.a(EventSendingController.o)
"_OBJC_CLASS_$_WebUndefined", referenced from:
objc-class-ref in libDumpRenderTree.a(ObjCController.o)
We need to export the classes WebScriptObject and WebEvent.
- bindings/objc/WebScriptObject.h:
- platform/ios/WebEvent.h:
- 12:40 PM Changeset in webkit [180070] by
-
- 2 edits in branches/safari-600.1.4.15-branch/Source/WebCore
Merged r180053. rdar://problem/19812624
- 12:37 PM Changeset in webkit [180069] by
-
- 2 edits in branches/safari-600.1.4.15-branch/Source/WebCore
Merged r179737. rdar://problem/19812624
- 12:30 PM Changeset in webkit [180068] by
-
- 2 edits in trunk/Source/WebCore
CachedResource::clearLoader() should self-destruct if nothing else retains the CachedResource.
<https://webkit.org/b/141568>
<rdar://problem/19800310>
Reviewed by Antti Koivisto.
Anything that may cause CachedResource::canDelete() to return true must also make sure
to call CachedResource::deleteIfPossible(), or we risk leaking the CachedResource.
This is because CachedResource employs an extremely convoluted lifetime mechanism that
depends on its presence in a number of collections, as well as internal counters and
state.
This is a speculative fix for a potential CachedResource leak that I'm not sure exists
in practice, but let's be good citizens here.
- loader/cache/CachedResource.cpp:
(WebCore::CachedResource::clearLoader):
- 11:57 AM Changeset in webkit [180067] by
-
- 1 edit1 add in trunk/Source/WebInspectorUI
Web Inspector: support better git diff of localizedStrings.js
https://bugs.webkit.org/show_bug.cgi?id=137237
Reviewed by Joseph Pecoraro.
- .gitattributes: Added.
- 11:52 AM Changeset in webkit [180066] by
-
- 4 edits in trunk
Add a move constructor and move assignment operator to Deque
https://bugs.webkit.org/show_bug.cgi?id=141571
Reviewed by Andreas Kling.
Source/WTF:
- wtf/Deque.h:
(WTF::inlineCapacity>::Deque):
(WTF::=): Deleted.
Tools:
- TestWebKitAPI/Tests/WTF/Deque.cpp:
(TestWebKitAPI::TEST):
- 11:50 AM Changeset in webkit [180065] by
-
- 2 edits in trunk/Source/WebCore
[MSE][Mac] Crash at WebCore::SourceBufferPrivateAVFObjC::didParseStreamDataAsAsset + 2357
https://bugs.webkit.org/show_bug.cgi?id=141566
rdar://problem/19826075
Reviewed by Andreas Kling.
Null check m_mediaSource before dereferencing.
- platform/graphics/avfoundation/objc/SourceBufferPrivateAVFObjC.mm:
(WebCore::SourceBufferPrivateAVFObjC::didParseStreamDataAsAsset):
- 11:28 AM Changeset in webkit [180064] by
-
- 3 edits2 adds in trunk
REGRESSION (r174761): Invalid cast in WebCore::lastRubyRun / WebCore::RenderRubyAsBlock::addChild
https://bugs.webkit.org/show_bug.cgi?id=137929
rdar://problem/18723492
Reviewed by Chris Dumez.
Ensure that the child renderer is a RenderRubyRun.
Source/WebCore:
Test: fast/ruby/crash-with-before-after-on-ruby.html
- rendering/RenderRuby.cpp:
(WebCore::lastRubyRun):
LayoutTests:
- fast/ruby/crash-with-before-after-on-ruby-expected.txt: Added.
- fast/ruby/crash-with-before-after-on-ruby.html: Added.
- 11:04 AM Changeset in webkit [180063] by
-
- 4 edits in trunk/Source/WebCore
Crashes under RenderLayer::hitTestLayer under determinePrimarySnapshottedPlugIn()
https://bugs.webkit.org/show_bug.cgi?id=141551
Reviewed by Zalan Bujtas.
It's possible for a layout to dirty the parent frame's state, via the calls to
ownerElement()->scheduleSetNeedsStyleRecalc() that RenderLayerCompositor does when
iframes toggle their compositing mode.
That could cause FrameView::updateLayoutAndStyleIfNeededRecursive() to fail to
leave all the frames in a clean state. Later on, we could enter hit testing,
which calls document().updateLayout() on each frame's document. Document::updateLayout()
does layout on all ancestor documents, so in the middle of hit testing, we could
layout a subframe (dirtying an ancestor frame), then layout another frame, which
would forcing that ancestor to be laid out while we're hit testing it, thus
corrupting the RenderLayer tree while it's being iterated over.
Fix by having FrameView::updateLayoutAndStyleIfNeededRecursive() do a second
layout after laying out subframes, which most of the time will be a no-op.
Also add a stronger assertion, that this frame and all subframes are clean
at the end of FrameView::updateLayoutAndStyleIfNeededRecursive() for the
main frame.
Various existing frames tests hit the new assertion if the code change is removed,
so this is covered by existing tests.
- page/FrameView.cpp:
(WebCore::FrameView::needsStyleRecalcOrLayout):
(WebCore::FrameView::updateLayoutAndStyleIfNeededRecursive):
- page/FrameView.h:
- rendering/RenderWidget.cpp:
(WebCore::RenderWidget::willBeDestroyed):
- 11:04 AM Changeset in webkit [180062] by
-
- 7 edits in trunk/Source
determinePrimarySnapshottedPlugIn() should only traverse visible Frames
https://bugs.webkit.org/show_bug.cgi?id=141547
Part of rdar://problem/18445733.
Reviewed by Anders Carlsson.
Source/WebCore:
There's an expectation from clients that FrameView::updateLayoutAndStyleIfNeededRecursive()
updates layout in all frames, but it uses the widget tree, so only hits frames
that are parented via renderers (i.e. not display:none frames or their descendants).
Moving towards a future where we remove Widgets, fix by adding a FrameTree
traversal function that only finds rendered frames (those with an ownerRenderer).
Not testable.
- page/FrameTree.cpp:
(WebCore::FrameTree::firstRenderedChild):
(WebCore::FrameTree::nextRenderedSibling):
(WebCore::FrameTree::traverseNextRendered):
(printFrames):
- page/FrameTree.h:
- page/FrameView.cpp:
(WebCore::FrameView::updateLayoutAndStyleIfNeededRecursive):
Source/WebKit2:
Use FrameTree::traverseNextRendered() to avoid doing things in unrendered frames
which are not guaranteed to have been laid out.
- WebProcess/WebPage/WebPage.cpp:
(WebKit::WebPage::determinePrimarySnapshottedPlugIn):
- 11:01 AM Changeset in webkit [180061] by
-
- 2 edits in trunk/Tools
run-jsc-stress-tests --remote should skip profiler tests
https://bugs.webkit.org/show_bug.cgi?id=141470
Reviewed by Filip Pizlo.
- Scripts/run-jsc-stress-tests:
- 10:57 AM Changeset in webkit [180060] by
-
- 6 edits3 adds in trunk
Google doc spreadsheet reproducibly crashes when sorting
https://bugs.webkit.org/show_bug.cgi?id=141098
Reviewed by Oliver Hunt.
Source/JavaScriptCore:
Moved the stack check to before the callee registers are allocated in the
prologue() by movving it from the functionInitialization() macro. This
way we can check the stack before moving the stack pointer, avoiding a
crash during a "call" instruction. Before this change, we weren't even
checking the stack for program and eval execution.
Made a couple of supporting changes.
- llint/LLIntSlowPaths.cpp:
(JSC::LLInt::llint_stack_check): We can't just go up one frame as we
may be processing an exception to an entry frame.
- llint/LowLevelInterpreter.asm:
- llint/LowLevelInterpreter32_64.asm:
- llint/LowLevelInterpreter64.asm:
(llint_throw_from_slow_path_trampoline): Changed method to get the vm
from the code block to not use the codeBlock, since we may need to
continue from an exception in a native function.
LayoutTests:
New test.
- js/regress-141098-expected.txt: Added.
- js/regress-141098.html: Added.
- js/script-tests/regress-141098.js: Added.
(probeAndRecurse):
- 10:46 AM Changeset in webkit [180059] by
-
- 4 edits in trunk
Add an initializer list constructor to Deque
https://bugs.webkit.org/show_bug.cgi?id=141565
Reviewed by Andreas Kling.
Source/WTF:
- wtf/Deque.h:
(WTF::inlineCapacity>::Deque):
Tools:
- TestWebKitAPI/Tests/WTF/Deque.cpp:
(TestWebKitAPI::TEST):
- 10:28 AM Changeset in webkit [180058] by
-
- 2 edits in trunk/Source/WebCore
TimerBase::m_heapInsertionOrder calculation is racy
https://bugs.webkit.org/show_bug.cgi?id=141554
Reviewed by Anders Carlsson.
Use an atomic increment.
- platform/Timer.cpp: (WebCore::TimerBase::setNextFireTime):
- 10:08 AM Changeset in webkit [180057] by
-
- 2 edits in branches/safari-600.1.4.15-branch
Merge r179771. rdar://problem/19395131
- 10:08 AM Changeset in webkit [180056] by
-
- 7 edits2 adds in branches/safari-600.1.4.15-branch
Merge r179994. rdar://problem/19711488
- 10:05 AM Changeset in webkit [180055] by
-
- 2 edits in trunk/Source/WTF
Re-indent Deque.h
Rubber-stamped by Antti Koivisto.
- wtf/Deque.h:
- 9:50 AM Changeset in webkit [180054] by
-
- 20 edits in trunk/Source/WebKit2
WorkQueue should support concurrent queues
https://bugs.webkit.org/show_bug.cgi?id=141559
Reviewed by Anders Carlsson.
- DatabaseProcess/DatabaseProcess.cpp:
(WebKit::DatabaseProcess::DatabaseProcess):
- Platform/IPC/Connection.h:
- Platform/IPC/mac/ConnectionMac.mm:
(IPC::Connection::open):
- Platform/WorkQueue.cpp:
(WorkQueue::create):
Also make create return a Ref.
(WorkQueue::WorkQueue):
- Platform/WorkQueue.h:
Add queue type enum.
- Platform/efl/WorkQueueEfl.cpp:
(WorkQueue::platformInitialize):
- Platform/gtk/WorkQueueGtk.cpp:
(WorkQueue::platformInitialize):
- Platform/mac/WorkQueueMac.cpp:
(WorkQueue::platformInitialize):
- Shared/mac/SecItemShim.cpp:
(WebKit::SecItemShim::initializeConnection):
- Shared/mac/SecItemShim.h:
- UIProcess/Launcher/ProcessLauncher.cpp:
(WebKit::processLauncherWorkQueue):
- UIProcess/mac/SecItemShimProxy.cpp:
(WebKit::SecItemShimProxy::initializeConnection):
- UIProcess/mac/SecItemShimProxy.h:
- WebProcess/Plugins/PluginProcessConnectionManager.cpp:
(WebKit::PluginProcessConnectionManager::initializeConnection):
- WebProcess/Plugins/PluginProcessConnectionManager.h:
- WebProcess/WebPage/EventDispatcher.cpp:
(WebKit::EventDispatcher::EventDispatcher):
(WebKit::EventDispatcher::initializeConnection):
- WebProcess/WebPage/EventDispatcher.h:
- WebProcess/WebPage/ViewUpdateDispatcher.h:
- 9:38 AM Changeset in webkit [180053] by
-
- 2 edits in trunk/Source/WebCore
[iOS] Refine GC behavior in response to process suspension and memory pressure.
<https://webkit.org/b/141543>
<rdar://problem/19738024>
Reviewed by Geoffrey Garen.
Do an immediate full garbage collection when the web process is about to
be suspended (when another tab is moving to the foreground on iOS.)
This ensures that we make a best effort to reduce the process footprint
before we lose the ability to execute code.
When receiving a memory pressure warning, tell the garbage collector to
accelerate its next collection (by calling garbageCollectSoon().)
This gives us some confidence that a collection will happen within a
reasonable timeframe, but doesn't risk dooming us to a loop of endless
garbage collections.
- platform/cocoa/MemoryPressureHandlerCocoa.mm:
(WebCore::MemoryPressureHandler::platformReleaseMemory):
- 9:35 AM Changeset in webkit [180052] by
-
- 4 edits1 move in trunk/Source/WebKit2
Clean up the framework postprocessing phase
https://bugs.webkit.org/show_bug.cgi?id=141544
Reviewed by Dan Bernstein.
Rename WK_API_AVAILABILITY_ENABLED to WK_FRAMEWORK_HEADER_POSTPROCESSING_ENABLED to better match what
it is we're doing. Also, rename the associated script. Remove a couple of now unnecessary #defines, and
change everything we can to be feature-based instead of OS based.
- Configurations/WebKit.xcconfig:
- Shared/API/Cocoa/WKFoundation.h:
- WebKit2.xcodeproj/project.pbxproj:
- mac/postprocess-framework-headers.sh: Renamed from Source/WebKit2/mac/rewrite-availability-macros.sh.
- 9:32 AM Changeset in webkit [180051] by
-
- 3 edits in trunk/Source/WebCore
Add some RELEASE_ASSERTs to try to catch crashes in StyleResolver::loadPendingImages
https://bugs.webkit.org/show_bug.cgi?id=141561
Reviewed by Simon Fraser.
One possibility is that loads triggered by loadPendingImages end up synchronously destroying or re-entering
style resolver. Try to catch these in release builds.
- css/StyleResolver.cpp:
(WebCore::StyleResolver::~StyleResolver):
(WebCore::StyleResolver::styleForElement):
(WebCore::StyleResolver::styleForKeyframe):
(WebCore::StyleResolver::styleForPage):
(WebCore::StyleResolver::loadPendingImages):
- css/StyleResolver.h:
- 9:15 AM Changeset in webkit [180050] by
-
- 4 edits2 adds in trunk
Div having contentEditable and display:grid cannot be edited if it is empty.
https://bugs.webkit.org/show_bug.cgi?id=141465
Reviewed by Ryosuke Niwa.
Source/WebCore:
This bug is quite similar to webkit.org/b/141218. RenderGrid should be also treated as a candidate
for visible position as like RenderFlexibleBox. The only different situation between them is
that RenderGrid has a bug setting "0px" for logicalHeight when it is empty. RenderGrid should also have
a minimum height of a single line if it is editable as well as RenderFlexibleBox does.
Test: fast/events/key-events-in-editable-gridbox.html
- dom/Position.cpp:
(WebCore::Position::isCandidate):
(WebCore::Position::isRenderedCharacter):
- rendering/RenderGrid.cpp:
(WebCore::RenderGrid::layoutGridItems):
LayoutTests:
This tests if an editable grid box can be reedited after erasing all texts in it.
- fast/events/key-events-in-editable-gridbox-expected.txt: Added.
- fast/events/key-events-in-editable-gridbox.html: Added.
- 9:10 AM Changeset in webkit [180049] by
-
- 2 edits in trunk/LayoutTests
[Win] Skip accessibility tests failing due to debug assertions.
- platform/win/TestExpectations:
- 9:02 AM Changeset in webkit [180048] by
-
- 4 edits2 adds in branches/safari-600.1.4.15-branch
Merge r179771. rdar://problem/19395131
- 9:02 AM Changeset in webkit [180047] by
-
- 2 edits in branches/safari-600.1.4.15-branch/Source/WebCore
Merge r179750. rdar://problem/19719085
- 9:02 AM Changeset in webkit [180046] by
-
- 3 edits2 adds in branches/safari-600.1.4.15-branch
Merge r179706. rdar://problem/19719085
- 8:43 AM Changeset in webkit [180045] by
-
- 7 edits in branches/safari-600.1.4.15-branch/Source
Merge r179580. rdar://problem/19709200
- 8:15 AM Changeset in webkit [180044] by
-
- 5 edits in branches/safari-600.1.4.15-branch/Source/WebKit2
Merge r180031. rdar://problem/19432928
- 8:15 AM Changeset in webkit [180043] by
-
- 4 edits in branches/safari-600.1.4.15-branch/Source/WebKit2
Merge r179885. rdar://problem/19812617
- 8:15 AM Changeset in webkit [180042] by
-
- 2 edits in branches/safari-600.1.4.15-branch/Source/WebKit2
Merge r179847. rdar://problem/19711485
- 8:15 AM Changeset in webkit [180041] by
-
- 6 edits2 adds in branches/safari-600.1.4.15-branch
Merge r179691. rdar://problem/19719089
- 8:15 AM Changeset in webkit [180040] by
-
- 3 edits in branches/safari-600.1.4.15-branch/Source/WebKit/ios
Merge r174284. rdar://problem/19812620
- 6:37 AM Changeset in webkit [180039] by
-
- 2 edits in trunk/Tools
[buildbot] Remove buildJSCTool cruft from master.cfg
https://bugs.webkit.org/show_bug.cgi?id=141513
Reviewed by Alexey Proskuryakov.
- BuildSlaveSupport/build.webkit.org-config/master.cfg:
(RunJavaScriptCoreTests):
(RunJavaScriptCoreTests.start):
(RunWebKitTests):
(RunWebKitTests.start):
(TestFactory.init):
(TestLeaksFactory.init):
(RunJavaScriptCoreTests.init): Deleted.
(RunWebKitTests.init): Deleted.
- 4:48 AM Changeset in webkit [180038] by
-
- 7 edits2 adds in trunk
REGRESSION (r176262): Invalid cast in WebCore`WebCore::RootInlineBox::selectionTop
https://bugs.webkit.org/show_bug.cgi?id=138992
Reviewed by Dave Hyatt.
RenderRubyText requires the ancestor chain to be (RenderRubyAsInline | RenderRubyAsBlock) -> RenderRubyRun -> RenderRubyText.
This patch ensures that we create RenderRubyText for an <rt> element only when
the expected ancestor chain is guaranteed.
Source/WebCore:
Test: fast/ruby/crash-when-ruby-is-set-to-inline-block.html
- rendering/RenderElement.cpp:
(WebCore::RenderElement::createFor): treat inline-block <ruby> as block.
- rendering/RenderObject.h:
(WebCore::RenderObject::isRubyInline):
(WebCore::RenderObject::isRubyBlock):
(WebCore::RenderObject::isRuby): Deleted.
- rendering/RenderRuby.cpp:
(WebCore::isAnonymousRubyInlineBlock):
- rendering/RenderRuby.h: add support for is<RenderRubyAsBlock>(renderer) and for is<RenderRubyAsInline>(renderer)
The isRuby() method does not follow the is*() -> maps to one class pattern.
(WebCore::isRuby):
- rendering/RenderRubyRun.cpp:
(WebCore::RenderRubyRun::addChild):
(WebCore::RenderRubyRun::staticCreateRubyRun):
LayoutTests:
- fast/ruby/crash-when-ruby-is-set-to-inline-block-expected.txt: Added.
- fast/ruby/crash-when-ruby-is-set-to-inline-block.html: Added.
- 1:43 AM Changeset in webkit [180037] by
-
- 3 edits in trunk/Source/bmalloc
[BMalloc] Add a FIXME comment for memory alignas
https://bugs.webkit.org/show_bug.cgi?id=141556
Reviewed by Csaba Osztrogonác.
- bmalloc/Chunk.h: Add a FIXME comment.
- bmalloc/LargeChunk.h: ditto.
- 1:19 AM Changeset in webkit [180036] by
-
- 2 edits in trunk/Source/WebCore
Fix the gperf related build issue on the WinCairo bot
https://bugs.webkit.org/show_bug.cgi?id=141507
Reviewed by Alex Christensen.
- platform/ColorData.gperf: Modified property svn:eol-style.
- 1:15 AM Changeset in webkit [180035] by
-
- 6 edits92 deletes in trunk
Unreviewed, remove empty directories.
.:
- ManualTests/qt: Removed.
Source/WebCore:
- loader/icon/wince: Removed.
- platform/graphics/wince: Removed.
- storage/wince: Removed.
Tools:
- DumpRenderTree/gtk: Removed.
- DumpRenderTree/gtk/fonts: Removed.
- DumpRenderTree/unix: Removed.
- DumpRenderTree/unix/TestNetscapePlugin: Removed.
- GtkLauncher: Removed.
- TestWebKitAPI/Tests/WebKitGtk: Removed.
- TestWebKitAPI/Tests/WebKitGtk/resources: Removed.
- WebKitTestRunner/WebKitTestRunnerAppTests: Removed.
- WebKitTestRunner/WebKitTestRunnerAppTests/en.lproj: Removed.
Websites/perf.webkit.org:
- public/data: Removed.
Websites/webkit.org:
- blog/wp-includes/js/codepress: Removed.
LayoutTests:
- fast/forms/date-multiple-fields: Removed.
- fast/forms/datetimelocal-multiple-fields: Removed.
- fast/forms/month-multiple-fields: Removed.
- fast/forms/time-multiple-fields: Removed.
- fast/forms/week-multiple-fields: Removed.
- fast/js: Removed.
- http/tests/inspector-enabled: Removed.
- http/tests/inspector-enabled/resources: Removed.
- http/tests/inspector/appcache: Removed.
- http/tests/inspector/appcache/resources: Removed.
- http/tests/inspector/audits: Removed.
- http/tests/inspector/audits/resources: Removed.
- http/tests/inspector/indexeddb: Removed.
- http/tests/inspector/indexeddb/resources: Removed.
- http/tests/inspector/network: Removed.
- http/tests/inspector/network/resources: Removed.
- http/tests/inspector/resource-tree: Removed.
- http/tests/inspector/resource-tree/resources: Removed.
- http/tests/inspector/resources: Removed.
- http/tests/inspector/search: Removed.
- http/tests/inspector/search/resources: Removed.
- http/tests/inspector/stacktraces: Removed.
- http/tests/inspector/stacktraces/resources: Removed.
- http/tests/inspector/styles: Removed.
- http/tests/inspector/styles/resources: Removed.
- http/tests/inspector/websocket: Removed.
- http/tests/webgl/1.0.1: Removed.
- inspector-protocol/heap-profiler: Removed.
- inspector-protocol/heap-profiler/resources: Removed.
- inspector/console: Removed.
- inspector/console/resources: Removed.
- inspector/elements: Removed.
- inspector/elements/resources: Removed.
- inspector/profiler: Removed.
- inspector/runtime: Removed.
- inspector/styles: Removed.
- inspector/styles/resources: Removed.
- js/regress/resources: Removed.
- media/track/opera/track/webvtt/rendering/reftest/media: Removed.
- networkinformation: Removed.
- networkinformation/resources: Removed.
- networkinformation/script-tests: Removed.
- platform/efl-wk2/dom: Removed.
- platform/efl-wk2/dom/xhtml: Removed.
- platform/efl-wk2/inspector: Removed.
- platform/efl-wk2/tables: Removed.
- platform/efl-wk2/tables/mozilla: Removed.
- platform/gtk-wk1: Removed.
- platform/gtk-wk1/fast: Removed.
- platform/gtk-wk1/fast/css: Removed.
- platform/gtk-wk1/fast/loader: Removed.
- platform/gtk-wk1/fast/media: Removed.
- platform/gtk-wk1/fast/writing-mode: Removed.
- platform/gtk-wk1/http: Removed.
- platform/gtk-wk1/http/tests: Removed.
- platform/gtk-wk1/http/tests/cache: Removed.
- platform/gtk-wk1/http/tests/security: Removed.
- platform/gtk-wk1/http/tests/security/contentSecurityPolicy: Removed.
- platform/gtk-wk1/js: Removed.
- platform/gtk-wk1/js/dom: Removed.
- platform/gtk-wk1/loader: Removed.
- platform/gtk-wk1/scrollingcoordinator: Removed.
- platform/gtk-wk2/accessibility: Removed.
- platform/gtk-wk2/fast: Removed.
- platform/gtk-wk2/fast/events: Removed.
- platform/gtk-wk2/http: Removed.
- platform/gtk-wk2/http/tests: Removed.
- platform/gtk-wk2/http/tests/loading: Removed.
- platform/gtk-wk2/http/tests/misc: Removed.
- platform/gtk-wk2/http/tests/misc/authentication-redirect-1: Removed.
- platform/gtk-wk2/http/tests/misc/authentication-redirect-2: Removed.
- platform/gtk-wk2/http/tests/misc/authentication-redirect-4: Removed.
- platform/gtk-wk2/http/tests/security: Removed.
- platform/gtk-wk2/http/tests/security/401-logout: Removed.
- platform/gtk-wk2/http/tests/xmlhttprequest: Removed.
- platform/gtk-wk2/js: Removed.
- platform/gtk-wk2/js/dom: Removed.
- platform/gtk-wk2/plugins: Removed.
- platform/gtk-wk2/plugins/npruntime: Removed.
- platform/gtk/platform: Removed.
- platform/ios-simulator/editing/editability: Removed.
- platform/ios-simulator/editing/input: Removed.
- platform/ios-simulator/editing/secure-input: Removed.
- platform/ios-simulator/editing/shadow: Removed.
- platform/ios-simulator/editing/spelling: Removed.
- platform/ios-simulator/editing/text-iterator: Removed.
- platform/ios-simulator/editing/undo: Removed.
- platform/ios-simulator/editing/unsupported-content: Removed.
- platform/ios-simulator/fast/canvas/webgl: Removed.
- platform/ios-simulator/fast/css-grid-layout: Removed.
- platform/ios-simulator/fast/css/content: Removed.
- platform/ios-simulator/fast/css/image-orientation: Removed.
- platform/ios-simulator/fast/css/image-resolution: Removed.
- platform/ios-simulator/fast/css/style-scoped: Removed.
- platform/ios-simulator/fast/css3-text/css3-text-indent: Removed.
- platform/ios-simulator/fast/css3-text/css3-text-indent/getComputedStyle: Removed.
- platform/ios-simulator/fast/dom/Document: Removed.
- platform/ios-simulator/fast/dom/Document/CaretRangeFromPoint: Removed.
- platform/ios-simulator/fast/dom/Geolocation: Removed.
- platform/ios-simulator/fast/dom/HTMLAnchorElement: Removed.
- platform/ios-simulator/fast/dom/HTMLImageElement/sizes: Removed.
- platform/ios-simulator/fast/dom/MutationObserver: Removed.
- platform/ios-simulator/fast/dom/NavigatorContentUtils: Removed.
- platform/ios-simulator/fast/dom/Orientation: Removed.
- platform/ios-simulator/fast/dom/custom: Removed.
- platform/ios-simulator/fast/events/constructors: Removed.
- platform/ios-simulator/fast/events/scoped: Removed.
- platform/iphone-simulator: Removed.
- platform/iphone-simulator/fast: Removed.
- platform/iphone-simulator/fast/events: Removed.
- platform/mac-mavericks/fast/borders: Removed.
- platform/mac-wk2/fast/writing-mode: Removed.
- platform/mac-wk2/http/tests/plugins: Removed.
- platform/mac-wk2/js: Removed.
- platform/mac-wk2/js/dom: Removed.
- platform/mac-wk2/tiled-drawing/fixed: Removed.
- platform/mac-wk2/tiled-drawing/fixed-background: Removed.
- platform/mac-wk2/tiled-drawing/sticky: Removed.
- platform/mac/dom: Removed.
- platform/mac/dom/xhtml: Removed.
- platform/mac/fast/backgrounds/resources: Removed.
- platform/mac/media/W3C/audio: Removed.
- platform/mac/media/W3C/audio/canPlayType: Removed.
- platform/mac/platform/mac-wk2: Removed.
- platform/mac/platform/mac/editing/spelling: Removed.
- platform/mac/platform/mac/fast/forms: Removed.
- platform/win-wk2: Removed.
- platform/win-wk2/fast: Removed.
- platform/win-wk2/http: Removed.
- platform/win-wk2/platform: Removed.
- platform/win-wk2/svg: Removed.
- platform/win/canvas: Removed.
- platform/win/fast/xmlhttprequest: Removed.
- platform/win/loader: Removed.
- platform/wk2/tables: Removed.
- 12:48 AM Changeset in webkit [180034] by
-
- 2 edits in trunk/Tools
Remove Windows cruft from Tools/Scripts/build-webkittestrunner
https://bugs.webkit.org/show_bug.cgi?id=141519
Reviewed by Alexey Proskuryakov.
- Scripts/build-webkittestrunner:
Feb 12, 2015:
- 11:15 PM Changeset in webkit [180033] by
-
- 2 edits in branches/safari-600.5-branch/Source/WebCore
Merged r180028. rdar://problem/19180955
- 11:14 PM Changeset in webkit [180032] by
-
- 2 edits in branches/safari-600.4-branch/Source/WebCore
Merged r180028. rdar://problem/19180955
- 10:48 PM Changeset in webkit [180031] by
-
- 5 edits in trunk/Source/WebKit2
Crash removing RemoteLayerTreeDisplayRefreshMonitor from the drawing area's map
https://bugs.webkit.org/show_bug.cgi?id=141545
<rdar://problem/17712200>
Reviewed by Simon Fraser.
- WebProcess/WebPage/WebPage.cpp:
(WebKit::WebPage::windowScreenDidChange):
- WebProcess/WebPage/WebPage.h:
- WebProcess/WebPage/WebPage.messages.in:
windowScreenDidChange should take uint32_t, since that's what PlatformDisplayID is.
- WebProcess/WebPage/mac/RemoteLayerTreeDrawingArea.mm:
(WebKit::RemoteLayerTreeDrawingArea::RemoteLayerTreeDrawingArea):
RemoteLayerTreeDisplayRefreshMonitor is driven by the page's
RemoteLayerTreeDrawingArea. Because of this, we cannot have the same
sharing of DisplayRefreshMonitors that other platforms use.
In order to ensure that we get a unique DisplayRefreshMonitor
per-DrawingArea, give each page a unique DisplayID derived
from WebPage's unique ID.
- 10:03 PM Changeset in webkit [180030] by
-
- 2 edits in trunk/LayoutTests
[Win] Mark some debug assertions.
- platform/win/TestExpectations: Mark several debug assertions so
we don't end testing early.
- 9:53 PM Changeset in webkit [180029] by
-
- 2 edits2 adds in trunk/LayoutTests
[Win] More media rebaselining.
- platform/win/TestExpectations:
- platform/win/fast/css/focus-ring-exists-for-search-field-expected.png: Added.
- platform/win/fast/css/focus-ring-exists-for-search-field-expected.txt: Added.
- 7:59 PM Changeset in webkit [180028] by
-
- 2 edits in trunk/Source/WebCore
Crashes under detectItemAroundHitTestResult when DataDetectors is not available
https://bugs.webkit.org/show_bug.cgi?id=141549
<rdar://problem/19180955>
Reviewed by Dan Bernstein.
- editing/mac/DataDetection.mm:
(WebCore::DataDetection::detectItemAroundHitTestResult):
Bail out from data detection if either of the relevant frameworks aren't loaded.
- 6:30 PM Changeset in webkit [180027] by
-
- 31 edits3 adds in trunk/LayoutTests
[EFL] Need to have new baseline for some tests of fast/ruby since r172874
https://bugs.webkit.org/show_bug.cgi?id=141462
Unreviewed, EFL gardening. Rebaseline after r172874
Patch by Hunseop Jeong <Hunseop Jeong> on 2015-02-12
- platform/efl/TestExpectations:
- platform/efl/fast/ruby/bopomofo-expected.png: Added.
- platform/efl/fast/ruby/bopomofo-expected.txt:
- platform/efl/fast/ruby/bopomofo-letter-spacing-expected.png: Added.
- platform/efl/fast/ruby/bopomofo-letter-spacing-expected.txt:
- platform/efl/fast/ruby/bopomofo-rl-expected.png: Added.
- platform/efl/fast/ruby/bopomofo-rl-expected.txt:
- platform/efl/fast/ruby/ruby-beforeafter-expected.png:
- platform/efl/fast/ruby/ruby-beforeafter-expected.txt:
- platform/efl/fast/ruby/ruby-length-expected.png:
- platform/efl/fast/ruby/ruby-length-expected.txt:
- platform/efl/fast/ruby/ruby-run-break-expected.png:
- platform/efl/fast/ruby/ruby-run-break-expected.txt:
- platform/efl/fast/ruby/ruby-runs-expected.png:
- platform/efl/fast/ruby/ruby-runs-expected.txt:
- platform/efl/fast/ruby/ruby-runs-spans-expected.png:
- platform/efl/fast/ruby/ruby-runs-spans-expected.txt:
- platform/efl/fast/ruby/ruby-trailing-expected.png:
- platform/efl/fast/ruby/ruby-trailing-expected.txt:
- platform/efl/fast/ruby/rubyDOM-insert-rt-expected.png:
- platform/efl/fast/ruby/rubyDOM-insert-rt-expected.txt:
- platform/efl/fast/ruby/rubyDOM-insert-text1-expected.png:
- platform/efl/fast/ruby/rubyDOM-insert-text1-expected.txt:
- platform/efl/fast/ruby/rubyDOM-insert-text2-expected.png:
- platform/efl/fast/ruby/rubyDOM-insert-text2-expected.txt:
- platform/efl/fast/ruby/rubyDOM-insert-text3-expected.png:
- platform/efl/fast/ruby/rubyDOM-insert-text3-expected.txt:
- platform/efl/fast/ruby/rubyDOM-remove-rt1-expected.png:
- platform/efl/fast/ruby/rubyDOM-remove-rt1-expected.txt:
- platform/efl/fast/ruby/rubyDOM-remove-rt2-expected.png:
- platform/efl/fast/ruby/rubyDOM-remove-rt2-expected.txt:
- platform/efl/fast/ruby/rubyDOM-remove-text1-expected.png:
- platform/efl/fast/ruby/rubyDOM-remove-text1-expected.txt:
- 5:23 PM Changeset in webkit [180026] by
-
- 5 edits in branches/safari-600.5-branch/Source
Versioning.
- 5:21 PM Changeset in webkit [180025] by
-
- 1 copy in tags/Safari-600.5.6
New tag.
- 5:04 PM Changeset in webkit [180024] by
-
- 3 edits in trunk/Source/WebCore
[iOS Media] Audio should hide timeline scrubber until playback starts
https://bugs.webkit.org/show_bug.cgi?id=141542
<rdar://problem/19820685>
Reviewed by Eric Carlson.
We regressed from the system behaviour in iOS 7, where the timeline
scrubber for an audio element is not shown until the user starts
playback.
- Modules/mediacontrols/mediaControlsiOS.css:
(video::-webkit-media-controls-panel): Fix the alignment of the flexbox.
(audio::-webkit-media-controls-status-display): Add the style for the status display.
- Modules/mediacontrols/mediaControlsiOS.js:
(ControllerIOS.prototype.configureInlineControls): Start hidden if we are an <audio>.
(ControllerIOS.prototype.configureFullScreenControls): Drive by comment change.
(ControllerIOS.prototype.setPlaying): Unhide the timeline.
- 5:00 PM Changeset in webkit [180023] by
-
- 2 edits in trunk/LayoutTests
[Win] Unreviewed updates to test expectations to reduce failures on Windows bots.
- platform/win/TestExpectations: Mark a number of time-outs for Media tests.
- 4:49 PM Changeset in webkit [180022] by
-
- 2 edits in trunk/Source/WebKit2
Crash under RemoteLayerTreeDrawingArea::flushLayers() when closing a tab
https://bugs.webkit.org/show_bug.cgi?id=141541
Reviewed by Benjamin Poulain.
- WebProcess/WebPage/mac/RemoteLayerTreeDrawingArea.mm:
(WebKit::RemoteLayerTreeDrawingArea::flushLayers):
The WebPage pointer can become stale between when it is retrieved
and dereferenced. Retrieve it by ID inside the dispatch_async block
instead of outside of it.
- 4:13 PM Changeset in webkit [180021] by
-
- 6 edits in trunk/LayoutTests
Re-baselines after http://trac.webkit.org/changeset/180018
- platform/mac-wk2/tiled-drawing/scrolling/fast-scroll-div-latched-div-with-handler-expected.txt:
- platform/mac-wk2/tiled-drawing/scrolling/fast-scroll-div-latched-mainframe-with-handler-expected.txt:
- platform/mac-wk2/tiled-drawing/scrolling/fast-scroll-iframe-latched-mainframe-with-handler-expected.txt:
- platform/mac-wk2/tiled-drawing/scrolling/fast-scroll-select-latched-mainframe-with-handler-expected.txt:
- platform/mac-wk2/tiled-drawing/scrolling/fast-scroll-select-latched-select-with-handler-expected.txt:
- 3:23 PM Changeset in webkit [180020] by
-
- 11 edits in trunk
Drop the quirks-mode exception for CSS MIME types
https://bugs.webkit.org/show_bug.cgi?id=141501
Reviewed by Alexey Proskuryakov.
Source/WebCore:
Drop the quirks-mode exception for CSS MIME types.
This matches Firefox's behavior.
This patch is based on the following Blink revision:
http://src.chromium.org/viewvc/blink?view=revision&revision=189669
Tests:
- http/tests/misc/css-accept-any-type.html
- http/tests/security/cross-origin-css.html
- css/StyleSheetContents.cpp:
(WebCore::StyleSheetContents::parseAuthorStyleSheet):
- dom/ProcessingInstruction.cpp:
(WebCore::ProcessingInstruction::setCSSStyleSheet):
- inspector/InspectorPageAgent.cpp:
(WebCore::InspectorPageAgent::cachedResourceContent):
- loader/cache/CachedCSSStyleSheet.cpp:
(WebCore::CachedCSSStyleSheet::sheetText):
(WebCore::CachedCSSStyleSheet::canUseSheet):
- loader/cache/CachedCSSStyleSheet.h:
LayoutTests:
Update a couple of layout tests now that we dropped the quirks-mode
exception for CSS MIME types.
- http/tests/misc/css-accept-any-type-expected.txt:
- http/tests/misc/css-accept-any-type.html:
- http/tests/security/cross-origin-css-expected.txt:
- http/tests/security/cross-origin-css.html:
- 3:10 PM Changeset in webkit [180019] by
-
- 2 edits in branches/safari-600.5-branch/Tools
Merge r179996 to correct test system failures.
2015-02-12 Brent Fulgham <Brent Fulgham>
[Win] Test bot infrastructure should not try to find/use old Safari DLLs
https://bugs.webkit.org/show_bug.cgi?id=141517
Reviewed by Anders Carlsson.
We do not need to locate or use the DLL's that came with Safari, so stop
failing when we cannot find them. Also remove some old Qt cruft from this
script.
- BuildSlaveSupport/built-product-archive: (archiveBuiltProduct): (extractBuiltProduct):
- 3:09 PM Changeset in webkit [180018] by
-
- 3 edits2 adds in trunk
REGRESSION: Should not send wheel events with a delta of 0
https://bugs.webkit.org/show_bug.cgi?id=141537
-and corresponding-
rdar://problem/18903118
Reviewed by Simon Fraser.
Source/WebCore:
This code should not have been removed entirely to accommodate rubber-banding, it
just needs to return false instead of true in order to allow the MayBegin and
Ended phases to be handled by the ElasticityController.
- dom/Element.cpp:
(WebCore::Element::dispatchWheelEvent):
LayoutTests:
- platform/mac-wk2/tiled-drawing/scrolling/wheel-events-with-no-delta-expected.txt: Added.
- platform/mac-wk2/tiled-drawing/scrolling/wheel-events-with-no-delta.html: Added.
- 3:08 PM Changeset in webkit [180017] by
-
- 5 edits in branches/safari-600.1.17-branch/Source
Versioning.
- 3:07 PM Changeset in webkit [180016] by
-
- 1 copy in tags/Safari-600.1.17.9
New Tag.
- 3:07 PM Changeset in webkit [180015] by
-
- 4 edits in branches/safari-600.5-branch
[Win] Unreviewed branch build fix.
Source/WebKit:
- WebKit.vcxproj/WebKitExportGenerator/WebKitExports.def.in: Fix
symbol export definition.
Tools:
- DumpRenderTree/AccessibilityUIElement.cpp:
(AccessibilityUIElement::setValue): This needs a default body.
- 2:24 PM Changeset in webkit [180014] by
-
- 1 edit1 add in branches/safari-600.5-branch/Source/JavaScriptCore
Correct merge error from r180011.
- JavaScriptCore.vcxproj/JavaScriptCoreDLL.cpp: Added.
(DllMain):
- 2:20 PM Changeset in webkit [180013] by
-
- 4 edits in trunk/Source/WebCore
[iOS Media] Implement 3-style media timeline (buffered, played, unavailable)
https://bugs.webkit.org/show_bug.cgi?id=141526
<rdar://problem/19603337>
Reviewed by Eric Carlson and Antoine Quint.
Implement a 3-style media scrubber for iOS. This draws white for
the region that has been played (before the playhead), black
for the region that is buffered, and is hollow for the rest.
- Modules/mediacontrols/mediaControlsApple.js:
(Controller.prototype.updateProgress): Update progress is now completely
shared between OS X and iOS. This calls the implementation specific
drawTimelineBackground.
(Controller.prototype.drawTimelineBackground): Move the OS X code to here.
- Modules/mediacontrols/mediaControlsiOS.css:
(audio::-webkit-media-controls-timeline): New styles for the timeline. Updated thumb image, etc.
(video::-webkit-media-controls-timeline::-webkit-slider-runnable-track):
(audio::-webkit-media-controls-timeline::-webkit-slider-thumb):
(audio::-webkit-media-controls-timeline::-webkit-slider-thumb:active):
(video::-webkit-media-controls-time-remaining-display):
- Modules/mediacontrols/mediaControlsiOS.js:
(ControllerIOS): Create a globally unique canvas context name in the constructor.
(ControllerIOS.prototype.createControls): Set the background of the timeline to a canvas.
(ControllerIOS.prototype.addRoundedRect): Helper function.
(ControllerIOS.prototype.drawTimelineBackground): Draw the 3-style content.
(ControllerIOS.prototype.updateProgress): Deleted.
- 2:20 PM Changeset in webkit [180012] by
-
- 5 edits in trunk/Source/JavaScriptCore
Simplify the initialization of BytecodeGenerator a bit
https://bugs.webkit.org/show_bug.cgi?id=141505
Reviewed by Anders Carlsson.
- bytecompiler/BytecodeGenerator.cpp:
(JSC::BytecodeGenerator::BytecodeGenerator):
- bytecompiler/BytecodeGenerator.h:
Setup the default initialization at the declaration level
instead of the constructor.
Also made m_scopeNode and m_codeType const to make it explicit
that they are invariant after construction.
- parser/Nodes.cpp:
- runtime/Executable.cpp:
Remove 2 useless #includes.
- 2:19 PM Changeset in webkit [180011] by
-
- 18 edits1 add in branches/safari-600.5-branch
Merge r179993. <rdar://problem/19813975>
2015-02-11 Brent Fulgham <Brent Fulgham>
[Win] [64-bit] Work around MSVC2013 Runtime Bug
https://bugs.webkit.org/show_bug.cgi?id=141498
<rdar://problem/19803642>
Reviewed by Anders Carlsson.
Disable FMA3 instruction use in the MSVC math library to
work around a VS2013 runtime crash. We can remove this
workaround when we switch to VS2015.
Source/JavaScriptCore:
- API/tests/testapi.c: Call _set_FMA3_enable(0) to disable FMA3 support.
- JavaScriptCore.vcxproj/JavaScriptCore.vcxproj: Add new files.
- JavaScriptCore.vcxproj/JavaScriptCore.vcxproj.filters: Ditto.
- JavaScriptCore.vcxproj/JavaScriptCoreDLL.cpp: Added.
- JavaScriptCore.vcxproj/jsc/DLLLauncherMain.cpp: Call _set_FMA3_enable(0) to disable FMA3 support.
- jsc.cpp: Ditto.
- testRegExp.cpp: Ditto.
Source/WebKit/win:
- WebKitDLL.cpp: Call _set_FMA3_enable(0) to disable FMA3 support.
Source/WTF:
- WTF.vcxproj/WTF.vcxproj: Add new files.
- WTF.vcxproj/WTF.vcxproj.filters: Ditto.
- wtf/win/WTFDLL.cpp: Added.
Tools:
- DumpRenderTree/cg/ImageDiffCG.cpp: Call _set_FMA3_enable(0) to disable FMA3 support.
- DumpRenderTree/win/DumpRenderTree.cpp: Ditto.
- DumpRenderTree/win/ImageDiffCairo.cpp: Ditto.
- DumpRenderTree/win/TestRunnerWin.cpp: Ditto.
- Scripts/webkitperl/httpd.pm: Ditto.
- TestWebKitAPI/win/main.cpp: Ditto.
- WinLauncher/Common.cpp: Ditto.
- win/DLLLauncher/DLLLauncherMain.cpp: Ditto.
- 2:18 PM Changeset in webkit [180010] by
-
- 5 edits in trunk/Source/JavaScriptCore
Move the generators for GetScope and SkipScope to the common core in DFGSpeculativeJIT
https://bugs.webkit.org/show_bug.cgi?id=141506
Reviewed by Michael Saboff.
The generators for the nodes GetScope and SkipScope were
completely identical between 32 and 64bits.
This patch moves the duplicated code to DFGSpeculativeJIT.
- dfg/DFGSpeculativeJIT.cpp:
(JSC::DFG::SpeculativeJIT::compileGetScope):
(JSC::DFG::SpeculativeJIT::compileSkipScope):
- dfg/DFGSpeculativeJIT.h:
- dfg/DFGSpeculativeJIT32_64.cpp:
(JSC::DFG::SpeculativeJIT::compile):
- dfg/DFGSpeculativeJIT64.cpp:
(JSC::DFG::SpeculativeJIT::compile):
- 1:33 PM Changeset in webkit [180009] by
-
- 2 edits in branches/safari-600.1.17-branch/Tools
Merge r179996.
2015-02-12 Brent Fulgham <Brent Fulgham>
[Win] Test bot infrastructure should not try to find/use old Safari DLLs
https://bugs.webkit.org/show_bug.cgi?id=141517
Reviewed by Anders Carlsson.
We do not need to locate or use the DLL's that came with Safari, so stop
failing when we cannot find them. Also remove some old Qt cruft from this
script.
- BuildSlaveSupport/built-product-archive: (archiveBuiltProduct): (extractBuiltProduct):
- 1:31 PM Changeset in webkit [180008] by
-
- 3 edits in trunk/Source/WebKit2
WebKit persistent cache uses a lot of threads
https://bugs.webkit.org/show_bug.cgi?id=141520
Reviewed by Andreas Kling.
Parallel retrieves are limited but stores are not. We may end up with lots of backround io
dispatch queue threads if they don't complete fast enough.
This patch adds pending state for write operations similar to what retrieves already have
and limits to number of active operations.
- NetworkProcess/cache/NetworkCacheStorage.h:
Combine StoreOperation and UpdateOperation and rename to WriteOperation.
Rename RetrieveOperation to ReadOperation.
The idea is to emphasize that these are disk operations.
- NetworkProcess/cache/NetworkCacheStorageCocoa.mm:
(WebKit::NetworkCacheStorage::dispatchReadOperation):
(WebKit::NetworkCacheStorage::dispatchPendingReadOperations):
(WebKit::retrieveFromMemory):
(WebKit::NetworkCacheStorage::retrieve):
(WebKit::NetworkCacheStorage::store):
(WebKit::NetworkCacheStorage::update):
(WebKit::NetworkCacheStorage::dispatchPendingWriteOperations):
Only allow 3 parallel writes.
(WebKit::NetworkCacheStorage::dispatchFullWriteOperation):
(WebKit::NetworkCacheStorage::dispatchHeaderWriteOperation):
More informative names.
(WebKit::NetworkCacheStorage::dispatchRetrieveOperation): Deleted.
(WebKit::NetworkCacheStorage::dispatchPendingRetrieveOperations): Deleted.
(WebKit::retrieveActive): Deleted.
- 1:18 PM Changeset in webkit [180007] by
-
- 4 edits in trunk/Source/WebCore
Fix some Mac linker warnings
https://bugs.webkit.org/show_bug.cgi?id=141522
Patch by Daniel Bates <dabates@apple.com> on 2015-02-12
Reviewed by Alex Christensen.
Following <https://trac.webkit.org/changeset/179945>, the linker warns about the following hidden WebCore symbols:
ld: warning: cannot export hidden symbol ZN7WebCore20LogNotYetImplementedE from /.../OpenSource/WebKitBuild/WebCore.build/Debug/WebCore.build/Objects-normal/x86_64/Logging.o
ld: warning: cannot export hidden symbol _wkCreateURLPasteboardFlavorTypeName from /.../OpenSource/WebKitBuild/WebCore.build/Debug/WebCore.build/Objects-normal/x86_64/WebCoreSystemInterface.o
ld: warning: cannot export hidden symbol _wkCreateURLNPasteboardFlavorTypeName from /.../OpenSource/WebKitBuild/WebCore.build/Debug/WebCore.build/Objects-normal/x86_64/WebCoreSystemInterface.o
- WebCore.exp.in: Remove symbols for functions wkCreateURLPasteboardFlavorTypeName and wkCreateURLNPasteboardFlavorTypeName.
- platform/Logging.h: Export WTFLogChannel for LogNotYetImplemented, which is referenced from
notImplementedLoggingChannel() in file NotImplemented.cpp. We make use of the associated
header, NotImplemented.h, in both WebKit and WebKit2.
- platform/mac/WebCoreSystemInterface.mm: Remove declarations for functions
wkCreateURLPasteboardFlavorTypeName and wkCreateURLNPasteboardFlavorTypeName that
were inadvertently not removed in r25494.
- 1:13 PM Changeset in webkit [180006] by
-
- 18 edits2 adds in branches/safari-600.1.17-branch
Merge r179993. <rdar://problem/19803642>
2015-02-11 Brent Fulgham <Brent Fulgham>
[Win] [64-bit] Work around MSVC2013 Runtime Bug
https://bugs.webkit.org/show_bug.cgi?id=141498
<rdar://problem/19803642>
Reviewed by Anders Carlsson.
Disable FMA3 instruction use in the MSVC math library to
work around a VS2013 runtime crash. We can remove this
workaround when we switch to VS2015.
Source/JavaScriptCore:
- API/tests/testapi.c: Call _set_FMA3_enable(0) to disable FMA3 support.
- JavaScriptCore.vcxproj/JavaScriptCore.vcxproj: Add new files.
- JavaScriptCore.vcxproj/JavaScriptCore.vcxproj.filters: Ditto.
- JavaScriptCore.vcxproj/JavaScriptCoreDLL.cpp: Added.
- JavaScriptCore.vcxproj/jsc/DLLLauncherMain.cpp: Call _set_FMA3_enable(0) to disable FMA3 support.
- jsc.cpp: Ditto.
- testRegExp.cpp: Ditto.
Source/WebKit/win:
- WebKitDLL.cpp: Call _set_FMA3_enable(0) to disable FMA3 support.
Source/WTF:
- WTF.vcxproj/WTF.vcxproj: Add new files.
- WTF.vcxproj/WTF.vcxproj.filters: Ditto.
- wtf/win/WTFDLL.cpp: Added.
Tools:
- DumpRenderTree/cg/ImageDiffCG.cpp: Call _set_FMA3_enable(0) to disable FMA3 support.
- DumpRenderTree/win/DumpRenderTree.cpp: Ditto.
- DumpRenderTree/win/ImageDiffCairo.cpp: Ditto.
- DumpRenderTree/win/TestRunnerWin.cpp: Ditto.
- TestWebKitAPI/win/main.cpp: Ditto.
- WinLauncher/Common.cpp: Ditto.
- win/DLLLauncher/DLLLauncherMain.cpp: Ditto.
- 12:47 PM Changeset in webkit [180005] by
-
- 6 edits2 adds in trunk
Web Inspector: Crash inspecting styles of element with mutated stylesheet
https://bugs.webkit.org/show_bug.cgi?id=141451
Patch by Joseph Pecoraro <Joseph Pecoraro> on 2015-02-12
Reviewed by Timothy Hatcher.
Source/WebCore:
Test: inspector/css/stylesheet-with-mutations.html
- css/CSSStyleSheet.h:
- css/CSSStyleSheet.cpp:
(WebCore::CSSStyleSheet::CSSStyleSheet):
(WebCore::CSSStyleSheet::didMutateRules):
Keep a flag on the CSSStyleSheet to know when it has had a rule
mutation from the contents.
- inspector/InspectorStyleSheet.h:
- inspector/InspectorStyleSheet.cpp:
(WebCore::InspectorStyleSheet::reparseStyleSheet):
When completely replacing the stylesheet's contents, we will have
destroyed any previous rules, so clear the has mutations flag.
(WebCore::selectorsFromSource):
Previously there was a mismatch between the rules the parsed
SourceData had, and the actual CSSSelectors we were iterating.
We use the SourceData so we can get the exact user authored
text instead of generated longhands from actualy selector objects.
Add an ASSERT and bail to catch and more gracefully handle
any possible mismatches in the future.
(WebCore::InspectorStyleSheet::styleSheetMutated):
(WebCore::InspectorStyleSheet::ensureParsedDataReady):
When a CSSStyleSheet has been mutated beyond the inspector's
knowledge right now, fall back to readonly generated selectors.
We should better handle this in the future:
<https://webkit.org/b/141450> Web Inspector: Better support for CSSOM StyleSheet mutations (insertRule/deleteRule)
LayoutTests:
- inspector/css/stylesheet-with-mutations-expected.txt: Added.
- inspector/css/stylesheet-with-mutations.html: Added.
Previously this test would crash, or in debug builds show one
of the two selectors in the rule.
- 12:30 PM Changeset in webkit [180004] by
-
- 3 edits in trunk/Source/WebInspectorUI
Web Inspector: add keyboard shortcut to enable/disable all breakpoints
https://bugs.webkit.org/show_bug.cgi?id=141527
Reviewed by Joseph Pecoraro.
- Localizations/en.lproj/localizedStrings.js:
- UserInterface/Views/DebuggerSidebarPanel.js:
(WebInspector.DebuggerSidebarPanel): Bind Command/Control + Y to toggle breakpoints button.
- 12:00 PM Changeset in webkit [180003] by
-
- 7 edits2 deletes in trunk
Unreviewed, rolling out r179987.
https://bugs.webkit.org/show_bug.cgi?id=141525
caused debug tests to fail (Requested by alexchristensen on
#webkit).
Reverted changeset:
"[CSS Grid Layout] Invalid initialization of track sizes with
non spanning grid items"
https://bugs.webkit.org/show_bug.cgi?id=140763
http://trac.webkit.org/changeset/179987
- 11:57 AM Changeset in webkit [180002] by
-
- 1 edit1 delete in trunk/Tools
Removed empty directory.
- WinCELauncher: Removed.
- 11:00 AM Changeset in webkit [180001] by
-
- 17 edits2 moves in trunk/Source/WebInspectorUI
Web Inspector: support adding and removing timelines to the timeline sidebar panel and overview
https://bugs.webkit.org/show_bug.cgi?id=138434
Reviewed by Timothy Hatcher.
A timeline recording's timelines should not be static. This patch adds relevant machinery to
dynamically add and remove timelines from the Timeline model object and its various views.
From the model side, this is a simple change that adds TimelineAdded and TimelineRemoved events.
The timeline views and timeline sidebar require more extensive changes to support this functionality.
Instead of keeping a fixed set of timeline tree elements, the sidebar view now adds and removes
tree elements to reflect the available timelines for the displayed timeline recording.
This change also includes several minor cleanups, such as appending 'Element' to view properties that
are DOM elements, and renaming TimelineContentView to TimelineRecordingContentView.
- UserInterface/Controllers/TimelineManager.js:
(WebInspector.TimelineManager.prototype._loadNewRecording):
Populate predefined network, layout, and script timelines here after constructing the recording.
- UserInterface/Main.html:
- UserInterface/Models/Timeline.js:
Move hardcoded class names and localized strings to these base class methods. This is not implemented
using overridden methods because not all timeline types have their own subclasses. Add a dummy
implementation of saveIdentityToCookie() to avoid warnings when saving sidebar panel selection state.
(WebInspector.Timeline.prototype.get type):
(WebInspector.Timeline.prototype.get displayName): Added.
(WebInspector.Timeline.prototype.get iconClassName): Added.
(WebInspector.Timeline.prototype.addRecord):
- UserInterface/Models/TimelineRecording.js:
(WebInspector.TimelineRecording):
(WebInspector.TimelineRecording.prototype.addTimeline.set timeline): Added.
(WebInspector.TimelineRecording.prototype.addTimeline): Added.
(WebInspector.TimelineRecording.prototype.removeTimeline.get this): Added.
(WebInspector.TimelineRecording.prototype.removeTimeline): Added.
- UserInterface/Views/ContentView.js:
(WebInspector.ContentView):
- UserInterface/Views/LayoutTimelineView.css:
(.sidebar > .panel.navigation.timeline.timeline-recording-content-view-showing .navigation-sidebar-panel-content-tree-outline.layout .item .subtitle):
(.sidebar > .panel.navigation.timeline.timeline-content-view-showing .navigation-sidebar-panel-content-tree-outline.layout .item .subtitle): Deleted.
- UserInterface/Views/LayoutTimelineView.js:
(WebInspector.LayoutTimelineView.prototype._treeElementSelected):
(WebInspector.LayoutTimelineView.prototype._closeStatusButtonClicked):
- UserInterface/Views/NavigationSidebarPanel.js:
(WebInspector.NavigationSidebarPanel.prototype.saveStateToCookie): Fix error message formatting.
(WebInspector.NavigationSidebarPanel.prototype._updateFilter):
- UserInterface/Views/NetworkTimelineView.css:
(.sidebar > .panel.navigation.timeline.timeline-recording-content-view-showing .navigation-sidebar-panel-content-tree-outline.network .item .subtitle):
(.sidebar > .panel.navigation.timeline.timeline-content-view-showing .navigation-sidebar-panel-content-tree-outline.network .item .subtitle): Deleted.
- UserInterface/Views/NetworkTimelineView.js:
(WebInspector.NetworkTimelineView.prototype._closeStatusButtonClicked):
- UserInterface/Views/ScriptTimelineView.css:
(.sidebar > .panel.navigation.timeline.timeline-recording-content-view-showing .navigation-sidebar-panel-content-tree-outline.script .item .subtitle):
(.sidebar > .panel.navigation.timeline.timeline-content-view-showing .navigation-sidebar-panel-content-tree-outline.script .item .subtitle): Deleted.
- UserInterface/Views/ScriptTimelineView.js:
(WebInspector.ScriptTimelineView.prototype._treeElementSelected):
(WebInspector.ScriptTimelineView.prototype._closeStatusButtonClicked):
- UserInterface/Views/TimelineOverview.js:
Convert this class to use the representedObject class pattern. It manages its own mapping from timelines
to overview graph views. Append 'Element' to some properties holding DOM elements.
(WebInspector.TimelineOverview):
(WebInspector.TimelineOverview.prototype.get visibleDuration):
(WebInspector.TimelineOverview.prototype.reset):
(WebInspector.TimelineOverview.prototype.updateLayout):
(WebInspector.TimelineOverview.prototype._handleScrollEvent):
(WebInspector.TimelineOverview.prototype._handleWheelEvent):
(WebInspector.TimelineOverview.prototype._timelineRemoved):
(WebInspector.TimelineOverview.prototype.updateLayoutIfNeeded):
(WebInspector.TimelineOverview.prototype._timeRangeSelectionChanged):
- UserInterface/Views/TimelineRecordingContentView.css: Renamed from Source/WebInspectorUI/UserInterface/Views/TimelineContentView.css.
Remove hard-coded top offsets and heights that need to be dynamically calculated.
(.content-view.timeline-recording > .timeline-overview):
(.content-view.timeline-recording > .view-container):
(.content-view.timeline-recording > .view-container > .timeline-view > .data-grid td):
(.content-view.timeline-recording > .view-container > .timeline-view > .data-grid table.data):
- UserInterface/Views/TimelineRecordingContentView.js: Renamed from Source/WebInspectorUI/UserInterface/Views/TimelineContentView.js.
Manage timeline views and path components dynamically. Adjust view heights as necessary. Append 'Element'
to some properties holding DOM elements.
(WebInspector.TimelineRecordingContentView):
(WebInspector.TimelineRecordingContentView.prototype.showOverviewTimelineView):
(WebInspector.TimelineRecordingContentView.prototype.showTimelineViewForTimeline):
(WebInspector.TimelineRecordingContentView.prototype.get allowedNavigationSidebarPanels):
(WebInspector.TimelineRecordingContentView.prototype.get supportsSplitContentBrowser):
(WebInspector.TimelineRecordingContentView.prototype.get navigationItems):
(WebInspector.TimelineRecordingContentView.prototype.get currentTimelineView):
(WebInspector.TimelineRecordingContentView.prototype.shown):
(WebInspector.TimelineRecordingContentView.prototype.hidden):
(WebInspector.TimelineRecordingContentView.prototype.filterDidChange):
(WebInspector.TimelineRecordingContentView.prototype.updateLayout):
(WebInspector.TimelineRecordingContentView.prototype.saveToCookie):
(WebInspector.TimelineRecordingContentView.prototype.restoreFromCookie):
(WebInspector.TimelineRecordingContentView.prototype.get matchTreeElementAgainstCustomFilters.checkTimeBounds):
(WebInspector.TimelineRecordingContentView.prototype.get matchTreeElementAgainstCustomFilters):
(WebInspector.TimelineRecordingContentView.prototype._pathComponentSelected):
(WebInspector.TimelineRecordingContentView.prototype._timelineViewSelectionPathComponentsDidChange):
(WebInspector.TimelineRecordingContentView.prototype._showTimelineView):
(WebInspector.TimelineRecordingContentView.prototype._update):
(WebInspector.TimelineRecordingContentView.prototype._updateTimes):
(WebInspector.TimelineRecordingContentView.prototype._startUpdatingCurrentTime):
(WebInspector.TimelineRecordingContentView.prototype._stopUpdatingCurrentTime):
(WebInspector.TimelineRecordingContentView.prototype._capturingStarted):
(WebInspector.TimelineRecordingContentView.prototype._capturingStopped):
(WebInspector.TimelineRecordingContentView.prototype._debuggerPaused):
(WebInspector.TimelineRecordingContentView.prototype._debuggerResumed):
Fix a minor regression where we make a useless call to stop updating the time.
(WebInspector.TimelineRecordingContentView.prototype._recordingTimesUpdated):
(WebInspector.TimelineRecordingContentView.prototype._clearTimeline):
(WebInspector.TimelineRecordingContentView.prototype._timelineRemoved):
(WebInspector.TimelineRecordingContentView.prototype._timelineCountChanged):
(WebInspector.TimelineRecordingContentView.prototype._recordingReset):
(WebInspector.TimelineRecordingContentView.prototype._recordingUnloaded):
(WebInspector.TimelineRecordingContentView.prototype._timeRangeSelectionChanged):
- UserInterface/Views/TimelineSidebarPanel.css:
(.sidebar > .panel.navigation.timeline.timeline-recording-content-view-showing > .content):
(.sidebar > .panel.navigation.timeline.timeline-content-view-showing > .content): Deleted.
- UserInterface/Views/TimelineSidebarPanel.js:
Manage timeline tree elements dynamically. Adjust view heights as necessary. Append 'Element'
to some properties holding DOM elements. Keep track of the displayed recording and view explicitly.
(WebInspector.TimelineSidebarPanel):
(WebInspector.TimelineSidebarPanel.prototype.shown):
(WebInspector.TimelineSidebarPanel.prototype.showDefaultContentView):
(WebInspector.TimelineSidebarPanel.prototype.treeElementForRepresentedObject.looselyCompareRepresentedObjects):
(WebInspector.TimelineSidebarPanel.prototype.showTimelineOverview):
(WebInspector.TimelineSidebarPanel.prototype.updateFilter):
(WebInspector.TimelineSidebarPanel.prototype.matchTreeElementAgainstCustomFilters):
(WebInspector.TimelineSidebarPanel.prototype.canShowDifferentContentView):
(WebInspector.TimelineSidebarPanel.prototype.saveStateToCookie):
(WebInspector.TimelineSidebarPanel.prototype.get if):
(WebInspector.TimelineSidebarPanel.prototype._contentBrowserCurrentContentViewDidChange):
(WebInspector.TimelineSidebarPanel.prototype._recordingCountChanged):
(WebInspector.TimelineSidebarPanel.prototype._recordingSelected):
(WebInspector.TimelineSidebarPanel.prototype._recordingLoaded):
(WebInspector.TimelineSidebarPanel.prototype._timelineRemoved):
(WebInspector.TimelineSidebarPanel.prototype._timelineCountChanged):
(WebInspector.TimelineSidebarPanel.createTimelineTreeElement): Deleted.
(WebInspector.TimelineSidebarPanel.prototype.restoreStateFromCookie):
- UserInterface/Views/TimelineView.css:
(.panel.navigation.timeline.timeline-recording-content-view-showing > .content > .navigation-sidebar-panel-content-tree-outline):
(.panel.navigation.timeline.timeline-content-view-showing > .content > .navigation-sidebar-panel-content-tree-outline): Deleted.
- 10:50 AM Changeset in webkit [180000] by
-
- 7 edits in trunk/Websites/perf.webkit.org
Perf dashboard should show the results of A/B testing
https://bugs.webkit.org/show_bug.cgi?id=141500
Reviewed by Chris Dumez.
Added the support for fetching test_runs for a specific test group in /api/runs/, and used it in the
analysis task page to fetch results for each test group.
Merged App.createChartData into App.Manifest.fetchRunsWithPlatformAndMetric so that App.BuildRequest
can use the formatter.
- public/api/runs.php:
(fetch_runs_for_config_and_test_group): Added.
(fetch_runs_for_config): Just return the fetched rows since main will format them with RunsGenerator.
(main): Use fetch_runs_for_config_and_test_group to fetch rows when a test group id is specified. Also
use RunsGenerator to format results.
(RunsGenerator): Added.
(RunsGenerator::construct): Added.
(RunsGenerator::add_runs): Added.
(RunsGenerator::format_run): Moved.
(RunsGenerator::parse_revisions_array): Moved.
- public/v2/analysis.js:
(App.TestGroup): Fixed a typo. The property on a test group that refers to an analysis task is "task".
(App.TestGroup._fetchChartData): Added. Fetches all A/B testing results for this group.
(App.BuildRequest.configLetter): Renamed from config since this returns a letter that identifies the
configuration associated with this build request such as "A" and "B".
(App.BuildRequest.statusLabel): Added the missing label for failed build requests.
(App.BuildRequest.url): Added. Returns the URL associated with this build request.
(App.BuildRequest._meanFetched): Added. Retrieve the mean and the build number for this request via
_fetchChartData.
- public/v2/app.js:
(App.Pane._fetch): Set chartData directly here.
(App.Pane._updateMovingAverageAndEnvelope): Renamed from _computeChartData. No longer sets chartData
now that it's done in App.Pane._fetch.
(App.AnalysisTaskController._fetchedRuns): Updated per createChartData merge.
- public/v2/data.js:
(Measurement.prototype.buildId): Added.
(TimeSeries.prototype.findPointByBuild): Added.
- public/v2/index.html: Fixed a bug that build status URL was broken. We can't use link-to helper since
url is not an Ember routed path.
- public/v2/manifest.js:
(App.Manifest.fetchRunsWithPlatformAndMetric): Takes testGroupId as the third argument. Merged
App.createChartData here so that App.BuildRequest can use the formatter
- 10:42 AM Changeset in webkit [179999] by
-
- 9 edits in trunk/Source/WebInspectorUI
Web Inspector: Large background image fails to load in inspector
https://bugs.webkit.org/show_bug.cgi?id=141405
Reviewed by Timothy Hatcher.
Convert base64-encoded resource data to binary data using Uint8Array and Blobs.
Use object URLs for blobs instead of data URLs when setting image 'src' attribute
and in @font-face urls. This conversion happens when the backend indicates that
resource content is base64-encoded.
Update image and font clients of resource-fetching functionality so that they use
object URLs correctly. Add some plumbing so that Blobs can be constructed with
the correct mime-type.
- UserInterface/Base/Utilities.js:
(decodeBase64ToBlob): Added.
- UserInterface/Models/CSSStyleSheet.js:
(WebInspector.CSSStyleSheet.prototype.get mimeType): Added.
- UserInterface/Models/Resource.js:
(WebInspector.Resource.prototype.createObjectURL): Added.
(WebInspector.Resource.prototype.getImageSize.imageDidLoad):
(WebInspector.Resource.prototype.getImageSize):
(WebInspector.Resource.prototype.get contentURL): Deleted.
- UserInterface/Models/Script.js:
(WebInspector.Script.prototype.get mimeType): Added.
- UserInterface/Models/SourceCode.js:
(WebInspector.SourceCode.prototype.get mimeType): Added.
(WebInspector.SourceCode.prototype._processContent):
(WebInspector.SourceCode.prototype.get contentIsBase64Encoded): Deleted.
- UserInterface/Models/SourceCodeRevision.js:
(WebInspector.SourceCodeRevision):
(WebInspector.SourceCodeRevision.prototype.copy):
(WebInspector.SourceCodeRevision.prototype.get contentIsBase64Encoded): Deleted.
(WebInspector.SourceCodeRevision.prototype.set contentIsBase64Encoded): Deleted.
- UserInterface/Views/FontResourceContentView.js:
(WebInspector.FontResourceContentView.prototype.hidden):
(WebInspector.FontResourceContentView.prototype.closed):
- UserInterface/Views/ImageResourceContentView.js:
(WebInspector.ImageResourceContentView.prototype.contentAvailable):
- 10:35 AM Changeset in webkit [179998] by
-
- 2 edits in trunk/Tools
run-jsc-stress-tests --remote should be able add the necessary libraries to the bundle on Linux
https://bugs.webkit.org/show_bug.cgi?id=141202
Reviewed by Darin Adler.
- Scripts/run-jsc-stress-tests:
- 10:21 AM Changeset in webkit [179997] by
-
- 2 edits in trunk/Source/WebCore
REGRESSION(r178060): empty arguments are passed when logging to system console
https://bugs.webkit.org/show_bug.cgi?id=141511
Reviewed by Timothy Hatcher.
- page/PageConsoleClient.cpp:
(WebCore::PageConsoleClient::messageWithTypeAndLevel): Change premature move to copyRef.
- 9:48 AM Changeset in webkit [179996] by
-
- 2 edits in trunk/Tools
[Win] Test bot infrastructure should not try to find/use old Safari DLLs
https://bugs.webkit.org/show_bug.cgi?id=141517
Reviewed by Anders Carlsson.
We do not need to locate or use the DLL's that came with Safari, so stop
failing when we cannot find them. Also remove some old Qt cruft from this
script.
- BuildSlaveSupport/built-product-archive:
(archiveBuiltProduct):
(extractBuiltProduct):
- 9:44 AM Changeset in webkit [179995] by
-
- 3 edits in branches/safari-600.5-branch/Source/WebCore
<rdar://problem/17889529> CrashTracer: [USER] com.apple.WebKit.WebContent at com.apple.WebCore: WebCore::SharedBuffer::createPurgeableBuffer const + 14
Reviewed by NOBODY (OOPS!).
Speculative fix for the above crash. The current theory is that a timer is being fired for a
CachedResource that has been freed. Add a marker field in CachedResource to do an early
return from CachedResource::makePurgeable() when this happens.
No new tests because I can't reproduce the crash.
- loader/cache/CachedResource.cpp:
(WebCore::CachedResource::CachedResource):
(WebCore::CachedResource::~CachedResource):
(WebCore::CachedResource::makePurgeable):
- loader/cache/CachedResource.h:
- 9:37 AM Changeset in webkit [179994] by
-
- 7 edits2 adds in branches/safari-600.5-branch
Merge r179883. rdar://problem/19790645
- 9:30 AM WebKitGTK/Eclipse edited by
- (diff)
- 9:20 AM Changeset in webkit [179993] by
-
- 19 edits2 adds in trunk
[Win] [64-bit] Work around MSVC2013 Runtime Bug
https://bugs.webkit.org/show_bug.cgi?id=141498
<rdar://problem/19803642>
Reviewed by Anders Carlsson.
Disable FMA3 instruction use in the MSVC math library to
work around a VS2013 runtime crash. We can remove this
workaround when we switch to VS2015.
Source/JavaScriptCore:
- API/tests/testapi.c: Call _set_FMA3_enable(0) to disable
FMA3 support.
- JavaScriptCore.vcxproj/JavaScriptCore.vcxproj: Add new files.
- JavaScriptCore.vcxproj/JavaScriptCore.vcxproj.filters: Ditto.
- JavaScriptCore.vcxproj/JavaScriptCoreDLL.cpp: Added.
- JavaScriptCore.vcxproj/jsc/DLLLauncherMain.cpp: Call _set_FMA3_enable(0)
to disable FMA3 support.
- jsc.cpp: Ditto.
- testRegExp.cpp: Ditto.
Source/WebKit/win:
- WebKitDLL.cpp: Call _set_FMA3_enable(0) to disable FMA3 support.
Source/WTF:
- WTF.vcxproj/WTF.vcxproj: Add new files.
- WTF.vcxproj/WTF.vcxproj.filters: Ditto.
- wtf/win/WTFDLL.cpp: Added.
Tools:
- DumpRenderTree/cg/ImageDiffCG.cpp: Call _set_FMA3_enable(0) to
disable FMA3 support.
- DumpRenderTree/win/DumpRenderTree.cpp: Ditto.
- DumpRenderTree/win/ImageDiffCairo.cpp: Ditto.
- DumpRenderTree/win/TestRunnerWin.cpp: Ditto.
- Scripts/webkitperl/httpd.pm: Ditto.
- TestWebKitAPI/win/main.cpp: Ditto.
- WinLauncher/Common.cpp: Ditto.
- win/DLLLauncher/DLLLauncherMain.cpp: Ditto.
- 9:12 AM Changeset in webkit [179992] by
-
- 2 edits in branches/safari-600.5-branch/Source/WebKit
Merge r178470. rdar://problem/19739095
- 9:12 AM WebKitGTK/Eclipse edited by
- (diff)
- 8:48 AM Changeset in webkit [179991] by
-
- 2 edits in trunk/Source/WebCore
[Win] Unreviewed debug build fix after r179980.
- svg/SVGSVGElement.cpp:
(WebCore::SVGSVGElement::currentViewportSize):
Explicitly call constructor.
- 8:35 AM Changeset in webkit [179990] by
-
- 2 edits in trunk/LayoutTests
Unreviewed, EFL gardening. Two tests have been failing since r148090.
- platform/efl/TestExpectations: Skip 2 failure tests.
- 8:32 AM Changeset in webkit [179989] by
-
- 2 edits in trunk/Websites/perf.webkit.org
v2 UI should adjust the number of ticks on dashboards based on screen size
https://bugs.webkit.org/show_bug.cgi?id=141502
Reviewed by Chris Dumez.
- public/v2/interactive-chart.js:
(App.InteractiveChartComponent._updateDimensionsIfNeeded): Compute the number of ticks based on the
content size.
- 8:25 AM Changeset in webkit [179988] by
-
- 2 edits in trunk/LayoutTests
Unreviewed, EFL gardening.
- platform/efl/TestExpectations: Mark two tests failing since r177664.
- 8:14 AM Changeset in webkit [179987] by
-
- 7 edits2 adds in trunk
[CSS Grid Layout] Invalid initialization of track sizes with non spanning grid items
https://bugs.webkit.org/show_bug.cgi?id=140763
Reviewed by Antti Koivisto.
Source/WebCore:
Content sized tracks with non-spanning grid items were not
properly sized because the growth limit was sometimes infinity
(-1) after calling resolveContentBasedTrackSizingFunctions() when
it should not. This patch adds an special initialization phase for
non-spanning grid items as the new track sizing algorithm
describes.
Granted, that was handled in the old algorithm in
distributeSpaceToTracks() as a special case. The problem is that
it regressed after the optimization added in r173868 because that
method is no longer called when the space to distribute is 0.
That's why we could fix this by allowing calls to
distributeSpaceToTracks() with spaceToDistribute>=0 but by fixing
it with an explicit initialization our implementation becomes
closer to the new algorithm and the initialization is now explicit
in the code instead of a side effect of calling
distributeSpaceToTracks() with no space to be distributed. It also
brings a slight performance improvement as we save sorts and hash
lookups.
I also took the change to add caching to several GridTrackSize
methods that were hot on the profiler (each one accounted for ~1%
of the total time, now they account for ~0.3% each).
Test: fast/css-grid-layout/grid-initialize-span-one-items.html
- rendering/RenderGrid.cpp:
(WebCore::GridItemWithSpan::span): New helper method for ASSERTs.
(WebCore::RenderGrid::resolveContentBasedTrackSizingFunctions):
Exclude non spanning grid items from the calls to
resolveContentBasedTrackSizingFunctionsForItems().
(WebCore::RenderGrid::resolveContentBasedTrackSizingFunctionsForNonSpanningItems):
New method to resolve track sizes only using non-spanning grid
items.
(WebCore::RenderGrid::resolveContentBasedTrackSizingFunctionsForItems):
Ensure that it isn't called for non-spanning grid items.
- rendering/RenderGrid.h:
- rendering/style/GridTrackSize.h:
(WebCore::GridTrackSize::GridTrackSize): Cache return values.
(WebCore::GridTrackSize::setLength): Ditto.
(WebCore::GridTrackSize::setMinMax): Ditto.
(WebCore::GridTrackSize::cacheMinMaxTrackBreadthTypes): New method
that caches the return values for hasXXXTrackBreadth() methods.
(WebCore::GridTrackSize::hasMinOrMaxContentMinTrackBreadth): Use
the cached return value.
(WebCore::GridTrackSize::hasMaxContentMaxTrackBreadth): Ditto.
(WebCore::GridTrackSize::hasMinContentMaxTrackBreadth): Ditto.
(WebCore::GridTrackSize::hasMinOrMaxContentMaxTrackBreadth): Ditto.
(WebCore::GridTrackSize::hasMaxContentMinTrackBreadth): Ditto.
(WebCore::GridTrackSize::hasMinContentMinTrackBreadth): Ditto.
(WebCore::GridTrackSize::hasMinContentMinTrackBreadthAndMinOrMaxContentMaxTrackBreadth):
Ditto.
(WebCore::GridTrackSize::hasMaxContentMinTrackBreadthAndMaxContentMaxTrackBreadth):
Ditto.
LayoutTests:
- fast/css-grid-layout/grid-content-sized-columns-resolution-expected.txt:
- fast/css-grid-layout/grid-content-sized-columns-resolution.html:
- fast/css-grid-layout/grid-initialize-span-one-items-expected.txt: Added.
- fast/css-grid-layout/grid-initialize-span-one-items.html: Added.
- 8:08 AM Changeset in webkit [179986] by
-
- 3 edits in trunk/Tools
run-jsc-stress-tests should accept remote host config from JSON file
https://bugs.webkit.org/show_bug.cgi?id=141509
Reviewed by Michael Saboff.
- Scripts/run-javascriptcore-tests:
(runJSCStressTests):
- Scripts/run-jsc-stress-tests:
- 8:04 AM HackingWebInspector edited by
- (diff)
- 6:25 AM Changeset in webkit [179985] by
-
- 2 edits in trunk/Source/WebCore
Remove the USE(ACCELERATED_COMPOSITING) guard from the
MediaPlayerPrivateGStreamerBase destructor.
The guard was removed in r163079. It was still defined through CMake
for the EFL and GTK ports when the guarded code was introduced in
r172828. In r171741 the macro was finally removed for the GTK and EFL
ports as well, but the guards for the affected code weren't updated
so TextureMapperPlatformLayer::Client::platformLayerWillBeDestroyed()
wasn't being called anymore.
Rubber-stamped by Philippe Normand.
- platform/graphics/gstreamer/MediaPlayerPrivateGStreamerBase.cpp:
(WebCore::MediaPlayerPrivateGStreamerBase::~MediaPlayerPrivateGStreamerBase):
- 5:54 AM WebKitGTK edited by
- (diff)
- 5:49 AM WebKitGTK/Eclipse edited by
- (diff)
- 5:41 AM WebKitGTK/Eclipse created by
- 5:11 AM WebKitGTK/Roadmap edited by
- (diff)
- 3:29 AM WebKitGTK/SpeedUpBuild edited by
- (diff)
- 2:48 AM WebKitGTK/SpeedUpBuild edited by
- (diff)
- 12:50 AM Changeset in webkit [179984] by
-
- 12 edits5 adds in trunk
[GTK] Add API to allow overriding the default color chooser implementation
https://bugs.webkit.org/show_bug.cgi?id=98935
Reviewed by Gustavo Noronha Silva.
Source/WebKit2:
Add API to allow the user to override the default UI of color
chooser. Similar to the file chooser API, a new run-color-chooser
signal is added to WebKitWebView that passes a WebKitColorChooserRequest
parameter to the signal.
Initial patch by Arnaud Renevier <a.renevier@sisa.samsung.com>.
- PlatformGTK.cmake: Add new files to compilation.
- UIProcess/API/gtk/PageClientImpl.cpp:
(WebKit::PageClientImpl::createColorPicker): Create a
WebKitColorChooser is the view widget is a WebKitWebView or a
WebColorPicker otherwise,
- UIProcess/API/gtk/WebKitColorChooser.cpp: Added.
(WebKit::WebKitColorChooser::create):
(WebKit::WebKitColorChooser::WebKitColorChooser):
(WebKit::WebKitColorChooser::~WebKitColorChooser):
(WebKit::WebKitColorChooser::endPicker): Chain up to parent class
when not having a request, or finish the request if we have one.
(WebKit::WebKitColorChooser::colorChooserRequestFinished):
(WebKit::WebKitColorChooser::colorChooserRequestRGBAChanged):
(WebKit::WebKitColorChooser::showColorPicker): Create a new
WebKitColorChooserRequest and emit WebKitWebView::run-color-chooser.
If the signal is not handled, fallback to the default color
chooser dialog implemented by the parent class.
- UIProcess/API/gtk/WebKitColorChooser.h: Added.
- UIProcess/API/gtk/WebKitColorChooserRequest.cpp: Added.
(webkitColorChooserRequestGetProperty):
(webkitColorChooserRequestSetProperty):
(webkit_color_chooser_request_class_init):
(webkit_color_chooser_request_set_rgba):
(webkit_color_chooser_request_get_rgba):
(webkit_color_chooser_request_get_element_rectangle):
(webkit_color_chooser_request_finish):
(webkit_color_chooser_request_cancel):
(webkitColorChooserRequestCreate):
- UIProcess/API/gtk/WebKitColorChooserRequest.h: Added.
- UIProcess/API/gtk/WebKitColorChooserRequestPrivate.h: Added.
- UIProcess/API/gtk/WebKitWebView.cpp:
(webkit_web_view_class_init): Add run-color-chooser signal.
(webkitWebViewEmitRunColorChooser): Emit run-color-chooser signal.
- UIProcess/API/gtk/WebKitWebView.h:
- UIProcess/API/gtk/WebKitWebViewPrivate.h:
- UIProcess/API/gtk/docs/webkit2gtk-4.0-sections.txt: Add new symbols.
- UIProcess/API/gtk/docs/webkit2gtk-docs.sgml: Add new section for WebKitColorChooserRequest.
- UIProcess/API/gtk/webkit2.h: Include WebKitColorChooserRequest.h
Tools:
Add a color chooser implementation for MiniBrowser using
GtkPopover when the GTK+ version is recent enough.
And a test case to check the new color chooser API.
- MiniBrowser/gtk/BrowserWindow.c:
(colorChooserRGBAChanged):
(popoverColorClosed):
(colorChooserRequestFinished):
(runColorChooserCallback):
(browserWindowConstructed):
- TestWebKitAPI/Tests/WebKit2Gtk/TestUIClient.cpp:
(testWebViewColorChooserRequest):
(beforeAll):
Feb 11, 2015:
- 11:08 PM Changeset in webkit [179983] by
-
- 2 edits1 move in trunk/LayoutTests
AX: Undeprecate ios-simulator accessibility tests
https://bugs.webkit.org/show_bug.cgi?id=141407
Reviewed by Daniel Bates.
Undeprecate ios accessibility tests.
Modified one failing test (svg-group-element-with-title.html) slightly to get it to pass.
- platform/ios-sim-deprecated/accessibility: Removed.
- platform/ios-sim-deprecated/accessibility/accessibility-aria-table-children-expected.txt: Removed.
- platform/ios-sim-deprecated/accessibility/accessibility-aria-table-children.html: Removed.
- platform/ios-sim-deprecated/accessibility/accessibility-crash-in-axcontainer-expected.txt: Removed.
- platform/ios-sim-deprecated/accessibility/accessibility-crash-in-axcontainer.html: Removed.
- platform/ios-sim-deprecated/accessibility/accessibility-hint-expected.txt: Removed.
- platform/ios-sim-deprecated/accessibility/accessibility-hint.html: Removed.
- platform/ios-sim-deprecated/accessibility/aria-label-with-internal-text-expected.txt: Removed.
- platform/ios-sim-deprecated/accessibility/aria-label-with-internal-text.html: Removed.
- platform/ios-sim-deprecated/accessibility/aria-pressed-state-expected.txt: Removed.
- platform/ios-sim-deprecated/accessibility/aria-pressed-state.html: Removed.
- platform/ios-sim-deprecated/accessibility/centerpoint-expected.txt: Removed.
- platform/ios-sim-deprecated/accessibility/centerpoint.html: Removed.
- platform/ios-sim-deprecated/accessibility/disabled-states-expected.txt: Removed.
- platform/ios-sim-deprecated/accessibility/disabled-states.html: Removed.
- platform/ios-sim-deprecated/accessibility/dom-focus-fires-on-correct-element-expected.txt: Removed.
- platform/ios-sim-deprecated/accessibility/dom-focus-fires-on-correct-element.html: Removed.
- platform/ios-sim-deprecated/accessibility/element-paths-expected.txt: Removed.
- platform/ios-sim-deprecated/accessibility/element-paths.html: Removed.
- platform/ios-sim-deprecated/accessibility/file-upload-button-expected.txt: Removed.
- platform/ios-sim-deprecated/accessibility/file-upload-button.html: Removed.
- platform/ios-sim-deprecated/accessibility/focus-change-notifications-expected.txt: Removed.
- platform/ios-sim-deprecated/accessibility/focus-change-notifications.html: Removed.
- platform/ios-sim-deprecated/accessibility/header-elements-expected.txt: Removed.
- platform/ios-sim-deprecated/accessibility/header-elements.html: Removed.
- platform/ios-sim-deprecated/accessibility/identifier-expected.txt: Removed.
- platform/ios-sim-deprecated/accessibility/identifier.html: Removed.
- platform/ios-sim-deprecated/accessibility/internal-link-expected.txt: Removed.
- platform/ios-sim-deprecated/accessibility/internal-link.html: Removed.
- platform/ios-sim-deprecated/accessibility/landmark-type-expected.txt: Removed.
- platform/ios-sim-deprecated/accessibility/landmark-type.html: Removed.
- platform/ios-sim-deprecated/accessibility/link-with-images-text-expected.txt: Removed.
- platform/ios-sim-deprecated/accessibility/link-with-images-text.html: Removed.
- platform/ios-sim-deprecated/accessibility/link-with-only-image-expected.txt: Removed.
- platform/ios-sim-deprecated/accessibility/link-with-only-image.html: Removed.
- platform/ios-sim-deprecated/accessibility/math-expected.txt: Removed.
- platform/ios-sim-deprecated/accessibility/math.html: Removed.
- platform/ios-sim-deprecated/accessibility/mixed-checkboxes-expected.txt: Removed.
- platform/ios-sim-deprecated/accessibility/mixed-checkboxes.html: Removed.
- platform/ios-sim-deprecated/accessibility/no-child-link-header-expected.txt: Removed.
- platform/ios-sim-deprecated/accessibility/no-child-link-header.html: Removed.
- platform/ios-sim-deprecated/accessibility/non-contiguous-link-expected.txt: Removed.
- platform/ios-sim-deprecated/accessibility/non-contiguous-link.html: Removed.
- platform/ios-sim-deprecated/accessibility/out-of-bounds-child-access-expected.txt: Removed.
- platform/ios-sim-deprecated/accessibility/out-of-bounds-child-access.html: Removed.
- platform/ios-sim-deprecated/accessibility/password-value-expected.txt: Removed.
- platform/ios-sim-deprecated/accessibility/password-value.html: Removed.
- platform/ios-sim-deprecated/accessibility/placeholder-value-expected.txt: Removed.
- platform/ios-sim-deprecated/accessibility/placeholder-value.html: Removed.
- platform/ios-sim-deprecated/accessibility/popup-button-value-label-expected.txt: Removed.
- platform/ios-sim-deprecated/accessibility/popup-button-value-label.html: Removed.
- platform/ios-sim-deprecated/accessibility/progressbar-expected.txt: Removed.
- platform/ios-sim-deprecated/accessibility/progressbar.html: Removed.
- platform/ios-sim-deprecated/accessibility/radio-button-expected.txt: Removed.
- platform/ios-sim-deprecated/accessibility/radio-button.html: Removed.
- platform/ios-sim-deprecated/accessibility/secure-text-field-expected.txt: Removed.
- platform/ios-sim-deprecated/accessibility/secure-text-field.html: Removed.
- platform/ios-sim-deprecated/accessibility/selected-buttons-expected.txt: Removed.
- platform/ios-sim-deprecated/accessibility/selected-buttons.html: Removed.
- platform/ios-sim-deprecated/accessibility/selected-text-expected.txt: Removed.
- platform/ios-sim-deprecated/accessibility/selected-text.html: Removed.
- platform/ios-sim-deprecated/accessibility/svg-group-element-with-title-expected.txt: Removed.
- platform/ios-sim-deprecated/accessibility/svg-group-element-with-title.html: Removed.
- platform/ios-sim-deprecated/accessibility/svg-path-crash-expected.txt: Removed.
- platform/ios-sim-deprecated/accessibility/svg-path-crash.html: Removed.
- platform/ios-sim-deprecated/accessibility/tab-role-expected.txt: Removed.
- platform/ios-sim-deprecated/accessibility/tab-role.html: Removed.
- platform/ios-sim-deprecated/accessibility/table-cell-for-row-col-expected.txt: Removed.
- platform/ios-sim-deprecated/accessibility/table-cell-for-row-col.html: Removed.
- platform/ios-sim-deprecated/accessibility/table-cell-ranges-expected.txt: Removed.
- platform/ios-sim-deprecated/accessibility/table-cell-ranges.html: Removed.
- platform/ios-sim-deprecated/accessibility/table-with-mismatch-column-count-in-initial-section-expected.txt: Removed.
- platform/ios-sim-deprecated/accessibility/tables-lists-expected.txt: Removed.
- platform/ios-sim-deprecated/accessibility/tables-lists.html: Removed.
- platform/ios-sim-deprecated/accessibility/text-line-no-ignored-elements-expected.txt: Removed.
- platform/ios-sim-deprecated/accessibility/text-line-no-ignored-elements.html: Removed.
- platform/ios-sim-deprecated/accessibility/text-marker-list-item-expected.txt: Removed.
- platform/ios-sim-deprecated/accessibility/text-marker-list-item.html: Removed.
- platform/ios-sim-deprecated/accessibility/text-marker-validation-expected.txt: Removed.
- platform/ios-sim-deprecated/accessibility/text-marker-validation.html: Removed.
- platform/ios-sim-deprecated/accessibility/text-role-expected.txt: Removed.
- platform/ios-sim-deprecated/accessibility/text-role.html: Removed.
- platform/ios-sim-deprecated/accessibility/textfield-in-axvalue-expected.txt: Removed.
- platform/ios-sim-deprecated/accessibility/textfield-in-axvalue.html: Removed.
- platform/ios-sim-deprecated/accessibility/toggle-button-expected.txt: Removed.
- platform/ios-sim-deprecated/accessibility/toggle-button.html: Removed.
- platform/ios-sim-deprecated/accessibility/url-test-expected.txt: Removed.
- platform/ios-sim-deprecated/accessibility/url-test.html: Removed.
- platform/ios-simulator/accessibility: Copied from LayoutTests/platform/ios-sim-deprecated/accessibility.
- platform/ios-simulator/accessibility/svg-group-element-with-title.html:
- 9:09 PM Changeset in webkit [179982] by
-
- 22 edits in trunk/Source/WebCore
Streamline and simplify SVGSVGElement and related classes
https://bugs.webkit.org/show_bug.cgi?id=141463
Reviewed by Antti Koivisto.
- dom/Document.cpp: Removed unneeded include of SVGSVGElement.h.
- loader/FrameLoader.cpp: Ditto.
- rendering/RenderBoxModelObject.cpp: Ditto.
- rendering/svg/RenderSVGForeignObject.cpp: Ditto.
- svg/SVGImageElement.cpp: Ditto.
- svg/SVGMarkerElement.cpp: Ditto.
- svg/SVGPathElement.cpp: Ditto.
- svg/SVGPatternElement.cpp: Ditto.
- svg/SVGTransform.cpp: Ditto.
- page/FrameView.cpp:
(WebCore::FrameView::scrollToAnchor): Updated to call the SVGSVGElement function
named scrollToAnchor (was setupInitialView). Also refactored to avoid repeating
the expression frame().document() so often.
- platform/graphics/FloatSize.h: Added operator/.
- svg/SVGDocument.cpp:
(WebCore::SVGDocument::rootElement): Rearranged to use early return.
(WebCore::SVGDocument::zoomAndPanEnabled): Ditto.
(WebCore::SVGDocument::startPan): Ditto. Also used FloatPoint math instead
of breaking things apart into x and y.
(WebCore::SVGDocument::updatePan): Ditto.
- svg/SVGDocument.h: Removed some unneeded includes and forward declarations.
Moved function bodies out of the class to make it more readable. Renamed
m_translate to m_panningOffset and made it a FloatSize instead of FloatPoint.
- svg/SVGDocumentExtensions.cpp:
(WebCore::SVGDocumentExtensions::startAnimations): Use a modern for loop
and update since timeContainer() now returns a reference.
- svg/SVGSVGElement.cpp: Removed many unneeded includes.
(WebCore::SVGSVGElement::SVGSVGElement): Moved initialization of scalars to
the class definition. Updated since m_timerContainer is a Ref (later should
change it to be a std::unique_ptr or just directly contained in SVGSVGElement).
(WebCore::SVGSVGElement::~SVGSVGElement): Removed comment that doesn't say much,
but is worded in a confusing way.
(WebCore::SVGSVGElement::contentScriptType): Use NeverDestroyed instead of
DEPRECATED_DEFINE_STATIC_LOCAL.
(WebCore::SVGSVGElement::contentStyleType): Ditto.
(WebCore::SVGSVGElement::viewport): Streamlined a bit.
(WebCore::SVGSVGElement::pixelUnitToMillimeterX): Ditto.
(WebCore::SVGSVGElement::pixelUnitToMillimeterY): Ditto.
(WebCore::SVGSVGElement::currentView): Changed to return a reference.
(WebCore::SVGSVGElement::frameForCurrentScale): Added. Helper for the two
functions below.
(WebCore::SVGSVGElement::currentScale): Updated to use frameForCurrentScale.
(WebCore::SVGSVGElement::setCurrentScale): Ditto.
(WebCore::SVGSVGElement::setCurrentTranslate): Optimized the case where there
is no change to the translation. Also changed the data member's name to
m_currentTranslate to more directly match the DOM API naming.
(WebCore::SVGSVGElement::parseAttribute): Simplified the parsing a bit, and
changed to pass a reference to SVGZoomAndPan::parseAttribute.
(WebCore::SVGSVGElement::svgAttributeChanged): Tweaked formatting and removed
code that converts the renderer pointer to a RenderObject* unnecessarily.
(WebCore::SVGSVGElement::suspendRedraw): Removed FIXME about implementing this,
since we don't really plan to do that.
(WebCore::SVGSVGElement::unsuspendRedraw): Ditto.
(WebCore::SVGSVGElement::unsuspendRedrawAll): Ditto.
(WebCore::SVGSVGElement::forceRedraw): Ditto.
(WebCore::SVGSVGElement::collectIntersectionOrEnclosureList): Changed return
type to Ref, and pass in a function instead of an enumeration value.
(WebCore::SVGSVGElement::getIntersectionList): Updated for above.
(WebCore::SVGSVGElement::getEnclosureList): Ditto.
(WebCore::SVGSVGElement::checkIntersection): Use && instead of early return
for the null check.
(WebCore::SVGSVGElement::checkEnclosure): Ditto.
(WebCore::SVGSVGElement::createSVGLength): Use initialization list syntax
for greater brevity.
(WebCore::SVGSVGElement::createSVGAngle): Ditto.
(WebCore::SVGSVGElement::createSVGPoint): Ditto.
(WebCore::SVGSVGElement::createSVGMatrix): Ditto.
(WebCore::SVGSVGElement::createSVGRect): Ditto.
(WebCore::SVGSVGElement::createSVGTransform): Removed unneeded explicit
constructor invocation.
(WebCore::SVGSVGElement::createSVGTransformFromMatrix): Removed unneeded cast.
(WebCore::SVGSVGElement::insertedInto): Updated since timeContainer() now
returns a reference.
(WebCore::SVGSVGElement::setCurrentTime): Use isfinite instead of isnan, so
we won't store infinity as the current time.
(WebCore::SVGSVGElement::currentViewBoxRect): Renamed locals and changed
to use initializer lists for brevity.
(WebCore::SVGSVGElement::currentViewportSize): Streamlined code by using
initializer lists, using local variables only for things used more than once,
and using the / operator defined above.
(WebCore::SVGSVGElement::viewBoxToViewTransform): Use the concatenate function
in a way that is more straightforward, rather than doing it with a temporary.
(WebCore::SVGSVGElement::scrollToAnchor): Renamed this from
setupInitialView.
(WebCore::SVGSVGElement::inheritViewAttributes): Changed this to work with
references rather than pointers.
(WebCore::SVGSVGElement::getElementById): Use getAllElementsById as the
FIXME comment suggested for better efficiency when there are a lot of descendants
of the <svg> element at the cost of making it less efficient if there are
a lot of elements all with the desired ID, but not inside the <svg> element.
(WebCore::SVGSVGElement::isValid): Moved here from the header.
- svg/SVGSVGElement.h: Removed unneeded forward declarations. Moved the
animated properties to the top of the file, then other DOM, the other
public members. Changed return types to references and Ref. Moved
function bodies out of the class definition. Made createSVGNumber inline.
- svg/SVGViewElement.cpp: Removed unneeded includes.
(WebCore::SVGViewElement::isSupportedAttribute): Deleted.
(WebCore::SVGViewElement::parseAttribute): Changed to not rely on
isSupportedAttribute. Eventually I will do this in all the SVG classes.
- svg/SVGViewSpec.cpp: Removed unneeded include of SVGSVGElement.h.
(WebCore::SVGViewSpec::parseViewSpec): Updated for change in the name
of the parse function in the SVGZoomAndPan class.
- svg/SVGZoomAndPan.cpp:
(WebCore::SVGZoomAndPan::isKnownAttribute): Deleted.
(WebCore::SVGZoomAndPan::addSupportedAttributes): Deleted.
(WebCore::SVGZoomAndPan::parse): Renamed from parseZoomAndPan since
this is a member of the SVGZoomAndPan class and also has an argument
of SVGZoomAndPanType.
(WebCore::SVGZoomAndPan::parseAttributeValue): Added. Helper so the
template function is not so big. Also write it in a simpler way.
(WebCore::SVGZoomAndPan::ref): Deleted.
(WebCore::SVGZoomAndPan::deref): Deleted.
(WebCore::SVGZoomAndPan::setZoomAndPan): Deleted.
- svg/SVGZoomAndPan.h: Removed unneeded includes. Reformatted the enum.
Moved function bodies out of the class definition. Removed many unneeded
functions.
- svg/animation/SVGSMILElement.cpp:
(WebCore::SVGSMILElement::insertedInto): Updated since timeContainer
now returns a reference rather than a pointer.
- 8:37 PM Changeset in webkit [179981] by
-
- 2 edits in trunk/LayoutTests
Unreviewed, EFL gardenging. Need to mark pointer/hover media query tests as failure.
These tests have been failed since r179055.
- platform/efl/TestExpectations:
- 7:53 PM Changeset in webkit [179980] by
-
- 18 edits2 adds10 deletes in trunk
SVGUseElement follow-up improvements
https://bugs.webkit.org/show_bug.cgi?id=141382
Reviewed by Antti Koivisto.
Source/WebCore:
- loader/cache/CachedSVGDocumentClient.h: Removed unneeded forward declaration.
- page/EventHandler.cpp: Removed unneeded include of SVGUseElement.h.
- rendering/svg/RenderSVGViewportContainer.cpp: Ditto.
- svg/SVGDocumentExtensions.cpp:
(WebCore::SVGDocumentExtensions::clearTargetDependencies): Removed too-specific
check that assumed that SVG elements in shadow trees are always for <use> elements.
This amounted to an unneeded optimization that could be removed with no bad effect.
- svg/SVGElement.cpp:
(WebCore::SVGElement::correspondingElement): Removed the assertions so this could
be used more freely outside of cases where the shadow tree state is fully consistent.
It's fine to have this just be a mechanical getter; there's nothing super-tricky
here that needs to be caught by the assertion.
(WebCore::SVGElement::title): Removed unneeded special handling for titles inside
the shadow tree.
- svg/SVGGElement.cpp:
(WebCore::SVGGElement::create): Added an overload that doesn't require explicitly
passing in the tag name.
- svg/SVGGElement.h: Ditto.
- svg/SVGSVGElement.cpp:
(WebCore::SVGSVGElement::create): Ditto.
- svg/SVGSVGElement.h: Ditto.
- svg/SVGUseElement.cpp: Removed a lot of unneeded includes.
(WebCore::SVGUseElement::SVGUseElement): Removed code to initialize some booleans.
We do that in the class definition now.
(WebCore::SVGUseElement::create): Removed the code that calls the
ensureUserAgentShadowRoot function unconditionally. That's properly done when
needed; no need to do it here.
(WebCore::SVGUseElement::~SVGUseElement): Removed unneeded code to destroy the
shadow tree (that happens automatically) and simplified the code to stop loading
the external document.
(WebCore::SVGUseElement::isSupportedAttribute): Deleted.
(WebCore::SVGUseElement::parseAttribute): Simplified this. Removed assumptions
about the intersection of various sets of attributes, and also removed the
isSupportedAttribute function. This seems to serve no purpose here, or in any
other SVG element class. I plan to remove it everywhere over time.
(WebCore::isWellFormedDocument): Deleted.
(WebCore::SVGUseElement::insertedInto): Simplified code by removing all the
special cases during initial parsing, and did the invalidation here rather than
deferring it to didNotifySubtreeInsertions. Added a call to the new function,
updateExternalDocument, since that won't do anything when the element is not
in a document.
(WebCore::SVGUseElement::didNotifySubtreeInsertions): Deleted.
(WebCore::SVGUseElement::removedFrom): Added code to call clearShadowTree and
updateExternalDocument. Both are efficient when doing nothing, and both are
appropriate since the element is no longer in a document.
(WebCore::SVGUseElement::referencedDocument): Deleted. No longer needed.
(WebCore::SVGUseElement::externalDocument): Streamlined the logic here, removing
multiple unneeded checks.
(WebCore::SVGUseElement::transferSizeAttributesToTargetClone): Renamed since
"target clone" is clear enough within this class, without explicitly stating
"shadow tree". All the clones are in the shadow tree.
(WebCore::SVGUseElement::svgAttributeChanged): Removed unneeded code calling
isSupportedAttribute. Changed the code that detects changes in href to just
call updateExternalDocument (for the document URL) and invalidateShadowTree
(for the fragment). Also updated the transferSizeAttributesToTargetClone logic
to only trigger on width and height and updated names.
(WebCore::SVGUseElement::willAttachRenderers): Updated for the new name of
m_shouldRebuildShadowTree and added a call through to the base class.
(WebCore::createAllowedElementSet): Added. A more efficient way to implement
the initialization of the set for isDisallowedElement.
(WebCore::isDisallowedElement): Simplified this by using the function above,
and also overloaded for both SVGElement and Element for a tiny efficiency boost.
(WebCore::SVGUseElement::clearShadowTree): Renamed form clearResourceReferences.
This is a much more straightforward name. Also deleted the code that sets the
m_needsShadowTreeRecreation flag to false. That should be done by the build
function, not here.
(WebCore::SVGUseElement::buildPendingResource): Made this just invalidate the
shadow tree now instead of explicitly building it.
(WebCore::SVGUseElement::updateShadowTree): Moved the code to create a shadow
tree here from buildPendingResource. ALso changed the logic so that we
always blow away the old shadow tree. Moved the comment about rebuilding things
every time here. Updated the code to use the findTarget and cloneTarget functions,
eliminating the buildShadowTree function entirely. Moved the call to
transferSizeAttributesToShadowTreeTargetClone inside cloneTarget. Also updated
for the name change for m_shouldRebuildShadowTree.
(WebCore::SVGUseElement::targetClone): Renamed from shadowTreeTargetClone.
No need to emphasize "shadow tree" since that's where all clones are.
(WebCore::isDirectReference): Streamlined a bit using "using namespace".
(WebCore::SVGUseElement::toClipPath): Rewrote to use early return and updated
for name changes. Also used ASCIILiteral.
(WebCore::SVGUseElement::rendererClipChild): Changed local variable names.
(WebCore::removeDisallowedElementsFromSubtree): Wrote the iteration in a
slightly more idiomatic style.
(WebCore::SVGUseElement::findTarget): Added. This new function implements
the rule for finding a valid target for a use element. This replaces logic
that was duplicated in two different places and it also includes all the
rules that were formerly in the isValidTarget function. Also, this implements
a correct check for a cycle that handles cases the code in isValidTarget did not.
(WebCore::SVGUseElement::isValidTarget): Deleted.
(WebCore::SVGUseElement::cloneTarget): Added. Helper function used both when
cloning the target of the top level <use> elements and for other <use> elements
inside the shadow tree.
(WebCore::cloneDataAndChildren): Added. Helper function that allows both the
<use> and <symbol> element expanding functions to be shorter and share more code.
(WebCore::SVGUseElement::expandUseElementsInShadowTree): Removed unneeded checks
of cachedDocumentIsStillLoading. Used the new findTarget function, which handles
finding the target cross-document correctly. Removed the incorrect use of
referencedDocument when creating new elements and finding targets. Refactored
to use the new cloneDataAndChildren function and also moved the code that removes
the special attributes here, replacing the transferAttributesToShadowTreeReplacement
function. Made a few other simplifications.
(WebCore::SVGUseElement::expandSymbolElementsInShadowTree): Ditto, just like the
<use> changes only simpler.
(WebCore::SVGUseElement::transferEventListenersToShadowTree): Made this const.
Removed unneeded assertions.
(WebCore::SVGUseElement::invalidateShadowTree): Updated for name change.
(WebCore::SVGUseElement::invalidateDependentShadowTrees): Removed assertion.
(WebCore::SVGUseElement::transferAttributesToShadowTreeReplacement): Deleted.
(WebCore::SVGUseElement::selfHasRelativeLengths): Tweaked names.
(WebCore::SVGUseElement::notifyFinished): Removed the inDocument check, since
this function will only be called for elements that are in a document.
(WebCore::SVGUseElement::cachedDocumentIsStillLoading): Deleted.
(WebCore::SVGUseElement::finishParsingChildren): Removed the code that calls
buildPendingResource here. Shadow tree updating is driven solely by renderer
generation now.
(WebCore::SVGUseElement::updateExternalDocument): Replaced setCachedDocument
with this. This function knows how to load a different document if the URL
has changed, or leave it alone if not, and also stop the load if it should.
(WebCore::SVGUseElement::isValid): Moved this here from the header, since it's
always being called virtually.
(WebCore::SVGUseElement::haveLoadedRequiredResources): Ditto.
(WebCore::SVGUseElement::setHaveFiredLoadEvent): Ditto.
(WebCore::SVGUseElement::haveFiredLoadEvent): Ditto.
(WebCore::SVGUseElement::svgLoadEventTimer): Ditto.
- svg/SVGUseElement.h: Removed unneeded include. Moved the animated properties
to the top of the class because they are public DOM API and so are logical to
list first. I'd like to do that for other classes too over time. Changed to
derive privately from CachedSVGDocumentClient. Made the function
invalidateDependentShadowTrees private. Removed didNotifySubtreeInsertions,
isSupportedAttribute, clearResourceReferences, buildShadowTree,
transferAttributesToShadowTreeReplacement, isParserInserted, and
m_wasInsertedByParser. Added updateExternalDocument, cloneTarget, targetClone,
updateShadowTree, and clearShadowTree. Also did a couple other renames,
including renaming m_cachedDocument to m_externalDocument.
- svg/svgtags.in: Removed constructorNeedsCreatedByParser from the <use>
element since we don't have to handle constructing by the parser specially.
LayoutTests:
Modified some tests to be reference tests since the change in implementation slightly changed
the behavior, but not in a way that matters. Other similar updates.
- TestExpectations: Expect a progression in imported/mozilla/svg/dynamic-use-02.svg.
- platform/gtk/svg/custom/relative-sized-shadow-tree-content-with-symbol-expected.png: Removed.
- platform/gtk/svg/custom/relative-sized-shadow-tree-content-with-symbol-expected.txt: Removed.
- platform/gtk/svg/custom/use-property-synchronization-crash-expected.png: Removed.
- platform/ios-sim-deprecated/svg/custom/relative-sized-shadow-tree-content-with-symbol-expected.txt: Removed.
- platform/ios-simulator/svg/custom/relative-sized-shadow-tree-content-with-symbol-expected.txt: Removed.
- platform/mac-mountainlion/svg/custom/relative-sized-shadow-tree-content-with-symbol-expected.txt: Removed.
- platform/mac/svg/custom/relative-sized-shadow-tree-content-with-symbol-expected.png: Removed.
- platform/mac/svg/custom/relative-sized-shadow-tree-content-with-symbol-expected.txt: Removed.
- platform/mac/svg/custom/use-property-synchronization-crash-expected.png: Removed.
- svg/animations/use-animate-width-and-height-expected.txt: Updated to expect the new expression
name from the modified test.
- svg/animations/use-animate-width-and-height.html: THe state of the shadow root now depends on
layout, so force layout before inspecting it.
- svg/custom/relative-sized-shadow-tree-content-with-symbol-expected.xhtml: Added.
- svg/custom/relative-sized-shadow-tree-content-with-symbol.xhtml: Made this no longer be a
repaint test. Not sure why we are using those in so many cases. Also made it be a reference test.
- svg/custom/use-property-synchronization-crash-expected.svg: Added.
- svg/custom/use-property-synchronization-crash-expected.txt: Removed.
- 7:46 PM Changeset in webkit [179979] by
-
- 2 edits in trunk/LayoutTests
Unreviewed, EFL gardening. Mark multi-column failing tests to Failure.
Those tests have been failing since r167808.
- platform/efl/TestExpectations:
- 6:33 PM Changeset in webkit [179978] by
-
- 3 edits2 adds in trunk
CSS selector JIT compilation support for :lang()
https://bugs.webkit.org/show_bug.cgi?id=140818
Reviewed by Benjamin Poulain.
Source/WebCore:
Add the JIT support for :lang(), to ensure :lang() no longer force the engine to
the slow path.
Test: fast/selectors/lang-chained-multiple.html
- cssjit/SelectorCompiler.cpp:
(WebCore::SelectorCompiler::addPseudoClassType):
Get rid of the FunctionType::CannotCompile
(WebCore::SelectorCompiler::SelectorCodeGenerator::generateElementMatching):
(WebCore::SelectorCompiler::SelectorCodeGenerator::generateElementIsInLanguage):
LayoutTests:
- fast/selectors/lang-chained-multiple-expected.txt: Added.
- fast/selectors/lang-chained-multiple.html: Added.
- 6:25 PM Changeset in webkit [179977] by
-
- 3 edits in trunk/Source/WebCore
Fix more weak external symbol errors.
- bindings/scripts/CodeGeneratorJS.pm:
(GenerateHeader):
- page/make_settings.pl:
(printGetterAndSetter):
Remove some WEBCORE_EXPORT macros.
- 5:46 PM Changeset in webkit [179976] by
-
- 14 edits in branches/safari-600.5-branch
Merged r177377. rdar://problem/19526158
- 5:25 PM Changeset in webkit [179975] by
-
- 2 edits in trunk/Source/WebKit2
iOS build fix.
Unreviewed.
- UIProcess/ios/WKContentViewInteraction.mm:
(-[WKContentView inputAssistantItem]):
(-[WKContentView _inputAssistantItem]):
- 5:09 PM Changeset in webkit [179974] by
-
- 7 edits in trunk/Source/WebCore
Do not export functions defined in headers.
This prevents weak external symbol errors in some configurations.
- dom/Range.h:
(WebCore::Range::startContainer):
(WebCore::Range::startOffset):
(WebCore::Range::endContainer):
(WebCore::Range::endOffset):
- inspector/InspectorController.h:
- page/Settings.h:
(WebCore::Settings::setMediaKeysStorageDirectory):
- page/make_settings.pl:
(printGetterAndSetter):
- platform/network/cocoa/ProtectionSpaceCocoa.h:
(WebCore::ProtectionSpace::encodingRequiresPlatformData):
- platform/text/TextEncoding.h:
(WebCore::TextEncoding::decode):
Removed WEBCORE_EXPORT macros.
- 4:46 PM Changeset in webkit [179973] by
-
- 2 edits in trunk/Source/WebInspectorUI
Web Inspector: REGRESSION: Clicking selected item in the sidebar second time should scroll to the corresponding line
https://bugs.webkit.org/show_bug.cgi?id=140280
Patch by Nikita Vasilyev <Nikita Vasilyev> on 2015-02-11
Reviewed by Brian Burg.
- UserInterface/Views/ContentViewContainer.js:
(WebInspector.ContentViewContainer.prototype.showContentView):
Perform scolling regardless of the navigation history. Since scrolling source code doesn't introduce a BackForwardEntry,
it's possible that the actual content view has a different scroll position.
- 3:57 PM Changeset in webkit [179972] by
-
- 16 edits in trunk/Source
[WK2] Add logging to validate the network cache efficacy (Part 2)
https://bugs.webkit.org/show_bug.cgi?id=141345
<rdar://problem/19632080>
Reviewed by Sam Weinig.
Source/WebCore:
Add a few more diagnostic logging keys for the network cache efficacy
logging.
Source/WebKit2:
Add diagnostic logging messages to validate the network cache efficacy.
The following 4 messages are added:
- networkCache / retrieval / success
- networkCache / retrieval / unhandledRequestFailure
- networkCache / retrieval / noLongerInCacheFailure
- networkCache / retrieval / unusableCachedEntryFailure
The messages are sent via IPC from the NetworkProcess to the UIProcess,
where the WebPageProxy code already handles diagnostic messages sent by
the WebProcess.
- 3:40 PM Changeset in webkit [179971] by
-
- 3 edits2 adds in trunk
REGRESSION(r179166): Crash when accessing document.dir on a document with no elements
<rdar://problem/19804351>
https://bugs.webkit.org/show_bug.cgi?id=141480
Reviewed by Chris Dumez.
Source/WebCore:
Test: fast/dom/document-dir-empty-document-crash.html
- html/HTMLDocument.cpp:
(WebCore::HTMLDocument::dir):
Don't unnecessarily dereference the documentElement. is<HTMLHtmlElement>() will handle null just fine.
LayoutTests:
- fast/dom/document-dir-empty-document-crash-expected.txt: Added.
- fast/dom/document-dir-empty-document-crash.html: Added.
- 3:14 PM Changeset in webkit [179970] by
-
- 2 edits in trunk/Source/WebCore
<https://webkit.org/b/141492> extract-localizable-strings.pl shouldn't update the target file if the contents haven't changed
Avoid updating the target file if the contents haven't changed. This prevents Xcode from copying the identical
file into the framework and resigning it, which avoids the resulting relinking of all targets that depend on
the framework.
Reviewed by Dan Bernstein.
- extract-localizable-strings.pl: Write our output to a temporary file. If the output differs from the
existing contents of the target file, move the temporary file over the target file. Otherwise, delete
the temporary file.
- 3:12 PM Changeset in webkit [179969] by
-
- 2 edits in branches/safari-600.5-branch/Source/WebCore
Merged r179750. rdar://problem/19739067
- 3:06 PM Changeset in webkit [179968] by
-
- 3 edits2 copies in branches/safari-600.5-branch
Merged r179706. rdar://problem/19739067
- 3:02 PM Changeset in webkit [179967] by
-
- 5 edits in trunk/Source/JavaScriptCore
The callee frame helpers in DFG::SpeculativeJIT should be available to other JITs
https://bugs.webkit.org/show_bug.cgi?id=141493
Reviewed by Michael Saboff.
- dfg/DFGSpeculativeJIT.h:
(JSC::DFG::SpeculativeJIT::calleeFrameSlot): Deleted.
(JSC::DFG::SpeculativeJIT::calleeArgumentSlot): Deleted.
(JSC::DFG::SpeculativeJIT::calleeFrameTagSlot): Deleted.
(JSC::DFG::SpeculativeJIT::calleeFramePayloadSlot): Deleted.
(JSC::DFG::SpeculativeJIT::calleeArgumentTagSlot): Deleted.
(JSC::DFG::SpeculativeJIT::calleeArgumentPayloadSlot): Deleted.
(JSC::DFG::SpeculativeJIT::calleeFrameCallerFrame): Deleted.
- dfg/DFGSpeculativeJIT32_64.cpp:
(JSC::DFG::SpeculativeJIT::emitCall):
- dfg/DFGSpeculativeJIT64.cpp:
(JSC::DFG::SpeculativeJIT::emitCall):
- jit/AssemblyHelpers.h:
(JSC::AssemblyHelpers::calleeFrameSlot):
(JSC::AssemblyHelpers::calleeArgumentSlot):
(JSC::AssemblyHelpers::calleeFrameTagSlot):
(JSC::AssemblyHelpers::calleeFramePayloadSlot):
(JSC::AssemblyHelpers::calleeArgumentTagSlot):
(JSC::AssemblyHelpers::calleeArgumentPayloadSlot):
(JSC::AssemblyHelpers::calleeFrameCallerFrame):
- 3:01 PM Changeset in webkit [179966] by
-
- 3 edits in branches/safari-600.5-branch/Source/JavaScriptCore
- 2:56 PM Changeset in webkit [179965] by
-
- 4 edits in trunk/Websites/perf.webkit.org
New perf dashboard shows too much space around interesting data points
https://bugs.webkit.org/show_bug.cgi?id=141487
Reviewed by Chris Dumez.
Revise the y-axis range adjustment algorithm in r179913. Instead of showing the entire moving average,
show the current time series excluding points in the series outside the moving average envelope.
- public/v2/app.js:
(App.Pane._computeChartData): Don't deal with missing moving average or enveloping strategy here.
(App.Pane._computeMovingAverageAndOutliers): Set isOutliner to true on all data points in the current
time series if the point lies outside the moving average envelope. Don't expose the moving average or
the envelope computed for this purpose if they're not set by the user.
- public/v2/data.js:
(TimeSeries.prototype.minMaxForTimeRange): Takes a boolean argument, ignoreOutlier. When the flag is set
to true, min/max computation will ignore any point in the series with non-falsy "isOutliner" property.
- public/v2/interactive-chart.js:
(App.InteractiveChartComponent._constructGraphIfPossible): Unsupport hideMovingAverage and hideEnvelope.
(App.InteractiveChartComponent._computeYAxisDomain): Removed the commented out code. Also moved the code
to deal with showFullYAxis here.
(App.InteractiveChartComponent._minMaxForAllTimeSeries): Rewrote the code. Takes ignoreOutliners as an
argument instead of directly inspecting showFullYAxis.
- 2:51 PM Changeset in webkit [179964] by
-
- 2 edits in branches/safari-600.5-branch/Source/WebKit/win
Merged r179841. rdar://problem/19771791
- 2:48 PM Changeset in webkit [179963] by
-
- 5 edits2 copies in branches/safari-600.5-branch
Merged r179772. rdar://problem/19771772
- 2:48 PM Changeset in webkit [179962] by
-
- 4 edits in trunk/Source/WebKit/win
[WinCairo] Accelerated compositing should be disabled when graphics card does not support it.
https://bugs.webkit.org/show_bug.cgi?id=140667
Patch by peavo@outlook.com <peavo@outlook.com> on 2015-02-11
Reviewed by Alex Christensen.
- WebCoreSupport/AcceleratedCompositingContext.cpp:
(AcceleratedCompositingContext::scrollNonCompositedContents):
(AcceleratedCompositingContext::acceleratedCompositingAvailable):
- WebCoreSupport/AcceleratedCompositingContext.h:
- WebView.cpp:
(WebView::notifyPreferencesChanged):
- 2:43 PM Changeset in webkit [179961] by
-
- 2 edits in branches/safari-600.5-branch/Source/WebKit2
Merged r179736. rdar://problem/19771783
- 2:40 PM Changeset in webkit [179960] by
-
- 1 edit in branches/safari-600.5-branch/Source/WebCore/rendering/RenderQuote.cpp
Clarify logic after r179959.
- 2:24 PM Changeset in webkit [179959] by
-
- 6 edits2 copies in branches/safari-600.5-branch
Merged r179691. rdar://problem/19739061
- 2:15 PM Changeset in webkit [179958] by
-
- 2 edits in trunk/Source/WebCore
Turn recent assertions into release assertions to help track down crash in DocumentLoader::stopLoadingForPolicyChange()
https://bugs.webkit.org/show_bug.cgi?id=141484
<rdar://problem/13811738>
Reviewed by Andy Estes.
Turn recent assertions into release assertions to help track down crash in
DocumentLoader::stopLoadingForPolicyChange(). This should increase the
likelyhood of tripping them so that we better understand why this happens.
- loader/DocumentLoader.cpp:
(WebCore::DocumentLoader::~DocumentLoader):
(WebCore::DocumentLoader::detachFromFrame):
- 1:48 PM Changeset in webkit [179957] by
-
- 5 edits in trunk/Source/JavaScriptCore
SetupVarargsFrame should not assume that an inline stack frame would have identical layout to a normal stack frame
https://bugs.webkit.org/show_bug.cgi?id=141485
Reviewed by Oliver Hunt.
The inlineStackOffset argument was meant to make it easy for the DFG to use this helper for
vararg calls from inlined code, but that doesn't work since the DFG inline call frame
doesn't actually put the argument count at the JSStack::ArgumentCount offset. In fact there
is really no such thing as an inlineStackOffset except when we OSR exit; while the code is
running the stack layout is compacted so that the stackOffset is not meaningful.
- jit/JITCall.cpp:
(JSC::JIT::compileSetupVarargsFrame):
- jit/JITCall32_64.cpp:
(JSC::JIT::compileSetupVarargsFrame):
- jit/SetupVarargsFrame.cpp:
(JSC::emitSetupVarargsFrameFastCase):
- jit/SetupVarargsFrame.h:
- 1:45 PM Changeset in webkit [179956] by
-
- 2 edits in trunk/Source/WebCore
[MSE] SampleMap::addRange() returns an inverted iterator_range, possibly causing a crash when that iterator_range is traversed.
https://bugs.webkit.org/show_bug.cgi?id=141479
rdar://problem/19067597
Reviewed by Chris Dumez.
When looking backwards through a presentationOrder map to find samples, we then reverse our iterators
and put them in an iterator_range to return to the caller. But in addition to reversing the iterators
themselves, we also need to put them in the iterator_range in reverse order, so that when the caller
iterates from iterator_range.first -> iterator_range.second, they don't end up off the end of the
the underlying storage.
- Modules/mediasource/SampleMap.cpp:
(WebCore::PresentationOrderSampleMap::findSamplesWithinPresentationRangeFromEnd):
- 1:36 PM Changeset in webkit [179955] by
-
- 5 edits2 adds in trunk
REGRESSION (iOS 8): PDFs embedded in <object>/<embed> tag don't load at all
https://bugs.webkit.org/show_bug.cgi?id=141457
<rdar://problem/18221851>
Reviewed by Alexey Proskuryakov.
- WebProcess/WebCoreSupport/WebFrameLoaderClient.cpp:
(WebKit::WebFrameLoaderClient::objectContentType):
On iOS, if we don't know what to do with a PDF-in-object, treat it as an image.
- TestExpectations:
- fast/replaced/pdf-as-object-and-embed-expected.html: Added.
- fast/replaced/pdf-as-object-and-embed.html: Added.
- platform/ios-simulator-wk2/TestExpectations:
Add a test that ensures that WebKit2 on iOS treats <object>, <embed>,
and <img> the same when rendering a PDF.
- 1:33 PM Changeset in webkit [179954] by
-
- 10 edits2 copies in branches/safari-600.5-branch
Merged r179627. rdar://problem/19720096
- 1:28 PM Changeset in webkit [179953] by
-
- 4 edits2 copies in branches/safari-600.5-branch
Merged r179597. rdar://problem/19720061
- 1:14 PM Changeset in webkit [179952] by
-
- 4 edits in branches/safari-600.5-branch/Source/WebKit2
Merged r179586. rdar://problem/19771799
- 1:12 PM Changeset in webkit [179951] by
-
- 1 edit1 copy in branches/safari-600.5-branch/LayoutTests
Merged r179587. rdar://problem/19709589
- 1:11 PM Changeset in webkit [179950] by
-
- 7 edits1 copy in branches/safari-600.5-branch
Merged r179569. rdar://problem/19709589
- 1:00 PM Changeset in webkit [179949] by
-
- 5 edits2 copies in branches/safari-600.5-branch
Merged r179711. rdar://problem/19709603
- 12:54 PM Changeset in webkit [179948] by
-
- 2 edits in branches/safari-600.5-branch/Source/WebCore
Merged r179563. <rdar://problem/19739078>
- 12:34 PM Changeset in webkit [179947] by
-
- 3 edits in trunk/Source/WebCore
Improve the showFrameTree() output slightly
https://bugs.webkit.org/show_bug.cgi?id=141482
Reviewed by Zalan Bujtas.
Have showFrameTree() show the Frame's RenderView so it can be correlated with
layer dumps, and show styleRecalc and needsLayout state.
- page/FrameTree.cpp:
(printFrames):
- page/FrameView.cpp:
(WebCore::FrameView::updateLayoutAndStyleIfNeededRecursive): Add an explanatory
comment about why we don't walk the Frame tree.
- 12:31 PM Changeset in webkit [179946] by
-
- 3 edits in trunk/Tools
Make run-jsc-stress-tests --remote work with make-based runner
https://bugs.webkit.org/show_bug.cgi?id=141079
Reviewed by Darin Adler.
- Scripts/run-javascriptcore-tests:
(runJSCStressTests): Pass through the --shell-runner and --make-runner arguments.
- Scripts/run-jsc-stress-tests: Made the make-based runner the default runner,
except for remote testing on Darwin. Unified runShellTestRunner and runMakeTestRunner
to avoid code duplication.
- 12:29 PM Changeset in webkit [179945] by
-
- 3 edits in trunk/Source/WebCore
Stop using WebCore.exp.in on Mac and iOS.
https://bugs.webkit.org/show_bug.cgi?id=141413
Reviewed by Dan Bates.
- Configurations/Base.xcconfig:
Make symbols hidden by default unless exported by WEBCORE_EXPORT macros.
- platform/PlatformExportMacros.h:
Start using WEBCORE_EXPORT on Mac and iOS (but not Windows yet).
- 12:13 PM Changeset in webkit [179944] by
-
- 3 edits2 adds in trunk
Div having contentEditable and display:flex cannot be edited if it is empty.
https://bugs.webkit.org/show_bug.cgi?id=141218
Reviewed by Ryosuke Niwa.
Source/WebCore:
RenderFlexibleBox should be treated as a candidate for visible position. Visible selection
in an editable area is recalculated whenever an inner value of div is changed. If the inner value
is empty, the visible selection recalculated with DeleteSelectionCommand::m_endingPostion
is not correct. Because RenderBlockFlow is only considered but not RenderFlexibleBox
in Position::isCandidate so that a calculated ending VisiblePosition for an editable div
having "display : flex" goes weird and VisibleSelection is empty accordingly.
Test: fast/events/key-events-in-editable-flexbox.html
- dom/Position.cpp:
(WebCore::Position::isCandidate):
LayoutTests:
This tests if an editable flexbox can be reedited after erasing all texts in it.
- fast/events/key-events-in-editable-flexbox-expected.txt: Added.
- fast/events/key-events-in-editable-flexbox.html: Added.
- 12:12 PM Changeset in webkit [179943] by
-
- 2 edits in trunk
[CMake] Don't set flag fuse-ld on the C/C++ compiler flags, but on the linker flags.
https://bugs.webkit.org/show_bug.cgi?id=141481
Reviewed by Csaba Osztrogonác.
- Source/cmake/OptionsCommon.cmake: Move -fuse-ld=gold to the linker flags.
- 12:09 PM Changeset in webkit [179942] by
-
- 7 edits in trunk/Source
Activate ReliefLogger of a memory pressure handler for linux system.
https://bugs.webkit.org/show_bug.cgi?id=123611
Reviewed by Anders Carlsson.
Source/WebCore:
Put more logs for the time when a system goes under memory pressure or viceversa.
No new tests since no engine behavior changed.
- platform/linux/MemoryPressureHandlerLinux.cpp:
(WebCore::MemoryPressureHandler::waitForMemoryPressureEvent): Wait a memory pressure event
from cgroup in a seperated thread. Once a pressure event happens, respondToMemoryPressure()
would be called to get back some resources.
(WebCore::MemoryPressureHandler::install): Install memoryPressureHandler module
to make it work.
Source/WebKit2:
- Shared/WebProcessCreationParameters.cpp:
(WebKit::WebProcessCreationParameters::WebProcessCreationParameters):
Make the parameter, shouldEnableMemoryPressureReliefLogging shareable with COCOA else systems.
(WebKit::WebProcessCreationParameters::encode): ditto.
(WebKit::WebProcessCreationParameters::decode): ditto.
- Shared/WebProcessCreationParameters.h:
- UIProcess/WebProcessPool.cpp:
(WebKit::WebProcessPool::createNewWebProcess): Set shouldEnableMemoryPressureReliefLogging
to true for linux system.
- WebProcess/WebProcess.cpp:
(WebKit::WebProcess::initializeWebProcess): Pass the shouldEnableMemoryPressureReliefLogging value
to MemoryPressureHandler::ReliefLogger::setLoggingEnabled().
- 11:44 AM Changeset in webkit [179941] by
-
- 2 edits in trunk/Source/WebCore
Final preparations to switch to WEBCORE_EXPORT.
- platform/ios/WebVideoFullscreenInterfaceAVKit.h:
Export the vtable and a function.
- 11:43 AM Changeset in webkit [179940] by
-
- 5 edits in branches/safari-600.1.17-branch/Source
Versioning.
- 11:42 AM Changeset in webkit [179939] by
-
- 1 copy in tags/Safari-600.1.17.8
New Tag.
- 10:42 AM Changeset in webkit [179938] by
-
- 2 edits in branches/safari-600.1.17-branch/Source/WebCore
Merged r174295.
- 10:36 AM Changeset in webkit [179937] by
-
- 2 edits in trunk/Source/WebCore
performance.now can crash if accessed from a window that has navigated
<rdar://problem/16892506>
https://bugs.webkit.org/show_bug.cgi?id=141478
Reviewed by Alexey Proskuryakov.
Test: fast/performance/performance-now-crash-on-navigated-window.html
- page/Performance.cpp:
(WebCore::Performance::now):
Check for a null frame, which can happen when the window has been navigated.
- 10:33 AM Changeset in webkit [179936] by
-
- 1 edit5 adds in trunk/LayoutTests
performance.now can crash if accessed from a window that has navigated
<rdar://problem/16892506>
https://bugs.webkit.org/show_bug.cgi?id=141478
Reviewed by Alexey Proskuryakov.
- fast/performance/performance-now-crash-on-navigated-window-expected.txt: Added.
- fast/performance/performance-now-crash-on-navigated-window.html: Added.
- fast/performance/resources: Added.
- fast/performance/resources/initialFrame.html: Added.
- fast/performance/resources/secondFrame.html: Added.
Add test for calling performance.now() on from a navigated window.
- 10:09 AM Changeset in webkit [179935] by
-
- 5 edits in trunk/Source
Versioning.
- 10:07 AM Changeset in webkit [179934] by
-
- 1 copy in tags/Safari-601.1.18
New tag.
- 10:03 AM Changeset in webkit [179933] by
-
- 7 edits4 adds2 deletes in trunk
URL::setUser and URL::setPass don't percent encode
https://bugs.webkit.org/show_bug.cgi?id=141453
rdar://problem/14844503&16551802&19623145
Reviewed by Darin Adler.
Source/WebCore:
Tests: fast/url/url-credentials-escaping.html
http/tests/xmlhttprequest/basic-auth-credentials-escaping.html
Start adding some code that performs escaping in a way that matches the URL Standard.
Right now, it's only used where we failed to do any escaping at all, and over time,
we'll be moving towards a new implementation.
- html/URLUtils.h:
(WebCore::URLUtils<T>::username):
(WebCore::URLUtils<T>::password):
- platform/URL.cpp:
(WebCore::isSchemeFirstChar):
(WebCore::URL::user):
(WebCore::URL::pass):
(WebCore::URL::encodedUser):
(WebCore::URL::encodedPass):
(WebCore::URL::setUser):
(WebCore::URL::setPass):
(WebCore::encodeWithURLEscapeSequences):
- platform/URL.h:
LayoutTests:
- fast/url/url-credentials-escaping-expected.txt: Added.
- fast/url/url-credentials-escaping.html: Added.
This change is most directly testable via URL API.
- http/tests/xmlhttprequest/basic-auth-credentials-escaping-expected.txt: Added.
- http/tests/xmlhttprequest/basic-auth-credentials-escaping.html: Added.
Verify that this doesn't break XMLHttpRequest authentication.
- fast/dom/DOMURL/invalid-url-getters-expected.txt: Removed.
- fast/dom/DOMURL/invalid-url-getters.html: Removed.
- fast/dom/DOMURL/url-origin-expected.txt:
- fast/dom/DOMURL/url-origin.html:
Removed tests for invalid URLs, there is no such thing with URL API.
- 9:58 AM Changeset in webkit [179932] by
-
- 2 edits in trunk/Source/WebKit2
[GTK][EFL] Suppress a warning of unused params
https://bugs.webkit.org/show_bug.cgi?id=141474
Reviewed by Darin Adler.
Drop unused argument name,
directoryName.
- UIProcess/API/APIWebsiteDataStore.cpp:
(API::WebsiteDataStore::websiteDataDirectoryFileSystemRepresentation):
- 9:41 AM WebKitGTK/SpeedUpBuild edited by
- (diff)
- 9:31 AM WebKitGTK/SpeedUpBuild edited by
- (diff)
- 9:11 AM Changeset in webkit [179931] by
-
- 3 edits in trunk/Tools
run-jsc-stress tests should detect the number of processors on the remote machine too
https://bugs.webkit.org/show_bug.cgi?id=141196
Reviewed by Darin Adler.
- Scripts/run-javascriptcore-tests:
(runJSCStressTests): Pass through the --child-processes argument.
- Scripts/run-jsc-stress-tests: Added determineNumberOfProcessors function
to detect the number of local and remote processors with the same algorithm.
Additionally introduced $numChildProcesses to emphasize it isn't necessarily
same as $numProcessors.
- 9:10 AM Changeset in webkit [179930] by
-
- 2 edits in trunk/Tools
run-jsc-stress-tests should pass JSC_timeout to remote hosts
https://bugs.webkit.org/show_bug.cgi?id=141164
Reviewed by Darin Adler.
Additonally removed IMPORTANT_ENVS, since JSC_timeout was the only one.
- Scripts/run-jsc-stress-tests:
- 9:09 AM Changeset in webkit [179929] by
-
- 4 edits in trunk/Source/WebCore
Add a few more WEBCORE_EXPORT macros for debug and iOS builds.
- page/EventHandler.h:
- page/ios/WebEventRegion.h:
- platform/sql/SQLiteDatabaseTracker.h:
- 8:41 AM Changeset in webkit [179928] by
-
- 2 edits in trunk/Source/WebCore
REGRESSION(r179476): It broke the !ENABLE(PICTURE_SIZES) build
https://bugs.webkit.org/show_bug.cgi?id=141327
Reviewed by Csaba Osztrogonác.
- html/parser/HTMLPreloadScanner.cpp:
(WebCore::TokenPreloadScanner::StartTagScanner::processAttributes):
Put #if around use of m_sizesAttribute.
- 8:39 AM WebKitGTK/SpeedUpBuild edited by
- (diff)
- 8:35 AM WebKitGTK/SpeedUpBuild edited by
- (diff)
- 8:12 AM Changeset in webkit [179927] by
-
- 4 edits in trunk/Source/bmalloc
bmalloc buildfix on 32 bit Linux (x86/ARM)
https://bugs.webkit.org/show_bug.cgi?id=141472
Reviewed by Gyuyoung Kim.
- bmalloc/Algorithm.h:
(bmalloc::roundUpToMultipleOf):
- bmalloc/FixedVector.h:
(bmalloc::FixedVector::clear):
- bmalloc/Sizes.h:
(bmalloc::Sizes::sizeClass):
- 7:34 AM Changeset in webkit [179926] by
-
- 3 edits2 deletes in trunk
Unreviewed, rolling out r179921.
https://bugs.webkit.org/show_bug.cgi?id=141473
caused about 30 test failures on yosemite and mavericks
(Requested by alexchristensen on #webkit).
Reverted changeset:
"Div having contentEditable and display:flex cannot be edited
if it is empty."
https://bugs.webkit.org/show_bug.cgi?id=141218
http://trac.webkit.org/changeset/179921
- 7:05 AM Changeset in webkit [179925] by
-
- 2 edits in trunk/Source/WebKit
[Win] Unreviewed build fix.
- WebKit.vcxproj/WebKitExportGenerator/WebKitExports.def.in:
- 5:54 AM Changeset in webkit [179924] by
-
- 6 edits10 adds in trunk/LayoutTests
Unreviewed GTK+ Gardening 11 February.
Patch by Marcos Chavarría Teijeiro <chavarria1991@gmail.com> on 2015-02-11
- platform/gtk/TestExpectations:
- platform/gtk/editing/pasteboard/onpaste-text-html-expected.txt: Rebaselined after r179168.
- platform/gtk/fast/css/outline-auto-empty-rects-expected.txt: Added. Rebaselined after r177774.
- platform/gtk/fast/css/pseudo-invalid-form-invalidation-optimization-expected.txt: Added. Rebaselined after r177664.
- platform/gtk/fast/css/pseudo-valid-form-invalidation-optimization-expected.txt: Added. Rebaselined after r177664.
- platform/gtk/fast/forms/basic-buttons-expected.txt: Rebaselined after r178078.
- platform/gtk/fast/line-grid/line-align-right-edges-expected.txt: Rebaselined after r177128.
- platform/gtk/fast/sub-pixel/selection/selection-rect-in-sub-pixel-table-expected.txt: Added. Rebaselined after r177774.
- platform/gtk/fast/table/022-expected.txt: Added. Rebaselined after r177774.
- platform/gtk/fast/text/wbr-pre-expected.txt: Added. Rebaselined after r177774.
- platform/gtk/fast/text/whitespace/019-expected.txt: Added. Rebaselined after r177774.
- platform/gtk/fast/text/whitespace/023-expected.txt: Added. Rebaselined after r177774.
- platform/gtk/svg/W3C-SVG-1.1/fonts-glyph-03-t-expected.txt: Added. Rebaselined after r177490.
- platform/gtk/svg/custom/glyph-selection-arabic-forms-expected.txt: Added. Rebaselined after r177774.
- platform/gtk/svg/custom/glyph-selection-lang-attribute-expected.txt: Rebaselined after r177490.
- 4:50 AM WebKitGTK/Gardening/Calendar edited by
- (diff)
- 4:15 AM Changeset in webkit [179923] by
-
- 20 edits1 add in trunk
[EFL][GTK] Use bmalloc instead of tcmalloc
https://bugs.webkit.org/show_bug.cgi?id=140162
Reviewed by Carlos Garcia Campos.
.:
Add bmalloc directory to build list.
- CMakeLists.txt: Define BMALLOC_DIR directory.
- Source/CMakeLists.txt: Add bmalloc path to sub-directories list.
Source/bmalloc:
Support to use bmalloc on EFL and GTK ports.
- CMakeLists.txt: Added.
- bmalloc/Allocator.cpp:
(bmalloc::Allocator::allocate):
Fix unused return value caused by posix_memalign().
- bmalloc/AsyncTask.h:
- bmalloc/BoundaryTag.h:
(bmalloc::BoundaryTag::clear):
- bmalloc/Chunk.h:
Change Traits::Page with Traits::PageType in order to fix
-fpermitive build error on EFL and GTK port.
- bmalloc/EndTag.h:
(bmalloc::EndTag::operator=):
- bmalloc/Line.h: ditto.
- bmalloc/MediumTraits.h:
- bmalloc/Page.h: ditto.
- bmalloc/PerThread.h:
EFL port doesn't support has_include definition yet.
Define HAVE_PTHREAD_MACHDEP_H according to check if has_include is supported.
- bmalloc/SmallTraits.h: ditto.
- bmalloc/VMAllocate.h:
(bmalloc::vmDeallocatePhysicalPages):
(bmalloc::vmAllocatePhysicalPages):
- bmalloc/Vector.h:
(bmalloc::Vector<T>::push):
(bmalloc::Vector<T>::reallocateBuffer):
Source/WTF:
- wtf/CMakeLists.txt:
- wtf/FastMalloc.cpp:
Enable bmalloc on GTK ports, but EFL port will enable it when bmalloc supports
memory statistics.
- 2:02 AM WebKitGTK/2.4.x edited by
- (diff)
- 1:03 AM Changeset in webkit [179922] by
-
- 9 edits4 adds in trunk
[GTK] Add default color chooser implementation using GtkColorChooserDialog
https://bugs.webkit.org/show_bug.cgi?id=141392
Reviewed by Gustavo Noronha Silva.
.:
Enable INPUT_TYPE_COLOR by default for GTK+ port.
- Source/cmake/OptionsGTK.cmake:
Source/WebKit2:
- PlatformGTK.cmake: Add new files to compilation.
- UIProcess/API/gtk/PageClientImpl.cpp:
(WebKit::PageClientImpl::createColorPicker): Call WebColorPickerGtk::create().
- UIProcess/gtk/WebColorPickerGtk.cpp: Added.
(WebKit::WebColorPickerGtk::create):
(WebKit::WebColorPickerGtk::WebColorPickerGtk):
(WebKit::WebColorPickerGtk::~WebColorPickerGtk):
(WebKit::WebColorPickerGtk::cancel): Set initial color.
(WebKit::WebColorPickerGtk::endPicker): Destroy the color chooser dialog.
(WebKit::WebColorPickerGtk::didChooseColor): Notify the client.
(WebKit::WebColorPickerGtk::colorChooserDialogRGBAChangedCallback):
Update the selected color.
(WebKit::WebColorPickerGtk::colorChooserDialogResponseCallback):
Cancel or finish the operation depending on the dialog response.
(WebKit::WebColorPickerGtk::showColorPicker): Create a
GtkColorChooserDialog to handle the color picker operation.
- UIProcess/gtk/WebColorPickerGtk.h: Added.
(WebKit::WebColorPickerGtk::initialColor):
Tools:
Enable INPUT_TYPE_COLOR by default for GTK+ port.
- Scripts/webkitperl/FeatureList.pm:
LayoutTests:
Unskip input color tests.
- platform/gtk/TestExpectations:
- platform/gtk/fast/forms/color/input-appearance-color-expected.txt: Added.
- 12:51 AM Changeset in webkit [179921] by
-
- 3 edits2 adds in trunk
Div having contentEditable and display:flex cannot be edited if it is empty.
https://bugs.webkit.org/show_bug.cgi?id=141218
Reviewed by Ryosuke Niwa.
Source/WebCore:
RenderFlexibleBox should be treated as a candidate for visible position. Visible selection
in an editable area is recalculated whenever an inner value of div is changed. If the inner value
is empty, the visible selection recalculated with DeleteSelectionCommand::m_endingPostion
is not correct. Because RenderBlockFlow is only considered but not RenderFlexibleBox
in Position::isCandidate so that a calculated ending VisiblePosition for an editable div
having "display : flex" goes weird and VisibleSelection is empty accordingly.
Test: fast/events/key-events-in-editable-flexbox.html
- dom/Position.cpp:
(WebCore::Position::isCandidate):
LayoutTests:
This tests if an editable flexbox can be reedited after erasing all texts in it.
- fast/events/key-events-in-editable-flexbox-expected.txt: Added.
- fast/events/key-events-in-editable-flexbox.html: Added.
- 12:36 AM Changeset in webkit [179920] by
-
- 2 edits in trunk/LayoutTests
http/tests/cache/disk-cache-validation.html generates a lot of Perl errors
https://bugs.webkit.org/show_bug.cgi?id=141393
Reviewed by Darin Adler.
- http/tests/cache/resources/generate-response.cgi:
- 12:25 AM WebKitGTK/Gardening/Calendar edited by
- (diff)
- 12:01 AM Changeset in webkit [179919] by
-
- 17 edits in trunk/Source
Unreviewed, rolling out r179910.
https://bugs.webkit.org/show_bug.cgi?id=141464
Caused assertions on debug bots (Requested by ap on #webkit).
Reverted changeset:
"[WK2] Add logging to validate the network cache efficacy
(Part 2)"
https://bugs.webkit.org/show_bug.cgi?id=141345
http://trac.webkit.org/changeset/179910
Feb 10, 2015:
- 10:16 PM Changeset in webkit [179918] by
-
- 2 edits in trunk/LayoutTests
Unreviewed, EFL gardening. Tests of fast/ruby needs to have new baseline since r172874.
- platform/efl/TestExpectations:
- 10:02 PM Changeset in webkit [179917] by
-
- 2 edits in trunk/Source/WebCore
Optimize MutableStyleProperties::removePropertiesInSet()
https://bugs.webkit.org/show_bug.cgi?id=141460
Reviewed by Andreas Kling.
Optimize MutableStyleProperties::removePropertiesInSet() by doing an
in-place removal of the vector properties, using the new and efficient
Vector::removalAllMatching().
I see a ~11% speed-up on CSS/CSSPropertySetterGetter.html performance
test.
This change was inspired by the following Blink revision:
https://src.chromium.org/viewvc/blink?view=rev&revision=189387
Test: PerformanceTests/CSS/CSSPropertySetterGetter.html
- css/StyleProperties.cpp:
(WebCore::MutableStyleProperties::removePropertiesInSet):
- 9:48 PM Changeset in webkit [179916] by
-
- 2 edits in trunk/Source/WebCore
[Win] Fix debug build after r179807.
https://bugs.webkit.org/show_bug.cgi?id=141461
Reviewed by Benjamin Poulain.
- dom/TypedElementDescendantIterator.h:
Explicitly call Iterator constructor to reduce complexity when iterating descendantsOfType.
- 8:54 PM Changeset in webkit [179915] by
-
- 2 edits in trunk/Source/WTF
Add the crash-on-failure behavior to bmalloc-based fastMalloc
https://bugs.webkit.org/show_bug.cgi?id=141434
Reviewed by Alexey Proskuryakov.
- wtf/FastMalloc.cpp: Removed unneeded includes.
(WTF::fastMalloc): Added null check and CRASH.
(WTF::fastCalloc): Added overflow checking, null check, and CRASH.
(WTF::fastRealloc): Added null check and CRASH.
(WTF::fastMallocSize): Added FIXME comment.
(WTF::fastMallocGoodSize): Added FIXME comment.
(WTF::fastAlignedMalloc): Added null check and CRASH.
(WTF::tryFastMalloc): Changed to call bmalloc directly instead of
calling fastMalloc, since fastMalloc will now crash on failure.
(WTF::tryFastRealloc): Changed to call bmalloc directly instead of
calling fastRealloc, since fastRealloc will now crash on failure.
(WTF::tryFastCalloc): Added overflow checking.
(WTF::fastMallocStatistics): Added FIXME comment.
- 8:01 PM Changeset in webkit [179914] by
-
- 2 edits in trunk/LayoutTests
Unreviewed EFL gardening. Some tests of imported/mozilla/svg/ have been failed since r177330.
Mark those tests to ImageOnlyFailure.
- platform/efl/TestExpectations:
- 6:45 PM Changeset in webkit [179913] by
-
- 6 edits in trunk/Websites/perf.webkit.org
New perf dashboard shouldn't always show outliners
https://bugs.webkit.org/show_bug.cgi?id=141445
Reviewed by Chris Dumez.
Use the simple moving average with an average difference envelope to compute the y-axis range to show
to avoid expanding it spuriously to show one off outlier.
- public/v2/app.js:
(App.Pane): Don't show the full y-axis range by default.
(App.Pane._computeChartData): Use the first strategies for the moving average and the enveloping if
one is not specified by the user but without showing them in the charts.
(App.Pane._computeMovingAverage): Takes moving average and enveloping strategies as arguments instead
of retrieving via chosenMovingAverageStrategy and chosenEnvelopingStrategy.
(App.ChartsController._parsePaneList): Added showFullYAxis as a query string argument to each pane.
(App.ChartsController._serializePaneList): Ditto.
- public/v2/chart-pane.css: Added a CSS rule for when y-axis is clickable.
- public/v2/index.html: Pass in showFullYAxis as an argument to the main interactive chart.
- public/v2/interactive-chart.js:
(App.InteractiveChartComponent._constructGraphIfPossible): Add an event listener on y-axis labels when
the chart is interactive so that toggle showFullYAxis. Also hide the moving average and/or the envelope
if they are not specified by the user (i.e. only used to adjust y-axis range).
(App.InteractiveChartComponent._updateDomain): Don't exit early if y-axis domains are different even if
x-axis domain remained the same. Without this change, the charts would never redraw.
(App.InteractiveChartComponent._minMaxForAllTimeSeries): Use the moving average instead of the current
time series to compute the y-axis range if showFullYAxis is false. When showFullYAxis is true, expand
y-axis all the way down to 0 or the minimum value in the current time series whichever is smaller.
- public/v2/js/statistics.js:
(Statistics.MovingAverageStrategies): Use a wider window in Simple Moving Average by default.
- 6:41 PM Changeset in webkit [179912] by
-
- 7 edits2 adds in trunk/Source/JavaScriptCore
Split FTL::JSCall into the part that knows about call inline caching and the part that interacts with LLVM patchpoints
https://bugs.webkit.org/show_bug.cgi?id=141455
Reviewed by Mark Lam.
The newly introduced FTL::JSCallBase can be used to build other things, like the FTL portion
of https://bugs.webkit.org/show_bug.cgi?id=141332.
- CMakeLists.txt:
- JavaScriptCore.vcxproj/JavaScriptCore.vcxproj:
- JavaScriptCore.xcodeproj/project.pbxproj:
- bytecode/CallLinkInfo.h:
(JSC::CallLinkInfo::specializationKindFor):
(JSC::CallLinkInfo::specializationKind):
- ftl/FTLJSCall.cpp:
(JSC::FTL::JSCall::JSCall):
(JSC::FTL::JSCall::emit): Deleted.
(JSC::FTL::JSCall::link): Deleted.
- ftl/FTLJSCall.h:
- ftl/FTLJSCallBase.cpp: Added.
(JSC::FTL::JSCallBase::JSCallBase):
(JSC::FTL::JSCallBase::emit):
(JSC::FTL::JSCallBase::link):
- ftl/FTLJSCallBase.h: Added.
- 6:39 PM Changeset in webkit [179911] by
-
- 2 edits in trunk/Source/WebInspectorUI
Web Inspector: Better handle unknown release versions
https://bugs.webkit.org/show_bug.cgi?id=141454
Patch by Joseph Pecoraro <Joseph Pecoraro> on 2015-02-10
Reviewed by Timothy Hatcher.
- UserInterface/Base/Platform.js:
If given an unknown Mac OS X version, at least parse
the version number in WebInspector.Platform.
- 6:31 PM Changeset in webkit [179910] by
-
- 17 edits in trunk/Source
[WK2] Add logging to validate the network cache efficacy (Part 2)
https://bugs.webkit.org/show_bug.cgi?id=141345
Source/WebCore:
Reviewed by Antti Koivisto.
Add a few more diagnostic logging keys for the network cache efficacy
logging.
Source/WebKit2:
<rdar://problem/19632080>
Reviewed by Antti Koivisto.
Add diagnostic logging messages to validate the network cache efficacy.
The following 4 messages are added:
- networkCache / retrieval / success
- networkCache / retrieval / unhandledRequestFailure
- networkCache / retrieval / noLongerInCacheFailure
- networkCache / retrieval / unusableCachedEntryFailure
The messages are sent via IPC from the NetworkProcess to the UIProcess,
where the WebPageProxy code already handles diagnostic messages sent by
the WebProcess.
- 6:08 PM Changeset in webkit [179909] by
-
- 4 edits in trunk/Source/WebKit2
Clients need the ability to hook into immediate action lifecycle
https://bugs.webkit.org/show_bug.cgi?id=141449
Reviewed by Tim Horton.
- UIProcess/API/Cocoa/WKViewPrivate.h:
Expose new methods.
- UIProcess/API/mac/WKView.mm:
(-[WKView _prepareForImmediateActionAnimation]):
(-[WKView _cancelImmediateActionAnimation]):
(-[WKView _completeImmediateActionAnimation]):
Stub implementations.
- UIProcess/mac/WKImmediateActionController.mm:
(-[WKImmediateActionController immediateActionRecognizerWillPrepare:]):
Send -_prepareForImmediateActionAnimation.
(-[WKImmediateActionController immediateActionRecognizerDidCancelAnimation:]):
Send -_cancelImmediateActionAnimation.
(-[WKImmediateActionController immediateActionRecognizerDidCompleteAnimation:]):
Send -_completeImmediateActionAnimation.
- 5:50 PM Changeset in webkit [179908] by
-
- 5 edits in branches/safari-600.5-branch/Source
Versioning.
- 5:42 PM Changeset in webkit [179907] by
-
- 3 edits in trunk/Source/WebCore
Unreviewed, rolling out r179896.
https://bugs.webkit.org/show_bug.cgi?id=141452
broke linking debug builds (Requested by thorton on #webkit).
Reverted changeset:
"Stop using WebCore.exp.in on Mac and iOS."
https://bugs.webkit.org/show_bug.cgi?id=141413
http://trac.webkit.org/changeset/179896
- 5:33 PM Changeset in webkit [179906] by
-
- 16 edits2 copies in branches/safari-600.5-branch
Merged r179722. rdar://problem/19670548
- 5:06 PM Changeset in webkit [179905] by
-
- 3 edits in branches/safari-600.1.4.15-branch/Source/WebCore
Merged r179843. rdar://problem/19432919
- 4:54 PM Changeset in webkit [179904] by
-
- 2 edits in trunk/Source/WebKit2
<rdar://problem/19770264> Starting a navigation gesture dismisses the keyboard
Reviewed by Tim Horton.
- UIProcess/ios/ViewGestureControllerIOS.mm:
(WebKit::ViewGestureController::beginSwipeGesture): Reordered the view hierarchy changes,
such that m_liveSwipeView will not momentarily move out of the window, because that causes
the window’s first responder to be set to nil (if it was a descendant of m_liveSwipView).
- 4:19 PM Changeset in webkit [179903] by
-
- 20 edits in trunk/Source/WebCore
Fix bindings tests after r179886.
- bindings/scripts/test/ObjC/DOMFloat64Array.h:
- bindings/scripts/test/ObjC/DOMTestActiveDOMObject.h:
- bindings/scripts/test/ObjC/DOMTestCallback.h:
- bindings/scripts/test/ObjC/DOMTestCustomNamedGetter.h:
- bindings/scripts/test/ObjC/DOMTestEventConstructor.h:
- bindings/scripts/test/ObjC/DOMTestEventTarget.h:
- bindings/scripts/test/ObjC/DOMTestException.h:
- bindings/scripts/test/ObjC/DOMTestGenerateIsReachable.h:
- bindings/scripts/test/ObjC/DOMTestInterface.h:
- bindings/scripts/test/ObjC/DOMTestMediaQueryListListener.h:
- bindings/scripts/test/ObjC/DOMTestNamedConstructor.h:
- bindings/scripts/test/ObjC/DOMTestNode.h:
- bindings/scripts/test/ObjC/DOMTestNondeterministic.h:
- bindings/scripts/test/ObjC/DOMTestObj.h:
- bindings/scripts/test/ObjC/DOMTestOverloadedConstructors.h:
- bindings/scripts/test/ObjC/DOMTestSerializedScriptValueInterface.h:
- bindings/scripts/test/ObjC/DOMTestTypedefs.h:
- bindings/scripts/test/ObjC/DOMattribute.h:
- bindings/scripts/test/ObjC/DOMreadonly.h:
- 4:16 PM Changeset in webkit [179902] by
-
- 4 edits in branches/safari-600.1.4.15-branch/Source/WebKit2
Merged r179578. rdar://problem/19709204
- 4:13 PM Changeset in webkit [179901] by
-
- 27 edits in branches/safari-600.5-branch
Merged r177849. rdar://problem/19739101
- 4:12 PM Changeset in webkit [179900] by
-
- 3 edits in branches/safari-600.1.4.15-branch/Source/JavaScriptCore
- 4:10 PM Changeset in webkit [179899] by
-
- 2 edits in trunk/LayoutTests
REGRESSION(?): inspector/css/selector-dynamic-specificity.html sometimes crashes
https://bugs.webkit.org/show_bug.cgi?id=141118
- platform/mac-wk2/TestExpectations: Skpping the test, I'm concerned that it may
be affecting other tests.
- 4:06 PM Changeset in webkit [179898] by
-
- 2 edits in trunk/Source/JavaScriptCore
Unreviewed, fix build.
- jit/CCallHelpers.h:
(JSC::CCallHelpers::setupArgumentsWithExecState):
- 4:05 PM Changeset in webkit [179897] by
-
- 4 edits in branches/safari-600.5-branch/Source/WebCore
Merged r178298. rdar://problem/19526083
- 4:00 PM Changeset in webkit [179896] by
-
- 3 edits in trunk/Source/WebCore
Stop using WebCore.exp.in on Mac and iOS.
https://bugs.webkit.org/show_bug.cgi?id=141413
Reviewed by Dan Bates.
- Configurations/Base.xcconfig:
Make symbols hidden by default unless exported by WEBCORE_EXPORT macros.
- platform/PlatformExportMacros.h:
Start using WEBCORE_EXPORT on Mac and iOS (but not Windows yet).
- 3:59 PM Changeset in webkit [179895] by
-
- 2 edits in trunk/Source/WebCore
Add another assertion to help track down crash in DocumentLoader::stopLoadingForPolicyChange()
https://bugs.webkit.org/show_bug.cgi?id=141447
<rdar://problem/13811738>
Reviewed by Alexey Proskuryakov.
Add another assertion to help track down crash in
DocumentLoader::stopLoadingForPolicyChange().
The trace seems to hint that frameLoader() returns null when
stopLoadingForPolicyChange() is called. frameLoader() can only return
null after DocumentLoader::detachFromFrame() has been called.
Also, stopLoadingForPolicyChange() here is called from the
DocumentLoader::continueAfterContentPolicy() policy callback which
requires m_waitingForContentPolicy to be true. Therefore, we should
assert that m_waitingForContentPolicy is false when m_frame is cleared
in DocumentLoader::detachFromFrame().
- loader/DocumentLoader.cpp:
(WebCore::DocumentLoader::detachFromFrame):
- 3:52 PM Changeset in webkit [179894] by
-
- 2 edits in branches/safari-600.1.4.15-branch/Source/WebCore
Merged r178571. rdar://problem/19670953
- 3:49 PM Changeset in webkit [179893] by
-
- 3 edits4 copies in branches/safari-600.1.4.15-branch
Merged r178490. rdar://problem/19709240
- 3:45 PM Changeset in webkit [179892] by
-
- 3 edits2 copies in branches/safari-600.1.4.15-branch
Merged r178426. rdar://problem/19564743
- 3:42 PM Changeset in webkit [179891] by
-
- 2 edits in branches/safari-600.1.4.15-branch/Source/WebCore
Merged r176240. rdar://problem/19670909
- 3:33 PM Changeset in webkit [179890] by
-
- 2 edits in branches/safari-600.1.4.15-branch/Source/JavaScriptCore
Merged r179187. rdar://problem/19670901
- 3:27 PM Changeset in webkit [179889] by
-
- 3 edits in trunk/Source/WebInspectorUI
Web Inspector: Make collapsed sidebars undraggable
https://bugs.webkit.org/show_bug.cgi?id=140664
Patch by Nikita Vasilyev <Nikita Vasilyev> on 2015-02-10
Reviewed by Timothy Hatcher.
- UserInterface/Views/Sidebar.css:
(.sidebar.collapsed):
(.sidebar.collapsed.right > .resizer): Deleted.
(.sidebar.collapsed.left > .resizer): Deleted.
- UserInterface/Views/Sidebar.js:
(WebInspector.Sidebar):
(WebInspector.Sidebar.prototype._resizerMouseDown):
(WebInspector.Sidebar.prototype._resizerMouseMoved):
(WebInspector.Sidebar.prototype._resizerDoubleClicked): Deleted.
- 3:24 PM Changeset in webkit [179888] by
-
- 4 edits in branches/safari-600.1.4.15-branch/Source/JavaScriptCore
Merged r176083. rdar://problem/19670901
- 3:16 PM Changeset in webkit [179887] by
-
- 18 edits1 add in trunk/Source/JavaScriptCore
op_call_varargs should only load the length once
https://bugs.webkit.org/show_bug.cgi?id=141440
rdar://problem/19761683
Reviewed by Michael Saboff.
Refactors the pair of calls that set up the varargs frame so that the first call returns the
length, and the second call uses the length returned by the first one. It turns out that this
gave me an opportunity to shorten a lot of the code.
- interpreter/Interpreter.cpp:
(JSC::sizeFrameForVarargs):
(JSC::loadVarargs):
(JSC::setupVarargsFrame):
(JSC::setupVarargsFrameAndSetThis):
- interpreter/Interpreter.h:
(JSC::calleeFrameForVarargs):
- jit/CCallHelpers.h:
(JSC::CCallHelpers::setupArgumentsWithExecState):
- jit/JIT.h:
- jit/JITCall.cpp:
(JSC::JIT::compileSetupVarargsFrame):
- jit/JITCall32_64.cpp:
(JSC::JIT::compileSetupVarargsFrame):
- jit/JITInlines.h:
(JSC::JIT::callOperation):
- jit/JITOperations.cpp:
- jit/JITOperations.h:
- jit/SetupVarargsFrame.cpp:
(JSC::emitSetVarargsFrame):
(JSC::emitSetupVarargsFrameFastCase):
- jit/SetupVarargsFrame.h:
- llint/LLIntSlowPaths.cpp:
(JSC::LLInt::LLINT_SLOW_PATH_DECL):
- runtime/Arguments.cpp:
(JSC::Arguments::copyToArguments):
- runtime/Arguments.h:
- runtime/JSArray.cpp:
(JSC::JSArray::copyToArguments):
- runtime/JSArray.h:
- runtime/VM.h:
- tests/stress/call-varargs-length-effects.js: Added.
(foo):
(bar):
- 3:07 PM Changeset in webkit [179886] by
-
- 26 edits in trunk/Source/WebCore
Add more WEBCORE_EXPORT macros for iOS.
https://bugs.webkit.org/show_bug.cgi?id=141430
Reviewed by Tim Horton.
- bindings/objc/WebScriptObject.h:
- bindings/scripts/CodeGeneratorObjC.pm:
(GenerateHeader):
- dom/Event.h:
- dom/Node.h:
- editing/FrameSelection.h:
- html/HTMLFormControlElement.h:
- html/HTMLMediaSession.h:
- page/Frame.h:
- page/FrameView.h:
- platform/ScrollView.h:
- platform/ios/PasteboardIOS.mm:
- platform/ios/wak/WAKAppKitStubs.h:
- platform/ios/wak/WAKClipView.h:
- platform/ios/wak/WAKScrollView.h:
- platform/ios/wak/WAKView.h:
- platform/ios/wak/WAKWindow.h:
- platform/ios/wak/WAKWindow.mm:
- platform/ios/wak/WKContentObservation.h:
- platform/ios/wak/WKGraphics.h:
- platform/ios/wak/WebCoreThread.h:
- platform/ios/wak/WebCoreThread.mm:
- platform/ios/wak/WebCoreThreadRun.h:
- platform/ios/wak/WebCoreThreadSystemInterface.h:
- platform/network/ios/QuickLook.h:
- platform/text/TextBreakIterator.h:
- 3:02 PM Changeset in webkit [179885] by
-
- 4 edits in trunk/Source/WebKit2
Hit testing is misplaced when selecting on wikipedia page.
https://bugs.webkit.org/show_bug.cgi?id=141410
rdar://problem/19768211
Reviewed by Simon Fraser.
When loading a new document into the same WebPage object, we need
to clear the assisted node, since there is no guarantee that the
node will be blurred. The bug was caused by the use of a stale
assisted node to constrain the point where the gesture occurs.
The node was still valid, since the previous page was in the cache.
I've added a ASSERT to check that, when constraining the point to
node rectangle, the node is in the same document as the one we
are performing hit test on. When the frame is detached, we clear
the assisted node if the node is part of the document is the frame
being detached.
- WebProcess/WebCoreSupport/WebFrameLoaderClient.cpp:
(WebKit::WebFrameLoaderClient::detachedFromParent2):
- WebProcess/WebPage/WebPage.h:
- WebProcess/WebPage/ios/WebPageIOS.mm:
(WebKit::constrainPoint):
(WebKit::WebPage::resetAssistedNodeForFrame):
- 2:40 PM Changeset in webkit [179884] by
-
- 4 edits1 copy in branches/safari-600.5-branch
Merged r176899. rdar://problem/19739097
- 2:35 PM Changeset in webkit [179883] by
-
- 7 edits2 adds in trunk
text-underline-position: under is broken
https://bugs.webkit.org/show_bug.cgi?id=141400
<rdar://problem/18569583>
Reviewed by Simon Fraser.
Source/WebCore:
Added fast/text/text-underline-position-under.html
- rendering/InlineFlowBox.cpp:
(WebCore::InlineFlowBox::computeMaxLogicalBottom):
(WebCore::InlineFlowBox::computeMaxLogicalTop): Deleted.
- rendering/InlineFlowBox.h:
Switch to using the bottom to compute the offset. Using the top is incorrect, since
the heights of boxes can vary.
Fix a bug where the y() of the box was being used instead of the logical value, making the result
wrong for vertical text.
- rendering/RootInlineBox.cpp:
(WebCore::RootInlineBox::maxLogicalBottom):
(WebCore::RootInlineBox::maxLogicalTop): Deleted.
Switch to using the bottom instead of the top. Make sure the root box contributes its own bottom,
since the old code just ignored the root's placement.
- rendering/RootInlineBox.h:
- style/InlineTextBoxStyle.cpp:
(WebCore::computeUnderlineOffset):
Call the bottom function now instead of the top.
LayoutTests:
- fast/text/text-underline-position-under-expected.html: Added.
- fast/text/text-underline-position-under.html: Added.
text-underline-position: under is broken
https://bugs.webkit.org/show_bug.cgi?id=141400
<rdar://problem/18569583>
Reviewed by Simon Fraser.
Source/WebCore:
Added fast/text/text-underline-position-under.html
- rendering/InlineFlowBox.cpp:
(WebCore::InlineFlowBox::computeMaxLogicalBottom):
(WebCore::InlineFlowBox::computeMaxLogicalTop): Deleted.
- rendering/InlineFlowBox.h:
Switch to using the bottom to compute the offset. Using the top is incorrect, since
the heights of boxes can vary.
Fix a bug where the y() of the box was being used instead of the logical value, making the result
wrong for vertical text.
- rendering/RootInlineBox.cpp:
(WebCore::RootInlineBox::maxLogicalBottom):
(WebCore::RootInlineBox::maxLogicalTop): Deleted.
Switch to using the bottom instead of the top. Make sure the root box contributes its own bottom,
since the old code just ignored the root's placement.
- rendering/RootInlineBox.h:
- style/InlineTextBoxStyle.cpp:
(WebCore::computeUnderlineOffset):
Call the bottom function now instead of the top.
LayoutTests:
- fast/text/text-underline-position-under-expected.html: Added.
- fast/text/text-underline-position-under.html: Added.
text-underline-position: under is broken
https://bugs.webkit.org/show_bug.cgi?id=141400
<rdar://problem/18569583>
Reviewed by Simon Fraser.
Source/WebCore:
Added fast/text/text-underline-position-under.html
- rendering/InlineFlowBox.cpp:
(WebCore::InlineFlowBox::computeMaxLogicalBottom):
(WebCore::InlineFlowBox::computeMaxLogicalTop): Deleted.
- rendering/InlineFlowBox.h:
Switch to using the bottom to compute the offset. Using the top is incorrect, since
the heights of boxes can vary.
Fix a bug where the y() of the box was being used instead of the logical value, making the result
wrong for vertical text.
- rendering/RootInlineBox.cpp:
(WebCore::RootInlineBox::maxLogicalBottom):
(WebCore::RootInlineBox::maxLogicalTop): Deleted.
Switch to using the bottom instead of the top. Make sure the root box contributes its own bottom,
since the old code just ignored the root's placement.
- rendering/RootInlineBox.h:
- style/InlineTextBoxStyle.cpp:
(WebCore::computeUnderlineOffset):
Call the bottom function now instead of the top.
LayoutTests:
- fast/text/text-underline-position-under-expected.html: Added.
- fast/text/text-underline-position-under.html: Added.
text-underline-position: under is broken
https://bugs.webkit.org/show_bug.cgi?id=141400
<rdar://problem/18569583>
Reviewed by Simon Fraser.
Source/WebCore:
Added fast/text/text-underline-position-under.html
- rendering/InlineFlowBox.cpp:
(WebCore::InlineFlowBox::computeMaxLogicalBottom):
(WebCore::InlineFlowBox::computeMaxLogicalTop): Deleted.
- rendering/InlineFlowBox.h:
Switch to using the bottom to compute the offset. Using the top is incorrect, since
the heights of boxes can vary.
Fix a bug where the y() of the box was being used instead of the logical value, making the result
wrong for vertical text.
- rendering/RootInlineBox.cpp:
(WebCore::RootInlineBox::maxLogicalBottom):
(WebCore::RootInlineBox::maxLogicalTop): Deleted.
Switch to using the bottom instead of the top. Make sure the root box contributes its own bottom,
since the old code just ignored the root's placement.
- rendering/RootInlineBox.h:
- style/InlineTextBoxStyle.cpp:
(WebCore::computeUnderlineOffset):
Call the bottom function now instead of the top.
LayoutTests:
- fast/text/text-underline-position-under-expected.html: Added.
- fast/text/text-underline-position-under.html: Added.
text-underline-position: under is broken
https://bugs.webkit.org/show_bug.cgi?id=141400
<rdar://problem/18569583>
Reviewed by Simon Fraser.
Source/WebCore:
Added fast/text/text-underline-position-under.html
- rendering/InlineFlowBox.cpp:
(WebCore::InlineFlowBox::computeMaxLogicalBottom):
(WebCore::InlineFlowBox::computeMaxLogicalTop): Deleted.
- rendering/InlineFlowBox.h:
Switch to using the bottom to compute the offset. Using the top is incorrect, since
the heights of boxes can vary.
Fix a bug where the y() of the box was being used instead of the logical value, making the result
wrong for vertical text.
- rendering/RootInlineBox.cpp:
(WebCore::RootInlineBox::maxLogicalBottom):
(WebCore::RootInlineBox::maxLogicalTop): Deleted.
Switch to using the bottom instead of the top. Make sure the root box contributes its own bottom,
since the old code just ignored the root's placement.
- rendering/RootInlineBox.h:
- style/InlineTextBoxStyle.cpp:
(WebCore::computeUnderlineOffset):
Call the bottom function now instead of the top.
LayoutTests:
- fast/text/text-underline-position-under-expected.html: Added.
- fast/text/text-underline-position-under.html: Added.
text-underline-position: under is broken
https://bugs.webkit.org/show_bug.cgi?id=141400
<rdar://problem/18569583>
Reviewed by Simon Fraser.
Source/WebCore:
Added fast/text/text-underline-position-under.html
- rendering/InlineFlowBox.cpp:
(WebCore::InlineFlowBox::computeMaxLogicalBottom):
(WebCore::InlineFlowBox::computeMaxLogicalTop): Deleted.
- rendering/InlineFlowBox.h:
Switch to using the bottom to compute the offset. Using the top is incorrect, since
the heights of boxes can vary.
Fix a bug where the y() of the box was being used instead of the logical value, making the result
wrong for vertical text.
- rendering/RootInlineBox.cpp:
(WebCore::RootInlineBox::maxLogicalBottom):
(WebCore::RootInlineBox::maxLogicalTop): Deleted.
Switch to using the bottom instead of the top. Make sure the root box contributes its own bottom,
since the old code just ignored the root's placement.
- rendering/RootInlineBox.h:
- style/InlineTextBoxStyle.cpp:
(WebCore::computeUnderlineOffset):
Call the bottom function now instead of the top.
LayoutTests:
- fast/text/text-underline-position-under-expected.html: Added.
- fast/text/text-underline-position-under.html: Added.
text-underline-position: under is broken
https://bugs.webkit.org/show_bug.cgi?id=141400
<rdar://problem/18569583>
Reviewed by Simon Fraser.
Source/WebCore:
Added fast/text/text-underline-position-under.html
- rendering/InlineFlowBox.cpp:
(WebCore::InlineFlowBox::computeMaxLogicalBottom):
(WebCore::InlineFlowBox::computeMaxLogicalTop): Deleted.
- rendering/InlineFlowBox.h:
Switch to using the bottom to compute the offset. Using the top is incorrect, since
the heights of boxes can vary.
Fix a bug where the y() of the box was being used instead of the logical value, making the result
wrong for vertical text.
- rendering/RootInlineBox.cpp:
(WebCore::RootInlineBox::maxLogicalBottom):
(WebCore::RootInlineBox::maxLogicalTop): Deleted.
Switch to using the bottom instead of the top. Make sure the root box contributes its own bottom,
since the old code just ignored the root's placement.
- rendering/RootInlineBox.h:
- style/InlineTextBoxStyle.cpp:
(WebCore::computeUnderlineOffset):
Call the bottom function now instead of the top.
LayoutTests:
- fast/text/text-underline-position-under-expected.html: Added.
- fast/text/text-underline-position-under.html: Added.
text-underline-position: under is broken
https://bugs.webkit.org/show_bug.cgi?id=141400
<rdar://problem/18569583>
Reviewed by Simon Fraser.
Source/WebCore:
Added fast/text/text-underline-position-under.html
- rendering/InlineFlowBox.cpp:
(WebCore::InlineFlowBox::computeMaxLogicalBottom):
(WebCore::InlineFlowBox::computeMaxLogicalTop): Deleted.
- rendering/InlineFlowBox.h:
Switch to using the bottom to compute the offset. Using the top is incorrect, since
the heights of boxes can vary.
Fix a bug where the y() of the box was being used instead of the logical value, making the result
wrong for vertical text.
- rendering/RootInlineBox.cpp:
(WebCore::RootInlineBox::maxLogicalBottom):
(WebCore::RootInlineBox::maxLogicalTop): Deleted.
Switch to using the bottom instead of the top. Make sure the root box contributes its own bottom,
since the old code just ignored the root's placement.
- rendering/RootInlineBox.h:
- style/InlineTextBoxStyle.cpp:
(WebCore::computeUnderlineOffset):
Call the bottom function now instead of the top.
LayoutTests:
- fast/text/text-underline-position-under-expected.html: Added.
- fast/text/text-underline-position-under.html: Added.
text-underline-position: under is broken
https://bugs.webkit.org/show_bug.cgi?id=141400
<rdar://problem/18569583>
Reviewed by Simon Fraser.
Source/WebCore:
Added fast/text/text-underline-position-under.html
- rendering/InlineFlowBox.cpp:
(WebCore::InlineFlowBox::computeMaxLogicalBottom):
(WebCore::InlineFlowBox::computeMaxLogicalTop): Deleted.
- rendering/InlineFlowBox.h:
Switch to using the bottom to compute the offset. Using the top is incorrect, since
the heights of boxes can vary.
Fix a bug where the y() of the box was being used instead of the logical value, making the result
wrong for vertical text.
- rendering/RootInlineBox.cpp:
(WebCore::RootInlineBox::maxLogicalBottom):
(WebCore::RootInlineBox::maxLogicalTop): Deleted.
Switch to using the bottom instead of the top. Make sure the root box contributes its own bottom,
since the old code just ignored the root's placement.
- rendering/RootInlineBox.h:
- style/InlineTextBoxStyle.cpp:
(WebCore::computeUnderlineOffset):
Call the bottom function now instead of the top.
LayoutTests:
- fast/text/text-underline-position-under-expected.html: Added.
- fast/text/text-underline-position-under.html: Added.
text-underline-position: under is broken
https://bugs.webkit.org/show_bug.cgi?id=141400
<rdar://problem/18569583>
Reviewed by Simon Fraser.
Source/WebCore:
Added fast/text/text-underline-position-under.html
- rendering/InlineFlowBox.cpp:
(WebCore::InlineFlowBox::computeMaxLogicalBottom):
(WebCore::InlineFlowBox::computeMaxLogicalTop): Deleted.
- rendering/InlineFlowBox.h:
Switch to using the bottom to compute the offset. Using the top is incorrect, since
the heights of boxes can vary.
Fix a bug where the y() of the box was being used instead of the logical value, making the result
wrong for vertical text.
- rendering/RootInlineBox.cpp:
(WebCore::RootInlineBox::maxLogicalBottom):
(WebCore::RootInlineBox::maxLogicalTop): Deleted.
Switch to using the bottom instead of the top. Make sure the root box contributes its own bottom,
since the old code just ignored the root's placement.
- rendering/RootInlineBox.h:
- style/InlineTextBoxStyle.cpp:
(WebCore::computeUnderlineOffset):
Call the bottom function now instead of the top.
LayoutTests:
- fast/text/text-underline-position-under-expected.html: Added.
- fast/text/text-underline-position-under.html: Added.
text-underline-position: under is broken
https://bugs.webkit.org/show_bug.cgi?id=141400
<rdar://problem/18569583>
Reviewed by Simon Fraser.
Source/WebCore:
Added fast/text/text-underline-position-under.html
- rendering/InlineFlowBox.cpp:
(WebCore::InlineFlowBox::computeMaxLogicalBottom):
(WebCore::InlineFlowBox::computeMaxLogicalTop): Deleted.
- rendering/InlineFlowBox.h:
Switch to using the bottom to compute the offset. Using the top is incorrect, since
the heights of boxes can vary.
Fix a bug where the y() of the box was being used instead of the logical value, making the result
wrong for vertical text.
- rendering/RootInlineBox.cpp:
(WebCore::RootInlineBox::maxLogicalBottom):
(WebCore::RootInlineBox::maxLogicalTop): Deleted.
Switch to using the bottom instead of the top. Make sure the root box contributes its own bottom,
since the old code just ignored the root's placement.
- rendering/RootInlineBox.h:
- style/InlineTextBoxStyle.cpp:
(WebCore::computeUnderlineOffset):
Call the bottom function now instead of the top.
LayoutTests:
- fast/text/text-underline-position-under-expected.html: Added.
- fast/text/text-underline-position-under.html: Added.
text-underline-position: under is broken
https://bugs.webkit.org/show_bug.cgi?id=141400
<rdar://problem/18569583>
Reviewed by Simon Fraser.
Source/WebCore:
Added fast/text/text-underline-position-under.html
- rendering/InlineFlowBox.cpp:
(WebCore::InlineFlowBox::computeMaxLogicalBottom):
(WebCore::InlineFlowBox::computeMaxLogicalTop): Deleted.
- rendering/InlineFlowBox.h:
Switch to using the bottom to compute the offset. Using the top is incorrect, since
the heights of boxes can vary.
Fix a bug where the y() of the box was being used instead of the logical value, making the result
wrong for vertical text.
- rendering/RootInlineBox.cpp:
(WebCore::RootInlineBox::maxLogicalBottom):
(WebCore::RootInlineBox::maxLogicalTop): Deleted.
Switch to using the bottom instead of the top. Make sure the root box contributes its own bottom,
since the old code just ignored the root's placement.
- rendering/RootInlineBox.h:
- style/InlineTextBoxStyle.cpp:
(WebCore::computeUnderlineOffset):
Call the bottom function now instead of the top.
LayoutTests:
- fast/text/text-underline-position-under-expected.html: Added.
- fast/text/text-underline-position-under.html: Added.
text-underline-position: under is broken
https://bugs.webkit.org/show_bug.cgi?id=141400
<rdar://problem/18569583>
Reviewed by Simon Fraser.
Source/WebCore:
Added fast/text/text-underline-position-under.html
- rendering/InlineFlowBox.cpp:
(WebCore::InlineFlowBox::computeMaxLogicalBottom):
(WebCore::InlineFlowBox::computeMaxLogicalTop): Deleted.
- rendering/InlineFlowBox.h:
Switch to using the bottom to compute the offset. Using the top is incorrect, since
the heights of boxes can vary.
Fix a bug where the y() of the box was being used instead of the logical value, making the result
wrong for vertical text.
- rendering/RootInlineBox.cpp:
(WebCore::RootInlineBox::maxLogicalBottom):
(WebCore::RootInlineBox::maxLogicalTop): Deleted.
Switch to using the bottom instead of the top. Make sure the root box contributes its own bottom,
since the old code just ignored the root's placement.
- rendering/RootInlineBox.h:
- style/InlineTextBoxStyle.cpp:
(WebCore::computeUnderlineOffset):
Call the bottom function now instead of the top.
LayoutTests:
- fast/text/text-underline-position-under-expected.html: Added.
- fast/text/text-underline-position-under.html: Added.
text-underline-position: under is broken
https://bugs.webkit.org/show_bug.cgi?id=141400
<rdar://problem/18569583>
Reviewed by Simon Fraser.
Source/WebCore:
Added fast/text/text-underline-position-under.html
- rendering/InlineFlowBox.cpp:
(WebCore::InlineFlowBox::computeMaxLogicalBottom):
(WebCore::InlineFlowBox::computeMaxLogicalTop): Deleted.
- rendering/InlineFlowBox.h:
Switch to using the bottom to compute the offset. Using the top is incorrect, since
the heights of boxes can vary.
Fix a bug where the y() of the box was being used instead of the logical value, making the result
wrong for vertical text.
- rendering/RootInlineBox.cpp:
(WebCore::RootInlineBox::maxLogicalBottom):
(WebCore::RootInlineBox::maxLogicalTop): Deleted.
Switch to using the bottom instead of the top. Make sure the root box contributes its own bottom,
since the old code just ignored the root's placement.
- rendering/RootInlineBox.h:
- style/InlineTextBoxStyle.cpp:
(WebCore::computeUnderlineOffset):
Call the bottom function now instead of the top.
LayoutTests:
- fast/text/text-underline-position-under-expected.html: Added.
- fast/text/text-underline-position-under.html: Added.
text-underline-position: under is broken
https://bugs.webkit.org/show_bug.cgi?id=141400
<rdar://problem/18569583>
Reviewed by Simon Fraser.
Source/WebCore:
Added fast/text/text-underline-position-under.html
- rendering/InlineFlowBox.cpp:
(WebCore::InlineFlowBox::computeMaxLogicalBottom):
(WebCore::InlineFlowBox::computeMaxLogicalTop): Deleted.
- rendering/InlineFlowBox.h:
Switch to using the bottom to compute the offset. Using the top is incorrect, since
the heights of boxes can vary.
Fix a bug where the y() of the box was being used instead of the logical value, making the result
wrong for vertical text.
- rendering/RootInlineBox.cpp:
(WebCore::RootInlineBox::maxLogicalBottom):
(WebCore::RootInlineBox::maxLogicalTop): Deleted.
Switch to using the bottom instead of the top. Make sure the root box contributes its own bottom,
since the old code just ignored the root's placement.
- rendering/RootInlineBox.h:
- style/InlineTextBoxStyle.cpp:
(WebCore::computeUnderlineOffset):
Call the bottom function now instead of the top.
LayoutTests:
- fast/text/text-underline-position-under-expected.html: Added.
- fast/text/text-underline-position-under.html: Added.
text-underline-position: under is broken
https://bugs.webkit.org/show_bug.cgi?id=141400
<rdar://problem/18569583>
Reviewed by Simon Fraser.
Source/WebCore:
Added fast/text/text-underline-position-under.html
- rendering/InlineFlowBox.cpp:
(WebCore::InlineFlowBox::computeMaxLogicalBottom):
(WebCore::InlineFlowBox::computeMaxLogicalTop): Deleted.
- rendering/InlineFlowBox.h:
Switch to using the bottom to compute the offset. Using the top is incorrect, since
the heights of boxes can vary.
Fix a bug where the y() of the box was being used instead of the logical value, making the result
wrong for vertical text.
- rendering/RootInlineBox.cpp:
(WebCore::RootInlineBox::maxLogicalBottom):
(WebCore::RootInlineBox::maxLogicalTop): Deleted.
Switch to using the bottom instead of the top. Make sure the root box contributes its own bottom,
since the old code just ignored the root's placement.
- rendering/RootInlineBox.h:
- style/InlineTextBoxStyle.cpp:
(WebCore::computeUnderlineOffset):
Call the bottom function now instead of the top.
LayoutTests:
- fast/text/text-underline-position-under-expected.html: Added.
- fast/text/text-underline-position-under.html: Added.
text-underline-position: under is broken
https://bugs.webkit.org/show_bug.cgi?id=141400
<rdar://problem/18569583>
Reviewed by Simon Fraser.
Source/WebCore:
Added fast/text/text-underline-position-under.html
- rendering/InlineFlowBox.cpp:
(WebCore::InlineFlowBox::computeMaxLogicalBottom):
(WebCore::InlineFlowBox::computeMaxLogicalTop): Deleted.
- rendering/InlineFlowBox.h:
Switch to using the bottom to compute the offset. Using the top is incorrect, since
the heights of boxes can vary.
Fix a bug where the y() of the box was being used instead of the logical value, making the result
wrong for vertical text.
- rendering/RootInlineBox.cpp:
(WebCore::RootInlineBox::maxLogicalBottom):
(WebCore::RootInlineBox::maxLogicalTop): Deleted.
Switch to using the bottom instead of the top. Make sure the root box contributes its own bottom,
since the old code just ignored the root's placement.
- rendering/RootInlineBox.h:
- style/InlineTextBoxStyle.cpp:
(WebCore::computeUnderlineOffset):
Call the bottom function now instead of the top.
LayoutTests:
- fast/text/text-underline-position-under-expected.html: Added.
- fast/text/text-underline-position-under.html: Added.
text-underline-position: under is broken
https://bugs.webkit.org/show_bug.cgi?id=141400
<rdar://problem/18569583>
Reviewed by Simon Fraser.
Source/WebCore:
Added fast/text/text-underline-position-under.html
- rendering/InlineFlowBox.cpp:
(WebCore::InlineFlowBox::computeMaxLogicalBottom):
(WebCore::InlineFlowBox::computeMaxLogicalTop): Deleted.
- rendering/InlineFlowBox.h:
Switch to using the bottom to compute the offset. Using the top is incorrect, since
the heights of boxes can vary.
Fix a bug where the y() of the box was being used instead of the logical value, making the result
wrong for vertical text.
- rendering/RootInlineBox.cpp:
(WebCore::RootInlineBox::maxLogicalBottom):
(WebCore::RootInlineBox::maxLogicalTop): Deleted.
Switch to using the bottom instead of the top. Make sure the root box contributes its own bottom,
since the old code just ignored the root's placement.
- rendering/RootInlineBox.h:
- style/InlineTextBoxStyle.cpp:
(WebCore::computeUnderlineOffset):
Call the bottom function now instead of the top.
LayoutTests:
- fast/text/text-underline-position-under-expected.html: Added.
- fast/text/text-underline-position-under.html: Added.
text-underline-position: under is broken
https://bugs.webkit.org/show_bug.cgi?id=141400
<rdar://problem/18569583>
Reviewed by Simon Fraser.
Source/WebCore:
Added fast/text/text-underline-position-under.html
- rendering/InlineFlowBox.cpp:
(WebCore::InlineFlowBox::computeMaxLogicalBottom):
(WebCore::InlineFlowBox::computeMaxLogicalTop): Deleted.
- rendering/InlineFlowBox.h:
Switch to using the bottom to compute the offset. Using the top is incorrect, since
the heights of boxes can vary.
Fix a bug where the y() of the box was being used instead of the logical value, making the result
wrong for vertical text.
- rendering/RootInlineBox.cpp:
(WebCore::RootInlineBox::maxLogicalBottom):
(WebCore::RootInlineBox::maxLogicalTop): Deleted.
Switch to using the bottom instead of the top. Make sure the root box contributes its own bottom,
since the old code just ignored the root's placement.
- rendering/RootInlineBox.h:
- style/InlineTextBoxStyle.cpp:
(WebCore::computeUnderlineOffset):
Call the bottom function now instead of the top.
LayoutTests:
- fast/text/text-underline-position-under-expected.html: Added.
- fast/text/text-underline-position-under.html: Added.
text-underline-position: under is broken
https://bugs.webkit.org/show_bug.cgi?id=141400
<rdar://problem/18569583>
Reviewed by Simon Fraser.
Source/WebCore:
Added fast/text/text-underline-position-under.html
- rendering/InlineFlowBox.cpp:
(WebCore::InlineFlowBox::computeMaxLogicalBottom):
(WebCore::InlineFlowBox::computeMaxLogicalTop): Deleted.
- rendering/InlineFlowBox.h:
Switch to using the bottom to compute the offset. Using the top is incorrect, since
the heights of boxes can vary.
Fix a bug where the y() of the box was being used instead of the logical value, making the result
wrong for vertical text.
- rendering/RootInlineBox.cpp:
(WebCore::RootInlineBox::maxLogicalBottom):
(WebCore::RootInlineBox::maxLogicalTop): Deleted.
Switch to using the bottom instead of the top. Make sure the root box contributes its own bottom,
since the old code just ignored the root's placement.
- rendering/RootInlineBox.h:
- style/InlineTextBoxStyle.cpp:
(WebCore::computeUnderlineOffset):
Call the bottom function now instead of the top.
LayoutTests:
- fast/text/text-underline-position-under-expected.html: Added.
- fast/text/text-underline-position-under.html: Added.
text-underline-position: under is broken
https://bugs.webkit.org/show_bug.cgi?id=141400
<rdar://problem/18569583>
Reviewed by Simon Fraser.
Source/WebCore:
Added fast/text/text-underline-position-under.html
- rendering/InlineFlowBox.cpp:
(WebCore::InlineFlowBox::computeMaxLogicalBottom):
(WebCore::InlineFlowBox::computeMaxLogicalTop): Deleted.
- rendering/InlineFlowBox.h:
Switch to using the bottom to compute the offset. Using the top is incorrect, since
the heights of boxes can vary.
Fix a bug where the y() of the box was being used instead of the logical value, making the result
wrong for vertical text.
- rendering/RootInlineBox.cpp:
(WebCore::RootInlineBox::maxLogicalBottom):
(WebCore::RootInlineBox::maxLogicalTop): Deleted.
Switch to using the bottom instead of the top. Make sure the root box contributes its own bottom,
since the old code just ignored the root's placement.
- rendering/RootInlineBox.h:
- style/InlineTextBoxStyle.cpp:
(WebCore::computeUnderlineOffset):
Call the bottom function now instead of the top.
LayoutTests:
- fast/text/text-underline-position-under-expected.html: Added.
- fast/text/text-underline-position-under.html: Added.
text-underline-position: under is broken
https://bugs.webkit.org/show_bug.cgi?id=141400
<rdar://problem/18569583>
Reviewed by Simon Fraser.
Source/WebCore:
Added fast/text/text-underline-position-under.html
- rendering/InlineFlowBox.cpp:
(WebCore::InlineFlowBox::computeMaxLogicalBottom):
(WebCore::InlineFlowBox::computeMaxLogicalTop): Deleted.
- rendering/InlineFlowBox.h:
Switch to using the bottom to compute the offset. Using the top is incorrect, since
the heights of boxes can vary.
Fix a bug where the y() of the box was being used instead of the logical value, making the result
wrong for vertical text.
- rendering/RootInlineBox.cpp:
(WebCore::RootInlineBox::maxLogicalBottom):
(WebCore::RootInlineBox::maxLogicalTop): Deleted.
Switch to using the bottom instead of the top. Make sure the root box contributes its own bottom,
since the old code just ignored the root's placement.
- rendering/RootInlineBox.h:
- style/InlineTextBoxStyle.cpp:
(WebCore::computeUnderlineOffset):
Call the bottom function now instead of the top.
LayoutTests:
- fast/text/text-underline-position-under-expected.html: Added.
- fast/text/text-underline-position-under.html: Added.
text-underline-position: under is broken
https://bugs.webkit.org/show_bug.cgi?id=141400
<rdar://problem/18569583>
Reviewed by Simon Fraser.
Source/WebCore:
Added fast/text/text-underline-position-under.html
- rendering/InlineFlowBox.cpp:
(WebCore::InlineFlowBox::computeMaxLogicalBottom):
(WebCore::InlineFlowBox::computeMaxLogicalTop): Deleted.
- rendering/InlineFlowBox.h:
Switch to using the bottom to compute the offset. Using the top is incorrect, since
the heights of boxes can vary.
Fix a bug where the y() of the box was being used instead of the logical value, making the result
wrong for vertical text.
- rendering/RootInlineBox.cpp:
(WebCore::RootInlineBox::maxLogicalBottom):
(WebCore::RootInlineBox::maxLogicalTop): Deleted.
Switch to using the bottom instead of the top. Make sure the root box contributes its own bottom,
since the old code just ignored the root's placement.
- rendering/RootInlineBox.h:
- style/InlineTextBoxStyle.cpp:
(WebCore::computeUnderlineOffset):
Call the bottom function now instead of the top.
LayoutTests:
- fast/text/text-underline-position-under-expected.html: Added.
- fast/text/text-underline-position-under.html: Added.
text-underline-position: under is broken
https://bugs.webkit.org/show_bug.cgi?id=141400
<rdar://problem/18569583>
Reviewed by Simon Fraser.
Source/WebCore:
Added fast/text/text-underline-position-under.html
- rendering/InlineFlowBox.cpp:
(WebCore::InlineFlowBox::computeMaxLogicalBottom):
(WebCore::InlineFlowBox::computeMaxLogicalTop): Deleted.
- rendering/InlineFlowBox.h:
Switch to using the bottom to compute the offset. Using the top is incorrect, since
the heights of boxes can vary.
Fix a bug where the y() of the box was being used instead of the logical value, making the result
wrong for vertical text.
- rendering/RootInlineBox.cpp:
(WebCore::RootInlineBox::maxLogicalBottom):
(WebCore::RootInlineBox::maxLogicalTop): Deleted.
Switch to using the bottom instead of the top. Make sure the root box contributes its own bottom,
since the old code just ignored the root's placement.
- rendering/RootInlineBox.h:
- style/InlineTextBoxStyle.cpp:
(WebCore::computeUnderlineOffset):
Call the bottom function now instead of the top.
LayoutTests:
- fast/text/text-underline-position-under-expected.html: Added.
- fast/text/text-underline-position-under.html: Added.
text-underline-position: under is broken
https://bugs.webkit.org/show_bug.cgi?id=141400
<rdar://problem/18569583>
Reviewed by Simon Fraser.
Source/WebCore:
Added fast/text/text-underline-position-under.html
- rendering/InlineFlowBox.cpp:
(WebCore::InlineFlowBox::computeMaxLogicalBottom):
(WebCore::InlineFlowBox::computeMaxLogicalTop): Deleted.
- rendering/InlineFlowBox.h:
Switch to using the bottom to compute the offset. Using the top is incorrect, since
the heights of boxes can vary.
Fix a bug where the y() of the box was being used instead of the logical value, making the result
wrong for vertical text.
- rendering/RootInlineBox.cpp:
(WebCore::RootInlineBox::maxLogicalBottom):
(WebCore::RootInlineBox::maxLogicalTop): Deleted.
Switch to using the bottom instead of the top. Make sure the root box contributes its own bottom,
since the old code just ignored the root's placement.
- rendering/RootInlineBox.h:
- style/InlineTextBoxStyle.cpp:
(WebCore::computeUnderlineOffset):
Call the bottom function now instead of the top.
LayoutTests:
- fast/text/text-underline-position-under-expected.html: Added.
- fast/text/text-underline-position-under.html: Added.
text-underline-position: under is broken
https://bugs.webkit.org/show_bug.cgi?id=141400
<rdar://problem/18569583>
Reviewed by Simon Fraser.
Source/WebCore:
Added fast/text/text-underline-position-under.html
- rendering/InlineFlowBox.cpp:
(WebCore::InlineFlowBox::computeMaxLogicalBottom):
(WebCore::InlineFlowBox::computeMaxLogicalTop): Deleted.
- rendering/InlineFlowBox.h:
Switch to using the bottom to compute the offset. Using the top is incorrect, since
the heights of boxes can vary.
Fix a bug where the y() of the box was being used instead of the logical value, making the result
wrong for vertical text.
- rendering/RootInlineBox.cpp:
(WebCore::RootInlineBox::maxLogicalBottom):
(WebCore::RootInlineBox::maxLogicalTop): Deleted.
Switch to using the bottom instead of the top. Make sure the root box contributes its own bottom,
since the old code just ignored the root's placement.
- rendering/RootInlineBox.h:
- style/InlineTextBoxStyle.cpp:
(WebCore::computeUnderlineOffset):
Call the bottom function now instead of the top.
LayoutTests:
- fast/text/text-underline-position-under-expected.html: Added.
- fast/text/text-underline-position-under.html: Added.
text-underline-position: under is broken
https://bugs.webkit.org/show_bug.cgi?id=141400
<rdar://problem/18569583>
Reviewed by Simon Fraser.
Source/WebCore:
Added fast/text/text-underline-position-under.html
- rendering/InlineFlowBox.cpp:
(WebCore::InlineFlowBox::computeMaxLogicalBottom):
(WebCore::InlineFlowBox::computeMaxLogicalTop): Deleted.
- rendering/InlineFlowBox.h:
Switch to using the bottom to compute the offset. Using the top is incorrect, since
the heights of boxes can vary.
Fix a bug where the y() of the box was being used instead of the logical value, making the result
wrong for vertical text.
- rendering/RootInlineBox.cpp:
(WebCore::RootInlineBox::maxLogicalBottom):
(WebCore::RootInlineBox::maxLogicalTop): Deleted.
Switch to using the bottom instead of the top. Make sure the root box contributes its own bottom,
since the old code just ignored the root's placement.
- rendering/RootInlineBox.h:
- style/InlineTextBoxStyle.cpp:
(WebCore::computeUnderlineOffset):
Call the bottom function now instead of the top.
LayoutTests:
- fast/text/text-underline-position-under-expected.html: Added.
- fast/text/text-underline-position-under.html: Added.
text-underline-position: under is broken
https://bugs.webkit.org/show_bug.cgi?id=141400
<rdar://problem/18569583>
Reviewed by Simon Fraser.
Source/WebCore:
Added fast/text/text-underline-position-under.html
- rendering/InlineFlowBox.cpp:
(WebCore::InlineFlowBox::computeMaxLogicalBottom):
(WebCore::InlineFlowBox::computeMaxLogicalTop): Deleted.
- rendering/InlineFlowBox.h:
Switch to using the bottom to compute the offset. Using the top is incorrect, since
the heights of boxes can vary.
Fix a bug where the y() of the box was being used instead of the logical value, making the result
wrong for vertical text.
- rendering/RootInlineBox.cpp:
(WebCore::RootInlineBox::maxLogicalBottom):
(WebCore::RootInlineBox::maxLogicalTop): Deleted.
Switch to using the bottom instead of the top. Make sure the root box contributes its own bottom,
since the old code just ignored the root's placement.
- rendering/RootInlineBox.h:
- style/InlineTextBoxStyle.cpp:
(WebCore::computeUnderlineOffset):
Call the bottom function now instead of the top.
LayoutTests:
- fast/text/text-underline-position-under-expected.html: Added.
- fast/text/text-underline-position-under.html: Added.
text-underline-position: under is broken
https://bugs.webkit.org/show_bug.cgi?id=141400
<rdar://problem/18569583>
Reviewed by Simon Fraser.
Source/WebCore:
Added fast/text/text-underline-position-under.html
- rendering/InlineFlowBox.cpp:
(WebCore::InlineFlowBox::computeMaxLogicalBottom):
(WebCore::InlineFlowBox::computeMaxLogicalTop): Deleted.
- rendering/InlineFlowBox.h:
Switch to using the bottom to compute the offset. Using the top is incorrect, since
the heights of boxes can vary.
Fix a bug where the y() of the box was being used instead of the logical value, making the result
wrong for vertical text.
- rendering/RootInlineBox.cpp:
(WebCore::RootInlineBox::maxLogicalBottom):
(WebCore::RootInlineBox::maxLogicalTop): Deleted.
Switch to using the bottom instead of the top. Make sure the root box contributes its own bottom,
since the old code just ignored the root's placement.
- rendering/RootInlineBox.h:
- style/InlineTextBoxStyle.cpp:
(WebCore::computeUnderlineOffset):
Call the bottom function now instead of the top.
LayoutTests:
- fast/text/text-underline-position-under-expected.html: Added.
- fast/text/text-underline-position-under.html: Added.
text-underline-position: under is broken
https://bugs.webkit.org/show_bug.cgi?id=141400
<rdar://problem/18569583>
Reviewed by Simon Fraser.
Source/WebCore:
Added fast/text/text-underline-position-under.html
- rendering/InlineFlowBox.cpp:
(WebCore::InlineFlowBox::computeMaxLogicalBottom):
(WebCore::InlineFlowBox::computeMaxLogicalTop): Deleted.
- rendering/InlineFlowBox.h:
Switch to using the bottom to compute the offset. Using the top is incorrect, since
the heights of boxes can vary.
Fix a bug where the y() of the box was being used instead of the logical value, making the result
wrong for vertical text.
- rendering/RootInlineBox.cpp:
(WebCore::RootInlineBox::maxLogicalBottom):
(WebCore::RootInlineBox::maxLogicalTop): Deleted.
Switch to using the bottom instead of the top. Make sure the root box contributes its own bottom,
since the old code just ignored the root's placement.
- rendering/RootInlineBox.h:
- style/InlineTextBoxStyle.cpp:
(WebCore::computeUnderlineOffset):
Call the bottom function now instead of the top.
LayoutTests:
- fast/text/text-underline-position-under-expected.html: Added.
- fast/text/text-underline-position-under.html: Added.
text-underline-position: under is broken
https://bugs.webkit.org/show_bug.cgi?id=141400
<rdar://problem/18569583>
Reviewed by Simon Fraser.
Source/WebCore:
Added fast/text/text-underline-position-under.html
- rendering/InlineFlowBox.cpp:
(WebCore::InlineFlowBox::computeMaxLogicalBottom):
(WebCore::InlineFlowBox::computeMaxLogicalTop): Deleted.
- rendering/InlineFlowBox.h:
Switch to using the bottom to compute the offset. Using the top is incorrect, since
the heights of boxes can vary.
Fix a bug where the y() of the box was being used instead of the logical value, making the result
wrong for vertical text.
- rendering/RootInlineBox.cpp:
(WebCore::RootInlineBox::maxLogicalBottom):
(WebCore::RootInlineBox::maxLogicalTop): Deleted.
Switch to using the bottom instead of the top. Make sure the root box contributes its own bottom,
since the old code just ignored the root's placement.
- rendering/RootInlineBox.h:
- style/InlineTextBoxStyle.cpp:
(WebCore::computeUnderlineOffset):
Call the bottom function now instead of the top.
LayoutTests:
- fast/text/text-underline-position-under-expected.html: Added.
- fast/text/text-underline-position-under.html: Added.
text-underline-position: under is broken
https://bugs.webkit.org/show_bug.cgi?id=141400
<rdar://problem/18569583>
Reviewed by Simon Fraser.
Source/WebCore:
Added fast/text/text-underline-position-under.html
- rendering/InlineFlowBox.cpp:
(WebCore::InlineFlowBox::computeMaxLogicalBottom):
(WebCore::InlineFlowBox::computeMaxLogicalTop): Deleted.
- rendering/InlineFlowBox.h:
Switch to using the bottom to compute the offset. Using the top is incorrect, since
the heights of boxes can vary.
Fix a bug where the y() of the box was being used instead of the logical value, making the result
wrong for vertical text.
- rendering/RootInlineBox.cpp:
(WebCore::RootInlineBox::maxLogicalBottom):
(WebCore::RootInlineBox::maxLogicalTop): Deleted.
Switch to using the bottom instead of the top. Make sure the root box contributes its own bottom,
since the old code just ignored the root's placement.
- rendering/RootInlineBox.h:
- style/InlineTextBoxStyle.cpp:
(WebCore::computeUnderlineOffset):
Call the bottom function now instead of the top.
LayoutTests:
- fast/text/text-underline-position-under-expected.html: Added.
- fast/text/text-underline-position-under.html: Added.
text-underline-position: under is broken
https://bugs.webkit.org/show_bug.cgi?id=141400
<rdar://problem/18569583>
Reviewed by Simon Fraser.
Source/WebCore:
Added fast/text/text-underline-position-under.html
- rendering/InlineFlowBox.cpp:
(WebCore::InlineFlowBox::computeMaxLogicalBottom):
(WebCore::InlineFlowBox::computeMaxLogicalTop): Deleted.
- rendering/InlineFlowBox.h:
Switch to using the bottom to compute the offset. Using the top is incorrect, since
the heights of boxes can vary.
Fix a bug where the y() of the box was being used instead of the logical value, making the result
wrong for vertical text.
- rendering/RootInlineBox.cpp:
(WebCore::RootInlineBox::maxLogicalBottom):
(WebCore::RootInlineBox::maxLogicalTop): Deleted.
Switch to using the bottom instead of the top. Make sure the root box contributes its own bottom,
since the old code just ignored the root's placement.
- rendering/RootInlineBox.h:
- style/InlineTextBoxStyle.cpp:
(WebCore::computeUnderlineOffset):
Call the bottom function now instead of the top.
LayoutTests:
- fast/text/text-underline-position-under-expected.html: Added.
- fast/text/text-underline-position-under.html: Added.
text-underline-position: under is broken
https://bugs.webkit.org/show_bug.cgi?id=141400
<rdar://problem/18569583>
Reviewed by Simon Fraser.
Source/WebCore:
Added fast/text/text-underline-position-under.html
- rendering/InlineFlowBox.cpp:
(WebCore::InlineFlowBox::computeMaxLogicalBottom):
(WebCore::InlineFlowBox::computeMaxLogicalTop): Deleted.
- rendering/InlineFlowBox.h:
Switch to using the bottom to compute the offset. Using the top is incorrect, since
the heights of boxes can vary.
Fix a bug where the y() of the box was being used instead of the logical value, making the result
wrong for vertical text.
- rendering/RootInlineBox.cpp:
(WebCore::RootInlineBox::maxLogicalBottom):
(WebCore::RootInlineBox::maxLogicalTop): Deleted.
Switch to using the bottom instead of the top. Make sure the root box contributes its own bottom,
since the old code just ignored the root's placement.
- rendering/RootInlineBox.h:
- style/InlineTextBoxStyle.cpp:
(WebCore::computeUnderlineOffset):
Call the bottom function now instead of the top.
LayoutTests:
- fast/text/text-underline-position-under-expected.html: Added.
- fast/text/text-underline-position-under.html: Added.
text-underline-position: under is broken
https://bugs.webkit.org/show_bug.cgi?id=141400
<rdar://problem/18569583>
Reviewed by Simon Fraser.
Source/WebCore:
Added fast/text/text-underline-position-under.html
- rendering/InlineFlowBox.cpp:
(WebCore::InlineFlowBox::computeMaxLogicalBottom):
(WebCore::InlineFlowBox::computeMaxLogicalTop): Deleted.
- rendering/InlineFlowBox.h:
Switch to using the bottom to compute the offset. Using the top is incorrect, since
the heights of boxes can vary.
Fix a bug where the y() of the box was being used instead of the logical value, making the result
wrong for vertical text.
- rendering/RootInlineBox.cpp:
(WebCore::RootInlineBox::maxLogicalBottom):
(WebCore::RootInlineBox::maxLogicalTop): Deleted.
Switch to using the bottom instead of the top. Make sure the root box contributes its own bottom,
since the old code just ignored the root's placement.
- rendering/RootInlineBox.h:
- style/InlineTextBoxStyle.cpp:
(WebCore::computeUnderlineOffset):
Call the bottom function now instead of the top.
LayoutTests:
- fast/text/text-underline-position-under-expected.html: Added.
- fast/text/text-underline-position-under.html: Added.
text-underline-position: under is broken
https://bugs.webkit.org/show_bug.cgi?id=141400
<rdar://problem/18569583>
Reviewed by Simon Fraser.
Source/WebCore:
Added fast/text/text-underline-position-under.html
- rendering/InlineFlowBox.cpp:
(WebCore::InlineFlowBox::computeMaxLogicalBottom):
(WebCore::InlineFlowBox::computeMaxLogicalTop): Deleted.
- rendering/InlineFlowBox.h:
Switch to using the bottom to compute the offset. Using the top is incorrect, since
the heights of boxes can vary.
Fix a bug where the y() of the box was being used instead of the logical value, making the result
wrong for vertical text.
- rendering/RootInlineBox.cpp:
(WebCore::RootInlineBox::maxLogicalBottom):
(WebCore::RootInlineBox::maxLogicalTop): Deleted.
Switch to using the bottom instead of the top. Make sure the root box contributes its own bottom,
since the old code just ignored the root's placement.
- rendering/RootInlineBox.h:
- style/InlineTextBoxStyle.cpp:
(WebCore::computeUnderlineOffset):
Call the bottom function now instead of the top.
LayoutTests:
- fast/text/text-underline-position-under-expected.html: Added.
- fast/text/text-underline-position-under.html: Added.
text-underline-position: under is broken
https://bugs.webkit.org/show_bug.cgi?id=141400
<rdar://problem/18569583>
Reviewed by Simon Fraser.
Source/WebCore:
Added fast/text/text-underline-position-under.html
- rendering/InlineFlowBox.cpp:
(WebCore::InlineFlowBox::computeMaxLogicalBottom):
(WebCore::InlineFlowBox::computeMaxLogicalTop): Deleted.
- rendering/InlineFlowBox.h:
Switch to using the bottom to compute the offset. Using the top is incorrect, since
the heights of boxes can vary.
Fix a bug where the y() of the box was being used instead of the logical value, making the result
wrong for vertical text.
- rendering/RootInlineBox.cpp:
(WebCore::RootInlineBox::maxLogicalBottom):
(WebCore::RootInlineBox::maxLogicalTop): Deleted.
Switch to using the bottom instead of the top. Make sure the root box contributes its own bottom,
since the old code just ignored the root's placement.
- rendering/RootInlineBox.h:
- style/InlineTextBoxStyle.cpp:
(WebCore::computeUnderlineOffset):
Call the bottom function now instead of the top.
LayoutTests:
- fast/text/text-underline-position-under-expected.html: Added.
- fast/text/text-underline-position-under.html: Added.
text-underline-position: under is broken
https://bugs.webkit.org/show_bug.cgi?id=141400
<rdar://problem/18569583>
Reviewed by Simon Fraser.
Source/WebCore:
Added fast/text/text-underline-position-under.html
- rendering/InlineFlowBox.cpp:
(WebCore::InlineFlowBox::computeMaxLogicalBottom):
(WebCore::InlineFlowBox::computeMaxLogicalTop): Deleted.
- rendering/InlineFlowBox.h:
Switch to using the bottom to compute the offset. Using the top is incorrect, since
the heights of boxes can vary.
Fix a bug where the y() of the box was being used instead of the logical value, making the result
wrong for vertical text.
- rendering/RootInlineBox.cpp:
(WebCore::RootInlineBox::maxLogicalBottom):
(WebCore::RootInlineBox::maxLogicalTop): Deleted.
Switch to using the bottom instead of the top. Make sure the root box contributes its own bottom,
since the old code just ignored the root's placement.
- rendering/RootInlineBox.h:
- style/InlineTextBoxStyle.cpp:
(WebCore::computeUnderlineOffset):
Call the bottom function now instead of the top.
LayoutTests:
- fast/text/text-underline-position-under-expected.html: Added.
- fast/text/text-underline-position-under.html: Added.
text-underline-position: under is broken
https://bugs.webkit.org/show_bug.cgi?id=141400
<rdar://problem/18569583>
Reviewed by Simon Fraser.
Source/WebCore:
Added fast/text/text-underline-position-under.html
- rendering/InlineFlowBox.cpp:
(WebCore::InlineFlowBox::computeMaxLogicalBottom):
(WebCore::InlineFlowBox::computeMaxLogicalTop): Deleted.
- rendering/InlineFlowBox.h:
Switch to using the bottom to compute the offset. Using the top is incorrect, since
the heights of boxes can vary.
Fix a bug where the y() of the box was being used instead of the logical value, making the result
wrong for vertical text.
- rendering/RootInlineBox.cpp:
(WebCore::RootInlineBox::maxLogicalBottom):
(WebCore::RootInlineBox::maxLogicalTop): Deleted.
Switch to using the bottom instead of the top. Make sure the root box contributes its own bottom,
since the old code just ignored the root's placement.
- rendering/RootInlineBox.h:
- style/InlineTextBoxStyle.cpp:
(WebCore::computeUnderlineOffset):
Call the bottom function now instead of the top.
LayoutTests:
- fast/text/text-underline-position-under-expected.html: Added.
- fast/text/text-underline-position-under.html: Added.
text-underline-position: under is broken
https://bugs.webkit.org/show_bug.cgi?id=141400
<rdar://problem/18569583>
Reviewed by Simon Fraser.
Source/WebCore:
Added fast/text/text-underline-position-under.html
- rendering/InlineFlowBox.cpp:
(WebCore::InlineFlowBox::computeMaxLogicalBottom):
(WebCore::InlineFlowBox::computeMaxLogicalTop): Deleted.
- rendering/InlineFlowBox.h:
Switch to using the bottom to compute the offset. Using the top is incorrect, since
the heights of boxes can vary.
Fix a bug where the y() of the box was being used instead of the logical value, making the result
wrong for vertical text.
- rendering/RootInlineBox.cpp:
(WebCore::RootInlineBox::maxLogicalBottom):
(WebCore::RootInlineBox::maxLogicalTop): Deleted.
Switch to using the bottom instead of the top. Make sure the root box contributes its own bottom,
since the old code just ignored the root's placement.
- rendering/RootInlineBox.h:
- style/InlineTextBoxStyle.cpp:
(WebCore::computeUnderlineOffset):
Call the bottom function now instead of the top.
LayoutTests:
- fast/text/text-underline-position-under-expected.html: Added.
- fast/text/text-underline-position-under.html: Added.
text-underline-position: under is broken
https://bugs.webkit.org/show_bug.cgi?id=141400
<rdar://problem/18569583>
Reviewed by Simon Fraser.
Source/WebCore:
Added fast/text/text-underline-position-under.html
- rendering/InlineFlowBox.cpp:
(WebCore::InlineFlowBox::computeMaxLogicalBottom):
(WebCore::InlineFlowBox::computeMaxLogicalTop): Deleted.
- rendering/InlineFlowBox.h:
Switch to using the bottom to compute the offset. Using the top is incorrect, since
the heights of boxes can vary.
Fix a bug where the y() of the box was being used instead of the logical value, making the result
wrong for vertical text.
- rendering/RootInlineBox.cpp:
(WebCore::RootInlineBox::maxLogicalBottom):
(WebCore::RootInlineBox::maxLogicalTop): Deleted.
Switch to using the bottom instead of the top. Make sure the root box contributes its own bottom,
since the old code just ignored the root's placement.
- rendering/RootInlineBox.h:
- style/InlineTextBoxStyle.cpp:
(WebCore::computeUnderlineOffset):
Call the bottom function now instead of the top.
LayoutTests:
- fast/text/text-underline-position-under-expected.html: Added.
- fast/text/text-underline-position-under.html: Added.
text-underline-position: under is broken
https://bugs.webkit.org/show_bug.cgi?id=141400
<rdar://problem/18569583>
Reviewed by Simon Fraser.
Source/WebCore:
Added fast/text/text-underline-position-under.html
- rendering/InlineFlowBox.cpp:
(WebCore::InlineFlowBox::computeMaxLogicalBottom):
(WebCore::InlineFlowBox::computeMaxLogicalTop): Deleted.
- rendering/InlineFlowBox.h:
Switch to using the bottom to compute the offset. Using the top is incorrect, since
the heights of boxes can vary.
Fix a bug where the y() of the box was being used instead of the logical value, making the result
wrong for vertical text.
- rendering/RootInlineBox.cpp:
(WebCore::RootInlineBox::maxLogicalBottom):
(WebCore::RootInlineBox::maxLogicalTop): Deleted.
Switch to using the bottom instead of the top. Make sure the root box contributes its own bottom,
since the old code just ignored the root's placement.
- rendering/RootInlineBox.h:
- style/InlineTextBoxStyle.cpp:
(WebCore::computeUnderlineOffset):
Call the bottom function now instead of the top.
LayoutTests:
- fast/text/text-underline-position-under-expected.html: Added.
- fast/text/text-underline-position-under.html: Added.
text-underline-position: under is broken
https://bugs.webkit.org/show_bug.cgi?id=141400
<rdar://problem/18569583>
Reviewed by Simon Fraser.
Source/WebCore:
Added fast/text/text-underline-position-under.html
- rendering/InlineFlowBox.cpp:
(WebCore::InlineFlowBox::computeMaxLogicalBottom):
(WebCore::InlineFlowBox::computeMaxLogicalTop): Deleted.
- rendering/InlineFlowBox.h:
Switch to using the bottom to compute the offset. Using the top is incorrect, since
the heights of boxes can vary.
Fix a bug where the y() of the box was being used instead of the logical value, making the result
wrong for vertical text.
- rendering/RootInlineBox.cpp:
(WebCore::RootInlineBox::maxLogicalBottom):
(WebCore::RootInlineBox::maxLogicalTop): Deleted.
Switch to using the bottom instead of the top. Make sure the root box contributes its own bottom,
since the old code just ignored the root's placement.
- rendering/RootInlineBox.h:
- style/InlineTextBoxStyle.cpp:
(WebCore::computeUnderlineOffset):
Call the bottom function now instead of the top.
LayoutTests:
- fast/text/text-underline-position-under-expected.html: Added.
- fast/text/text-underline-position-under.html: Added.
text-underline-position: under is broken
https://bugs.webkit.org/show_bug.cgi?id=141400
<rdar://problem/18569583>
Reviewed by Simon Fraser.
Source/WebCore:
Added fast/text/text-underline-position-under.html
- rendering/InlineFlowBox.cpp:
(WebCore::InlineFlowBox::computeMaxLogicalBottom):
(WebCore::InlineFlowBox::computeMaxLogicalTop): Deleted.
- rendering/InlineFlowBox.h:
Switch to using the bottom to compute the offset. Using the top is incorrect, since
the heights of boxes can vary.
Fix a bug where the y() of the box was being used instead of the logical value, making the result
wrong for vertical text.
- rendering/RootInlineBox.cpp:
(WebCore::RootInlineBox::maxLogicalBottom):
(WebCore::RootInlineBox::maxLogicalTop): Deleted.
Switch to using the bottom instead of the top. Make sure the root box contributes its own bottom,
since the old code just ignored the root's placement.
- rendering/RootInlineBox.h:
- style/InlineTextBoxStyle.cpp:
(WebCore::computeUnderlineOffset):
Call the bottom function now instead of the top.
LayoutTests:
- fast/text/text-underline-position-under-expected.html: Added.
- fast/text/text-underline-position-under.html: Added.
text-underline-position: under is broken
https://bugs.webkit.org/show_bug.cgi?id=141400
<rdar://problem/18569583>
Reviewed by Simon Fraser.
Source/WebCore:
Added fast/text/text-underline-position-under.html
- rendering/InlineFlowBox.cpp:
(WebCore::InlineFlowBox::computeMaxLogicalBottom):
(WebCore::InlineFlowBox::computeMaxLogicalTop): Deleted.
- rendering/InlineFlowBox.h:
Switch to using the bottom to compute the offset. Using the top is incorrect, since
the heights of boxes can vary.
Fix a bug where the y() of the box was being used instead of the logical value, making the result
wrong for vertical text.
- rendering/RootInlineBox.cpp:
(WebCore::RootInlineBox::maxLogicalBottom):
(WebCore::RootInlineBox::maxLogicalTop): Deleted.
Switch to using the bottom instead of the top. Make sure the root box contributes its own bottom,
since the old code just ignored the root's placement.
- rendering/RootInlineBox.h:
- style/InlineTextBoxStyle.cpp:
(WebCore::computeUnderlineOffset):
Call the bottom function now instead of the top.
LayoutTests:
- fast/text/text-underline-position-under-expected.html: Added.
- fast/text/text-underline-position-under.html: Added.
text-underline-position: under is broken
https://bugs.webkit.org/show_bug.cgi?id=141400
<rdar://problem/18569583>
Reviewed by Simon Fraser.
Source/WebCore:
Added fast/text/text-underline-position-under.html
- rendering/InlineFlowBox.cpp:
(WebCore::InlineFlowBox::computeMaxLogicalBottom):
(WebCore::InlineFlowBox::computeMaxLogicalTop): Deleted.
- rendering/InlineFlowBox.h:
Switch to using the bottom to compute the offset. Using the top is incorrect, since
the heights of boxes can vary.
Fix a bug where the y() of the box was being used instead of the logical value, making the result
wrong for vertical text.
- rendering/RootInlineBox.cpp:
(WebCore::RootInlineBox::maxLogicalBottom):
(WebCore::RootInlineBox::maxLogicalTop): Deleted.
Switch to using the bottom instead of the top. Make sure the root box contributes its own bottom,
since the old code just ignored the root's placement.
- rendering/RootInlineBox.h:
- style/InlineTextBoxStyle.cpp:
(WebCore::computeUnderlineOffset):
Call the bottom function now instead of the top.
LayoutTests:
- fast/text/text-underline-position-under-expected.html: Added.
- fast/text/text-underline-position-under.html: Added.
text-underline-position: under is broken
https://bugs.webkit.org/show_bug.cgi?id=141400
<rdar://problem/18569583>
Reviewed by Simon Fraser.
Source/WebCore:
Added fast/text/text-underline-position-under.html
- rendering/InlineFlowBox.cpp:
(WebCore::InlineFlowBox::computeMaxLogicalBottom):
(WebCore::InlineFlowBox::computeMaxLogicalTop): Deleted.
- rendering/InlineFlowBox.h:
Switch to using the bottom to compute the offset. Using the top is incorrect, since
the heights of boxes can vary.
Fix a bug where the y() of the box was being used instead of the logical value, making the result
wrong for vertical text.
- rendering/RootInlineBox.cpp:
(WebCore::RootInlineBox::maxLogicalBottom):
(WebCore::RootInlineBox::maxLogicalTop): Deleted.
Switch to using the bottom instead of the top. Make sure the root box contributes its own bottom,
since the old code just ignored the root's placement.
- rendering/RootInlineBox.h:
- style/InlineTextBoxStyle.cpp:
(WebCore::computeUnderlineOffset):
Call the bottom function now instead of the top.
LayoutTests:
- fast/text/text-underline-position-under-expected.html: Added.
- fast/text/text-underline-position-under.html: Added.
- 1:59 PM Changeset in webkit [179882] by
-
- 3 edits in trunk/Source/JavaScriptCore
Crash in JSC::FTL::LowerDFGToLLVM::compileCompareStrictEq
https://bugs.webkit.org/show_bug.cgi?id=139398
Reviewed by Filip Pizlo.
Due to CFA analysis, the CompareStrictEq node was determined to be unreachable, but later
was determined to be reachable. When we go to lower to LLVM, the edges for the CompareStrictEq
node are UntypedUse which we can't compile. Fixed this by checking that the IR before
lowering can still be handled by the FTL.
Had to add GetArgument as a node that the FTL can compile as the SSA conversion phase converts
a SetArgument to a GetArgument. Before this change FTL::canCompile() would never see a GetArgument
node. With the check right before lowering, we see this node.
- dfg/DFGPlan.cpp:
(JSC::DFG::Plan::compileInThreadImpl): Added a final FTL::canCompile() check before lowering
to verify that after all the transformations we still have valid IR for the FTL.
- ftl/FTLCapabilities.cpp:
(JSC::FTL::canCompile): Added GetArgument as a node the FTL can compile.
- 1:56 PM Changeset in webkit [179881] by
-
- 2 edits in trunk/Websites/perf.webkit.org
Unreviewed build fix.
- public/v2/app.js:
(set get App.Pane.Ember.Object.extend):
- 1:44 PM Changeset in webkit [179880] by
-
- 2 edits in trunk/Source/WebCore
Add assertion to help track down WebCore::DocumentLoader::stopLoadingForPolicyChange() crash
https://bugs.webkit.org/show_bug.cgi?id=141441
<rdar://problem/13811738>
Reviewed by Alexey Proskuryakov.
Add assertion to help track down a crash in
WebCore::DocumentLoader::stopLoadingForPolicyChange().
- loader/DocumentLoader.cpp:
(WebCore::DocumentLoader::~DocumentLoader):
Make sure the DocumentLoader is not waiting for a content policy
response when it is destroyed. If this were to happen, then the
lambda function passed to PolicyChecker::checkContentPolicy()
would outlive the DocumentLoader. This is an issue because
that lambda function captures [this], which is the DocumentLoader.
This would cause DocumentLoader::continueAfterContentPolicy() to
be called after the DocumentLoader has been destroyed, which would
explain the crash.
- 1:41 PM Changeset in webkit [179879] by
-
- 3 edits in trunk/Source/WebInspectorUI
Web Inspector: CSS Pretty Printing: still fails fails to put space between rgb(...) and numbers in minified gradients
https://bugs.webkit.org/show_bug.cgi?id=141411
Patch by Joseph Pecoraro <Joseph Pecoraro> on 2015-02-10
Reviewed by Timothy Hatcher.
- Tools/PrettyPrinting/CodeMirrorFormatters.js:
- UserInterface/Views/CodeMirrorFormatters.js:
Token actually was "number m-css" so we should check token strings
more robustly, like elsewhere in the formatters.
- 1:39 PM Changeset in webkit [179878] by
-
- 7 edits in trunk/Websites/perf.webkit.org
New perf dashboard should have the ability to overlay moving average with an envelope
https://bugs.webkit.org/show_bug.cgi?id=141438
Reviewed by Andreas Kling.
This patch adds three kinds of moving average strategies and two kinds of enveloping strategies:
Simple Moving Average - The moving average x̄_i of x_i is computed as the arithmetic mean of values
from x_(i - n) though x_(i + m) where n is a non-negative integer and m is a positive integer. It takes
n, backward window size, and m, forward window size, as an argument.
Cumulative Moving Average - x̄_i is computed as the arithmetic mean of all values x_0 though x_i.
That is, x̄_1 = x_1 and x̄_i = ((i - 1) * M_(i - 1) + x_i) / i for all i > 1.
Exponential Moving Average - x̄_i is computed as the weighted average of x_i and x̄_(i - 1) with α as
an argument specifying x_i's weight. To be precise, x̄_1 = x_1 and x̄_i = α * x_i + (α - 1) x̄_(i-1).
Average Difference - The enveloping delta d is computed as the arithmetic mean of the difference
between each x_i and x̄_i.
Moving Average Standard Deviation - d is computed like the standard deviation except the deviation
for each term is measured from the moving average instead of the sample arithmetic mean. i.e. it uses
the average of (x_i - x̄_i)2 as the "sample variance" instead of the conventional (x_i - x̄)2 where
x̄ is the sample mean of all x_i's. This change was necessary since our time series is non-stationary.
Each strategy is cloned for an App.Pane instance so that its parameterList can be configured per pane.
The configuration of the currently chosen strategies is saved in the query string for convenience.
Also added the "stat pane" to choose a moving average strategy and a enveloping strategy in each pane.
- public/v2/app.css: Specify the fill color for all SVG groups in the pane toolbar icons.
- public/v2/app.js:
(App.Pane._fetch): Delegate the creation of 'chartData' to _computeChartData.
(App.Pane.updateStatisticsTools): Added. Clones moving average and enveloping strategies for this pane.
(App.Pane._cloneStrategy): Added. Clones a strategy for a new pane.
(App.Pane._configureStrategy): Added. Finds and configures a strategy from the configuration retrieved
from the query string via ChartsController.
(App.Pane._computeChartData): Added. Creates chartData from fetchedData.
(App.Pane._computeMovingAverage): Added. Computes the moving average and the envelope.
(App.Pane._executeStrategy): Added.
(App.Pane._updateStrategyConfigIfNeeded): Pushes the strategy configurations to the query string via
ChartsController.
(App.ChartsController._parsePaneList): Merged the query string arguments for the range and point
selections, and added two new arguments for the moving average and the enveloping configurations.
(App.ChartsController._serializePaneList): Ditto.
(App.ChartsController._scheduleQueryStringUpdate): Observes strategy configurations.
(App.PaneController.actions.toggleBugsPane): Hides the stat pane.
(App.PaneController.actions.toggleSearchPane): Hides the stat pane.
(App.PaneController.actions.toggleStatPane): Added.
- public/v2/chart-pane.css: Added CSS rules for the new stat pane. Also added .foreground class for the
current (as opposed to baseline and target) time series for when it's the most foreground graph without
moving average and its envelope overlapping on top of it.
- public/v2/index.html: Added the templates for the stat pane and the corresponding icon (Σ).
- public/v2/interactive-chart.js:
(App.InteractiveChartComponent.chartDataDidChange): Unset _totalWidth and _totalHeight to avoid exiting
early inside _updateDimensionsIfNeeded when chartData changes after the initial layout.
(App.InteractiveChartComponent.didInsertElement): Attach event listeners here instead of inside
_constructGraphIfPossible since that could be called multiple times on the same SVG element.
(App.InteractiveChartComponent._constructGraphIfPossible): Clear down the old SVG element we created
but don't bother removing individual paths and circles. Added the code to show the moving average time
series when there is one. Also add "foreground" class on SVG elements for the current time series when
we're not showing the moving average. chart-pane.css has been updated to "dim down" the current time
series when "foreground" is not set.
(App.InteractiveChartComponent._minMaxForAllTimeSeries): Take the moving average time series into
account when computing the y-axis range.
(App.InteractiveChartComponent._brushChanged): Removed 'selectionIsLocked' argument as it's useless.
- public/v2/js/statistics.js:
(Statistics.MovingAverageStrategies): Added.
(Statistics.EnvelopingStrategies): Added.
- 12:27 PM Changeset in webkit [179877] by
-
- 3 edits2 adds in trunk
REGRESSION (r168046): Crash in WebCore::InlineBox::renderer / WebCore::RenderFlowThread::checkLinesConsistency
https://bugs.webkit.org/show_bug.cgi?id=133462
Reviewed by David Hyatt.
RenderFlowThread::m_lineToRegionMap stores pointers to the root inlineboxes in the block flow.
Normally root inlineboxes remove themselves from this map in their dtors. However when collapsing an anonymous block,
we detach the inline tree first and destroy them after. The detached root boxes can't access
the flowthread containing block and we end up with dangling pointers in this map.
Call removeFlowChildInfo() before detaching the subtree to ensure proper pointer removal.
Source/WebCore:
Test: fast/multicol/newmulticol/crash-when-switching-to-floating.html
- rendering/RenderBlock.cpp:
(WebCore::RenderBlock::collapseAnonymousBoxChild):
LayoutTests:
- fast/multicol/newmulticol/crash-when-switching-to-floating-expected.txt: Added.
- fast/multicol/newmulticol/crash-when-switching-to-floating.html: Added.
- 12:26 PM Changeset in webkit [179876] by
-
- 5 edits in branches/safari-600.1.4.15-branch/Source
Versioning.
- 11:48 AM Changeset in webkit [179875] by
-
- 2 edits in trunk/Source/JavaScriptCore
Remove unused DFG::SpeculativeJIT::calleeFrameOffset().
Rubber stamped by Michael Saboff.
Not only was this not used, I believe that the math was wrong. The callee frame doesn't
actually land past m_nextMachineLocal; instead it lands just below wherever we put SP and
that decision is made elsewhere. Also, it makes no sense to subtract 1 from
m_nextMachineLocal when trying to deduce the number of in-use stack slots.
- dfg/DFGSpeculativeJIT.h:
(JSC::DFG::SpeculativeJIT::calleeFrameOffset): Deleted.
- 11:11 AM Changeset in webkit [179874] by
-
- 2 edits in trunk/Source/WebKit2
[Cocoa] NSData over retain in WKBrowsingContextController loadData
https://bugs.webkit.org/show_bug.cgi?id=141437
Patch by Joseph Pecoraro <Joseph Pecoraro> on 2015-02-10
Reviewed by Anders Carlsson.
- UIProcess/API/Cocoa/WKBrowsingContextController.mm:
(-[WKBrowsingContextController loadData:MIMEType:textEncodingName:baseURL:userData:]):
API::Data::createWithoutCopying will automatically do the retaining / releasing for us.
- 11:01 AM Changeset in webkit [179873] by
-
- 4 edits in trunk/Source/JavaScriptCore
Parser::parseVarDeclarationList gets the wrong JSToken for the last identifier
https://bugs.webkit.org/show_bug.cgi?id=141272
Reviewed by Oliver Hunt.
This patch fixes a bug where the wrong text location would be
assigned to a variable declaration inside a ForIn/ForOf loop.
It also fixes a bug in the type profiler where the type profiler
emits the wrong text offset for a ForIn loop's variable declarator
when it's not a pattern node.
- bytecompiler/NodesCodegen.cpp:
(JSC::ForInNode::emitLoopHeader):
- parser/Parser.cpp:
(JSC::Parser<LexerType>::parseVarDeclarationList):
- tests/typeProfiler/loop.js:
(testForIn):
(testForOf):
- 9:28 AM Changeset in webkit [179872] by
-
- 2 edits in trunk/LayoutTests
[iOS] Gardening: fast/loader/subframe-navigate-during-main-frame-load.html crashes running all tests
- platform/ios-simulator-wk2/TestExpectations: Mark
fast/loader/subframe-navigate-during-main-frame-load.html as
flakey.
- 9:05 AM Changeset in webkit [179871] by
-
- 4 edits2 adds in trunk
Render: properly update body's background image
https://bugs.webkit.org/show_bug.cgi?id=140183
When HTML and BODY renderers are both composited the
skipBodyBackground condition should also take into account
if the HTML's layer can draw its contents.
Patch by Julien Isorce <j.isorce@samsung.com> on 2015-02-10
Reviewed by Darin Adler.
Source/WebCore:
Test: animations/animation-background-image.html
- rendering/RenderBox.cpp:
(WebCore::skipBodyBackground): Do not skip
if document's layer cannot draw its content.
Previously both body and html did not paint the background
when they are both composited.
- rendering/RenderLayerBacking.cpp:
(WebCore::RenderLayerBacking::contentChanged): Also redisplay
the content.
LayoutTests:
- compositing/backgrounds/background-image-with-negative-zindex-expected.html: Added.
- compositing/backgrounds/background-image-with-negative-zindex.html: Added.
- 8:01 AM Changeset in webkit [179870] by
-
- 10 edits in trunk/Source/WebKit2
[WK2] Add support for registering url schemes to bypass Content Security Policy.
https://bugs.webkit.org/show_bug.cgi?id=140736
Patch by Zhuo Li <zachli@apple.com> on 2015-02-10
Reviewed by Anders Carlsson.
- Shared/WebProcessCreationParameters.cpp:
(WebKit::WebProcessCreationParameters::encode):
(WebKit::WebProcessCreationParameters::decode):
- Shared/WebProcessCreationParameters.h:
- UIProcess/API/C/WKContext.cpp:
(WKContextRegisterURLSchemeAsBypassingContentSecurityPolicy):
- UIProcess/API/C/WKContextPrivate.h:
- UIProcess/WebProcessPool.cpp:
(WebKit::WebProcessPool::createNewWebProcess):
(WebKit::WebProcessPool::registerURLSchemeAsBypassingContentSecurityPolicy):
- UIProcess/WebProcessPool.h:
- WebProcess/WebProcess.cpp:
(WebKit::WebProcess::initializeWebProcess):
(WebKit::WebProcess::registerURLSchemeAsBypassingContentSecurityPolicy):
- WebProcess/WebProcess.h:
- WebProcess/WebProcess.messages.in:
- 7:47 AM Changeset in webkit [179869] by
-
- 6 edits in trunk
[iOS] don't get out of sync when interrupt/resume calls are not balanced
https://bugs.webkit.org/show_bug.cgi?id=141310
Reviewed by Jer Noble.
Source/WebCore:
No new tests, updated media/video-interruption-with-resume-allowing-play.html.
- platform/audio/MediaSession.cpp:
(WebCore::MediaSession::beginInterruption): Count interruptions.
(WebCore::MediaSession::endInterruption): Ignore calls when m_interruptionCount is already zero.
- platform/audio/MediaSession.h:
LayoutTests:
- media/video-interruption-with-resume-allowing-play-expected.txt:
- media/video-interruption-with-resume-allowing-play.html: Updated to test unbalanced calls
to begin/end interruption.
- 6:42 AM Changeset in webkit [179868] by
-
- 2 edits1 add in trunk/LayoutTests
Unreviewed GTK Gardening.
Patch by Marcos Chavarría Teijeiro <chavarria1991@gmail.com> on 2015-02-10
- platform/gtk/TestExpectations:
- platform/gtk/fast/css/font-face-in-media-rule-expected.txt: Added. Rebaselined after r177774.
- 5:19 AM WebKitGTK/Gardening/Calendar edited by
- (diff)
- 2:40 AM WebKitGTK/Gardening/Howto edited by
- (diff)
- 12:34 AM WebKitGTK/Gardening/Calendar edited by
- (diff)
- 12:24 AM Changeset in webkit [179867] by
-
- 2 edits in trunk/Source/WebKit2
[EFL] Use a variable to set flicking time instead of hardcoding
https://bugs.webkit.org/show_bug.cgi?id=138950
Patch by Sanghyup Lee <sh53.lee@samsung.com> on 2015-02-10
Reviewed by Gyuyoung Kim.
Currently time of flick gesture animation is hardcoded as one second.
This patch added a static variable to increase readability.
- UIProcess/API/efl/GestureRecognizer.cpp:
(WebKit::GestureHandler::GestureHandler):
(WebKit::GestureHandler::flickAnimatorCallback):
(WebKit::GestureHandler::handleFlick):
- 12:16 AM Changeset in webkit [179866] by
-
- 7 edits in trunk/Source
[GTK] GMutexLocker build issue
https://bugs.webkit.org/show_bug.cgi?id=141381
Reviewed by Žan Doberšek.
Source/WebCore:
Use always WTF::GMutexLocker because newer glib versions have a
GMutexLocker in the public API.
- platform/audio/gstreamer/AudioSourceProviderGStreamer.cpp:
(WebCore::AudioSourceProviderGStreamer::provideInput):
(WebCore::AudioSourceProviderGStreamer::handleAudioBuffer):
(WebCore::AudioSourceProviderGStreamer::clearAdapters):
- platform/graphics/gstreamer/MediaPlayerPrivateGStreamerBase.cpp:
(WebCore::MediaPlayerPrivateGStreamerBase::naturalSize):
(WebCore::MediaPlayerPrivateGStreamerBase::updateTexture):
(WebCore::MediaPlayerPrivateGStreamerBase::triggerRepaint):
(WebCore::MediaPlayerPrivateGStreamerBase::paint):
- platform/graphics/gstreamer/VideoSinkGStreamer.cpp:
(webkitVideoSinkTimeoutCallback):
(webkitVideoSinkRender):
(unlockSampleMutex):
(webkitVideoSinkUnlockStop):
(webkitVideoSinkStart):
- platform/graphics/gstreamer/WebKitWebSourceGStreamer.cpp:
(webKitWebSrcGetProperty):
(webKitWebSrcStop):
(webKitWebSrcStart):
(webKitWebSrcChangeState):
(webKitWebSrcQueryWithParent):
(webKitWebSrcGetUri):
(webKitWebSrcSetUri):
(webKitWebSrcNeedDataMainCb):
(webKitWebSrcNeedDataCb):
(webKitWebSrcEnoughDataMainCb):
(webKitWebSrcEnoughDataCb):
(webKitWebSrcSeekDataCb):
(webKitWebSrcSetMediaPlayer):
(StreamingClient::createReadBuffer):
(StreamingClient::handleResponseReceived):
(StreamingClient::handleDataReceived):
(StreamingClient::handleNotifyFinished):
(ResourceHandleStreamingClient::wasBlocked):
(ResourceHandleStreamingClient::cannotShowURL):
Source/WTF:
Remove using WTF::GMutexLocker from the header, since newer glib
versions have a GMutexLocker in the public API.
- wtf/gobject/GMutexLocker.h:
