Timeline
Aug 7, 2017:
- 11:43 PM Changeset in webkit [220388] by
-
- 7 edits in trunk/Source/WebDriver
WebDriver: implement unhandled prompt behavior
https://bugs.webkit.org/show_bug.cgi?id=175184
Reviewed by Brian Burg.
Handle user prompts before running some of the commands according to the specification.
- Capabilities.h: Add UnhandledPromptBehavior capability.
- CommandResult.cpp:
(WebDriver::CommandResult::httpStatusCode const): Add UnexpectedAlertOpen error.
(WebDriver::CommandResult::errorString const): Ditto.
- CommandResult.h:
(WebDriver::CommandResult::setAdditonalErrorData): New method to set an additional data object that will be sent
as part of the result error message.
(WebDriver::CommandResult::additionalErrorData const): Return the additional data object.
- Session.cpp:
(WebDriver::Session::handleUserPrompts): Check if there's an active JavaScript dialog and deal with it depeding
on the unhandled prompt behavior.
(WebDriver::Session::reportUnexpectedAlertOpen): Generate an error message with UnexpectedAlertOpen error and
including the alert text as additional error data.
(WebDriver::Session::go): Handle user prompts before running the command.
(WebDriver::Session::getCurrentURL): Ditto.
(WebDriver::Session::back): Ditto.
(WebDriver::Session::forward): Ditto.
(WebDriver::Session::refresh): Ditto.
(WebDriver::Session::getTitle): Ditto.
(WebDriver::Session::closeWindow): Ditto.
(WebDriver::Session::switchToFrame): Ditto.
(WebDriver::Session::switchToParentFrame): Ditto.
(WebDriver::Session::isElementSelected): Ditto.
(WebDriver::Session::getElementText): Ditto.
(WebDriver::Session::getElementTagName): Ditto.
(WebDriver::Session::getElementRect): Ditto.
(WebDriver::Session::isElementEnabled): Ditto.
(WebDriver::Session::isElementDisplayed): Ditto.
(WebDriver::Session::getElementAttribute): Ditto.
(WebDriver::Session::elementSendKeys): Ditto.
(WebDriver::Session::elementSubmit): Ditto.
(WebDriver::Session::executeScript): Ditto.
- Session.h:
- WebDriverService.cpp:
(WebDriver::WebDriverService::sendResponse const): Send data object as part of the result error message if present.
(WebDriver::deserializeUnhandledPromptBehavior):
(WebDriver::WebDriverService::parseCapabilities const):
(WebDriver::WebDriverService::validatedCapabilities const):
(WebDriver::WebDriverService::newSession):
- 11:25 PM Changeset in webkit [220387] by
-
- 12 edits1 copy1 add in trunk/Source
[GTK] Implement JavaScript dialog methods of API::AutomationSessionClient
https://bugs.webkit.org/show_bug.cgi?id=175259
Reviewed by Michael Catanzaro.
Source/WebCore/platform/gtk/po:
- POTFILES.in: Add WebKitScriptDialogGtk.cpp.
Source/WebKit:
Move the default implementation of WebKitScriptDialog from WebKitWebView platform specific files to their own
files. Implement all JavaScript dialog methods of API::AutomationSessionClient in WebKitAutomationSession. For
now it only works when the user doesn't override WebKitWebView::script-dialog signal and default implementation
is used.
- PlatformGTK.cmake: Add new files to compilation.
- PlatformWPE.cmake: Ditto.
- UIProcess/API/glib/WebKitAutomationSession.cpp:
(webkitAutomationSessionCreate): Pass the WebKitWebContext to the constructor and keep a pointer to it in session.
- UIProcess/API/glib/WebKitAutomationSessionPrivate.h:
- UIProcess/API/glib/WebKitScriptDialogPrivate.h:
- UIProcess/API/glib/WebKitWebContext.cpp:
- UIProcess/API/glib/WebKitWebView.cpp:
(webkitWebViewRunJavaScriptAlert): Set the currently script dialog for the scope of the function.
(webkitWebViewRunJavaScriptConfirm): Ditto.
(webkitWebViewRunJavaScriptPrompt): Ditto.
(webkitWebViewRunJavaScriptBeforeUnloadConfirm): Ditto.
(webkitWebViewIsShowingScriptDialog): Ask current dialog if there's one.
(webkitWebViewGetCurrentScriptDialogMessage): Ditto.
(webkitWebViewSetCurrentScriptDialogUserInput): Ditto.
(webkitWebViewAcceptCurrentScriptDialog): Ditto.
(webkitWebViewDismissCurrentScriptDialog): Ditto.
- UIProcess/API/glib/WebKitWebViewPrivate.h:
- UIProcess/API/gtk/WebKitScriptDialogGtk.cpp: Added.
(webkitWebViewCreateJavaScriptDialog): Moved from WebKitWebViewGtk.cpp.
(webkitScriptDialogRun): Run the dialog, this code is moved from WebKitWebViewGtk.cpp.
(webkitScriptDialogIsRunning): Return true if the script dialog has a native dialog running.
(webkitScriptDialogAccept): Send Ok or Close response to the native dialog depending on the dialog type.
(webkitScriptDialogDismiss): Send Close response to the native dialog.
(webkitScriptDialogSetUserInput): Set the given text on the prompt dialog entry.
- UIProcess/API/gtk/WebKitWebViewGtk.cpp:
(webkitWebViewScriptDialog): Simply call webkitScriptDialogRun() now.
- UIProcess/API/wpe/WebKitScriptDialogWPE.cpp: Copied from Source/WebKit/UIProcess/API/glib/WebKitAutomationSessionPrivate.h.
(webkitScriptDialogRun):
(webkitScriptDialogIsRunning):
(webkitScriptDialogAccept):
(webkitScriptDialogDismiss):
(webkitScriptDialogSetUserInput):
- 11:22 PM Changeset in webkit [220386] by
-
- 7 edits in trunk/Source/WebDriver
WebDriver: implement user prompt commands
https://bugs.webkit.org/show_bug.cgi?id=174614
Reviewed by Brian Burg.
- CommandResult.cpp:
(WebDriver::CommandResult::CommandResult): Handle NoJavaScriptDialog protocol error.
(WebDriver::CommandResult::httpStatusCode const): Add NoSuchAlert.
(WebDriver::CommandResult::errorString const): Ditto.
- CommandResult.h:
- Session.cpp:
(WebDriver::Session::dismissAlert):
(WebDriver::Session::acceptAlert):
(WebDriver::Session::getAlertText):
(WebDriver::Session::sendAlertText):
- Session.h:
- WebDriverService.cpp:
(WebDriver::WebDriverService::dismissAlert):
(WebDriver::WebDriverService::acceptAlert):
(WebDriver::WebDriverService::getAlertText):
(WebDriver::WebDriverService::sendAlertText):
- WebDriverService.h:
- 10:48 PM Changeset in webkit [220385] by
-
- 2 edits in branches/safari-604.1.38.1-branch/Source/WebCore
Cherry-pick r220333. rdar://problem/33601173
- 8:20 PM Changeset in webkit [220384] by
-
- 2 edits in trunk/PerformanceTests
Use more accurate Russian keywords for RexBench/FlightPlanner Unicode mode
https://bugs.webkit.org/show_bug.cgi?id=175298
Reviewed by Saam Barati.
With the help of Alexey Proskuryakov updated the Russian keywords.
- RexBench/FlightPlanner/use_unicode.js:
- 7:52 PM Changeset in webkit [220383] by
-
- 3 edits in trunk/Source/WebCore
RenderStyle:diff() was inadvertently doing deep compares of StyleRareNonInheritedData etc
https://bugs.webkit.org/show_bug.cgi?id=175304
Reviewed by Tim Horton.
r210758 changed DataRef::get() from returning a pointer to a reference. This caused all the places
in RenderStyle::diff() and related functions, where we intended to do a quick pointer comparison,
to doing deep compares via operator!=. This made the code slightly slower.
Fix by exposing ptr() on DataRef and using it wherever we wish to do pointer comparison.
- rendering/style/DataRef.h:
(WebCore::DataRef::ptr const):
- rendering/style/RenderStyle.cpp:
(WebCore::RenderStyle::inheritedDataShared const):
(WebCore::RenderStyle::changeAffectsVisualOverflow const):
(WebCore::RenderStyle::changeRequiresLayout const):
(WebCore::RenderStyle::changeRequiresRecompositeLayer const):
(WebCore::RenderStyle::listStyleImage const): Expand the function onto multiple lines.
- 7:52 PM Changeset in webkit [220382] by
-
- 2 edits in trunk/Source/WebCore
Add a fast path for rotate() and rotateZ() transform parsing
https://bugs.webkit.org/show_bug.cgi?id=175308
Reviewed by Zalan Bujtas.
Fast paths only existed for translate-related functions, matrix3d() and scale3d(). Add
rotate() and rotateX(), which gives a small boost to one of the MotionMark tests.
- css/parser/CSSParserFastPaths.cpp:
(WebCore::parseSimpleAngle):
(WebCore::parseTransformAngleArgument):
(WebCore::parseSimpleTransformValue):
(WebCore::transformCanLikelyUseFastPath):
- 7:52 PM Changeset in webkit [220381] by
-
- 2 edits in trunk/Source/WebCore
Re-order the tests in RenderLayerCompositor::requiresCompositingLayer() for performance
https://bugs.webkit.org/show_bug.cgi?id=175306
Reviewed by Tim Horton.
Re-order the tests for compositing reasons so that reasons more likely to happen are higher in the list.
- rendering/RenderLayerCompositor.cpp:
(WebCore::RenderLayerCompositor::requiresCompositingLayer const):
(WebCore::RenderLayerCompositor::requiresOwnBackingStore const):
- 7:52 PM Changeset in webkit [220380] by
-
- 2 edits in trunk/Source/WebCore
Avoid a hash lookup in FilterInfo::remove()
https://bugs.webkit.org/show_bug.cgi?id=175301
Reviewed by Sam Weinig.
FilterInfo::remove() always called map().remove(&layer)), even when layer.m_hasFilterInfo
was false (and even asserted that layer.m_hasFilterInfo == map().contains(&layer)).
So we can early return if layer.m_hasFilterInfo is false.
- rendering/RenderLayerFilterInfo.cpp:
(WebCore::RenderLayer::FilterInfo::getIfExists):
(WebCore::RenderLayer::FilterInfo::remove):
- 7:46 PM Changeset in webkit [220379] by
-
- 14 edits in trunk/Source/WebCore
Make TransformOperation::type() non-virtual
https://bugs.webkit.org/show_bug.cgi?id=175297
Reviewed by Sam Weinig.
Store the OperationType in the base class so that type() and isSameType() can
be non-virtual.
Small perf win on some benchmarks.
- platform/graphics/transforms/IdentityTransformOperation.h:
- platform/graphics/transforms/Matrix3DTransformOperation.h:
- platform/graphics/transforms/MatrixTransformOperation.h:
- platform/graphics/transforms/PerspectiveTransformOperation.h:
- platform/graphics/transforms/RotateTransformOperation.cpp:
(WebCore::RotateTransformOperation::blend):
- platform/graphics/transforms/RotateTransformOperation.h:
- platform/graphics/transforms/ScaleTransformOperation.cpp:
(WebCore::ScaleTransformOperation::blend):
- platform/graphics/transforms/ScaleTransformOperation.h:
- platform/graphics/transforms/SkewTransformOperation.cpp:
(WebCore::SkewTransformOperation::blend):
- platform/graphics/transforms/SkewTransformOperation.h:
- platform/graphics/transforms/TransformOperation.h:
(WebCore::TransformOperation::TransformOperation):
(WebCore::TransformOperation::type const):
(WebCore::TransformOperation::isSameType const):
- platform/graphics/transforms/TranslateTransformOperation.cpp:
(WebCore::TranslateTransformOperation::blend):
- platform/graphics/transforms/TranslateTransformOperation.h:
- 7:46 PM Changeset in webkit [220378] by
-
- 4 edits in trunk/Source/WebCore
Inline ~Color and Color::isExtended()
https://bugs.webkit.org/show_bug.cgi?id=175293
Reviewed by Zalan Bujtas.
The Color destructor and Color::isExtended() show up on profiles, so inline them.
Before r207265 the destructor was inlined.
Also make sure that LengthSize::operator== is inlined, which it was not (according
to profiles).
- platform/LengthSize.h:
(WebCore::operator==):
- platform/graphics/Color.cpp:
(WebCore::Color::~Color): Deleted.
(WebCore::Color::isExtended const): Deleted.
- platform/graphics/Color.h:
(WebCore::Color::~Color):
(WebCore::Color::isExtended const):
- 7:29 PM Changeset in webkit [220377] by
-
- 3 edits1 add in trunk
GetOwnProperty of TypedArray indexed fields is wrongly configurable
https://bugs.webkit.org/show_bug.cgi?id=175307
Patch by Robin Morisset <rmorisset@apple.com> on 2017-08-07
Reviewed by Saam Barati.
JSTests:
- stress/typedarray-getownproperty-not-configurable.js: Added.
(assert):
(foo):
Source/JavaScriptCore:
`
let a = new Uint8Array(10);
let b = Object.getOwnPropertyDescriptor(a, 0);
assert(b.configurable === false);
`
should not fail: by section 9.4.5.1 (https://tc39.github.io/ecma262/#sec-integer-indexed-exotic-objects-getownproperty-p)
that applies to integer indexed exotic objects, and section 22.2.7 (https://tc39.github.io/ecma262/#sec-properties-of-typedarray-instances)
that says that typed arrays are integer indexed exotic objects.
- runtime/JSGenericTypedArrayViewInlines.h:
(JSC::JSGenericTypedArrayView<Adaptor>::getOwnPropertySlotByIndex):
- 7:28 PM Changeset in webkit [220376] by
-
- 5 edits in trunk
Source/WebCore:
Implement the HTML5 same-origin restriction specification
https://bugs.webkit.org/show_bug.cgi?id=175226
<rdar://problem/11079948>
Reviewed by Chris Dumez.
Follow the algorithms defined in the HTML5 specification for relaxing
the same-origin restriction. We were missing a few steps related to
checking for public suffix and presence of a browsing context.
Tested by new TestWebKitAPI tests.
- dom/Document.cpp:
(WebCore::Document::domainIsRegisterable): Added helper function.
(WebCore::Document::setDomain):
Tools:
Prevent domain from being set to a TLD
https://bugs.webkit.org/show_bug.cgi?id=175226
<rdar://problem/11079948>
Reviewed by Chris Dumez.
Extend the public suffix tests to include cases used by the
Public Domain 'Public Suffix List'.
- TestWebKitAPI/Tests/mac/PublicSuffix.cpp:
(TestWebKitAPI::TEST):
- 6:05 PM Changeset in webkit [220375] by
-
- 2 edits in trunk/LayoutTests
Marked media/modern-media-controls/fullscreen-support/fullscreen-support-press.html as flaky.
https://bugs.webkit.org/show_bug.cgi?id=173946
Unreviewed test gardening.
- platform/mac-wk2/TestExpectations:
- 5:53 PM Changeset in webkit [220374] by
-
- 1 edit3 adds in trunk/LayoutTests
Rebaselined js/dom/global-constructors-attributes.html on El Capitan.
https://bugs.webkit.org/show_bug.cgi?id=175201
Unreviewed test gardening.
- platform/mac-elcapitan-wk2/js/dom/global-constructors-attributes-dedicated-worker-expected.txt: Added.
- 5:47 PM Changeset in webkit [220373] by
-
- 11 edits10 adds2 deletes in trunk/PerformanceTests
Speedometer: Update to modern production Ember version
https://bugs.webkit.org/show_bug.cgi?id=175278
Patch by Mathias Bynens <mathias@qiwi.be> on 2017-08-07
Reviewed by Ryosuke Niwa.
- Speedometer/resources/todomvc/architecture-examples/emberjs/README.md: Added.
- Speedometer/resources/todomvc/architecture-examples/emberjs/index.html: Made title consistent.
- Speedometer/resources/todomvc/architecture-examples/emberjs/source/README.md: Documented build process.
- Speedometer/resources/todomvc/architecture-examples/emberjs/source/app/components/.gitkeep: Added.
- Speedometer/resources/todomvc/architecture-examples/emberjs/source/app/index.html: Made title consistent.
- Speedometer/resources/todomvc/architecture-examples/emberjs/source/app/instance-initializers/global.js: Removed.
- Speedometer/resources/todomvc/architecture-examples/emberjs/source/app/models/.gitkeep: Added.
- Speedometer/resources/todomvc/architecture-examples/emberjs/source/app/router.js: Update to latest version.
- Speedometer/resources/todomvc/architecture-examples/emberjs/source/app/routes/.gitkeep: Added.
- Speedometer/resources/todomvc/architecture-examples/emberjs/source/app/services/repo.js: Update to latest version
- Speedometer/resources/todomvc/architecture-examples/emberjs/source/app/styles/app.css: Added.
- Speedometer/resources/todomvc/architecture-examples/emberjs/source/app/templates/components/.gitkeep: Added.
- Speedometer/resources/todomvc/architecture-examples/emberjs/source/bower.json: Update to latest version.
- Speedometer/resources/todomvc/architecture-examples/emberjs/source/config/environment.js: Update to latest version.
- Speedometer/resources/todomvc/architecture-examples/emberjs/source/dist/*: Added generated files per build instructions.
- Speedometer/resources/todomvc/architecture-examples/emberjs/source/package-lock.json: Pinned dependencies.
- Speedometer/resources/todomvc/architecture-examples/emberjs/source/package.json: Update to latest version.
- Speedometer/resources/todomvc/architecture-examples/emberjs/source/tests/.jshintrc: Removed.
- Speedometer/resources/todomvc/architecture-examples/emberjs/source/tests/helpers/module-for-acceptance.js: Updated to latest version.
- Speedometer/resources/todomvc/architecture-examples/emberjs/source/tests/helpers/start-app.js: Updated to latest version.
- Speedometer/resources/todomvc/architecture-examples/emberjs/source/tests/index.html: Updated to latest version.
- Speedometer/resources/todomvc/architecture-examples/emberjs/source/tests/integration/.gitkeep: Added.
- Speedometer/resources/todomvc/architecture-examples/emberjs/source/tests/unit/.gitkeep: Added.
- Speedometer/resources/todomvc/architecture-examples/emberjs/source/vendor/.gitkeep: Added.
- 5:25 PM Changeset in webkit [220372] by
-
- 8 edits1 add in trunk/Tools
[XCode] webkit-patch should run sort-Xcode-project-file
https://bugs.webkit.org/show_bug.cgi?id=174036
<rdar://problem/33732709>
Patch by Stephan Szabo <stephan.szabo@sony.com> on 2017-08-07
Reviewed by Simon Fraser.
- Scripts/webkitpy/common/config/ports.py:
- Scripts/webkitpy/tool/commands/download.py:
- Scripts/webkitpy/tool/commands/download_unittest.py:
- Scripts/webkitpy/tool/commands/upload.py:
- Scripts/webkitpy/tool/commands/upload_unittest.py:
- Scripts/webkitpy/tool/steps/init.py:
- Scripts/webkitpy/tool/steps/options.py:
- Scripts/webkitpy/tool/steps/sortxcodeprojectfiles.py: Added.
- 4:56 PM Changeset in webkit [220371] by
-
- 2 edits in trunk/LayoutTests
Changed expectations for webrtc/video-rotation.html.
https://bugs.webkit.org/show_bug.cgi?id=175305
Unreviewed test gardening.
- 4:55 PM Changeset in webkit [220370] by
-
- 36 edits8 adds in trunk
Web Inspector: Preview Canvas path when viewing a recording
https://bugs.webkit.org/show_bug.cgi?id=174967
Reviewed by Brian Burg.
Source/WebCore:
Tests: fast/canvas/2d.currentPoint.html
fast/canvas/2d.getPath.modification.html
fast/canvas/2d.getPath.newobject.html
fast/canvas/2d.setPath.html
- html/canvas/CanvasPath.idl:
- html/canvas/CanvasPath.h:
- html/canvas/CanvasPath.cpp:
(WebCore::CanvasPath::currentX const):
(WebCore::CanvasPath::currentY const):
- html/canvas/CanvasRenderingContext2D.idl:
- html/canvas/CanvasRenderingContext2D.h:
- html/canvas/CanvasRenderingContext2D.cpp:
(WebCore::CanvasRenderingContext2D::setPath):
(WebCore::CanvasRenderingContext2D::getPath const):
- page/RuntimeEnabledFeatures.h:
(WebCore::RuntimeEnabledFeatures::setInspectorAdditionsEnabled):
(WebCore::RuntimeEnabledFeatures::inspectorAdditionsEnabled const):
Add runtime flag for added IDL items above so that they are only usable within the inspector
process. The runtime flag is not enabled from anywhere else as of now.
- inspector/InspectorCanvas.cpp:
(WebCore::InspectorCanvas::buildInitialState):
Send current path as part of the InitialState.
Drive-by: deduplicate more string values.
Source/WebInspectorUI:
- Localizations/en.lproj/localizedStrings.js:
- UserInterface/Base/Setting.js:
- UserInterface/Views/RecordingContentView.js:
(WI.RecordingContentView):
(WI.RecordingContentView.supportsCanvasPathDebugging):
(WI.RecordingContentView.prototype.get navigationItems):
(WI.RecordingContentView.prototype.shown):
(WI.RecordingContentView.prototype._generateContentCanvas2D):
(WI.RecordingContentView.prototype._actionModifiesPath):
(WI.RecordingContentView.prototype._updateCanvasPath):
(WI.RecordingContentView.prototype._showPathButtonClicked):
Show each segment of the current path as an overlay when the setting is enabled.
Drive-by: fix forgotten function rename.
- UserInterface/Views/RecordingContentView.css:
(.content-view:not(.tab).recording canvas.path):
- UserInterface/Views/RecordingStateDetailsSidebarPanel.js:
(WI.RecordingStateDetailsSidebarPanel.prototype._generateDetailsCanvas2D):
Show the currentX/currentY in the current state.
- UserInterface/Models/RecordingAction.js:
(WI.RecordingAction.isFunctionForType):
(WI.RecordingAction.prototype.swizzle):
Use Sets for better performance.
Source/WebKit:
- Shared/WebPreferencesDefinitions.h:
- UIProcess/API/C/WKPreferencesRefPrivate.h:
- UIProcess/API/C/WKPreferences.cpp:
(WKPreferencesSetInspectorAdditionsEnabled):
(WKPreferencesGetInspectorAdditionsEnabled):
- WebProcess/WebPage/WebPage.cpp:
(WebKit::WebPage::updatePreferences):
Add plumbing for new InspectorAdditions runtime flag.
- WebProcess/WebPage/WebInspectorUI.cpp:
(WebKit::WebInspectorUI::WebInspectorUI):
Enable InspectorAdditions by default in the WebInspector page.
Source/WebKitLegacy/mac:
- WebView/WebPreferenceKeysPrivate.h:
- WebView/WebPreferences.mm:
(+[WebPreferences initialize]):
(-[WebPreferences inspectorAdditionsEnabled]):
(-[WebPreferences setInspectorAdditionsEnabled:]):
- WebView/WebPreferencesPrivate.h:
- WebView/WebView.mm:
(-[WebView _preferencesChanged:]):
Add plumbing for new InspectorAdditions runtime flag.
Tools:
- DumpRenderTree/TestOptions.h:
- DumpRenderTree/TestOptions.mm:
(TestOptions::TestOptions):
- DumpRenderTree/mac/DumpRenderTree.mm:
(setWebPreferencesForTestOptions):
- WebKitTestRunner/TestController.cpp:
(WTR::TestController::resetPreferencesToConsistentValues):
(WTR::updateTestOptionsFromTestHeader):
- WebKitTestRunner/TestOptions.h:
(WTR::TestOptions::hasSameInitializationOptions const):
Add plumbing for new InspectorAdditions runtime flag.
LayoutTests:
- fast/canvas/2d.currentPoint-expected.txt: Added.
- fast/canvas/2d.currentPoint.html: Added.
- fast/canvas/2d.getPath.modification-expected.txt: Added.
- fast/canvas/2d.getPath.modification.html: Added.
- fast/canvas/2d.getPath.newobject-expected.txt: Added.
- fast/canvas/2d.getPath.newobject.html: Added.
- fast/canvas/2d.setPath-expected.txt: Added.
- fast/canvas/2d.setPath.html: Added.
- inspector/canvas/recording-2d-expected.txt:
- inspector/canvas/recording-2d.html:
Updated for additional deduplication in InitialState.
- 4:33 PM Changeset in webkit [220369] by
-
- 3 edits in trunk/Tools
Sorted EWS Queues on Patch page
https://bugs.webkit.org/show_bug.cgi?id=173667
Patch by obinna obike <oobike@apple.com> on 2017-08-07
Reviewed by Aakash Jain.
- QueueStatusServer/handlers/patch.py:
(Patch.get): Sorted queue_status.
- QueueStatusServer/templates/patch.html:
Changed queue_status.items to queue_status because it's a list.
- 4:30 PM Changeset in webkit [220368] by
-
- 14 edits in trunk/Source
Baseline JIT should do caging
https://bugs.webkit.org/show_bug.cgi?id=175037
Reviewed by Mark Lam.
Source/bmalloc:
This centralizes the notion of permanently enabling the primitive gigacage, which we only do in jsc
and WebProcess.
This saves the baseline JIT from emitting some code. Otherwise it would always have to emit enabled
checks on each typed array access.
- bmalloc/Gigacage.cpp:
(Gigacage::primitiveGigacageDisabled):
(Gigacage::disableDisablingPrimitiveGigacageIfShouldBeEnabled):
(Gigacage::isDisablingPrimitiveGigacageDisabled):
- bmalloc/Gigacage.h:
(Gigacage::isPrimitiveGigacagePermanentlyEnabled):
(Gigacage::canPrimitiveGigacageBeDisabled):
Source/JavaScriptCore:
Adds a AssemblyHelpers::cage and cageConditionally. Uses it in the baseline JIT.
Also modifies FTL caging to be more defensive when caging is disabled.
- ftl/FTLLowerDFGToB3.cpp:
(JSC::FTL::DFG::LowerDFGToB3::caged):
- jit/AssemblyHelpers.h:
(JSC::AssemblyHelpers::cage):
(JSC::AssemblyHelpers::cageConditionally):
- jit/JITPropertyAccess.cpp:
(JSC::JIT::emitDoubleLoad):
(JSC::JIT::emitContiguousLoad):
(JSC::JIT::emitArrayStorageLoad):
(JSC::JIT::emitGenericContiguousPutByVal):
(JSC::JIT::emitArrayStoragePutByVal):
(JSC::JIT::emit_op_get_from_scope):
(JSC::JIT::emit_op_put_to_scope):
(JSC::JIT::emitIntTypedArrayGetByVal):
(JSC::JIT::emitFloatTypedArrayGetByVal):
(JSC::JIT::emitIntTypedArrayPutByVal):
(JSC::JIT::emitFloatTypedArrayPutByVal):
- jsc.cpp:
(jscmain):
(primitiveGigacageDisabled): Deleted.
Source/WebKit:
Use a better API to disable disabling the primitive gigacage.
- WebProcess/WebProcess.cpp:
(WebKit::m_webSQLiteDatabaseTracker):
(WebKit::primitiveGigacageDisabled): Deleted.
Source/WTF:
- wtf/Gigacage.h:
(Gigacage::disableDisablingPrimitiveGigacageIfShouldBeEnabled):
(Gigacage::isDisablingPrimitiveGigacageDisabled):
(Gigacage::isPrimitiveGigacagePermanentlyEnabled):
(Gigacage::canPrimitiveGigacageBeDisabled):
- 4:25 PM Changeset in webkit [220367] by
-
- 3 edits in trunk/Tools
check-webkit-style: NS_ERROR_ENUM and NS_OPTIONS should not be handled as function definitions.
https://bugs.webkit.org/show_bug.cgi?id=175286
Patch by Yoshiaki Jitsukawa <Yoshiaki.Jitsukawa@sony.com> on 2017-08-07
Reviewed by Myles C. Maxfield.
- Scripts/webkitpy/style/checkers/cpp.py:
(check_braces):
- Scripts/webkitpy/style/checkers/cpp_unittest.py:
(WebKitStyleTest.test_braces.NS_ERROR_ENUM):
(WebKitStyleTest.test_braces):
(WebKitStyleTest.test_braces.NS_OPTIONS):
- 4:22 PM Changeset in webkit [220366] by
-
- 2 edits in trunk/Source/WebCore
Update sendBeacon() to rely on FetchBody instead of the whole FetchRequest
https://bugs.webkit.org/show_bug.cgi?id=175280
Reviewed by Youenn Fablet.
Update sendBeacon() to rely on FetchBody instead of the whole FetchRequest. FetchBody
for data extraction is really the only thing we need at the moment.
The new code also properly sets the CORS mode, which will be needed for Bug 175264.
- Modules/beacon/NavigatorBeacon.cpp:
(WebCore::NavigatorBeacon::sendBeacon):
- 4:18 PM Changeset in webkit [220365] by
-
- 7 edits in branches/safari-604.1.38.0-branch/Source
Versioning.
- 4:16 PM Changeset in webkit [220364] by
-
- 7 edits in branches/safari-604.1.38.1-branch/Source
Versioning.
- 4:12 PM Changeset in webkit [220363] by
-
- 1 copy in tags/Safari-604.1.38.0.1
Tag Safari-604.1.38.0.1.
- 4:10 PM Changeset in webkit [220362] by
-
- 1 copy in tags/Safari-604.1.38.1.1
Tag Safari-604.1.38.1.1.
- 3:43 PM Changeset in webkit [220361] by
-
- 2 edits in trunk/PerformanceTests
REGRESSION(r220307): Perf bot failure trying to run RexBench tests
https://bugs.webkit.org/show_bug.cgi?id=175289
Reviewed by Filip Pizlo.
Add RexBench to the list of benchmarks we skip on the perf bots.
- Skipped:
- 3:31 PM Changeset in webkit [220360] by
-
- 2 edits in trunk/Websites/webkit.org
Fixed superscript rendering for blog posts
https://bugs.webkit.org/show_bug.cgi?id=175285
Reviewed by Filip Pizlo.
- wp-content/themes/webkit/style.css:
(sup): Added.
- 3:26 PM Changeset in webkit [220359] by
-
- 7 edits in branches/safari-604.1.38.1-branch/Source
Versioning.
- 3:24 PM Changeset in webkit [220358] by
-
- 1 copy in branches/safari-604.1.38.1-branch
New branch.
- 3:22 PM Changeset in webkit [220357] by
-
- 2 edits in branches/safari-604.1.38.0-branch/Source/WebKit
Cherry-pick r220273. rdar://problem/33732757
- 3:20 PM Changeset in webkit [220356] by
-
- 7 edits in branches/safari-604.1.38.0-branch/Source
Versioning.
- 3:09 PM Changeset in webkit [220355] by
-
- 1 copy in branches/safari-604.1.38.0-branch
New branch.
- 3:01 PM Changeset in webkit [220354] by
-
- 2 edits in trunk/Source/WebCore
-Wimplicit-fallthrough warning in ComputedStyleExtractor::propertyValue
https://bugs.webkit.org/show_bug.cgi?id=174469
<rdar://problem/33311638>
Reviewed by Simon Fraser.
Add a RELEASE_ASSERT_NOT_REACHED to silence this warning.
- css/CSSComputedStyleDeclaration.cpp:
(WebCore::ComputedStyleExtractor::propertyValue):
- 2:53 PM Changeset in webkit [220353] by
-
- 3 edits in trunk/Source/WTF
Unreviewed, try to fix Windows build.
- wtf/Gigacage.cpp:
- wtf/Gigacage.h:
- 2:31 PM Changeset in webkit [220352] by
-
- 54 edits1 move1 add in trunk/Source
Primitive auxiliaries and JSValue auxiliaries should have separate gigacages
https://bugs.webkit.org/show_bug.cgi?id=174919
Reviewed by Keith Miller.
Source/bmalloc:
This introduces two kinds of Gigacage, Primitive and JSValue. This translates to two kinds of
HeapKind, PrimitiveGigacage and JSValueGigacage.
The new support functionality required turning Inline.h into BInline.h, and INLINE into BINLINE, and
NO_INLINE into BNO_INLINE.
- bmalloc.xcodeproj/project.pbxproj:
- bmalloc/Allocator.cpp:
(bmalloc::Allocator::refillAllocatorSlowCase):
(bmalloc::Allocator::refillAllocator):
(bmalloc::Allocator::allocateLarge):
(bmalloc::Allocator::allocateLogSizeClass):
- bmalloc/AsyncTask.h:
- bmalloc/BInline.h: Copied from Source/bmalloc/bmalloc/Inline.h.
- bmalloc/Cache.cpp:
(bmalloc::Cache::tryAllocateSlowCaseNullCache):
(bmalloc::Cache::allocateSlowCaseNullCache):
(bmalloc::Cache::deallocateSlowCaseNullCache):
(bmalloc::Cache::reallocateSlowCaseNullCache):
- bmalloc/Deallocator.cpp:
- bmalloc/Gigacage.cpp:
(Gigacage::PrimitiveDisableCallbacks::PrimitiveDisableCallbacks):
(Gigacage::ensureGigacage):
(Gigacage::disablePrimitiveGigacage):
(Gigacage::addPrimitiveDisableCallback):
(Gigacage::removePrimitiveDisableCallback):
(Gigacage::Callbacks::Callbacks): Deleted.
(Gigacage::disableGigacage): Deleted.
(Gigacage::addDisableCallback): Deleted.
(Gigacage::removeDisableCallback): Deleted.
- bmalloc/Gigacage.h:
(Gigacage::name):
(Gigacage::basePtr):
(Gigacage::forEachKind):
(Gigacage::caged):
(Gigacage::isCaged):
- bmalloc/Heap.cpp:
(bmalloc::Heap::Heap):
(bmalloc::Heap::usingGigacage):
(bmalloc::Heap::gigacageBasePtr):
- bmalloc/Heap.h:
- bmalloc/HeapKind.h:
(bmalloc::isGigacage):
(bmalloc::gigacageKind):
(bmalloc::heapKind):
- bmalloc/Inline.h: Removed.
- bmalloc/Map.h:
- bmalloc/PerProcess.h:
(bmalloc::PerProcess<T>::getFastCase):
(bmalloc::PerProcess<T>::get):
(bmalloc::PerProcess<T>::getSlowCase):
- bmalloc/PerThread.h:
(bmalloc::PerThread<T>::getFastCase):
- bmalloc/Vector.h:
(bmalloc::Vector<T>::push):
(bmalloc::Vector<T>::shrinkCapacity):
(bmalloc::Vector<T>::growCapacity):
Source/JavaScriptCore:
This adapts JSC to there being two gigacages.
To make matters simpler, this turns AlignedMemoryAllocators into per-VM instances rather than
singletons. I don't think we were gaining anything by making them be singletons.
This makes it easy to teach GigacageAlignedMemoryAllocator that there are multiple kinds of
gigacages. We'll have one of those allocators per cage.
From there, this change teaches everyone who previously knew about cages that there are two cages.
This means having to specify either Gigacage::Primitive or Gigacage::JSValue. In most places, this is
easy: typed arrays are Primitive and butterflies are JSValue. But there are a few places where it's
not so obvious, so this change introduces some helpers to make it easy to define what cage you want
to use in one place and refer to it abstractly. We do this in DirectArguments and GenericArguments.h
A lot of the magic of this change is due to CagedBarrierPtr, which combines AuxiliaryBarrier and
CagedPtr. This removes one layer of "get()" calls from a bunch of places.
- JavaScriptCore.xcodeproj/project.pbxproj:
- bytecode/AccessCase.cpp:
(JSC::AccessCase::generateImpl):
- dfg/DFGSpeculativeJIT.cpp:
(JSC::DFG::SpeculativeJIT::emitAllocateRawObject):
(JSC::DFG::SpeculativeJIT::compileAllocatePropertyStorage):
(JSC::DFG::SpeculativeJIT::compileReallocatePropertyStorage):
(JSC::DFG::SpeculativeJIT::compileNewTypedArray):
(JSC::DFG::SpeculativeJIT::emitAllocateButterfly):
- ftl/FTLLowerDFGToB3.cpp:
(JSC::FTL::DFG::LowerDFGToB3::compileGetButterfly):
(JSC::FTL::DFG::LowerDFGToB3::compileGetIndexedPropertyStorage):
(JSC::FTL::DFG::LowerDFGToB3::compileNewTypedArray):
(JSC::FTL::DFG::LowerDFGToB3::compileGetDirectPname):
(JSC::FTL::DFG::LowerDFGToB3::compileMaterializeNewObject):
(JSC::FTL::DFG::LowerDFGToB3::allocatePropertyStorageWithSizeImpl):
(JSC::FTL::DFG::LowerDFGToB3::allocateJSArray):
(JSC::FTL::DFG::LowerDFGToB3::caged):
- heap/FastMallocAlignedMemoryAllocator.cpp:
(JSC::FastMallocAlignedMemoryAllocator::instance): Deleted.
- heap/FastMallocAlignedMemoryAllocator.h:
- heap/GigacageAlignedMemoryAllocator.cpp:
(JSC::GigacageAlignedMemoryAllocator::GigacageAlignedMemoryAllocator):
(JSC::GigacageAlignedMemoryAllocator::tryAllocateAlignedMemory):
(JSC::GigacageAlignedMemoryAllocator::freeAlignedMemory):
(JSC::GigacageAlignedMemoryAllocator::dump const):
(JSC::GigacageAlignedMemoryAllocator::instance): Deleted.
- heap/GigacageAlignedMemoryAllocator.h:
- jsc.cpp:
(primitiveGigacageDisabled):
(jscmain):
(gigacageDisabled): Deleted.
- llint/LowLevelInterpreter64.asm:
- runtime/ArrayBuffer.cpp:
(JSC::ArrayBufferContents::tryAllocate):
(JSC::ArrayBuffer::createAdopted):
(JSC::ArrayBuffer::createFromBytes):
- runtime/AuxiliaryBarrier.h:
- runtime/ButterflyInlines.h:
(JSC::Butterfly::createUninitialized):
(JSC::Butterfly::tryCreate):
(JSC::Butterfly::growArrayRight):
- runtime/CagedBarrierPtr.h: Added.
(JSC::CagedBarrierPtr::CagedBarrierPtr):
(JSC::CagedBarrierPtr::clear):
(JSC::CagedBarrierPtr::set):
(JSC::CagedBarrierPtr::get const):
(JSC::CagedBarrierPtr::getMayBeNull const):
(JSC::CagedBarrierPtr::operator== const):
(JSC::CagedBarrierPtr::operator!= const):
(JSC::CagedBarrierPtr::operator bool const):
(JSC::CagedBarrierPtr::setWithoutBarrier):
(JSC::CagedBarrierPtr::operator* const):
(JSC::CagedBarrierPtr::operator-> const):
(JSC::CagedBarrierPtr::operator[] const):
- runtime/DirectArguments.cpp:
(JSC::DirectArguments::overrideThings):
(JSC::DirectArguments::unmapArgument):
- runtime/DirectArguments.h:
(JSC::DirectArguments::isMappedArgument const):
- runtime/GenericArguments.h:
- runtime/GenericArgumentsInlines.h:
(JSC::GenericArguments<Type>::initModifiedArgumentsDescriptor):
(JSC::GenericArguments<Type>::setModifiedArgumentDescriptor):
(JSC::GenericArguments<Type>::isModifiedArgumentDescriptor):
- runtime/HashMapImpl.cpp:
(JSC::HashMapImpl<HashMapBucket>::visitChildren):
- runtime/HashMapImpl.h:
(JSC::HashMapBuffer::create):
(JSC::HashMapImpl::buffer const):
(JSC::HashMapImpl::rehash):
- runtime/JSArray.cpp:
(JSC::JSArray::tryCreateUninitializedRestricted):
(JSC::JSArray::unshiftCountSlowCase):
(JSC::JSArray::setLength):
(JSC::JSArray::pop):
(JSC::JSArray::push):
(JSC::JSArray::fastSlice):
(JSC::JSArray::shiftCountWithArrayStorage):
(JSC::JSArray::shiftCountWithAnyIndexingType):
(JSC::JSArray::unshiftCountWithAnyIndexingType):
(JSC::JSArray::fillArgList):
(JSC::JSArray::copyToArguments):
- runtime/JSArray.h:
(JSC::JSArray::tryCreate):
- runtime/JSArrayBufferView.cpp:
(JSC::JSArrayBufferView::ConstructionContext::ConstructionContext):
(JSC::JSArrayBufferView::finalize):
- runtime/JSLock.cpp:
(JSC::JSLock::didAcquireLock):
- runtime/JSObject.cpp:
(JSC::JSObject::heapSnapshot):
(JSC::JSObject::getOwnPropertySlotByIndex):
(JSC::JSObject::putByIndex):
(JSC::JSObject::enterDictionaryIndexingMode):
(JSC::JSObject::createInitialIndexedStorage):
(JSC::JSObject::createArrayStorage):
(JSC::JSObject::convertUndecidedToInt32):
(JSC::JSObject::convertUndecidedToDouble):
(JSC::JSObject::convertUndecidedToContiguous):
(JSC::JSObject::constructConvertedArrayStorageWithoutCopyingElements):
(JSC::JSObject::convertUndecidedToArrayStorage):
(JSC::JSObject::convertInt32ToDouble):
(JSC::JSObject::convertInt32ToContiguous):
(JSC::JSObject::convertInt32ToArrayStorage):
(JSC::JSObject::convertDoubleToContiguous):
(JSC::JSObject::convertDoubleToArrayStorage):
(JSC::JSObject::convertContiguousToArrayStorage):
(JSC::JSObject::setIndexQuicklyToUndecided):
(JSC::JSObject::ensureArrayStorageExistsAndEnterDictionaryIndexingMode):
(JSC::JSObject::deletePropertyByIndex):
(JSC::JSObject::getOwnPropertyNames):
(JSC::JSObject::putIndexedDescriptor):
(JSC::JSObject::defineOwnIndexedProperty):
(JSC::JSObject::putByIndexBeyondVectorLengthWithoutAttributes):
(JSC::JSObject::putDirectIndexSlowOrBeyondVectorLength):
(JSC::JSObject::getNewVectorLength):
(JSC::JSObject::ensureLengthSlow):
(JSC::JSObject::reallocateAndShrinkButterfly):
(JSC::JSObject::allocateMoreOutOfLineStorage):
(JSC::JSObject::getEnumerableLength):
- runtime/JSObject.h:
(JSC::JSObject::getArrayLength const):
(JSC::JSObject::getVectorLength):
(JSC::JSObject::putDirectIndex):
(JSC::JSObject::canGetIndexQuickly):
(JSC::JSObject::getIndexQuickly):
(JSC::JSObject::tryGetIndexQuickly const):
(JSC::JSObject::canSetIndexQuickly):
(JSC::JSObject::setIndexQuickly):
(JSC::JSObject::initializeIndex):
(JSC::JSObject::initializeIndexWithoutBarrier):
(JSC::JSObject::hasSparseMap):
(JSC::JSObject::inSparseIndexingMode):
(JSC::JSObject::butterfly const):
(JSC::JSObject::butterfly):
(JSC::JSObject::outOfLineStorage const):
(JSC::JSObject::outOfLineStorage):
(JSC::JSObject::ensureInt32):
(JSC::JSObject::ensureDouble):
(JSC::JSObject::ensureContiguous):
(JSC::JSObject::ensureArrayStorage):
(JSC::JSObject::arrayStorage):
(JSC::JSObject::arrayStorageOrNull):
(JSC::JSObject::ensureLength):
- runtime/RegExpMatchesArray.h:
(JSC::tryCreateUninitializedRegExpMatchesArray):
- runtime/VM.cpp:
(JSC::VM::VM):
(JSC::VM::~VM):
(JSC::VM::primitiveGigacageDisabledCallback):
(JSC::VM::primitiveGigacageDisabled):
(JSC::VM::gigacageDisabledCallback): Deleted.
(JSC::VM::gigacageDisabled): Deleted.
- runtime/VM.h:
(JSC::VM::gigacageAuxiliarySpace):
(JSC::VM::firePrimitiveGigacageEnabledIfNecessary):
(JSC::VM::primitiveGigacageEnabled):
(JSC::VM::fireGigacageEnabledIfNecessary): Deleted.
(JSC::VM::gigacageEnabled): Deleted.
- wasm/WasmMemory.cpp:
(JSC::Wasm::Memory::create):
(JSC::Wasm::Memory::~Memory):
(JSC::Wasm::Memory::grow):
Source/WebCore:
No new tests because no change in behavior.
Adapting to API changes - we now specify the AlignedMemoryAllocator differently and we need to be
specific about which Gigacage we're using.
- bindings/js/WebCoreJSClientData.cpp:
(WebCore::JSVMClientData::JSVMClientData):
- platform/graphics/cocoa/GPUBufferMetal.mm:
(WebCore::GPUBuffer::GPUBuffer):
Source/WebKit:
The disable callback is all about the primitive gigacage.
- WebProcess/WebProcess.cpp:
(WebKit::primitiveGigacageDisabled):
(WebKit::m_webSQLiteDatabaseTracker):
(WebKit::gigacageDisabled): Deleted.
Source/WTF:
This mirrors the changes from bmalloc/Gigacage.h.
Also it teaches CagedPtr how to reason about multiple gigacages.
- wtf/CagedPtr.h:
(WTF::CagedPtr::get const):
(WTF::CagedPtr::operator[] const):
- wtf/Gigacage.cpp:
(Gigacage::tryMalloc):
(Gigacage::tryAllocateVirtualPages):
(Gigacage::freeVirtualPages):
(Gigacage::tryAlignedMalloc):
(Gigacage::alignedFree):
(Gigacage::free):
- wtf/Gigacage.h:
(Gigacage::disablePrimitiveGigacage):
(Gigacage::addPrimitiveDisableCallback):
(Gigacage::removePrimitiveDisableCallback):
(Gigacage::name):
(Gigacage::basePtr):
(Gigacage::caged):
(Gigacage::isCaged):
(Gigacage::tryAlignedMalloc):
(Gigacage::alignedFree):
(Gigacage::free):
(Gigacage::disableGigacage): Deleted.
(Gigacage::addDisableCallback): Deleted.
(Gigacage::removeDisableCallback): Deleted.
- 1:58 PM Changeset in webkit [220351] by
-
- 4 edits1 add in trunk/Source/WebCore
[Curl] Add abstraction layer of cookie jar implementation for Curl port
https://bugs.webkit.org/show_bug.cgi?id=174943
Patch by Basuke Suzuki <Basuke Suzuki> on 2017-08-07
Reviewed by Brent Fulgham.
- platform/network/curl/CookieJarCurl.cpp:
(WebCore::CookieJarCurlFileSystem::setCookiesFromDOM):
(WebCore::CookieJarCurlFileSystem::cookiesForDOM):
(WebCore::CookieJarCurlFileSystem::cookieRequestHeaderFieldValue):
(WebCore::CookieJarCurlFileSystem::cookiesEnabled):
(WebCore::CookieJarCurlFileSystem::getRawCookies):
(WebCore::CookieJarCurlFileSystem::deleteCookie):
(WebCore::CookieJarCurlFileSystem::getHostnamesWithCookies):
(WebCore::CookieJarCurlFileSystem::deleteCookiesForHostnames):
(WebCore::CookieJarCurlFileSystem::deleteAllCookies):
(WebCore::CookieJarCurlFileSystem::deleteAllCookiesModifiedSince):
(WebCore::cookiesForDOM):
(WebCore::setCookiesFromDOM):
(WebCore::cookieRequestHeaderFieldValue):
(WebCore::cookiesEnabled):
(WebCore::getRawCookies):
(WebCore::deleteCookie):
(WebCore::getHostnamesWithCookies):
(WebCore::deleteCookiesForHostnames):
(WebCore::deleteAllCookies):
(WebCore::deleteAllCookiesModifiedSince):
- platform/network/curl/CookieJarCurl.h: Added.
- platform/network/curl/CurlContext.cpp:
- platform/network/curl/CurlContext.h:
(WebCore::CurlContext::cookieJar):
- 1:30 PM Changeset in webkit [220350] by
-
- 2 edits in trunk/LayoutTests
Skip workers/wasm-hashset-many.html and workers/wasm-hashset-many-2.html on El Capitan.
https://bugs.webkit.org/show_bug.cgi?id=175102
Unreviewed test gardening.
- platform/mac/TestExpectations:
- 11:58 AM Changeset in webkit [220349] by
-
- 2 edits in branches/safari-604.1-branch/Source/WebCore
Cherry-pick r220248. rdar://problem/33754446
- 11:55 AM Changeset in webkit [220348] by
-
- 2 edits in branches/safari-604.1-branch/Source/WebKit
Cherry-pick r220273. rdar://problem/33732754
- 11:55 AM Changeset in webkit [220347] by
-
- 4 edits in branches/safari-604.1-branch/Source/WebCore
Cherry-pick r220247. rdar://problem/33754440
- 11:42 AM Changeset in webkit [220346] by
-
- 3 edits in trunk/Source/JavaScriptCore
Unreviewed, rolling out r220144.
https://bugs.webkit.org/show_bug.cgi?id=175276
"It did not actually speed things up in the way I expected"
(Requested by saamyjoon on #webkit).
Reverted changeset:
"On memory-constrained iOS devices, reduce the rate at which
the JS heap grows before a GC to try to keep more memory
available for the system"
https://bugs.webkit.org/show_bug.cgi?id=175041
http://trac.webkit.org/changeset/220144
- 11:25 AM Changeset in webkit [220345] by
-
- 2 edits in trunk/LayoutTests
Marking imported/w3c/web-platform-tests/html/browsers/windows/noreferrer-window-name.html as failing on iOS.
https://bugs.webkit.org/show_bug.cgi?id=175273
Unreviewed test gardening.
- platform/ios/TestExpectations:
- 11:22 AM Changeset in webkit [220344] by
-
- 62 edits4 copies in trunk
Implement most of ServiceWorkerContainer::addRegistration.
https://bugs.webkit.org/show_bug.cgi?id=175237
Reviewed by Andy Estes.
LayoutTests/imported/w3c:
- web-platform-tests/FileAPI/historical.https-expected.txt:
- web-platform-tests/background-fetch/interfaces-worker.https-expected.txt:
- web-platform-tests/fetch/api/policies/referrer-no-referrer-service-worker.https-expected.txt:
- web-platform-tests/fetch/api/policies/referrer-origin-service-worker.https-expected.txt:
- web-platform-tests/fetch/api/policies/referrer-origin-when-cross-origin-service-worker.https-expected.txt:
- web-platform-tests/fetch/api/policies/referrer-unsafe-url-service-worker.https-expected.txt:
- web-platform-tests/html/webappapis/scripting/events/messageevent-constructor.https-expected.txt:
- web-platform-tests/html/webappapis/scripting/processing-model-2/integration-with-the-javascript-agent-formalism/canblock-serviceworker.https-expected.txt:
- web-platform-tests/service-workers/cache-storage/serviceworker/cache-add.https-expected.txt:
- web-platform-tests/service-workers/cache-storage/serviceworker/cache-delete.https-expected.txt:
- web-platform-tests/service-workers/cache-storage/serviceworker/cache-keys.https-expected.txt:
- web-platform-tests/service-workers/cache-storage/serviceworker/cache-match.https-expected.txt:
- web-platform-tests/service-workers/cache-storage/serviceworker/cache-matchAll.https-expected.txt:
- web-platform-tests/service-workers/cache-storage/serviceworker/cache-put.https-expected.txt:
- web-platform-tests/service-workers/cache-storage/serviceworker/cache-storage-keys.https-expected.txt:
- web-platform-tests/service-workers/cache-storage/serviceworker/cache-storage-match.https-expected.txt:
- web-platform-tests/service-workers/cache-storage/serviceworker/cache-storage.https-expected.txt:
- web-platform-tests/streams/byte-length-queuing-strategy.serviceworker.https-expected.txt:
- web-platform-tests/streams/count-queuing-strategy.serviceworker.https-expected.txt:
- web-platform-tests/streams/piping/close-propagation-backward.serviceworker.https-expected.txt:
- web-platform-tests/streams/piping/close-propagation-forward.serviceworker.https-expected.txt:
- web-platform-tests/streams/piping/error-propagation-backward.serviceworker.https-expected.txt:
- web-platform-tests/streams/piping/error-propagation-forward.serviceworker.https-expected.txt:
- web-platform-tests/streams/piping/flow-control.serviceworker.https-expected.txt:
- web-platform-tests/streams/piping/general.serviceworker.https-expected.txt:
- web-platform-tests/streams/piping/multiple-propagation.serviceworker.https-expected.txt:
- web-platform-tests/streams/piping/pipe-through.serviceworker.https-expected.txt:
- web-platform-tests/streams/piping/transform-streams.serviceworker.https-expected.txt:
- web-platform-tests/streams/readable-byte-streams/general.serviceworker.https-expected.txt:
- web-platform-tests/streams/readable-streams/bad-strategies.serviceworker.https-expected.txt:
- web-platform-tests/streams/readable-streams/bad-underlying-sources.serviceworker.https-expected.txt:
- web-platform-tests/streams/readable-streams/brand-checks.serviceworker.https-expected.txt:
- web-platform-tests/streams/readable-streams/cancel.serviceworker.https-expected.txt:
- web-platform-tests/streams/readable-streams/count-queuing-strategy-integration.serviceworker.https-expected.txt:
- web-platform-tests/streams/readable-streams/default-reader.serviceworker.https-expected.txt:
- web-platform-tests/streams/readable-streams/floating-point-total-queue-size.serviceworker.https-expected.txt:
- web-platform-tests/streams/readable-streams/garbage-collection.serviceworker.https-expected.txt:
- web-platform-tests/streams/readable-streams/general.serviceworker.https-expected.txt:
- web-platform-tests/streams/readable-streams/pipe-through.serviceworker.https-expected.txt:
- web-platform-tests/streams/readable-streams/readable-stream-reader.serviceworker.https-expected.txt:
- web-platform-tests/streams/readable-streams/tee.serviceworker.https-expected.txt:
- web-platform-tests/streams/readable-streams/templated.serviceworker.https-expected.txt:
Source/WebCore:
No new tests (Covered by changes to existing tests).
There's still so much supporting infrastructure to add with these early patches
that I'm still moving them in baby steps for now, hence not implementing
register() all in one shot.
Things will start moving very quickly once we no longer need to add lots of new
primitives in each change.
- CMakeLists.txt:
- DerivedSources.make:
- WebCore.xcodeproj/project.pbxproj:
- page/DOMWindow.cpp:
(WebCore::DOMWindow::navigator const):
- page/Navigator.cpp:
(WebCore::Navigator::Navigator): Make the constructor take a ScriptExecutionContext for
creation of objects where its import (e.g. ServiceWorkerContainer).
- page/Navigator.h:
- page/WorkerNavigator.cpp:
(WebCore::WorkerNavigator::WorkerNavigator): Ditto.
- page/WorkerNavigator.h:
- page/NavigatorBase.cpp:
(WebCore::NavigatorBase::NavigatorBase): Create the ServiceWorkerContainer upfront with
the passed-in ScriptExecutionContext.
(WebCore::NavigatorBase::serviceWorker):
- page/NavigatorBase.h:
Make ServiceWorkerContainer into an ActiveDOMObject. This will eventually be necessary for
Document suspension reasons, but is also necessary because it also needs to be a
ContextDestructionObserver (which ActiveDOMObject is):
- workers/ServiceWorkerContainer.cpp:
(WebCore::ServiceWorkerContainer::ServiceWorkerContainer):
(WebCore::ServiceWorkerContainer::ready):
(WebCore::ServiceWorkerContainer::addRegistration):
(WebCore::ServiceWorkerContainer::getRegistration):
(WebCore::ServiceWorkerContainer::getRegistrations):
(WebCore::ServiceWorkerContainer::activeDOMObjectName const):
(WebCore::ServiceWorkerContainer::canSuspendForDocumentSuspension const):
(WebCore::rejectLater): Deleted.
(WebCore::ServiceWorkerContainer::eventTargetInterface const): Deleted.
(WebCore::ServiceWorkerContainer::scriptExecutionContext const): Deleted.
- workers/ServiceWorkerContainer.h:
- workers/ServiceWorkerContainer.idl:
Add updateViaCache, etc:
- workers/ServiceWorkerRegistration.cpp:
(WebCore::ServiceWorkerRegistration::updateViaCache const):
(WebCore::ServiceWorkerRegistration::update):
(WebCore::ServiceWorkerRegistration::unregister):
- workers/ServiceWorkerRegistration.h:
- workers/ServiceWorkerRegistration.idl:
- workers/ServiceWorkerUpdateViaCache.h: Copied from Source/WebCore/workers/ServiceWorkerRegistration.idl.
- workers/ServiceWorkerUpdateViaCache.idl: Copied from Source/WebCore/workers/ServiceWorkerRegistration.idl.
- workers/WorkerGlobalScope.cpp:
(WebCore::WorkerGlobalScope::navigator):
(WebCore::WorkerGlobalScope::navigator const): Deleted.
- workers/WorkerGlobalScope.h:
- workers/WorkerType.h: Copied from Source/WebCore/workers/ServiceWorkerRegistration.idl.
- workers/WorkerType.idl: Copied from Source/WebCore/workers/ServiceWorkerRegistration.idl.
- 10:06 AM Changeset in webkit [220343] by
-
- 3 edits in trunk/LayoutTests
Rebaseline js/dom/global-constructors-attributes.html.
Unreviewed test gardening.
- platform/mac-elcapitan/js/dom/global-constructors-attributes-expected.txt:
- platform/mac/js/dom/global-constructors-attributes-expected.txt:
- 9:44 AM Changeset in webkit [220342] by
-
- 12 edits1 delete in trunk
Unreviewed, rolling out r220299.
This change caused LayoutTest inspector/dom-debugger/dom-
breakpoints.html to fail.
Reverted changeset:
"Web Inspector: capture async stack trace when workers/main
context posts a message"
https://bugs.webkit.org/show_bug.cgi?id=167084
http://trac.webkit.org/changeset/220299
- 9:42 AM Changeset in webkit [220341] by
-
- 2 edits in branches/safari-604-branch/Source/WebKit
Cherry-pick r220273. rdar://problem/33732762
- 9:42 AM Changeset in webkit [220340] by
-
- 6 edits in branches/safari-604-branch
Cherry-pick r220185. rdar://problem/33713534
- 9:42 AM Changeset in webkit [220339] by
-
- 4 edits3 adds in branches/safari-604-branch
Cherry-pick r220163. rdar://problem/33711032
- 9:42 AM Changeset in webkit [220338] by
-
- 2 edits in branches/safari-604-branch/Source/WebCore
Cherry-pick r220153. rdar://problem/33711047
- 9:32 AM Changeset in webkit [220337] by
-
- 6 edits in trunk/Source/WebCore
Cleanup: simplify WebSockets code for RuntimeEnabledFeatures
https://bugs.webkit.org/show_bug.cgi?id=175190
Reviewed by Sam Weinig.
This runtime enabled flag is not used by anything. It was added
for V8 in <https://bugs.webkit.org/show_bug.cgi?id=29896>.
- Modules/websockets/WebSocket.cpp:
(WebCore::WebSocket::setIsAvailable): Deleted.
(WebCore::WebSocket::isAvailable): Deleted.
- Modules/websockets/WebSocket.h:
- Modules/websockets/WebSocket.idl:
- page/RuntimeEnabledFeatures.cpp:
(WebCore::RuntimeEnabledFeatures::webSocketEnabled const): Deleted.
- page/RuntimeEnabledFeatures.h:
- 9:29 AM Changeset in webkit [220336] by
-
- 24 edits in trunk
Remove CANVAS_PATH compilation guard
https://bugs.webkit.org/show_bug.cgi?id=175207
Reviewed by Sam Weinig.
.:
- Source/cmake/OptionsGTK.cmake:
- Source/cmake/OptionsMac.cmake:
- Source/cmake/OptionsWin.cmake:
- Source/cmake/WebKitFeatures.cmake:
- Source/cmake/tools/vsprops/FeatureDefines.props:
- Source/cmake/tools/vsprops/FeatureDefinesCairo.props:
Source/JavaScriptCore:
- Configurations/FeatureDefines.xcconfig:
Source/WebCore:
- Configurations/FeatureDefines.xcconfig:
- html/canvas/DOMPath.h:
- html/canvas/DOMPath.idl:
Source/WebCore/PAL:
- Configurations/FeatureDefines.xcconfig:
Source/WebKit:
- Configurations/FeatureDefines.xcconfig:
Source/WebKitLegacy/mac:
- Configurations/FeatureDefines.xcconfig:
Source/WTF:
- wtf/FeatureDefines.h:
Tools:
- Scripts/webkitperl/FeatureList.pm:
- TestWebKitAPI/Configurations/FeatureDefines.xcconfig:
- 9:22 AM Changeset in webkit [220335] by
-
- 2 edits in trunk/Tools
[Win] Use Visual Studio 2017 if installed.
https://bugs.webkit.org/show_bug.cgi?id=175188
Reviewed by Sam Weinig.
If installed, use VS2017 to build WebKit. If not, fall back to VS2015.
- Scripts/webkitdirs.pm:
(visualStudioInstallDir):
(msBuildInstallDir):
- 8:44 AM Changeset in webkit [220334] by
-
- 4 edits in trunk/Source
[WPE][CMake] Only pass pkg-config CFLAGS from (E)GL when not using libepoxy.
https://bugs.webkit.org/show_bug.cgi?id=175125
Reviewed by Michael Catanzaro.
Source/WebCore:
No change of behavior, covered by existing tests.
- CMakeLists.txt: Instead of negating 3 times the use of libepoxy
move the logic for (E)GL detection inside an else() block.
Source/WebKit:
- CMakeLists.txt: When using libepoxy avoid passing the CFLAGS from (E)GL,
and instead pass the libepoxy ones.
- 6:14 AM Changeset in webkit [220333] by
-
- 2 edits in trunk/Source/WebCore
REGRESSION (r219121): Airmail 3 prints header part only.
https://bugs.webkit.org/show_bug.cgi?id=175258
<rdar://problem/33601173>
Reviewed by Andreas Kling.
When a WK1 WebViw is printed via AppKit view hierarchy it won't explictly set the page width
but uses the existing width. r219121 assumes that all printing code paths set the page width.
No test, there appears to be no good way to test AppKit printing behaviors without adding complicated
new testing infrastructure.
- rendering/RenderView.cpp:
(WebCore::RenderView::layout):
If we are in printing layout and don't have page width set yet then use the current view width.
This matches the behavior prior r219121.
- 6:08 AM Changeset in webkit [220332] by
-
- 2 edits in trunk/LayoutTests
Remove obsolete failure expectations for wpt browsers tests.
https://bugs.webkit.org/show_bug.cgi?id=175073
Unreviewed test gardening.
Patch by Ms2ger <Ms2ger@igalia.com> on 2017-08-07
- 3:03 AM WebKitGTK/2.16.x edited by
- (diff)
- 2:53 AM Changeset in webkit [220331] by
-
- 2 edits in trunk/Source/WebKit
[GTK][WPE] CFLAGS from pkg-config for (E)GL are not passed to WebKit
https://bugs.webkit.org/show_bug.cgi?id=175125
Unreviewed follow-up fix after r220326.
- CMakeLists.txt: Move the block appending to WebKit2_LIBRARIES
below where the list itself is created.
- 12:03 AM Changeset in webkit [220330] by
-
- 2 edits in trunk/Source/JavaScriptCore
REGRESSION: wasm.yaml/wasm/js-api/dont-mmap-zero-byte-memory.js failing on JSC Debug bots
https://bugs.webkit.org/show_bug.cgi?id=175256
Reviewed by Saam Barati.
The check in createFromBytes just needed to check that the buffer was not null before
calling isCaged.
- runtime/ArrayBuffer.cpp:
(JSC::ArrayBuffer::createFromBytes):
Aug 6, 2017:
- 11:06 PM Changeset in webkit [220329] by
-
- 24 edits3 adds in trunk
[GTK][WPE] Add API to provide browser information required by automation
https://bugs.webkit.org/show_bug.cgi?id=175130
Source/JavaScriptCore:
Reviewed by Brian Burg.
Add browserName and browserVersion to RemoteInspector::Client::Capabilities and virtual methods to the Client to
get them.
- inspector/remote/RemoteInspector.cpp:
(Inspector::RemoteInspector::updateClientCapabilities): Update also browserName and browserVersion.
- inspector/remote/RemoteInspector.h:
- inspector/remote/glib/RemoteInspectorGlib.cpp:
(Inspector::RemoteInspector::requestAutomationSession): Call updateClientCapabilities() after the session is
requested to ensure they are updated before StartAutomationSession reply is sent.
- inspector/remote/glib/RemoteInspectorServer.cpp: Add browserName and browserVersion as return values of
StartAutomationSession mesasage.
Source/WebDriver:
Reviewed by Brian Burg.
- Session.cpp:
(WebDriver::Session::createTopLevelBrowsingContext): Check if startAutomationSession and complete the command
with error in that case.
- SessionHost.h:
- glib/SessionHostGlib.cpp:
(WebDriver::SessionHost::matchCapabilities): Match the capabilities that are known only after the browser has
been launched.
(WebDriver::SessionHost::startAutomationSession): Handle the StartAutomationSession response, extracting the
capabilities and calling matchCapabilities() to match them.
(WebDriver::SessionHost::setTargetList): Return early if the session was rejected before due to invalid
capabilities.
Source/WebKit:
Reviewed by Michael Catanzaro.
When a new automation session is started, the web driver receives some required capabilities from the client,
like browser name and version. The session should be rejected if those required capabilities don't match with
the actual browser that is launched. We don't know that information in WebKit, so we need to add API so that
users can provide it when a new session request is made. This patch adds boxed object WebKitApplicationInfo that
can be used to set the application name and version. This object can be set to a WebKitAutomationSession when
WebKitWebContext::automation-started signal is emitted.
- PlatformGTK.cmake:
- PlatformWPE.cmake:
- UIProcess/API/glib/WebKitApplicationInfo.cpp: Added.
(webkit_application_info_new):
(webkit_application_info_ref):
(webkit_application_info_unref):
(webkit_application_info_set_name):
(webkit_application_info_get_name):
(webkit_application_info_set_version):
(webkit_application_info_get_version):
- UIProcess/API/glib/WebKitAutomationSession.cpp:
(webkitAutomationSessionDispose):
(webkit_automation_session_class_init):
(webkitAutomationSessionGetBrowserName):
(webkitAutomationSessionGetBrowserVersion):
(webkit_automation_session_set_application_info):
(webkit_automation_session_get_application_info):
- UIProcess/API/glib/WebKitAutomationSessionPrivate.h:
- UIProcess/API/glib/WebKitWebContext.cpp:
- UIProcess/API/gtk/WebKitApplicationInfo.h: Added.
- UIProcess/API/gtk/WebKitAutomationSession.h:
- UIProcess/API/gtk/docs/webkit2gtk-4.0-sections.txt:
- UIProcess/API/gtk/docs/webkit2gtk-docs.sgml:
- UIProcess/API/gtk/webkit2.h:
- UIProcess/API/wpe/WebKitApplicationInfo.h: Added.
- UIProcess/API/wpe/WebKitAutomationSession.h:
- UIProcess/API/wpe/webkit.h:
Tools:
Reviewed by Michael Catanzaro.
- MiniBrowser/gtk/main.c:
(automationStartedCallback): Set browser information when a new automation session is started.
- TestWebKitAPI/Tests/WebKitGLib/TestAutomationSession.cpp:
(testAutomationSessionApplicationInfo):
(beforeAll):
- 8:24 PM Changeset in webkit [220328] by
-
- 6 edits1 delete in trunk/Tools
Unreviewed, rolling out r220295.
This change introduced 4 errors in webkitpy tests.
Reverted changeset:
"[XCode] webkit-patch should run sort-Xcode-project-file"
https://bugs.webkit.org/show_bug.cgi?id=174036
http://trac.webkit.org/changeset/220295
- 8:00 PM Changeset in webkit [220327] by
-
- 2 edits in trunk/Tools
Disable API test NowPlayingControlsTests.NowPlayingControlsIOS.
Unreviewed test gardening.
- TestWebKitAPI/Tests/WebKit2Cocoa/NowPlayingControlsTests.mm:
(TestWebKitAPI::TEST):
- 5:06 PM WebKitGTK/2.16.x edited by
- (diff)
- 4:34 PM Changeset in webkit [220326] by
-
- 2 edits in trunk/Source/WebKit
[GTK][WPE] CFLAGS from pkg-config for (E)GL are not passed to WebKit
https://bugs.webkit.org/show_bug.cgi?id=175125
Reviewed by Michael Catanzaro.
Some platforms define cflags on the (E)GL pkg-config files that we
need to pass to the build system when including the (E)GL headers.
And we are already doing this when building WebCore (after r184954).
But now we need to do this also on WebKit (former WebKit2) because
we include (E)GL headers (and make use of them) on the UIProcess.
- CMakeLists.txt:
- 4:24 PM Changeset in webkit [220325] by
-
- 3 edits in trunk/Websites/webkit.org
2017-08-06 James Craig <jcraig@apple.com>
Update prefers-reduced-motion demos to link back to blog post
https://bugs.webkit.org/show_bug.cgi?id=175251
Unreviewed, added some cross links to older demo files.
- blog-files/prefers-reduced-motion/axi.htm:
- blog-files/prefers-reduced-motion/prm.htm:
- 10:57 AM Changeset in webkit [220324] by
-
- 4 edits1 add in trunk
Promise resolve and reject function should have length = 1
https://bugs.webkit.org/show_bug.cgi?id=175242
Reviewed by Saam Barati.
JSTests:
- stress/builtin-function-length.js: Added.
(shouldBe):
(shouldThrow):
(shouldBe.JSON.stringify.Object.getOwnPropertyDescriptor):
(shouldBe.JSON.stringify.Object.getOwnPropertyNames.Array.prototype.filter.sort):
Source/JavaScriptCore:
Previously we have separate system for "length" and "name" for builtin functions.
The builtin functions do not use lazy reifying system. Instead, they have direct
properties when instantiating it. While the function created for properties (like
Array.prototype.filter) is created by JSFunction::createBuiltin(), function inside
these builtin functions are just created by JSFunction::create(). Since it does
not set any values for "length", these functions do not have "length" property.
So, the resolve and reject functions passed to Promise's executor do not have
"length" property.
This patch make builtin functions use standard lazy reifying system for "length".
So, "length" property of the builtin function just works as if the normal functions
do.
- runtime/JSFunction.cpp:
(JSC::JSFunction::createBuiltinFunction):
(JSC::JSFunction::getOwnPropertySlot):
(JSC::JSFunction::getOwnNonIndexPropertyNames):
(JSC::JSFunction::put):
(JSC::JSFunction::deleteProperty):
(JSC::JSFunction::defineOwnProperty):
(JSC::JSFunction::reifyLazyPropertyIfNeeded):
(JSC::JSFunction::reifyLazyPropertyForHostOrBuiltinIfNeeded):
(JSC::JSFunction::reifyLazyLengthIfNeeded):
(JSC::JSFunction::reifyLazyBoundNameIfNeeded):
(JSC::JSFunction::reifyBoundNameIfNeeded): Deleted.
- runtime/JSFunction.h:
- 6:04 AM Changeset in webkit [220323] by
-
- 8 edits1 add in trunk
[Next] Async iteration - Implement Async Generator - parser
https://bugs.webkit.org/show_bug.cgi?id=175210
Reviewed by Yusuke Suzuki.
JSTests:
- stress/async-await-syntax.js:
(testTopLevelAsyncAwaitSyntaxSloppyMode.testSyntax):
- stress/async-iteration-syntax.js: Added.
(assert):
(checkSyntax):
(checkSyntaxError):
(checkSimpleAsyncGeneratorSloppyMode):
(checkSimpleAsyncGeneratorStrictMode):
(checkNestedAsyncGenerators):
(checkSimpleAsyncGeneratorSyntaxErrorInStrictMode):
- stress/generator-class-methods-syntax.js:
Source/JavaScriptCore:
Current implementation is draft version of Async Iteration.
Link to spec https://tc39.github.io/proposal-async-iteration/
Current patch implement only parser part of the Async generator
Runtime part will be in next ptches
- parser/ASTBuilder.h:
(JSC::ASTBuilder::createFunctionMetadata):
- parser/Parser.cpp:
(JSC::getAsynFunctionBodyParseMode):
(JSC::Parser<LexerType>::parseInner):
(JSC::Parser<LexerType>::parseAsyncFunctionSourceElements):
(JSC::Parser<LexerType>::parseAsyncGeneratorFunctionSourceElements):
(JSC::stringArticleForFunctionMode):
(JSC::stringForFunctionMode):
(JSC::Parser<LexerType>::parseFunctionInfo):
(JSC::Parser<LexerType>::parseAsyncFunctionDeclaration):
(JSC::Parser<LexerType>::parseClass):
(JSC::Parser<LexerType>::parseProperty):
(JSC::Parser<LexerType>::parsePropertyMethod):
(JSC::Parser<LexerType>::parseAsyncFunctionExpression):
- parser/Parser.h:
(JSC::Scope::setSourceParseMode):
- parser/ParserModes.h:
(JSC::isFunctionParseMode):
(JSC::isAsyncFunctionParseMode):
(JSC::isAsyncArrowFunctionParseMode):
(JSC::isAsyncGeneratorFunctionParseMode):
(JSC::isAsyncFunctionOrAsyncGeneratorWrapperParseMode):
(JSC::isAsyncFunctionWrapperParseMode):
(JSC::isAsyncFunctionBodyParseMode):
(JSC::isGeneratorMethodParseMode):
(JSC::isAsyncMethodParseMode):
(JSC::isAsyncGeneratorMethodParseMode):
(JSC::isMethodParseMode):
(JSC::isGeneratorOrAsyncFunctionBodyParseMode):
(JSC::isGeneratorOrAsyncFunctionWrapperParseMode):
Aug 5, 2017:
- 9:43 PM Changeset in webkit [220322] by
-
- 18 edits2 adds in trunk
REGRESSION (r219895-219897): Number of leaks on Open Source went from 9240 to 235983 and is now at 302372
https://bugs.webkit.org/show_bug.cgi?id=175083
Reviewed by Oliver Hunt.
Source/JavaScriptCore:
This fixes the leak by making MarkedBlock::specializedSweep call destructors when the block is empty,
even if we are using the pop path.
Also, this fixes HeapCellInlines.h to no longer include MarkedBlockInlines.h. That's pretty
important, since MarkedBlockInlines.h is the GC's internal guts - we don't want to have to recompile
the world just because we changed it.
Finally, this adds a new testing SPI for waiting for all VMs to finish destructing. This makes it
easier to debug leaks.
- bytecode/AccessCase.cpp:
- bytecode/PolymorphicAccess.cpp:
- heap/HeapCell.cpp:
(JSC::HeapCell::isLive):
- heap/HeapCellInlines.h:
(JSC::HeapCell::isLive): Deleted.
- heap/MarkedAllocator.cpp:
(JSC::MarkedAllocator::tryAllocateWithoutCollecting):
(JSC::MarkedAllocator::endMarking):
- heap/MarkedBlockInlines.h:
(JSC::MarkedBlock::Handle::specializedSweep):
- jit/AssemblyHelpers.cpp:
- jit/Repatch.cpp:
- runtime/TestRunnerUtils.h:
- runtime/VM.cpp:
(JSC::waitForVMDestruction):
(JSC::VM::~VM):
Source/WTF:
Adds a classic ReadWriteLock class. I wrote my own because I can never remember if the pthread one is
guaranted to bias in favor of writers or not.
- WTF.xcodeproj/project.pbxproj:
- wtf/Condition.h:
(WTF::ConditionBase::construct):
(WTF::Condition::Condition):
- wtf/Lock.h:
(WTF::LockBase::construct):
(WTF::Lock::Lock):
- wtf/ReadWriteLock.cpp: Added.
(WTF::ReadWriteLockBase::construct):
(WTF::ReadWriteLockBase::readLock):
(WTF::ReadWriteLockBase::readUnlock):
(WTF::ReadWriteLockBase::writeLock):
(WTF::ReadWriteLockBase::writeUnlock):
- wtf/ReadWriteLock.h: Added.
(WTF::ReadWriteLockBase::ReadLock::tryLock):
(WTF::ReadWriteLockBase::ReadLock::lock):
(WTF::ReadWriteLockBase::ReadLock::unlock):
(WTF::ReadWriteLockBase::WriteLock::tryLock):
(WTF::ReadWriteLockBase::WriteLock::lock):
(WTF::ReadWriteLockBase::WriteLock::unlock):
(WTF::ReadWriteLockBase::read):
(WTF::ReadWriteLockBase::write):
(WTF::ReadWriteLock::ReadWriteLock):
Tools:
Leaks results are super confusing if leaks runs while some VMs are destructing. This calls a new SPI
to wait for VM destructions to finish before running the next test. This makes it easier to
understand leaks results from workers tests, and leads to fewer reported leaks.
- DumpRenderTree/mac/DumpRenderTree.mm:
(runTest):
- 4:48 PM Changeset in webkit [220321] by
-
- 2 edits in trunk/Tools
check-webkit-style: fix path-specific rules for WebKit2 rename
https://bugs.webkit.org/show_bug.cgi?id=175182
Patch by Yoshiaki Jitsukawa <Yoshiaki.Jitsukawa@sony.com> on 2017-08-05
Reviewed by David Kilzer.
- Scripts/webkitpy/style/checker.py:
- 3:11 PM Changeset in webkit [220320] by
-
- 15 edits in trunk
[Fetch API] Response should keep all ResourceResponse information
https://bugs.webkit.org/show_bug.cgi?id=175099
Patch by Youenn Fablet <youenn@apple.com> on 2017-08-05
Reviewed by Sam Weinig.
Source/WebCore:
No change of behavior, covered by existing tests.
Disabling filtering of resource response at DocumentThreadableLoader for fetch API and doing the filtering at FetchResponse level.
This requires passing the tainting parameter to FetchResponse. For that purpose, we store the tainting on the ResourceResponse itself.
This allows mimicking the concept of internal response from the fetch spec.
This might be useful for future developments related to caching the responses.
The body is now also stored in FetchResponse so a flag is added to ensure we only expose the body if allowed.
Changing storage of opaque redirect information to keep the redirection information in the response.
- Modules/fetch/FetchBodyOwner.cpp:
(WebCore::FetchBodyOwner::blob):
(WebCore::FetchBodyOwner::consumeNullBody):
- Modules/fetch/FetchBodyOwner.h:
- Modules/fetch/FetchLoader.cpp:
(WebCore::FetchLoader::start):
- Modules/fetch/FetchResponse.cpp:
(WebCore::FetchResponse::BodyLoader::didReceiveResponse):
(WebCore::FetchResponse::consume):
(WebCore::FetchResponse::consumeBodyAsStream):
(WebCore::FetchResponse::createReadableStreamSource):
- Modules/fetch/FetchResponse.h:
- loader/DocumentThreadableLoader.cpp:
(WebCore::DocumentThreadableLoader::responseReceived):
(WebCore::DocumentThreadableLoader::didReceiveResponse):
(WebCore::DocumentThreadableLoader::didFinishLoading):
(WebCore::DocumentThreadableLoader::loadRequest):
- loader/DocumentThreadableLoader.h:
- loader/SubresourceLoader.cpp:
(WebCore::SubresourceLoader::willSendRequestInternal):
- loader/cache/CachedResource.cpp:
(WebCore::CachedResource::setBodyDataFrom):
(WebCore::CachedResource::setResponse):
- platform/network/ResourceResponseBase.cpp:
(WebCore::ResourceResponseBase::crossThreadData const):
(WebCore::ResourceResponseBase::fromCrossThreadData):
(WebCore::ResourceResponseBase::filter):
- platform/network/ResourceResponseBase.h:
(WebCore::ResourceResponseBase::setTainting):
(WebCore::ResourceResponseBase::tainting const):
(WebCore::ResourceResponseBase::encode const):
(WebCore::ResourceResponseBase::decode):
LayoutTests:
Updating test now that we are no longer cancelling the load in case of opaque responses.
- http/tests/inspector/network/fetch-network-data-expected.txt:
- http/tests/inspector/network/fetch-network-data.html:
- 11:59 AM Changeset in webkit [220319] by
-
- 10 edits in trunk/LayoutTests/imported/w3c
[Cache API] Add Cache and CacheStorage IDL definitions
https://bugs.webkit.org/show_bug.cgi?id=175201
<rdar://problem/33738001>
Unreviewed.
Rebasing test expectations after http://trac.webkit.org/changeset/220310.
Patch by Youenn Fablet <youenn@apple.com> on 2017-08-05
- web-platform-tests/service-workers/cache-storage/serviceworker/cache-add.https-expected.txt:
- web-platform-tests/service-workers/cache-storage/serviceworker/cache-delete.https-expected.txt:
- web-platform-tests/service-workers/cache-storage/serviceworker/cache-keys.https-expected.txt:
- web-platform-tests/service-workers/cache-storage/serviceworker/cache-match.https-expected.txt:
- web-platform-tests/service-workers/cache-storage/serviceworker/cache-matchAll.https-expected.txt:
- web-platform-tests/service-workers/cache-storage/serviceworker/cache-put.https-expected.txt:
- web-platform-tests/service-workers/cache-storage/serviceworker/cache-storage-keys.https-expected.txt:
- web-platform-tests/service-workers/cache-storage/serviceworker/cache-storage-match.https-expected.txt:
- web-platform-tests/service-workers/cache-storage/serviceworker/cache-storage.https-expected.txt:
- 7:10 AM Changeset in webkit [220318] by
-
- 5 edits1 delete in trunk/Source/JavaScriptCore
Move DFG::OSRExitCompiler methods into DFG::OSRExit [step 3].
https://bugs.webkit.org/show_bug.cgi?id=175228
<rdar://problem/33735737>
Reviewed by Saam Barati.
Merge the 32-bit OSRExit::compileExit() method into the 64-bit version, and
delete OSRExit32_64.cpp.
- CMakeLists.txt:
- JavaScriptCore.xcodeproj/project.pbxproj:
- dfg/DFGOSRExit.cpp:
(JSC::DFG::OSRExit::compileExit):
- dfg/DFGOSRExit32_64.cpp: Removed.
- jit/GPRInfo.h:
(JSC::JSValueSource::payloadGPR const):
- 4:14 AM Changeset in webkit [220317] by
-
- 9 edits in trunk/Source
WebDriver: Implement page load strategy
https://bugs.webkit.org/show_bug.cgi?id=175183
Reviewed by Brian Burg.
Source/WebDriver:
Validate and parse page load strategy when processing capabilities.
- Capabilities.h:
- Session.cpp:
(WebDriver::Session::pageLoadStrategyString const): Helper to get the page load strategy as a String to be
passed to Automation.
(WebDriver::Session::go): Pass page load strategy if present.
(WebDriver::Session::back): Ditto.
(WebDriver::Session::forward): Ditto.
(WebDriver::Session::refresh): Ditto.
(WebDriver::Session::waitForNavigationToComplete): Ditto.
- Session.h:
- WebDriverService.cpp:
(WebDriver::deserializePageLoadStrategy):
(WebDriver::WebDriverService::parseCapabilities const):
(WebDriver::WebDriverService::validatedCapabilities const):
(WebDriver::WebDriverService::newSession):
Source/WebKit:
Split pending navigation maps into normal and eager, and use one or the other depending on the received page
load strategy. We need to keep different maps for every page load strategy because every command could use a
different strategy.
- UIProcess/Automation/WebAutomationSession.cpp:
(WebKit::WebAutomationSession::waitForNavigationToComplete): Extract page load strategy from parameter and pass
it to waitForNavigationToCompleteOnPage() and waitForNavigationToCompleteOnFrame().
(WebKit::WebAutomationSession::waitForNavigationToCompleteOnPage): Return early if page load strategy is
none. Otherwise at the pening callback to the normal or eager map depeding on the page load straegy.
(WebKit::WebAutomationSession::waitForNavigationToCompleteOnFrame): Ditto.
(WebKit::respondToPendingNavigationCallbacksWithTimeout): Helper to send pening navigation callback in case of
timeout failure.
(WebKit::WebAutomationSession::loadTimerFired): Call finishPendingNavigationsWithTimeoutFailure() with all the maps.
(WebKit::WebAutomationSession::navigateBrowsingContext): Extract page load strategy from parameter and pass it
to waitForNavigationToCompleteOnPage().
(WebKit::WebAutomationSession::goBackInBrowsingContext): Ditto.
(WebKit::WebAutomationSession::goForwardInBrowsingContext): Ditto.
(WebKit::WebAutomationSession::reloadBrowsingContext): Ditto.
(WebKit::WebAutomationSession::navigationOccurredForFrame): Use the normal maps.
(WebKit::WebAutomationSession::documentLoadedForFrame): Stop timeout timer and dispatch eager pending navigations.
- UIProcess/Automation/WebAutomationSession.h:
- UIProcess/WebPageProxy.cpp:
(WebKit::WebPageProxy::didFinishDocumentLoadForFrame): Notify the automation session.
- 3:32 AM Changeset in webkit [220316] by
-
- 2 edits in trunk/Source/WebDriver
Unreviewed. Try to fix build with clang after r220315.
- WebDriverService.cpp:
(WebDriver::WebDriverService::validatedCapabilities const):
(WebDriver::WebDriverService::mergeCapabilities const):
- 2:27 AM Changeset in webkit [220315] by
-
- 7 edits in trunk/Source/WebDriver
WebDriver: properly handle capabilities and process firstMatch too
https://bugs.webkit.org/show_bug.cgi?id=174618
Reviewed by Brian Burg.
Implement processing of capabilities following the spec. This patch adds validation, merging and matching of
capabilities.
7.2 Processing Capabilities.
https://w3c.github.io/webdriver/webdriver-spec.html#processing-capabilities
- Capabilities.h: Make all capabilities optional and move Timeouts struct here.
- Session.h:
- WebDriverService.cpp:
(WebDriver::deserializeTimeouts): Helper to extract timeouts from JSON object.
(WebDriver::WebDriverService::parseCapabilities const): At this point capabilities have already been validated,
so we just need to get them without checking the value type.
(WebDriver::WebDriverService::validatedCapabilities const): Validate the given capabilities, ensuring types of
values are the expected one.
(WebDriver::WebDriverService::mergeCapabilities const): Merge the alwaysMatch and firstMatch capabilities into a
single object ensuring that the same capability is not in both.
(WebDriver::WebDriverService::matchCapabilities const): Try to match the merged capabilities againt the platform
expected capabilities.
(WebDriver::WebDriverService::processCapabilities const): Validate, merge and match the capabilities.
(WebDriver::WebDriverService::newSession): Use processCapabilities(). Also initialize the timeouts from
capabilities and add all capabilities to the command result.
(WebDriver::WebDriverService::setTimeouts): Use deserializeTimeouts().
- WebDriverService.h:
- glib/SessionHostGlib.cpp:
(WebDriver::SessionHost::launchBrowser): Updated to properly access the capabilities that are now optional.
(WebDriver::SessionHost::startAutomationSession): Add FIXME.
- gtk/WebDriverServiceGtk.cpp:
(WebDriver::WebDriverService::platformCapabilities): Return the Capabilities with the known required ones filled.
(WebDriver::WebDriverService::platformValidateCapability const): Validate webkitgtk:browserOptions.
(WebDriver::WebDriverService::platformMatchCapability const): This does nothing for now.
(WebDriver::WebDriverService::platformCompareBrowserVersions): Compare the given browser versions.
(WebDriver::WebDriverService::platformParseCapabilities const): Updated now that capabilites have already been
validated before.
- 1:11 AM Changeset in webkit [220314] by
-
- 17 edits in trunk/Source
WebDriver: use in-view center point for clicks instead of bounding box center point
https://bugs.webkit.org/show_bug.cgi?id=174863
Reviewed by Simon Fraser.
Source/WebCore:
Make DOMRect, and FloatPoint::narrowPrecision() available to WebKit layer. Also add
FrameView::clientToDocumentPoint().
- WebCore.xcodeproj/project.pbxproj:
- dom/Element.h:
- page/FrameView.h:
- platform/graphics/FloatPoint.h:
Source/WebDriver:
The center of the element bounding box is not always part of the element, like in multiline links, for example.
11.1 Element Interactability.
https://www.w3.org/TR/webdriver/#dfn-in-view-center-point
- CommandResult.cpp:
(WebDriver::CommandResult::httpStatusCode): Add ElementClickIntercepted and ElementNotInteractable errors.
(WebDriver::CommandResult::errorString): Ditto.
- CommandResult.h: Ditto.
- Session.cpp:
(WebDriver::Session::computeElementLayout): Get the in-view center point and isObscured from the result too.
(WebDriver::Session::getElementRect): Ignore in-view center point and isObscured.
(WebDriver::Session::elementClick): Fail in case the element is not interactable or is obscured.
- Session.h:
Source/WebKit:
Change computeElementLayout to also return the in-view center point and whether it's obscured by another
element.
- UIProcess/Automation/Automation.json: Add optional inViewCenterPoint to the result and isObscured.
- UIProcess/Automation/WebAutomationSession.cpp:
(WebKit::WebAutomationSession::didComputeElementLayout): Handle inViewCenterPoint and isObscured.
- UIProcess/Automation/WebAutomationSession.h:
- UIProcess/Automation/WebAutomationSession.messages.in:
- WebProcess/Automation/WebAutomationSessionProxy.cpp:
(WebKit::elementInViewClientCenterPoint): Get the client in-view center point and whether it's obscured
according to the spec.
(WebKit::WebAutomationSessionProxy::computeElementLayout): Pass inViewCenterPoint and isObscured to
DidComputeElementLayout message.
- 1:08 AM Changeset in webkit [220313] by
-
- 3 edits6 adds in trunk
getClientRects doesn't work with list box option elements
https://bugs.webkit.org/show_bug.cgi?id=175016
Reviewed by Darin Adler.
Source/WebCore:
Since HTMLOptionElement and HTMLOptGroupElement don't have a renderer, we are always returning an empty list
from getClientRects. This is working fine in both chromium and firefox, option elements return its own bounding
box and group elements return the bounding box of the group label and all its children items.
Test: fast/dom/HTMLSelectElement/listbox-items-client-rects.html
- dom/Element.cpp:
(WebCore::listBoxElementBoundingBox): Helper function to return the bounding box of a HTMLOptionElement or
HTMLOptGroupElement element.
(WebCore::Element::getClientRects): Use listBoxElementBoundingBox() in case of HTMLOptionElement or
HTMLOptGroupElement.
(WebCore::Element::boundingClientRect): Ditto.
LayoutTests:
Add new test to check list box option elements client rects.
- fast/dom/HTMLSelectElement/listbox-items-client-rects-expected.txt: Added.
- fast/dom/HTMLSelectElement/listbox-items-client-rects.html: Added.
- platform/ios-simulator-wk2/fast/dom/HTMLSelectElement/listbox-items-client-rects-expected.txt: Added.
- 12:57 AM Changeset in webkit [220312] by
-
- 2 edits in trunk/Source/WebKit
Web Automation: files selected for upload should be matched against 'accept' attribute values case-insensitively
https://bugs.webkit.org/show_bug.cgi?id=175191
<rdar://problem/33725790>
Reviewed by Carlos Garcia Campos.
Values of the "accept" attribute are to be compared in a case-insensitive manner, per
https://html.spec.whatwg.org/multipage/input.html#file-upload-state-(type=file)
Except for converting MIME types and extensions to lowercase, most of these changes
were lost in a rebase prior to landing the patch.
- UIProcess/Automation/WebAutomationSession.cpp:
(WebKit::fileCanBeAcceptedForUpload): Fix some issues:
- Handle a file ending in a period.
- Handle MIME type inference failing.
- Convert extensions and MIMEs to lower case, per specification.
(WebKit::WebAutomationSession::handleRunOpenPanel):
- Strip the leading period from file extensions.
- These range converters crash unless the API::Array is retained by a local variable.
Aug 4, 2017:
- 10:06 PM Changeset in webkit [220311] by
-
- 29 edits1 copy47 adds in trunk
[Cache API] Add Cache and CacheStorage IDL definitions
https://bugs.webkit.org/show_bug.cgi?id=175201
Patch by Youenn Fablet <youenn@apple.com> on 2017-08-04
Reviewed by Brady Eidson.
LayoutTests/imported/w3c:
- web-platform-tests/service-workers/cache-storage/common.https-expected.txt: Added.
- web-platform-tests/service-workers/cache-storage/serviceworker/cache-add.https-expected.txt: Added.
- web-platform-tests/service-workers/cache-storage/serviceworker/cache-delete.https-expected.txt: Added.
- web-platform-tests/service-workers/cache-storage/serviceworker/cache-keys.https-expected.txt: Added.
- web-platform-tests/service-workers/cache-storage/serviceworker/cache-match.https-expected.txt: Added.
- web-platform-tests/service-workers/cache-storage/serviceworker/cache-matchAll.https-expected.txt: Added.
- web-platform-tests/service-workers/cache-storage/serviceworker/cache-put.https-expected.txt: Added.
- web-platform-tests/service-workers/cache-storage/serviceworker/cache-storage-keys.https-expected.txt: Added.
- web-platform-tests/service-workers/cache-storage/serviceworker/cache-storage-match.https-expected.txt: Added.
- web-platform-tests/service-workers/cache-storage/serviceworker/cache-storage.https-expected.txt: Added.
- web-platform-tests/service-workers/cache-storage/serviceworker/credentials.https-expected.txt: Added.
- web-platform-tests/service-workers/cache-storage/window/cache-add.https-expected.txt: Added.
- web-platform-tests/service-workers/cache-storage/window/cache-delete.https-expected.txt: Added.
- web-platform-tests/service-workers/cache-storage/window/cache-keys.https-expected.txt: Added.
- web-platform-tests/service-workers/cache-storage/window/cache-match.https-expected.txt: Added.
- web-platform-tests/service-workers/cache-storage/window/cache-matchAll.https-expected.txt: Added.
- web-platform-tests/service-workers/cache-storage/window/cache-put.https-expected.txt: Added.
- web-platform-tests/service-workers/cache-storage/window/cache-storage-keys.https-expected.txt: Added.
- web-platform-tests/service-workers/cache-storage/window/cache-storage-match.https-expected.txt: Added.
- web-platform-tests/service-workers/cache-storage/window/cache-storage.https-expected.txt: Added.
- web-platform-tests/service-workers/cache-storage/window/sandboxed-iframes.https-expected.txt: Added.
- web-platform-tests/service-workers/cache-storage/worker/cache-add.https-expected.txt: Added.
- web-platform-tests/service-workers/cache-storage/worker/cache-delete.https-expected.txt: Added.
- web-platform-tests/service-workers/cache-storage/worker/cache-keys.https-expected.txt: Added.
- web-platform-tests/service-workers/cache-storage/worker/cache-match.https-expected.txt: Added.
- web-platform-tests/service-workers/cache-storage/worker/cache-matchAll.https-expected.txt: Added.
- web-platform-tests/service-workers/cache-storage/worker/cache-put.https-expected.txt: Added.
- web-platform-tests/service-workers/cache-storage/worker/cache-storage-keys.https-expected.txt: Added.
- web-platform-tests/service-workers/cache-storage/worker/cache-storage-match.https-expected.txt: Added.
- web-platform-tests/service-workers/cache-storage/worker/cache-storage.https-expected.txt: Added.
- web-platform-tests/service-workers/stub-4.6.2-cache-expected.txt: Added.
- web-platform-tests/service-workers/stub-4.6.3-cache-storage-expected.txt: Added.
Source/JavaScriptCore:
- runtime/CommonIdentifiers.h:
Source/WebCore:
Covered by activated tests.
Adding IDLs as per https://www.w3.org/TR/service-workers-1/#idl-index.
Implementation is guarded by a runtime flag which is off by default.
It is off for DRT but on for WTR.
- CMakeLists.txt:
- DerivedSources.make:
- Modules/cache/Cache.cpp: Added.
(WebCore::Cache::match):
(WebCore::Cache::matchAll):
(WebCore::Cache::add):
(WebCore::Cache::addAll):
(WebCore::Cache::put):
(WebCore::Cache::remove):
(WebCore::Cache::keys):
- Modules/cache/Cache.h: Added.
(WebCore::Cache::create):
(WebCore::Cache::Cache):
- Modules/cache/Cache.idl: Added.
- Modules/cache/CacheQueryOptions.h: Added.
- Modules/cache/CacheQueryOptions.idl: Added.
- Modules/cache/CacheStorage.cpp: Added.
(WebCore::CacheStorage::match):
(WebCore::CacheStorage::has):
(WebCore::CacheStorage::open):
(WebCore::CacheStorage::remove):
(WebCore::CacheStorage::keys):
- Modules/cache/CacheStorage.h: Added.
(WebCore::CacheStorage::create):
- Modules/cache/CacheStorage.idl: Added.
- Modules/cache/DOMWindowCaches.cpp: Added.
(WebCore::DOMWindowCaches::DOMWindowCaches):
(WebCore::DOMWindowCaches::supplementName):
(WebCore::DOMWindowCaches::from):
(WebCore::DOMWindowCaches::caches):
(WebCore::DOMWindowCaches::caches const):
- Modules/cache/DOMWindowCaches.h: Added.
- Modules/cache/DOMWindowCaches.idl: Added.
- Modules/cache/WorkerGlobalScopeCaches.cpp: Added.
(WebCore::WorkerGlobalScopeCaches::supplementName):
(WebCore::WorkerGlobalScopeCaches::from):
(WebCore::WorkerGlobalScopeCaches::caches):
(WebCore::WorkerGlobalScopeCaches::caches const):
- Modules/cache/WorkerGlobalScopeCaches.h: Added.
- Modules/cache/WorkerGlobalScopeCaches.idl: Added.
- WebCore.xcodeproj/project.pbxproj:
- page/RuntimeEnabledFeatures.h:
(WebCore::RuntimeEnabledFeatures::setCacheAPIEnabled):
(WebCore::RuntimeEnabledFeatures::cacheAPIEnabled const):
Source/WebKit:
- Shared/WebPreferencesDefinitions.h:
- WebProcess/InjectedBundle/InjectedBundle.cpp:
(WebKit::InjectedBundle::overrideBoolPreferenceForTestRunner):
- WebProcess/WebPage/WebPage.cpp:
(WebKit::WebPage::updatePreferences):
Source/WebKitLegacy/mac:
- WebView/WebPreferenceKeysPrivate.h:
- WebView/WebPreferences.mm:
(+[WebPreferences initialize]):
(-[WebPreferences cacheAPIEnabled]):
(-[WebPreferences setCacheAPIEnabled:]):
- WebView/WebPreferencesPrivate.h:
- WebView/WebView.mm:
(-[WebView _preferencesChanged:]):
Tools:
- DumpRenderTree/mac/DumpRenderTree.mm:
(enableExperimentalFeatures):
(resetWebPreferencesToConsistentValues):
- WebKitTestRunner/InjectedBundle/InjectedBundle.cpp:
(WTR::InjectedBundle::beginTesting):
- WebKitTestRunner/InjectedBundle/TestRunner.cpp:
(WTR::TestRunner::setCacheAPIEnabled):
- WebKitTestRunner/InjectedBundle/TestRunner.h:
LayoutTests:
- TestExpectations:
- platform/ios-wk1/TestExpectations:
- platform/mac-wk1/TestExpectations:
- js/dom/global-constructors-attributes-dedicated-worker-expected.txt:
- platform/mac-highsierra/js/dom/global-constructors-attributes-expected.txt:
- 9:59 PM Changeset in webkit [220310] by
-
- 40 edits1 copy1 add in trunk
Have navigator.serviceWorker() actually return a ServiceWorkerContainer object.
https://bugs.webkit.org/show_bug.cgi?id=175215
Reviewed by Youenn Fablet.
LayoutTests/imported/w3c:
- web-platform-tests/FileAPI/historical.https-expected.txt:
- web-platform-tests/background-fetch/interfaces-worker.https-expected.txt:
- web-platform-tests/fetch/api/policies/referrer-no-referrer-service-worker.https-expected.txt:
- web-platform-tests/fetch/api/policies/referrer-origin-service-worker.https-expected.txt:
- web-platform-tests/fetch/api/policies/referrer-origin-when-cross-origin-service-worker.https-expected.txt:
- web-platform-tests/fetch/api/policies/referrer-unsafe-url-service-worker.https-expected.txt:
- web-platform-tests/html/webappapis/scripting/events/messageevent-constructor.https-expected.txt:
- web-platform-tests/html/webappapis/scripting/processing-model-2/integration-with-the-javascript-agent-formalism/canblock-serviceworker.https-expected.txt:
- web-platform-tests/streams/byte-length-queuing-strategy.serviceworker.https-expected.txt:
- web-platform-tests/streams/count-queuing-strategy.serviceworker.https-expected.txt:
- web-platform-tests/streams/piping/close-propagation-backward.serviceworker.https-expected.txt:
- web-platform-tests/streams/piping/close-propagation-forward.serviceworker.https-expected.txt:
- web-platform-tests/streams/piping/error-propagation-backward.serviceworker.https-expected.txt:
- web-platform-tests/streams/piping/error-propagation-forward.serviceworker.https-expected.txt:
- web-platform-tests/streams/piping/flow-control.serviceworker.https-expected.txt:
- web-platform-tests/streams/piping/general.serviceworker.https-expected.txt:
- web-platform-tests/streams/piping/multiple-propagation.serviceworker.https-expected.txt:
- web-platform-tests/streams/piping/pipe-through.serviceworker.https-expected.txt:
- web-platform-tests/streams/piping/transform-streams.serviceworker.https-expected.txt:
- web-platform-tests/streams/readable-byte-streams/general.serviceworker.https-expected.txt:
- web-platform-tests/streams/readable-streams/bad-strategies.serviceworker.https-expected.txt:
- web-platform-tests/streams/readable-streams/bad-underlying-sources.serviceworker.https-expected.txt:
- web-platform-tests/streams/readable-streams/brand-checks.serviceworker.https-expected.txt:
- web-platform-tests/streams/readable-streams/cancel.serviceworker.https-expected.txt:
- web-platform-tests/streams/readable-streams/count-queuing-strategy-integration.serviceworker.https-expected.txt:
- web-platform-tests/streams/readable-streams/default-reader.serviceworker.https-expected.txt:
- web-platform-tests/streams/readable-streams/floating-point-total-queue-size.serviceworker.https-expected.txt:
- web-platform-tests/streams/readable-streams/garbage-collection.serviceworker.https-expected.txt:
- web-platform-tests/streams/readable-streams/general.serviceworker.https-expected.txt:
- web-platform-tests/streams/readable-streams/pipe-through.serviceworker.https-expected.txt:
- web-platform-tests/streams/readable-streams/readable-stream-reader.serviceworker.https-expected.txt:
- web-platform-tests/streams/readable-streams/tee.serviceworker.https-expected.txt:
- web-platform-tests/streams/readable-streams/templated.serviceworker.https-expected.txt:
Source/WebCore:
- page/NavigatorBase.cpp:
(WebCore::NavigatorBase::serviceWorker): Actually create and remember an object.
- page/NavigatorBase.h:
- workers/ServiceWorkerContainer.cpp:
(WebCore::rejectLater): Asynchronously reject the given promise with an error message.
(WebCore::ServiceWorkerContainer::ServiceWorkerContainer):
(WebCore::ServiceWorkerContainer::refEventTarget): Ref the underlying Navigator.
(WebCore::ServiceWorkerContainer::derefEventTarget): Deref the underlying Navigator.
(WebCore::ServiceWorkerContainer::ready): rejectLater the promise.
(WebCore::ServiceWorkerContainer::addRegistration): Ditto.
(WebCore::ServiceWorkerContainer::getRegistration): Ditto.
(WebCore::ServiceWorkerContainer::getRegistrations): Ditto.
- workers/ServiceWorkerContainer.h:
LayoutTests:
- platform/mac-wk1/imported/w3c/web-platform-tests/FileAPI/historical.https-expected.txt: Copied from LayoutTests/imported/w3c/web-platform-tests/FileAPI/historical.https-expected.txt.
- 7:14 PM Changeset in webkit [220309] by
-
- 3 edits in trunk/Source/WebKit
Expose WKPreferences SPI for toggling mock capture devices prompt
https://bugs.webkit.org/show_bug.cgi?id=175227
<rdar://problem/33734528>
Reviewed by Youenn Fablet.
- UIProcess/API/Cocoa/WKPreferences.mm:
(-[WKPreferences _mockCaptureDevicesPromptEnabled]):
(-[WKPreferences _setMockCaptureDevicesPromptEnabled:]):
- UIProcess/API/Cocoa/WKPreferencesPrivate.h:
- 6:01 PM Changeset in webkit [220308] by
-
- 2 edits in trunk/Source/JavaScriptCore
Fix typo in testmasm.cpp: ENABLE(JSVALUE64) should be USE(JSVALUE64).
https://bugs.webkit.org/show_bug.cgi?id=175230
<rdar://problem/33735857>
Reviewed by Saam Barati.
- assembler/testmasm.cpp:
(JSC::testProbeReadsArgumentRegisters):
(JSC::testProbeWritesArgumentRegisters):
- 5:20 PM Changeset in webkit [220307] by
-
- 1 edit39 adds in trunk/PerformanceTests
Create a new JavaScript RegExp benchmark
https://bugs.webkit.org/show_bug.cgi?id=175225
Reviewed by JF Bastien.
This is a new benchmark for Regular Expressions. It borrows regex-dna from
SunSpider, the regexp test from Octane2, the BASIC parser from ARES-6/Basic,
and adds a new flight planner benchmark that uses RegExp's for textual parsing.
There needs to be some additions and changes to some of the textual content.
This includes adding more BASIC programs to the Basic test and increasing
keyword usage in the Flight Planner to increase the 16-bit coverage.
The intent is to grow this benchmark by adding a JavaScript implementation of the
offline assembler lexer and parser as well as adding some targeted micro benchmark
tests.
- RexBench: Added.
- RexBench/Basic: Added.
- RexBench/Basic/ast.js: Added.
- RexBench/Basic/basic.js: Added.
- RexBench/Basic/benchmark.js: Added.
- RexBench/Basic/caseless_map.js: Added.
- RexBench/Basic/lexer.js: Added.
- RexBench/Basic/number.js: Added.
- RexBench/Basic/parser.js: Added.
- RexBench/Basic/random.js: Added.
- RexBench/Basic/state.js: Added.
- RexBench/Basic/stress-test.js: Added.
- RexBench/Basic/util.js: Added.
- RexBench/FlightPlanner: Added.
- RexBench/FlightPlanner/airways.js: Added.
- RexBench/FlightPlanner/benchmark.js: Added.
- RexBench/FlightPlanner/convert-nfdc.py: Added.
- RexBench/FlightPlanner/expectations.js: Added.
- RexBench/FlightPlanner/flight_planner.js: Added.
- RexBench/FlightPlanner/use_unicode.js: Added.
- RexBench/FlightPlanner/waypoints.js: Added.
- RexBench/Octane2: Added.
- RexBench/Octane2/base.js: Added.
- RexBench/Octane2/regexp.js: Added.
- RexBench/SunSpider: Added.
- RexBench/SunSpider/regex-dna.js: Added.
- RexBench/about.html: Added.
- RexBench/basic_benchmark.js: Added.
- RexBench/cli.js: Added.
- RexBench/driver.js: Added.
- RexBench/flightplan_benchmark.js: Added.
- RexBench/flightplan_unicode_benchmark.js: Added.
- RexBench/glue.js: Added.
- RexBench/index.html: Added.
- RexBench/octane2_benchmark.js: Added.
- RexBench/results.js: Added.
- RexBench/stats.js: Added.
- RexBench/styles.css: Added.
- RexBench/sunspider_benchmark.js: Added.
- 5:04 PM Changeset in webkit [220306] by
-
- 9 edits1 move3 deletes in trunk/Source/JavaScriptCore
Move DFG::OSRExitCompiler methods into DFG::OSRExit [step 2].
https://bugs.webkit.org/show_bug.cgi?id=175214
<rdar://problem/33733308>
Rubber-stamped by Michael Saboff.
Copy the 64-bit and common methods into DFGOSRExit.cpp, and delete the unused
DFGOSRExitCompiler files.
Also renamed DFGOSRExitCompiler32_64.cpp to DFGOSRExit32_64.cpp.
Also move debugOperationPrintSpeculationFailure() into DFGOSRExit.cpp. It's only
used by compileOSRExit(), and will be changed to not be a DFG operation function
when we use JIT probes for DFG OSR exits later in
https://bugs.webkit.org/show_bug.cgi?id=175144.
- CMakeLists.txt:
- JavaScriptCore.xcodeproj/project.pbxproj:
- dfg/DFGJITCompiler.cpp:
- dfg/DFGOSRExit.cpp:
(JSC::DFG::OSRExit::emitRestoreArguments):
(JSC::DFG::OSRExit::compileOSRExit):
(JSC::DFG::OSRExit::compileExit):
(JSC::DFG::OSRExit::debugOperationPrintSpeculationFailure):
- dfg/DFGOSRExit.h:
- dfg/DFGOSRExit32_64.cpp: Copied from Source/JavaScriptCore/dfg/DFGOSRExitCompiler32_64.cpp.
- dfg/DFGOSRExitCompiler.cpp: Removed.
- dfg/DFGOSRExitCompiler.h: Removed.
- dfg/DFGOSRExitCompiler32_64.cpp: Removed.
- dfg/DFGOSRExitCompiler64.cpp: Removed.
- dfg/DFGOperations.cpp:
- dfg/DFGOperations.h:
- dfg/DFGThunks.cpp:
- 4:28 PM Changeset in webkit [220305] by
-
- 3 edits in trunk/Source/WebKit
Fix typo in WKPreferences _iceCandidateFiltertingEnabled property
https://bugs.webkit.org/show_bug.cgi?id=175224
Reviewed by Tim Horton.
- UIProcess/API/Cocoa/WKPreferences.mm:
(-[WKPreferences _iceCandidateFilteringEnabled]):
(-[WKPreferences _iceCandidateFiltertingEnabled]): Deleted.
- UIProcess/API/Cocoa/WKPreferencesPrivate.h:
- 4:22 PM Changeset in webkit [220304] by
-
- 1 edit4 adds in trunk/LayoutTests
Add test coverage for sendBeacon() keepalive flag
https://bugs.webkit.org/show_bug.cgi?id=175212
Reviewed by Youenn Fablet.
- http/wpt/beacon/keepalive-after-navigation-expected.txt: Added.
- http/wpt/beacon/keepalive-after-navigation.html: Added.
- http/wpt/beacon/support/sendBeacon-onunload-iframe.html: Added.
- 4:09 PM Changeset in webkit [220303] by
-
- 21 edits in trunk
[Beacon] Update sendBeacon to use the CachedResourceLoader
https://bugs.webkit.org/show_bug.cgi?id=175192
<rdar://problem/33725923>
Reviewed by Youenn Fablet.
LayoutTests/imported/w3c:
Rebaseline test as our Content-Type header has changed for ArrayBuffer / ArrayBufferView
payloads.
- web-platform-tests/beacon/headers/header-content-type-expected.txt:
Source/WebCore:
Update sendBeacon to use the FetchRequest / CachedResourceLoader instead of
the PingLoader. This gets us closer to the specification which is based on
Fetch and reduces code duplication. This also fixes an issue where our
Origin header was not properly set on Beacon resquests.
In a follow-up, we will implement in CachedResourceLoader Fetch's quota for
inflight keepalive requests which is needed to fully support sendBeacon().
- Modules/beacon/NavigatorBeacon.cpp:
(WebCore::NavigatorBeacon::sendBeacon):
- Modules/beacon/NavigatorBeacon.h:
- loader/LinkLoader.cpp:
(WebCore::createLinkPreloadResourceClient):
- loader/PingLoader.cpp:
- loader/PingLoader.h:
- loader/ResourceLoadInfo.cpp:
(WebCore::toResourceType):
- loader/SubresourceLoader.cpp:
(WebCore::logResourceLoaded):
- loader/cache/CachedResource.cpp:
(WebCore::CachedResource::defaultPriorityForResourceType):
(WebCore::CachedResource::load):
- loader/cache/CachedResource.h:
- loader/cache/CachedResourceLoader.cpp:
(WebCore::createResource):
(WebCore::CachedResourceLoader::requestBeaconResource):
(WebCore::contentTypeFromResourceType):
(WebCore::CachedResourceLoader::checkInsecureContent const):
(WebCore::CachedResourceLoader::allowedByContentSecurityPolicy const):
(WebCore::isResourceSuitableForDirectReuse):
- loader/cache/CachedResourceLoader.h:
Source/WebKit:
Deal with new Beacon CachedResource type.
- WebProcess/Network/WebLoaderStrategy.cpp:
(WebKit::maximumBufferingTime):
LayoutTests:
Rebaseline a few tests now that the Origin header is properly set of our Beacon
requests. This is a progression and matches the results from Blink.
Our Content-Type header for ArrayBuffer / ArrayBufferView payloads has also
changed. It is unclear which one is best but at least we are now consistent
with Fetch.
- http/tests/blink/sendbeacon/beacon-cookie-expected.txt:
- http/tests/blink/sendbeacon/beacon-cross-origin-expected.txt:
- http/tests/blink/sendbeacon/beacon-same-origin-expected.txt:
- http/wpt/beacon/headers/header-content-type-same-origin.html:
- 3:34 PM Changeset in webkit [220302] by
-
- 14 edits2 adds in trunk
Resource Load Statistics: Report user interaction immediately, but only when needed
https://bugs.webkit.org/show_bug.cgi?id=175090
<rdar://problem/33685546>
Reviewed by Chris Dumez.
Source/WebCore:
Test: http/tests/loading/resourceLoadStatistics/user-interaction-only-reported-once-within-short-period-of-time.html
- loader/ResourceLoadObserver.cpp:
(WebCore::ResourceLoadObserver::ResourceLoadObserver):
(WebCore::ResourceLoadObserver::logUserInteractionWithReducedTimeResolution):
Now tells the UI process immediately but also records that it has
done so to avoid doing it when not needed.
(WebCore::ResourceLoadObserver::scheduleNotificationIfNeeded):
Conditional throttling gone, now always throttles.
(WebCore::ResourceLoadObserver::notifyObserver):
Renamed from ResourceLoadObserver::notificationTimerFired().
(WebCore::ResourceLoadObserver::clearState):
New function to allow the test runner to reset the web process'
statistics state now that we keep track of whether or not we've
reported user interaction to the UI process.
(WebCore::ResourceLoadObserver::setShouldThrottleObserverNotifications): Deleted.
(WebCore::ResourceLoadObserver::notificationTimerFired): Deleted.
- loader/ResourceLoadObserver.h:
(): Deleted.
- testing/Internals.cpp:
(WebCore::Internals::resetToConsistentState):
(WebCore::Internals::setResourceLoadStatisticsShouldThrottleObserverNotifications): Deleted.
No longer needed since user interaction is always communicated
immediately.
- testing/Internals.h:
- testing/Internals.idl:
Source/WebKit:
- WebProcess/InjectedBundle/API/c/WKBundle.cpp:
(WKBundleClearResourceLoadStatistics):
Test infrastructure. Ends up calling
WebCore::ResourceLoadObserver::clearState().
- WebProcess/InjectedBundle/API/c/WKBundlePrivate.h:
Tools:
- WebKitTestRunner/InjectedBundle/InjectedBundle.cpp:
(WTR::InjectedBundle::beginTesting):
Now calls WebCore::ResourceLoadObserver::clearState().
LayoutTests:
- http/tests/loading/resourceLoadStatistics/user-interaction-in-cross-origin-sub-frame.html:
Now no longer needs to disable throttling since reports of
user interaction happen immediately (when needed).
- http/tests/loading/resourceLoadStatistics/user-interaction-only-reported-once-within-short-period-of-time-expected.txt: Added.
- http/tests/loading/resourceLoadStatistics/user-interaction-only-reported-once-within-short-period-of-time.html: Added.
- platform/mac-wk2/TestExpectations:
user-interaction-only-reported-once-within-short-period-of-time.html marked as [ Pass ].
- 3:19 PM Changeset in webkit [220301] by
-
- 7 edits in branches/safari-604.1-branch/Source
Versioning.
- 3:14 PM Changeset in webkit [220300] by
-
- 1 copy in tags/Safari-604.1.38
Tag Safari-604.1.38.
- 3:12 PM Changeset in webkit [220299] by
-
- 12 edits1 add in trunk
Web Inspector: capture async stack trace when workers/main context posts a message
https://bugs.webkit.org/show_bug.cgi?id=167084
<rdar://problem/30033673>
Reviewed by Brian Burg.
Source/JavaScriptCore:
- inspector/agents/InspectorDebuggerAgent.h:
Add
PostMessageasync call type.
Source/WebCore:
Add instrumentation to DOMWindow to support showing asynchronous
stack traces when the debugger pauses in a MessageEvent handler.
Test: inspector/debugger/async-stack-trace.html
- inspector/InspectorInstrumentation.cpp:
(WebCore::InspectorInstrumentation::didPostMessageImpl):
(WebCore::InspectorInstrumentation::didFailPostMessageImpl):
(WebCore::InspectorInstrumentation::willDispatchPostMessageImpl):
(WebCore::InspectorInstrumentation::didDispatchPostMessageImpl):
- inspector/InspectorInstrumentation.h:
(WebCore::InspectorInstrumentation::didPostMessage):
(WebCore::InspectorInstrumentation::didFailPostMessage):
(WebCore::InspectorInstrumentation::willDispatchPostMessage):
(WebCore::InspectorInstrumentation::didDispatchPostMessage):
- inspector/PageDebuggerAgent.cpp:
(WebCore::PageDebuggerAgent::didClearAsyncStackTraceData):
(WebCore::PageDebuggerAgent::didPostMessage):
(WebCore::PageDebuggerAgent::didFailPostMessage):
(WebCore::PageDebuggerAgent::willDispatchPostMessage):
(WebCore::PageDebuggerAgent::didDispatchPostMessage):
- inspector/PageDebuggerAgent.h:
- page/DOMWindow.cpp:
(WebCore::DOMWindow::postMessage):
(WebCore::DOMWindow::postMessageTimerFired):
LayoutTests:
Add a test to check for asynchronous stack trace data when the debugger
pauses inside a MessageEvent handler.
- inspector/debugger/async-stack-trace-expected.txt:
- inspector/debugger/async-stack-trace.html:
- inspector/debugger/resources/postMessage-echo.html: Added.
- 3:03 PM Changeset in webkit [220298] by
-
- 7 edits in trunk/Source/JavaScriptCore
Move DFG::OSRExitCompiler methods into DFG::OSRExit [step 1].
https://bugs.webkit.org/show_bug.cgi?id=175208
<rdar://problem/33732402>
Reviewed by Saam Barati.
This will minimize the code diff and make it easier to review the patch for
https://bugs.webkit.org/show_bug.cgi?id=175144 later. We'll do this patch in 3
steps:
- Do the code changes to move methods into OSRExit.
- Copy the 64-bit and common methods into DFGOSRExit.cpp, and delete the unused DFGOSRExitCompiler files.
- Merge the 32-bit OSRExitCompiler methods into the 64-bit version, and delete DFGOSRExitCompiler32_64.cpp.
Splitting this refactoring into these 3 steps also makes it easier to review this
patch and understand what is being changed.
- dfg/DFGOSRExit.h:
- dfg/DFGOSRExitCompiler.cpp:
(JSC::DFG::OSRExit::emitRestoreArguments):
(JSC::DFG::OSRExit::compileOSRExit):
(JSC::DFG::OSRExitCompiler::emitRestoreArguments): Deleted.
(): Deleted.
- dfg/DFGOSRExitCompiler.h:
(JSC::DFG::OSRExitCompiler::OSRExitCompiler): Deleted.
(): Deleted.
- dfg/DFGOSRExitCompiler32_64.cpp:
(JSC::DFG::OSRExit::compileExit):
(JSC::DFG::OSRExitCompiler::compileExit): Deleted.
- dfg/DFGOSRExitCompiler64.cpp:
(JSC::DFG::OSRExit::compileExit):
(JSC::DFG::OSRExitCompiler::compileExit): Deleted.
- dfg/DFGThunks.cpp:
(JSC::DFG::osrExitGenerationThunkGenerator):
- 2:57 PM Changeset in webkit [220297] by
-
- 2 edits in trunk/LayoutTests
LayoutTest imported/w3c/web-platform-tests/beacon/beacon-basic-string.html is a flaky failure (harness timeout)
https://bugs.webkit.org/show_bug.cgi?id=175202
Unreviewed, mark test as flaky.
- 2:46 PM Changeset in webkit [220296] by
-
- 2 edits in trunk/Tools
Add an API test for r220286
https://bugs.webkit.org/show_bug.cgi?id=175206
Reviewed by Simon Fraser.
- TestWebKitAPI/Tests/WebKit2Cocoa/AnimatedResize.mm:
(-[AnimatedResizeWebView _webView:didChangeSafeAreaShouldAffectObscuredInsets:]):
(createAnimatedResizeWebView):
(TEST):
Add a test to ensure that we don't call
_webView:didChangeSafeAreaShouldAffectObscuredInsets: during an
animated resize.
- 2:45 PM Changeset in webkit [220295] by
-
- 6 edits1 add in trunk/Tools
[XCode] webkit-patch should run sort-Xcode-project-file
https://bugs.webkit.org/show_bug.cgi?id=174036
Patch by Stephan Szabo <stephan.szabo@sony.com> on 2017-08-04
Reviewed by Simon Fraser.
- Scripts/webkitpy/common/config/ports.py:
- Scripts/webkitpy/tool/commands/download.py:
- Scripts/webkitpy/tool/commands/upload.py:
- Scripts/webkitpy/tool/steps/init.py:
- Scripts/webkitpy/tool/steps/options.py:
- Scripts/webkitpy/tool/steps/sortxcodeprojectfiles.py: Added.
- 2:39 PM Changeset in webkit [220294] by
-
- 27 edits2 copies6 adds in trunk
Web Inspector: add source view for WebGL shader programs
https://bugs.webkit.org/show_bug.cgi?id=138593
<rdar://problem/18936194>
Reviewed by Matt Baker.
Source/JavaScriptCore:
- inspector/protocol/Canvas.json:
- Add
ShaderTypeenum that contains "vertex" and "fragment". - Add
requestShaderSourcecommand that will return the original source code for a given shader program and shader type.
- Add
Source/WebCore:
Test: inspector/canvas/requestShaderSource.html
- inspector/InspectorCanvasAgent.h:
- inspector/InspectorCanvasAgent.cpp:
(WebCore::InspectorCanvasAgent::requestShaderSource):
- inspector/InspectorShaderProgram.h:
- inspector/InspectorShaderProgram.cpp:
(WebCore::InspectorShaderProgram::shaderForType):
Source/WebInspectorUI:
Shader programs are now listed in the Resources sidebar as items within the owner Canvas
context. When selected, the shown content view has two editors, one for the Vertex shader
and one for the Fragment shader. These editors use CodeMirror's "clike" mode for GLSL syntax.
- Localizations/en.lproj/localizedStrings.js:
- WebInspectorUI.vcxproj/WebInspectorUI.vcxproj:
- WebInspectorUI.vcxproj/WebInspectorUI.vcxproj.filters:
- UserInterface/Main.html:
- UserInterface/Controllers/CanvasManager.js:
(WI.CanvasManager):
(WI.CanvasManager.prototype.canvasRemoved):
(WI.CanvasManager.prototype.programCreated):
(WI.CanvasManager.prototype.programDeleted):
(WI.CanvasManager.prototype._mainResourceDidChange):
- UserInterface/Models/Collection.js:
- UserInterface/Models/Canvas.js:
(WI.Canvas.prototype.get shaderProgramCollection)
- UserInterface/Models/ShaderProgram.js:
(WI.ShaderProgram.prototype.requestVertexShaderSource):
(WI.ShaderProgram.prototype.requestFragmentShaderSource):
(WI.ShaderProgram.prototype._requestShaderSource):
- UserInterface/Views/CanvasTreeElement.js:
(WI.CanvasTreeElement):
(WI.CanvasTreeElement.prototype.onattach):
(WI.CanvasTreeElement.prototype.ondetach):
(WI.CanvasTreeElement.prototype.onpopulate):
(WI.CanvasTreeElement.prototype._shaderProgramAdded):
(WI.CanvasTreeElement.prototype._shaderProgramRemoved):
- UserInterface/Views/ShaderProgramTreeElement.js: Added.
(WI.ShaderProgramTreeElement):
- UserInterface/Views/ShaderProgramContentView.js: Added.
(WI.ShaderProgramContentView):
(WI.ShaderProgramContentView.prototype.shown):
(WI.ShaderProgramContentView.prototype.hidden):
(WI.ShaderProgramContentView.prototype.closed):
(WI.ShaderProgramContentView.prototype.get supportsSave):
(WI.ShaderProgramContentView.prototype.get saveData):
(WI.ShaderProgramContentView.prototype.get supportsSearch):
(WI.ShaderProgramContentView.prototype.get numberOfSearchResults):
(WI.ShaderProgramContentView.prototype.get hasPerformedSearch):
(WI.ShaderProgramContentView.prototype.set automaticallyRevealFirstSearchResult):
(WI.ShaderProgramContentView.prototype.performSearch):
(WI.ShaderProgramContentView.prototype.searchCleared):
(WI.ShaderProgramContentView.prototype.searchQueryWithSelection):
(WI.ShaderProgramContentView.prototype.revealPreviousSearchResult):
(WI.ShaderProgramContentView.prototype.revealNextSearchResult):
(WI.ShaderProgramContentView.prototype.revealPosition):
(WI.ShaderProgramContentView.prototype._editorFocused):
(WI.ShaderProgramContentView.prototype._numberOfSearchResultsDidChange):
- UserInterface/Views/ShaderProgramContentView.css: Added.
(.content-view.shader-program > .text-editor.shader):
(body[dir=ltr] .content-view.shader-program > .text-editor.shader.vertex,):
(body[dir=ltr] .content-view.shader-program > .text-editor.shader.fragment,):
(body[dir=ltr] .content-view.shader-program > .text-editor.shader + .text-editor.shader):
(body[dir=rtl] .content-view.shader-program > .text-editor.shader + .text-editor.shader):
(.content-view.shader-program > .text-editor.shader > .type-title):
(.content-view.shader-program > .text-editor.shader > .CodeMirror):
- UserInterface/Images/DocumentGL.png: Added.
- UserInterface/Images/DocumentGL@2x.png: Added.
- UserInterface/Views/ResourceIcons.css:
(.shader-program .icon):
- UserInterface/Base/Main.js:
- UserInterface/Views/ContentView.js:
(WI.ContentView.createFromRepresentedObject):
(WI.ContentView.isViewable):
- UserInterface/Views/ResourceSidebarPanel.js:
(WI.ResourceSidebarPanel.prototype.matchTreeElementAgainstCustomFilters.match):
(WI.ResourceSidebarPanel.prototype.matchTreeElementAgainstCustomFilters):
(WI.ResourceSidebarPanel.prototype._treeSelectionDidChange):
- UserInterface/Views/ResourcesTabContentView.js:
(WI.ResourcesTabContentView.prototype.canShowRepresentedObject):
Plumbing for displaying ShaderProgram content.
- UserInterface/Views/TextEditor.js:
(WebInspector.TextEditor):
(WebInspector.TextEditor.prototype._editorFocused):
- Scripts/update-codemirror-resources.rb:
- UserInterface/External/CodeMirror/clike.js: Added.
Add C-like mode for highlighting GLSL.
LayoutTests:
- inspector/canvas/requestShaderSource-expected.txt: Added.
- inspector/canvas/requestShaderSource.html: Added.
- inspector/canvas/resources/shaderProgram-utilities.js:
(linkProgram):
(linkProgram.typeForScript):
(linkProgram.createShaderFromScript):
- platform/win/TestExpectations:
- 2:37 PM Changeset in webkit [220293] by
-
- 6 edits in trunk/Source
Unreviewed, rolling out r220288.
This broke multiple builds.
Reverted changeset:
"Use MPAVRoutingController instead of deprecated versions."
https://bugs.webkit.org/show_bug.cgi?id=175063
http://trac.webkit.org/changeset/220288
- 2:18 PM Changeset in webkit [220292] by
-
- 2 edits in trunk/Tools
REGRESSION(r219857): run-benchmark --allplans broken
https://bugs.webkit.org/show_bug.cgi?id=175186
Reviewed by Saam Barati.
r219857 forgot to update also the calls to BenchmarkRunner() that
is done when the script is run with --allplans.
To fix this (and avoid future issues like this), let's factorize
the calls to the benchhmark runner in a run_benchmark_plan()
function.
- Scripts/webkitpy/benchmark_runner/run_benchmark.py:
(run_benchmark_plan):
(start):
- 1:50 PM Changeset in webkit [220291] by
-
- 21 edits7 adds2 deletes in trunk/Source
The allocator used to allocate memory for MarkedBlocks and LargeAllocations should not be the Subspace itself
https://bugs.webkit.org/show_bug.cgi?id=175141
Reviewed by Mark Lam.
Source/JavaScriptCore:
To make it easier to have multiple gigacages and maybe even fancier methods of allocating, this
decouples the allocator used to allocate memory from the GC Subspace. This means we no longer have
to create a new Subspace subclass to allocate memory a different way. Instead, the allocator is now
determined by the AlignedMemoryAllocator object.
This also simplifies trading of blocks. Before, Subspaces had to determine if other Subspaces could
trade blocks with them using canTradeBlocksWith(). This makes it difficult for two different
Subspaces that both use the same underlying allocator to realize that they can trade blocks with
each other. Now, you just need to ask the block being stolen and the subspace doing the stealing if
they use the same AlignedMemoryAllocator.
- CMakeLists.txt:
- JavaScriptCore.xcodeproj/project.pbxproj:
- heap/AlignedMemoryAllocator.cpp: Added.
(JSC::AlignedMemoryAllocator::AlignedMemoryAllocator):
(JSC::AlignedMemoryAllocator::~AlignedMemoryAllocator):
- heap/AlignedMemoryAllocator.h: Added.
- heap/FastMallocAlignedMemoryAllocator.cpp: Added.
(JSC::FastMallocAlignedMemoryAllocator::singleton):
(JSC::FastMallocAlignedMemoryAllocator::FastMallocAlignedMemoryAllocator):
(JSC::FastMallocAlignedMemoryAllocator::~FastMallocAlignedMemoryAllocator):
(JSC::FastMallocAlignedMemoryAllocator::tryAllocateAlignedMemory):
(JSC::FastMallocAlignedMemoryAllocator::freeAlignedMemory):
(JSC::FastMallocAlignedMemoryAllocator::dump const):
- heap/FastMallocAlignedMemoryAllocator.h: Added.
- heap/GigacageAlignedMemoryAllocator.cpp: Added.
(JSC::GigacageAlignedMemoryAllocator::singleton):
(JSC::GigacageAlignedMemoryAllocator::GigacageAlignedMemoryAllocator):
(JSC::GigacageAlignedMemoryAllocator::~GigacageAlignedMemoryAllocator):
(JSC::GigacageAlignedMemoryAllocator::tryAllocateAlignedMemory):
(JSC::GigacageAlignedMemoryAllocator::freeAlignedMemory):
(JSC::GigacageAlignedMemoryAllocator::dump const):
- heap/GigacageAlignedMemoryAllocator.h: Added.
- heap/GigacageSubspace.cpp: Removed.
- heap/GigacageSubspace.h: Removed.
- heap/LargeAllocation.cpp:
(JSC::LargeAllocation::tryCreate):
(JSC::LargeAllocation::destroy):
- heap/MarkedAllocator.cpp:
(JSC::MarkedAllocator::tryAllocateWithoutCollecting):
- heap/MarkedBlock.cpp:
(JSC::MarkedBlock::tryCreate):
(JSC::MarkedBlock::Handle::Handle):
(JSC::MarkedBlock::Handle::~Handle):
(JSC::MarkedBlock::Handle::didAddToAllocator):
(JSC::MarkedBlock::Handle::subspace const):
- heap/MarkedBlock.h:
(JSC::MarkedBlock::Handle::alignedMemoryAllocator const):
(JSC::MarkedBlock::Handle::subspace const): Deleted.
- heap/Subspace.cpp:
(JSC::Subspace::Subspace):
(JSC::Subspace::findEmptyBlockToSteal):
(JSC::Subspace::canTradeBlocksWith): Deleted.
(JSC::Subspace::tryAllocateAlignedMemory): Deleted.
(JSC::Subspace::freeAlignedMemory): Deleted.
- heap/Subspace.h:
(JSC::Subspace::name const):
(JSC::Subspace::alignedMemoryAllocator const):
- runtime/JSDestructibleObjectSubspace.cpp:
(JSC::JSDestructibleObjectSubspace::JSDestructibleObjectSubspace):
- runtime/JSDestructibleObjectSubspace.h:
- runtime/JSSegmentedVariableObjectSubspace.cpp:
(JSC::JSSegmentedVariableObjectSubspace::JSSegmentedVariableObjectSubspace):
- runtime/JSSegmentedVariableObjectSubspace.h:
- runtime/JSStringSubspace.cpp:
(JSC::JSStringSubspace::JSStringSubspace):
- runtime/JSStringSubspace.h:
- runtime/VM.cpp:
(JSC::VM::VM):
- runtime/VM.h:
- wasm/js/JSWebAssemblyCodeBlockSubspace.cpp:
(JSC::JSWebAssemblyCodeBlockSubspace::JSWebAssemblyCodeBlockSubspace):
- wasm/js/JSWebAssemblyCodeBlockSubspace.h:
Source/WebCore:
No new tests because no new behavior.
Just adapting to an API change.
- ForwardingHeaders/heap/FastMallocAlignedMemoryAllocator.h: Added.
- bindings/js/WebCoreJSClientData.cpp:
(WebCore::JSVMClientData::JSVMClientData):
- 1:49 PM Changeset in webkit [220290] by
-
- 11 edits2 adds in trunk
Match newly-clarified spec on textarea defaultValue/value/child text content
https://bugs.webkit.org/show_bug.cgi?id=173878
Reviewed by Darin Adler.
LayoutTests/imported/w3c:
Re-sync WPT test from upstream and rebaseline to improve test coverage.
- web-platform-tests/html/semantics/forms/the-textarea-element/value-defaultValue-textContent-expected.txt:
- web-platform-tests/html/semantics/forms/the-textarea-element/value-defaultValue-textContent.html:
Source/WebCore:
Update HTMLTextArea.defaultValue to match align with other browsers and match the
latest HTML specification:
The defaultValue getter should return the child text content:
Our code was traversing all Text descendants, not just the children.
The defaultValue setter should act as the setter of the Element's textContent
IDL attribute. Previously, we had a custom logic that was only removing the
text children.
Test: imported/w3c/web-platform-tests/html/semantics/forms/the-textarea-element/value-defaultValue-textContent.html
- dom/ScriptElement.cpp:
(WebCore::ScriptElement::scriptContent const):
- dom/TextNodeTraversal.cpp:
(WebCore::TextNodeTraversal::childTextContent):
- dom/TextNodeTraversal.h:
- html/HTMLTextAreaElement.cpp:
(WebCore::HTMLTextAreaElement::defaultValue const):
(WebCore::HTMLTextAreaElement::setDefaultValue):
- html/HTMLTitleElement.cpp:
(WebCore::HTMLTitleElement::text const):
- 1:42 PM Changeset in webkit [220289] by
-
- 3 edits2 adds in trunk
RenderImageResourceStyleImage::image() should return the nullImage() if the image is not available
https://bugs.webkit.org/show_bug.cgi?id=174874
<rdar://problem/33530130>
Patch by Said Abou-Hallawa <sabouhallawa@apple.com> on 2017-08-04
Reviewed by Simon Fraser.
Source/WebCore:
If an <img> element has a non-CachedImage content data, e.g. -webkit-named-image,
RenderImageResourceStyleImage will be created and attached to the RenderImage.
RenderImageResourceStyleImage::m_cachedImage will be set to null at the
beginning because the m_styleImage->isCachedImage() is false in this case.
When ImageLoader finishes loading the url of the src attribute,
RenderImageResource::setCachedImage() will be called to set m_cachedImage.
A crash will happen when the RenderImage is destroyed. Destroying the
RenderImage calls RenderImageResourceStyleImage::shutdown() which checks
m_cachedImage and finds it not null, so it calls RenderImageResourceStyleImage::image()
which ends up calling CSSNamedImageValue::image() which returns a null pointer
because the size is empty. RenderImageResourceStyleImage::shutdown() calls
image()->stopAnimation() without checking the return value of image().
Another crash will happen later when deleting the CachedImage from the memory
cache if CachedImage::canDestroyDecodedData() is called because the client
it gets from m_clients is a freed pointer. This happens because RenderImageResourceStyleImage
has m_styleImage of type StyleGeneratedImage but its m_cachedImage is set
by RenderImageResource::setCachedImage(). When RenderImageResourceStyleImage::shutdown()
is called, it calls StyleGeneratedImage::removeClient() which does not
know anything about RenderImageResourceStyleImage::m_cachedImage. So we
end up having a freed pointer in the m_clients of the CachedImage.
Test: fast/images/image-element-image-content-data.html
- rendering/RenderImageResourceStyleImage.cpp:
(WebCore::RenderImageResourceStyleImage::shutdown): Revert back the changes
of r208511 in this function. Add a call to image()->stopAnimation() without
checking the return of image() since it will return the nullImage() if
the image not available. There is no need to check m_cachedImage before
calling image() because image() does not check or access m_cachedImage.
If m_styleImage is not a CachedStyleImage but m_cachedImage is not null,
we need to remove m_renderer from the set of the clients of this m_cachedImage.
(WebCore::RenderImageResourceStyleImage::image const): The base class method
RenderImageResource::image() returns the nullImage() if the image not
available. This is because CachedImage::imageForRenderer() returns
the nullImage() if the image is not available; see CachedImage.h. We should
do the same for the derived class for consistency.
LayoutTests:
- fast/images/image-element-image-content-data-expected.txt: Added.
- fast/images/image-element-image-content-data.html: Added.
- 1:07 PM Changeset in webkit [220288] by
-
- 6 edits in trunk/Source
Use MPAVRoutingController instead of deprecated versions.
https://bugs.webkit.org/show_bug.cgi?id=175063
Source/WebCore:
Patch by Jeremy Jones <jeremyj@apple.com> on 2017-08-04
Reviewed by Tim Horton.
No new tests because no behavior change. This uses a different platform class to present
an interface.
Remove deprecated MPAudioVideoRoutingPopoverController and MPAVRoutingSheet
Add MPMediaControlsViewController.
- platform/spi/ios/MediaPlayerSPI.h:
Source/WebKit:
rdar://problem/33301230
Patch by Jeremy Jones <jeremyj@apple.com> on 2017-08-04
Reviewed by Tim Horton.
Remove dependence on deprecated classes MPAVRoutingSheet and MPAudioVideoRoutingPopoverController.
- UIProcess/ios/forms/WKAirPlayRoutePicker.h:
- UIProcess/ios/forms/WKAirPlayRoutePicker.mm:
(-[WKAirPlayRoutePicker dealloc]):
(-[WKAirPlayRoutePicker show:fromRect:]):
(-[WKAirPlayRoutePicker popoverControllerDidDismissPopover:]): Deleted.
(-[WKAirPlayRoutePicker _presentAirPlayPopoverAnimated:fromRect:]): Deleted.
(-[WKAirPlayRoutePicker _windowWillRotate:]): Deleted.
(-[WKAirPlayRoutePicker _windowDidRotate:]): Deleted.
(-[WKAirPlayRoutePicker _dismissAirPlayRoutePickerIPad]): Deleted.
(-[WKAirPlayRoutePicker showAirPlayPickerIPad:fromRect:]): Deleted.
(-[WKAirPlayRoutePicker showAirPlayPickerIPhone:]): Deleted.
- 1:01 PM Changeset in webkit [220287] by
-
- 2 edits in trunk/Tools
Dashboard bubbles sometimes show failure count instead of crash count
https://bugs.webkit.org/show_bug.cgi?id=175157
<rdar://problem/33709009>
Reviewed by Alexey Proskuryakov.
- BuildSlaveSupport/build.webkit.org-config/public_html/dashboard/Scripts/BuildbotTestResults.js:
(BuildbotTestResults.prototype.resultSummarizer): Use the regex for the precise match first.
- 12:50 PM Changeset in webkit [220286] by
-
- 12 edits in trunk/Source/WebKit
viewport-fit changes during animated resize can cause layout size to get stuck
https://bugs.webkit.org/show_bug.cgi?id=175169
<rdar://problem/33684697>
Reviewed by Simon Fraser.
If we get a commit that changes viewport-fit state while an animated
resize is underway, and that change causes the client to push a new
minimumLayoutSizeOverride, the new size will be lost forever, and we
will get stuck laying out at the wrong size.
This is because we unconditionally apply avoidsUnsafeArea for every commit,
while most other changes that come in from a commit are ignored if we're
inside animated resize. To fix, also ignore avoidsUnsafeArea changes during
animated resize, by moving the code that keeps track of it into WKWebView
like all of the rest, and read it out of the commit in didCommitLayerTree
*after* the animated-resize early-return.
Also, fix the associated layout tests by adding a missing assignment
in _computedContentInset, which was broken in r220138.
- UIProcess/API/Cocoa/WKWebView.mm:
(-[WKWebView _initializeWithConfiguration:]):
(-[WKWebView _setHasCustomContentView:loadedMIMEType:]):
(-[WKWebView _processDidExit]):
(-[WKWebView _didCommitLayerTree:]):
(-[WKWebView _setAvoidsUnsafeArea:]):
(-[WKWebView _safeAreaShouldAffectObscuredInsets]):
(-[WKWebView _didChangeAvoidsUnsafeArea:]): Deleted.
- UIProcess/API/Cocoa/WKWebViewInternal.h:
- UIProcess/PageClient.h:
- UIProcess/WebPageProxy.cpp:
(WebKit::WebPageProxy::resetState):
(WebKit::WebPageProxy::setAvoidsUnsafeArea): Deleted.
- UIProcess/WebPageProxy.h:
(WebKit::WebPageProxy::avoidsUnsafeArea const): Deleted.
- UIProcess/ios/PageClientImplIOS.h:
- UIProcess/ios/PageClientImplIOS.mm:
(WebKit::PageClientImpl::didChangeAvoidsUnsafeArea): Deleted.
- UIProcess/ios/WebPageProxyIOS.mm:
(WebKit::WebPageProxy::didCommitLayerTree):
- 12:44 PM Changeset in webkit [220285] by
-
- 2 edits in trunk/LayoutTests
Rebaslining fast/text/font-selection-font-loading-api-parse.html for iOS 11.
Unreviewed test gardening.
- platform/ios-11/fast/text/font-selection-font-loading-api-parse-expected.txt:
- 12:04 PM Changeset in webkit [220284] by
-
- 5 edits3 copies2 adds in trunk/Tools
Add tests for NeverDestroyed
https://bugs.webkit.org/show_bug.cgi?id=175146
Patch by Joseph Pecoraro <Joseph Pecoraro> on 2017-08-04
Reviewed by Darin Adler.
- CMakeLists.txt:
- TestWebKitAPI/TestWebKitAPI.xcodeproj/project.pbxproj:
New files and sort.
- TestWebKitAPI/Tests/WTF/Logger.h:
- TestWebKitAPI/Tests/WTF/Logger.cpp:
(TestWebKitAPI::log):
(TestWebKitAPI::takeLogStr):
- TestWebKitAPI/Tests/WTF/RefLogger.h:
- TestWebKitAPI/Tests/WTF/RefLogger.cpp:
(TestWebKitAPI::log): Deleted.
(TestWebKitAPI::takeLogStr): Deleted.
Extract log() / takeLogStr() from RefLogger to a general Logger.h/cpp.
- TestWebKitAPI/Tests/WTF/LifecycleLogger.h:
- TestWebKitAPI/Tests/WTF/LifecycleLogger.cpp: Added.
(TestWebKitAPI::LifecycleLogger::LifecycleLogger):
(TestWebKitAPI::LifecycleLogger::operator=):
(TestWebKitAPI::LifecycleLogger::~LifecycleLogger):
(TestWebKitAPI::LifecycleLogger::setName):
(TestWebKitAPI::TEST):
Add a class that logs construction / assignment / modifications.
- TestWebKitAPI/Tests/WTF/NeverDestroyed.cpp: Added.
(TestWebKitAPI::TEST):
(TestWebKitAPI::list):
Test construction behavior is as expected both directly and using makeNeverDestroyed.
- 12:03 PM Changeset in webkit [220283] by
-
- 2 edits in trunk/Tools
minification logic is not implemented for ios builds
<rdar://problem/33726561>
Reviewed by Dean Johnson
- BuildSlaveSupport/built-product-archive:
(minifyDirectory): refactor minifySource -> minifyDirectory
(archiveBuiltProduct): add ios minification logic
(minifySource): Deleted.
- 12:01 PM Changeset in webkit [220282] by
-
- 2 edits in trunk/Source/JavaScriptCore
[ESNext] Async iteration - update feature.json
https://bugs.webkit.org/show_bug.cgi?id=175197
Reviewed by Yusuke Suzuki.
Update feature.json to add status of the Async Iteration
- features.json:
- 11:57 AM Changeset in webkit [220281] by
-
- 2 edits in trunk/Source/WebCore
Web Inspector: REGRESSION (r220233): Check for null pointer passed to WebGLRenderingContextBase::deleteProgram
https://bugs.webkit.org/show_bug.cgi?id=175196
<rdar://problem/33727603>
Reviewed by Devin Rousso.
- html/canvas/WebGLRenderingContextBase.cpp:
(WebCore::WebGLRenderingContextBase::deleteProgram):
- 11:32 AM Changeset in webkit [220280] by
-
- 2 edits in trunk/Websites/webkit.org
Fixed loading feature status page queries and anchor link URLs
https://bugs.webkit.org/show_bug.cgi?id=175156
Reviewed by Simon Fraser.
- wp-content/themes/webkit/status.php:
- 11:28 AM Changeset in webkit [220279] by
-
- 61 edits in trunk
Unreviewed, rolling out r220271.
Rolling out due to Layout Test failing on iOS Simulator.
Reverted changeset:
"Remove STREAMS_API compilation guard"
https://bugs.webkit.org/show_bug.cgi?id=175165
http://trac.webkit.org/changeset/220271
- 10:59 AM Changeset in webkit [220278] by
-
- 10 edits2 deletes in trunk/Source
[Cleanup] Remove ScriptGlobalObject
https://bugs.webkit.org/show_bug.cgi?id=175173
Reviewed by Darin Adler.
Source/WebCore:
ScriptGlobalObject's two functions were only being used in
four places. Three of those uses (ScriptGlobalObject::set in
InspectorFrontendClientLocal, WebInspectorUI, and RemoteWebInspectorUI)
were merged into the new function addSelfToGlobalObjectInWorld on
InspectorFrontendHost. The remaining function (ScriptGlobalObject::get
in InspectorFrontendHost) was easily inlined.
- CMakeLists.txt:
- WebCore.xcodeproj/project.pbxproj:
- bindings/js/JSBindingsAllInOne.cpp:
- bindings/js/ScriptGlobalObject.cpp: Removed.
- bindings/js/ScriptGlobalObject.h: Removed.
Removed ScriptGlobalObject.
- inspector/InspectorFrontendClientLocal.cpp:
(WebCore::InspectorFrontendClientLocal::windowObjectCleared):
Remove call to ScriptGlobalObject::set and use addSelfToGlobalObjectInWorld instead.
- inspector/InspectorFrontendHost.h:
- inspector/InspectorFrontendHost.cpp:
(WebCore::InspectorFrontendHost::showContextMenu):
Inline ScriptGlobalObject::get.
(WebCore::InspectorFrontendHost::addSelfToGlobalObjectInWorld):
Add helper which inlines ScriptGlobalObject::set and works
as a helper for the three inspector frontends.
Source/WebKit:
- WebProcess/WebPage/RemoteWebInspectorUI.cpp:
(WebKit::RemoteWebInspectorUI::windowObjectCleared):
- WebProcess/WebPage/WebInspectorUI.cpp:
(WebKit::WebInspectorUI::windowObjectCleared):
Remove call to ScriptGlobalObject::set and use addSelfToGlobalObjectInWorld instead.
- 10:10 AM Changeset in webkit [220277] by
-
- 8 edits in branches/safari-604.1-branch
Revert r219896. rdar://problem/33711000
- 9:45 AM Changeset in webkit [220276] by
-
- 3 edits in trunk/LayoutTests
PROGRESSION? Multiple imported/w3c/web-platform-tests/fetch/api/ test have started to fail.
https://bugs.webkit.org/show_bug.cgi?id=175061
Unreviewed, mark Fetch/Cors tests as failing on wk2 ElCapitan only, as those tests seem to be passing
everywhere else.
- platform/mac-wk2/TestExpectations:
- platform/mac/TestExpectations:
- 9:44 AM Changeset in webkit [220275] by
-
- 2 edits in trunk/Source/WebKit
[Cocoa] Web Automation: copy JavaScript atoms to WebKit.framework private headers
https://bugs.webkit.org/show_bug.cgi?id=175088
<rdar://problem/33685818>
Reviewed by Joseph Pecoraro.
- WebKit.xcodeproj/project.pbxproj:
- Add a Copy Files phase to WebKit.framework.
- Copy atoms to the atoms/ directory in WebKit.framework/PrivateHeaders/.
- 9:17 AM Changeset in webkit [220274] by
-
- 14 edits2 deletes in trunk
Unreviewed, rolling out r220268.
This change caused assertion failures on macOS and iOS Debug
WK2.
Reverted changeset:
"Resource Load Statistics: Report user interaction
immediately, but only when needed"
https://bugs.webkit.org/show_bug.cgi?id=175090
http://trac.webkit.org/changeset/220268
- 9:02 AM Changeset in webkit [220273] by
-
- 2 edits in trunk/Source/WebKit
NetworkLoad should always invoke its redirect completion handler
https://bugs.webkit.org/show_bug.cgi?id=175179
<rdar://problem/33464999>
Reviewed by Chris Dumez.
Make sure the redirect completion handler is always invoked,
just like we already did for the response completion handler.
- NetworkProcess/NetworkLoad.cpp:
(WebKit::NetworkLoad::~NetworkLoad):
- 8:39 AM Changeset in webkit [220272] by
-
- 2 edits in trunk/LayoutTests
Mark beacon-basic-string.html as slow.
- 8:20 AM Changeset in webkit [220271] by
-
- 61 edits in trunk
Remove STREAMS_API compilation guard
https://bugs.webkit.org/show_bug.cgi?id=175165
Patch by Youenn Fablet <youenn@apple.com> on 2017-08-04
Reviewed by Darin Adler.
Source/JavaScriptCore:
- Configurations/FeatureDefines.xcconfig:
Source/WebCore:
No change of behavior.
- Configurations/FeatureDefines.xcconfig:
- Modules/fetch/FetchBody.cpp:
(WebCore::FetchBody::consumeAsStream):
- Modules/fetch/FetchBody.h:
- Modules/fetch/FetchBodyOwner.cpp:
(WebCore::FetchBodyOwner::isDisturbedOrLocked const):
(WebCore::FetchBodyOwner::blobLoadingSucceeded):
(WebCore::FetchBodyOwner::blobLoadingFailed):
(WebCore::FetchBodyOwner::blobChunk):
- Modules/fetch/FetchBodyOwner.h:
- Modules/fetch/FetchResponse.cpp:
(WebCore::FetchResponse::BodyLoader::didSucceed):
(WebCore::FetchResponse::BodyLoader::didFail):
(WebCore::FetchResponse::BodyLoader::didReceiveData):
- Modules/fetch/FetchResponse.h:
- Modules/fetch/FetchResponse.idl:
- Modules/fetch/FetchResponse.js:
(initializeFetchResponse):
- Modules/fetch/FetchResponseSource.cpp:
- Modules/fetch/FetchResponseSource.h:
- Modules/streams/ByteLengthQueuingStrategy.idl:
- Modules/streams/ByteLengthQueuingStrategy.js:
- Modules/streams/CountQueuingStrategy.idl:
- Modules/streams/CountQueuingStrategy.js:
- Modules/streams/ReadableByteStreamController.idl:
- Modules/streams/ReadableByteStreamController.js:
- Modules/streams/ReadableByteStreamInternals.js:
- Modules/streams/ReadableStream.idl:
- Modules/streams/ReadableStream.js:
- Modules/streams/ReadableStreamBYOBReader.idl:
- Modules/streams/ReadableStreamBYOBReader.js:
- Modules/streams/ReadableStreamBYOBRequest.idl:
- Modules/streams/ReadableStreamBYOBRequest.js:
- Modules/streams/ReadableStreamDefaultController.idl:
- Modules/streams/ReadableStreamDefaultController.js:
- Modules/streams/ReadableStreamDefaultReader.idl:
- Modules/streams/ReadableStreamDefaultReader.js:
- Modules/streams/ReadableStreamInternals.js:
- Modules/streams/ReadableStreamSource.h:
- Modules/streams/ReadableStreamSource.idl:
- Modules/streams/StreamInternals.js:
- Modules/streams/WritableStream.idl:
- Modules/streams/WritableStream.js:
- Modules/streams/WritableStreamInternals.js:
- bindings/js/JSDOMGlobalObject.cpp:
(WebCore::isReadableByteStreamAPIEnabled):
(WebCore::JSDOMGlobalObject::addBuiltinGlobals):
- bindings/js/JSReadableStreamPrivateConstructors.cpp:
- bindings/js/JSReadableStreamPrivateConstructors.h:
- bindings/js/JSReadableStreamSourceCustom.cpp:
- bindings/js/ReadableStreamDefaultController.cpp:
- bindings/js/ReadableStreamDefaultController.h:
- page/RuntimeEnabledFeatures.h:
- testing/Internals.cpp:
- testing/Internals.h:
- testing/Internals.idl:
Source/WebCore/PAL:
- Configurations/FeatureDefines.xcconfig:
Source/WebKit:
- Configurations/FeatureDefines.xcconfig:
- WebProcess/InjectedBundle/InjectedBundle.cpp:
(WebKit::InjectedBundle::overrideBoolPreferenceForTestRunner):
Source/WebKitLegacy/mac:
- Configurations/FeatureDefines.xcconfig:
- WebView/WebPreferences.mm:
(+[WebPreferences initialize]):
- WebView/WebView.mm:
(-[WebView _preferencesChanged:]):
Source/WTF:
- wtf/FeatureDefines.h:
Tools:
- TestWebKitAPI/Configurations/FeatureDefines.xcconfig:
- 8:16 AM Changeset in webkit [220270] by
-
- 39 edits34 copies10 adds in trunk
Enable ServiceWorkers at runtime for WebKitTestRunner.
https://bugs.webkit.org/show_bug.cgi?id=175174
Reviewed by Youenn Fablet.
LayoutTests/imported/w3c:
- web-platform-tests/background-fetch/interfaces-expected.txt:
- web-platform-tests/background-fetch/interfaces-worker.https-expected.txt:
- web-platform-tests/fetch/api/policies/referrer-no-referrer-service-worker.https-expected.txt:
- web-platform-tests/fetch/api/policies/referrer-origin-service-worker.https-expected.txt:
- web-platform-tests/fetch/api/policies/referrer-origin-when-cross-origin-service-worker.https-expected.txt:
- web-platform-tests/fetch/api/policies/referrer-unsafe-url-service-worker.https-expected.txt:
- web-platform-tests/html/webappapis/scripting/events/messageevent-constructor.https-expected.txt:
- web-platform-tests/html/webappapis/scripting/processing-model-2/integration-with-the-javascript-agent-formalism/canblock-serviceworker.https-expected.txt:
- web-platform-tests/streams/byte-length-queuing-strategy.serviceworker.https-expected.txt:
- web-platform-tests/streams/count-queuing-strategy.serviceworker.https-expected.txt:
- web-platform-tests/streams/piping/close-propagation-backward.serviceworker.https-expected.txt:
- web-platform-tests/streams/piping/close-propagation-forward.serviceworker.https-expected.txt:
- web-platform-tests/streams/piping/error-propagation-backward.serviceworker.https-expected.txt:
- web-platform-tests/streams/piping/error-propagation-forward.serviceworker.https-expected.txt:
- web-platform-tests/streams/piping/flow-control.serviceworker.https-expected.txt:
- web-platform-tests/streams/piping/general.serviceworker.https-expected.txt:
- web-platform-tests/streams/piping/multiple-propagation.serviceworker.https-expected.txt:
- web-platform-tests/streams/piping/pipe-through.serviceworker.https-expected.txt:
- web-platform-tests/streams/piping/transform-streams.serviceworker.https-expected.txt:
- web-platform-tests/streams/readable-byte-streams/general.serviceworker.https-expected.txt:
- web-platform-tests/streams/readable-streams/bad-strategies.serviceworker.https-expected.txt:
- web-platform-tests/streams/readable-streams/bad-underlying-sources.serviceworker.https-expected.txt:
- web-platform-tests/streams/readable-streams/brand-checks.serviceworker.https-expected.txt:
- web-platform-tests/streams/readable-streams/cancel.serviceworker.https-expected.txt:
- web-platform-tests/streams/readable-streams/count-queuing-strategy-integration.serviceworker.https-expected.txt:
- web-platform-tests/streams/readable-streams/default-reader.serviceworker.https-expected.txt:
- web-platform-tests/streams/readable-streams/floating-point-total-queue-size.serviceworker.https-expected.txt:
- web-platform-tests/streams/readable-streams/garbage-collection.serviceworker.https-expected.txt:
- web-platform-tests/streams/readable-streams/general.serviceworker.https-expected.txt:
- web-platform-tests/streams/readable-streams/pipe-through.serviceworker.https-expected.txt:
- web-platform-tests/streams/readable-streams/readable-stream-reader.serviceworker.https-expected.txt:
- web-platform-tests/streams/readable-streams/tee.serviceworker.https-expected.txt:
- web-platform-tests/streams/readable-streams/templated.serviceworker.https-expected.txt:
Source/WebKit:
- UIProcess/WebPreferences.cpp:
(WebKit::WebPreferences::enableAllExperimentalFeatures):
LayoutTests:
- fast/dom/navigator-detached-no-crash-expected.txt:
- platform/mac-wk1/fast/dom/navigator-detached-no-crash-expected.txt: Copied from LayoutTests/fast/dom/navigator-detached-no-crash-expected.txt.
- platform/mac-wk1/imported/w3c/web-platform-tests/background-fetch/interfaces-expected.txt: Copied from LayoutTests/imported/w3c/web-platform-tests/background-fetch/interfaces-expected.txt.
- platform/mac-wk1/imported/w3c/web-platform-tests/background-fetch/interfaces-worker.https-expected.txt: Copied from LayoutTests/imported/w3c/web-platform-tests/background-fetch/interfaces-worker.https-expected.txt.
- platform/mac-wk1/imported/w3c/web-platform-tests/fetch/api/policies/referrer-no-referrer-service-worker.https-expected.txt: Copied from LayoutTests/imported/w3c/web-platform-tests/fetch/api/policies/referrer-no-referrer-service-worker.https-expected.txt.
- platform/mac-wk1/imported/w3c/web-platform-tests/fetch/api/policies/referrer-origin-service-worker.https-expected.txt: Copied from LayoutTests/imported/w3c/web-platform-tests/fetch/api/policies/referrer-origin-service-worker.https-expected.txt.
- platform/mac-wk1/imported/w3c/web-platform-tests/fetch/api/policies/referrer-origin-when-cross-origin-service-worker.https-expected.txt: Copied from LayoutTests/imported/w3c/web-platform-tests/fetch/api/policies/referrer-origin-when-cross-origin-service-worker.https-expected.txt.
- platform/mac-wk1/imported/w3c/web-platform-tests/fetch/api/policies/referrer-unsafe-url-service-worker.https-expected.txt: Copied from LayoutTests/imported/w3c/web-platform-tests/fetch/api/policies/referrer-unsafe-url-service-worker.https-expected.txt.
- platform/mac-wk1/imported/w3c/web-platform-tests/html/webappapis/scripting/events/messageevent-constructor.https-expected.txt: Copied from LayoutTests/imported/w3c/web-platform-tests/html/webappapis/scripting/events/messageevent-constructor.https-expected.txt.
- platform/mac-wk1/imported/w3c/web-platform-tests/html/webappapis/scripting/processing-model-2/integration-with-the-javascript-agent-formalism/canblock-serviceworker.https-expected.txt: Copied from LayoutTests/imported/w3c/web-platform-tests/html/webappapis/scripting/processing-model-2/integration-with-the-javascript-agent-formalism/canblock-serviceworker.https-expected.txt.
- platform/mac-wk1/imported/w3c/web-platform-tests/streams/byte-length-queuing-strategy.serviceworker.https-expected.txt: Copied from LayoutTests/imported/w3c/web-platform-tests/streams/byte-length-queuing-strategy.serviceworker.https-expected.txt.
- platform/mac-wk1/imported/w3c/web-platform-tests/streams/count-queuing-strategy.serviceworker.https-expected.txt: Copied from LayoutTests/imported/w3c/web-platform-tests/streams/count-queuing-strategy.serviceworker.https-expected.txt.
- platform/mac-wk1/imported/w3c/web-platform-tests/streams/piping/close-propagation-backward.serviceworker.https-expected.txt: Copied from LayoutTests/imported/w3c/web-platform-tests/streams/piping/close-propagation-backward.serviceworker.https-expected.txt.
- platform/mac-wk1/imported/w3c/web-platform-tests/streams/piping/close-propagation-forward.serviceworker.https-expected.txt: Copied from LayoutTests/imported/w3c/web-platform-tests/streams/piping/close-propagation-forward.serviceworker.https-expected.txt.
- platform/mac-wk1/imported/w3c/web-platform-tests/streams/piping/error-propagation-backward.serviceworker.https-expected.txt: Copied from LayoutTests/imported/w3c/web-platform-tests/streams/piping/error-propagation-backward.serviceworker.https-expected.txt.
- platform/mac-wk1/imported/w3c/web-platform-tests/streams/piping/error-propagation-forward.serviceworker.https-expected.txt: Copied from LayoutTests/imported/w3c/web-platform-tests/streams/piping/error-propagation-forward.serviceworker.https-expected.txt.
- platform/mac-wk1/imported/w3c/web-platform-tests/streams/piping/flow-control.serviceworker.https-expected.txt: Copied from LayoutTests/imported/w3c/web-platform-tests/streams/piping/flow-control.serviceworker.https-expected.txt.
- platform/mac-wk1/imported/w3c/web-platform-tests/streams/piping/general.serviceworker.https-expected.txt: Copied from LayoutTests/imported/w3c/web-platform-tests/streams/piping/general.serviceworker.https-expected.txt.
- platform/mac-wk1/imported/w3c/web-platform-tests/streams/piping/multiple-propagation.serviceworker.https-expected.txt: Copied from LayoutTests/imported/w3c/web-platform-tests/streams/piping/multiple-propagation.serviceworker.https-expected.txt.
- platform/mac-wk1/imported/w3c/web-platform-tests/streams/piping/pipe-through.serviceworker.https-expected.txt: Copied from LayoutTests/imported/w3c/web-platform-tests/streams/piping/pipe-through.serviceworker.https-expected.txt.
- platform/mac-wk1/imported/w3c/web-platform-tests/streams/piping/transform-streams.serviceworker.https-expected.txt: Copied from LayoutTests/imported/w3c/web-platform-tests/streams/piping/transform-streams.serviceworker.https-expected.txt.
- platform/mac-wk1/imported/w3c/web-platform-tests/streams/readable-byte-streams/general.serviceworker.https-expected.txt: Copied from LayoutTests/imported/w3c/web-platform-tests/streams/piping/general.serviceworker.https-expected.txt.
- platform/mac-wk1/imported/w3c/web-platform-tests/streams/readable-streams/bad-strategies.serviceworker.https-expected.txt: Copied from LayoutTests/imported/w3c/web-platform-tests/streams/readable-streams/bad-strategies.serviceworker.https-expected.txt.
- platform/mac-wk1/imported/w3c/web-platform-tests/streams/readable-streams/bad-underlying-sources.serviceworker.https-expected.txt: Copied from LayoutTests/imported/w3c/web-platform-tests/streams/readable-streams/bad-underlying-sources.serviceworker.https-expected.txt.
- platform/mac-wk1/imported/w3c/web-platform-tests/streams/readable-streams/brand-checks.serviceworker.https-expected.txt: Copied from LayoutTests/imported/w3c/web-platform-tests/streams/readable-streams/brand-checks.serviceworker.https-expected.txt.
- platform/mac-wk1/imported/w3c/web-platform-tests/streams/readable-streams/cancel.serviceworker.https-expected.txt: Copied from LayoutTests/imported/w3c/web-platform-tests/streams/readable-streams/cancel.serviceworker.https-expected.txt.
- platform/mac-wk1/imported/w3c/web-platform-tests/streams/readable-streams/count-queuing-strategy-integration.serviceworker.https-expected.txt: Copied from LayoutTests/imported/w3c/web-platform-tests/streams/readable-streams/count-queuing-strategy-integration.serviceworker.https-expected.txt.
- platform/mac-wk1/imported/w3c/web-platform-tests/streams/readable-streams/default-reader.serviceworker.https-expected.txt: Copied from LayoutTests/imported/w3c/web-platform-tests/streams/readable-streams/default-reader.serviceworker.https-expected.txt.
- platform/mac-wk1/imported/w3c/web-platform-tests/streams/readable-streams/floating-point-total-queue-size.serviceworker.https-expected.txt: Copied from LayoutTests/imported/w3c/web-platform-tests/streams/readable-streams/floating-point-total-queue-size.serviceworker.https-expected.txt.
- platform/mac-wk1/imported/w3c/web-platform-tests/streams/readable-streams/garbage-collection.serviceworker.https-expected.txt: Copied from LayoutTests/imported/w3c/web-platform-tests/streams/readable-streams/garbage-collection.serviceworker.https-expected.txt.
- platform/mac-wk1/imported/w3c/web-platform-tests/streams/readable-streams/general.serviceworker.https-expected.txt: Copied from LayoutTests/imported/w3c/web-platform-tests/streams/piping/general.serviceworker.https-expected.txt.
- platform/mac-wk1/imported/w3c/web-platform-tests/streams/readable-streams/pipe-through.serviceworker.https-expected.txt: Copied from LayoutTests/imported/w3c/web-platform-tests/streams/piping/pipe-through.serviceworker.https-expected.txt.
- platform/mac-wk1/imported/w3c/web-platform-tests/streams/readable-streams/readable-stream-reader.serviceworker.https-expected.txt: Copied from LayoutTests/imported/w3c/web-platform-tests/streams/readable-streams/readable-stream-reader.serviceworker.https-expected.txt.
- platform/mac-wk1/imported/w3c/web-platform-tests/streams/readable-streams/tee.serviceworker.https-expected.txt: Copied from LayoutTests/imported/w3c/web-platform-tests/streams/readable-streams/tee.serviceworker.https-expected.txt.
- platform/mac-wk1/imported/w3c/web-platform-tests/streams/readable-streams/templated.serviceworker.https-expected.txt: Copied from LayoutTests/imported/w3c/web-platform-tests/streams/readable-streams/templated.serviceworker.https-expected.txt.
- platform/mac/js/dom/global-constructors-attributes-expected.txt:
- 8:04 AM Changeset in webkit [220269] by
-
- 41 edits in trunk
[EME][GStreamer] Register ClearKey CDMFactory
https://bugs.webkit.org/show_bug.cgi?id=175136
Reviewed by Xabier Rodriguez-Calvar.
Source/WebCore:
Register the ClearKey CDM factory in CDMFactoryGStreamer. A CDMFactoryClearKey
singleton object is introduced and used for that registration.
The basic CDMFactoryClearKey and CDMPrivateClearKey methods are implemented,
markingorg.w3.clearkeyas the supported key system andkeyidsas the
supported init data type. Additional logic around key system configurations,
distinctive identifiers, persistent state and related restrictions and
requirements is implemented.
This improves the ClearKey EME tests a bit, now progressing to the point of
failing with a NotAllowedError exception due to the CDMInstance object failing
to properly initialize because of missing implementation.
No new tests -- relevant tests have underlying baselines updated to reflect
changes in behavior.
- platform/encryptedmedia/clearkey/CDMClearKey.cpp:
(WebCore::CDMFactoryClearKey::singleton):
(WebCore::CDMFactoryClearKey::supportsKeySystem):
(WebCore::CDMPrivateClearKey::supportsInitDataType const):
(WebCore::CDMPrivateClearKey::supportsConfiguration const):
(WebCore::CDMPrivateClearKey::supportsConfigurationWithRestrictions const):
(WebCore::CDMPrivateClearKey::supportsSessionTypeWithConfiguration const):
(WebCore::CDMPrivateClearKey::supportsRobustness const):
(WebCore::CDMPrivateClearKey::distinctiveIdentifiersRequirement const):
(WebCore::CDMPrivateClearKey::persistentStateRequirement const):
- platform/encryptedmedia/clearkey/CDMClearKey.h:
- platform/graphics/gstreamer/eme/CDMFactoryGStreamer.cpp:
(WebCore::CDMFactory::platformRegisterFactories):
LayoutTests:
Update WPE baselines for EME ClearKey tests following some advancements in
ClearKey support.
- platform/wpe/imported/w3c/web-platform-tests/encrypted-media/clearkey-events-expected.txt:
- platform/wpe/imported/w3c/web-platform-tests/encrypted-media/clearkey-events-session-closed-event-expected.txt:
- platform/wpe/imported/w3c/web-platform-tests/encrypted-media/clearkey-generate-request-disallowed-input-expected.txt:
- platform/wpe/imported/w3c/web-platform-tests/encrypted-media/clearkey-invalid-license-expected.txt:
- platform/wpe/imported/w3c/web-platform-tests/encrypted-media/clearkey-keystatuses-expected.txt:
- platform/wpe/imported/w3c/web-platform-tests/encrypted-media/clearkey-keystatuses-multiple-sessions-expected.txt:
- platform/wpe/imported/w3c/web-platform-tests/encrypted-media/clearkey-mp4-playback-temporary-clear-encrypted-expected.txt:
- platform/wpe/imported/w3c/web-platform-tests/encrypted-media/clearkey-mp4-playback-temporary-encrypted-clear-expected.txt:
- platform/wpe/imported/w3c/web-platform-tests/encrypted-media/clearkey-mp4-playback-temporary-encrypted-clear-sources-expected.txt:
- platform/wpe/imported/w3c/web-platform-tests/encrypted-media/clearkey-mp4-playback-temporary-events-expected.txt:
- platform/wpe/imported/w3c/web-platform-tests/encrypted-media/clearkey-mp4-playback-temporary-expected.txt:
- platform/wpe/imported/w3c/web-platform-tests/encrypted-media/clearkey-mp4-playback-temporary-multikey-expected.txt:
- platform/wpe/imported/w3c/web-platform-tests/encrypted-media/clearkey-mp4-playback-temporary-multikey-sequential-expected.txt:
- platform/wpe/imported/w3c/web-platform-tests/encrypted-media/clearkey-mp4-playback-temporary-multikey-sequential-readyState-expected.txt:
- platform/wpe/imported/w3c/web-platform-tests/encrypted-media/clearkey-mp4-playback-temporary-multisession-expected.txt:
- platform/wpe/imported/w3c/web-platform-tests/encrypted-media/clearkey-mp4-playback-temporary-setMediaKeys-after-src-expected.txt:
- platform/wpe/imported/w3c/web-platform-tests/encrypted-media/clearkey-mp4-playback-temporary-setMediaKeys-after-update-expected.txt:
- platform/wpe/imported/w3c/web-platform-tests/encrypted-media/clearkey-mp4-playback-temporary-setMediaKeys-immediately-expected.txt:
- platform/wpe/imported/w3c/web-platform-tests/encrypted-media/clearkey-mp4-playback-temporary-setMediaKeys-onencrypted-expected.txt:
- platform/wpe/imported/w3c/web-platform-tests/encrypted-media/clearkey-mp4-playback-temporary-two-videos-expected.txt:
- platform/wpe/imported/w3c/web-platform-tests/encrypted-media/clearkey-mp4-playback-temporary-waitingforkey-expected.txt:
- platform/wpe/imported/w3c/web-platform-tests/encrypted-media/clearkey-mp4-requestmediakeysystemaccess-expected.txt:
- platform/wpe/imported/w3c/web-platform-tests/encrypted-media/clearkey-mp4-reset-src-after-setmediakeys-expected.txt:
- platform/wpe/imported/w3c/web-platform-tests/encrypted-media/clearkey-mp4-setmediakeys-again-after-playback-expected.txt:
- platform/wpe/imported/w3c/web-platform-tests/encrypted-media/clearkey-mp4-setmediakeys-again-after-resetting-src-expected.txt:
- platform/wpe/imported/w3c/web-platform-tests/encrypted-media/clearkey-mp4-setmediakeys-at-same-time-expected.txt:
- platform/wpe/imported/w3c/web-platform-tests/encrypted-media/clearkey-mp4-setmediakeys-expected.txt:
- platform/wpe/imported/w3c/web-platform-tests/encrypted-media/clearkey-mp4-setmediakeys-multiple-times-with-the-same-mediakeys-expected.txt:
- platform/wpe/imported/w3c/web-platform-tests/encrypted-media/clearkey-mp4-setmediakeys-to-multiple-video-elements-expected.txt:
- platform/wpe/imported/w3c/web-platform-tests/encrypted-media/clearkey-mp4-syntax-mediakeys-expected.txt:
- platform/wpe/imported/w3c/web-platform-tests/encrypted-media/clearkey-mp4-syntax-mediakeysession-expected.txt:
- platform/wpe/imported/w3c/web-platform-tests/encrypted-media/clearkey-mp4-syntax-mediakeysystemaccess-expected.txt:
- platform/wpe/imported/w3c/web-platform-tests/encrypted-media/clearkey-mp4-update-disallowed-input-expected.txt:
- platform/wpe/imported/w3c/web-platform-tests/encrypted-media/clearkey-mp4-waiting-for-a-key-expected.txt:
- platform/wpe/imported/w3c/web-platform-tests/encrypted-media/clearkey-not-callable-after-createsession-expected.txt:
- platform/wpe/imported/w3c/web-platform-tests/encrypted-media/clearkey-update-non-ascii-input-expected.txt:
- 7:20 AM Changeset in webkit [220268] by
-
- 14 edits2 adds in trunk
Resource Load Statistics: Report user interaction immediately, but only when needed
https://bugs.webkit.org/show_bug.cgi?id=175090
<rdar://problem/33685546>
Reviewed by Chris Dumez.
Source/WebCore:
Test: http/tests/loading/resourceLoadStatistics/user-interaction-only-reported-once-within-short-period-of-time.html
- loader/ResourceLoadObserver.cpp:
(WebCore::ResourceLoadObserver::ResourceLoadObserver):
(WebCore::ResourceLoadObserver::logUserInteractionWithReducedTimeResolution):
Now tells the UI process immediately but also records that it has
done so to avoid doing it when not needed.
(WebCore::ResourceLoadObserver::scheduleNotificationIfNeeded):
Conditional throttling gone, now always throttles.
(WebCore::ResourceLoadObserver::notifyObserver):
Renamed from ResourceLoadObserver::notificationTimerFired().
(WebCore::ResourceLoadObserver::clearState):
New function to allow the test runner to reset the web process'
statistics state now that we keep track of whether or not we've
reported user interaction to the UI process.
(WebCore::ResourceLoadObserver::setShouldThrottleObserverNotifications): Deleted.
(WebCore::ResourceLoadObserver::notificationTimerFired): Deleted.
- loader/ResourceLoadObserver.h:
(): Deleted.
- testing/Internals.cpp:
(WebCore::Internals::resetToConsistentState):
(WebCore::Internals::setResourceLoadStatisticsShouldThrottleObserverNotifications): Deleted.
No longer needed since user interaction is always communicated
immediately.
- testing/Internals.h:
- testing/Internals.idl:
Source/WebKit:
- WebProcess/InjectedBundle/API/c/WKBundle.cpp:
(WKBundleClearResourceLoadStatistics):
Test infrastructure. Ends up calling
WebCore::ResourceLoadObserver::clearState().
- WebProcess/InjectedBundle/API/c/WKBundlePrivate.h:
Tools:
- WebKitTestRunner/InjectedBundle/InjectedBundle.cpp:
(WTR::InjectedBundle::beginTesting):
Now calls WebCore::ResourceLoadObserver::clearState().
LayoutTests:
- http/tests/loading/resourceLoadStatistics/user-interaction-in-cross-origin-sub-frame.html:
Now no longer needs to disable throttling since reports of
user interaction happen immediately (when needed).
- http/tests/loading/resourceLoadStatistics/user-interaction-only-reported-once-within-short-period-of-time-expected.txt: Added.
- http/tests/loading/resourceLoadStatistics/user-interaction-only-reported-once-within-short-period-of-time.html: Added.
- platform/mac-wk2/TestExpectations:
user-interaction-only-reported-once-within-short-period-of-time.html marked as [ Pass ].
- 6:25 AM Changeset in webkit [220267] by
-
- 22 edits in trunk/Source/WebKit
Network cache should be usable as non-singleton
https://bugs.webkit.org/show_bug.cgi?id=175139
Reviewed by Sam Weinig.
We might want to use it as a non-singleton in the future (for example as a backend for the cache API).
This patch makes NetworkCache::Cache and NetworkCache::Storage refcounted objects and takes
care to ref them properly during asynchronous operations.
The patch doesn't actually create any non-shared instances, it just adds the capability.
- NetworkProcess/NetworkConnectionToWebProcess.cpp:
(WebKit::NetworkConnectionToWebProcess::storeDerivedDataToCache):
- NetworkProcess/NetworkProcess.cpp:
(WebKit::fetchDiskCacheEntries):
(WebKit::clearDiskCacheEntries):
(WebKit::NetworkProcess::setCacheModel):
- NetworkProcess/NetworkProcess.h:
(WebKit::NetworkProcess::cache):
Move the shared cache instance to the network process singleton.
- NetworkProcess/NetworkResourceLoader.cpp:
(WebKit::NetworkResourceLoader::NetworkResourceLoader):
Include shared cache as a reffed member for non-ephemeral instances.
(WebKit::NetworkResourceLoader::canUseCache const):
(WebKit::NetworkResourceLoader::retrieveCacheEntry):
(WebKit::NetworkResourceLoader::abort):
(WebKit::NetworkResourceLoader::didReceiveResponse):
(WebKit::NetworkResourceLoader::willSendRedirectedRequest):
(WebKit::NetworkResourceLoader::tryStoreAsCacheEntry):
(WebKit::NetworkResourceLoader::didRetrieveCacheEntry):
- NetworkProcess/NetworkResourceLoader.h:
- NetworkProcess/cache/NetworkCache.cpp:
(WebKit::NetworkCache::Cache::open):
Open now returns null if it fails.
Add RegisterNotify option to set up notify trigger for the shared instance.
(WebKit::NetworkCache::Cache::Cache):
(WebKit::NetworkCache::Cache::~Cache):
Add destructor.
(WebKit::NetworkCache::dumpFileChanged):
(WebKit::NetworkCache::Cache::setCapacity):
(WebKit::NetworkCache::Cache::retrieve):
Protect the cache instance during asynchronous operations.
(WebKit::NetworkCache::Cache::store):
(WebKit::NetworkCache::Cache::storeRedirect):
(WebKit::NetworkCache::Cache::remove):
(WebKit::NetworkCache::Cache::traverse):
(WebKit::NetworkCache::Cache::dumpContentsToFile):
(WebKit::NetworkCache::Cache::clear):
(WebKit::NetworkCache::Cache::recordsPath const):
(WebKit::NetworkCache::Cache::retrieveData):
(WebKit::NetworkCache::Cache::storeData):
(WebKit::NetworkCache::singleton): Deleted.
(WebKit::NetworkCache::Cache::initialize): Deleted.
- NetworkProcess/cache/NetworkCache.h:
(WebKit::NetworkCache::Cache::canUseSharedMemoryForBodyData const):
(WebKit::NetworkCache::Cache::isEnabled const): Deleted.
Remove isEnabled() state as a cache object now always represents an enabled cache.
- NetworkProcess/cache/NetworkCacheEntry.cpp:
(WebKit::NetworkCache::Entry::initializeShareableResourceHandleFromStorageRecord const):
- NetworkProcess/cache/NetworkCacheSpeculativeLoad.cpp:
(WebKit::NetworkCache::SpeculativeLoad::willSendRedirectedRequest):
(WebKit::NetworkCache::SpeculativeLoad::didReceiveResponse):
(WebKit::NetworkCache::SpeculativeLoad::didFinishLoading):
- NetworkProcess/cache/NetworkCacheStatistics.cpp:
(WebKit::NetworkCache::Statistics::initialize):
(WebKit::NetworkCache::Statistics::shrinkIfNeeded):
- NetworkProcess/cache/NetworkCacheStorage.cpp:
(WebKit::NetworkCache::Storage::ReadOperation::ReadOperation):
(WebKit::NetworkCache::Storage::WriteOperation::WriteOperation):
(WebKit::NetworkCache::Storage::TraverseOperation::TraverseOperation):
Operations now ref the storage. They are already deleted in the main thread so
proper destruction is taken care of.
(WebKit::NetworkCache::Storage::open):
(WebKit::NetworkCache::Storage::~Storage):
(WebKit::NetworkCache::Storage::synchronize):
This and other asynchronous methods now protect the Storage instance.
(WebKit::NetworkCache::Storage::remove):
(WebKit::NetworkCache::Storage::retrieve):
(WebKit::NetworkCache::Storage::store):
(WebKit::NetworkCache::Storage::traverse):
(WebKit::NetworkCache::Storage::clear):
(WebKit::NetworkCache::Storage::shrink):
(WebKit::NetworkCache::Storage::deleteOldVersions):
- NetworkProcess/cache/NetworkCacheStorage.h:
- NetworkProcess/cocoa/NetworkProcessCocoa.mm:
(WebKit::NetworkProcess::platformInitializeNetworkProcessCocoa):
(WebKit::NetworkProcess::clearDiskCache):
- NetworkProcess/cocoa/NetworkSessionCocoa.mm:
(WebKit::NetworkSessionCocoa::NetworkSessionCocoa):
- NetworkProcess/soup/NetworkProcessSoup.cpp:
(WebKit::NetworkProcess::platformInitializeNetworkProcess):
(WebKit::NetworkProcess::clearDiskCache):
- 3:12 AM Changeset in webkit [220266] by
-
- 2 edits1 delete in trunk/LayoutTests
[GTK] Test gardening around MOUSE_CURSOR_SCALE.
https://bugs.webkit.org/show_bug.cgi?id=109469
Patch by Ms2ger <Ms2ger@igalia.com> on 2017-08-04
Reviewed by Carlos Alberto Lopez Perez.
Remove obsolete expectation for cursor-parsing-image-set.html; it may have
started passing in r209396.
Mark mouse-cursor-image-set.html as failing rather than using an incorrect
-expected file.
- platform/gtk/TestExpectations:
- platform/gtk/fast/events/mouse-cursor-image-set-expected.txt: Removed.
- 3:01 AM Changeset in webkit [220265] by
-
- 2 edits in trunk/JSTests
JSC test wasm/js-api/test_memory_constructor.js should be skipped on memoryLimited
https://bugs.webkit.org/show_bug.cgi?id=175150
Unreviewed test gardening.
- wasm/js-api/test_memory_constructor.js:
- 2:54 AM Changeset in webkit [220264] by
-
- 10 edits3 adds in trunk/Source/WebCore
[EME] Push CDMFactory into the platform layer
https://bugs.webkit.org/show_bug.cgi?id=175129
Reviewed by Xabier Rodriguez-Calvar.
This is a follow-up to r219678 that moved the majority of CDM abstraction
classes into the platform layer, but missed the CDMFactory class.
The CDMFactory abstraction is now also placed in the platform layer. Only
change to the interface is that the createCDM() method can't accept a CDM
object reference anymore since that class is cemented into the WebCore
layer, and no current implementation used it anyway.
Additionally, the static Vector object of registered factories is moved
under the CDMFactory class, along with the register and unregister
functions. The platformRegisterFactories() function is added to allow for
platform-specific factory registrations to occur when the registered
factories are queried for the first time. Empty implementation for this
function is provided for non-GStreamer platforms, while for GStreamer
the implementation is kept in CDMFactoryGStreamer.cpp. It's still empty
for now, but it will register the ClearKey factory there in the near
future.
No new tests -- none affected, only refactoring.
- CMakeLists.txt:
- Modules/encryptedmedia/CDM.cpp:
(WebCore::createCDMPrivateForKeySystem):
(WebCore::CDM::supportsKeySystem):
(WebCore::CDM::CDM):
(): Deleted.
(WebCore::CDM::registerCDMFactory): Deleted.
(WebCore::CDM::unregisterCDMFactory): Deleted.
- Modules/encryptedmedia/CDM.h:
(WebCore::CDMFactory::~CDMFactory): Deleted.
- PlatformWPE.cmake:
- platform/GStreamer.cmake:
- platform/encryptedmedia/CDMFactory.cpp: Added.
(WebCore::CDMFactory::registerFactory):
(WebCore::CDMFactory::unregisterFactory):
(WebCore::CDMFactory::platformRegisterFactories):
- platform/encryptedmedia/CDMFactory.h: Added.
(WebCore::CDMFactory::~CDMFactory):
- platform/encryptedmedia/clearkey/CDMClearKey.cpp:
(WebCore::CDMFactoryClearKey::createCDM):
- platform/encryptedmedia/clearkey/CDMClearKey.h:
- platform/encryptedmedia/gstreamer/CDMFactoryGStreamer.cpp: Added.
(WebCore::CDMFactory::platformRegisterFactories):
- testing/MockCDMFactory.cpp:
(WebCore::m_weakPtrFactory):
(WebCore::MockCDMFactory::unregister):
(WebCore::MockCDMFactory::createCDM):
- testing/MockCDMFactory.h:
- 2:08 AM Changeset in webkit [220263] by
-
- 2 edits15 adds in trunk/LayoutTests
Unreviewed GTK+ gardening. Update test expectations and layout test baselines
for Web Crypto tests now that the implementation is complete.
- platform/gtk/TestExpectations:
- platform/gtk/imported/w3c/web-platform-tests/WebCryptoAPI/encrypt_decrypt: Added.
- platform/gtk/imported/w3c/web-platform-tests/WebCryptoAPI/encrypt_decrypt/aes_cbc.worker-expected.txt: Added.
- platform/gtk/imported/w3c/web-platform-tests/WebCryptoAPI/encrypt_decrypt/test_aes_cbc.https-expected.txt: Added.
- platform/gtk/imported/w3c/web-platform-tests/WebCryptoAPI/generateKey/successes.worker-expected.txt: Added.
- platform/gtk/imported/w3c/web-platform-tests/WebCryptoAPI/import_export: Added.
- platform/gtk/imported/w3c/web-platform-tests/WebCryptoAPI/import_export/rsa_importKey.worker-expected.txt: Added.
- platform/gtk/imported/w3c/web-platform-tests/WebCryptoAPI/import_export/test_rsa_importKey.https-expected.txt: Added.
- platform/gtk/imported/w3c/web-platform-tests/WebCryptoAPI/sign_verify: Added.
- platform/gtk/imported/w3c/web-platform-tests/WebCryptoAPI/sign_verify/rsa_pkcs.worker-expected.txt: Added.
- platform/gtk/imported/w3c/web-platform-tests/WebCryptoAPI/sign_verify/rsa_pss.worker-expected.txt: Added.
- platform/gtk/imported/w3c/web-platform-tests/WebCryptoAPI/sign_verify/test_rsa_pkcs.https-expected.txt: Added.
- platform/gtk/imported/w3c/web-platform-tests/WebCryptoAPI/sign_verify/test_rsa_pss.https-expected.txt: Added.
- platform/gtk/imported/w3c/web-platform-tests/WebCryptoAPI/wrapKey_unwrapKey: Added.
- platform/gtk/imported/w3c/web-platform-tests/WebCryptoAPI/wrapKey_unwrapKey/test_wrapKey_unwrapKey.https-expected.txt: Added.
- platform/gtk/imported/w3c/web-platform-tests/WebCryptoAPI/wrapKey_unwrapKey/wrapKey_unwrapKey.worker-expected.txt: Added.
- 1:51 AM Changeset in webkit [220262] by
-
- 2 edits in trunk/Source/JavaScriptCore
[EsNext] Async iteration - Add feature flag
https://bugs.webkit.org/show_bug.cgi?id=166694
Reviewed by Yusuke Suzuki.
Add feature flag to JSC to switch on/off Async Iterator
- runtime/Options.h:
- 1:08 AM Changeset in webkit [220261] by
-
- 7 edits2 adds in trunk
ScrollingTreeOverflowScrollingNodeIOS uses the wrong fixed position rectangle
https://bugs.webkit.org/show_bug.cgi?id=175135
Patch by Frederic Wang <fwang@igalia.com> on 2017-08-04
Reviewed by Simon Fraser.
Source/WebCore:
This patch modifies ScrollingTreeOverflowScrollingNodeIOS::updateChildNodesAfterScroll so
that it uses the fixed position rectangle relative of the first frame ancestor instead of
the one of the main frame. This makes it consistent with ScrollingTreeFrameScrollingNodeIOS
and RenderLayerCompositor. This fixes some flickering issues on iOS.
Test: fast/scrolling/ios/fixed-inside-overflow-inside-iframe.html
- page/scrolling/ScrollingTreeFrameScrollingNode.h:
(WebCore::ScrollingTreeFrameScrollingNode::fixedPositionRect): Helper function to get the
fixed position rect to use for that frame.
- page/scrolling/ScrollingTreeNode.cpp:
(WebCore::ScrollingTreeNode::enclosingFrameNode const): Helper function to get the enclosing
frame for this scrolling node or null if there is none.
- page/scrolling/ScrollingTreeNode.h: Declare enclosingFrameNode.
Source/WebKit:
This patch modifies ScrollingTreeOverflowScrollingNodeIOS::updateChildNodesAfterScroll so
that it uses the fixed position rectangle relative of the first frame ancestor instead of
the one of the main frame. This makes it consistent with ScrollingTreeFrameScrollingNodeIOS
and RenderLayerCompositor. This fixes some flickering issues on iOS.
- UIProcess/Scrolling/ios/ScrollingTreeOverflowScrollingNodeIOS.mm:
(WebKit::ScrollingTreeOverflowScrollingNodeIOS::updateChildNodesAfterScroll): Use the fixed
position rect of a non-main frame if there is one.
LayoutTests:
This patch adds a new test for a position:fixed element inside an overflow node inside an
iframe. When scrolling the overflow node, the position of such an element should remain fixed
relative to the inner frame. Before that change, ScrollingTreeOverflowScrollingNodeIOS used
to take the main frame as a reference instead, causing the element to flicker and even to
disappear when the user scrolls that overflow node. We add a reftest to verify that the
element is visible and positioned at the correct location when the user scrolls.
- fast/scrolling/ios/fixed-inside-overflow-inside-iframe-expected.html: Added.
- fast/scrolling/ios/fixed-inside-overflow-inside-iframe.html: Added.
- 12:56 AM Changeset in webkit [220260] by
-
- 3 edits in trunk/Source/WebCore
Unreviewed. Removing redundant NotImplemented.h header inclusions
and cleaning up whitespace issues in libgcrypt-specific CryptoKeyEC
and CryptoKeyRSA implementation files.
- crypto/gcrypt/CryptoKeyECGCrypt.cpp:
- crypto/gcrypt/CryptoKeyRSAGCrypt.cpp:
- 12:52 AM Changeset in webkit [220259] by
-
- 2 edits12 adds in trunk/LayoutTests
Unreviewed WPE gardening. Update test expectations and layout test baselines
for Web Crypto tests now that the implementation is complete.
- platform/wpe/TestExpectations:
- platform/wpe/imported/w3c/web-platform-tests/WebCryptoAPI/generateKey/successes.worker-expected.txt: Added.
- platform/wpe/imported/w3c/web-platform-tests/WebCryptoAPI/import_export: Added.
- platform/wpe/imported/w3c/web-platform-tests/WebCryptoAPI/import_export/rsa_importKey.worker-expected.txt: Added.
- platform/wpe/imported/w3c/web-platform-tests/WebCryptoAPI/import_export/test_rsa_importKey.https-expected.txt: Added.
- platform/wpe/imported/w3c/web-platform-tests/WebCryptoAPI/sign_verify: Added.
- platform/wpe/imported/w3c/web-platform-tests/WebCryptoAPI/sign_verify/rsa_pkcs.worker-expected.txt: Added.
- platform/wpe/imported/w3c/web-platform-tests/WebCryptoAPI/sign_verify/rsa_pss.worker-expected.txt: Added.
- platform/wpe/imported/w3c/web-platform-tests/WebCryptoAPI/sign_verify/test_rsa_pkcs.https-expected.txt: Added.
- platform/wpe/imported/w3c/web-platform-tests/WebCryptoAPI/sign_verify/test_rsa_pss.https-expected.txt: Added.
- platform/wpe/imported/w3c/web-platform-tests/WebCryptoAPI/wrapKey_unwrapKey: Added.
- platform/wpe/imported/w3c/web-platform-tests/WebCryptoAPI/wrapKey_unwrapKey/test_wrapKey_unwrapKey.https-expected.txt: Added.
- platform/wpe/imported/w3c/web-platform-tests/WebCryptoAPI/wrapKey_unwrapKey/wrapKey_unwrapKey.worker-expected.txt: Added.
- 12:36 AM Changeset in webkit [220258] by
-
- 2 edits in trunk/Source/WebCore
Remove unnecesary call to status bar SPI.
https://bugs.webkit.org/show_bug.cgi?id=175176
rdar://problem/20887306
Patch by Jeremy Jones <jeremyj@apple.com> on 2017-08-04
Reviewed by Darin Adler.
No new tests because no behavior change.
This removes an obsolete call to SPI.
- platform/ios/VideoFullscreenInterfaceAVKit.mm:
(VideoFullscreenInterfaceAVKit::cleanupFullscreen):
Aug 3, 2017:
- 11:38 PM Changeset in webkit [220257] by
-
- 6 edits in branches/safari-604.1-branch
Cherry-pick r220185. rdar://problem/33713528
- 11:38 PM Changeset in webkit [220256] by
-
- 4 edits3 adds in branches/safari-604.1-branch
Cherry-pick r220163. rdar://problem/33711018
- 11:38 PM Changeset in webkit [220255] by
-
- 2 edits in branches/safari-604.1-branch/Source/WebCore
Cherry-pick r220153. rdar://problem/33711038
- 11:38 PM Changeset in webkit [220254] by
-
- 8 edits in branches/safari-604.1-branch
Cherry-pick r219896. rdar://problem/33711000
- 11:13 PM Changeset in webkit [220253] by
-
- 5 edits in trunk
[GCrypt] Implement CryptoKeyEC PKCS#8 imports
https://bugs.webkit.org/show_bug.cgi?id=173647
Reviewed by Jiewen Tan.
Source/WebCore:
No new tests -- affected tests are now passing and are unskipped.
Implement libgcrypt-based support for PKCS#8 imports of EC keys.
Existing libtasn1 utilities are used to achieve this. First, the provided key data
is decoded against the PrivateKeyInfo ASN.1 definition. First, the version member
of that structure is validated, followed by the algorithm member. The latter is
also properly tested depending on this being an import of an ECDSA or ECDH key.
Data of the parameters member is decoded against the ECParameters ASN.1 definition,
and the namedCurve object identifier is validated, making sure it represents a
valid EC curve and that this curve maches the one specified for the import
operation.
Data of the privateKey member is decoded against the ECPrivateKey ASN.1 definition.
The version member of that structure is properly validated. The optional parameters
member of that structure is already decoded against the ECParameters ASN.1
definition. If present, it is checked to contain a valid EC curve identifier that
matches the specified curve.
The optional publicKey member of the ECPrivateKey structure is validated, testing
that its data matches in size an uncompressed EC point, and that the first byte
of this data is 0x04, as expected for an uncompressed EC point.
What's left is the private key data on the initial ECPrivateKey structure. That
data is retrieved and validated, making sure its size matches the size of the
specified curve. Theprivate-keys-expression is then constructed, embedding
the curve name and the validated private key data. This s-expression is then used
to construct an EC context.
If the optional publicKey data was provided, it's used to set the
qparameter
for this EC context. Otherwise, the value forqis computed on-the-fly for the
specified EC and the provided private key. Theqpoint is then tested through
the gcry_mpi_ec_curve_point() function, making sure that the derived point is
indeed located on the given EC.
Finally, with the private key properly validated, a new CryptoKeyEC object is
constructed, using theprivate-keys-expression and the parameters that were
specified for this import operation.
- crypto/gcrypt/CryptoKeyECGCrypt.cpp:
(WebCore::CryptoKeyEC::platformImportPkcs8):
- crypto/gcrypt/GCryptUtilities.h:
LayoutTests:
- platform/wpe/TestExpectations:
Unskip passing Web Crypto tests that cover PKCS#8 imports of EC keys.
- 9:37 PM Changeset in webkit [220252] by
-
- 12 edits1 move2 adds1 delete in trunk
Fix parsing of <meta http-equiv=refresh> to allow time starting with a '.' without a leading 0
https://bugs.webkit.org/show_bug.cgi?id=175132
Reviewed by Darin Adler.
LayoutTests/imported/w3c:
Re-sync WPT tests from upstream c8bf1bbe9296. This extends test coverage.
- web-platform-tests/html/semantics/document-metadata/the-meta-element/pragma-directives/attr-meta-http-equiv-refresh/allow-scripts-flag-changing-1-expected.txt:
- web-platform-tests/html/semantics/document-metadata/the-meta-element/pragma-directives/attr-meta-http-equiv-refresh/allow-scripts-flag-changing-2-expected.txt:
- web-platform-tests/html/semantics/document-metadata/the-meta-element/pragma-directives/attr-meta-http-equiv-refresh/moving-documents-expected.txt: Removed.
- web-platform-tests/html/semantics/document-metadata/the-meta-element/pragma-directives/attr-meta-http-equiv-refresh/parsing-expected.txt:
- web-platform-tests/html/semantics/document-metadata/the-meta-element/pragma-directives/attr-meta-http-equiv-refresh/parsing.html:
- web-platform-tests/html/semantics/document-metadata/the-meta-element/pragma-directives/attr-meta-http-equiv-refresh/remove-from-document-expected.txt: Added.
- web-platform-tests/html/semantics/document-metadata/the-meta-element/pragma-directives/attr-meta-http-equiv-refresh/remove-from-document.html: Renamed from LayoutTests/imported/w3c/web-platform-tests/html/semantics/document-metadata/the-meta-element/pragma-directives/attr-meta-http-equiv-refresh/moving-documents.html.
- web-platform-tests/html/semantics/document-metadata/the-meta-element/pragma-directives/attr-meta-http-equiv-refresh/support/refresh.sub.html: Added.
- web-platform-tests/html/semantics/document-metadata/the-meta-element/pragma-directives/attr-meta-http-equiv-refresh/support/w3c-import.log:
- web-platform-tests/html/semantics/document-metadata/the-meta-element/pragma-directives/attr-meta-http-equiv-refresh/w3c-import.log:
Source/WebCore:
Fix parsing of <meta http-equiv=refresh> to allow time starting with a '.', without
a leading 0. This is as per https://github.com/whatwg/html/pull/2852.
The latest spec is at:
Test: imported/w3c/web-platform-tests/html/semantics/document-metadata/the-meta-element/pragma-directives/attr-meta-http-equiv-refresh/parsing.html
- html/parser/HTMLParserIdioms.cpp:
(WebCore::parseHTTPRefreshInternal):
LayoutTests:
Mark as flaky tests that used to not run because they were missing a subresource. Now that I imported
this subresource, the tests run but are failing. When they fail, those tests are flaky due to the lines
they log.
- tests-options.json:
- 8:57 PM Changeset in webkit [220251] by
-
- 2 edits in trunk/Source/WebInspectorUI
JSContext Inspector: Scripts sometimes do not show in resources tab
https://bugs.webkit.org/show_bug.cgi?id=175153
<rdar://problem/33708683>
Patch by Joseph Pecoraro <Joseph Pecoraro> on 2017-08-03
Reviewed by Matt Baker.
- UserInterface/Views/ResourceSidebarPanel.js:
(WI.ResourceSidebarPanel.prototype.initialLayout):
When the ResourceSidebar is lazily created, be sure to add any scripts
to the sidebar that are not backed by Resources.
(WI.ResourceSidebarPanel.prototype._scriptWasAdded):
(WI.ResourceSidebarPanel.prototype._addScript):
Extract so it can be used outside of an event handler.
- 7:41 PM Changeset in webkit [220250] by
-
- 66 edits in trunk
Remove ENABLE(WEB_SOCKET) guards
https://bugs.webkit.org/show_bug.cgi?id=167044
Reviewed by Joseph Pecoraro.
.:
- Source/cmake/OptionsMac.cmake:
- Source/cmake/OptionsWin.cmake:
- Source/cmake/WebKitFeatures.cmake:
- Source/cmake/tools/vsprops/FeatureDefines.props:
- Source/cmake/tools/vsprops/FeatureDefinesCairo.props:
Source/JavaScriptCore:
- Configurations/FeatureDefines.xcconfig:
Source/WebCore:
- Configurations/FeatureDefines.xcconfig:
- Modules/websockets/ThreadableWebSocketChannel.cpp:
- Modules/websockets/ThreadableWebSocketChannel.h:
- Modules/websockets/ThreadableWebSocketChannelClientWrapper.cpp:
- Modules/websockets/ThreadableWebSocketChannelClientWrapper.h:
- Modules/websockets/WebSocket.cpp:
- Modules/websockets/WebSocket.h:
- Modules/websockets/WebSocket.idl:
- Modules/websockets/WebSocketChannel.cpp:
- Modules/websockets/WebSocketChannel.h:
- Modules/websockets/WebSocketChannelClient.h:
- Modules/websockets/WebSocketDeflateFramer.cpp:
- Modules/websockets/WebSocketDeflateFramer.h:
- Modules/websockets/WebSocketDeflater.cpp:
- Modules/websockets/WebSocketDeflater.h:
- Modules/websockets/WebSocketExtensionDispatcher.cpp:
- Modules/websockets/WebSocketExtensionDispatcher.h:
- Modules/websockets/WebSocketExtensionParser.cpp:
- Modules/websockets/WebSocketExtensionParser.h:
- Modules/websockets/WebSocketExtensionProcessor.h:
- Modules/websockets/WebSocketFrame.cpp:
- Modules/websockets/WebSocketFrame.h:
- Modules/websockets/WebSocketHandshake.cpp:
- Modules/websockets/WebSocketHandshake.h:
- Modules/websockets/WorkerThreadableWebSocketChannel.cpp:
- Modules/websockets/WorkerThreadableWebSocketChannel.h:
- dom/Document.cpp:
(WebCore::Document::Document):
(WebCore::Document::idbConnectionProxy):
(WebCore::Document::socketProvider):
- dom/Document.h:
- dom/ScriptExecutionContext.h:
- inspector/InspectorInstrumentation.cpp:
(WebCore::InspectorInstrumentation::didSendWebSocketFrameImpl):
- inspector/InspectorInstrumentation.h:
(WebCore::InspectorInstrumentation::didSendWebSocketFrame):
- inspector/InspectorNetworkAgent.cpp:
- inspector/InspectorNetworkAgent.h:
- page/RuntimeEnabledFeatures.cpp:
(WebCore::RuntimeEnabledFeatures::webSocketEnabled const):
- page/RuntimeEnabledFeatures.h:
- page/SocketProvider.cpp:
- page/SocketProvider.h:
- workers/WorkerGlobalScope.cpp:
(WebCore::WorkerGlobalScope::WorkerGlobalScope):
- workers/WorkerGlobalScope.h:
- workers/WorkerMessagingProxy.cpp:
(WebCore::WorkerMessagingProxy::startWorkerGlobalScope):
- workers/WorkerThread.cpp:
(WebCore::WorkerThread::WorkerThread):
(WebCore::WorkerThread::socketProvider):
- workers/WorkerThread.h:
Source/WebCore/PAL:
- Configurations/FeatureDefines.xcconfig:
Source/WebKit:
- Configurations/FeatureDefines.xcconfig:
- WebProcess/Network/WebSocketProvider.cpp:
- WebProcess/Network/WebSocketProvider.h:
Source/WebKitLegacy/mac:
- Configurations/FeatureDefines.xcconfig:
- WebView/WebPreferencesPrivate.h:
Source/WTF:
- wtf/FeatureDefines.h:
Tools:
- Scripts/webkitperl/FeatureList.pm:
- TestWebKitAPI/Configurations/FeatureDefines.xcconfig:
- 7:37 PM Changeset in webkit [220249] by
-
- 6 edits1 add in trunk/Source/WebCore
Remove LayoutUnit dependency in TextStream
https://bugs.webkit.org/show_bug.cgi?id=175110
Reviewed by Zalan Bujtas.
No new tests. No change in behavior.
- CMakeLists.txt:
- WebCore.xcodeproj/project.pbxproj:
- platform/LayoutUnit.cpp: Added.
(WebCore::operator<<):
- platform/LayoutUnit.h:
- platform/text/TextStream.cpp:
- platform/text/TextStream.h:
- 6:58 PM Changeset in webkit [220248] by
-
- 2 edits in trunk/Source/WebCore
Improve WebKitLegacy video fullscreen animation begin and end rects.
https://bugs.webkit.org/show_bug.cgi?id=175152
rdar://problem/32840576
Patch by Jeremy Jones <jeremyj@apple.com> on 2017-08-03
Reviewed by Eric Carlson.
No new tests, becuase this change has no effect on the DOM.
This change uses different rects for fullscreen animation to prevent the animation
from failing, and to improve the aesthetics of the animation.
- platform/mac/WebVideoFullscreenController.mm:
(frameExpandedToRatioOfFrame):
(-[WebVideoFullscreenController enterFullscreen:]):
(-[WebVideoFullscreenController exitFullscreen]):
(-[WebVideoFullscreenWindow animateFromRect:toRect:withSubAnimation:controllerAction:]):
(constrainFrameToRatioOfFrame): Deleted.
- 6:44 PM WebKitGTK/2.16.x edited by
- (diff)
- 6:38 PM WebKitGTK/2.16.x edited by
- (diff)
- 6:13 PM Changeset in webkit [220247] by
-
- 4 edits in trunk/Source/WebCore
[EME][Mac] SecureStop left on disk in Private Browsing mode.
https://bugs.webkit.org/show_bug.cgi?id=175162
Reviewed by Eric Carlson.
Return an empty string from mediaKeysStorageDirectory() when the page indicates that storage should
be ephemeral(). Previously, an empty string in this case would be treated as an error. Instead, treat
an empty string as valid, and do not try to store or retrieve session information to disk in that case.
- Modules/encryptedmedia/legacy/WebKitMediaKeySession.cpp:
(WebCore::WebKitMediaKeySession::mediaKeysStorageDirectory const):
- platform/graphics/avfoundation/objc/CDMSessionAVContentKeySession.mm:
(WebCore::CDMSessionAVContentKeySession::releaseKeys):
(WebCore::CDMSessionAVContentKeySession::update):
(WebCore::CDMSessionAVContentKeySession::generateKeyReleaseMessage):
(WebCore::CDMSessionAVContentKeySession::contentKeySession):
- platform/graphics/avfoundation/objc/CDMSessionMediaSourceAVFObjC.mm:
(WebCore::CDMSessionMediaSourceAVFObjC::storagePath const):
- 6:12 PM Changeset in webkit [220246] by
-
- 3 edits in trunk/Tools
REGRESSION(r219850): run-benchmark script broken on Linux
https://bugs.webkit.org/show_bug.cgi?id=175126
Reviewed by Stephanie Lewis.
The run-benchmark script dynamically generates the list of supported
browsers and platforms (currently Linux and OSX) by loading all
python files from Tools/Scripts/webkitpy/benchmark_runner/browser_driver
and getting the browser_name and platform variables from the
classes defined there.
This means that this classes should not raise an exception when
loaded on other platforms or otherwise they will broke the whole
script. Its fine if they raise an exception when executing any of
the methods they implement, but not when just loading/importing
the class.
Move the argument variable definitions that call on the platform
specific OSXBrowserDriver._screen_size() function from beeing
variables that are evaluated when loading the file, to be functions
that are only evaluated when the actual functionality needs to be
executed.
- Scripts/webkitpy/benchmark_runner/browser_driver/osx_chrome_driver.py:
(OSXChromeDriver.launch_url):
(OSXChromeCanaryDriver.launch_url):
(create_args):
(create_chrome_options):
(create_window_size_arg):
- Scripts/webkitpy/benchmark_runner/browser_driver/osx_firefox_driver.py:
(OSXFirefoxDriver.launch_url):
(OSXFirefoxNightlyDriver.launch_url):
(OSXFirefoxNightlyDriver.launch_driver):
(create_args):
- 6:07 PM Changeset in webkit [220245] by
-
- 11 edits7 deletes in trunk
Multiple Layout tests from web-platform-tests/beacon/ are timing out.
https://bugs.webkit.org/show_bug.cgi?id=175076
<rdar://problem/33704752>
Reviewed by Alexey Proskuryakov.
LayoutTests/imported/w3c:
Rebaseline tests that are now passing.
- web-platform-tests/fetch/api/cors/cors-basic.any-expected.txt:
- web-platform-tests/fetch/api/cors/cors-basic.any.worker-expected.txt:
- web-platform-tests/fetch/api/cors/cors-no-preflight.any-expected.txt:
- web-platform-tests/fetch/api/cors/cors-origin.any-expected.txt:
- web-platform-tests/fetch/api/cors/cors-origin.any.worker-expected.txt:
Source/WebKit:
- NetworkProcess/NetworkLoad.cpp:
(WebKit::NetworkLoad::completeAuthenticationChallenge):
In the NETWORK_SESSION code path, we did not ask the client about server trust evaluation
when the clientCredentialPolicy was CannotAskClientForCredentials. This is because the
same delegate is used for HTTP authentication and server trust evaluation in the
NETWORK_SESSION code path. To align both code paths, we now ask the client about server
trust evaluation even if the policy CannotAskClientForCredentials. This allows WKTR
to trust certificates for localhost / 127.0.0.1 unconditionally and consistently.
LayoutTests:
- platform/ios-wk2/imported/w3c/web-platform-tests/fetch/api/basic/mode-no-cors-expected.txt: Removed.
- platform/ios-wk2/imported/w3c/web-platform-tests/fetch/api/basic/mode-no-cors-worker-expected.txt: Removed.
- platform/mac-wk1/imported/w3c/web-platform-tests/fetch/api/cors/cors-basic.any-expected.txt: Removed.
- platform/mac-wk1/imported/w3c/web-platform-tests/fetch/api/cors/cors-basic.any.worker-expected.txt: Removed.
- platform/mac-wk1/imported/w3c/web-platform-tests/fetch/api/cors/cors-no-preflight.any-expected.txt: Removed.
- platform/mac-wk1/imported/w3c/web-platform-tests/fetch/api/cors/cors-origin.any-expected.txt: Removed.
- platform/mac-wk1/imported/w3c/web-platform-tests/fetch/api/cors/cors-origin.any.worker-expected.txt: Removed.
- platform/mac-wk2/imported/w3c/web-platform-tests/fetch/api/basic/mode-no-cors-expected.txt: Removed.
- platform/mac-wk2/imported/w3c/web-platform-tests/fetch/api/basic/mode-no-cors-worker-expected.txt: Removed.
Drop platform-specific expectations as those tests are now passing everywhere.
- platform/wk2/TestExpectations:
Unskip tests that are now passing.
- 6:00 PM Changeset in webkit [220244] by
-
- 8 edits2 adds in trunk
[Fetch API] Add support for Request keepalive getter
https://bugs.webkit.org/show_bug.cgi?id=175151
Patch by Youenn Fablet <youenn@apple.com> on 2017-08-03
Reviewed by Chris Dumez.
LayoutTests/imported/w3c:
- web-platform-tests/fetch/api/request/request-idl-expected.txt:
- web-platform-tests/fetch/api/request/request-idl.html:
- web-platform-tests/fetch/api/request/request-keepalive-expected.txt: Added.
- web-platform-tests/fetch/api/request/request-keepalive.html: Added.
Source/WebCore:
Test: imported/w3c/web-platform-tests/fetch/api/request/request-keepalive.html
Adding keepalive as a fetch option.
Adding initialization and getter of keepalive into FetchRequest.
- Modules/fetch/FetchRequest.cpp:
(WebCore::buildOptions):
- Modules/fetch/FetchRequest.h:
- Modules/fetch/FetchRequest.idl:
- loader/FetchOptions.h:
- 5:20 PM Changeset in webkit [220243] by
-
- 69 edits6 copies2 adds1 delete in trunk
[PAL] Move spi/cf directory into PAL
https://bugs.webkit.org/show_bug.cgi?id=175057
Patch by Yoshiaki Jitsukawa <jitsu@rd.scei.sony.co.jp> on 2017-08-03
Reviewed by Antti Koivisto.
Source/WebCore:
- WebCore.xcodeproj/project.pbxproj:
- loader/cocoa/DiskCacheMonitorCocoa.mm:
- loader/cocoa/SubresourceLoaderCocoa.mm:
- loader/mac/ResourceLoaderMac.mm:
- platform/cf/CoreMediaSoftLink.cpp:
- platform/cf/CoreMediaSoftLink.h:
- platform/mac/PluginBlacklist.mm:
- platform/mac/WebCoreNSStringExtras.mm:
- platform/mac/WebGLBlacklist.mm:
- platform/mediastream/mac/CoreAudioCaptureSource.cpp:
- platform/network/NetworkStorageSession.h:
- platform/network/cf/CookieJarCFNet.cpp:
- platform/network/cf/CredentialStorageCFNet.cpp:
- platform/network/cf/ResourceHandleCFNet.cpp:
- platform/network/cf/ResourceHandleCFURLConnectionDelegate.cpp:
- platform/network/cf/ResourceHandleCFURLConnectionDelegateWithOperationQueue.cpp:
- platform/network/cf/ResourceRequestCFNet.cpp:
- platform/network/cf/ResourceRequestCFNet.h:
- platform/network/cf/ResourceResponse.h:
- platform/network/cf/ResourceResponseCFNet.cpp:
- platform/network/cf/SocketStreamHandleImplCFNet.cpp:
- platform/network/cf/SynchronousResourceHandleCFURLConnectionDelegate.cpp:
- platform/network/cocoa/CookieStorageObserver.h:
- platform/network/cocoa/CredentialCocoa.h:
- platform/network/cocoa/NetworkStorageSessionCocoa.mm:
- platform/network/cocoa/ResourceRequestCocoa.mm:
- platform/network/cocoa/ResourceResponseCocoa.mm:
- platform/network/ios/ResourceRequestIOS.mm:
- platform/network/mac/CookieJarMac.mm:
- platform/network/mac/FormDataStreamMac.mm:
- platform/network/mac/ResourceHandleMac.mm:
- platform/network/mac/ResourceRequestMac.mm:
- platform/network/mac/WebCoreResourceHandleAsDelegate.mm:
- platform/network/mac/WebCoreResourceHandleAsOperationQueueDelegate.mm:
- platform/network/mac/WebCoreURLResponse.h:
- testing/cocoa/WebArchiveDumpSupport.mm:
Source/WebCore/PAL:
- PAL.xcodeproj/project.pbxproj:
- pal/spi/cf/CFLocaleSPI.h: Renamed from Source/WebCore/platform/spi/cf/CFLocaleSPI.h.
- pal/spi/cf/CFNetworkConnectionCacheSPI.h: Renamed from Source/WebCore/platform/spi/cf/CFNetworkConnectionCacheSPI.h.
- pal/spi/cf/CFNetworkSPI.h: Renamed from Source/WebCore/platform/spi/cf/CFNetworkSPI.h.
- pal/spi/cf/CFUtilitiesSPI.h: Renamed from Source/WebCore/platform/spi/cf/CFUtilitiesSPI.h.
- pal/spi/cf/CoreAudioSPI.h: Renamed from Source/WebCore/platform/spi/cf/CoreAudioSPI.h.
- pal/spi/cf/CoreMediaSPI.h: Renamed from Source/WebCore/platform/spi/cf/CoreMediaSPI.h.
Source/WebKit:
- NetworkProcess/cocoa/NetworkDataTaskCocoa.mm:
- NetworkProcess/cocoa/NetworkProcessCocoa.mm:
- NetworkProcess/cocoa/NetworkSessionCocoa.mm:
- NetworkProcess/ios/NetworkProcessIOS.mm:
- NetworkProcess/mac/NetworkLoadMac.mm:
- NetworkProcess/mac/NetworkProcessMac.mm:
- Shared/cf/CookieStorageUtilsCF.h:
- Shared/mac/ChildProcessMac.mm:
- Shared/mac/CookieStorageShim.mm:
- UIProcess/API/Cocoa/WKHTTPCookieStore.mm:
- UIProcess/API/Cocoa/WKProcessPool.mm:
- UIProcess/Cocoa/WebProcessPoolCocoa.mm:
- UIProcess/Plugins/mac/PluginProcessProxyMac.mm:
- UIProcess/WebsiteData/Cocoa/WebsiteDataStoreCocoa.mm:
- UIProcess/WebsiteData/WebsiteDataRecord.cpp:
- UIProcess/WebsiteData/WebsiteDataStore.h:
- UIProcess/mac/WebCookieManagerProxyMac.mm:
- WebProcess/WebCoreSupport/mac/WebFrameNetworkingContext.mm:
- WebProcess/cocoa/WebProcessCocoa.mm:
Source/WebKitLegacy/mac:
- Plugins/Hosted/HostedNetscapePluginStream.mm:
- Plugins/WebNetscapePluginStream.mm:
- WebCoreSupport/WebFrameNetworkingContext.mm:
- WebView/WebPreferences.mm:
- WebView/WebView.mm:
Tools:
- Scripts/webkitpy/style/checker.py:
- Scripts/webkitpy/style/checker_unittest.py:
(GlobalVariablesTest.test_path_rules_specifier):
Ignore "readability/naming/underscores" style errors for sources
under the WebCore/PAL/pal/spi directory.
- TestWebKitAPI/Tests/WebKit2Cocoa/CookieAcceptPolicy.mm:
- 5:14 PM Changeset in webkit [220242] by
-
- 7 edits in branches/safari-604-branch/Source
Versioning.
- 4:54 PM Changeset in webkit [220241] by
-
- 47 edits in trunk
Remove FETCH_API compilation guard
https://bugs.webkit.org/show_bug.cgi?id=175154
Patch by Youenn Fablet <youenn@apple.com> on 2017-08-03
Reviewed by Chris Dumez.
Source/JavaScriptCore:
- Configurations/FeatureDefines.xcconfig:
Source/WebCore:
No change of behavior.
- Configurations/FeatureDefines.xcconfig:
- Modules/fetch/DOMWindowFetch.cpp:
- Modules/fetch/DOMWindowFetch.h:
- Modules/fetch/DOMWindowFetch.idl:
- Modules/fetch/FetchBody.cpp:
- Modules/fetch/FetchBody.h:
- Modules/fetch/FetchBody.idl:
- Modules/fetch/FetchBodyConsumer.cpp:
- Modules/fetch/FetchBodyConsumer.h:
- Modules/fetch/FetchBodyOwner.cpp:
- Modules/fetch/FetchBodyOwner.h:
- Modules/fetch/FetchHeaders.cpp:
- Modules/fetch/FetchHeaders.h:
- Modules/fetch/FetchHeaders.idl:
- Modules/fetch/FetchInternals.js:
- Modules/fetch/FetchLoader.cpp:
- Modules/fetch/FetchLoader.h:
- Modules/fetch/FetchLoaderClient.h:
- Modules/fetch/FetchRequest.cpp:
- Modules/fetch/FetchRequest.h:
- Modules/fetch/FetchRequest.idl:
- Modules/fetch/FetchResponse.cpp:
- Modules/fetch/FetchResponse.h:
- Modules/fetch/FetchResponse.idl:
- Modules/fetch/FetchResponse.js:
- Modules/fetch/FetchResponseSource.cpp:
- Modules/fetch/FetchResponseSource.h:
- Modules/fetch/WorkerGlobalScopeFetch.cpp:
- Modules/fetch/WorkerGlobalScopeFetch.h:
- Modules/fetch/WorkerGlobalScopeFetch.idl:
- page/RuntimeEnabledFeatures.h:
(WebCore::RuntimeEnabledFeatures::fetchAPIEnabled const):
Source/WebCore/PAL:
- Configurations/FeatureDefines.xcconfig:
Source/WebKit:
- Configurations/FeatureDefines.xcconfig:
- WebProcess/WebPage/WebPage.cpp:
(WebKit::WebPage::updatePreferences):
Source/WebKitLegacy/mac:
- Configurations/FeatureDefines.xcconfig:
- WebView/WebPreferences.mm:
(+[WebPreferences initialize]):
- WebView/WebView.mm:
(-[WebView _preferencesChanged:]):
Source/WTF:
- wtf/FeatureDefines.h:
Tools:
- TestWebKitAPI/Configurations/FeatureDefines.xcconfig:
- 4:38 PM Changeset in webkit [220240] by
-
- 1 copy in tags/Safari-604.2.1
Tag Safari-604.2.1.
- 4:27 PM Changeset in webkit [220239] by
-
- 1 copy in tags/Safari-604.1.37
Tag Safari-604.1.37.
- 4:17 PM Changeset in webkit [220238] by
-
- 2 edits in trunk/Tools
- 4:11 PM Changeset in webkit [220237] by
-
- 7 edits in branches/safari-604.1-branch/Source
Versioning.
- 4:11 PM Changeset in webkit [220236] by
-
- 7 edits in branches/safari-604.1-branch/Source
Versioning.
- 4:10 PM Changeset in webkit [220235] by
-
- 10 edits in trunk/Source
Web Inspector: add button to open Inspector2
https://bugs.webkit.org/show_bug.cgi?id=175108
Reviewed by Brian Burg.
Source/WebCore:
This patch just exposes a function to the inspector page. No new functionality was added.
- inspector/InspectorFrontendHost.idl:
- inspector/InspectorFrontendHost.h:
- inspector/InspectorFrontendHost.cpp:
(WebCore::InspectorFrontendHost::inspectInspector):
Source/WebInspectorUI:
- UserInterface/Debug/Bootstrap.js:
(updateDebugUI):
(WI.runBootstrapOperations):
- UserInterface/Views/ButtonToolbarItem.js:
(WI.ButtonToolbarItem):
(WI.ButtonToolbarItem.prototype.get label): Deleted.
(WI.ButtonToolbarItem.prototype.set label): Deleted.
- UserInterface/Views/ButtonToolbarItem.css:
(.toolbar .item.button):
(.toolbar .item.button:not(.disabled):active):
(.toolbar .item.button:not(.disabled):matches(:focus, .activate.activated)):
(.toolbar .item.button:not(.disabled):active:matches(:focus, .activate.activated)):
(.toolbar .item.button > .glyph):
(.toolbar .item.button:not(.disabled):active > .glyph): Deleted.
(.toolbar .item.button:not(.disabled):matches(:focus, .activate.activated) > .glyph): Deleted.
(.toolbar .item.button:not(.disabled):active:matches(:focus, .activate.activated) > .glyph): Deleted.
(.toolbar .item.button > .label): Deleted.
- UserInterface/Views/ActivateButtonToolbarItem.js:
(WI.ActivateButtonToolbarItem):
(WI.ActivateButtonToolbarItem.prototype.get label): Deleted.
(WI.ActivateButtonToolbarItem.prototype.set label): Deleted.
- UserInterface/Base/Main.js:
(WI.contentLoaded):
Remove unusedlabelparameter from Toolbar objects.
- 4:09 PM Changeset in webkit [220234] by
-
- 2 edits in branches/safari-604.1-branch/Source/WebKitLegacy
Cherry-pick r220205. rdar://problem/33711361
- 3:31 PM Changeset in webkit [220233] by
-
- 20 edits1 copy7 adds in trunk
Web Inspector: Instrument WebGLProgram created/deleted
https://bugs.webkit.org/show_bug.cgi?id=175059
Reviewed by Devin Rousso.
Source/JavaScriptCore:
Extend the Canvas protocol with types/events for tracking WebGLPrograms.
- inspector/protocol/Canvas.json:
Source/WebCore:
Tests: inspector/canvas/shaderProgram-add-remove-webgl.html
inspector/canvas/shaderProgram-add-remove-webgl2.html
This patch adds instrumentation to WebGLRenderingContextBase for tracking
WebGLPrograms. A new helper class, InspectorShaderProgram, is used by
the CanvasAgent to hold related data.
- CMakeLists.txt:
- WebCore.xcodeproj/project.pbxproj:
- html/canvas/WebGLRenderingContextBase.cpp:
(WebCore::WebGLRenderingContextBase::createProgram):
(WebCore::WebGLRenderingContextBase::deleteProgram):
- inspector/InspectorCanvasAgent.cpp:
(WebCore::InspectorCanvasAgent::enable):
(WebCore::InspectorCanvasAgent::frameNavigated):
(WebCore::InspectorCanvasAgent::didCreateProgram):
(WebCore::InspectorCanvasAgent::willDeleteProgram):
(WebCore::InspectorCanvasAgent::clearCanvasData):
(WebCore::InspectorCanvasAgent::unbindCanvas):
(WebCore::InspectorCanvasAgent::unbindProgram):
(WebCore::InspectorCanvasAgent::assertInspectorProgram):
(WebCore::InspectorCanvasAgent::findInspectorProgram):
- inspector/InspectorCanvasAgent.h:
- inspector/InspectorInstrumentation.cpp:
(WebCore::InspectorInstrumentation::didCreateCSSCanvasImpl):
(WebCore::InspectorInstrumentation::didChangeCSSCanvasClientNodesImpl):
(WebCore::InspectorInstrumentation::didCreateCanvasRenderingContextImpl):
(WebCore::InspectorInstrumentation::didChangeCanvasMemoryImpl):
(WebCore::InspectorInstrumentation::recordCanvasActionImpl):
(WebCore::InspectorInstrumentation::didFinishRecordingCanvasFrameImpl):
(WebCore::InspectorInstrumentation::didCreateProgramImpl):
(WebCore::InspectorInstrumentation::willDeleteProgramImpl):
- inspector/InspectorInstrumentation.h:
(WebCore::InspectorInstrumentation::recordCanvasActionImpl):
(WebCore::InspectorInstrumentation::didCreateCSSCanvas):
(WebCore::InspectorInstrumentation::didChangeCSSCanvasClientNodes):
(WebCore::InspectorInstrumentation::didCreateCanvasRenderingContext):
(WebCore::InspectorInstrumentation::didChangeCanvasMemory):
(WebCore::InspectorInstrumentation::recordCanvasAction):
(WebCore::InspectorInstrumentation::didFinishRecordingCanvasFrame):
(WebCore::InspectorInstrumentation::didCreateProgram):
(WebCore::InspectorInstrumentation::willDeleteProgram):
- inspector/InspectorShaderProgram.cpp: Added.
(WebCore::InspectorShaderProgram::create):
(WebCore::InspectorShaderProgram::InspectorShaderProgram):
(WebCore::InspectorShaderProgram::context const):
- inspector/InspectorShaderProgram.h: Added.
Source/WebInspectorUI:
This patch adds frontend support for shader program instrumentation.
The frontend creates a ShaderProgram model object for each WebGLProgram.
Since only canvases with a WebGL context have programs, the Canvas model
object does not contain any logic specific to programs. CanvasManager
dispatches program added/removed events, and the parent Canvas can be
accessed from ShaderProgram but not the other way around.
- UserInterface/Controllers/CanvasManager.js:
(WI.CanvasManager):
(WI.CanvasManager.prototype.get shaderPrograms):
(WI.CanvasManager.prototype.canvasRemoved):
(WI.CanvasManager.prototype.programCreated):
(WI.CanvasManager.prototype.programDeleted):
(WI.CanvasManager.prototype._mainResourceDidChange):
(WI.CanvasManager.prototype._dispatchShaderProgramRemoved):
- UserInterface/Main.html:
- UserInterface/Models/Canvas.js:
(WI.Canvas.prototype.nextShaderProgramDisplayNumber):
(WI.Canvas):
- UserInterface/Models/ShaderProgram.js: Added.
(WI.ShaderProgram):
(WI.ShaderProgram.prototype.get identifier):
(WI.ShaderProgram.prototype.get canvas):
(WI.ShaderProgram.prototype.get displayName):
- UserInterface/Protocol/CanvasObserver.js:
(WI.CanvasObserver.prototype.programCreated):
(WI.CanvasObserver.prototype.programDeleted):
(WI.CanvasObserver):
- UserInterface/Test.html:
LayoutTests:
Add tests for CanvasManager shader program events and ShaderProgram model object.
WebGL and WebGL2 contexts are tested separately based on platform support.
- inspector/canvas/resources/shaderProgram-utilities.js: Added.
(createProgram):
(deleteProgram):
(deleteContext):
(TestPage.registerInitializer.awaitProgramAdded):
(TestPage.registerInitializer):
(TestPage.registerInitializer.window.initializeTestSuite):
(TestPage.registerInitializer.window.addSimpleTestCase):
(TestPage.registerInitializer.window.addParentCanvasRemovedTestCase):
- inspector/canvas/shaderProgram-add-remove-webgl-expected.txt: Added.
- inspector/canvas/shaderProgram-add-remove-webgl.html: Added.
- inspector/canvas/shaderProgram-add-remove-webgl2-expected.txt: Added.
- inspector/canvas/shaderProgram-add-remove-webgl2.html: Added.
- platform/gtk/TestExpectations:
- platform/mac/TestExpectations:
- platform/win/TestExpectations:
- 3:18 PM Changeset in webkit [220232] by
-
- 2 edits in trunk/Source/WebInspectorUI
JSContext Inspector: Recording tab should not be available in New Tab picker
https://bugs.webkit.org/show_bug.cgi?id=175155
Patch by Joseph Pecoraro <Joseph Pecoraro> on 2017-08-03
Reviewed by Brian Burg.
- UserInterface/Views/RecordingTabContentView.js:
(WI.RecordingTabContentView.isTabAllowed):
Only allow the Recording Tab if we have a CanvasAgent.
- 1:28 PM Changeset in webkit [220231] by
-
- 1 edit in branches/safari-604-branch/Source/JavaScriptCore/ChangeLog
Cherry-pick r220144. rdar://problem/33692161
- 1:27 PM Changeset in webkit [220230] by
-
- 5 edits in branches/safari-604-branch/Source/WebKit
Cherry-pick r220138. rdar://problem/33692550
- 1:27 PM Changeset in webkit [220229] by
-
- 2 edits in branches/safari-604-branch
Cherry-pick r220112. rdar://problem/33692164
- 1:27 PM Changeset in webkit [220228] by
-
- 1 edit in branches/safari-604-branch/Source/WebCore/ChangeLog
Cherry-pick r220085. rdar://problem/33692157
- 1:27 PM Changeset in webkit [220227] by
-
- 1 edit in branches/safari-604-branch/Source/WebCore/ChangeLog
Cherry-pick r220084. rdar://problem/33692167
- 1:27 PM Changeset in webkit [220226] by
-
- 6 edits in trunk/Source
Unreviewed, rolling out r220209.
This caused internal build failures.
Reverted changeset:
"Use MPAVRoutingController instead of deprecated versions."
https://bugs.webkit.org/show_bug.cgi?id=175063
http://trac.webkit.org/changeset/220209
- 1:27 PM Changeset in webkit [220225] by
-
- 1 edit in branches/safari-604-branch/Source/WebCore/ChangeLog
Cherry-pick r220077. rdar://problem/33692157
- 1:27 PM Changeset in webkit [220224] by
-
- 2 edits in branches/safari-604-branch
Cherry-pick r220035. rdar://problem/33692157
- 1:21 PM Changeset in webkit [220223] by
-
- 164 edits13 copies1 move302 adds1 delete in trunk/LayoutTests
Import WPT service worker tests
https://bugs.webkit.org/show_bug.cgi?id=175053
Patch by Youenn Fablet <youenn@apple.com> on 2017-08-03
Reviewed by Brady Eidson.
LayoutTests/imported/w3c:
Importing service-worker tests up to cfdfb48329b20e19b6492a317ac5181a99506fd2.
- resources/resource-files.json:
- resources/import-expectations.json:
- web-platform-tests/service-workers/:
LayoutTests:
- TestExpectations: skipping service worker tests.
- 1:19 PM Changeset in webkit [220222] by
-
- 2 edits in trunk/Source/WebKit
Web Automation: consider file extensions in the "accept" attribute when deciding if a file can be uploaded
https://bugs.webkit.org/show_bug.cgi?id=175081
Reviewed by Joseph Pecoraro.
- UIProcess/Automation/WebAutomationSession.cpp:
(WebKit::fileCanBeAcceptedForUpload):
(WebKit::WebAutomationSession::handleRunOpenPanel):
In cases where a file has an extension, try to match it against the
accepted extensions list. Give up if there's no extension. Otherwise,
proceed as normal to infer a MIME type based on the extension.
In cases where extension-less files are inside hidden folders, the MIME
inference will fail as well.
- 12:37 PM Changeset in webkit [220221] by
-
- 2 edits in trunk/Tools
[GTK][WKE] Pass the --memory-limited option on the GTK and WPE buildbots for the JSC tests.
https://bugs.webkit.org/show_bug.cgi?id=175140
Reviewed by Alexey Proskuryakov.
We are having lately issues with JSC tests causing problems on
the GTK+ and WPE bots due to the high amount of memory some tests
need to run.
The best thing we can do now is to workaround this by disabling
all the tests marked as memoryLimited on the GTK and WPE bots.
We may revise this on the future.
- BuildSlaveSupport/build.webkit.org-config/master.cfg:
(RunJavaScriptCoreTests.start):
- 12:21 PM Changeset in webkit [220220] by
-
- 36 edits14 adds in trunk
Add SW IDLs and stub out basic functionality.
https://bugs.webkit.org/show_bug.cgi?id=175115
Reviewed by Chris Dumez.
.:
- Source/cmake/WebKitFeatures.cmake:
- Source/cmake/tools/vsprops/FeatureDefines.props:
- Source/cmake/tools/vsprops/FeatureDefinesCairo.props:
Source/JavaScriptCore:
- Configurations/FeatureDefines.xcconfig:
- runtime/CommonIdentifiers.h:
Source/WebCore:
No new tests (Currently no behavior change).
Overall note: This feature is EnabledAtRuntime as opposed to EnabledBySetting because
the Settings-based code generation is completely broken for non-Document contexts,
whereas the RuntimeEnabledFeatures-based generation is not.
- CMakeLists.txt:
- Configurations/FeatureDefines.xcconfig:
- DerivedSources.make:
- WebCore.xcodeproj/project.pbxproj:
- bindings/scripts/preprocess-idls.pl: Handle the new global scope c'tor file.
- bindings/js/JSServiceWorkerContainerCustom.cpp: Added.
(WebCore::JSServiceWorkerContainer::ready const):
- bindings/js/JSWorkerGlobalScopeBase.cpp:
(WebCore::toJSWorkerGlobalScope): Refactor to handle both types of derived workers.
(WebCore::toJSServiceWorkerGlobalScope):
- bindings/js/JSWorkerGlobalScopeBase.h:
- dom/EventNames.h:
- dom/EventTargetFactory.in:
- features.json: Change status of feature.
- page/Navigator.idl:
- page/NavigatorBase.cpp:
(WebCore::NavigatorBase::serviceWorker):
- page/NavigatorBase.h:
- page/NavigatorServiceWorker.idl: Added.
- page/RuntimeEnabledFeatures.h:
(WebCore::RuntimeEnabledFeatures::serviceWorkerEnabled const):
(WebCore::RuntimeEnabledFeatures::setServiceWorkerEnabled):
- workers/ServiceWorker.cpp: Added.
(WebCore::ServiceWorker::postMessage):
(WebCore::ServiceWorker::~ServiceWorker):
(WebCore::ServiceWorker::scriptURL const):
(WebCore::ServiceWorker::state const):
(WebCore::ServiceWorker::eventTargetInterface const):
(WebCore::ServiceWorker::scriptExecutionContext const):
- workers/ServiceWorker.h: Added.
- workers/ServiceWorker.idl: Added.
- workers/ServiceWorkerContainer.cpp: Added.
(WebCore::ServiceWorkerContainer::~ServiceWorkerContainer):
(WebCore::ServiceWorkerContainer::controller const):
(WebCore::ServiceWorkerContainer::ready):
(WebCore::ServiceWorkerContainer::addRegistration):
(WebCore::ServiceWorkerContainer::getRegistration):
(WebCore::ServiceWorkerContainer::getRegistrations):
(WebCore::ServiceWorkerContainer::startMessages):
(WebCore::ServiceWorkerContainer::eventTargetInterface const):
(WebCore::ServiceWorkerContainer::scriptExecutionContext const):
- workers/ServiceWorkerContainer.h: Added.
- workers/ServiceWorkerContainer.idl: Added.
- workers/ServiceWorkerGlobalScope.cpp: Added.
(WebCore::ServiceWorkerGlobalScope::registration):
(WebCore::ServiceWorkerGlobalScope::skipWaiting):
- workers/ServiceWorkerGlobalScope.h: Added.
- workers/ServiceWorkerGlobalScope.idl: Added.
- workers/ServiceWorkerRegistration.cpp: Added.
(WebCore::ServiceWorkerRegistration::~ServiceWorkerRegistration):
(WebCore::ServiceWorkerRegistration::installing):
(WebCore::ServiceWorkerRegistration::waiting):
(WebCore::ServiceWorkerRegistration::active):
(WebCore::ServiceWorkerRegistration::scope const):
(WebCore::ServiceWorkerRegistration::update):
(WebCore::ServiceWorkerRegistration::unregister):
(WebCore::ServiceWorkerRegistration::eventTargetInterface const):
(WebCore::ServiceWorkerRegistration::scriptExecutionContext const):
- workers/ServiceWorkerRegistration.h: Added.
- workers/ServiceWorkerRegistration.idl: Added.
Source/WebCore/PAL:
- Configurations/FeatureDefines.xcconfig:
Source/WebKit:
- Configurations/FeatureDefines.xcconfig:
- Shared/WebPreferencesDefinitions.h:
- UIProcess/WebPreferences.cpp:
(WebKit::WebPreferences::enableAllExperimentalFeatures): Explicitly skip SW for now.
The ramifications to layouttests are complicated, and we'd like to follow up in a
separate patch.
- WebProcess/WebPage/WebPage.cpp:
(WebKit::WebPage::updatePreferences):
Source/WebKitLegacy/mac:
- Configurations/FeatureDefines.xcconfig:
Source/WTF:
- wtf/FeatureDefines.h:
Tools:
- TestWebKitAPI/Configurations/FeatureDefines.xcconfig:
- Scripts/webkitpy/bindings/main.py:
- 12:17 PM Changeset in webkit [220219] by
-
- 11 edits in trunk/Source/JavaScriptCore
Rename ScratchBuffer::activeLengthPtr to addressOfActiveLength.
https://bugs.webkit.org/show_bug.cgi?id=175142
<rdar://problem/33704528>
Reviewed by Filip Pizlo.
The convention in the rest of of JSC for such methods which return the address of
a field is to name them "addressOf<field name>". We'll rename
ScratchBuffer::activeLengthPtr to be consistent with this convention.
- dfg/DFGSpeculativeJIT.cpp:
(JSC::DFG::SpeculativeJIT::compileNewArrayWithSpread):
- dfg/DFGSpeculativeJIT32_64.cpp:
(JSC::DFG::SpeculativeJIT::compile):
- dfg/DFGSpeculativeJIT64.cpp:
(JSC::DFG::SpeculativeJIT::compile):
- dfg/DFGThunks.cpp:
(JSC::DFG::osrExitGenerationThunkGenerator):
- ftl/FTLLowerDFGToB3.cpp:
(JSC::FTL::DFG::LowerDFGToB3::compileNewArray):
(JSC::FTL::DFG::LowerDFGToB3::compileNewArrayWithSpread):
- ftl/FTLThunks.cpp:
(JSC::FTL::genericGenerationThunkGenerator):
- jit/AssemblyHelpers.cpp:
(JSC::AssemblyHelpers::debugCall):
- jit/ScratchRegisterAllocator.cpp:
(JSC::ScratchRegisterAllocator::preserveUsedRegistersToScratchBufferForCall):
(JSC::ScratchRegisterAllocator::restoreUsedRegistersFromScratchBufferForCall):
- runtime/VM.h:
(JSC::ScratchBuffer::addressOfActiveLength):
(JSC::ScratchBuffer::activeLengthPtr): Deleted.
- wasm/WasmBinding.cpp:
(JSC::Wasm::wasmToJs):
- 12:13 PM Changeset in webkit [220218] by
-
- 4 edits1 add in trunk/LayoutTests
Test gardening.
https://bugs.webkit.org/show_bug.cgi?id=175137
Unreviewed test gardening.
Patch by Ms2ger <Ms2ger@igalia.com> on 2017-08-03
- platform/gtk/TestExpectations:
- platform/gtk/fast/images/async-image-multiple-clients-repaint-expected.txt: Added.
- platform/gtk/fast/text/atsui-pointtooffset-calls-cg-expected.txt:
- platform/win/TestExpectations:
- 11:59 AM Changeset in webkit [220217] by
-
- 2 edits in trunk/Source/WTF
Unreviewed, build fix for Windows port
https://bugs.webkit.org/show_bug.cgi?id=175013
- wtf/Threading.h:
- 11:58 AM Changeset in webkit [220216] by
-
- 7 edits in branches/safari-604-branch/Source
Versioning.
- 11:55 AM Changeset in webkit [220215] by
-
- 7 edits in branches/safari-604.1-branch/Source
Versioning.
- 11:29 AM Changeset in webkit [220214] by
-
- 7 edits4 deletes in trunk/Source/WTF
Merge ThreadHolder to WTF::Thread itself
https://bugs.webkit.org/show_bug.cgi?id=175013
Reviewed by Mark Lam.
Currently, we store ThreadHolder* to the TLS, and ThreadHolder* holds Ref<Thread>.
When we get Thread& from the current thread TLS, we need to dereference the ThreadHolder*.
However, ideally, we can store Thread* directly to the current thread TLS.
While the ThreadHolder design is beautiful, it's worth optimizing by storing Thread* directly
since Thread::current() is so frequently executed.
This patch merges ThreadHolder to Thread. And we now store Thread* directly in the TLS.
When storing it to TLS, we call leakRef() to keep Thread ref count incremented by the TLS.
And when destroying the TLS, we callderef()to ensure that Thread* is dereferenced from
the TLS.
- WTF.xcodeproj/project.pbxproj:
- wtf/CMakeLists.txt:
- wtf/ThreadHolder.cpp: Removed.
- wtf/ThreadHolder.h: Removed.
- wtf/ThreadHolderPthreads.cpp: Removed.
- wtf/ThreadHolderWin.cpp: Removed.
- wtf/Threading.cpp:
(WTF::Thread::entryPoint):
(WTF::initializeThreading):
- wtf/Threading.h:
(WTF::Thread::currentMayBeNull):
(WTF::Thread::current):
- wtf/ThreadingPthreads.cpp:
(WTF::Thread::waitForCompletion):
(WTF::Thread::initializeCurrentTLS):
(WTF::Thread::initializeTLSKey):
(WTF::Thread::initializeTLS):
(WTF::Thread::destructTLS):
(WTF::Thread::createCurrentThread): Deleted.
- wtf/ThreadingWin.cpp:
(WTF::Thread::initializeCurrentTLS):
(WTF::threadMapMutex):
(WTF::Thread::initializeTLSKey):
(WTF::Thread::currentDying):
(WTF::Thread::get):
(WTF::Thread::initializeTLS):
(WTF::Thread::destructTLS):
(WTF::waitForThreadCompletion):
(WTF::Thread::createCurrentThread): Deleted.
- 11:06 AM Changeset in webkit [220213] by
-
- 1 copy in branches/safari-604.1-branch
New Tag.
- 11:03 AM Changeset in webkit [220212] by
-
- 2 edits in trunk/Source/WebKit
[WebKit] Sort Xcode project file
https://bugs.webkit.org/show_bug.cgi?id=175122
Patch by Yoshiaki Jitsukawa <jitsu@rd.scei.sony.co.jp> on 2017-08-03
Reviewed by Antti Koivisto.
- WebKit.xcodeproj/project.pbxproj:
- 11:03 AM Changeset in webkit [220211] by
-
- 4 edits in trunk/Source/WebCore
[WebCore] Sort Xcode project files
https://bugs.webkit.org/show_bug.cgi?id=175121
Patch by Yoshiaki Jitsukawa <jitsu@rd.scei.sony.co.jp> on 2017-08-03
Reviewed by Antti Koivisto.
Source/WebCore:
- WebCore.xcodeproj/project.pbxproj:
Source/WebCore/PAL:
- PAL.xcodeproj/project.pbxproj:
- 10:51 AM Changeset in webkit [220210] by
-
- 59 edits4 adds2 deletes in trunk
Source/WebCore:
[WebIDL] Convert MutationCallback to be a normal generated callback
https://bugs.webkit.org/show_bug.cgi?id=174140
Reviewed by Darin Adler.
To make this work more nicely, I:
- Added the ability to for non-nullable interfaces in sequences to be passed via a Ref<> rather than a RefPtr<> as a parameter to a callback function. (e.g. callback MyCallback = void (sequence<Foo> foos) will now have the signature, CallbackResult<void> handleEvent(const Vector<Ref<Foo>>&) rather than CallbackResult<void> handleEvent(const Vector<RefPtr<Foo>>&).
- Added a new extended attribute for callback functions called [CallbackThisObject=Type] which allows you to specify that the callback needs a this object in addition to its arguments. When specified, the first argument of the C++ implementation function will now correspond to the this object, with the remaining arguments shifted over one.
- Converted callback objects to all inherit directly from ActiveDOMCallback rather than having the generated JS callback derived class inherit from it. This allows us to have access to a callback's canInvokeCallback() function anywhere (needed for MutationCallback) as well as giving a place to put an optional virtual visitJSFunction to allow marking weak callbacks (while not an ideal layering, this matches what we do in EventListener). This change requires each callback to have a bit more code to import the ActiveDOMCallback's constructor and requires non-JS derived callbacks to pass a ScriptExecutionContext (e.g. the Document).
- CMakeLists.txt:
- DerivedSources.make:
- WebCore.xcodeproj/project.pbxproj:
- bindings/js/JSMutationCallback.cpp: Removed.
- bindings/js/JSMutationCallback.h: Removed.
Remove custom JSMutationCallback.h/cpp
- Modules/geolocation/PositionCallback.h:
- Modules/geolocation/PositionErrorCallback.h:
- Modules/notifications/NotificationPermissionCallback.h:
- Modules/webaudio/AudioBufferCallback.h:
- Modules/webdatabase/DatabaseCallback.h:
- Modules/webdatabase/SQLStatementCallback.h:
- Modules/webdatabase/SQLStatementErrorCallback.h:
- Modules/webdatabase/SQLTransactionCallback.h:
- Modules/webdatabase/SQLTransactionErrorCallback.h:
- css/MediaQueryListListener.h:
- dom/NodeFilter.h:
- dom/RequestAnimationFrameCallback.h:
- dom/StringCallback.h:
- fileapi/BlobCallback.h:
- html/VoidCallback.h:
- page/IntersectionObserverCallback.h:
- page/PerformanceObserverCallback.h:
Add ActiveDOMCallback as a base class. Import the ActiveDOMCallback constructor.
- Modules/mediastream/MediaDevicesRequest.cpp:
(WebCore::MediaDevicesRequest::filterDeviceList):
(WebCore::MediaDevicesRequest::start):
- Modules/mediastream/MediaDevicesRequest.h:
Change filterDeviceList to take a Vector of Refs.
- bindings/IDLTypes.h:
Add InnerParameterType and NullableInnerParameterType type hooks
and specialize wrappers to use Ref for InnerParameterType, and RefPtr
for NullableInnerParameterType.
- bindings/js/JSCallbackData.cpp:
- bindings/js/JSCallbackData.h:
Add support for passing a this object and give JSCallbackDataWeak a visitJSFunction
to allow marking the underlying function.
- bindings/js/JSMutationObserverCustom.cpp:
(WebCore::JSMutationObserver::visitAdditionalChildren):
(WebCore::constructJSMutationObserver): Deleted.
Remove the custom constructor and replace it with a custom visitAdditionalChildren
that calls the new ActiveDOMObject's visitJSFunction.
- bindings/scripts/CodeGenerator.pm:
(ParseType):
Add helper to parse a type and cache the result.
- bindings/scripts/CodeGeneratorJS.pm:
(GenerateCallbackHeaderContent):
(GenerateCallbackImplementationContent):
(GetJSCallbackDataType): Deleted.
- Add support for [CallbackThisObject]. When [CallbackThisObject] is not specified, use jsUndefined() as the this object as specified by WebIDL.
- Stop inheriting from ActiveDOMCallback now that callbacks need to do this themselves.
- Add a visitJSFunction override for weak callback functions which calls into the callback data.
- bindings/scripts/IDLAttributes.json:
Add [CallbackThisObject].
- bindings/scripts/IDLParser.pm:
(ParseType):
Add entry point to parse a single type.
- css/FontFaceSet.h:
Use Ref rather than RefPtr for the faces sequence.
- dom/ActiveDOMCallback.h:
(WebCore::ActiveDOMCallback::visitJSFunction):
Add an optional visitJSFunction virtual function so that derived classes
have a way of marking underlying function objects.
- dom/MutationCallback.h:
Convert to support generation (return a CallbackResult, inherit from ActiveDOMObject).
- dom/MutationCallback.idl: Added.
Added to generate the callback. Uses the new [CallbackThisObject].
- dom/MutationObserver.cpp:
(WebCore::MutationObserver::deliver):
Switch to call idiomatic handleEvent, and pass *this as the first parameter
which will be translated into the this object.
- dom/MutationObserver.h:
(WebCore::MutationObserver::callback):
Expose the callback so it can marked during GC.
- dom/MutationObserver.idl:
Remove CustomConstructor and replace it with a custom mark function.
- dom/NativeNodeFilter.cpp:
- dom/NativeNodeFilter.h:
- inspector/InspectorDatabaseAgent.cpp:
Pass now needed ScriptExecutionContext to non-js based callbacks.
- bindings/scripts/test/JS/JSTestCallbackFunction.cpp:
- bindings/scripts/test/JS/JSTestCallbackFunctionRethrow.cpp:
- bindings/scripts/test/JS/JSTestCallbackFunctionWithThisObject.cpp: Added.
- bindings/scripts/test/JS/JSTestCallbackFunctionWithThisObject.h: Added.
- bindings/scripts/test/JS/JSTestCallbackFunctionWithTypedefs.cpp:
- bindings/scripts/test/JS/JSTestCallbackInterface.cpp:
- bindings/scripts/test/JS/JSTestCallbackInterface.h:
- bindings/scripts/test/JS/JSTestVoidCallbackFunction.cpp:
- bindings/scripts/test/TestCallbackFunctionWithThisObject.idl: Added.
- bindings/scripts/test/TestCallbackInterface.idl:
Add/update tests.
Source/WebKit:
[WebIDL] Convert MutationCallback to be a normal generated callback
https://bugs.webkit.org/show_bug.cgi?id=174140
Reviewed by Darin Adler.
- WebProcess/InjectedBundle/API/gtk/DOM/WebKitDOMDocument.cpp:
- WebProcess/InjectedBundle/API/gtk/DOM/WebKitDOMNodeFilter.cpp:
- WebProcess/InjectedBundle/API/gtk/DOM/WebKitDOMNodeFilterPrivate.h:
Pass, now necessary, Document to NativeNodeFilter constructor.
Source/WebKitLegacy/mac:
[WebIDL] Convert MutationCallback to be a normal generated callback
https://bugs.webkit.org/show_bug.cgi?id=174140
Reviewed by Darin Adler.
- DOM/DOMDocument.mm:
(-[DOMDocument createNodeIterator:whatToShow:filter:expandEntityReferences:]):
(-[DOMDocument createTreeWalker:whatToShow:filter:expandEntityReferences:]):
Pass, now necessary, Document to NativeNodeFilter constructor.
LayoutTests:
[WebIDL] Convert MutationCallback to be a normal generate callback
https://bugs.webkit.org/show_bug.cgi?id=174140
Reviewed by Darin Adler.
- fast/dom/MutationObserver/mutation-observer-constructor-expected.txt:
Update results for standard error messages.
- 10:37 AM Changeset in webkit [220209] by
-
- 6 edits in trunk/Source
Use MPAVRoutingController instead of deprecated versions.
https://bugs.webkit.org/show_bug.cgi?id=175063
Source/WebCore:
Patch by Jeremy Jones <jeremyj@apple.com> on 2017-08-03
Reviewed by Tim Horton.
No new tests because no behavior change. This uses a different platform class to present
an interface.
Remove deprecated MPAudioVideoRoutingPopoverController and MPAVRoutingSheet
Add MPMediaControlsViewController.
- platform/spi/ios/MediaPlayerSPI.h:
Source/WebKit:
rdar://problem/33301230
Patch by Jeremy Jones <jeremyj@apple.com> on 2017-08-03
Reviewed by Tim Horton.
Remove dependence on deprecated classes MPAVRoutingSheet and MPAudioVideoRoutingPopoverController.
- UIProcess/ios/forms/WKAirPlayRoutePicker.h:
- UIProcess/ios/forms/WKAirPlayRoutePicker.mm:
(-[WKAirPlayRoutePicker dealloc]):
(-[WKAirPlayRoutePicker show:fromRect:]):
(-[WKAirPlayRoutePicker popoverControllerDidDismissPopover:]): Deleted.
(-[WKAirPlayRoutePicker _presentAirPlayPopoverAnimated:fromRect:]): Deleted.
(-[WKAirPlayRoutePicker _windowWillRotate:]): Deleted.
(-[WKAirPlayRoutePicker _windowDidRotate:]): Deleted.
(-[WKAirPlayRoutePicker _dismissAirPlayRoutePickerIPad]): Deleted.
(-[WKAirPlayRoutePicker showAirPlayPickerIPad:fromRect:]): Deleted.
(-[WKAirPlayRoutePicker showAirPlayPickerIPhone:]): Deleted.
- 10:19 AM Changeset in webkit [220208] by
-
- 29 edits31 adds in trunk
Improve our support for referrer policies
https://bugs.webkit.org/show_bug.cgi?id=175069
<rdar://problem/33677313>
Reviewed by Darin Adler.
LayoutTests/imported/w3c:
Rebaseline several WPT tests now that more checks are passing.
- web-platform-tests/beacon/headers/header-referrer-origin-when-cross-origin-expected.txt:
- web-platform-tests/beacon/headers/header-referrer-same-origin-expected.txt:
- web-platform-tests/beacon/headers/header-referrer-strict-origin-when-cross-origin.https-expected.txt:
- web-platform-tests/beacon/headers/header-referrer-strict-origin.https-expected.txt:
- web-platform-tests/beacon/headers/header-referrer-unsafe-url.https-expected.txt:
- web-platform-tests/fetch/api/redirect/redirect-referrer-expected.txt:
- web-platform-tests/fetch/api/redirect/redirect-referrer-worker-expected.txt:
- web-platform-tests/fetch/api/request/request-init-001.sub-expected.txt:
Source/WebCore:
Improve our support for referrer policies. In particular, we now support the
additional following ones: "same-origin", "origin-when-cross-origin" and
"strict-origin-when-cross-origin".
This is as per the following specification:
Also refactor the code a bit for clarity: I merged the ReferrerPolicy enum and the
FetchOptions::ReferrerPolicy one.
Tests: http/tests/referrer-policy/origin-when-cross-origin/cross-origin-http-http.html
http/tests/referrer-policy/origin-when-cross-origin/cross-origin-http.https.html
http/tests/referrer-policy/origin-when-cross-origin/same-origin.html
http/tests/referrer-policy/same-origin/cross-origin-http-http.html
http/tests/referrer-policy/same-origin/cross-origin-http.https.html
http/tests/referrer-policy/same-origin/same-origin.html
http/tests/referrer-policy/strict-origin-when-cross-origin/cross-origin-http-http.html
http/tests/referrer-policy/strict-origin-when-cross-origin/cross-origin-http.https.html
http/tests/referrer-policy/strict-origin-when-cross-origin/same-origin.html
http/tests/referrer-policy/strict-origin/cross-origin-http-http.html
http/tests/referrer-policy/strict-origin/cross-origin-http.https.html
http/tests/referrer-policy/strict-origin/same-origin.html
- Modules/fetch/FetchLoader.cpp:
(WebCore::FetchLoader::start):
- Modules/fetch/FetchReferrerPolicy.h:
- Modules/fetch/FetchReferrerPolicy.idl:
- Modules/fetch/FetchRequest.h:
- Modules/fetch/FetchRequestInit.h:
- dom/Document.cpp:
(WebCore::Document::processReferrerPolicy):
(WebCore::Document::applyQuickLookSandbox):
(WebCore::Document::applyContentDispositionAttachmentSandbox):
- dom/Document.h:
- loader/FetchOptions.h:
- loader/FrameNetworkingContext.h:
- loader/PingLoader.cpp:
(WebCore::PingLoader::sendBeacon):
Drop explicit call to SecurityPolicy::shouldHideReferrer(). This is already called inside
SecurityPolicy::generateReferrerHeader() and used only when needed, depending on the
actual referrer policy.
- loader/cache/CachedResourceLoader.cpp:
(WebCore::CachedResourceLoader::updateHTTPRequestHeaders):
- loader/cache/CachedResourceRequest.cpp:
(WebCore::CachedResourceRequest::updateReferrerOriginAndUserAgentHeaders):
- page/SecurityPolicy.cpp:
(WebCore::referrerToOriginString):
(WebCore::SecurityPolicy::generateReferrerHeader):
- page/SecurityPolicy.h:
- platform/ReferrerPolicy.h:
Source/WebKit:
- WebProcess/Network/WebLoaderStrategy.cpp:
(WebKit::WebLoaderStrategy::loadResource):
(WebKit::WebLoaderStrategy::schedulePluginStreamLoad):
LayoutTests:
- http/tests/referrer-policy/origin-when-cross-origin/cross-origin-http-http-expected.txt: Added.
- http/tests/referrer-policy/origin-when-cross-origin/cross-origin-http-http.html: Added.
- http/tests/referrer-policy/origin-when-cross-origin/cross-origin-http.https-expected.txt: Added.
- http/tests/referrer-policy/origin-when-cross-origin/cross-origin-http.https.html: Added.
- http/tests/referrer-policy/origin-when-cross-origin/same-origin-expected.txt: Added.
- http/tests/referrer-policy/origin-when-cross-origin/same-origin.html: Added.
- http/tests/referrer-policy/resources/document.html: Added.
- http/tests/referrer-policy/same-origin/cross-origin-http-http-expected.txt: Added.
- http/tests/referrer-policy/same-origin/cross-origin-http-http.html: Added.
- http/tests/referrer-policy/same-origin/cross-origin-http.https-expected.txt: Added.
- http/tests/referrer-policy/same-origin/cross-origin-http.https.html: Added.
- http/tests/referrer-policy/same-origin/same-origin-expected.txt: Added.
- http/tests/referrer-policy/same-origin/same-origin.html: Added.
- http/tests/referrer-policy/strict-origin-when-cross-origin/cross-origin-http-http-expected.txt: Added.
- http/tests/referrer-policy/strict-origin-when-cross-origin/cross-origin-http-http.html: Added.
- http/tests/referrer-policy/strict-origin-when-cross-origin/cross-origin-http.https-expected.txt: Added.
- http/tests/referrer-policy/strict-origin-when-cross-origin/cross-origin-http.https.html: Added.
- http/tests/referrer-policy/strict-origin-when-cross-origin/same-origin-expected.txt: Added.
- http/tests/referrer-policy/strict-origin-when-cross-origin/same-origin.html: Added.
- http/tests/referrer-policy/strict-origin/cross-origin-http-http-expected.txt: Added.
- http/tests/referrer-policy/strict-origin/cross-origin-http-http.html: Added.
- http/tests/referrer-policy/strict-origin/cross-origin-http.https-expected.txt: Added.
- http/tests/referrer-policy/strict-origin/cross-origin-http.https.html: Added.
- http/tests/referrer-policy/strict-origin/same-origin-expected.txt: Added.
- http/tests/referrer-policy/strict-origin/same-origin.html: Added.
Add layout test coverage.
- http/tests/security/referrer-policy-invalid-expected.txt:
Rebaseline test now that console message has changed.
- 9:16 AM Changeset in webkit [220207] by
-
- 11 edits12 adds in trunk
Support ::marker pseudo-element
https://bugs.webkit.org/show_bug.cgi?id=141477
Reviewed by David Hyatt.
Source/WebCore:
Implements the ::marker pseudo element as per the CSS Pseudo-Element Module Level 4
spec., <https://drafts.csswg.org/css-pseudo-4> (Editor's Draft, 24 July 2017).
The ::marker pseudo element is a convenience pseudo element that allows a person to
style the appearance of a list item marker. For example, to render all list item
markers in bolded, blue text you would define a stylesheet with the following content:
li::marker {
color: blue;
font-weight: bold;
}
and this could be applied to a page that contains markup of the form:
<ol>
<li>Item 1</li>
<li>Item 2</li>
...
<li>Item N-1</li>
<li>Item N</li>
</ol>
Formerly to the achieve the same effect you would need to use a stylesheet of the form:
li {
color: blue;
font-weight: bold;
}
.list-item-content {
all: initial;
}
and then write your markup to have the form:
<ol>
<li><span class="list-item-content">Item 1</span></li>
<li><span class="list-item-content">Item 2</span></li>
...
<li><span class="list-item-content">Item N-1</span></li>
<li><span class="list-item-content">Item N</span></li>
</ol>
The ::marker pseudo element only supports stylizing all font properties and the color property
of a list item marker.
Tests: fast/lists/list-marker-with-display.html
http/wpt/css/css-pseudo-4/marker-and-other-pseudo-elements.html
http/wpt/css/css-pseudo-4/marker-color.html
http/wpt/css/css-pseudo-4/marker-font-properties.html
http/wpt/css/css-pseudo-4/marker-inherit-values.html
- css/CSSSelector.cpp:
(WebCore::CSSSelector::pseudoId): Return the pseudo id for the ::marker pseudo element.
- css/CSSSelector.h: Add enumerator PseudoElementMarker to the pseudo element enum.
- css/RuleSet.cpp:
(WebCore::determinePropertyWhitelistType): Return whitelist type PropertyWhitelistMarker for ::marker
so that we match rules against the acceptable rules for ::marker.
- css/RuleSet.h: Add enumerator PropertyWhitelistMarker to the property whitelist type enum.
- css/SelectorPseudoElementTypeMap.in: Add "marker" to the list of pseudo element types.
- css/StyleResolver.cpp:
(WebCore::isValidMarkerStyleProperty): Determines if the specified CSS property is valid inside ::marker.
(WebCore::StyleResolver::CascadedProperties::addMatch): Only recognize CSS properties in the content block
of ::marker that match the ::marker whitelist policy.
- rendering/RenderListItem.cpp:
(WebCore::RenderListItem::computeMarkerStyle): Computes the style object for the list item marker. We
apply the user-agent style to the marker here as opposed to defining ::marker in the UA sheet as per
the spec. as an optimization to avoid having the style resolver apply the pseudo element to all elements.
For now, we always inherit style from the originating element (list item). Added FIXME to selectively
inherit styles.
(WebCore::RenderListItem::styleDidChange): Always apply the list marker style to the list marker renderer.
- rendering/RenderListItem.h:
- rendering/style/RenderStyleConstants.h: Add pseudo ID for the ::marker pseudo element.
LayoutTests:
Add tests that check we respect ::marker when rendering the list item marker. I will
submit all the tests in http/wpt/css/css-pseudo-4 to the Web Platform Tests repository
shortly and then import them into the WebKit repository in a subsequent commit.
- fast/lists/list-marker-with-display-expected.html: Added.
- fast/lists/list-marker-with-display.html: Added.
- http/wpt/css/css-pseudo-4/marker-and-other-pseudo-elements-expected.html: Added.
- http/wpt/css/css-pseudo-4/marker-and-other-pseudo-elements.html: Added.
- http/wpt/css/css-pseudo-4/marker-color-expected.html: Added.
- http/wpt/css/css-pseudo-4/marker-color.html: Added.
- http/wpt/css/css-pseudo-4/marker-font-properties-expected.html: Added.
- http/wpt/css/css-pseudo-4/marker-font-properties.html: Added.
- http/wpt/css/css-pseudo-4/marker-inherit-values-expected.html: Added.
- http/wpt/css/css-pseudo-4/marker-inherit-values.html: Added.
- 9:01 AM Changeset in webkit [220206] by
-
- 2 edits in trunk/Tools
[Win] The test http/tests/security/contentSecurityPolicy/upgrade-insecure-requests/basic-upgrade.https.html is failing.
https://bugs.webkit.org/show_bug.cgi?id=159510
Reviewed by Daniel Bates.
Allow any https certificate when running tests.
- DumpRenderTree/win/DumpRenderTree.cpp:
(runTest):
- 7:47 AM Changeset in webkit [220205] by
-
- 2 edits in trunk/Source/WebKitLegacy
[Win] WebKit COM header file is not placed in the correct location.
https://bugs.webkit.org/show_bug.cgi?id=175101
Reviewed by Brent Fulgham.
After the transition to WebKitLegacy, the generated WebKit header files should still
be placed in the WebKit folder, so WebKit clients will find the header files in the
same place.
- WebKitLegacy.vcxproj/WebKitLegacy.proj:
- 7:45 AM Changeset in webkit [220204] by
-
- 5 edits1 add in trunk/Source/WebCore
Factor common code in Style::*ChangeInvalidation into helper functions
https://bugs.webkit.org/show_bug.cgi?id=174312
Reviewed by Andreas Kling.
There is a lot of copy code here.
- Style/StyleInvalidationFunctions.h: Added.
(WebCore::Style::traverseRuleFeaturesInShadowTree):
(WebCore::Style::traverseRuleFeaturesForSlotted):
(WebCore::Style::traverseRuleFeatures):
Add functions for traversing rule features that may affect style of an element.
Use lambdas to implement client-specific behavior.
- WebCore.xcodeproj/project.pbxproj:
- style/AttributeChangeInvalidation.cpp:
(WebCore::Style::mayBeAffectedByAttributeChange):
(WebCore::Style::AttributeChangeInvalidation::invalidateStyle):
(WebCore::Style::mayBeAffectedByHostRules): Deleted.
(WebCore::Style::mayBeAffectedBySlottedRules): Deleted.
- style/ClassChangeInvalidation.cpp:
(WebCore::Style::ClassChangeInvalidation::invalidateStyle):
(WebCore::Style::mayBeAffectedByHostRules): Deleted.
(WebCore::Style::mayBeAffectedBySlottedRules): Deleted.
- style/IdChangeInvalidation.cpp:
(WebCore::Style::IdChangeInvalidation::invalidateStyle):
(WebCore::Style::mayBeAffectedByHostRules): Deleted.
(WebCore::Style::mayBeAffectedBySlottedRules): Deleted.
- 7:40 AM Changeset in webkit [220203] by
-
- 2 edits in trunk/Source/WebCore
[EME] CDM constructor assigns CDMPrivate member multiple times
https://bugs.webkit.org/show_bug.cgi?id=175128
Reviewed by Xabier Rodriguez-Calvar.
In the CDM class constructor, iterate over the registered CDM
factories, finding one that supports the specified key system.
A CDMPrivate object is created through that factory, and the
iteration is now stopped at that point, while previously it
contined to potentially create CDMPrivate objects through
other factories.
Helper createCDMPrivateForKeySystem() function is removed.
- Modules/encryptedmedia/CDM.cpp:
(WebCore::CDM::CDM):
(WebCore::createCDMPrivateForKeySystem): Deleted.
- 6:20 AM Changeset in webkit [220202] by
-
- 5 edits in trunk/Source/WebCore
Don't always recalc the style of display: contents elements.
https://bugs.webkit.org/show_bug.cgi?id=172753
Patch by Emilio Cobos Álvarez <ecobos@igalia.com> on 2017-08-03
Reviewed by Antti Koivisto.
No new tests (no functionality change). This only removes an
inefficiency.
- dom/Element.cpp:
(WebCore::Element::existingComputedStyle):
- dom/Element.h:
- style/RenderTreeUpdater.cpp:
(WebCore::RenderTreeUpdater::updateRenderTree):
(WebCore::RenderTreeUpdater::updateElementRenderer):
- style/StyleTreeResolver.cpp:
(WebCore::Style::renderOrDisplayContentsStyle):
(WebCore::Style::TreeResolver::resolveElement):
(WebCore::Style::TreeResolver::createAnimatedElementUpdate):
(WebCore::Style::shouldResolveElement):
(WebCore::Style::TreeResolver::resolveComposedTree):
- 4:17 AM Changeset in webkit [220201] by
-
- 4 edits in trunk/Source/WTF
[Linux][WTF] Use one global semaphore to notify thread suspend and resume completion
https://bugs.webkit.org/show_bug.cgi?id=175124
Reviewed by Carlos Garcia Campos.
POSIX sem_t is used to notify thread suspend and resume completion in Linux ports
since sem_post is async-signal-safe function. Since we guard suspend() and resume()
with one global lock, this semaphore is also guarded by this lock. So we do not need
to have semaphore per WTF::Thread.
This patch introduces one global Semaphore. And drop per thread semaphore.
- wtf/Threading.h:
- wtf/ThreadingPthreads.cpp:
(WTF::Thread::~Thread):
(WTF::Semaphore::Semaphore):
(WTF::Semaphore::~Semaphore):
(WTF::Semaphore::wait):
(WTF::Semaphore::post):
(WTF::Thread::signalHandlerSuspendResume):
(WTF::Thread::initializePlatformThreading):
(WTF::Thread::suspend):
(WTF::Thread::resume):
(WTF::Thread::Thread): Deleted.
- wtf/ThreadingWin.cpp:
(WTF::Thread::Thread): Deleted.
- 3:03 AM Changeset in webkit [220200] by
-
- 2 edits in trunk/LayoutTests
[GTK] Remove obsolete failure expectation for narrow-non-breaking-space.html.
https://bugs.webkit.org/show_bug.cgi?id=139493
Unreviewed test gardening.
It likely started passing in r205826 due to its change in Font.cpp.
Patch by Ms2ger <Ms2ger@igalia.com> on 2017-08-03
- platform/gtk/TestExpectations:
- 2:33 AM Changeset in webkit [220199] by
-
- 2 edits in trunk/LayoutTests
[GTK] Skip timezone-dependent Sputnik tests.
https://bugs.webkit.org/show_bug.cgi?id=175120
Unreviewed test gardening.
These tests are skipped in the platform-neutral TestExpectations file because
they only pass in Pacific Time (see bug 42625). There doesn't seem to be a
reason for gtk to run them, and they do pass on the buildbot.
Patch by Ms2ger <Ms2ger@igalia.com> on 2017-08-03
- platform/gtk/TestExpectations:
- 1:53 AM Changeset in webkit [220198] by
-
- 3 edits in trunk/Source/WTF
[Linux][WTF] Reduce sizeof(WTF::Thread) by using a pointer to PlatformRegisters
https://bugs.webkit.org/show_bug.cgi?id=175119
Reviewed by Carlos Garcia Campos.
sizeof(PlatformRegisters) is so large. In my Linux box, it is 256. It enlarges the sizeof(WTF::Thread).
However, it is not necessary to hold it in WTF::Thread member. Thread's ucontext data and its stack is
effective while suspending the thread. So, we can just use the pointer to the PlatformRegister instead
of copying it to the member of the WTF::Thread.
- wtf/Threading.h:
- wtf/ThreadingPthreads.cpp:
(WTF::Thread::signalHandlerSuspendResume):
(WTF::Thread::getRegisters):
- 12:03 AM Changeset in webkit [220197] by
-
- 7 edits in branches/safari-604.1.36-branch/Source
Versioning.
- 12:01 AM Changeset in webkit [220196] by
-
- 7 edits in tags/Safari-604.1.36/Source
Revert "Versioning."
This reverts commit 47a02c7900432c88ffc054f87447bf2ced2bc7bc.
