Timeline
Jul 5, 2020:
- 5:59 PM Changeset in webkit [263961] by
-
- 34 edits4 deletes in trunk
Unreviewed, reverting r263960.
https://bugs.webkit.org/show_bug.cgi?id=213980
Re-land, because r263959 somehow fixed the build issue caused
by r263953
Reverted changeset:
"Unreviewed, reverting r263953 and r263959."
https://bugs.webkit.org/show_bug.cgi?id=213979
https://trac.webkit.org/changeset/263960
- 5:46 PM Changeset in webkit [263960] by
-
- 34 edits4 adds in trunk
Unreviewed, reverting r263953 and r263959.
https://bugs.webkit.org/show_bug.cgi?id=213979
Broke internal build
Reverted changesets:
"[Cocoa] Move almost all features from FeatureDefines.xcconfig
to PlatformEnableCocoa.h"
https://bugs.webkit.org/show_bug.cgi?id=212542
https://trac.webkit.org/changeset/263953
"[Cocoa] Remove FEATURE_DEFINES from the Cocoa/Xcode build
system"
https://bugs.webkit.org/show_bug.cgi?id=213976
https://trac.webkit.org/changeset/263959
- 4:34 PM Changeset in webkit [263959] by
-
- 32 edits4 deletes in trunk
[Cocoa] Remove FEATURE_DEFINES from the Cocoa/Xcode build system
https://bugs.webkit.org/show_bug.cgi?id=213976
Reviewed by Sam Weinig.
Source/JavaScriptCore:
- Configurations/Base.xcconfig: Removed FEATURE_DEFINES.
- Configurations/FeatureDefines.xcconfig: Removed.
- Configurations/JSC.xcconfig: Removed include of FeatureDefines.xcconfig.
- Configurations/JavaScriptCore.xcconfig: Ditto.
- Configurations/ToolExecutable.xcconfig: Ditto.
- DerivedSources-input.xcfilelist: Removed FeatureDefines.xcconfig.
- DerivedSources.make: Removed FEATURE_DEFINES and FEATURE_DEFINE_FLAGS.
- JavaScriptCore.xcodeproj/project.pbxproj: Removed FeatureDefines.xcconfig.
Source/WebCore:
- Configurations/FeatureDefines.xcconfig: Removed everything except for ENABLE_WEB_RTC.
- Configurations/WebCore.xcconfig: Removed FEATURE_DEFINES.
- DerivedSources-input.xcfilelist: Removed FeatureDefines.xcconfig.
- DerivedSources.make: Removed FEATURE_DEFINES, FEATURE_DEFINE_FLAGS, and the dependency
on FeatureDefines.xcconfig.
Source/WebCore/PAL:
- Configurations/FeatureDefines.xcconfig: Removed.
- Configurations/PAL.xcconfig: Removed FEATURE_DEFINES and the include
of FeatureDefines.xcconfig.
- PAL.xcodeproj/project.pbxproj: Removed FeatureDefines.xcconfig.
Source/WebKit:
- Configurations/BaseTarget.xcconfig: Removed FEATURE_DEFINES, and added
WK_ENABLE_EXPERIMENTAL_FEATURES,
- Configurations/FeatureDefines.xcconfig: Removed everything except for ENABLE_WEB_RTC.
Source/WebKitLegacy:
- WebKitLegacy.xcodeproj/project.pbxproj: Removed FeatureDefines.xconfig.
Source/WebKitLegacy/mac:
- Configurations/FeatureDefines.xcconfig: Removed.
- Configurations/WebKitLegacy.xcconfig: Removed FEATURE_DEFINES and the include of
FeatureDefines.xcconfig.
- MigrateHeaders.make: Removed FEATURE_DEFINES, FEATURE_DEFINE_FLAGS, and
FEATURE_AND_PLATFORM_DEFINE_DEPENDENCIES. FEATURE_AND_PLATFORM_DEFINES is now used
solely to determine if this is an internal SDK build, which should be cleaned up later.
Tools:
- Scripts/webkitpy/style/checkers/featuredefines.py: Removed all FeatureDefines.xcconfig
files except for the two that remain, WebCore and WebKit.
- TestWebKitAPI/Configurations/Base.xcconfig: Removed FEATURE_DEFINES.
- TestWebKitAPI/Configurations/FeatureDefines.xcconfig: Removed.
- TestWebKitAPI/Configurations/TestWTF.xcconfig: Removed include of FeatureDefines.xcconfig.
- TestWebKitAPI/Configurations/TestWTFLibrary.xcconfig: Ditto.
- TestWebKitAPI/Configurations/TestWebKitAPILibrary.xcconfig: Ditto.
- TestWebKitAPI/Configurations/WebProcessPlugIn.xcconfig: Ditto.
- TestWebKitAPI/TestWebKitAPI.xcodeproj/project.pbxproj: Removed FeatureDefines.xcconfig.
- 4:32 PM Changeset in webkit [263958] by
-
- 4 edits in trunk
Text manipulation: add a new heuristic to decide paragraph boundary
https://bugs.webkit.org/show_bug.cgi?id=213918
Patch by Sihui Liu <sihui_liu@appe.com> on 2020-07-05
Reviewed by Wenson Hsieh.
Source/WebCore:
Modified test: TextManipulation.StartTextManipulationTreatsInlineBlockLinksAndButtonsAndSpansAsParagraphs.
- editing/TextManipulationController.cpp:
(WebCore::isEnclosingItemBoundaryElement):
Tools:
- TestWebKitAPI/Tests/WebKitCocoa/TextManipulation.mm:
(TestWebKitAPI::TEST):
- 4:10 PM Changeset in webkit [263957] by
-
- 2 edits in trunk/Source/WebCore
[Cocoa] REGRESSION (r263914): Non-Apple-Pay Cocoa builds failing
https://bugs.webkit.org/show_bug.cgi?id=213977
- Modules/applepay/cocoa/PaymentMethodCocoa.mm: Fixed messed up #endif and
unnecessary #undef at end of file.
- 4:06 PM Changeset in webkit [263956] by
-
- 8 edits1 add in trunk/Source
[iOS] La Banque Postale app does not respect safe area insets on iOS 14
https://bugs.webkit.org/show_bug.cgi?id=213974
<rdar://problem/65109197>
Reviewed by Darin Adler.
Source/WebCore:
- platform/RuntimeApplicationChecks.h:
- platform/cocoa/RuntimeApplicationChecksCocoa.mm:
(WebCore::IOSApplication::isLaBanquePostale): Added.
Source/WebKitLegacy:
- WebKitLegacy.xcodeproj/project.pbxproj:
Source/WebKitLegacy/mac:
La Banque Postale uses a fixed list of iOS versions that support safe area insets (versions
11 through 13 at the time of writing). Since iOS 14 is not in this list, the app fails to
apply top and bottom safe area insets to its web content. Work around this by adding the
'device-ios12p' class to <body> as if the iOS major version were 13 (the app currently uses
'device-ios12p' to represent both iOS 12 and iOS 13). This quirk will be disabled on
versions of La Banque Postale that link against the iOS 14 SDK (or later).
- Configurations/WebKitLegacy.xcconfig: Excluded LaBanquePostaleQuirks.js on Mac.
- Resources/LaBanquePostaleQuirks.js: Added.
- WebView/WebView.mm:
(needsLaBanquePostaleQuirks): Added. Checks if the quirk is needed.
(leakLaBanquePostaleQuirksScript): Added. Leaks an NSString containing the quirk script.
(-[WebView _injectLaBanquePostaleQuirks]): Added. Injects the quirk as a user script.
(-[WebView _commonInitializationWithFrameName:groupName:]):
- 3:23 PM Changeset in webkit [263955] by
-
- 1 copy in tags/Safari-610.1.20.4
Tag Safari-610.1.20.4.
- 2:57 PM Changeset in webkit [263954] by
-
- 8 edits in branches/safari-610.1.20-branch/Source
Versioning.
WebKit-610.1.20.4
- 12:31 PM Changeset in webkit [263953] by
-
- 14 edits in trunk
[Cocoa] Move almost all features from FeatureDefines.xcconfig to PlatformEnableCocoa.h
https://bugs.webkit.org/show_bug.cgi?id=212542
Reviewed by Sam Weinig.
Source/JavaScriptCore:
- Configurations/FeatureDefines.xcconfig: Delete everything except
ENABLE_EXPERIMENTAL_FEATURES and ENABLE_WEBRTC.
Source/WebCore:
- Configurations/FeatureDefines.xcconfig: Delete everything except
ENABLE_EXPERIMENTAL_FEATURES and ENABLE_WEBRTC.
Source/WebCore/PAL:
- Configurations/FeatureDefines.xcconfig: Delete everything except
ENABLE_EXPERIMENTAL_FEATURES and ENABLE_WEBRTC.
Source/WebKit:
- Configurations/FeatureDefines.xcconfig: Delete everything except
ENABLE_EXPERIMENTAL_FEATURES and ENABLE_WEBRTC.
- Configurations/WebKit.xcconfig: Added include of WebKitTargetConditionals.xcconfig.
Source/WebKitLegacy/mac:
- Configurations/FeatureDefines.xcconfig: Delete everything except
ENABLE_EXPERIMENTAL_FEATURES and ENABLE_WEBRTC.
Tools:
- TestWebKitAPI/Configurations/FeatureDefines.xcconfig: Delete everything except
ENABLE_EXPERIMENTAL_FEATURES and ENABLE_WEBRTC.
- TestWebKitAPI/Configurations/TestWebKitAPI.xcconfig: Added include of
WebKitTargetConditionals.xcconfig.
- 12:00 PM Changeset in webkit [263952] by
-
- 8 edits in trunk/Source/WebCore
[macOS 11] Indeterminate progress bar animation periodically jumps
https://bugs.webkit.org/show_bug.cgi?id=213967
Reviewed by Sam Weinig.
Source/WebCore:
The animation for an indeterminate progress bar occasionally jumps to the starting frame on macOS 11. This is
because the indeterinate progress bar animation only lasts for 120 frames on macOS 11, so our current hard-coded
value of 256 frames results in 16 extra frames of animation, before abruptly looping back to the starting frame.
To fix this, instead of specifying the animation frame index when painting progress bars on macOS, specify the
starting time and current time for the animation, and let CoreUI determine which frame to use.
- rendering/RenderProgress.cpp:
(WebCore::RenderProgress::animationProgress):
Assert that this never gets called with a zero animation duration.
(WebCore::RenderProgress::updateAnimationState):
Instead of checking for the animation duration to determine whether or not we should try and animate progress
bar painting, check for the animation repeat interval instead. This is because on macOS, we no longer hard-code
the animation duration, but still want to be able to animate progress bars.
- rendering/RenderThemeIOS.h:
- rendering/RenderThemeIOS.mm:
(WebCore::RenderThemeIOS::animationRepeatIntervalForProgressBar const): Deleted.
(WebCore::RenderThemeIOS::animationDurationForProgressBar const): Deleted.
Drive-by fix: remove some unnecessary overridden method implementations on RenderThemeIOS.
- rendering/RenderThemeMac.h:
- rendering/RenderThemeMac.mm:
(WebCore::RenderThemeMac::paintProgressBar):
(WebCore::RenderThemeMac::animationDurationForProgressBar const): Deleted.
Stop overriding this method, and remove the
progressAnimationNumFramesconstant declared in this file.
Source/WebCore/PAL:
- pal/spi/mac/CoreUISPI.h: Add declarations for
kCUIAnimationStartTimeKeyandkCUIAnimationTimeKey.
- 7:23 AM Changeset in webkit [263951] by
-
- 3 edits in trunk/Source/JavaScriptCore
Web Inspector: Fix python3 build warnings
https://bugs.webkit.org/show_bug.cgi?id=213971
Patch by Philippe Normand <pnormand@igalia.com> on 2020-07-05
Reviewed by Sam Weinig.
Fix Python3 syntax warnings. Using 'is' with string literals triggers those. Adopt the ==
operator instead, which is more idiomatic anyway.
- inspector/scripts/codegen/cpp_generator.py:
(CppGenerator.cpp_getter_method_for_type):
(CppGenerator.cpp_setter_method_for_type):
- inspector/scripts/codegen/objc_generator.py:
(ObjCTypeCategory.category_for_type):
(ObjCGenerator.objc_type_for_raw_name):
(ObjCGenerator.objc_class_for_raw_name):
(ObjCGenerator.protocol_type_for_raw_name):
(ObjCGenerator.objc_protocol_export_expression_for_variable):
(ObjCGenerator.objc_protocol_import_expression_for_variable):
(ObjCGenerator.objc_to_protocol_expression_for_member):
(ObjCGenerator.protocol_to_objc_expression_for_member):
(ObjCGenerator.payload_to_objc_expression_for_member):
(ObjCGenerator.objc_setter_method_for_member_internal):
(ObjCGenerator.objc_getter_method_for_member_internal):
(ObjCGenerator.objc_protocol_export_expression_for_variable.is): Deleted.
(ObjCGenerator.objc_protocol_import_expression_for_variable.is): Deleted.
(ObjCGenerator.objc_to_protocol_expression_for_member.is): Deleted.
(ObjCGenerator.protocol_to_objc_expression_for_member.is): Deleted.
- 7:15 AM Changeset in webkit [263950] by
-
- 2 edits in trunk/Tools
[WPE] run-wpe-tests Python3 compatibility
https://bugs.webkit.org/show_bug.cgi?id=213969
Patch by Philippe Normand <pnormand@igalia.com> on 2020-07-05
Reviewed by Sam Weinig.
- Scripts/run-wpe-tests:
- 3:02 AM Changeset in webkit [263949] by
-
- 15 edits in trunk
[macOS] Allow selecting HEIF images if the 'accept' attribute includes an image MIME type that the platform can transcode the HEIF to
https://bugs.webkit.org/show_bug.cgi?id=212489
Patch by Said Abou-Hallawa <sabouhallawa@apple.com> on 2020-07-05
Reviewed by Darin Adler.
Source/WebCore:
Add some utility functions to MIMETypeRegistry. These new functions will
deal with a list of MIME types combined with a list of file extensions.
They are going to be used the FileChooserSettings which reflects the
'accept' attribute of the <input> file element.
- platform/MIMETypeRegistry.cpp:
(WebCore::trimmedExtension):
(WebCore::MIMETypeRegistry::preferredImageMIMETypeForEncoding):
(WebCore::MIMETypeRegistry::containsImageMIMETypeForEncoding):
(WebCore::MIMETypeRegistry::allowedMIMETypes):
It returns the set of mime types which is stated by a list of mime types
and a list of file extensions.
(WebCore::MIMETypeRegistry::allowedFileExtensions):
It returns the set of extensions which is stated by a list of mime types
and a list of file extensions.
- platform/MIMETypeRegistry.h:
Source/WebKit:
Make WKOpenPanelParameters call allowedFileExtensions() and allowedMIMETypes()
of MIMETypeRegistry via OpenPanelParameters.
UIDelegate will get allowedMIMETypes() and pass it to WebOpenPanelResultListenerProxy
which will pass it to WebPageProxy. In a following patch, WebPageProxy
will use allowedMIMETypes to transcode the selected images before sending
them to WebPage.
- UIProcess/API/APIOpenPanelParameters.cpp:
(API::OpenPanelParameters::allowedMIMETypes const):
(API::OpenPanelParameters::allowedFileExtensions const):
(API::OpenPanelParameters::mediaCaptureType const): Deleted.
- UIProcess/API/APIOpenPanelParameters.h:
(API::OpenPanelParameters::mediaCaptureType const):
- UIProcess/API/Cocoa/WKOpenPanelParameters.mm:
(-[WKOpenPanelParameters _allowedFileExtensions]):
(-[WKOpenPanelParameters _allowedFileExtensionsTitles]): Deleted.
- UIProcess/API/Cocoa/WKOpenPanelParametersPrivate.h:
- UIProcess/Cocoa/UIDelegate.mm:
(WebKit::UIDelegate::UIClient::runOpenPanel):
- UIProcess/WebOpenPanelResultListenerProxy.cpp:
(WebKit::WebOpenPanelResultListenerProxy::chooseFiles):
- UIProcess/WebOpenPanelResultListenerProxy.h:
(WebKit::WebOpenPanelResultListenerProxy::chooseFiles):
- UIProcess/WebPageProxy.cpp:
(WebKit::WebPageProxy::didChooseFilesForOpenPanel):
- UIProcess/WebPageProxy.h:
Tools:
The scope of this enhancement will be scaled down. No UI change will be
done for now. Only allow HEIF/HEIC images to be selected if the 'accept'
attribute include a mime type which CG supports transcoding the HEIF/HEIC
images to.
- MiniBrowser/mac/WK2BrowserWindowController.m:
(-[WK2BrowserWindowController dealloc]):
(-[WK2BrowserWindowController webView:runOpenPanelWithParameters:initiatedByFrame:completionHandler:webView:runOpenPanelWithParameters:initiatedByFrame:completionHandler:]):
(-[FileExtensionsPopupTarget initWithOpenPanel:allowedFileExtensions:]): Deleted.
(-[FileExtensionsPopupTarget popupAction:]): Deleted.
(-[FileExtensionsPopupTarget dealloc]): Deleted.
(-[WK2BrowserWindowController createFilterView:popupTarget:]): Deleted.
Jul 4, 2020:
- 6:54 PM Changeset in webkit [263948] by
-
- 2 edits in trunk/Source/WTF
[Cocoa] Add almost everything from FeatureDefines.xcconfig to PlatformEnableCocoa.h
https://bugs.webkit.org/show_bug.cgi?id=213964
Reviewed by Sam Weinig.
Added logic that matches what FeatureDefines.xcconfig does for everything
except for ENABLE_EXPERIMENTAL_FEATURES.
Changed format so we mostly list platforms where we do not want to enable
a certain feature; also order it pretty consistently macOS, iOS, Mac Catalyst,
watchOS, then tvOS. This format highlights possible mistakes when not enabling
a feature on some platforms in an easier to understand way.
- wtf/PlatformEnableCocoa.h:
(ENABLE_AIRPLAY_PICKER): List platforms that do *not* enable this.
(ENABLE_APPLE_PAY): All except Mac Catalyst, watchOS, tvOS.
Not sure why disabled in Mac Catalyst, but replicates FeatureDefines.xcconfig.
(ENABLE_APPLE_PAY_REMOTE_UI): List platforms that do *not* enable this.
(ENABLE_AUTOCAPITALIZE): List platforms that do *not* enable this.
(ENABLE_AUTOCORRECT): List platforms that do *not* enable this.
(ENABLE_CONTENT_FILTERING): All except older watchOS simulator and tvOS.
(ENABLE_CORE_IMAGE_ACCELERATED_FILTER_RENDER): Re-sorted this.
(ENABLE_CSS_CONIC_GRADIENTS): All except older macOS and tvOS. The tvOS
exception seems questionable, so added a FIXME.
(ENABLE_DARK_MODE_CSS): All except watchOS and tvOS.
(ENABLE_DATA_INTERACTION): All except macOS, watchOS, and tvOS.
(ENABLE_DATA_DETECTION): All except Mac Catalyst, watchOS, and tvOS.
(ENABLE_EDITABLE_REGION): Only iOS.
(ENABLE_ENCRYPTED_MEDIA): All except older macOS and Mac Catalyst.
(ENABLE_FULLSCREEN_API): All except Mac Catalyst, watchOS, and tvOS.
(ENABLE_FULL_KEYBOARD_ACCESS): All except Mac Catalyst, watchOS, and tvOS.
(ENABLE_GAMEPAD): All except watchOS.
(ENABLE_INPUT_TYPE_DATE): All except macOS, Mac Catalyst, and tvOS.
(ENABLE_INPUT_TYPE_DATETIMELOCAL): Ditto.
(ENABLE_INPUT_TYPE_MONTH): Ditto.
(ENABLE_INPUT_TYPE_TIME): Ditto.
(ENABLE_INPUT_TYPE_WEEK): Ditto.
(ENABLE_MEDIA_CAPTURE): All except macOS.
(ENABLE_MEDIA_SOURCE): All except Mac Catalyst, watchOS, tvOS, and
all simulators.
(ENABLE_MEDIA_STREAM): All except Mac Catalyst, watchOS, and tvOS.
(ENABLE_NAVIGATOR_STANDALONE): All iOS family. (Considered writing as
all except macOS, but left like this to match some .cpp code for now.)
(ENABLE_OVERFLOW_SCROLLING_TOUCH): All iOS family.
(ENABLE_PICTURE_IN_PICTURE_API): All except Mac Catalyst, watchOS, and tvOS.
(ENABLE_SPEECH_SYNTHESIS): All except Mac Catalyst.
(ENABLE_REVEAL): All except older macOS, watchOS, and tvOS.
(ENABLE_SIGILL_CRASH_ANALYZER): All except macOS and watchOS.
(ENABLE_TAKE_UNBOUNDED_NETWORKING_ASSERTION): All except macOS, watchOS, and tvOS.
(ENABLE_VIDEO_PRESENTATION_MODE): All except simulators and tvOS.
(ENABLE_WEBGPU): All except older macOS, Mac Catalyst, and simulators.
(ENABLE_WEB_API_STATISTICS): Re-sorted this.
(ENABLE_WEB_AUTHN): All except Mac Catalyst, watchOS, and tvOS.
(ENABLE_WEB_RTC): All except Mac Catalyst, watchOS, and tvOS.
- 2:32 PM Changeset in webkit [263947] by
-
- 14 edits in trunk
[Cocoa] Remove all features from FeatureDefines.xcconfig that are already mentioned in PlatformEnableCocoa.h
https://bugs.webkit.org/show_bug.cgi?id=213962
Reviewed by Sam Weinig.
Source/JavaScriptCore:
- Configurations/FeatureDefines.xcconfig: Removed all features that were mentioned
in PlatformEnableCocoa.h; the rules in that file now define whether they are enabled.
Source/WebCore:
- Configurations/FeatureDefines.xcconfig: Removed all features that were mentioned
in PlatformEnableCocoa.h; the rules in that file now define whether they are enabled.
Source/WebCore/PAL:
- Configurations/FeatureDefines.xcconfig: Removed all features that were mentioned
in PlatformEnableCocoa.h; the rules in that file now define whether they are enabled.
Source/WebKit:
- Configurations/FeatureDefines.xcconfig: Removed all features that were mentioned
in PlatformEnableCocoa.h; the rules in that file now define whether they are enabled.
Source/WebKitLegacy/mac:
- Configurations/FeatureDefines.xcconfig: Removed all features that were mentioned
in PlatformEnableCocoa.h; the rules in that file now define whether they are enabled.
Source/WTF:
- wtf/PlatformEnableCocoa.h:
(ENABLE_DRAG_SUPPORT): Before, this was enabled for iOS, iOS simulator, and Mac Catalyst in the
FeatureDefines.xcconfig file, but then disabled for any other iOS family platform here, then enabled
by default in PlatformEnable.h, which covered Mac. Instead, disable here only for watchOS and tvOS,
which gives the same result in a more straightforward way.
(ENABLE_GEOLOCATION): Before, this was enabled for iOS, iOS simulator, Mac Catalyst, and Mac in the
FeatureDefines.xcconfig file, then enabled for any other iOS family platform here. Instead, just
enable here for all Cocoa platforms, which gives the same result in a more straightforward way.
Tools:
- TestWebKitAPI/Configurations/FeatureDefines.xcconfig: Removed all features that were mentioned
in PlatformEnableCocoa.h; the rules in that file now define whether they are enabled.
- 2:17 PM Changeset in webkit [263946] by
-
- 8 edits in trunk/Source/WebCore
[LFC][BFC] Move PositiveAndNegativeVerticalMargin to UsedVerticalMargin
https://bugs.webkit.org/show_bug.cgi?id=213963
Reviewed by Antti Koivisto.
Now that UsedVerticalMargin is not used in Display::Box, we can merged these 2 layout structures.
- layout/MarginTypes.h:
(WebCore::Layout::UsedVerticalMargin::PositiveAndNegativePair::Values::isNonZero const):
(WebCore::Layout::PositiveAndNegativeVerticalMargin::Values::isNonZero const): Deleted.
- layout/blockformatting/BlockFormattingContext.cpp:
(WebCore::Layout::BlockFormattingContext::precomputeVerticalPositionForBoxAndAncestors):
(WebCore::Layout::BlockFormattingContext::computeHeightAndMargin):
- layout/blockformatting/BlockFormattingContext.h:
- layout/blockformatting/BlockFormattingState.h:
(WebCore::Layout::BlockFormattingState::hasUsedVerticalMargin const):
(WebCore::Layout::BlockFormattingState::setPositiveAndNegativeVerticalMargin): Deleted.
(WebCore::Layout::BlockFormattingState::hasPositiveAndNegativeVerticalMargin const): Deleted.
(WebCore::Layout::BlockFormattingState::positiveAndNegativeVerticalMargin const): Deleted.
- layout/blockformatting/BlockMarginCollapse.cpp:
(WebCore::Layout::BlockFormattingContext::MarginCollapse::computedPositiveAndNegativeMargin const):
(WebCore::Layout::BlockFormattingContext::MarginCollapse::marginValue const):
(WebCore::Layout::BlockFormattingContext::MarginCollapse::updateMarginAfterForPreviousSibling):
(WebCore::Layout::BlockFormattingContext::MarginCollapse::positiveNegativeValues const):
(WebCore::Layout::BlockFormattingContext::MarginCollapse::positiveNegativeMarginBefore const):
(WebCore::Layout::BlockFormattingContext::MarginCollapse::positiveNegativeMarginAfter const):
(WebCore::Layout::BlockFormattingContext::MarginCollapse::collapsedVerticalValues):
- layout/blockformatting/PrecomputedBlockMarginCollapse.cpp:
(WebCore::Layout::BlockFormattingContext::MarginCollapse::precomputedPositiveNegativeValues const):
(WebCore::Layout::BlockFormattingContext::MarginCollapse::precomputedPositiveNegativeMarginBefore const):
- layout/blockformatting/tablewrapper/TableWrapperBlockFormattingContext.cpp:
(WebCore::Layout::TableWrapperBlockFormattingContext::computeHeightAndMarginForTableBox):
- 2:09 PM Changeset in webkit [263945] by
-
- 2 edits in trunk/Source/WebCore
[LFC][BFC] Remove redundant out-of-flow height-and-margin handling
https://bugs.webkit.org/show_bug.cgi?id=213959
Reviewed by Antti Koivisto.
We will never end up here with an out-of-flow box.
- layout/blockformatting/BlockFormattingContext.cpp:
(WebCore::Layout::BlockFormattingContext::computeHeightAndMargin):
- 1:06 PM Changeset in webkit [263944] by
-
- 5 edits3 adds in trunk
%TypedArray%.prototype.{indexOf,lastIndexOf} are not spec-perfect
https://bugs.webkit.org/show_bug.cgi?id=213715
Reviewed by Yusuke Suzuki.
JSTests:
- microbenchmarks/array-prototype-indexOf-empty.js: Added.
- microbenchmarks/typed-array-indexOf-empty.js: Added.
- stress/array-indexof-tointeger-order.js: Added.
- test262/expectations.yaml: Mark 10 test cases as passing.
Source/JavaScriptCore:
This patch:
- Implements step 3 of {Array,%TypedArray%}.prototype.indexOf [1] and %TypedArray%.prototype.lastIndexOf [2] since it is observable when second argument is an object with userland toString() or valueOf() method. Advances provided microbenchmark by 100% for Array and by 25% for %TypedArray%.
- Removes argument count check from %TypedArray%.prototype.{indexOf,lastIndexOf},
allowing these methods to be invoked w/o arguments. The spec treats missing
arguments as
undefined, always returning -1 for typed arrays.
Both changes align JSC with V8 and SpiderMonkey.
[1]: https://tc39.es/ecma262/#sec-array.prototype.indexof
[2]: https://tc39.es/ecma262/#sec-%typedarray%.prototype.lastindexof
- runtime/ArrayPrototype.cpp:
(JSC::arrayProtoFuncIndexOf):
(JSC::arrayProtoFuncLastIndexOf):
- runtime/JSGenericTypedArrayViewPrototypeFunctions.h:
(JSC::genericTypedArrayViewProtoFuncIndexOf):
(JSC::genericTypedArrayViewProtoFuncLastIndexOf):
- 1:06 PM Changeset in webkit [263943] by
-
- 15 edits in trunk/Source/WebCore
[LFC][BFC] Store used vertical margin values in the formatting state
https://bugs.webkit.org/show_bug.cgi?id=213958
Reviewed by Antti Koivisto.
Layout related vertical margin values (collapsed vs. non-collapsed) should all go to the FormattingState, while
Display::Box should only have the final values.
- layout/FormattingContext.cpp:
(WebCore::Layout::FormattingContext::computeOutOfFlowVerticalGeometry):
- layout/FormattingContextGeometry.cpp:
(WebCore::Layout::FormattingContext::Geometry::staticVerticalPositionForOutOfFlowPositioned const):
- layout/MarginTypes.h:
(WebCore::Layout::marginBefore):
(WebCore::Layout::marginAfter):
(WebCore::Layout::UsedVerticalMargin::before const): Deleted.
(WebCore::Layout::UsedVerticalMargin::after const): Deleted.
(WebCore::Layout::UsedVerticalMargin::isCollapsedThrough const): Deleted.
(WebCore::Layout::UsedVerticalMargin::nonCollapsedValues const): Deleted.
(WebCore::Layout::UsedVerticalMargin::collapsedValues const): Deleted.
(WebCore::Layout::UsedVerticalMargin::hasCollapsedValues const): Deleted.
(WebCore::Layout::UsedVerticalMargin::setCollapsedValues): Deleted.
(WebCore::Layout::UsedVerticalMargin::UsedVerticalMargin): Deleted.
- layout/Verification.cpp:
(WebCore::Layout::outputMismatchingBlockBoxInformationIfNeeded):
- layout/blockformatting/BlockFormattingContext.cpp:
(WebCore::Layout::BlockFormattingContext::precomputeVerticalPositionForBoxAndAncestors):
(WebCore::Layout::BlockFormattingContext::computeHeightAndMargin):
(WebCore::Layout::BlockFormattingContext::verticalPositionWithMargin const):
- layout/blockformatting/BlockFormattingContextGeometry.cpp:
(WebCore::Layout::BlockFormattingContext::Geometry::inFlowNonReplacedHeightAndMargin):
- layout/blockformatting/BlockFormattingState.h:
(WebCore::Layout::BlockFormattingState::setUsedVerticalMargin):
(WebCore::Layout::BlockFormattingState::usedVerticalMargin const):
- layout/blockformatting/BlockMarginCollapse.cpp:
(WebCore::Layout::BlockFormattingContext::MarginCollapse::updateMarginAfterForPreviousSibling):
- layout/blockformatting/tablewrapper/TableWrapperBlockFormattingContext.cpp:
(WebCore::Layout::TableWrapperBlockFormattingContext::computeHeightAndMarginForTableBox):
- layout/displaytree/DisplayBox.cpp:
(WebCore::Display::Box::Box):
(WebCore::Display::Box::nonCollapsedMarginBox const): Deleted.
- layout/displaytree/DisplayBox.h:
(WebCore::Display::Box::rectWithMargin const):
(WebCore::Display::Box::setHasValidVerticalMargin):
(WebCore::Display::Box::setVerticalMargin):
(WebCore::Display::Box::marginBefore const):
(WebCore::Display::Box::marginAfter const):
(WebCore::Display::Box::VerticalMargin::before const): Deleted.
(WebCore::Display::Box::VerticalMargin::after const): Deleted.
(WebCore::Display::Box::VerticalMargin::isCollapsedThrough const): Deleted.
(WebCore::Display::Box::VerticalMargin::nonCollapsedValues const): Deleted.
(WebCore::Display::Box::VerticalMargin::collapsedValues const): Deleted.
(WebCore::Display::Box::VerticalMargin::hasCollapsedValues const): Deleted.
(WebCore::Display::Box::VerticalMargin::setCollapsedValues): Deleted.
(WebCore::Display::Box::hasCollapsedThroughMargin const): Deleted.
(WebCore::Display::Box::setHasValidVerticalNonCollapsedMargin): Deleted.
(WebCore::Display::Box::VerticalMargin::VerticalMargin): Deleted.
(WebCore::Display::Box::VerticalMargin::NonCollapsedValues::NonCollapsedValues): Deleted.
(WebCore::Display::Box::VerticalMargin::CollapsedValues::CollapsedValues): Deleted.
(WebCore::Display::Box::nonCollapsedMarginBefore const): Deleted.
(WebCore::Display::Box::nonCollapsedMarginAfter const): Deleted.
- layout/floats/FloatingContext.cpp:
(WebCore::Layout::FloatingContext::verticalPositionWithClearance const):
- layout/inlineformatting/InlineFormattingContext.cpp:
(WebCore::Layout::InlineFormattingContext::layoutInFlowContent):
(WebCore::Layout::InlineFormattingContext::computeHeightAndMargin):
- layout/tableformatting/TableFormattingContext.cpp:
(WebCore::Layout::TableFormattingContext::setUsedGeometryForRows):
(WebCore::Layout::TableFormattingContext::setUsedGeometryForSections):
(WebCore::Layout::TableFormattingContext::layoutCell):
- 12:21 PM Changeset in webkit [263942] by
-
- 12 edits in trunk
[Cocoa] Remove unconditional features from FeatureDefines.xcconfig, making sure they are covered in PlatformEnableCocoa.h
https://bugs.webkit.org/show_bug.cgi?id=212418
Reviewed by Sam Weinig.
Source/JavaScriptCore:
- Configurations/FeatureDefines.xcconfig: Removed features that are either unconditionally not enabled,
or unconditionally enabled. Double checked that all the enabled ones are either in PlatformEnable.h or
PlatformEnableCocoa.h.
Source/WebCore:
- Configurations/FeatureDefines.xcconfig: Removed features that are either unconditionally not enabled,
or unconditionally enabled. Double checked that all the enabled ones are either in PlatformEnable.h or
PlatformEnableCocoa.h.
Source/WebCore/PAL:
- Configurations/FeatureDefines.xcconfig: Removed features that are either unconditionally not enabled,
or unconditionally enabled. Double checked that all the enabled ones are either in PlatformEnable.h or
PlatformEnableCocoa.h.
Source/WebKit:
- Configurations/FeatureDefines.xcconfig: Removed features that are either unconditionally not enabled,
or unconditionally enabled. Double checked that all the enabled ones are either in PlatformEnable.h or
PlatformEnableCocoa.h.
Source/WebKitLegacy/mac:
- Configurations/FeatureDefines.xcconfig: Removed features that are either unconditionally not enabled,
or unconditionally enabled. Double checked that all the enabled ones are either in PlatformEnable.h or
PlatformEnableCocoa.h.
Tools:
- TestWebKitAPI/Configurations/FeatureDefines.xcconfig: Removed features that are either unconditionally not enabled,
or unconditionally enabled. Double checked that all the enabled ones are either in PlatformEnable.h or
PlatformEnableCocoa.h.
- 12:20 PM Changeset in webkit [263941] by
-
- 45 edits1 delete in trunk
Part 1 of SimpleColor and SRGBA<uint8_t> are essentially the same - let's converge them
https://bugs.webkit.org/show_bug.cgi?id=213948
Reviewed by Darin Adler.
Source/WebCore:
Begin converging SimpleColor and SRGBA<uint8_t>, starting with removing usages that
were getting SimpleColors to access or operate on the color's components.
- Replace toSRGBASimpleColorLossy() with toSRGBALossy<uint8_t>
- Replace toSRGBALossy() with toSRGBALossy<float>().
- Remove direct color component accessors from SimpleColor.
- Add new ARGB type to support explicit conversion from packed ARGB bits to a color type.
- Update premulitplication functions to operate on SRGBA<uint8_t> and rename them to match existing function operating on SRGBA<float>.
- Sources.txt:
- WebCore.xcodeproj/project.pbxproj:
Removes SimpleColor.cpp
- accessibility/AccessibilityNodeObject.cpp:
(WebCore::AccessibilityNodeObject::colorValue const):
Replace toSRGBASimpleColorLossy() with toSRGBALossy<uint8_t>.
- accessibility/atk/WebKitAccessibleInterfaceText.cpp:
(getAttributeSetForAccessibilityObject):
Replace toSRGBASimpleColorLossy() with toSRGBALossy<uint8_t>.
- accessibility/isolatedtree/AXIsolatedObject.cpp:
(WebCore::AXIsolatedObject::colorValue const):
Replace toSRGBASimpleColorLossy() with toSRGBALossy<uint8_t>.
- css/DeprecatedCSSOMRGBColor.h:
Replace toSRGBASimpleColorLossy() with toSRGBALossy<uint8_t>.
- editing/cocoa/DataDetection.mm:
(WebCore::DataDetection::detectContentInRange):
Replace toSRGBALossy() with toSRGBALossy<float>().
- platform/graphics/Color.cpp:
(WebCore::Color::lightened const):
(WebCore::Color::darkened const):
(WebCore::Color::lightness const):
(WebCore::Color::luminance const):
(WebCore::Color::semanticColor const):
(WebCore::Color::toSRGBASimpleColorLossy const): Deleted.
(WebCore::Color::toSRGBALossy const): Deleted.
- platform/graphics/Color.h:
Remove toSRGBASimpleColorLossy() and templatize toSRGBALossy() to support both
lossy conversion to byte based components as well.
- platform/graphics/ColorBlending.cpp:
(WebCore::blendSourceOver):
(WebCore::blendWithWhite):
Replace toSRGBASimpleColorLossy() with toSRGBALossy<uint8_t>.
(WebCore::blend):
(WebCore::blendWithoutPremultiply):
Replace toSRGBASimpleColorLossy() with toSRGBALossy<uint8_t>. Adopt new premultipliedCeiling
and unpremultiplied functions that operate on SRGBA<uint8_t> (rather than SimpleColor) and
clamping conversion to SRGBA<uint8_t> via convertToComponentBytes<SRGBA>. premultipliedCeiling
now works correctly for 0 alpha, so no workaround is needed anymore.
- platform/graphics/ColorTypes.h:
(WebCore::asARGB):
(WebCore::asSRGBA):
Add new type of color struct representing a format packing. The initial one is ARGB to support
the few places using SimpleColor to manipulate ARGB pixel data.
- platform/graphics/ColorUtilities.cpp:
(WebCore::unpremultiplied):
(WebCore::premultipliedFlooring):
(WebCore::premultipliedCeiling):
(WebCore::unpremultipliedComponentByte):
- platform/graphics/ColorUtilities.h:
(WebCore::convertToComponentBytes):
(WebCore::convertToComponentFloats):
Move remaining premultiplication functions here (from SimpleColor.h/cpp) and reimplement
to operate on SRGBA<uint8_t>. Also slightly rename, premultiply -> premultiplied, to
match existing premultiplied functions. While moving them, also fix the premultiplied
functions to better support 0 alpha to avoid workarounds in callers.
Additional, this adds convertToComponentBytes/convertToComponentFloats helper functions
which convert to/from uint8_t/float based color types, and support creating color types
from values potentially outside of their supported value range. Right now, that value
range is always considerer to be 0-255 for uint8_t and 0-1 for float, but that can be
chnaged in the future by allowing color type structs to indicate their supported ranges.
- platform/graphics/ExtendedColor.cpp:
(WebCore::ExtendedColor::toSRGBAFloatComponentsLossy const):
(WebCore::ExtendedColor::toSRGBALossy const): Deleted.
- platform/graphics/ExtendedColor.h:
(WebCore::ExtendedColor::toSRGBALossy const):
Rename existing toSRGBALossy() to toSRGBAFloatComponentsLossy() and replace templatize
toSRGBALossy(), with the uint8_t case just down converting via new convertToComponentBytes.
- platform/graphics/ImageBackingStore.h:
(WebCore::ImageBackingStore::fillRect):
(WebCore::ImageBackingStore::setPixel):
(WebCore::ImageBackingStore::blendPixel):
(WebCore::ImageBackingStore::pixelValue const):
Replace usage of SimpleColor with new ARGB type for conversion from ARGB to SRGBA<uint8_t>.
Update to use new premulitplication functions. Also, update the interface to take pixel
components as uint8_t, rather than unsigned, as that is what the callers are passing anyway,
and it avoids us having to clamp/cast manually all over the case.
- platform/graphics/SimpleColor.cpp: Removed.
- platform/graphics/SimpleColor.h:
(WebCore::SimpleColor::alphaComponent const):
(WebCore::SimpleColor::asSRGBA const):
(WebCore::SimpleColor::redComponent const):
(WebCore::SimpleColor::greenComponent const):
(WebCore::SimpleColor::blueComponent const):
(WebCore::operator==):
(WebCore::makeSimpleColor):
Simplify interface, making red, green and blue component accessors private and having
callers all use SRGBA directly. Remove premultiplication functions in favor of new ones
in ColorUtilities.
- platform/graphics/ca/cocoa/PlatformCAAnimationCocoa.mm:
(WebCore::PlatformCAAnimationCocoa::setFromValue):
(WebCore::PlatformCAAnimationCocoa::setToValue):
(WebCore::PlatformCAAnimationCocoa::setValues):
Replace toSRGBASimpleColorLossy() with toSRGBALossy<uint8_t>.
- platform/graphics/ca/win/PlatformCAAnimationWin.cpp:
(PlatformCAAnimationWin::setFromValue):
(PlatformCAAnimationWin::setToValue):
(PlatformCAAnimationWin::setValues):
Replace toSRGBASimpleColorLossy() with toSRGBALossy<uint8_t>.
- platform/graphics/cairo/CairoUtilities.cpp:
(WebCore::setSourceRGBAFromColor):
Replace toSRGBALossy() with toSRGBALossy<float>().
- platform/graphics/cairo/GradientCairo.cpp:
(WebCore::addColorStopRGBA):
(WebCore::setCornerColorRGBA):
(WebCore::interpolateColorStop):
Replace toSRGBALossy() with toSRGBALossy<float>().
- platform/graphics/cairo/ImageBufferCairoImageSurfaceBackend.cpp:
(WebCore::ImageBufferCairoImageSurfaceBackend::platformTransformColorSpace):
Replace usage of SimpleColor with new ARGB type for conversion from ARGB to SRGBA<uint8_t>.
Update to use new premulitplication functions.
- platform/graphics/cairo/NativeImageCairo.cpp:
(WebCore::nativeImageSinglePixelSolidColor):
Replace usage of SimpleColor with new ARGB type for conversion from ARGB to SRGBA<uint8_t>.
Update to use new premulitplication functions.
- platform/graphics/cpu/arm/filters/FELightingNEON.h:
(WebCore::FELighting::platformApplyNeon):
Replace toSRGBASimpleColorLossy() with toSRGBALossy<uint8_t>.
- platform/graphics/filters/FELighting.cpp:
(WebCore::FELighting::drawLighting):
Replace toSRGBALossy() with toSRGBALossy<float>().
- platform/graphics/filters/FilterOperations.cpp:
(WebCore::FilterOperations::transformColor const):
(WebCore::FilterOperations::inverseTransformColor const):
Replace toSRGBALossy() with toSRGBALossy<float>().
- platform/graphics/gtk/ColorGtk.cpp:
(WebCore::Color::operator GdkRGBA const):
Replace toSRGBALossy() with toSRGBALossy<float>().
- platform/graphics/texmap/TextureMapperGL.cpp:
(WebCore::TextureMapperGL::drawBorder):
(WebCore::TextureMapperGL::drawNumber):
(WebCore::prepareFilterProgram):
(WebCore::TextureMapperGL::drawSolidColor):
(WebCore::TextureMapperGL::clearColor):
Replace toSRGBALossy() with toSRGBALossy<float>().
- platform/graphics/win/ColorDirect2D.cpp:
(WebCore::Color::operator D2D1_COLOR_F const):
(WebCore::Color::operator D2D1_VECTOR_4F const):
Replace toSRGBALossy() with toSRGBALossy<float>().
- platform/graphics/win/GradientDirect2D.cpp:
(WebCore::Gradient::generateGradient):
Replace toSRGBALossy() with toSRGBALossy<float>().
- platform/graphics/win/GraphicsContextCGWin.cpp:
(WebCore::GraphicsContext::drawDotsForDocumentMarker):
Replace direct usage component access from SimpleColor with usage of accessed SRGBA<uint8_t>.
- platform/graphics/win/GraphicsContextDirect2D.cpp:
(WebCore::GraphicsContext::colorWithGlobalAlpha const):
Replace toSRGBALossy() with toSRGBALossy<float>().
- rendering/RenderTheme.cpp:
(WebCore::RenderTheme::disabledTextColor const):
Replace toSRGBALossy() with toSRGBALossy<float>().
- rendering/RenderThemeIOS.mm:
(WebCore::shouldUseConvexGradient):
Replace toSRGBALossy() with toSRGBALossy<float>().
- rendering/TextPaintStyle.cpp:
(WebCore::textColorIsLegibleAgainstBackgroundColor):
Replace toSRGBALossy() with toSRGBALossy<float>().
- svg/properties/SVGAnimationAdditiveValueFunctionImpl.h:
(WebCore::SVGAnimationColorFunction::animate):
Replace toSRGBASimpleColorLossy() with toSRGBALossy<uint8_t>.
Source/WebKit:
- UIProcess/API/wpe/WebKitColor.cpp:
(webkitColorFillFromWebCoreColor):
- UIProcess/gtk/ViewGestureControllerGtk.cpp:
(WebKit::ViewGestureController::beginSwipeGesture):
- WebProcess/WebPage/RemoteLayerTree/PlatformCAAnimationRemote.mm:
(WebKit::animationValueFromKeyframeValue):
- WebProcess/WebPage/WebFrame.cpp:
(WebKit::WebFrame::getDocumentBackgroundColor):
Adapt to use templatized toSRGBALossy<float/uint8_t>.
Tools:
- TestWebKitAPI/Tests/WebCore/ColorTests.cpp:
(TestWebKitAPI::TEST):
- TestWebKitAPI/Tests/WebCore/ExtendedColorTests.cpp:
(TestWebKitAPI::TEST):
Adapt tests to use templatized toSRGBALossy<float/uint8_t>.
- 12:17 PM Changeset in webkit [263940] by
-
- 6 edits in trunk/Source/WebCore
[LFC] Remove redundant Display::Box::HorizontalMargin c'tors
https://bugs.webkit.org/show_bug.cgi?id=213957
Reviewed by Antti Koivisto.
Let's just have a very simple Display::Box:HorizontalMargin struct.
- layout/FormattingContext.cpp:
(WebCore::Layout::FormattingContext::computeOutOfFlowHorizontalGeometry):
- layout/blockformatting/BlockFormattingContext.cpp:
(WebCore::Layout::BlockFormattingContext::computeWidthAndMargin):
- layout/blockformatting/tablewrapper/TableWrapperBlockFormattingContext.cpp:
(WebCore::Layout::TableWrapperBlockFormattingContext::computeWidthAndMarginForTableBox):
- layout/displaytree/DisplayBox.h:
(WebCore::Display::Box::HorizontalMargin::HorizontalMargin): Deleted.
- layout/inlineformatting/InlineFormattingContext.cpp:
(WebCore::Layout::InlineFormattingContext::computeWidthAndMargin):
- 11:56 AM Changeset in webkit [263939] by
-
- 4 edits in trunk
[Flatpak SDK] Require OpenXR 1.0.9
https://bugs.webkit.org/show_bug.cgi?id=213930
Reviewed by Philippe Normand.
.:
Require version 1.0.9 because is the first one that include EGL support for Monado.
- Source/cmake/OptionsWPE.cmake:
Tools:
- buildstream/elements/sdk/openxr.bst:
- 11:37 AM Changeset in webkit [263938] by
-
- 16 edits in trunk/Source/WebCore
[LFC] Do not cache computed horizontal margin values
https://bugs.webkit.org/show_bug.cgi?id=213956
Reviewed by Antti Koivisto.
Outside of the width + margin computation, computed horizontal margin values are only required by
float positioning. Let's just re-computed them on demand (which is basically just a style.marginStart/end resolve).
- layout/FormattingContext.cpp:
(WebCore::Layout::FormattingContext::computeOutOfFlowHorizontalGeometry):
- layout/FormattingContextGeometry.cpp:
(WebCore::Layout::FormattingContext::Geometry::outOfFlowNonReplacedHorizontalGeometry):
(WebCore::Layout::FormattingContext::Geometry::outOfFlowReplacedHorizontalGeometry):
(WebCore::Layout::FormattingContext::Geometry::floatingNonReplacedWidthAndMargin):
(WebCore::Layout::FormattingContext::Geometry::inlineReplacedWidthAndMargin):
- layout/LayoutContext.cpp:
(WebCore::Layout::LayoutContext::layout):
- layout/LayoutUnits.h:
- layout/Verification.cpp:
(WebCore::Layout::outputMismatchingBlockBoxInformationIfNeeded):
- layout/blockformatting/BlockFormattingContext.cpp:
(WebCore::Layout::BlockFormattingContext::computePositionToAvoidFloats):
(WebCore::Layout::BlockFormattingContext::computeWidthAndMargin):
- layout/blockformatting/BlockFormattingContextGeometry.cpp:
(WebCore::Layout::BlockFormattingContext::Geometry::inFlowNonReplacedWidthAndMargin):
(WebCore::Layout::BlockFormattingContext::Geometry::inFlowReplacedWidthAndMargin):
- layout/blockformatting/tablewrapper/TableWrapperBlockFormattingContext.cpp:
(WebCore::Layout::TableWrapperBlockFormattingContext::computeWidthAndMarginForTableBox):
- layout/displaytree/DisplayBox.cpp:
(WebCore::Display::Box::Box):
- layout/displaytree/DisplayBox.h:
(WebCore::Display::Box::setHasValidVerticalNonCollapsedMargin):
(WebCore::Display::Box::setHasValidHorizontalComputedMargin): Deleted.
(WebCore::Display::Box::ComputedHorizontalMargin::ComputedHorizontalMargin): Deleted.
(WebCore::Display::Box::setHorizontalComputedMargin): Deleted.
(WebCore::Display::Box::computedMarginStart const): Deleted.
(WebCore::Display::Box::computedMarginEnd const): Deleted.
- layout/floats/FloatingContext.cpp:
(WebCore::Layout::computedHorizontalMargin):
(WebCore::Layout::FloatingContext::positionForFloat const):
(WebCore::Layout::FloatingContext::positionForNonFloatingFloatAvoider const):
- layout/floats/FloatingContext.h:
- layout/inlineformatting/InlineFormattingContext.cpp:
(WebCore::Layout::InlineFormattingContext::computeHorizontalMargin):
(WebCore::Layout::InlineFormattingContext::computeWidthAndMargin):
- layout/inlineformatting/InlineFormattingContextGeometry.cpp:
(WebCore::Layout::InlineFormattingContext::Geometry::inlineBlockWidthAndMargin):
- layout/tableformatting/TableFormattingContext.cpp:
(WebCore::Layout::TableFormattingContext::setUsedGeometryForRows):
(WebCore::Layout::TableFormattingContext::setUsedGeometryForSections):
(WebCore::Layout::TableFormattingContext::layoutCell):
- 10:18 AM Changeset in webkit [263937] by
-
- 2 edits in trunk/Tools
Make generate-unified-sources.sh not depend on features being listed in FEATURE_DEFINES environment variable
https://bugs.webkit.org/show_bug.cgi?id=212420
- Scripts/generate-unified-sources.sh: Removed --feature-flags from the
arguments passed to generate-unified-source-bundles.rb. Also removed many
unneeded quotes.
- 9:43 AM Changeset in webkit [263936] by
-
- 2 edits in trunk/WebDriverTests
[WebDriver][WPE][GTK] Garden post update failures
Unreviewed test gardening.
- TestExpectations.json:
- 9:41 AM Changeset in webkit [263935] by
-
- 16 edits1 add in trunk
Make generate-unified-sources.sh not depend on features being listed in FEATURE_DEFINES environment variable
https://bugs.webkit.org/show_bug.cgi?id=212420
Reviewed by Don Olmstead.
.:
- Source/cmake/WebKitCommon.cmake: Removed call to SET_CONFIGURATION_FOR_UNIFIED_SOURCE_LISTS.
- Source/cmake/WebKitFeatures.cmake: Removed SET_CONFIGURATION_FOR_UNIFIED_SOURCE_LISTS.
- Source/cmake/WebKitMacros.cmake: Removed use of "--feature-flags" when invoking the
generate-unified-source-bundles.rb, which also means no longer depending on the
UNIFIED_SOURCE_LIST_ENABLED_FEATURES variable, which was set by the
SET_CONFIGURATION_FOR_UNIFIED_SOURCE_LISTS function.
Source/JavaScriptCore:
- Scripts/generate-unified-sources.sh: Removed many unneeded quote marks from the
invocation of generate-unified-source-bundles.rb.
Source/WebCore:
- Scripts/generate-unified-sources.sh: Added code to include sources from
SourcesCocoaInternalSDK.txt, only if USE_INTERNAL_SDK is YES. Removed --feature-flags
argument from invocation of generate-unified-source-bundles.rb. Removed many
unnecessary quotes.
- SourcesCocoa.txt: Removed the names of the files from the internal SDK.
- SourcesCocoaInternalSDK.txt: Added. Moved those names here.
- html/MediaElementSession.cpp: Added some includes, apparently needed because of different
groupings of unified sources.
Source/WebKit:
- Scripts/generate-unified-sources.sh: Removed --feature-flags from the
arguments passed to generate-unified-source-bundles.rb. Also removed many
unneeded quotes.
Source/WebKitLegacy:
- scripts/generate-unified-sources.sh: Removed --feature-flags from the
arguments passed to generate-unified-source-bundles.rb. Also removed many
unneeded quotes.
Source/WTF:
- Scripts/generate-unified-source-bundles.rb: Removed the --feature-flags
command line option along with the code that implements it.
- 9:33 AM Changeset in webkit [263934] by
-
- 6 edits in trunk/Source/WebCore
[LFC] Used dedicated paint margin structures
https://bugs.webkit.org/show_bug.cgi?id=213955
Reviewed by Antti Koivisto.
This is in preparation for having only final margin values in Display::Box.
- layout/FormattingContext.cpp:
(WebCore::Layout::FormattingContext::computeOutOfFlowHorizontalGeometry):
(WebCore::Layout::FormattingContext::computeOutOfFlowVerticalGeometry):
- layout/blockformatting/BlockFormattingContext.cpp:
(WebCore::Layout::BlockFormattingContext::computeWidthAndMargin):
- layout/displaytree/DisplayBox.h:
(WebCore::Display::Box::VerticalMargin::before const):
(WebCore::Display::Box::VerticalMargin::after const):
(WebCore::Display::Box::VerticalMargin::isCollapsedThrough const):
(WebCore::Display::Box::VerticalMargin::nonCollapsedValues const):
(WebCore::Display::Box::VerticalMargin::collapsedValues const):
(WebCore::Display::Box::VerticalMargin::hasCollapsedValues const):
(WebCore::Display::Box::VerticalMargin::setCollapsedValues):
(WebCore::Display::Box::VerticalMargin::VerticalMargin):
(WebCore::Display::Box::VerticalMargin::NonCollapsedValues::NonCollapsedValues):
(WebCore::Display::Box::VerticalMargin::CollapsedValues::CollapsedValues):
(WebCore::Display::Box::ComputedHorizontalMargin::ComputedHorizontalMargin):
(WebCore::Display::Box::HorizontalMargin::HorizontalMargin):
(WebCore::Display::Box::setHorizontalMargin):
(WebCore::Display::Box::setVerticalMargin):
(WebCore::Display::Box::setHorizontalComputedMargin):
(WebCore::Display::Box::verticalMargin const):
(WebCore::Display::Box::horizontalMargin const):
- layout/floats/FloatingState.h:
(WebCore::Layout::FloatingState::FloatItem::horizontalMargin const):
- layout/inlineformatting/InlineFormattingContext.cpp:
(WebCore::Layout::InlineFormattingContext::computeWidthAndMargin):
(WebCore::Layout::InlineFormattingContext::computeHeightAndMargin):
- 9:04 AM Changeset in webkit [263933] by
-
- 2 edits in trunk/Tools
[WebDriver][WPE] Allow running Selenium tests with the WPE WebDriver
https://bugs.webkit.org/show_bug.cgi?id=213500
Reviewed by Carlos Garcia Campos.
Add missing method used to discover the selenium name of
the WPE webdriver.
Proper gardening/triaging of failures will come in following commits.
- Scripts/webkitpy/webdriver_tests/webdriver_driver_wpe.py:
(WebDriverWPE.selenium_name):
- 6:32 AM Changeset in webkit [263932] by
-
- 12 edits in trunk
Update comment in FeatureDefines.xcconfig since PlatformEnableCocoa.h should be used instead
https://bugs.webkit.org/show_bug.cgi?id=213952
Reviewed by Sam Weinig.
Source/JavaScriptCore:
- Configurations/FeatureDefines.xcconfig: Updated comment.
Source/WebCore:
- Configurations/FeatureDefines.xcconfig: Updated comment.
Source/WebCore/PAL:
- Configurations/FeatureDefines.xcconfig: Updated comment.
Source/WebKit:
- Configurations/FeatureDefines.xcconfig: Updated comment.
Source/WebKitLegacy/mac:
- Configurations/FeatureDefines.xcconfig: Updated comment.
Tools:
- TestWebKitAPI/Configurations/FeatureDefines.xcconfig: Updated comment.
- 2:18 AM Changeset in webkit [263931] by
-
- 13 edits in trunk/Source
Enable VTB required low latency code path
https://bugs.webkit.org/show_bug.cgi?id=210609
<rdar://problem/61890332>
Reviewed by Darin Adler.
Source/ThirdParty/libwebrtc:
Enable low latency mode for latest MacOS and iOS versions.
Add a boolean controlling whether to opt in that code path.
Export the setter to that boolean so that it can be controlled by a WebCore runtime flag.
- Configurations/libwebrtc.iOS.exp:
- Configurations/libwebrtc.iOSsim.exp:
- Configurations/libwebrtc.mac.exp:
- Source/webrtc/sdk/WebKit/VideoProcessingSoftLink.h:
- Source/webrtc/sdk/WebKit/WebKitEncoder.mm:
(webrtc::setH264LowLatencyEncoderEnabled):
(webrtc::isH264LowLatencyEncoderEnabled):
- Source/webrtc/sdk/WebKit/WebKitUtilities.h:
- Source/webrtc/sdk/objc/components/video_codec/RTCVideoEncoderH264.mm:
(-[RTCVideoEncoderH264 resetCompressionSessionWithPixelFormat:]):
Source/WebCore:
Add new runtime flag for the low latency code path.
Call webrtc newly introduced setter whenever creating a new encoder factory.
Manually tested.
- page/RuntimeEnabledFeatures.h:
(WebCore::RuntimeEnabledFeatures::webRTCH264LowLatencyEncoderEnabled const):
(WebCore::RuntimeEnabledFeatures::setWebRTCH264LowLatencyEncoderEnabled):
- platform/mediastream/libwebrtc/LibWebRTCProviderCocoa.cpp:
(WebCore::LibWebRTCProviderCocoa::createEncoderFactory):
Source/WebKit:
Add a runtime flag to enable/disable the low latency code path.
- Shared/WebPreferences.yaml:
Jul 3, 2020:
- 11:42 PM Changeset in webkit [263930] by
-
- 3 edits1 add in trunk
[JSC] Promise should check whether a user-provided function is set by using
@isUndefinedOrNull
https://bugs.webkit.org/show_bug.cgi?id=213951
Reviewed by Ross Kirsling.
JSTests:
- stress/masquerader-as-function-in-promise.js: Added.
Source/JavaScriptCore:
If a user-provided function is masquerade-as-undefined value,
if (!xxx.@onRejected)returns wrong
value since this function object is considered as undefined in this context. We should use@isUndefinedOrNull
here instead since this if-branch is checking whether this property is null/undefined actually.
Andif (@isUndefinedOrNull(...))is efficient since we havejundefined_or_null/jnundefined_or_nullbytecodes.
- builtins/PromiseOperations.js:
(globalPrivate.promiseReactionJob):
- 11:38 PM Changeset in webkit [263929] by
-
- 2 edits in trunk/Tools
[macOS] Tests that show system menu popups may fail when run concurrently
https://bugs.webkit.org/show_bug.cgi?id=213950
<rdar://problem/64551566>
Reviewed by Darin Adler.
Make it possible to run tests that present system modal popups simultaneously, without the possibility of these
tests interfering with each other. Examples include context menus, select menus, and the DOM paste popup. To do
this, we swizzle out codepaths for showing and dimissing popups, and instead dispatch notifications and the
appropriate NSMenuDelegate methods ourselves.
This change has the side effect of fixing several layout tests that are currently timing out only on arm64e
macOS, due to reasons detailed in <rdar://problem/65084503>.
- WebKitTestRunner/mac/TestControllerMac.mm:
(WTR::setSwizzledPopUpMenu):
(WTR::swizzledPopUpContextMenu):
(WTR::swizzledPopUpMenu):
(WTR::swizzledCancelTracking):
(WTR::TestController::platformInitialize):
- 10:15 PM Changeset in webkit [263928] by
-
- 25 edits3 copies23 adds in trunk
MediaRecorder should support peer connection remote video tracks
https://bugs.webkit.org/show_bug.cgi?id=213924
Reviewed by Darin Adler.
LayoutTests/imported/w3c:
Resync to WPT ToT.
- resources/resource-files.json:
- web-platform-tests/mediacapture-record/BlobEvent-constructor.html:
- web-platform-tests/mediacapture-record/MediaRecorder-bitrate.https-expected.txt: Added.
- web-platform-tests/mediacapture-record/MediaRecorder-bitrate.https.html: Added.
- web-platform-tests/mediacapture-record/MediaRecorder-creation.https-expected.txt: Added.
- web-platform-tests/mediacapture-record/MediaRecorder-creation.https.html: Added.
- web-platform-tests/mediacapture-record/MediaRecorder-detached-context-expected.txt: Added.
- web-platform-tests/mediacapture-record/MediaRecorder-detached-context.html: Added.
- web-platform-tests/mediacapture-record/MediaRecorder-disabled-tracks.https-expected.txt: Added.
- web-platform-tests/mediacapture-record/MediaRecorder-disabled-tracks.https.html: Added.
- web-platform-tests/mediacapture-record/MediaRecorder-error.html:
- web-platform-tests/mediacapture-record/MediaRecorder-events-and-exceptions-expected.txt: Added.
- web-platform-tests/mediacapture-record/MediaRecorder-events-and-exceptions.html: Added.
- web-platform-tests/mediacapture-record/MediaRecorder-mimetype-expected.txt: Added.
- web-platform-tests/mediacapture-record/MediaRecorder-mimetype.html: Added.
- web-platform-tests/mediacapture-record/MediaRecorder-no-sink.https-expected.txt: Added.
- web-platform-tests/mediacapture-record/MediaRecorder-no-sink.https.html: Added.
- web-platform-tests/mediacapture-record/MediaRecorder-pause-resume-expected.txt: Added.
- web-platform-tests/mediacapture-record/MediaRecorder-pause-resume.html: Added.
- web-platform-tests/mediacapture-record/MediaRecorder-peerconnection.https-expected.txt: Added.
- web-platform-tests/mediacapture-record/MediaRecorder-peerconnection.https.html: Added.
- web-platform-tests/mediacapture-record/MediaRecorder-stop-expected.txt:
- web-platform-tests/mediacapture-record/MediaRecorder-stop.html:
- web-platform-tests/mediacapture-record/idlharness.window-expected.txt:
- web-platform-tests/mediacapture-record/idlharness.window.js:
- web-platform-tests/mediacapture-record/passthrough/MediaRecorder-passthrough.https-expected.txt: Added.
- web-platform-tests/mediacapture-record/passthrough/MediaRecorder-passthrough.https.html: Added.
- web-platform-tests/mediacapture-record/passthrough/w3c-import.log: Copied from LayoutTests/imported/w3c/web-platform-tests/mediacapture-record/w3c-import.log.
- web-platform-tests/mediacapture-record/support/w3c-import.log: Copied from LayoutTests/imported/w3c/web-platform-tests/mediacapture-record/w3c-import.log.
- web-platform-tests/mediacapture-record/utils/peerconnection.js: Added.
(async exchangeOfferAnswer):
(setTransceiverCodecPreference):
(async startConnection):
(async waitForReceivedFrames):
(async waitForReceivedCodec):
- web-platform-tests/mediacapture-record/utils/w3c-import.log: Copied from LayoutTests/imported/w3c/web-platform-tests/mediacapture-record/w3c-import.log.
- web-platform-tests/mediacapture-record/w3c-import.log:
Source/WebCore:
Previsouly we were checking whether width and height are given for a video track.
This is not always available, for instance for remote tracks.
And this can change over time.
Instead, pass a simple boolean (whether there is a video track or not).
Minor simplifications to stop storing unneeded members.
Tests: imported/w3c/web-platform-tests/mediacapture-record/MediaRecorder-bitrate.https.html
imported/w3c/web-platform-tests/mediacapture-record/MediaRecorder-creation.https.html
imported/w3c/web-platform-tests/mediacapture-record/MediaRecorder-detached-context.html
imported/w3c/web-platform-tests/mediacapture-record/MediaRecorder-disabled-tracks.https.html
imported/w3c/web-platform-tests/mediacapture-record/MediaRecorder-events-and-exceptions.html
imported/w3c/web-platform-tests/mediacapture-record/MediaRecorder-mimetype.html
imported/w3c/web-platform-tests/mediacapture-record/MediaRecorder-no-sink.https.html
imported/w3c/web-platform-tests/mediacapture-record/MediaRecorder-pause-resume.html
imported/w3c/web-platform-tests/mediacapture-record/MediaRecorder-peerconnection.https.html
imported/w3c/web-platform-tests/mediacapture-record/passthrough/MediaRecorder-passthrough.https.html
- platform/mediarecorder/MediaRecorderPrivateAVFImpl.cpp:
(WebCore::MediaRecorderPrivateAVFImpl::create):
(WebCore::MediaRecorderPrivateAVFImpl::MediaRecorderPrivateAVFImpl):
- platform/mediarecorder/MediaRecorderPrivateAVFImpl.h:
- platform/mediarecorder/cocoa/MediaRecorderPrivateWriterCocoa.h:
- platform/mediarecorder/cocoa/MediaRecorderPrivateWriterCocoa.mm:
(WebCore::MediaRecorderPrivateWriter::create):
Source/WebKit:
Pass a boolean instead of width/height for the video track.
- GPUProcess/webrtc/RemoteMediaRecorder.cpp:
(WebKit::RemoteMediaRecorder::create):
- GPUProcess/webrtc/RemoteMediaRecorder.h:
- GPUProcess/webrtc/RemoteMediaRecorderManager.cpp:
(WebKit::RemoteMediaRecorderManager::createRecorder):
- GPUProcess/webrtc/RemoteMediaRecorderManager.h:
- GPUProcess/webrtc/RemoteMediaRecorderManager.messages.in:
- WebProcess/GPU/webrtc/MediaRecorderPrivate.cpp:
(WebKit::MediaRecorderPrivate::startRecording):
- WebProcess/GPU/webrtc/MediaRecorderPrivate.h:
LayoutTests:
- TestExpectations:
- tests-options.json:
- 9:56 PM Changeset in webkit [263927] by
-
- 2 edits in trunk/Source/WebCore/PAL
Fix a build error by forward declaring _NSHTTPAlternativeServicesStorage
https://bugs.webkit.org/show_bug.cgi?id=213911
Patch by Saagar Jha <saagar@saagarjha.com> on 2020-07-03
Reviewed by Darin Adler.
The public 10.16 SDK does not define _NSHTTPAlternativeServicesStorage, and the interface
declaration comes after it is used. Forward declare it to fix the error.
- pal/spi/cf/CFNetworkSPI.h: Forward declare _NSHTTPAlternativeServicesStorage.
- 7:33 PM Changeset in webkit [263926] by
-
- 3 edits in trunk/Source/WebCore
WebCore/Sources.txt: Remove all remaining #if (ENABLE_WEBGL and ENABLE_WEBXR)
https://bugs.webkit.org/show_bug.cgi?id=213946
Reviewed by Sam Weinig.
- CMakeLists.txt: Move IDL-related ENABLE_WEBGL and ENABLE_WEBXR items out of if blocks.
Longer term, I think our direction should be to remove most feature conditionals from
CMakeLists.txt too, just like I am doing for Sources.txt. But for now, doing just enough
of this to keep the build working. Someone can tidy this later as desired.
- Sources.txt: Remove all remaining #if, and sort files, getting rid of paragraphing
per directory to just turn this into three sorted lists of filenames (separate ones for
derived sources and generated bindings at the end). The source files themselves have the
appropriate conditionals and soon we will not need to support #if in Sources.txt files.
- 7:07 PM Changeset in webkit [263925] by
-
- 2 edits in trunk/LayoutTests
[GStreamer] Garden media/vp9.html failure
Unreviewed test gardening.
- platform/glib/TestExpectations:
- 6:51 PM Changeset in webkit [263924] by
-
- 4 edits in trunk/LayoutTests
[GTK][WPE] Rebaseline imported dom/idlharness tests after r263903
Unreviewed test gardening.
- platform/glib/imported/w3c/web-platform-tests/html/dom/idlharness.worker-expected.txt:
- platform/gtk/imported/w3c/web-platform-tests/html/dom/idlharness.https-expected.txt:
- platform/wpe/imported/w3c/web-platform-tests/html/dom/idlharness.https-expected.txt:
- 5:42 PM Changeset in webkit [263923] by
-
- 2 edits in trunk/Source/WebCore
[Win] Unreviewed stack-overflow crash fix for r263890
https://bugs.webkit.org/show_bug.cgi?id=213912
<rdar://problem/65069686>
Renamed mimeTypeForExtension to mimeTypeForExtensionFromRegistry.
- platform/win/MIMETypeRegistryWin.cpp:
(WebCore::mimeTypeForExtensionFromRegistry):
(WebCore::MIMETypeRegistry::mimeTypeForExtension):
(WebCore::mimeTypeForExtension): Deleted.
- 5:30 PM Changeset in webkit [263922] by
-
- 2 edits in trunk/Source/WebCore
[Win] Unreviewed build fix for r263913
https://bugs.webkit.org/show_bug.cgi?id=213939
<rdar://problem/65084185>
- CMakeLists.txt: Process encrypted media IDL files even if ENABLE_ENCRYPTED_MEDIA is off.
- 5:16 PM Changeset in webkit [263921] by
-
- 1 copy in tags/Safari-610.1.20.3
Tag Safari-610.1.20.3.
- 5:12 PM Changeset in webkit [263920] by
-
- 2 edits in branches/safari-610.1.20-branch/Source/WebKit
Revert r263781. rdar://problem/65085169
- 5:12 PM Changeset in webkit [263919] by
-
- 3 edits in branches/safari-610.1.20-branch/Source/WebKit
Revert r263793. rdar://problem/65085169
- 4:56 PM Changeset in webkit [263918] by
-
- 4 edits in trunk/Source/WebCore
WebCore/Sources.txt: Remove #if ENABLE_WIRELESS_PLAYBACK_TARGET
https://bugs.webkit.org/show_bug.cgi?id=213945
Reviewed by Sam Weinig.
- CMakeLists.txt: Moved items from the pictureinpicture and remoteplayback module here
from PlatformMac.cmake. Even if these are currently only implemented on Mac, they are
not intrinsically Mac-specific, so they belong here.
- PlatformMac.cmake: Ditto.
- Sources.txt: Remove #if ENABLE_WIRELESS_PLAYBACK_TARGET and sort files in with the others.
The source files themselves have conditionals, we don't need conditionals in Sources.txt,
and we are moving towards removing the capability for #if ENABLE in the Sources.txt file.
- 4:51 PM Changeset in webkit [263917] by
-
- 8 edits in branches/safari-610.1.20-branch/Source
Versioning.
WebKit-610.1.20.3
- 4:43 PM Changeset in webkit [263916] by
-
- 32 edits2 deletes in trunk
Remove support for ENABLE_INPUT_TYPE_DATETIME_INCOMPLETE
https://bugs.webkit.org/show_bug.cgi?id=213932
Reviewed by Darin Adler.
Removes support for non-standard <input type="datetime">, currently being
guarded by the macro ENABLE_INPUT_TYPE_DATETIME_INCOMPLETE. This macro, was
added back in 2013 as a temporary measure to support some engines who shipped
support for <input type="datetime">. It is currently not enabled for any
ports so now seems like as good a time as any to remove it.
.:
- 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:
- Sources.txt:
- WebCore.xcodeproj/project.pbxproj:
- css/html.css:
(#endif):
- html/DateTimeInputType.cpp: Removed.
- html/DateTimeInputType.h: Removed.
- html/InputType.cpp:
(WebCore::createInputTypeFactoryMap):
- page/RuntimeEnabledFeatures.h:
(WebCore::RuntimeEnabledFeatures::inputTypeDateTimeEnabled const): Deleted.
(WebCore::RuntimeEnabledFeatures::setInputTypeDateTimeEnabled): Deleted.
- platform/DateComponents.cpp:
(WebCore::DateComponents::setMillisecondsSinceEpochForDateTimeLocal):
(WebCore::DateComponents::millisecondsSinceEpoch const):
(WebCore::DateComponents::toString const):
(WebCore::DateComponents::fromParsingDateTime): Deleted.
(WebCore::DateComponents::parseDateTime): Deleted.
(WebCore::DateComponents::fromMillisecondsSinceEpochForDateTime): Deleted.
(WebCore::DateComponents::setMillisecondsSinceEpochForDateTime): Deleted.
- platform/DateComponents.h:
- platform/text/PlatformLocale.cpp:
(WebCore::Locale::formatDateTime):
- platform/text/ios/LocalizedDateCache.mm:
(WebCore::LocalizedDateCache::createFormatterForType):
Source/WebCore/PAL:
- Configurations/FeatureDefines.xcconfig:
Source/WebKit:
- Configurations/FeatureDefines.xcconfig:
Source/WebKitLegacy/mac:
- Configurations/FeatureDefines.xcconfig:
Source/WTF:
- wtf/PlatformEnable.h:
Tools:
- Scripts/webkitperl/FeatureList.pm:
- TestWebKitAPI/Configurations/FeatureDefines.xcconfig:
- 4:25 PM Changeset in webkit [263915] by
-
- 3 edits in trunk/Source/WebCore
WebCore/Sources.txt: Remove #if ENABLE_USER_MESSAGE_HANDLERS
https://bugs.webkit.org/show_bug.cgi?id=213944
Reviewed by Sam Weinig.
- CMakeLists.txt: Move IDL-related ENABLE_USER_MESSAGE_HANDLERS items out of if blocks.
Longer term, I think our direction should be to remove most feature conditionals from
CMakeLists.txt too, just like I am doing for Sources.txt. But for now, doing just enough
of this to keep the build working. Someone can tidy this later as desired.
- Sources.txt: Remove #if ENABLE_USER_MESSAGE_HANDLERS and sort files in with the others.
The source files themselves have conditionals, we don't need conditionals in Sources.txt,
and we are moving towards removing the capability for #if ENABLE in the Sources.txt file.
- 4:23 PM Changeset in webkit [263914] by
-
- 2 edits in trunk/Source/WebCore
WebCore/SourcesCocoa.txt: Remove #if ENABLE_APPLE_PAY
https://bugs.webkit.org/show_bug.cgi?id=213943
Reviewed by Sam Weinig.
- SourcesCocoa.txt: Remove #if ENABLE_APPLE_PAY and sort files in with the others.
The source files themselves have conditionals, we don't need conditionals in Sources.txt,
and we are moving towards removing the capability for #if ENABLE in the Sources.txt file.
Also, sort the files and remove the separate per-directory paragraphs, making this closer
to just a sorted list of filenames with less "editorial" content. Also added comments
about the USE_INTERNAL_SDK part of the file.
- 3:17 PM Changeset in webkit [263913] by
-
- 2 edits in trunk/Source/WebCore
WebCore/Sources.txt: Remove #if ENABLE_ENCRYPTED_MEDIA
https://bugs.webkit.org/show_bug.cgi?id=213939
Reviewed by Sam Weinig.
- CMakeLists.txt: Move IDL-related ENABLE_ENCRYPTED_MEDIA items out of if blocks.
Longer term, I think our direction should be to remove most feature conditionals from
CMakeLists.txt too, just like I am doing for Sources.txt. But for now, doing just enough
of this to keep the build working. Someone can tidy this later as desired.
- Sources.txt: Remove #if ENABLE_LEGACY_ENCRYPTED_MEDIA and sort files in with the others.
The IDL files themselves have conditionals, we don't need conditionals in Sources.txt,
and we are moving towards removing the capability for #if ENABLE in the Sources.txt file.
- 2:50 PM Changeset in webkit [263912] by
-
- 3 edits in trunk/Source/WebCore
WebCore/Sources.txt: Remove #if ENABLE_LEGACY_ENCRYPTED_MEDIA
https://bugs.webkit.org/show_bug.cgi?id=213937
Reviewed by Sam Weinig.
- CMakeLists.txt: Move IDL-related ENABLE_LEGACY_ENCRYPTED_MEDIA items out of if blocks.
Longer term, I think our direction should be to remove most feature conditionals from
CMakeLists.txt too, just like I am doing for Sources.txt. But for now, doing just enough
of this to keep the build working. Someone can tidy this later as desired.
- Sources.txt: Remove #if ENABLE_LEGACY_ENCRYPTED_MEDIA and sort files in with the others.
The IDL files themselves have conditionals, we don't need conditionals in Sources.txt,
and we are moving towards removing the capability for #if ENABLE in the Sources.txt file.
- 2:45 PM Changeset in webkit [263911] by
-
- 3 edits in trunk/Source/WebCore
WebCore/Sources.txt: Remove #if ENABLE_QUOTA
https://bugs.webkit.org/show_bug.cgi?id=213941
Reviewed by Sam Weinig.
- CMakeLists.txt: Move IDL-related ENABLE_QUOTA items out of "if (ENABLE_QUOTA)".
Longer term, I think our direction should be to remove most feature conditionals from
CMakeLists.txt too, just like I am doing for Sources.txt. But for now, doing just enough
of this to keep the build working. Someone can tidy this later as desired.
- Sources.txt: Remove #if ENABLE_QUOTA and sort files in with the others.
The source files themselves have conditionals, we don't need conditionals in Sources.txt,
and we are moving towards removing the capability for #if ENABLE in the Sources.txt file.
- 2:43 PM Changeset in webkit [263910] by
-
- 3 edits in trunk/Source/WebCore
WebCore/Sources.txt: Remove #if ENABLE_GAMEPAD
https://bugs.webkit.org/show_bug.cgi?id=213936
Reviewed by Sam Weinig.
- CMakeLists.txt: Move IDL-related ENABLE_GAMEPAD items out of "if (ENABLE_GAMEPAD)".
Longer term, I think our direction should be to remove most feature conditionals from
CMakeLists.txt too, just like I am doing for Sources.txt. But for now, doing just enough
of this to keep the build working. Someone can tidy this later as desired.
- Sources.txt: Remove #if ENABLE_GAMEPAD and sort files in with the others.
The IDL files themselves have conditionals, we don't need conditionals in Sources.txt,
and we are moving towards removing the capability for #if ENABLE in the Sources.txt file.
- 2:40 PM Changeset in webkit [263909] by
-
- 2 edits in trunk/Source/WebCore
WebCore/Sources.txt: Remove #if ENABLE_MEDIA_SOURCE
https://bugs.webkit.org/show_bug.cgi?id=213940
Reviewed by Sam Weinig.
- Sources.txt: Remove #if ENABLE_MEDIA_SOURCE and sort files in with the others.
The IDL files themselves have conditionals, we don't need conditionals in Sources.txt,
and we are moving towards removing the capability for #if ENABLE in the Sources.txt file.
- 2:27 PM Changeset in webkit [263908] by
-
- 2 edits in trunk/Source/WebCore
[WPE] Fix build with clang after r263898
https://bugs.webkit.org/show_bug.cgi?id=213942
Unreviewed build fix.
- loader/appcache/ApplicationCacheManifestParser.cpp:
- 2:16 PM Changeset in webkit [263907] by
-
- 2 edits in trunk/Source/WebCore
WebCore/Sources.txt: Remove #if ENABLE_APPLICATION_MANIFEST
https://bugs.webkit.org/show_bug.cgi?id=213935
Reviewed by Sam Weinig.
- Sources.txt: Remove #if ENABLE_APPLICATION_MANIFEST and sort files in with the others.
The source files themselves have conditionals, we don't need conditionals in Sources.txt,
and we are moving towards removing the capability for #if ENABLE in the Sources.txt file.
- 1:30 PM Changeset in webkit [263906] by
-
- 17 edits in trunk/Source
Add "-Wliteral-conversion" warning to Xcode based builds and fix the issues it finds
https://bugs.webkit.org/show_bug.cgi?id=213931
Reviewed by Darin Adler.
- Configurations/Base.xcconfig:
Add -Wliteral-conversion.
Source/WebCore:
- platform/graphics/BitmapImage.cpp:
(WebCore::BitmapImage::draw):
Fix error due to passing a double when a uint8_t was expected. This probably means
that the debug borders looked wrong for a little bit, but should now be back to
their transparent splendor.
- 1:06 PM Changeset in webkit [263905] by
-
- 7 edits1 add in trunk
[JSC] Add exception checks before and after viewWithUnderlyingString
https://bugs.webkit.org/show_bug.cgi?id=213923
<rdar://problem/65068473>
Reviewed by Sam Weinig.
JSTests:
- stress/exception-checks-before-and-after-viewwithunderlyingstring.js: Added.
(shouldThrow):
Source/JavaScriptCore:
This patch inserts missing exception checks before and after viewWithUnderlyingString.
- jsc.cpp:
(printInternal):
(functionDebug):
- runtime/FunctionConstructor.cpp:
(JSC::constructFunctionSkippingEvalEnabledCheck):
- runtime/JSGlobalObjectFunctions.cpp:
(JSC::globalFuncParseFloat):
- runtime/JSONObject.cpp:
(JSC::JSONProtoFuncParse):
- runtime/StringPrototype.cpp:
(JSC::stringProtoFuncCharAt):
(JSC::stringProtoFuncCharCodeAt):
- 12:59 PM Changeset in webkit [263904] by
-
- 24 edits in trunk/Source
Unreviewed, reverting r263882.
https://bugs.webkit.org/show_bug.cgi?id=213933
Broke the build
Reverted changeset:
"Make _WKWebsiteDataStoreConfiguration SPI for HSTS storage to
replace _WKProcessPoolConfiguration.hstsStorageDirectory"
https://bugs.webkit.org/show_bug.cgi?id=213048
https://trac.webkit.org/changeset/263882
- 11:54 AM Changeset in webkit [263903] by
-
- 48 edits5 copies3 moves29 adds9 deletes in trunk/LayoutTests
Resync web-platform-tests/html/dom & web-platform-tests/html/infrastructure from upstream
https://bugs.webkit.org/show_bug.cgi?id=213908
Reviewed by Youenn Fablet.
LayoutTests/imported/w3c:
Resync web-platform-tests/html/dom & web-platform-tests/html/infrastructure from upstream 145839c323ba1ec8b.
- resources/import-expectations.json:
- resources/resource-files.json:
- web-platform-tests/html/dom/*: Updated.
- web-platform-tests/html/infrastructure/*: Updated.
LayoutTests:
- platform/mac-wk1/imported/w3c/web-platform-tests/html/dom/idlharness.https-expected.txt:
- platform/mac-wk1/imported/w3c/web-platform-tests/html/dom/usvstring-reflection.https-expected.txt:
- platform/mac-wk2/imported/w3c/web-platform-tests/html/dom/idlharness.https-expected.txt:
- 11:32 AM Changeset in webkit [263902] by
-
- 10 edits3 adds in trunk
[LFC][TFC][Quirk] Inflow child box quirk vertical margins should collapse with table cell.
https://bugs.webkit.org/show_bug.cgi?id=213926
Reviewed by Antti Koivisto.
Source/WebCore:
The BFC rootd (table cell box in this case) nornally do not collapse their margins with the first/last inflow child.
However in quirks mode, cell boxes collapse their (non-existing)margins with inflow quirk margins.
<table><tr><td><p>text content</td></tr></table> <- <p> box's top position is at 0px in quirks mode, while at 16px (1em) in strict mode.
This patch ensures that we collapse the quirk margins and take them into account when computing the cell's content height.
Test: fast/layoutformattingcontext/table-quirk-vertical-margin-simple.html
- Sources.txt:
- WebCore.xcodeproj/project.pbxproj:
- layout/blockformatting/BlockFormattingContext.cpp:
(WebCore::Layout::BlockFormattingContext::computeHeightAndMargin):
- layout/blockformatting/BlockFormattingContext.h:
- layout/blockformatting/BlockFormattingContextQuirks.cpp:
(WebCore::Layout::BlockFormattingContext::Quirks::shouldCollapseMarginBeforeWithParentMarginBefore const):
(WebCore::Layout::BlockFormattingContext::Quirks::shouldCollapseMarginAfterWithParentMarginAfter const):
- layout/blockformatting/BlockMarginCollapse.cpp:
(WebCore::Layout::BlockFormattingContext::MarginCollapse::marginBeforeCollapsesWithParentMarginBefore const):
(WebCore::Layout::BlockFormattingContext::MarginCollapse::marginAfterCollapsesWithParentMarginAfter const):
(WebCore::Layout::BlockFormattingContext::MarginCollapse::collapsedVerticalValues):
- layout/tableformatting/TableFormattingContext.h:
(WebCore::Layout::TableFormattingContext::Quirks::Quirks):
- layout/tableformatting/TableFormattingContextGeometry.cpp:
(WebCore::Layout::TableFormattingContext::Geometry::cellHeigh const):
- layout/tableformatting/TableFormattingContextQuirks.cpp: Added.
(WebCore::Layout::TableFormattingContext::Quirks::shouldIgnoreChildContentVerticalMargin const):
LayoutTests:
- fast/layoutformattingcontext/table-quirk-vertical-margin-simple-expected.html: Added.
- fast/layoutformattingcontext/table-quirk-vertical-margin-simple.html: Added.
- 9:55 AM Changeset in webkit [263901] by
-
- 22 edits2 copies in trunk
Split color conversion functions out of ColorUtilities.h/cpp into their own file.
https://bugs.webkit.org/show_bug.cgi?id=213803
Reviewed by Simon Fraser.
Source/WebCore:
Moves color conversion functions (both between color spaces like toDisplayP3(const SRGBA&)
and between different representations like toHSLA(const SRGBA&)) to their own files.
- Headers.cmake:
- Sources.txt:
- WebCore.xcodeproj/project.pbxproj:
Add new files.
- css/parser/CSSPropertyParserHelpers.cpp:
- editing/cocoa/DataDetection.mm:
- html/canvas/CanvasRenderingContext2DBase.cpp:
- html/canvas/CanvasStyle.cpp:
Add include of ColorConversion.h
- platform/graphics/ColorConversion.cpp: Added.
- platform/graphics/ColorConversion.h: Added.
- platform/graphics/ColorUtilities.cpp:
- platform/graphics/ColorUtilities.h:
Move functions to the new files.
- platform/graphics/ExtendedColor.cpp:
- platform/graphics/cairo/ImageBufferCairoBackend.cpp:
- platform/graphics/filters/FELighting.cpp:
Add include of ColorConversion.h, remove unncessary include of ColorUtilities.h
- platform/graphics/cairo/ImageBufferCairoSurfaceBackend.cpp:
- platform/graphics/filters/FEDisplacementMap.cpp:
- platform/graphics/filters/FEMorphology.cpp:
- platform/graphics/filters/FilterOperations.cpp:
Remove unncessary include of ColorUtilities.h
- platform/graphics/filters/FilterOperation.cpp:
Add include of ColorConversion.h and ColorTypes.h, remove unncessary include of ColorUtilities.h.
- platform/graphics/gtk/ColorGtk.cpp:
- platform/graphics/win/ColorDirect2D.cpp:
Remove unncessary include of ColorUtilities.h. Update to use standard float -> SimpleColor idiom
using SRGBA<float>.
Tools:
- TestWebKitAPI/Tests/WebCore/ColorTests.cpp:
Update to include ColorConversion.h and ColorTypes.h rather than ColorUtilities.h
- 9:45 AM Changeset in webkit [263900] by
-
- 21 edits in trunk/Source/WebCore
Convert DateComponents parsing code to use Optional based return values rather than out-parameters
https://bugs.webkit.org/show_bug.cgi?id=213440
Reviewed by Darin Adler.
Rework DateComponents and Date/Time related InputTypes to use Optional based programming
for parsing results. Also take the opportunity to remove unicode upconversion from
DateComponent parsing and instead separate UChar and LChar variants via templates.
- html/BaseDateAndTimeInputType.cpp:
- html/BaseDateAndTimeInputType.h:
- html/DateInputType.cpp:
- html/DateInputType.h:
- html/DateTimeInputType.cpp:
- html/DateTimeInputType.h:
- html/DateTimeLocalInputType.cpp:
- html/DateTimeLocalInputType.h:
- html/HTMLInputElement.cpp:
- html/HTMLInputElement.h:
- html/InputType.cpp:
- html/InputType.h:
- html/MonthInputType.cpp:
- html/MonthInputType.h:
- html/TimeInputType.cpp:
- html/TimeInputType.h:
- html/WeekInputType.cpp:
- html/WeekInputType.h:
- Removes parseToDateComponentsInternal. No need it and parseToDateComponents.
- Makes parseToDateComponents pure virtual. The old code had a default implementations down in InputType, but it had no callers.
- Remove iOS vs. non-iOS difference for dateType(). It is now available on InputType on all platforms.
- Make setMillisecondToDateComponents and parseToDateComponents return an Optional.
- platform/DateComponents.cpp:
- platform/DateComponents.h:
- Replace member function based interfaces for parsing/setting explicit time offsets with new factory functions that return Optional<DateComponents>.
- These factories are implemented using the existing member functions, which are now private.
- Make max/min constants constexpr.
- Make parse* member functions templates to allow factory parse functions to call them without upconverting.
- Replace header guard with #pragma once.
- 8:19 AM WebKitGTK/2.28.x edited by
- (diff)
- 8:09 AM Changeset in webkit [263899] by
-
- 2 edits in trunk/Source/WebKit
[WPE][GTK] Bubblewrap sandbox should not attempt to bind empty paths
https://bugs.webkit.org/show_bug.cgi?id=202880
Patch by Michael Catanzaro <Michael Catanzaro> on 2020-07-03
Reviewed by Carlos Garcia Campos.
If an empty path is set in LD_LIBRARY_PATH (e.g. /foo::/bar), we wind up passing
--ro-bind-try --ro-bind-try to bwrap, where the second --ro-bind-try is treated as the
source path to mount, and the next flag we pass gets treated as the destination path. The
web process crashes with this confusing error:
bwrap: Can't create file at /.flatpak-info: Read-only file system
- UIProcess/Launcher/glib/BubblewrapLauncher.cpp:
(WebKit::bindIfExists):
- 8:01 AM Changeset in webkit [263898] by
-
- 3 edits in trunk/Source/WebCore
Cleanup ApplicationCacheManifestParser.h/cpp
https://bugs.webkit.org/show_bug.cgi?id=213844
Reviewed by Alex Christensen.
Address some post review comments.
- loader/appcache/ApplicationCacheManifestParser.cpp:
(WebCore::parseApplicationCacheManifest):
Fix typos: manfest -> manifest.
- loader/appcache/ApplicationCacheManifestParser.h:
Stop including ApplicationCache.h, and only include / declare what's needed.
- 8:00 AM Changeset in webkit [263897] by
-
- 2 edits in trunk/Tools
[WPE][webkitpy] Use headless driver instead of wayland driver
https://bugs.webkit.org/show_bug.cgi?id=213914
Reviewed by Philippe Normand.
WPE now supports running tests completely headless, but we are
still using the webkitpy WaylandDriver to run the tests.
This causes an error if the environment doesn't have defined the
WAYLAND_DISPLAY or WAYLAND_SOCKET environment variables.
Switch the driver to HeadlessDriver.
- Scripts/webkitpy/port/wpe.py:
(WPEPort._driver_class):
- 7:55 AM Changeset in webkit [263896] by
-
- 7 edits in trunk
Support MediaRecorder.onstart
https://bugs.webkit.org/show_bug.cgi?id=213720
Reviewed by Darin Adler.
Source/WebCore:
Fire start event if MediaRecorder.start is successful.
Do some WebIDL clean-up, in particular change timeSlice from long to unsigned long, as per spec.
Covered by added test.
- Modules/mediarecorder/MediaRecorder.cpp:
(WebCore::MediaRecorder::startRecording):
- Modules/mediarecorder/MediaRecorder.h:
- Modules/mediarecorder/MediaRecorder.idl:
LayoutTests:
- http/wpt/mediarecorder/MediaRecorder-start-timeSlice-expected.txt:
- http/wpt/mediarecorder/MediaRecorder-start-timeSlice.html:
- 7:31 AM Changeset in webkit [263895] by
-
- 4 edits in trunk
REGRESSION(r261779): [GTK][WPE] http/tests/resourceLoadStatistics/third-party-cookie-blocking-ephemeral.html is failing
https://bugs.webkit.org/show_bug.cgi?id=212017
Reviewed by Michael Catanzaro.
Tools:
Use the ephemeral data store when useEphemeralSession option is enabled.
- WebKitTestRunner/TestController.cpp:
(WTR::TestController::platformInitializeDataStore):
LayoutTests:
Remove expectations for tests that are now passing.
- platform/gtk/TestExpectations:
- 7:21 AM Changeset in webkit [263894] by
-
- 20 edits1 copy4 adds in trunk
Allow registering VP9 as a VT decoder
https://bugs.webkit.org/show_bug.cgi?id=213837
<rdar://problem/64984881>
Reviewed by Jer Noble.
Source/ThirdParty/libwebrtc:
Implement a method that registers VP9 as a VT decoder.
Implement the binding code between VP9DecoderImpl and VT pluggable codec API.
Restrict this to internal SDK for now.
- Configurations/libwebrtc.iOS.exp:
- Configurations/libwebrtc.iOSsim.exp:
- Configurations/libwebrtc.mac.exp:
- Source/webrtc/sdk/WebKit/WebKitUtilities.h:
- Source/webrtc/sdk/WebKit/WebKitUtilities.mm:
(webrtc::createPixelBufferPool):
- Source/webrtc/sdk/WebKit/WebKitVP9Decoder.cpp: Added.
(webrtc::registerWebKitVP9Decoder):
(webrtc::createWebKitVP9Decoder):
(webrtc::invalidateVP9Decoder):
(webrtc::finalizeVP9Decoder):
(webrtc::copyVP9DecoderDebugDescription):
(webrtc::webKitVP9DecoderFromVTDecoder):
(webrtc::startVP9DecoderSession):
(webrtc::decodeVP9DecoderFrameFromContiguousBlock):
(webrtc::decodeVP9DecoderFrame):
(webrtc::WebKitVP9DecoderReceiver::WebKitVP9DecoderReceiver):
(webrtc::WebKitVP9DecoderReceiver::~WebKitVP9DecoderReceiver):
(webrtc::WebKitVP9DecoderReceiver::decoderFailed):
(webrtc::WebKitVP9DecoderReceiver::Decoded):
- Source/webrtc/sdk/WebKit/WebKitVP9Decoder.h: Added.
- libwebrtc.xcodeproj/project.pbxproj:
Source/WebKit:
Add a runtime flag to enable registration of VP9 as a VT decoder.
Register conditionally this decoder on creation of page based on this flag.
Test: media/vp9.html
- Scripts/GeneratePreferences.rb:
- Shared/WebPageCreationParameters.cpp:
(WebKit::WebPageCreationParameters::encode const):
(WebKit::WebPageCreationParameters::decode):
- Shared/WebPageCreationParameters.h:
- Shared/WebPreferences.yaml:
- UIProcess/WebPageProxy.cpp:
(WebKit::WebPageProxy::creationParameters):
- WebProcess/WebPage/WebPage.cpp:
(WebKit::m_limitsNavigationsToAppBoundDomains):
- WebProcess/WebProcess.cpp:
(WebKit::WebProcess::enableVP9Decoder):
- WebProcess/WebProcess.h:
LayoutTests:
- media/content/bipbop-vp9.mp4: Added.
- media/vp9-expected.txt: Added.
- media/vp9.html: Added.
- 6:12 AM Changeset in webkit [263893] by
-
- 81 edits1 copy3 adds5 deletes in trunk/WebDriverTests
[WebDriver][WPE] Update imported selenium to a version that enables WPE WebDriver tests
https://bugs.webkit.org/show_bug.cgi?id=213916
Reviewed by Carlos Garcia Campos.
Also updated the importer.json file with more paths to be skipped.
- imported/selenium/common/.classpath: Removed.
- imported/selenium/common/src/web/css/ui-lightness/images/ui-bg_diagonals-thick_18_b81900_40x40.png:
- imported/selenium/common/src/web/css/ui-lightness/images/ui-bg_diagonals-thick_20_666666_40x40.png:
- imported/selenium/common/src/web/css/ui-lightness/images/ui-bg_glass_100_f6f6f6_1x400.png:
- imported/selenium/common/src/web/css/ui-lightness/images/ui-bg_glass_100_fdf5ce_1x400.png:
- imported/selenium/common/src/web/css/ui-lightness/images/ui-bg_glass_65_ffffff_1x400.png:
- imported/selenium/common/src/web/css/ui-lightness/images/ui-bg_gloss-wave_35_f6a828_500x100.png:
- imported/selenium/common/src/web/css/ui-lightness/images/ui-bg_highlight-soft_100_eeeeee_1x100.png:
- imported/selenium/common/src/web/css/ui-lightness/images/ui-bg_highlight-soft_75_ffe45c_1x100.png:
- imported/selenium/common/src/web/css/ui-lightness/images/ui-icons_222222_256x240.png:
- imported/selenium/common/src/web/css/ui-lightness/images/ui-icons_228ef1_256x240.png:
- imported/selenium/common/src/web/css/ui-lightness/images/ui-icons_ef8c08_256x240.png:
- imported/selenium/common/src/web/css/ui-lightness/images/ui-icons_ffd27a_256x240.png:
- imported/selenium/common/src/web/css/ui-lightness/images/ui-icons_ffffff_256x240.png:
- imported/selenium/common/src/web/css/ui-lightness/jquery-ui-1.12.1.min.css: Added.
- imported/selenium/common/src/web/css/ui-lightness/jquery-ui-1.8.10.custom.css: Removed.
- imported/selenium/common/src/web/draggableLists.html:
- imported/selenium/common/src/web/droppableItems.html:
- imported/selenium/common/src/web/jquery-1.3.2.js: Removed.
- imported/selenium/common/src/web/js/jquery-1.4.4.min.js: Removed.
- imported/selenium/common/src/web/js/jquery-3.5.1.min.js: Added.
- imported/selenium/common/src/web/js/jquery-ui-1.12.1.min.js: Added.
- imported/selenium/common/src/web/js/jquery-ui-1.8.10.custom.min.js: Removed.
- imported/selenium/common/src/web/mousePositionTracker.html:
- imported/selenium/common/src/web/selectableItems.html:
- imported/selenium/importer.json:
- imported/selenium/py/conftest.py:
- imported/selenium/py/selenium/init.py:
- imported/selenium/py/selenium/webdriver/init.py:
- imported/selenium/py/selenium/webdriver/common/action_chains.py:
- imported/selenium/py/selenium/webdriver/common/actions/action_builder.py:
- imported/selenium/py/selenium/webdriver/common/actions/pointer_actions.py:
- imported/selenium/py/selenium/webdriver/common/actions/pointer_input.py:
- imported/selenium/py/selenium/webdriver/common/desired_capabilities.py:
- imported/selenium/py/selenium/webdriver/common/html5/application_cache.py:
- imported/selenium/py/selenium/webdriver/common/keys.py:
- imported/selenium/py/selenium/webdriver/common/options.py:
- imported/selenium/py/selenium/webdriver/common/service.py:
- imported/selenium/py/selenium/webdriver/common/timeouts.py:
- imported/selenium/py/selenium/webdriver/common/window.py:
- imported/selenium/py/selenium/webdriver/remote/file_detector.py:
- imported/selenium/py/selenium/webdriver/remote/remote_connection.py:
- imported/selenium/py/selenium/webdriver/remote/switch_to.py:
- imported/selenium/py/selenium/webdriver/remote/webdriver.py:
- imported/selenium/py/selenium/webdriver/remote/webelement.py:
- imported/selenium/py/selenium/webdriver/support/color.py:
- imported/selenium/py/selenium/webdriver/support/event_firing_webdriver.py:
- imported/selenium/py/selenium/webdriver/support/expected_conditions.py:
- imported/selenium/py/selenium/webdriver/support/relative_locator.py:
- imported/selenium/py/selenium/webdriver/support/select.py:
- imported/selenium/py/selenium/webdriver/support/wait.py:
- imported/selenium/py/selenium/webdriver/wpewebkit/options.py:
- imported/selenium/py/test/selenium/webdriver/common/alerts_tests.py:
- imported/selenium/py/test/selenium/webdriver/common/api_example_tests.py:
- imported/selenium/py/test/selenium/webdriver/common/appcache_tests.py:
- imported/selenium/py/test/selenium/webdriver/common/children_finding_tests.py:
- imported/selenium/py/test/selenium/webdriver/common/clear_tests.py:
- imported/selenium/py/test/selenium/webdriver/common/click_scrolling_tests.py:
- imported/selenium/py/test/selenium/webdriver/common/cookie_tests.py:
- imported/selenium/py/test/selenium/webdriver/common/correct_event_firing_tests.py:
- imported/selenium/py/test/selenium/webdriver/common/driver_element_finding_tests.py:
- imported/selenium/py/test/selenium/webdriver/common/element_attribute_tests.py:
- imported/selenium/py/test/selenium/webdriver/common/executing_async_javascript_tests.py:
- imported/selenium/py/test/selenium/webdriver/common/executing_javascript_tests.py:
- imported/selenium/py/test/selenium/webdriver/common/form_handling_tests.py:
- imported/selenium/py/test/selenium/webdriver/common/frame_switching_tests.py:
- imported/selenium/py/test/selenium/webdriver/common/google_one_box.py:
- imported/selenium/py/test/selenium/webdriver/common/implicit_waits_tests.py:
- imported/selenium/py/test/selenium/webdriver/common/interactions_tests.py:
- imported/selenium/py/test/selenium/webdriver/common/page_load_timeout_tests.py:
- imported/selenium/py/test/selenium/webdriver/common/page_loading_tests.py:
- imported/selenium/py/test/selenium/webdriver/common/position_and_size_tests.py:
- imported/selenium/py/test/selenium/webdriver/common/repr_tests.py:
- imported/selenium/py/test/selenium/webdriver/common/results_page.py:
- imported/selenium/py/test/selenium/webdriver/common/select_class_tests.py:
- imported/selenium/py/test/selenium/webdriver/common/stale_reference_tests.py:
- imported/selenium/py/test/selenium/webdriver/common/takes_screenshots_tests.py:
- imported/selenium/py/test/selenium/webdriver/common/text_handling_tests.py:
- imported/selenium/py/test/selenium/webdriver/common/timeout_tests.py:
- imported/selenium/py/test/selenium/webdriver/common/typing_tests.py:
- imported/selenium/py/test/selenium/webdriver/common/visibility_tests.py:
- imported/selenium/py/test/selenium/webdriver/common/w3c_interaction_tests.py:
- imported/selenium/py/test/selenium/webdriver/common/webdriverwait_tests.py:
- imported/selenium/py/test/selenium/webdriver/common/window_switching_tests.py:
- imported/selenium/py/test/selenium/webdriver/common/window_tests.py:
- imported/selenium/py/test/selenium/webdriver/remote/remote_hub_connection.py: Copied from WebDriverTests/imported/selenium/py/selenium/webdriver/common/window.py.
- imported/selenium/py/test/selenium/webdriver/safari/launcher_tests.py:
- imported/selenium/py/test/selenium/webdriver/support/event_firing_webdriver_tests.py:
- imported/selenium/py/test/selenium/webdriver/support/relative_by_tests.py:
- 5:58 AM Changeset in webkit [263892] by
-
- 4 edits2 adds in trunk
MediaRecorder.start() Method is Ignoring the "timeslice" Parameter
https://bugs.webkit.org/show_bug.cgi?id=202233
<rdar://problem/55720555>
Reviewed by Eric Carlson.
Source/WebCore:
Use a timer to implement timeSlice parameter.
Schedule timer either when start is called or as part of requestData callback.
This should ensure that, if requestData is called by the application, the timer will be rescheduled appropriately.
Test: http/wpt/mediarecorder/MediaRecorder-start-timeSlice.html
- Modules/mediarecorder/MediaRecorder.cpp:
(WebCore::MediaRecorder::MediaRecorder):
(WebCore::MediaRecorder::startRecording):
(WebCore::MediaRecorder::requestData):
- Modules/mediarecorder/MediaRecorder.h:
LayoutTests:
- http/wpt/mediarecorder/MediaRecorder-start-timeSlice-expected.txt: Added.
- http/wpt/mediarecorder/MediaRecorder-start-timeSlice.html: Added.
- 5:01 AM Changeset in webkit [263891] by
-
- 16 edits2 adds in trunk
MediaRecorder stopRecorder() returns empty Blob after first use
https://bugs.webkit.org/show_bug.cgi?id=212274
<rdar://problem/63601298>
Reviewed by Eric Carlson.
Source/WebCore:
Refactor code to create/destroy MediaRecorderPrivate on MediaRecorder start/stop.
This allows reusing a MediaRecorder after a stop and restarting with a clean state.
We introduce MediaRecorderPrivate::startRecording to do the initialization,
which allows to fix a potential ref cycle as part of the error callback handling.
Make some improvements to the platform implementation, in particular add default initialization to all fields.
Align the code using AudioConverterRef to what is done in AudioSampleDataSource.
Also call VTCompressionSessionInvalidate when destroying the VideoSampleBufferCompressor.
Test: http/wpt/mediarecorder/MediaRecorder-multiple-start-stop.html
- Modules/mediarecorder/MediaRecorder.cpp:
(WebCore::MediaRecorder::create):
(WebCore::MediaRecorder::MediaRecorder):
(WebCore::MediaRecorder::startRecording):
(WebCore::MediaRecorder::stopRecording):
(WebCore::MediaRecorder::requestData):
- Modules/mediarecorder/MediaRecorder.h:
- platform/mediarecorder/MediaRecorderPrivateMock.cpp:
(WebCore::MediaRecorderPrivateMock::fetchData):
- platform/mediarecorder/MediaRecorderPrivate.h:
(WebCore::MediaRecorderPrivate::startRecording):
- platform/mediarecorder/cocoa/AudioSampleBufferCompressor.h:
- platform/mediarecorder/cocoa/AudioSampleBufferCompressor.mm:
(WebCore::AudioSampleBufferCompressor::~AudioSampleBufferCompressor):
(WebCore::AudioSampleBufferCompressor::initAudioConverterForSourceFormatDescription):
(WebCore::AudioSampleBufferCompressor::attachPrimingTrimsIfNeeded):
(WebCore::AudioSampleBufferCompressor::gradualDecoderRefreshCount):
(WebCore::AudioSampleBufferCompressor::sampleBufferWithNumPackets):
(WebCore::AudioSampleBufferCompressor::processSampleBuffersUntilLowWaterTime):
- platform/mediarecorder/cocoa/MediaRecorderPrivateWriterCocoa.h:
- platform/mediarecorder/cocoa/MediaRecorderPrivateWriterCocoa.mm:
(WebCore::MediaRecorderPrivateWriter::stopRecording):
- platform/mediarecorder/cocoa/VideoSampleBufferCompressor.mm:
(WebCore::VideoSampleBufferCompressor::~VideoSampleBufferCompressor):
Source/WebKit:
Update implementation to do initialization as part of startRecording.
- GPUProcess/webrtc/RemoteMediaRecorderManager.cpp:
(WebKit::RemoteMediaRecorderManager::releaseRecorder):
Remove ASSERT as recorder creation in WebProcess is always ok while creation in GPUProcess may fail and m_recorders may not be populated.
- WebProcess/GPU/webrtc/MediaRecorderPrivate.cpp:
(WebKit::MediaRecorderPrivate::MediaRecorderPrivate):
(WebKit::MediaRecorderPrivate::startRecording):
- WebProcess/GPU/webrtc/MediaRecorderPrivate.h:
LayoutTests:
- http/wpt/mediarecorder/MediaRecorder-multiple-start-stop-expected.txt: Added.
- http/wpt/mediarecorder/MediaRecorder-multiple-start-stop.html: Added.
- 2:35 AM Changeset in webkit [263890] by
-
- 28 edits in trunk/Source
Change the names of MIMETypeRegistry methods to comply with webkit naming style
https://bugs.webkit.org/show_bug.cgi?id=213912
Patch by Said Abou-Hallawa <sabouhallawa@apple.com> on 2020-07-03
Reviewed by Youenn Fablet.
Source/WebCore:
Remove the verb 'get' from the name of some of the methods of MIMETypeRegistry.
- editing/cocoa/WebArchiveResourceFromNSAttributedString.mm:
(WebCore::if):
- fileapi/File.cpp:
(WebCore::File::computeNameAndContentType):
- html/HTMLPlugInElement.cpp:
(WebCore::pluginReplacementForType):
- loader/mac/LoaderNSURLExtras.mm:
(suggestedFilenameWithMIMEType):
- platform/MIMETypeRegistry.cpp:
(WebCore::commonMimeTypesMap):
(WebCore::MIMETypeRegistry::mediaMIMETypeForExtension):
(WebCore::MIMETypeRegistry::mimeTypeForPath):
(WebCore::MIMETypeRegistry::isSupportedImageMIMEType):
(WebCore::MIMETypeRegistry::normalizedMIMEType):
(WebCore::MIMETypeRegistry::appendFileExtensionIfNecessary):
(WebCore::MIMETypeRegistry::getMediaMIMETypeForExtension): Deleted.
(WebCore::MIMETypeRegistry::getMediaMIMETypesForExtension): Deleted.
Delete this function since it is not used.
(WebCore::MIMETypeRegistry::getMIMETypeForPath): Deleted.
(WebCore::MIMETypeRegistry::getNormalizedMIMEType): Deleted.
- platform/MIMETypeRegistry.h:
- platform/cocoa/MIMETypeRegistryCocoa.mm:
(WebCore::MIMETypeRegistry::mimeTypeForExtension):
(WebCore::MIMETypeRegistry::extensionsForMIMEType):
(WebCore::MIMETypeRegistry::preferredExtensionForMIMEType):
(WebCore::MIMETypeRegistry::getMIMETypeForExtension): Deleted.
(WebCore::MIMETypeRegistry::getExtensionsForMIMEType): Deleted.
(WebCore::MIMETypeRegistry::getPreferredExtensionForMIMEType): Deleted.
- platform/graphics/MediaPlayer.cpp:
(WebCore::MediaPlayer::load):
- platform/graphics/avfoundation/objc/ImageDecoderAVFObjC.mm:
(WebCore::ImageDecoderAVFObjC::filenameExtension const):
- platform/graphics/gstreamer/ImageDecoderGStreamer.h:
- platform/network/curl/CurlRequest.cpp:
(WebCore::CurlRequest::invokeDidReceiveResponseForFile):
- platform/playstation/MIMETypeRegistryPlayStation.cpp:
(WebCore::MIMETypeRegistry::mimeTypeForExtension):
(WebCore::MIMETypeRegistry::preferredExtensionForMIMEType):
(WebCore::MIMETypeRegistry::extensionsForMIMEType):
(WebCore::MIMETypeRegistry::getMIMETypeForExtension): Deleted.
(WebCore::MIMETypeRegistry::getPreferredExtensionForMIMEType): Deleted.
(WebCore::MIMETypeRegistry::getExtensionsForMIMEType): Deleted.
- platform/win/MIMETypeRegistryWin.cpp:
(WebCore::MIMETypeRegistry::preferredExtensionForMIMEType):
(WebCore::MIMETypeRegistry::mimeTypeForExtension):
(WebCore::MIMETypeRegistry::extensionsForMIMEType):
(WebCore::MIMETypeRegistry::getPreferredExtensionForMIMEType): Deleted.
(WebCore::MIMETypeRegistry::getMIMETypeForExtension): Deleted.
(WebCore::MIMETypeRegistry::getExtensionsForMIMEType): Deleted.
- platform/xdg/MIMETypeRegistryXdg.cpp:
(WebCore::MIMETypeRegistry::mimeTypeForExtension):
(WebCore::MIMETypeRegistry::preferredExtensionForMIMEType):
(WebCore::MIMETypeRegistry::extensionsForMIMEType):
(WebCore::MIMETypeRegistry::getMIMETypeForExtension): Deleted.
(WebCore::MIMETypeRegistry::getPreferredExtensionForMIMEType): Deleted.
(WebCore::MIMETypeRegistry::getExtensionsForMIMEType): Deleted.
Source/WebKit:
- NetworkProcess/soup/NetworkDataTaskSoup.cpp:
(WebKit::NetworkDataTaskSoup::didSendRequest):
- UIProcess/API/Cocoa/APIAttachmentCocoa.mm:
(API::mimeTypeInferredFromFileExtension):
(API::Attachment::setFileWrapperAndUpdateContentType):
- UIProcess/API/Cocoa/WKOpenPanelParameters.mm:
(-[WKOpenPanelParameters _allowedFileExtensions]):
- UIProcess/API/glib/WebKitURISchemeRequest.cpp:
(webkitURISchemeRequestReadCallback):
- UIProcess/Automation/WebAutomationSession.cpp:
(WebKit::fileCanBeAcceptedForUpload):
- UIProcess/Plugins/PluginInfoStore.cpp:
(WebKit::PluginInfoStore::findPlugin):
- UIProcess/ios/forms/WKFileUploadPanel.mm:
(-[WKFileUploadPanel presentWithParameters:resultListener:]):
- WebProcess/WebCoreSupport/WebFrameLoaderClient.cpp:
(WebKit::WebFrameLoaderClient::objectContentType):
Source/WebKitLegacy/ios:
- WebCoreSupport/WebMIMETypeRegistry.mm:
(+[WebMIMETypeRegistry mimeTypeForExtension:]):
(+[WebMIMETypeRegistry preferredExtensionForMIMEType:]):
Source/WebKitLegacy/win:
- WebView.cpp:
(WebView::MIMETypeForExtension):
- 2:18 AM Changeset in webkit [263889] by
-
- 4 edits1 add in trunk
[JSC] Add exception checks in JSStringBuilder and Array#join
https://bugs.webkit.org/show_bug.cgi?id=213915
<rdar://problem/64878225>
Reviewed by Saam Barati and Mark Lam.
JSTests:
- stress/out-of-memory-handle-in-join.js: Added.
(shouldThrow):
Source/JavaScriptCore:
This patch adds missing exception checks into Array#join's certain place and JSStringBuilder.
- runtime/ArrayPrototype.cpp:
(JSC::arrayProtoFuncToString):
- runtime/JSStringJoiner.h:
(JSC::JSStringJoiner::append):
- 2:06 AM Changeset in webkit [263888] by
-
- 2 edits in trunk/Source/WebCore
[GTK4] Enable threaded rendering by default
https://bugs.webkit.org/show_bug.cgi?id=213883
Patch by Carlos Garcia Campos <cgarcia@igalia.com> on 2020-07-03
Reviewed by Adrian Perez de Castro.
When building with GTK4 we no longer use the cairo context to render directly, so we can always enable the
threaded rendering. In WPE port threaded rendering is only enabled when env var WEBKIT_NICOSIA_PAINTING_THREADS
is present. This patch changes a bit the meaning of the env var, so that when 0 is passed the feature is
disabled too, instead of setting the number of threads to 4.
- platform/graphics/nicosia/NicosiaPaintingEngine.cpp:
(Nicosia::PaintingEngine::create):
- 1:37 AM Changeset in webkit [263887] by
-
- 5 edits in trunk/Source/WebKit
[GTK4] Support kinetic scrolling
https://bugs.webkit.org/show_bug.cgi?id=213791
Patch by Alexander Mikhaylenko <Alexander Mikhaylenko> on 2020-07-03
Reviewed by Carlos Garcia Campos.
GtkEventControllerScroll skips the last event and exposes it in a separate signal.
Normally one would add the KINETIC flag and listen to decelerate signal, but since
WebKit already has code for calculating the velocity for GTK3, just use the same
code path as scroll signal handler.
- Shared/gtk/NativeWebWheelEventGtk.cpp:
(WebKit::NativeWebWheelEvent::NativeWebWheelEvent):
- Shared/gtk/WebEventFactory.cpp:
(WebKit::WebEventFactory::createWebWheelEvent):
- Shared/gtk/WebEventFactory.h:
- UIProcess/API/gtk/WebKitWebViewBase.cpp:
(webkitWebViewBaseScrollEnd):
(webkitWebViewBaseConstructed):
- 12:42 AM Changeset in webkit [263886] by
-
- 25 edits6 adds4 deletes in trunk/LayoutTests
Unreviewed test gardening. Update test expectations and baselines after r263858.
Baselines need to be updated after r263856. Some PNG images moved to
GLIB as their text output was on GLIB as well.
Filed a test crashing after r263858.
- platform/glib/TestExpectations:
- platform/glib/fast/css/percentage-non-integer-expected.png: Added.
- platform/glib/fast/css/percentage-non-integer-expected.txt:
- platform/glib/tables/mozilla/bugs/bug100334-expected.png: Added.
- platform/glib/tables/mozilla/bugs/bug100334-expected.txt:
- platform/glib/tables/mozilla/bugs/bug3037-1-expected.png: Added.
- platform/glib/tables/mozilla/bugs/bug3037-1-expected.txt:
- platform/glib/tables/mozilla_expected_failures/other/empty_cells-expected.png: Added.
- platform/glib/tables/mozilla_expected_failures/other/empty_cells-expected.txt:
- platform/gtk/TestExpectations:
- platform/gtk/fast/css/percentage-non-integer-expected.png: Removed.
- platform/gtk/fast/dynamic/insert-before-table-part-in-continuation-expected.png:
- platform/gtk/fast/dynamic/insert-before-table-part-in-continuation-expected.txt:
- platform/gtk/fast/repaint/table-cell-move-expected.png:
- platform/gtk/fast/repaint/table-cell-move-expected.txt:
- platform/gtk/fast/table/cell-absolute-child-expected.png:
- platform/gtk/fast/table/cell-absolute-child-expected.txt:
- platform/gtk/fast/table/empty-cells-expected.png:
- platform/gtk/fast/table/empty-cells-expected.txt:
- platform/gtk/fast/table/prepend-in-anonymous-table-expected.png:
- platform/gtk/fast/table/prepend-in-anonymous-table-expected.txt:
- platform/gtk/tables/mozilla/bugs/bug100334-expected.png: Removed.
- platform/gtk/tables/mozilla/bugs/bug10633-expected.png:
- platform/gtk/tables/mozilla/bugs/bug10633-expected.txt:
- platform/gtk/tables/mozilla/bugs/bug1188-expected.png:
- platform/gtk/tables/mozilla/bugs/bug1188-expected.txt:
- platform/gtk/tables/mozilla/bugs/bug1302-expected.png:
- platform/gtk/tables/mozilla/bugs/bug1302-expected.txt:
- platform/gtk/tables/mozilla/bugs/bug16012-expected.png:
- platform/gtk/tables/mozilla/bugs/bug16012-expected.txt:
- platform/gtk/tables/mozilla/bugs/bug222336-expected.png: Added.
- platform/gtk/tables/mozilla/bugs/bug222336-expected.txt: Added.
- platform/gtk/tables/mozilla/bugs/bug3037-1-expected.png: Removed.
- platform/gtk/tables/mozilla_expected_failures/other/empty_cells-expected.png: Removed.
- 12:21 AM Changeset in webkit [263885] by
-
- 3 edits in trunk/Source/JavaScriptCore
Builtin internal wrapper implementation files wrap static global initialization code with incorrect guards
https://bugs.webkit.org/show_bug.cgi?id=213792
Reviewed by Youenn Fablet.
Streams API hadn't worked since r263700 for AppleWin and WinCairo
ports. r263700 removed the unused ENABLE_STREAMS_API.
Before r263700, the static global initialization code was wrapped
by "ENABLE(WEB_RTC) ENABLE(STREAMS_API)". After r263700, it was wrapped by "ENABLE(WEB_RTC)". AppleWin and WinCairo doesn't turn
on ENABLE_WEB_RTC. So, builtins for Streams API weren't properly
initialized for them.
- Scripts/tests/builtins/expected/WebCoreJSBuiltins.h-result: Rebaselined.
- Scripts/wkbuiltins/builtins_generate_internals_wrapper_implementation.py:
(BuiltinsInternalsWrapperImplementationGenerator.generate_initialize_method):
Removed calling wrap_with_guard for the value of _generate_initialize_static_globals().
- 12:15 AM Changeset in webkit [263884] by
-
- 7 edits in trunk/LayoutTests
[WinCairo] Unreviewed test gardening
- platform/wincairo/fast/css/acid2-expected.txt:
- platform/wincairo/fast/css/acid2-pixel-expected.txt:
- platform/wincairo/fast/css/percentage-non-integer-expected.txt:
- platform/wincairo/fast/dynamic/insert-before-table-part-in-continuation-expected.txt:
- platform/wincairo/http/tests/misc/acid2-expected.txt:
- platform/wincairo/http/tests/misc/acid2-pixel-expected.txt:
Jul 2, 2020:
- 10:51 PM Changeset in webkit [263883] by
-
- 7 edits in trunk/Source/WebCore
ReadableStream::create() should handle any exceptions that may be thrown during construction.
https://bugs.webkit.org/show_bug.cgi?id=213819
Reviewed by Youenn Fablet and Yusuke Suzuki.
Win EWS detected that ReadableStream::create() can throw exceptions, and we were
failing to handle it. This patch fixes that.
- Modules/cache/DOMCache.cpp:
(WebCore::DOMCache::put):
- Modules/fetch/FetchBodyOwner.cpp:
(WebCore::FetchBodyOwner::readableStream):
(WebCore::FetchBodyOwner::createReadableStream):
- Modules/fetch/FetchBodyOwner.h:
- Modules/fetch/FetchResponse.cpp:
(WebCore::FetchResponse::clone):
- bindings/js/ReadableStream.cpp:
(WebCore::ReadableStream::create):
- bindings/js/ReadableStream.h:
- 9:17 PM Changeset in webkit [263882] by
-
- 24 edits in trunk/Source
Make _WKWebsiteDataStoreConfiguration SPI for HSTS storage to replace _WKProcessPoolConfiguration.hstsStorageDirectory
https://bugs.webkit.org/show_bug.cgi?id=213048
Patch by Alex Christensen <achristensen@webkit.org> on 2020-07-02
Reviewed by Youenn Fablet.
Source/WebCore/PAL:
- pal/spi/cf/CFNetworkSPI.h:
Source/WebKit:
This uses CFNetwork SPI introduced in rdar://problem/50109631 to allow HSTS storage per NSURLSession.
To be complete, I also deprecated our UI process HSTS state removal attempt SPIs, WKContextResetHSTSHosts and
WKContextResetHSTSHostsAddedAfterDate, which had their last use removed in rdar://problem/64220838.
I manually verified that this new SPI puts HSTS data in the specified location, and I also verified that HSTS
state querying and removal works with the new CFNetwork SPI as it did with the old one.
- NetworkProcess/NetworkProcess.cpp:
(WebKit::NetworkProcess::fetchWebsiteData):
(WebKit::NetworkProcess::deleteWebsiteData):
(WebKit::NetworkProcess::deleteWebsiteDataForOrigins):
(WebKit::NetworkProcess::deleteAndRestrictWebsiteDataForRegistrableDomains):
(WebKit::NetworkProcess::registrableDomainsWithWebsiteData):
- NetworkProcess/NetworkProcess.h:
- NetworkProcess/NetworkSessionCreationParameters.cpp:
(WebKit::NetworkSessionCreationParameters::encode const):
(WebKit::NetworkSessionCreationParameters::decode):
- NetworkProcess/NetworkSessionCreationParameters.h:
- NetworkProcess/cocoa/NetworkProcessCocoa.mm:
(WebKit::NetworkProcess::hostNamesWithHSTSCache const):
(WebKit::NetworkProcess::deleteHSTSCacheForHostNames):
(WebKit::NetworkProcess::clearHSTSCache):
(WebKit::NetworkProcess::getHostNamesWithHSTSCache): Deleted.
- NetworkProcess/cocoa/NetworkSessionCocoa.h:
- NetworkProcess/cocoa/NetworkSessionCocoa.mm:
(WebKit::NetworkSessionCocoa::hstsStorage const):
(WebKit::NetworkSessionCocoa::NetworkSessionCocoa):
- NetworkProcess/soup/NetworkProcessSoup.cpp:
(WebKit::NetworkProcess::hostNamesWithHSTSCache const):
(WebKit::NetworkProcess::deleteHSTSCacheForHostNames):
(WebKit::NetworkProcess::clearHSTSCache):
(WebKit::NetworkProcess::getHostNamesWithHSTSCache): Deleted.
- UIProcess/API/C/mac/WKContextPrivateMac.h:
- UIProcess/API/C/mac/WKContextPrivateMac.mm:
(WKContextResetHSTSHosts):
(WKContextResetHSTSHostsAddedAfterDate):
- UIProcess/API/Cocoa/_WKProcessPoolConfiguration.h:
- UIProcess/API/Cocoa/_WKWebsiteDataStoreConfiguration.h:
- UIProcess/API/Cocoa/_WKWebsiteDataStoreConfiguration.mm:
(-[_WKWebsiteDataStoreConfiguration hstsStorageDirectory]):
(-[_WKWebsiteDataStoreConfiguration setHSTSStorageDirectory:]):
- UIProcess/Cocoa/WebProcessPoolCocoa.mm:
(WebKit::privateBrowsingSession): Deleted.
(WebKit::WebProcessPool::resetHSTSHosts): Deleted.
(WebKit::WebProcessPool::resetHSTSHostsAddedAfterDate): Deleted.
- UIProcess/WebProcessPool.h:
- UIProcess/WebsiteData/WebsiteDataStore.cpp:
(WebKit::WebsiteDataStore::resolveDirectoriesIfNecessary):
(WebKit::WebsiteDataStore::parameters):
- UIProcess/WebsiteData/WebsiteDataStore.h:
(WebKit::WebsiteDataStore::resolvedHSTSStorageDirectory const):
- UIProcess/WebsiteData/WebsiteDataStoreConfiguration.cpp:
(WebKit::WebsiteDataStoreConfiguration::copy const):
- UIProcess/WebsiteData/WebsiteDataStoreConfiguration.h:
Source/WTF:
- wtf/PlatformHave.h:
- 8:40 PM Changeset in webkit [263881] by
-
- 13 edits in trunk
Update Mac CMake build
.:
- Source/cmake/OptionsMac.cmake:
Source/JavaScriptCore:
- PlatformMac.cmake:
Source/WebCore:
- PlatformMac.cmake:
Source/WebKit:
- PlatformMac.cmake:
Source/WebKitLegacy:
- PlatformMac.cmake:
Tools:
- DumpRenderTree/PlatformMac.cmake:
- WebKitTestRunner/PlatformMac.cmake:
- 8:19 PM Changeset in webkit [263880] by
-
- 2 edits in trunk/Source/WebCore
MIMETypeRegistry::getExtensionsForMIMEType() needs to handle wildcard MIME types
https://bugs.webkit.org/show_bug.cgi?id=213826
Patch by Said Abou-Hallawa <sabouhallawa@apple.com> on 2020-07-02
Reviewed by Darin Adler.
Addressing post commit review comments for r263832.
- platform/cocoa/MIMETypeRegistryCocoa.mm:
(WebCore::extensionsForMIMETypeMap):
- 7:01 PM Changeset in webkit [263879] by
-
- 8 edits in trunk
Add testing infrastructure and SPI declaration for HTTP/2 ping
https://bugs.webkit.org/show_bug.cgi?id=213913
Patch by Alex Christensen <achristensen@webkit.org> on 2020-07-02
Reviewed by Jer Noble.
Source/WebCore/PAL:
- pal/spi/cf/CFNetworkSPI.h:
Source/WTF:
This is work towards rdar://problem/64495827
- wtf/PlatformHave.h:
Tools:
- TestWebKitAPI/Tests/WebKitCocoa/Preconnect.mm:
(-[SessionDelegate URLSession:task:didReceiveChallenge:completionHandler:]):
(TestWebKitAPI::pingPong):
(TestWebKitAPI::TEST):
- TestWebKitAPI/cocoa/HTTPServer.h:
(TestWebKitAPI::H2::Frame::Frame):
(TestWebKitAPI::H2::Frame::type const):
(TestWebKitAPI::H2::Frame::flags const):
(TestWebKitAPI::H2::Frame::streamID const):
(TestWebKitAPI::H2::Frame::payload const):
(TestWebKitAPI::H2::Connection::create):
(TestWebKitAPI::H2::Connection::Connection):
- TestWebKitAPI/cocoa/HTTPServer.mm:
(TestWebKitAPI::HTTPServer::listenerParameters):
(TestWebKitAPI::dataFromVector):
(TestWebKitAPI::vectorFromData):
(TestWebKitAPI::HTTPServer::request const):
(TestWebKitAPI::Connection::receiveBytes const):
(TestWebKitAPI::Connection::receiveHTTPRequest const):
(TestWebKitAPI::Connection::send const):
(TestWebKitAPI::H2::Connection::send const):
(TestWebKitAPI::H2::Connection::receive const):
- 6:05 PM Changeset in webkit [263878] by
-
- 3 edits3 adds in trunk/Source/WebCore
GameController.framework soft linking refactoring
https://bugs.webkit.org/show_bug.cgi?id=213909
Reviewed by Tim Horton.
No new tests (No behavior changes)
- WebCore.xcodeproj/project.pbxproj:
- platform/gamepad/cocoa/GameControllerGamepadProvider.mm:
(WebCore::GameControllerGamepadProvider::startMonitoringGamepads):
- platform/gamepad/cocoa/GameControllerSPI.h: Added.
- platform/gamepad/cocoa/GameControllerSoftLink.h: Added.
- platform/gamepad/cocoa/GameControllerSoftLink.mm: Added.
- 5:22 PM Changeset in webkit [263877] by
-
- 1 copy in tags/Safari-610.1.20.2
Tag Safari-610.1.20.2.
- 5:17 PM Changeset in webkit [263876] by
-
- 16 edits2 deletes in branches/safari-610.1.20-branch
Cherry-pick r263854. rdar://problem/65048370
Still seeing MediaRecorder test crashes after re-landing
r263633
Reverted changesets:
"MediaRecorder stopRecorder() returns empty Blob after first
use"
https://bugs.webkit.org/show_bug.cgi?id=212274
https://trac.webkit.org/changeset/263633
"MediaRecorder.start() Method is Ignoring the "timeslice"
Parameter"
https://bugs.webkit.org/show_bug.cgi?id=202233
https://trac.webkit.org/changeset/263651
"Support MediaRecorder.onstart"
https://bugs.webkit.org/show_bug.cgi?id=213720
https://trac.webkit.org/changeset/263671
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@263854 268f45cc-cd09-0410-ab3c-d52691b4dbfc
- 5:08 PM Changeset in webkit [263875] by
-
- 8 edits in branches/safari-610.1.20-branch/Source
Versioning.
WebKit-610.1.20.2
- 4:15 PM Changeset in webkit [263874] by
-
- 8 edits in trunk
Custom URL schemes should be treated as app-bound
https://bugs.webkit.org/show_bug.cgi?id=213889
<rdar://problem/64804671>
Reviewed by Brent Fulgham.
Source/WebKit:
For applications which opt-in to App-Bound Domains, allow
specification of app-bound custom URL schemes. All content loaded
using an app-bound scheme will have access to otherwise restricted
APIs. A custom scheme is specified by a colon at the end of a string
in the WKAppBoundDomains list. Custom schemes are included in the
count of 10 app-bound domains.
- UIProcess/API/Cocoa/WKWebsiteDataStore.mm:
(-[WKWebsiteDataStore _appBoundSchemes:]):
- UIProcess/API/Cocoa/WKWebsiteDataStorePrivate.h:
SPI for testing.
- UIProcess/WebsiteData/Cocoa/WebsiteDataStoreCocoa.mm:
(WebKit::appBoundSchemes):
(WebKit::WebsiteDataStore::initializeAppBoundDomains):
Change variable name from appBoundDomains to appBoundData now that
more than domains can be specified in the Info.plist.
(WebKit::WebsiteDataStore::ensureAppBoundDomains const):
Return both domains and schemes to avoid code duplication, at the expense
of occasionally returning an unused parameter if only domains or only
schemes are needed.
(WebKit::WebsiteDataStore::beginAppBoundDomainCheck):
(WebKit::WebsiteDataStore::getAppBoundDomains const):
(WebKit::WebsiteDataStore::getAppBoundSchemes const):
- UIProcess/WebsiteData/WebsiteDataStore.h:
Tools:
Added custom schemes to TestWebKitAPI's Info.plist for testing
duplicate values and the max of 10 domains/schemes. Added
two API tests to check that schemes are properly read from the
Info.plist and that content loaded using a specified app-bound scheme
has restricted API use.
- TestWebKitAPI/Info.plist:
- TestWebKitAPI/Tests/WebKitCocoa/InAppBrowserPrivacy.mm:
(TEST):
- 3:50 PM Changeset in webkit [263873] by
-
- 1 copy in tags/Safari-610.1.20.1
Tag Safari-610.1.20.1.
- 3:49 PM Changeset in webkit [263872] by
-
- 2 edits in trunk/Source/WebInspectorUI
Web Inspector: Allow selecting text of Response (DOM Tree)
https://bugs.webkit.org/show_bug.cgi?id=213617
Reviewed by Joseph Pecoraro.
- UserInterface/Views/DOMTreeOutline.css:
(.tree-outline.dom.non-selectable):
- 3:49 PM Changeset in webkit [263871] by
-
- 3 edits in branches/safari-610.1.20-branch/Source/ThirdParty/libwebrtc
Apply patch. rdar://problem/64901603
- 3:44 PM Changeset in webkit [263870] by
-
- 1 delete in tags/Safari-610.1.20.1
Delete tag.
- 3:43 PM Changeset in webkit [263869] by
-
- 2 edits in trunk/Source/WebKit
Crash in +[UIViewController _viewControllerForFullScreenPresentationFromView:] when WKContentView is deallocated
https://bugs.webkit.org/show_bug.cgi?id=213867
Patch by Austin Blackwood <ablackwoood@apple.com> on 2020-07-02
Reviewed by Darin Adler.
+[UIViewController _viewControllerForFullScreenPresentationFromView:]
asserts that the view parameter is non-nil. WKFileUploadPanel
passes its view (a weak pointer to the WKContentView), which may
have already been nil'd out when the content view tries to dismiss
the panel in its own dealloc.
- UIProcess/ios/forms/WKFileUploadPanel.mm:
(-[WKFileUploadPanel dismiss]):
Check for nil.
- 3:42 PM Changeset in webkit [263868] by
-
- 3 edits in branches/safari-610.1.20-branch/Source/ThirdParty/libwebrtc
Revert r263828. rdar://problem/64901603
- 3:37 PM Changeset in webkit [263867] by
-
- 1 copy in tags/Safari-610.1.20.1
Tag Safari-610.1.20.1.
- 3:36 PM Changeset in webkit [263866] by
-
- 3 edits in branches/safari-610.1.20-branch/Source/ThirdParty/libwebrtc
Cherry-pick r263828. rdar://problem/64901603
Fix the arm64 iOS simulator build
https://bugs.webkit.org/show_bug.cgi?id=213868
<rdar://problem/64901603>
- Configurations/libvpx.xcconfig:
- Configurations/libwebrtc.xcconfig:
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@263828 268f45cc-cd09-0410-ab3c-d52691b4dbfc
- 3:31 PM Changeset in webkit [263865] by
-
- 8 edits in branches/safari-610.1.20-branch/Source
Versioning.
WebKit-610.1.20.1
- 3:31 PM Changeset in webkit [263864] by
-
- 2 edits in trunk/LayoutTests
[ macOS wk1 ] imported/w3c/web-platform-tests/css/css-flexbox/overflow-auto-008.html is flaky failing
https://bugs.webkit.org/show_bug.cgi?id=213896
Unreviewed test gardening.
Added prefix with bug
Patch by Karl Rackler <Karl Rackler> on 2020-07-02
- platform/mac-wk1/TestExpectations:
- 3:30 PM Changeset in webkit [263863] by
-
- 16 edits2 adds in branches/safari-610.1.20-branch
Revert r263854. rdar://problem/65048370
- 2:38 PM Changeset in webkit [263862] by
-
- 1 copy in tags/Safari-610.1.20
Tag Safari-610.1.20.
- 2:11 PM Changeset in webkit [263861] by
-
- 16 edits2 deletes in branches/safari-610.1.20-branch
Cherry-pick r263854. rdar://problem/65048370
Still seeing MediaRecorder test crashes after re-landing
r263633
Reverted changesets:
"MediaRecorder stopRecorder() returns empty Blob after first
use"
https://bugs.webkit.org/show_bug.cgi?id=212274
https://trac.webkit.org/changeset/263633
"MediaRecorder.start() Method is Ignoring the "timeslice"
Parameter"
https://bugs.webkit.org/show_bug.cgi?id=202233
https://trac.webkit.org/changeset/263651
"Support MediaRecorder.onstart"
https://bugs.webkit.org/show_bug.cgi?id=213720
https://trac.webkit.org/changeset/263671
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@263854 268f45cc-cd09-0410-ab3c-d52691b4dbfc
- 2:08 PM Changeset in webkit [263860] by
-
- 3 edits2 adds in trunk
REGRESSION(r260276): menu items in footer of https://salo.ai are invisible until you hover or resize
https://bugs.webkit.org/show_bug.cgi?id=213712
<rdar://problem/64893190>
Reviewed by Zalan Bujtas.
Source/WebCore:
A negative z-index child triggers a foreground layer inside overflow scroll. This foreground layer
gets the PaintingOverflowContents flag but not the PaintingCompositingScrollingPhase flag, but we still need
to avoid the clip check that happens in RenderBlock::paint(), so change RenderLayer::paintLayerContents()
to just check for isPaintingOverflowContents.
Test: compositing/scrolling/async-overflow-scrolling/foreground-layer-paint-phase.html
- rendering/RenderLayer.cpp:
(WebCore::RenderLayer::paintLayerContents):
LayoutTests:
- compositing/scrolling/async-overflow-scrolling/foreground-layer-paint-phase-expected.html: Added.
- compositing/scrolling/async-overflow-scrolling/foreground-layer-paint-phase.html: Added.
- 1:35 PM Changeset in webkit [263859] by
-
- 2 edits in trunk/LayoutTests
[ macOS wk1 ] imported/w3c/web-platform-tests/css/css-flexbox/overflow-auto-008.html is flaky failing
https://bugs.webkit.org/show_bug.cgi?id=213896
Unreviewed test gardening.
Patch by Karl Rackler <Karl Rackler> on 2020-07-02
- platform/mac-wk1/TestExpectations:
- 1:35 PM Changeset in webkit [263858] by
-
- 3 edits2 adds in trunk
visualViewport.addEventListener("scroll"* fires permanently after zoom or orientation change
https://bugs.webkit.org/show_bug.cgi?id=211522
<rdar://problem/62939371>
Reviewed by Simon Fraser.
Source/WebCore:
Correctly set the flag indicating that a scroll event has been dispatched on the visual viewport.
Test: fast/visual-viewport/scroll-event-fired-during-scroll-alone.html
- dom/Document.cpp:
(WebCore::Document::runScrollSteps):
LayoutTests:
Add a test that checks that after a call to scrollTo() no further "scroll" events are dispatched on window.visualViewport.
- fast/visual-viewport/scroll-event-fired-during-scroll-alone-expected.txt: Added.
- fast/visual-viewport/scroll-event-fired-during-scroll-alone.html: Added.
- 1:26 PM Changeset in webkit [263857] by
-
- 3 edits in trunk/Source/WebKit
FloatingPointEnvironment::saveMainThreadEnvironment() is not called for macOS Catalyst apps on Apple Silicon
https://bugs.webkit.org/show_bug.cgi?id=213897
<rdar://problem/64607098>
Reviewed by Darin Adler.
FloatingPointEnvironment::saveMainThreadEnvironment() is not called for macOS Catalyst apps on Apple Silicon
This causes crashes because FloatingPointEnvironment::propagateMainThreadEnvironment() is called for worker
threads for such apps on this platform.
- Shared/ios/AuxiliaryProcessIOS.mm:
(WebKit::AuxiliaryProcess::initializeSandbox):
Drop some dead code for PLATFORM(MACCATALYST) since the whole file is protected with !PLATFORM(MACCATALYST).
Mac Catalyst uses AuxiliaryProcessMac.mm instead.
- Shared/mac/AuxiliaryProcessMac.mm:
(WebKit::AuxiliaryProcess::platformInitialize):
Call FloatingPointEnvironment::saveMainThreadEnvironment() on AuxiliaryProcessMac.mm since this
platformInitialize() is used for Mac Catalyst and Catalyst apps may now run on Apple Silicon / ARM.
Note that FloatingPointEnvironment::saveMainThreadEnvironment() is a no-op on platforms where it is
not needed.
- 12:54 PM Changeset in webkit [263856] by
-
- 54 edits2 copies43 adds in trunk/LayoutTests
Resync web-platform-tests/html/browsers from upstream
https://bugs.webkit.org/show_bug.cgi?id=213892
Reviewed by Geoffrey Garen.
LayoutTests/imported/w3c:
Resync web-platform-tests/html/browsers from upstream 145839c323ba1ec8b9737.
- resources/import-expectations.json:
- resources/resource-files.json:
- web-platform-tests/html/browsers/*: Updated.
LayoutTests:
- tests-options.json:
- 12:45 PM Changeset in webkit [263855] by
-
- 58 edits in trunk
Overlapping content on earny.co
https://bugs.webkit.org/show_bug.cgi?id=213864
<rdar://problem/61011802>
Reviewed by Simon Fraser.
LayoutTests/imported/w3c:
- web-platform-tests/css/css-writing-modes/orthogonal-parent-shrink-to-fit-001-expected.txt:
- web-platform-tests/css/css-writing-modes/orthogonal-parent-shrink-to-fit-001q-expected.txt:
- web-platform-tests/css/css-writing-modes/orthogonal-parent-shrink-to-fit-001r-expected.txt:
- web-platform-tests/css/css-writing-modes/orthogonal-parent-shrink-to-fit-001u-expected.txt:
- web-platform-tests/css/css-writing-modes/orthogonal-parent-shrink-to-fit-001v-expected.txt:
Source/WebCore:
Do not use a 1px value for initial min/max content width.
- rendering/AutoTableLayout.cpp:
(WebCore::AutoTableLayout::recalcColumn):
(WebCore::AutoTableLayout::calcEffectiveLogicalWidth):
(WebCore::AutoTableLayout::layout): I found 2 places where we rely on the initial 1px value for min/max width. This might cause some "regression".
LayoutTests:
Progressions. Some of them are visually observable and they match Chrome/FF.
- css2.1/20110323/abspos-containing-block-initial-004e-expected.txt:
- css2.1/20110323/abspos-containing-block-initial-004f-expected.txt:
- fast/table/auto-100-percent-width-expected.txt:
- fast/table/large-shrink-wrapped-width.html:
- fast/table/table-section-split-with-after-content-expected.txt:
- platform/mac/fast/css/acid2-expected.txt:
- platform/mac/fast/css/acid2-pixel-expected.txt:
- platform/mac/fast/css/percentage-non-integer-expected.txt:
- platform/mac/fast/dynamic/insert-before-table-part-in-continuation-expected.txt:
- platform/mac/fast/repaint/table-cell-move-expected.txt:
- platform/mac/fast/table/cell-absolute-child-expected.txt:
- platform/mac/fast/table/empty-cells-expected.txt:
- platform/mac/fast/table/prepend-in-anonymous-table-expected.txt:
- platform/mac/fast/table/table-cell-split-expected.txt:
- platform/mac/fast/table/table-insert-before-non-anonymous-block-expected.txt:
- platform/mac/http/tests/misc/acid2-expected.txt:
- platform/mac/http/tests/misc/acid2-pixel-expected.txt:
- platform/mac/http/tests/misc/acid3-expected.txt:
- platform/mac/tables/mozilla/bugs/bug100334-expected.txt:
- platform/mac/tables/mozilla/bugs/bug10633-expected.txt:
- platform/mac/tables/mozilla/bugs/bug1188-expected.txt:
- platform/mac/tables/mozilla/bugs/bug1302-expected.txt:
- platform/mac/tables/mozilla/bugs/bug16012-expected.txt:
- platform/mac/tables/mozilla/bugs/bug222336-expected.txt:
- platform/mac/tables/mozilla/bugs/bug3037-1-expected.txt:
- platform/mac/tables/mozilla_expected_failures/other/empty_cells-expected.txt:
- 12:34 PM Changeset in webkit [263854] by
-
- 17 edits4 deletes in trunk
Unreviewed, reverting r263633, r263651, and r263671.
Still seeing MediaRecorder test crashes after re-landing
r263633
Reverted changesets:
"MediaRecorder stopRecorder() returns empty Blob after first
use"
https://bugs.webkit.org/show_bug.cgi?id=212274
https://trac.webkit.org/changeset/263633
"MediaRecorder.start() Method is Ignoring the "timeslice"
Parameter"
https://bugs.webkit.org/show_bug.cgi?id=202233
https://trac.webkit.org/changeset/263651
"Support MediaRecorder.onstart"
https://bugs.webkit.org/show_bug.cgi?id=213720
https://trac.webkit.org/changeset/263671
- 11:56 AM Changeset in webkit [263853] by
-
- 3 edits2 adds in trunk
REGRESSION: Comments section at dpreview has overlapping names with comment on phone
https://bugs.webkit.org/show_bug.cgi?id=213890
Source/WebCore:
<rdar://problem/64693599>
Reviewed by Simon Fraser.
Test: fast/text-autosizing/ios/float-miscomputed-line-height.html
- rendering/TextAutoSizing.cpp:
(WebCore::TextAutoSizingValue::adjustTextNodeSizes):
Negative value is used to indicate unset line-height. We shouldn't make adjustments in this case.
This matches Style::BuilderCustom code.
LayoutTests:
Reviewed by Simon Fraser.
- fast/text-autosizing/ios/float-miscomputed-line-height-expected.html: Added.
- fast/text-autosizing/ios/float-miscomputed-line-height.html: Added.
- 11:45 AM Changeset in webkit [263852] by
-
- 2 edits in trunk/Source/WebKit
Crash under WebKit::NetworkProcessProxy::updateProcessAssertion()
https://bugs.webkit.org/show_bug.cgi?id=213891
<rdar://problem/65017909>
Reviewed by Alex Christensen.
The crash was due to NetworkProcessProxy::updateProcessAssertion() re-entering while
in the middle of them_activityFromWebProcesses = nullptr;assignment. Calling
the ProcessThrottler::BackgroundActivity destructor, could cause updateProcessAssertion()
to get called again, in which case we may dereference m_activityFromWebProcesses and
crash. To address the issue, usestd::exchange(m_activityFromWebProcesses, nullptr);
instead, so that m_activityFromWebProcesses becomes null BEFORE the BackgroundActivity
destructor gets called. updateProcessAssertion() will still re-enter but
m_activityFromWebProcesses will be nullptr and updateProcessAssertion() will do the
right thing.
- UIProcess/Network/NetworkProcessProxy.cpp:
(WebKit::NetworkProcessProxy::updateProcessAssertion):
- 10:56 AM Changeset in webkit [263851] by
-
- 1 edit8 deletes in trunk/LayoutTests/imported/w3c
Unreviewed, drop some outdated tests in WebCrypto API.
Those tests failed to get removed during the recent re-sync from upstream in r263598.
They are actually still present but under different names so we have duplicate
copies right now.
- web-platform-tests/WebCryptoAPI/derive_bits_keys/test_hkdf-expected.txt: Removed.
- web-platform-tests/WebCryptoAPI/derive_bits_keys/test_pbkdf2-expected.txt: Removed.
- web-platform-tests/WebCryptoAPI/getRandomValues.js: Removed.
- web-platform-tests/WebCryptoAPI/getRandomValues.worker-expected.txt: Removed.
- web-platform-tests/WebCryptoAPI/getRandomValues.worker.html: Removed.
- web-platform-tests/WebCryptoAPI/getRandomValues.worker.js: Removed.
- web-platform-tests/WebCryptoAPI/test_getRandomValues-expected.txt: Removed.
- web-platform-tests/WebCryptoAPI/test_getRandomValues.html: Removed.
- 10:30 AM Changeset in webkit [263850] by
-
- 2 edits in trunk/LayoutTests
Remove expectation for fast/canvas/canvas-radial-gradient-spreadMethod.html as it is passing. Added back imported/w3c/canvas/2d.gradient.radial.cone.bottom.html as it is failing.
https://bugs.webkit.org/show_bug.cgi?id=213840
Unreviewed test gardening.
Patch by Karl Rackler <Karl Rackler> on 2020-07-02
- platform/ios/TestExpectations:
- 10:17 AM Changeset in webkit [263849] by
-
- 2 edits in trunk/JSTests
Unreviewed, mark JSTests/test262/test/intl402/NumberFormat/prototype/format/numbering-systems.js fail since it requires ICU 66
https://bugs.webkit.org/show_bug.cgi?id=213872
This test in test262 requires ICU 66 while Catalina bot is using ICU 64.
- test262/config.yaml:
- 9:31 AM Changeset in webkit [263848] by
-
- 2 edits in trunk/Tools
REGRESSION(r263625): run-minibrowser fails on mac
https://bugs.webkit.org/show_bug.cgi?id=213876
Patch by Philippe Normand <pnormand@igalia.com> on 2020-07-02
Reviewed by Darin Adler.
- Scripts/webkitpy/port/base.py:
(Port.run_minibrowser): _run_script() expects a string as first argument. Command-line args
are passed through a keyword.
- 9:17 AM Changeset in webkit [263847] by
-
- 2 edits in trunk/LayoutTests
Unreviewed, unskip most webaudio WPT tests in Debug.
- 7:52 AM Changeset in webkit [263846] by
-
- 3 edits2 adds in trunk
[LFC][BFC] Remove redundant margin-border-padding when computing the intrinsic width
https://bugs.webkit.org/show_bug.cgi?id=213882
Reviewed by Antti Koivisto.
Source/WebCore:
The child intrinsic width already includes the margin-border-padding values.
Test: fast/layoutformattingcontext/table-with-margin-content-simple.html
- layout/blockformatting/BlockFormattingContextGeometry.cpp:
(WebCore::Layout::BlockFormattingContext::Geometry::intrinsicWidthConstraints):
LayoutTests:
- fast/layoutformattingcontext/table-with-margin-content-simple-expected.html: Added.
- fast/layoutformattingcontext/table-with-margin-content-simple.html: Added.
- 7:52 AM Changeset in webkit [263845] by
-
- 3 edits2 adds in trunk
[LFC][IFC] Use <tr> computed height as minimum height for the row
https://bugs.webkit.org/show_bug.cgi?id=213880
Reviewed by Antti Koivisto.
Source/WebCore:
Test: fast/layoutformattingcontext/table-fixed-row-height-simple.html
- layout/tableformatting/TableLayout.cpp:
(WebCore::Layout::TableFormattingContext::TableLayout::distributedVerticalSpace):
LayoutTests:
- fast/layoutformattingcontext/table-fixed-row-height-simple-expected.html: Added.
- fast/layoutformattingcontext/table-fixed-row-height-simple.html: Added.
- 7:05 AM Changeset in webkit [263844] by
-
- 11 edits2 adds in trunk
getUserMedia returns OverConstrained on Jitsi
https://bugs.webkit.org/show_bug.cgi?id=210932
<rdar://problem/64403675>
Reviewed by Philippe Normand.
Source/WebCore:
We compute the max width, max height and max frame rate across all presets.
In case a preset for the max resolution is different from the preset for the max frame rate,
we were selecting capture parameters that no preset can match.
To fix the issue, in case the frame rate constraint is not mandatory, and we do not find a preset
when starting to capture, we remove the frame rate constraint, pick the preset and choose the max frame rate from that preset.
Update mock sources to have a high resolution preset with low frame rate to enable writing a test.
Test: fast/mediastream/get-user-media-ideal-constraints.html
- platform/mediastream/RealtimeMediaSource.cpp:
(WebCore::RealtimeMediaSource::supportsSizeAndFrameRate):
- platform/mediastream/RealtimeVideoCaptureSource.cpp:
(WebCore::RealtimeVideoCaptureSource::bestSupportedSizeAndFrameRate):
(WebCore::RealtimeVideoCaptureSource::setSizeAndFrameRate):
- platform/mediastream/VideoPreset.h:
(WebCore::VideoPreset::maxFrameRate const):
- platform/mock/MockRealtimeMediaSourceCenter.cpp:
(WebCore::defaultDevices):
LayoutTests:
We bumped the max width/height for mock sources and have to update existing tests according to that.
- fast/mediastream/MediaStreamTrack-getCapabilities-expected.txt:
- fast/mediastream/apply-constraints-advanced-expected.txt:
- fast/mediastream/apply-constraints-advanced.html:
- fast/mediastream/apply-constraints-video-expected.txt:
- fast/mediastream/apply-constraints-video.html:
- fast/mediastream/get-user-media-ideal-constraints-expected.txt: Added.
- fast/mediastream/get-user-media-ideal-constraints.html: Added.
- 5:41 AM Changeset in webkit [263843] by
-
- 8 edits in trunk
Unreviewed. Fix GTK4 build
Source/WebCore:
- platform/gtk/GtkVersioning.h:
(gtk_scrolled_window_new):
Source/WebKit:
- UIProcess/API/gtk/WebKitScriptDialogImpl.cpp:
(webkitScriptDialogImplConstructed):
- UIProcess/gtk/WebDataListSuggestionsDropdownGtk.cpp:
(WebKit::WebDataListSuggestionsDropdownGtk::WebDataListSuggestionsDropdownGtk):
- UIProcess/gtk/WebPopupMenuProxyGtk.cpp:
(WebKit::WebPopupMenuProxyGtk::createPopupMenu):
Tools:
- MiniBrowser/gtk/BrowserSettingsDialog.c:
(browser_settings_dialog_init):
- 5:40 AM Changeset in webkit [263842] by
-
- 6 edits in trunk/LayoutTests
Remove expectation for fast/events/input-events-forecolor-data.html and fast/events/input-events-ime-composition.html and fast/events/input-events-selection-forecolor-data.html and fast/events/input-events-spell-checking-datatransfer.html and fast/events/offsetX-offsetY.html and fast/events/scale-and-scroll-iframe-body.html and fast/events/scale-and-scroll-iframe-window.html and fast/events/scroll-to-anchor-vertical-lr-writing-mode.html and fast/events/scroll-to-anchor-vertical-writing-mode.html and fast/forms/range/slider-transformed.html and fast/forms/range/slider-zoomed.html and fast/frames/content-opacity-1.html and fast/frames/content-opacity-2.html and fast/frames/iframe-text-contents.html and fast/frames/onlyCommentInIFrame.html and fast/frames/paint-iframe-background.html and fast/frames/sandboxed-iframe-close-top-noclose.html and fast/hidpi/image-srcset-invalid-descriptor.html and fast/hidpi/image-srcset-png-canvas.html and fast/html/listing.html as they are passing.
https://bugs.webkit.org/show_bug.cgi?id=213863
Unreviewed test gardening.
Patch by Karl Rackler <Karl Rackler> on 2020-07-02
- platform/ios-simulator-wk1/TestExpectations:
- platform/ios-simulator-wk2/TestExpectations:
- platform/ios-simulator/TestExpectations:
- platform/ios-wk2/TestExpectations:
- platform/ios/TestExpectations:
- 5:38 AM Changeset in webkit [263841] by
-
- 9 edits2 adds3 deletes in trunk
Fragment navigation involving fragment identifiers does not match specification
https://bugs.webkit.org/show_bug.cgi?id=83930
Patch by Rob Buis <rbuis@igalia.com> on 2020-07-02
Reviewed by Darin Adler.
LayoutTests/imported/w3c:
Update improved test results and import scroll-frag-non-utf8-encoded-document.html.
- web-platform-tests/html/browsers/browsing-the-web/scroll-to-fragid/fragment-and-encoding-2-expected.txt:
- web-platform-tests/html/browsers/browsing-the-web/scroll-to-fragid/fragment-and-encoding-expected.txt:
- web-platform-tests/html/browsers/browsing-the-web/scroll-to-fragid/scroll-frag-non-utf8-encoded-document-expected.txt: Added.
- web-platform-tests/html/browsers/browsing-the-web/scroll-to-fragid/scroll-frag-non-utf8-encoded-document.html: Added.
- web-platform-tests/html/browsers/browsing-the-web/scroll-to-fragid/scroll-to-anchor-name-expected.txt:
Source/WebCore:
Implement step 5 and 6 of [1] to allow fragment identifier lookup
using percent-decoding, after normal lookup failed.
Behavior matches Chrome and Firefox.
[1] https://html.spec.whatwg.org/multipage/browsing-the-web.html#the-indicated-part-of-the-document
Tests: imported/w3c/web-platform-tests/html/browsers/browsing-the-web/scroll-to-fragid/scroll-to-anchor-name.html
imported/w3c/web-platform-tests/html/browsers/browsing-the-web/scroll-to-fragid/fragment-and-encoding-2.html
imported/w3c/web-platform-tests/html/browsers/browsing-the-web/scroll-to-fragid/fragment-and-encoding.html
imported/w3c/web-platform-tests/html/browsers/browsing-the-web/scroll-to-fragid/scroll-frag-non-utf8-encoded-document.html
- page/FrameView.cpp:
(WebCore::FrameView::scrollToFragmentInternal):
LayoutTests:
Replace anchor-frames-gbk.html by scroll-frag-non-utf8-encoded-document.html.
- http/tests/navigation/anchor-frames-gbk-expected.txt: Removed.
- http/tests/navigation/anchor-frames-gbk.html: Removed.
- http/tests/navigation/resources/frame-with-anchor-gbk.html: Removed.
- platform/ios-wk1/TestExpectations:
- platform/ios-wk2/TestExpectations:
- 3:40 AM Changeset in webkit [263840] by
-
- 2 edits in trunk/LayoutTests
Unreviewed GTK multimedia gardening
https://bugs.webkit.org/show_bug.cgi?id=213877
Crashes in media/context-menu-actions.html are flaky instead of always reproducible.
- platform/gtk/TestExpectations:
- 2:26 AM Changeset in webkit [263839] by
-
- 3 edits1 add in trunk/Tools
[Flatpak SDK] Add perl-term-readkey
https://bugs.webkit.org/show_bug.cgi?id=213787
Patch by Philippe Normand <pnormand@igalia.com> on 2020-07-02
Reviewed by Adrian Perez de Castro.
This is yet another git-svn runtime dependency.
- buildstream/elements/sdk-platform.bst:
- buildstream/elements/sdk/perl-term-readkey.bst: Added.
- 2:19 AM Changeset in webkit [263838] by
-
- 4 edits in trunk/Source/WebCore
[GTK] Kinetic scrolling is still triggered if you wait before lifting fingers
https://bugs.webkit.org/show_bug.cgi?id=213789
Patch by Alexander Mikhaylenko <Alexander Mikhaylenko> on 2020-07-02
Reviewed by Carlos Garcia Campos.
The intended behavior for kinetic scrolling is that it's possible to not trigger it by
stopping moving fingers before lifting them, as opposed to doing a fling/swipe. However,
right now it's broken and trying to do this triggers kinetic scrolling as well.
Currently, the last scroll event with (0, 0) deltas isn't added to the scroll history for
kinetic scrolling. However, this is wrong, because the last event, or rather the timespan
between it and the previous scroll event, actually determines the kinetic scrolling velocity:
if this event follows the previous one quickly, the velocity is large, but if there was a long
pause, the velocity would be close to 0.
See https://gitlab.gnome.org/GNOME/gtk/-/merge_requests/1902 and
https://gitlab.gnome.org/GNOME/gtk/-/merge_requests/1903 for the equivalent GTK fixes.
- page/scrolling/nicosia/ScrollingTreeFrameScrollingNodeNicosia.cpp:
(WebCore::ScrollingTreeFrameScrollingNodeNicosia::handleWheelEvent):
- page/scrolling/nicosia/ScrollingTreeOverflowScrollingNodeNicosia.cpp:
(WebCore::ScrollingTreeOverflowScrollingNodeNicosia::handleWheelEvent):
- platform/generic/ScrollAnimatorGeneric.cpp:
(WebCore::ScrollAnimatorGeneric::handleWheelEvent):
- 1:47 AM Changeset in webkit [263837] by
-
- 4 edits1 add in trunk
[JSC] Configure option-offered numberingSystem in Intl.NumberFormat through locale
https://bugs.webkit.org/show_bug.cgi?id=213872
Reviewed by Ross Kirsling.
JSTests:
- stress/intl-numberformat-nu.js: Added.
(shouldBe):
- test262/expectations.yaml:
Source/JavaScriptCore:
We need to pass numberingSystem option to ICU through locale when constructing UNumberFormat.
We are passing it when we get "en-US-u-nu-hanidec" locale, but we are not passing it when
we are gettingnew Intl.NumberFormat("en-US", { numberingSystem: "hanidec" }).
- runtime/IntlNumberFormat.cpp:
(JSC::IntlNumberFormat::initializeNumberFormat):
- 1:20 AM Changeset in webkit [263836] by
-
- 4 edits in trunk/Source/WebCore
[GStreamer] Rewrite mediastreamsrc element
https://bugs.webkit.org/show_bug.cgi?id=213777
Reviewed by Xabier Rodriguez-Calvar.
The underlying functionality remains the same, design-wise. This is mostly a refactoring
switching to smart pointers as much as possible and trying to comply with WebKit code style
guidelines.
The previous version of this element was leaking pad templates and sticky stream-start
events (in webkitMediaStreamSrcTrackEnded).
The LIBWEBRTC ifdef guards were removed as well, because there's no actual libwebrtc
dependency involved in this code.
No new tests, existing webrtc and fast/mediastream tests cover this patch.
- platform/graphics/gstreamer/MediaPlayerPrivateGStreamer.cpp:
- platform/mediastream/gstreamer/GStreamerMediaStreamSource.cpp:
(mediaStreamTrackPrivateGetTags):
(webkitMediaStreamNew):
(WebKitMediaStreamTrackObserver::WebKitMediaStreamTrackObserver):
(WebKitMediaStreamObserver::WebKitMediaStreamObserver):
(InternalSource::InternalSource):
(InternalSource::~InternalSource):
(InternalSource::src const):
(InternalSource::pushSample):
(webkitMediaStreamSrcUriGetType):
(webkitMediaStreamSrcUriGetProtocols):
(webkitMediaStreamSrcUriGetUri):
(webkitMediaStreamSrcUriSetUri):
(webkitMediaStreamSrcUriHandlerInit):
(webkitMediaStreamSrcSetProperty):
(webkitMediaStreamSrcGetProperty):
(webkitMediaStreamSrcConstructed):
(stopObservingTracks):
(webkitMediaStreamSrcFinalize):
(webkitMediaStreamSrcChangeState):
(webkit_media_stream_src_class_init):
(webkitMediaStreamSrcChain):
(webkitMediaStreamSrcAddPad):
(ProbeData::ProbeData):
(webkitMediaStreamSrcPadProbeCb):
(webkitMediaStreamSrcSetupSrc):
(webkitMediaStreamSrcPostStreamCollection):
(webkitMediaStreamSrcAddTrack):
(webkitMediaStreamSrcRemoveTrackByType):
(webkitMediaStreamSrcSetStream):
(webkitMediaStreamSrcPushVideoSample):
(webkitMediaStreamSrcPushAudioSample):
(webkitMediaStreamSrcTrackEnded):
(webkitMediaStreamSrcNew):
- platform/mediastream/gstreamer/GStreamerMediaStreamSource.h:
- 1:14 AM Changeset in webkit [263835] by
-
- 5 edits in trunk/LayoutTests
[GTK] Layout test media/context-menu-actions.html is failing
https://bugs.webkit.org/show_bug.cgi?id=183611
Reviewed by Xabier Rodriguez-Calvar.
Changing to fullscreen is an async operation, so we now use testExpectedEventually() to wait for
the final value.
Simply setting an event handler on video.onwebkitfullscreenchange wouldn't have been enough, since the
event is triggered before the fullscreen operation has been completed and checking for
video.webkitDisplayingFullscreen == true would still fail by then.
Using the 'webkitbeginfullscreen' event isn't possible either, because it's never triggered when
FULLSCREEN_API is enabled.
- media/context-menu-actions.html: Use asynchronous testing.
- media/video-test.js:
(testExpectedEventually): Add an optional "timeout" parameter to avoid a test timeout (all the output
would be lost) in case we wait forever for the comparison to evaluate to the expected value.
(reportExpected): Add an optional "explanation" parameter for extra reporting.
- platform/gtk-wayland/TestExpectations: Unskipped the test.
- platform/gtk/TestExpectations: Ditto.
- 12:20 AM Changeset in webkit [263834] by
-
- 2 edits in trunk/LayoutTests
[GLIB] Unreviewed test gardening. Skip AX tests added in r263823 as they're timing out.
- platform/glib/TestExpectations:
Jul 1, 2020:
- 11:00 PM Changeset in webkit [263833] by
-
- 4 edits1 add in trunk
[JSC] Intl.Collator should set usage:"search" option through ICU locale
https://bugs.webkit.org/show_bug.cgi?id=213869
Reviewed by Ross Kirsling.
JSTests:
- stress/intl-collator-co-extension.js: Added.
(shouldBe):
(shouldBeArray):
(explicitTrueBeforeICU67):
- test262/expectations.yaml:
Source/JavaScriptCore:
Intl.Collator has usage:"search" option, and it affects on collation. However, UCollator does not have an interface to set this collation option,
and only way to configure UCollator is setting "-u-co-search" unicode extension to passed locale string. This patch adds "-u-co-search" unicode
extension if Usage::Search is specified.
- runtime/IntlCollator.cpp:
(JSC::IntlCollator::initializeCollator):
- 10:02 PM Changeset in webkit [263832] by
-
- 8 edits in trunk/Source
MIMETypeRegistry::getExtensionsForMIMEType() needs to handle wildcard MIME types
https://bugs.webkit.org/show_bug.cgi?id=213826
Patch by Said Abou-Hallawa <sabouhallawa@apple.com> on 2020-07-01
Reviewed by Darin Adler.
Source/WebCore:
Working towards webkit.org/b/213347, it needs to be possible for WebCore
to get the file extensions for wildcard MIME types, e.g. "image/*" or "video/*".
For Cocoa platforms, we will enumerate the UTIs of the system. Get the
MIMEType and the extensions of each UTI. Add the following pairs to a
singleton HashMap:
{ MIMEType, extension }
{ Type(MIMEType)/*, extension }
Change MIMETypeRegistry::getExtensionsForMIMEType() such that it calls
extensionsForWildcardMIMEType() if the MIMEType ends with "*".
- platform/MIMETypeRegistry.h:
- platform/cocoa/MIMETypeRegistryCocoa.mm:
(WebCore::extensionsForMIMETypeMap):
(WebCore::extensionsForWildcardMIMEType):
(WebCore::MIMETypeRegistry::getExtensionsForMIMEType):
- platform/playstation/MIMETypeRegistryPlayStation.cpp:
(WebCore::MIMETypeRegistry::getExtensionsForMIMEType):
- platform/win/MIMETypeRegistryWin.cpp:
(WebCore::MIMETypeRegistry::getExtensionsForMIMEType):
- platform/xdg/MIMETypeRegistryXdg.cpp:
(WebCore::MIMETypeRegistry::getExtensionsForMIMEType):
Source/WebKit:
Replace extensionsForMIMEType() with MIMETypeRegistry::getExtensionsForMIMEType().
- UIProcess/API/Cocoa/WKOpenPanelParameters.mm:
(-[WKOpenPanelParameters _allowedFileExtensions]):
- 9:04 PM Changeset in webkit [263831] by
-
- 2 edits in trunk/Source/WebKit
[SOUP] Build fix after r263797 for older soup versions.
Build was failing in LTS distributions due to -fpermissive
Unreviewed build fix.
- NetworkProcess/soup/WebSocketTaskSoup.cpp:
(WebKit::WebSocketTask::sendString):
- 8:08 PM Changeset in webkit [263830] by
-
- 31 edits in trunk/Source
Allow the File object to be created with a replacement file
https://bugs.webkit.org/show_bug.cgi?id=213825
Patch by Said Abou-Hallawa <sabouhallawa@apple.com> on 2020-07-01
Reviewed by Darin Adler.
Source/WebCore:
Working towards webkit.org/b/213347, it needs to be possible to create
the File object with an optional replacement file. Only the registered
BlobDataFileReference will be created with both the original file path
and the replacement file path. So it can delete the replacement file when
it is destroyed. Otherwise BlobDataFileReference will be created with the
replacement file path.
It is important to create the File object with the replacement file because
it needs to get the meta-data and the bytes of the replacement file not
the original file.
- fileapi/File.cpp:
(WebCore::File::create):
- fileapi/File.h:
- fileapi/ThreadableBlobRegistry.cpp:
(WebCore::ThreadableBlobRegistry::registerFileBlobURL):
- fileapi/ThreadableBlobRegistry.h:
- html/DirectoryFileListCreator.cpp:
(WebCore::createFileList):
- html/FileInputType.cpp:
(WebCore::FileInputType::filesFromFormControlState):
When the Files are created from a FormControlState, they will be created
without replacement files since they might have been deleted.
(WebCore::FileInputType::filesChosen):
(WebCore::FileInputType::receiveDroppedFiles):
- platform/FileChooser.cpp:
(WebCore::FileChooser::chooseFiles):
(WebCore::FileChooser::chooseMediaFiles):
- platform/FileChooser.h:
(WebCore::FileChooserFileInfo::isolatedCopy const):
(WebCore::FileChooser::chooseFiles):
(WebCore::FileChooserFileInfo::FileChooserFileInfo): Deleted.
- platform/network/BlobDataFileReference.cpp:
(WebCore::BlobDataFileReference::BlobDataFileReference):
(WebCore::BlobDataFileReference::~BlobDataFileReference):
(WebCore::BlobDataFileReference::path):
(WebCore::BlobDataFileReference::startTrackingModifications):
- platform/network/BlobDataFileReference.h:
- platform/network/BlobRegistry.h:
Source/WebKit:
The UIProcess passes a list of strings which represents the replacement
paths along with a list to the original paths to the WebProcess. The
WebProcess passes these two list to FileChooser which creates the File
objects and register the Blobs.
The WebProcess registers the Blobs in the NetworkProcess which creates
BlobDataFileReference objects with both the original path and the
replacement path.
The WebProcess unregisters the Blobs from the NetworkProcess which deletes
the corresponding BlobDataFileReference from its registry. Upon destroying
the BlobDataFileReference, the replacement file should be deleted.
- NetworkProcess/NetworkConnectionToWebProcess.cpp:
(WebKit::NetworkConnectionToWebProcess::registerFileBlobURL):
(WebKit::NetworkConnectionToWebProcess::registerBlobURLOptionallyFileBacked):
- NetworkProcess/NetworkConnectionToWebProcess.h:
- NetworkProcess/NetworkConnectionToWebProcess.messages.in:
- NetworkProcess/NetworkProcessPlatformStrategies.cpp:
(WebKit::NetworkProcessPlatformStrategies::createBlobRegistry):
- Shared/BlobDataFileReferenceWithSandboxExtension.cpp:
(WebKit::BlobDataFileReferenceWithSandboxExtension::BlobDataFileReferenceWithSandboxExtension):
- Shared/BlobDataFileReferenceWithSandboxExtension.h:
- UIProcess/WebPageProxy.cpp:
(WebKit::WebPageProxy::didChooseFilesForOpenPanel):
- WebProcess/FileAPI/BlobRegistryProxy.cpp:
(WebKit::BlobRegistryProxy::registerFileBlobURL):
- WebProcess/FileAPI/BlobRegistryProxy.h:
- WebProcess/WebPage/WebOpenPanelResultListener.cpp:
(WebKit::WebOpenPanelResultListener::didChooseFiles):
- WebProcess/WebPage/WebOpenPanelResultListener.h:
- WebProcess/WebPage/WebPage.cpp:
(WebKit::WebPage::didChooseFilesForOpenPanel):
- WebProcess/WebPage/WebPage.h:
- WebProcess/WebPage/WebPage.messages.in:
Source/WebKitLegacy/mac:
- WebCoreSupport/WebPlatformStrategies.mm:
Source/WebKitLegacy/win:
- WebCoreSupport/WebPlatformStrategies.cpp:
- 7:58 PM Changeset in webkit [263829] by
-
- 5 edits1 add1 delete in trunk
[CMake] Add WOFF2 targets
https://bugs.webkit.org/show_bug.cgi?id=213865
Reviewed by Fujii Hironori.
.:
Rework the Find module for WOFF2 to provide targets for libwoff2common and
libwoff2dec. Previously only libwoff2dec was specified in the find module. This
is because pkg-config works on that platform and because shared libraries are
supported. Neither is true for Windows so the module was expanded in anticipation
of support for WinCairo.
- Source/cmake/FindWOFF2.cmake: Added.
- Source/cmake/FindWOFF2Dec.cmake: Removed.
- Source/cmake/OptionsGTK.cmake:
- Source/cmake/OptionsWPE.cmake:
Source/WebCore:
Use the WOFF:: targets when USE_WOFF2 is set.
- CMakeLists.txt:
- 6:26 PM Changeset in webkit [263828] by
-
- 3 edits in trunk/Source/ThirdParty/libwebrtc
Fix the arm64 iOS simulator build
https://bugs.webkit.org/show_bug.cgi?id=213868
<rdar://problem/64901603>
- Configurations/libvpx.xcconfig:
- Configurations/libwebrtc.xcconfig:
- 5:43 PM Changeset in webkit [263827] by
-
- 3 edits in branches/safari-610.1.20-branch/Source/WebKit
Cherry-pick r263793. rdar://problem/65011200
Fix Swift overlay build after r263727.
https://bugs.webkit.org/show_bug.cgi?id=213823.
Unreviewed, follow up build fix.
- SwiftOverlay/Tests/WebKitTests.swift: (WebKitTests.testAPI): Fix tests to refer to new parameter name.
- UIProcess/API/Cocoa/WebKitSwiftOverlay.swift: (callAsyncJavaScript(_:arguments:in:in:completionHandler:Error:)): Use the imported ObjC name that the compiler expects. (evaluateJavaScript(_:in:in:completionHandler:Error:)): Ditto.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@263793 268f45cc-cd09-0410-ab3c-d52691b4dbfc
- 5:43 PM Changeset in webkit [263826] by
-
- 2 edits in branches/safari-610.1.20-branch/Source/WebKit
Cherry-pick r263781. rdar://problem/65011200
Fix Swift overlay build after r263727.
<rdar://problem/64962370> and https://bugs.webkit.org/show_bug.cgi?id=213823
Reviewed by Darin Adler.
No new tests (No behavior change).
Adapt the Swift overlay refinemnt to handle the inFrame: argument for these methods.
- UIProcess/API/Cocoa/WebKitSwiftOverlay.swift: (callAsyncJavaScript(_:arguments:in:in:completionHandler:Error:)): (evaluateJavaScript(_:in:in:completionHandler:Error:)): (callAsyncJavaScript(_:arguments:in:completionHandler:Error:)): Deleted. (evaluateJavaScript(_:in:completionHandler:Error:)): Deleted.
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@263781 268f45cc-cd09-0410-ab3c-d52691b4dbfc
- 5:32 PM Changeset in webkit [263825] by
-
- 28 edits5 adds in trunk
Swipe snapshot is removed too early when swiping away from a page that is still loading
https://bugs.webkit.org/show_bug.cgi?id=213763
<rdar://problem/64576811>
Reviewed by Darin Adler.
Source/WebKit:
Test: http/tests/swipe/swipe-back-with-outstanding-load-cancellation.html
When swiping back from a page that is still loading, when the swipe commits,
and we start navigating back to the previous page, the unfinished forward navigation
is cancelled.
This cancellation resulted in a "didFailLoadForMainFrame" making it to ViewGestureController,
which took it as an indication that the navigation *it* was interested in (the back navigation)
had failed, and immediately removed the snapshot.
Instead of listening to any random navigation's load notifications, keep track of the navigation
started by goToBackForwardItem, and only listen to notifications from it, ignoring the others.
This requires a bunch of plumbing to get the navigation from WebPageProxy to ViewGestureController,
but is otherwise fairly trivial.
- UIProcess/API/gtk/PageClientImpl.cpp:
(WebKit::PageClientImpl::didFinishNavigation):
(WebKit::PageClientImpl::didFailNavigation):
(WebKit::PageClientImpl::didFinishLoadForMainFrame): Deleted.
(WebKit::PageClientImpl::didFailLoadForMainFrame): Deleted.
- UIProcess/API/gtk/PageClientImpl.h:
- UIProcess/API/gtk/WebKitWebViewBase.cpp:
(webkitWebViewBaseDidFinishLoadForMainFrame):
(webkitWebViewBaseDidFailLoadForMainFrame):
- UIProcess/API/gtk/WebKitWebViewBasePrivate.h:
- UIProcess/API/ios/WKWebViewIOS.h:
- UIProcess/API/ios/WKWebViewIOS.mm:
(-[WKWebView _didFinishNavigation:]):
(-[WKWebView _didFailNavigation:]):
(-[WKWebView _didFinishLoadForMainFrame]): Deleted.
(-[WKWebView _didFailLoadForMainFrame]): Deleted.
- UIProcess/API/wpe/PageClientImpl.cpp:
(WebKit::PageClientImpl::didFinishNavigation):
(WebKit::PageClientImpl::didFailNavigation):
(WebKit::PageClientImpl::didFinishLoadForMainFrame): Deleted.
(WebKit::PageClientImpl::didFailLoadForMainFrame): Deleted.
- UIProcess/API/wpe/PageClientImpl.h:
- UIProcess/PageClient.h:
- UIProcess/ViewGestureController.cpp:
(WebKit::ViewGestureController::didReachNavigationTerminalState):
(WebKit::ViewGestureController::willEndSwipeGesture):
(WebKit::ViewGestureController::didReachMainFrameLoadTerminalState): Deleted.
- UIProcess/ViewGestureController.h:
(WebKit::ViewGestureController::didFinishNavigation):
(WebKit::ViewGestureController::didFailNavigation):
(WebKit::ViewGestureController::didFinishLoadForMainFrame): Deleted.
(WebKit::ViewGestureController::didFailLoadForMainFrame): Deleted.
- UIProcess/WebPageProxy.cpp:
(WebKit::WebPageProxy::didFinishLoadForFrame):
(WebKit::WebPageProxy::didFailLoadForFrame):
(WebKit::WebPageProxy::didSameDocumentNavigationForFrame):
- UIProcess/ios/PageClientImplIOS.h:
- UIProcess/ios/PageClientImplIOS.mm:
(WebKit::PageClientImpl::didFinishNavigation):
(WebKit::PageClientImpl::didFailNavigation):
(WebKit::PageClientImpl::didFinishLoadForMainFrame): Deleted.
(WebKit::PageClientImpl::didFailLoadForMainFrame): Deleted.
- UIProcess/ios/ViewGestureControllerIOS.mm:
(WebKit::ViewGestureController::willEndSwipeGesture):
(WebKit::ViewGestureController::resetState):
- UIProcess/mac/PageClientImplMac.h:
- UIProcess/mac/PageClientImplMac.mm:
(WebKit::PageClientImpl::didFinishNavigation):
(WebKit::PageClientImpl::didFailNavigation):
(WebKit::PageClientImpl::didFinishLoadForMainFrame): Deleted.
(WebKit::PageClientImpl::didFailLoadForMainFrame): Deleted.
- UIProcess/mac/ViewGestureControllerMac.mm:
(WebKit::ViewGestureController::resetState):
- UIProcess/playstation/PageClientImpl.cpp:
(WebKit::PageClientImpl::didFinishNavigation):
(WebKit::PageClientImpl::didFailNavigation):
(WebKit::PageClientImpl::didFinishLoadForMainFrame): Deleted.
(WebKit::PageClientImpl::didFailLoadForMainFrame): Deleted.
- UIProcess/playstation/PageClientImpl.h:
- UIProcess/win/PageClientImpl.cpp:
(WebKit::PageClientImpl::didFinishNavigation):
(WebKit::PageClientImpl::didFinishLoadForMainFrame): Deleted.
- UIProcess/win/PageClientImpl.h:
LayoutTests:
- http/tests/swipe/resources/swipe-test.js: Added.
- http/tests/swipe/swipe-back-with-outstanding-load-cancellation-expected.txt: Added.
- http/tests/swipe/swipe-back-with-outstanding-load-cancellation.html: Added.
Add a test that ensures that we don't remove the swipe snapshot before the "back" page is loaded.
Before this fix, this test fails the assertion that we're on the first page in didRemoveSwipeSnapshotCallback.
- 4:50 PM Changeset in webkit [263824] by
-
- 21 edits in trunk/Source/JavaScriptCore
Rename zeroExtend32ToPtr to zeroExtend32ToWord
https://bugs.webkit.org/show_bug.cgi?id=213866
Reviewed by Saam Barati.
The old name no longer makes sense now that we have configurations
where sizeof(void*) != sizeof(CPURegister).
- assembler/MacroAssemblerARM64.h:
(JSC::MacroAssemblerARM64::branchTruncateDoubleToInt32):
(JSC::MacroAssemblerARM64::zeroExtend32ToWord):
(JSC::MacroAssemblerARM64::branchMul32):
(JSC::MacroAssemblerARM64::zeroExtend32ToPtr): Deleted.
- assembler/MacroAssemblerARMv7.h:
(JSC::MacroAssemblerARMv7::zeroExtend32ToWord):
(JSC::MacroAssemblerARMv7::zeroExtend32ToPtr): Deleted.
- assembler/MacroAssemblerMIPS.h:
(JSC::MacroAssemblerMIPS::zeroExtend32ToWord):
(JSC::MacroAssemblerMIPS::zeroExtend32ToPtr): Deleted.
- assembler/MacroAssemblerX86Common.h:
(JSC::MacroAssemblerX86Common::add32):
(JSC::MacroAssemblerX86Common::and32):
(JSC::MacroAssemblerX86Common::mul32):
(JSC::MacroAssemblerX86Common::or32):
(JSC::MacroAssemblerX86Common::xor32):
(JSC::MacroAssemblerX86Common::zeroExtend32ToWord):
(JSC::MacroAssemblerX86Common::branchAdd32):
(JSC::MacroAssemblerX86Common::zeroExtend32ToPtr): Deleted.
- b3/air/AirOpcode.opcodes:
- bytecode/AccessCase.cpp:
(JSC::AccessCase::generateWithGuard):
- dfg/DFGOSRExit.cpp:
(JSC::DFG::OSRExit::compileExit):
- dfg/DFGSpeculativeJIT.cpp:
(JSC::DFG::SpeculativeJIT::compileStringSlice):
(JSC::DFG::SpeculativeJIT::compileValueToInt32):
(JSC::DFG::SpeculativeJIT::compileUInt32ToNumber):
(JSC::DFG::SpeculativeJIT::setIntTypedArrayLoadResult):
(JSC::DFG::SpeculativeJIT::compileArithDiv):
(JSC::DFG::SpeculativeJIT::compileCreateRest):
(JSC::DFG::SpeculativeJIT::compileArraySlice):
(JSC::DFG::SpeculativeJIT::compileArrayIndexOf):
(JSC::DFG::SpeculativeJIT::compileNewTypedArrayWithSize):
(JSC::DFG::SpeculativeJIT::emitAllocateButterfly):
(JSC::DFG::SpeculativeJIT::compileWeakMapGet):
(JSC::DFG::SpeculativeJIT::emitInitializeButterfly):
- dfg/DFGSpeculativeJIT64.cpp:
(JSC::DFG::SpeculativeJIT::fillSpeculateInt32Internal):
(JSC::DFG::SpeculativeJIT::compile):
- ftl/FTLOSRExitCompiler.cpp:
(JSC::FTL::reboxAccordingToFormat):
- jit/CallFrameShuffler64.cpp:
(JSC::CallFrameShuffler::emitBox):
- jit/JITOpcodes.cpp:
(JSC::JIT::emit_op_has_indexed_property):
- jit/JITOpcodes32_64.cpp:
(JSC::JIT::emit_op_has_indexed_property):
- jit/JITPropertyAccess.cpp:
(JSC::JIT::emit_op_put_by_val):
- jit/JSInterfaceJIT.h:
(JSC::JSInterfaceJIT::emitLoadInt32):
- wasm/js/JSToWasm.cpp:
(JSC::Wasm::boxWasmResult):
- wasm/js/WasmToJS.cpp:
(JSC::Wasm::wasmToJS):
- wasm/js/WebAssemblyFunction.cpp:
(JSC::WebAssemblyFunction::jsCallEntrypointSlow):
- yarr/YarrJIT.cpp:
- 4:35 PM Changeset in webkit [263823] by
-
- 15 edits4 adds in trunk
AX: Implement relevant simulated key presses for custom ARIA widgets for increment/decrement
https://bugs.webkit.org/show_bug.cgi?id=213744
Reviewed by Darin Adler.
Source/WebCore:
In order to allow custom ARIA widgets to work, we can post keyboard events for specific ax actions
that are not handled natively.
Spec: https://github.com/WICG/aom/blob/gh-pages/explainer.md#user-action-events-from-assistive-technology
Test: accessibility/keyevents-posted-for-increment-actions.html
accessibility/keyevents-for-increment-actions-with-node-removal.html
- accessibility/AccessibilityNodeObject.cpp:
(WebCore::AccessibilityNodeObject::postKeyboardKeysForValueChange):
(WebCore::AccessibilityNodeObject::setNodeValue):
(WebCore::AccessibilityNodeObject::changeValueByStep):
(WebCore::AccessibilityNodeObject::changeValueByPercent):
- accessibility/AccessibilityNodeObject.h:
- accessibility/AccessibilityObject.h:
- accessibility/AccessibilityObjectInterface.h:
- accessibility/AccessibilityRenderObject.cpp:
(WebCore::AccessibilityRenderObject::setValue):
- accessibility/AccessibilityRenderObject.h:
- accessibility/AccessibilityScrollbar.cpp:
(WebCore::AccessibilityScrollbar::setValue):
- accessibility/AccessibilityScrollbar.h:
- accessibility/AccessibilitySlider.cpp:
(WebCore::AccessibilitySlider::setValue):
- accessibility/AccessibilitySlider.h:
- accessibility/isolatedtree/AXIsolatedObject.cpp:
(WebCore::AXIsolatedObject::setValue):
- accessibility/isolatedtree/AXIsolatedObject.h:
LayoutTests:
- accessibility/keyevents-posted-for-increment-actions-expected.txt: Added.
- accessibility/keyevents-posted-for-increment-actions.html: Added.
- accessibility/keyevents-for-increment-actions-with-node-removal-expected.txt: Added.
- accessibility/keyevents-for-increment-actions-with-node-removal.htmk: Added.
- 2:46 PM Changeset in webkit [263822] by
-
- 4 edits in trunk/LayoutTests
[ iOS Debug and Mojave Debug ] http/tests/storage/storage-map-leaking.html is flaky timing out.
https://bugs.webkit.org/show_bug.cgi?id=213795
<rdar://problem/64937993>
Reviewed by Darin Adler.
Because our GC is conservative, a particular JS wrapper is not guaranteed to go away in a timely fashion.
To try and make the test more reliable, we now use 4 separate Storage wrappers instead of a single one.
We treat the test as passing if any of the 4 storage wrappers go away after removing the 4 subframes from
4 separate origins.
- http/tests/storage/storage-map-leaking.html:
- platform/ios-wk2/TestExpectations:
- platform/mac-wk2/TestExpectations:
- 2:04 PM Changeset in webkit [263821] by
-
- 2 edits in trunk/Source/WebKit
[iOS] Allow the WebContent sandbox to check the 'hw.product' sysctl
https://bugs.webkit.org/show_bug.cgi?id=213861
<rdar://problem/64988957>
Reviewed by Per Arne Vollan.
Telemetry shows that we need to read the 'hw.product' sysctl in the font subsystem,
primarily to enable certain font features (for various products) in different locales.
- Resources/SandboxProfiles/ios/com.apple.WebKit.WebContent.sb:
- 2:01 PM Changeset in webkit [263820] by
-
- 837 edits21 copies9 moves105 adds16 deletes in trunk
Add a functional WebRTC VP9 codec
https://bugs.webkit.org/show_bug.cgi?id=213778
Reviewed by Eric Carlson.
Source/ThirdParty/libwebrtc:
Update libvpx to synchronize VP9 support with the rest of libwebrtc.
Disable vp9_noop in the build system and use function vp9 encoder/decoder instead.
- Configurations/libvpx.xcconfig:
- Source/third_party/libvpx/BUILD.gn:
- Source/third_party/libvpx/OWNERS:
- Source/third_party/libvpx/README.chromium:
- Source/third_party/libvpx/generate_gni.sh:
- Source/third_party/libvpx/libvpx_srcs.gni:
- Source/third_party/libvpx/lint_config.sh:
- Source/third_party/libvpx/run_perl.py:
- Source/third_party/libvpx/source/config/ios/arm-neon/vp8_rtcd.h:
- Source/third_party/libvpx/source/config/ios/arm-neon/vp9_rtcd.h:
- Source/third_party/libvpx/source/config/ios/arm-neon/vpx_config.asm:
- Source/third_party/libvpx/source/config/ios/arm-neon/vpx_config.c:
- Source/third_party/libvpx/source/config/ios/arm-neon/vpx_config.h:
- Source/third_party/libvpx/source/config/ios/arm-neon/vpx_dsp_rtcd.h:
- Source/third_party/libvpx/source/config/ios/arm64/vp8_rtcd.h:
- Source/third_party/libvpx/source/config/ios/arm64/vp9_rtcd.h:
- Source/third_party/libvpx/source/config/ios/arm64/vpx_config.asm:
- Source/third_party/libvpx/source/config/ios/arm64/vpx_config.c:
- Source/third_party/libvpx/source/config/ios/arm64/vpx_config.h:
- Source/third_party/libvpx/source/config/ios/arm64/vpx_dsp_rtcd.h:
- Source/third_party/libvpx/source/config/linux/arm-neon-cpu-detect/vp8_rtcd.h:
- Source/third_party/libvpx/source/config/linux/arm-neon-cpu-detect/vp9_rtcd.h:
- Source/third_party/libvpx/source/config/linux/arm-neon-cpu-detect/vpx_config.asm:
- Source/third_party/libvpx/source/config/linux/arm-neon-cpu-detect/vpx_config.c:
- Source/third_party/libvpx/source/config/linux/arm-neon-cpu-detect/vpx_config.h:
- Source/third_party/libvpx/source/config/linux/arm-neon-cpu-detect/vpx_dsp_rtcd.h:
- Source/third_party/libvpx/source/config/linux/arm-neon-highbd/vp8_rtcd.h: Copied from Source/ThirdParty/libwebrtc/Source/third_party/libvpx/source/config/linux/arm-neon/vp8_rtcd.h.
- Source/third_party/libvpx/source/config/linux/arm-neon-highbd/vp9_rtcd.h: Copied from Source/ThirdParty/libwebrtc/Source/third_party/libvpx/source/config/linux/generic/vp9_rtcd.h.
- Source/third_party/libvpx/source/config/linux/arm-neon-highbd/vpx_config.asm: Copied from Source/ThirdParty/libwebrtc/Source/third_party/libvpx/source/config/linux/arm-neon/vpx_config.asm.
- Source/third_party/libvpx/source/config/linux/arm-neon-highbd/vpx_config.c: Copied from Source/ThirdParty/libwebrtc/Source/third_party/libvpx/source/config/ios/arm-neon/vpx_config.c.
(vpx_codec_build_config):
- Source/third_party/libvpx/source/config/linux/arm-neon-highbd/vpx_config.h: Copied from Source/ThirdParty/libwebrtc/Source/third_party/libvpx/source/config/linux/arm-neon/vpx_config.h.
- Source/third_party/libvpx/source/config/linux/arm-neon-highbd/vpx_dsp_rtcd.h: Copied from Source/ThirdParty/libwebrtc/Source/third_party/libvpx/source/config/linux/generic/vpx_dsp_rtcd.h.
- Source/third_party/libvpx/source/config/linux/arm-neon-highbd/vpx_scale_rtcd.h: Added.
- Source/third_party/libvpx/source/config/linux/arm-neon/vp8_rtcd.h:
- Source/third_party/libvpx/source/config/linux/arm-neon/vp9_rtcd.h:
- Source/third_party/libvpx/source/config/linux/arm-neon/vpx_config.asm:
- Source/third_party/libvpx/source/config/linux/arm-neon/vpx_config.c:
- Source/third_party/libvpx/source/config/linux/arm-neon/vpx_config.h:
- Source/third_party/libvpx/source/config/linux/arm-neon/vpx_dsp_rtcd.h:
- Source/third_party/libvpx/source/config/linux/arm/vp8_rtcd.h:
- Source/third_party/libvpx/source/config/linux/arm/vp9_rtcd.h:
- Source/third_party/libvpx/source/config/linux/arm/vpx_config.asm:
- Source/third_party/libvpx/source/config/linux/arm/vpx_config.c:
- Source/third_party/libvpx/source/config/linux/arm/vpx_config.h:
- Source/third_party/libvpx/source/config/linux/arm/vpx_dsp_rtcd.h:
- Source/third_party/libvpx/source/config/linux/arm64-highbd/vp8_rtcd.h: Copied from Source/ThirdParty/libwebrtc/Source/third_party/libvpx/source/config/linux/arm-neon/vp8_rtcd.h.
- Source/third_party/libvpx/source/config/linux/arm64-highbd/vp9_rtcd.h: Copied from Source/ThirdParty/libwebrtc/Source/third_party/libvpx/source/config/linux/generic/vp9_rtcd.h.
- Source/third_party/libvpx/source/config/linux/arm64-highbd/vpx_config.asm: Copied from Source/ThirdParty/libwebrtc/Source/third_party/libvpx/source/config/linux/arm64/vpx_config.asm.
- Source/third_party/libvpx/source/config/linux/arm64-highbd/vpx_config.c: Copied from Source/ThirdParty/libwebrtc/Source/third_party/libvpx/source/config/ios/arm64/vpx_config.c.
(vpx_codec_build_config):
- Source/third_party/libvpx/source/config/linux/arm64-highbd/vpx_config.h: Copied from Source/ThirdParty/libwebrtc/Source/third_party/libvpx/source/config/linux/arm64/vpx_config.h.
- Source/third_party/libvpx/source/config/linux/arm64-highbd/vpx_dsp_rtcd.h: Copied from Source/ThirdParty/libwebrtc/Source/third_party/libvpx/source/config/linux/generic/vpx_dsp_rtcd.h.
- Source/third_party/libvpx/source/config/linux/arm64-highbd/vpx_scale_rtcd.h: Added.
- Source/third_party/libvpx/source/config/linux/arm64/vp8_rtcd.h:
- Source/third_party/libvpx/source/config/linux/arm64/vp9_rtcd.h:
- Source/third_party/libvpx/source/config/linux/arm64/vpx_config.asm:
- Source/third_party/libvpx/source/config/linux/arm64/vpx_config.c:
- Source/third_party/libvpx/source/config/linux/arm64/vpx_config.h:
- Source/third_party/libvpx/source/config/linux/arm64/vpx_dsp_rtcd.h:
- Source/third_party/libvpx/source/config/linux/generic/vp8_rtcd.h:
- Source/third_party/libvpx/source/config/linux/generic/vp9_rtcd.h:
- Source/third_party/libvpx/source/config/linux/generic/vpx_config.asm:
- Source/third_party/libvpx/source/config/linux/generic/vpx_config.c:
- Source/third_party/libvpx/source/config/linux/generic/vpx_config.h:
- Source/third_party/libvpx/source/config/linux/generic/vpx_dsp_rtcd.h:
- Source/third_party/libvpx/source/config/linux/ia32/vp8_rtcd.h:
- Source/third_party/libvpx/source/config/linux/ia32/vp9_rtcd.h:
- Source/third_party/libvpx/source/config/linux/ia32/vpx_config.asm:
- Source/third_party/libvpx/source/config/linux/ia32/vpx_config.c:
- Source/third_party/libvpx/source/config/linux/ia32/vpx_config.h:
- Source/third_party/libvpx/source/config/linux/ia32/vpx_dsp_rtcd.h:
- Source/third_party/libvpx/source/config/linux/mips64el/vp8_rtcd.h:
- Source/third_party/libvpx/source/config/linux/mips64el/vp9_rtcd.h:
- Source/third_party/libvpx/source/config/linux/mips64el/vpx_config.c:
- Source/third_party/libvpx/source/config/linux/mips64el/vpx_config.h:
- Source/third_party/libvpx/source/config/linux/mips64el/vpx_dsp_rtcd.h:
- Source/third_party/libvpx/source/config/linux/mipsel/vp8_rtcd.h:
- Source/third_party/libvpx/source/config/linux/mipsel/vp9_rtcd.h:
- Source/third_party/libvpx/source/config/linux/mipsel/vpx_config.c:
- Source/third_party/libvpx/source/config/linux/mipsel/vpx_config.h:
- Source/third_party/libvpx/source/config/linux/mipsel/vpx_dsp_rtcd.h:
- Source/third_party/libvpx/source/config/linux/x64/vp8_rtcd.h:
- Source/third_party/libvpx/source/config/linux/x64/vp9_rtcd.h:
- Source/third_party/libvpx/source/config/linux/x64/vpx_config.asm:
- Source/third_party/libvpx/source/config/linux/x64/vpx_config.c:
- Source/third_party/libvpx/source/config/linux/x64/vpx_config.h:
- Source/third_party/libvpx/source/config/linux/x64/vpx_dsp_rtcd.h:
- Source/third_party/libvpx/source/config/mac/ia32/vp8_rtcd.h:
- Source/third_party/libvpx/source/config/mac/ia32/vp9_rtcd.h:
- Source/third_party/libvpx/source/config/mac/ia32/vpx_config.asm:
- Source/third_party/libvpx/source/config/mac/ia32/vpx_config.c:
- Source/third_party/libvpx/source/config/mac/ia32/vpx_config.h:
- Source/third_party/libvpx/source/config/mac/ia32/vpx_dsp_rtcd.h:
- Source/third_party/libvpx/source/config/mac/x64/vp8_rtcd.h:
- Source/third_party/libvpx/source/config/mac/x64/vp8_rtcd_no_acceleration.h:
- Source/third_party/libvpx/source/config/mac/x64/vp9_rtcd.h:
- Source/third_party/libvpx/source/config/mac/x64/vpx_config.asm:
- Source/third_party/libvpx/source/config/mac/x64/vpx_config.c:
- Source/third_party/libvpx/source/config/mac/x64/vpx_config.h:
- Source/third_party/libvpx/source/config/mac/x64/vpx_dsp_rtcd.h:
- Source/third_party/libvpx/source/config/mac/x64/vpx_dsp_rtcd_no_acceleration.h:
- Source/third_party/libvpx/source/config/nacl/vp8_rtcd.h:
- Source/third_party/libvpx/source/config/nacl/vp9_rtcd.h:
- Source/third_party/libvpx/source/config/nacl/vpx_config.c:
- Source/third_party/libvpx/source/config/nacl/vpx_config.h:
- Source/third_party/libvpx/source/config/nacl/vpx_dsp_rtcd.h:
- Source/third_party/libvpx/source/config/vpx_version.h:
- Source/third_party/libvpx/source/config/win/arm64/vp8_rtcd.h: Copied from Source/ThirdParty/libwebrtc/Source/third_party/libvpx/source/config/linux/arm-neon/vp8_rtcd.h.
- Source/third_party/libvpx/source/config/win/arm64/vp9_rtcd.h: Copied from Source/ThirdParty/libwebrtc/Source/third_party/libvpx/source/config/linux/generic/vp9_rtcd.h.
- Source/third_party/libvpx/source/config/win/arm64/vpx_config.asm: Copied from Source/ThirdParty/libwebrtc/Source/third_party/libvpx/source/config/ios/arm64/vpx_config.asm.
- Source/third_party/libvpx/source/config/win/arm64/vpx_config.c: Copied from Source/ThirdParty/libwebrtc/Source/third_party/libvpx/source/config/linux/mips64el/vpx_config.c.
(vpx_codec_build_config):
- Source/third_party/libvpx/source/config/win/arm64/vpx_config.h: Copied from Source/ThirdParty/libwebrtc/Source/third_party/libvpx/source/config/linux/arm64/vpx_config.h.
- Source/third_party/libvpx/source/config/win/arm64/vpx_dsp_rtcd.h: Copied from Source/ThirdParty/libwebrtc/Source/third_party/libvpx/source/config/linux/generic/vpx_dsp_rtcd.h.
- Source/third_party/libvpx/source/config/win/arm64/vpx_scale_rtcd.h: Added.
- Source/third_party/libvpx/source/config/win/ia32/vp8_rtcd.h:
- Source/third_party/libvpx/source/config/win/ia32/vp9_rtcd.h:
- Source/third_party/libvpx/source/config/win/ia32/vpx_config.asm:
- Source/third_party/libvpx/source/config/win/ia32/vpx_config.c:
- Source/third_party/libvpx/source/config/win/ia32/vpx_config.h:
- Source/third_party/libvpx/source/config/win/ia32/vpx_dsp_rtcd.h:
- Source/third_party/libvpx/source/config/win/x64/vp8_rtcd.h:
- Source/third_party/libvpx/source/config/win/x64/vp9_rtcd.h:
- Source/third_party/libvpx/source/config/win/x64/vpx_config.asm:
- Source/third_party/libvpx/source/config/win/x64/vpx_config.c:
- Source/third_party/libvpx/source/config/win/x64/vpx_config.h:
- Source/third_party/libvpx/source/config/win/x64/vpx_dsp_rtcd.h:
- libwebrtc.xcodeproj/project.pbxproj:
LayoutTests:
- webrtc/vp9-expected.txt:
- webrtc/vp9.html:
- 1:29 PM Changeset in webkit [263819] by
-
- 15 edits in trunk/Source
Upstream application accent color support
https://bugs.webkit.org/show_bug.cgi?id=213859
Reviewed by Wenson Hsieh.
Source/WebCore/PAL:
- pal/spi/mac/NSApplicationSPI.h:
Source/WebKit:
- Shared/WebPageCreationParameters.cpp:
(WebKit::WebPageCreationParameters::encode const):
(WebKit::WebPageCreationParameters::decode):
- Shared/WebPageCreationParameters.h:
- UIProcess/Cocoa/PageClientImplCocoa.h:
- UIProcess/PageClient.h:
- UIProcess/WebPageProxy.cpp:
(WebKit::WebPageProxy::creationParameters):
- UIProcess/mac/PageClientImplMac.h:
- UIProcess/mac/PageClientImplMac.mm:
(WebKit::PageClientImpl::accentColor):
- WebProcess/WebPage/WebPage.cpp:
(WebKit::m_limitsNavigationsToAppBoundDomains):
- WebProcess/WebPage/WebPage.h:
- WebProcess/WebPage/mac/WebPageMac.mm:
(WebKit::WebPage::setAccentColor):
Plumb accent color to the Web Content process so that it is used when
deciding the colors of selection and form controls.
Source/WTF:
- wtf/PlatformHave.h:
- 1:16 PM Changeset in webkit [263818] by
-
- 4 edits in trunk/PerformanceTests
Make it so JetStream2 can run with the d8 shell
https://bugs.webkit.org/show_bug.cgi?id=213856
Reviewed by Keith Miller.
We use d8's Realm API to create separate global objects for
each test to run in.
- JetStream2/JetStreamDriver.js:
(Driver.prototype.runCode.globalObject.loadString):
(Driver.prototype.runCode):
(prototype.async run):
- JetStream2/cli.js:
- JetStream2/index.html:
- 1:14 PM Changeset in webkit [263817] by
-
- 2 edits in trunk/Source/WebKit
Add WebXR to WebKit.plist preferences, off by default
https://bugs.webkit.org/show_bug.cgi?id=213860
<rdar://64411568>
Reviewed by Tim Horton.
Right now we look for "WebXR" in the plist, however, it's not in
the file. It so happens that, *I think*, we fall back to false for
missing keys. We should have an entry for readability and
consistency, though.
No tests needed because this shouldn't change behavior.
- FeatureFlags/WebKit.plist:
- 10:42 AM Changeset in webkit [263816] by
-
- 3 edits in trunk/Source/WebKit
Regression(r259242) WebKit no longer does client navigations at foreground priority for extensions
https://bugs.webkit.org/show_bug.cgi?id=213852
<rdar://problem/63300428>
Reviewed by Alex Christensen.
r259242 tried to address a ASSERT(window) debug assertion hit in applicationType()
by only calling applicationType() when the window is non-null and assuming the
process is an application when the window is null. However, this is not correct
for extensions. The first thing isApplicationVisible() does is call
_UIApplicationIsExtension() to determine if we are an extension. It does not need
the window to make this determination. As a result, I have reverted r259242 and
simply dropped the ASSERT(window) assertion in isApplicationVisible() instead.
This fixes the regression tracked by <rdar://problem/63300428> by reverting the
change that caused it (r259242) and drops the assertion that was hit before r259242.
- UIProcess/ApplicationStateTracker.mm:
(WebKit::applicationType):
- UIProcess/ios/PageClientImplIOS.mm:
(WebKit::PageClientImpl::isApplicationVisible):
- 10:26 AM Changeset in webkit [263815] by
-
- 9 edits3 adds in trunk/Source/WebInspectorUI
Web Inspector: add back support for 10.3.1
https://bugs.webkit.org/show_bug.cgi?id=213764
Reviewed by Brian Burg.
- UserInterface/Controllers/NetworkManager.js:
(WI.NetworkManager.prototype.initializeTarget):
(WI.NetworkManager.prototype.webSocketWillSendHandshakeRequest):
(WI.NetworkManager.prototype.markResourceRequestAsServedFromMemoryCache): Added.
(WI.NetworkManager.prototype.resourceRequestDidReceiveResponse):
- UserInterface/Models/Resource.js:
(WI.Resource.prototype.legacyMarkServedFromMemoryCache): Added.
(WI.Resource.prototype.legacyMarkServedFromDiskCache): Added.
- UserInterface/Models/WebSocketResource.js:
(WI.WebSocketResource.prototype.addFrame):
- UserInterface/Protocol/Legacy/10.3/InspectorBackendCommands.js: Added.
- UserInterface/Protocol/NetworkObserver.js:
(WI.NetworkObserver.prototype.requestServedFromCache): Added.
- UserInterface/Views/NetworkTableContentView.js:
(WI.NetworkTableContentView):
- UserInterface/Views/NetworkTimelineView.js:
(WI.NetworkTimelineView):
- UserInterface/Views/ResourceDetailsSidebarPanel.js:
(WI.ResourceDetailsSidebarPanel.prototype._refreshRequestAndResponse):
- UserInterface/Views/WebSocketContentView.js:
(WI.WebSocketContentView):
(WI.WebSocketContentView.prototype._addRow):
- Versions/Inspector-iOS-10.3.json: Added.
- 10:23 AM Changeset in webkit [263814] by
-
- 8 edits in trunk
Fix the WebKit build after <rdar://problem/64288191>
https://bugs.webkit.org/show_bug.cgi?id=213832
Reviewed by Tim Horton.
Source/WebCore:
Upcoming changes to <UIKit/UIWebDocumentView.h> will cause the WebKit build to break, due to conflicting
declarations of WKObject in both WebCore (WAK) code and the WebKit API. Avoid this by renaming the WAK version
to WAKObject, rather than WKObject.
Additionally, fix some minor style issues along the way.
- platform/ios/wak/WKTypes.h:
- platform/ios/wak/WKUtilities.c:
(WKRetain):
(WKRelease):
(_WAKObjectDealloc):
(WKGetClassInfo):
(_WKObjectDealloc): Deleted.
- platform/ios/wak/WKUtilities.h:
- platform/ios/wak/WKView.h:
- platform/ios/wak/WKView.mm:
(_WKViewDealloc):
Tools:
Additionally, fix a naming conflict that will result from
WebArchivePboardTypebeing declared in both
<WebKitLegacy/WebArchive.h>and DumpRenderTree, by making the latter a local variable instead.
- DumpRenderTree/mac/TestRunnerMac.mm:
(TestRunner::imageCountInGeneralPasteboard const):
- 10:08 AM Changeset in webkit [263813] by
-
- 8 edits in trunk
[iOS] Implement support for UIWKDocumentRequestSpatialAndCurrentSelection
https://bugs.webkit.org/show_bug.cgi?id=213704
<rdar://problem/59738878>
Reviewed by Wenson Hsieh.
Source/WebKit:
Implement support for the new request type UIWKDocumentRequestSpatialAndCurrentSelection.
Requests of this type return the contents inside the range that covers both the specified
rect and the current selection, if there is one.
The flag UIWKDocumentRequestSpatialAndCurrentSelection was added in <rdar://problem/64867540>.
- Platform/spi/ios/UIKitSPI.h: Same thing I did in TestWebKitAPI/ios/UIKitSPI.h. See Tools ChangeLog for why.
- Shared/DocumentEditingContext.h:
- UIProcess/ios/WKContentViewInteraction.mm:
(toWebDocumentRequestOptions):
- WebProcess/WebPage/ios/WebPageIOS.mm:
(WebKit::WebPage::requestDocumentEditingContext):
Tools:
Add some tests.
- TestWebKitAPI/Tests/WebKitCocoa/DocumentEditingContext.mm:
(TEST):
- TestWebKitAPI/ios/UIKitSPI.h: Define the enumerator using a macro just like what was done for
UIWKDocumentRequestMarkedTextRects. The reason for the macro instead of modifying the enumeration
definition in this file is to 1) avoid breaking the build for Apple engineers that haven't picked
up <rdar://problem/64867540> and to 2) avoid breaking the OpenSource build.
- 9:58 AM Changeset in webkit [263812] by
-
- 2 edits in trunk/LayoutTests
Remove expectation for fast/events/before-input-delete-empty-list-target-ranges.html and fast/events/before-input-events-prevent-insert-composition.html and fast/events/before-input-events-prevent-recomposition.html as they are passing.
https://bugs.webkit.org/show_bug.cgi?id=213848
Unreviewed test gardening.
Patch by Karl Rackler <Karl Rackler> on 2020-07-01
- platform/ios-simulator/TestExpectations:
- 9:41 AM Changeset in webkit [263811] by
-
- 2 edits in trunk/LayoutTests
Remove expectation for fast/events/fire-scroll-event.html as it is passing.
https://bugs.webkit.org/show_bug.cgi?id=213850
Unreviewed test gardening.
Patch by Karl Rackler <Karl Rackler> on 2020-07-01
- platform/ios-simulator/TestExpectations:
- 9:20 AM Changeset in webkit [263810] by
-
- 2 edits in trunk/Source/JavaScriptCore
Script to copy over testapi.js is redundant in xcodebuild
https://bugs.webkit.org/show_bug.cgi?id=213824
Reviewed by Keith Miller.
We're already copying over the entire testapiScripts directory.
No need to manually copy just one file from it, too.
- JavaScriptCore.xcodeproj/project.pbxproj:
- 9:19 AM Changeset in webkit [263809] by
-
- 2 edits in trunk/LayoutTests
[GLIB] Unreviewed test gardening. Remove unnecessary baseline after r263797.
- platform/glib/imported/w3c/web-platform-tests/websockets/interfaces/WebSocket/bufferedAmount/bufferedAmount-unicode-expected.txt:
- 9:15 AM Changeset in webkit [263808] by
-
- 2 edits in trunk/LayoutTests
Remove expectation for fast/events/before-input-prevent-cut.html as they are passing.
https://bugs.webkit.org/show_bug.cgi?id=213849
Unreviewed test gardening.
Patch by Karl Rackler <Karl Rackler> on 2020-07-01
- platform/ios/TestExpectations:
- 9:14 AM Changeset in webkit [263807] by
-
- 4 edits in trunk/Source/WebKit
Crash under IPC::isValidMessageName()
https://bugs.webkit.org/show_bug.cgi?id=213751
<rdar://problem/64773416>
Reviewed by Geoffrey Garen.
This is a speculative fix for the crashes tracked by <rdar://problem/64773416>.
The suspicion is that the compiler optimizes the code for IPC::isValidMessageName()
because its parameter is of type MessageName. It may therefore assume the parameter
is a valid MessageName enum value, which is ironic because the purpose of this
function is to check whether or not the enum value is valid.
To address the issue, I got rid of the #ifdefs in MessageName enum so that the
enum values have an underlying value in the range [0, MessageName::Last]. I then
updated the isValidEnum template specialization to verify that the input integer
is indeed in the range [0, MessageName::Last] before casting it to a MessageName
and calling IPC::isValidMessageName().
- Scripts/webkit/messages.py:
- 9:03 AM Changeset in webkit [263806] by
-
- 12 edits in trunk
WKUserScripts injecting into all frames seem to violate app-bound domains
https://bugs.webkit.org/show_bug.cgi?id=213816
<rdar://problem/64872296>
Reviewed by Brady Eidson.
Source/WebKit:
Patch to make app-bound domain checks on a per-frame basis instead
of per-page-based-on-main-frame. This will prevent WKUserScripts from
being injected in subframes when the main frame is app-bound. Now, all
ancestors of a frame must be app-bound in order for a frame to
evaluate user script.
- UIProcess/WebPageProxy.cpp:
(WebKit::WebPageProxy::setIsNavigatingToAppBoundDomainAndCheckIfPermitted):
Move 'isMainFrame' check to be inside of the check for
m_limitsNavigationsToAppBoundDomains. The navigation should fail if
the main frame is not app-bound and the WKWebView is marked with the
limitsNavigationToAppBoundDomains flag, but non-app-bound subframe
loads are allowed to continue.
(WebKit::WebPageProxy::decidePolicyForNavigationAction):
- UIProcess/WebPageProxy.h:
(WebKit::WebPageProxy::isTopFrameNavigatingToAppBoundDomain const):
- UIProcess/WebProcessPool.cpp:
(WebKit::WebProcessPool::download):
- WebProcess/Network/WebLoaderStrategy.cpp:
(WebKit::WebLoaderStrategy::scheduleLoadFromNetworkProcess):
(WebKit::WebLoaderStrategy::loadResourceSynchronously):
(WebKit::WebLoaderStrategy::startPingLoad):
(WebKit::WebLoaderStrategy::preconnectTo):
For reporting loads to the Network Process, we only care if the main
frame is app bound, so we need to store that info in the WebPage and
WebPageProxy.
- WebProcess/WebCoreSupport/WebFrameLoaderClient.cpp:
(WebKit::WebFrameLoaderClient::shouldEnableInAppBrowserPrivacyProtections const):
- WebProcess/WebPage/WebFrame.cpp:
(WebKit::WebFrame::startDownload):
(WebKit::WebFrame::convertMainResourceLoadToDownload):
(WebKit::WebFrame::shouldEnableInAppBrowserPrivacyProtections):
(WebKit::WebFrame::isTopFrameNavigatingToAppBoundDomain const):
- WebProcess/WebPage/WebFrame.h:
(WebKit::WebFrame::setIsNavigatingToAppBoundDomain):
(WebKit::WebFrame::isNavigatingToAppBoundDomain const):
- WebProcess/WebPage/WebPage.cpp:
(WebKit::WebPage::loadRequest):
(WebKit::WebPage::loadDataImpl):
(WebKit::WebPage::didReceivePolicyDecision):
(WebKit::WebPage::runJavaScript):
(WebKit::WebPage::setIsNavigatingToAppBoundDomain):
(WebKit::WebPage::shouldEnableInAppBrowserPrivacyProtections): Deleted.
- WebProcess/WebPage/WebPage.h:
(WebKit::WebPage::needsInAppBrowserPrivacyQuirks):
(WebKit::WebPage::isNavigatingToAppBoundDomain const): Deleted.
Move all app-bound domain logic from the WebPage to the WebFrame now
that checks are on a per-frame basis.
Tools:
Added API test coverage. Sets up a tree like so: [app-bound domain,
non-app-bound domain, app-bound domain]. Script injection should only
be successful in the top frame. Since message handlers are disabled for non-
app-bound domains, this test injects script which loads an image using a
custom url scheme. If the load succeeds, we increment a counter, and
expect only one successful script load (the main frame).
- TestWebKitAPI/Tests/WebKitCocoa/InAppBrowserPrivacy.mm:
(-[InAppBrowserSchemeHandler webView:startURLSchemeTask:]):
- 8:57 AM Changeset in webkit [263805] by
-
- 3 edits2 adds in trunk
[LFC][TFC] Add support for non-baseline aligned cell height
https://bugs.webkit.org/show_bug.cgi?id=213845
Reviewed by Antti Koivisto.
Source/WebCore:
Adjust max ascent/descent only when the cell is basline aligned.
Test: fast/layoutformattingcontext/table-cell-height-middle-align-simple.html
- layout/tableformatting/TableLayout.cpp:
(WebCore::Layout::TableFormattingContext::TableLayout::distributedVerticalSpace):
LayoutTests:
- fast/layoutformattingcontext/table-cell-height-middle-align-simple-expected.html: Added.
- fast/layoutformattingcontext/table-cell-height-middle-align-simple.html: Added.
- 8:47 AM Changeset in webkit [263804] by
-
- 1 edit2 moves2 adds in trunk/LayoutTests
pointerevents/ios/touch-action-none-on-iframe.html times out
https://bugs.webkit.org/show_bug.cgi?id=213846
<rdar://problem/60393092>
Reviewed by Antti Koivisto.
The test pointerevents/ios/touch-action-none-on-iframe.html was mistakenly assuming that setting
touch-action: noneon
an <iframe> should prevent scrolling on the outer document with a swipe gesture over the inner document. We're renaming
this test as pointerevents/ios/touch-action-none-on-iframe-outer.html and modifying it to expect scrolling to occur, and
also make it a promise_test to avoid further timeouts due to failures in an async_test.
We also add a new test pointerevents/ios/touch-action-none-on-iframe-inner.html which sets
touch-action: noneon the
inner document which should indeed prevent scrolling of both the inner and outer documents.
- pointerevents/ios/touch-action-none-on-iframe-inner-expected.txt: Added.
- pointerevents/ios/touch-action-none-on-iframe-inner.html: Added.
- pointerevents/ios/touch-action-none-on-iframe-outer-expected.txt: Renamed from LayoutTests/pointerevents/ios/touch-action-none-on-iframe-expected.txt.
- pointerevents/ios/touch-action-none-on-iframe-outer.html: Renamed from LayoutTests/pointerevents/ios/touch-action-none-on-iframe.html.
- 8:40 AM Changeset in webkit [263803] by
-
- 3 edits in trunk/LayoutTests
Remove expectation for fast/dom/Window/open-invalid-url.html and fast/dom/Range/simple-line-layout-getclientrects.html and fast/dom/attr_dead_doc.html and fast/dom/horizontal-scrollbar-when-dir-change.html and fast/dom/vertical-scrollbar-when-dir-change.html as they are passing.
https://bugs.webkit.org/show_bug.cgi?id=213847
Unreviewed test gardening.
Patch by Karl Rackler <Karl Rackler> on 2020-07-01
- platform/ios-simulator/TestExpectations:
- platform/ios/TestExpectations:
- 8:26 AM Changeset in webkit [263802] by
-
- 3 edits2 adds in trunk
[Media Controls] Tracks panel can show text tracks with mode "hidden" as selected
https://bugs.webkit.org/show_bug.cgi?id=213839
<rdar://problem/57989325>
Reviewed by Jer Noble.
Source/WebCore:
We only considered text tracks that had their "mode" set to "disabled" as tracks that weren't selected,
but there is also the "hidden" mode which should share the same UI state. We now check for "mode" to
be set to "showing" to consider a text track as selected in the tracks panel.
Test: media/modern-media-controls/tracks-support/tracks-support-hidden-tracks.html
- Modules/modern-media-controls/media/tracks-support.js:
(TracksSupport.prototype.tracksPanelIsTrackInSectionSelected):
LayoutTests:
Add a new test that dumps the selection state of tracks in the tracks panel when all of media.textTracks
have been set to mode = "hidden".
- media/modern-media-controls/tracks-support/tracks-support-hidden-tracks-expected.txt: Added.
- media/modern-media-controls/tracks-support/tracks-support-hidden-tracks.html: Added.
- 7:49 AM Changeset in webkit [263801] by
-
- 2 edits in trunk/LayoutTests
Remove expectation for fast/css/replaced-element-implicit-size.html as they are passing.
https://bugs.webkit.org/show_bug.cgi?id=213843
Unreviewed test gardening.
Patch by Karl Rackler <Karl Rackler> on 2020-07-01
- platform/ios/TestExpectations:
- 7:31 AM Changeset in webkit [263800] by
-
- 2 edits in trunk/Source/WebCore
Regression (r263788): Windows build broken: nameForRenderTreeAsText is not a member of WebCore::Color
https://bugs.webkit.org/show_bug.cgi?id=213841
<rdar://problem/64985276>
Fix Windows build.
- platform/graphics/ca/win/PlatformCALayerWin.cpp:
(printColor):
- 7:28 AM Changeset in webkit [263799] by
-
- 2 edits in trunk/LayoutTests
Remove expectation for fast/canvas/canvas-gradient-addStop-error.html and fast/canvas/canvas-radial-gradient-spreadMethod.html as they are passing.
https://bugs.webkit.org/show_bug.cgi?id=213840
Unreviewed test gardening.
Patch by Karl Rackler <Karl Rackler> on 2020-07-01
- platform/ios/TestExpectations:
- 7:27 AM Changeset in webkit [263798] by
-
- 2 edits in trunk/LayoutTests
Remove expectation for fast/css/font-calculated-value.html as it is passing.
https://bugs.webkit.org/show_bug.cgi?id=178952
Unreviewed test gardening.
Patch by Karl Rackler <Karl Rackler> on 2020-07-01
- platform/ios/TestExpectations:
- 7:23 AM Changeset in webkit [263797] by
-
- 19 edits in trunk/Source
Make NetworkSendQueue use CString instead of String for UTF-8 data
https://bugs.webkit.org/show_bug.cgi?id=213714
Reviewed by Darin Adler.
Source/WebCore:
Use CString instead of String in NetworkSendQueue as all data is to be encoded as UTF-8.
This allows converting and allocating the UTF-8 string only once.
Covered by existing tests.
- Modules/mediastream/RTCDataChannel.cpp:
(WebCore::RTCDataChannel::createMessageQueue):
(WebCore::RTCDataChannel::send):
- Modules/mediastream/libwebrtc/LibWebRTCDataChannelHandler.cpp:
(WebCore::LibWebRTCDataChannelHandler::sendStringData):
(WebCore::LibWebRTCDataChannelHandler::sendRawData):
- Modules/mediastream/libwebrtc/LibWebRTCDataChannelHandler.h:
- fileapi/NetworkSendQueue.cpp:
(WebCore::NetworkSendQueue::enqueue):
(WebCore::NetworkSendQueue::processMessages):
- fileapi/NetworkSendQueue.h:
- platform/mediastream/RTCDataChannelHandler.h:
- platform/mock/RTCDataChannelHandlerMock.cpp:
(WebCore::RTCDataChannelHandlerMock::sendStringData):
- platform/mock/RTCDataChannelHandlerMock.h:
Source/WebKit:
Update code to use CString/DataReference to transmit WebSocket text messages.
- NetworkProcess/NetworkSocketChannel.cpp:
(WebKit::NetworkSocketChannel::sendString):
- NetworkProcess/NetworkSocketChannel.h:
- NetworkProcess/NetworkSocketChannel.messages.in:
- NetworkProcess/WebSocketTask.h:
(WebKit::WebSocketTask::sendString):
- NetworkProcess/cocoa/WebSocketTaskCocoa.h:
- NetworkProcess/cocoa/WebSocketTaskCocoa.mm:
(WebKit::WebSocketTask::sendString):
- NetworkProcess/soup/WebSocketTaskSoup.h:
- NetworkProcess/soup/WebSocketTaskSoup.cpp:
(WebKit::WebSocketTask::sendString):
- WebProcess/Network/WebSocketChannel.cpp:
(WebKit::WebSocketChannel::createMessageQueue):
(WebKit::WebSocketChannel::send):
- 7:16 AM Changeset in webkit [263796] by
-
- 2 edits in trunk/LayoutTests
Remove expectation for fast/block/positioning/rtl-fixed-positioning.html as it is passing.
https://bugs.webkit.org/show_bug.cgi?id=213838
Unreviewed test gardening.
Patch by Karl Rackler <Karl Rackler> on 2020-07-01
- platform/ios-simulator/TestExpectations:
- 3:49 AM Changeset in webkit [263795] by
-
- 4 edits in trunk/Source
Use more efficient makeString() instead of StringBuilder
https://bugs.webkit.org/show_bug.cgi?id=213708
Reviewed by Sam Weinig.
Source/WebCore:
This patch refactors throwRequiredMemberTypeError() and throwArgument*Error()
methods, replacing all StringBuilder usages with more efficient makeString().
Extracts makeArgumentTypeErrorMessage(), similiar to existing helpers,
which can be reused if neccessary.
No new tests, no behavior change.
- bindings/js/JSDOMExceptionHandling.cpp:
(WebCore::makeArgumentTypeErrorMessage):
(WebCore::throwArgumentMustBeEnumError):
(WebCore::throwArgumentMustBeFunctionError):
(WebCore::throwArgumentMustBeObjectError):
(WebCore::throwArgumentTypeError):
(WebCore::throwRequiredMemberTypeError):
(WebCore::appendArgumentMustBe): Deleted.
Source/WTF:
Introduces StringTypeAdapter<std::tuple> overload, which enables using
makeString() instead of StringBuilder in more cases.
- wtf/text/StringConcatenate.h:
(WTF::StringTypeAdapter<std::tuple<StringTypes...>, void>): Added.
- 3:41 AM Changeset in webkit [263794] by
-
- 4 edits in trunk
[css-flexbox] Don't include scrollbar extents when computing sizes for percentage resolution
https://bugs.webkit.org/show_bug.cgi?id=213739
Reviewed by Javier Fernandez.
Source/WebCore:
Content override sizes do include scrollbars so they must be substracted when using the override size
to compute percentages.
- rendering/RenderFlexibleBox.cpp:
(WebCore::RenderFlexibleBox::crossSizeForPercentageResolution): Remove scrollbars.
(WebCore::RenderFlexibleBox::mainSizeForPercentageResolution): Ditto.
LayoutTests:
- TestExpectations: Unskipped percentage-heights-004.html.
- 3:36 AM Changeset in webkit [263793] by
-
- 3 edits in trunk/Source/WebKit
Fix Swift overlay build after r263727.
https://bugs.webkit.org/show_bug.cgi?id=213823.
Unreviewed, follow up build fix.
- SwiftOverlay/Tests/WebKitTests.swift:
(WebKitTests.testAPI): Fix tests to refer to new parameter name.
- UIProcess/API/Cocoa/WebKitSwiftOverlay.swift:
(callAsyncJavaScript(_:arguments:in:in:completionHandler:Error:)): Use
the imported ObjC name that the compiler expects.
(evaluateJavaScript(_:in:in:completionHandler:Error:)): Ditto.
- 3:36 AM Changeset in webkit [263792] by
-
- 3 edits in trunk/Source/WebCore
[css-flex] Remove death code paths when evaluating percentage resolution
https://bugs.webkit.org/show_bug.cgi?id=213809
Reviewed by Manuel Rego Casasnovas.
Both crossSizeForPercentageResolution() and mainSizeForPercentageResolution() are only called from
childLogicalHeightForPercentageResolution(). The former is called whenever hasOrthogonalFlow(child)
is false and the latter when it's true. However crossSizeForPercentageResolution() has a path for
hasOrthogonalFlow(child)==true which is impossible to reach. The same happens to
mainSizeForPercentageResolution() which has a path for hasOrthogonalFlow(child)==false which is
also impossible to reach.
Remove both death code paths and replace them by assertions. We're also making both methods
private since are not meant to be used from the outside.
- rendering/RenderFlexibleBox.cpp:
(WebCore::RenderFlexibleBox::crossSizeForPercentageResolution): Remove death code path.
(WebCore::RenderFlexibleBox::mainSizeForPercentageResolution): Ditto.
- rendering/RenderFlexibleBox.h: Make both calls private.
- 3:34 AM Changeset in webkit [263791] by
-
- 3 edits in trunk/LayoutTests
[GTK] Unreviewed test gardening. Garden flaky failures in EWS GTK-WK2 queue.
- platform/glib/TestExpectations:
- platform/gtk/TestExpectations:
- 12:09 AM Changeset in webkit [263790] by
-
- 11 edits in trunk/Source/JavaScriptCore
Use properly flag names for tests of Tools/Scripts/run-builtins-generator-tests
https://bugs.webkit.org/show_bug.cgi?id=213733
Patch by Tetsuharu Ohzeki <Tetsuharu Ohzeki> on 2020-07-01
Reviewed by Youenn Fablet.
Test cases under Source/JavaScriptCore/Scripts/tests/builtins/ uses exist compilation flags.
But they can take an arbitary flag name and don't have to use an exist flag.
I think it's better to rename them to more proper ones.
- Scripts/tests/builtins/WebCore-ArbitraryConditionalGuard-Separate.js:
- Scripts/tests/builtins/WebCore-DuplicateKeyValueAnnotation-Separate.js:
- Scripts/tests/builtins/WebCore-GuardedBuiltin-Separate.js:
- Scripts/tests/builtins/WebCore-GuardedInternalBuiltin-Separate.js:
- Scripts/tests/builtins/WebCore-xmlCasingTest-Separate.js:
- Scripts/tests/builtins/expected/WebCore-ArbitraryConditionalGuard-Separate.js-result:
- Scripts/tests/builtins/expected/WebCore-GuardedBuiltin-Separate.js-result:
- Scripts/tests/builtins/expected/WebCore-GuardedInternalBuiltin-Separate.js-result:
- Scripts/tests/builtins/expected/WebCore-xmlCasingTest-Separate.js-result:
- Scripts/tests/builtins/expected/WebCoreJSBuiltins.h-result:
- 12:08 AM Changeset in webkit [263789] by
-
- 16 edits3 deletes in trunk/Source/WebKit
Make NetworkRTCProvider receive IPC messages from a background thread
https://bugs.webkit.org/show_bug.cgi?id=213557
Reviewed by Geoffrey Garen.
To ensure webrtc packets do not get blocked on the main thread,
we process WebProcess IPC message directly in the socket thread.
For new sockets, we still go through the main thread, to check suspension or proxy values.
For sending packets/setting options/closing sockets, we stay in the background thread.
- DerivedSources-output.xcfilelist:
- DerivedSources.make:
- NetworkProcess/NetworkConnectionToWebProcess.cpp:
(WebKit::NetworkConnectionToWebProcess::didReceiveMessage):
- NetworkProcess/webrtc/LibWebRTCSocketClient.cpp:
(WebKit::LibWebRTCSocketClient::sendTo):
- NetworkProcess/webrtc/LibWebRTCSocketClient.h:
- NetworkProcess/webrtc/NetworkRTCProvider.cpp:
(WebKit::NetworkRTCProvider::NetworkRTCProvider):
(WebKit::NetworkRTCProvider::close):
(WebKit::NetworkRTCProvider::createSocket):
(WebKit::NetworkRTCProvider::createUDPSocket):
(WebKit::NetworkRTCProvider::createServerTCPSocket):
(WebKit::NetworkRTCProvider::createClientTCPSocket):
(WebKit::NetworkRTCProvider::wrapNewTCPConnection):
(WebKit::NetworkRTCProvider::sendToSocket):
(WebKit::NetworkRTCProvider::closeSocket):
(WebKit::NetworkRTCProvider::setSocketOption):
(WebKit::NetworkRTCProvider::addSocket):
(WebKit::NetworkRTCProvider::takeSocket):
(WebKit::NetworkRTCProvider::newConnection):
(WebKit::NetworkRTCProvider::dispatchToThread):
- NetworkProcess/webrtc/NetworkRTCProvider.h:
(WebKit::NetworkRTCProvider::didReceiveNetworkRTCMonitorMessage):
- NetworkProcess/webrtc/NetworkRTCProvider.messages.in:
- NetworkProcess/webrtc/NetworkRTCSocket.cpp: Removed.
- NetworkProcess/webrtc/NetworkRTCSocket.h: Removed.
- NetworkProcess/webrtc/NetworkRTCSocket.messages.in: Removed.
- SourcesCocoa.txt:
- SourcesGTK.txt:
- SourcesWPE.txt:
- WebKit.xcodeproj/project.pbxproj:
- WebProcess/Network/webrtc/LibWebRTCSocket.cpp:
(WebKit::LibWebRTCSocket::SendTo):
(WebKit::LibWebRTCSocket::Close):
(WebKit::LibWebRTCSocket::SetOption):
(WebKit::LibWebRTCSocket::suspend):
