Timeline
Jun 27, 2017:
- 11:42 PM Changeset in webkit [218868] by
-
- 15 edits4 adds in trunk
WebAssembly: running out of executable memory should throw OoM
https://bugs.webkit.org/show_bug.cgi?id=171537
<rdar://problem/32963338>
Reviewed by Saam Barati.
JSTests:
- wasm.yaml:
- wasm/lowExecutableMemory/executable-memory-oom.js: Added.
(const.invoke):
(failCount.0.catch):
(failCount.0.module.undefined.catch):
- wasm/lowExecutableMemory/exports-oom.js: Added.
(const.type):
(const.params):
(const.randomProgram):
(failCount.0.catch):
(failCount.0.module.undefined.catch):
- wasm/lowExecutableMemory/imports-oom.js: Added.
(const.type):
(const.params):
(const.randomProgram):
(f.imports.push):
(failCount.0.catch):
(failCount.0.module.undefined.catch):
Source/JavaScriptCore:
Both on first compile with BBQ as well as on tier-up with OMG,
running out of X memory shouldn't cause the entire program to
terminate. An exception will do when compiling initial code (since
we don't have any other fallback at the moment), and refusal to
tier up will do as well (it'll just be slower).
This is useful because programs which generate huge amounts of
code simply look like crashes, which developers report to
us. Getting a JavaScript exception instead is much clearer.
- jit/ExecutableAllocator.cpp:
(JSC::ExecutableAllocator::allocate):
- llint/LLIntSlowPaths.cpp:
(JSC::LLInt::shouldJIT):
- runtime/Options.h:
- wasm/WasmBBQPlan.cpp:
(JSC::Wasm::BBQPlan::prepare):
(JSC::Wasm::BBQPlan::complete):
- wasm/WasmBinding.cpp:
(JSC::Wasm::wasmToJs):
(JSC::Wasm::wasmToWasm):
- wasm/WasmBinding.h:
- wasm/WasmOMGPlan.cpp:
(JSC::Wasm::OMGPlan::work):
- wasm/js/JSWebAssemblyCodeBlock.cpp:
(JSC::JSWebAssemblyCodeBlock::JSWebAssemblyCodeBlock):
- wasm/js/JSWebAssemblyCodeBlock.h:
- wasm/js/JSWebAssemblyInstance.cpp:
(JSC::JSWebAssemblyInstance::finalizeCreation):
Tools:
- Scripts/run-jsc-stress-tests: add a configuration which runs the
tests under limited executable memory and avoids non-WebAssembly
code generation so that we more reliably run out of executable
memory in WebAssembly.
- 11:23 PM Changeset in webkit [218867] by
-
- 4 edits in trunk/Source/JavaScriptCore
JITStubRoutine::passesFilter should use isJITPC
https://bugs.webkit.org/show_bug.cgi?id=173906
Reviewed by JF Bastien.
This patch makes JITStubRoutine use the isJITPC abstraction defined
inside ExecutableAllocator.h. Before, JITStubRoutine was using a
hardcoded platform size constant. This means it'd do the wrong thing
if Options::jitMemoryReservationSize() was larger than the defined
constant for that platform. This patch also removes a bunch of
dead code in that file.
- jit/ExecutableAllocator.cpp:
- jit/ExecutableAllocator.h:
- jit/JITStubRoutine.h:
(JSC::JITStubRoutine::passesFilter):
(JSC::JITStubRoutine::canPerformRangeFilter): Deleted.
(JSC::JITStubRoutine::filteringStartAddress): Deleted.
(JSC::JITStubRoutine::filteringExtentSize): Deleted.
- 10:45 PM Changeset in webkit [218866] by
-
- 3 edits in trunk/Source/JavaScriptCore
Fix some stale comments in Wasm code base
https://bugs.webkit.org/show_bug.cgi?id=173814
Reviewed by Mark Lam.
- wasm/WasmBinding.cpp:
(JSC::Wasm::wasmToJs):
- wasm/WasmOMGPlan.cpp:
(JSC::Wasm::runOMGPlanForIndex):
- 10:14 PM Changeset in webkit [218865] by
-
- 12 edits in trunk/Source
[ResourceLoadStatistics] Update minimumTimeBetweeenDataRecordsRemoval to 1 hour instead of 1 minute
https://bugs.webkit.org/show_bug.cgi?id=173895
<rdar://problem/32984366>
Reviewed by Brent Fulgham.
Update minimumTimeBetweeenDataRecordsRemoval to 1 hour instead of 1 minute to save battery.
Also port code to modern time types.
Source/WebCore:
- loader/ResourceLoadObserver.cpp:
(WebCore::reduceTimeResolution):
(WebCore::ResourceLoadObserver::logUserInteractionWithReducedTimeResolution):
(WebCore::ResourceLoadObserver::setTimeToLiveUserInteraction):
(WebCore::ResourceLoadObserver::setTimeToLiveCookiePartitionFree):
(WebCore::ResourceLoadObserver::setMinimumTimeBetweeenDataRecordsRemoval):
(WebCore::ResourceLoadObserver::setReducedTimestampResolution):
(WebCore::ResourceLoadObserver::setGrandfatheringTime):
- loader/ResourceLoadObserver.h:
- loader/ResourceLoadStatistics.h:
(WebCore::ResourceLoadStatistics::mostRecentUserInteractionTime):
- loader/ResourceLoadStatisticsStore.cpp:
(WebCore::ResourceLoadStatisticsStore::createEncoderFromData):
(WebCore::ResourceLoadStatisticsStore::readDataFromDecoder):
(WebCore::shouldPartitionCookies):
(WebCore::ResourceLoadStatisticsStore::setTimeToLiveUserInteraction):
(WebCore::ResourceLoadStatisticsStore::setTimeToLiveCookiePartitionFree):
(WebCore::ResourceLoadStatisticsStore::setMinimumTimeBetweeenDataRecordsRemoval):
(WebCore::ResourceLoadStatisticsStore::setGrandfatheringTime):
(WebCore::ResourceLoadStatisticsStore::hasHadRecentUserInteraction):
(WebCore::ResourceLoadStatisticsStore::topPrivatelyControlledDomainsToRemoveWebsiteDataFor):
(WebCore::ResourceLoadStatisticsStore::handleFreshStartWithEmptyOrNoStore):
(WebCore::ResourceLoadStatisticsStore::shouldRemoveDataRecords):
(WebCore::ResourceLoadStatisticsStore::dataRecordsBeingRemoved):
- loader/ResourceLoadStatisticsStore.h:
Source/WebKit2:
- UIProcess/API/C/WKResourceLoadStatisticsManager.cpp:
(WKResourceLoadStatisticsManagerSetTimeToLiveUserInteraction):
(WKResourceLoadStatisticsManagerSetTimeToLiveCookiePartitionFree):
(WKResourceLoadStatisticsManagerSetMinimumTimeBetweeenDataRecordsRemoval):
(WKResourceLoadStatisticsManagerSetGrandfatheringTime):
- UIProcess/WebResourceLoadStatisticsManager.cpp:
(WebKit::WebResourceLoadStatisticsManager::setTimeToLiveUserInteraction):
(WebKit::WebResourceLoadStatisticsManager::setTimeToLiveCookiePartitionFree):
(WebKit::WebResourceLoadStatisticsManager::setMinimumTimeBetweeenDataRecordsRemoval):
(WebKit::WebResourceLoadStatisticsManager::setGrandfatheringTime):
- UIProcess/WebResourceLoadStatisticsManager.h:
- UIProcess/WebResourceLoadStatisticsStore.h:
- 10:06 PM Changeset in webkit [218864] by
-
- 5 edits in trunk/Source
Port HysteresisActivity to Seconds type
https://bugs.webkit.org/show_bug.cgi?id=173902
Reviewed by Simon Fraser.
Source/WebCore:
- platform/HysteresisActivity.h:
(WebCore::HysteresisActivity::HysteresisActivity):
(WebCore::HysteresisActivity::stop):
- platform/ios/WebSQLiteDatabaseTrackerClient.mm:
Source/WebKit2:
- WebProcess/WebPage/WebPage.cpp:
(WebKit::m_pageScrolledHysteresis):
- 9:52 PM Changeset in webkit [218863] by
-
- 2 edits in trunk/Source/WebCore/PAL
Build fix for PAL
https://bugs.webkit.org/show_bug.cgi?id=173896
Reviewed by Myles C. Maxfield.
- PAL.xcodeproj/project.pbxproj: Ensure that the SRCROOT is WebCore/PAL and not
just WebCore.
- 9:23 PM Changeset in webkit [218862] by
-
- 4 edits2 adds1 delete in trunk/LayoutTests
Web Inspector: Cleanup RemoteObject tests
https://bugs.webkit.org/show_bug.cgi?id=173885
Reviewed by Matt Baker.
- inspector/model/remote-object-dom.html: Added.
- inspector/model/remote-object-expected.txt:
- inspector/model/remote-object.html:
Split out DOM results (which differ across ports and features).
- inspector/model/resources/remote-object-utilities.js:
(TestPage.registerInitializer.window.runSteps):
This should have been a continue not a return.
- platform/mac/inspector/model/remote-object-dom-expected.txt: Added.
- platform/mac/inspector/model/remote-object-expected.txt: Removed.
Platform expectations are now limited to just the DOM test.
- 8:05 PM Changeset in webkit [218861] by
-
- 25 edits1 add in trunk
[ESnext] Implement Object Rest - Implementing Object Rest Destructuring
https://bugs.webkit.org/show_bug.cgi?id=167962
Reviewed by Saam Barati.
JSTests:
- stress/object-rest-deconstruct.js: Added.
(let.assert):
(let.assertPropDescriptor):
(catch):
(get 3):
(foo):
(let.src.get y):
(let.src.set y):
(let.gen):
Source/JavaScriptCore:
Object Rest/Spread Destructing proposal is in stage 3[1] and this
Patch is a prototype implementation of it. A simple change over the
parser was necessary to support the new '...' token on Object Pattern
destruction rule. In the bytecode generator side, We changed the
bytecode generated on ObjectPatternNode::bindValue to store in an
set the identifiers of already destructured properties, following spec draft
section[2], and then pass it as excludedNames to CopyDataProperties.
The rest destructuring calls copyDataProperties to perform the
copy of rest properties in rhs.
We also implemented CopyDataProperties as private JS global operation
on builtins/GlobalOperations.js following it's specification on [3].
It is implemented using Set object to verify if a property is on
excludedNames to keep this algorithm with O(n + m) complexity, where n
number of source's own properties and m = excludedNames.length.
In this implementation we aren't using excludeList as constant if
destructuring pattern contains computed property, i.e. we can
just determine the key to be excluded at runtime. If we can define all
identifiers in the pattern in compile time, we then create a
constant JSSet. This approach gives a good performance improvement,
since we allocate the excludeSet just once, reducing GC pressure.
[1] - https://github.com/tc39/proposal-object-rest-spread
[2] - https://tc39.github.io/proposal-object-rest-spread/#Rest-RuntimeSemantics-PropertyDestructuringAssignmentEvaluation
[3] - https://tc39.github.io/proposal-object-rest-spread/#AbstractOperations-CopyDataProperties
- builtins/BuiltinNames.h:
- builtins/GlobalOperations.js:
(globalPrivate.copyDataProperties):
- bytecode/CodeBlock.cpp:
(JSC::CodeBlock::finishCreation):
- bytecompiler/NodesCodegen.cpp:
(JSC::ObjectPatternNode::bindValue):
- parser/ASTBuilder.h:
(JSC::ASTBuilder::appendObjectPatternEntry):
(JSC::ASTBuilder::appendObjectPatternRestEntry):
(JSC::ASTBuilder::setContainsObjectRestElement):
- parser/Nodes.h:
(JSC::ObjectPatternNode::appendEntry):
(JSC::ObjectPatternNode::setContainsRestElement):
- parser/Parser.cpp:
(JSC::Parser<LexerType>::parseDestructuringPattern):
(JSC::Parser<LexerType>::parseProperty):
- parser/SyntaxChecker.h:
(JSC::SyntaxChecker::operatorStackPop):
- runtime/JSGlobalObject.cpp:
(JSC::JSGlobalObject::init):
- runtime/JSGlobalObject.h:
(JSC::JSGlobalObject::asyncFunctionStructure):
(JSC::JSGlobalObject::setStructure): Deleted.
- runtime/JSGlobalObjectFunctions.cpp:
(JSC::privateToObject):
- runtime/JSGlobalObjectFunctions.h:
- runtime/ObjectConstructor.cpp:
(JSC::ObjectConstructor::finishCreation):
- runtime/SetPrototype.cpp:
(JSC::SetPrototype::finishCreation):
Source/WTF:
- wtf/HashSet.h:
(WTF::=):
LayoutTests:
- js/parser-syntax-check-expected.txt:
- js/script-tests/parser-syntax-check.js:
- 6:47 PM Changeset in webkit [218860] by
-
- 2 edits in trunk/Source/WebKit2
Move WebsiteDataRecord processing off-thread in WebsiteDataStore::fetchDataForTopPrivatelyControlledDomains()
https://bugs.webkit.org/show_bug.cgi?id=173882
<rdar://problem/32984366>
Reviewed by Darin Adler.
Move WebsiteDataRecord processing off-thread in WebsiteDataStore::fetchDataForTopPrivatelyControlledDomains()
as we have data showing it is slow and hangs the UIProcess' main thread.
- UIProcess/WebsiteData/WebsiteDataStore.cpp:
(WebKit::WebsiteDataStore::fetchDataForTopPrivatelyControlledDomains):
- 6:09 PM Changeset in webkit [218859] by
-
- 5 edits in trunk/LayoutTests
Unreviewed, unmark workers/bomb.html flakiness after r218856
- platform/gtk-wayland/TestExpectations:
- platform/ios-device/TestExpectations:
- platform/ios-wk2/TestExpectations:
- platform/mac/TestExpectations:
- 6:06 PM Changeset in webkit [218858] by
-
- 2 edits in trunk/Source/WebKit2
[WK2][macOS] Expand sandbox to allow access to the CoreMedia volume control endpoint
https://bugs.webkit.org/show_bug.cgi?id=173898
<rdar://problem/32930754>
Reviewed by Ryosuke Niwa.
- WebProcess/com.apple.WebProcess.sb.in:
- 6:04 PM Changeset in webkit [218857] by
-
- 2 edits in trunk/Source/WebCore
MediaPlayerPrivate m_private may not yet be created when setPrivateBrowsingMode is called.
https://bugs.webkit.org/show_bug.cgi?id=173893
rdar://problem/32986872
Patch by Jeremy Jones <jeremyj@apple.com> on 2017-06-27
Reviewed by Ryosuke Niwa.
No new tests because no reproducable case.
setPrivateBrowsingMode can be called on MediaPlayer before the MediaPlayerPrivate is created.
The value should only be pushed down the m_private if it has been created.
- platform/graphics/MediaPlayer.cpp:
(WebCore::MediaPlayer::setPrivateBrowsingMode):
- 5:46 PM Changeset in webkit [218856] by
-
- 2 edits in trunk/Source/JavaScriptCore
[JSC] Do not touch VM after notifying Ready in DFG::Worklist
https://bugs.webkit.org/show_bug.cgi?id=173888
Reviewed by Saam Barati.
After notifying Plan::Ready and releasing Worklist lock, VM can be destroyed.
Thus, Plan::vm() can return a destroyed VM. Do not touch it.
This causes occasional SEGV / assertion failures in workers/bomb test.
- dfg/DFGWorklist.cpp:
- 5:36 PM Changeset in webkit [218855] by
-
- 12 edits1 add in trunk
[iOS DnD] Support dragging out of contenteditable areas without a prior selection
https://bugs.webkit.org/show_bug.cgi?id=173854
<rdar://problem/32236827>
Reviewed by Ryosuke Niwa and Tim Horton.
Source/WebCore:
Allows elements to be dragged from contenteditable areas for both WebKit1 and WebKit2 iOS. There are two main
changes in WebCore: move the touch point adjustment code into EventHandler::tryToBeginDataInteractionAtPoint, so
that the clientPosition specified will be adjusted to an appropriate clickable node if needed. This is necessary
because UIWebDocumentView and WKContentView no longer send adjusted points to WebCore when requesting drag
start. See <https://bugs.webkit.org/show_bug.cgi?id=173855> for a followup regarding the globalPosition and
clientPositions passed in to the MouseEvents when performing a drag or synthetic click.
Secondly, image elements in Mail's contenteditable area are not draggable unless the heuristic in
DragController::draggableElement is tweaked to not reject image dragging across the board if the
loadsImagesAutomatically setting is turned off. Instead, even if images are not automatically loaded, allow the
image drag to commence if the image renderer already has a cached image.
Test: DataInteractionTests.DragImageFromContentEditable
- page/DragController.cpp:
(WebCore::imageElementIsDraggable):
(WebCore::DragController::draggableElement):
- page/ios/EventHandlerIOS.mm:
(WebCore::EventHandler::tryToBeginDataInteractionAtPoint):
Source/WebKit/mac:
Vends some information from the DragItem passed into -[WebView _startDrag:] through the WebView as SPI for
WebKit1 clients. No behavior change with these changes alone -- see <rdar://problem/32991062> for more detail.
- WebView/WebView.mm:
(-[WebView _startDrag:]):
(-[WebView _dragSourceAction]):
(-[WebView _draggedLinkTitle]):
(-[WebView _draggedLinkURL]):
(-[WebView _draggedElementBounds]):
(-[WebView _endedDataInteraction:global:]):
- WebView/WebViewData.h:
- WebView/WebViewPrivate.h:
Source/WebKit2:
Instead of allowing a drag to occur only if a position information request discovers a clickable node, remove
the position information request entirely and just call into WebCore to try and begin the drag. Currently, the
position information request serves two purposes:
- To adjust the hit-test location to account for nearby clickable nodes.
- To obtain information about the content being dragged.
The first requirement is fulfilled by performing the drag location adjustment in WebCore instead (see
ChangeLogs for more detail). The second requirement is fulfilled by refactoring in
https://bugs.webkit.org/show_bug.cgi?id=173832 to fold all information relevant to starting a drag into the
DragItem struct, and then propagating the DragItem struct. The relevant information from the position
information update is now populated when the UI process receives the drag start response.
- UIProcess/ios/WKContentViewInteraction.h:
- UIProcess/ios/WKContentViewInteraction.mm:
(-[WKContentView _startDrag:item:]):
(-[WKContentView computeClientAndGlobalPointsForDropSession:outClientPoint:outGlobalPoint:]):
(-[WKContentView _dragInteraction:prepareForSession:completion:]):
(-[WKContentView _api_dropInteraction:sessionDidEnter:]):
(-[WKContentView _api_dropInteraction:sessionDidUpdate:]):
(positionInformationMayStartDataInteraction): Deleted.
Tools:
Adds a new test to check that an image can be dragged out of a contenteditable and dropped.
- TestWebKitAPI/TestWebKitAPI.xcodeproj/project.pbxproj:
- TestWebKitAPI/Tests/WebKit2Cocoa/contenteditable-and-target.html: Added.
- TestWebKitAPI/Tests/ios/DataInteractionTests.mm:
(TestWebKitAPI::TEST):
- 4:55 PM Changeset in webkit [218854] by
-
- 2 edits in trunk/LayoutTests
Marked http/tests/loading/resourceLoadStatistics/telemetry-generation.html as flaky.
https://bugs.webkit.org/show_bug.cgi?id=173499
Unreviewed test gardening.
- platform/wk2/TestExpectations:
- 4:48 PM Changeset in webkit [218853] by
-
- 14 edits in trunk
[Modern Media Controls] Accessibility labels should be formatted using NSDateComponentsFormatter
https://bugs.webkit.org/show_bug.cgi?id=173858
<rdar://problem/32643171>
Patch by Antoine Quint <Antoine Quint> on 2017-06-27
Reviewed by Dean Jackson.
Source/WebCore:
We shouldn't be manually trying to create a formatted string for media controls and instead rely
on NSDateComponentsFormatter to perform this task for us. So we remove the ad-hoc code in the JS
media controls code and instead add a new MediaControlsHost method to format durations which calls
into RenderTheme to provide a formatted duration string relevant to the current platform and locale.
- English.lproj/modern-media-controls-localized-strings.js:
- Modules/mediacontrols/MediaControlsHost.cpp:
(WebCore::MediaControlsHost::formattedStringForDuration):
- Modules/mediacontrols/MediaControlsHost.h:
- Modules/mediacontrols/MediaControlsHost.idl:
- Modules/modern-media-controls/controls/slider.js:
(Slider.prototype.set inputAccessibleLabel):
- Modules/modern-media-controls/controls/time-label.js:
(TimeLabel.prototype.commitProperty):
- Modules/modern-media-controls/main.js:
(createControls):
(formattedStringForDuration):
(formatTimeToString): Deleted.
- rendering/RenderTheme.h:
(WebCore::RenderTheme::mediaControlsFormattedStringForDuration):
- rendering/RenderThemeCocoa.h:
- rendering/RenderThemeCocoa.mm:
(WebCore::RenderThemeCocoa::mediaControlsFormattedStringForDuration):
LayoutTests:
- media/modern-media-controls/scrubber/scrubber-has-correct-ax-label-expected.txt:
- media/modern-media-controls/scrubber/scrubber-has-correct-ax-label.html:
- 4:40 PM Changeset in webkit [218852] by
-
- 2 edits in trunk/Source/WebCore
r218647 causes getUserMedia to fail on some machines
https://bugs.webkit.org/show_bug.cgi?id=173894
Reviewed by Youenn Fablet.
- platform/mediastream/mac/AVVideoCaptureSource.mm:
(WebCore::AVVideoCaptureSource::isFrameRateSupported): Change frame rate epsilon from 0.00001
to 0.001.
- 4:29 PM Changeset in webkit [218851] by
-
- 5 edits in trunk
Placard icons act like buttons (can get keyboard focus and shows up in VoiceOver)
https://bugs.webkit.org/show_bug.cgi?id=173891
<rdar://problem/33011855>
Patch by Antoine Quint <Antoine Quint> on 2017-06-27
Reviewed by Dean Jackson.
Source/WebCore:
Ensure that we disable buttons inside placards as they're only decorative and should
not be interactive.
- Modules/modern-media-controls/controls/placard.js:
(Placard.):
LayoutTests:
Update the Placard test to check that its <button> is disabled.
- media/modern-media-controls/placard/placard-expected.txt:
- media/modern-media-controls/placard/placard.html:
- 4:05 PM Changeset in webkit [218850] by
-
- 343 edits269 adds60 deletes in trunk/Source/ThirdParty/libwebrtc
Update boringssl to c8ff30cbe716c72279a6f6a9d7d7d0d4091220fa
https://bugs.webkit.org/show_bug.cgi?id=173676
Patch by Youenn Fablet <youenn@apple.com> on 2017-06-27
Reviewed by Alex Christensen.
- Configurations/boringssl.xcconfig: Enabling ASM.
- Source/third_party/boringssl/BUILD.generated.gni:
- Source/third_party/boringssl: Updated folder according new revision.
- WebKit/patch-boringssl: Added, needed to fix some files to disable warnings.
- libwebrtc.xcodeproj/project.pbxproj:
- 3:54 PM Changeset in webkit [218849] by
-
- 48 edits5 adds in trunk/Source/ThirdParty/libwebrtc
Refresh usrsctp to Source/ThirdParty/libwebrtc/WebKit/patch-usrsctp and libsrtp to ccf84786f8ef803cb9c75e919e5a3976b9f5a67
https://bugs.webkit.org/show_bug.cgi?id=173673
Patch by Youenn Fablet <youenn@apple.com> on 2017-06-27
Reviewed by Sam Weinig.
- Source/third_party/libsrtp/README.chromium:
- Source/third_party/libsrtp/srtp/srtp.c:
(srtp_stream_init_keys):
(srtp_calc_aead_iv_srtcp):
(srtp_protect_rtcp_aead):
(srtp_unprotect_rtcp_aead):
- Source/third_party/libsrtp/test/srtp_driver.c:
(srtp_validate_encrypted_extensions_headers_gcm):
- Source/third_party/usrsctp/usrsctplib/.gitignore: Added.
- Source/third_party/usrsctp/usrsctplib/CMakeLists.txt:
- Source/third_party/usrsctp/usrsctplib/Makefile.am:
- Source/third_party/usrsctp/usrsctplib/README.md:
- Source/third_party/usrsctp/usrsctplib/configure.ac:
- Source/third_party/usrsctp/usrsctplib/programs/CMakeLists.txt:
- Source/third_party/usrsctp/usrsctplib/programs/Makefile.am:
- Source/third_party/usrsctp/usrsctplib/programs/client.c:
(main):
- Source/third_party/usrsctp/usrsctplib/programs/datachan_serv.c:
(main):
- Source/third_party/usrsctp/usrsctplib/programs/ekr_loop_offload.c: Added.
(handle_packets):
- Source/third_party/usrsctp/usrsctplib/programs/test_timer.c: Added.
(main):
- Source/third_party/usrsctp/usrsctplib/usrsctp.pc.in: Added.
- Source/third_party/usrsctp/usrsctplib/usrsctplib/CMakeLists.txt:
- Source/third_party/usrsctp/usrsctplib/usrsctplib/netinet/sctp_asconf.c:
- Source/third_party/usrsctp/usrsctplib/usrsctplib/netinet/sctp_asconf.h:
- Source/third_party/usrsctp/usrsctplib/usrsctplib/netinet/sctp_auth.c:
- Source/third_party/usrsctp/usrsctplib/usrsctplib/netinet/sctp_auth.h:
- Source/third_party/usrsctp/usrsctplib/usrsctplib/netinet/sctp_bsd_addr.c:
- Source/third_party/usrsctp/usrsctplib/usrsctplib/netinet/sctp_bsd_addr.h:
- Source/third_party/usrsctp/usrsctplib/usrsctplib/netinet/sctp_cc_functions.c:
(sctp_cwnd_update_after_fr):
(sctp_hs_cwnd_update_after_fr):
(sctp_htcp_cwnd_update_after_fr):
- Source/third_party/usrsctp/usrsctplib/usrsctplib/netinet/sctp_constants.h:
- Source/third_party/usrsctp/usrsctplib/usrsctplib/netinet/sctp_crc32.c:
- Source/third_party/usrsctp/usrsctplib/usrsctplib/netinet/sctp_crc32.h:
- Source/third_party/usrsctp/usrsctplib/usrsctplib/netinet/sctp_header.h:
- Source/third_party/usrsctp/usrsctplib/usrsctplib/netinet/sctp_indata.c:
(sctp_build_readq_entry):
(sctp_place_control_in_stream):
(sctp_abort_in_reasm):
(sctp_queue_data_to_stream):
(sctp_build_readq_entry_from_ctl):
(sctp_handle_old_unordered_data):
(sctp_inject_old_unordered_data):
(sctp_deliver_reasm_check):
(sctp_add_chk_to_control):
(sctp_queue_data_for_reasm):
(sctp_find_reasm_entry):
(sctp_process_a_data_chunk):
(sctp_sack_check):
(sctp_process_segment_range):
(sctp_check_for_revoked):
- Source/third_party/usrsctp/usrsctplib/usrsctplib/netinet/sctp_indata.h:
- Source/third_party/usrsctp/usrsctplib/usrsctplib/netinet/sctp_input.c:
(sctp_process_init):
(sctp_process_cookie_existing):
- Source/third_party/usrsctp/usrsctplib/usrsctplib/netinet/sctp_input.h:
- Source/third_party/usrsctp/usrsctplib/usrsctplib/netinet/sctp_output.c:
- Source/third_party/usrsctp/usrsctplib/usrsctplib/netinet/sctp_output.h:
- Source/third_party/usrsctp/usrsctplib/usrsctplib/netinet/sctp_pcb.c:
- Source/third_party/usrsctp/usrsctplib/usrsctplib/netinet/sctp_pcb.h:
- Source/third_party/usrsctp/usrsctplib/usrsctplib/netinet/sctp_peeloff.h:
- Source/third_party/usrsctp/usrsctplib/usrsctplib/netinet/sctp_ss_functions.c:
(sctp_ss_rr_add):
(sctp_ss_fcfs_select):
- Source/third_party/usrsctp/usrsctplib/usrsctplib/netinet/sctp_structs.h:
- Source/third_party/usrsctp/usrsctplib/usrsctplib/netinet/sctp_sysctl.c:
- Source/third_party/usrsctp/usrsctplib/usrsctplib/netinet/sctp_timer.c:
(sctp_recover_sent_list):
- Source/third_party/usrsctp/usrsctplib/usrsctplib/netinet/sctp_uio.h:
- Source/third_party/usrsctp/usrsctplib/usrsctplib/netinet/sctp_usrreq.c:
(sctp_init):
(sctp_pathmtu_adjustment):
- Source/third_party/usrsctp/usrsctplib/usrsctplib/netinet/sctp_var.h:
- Source/third_party/usrsctp/usrsctplib/usrsctplib/netinet/sctputil.c:
(sctp_log_strm_del):
(sctp_init_asoc):
(sctp_notify_send_failed):
(sctp_notify_send_failed2):
- Source/third_party/usrsctp/usrsctplib/usrsctplib/netinet/sctputil.h:
- Source/third_party/usrsctp/usrsctplib/usrsctplib/netinet6/sctp6_usrreq.c:
- Source/third_party/usrsctp/usrsctplib/usrsctplib/netinet6/sctp6_var.h:
- Source/third_party/usrsctp/usrsctplib/usrsctplib/user_mbuf.c:
(m_get):
(mbuf_initialize):
- Source/third_party/usrsctp/usrsctplib/usrsctplib/user_mbuf.h:
- Source/third_party/usrsctp/usrsctplib/usrsctplib/user_socket.c:
- Source/third_party/usrsctp/usrsctplib/usrsctplib/usrsctp.h:
- WebKit/patch-usrsctp: Added.
- 3:43 PM Changeset in webkit [218848] by
-
- 2 edits in trunk/Source/JavaScriptCore
Remove an inaccurate comment inside DFGClobberize.h
https://bugs.webkit.org/show_bug.cgi?id=163874
Reviewed by Filip Pizlo.
The comment said that Clobberize may or may not be sound if run prior to
doing type inference. This is not correct, though. Clobberize *must* be sound
prior do doing type inference since we use it inside the BytecodeParser, which
is the very first thing the DFG does.
- dfg/DFGClobberize.h:
(JSC::DFG::clobberize):
- 3:39 PM Changeset in webkit [218847] by
-
- 4 edits2 adds in trunk
Disable m_temporarilyAllowingInlinePlaybackAfterFullscreen on pause
https://bugs.webkit.org/show_bug.cgi?id=173843
rdar://problem/32982431
Patch by Jeremy Jones <jeremyj@apple.com> on 2017-06-27
Reviewed by Eric Carlson.
Source/WebCore:
Test: media/media-fullscreen-pause-inline.html
Some pages may not have a fullscreen button, so disabled m_temporarilyAllowingInlinePlaybackAfterFullscreen on pause.
- html/HTMLMediaElement.cpp:
(WebCore::HTMLMediaElement::pause):
LayoutTests:
Some pages may not have a fullscreen button, so disabled m_temporarilyAllowingInlinePlaybackAfterFullscreen on pause.
- media/media-fullscreen-pause-inline-expected.txt: Added.
- media/media-fullscreen-pause-inline.html: Added.
- platform/mac-wk2/TestExpectations:
- 2:49 PM Changeset in webkit [218846] by
-
- 5 edits in trunk/Source
[iOS] Avoid taking / releasing process assertions too quickly due to database activity
https://bugs.webkit.org/show_bug.cgi?id=173879
<rdar://problem/32412701>
Reviewed by Antti Koivisto.
Source/WebCore:
Add HysteresisActivity to WebSQLiteDatabaseTrackerClient to avoid taking / releasing
process assertion too quickly due to database activity.
- platform/ios/WebSQLiteDatabaseTrackerClient.h:
- platform/ios/WebSQLiteDatabaseTrackerClient.mm:
(WebCore::WebSQLiteDatabaseTrackerClient::WebSQLiteDatabaseTrackerClient):
(WebCore::WebSQLiteDatabaseTrackerClient::willBeginFirstTransaction):
(WebCore::WebSQLiteDatabaseTrackerClient::didFinishLastTransaction):
(WebCore::WebSQLiteDatabaseTrackerClient::hysteresisUpdated):
Source/WebKit2:
Specify an activity name when taking the process assertion to facilitate
debugging in the future. It took me a while to figure out this assertion
was the one that was too aggressive because it was anonymous.
- UIProcess/API/Cocoa/WKWebView.mm:
(-[WKWebView _setUpSQLiteDatabaseTrackerClient]):
- 2:37 PM Changeset in webkit [218845] by
-
- 14 edits1 add in trunk
Function constructor needs to follow the spec and validate parameters and body independently
https://bugs.webkit.org/show_bug.cgi?id=173303
<rdar://problem/32732526>
Reviewed by Keith Miller.
JSTests:
- ChakraCore/test/Function/FuncBodyES5.baseline-jsc:
- stress/function-constructor-semantics.js: Added.
(assert):
(hasSyntaxError):
(foo):
(async.foo):
(testError):
(testOK.toString):
(toString):
LayoutTests/imported/w3c:
- web-platform-tests/html/webappapis/scripting/events/inline-event-handler-ordering-expected.txt:
- web-platform-tests/html/webappapis/scripting/events/invalid-uncompiled-raw-handler-compiled-late-expected.txt:
- web-platform-tests/html/webappapis/scripting/processing-model-2/compile-error-in-attribute-expected.txt:
- web-platform-tests/html/webappapis/scripting/processing-model-2/compile-error-in-body-onerror-expected.txt:
Source/JavaScriptCore:
The Function constructor must check the arguments and body strings
independently for syntax errors. People rely on this specified behavior
to verify that a particular string is a valid function body. We used
to check these things strings concatenated together, instead of
independently. For example, this used to be valid:Function("/*", "*/){").
However, we should throw a syntax error here since "(/*)" is not a valid
parameter list, and "*/){" is not a valid body.
To implement the specified behavior, we check the syntax independently of
both the body and the parameter list. To check that the parameter list has
valid syntax, we check that it is valid if in a function with an empty body.
To check that the body has valid syntax, we check it is valid in a function
with an empty parameter list.
- runtime/FunctionConstructor.cpp:
(JSC::constructFunctionSkippingEvalEnabledCheck):
LayoutTests:
- fast/dom/attribute-event-listener-errors-expected.txt:
- fast/events/attribute-listener-deletion-crash-expected.txt:
- fast/events/window-onerror-syntax-error-in-attr-expected.txt:
- js/dom/invalid-syntax-for-function-expected.txt:
- 2:35 PM Changeset in webkit [218844] by
-
- 4 edits in trunk/Source/WebCore
Using public logging for WebRTC release logging
https://bugs.webkit.org/show_bug.cgi?id=173881
Patch by Youenn Fablet <youenn@apple.com> on 2017-06-27
Reviewed by Eric Carlson.
No change of behavior.
- Modules/mediastream/PeerConnectionBackend.cpp:
(WebCore::PeerConnectionBackend::createOfferSucceeded):
(WebCore::PeerConnectionBackend::createOfferFailed):
(WebCore::PeerConnectionBackend::createAnswerSucceeded):
(WebCore::PeerConnectionBackend::createAnswerFailed):
(WebCore::PeerConnectionBackend::setLocalDescriptionFailed):
(WebCore::PeerConnectionBackend::setRemoteDescriptionFailed):
(WebCore::PeerConnectionBackend::addIceCandidateFailed):
(WebCore::PeerConnectionBackend::newICECandidate):
- Modules/mediastream/RTCPeerConnection.cpp:
(WebCore::RTCPeerConnection::queuedSetLocalDescription):
(WebCore::RTCPeerConnection::queuedSetRemoteDescription):
(WebCore::RTCPeerConnection::queuedAddIceCandidate):
(WebCore::RTCPeerConnection::updateIceGatheringState):
(WebCore::RTCPeerConnection::updateIceConnectionState):
- Modules/mediastream/libwebrtc/LibWebRTCMediaEndpoint.cpp:
(WebCore::LibWebRTCMediaEndpoint::OnStatsDelivered):
- 2:30 PM Changeset in webkit [218843] by
-
- 16 edits1 copy1 add in trunk
[PAL] Add symbol export macros for PAL
https://bugs.webkit.org/show_bug.cgi?id=171519
Reviewed by Konstantin Tokarev.
Source/WebCore:
No new tests. No change in behavior.
- CMakeLists.txt:
- config.h:
- platform/PlatformExportMacros.h:
Source/WebCore/PAL:
No new tests. No change in behavior.
- Configurations/CopyPALHeaders.xcconfig: Added.
- PAL.xcodeproj/project.pbxproj:
- config.h:
- pal/ExportMacros.h: Copied from Source/WebCore/platform/PlatformExportMacros.h.
- pal/PlatformWin.cmake:
- pal/crypto/CryptoDigest.h:
Source/WebKit2:
- config.h:
Tools:
- DumpRenderTree/config.h:
- TestWebKitAPI/CMakeLists.txt:
- TestWebKitAPI/config.h:
- WebKitTestRunner/config.h:
- 2:23 PM Changeset in webkit [218842] by
-
- 2 edits in trunk/Source/WebKit2
ProcessLauncherMac could leak a xpc connection when the process had failed to launch
https://bugs.webkit.org/show_bug.cgi?id=173865
Reviewed by Antti Koivisto.
Explicitly cancel the XPC connection even When the process launch had failed to avoid a leak.
- UIProcess/Launcher/mac/ProcessLauncherMac.mm:
(WebKit::ProcessLauncher::launchProcess):
- 2:10 PM Changeset in webkit [218841] by
-
- 30 edits4 adds in trunk
Resource Load Statistics: Add telemetry
https://bugs.webkit.org/show_bug.cgi?id=173499
<rdar://problem/32826094>
Reviewed by Brent Fulgham.
Source/WebCore:
Test: http/tests/loading/resourceLoadStatistics/telemetry-generation.html
- loader/ResourceLoadObserver.cpp:
(WebCore::ResourceLoadObserver::fireTelemetryHandler):
Test infrastructure.
- loader/ResourceLoadObserver.h:
- loader/ResourceLoadStatisticsStore.cpp:
(WebCore::ResourceLoadStatisticsStore::setFireTelemetryCallback):
(WebCore::ResourceLoadStatisticsStore::fireTelemetryHandler):
Test infrastructure.
(WebCore::ResourceLoadStatisticsStore::sortedPrevalentResourceTelemetry):
Convenience function for telemetry.
- loader/ResourceLoadStatisticsStore.h:
Added struct WebCore::PrevalentResourceTelemetry.
- page/DiagnosticLoggingKeys.cpp:
(WebCore::DiagnosticLoggingKeys::resourceLoadStatisticsTelemetryKey):
Added.
- page/DiagnosticLoggingKeys.h:
Source/WebKit2:
- PlatformGTK.cmake:
Added UIProcess/WebResourceLoadStatisticsTelemetry.cpp
- PlatformMac.cmake:
Added UIProcess/WebResourceLoadStatisticsTelemetry.cpp
- PlatformWPE.cmake:
Added UIProcess/WebResourceLoadStatisticsTelemetry.cpp
- UIProcess/API/C/WKResourceLoadStatisticsManager.cpp:
(WKResourceLoadStatisticsManagerFireTelemetryHandler):
(WKResourceLoadStatisticsManagerSetNotifyPagesWhenTelemetryWasCaptured):
Test infrastructure.
- UIProcess/API/C/WKResourceLoadStatisticsManager.h:
- UIProcess/WebProcessProxy.cpp:
(WebKit::WebProcessProxy::notifyPageStatisticsTelemetryFinished):
Test infrastructure.
- UIProcess/WebProcessProxy.h:
- UIProcess/WebResourceLoadStatisticsManager.cpp:
(WebKit::WebResourceLoadStatisticsManager::fireTelemetryHandler):
(WebKit::WebResourceLoadStatisticsManager::setNotifyPagesWhenTelemetryWasCaptured):
(WebKit::WebResourceLoadStatisticsManager::resetToConsistentState):
Test infrastructure.
- UIProcess/WebResourceLoadStatisticsManager.h:
- UIProcess/WebResourceLoadStatisticsStore.cpp:
(WebKit::WebResourceLoadStatisticsStore::WebResourceLoadStatisticsStore):
Configures a timer for telemetry capture. Fires 5 seconds after launch
and then every 24 hours.
(WebKit::WebResourceLoadStatisticsStore::setNotifyPagesWhenDataRecordsWereScanned):
Variable renamed notifyPages -> notifyPagesWhenDataRecordsWereScanned.
(WebKit::WebResourceLoadStatisticsStore::removeDataRecords):
Variable renamed notifyPages -> notifyPagesWhenDataRecordsWereScanned.
(WebKit::WebResourceLoadStatisticsStore::processStatisticsAndDataRecords):
Variable renamed notifyPages -> notifyPagesWhenDataRecordsWereScanned.
(WebKit::WebResourceLoadStatisticsStore::registerSharedResourceLoadObserver):
Variable renamed notifyPages -> notifyPagesWhenDataRecordsWereScanned.
(WebKit::WebResourceLoadStatisticsStore::grandfatherExistingWebsiteData):
Variable renamed notifyPages -> notifyPagesWhenDataRecordsWereScanned.
(WebKit::WebResourceLoadStatisticsStore::telemetryTimerFired):
Calls WebResourceLoadStatisticsTelemetry::calculateAndSubmit().
- UIProcess/WebResourceLoadStatisticsStore.h:
- UIProcess/WebResourceLoadStatisticsTelemetry.cpp: Added.
(WebKit::numberOfResourcesWithUserInteraction):
(WebKit::median):
(WebKit::nonEphemeralWebPageProxy):
(WebKit::submitTopList):
(WebKit::submitTopLists):
(WebKit::notifyPages):
(WebKit::WebResourceLoadStatisticsTelemetry::calculateAndSubmit):
(WebKit::WebResourceLoadStatisticsTelemetry::setNotifyPagesWhenTelemetryWasCaptured):
- UIProcess/WebResourceLoadStatisticsTelemetry.h: Added.
- WebKit2.xcodeproj/project.pbxproj:
Tools:
Adds three new testRunner functions:
- installStatisticsDidRunTelemetryCallback()
- statisticsFireTelemetryHandler()
- setStatisticsNotifyPagesWhenTelemetryWasCaptured()
- WebKitTestRunner/InjectedBundle/Bindings/TestRunner.idl:
- WebKitTestRunner/InjectedBundle/InjectedBundle.cpp:
(WTR::InjectedBundle::didReceiveMessageToPage):
- WebKitTestRunner/InjectedBundle/TestRunner.cpp:
(WTR::TestRunner::installStatisticsDidRunTelemetryCallback):
(WTR::TestRunner::statisticsDidRunTelemetryCallback):
(WTR::TestRunner::statisticsFireTelemetryHandler):
(WTR::TestRunner::setStatisticsNotifyPagesWhenTelemetryWasCaptured):
- WebKitTestRunner/InjectedBundle/TestRunner.h:
- WebKitTestRunner/TestController.cpp:
(WTR::TestController::resetStateToConsistentValues):
(WTR::TestController::statisticsFireTelemetryHandler):
(WTR::TestController::setStatisticsNotifyPagesWhenTelemetryWasCaptured):
- WebKitTestRunner/TestController.h:
- WebKitTestRunner/TestInvocation.cpp:
(WTR::TestInvocation::didReceiveSynchronousMessageFromInjectedBundle):
LayoutTests:
- http/tests/loading/resourceLoadStatistics/telemetry-generation-expected.txt: Added.
- http/tests/loading/resourceLoadStatistics/telemetry-generation.html: Added.
- platform/wk2/TestExpectations:
Enabled here since ResourceLoadStatistics is WK2-only.
- 2:06 PM Changeset in webkit [218840] by
-
- 13 edits in trunk
Add missing includes to fix compilation error on FreeBSD
https://bugs.webkit.org/show_bug.cgi?id=172919
Patch by Ting-Wei Lan <Ting-Wei Lan> on 2017-06-27
Reviewed by Mark Lam.
Source/JavaScriptCore:
- API/JSRemoteInspector.h:
- API/tests/GlobalContextWithFinalizerTest.cpp:
- API/tests/TypedArrayCTest.cpp:
Source/WebCore:
No new tests needed.
- crypto/gcrypt/CryptoKeyECGCrypt.cpp:
- platform/audio/ReverbAccumulationBuffer.cpp:
Source/WebKit2:
- UIProcess/API/C/WKContextPrivate.h:
- UIProcess/API/C/WKPagePrivate.h:
- UIProcess/ProcessAssertion.h:
Tools:
- ImageDiff/cairo/PlatformImageCairo.cpp:
- 2:01 PM Changeset in webkit [218839] by
-
- 7 edits1 move3 adds in trunk/Source/WebInspectorUI
Web Inspector: no discoverable way to dismiss the split console
https://bugs.webkit.org/show_bug.cgi?id=167034
<rdar://problem/30023436>
Reviewed by Devin Rousso.
This patch adds an Xcode-style toggle button to the left of the
split console navigation bar. Additionally, the split console code
has been broken out into it's own class to reduce the clutter in
Main.js/css, and simplify future console maintenance.
- Localizations/en.lproj/localizedStrings.js:
- UserInterface/Base/Main.js:
(WebInspector.loaded):
Replace the "split" ContentBrowser with a new ConsoleDrawer class,
which encapsulates the drawer height setting and resize behavior.
(WebInspector.contentLoaded):
The console drawer manages its own height.
- UserInterface/Images/HideConsoleDrawer.svg: Added.
- UserInterface/Images/ShowConsoleDrawer.svg: Added.
Console drawer toggle button images. Styled after corresponding Xcode button.
- UserInterface/Main.html:
- UserInterface/Views/ConsoleDrawer.css: Added.
Moved old #split-content-browser styles over from Main.css.
(.console-drawer):
(.console-drawer > .navigation-bar):
(.console-drawer > .navigation-bar > .item.button:not(.clear-log)):
Make button spacing less cramped.
(.console-drawer > .navigation-bar > .item:not(.flexible-space)):
(.console-drawer > .navigation-bar > :matches(.item.button, .log-scope-bar)):
(.console-drawer > .navigation-bar > .item > .glyph,):
- UserInterface/Views/ConsoleDrawer.js: Added.
New class responsible for console drawer expand / collapse and resizer
dragging, which was previously part of Main.js. The console drawer toolbar
now has a toggle button, debugging controls, and a call frames path item.
The "Show Console Tab" button has been removed.
(WebInspector.ConsoleDrawer):
(WebInspector.ConsoleDrawer.prototype.get collapsed):
(WebInspector.ConsoleDrawer.prototype.set collapsed):
(WebInspector.ConsoleDrawer.prototype.get height):
(WebInspector.ConsoleDrawer.prototype.shown):
(WebInspector.ConsoleDrawer.prototype.layout):
(WebInspector.ConsoleDrawer.prototype._consoleResizerMouseDown.dockedResizerDrag):
(WebInspector.ConsoleDrawer.prototype._consoleResizerMouseDown.dockedResizerDragEnd):
(WebInspector.ConsoleDrawer.prototype._consoleResizerMouseDown):
(WebInspector.ConsoleDrawer.prototype._restoreDrawerHeight):
(WebInspector.ConsoleDrawer.prototype._updateDrawerHeight):
(WebInspector.ConsoleDrawer.prototype._selectedTabContentViewDidChange):
When switching to a tab that doesn't support showing the console drawer,
hide it without altering the collapsed state.
- UserInterface/Views/LogContentView.js:
(WebInspector.LogContentView.prototype.get navigationItems):
- UserInterface/Views/Main.css:
Renamed #split-content-browser to #console-drawer, and moved related
styles to ConsoleDrawer.css.
(#split-content-browser): Deleted.
(#split-content-browser > .navigation-bar): Deleted.
(#split-content-browser > .navigation-bar :matches(.find-banner, .find-banner + .divider)): Deleted.
(#split-content-browser > .navigation-bar .item): Deleted.
(#split-content-browser > .navigation-bar .item:not(.flexible-space)): Deleted.
(#split-content-browser .hierarchical-path-component.log-icon .icon): Deleted.
(#split-content-browser .hierarchical-path-component.log-icon .title): Deleted.
(#split-content-browser .hierarchical-path-component.log-icon select): Deleted.
(#split-content-browser > .navigation-bar > :matches(.hierarchical-path, .log-scope-bar)): Deleted.
(#split-content-browser > .navigation-bar > .log-scope-bar > li): Deleted.
- UserInterface/Views/QuickConsole.js:
(WebInspector.QuickConsole):
(WebInspector.QuickConsole.prototype._toggleOrFocus):
(WebInspector.QuickConsole.prototype._updateStyles):
Update top border style in response to change in selected TabBrowser
tab or the console drawer collapsed state.
(WebInspector.QuickConsole.prototype.consoleLogVisibilityChanged): Deleted.
(WebInspector.QuickConsole.prototype.set keyboardShortcutDisabled): Deleted.
- 12:52 PM Changeset in webkit [218838] by
-
- 3 edits in trunk/Source/WebCore
Add RenderEmbeddedObject::getReplacementTextGeometry helper.
https://bugs.webkit.org/show_bug.cgi?id=173847
Reviewed by Simon Fraser.
...and remove getReplacementTextGeometry's redundant return value.
No change in functionality.
- rendering/RenderEmbeddedObject.cpp:
(WebCore::RenderEmbeddedObject::paintReplaced):
(WebCore::RenderEmbeddedObject::getReplacementTextGeometry):
(WebCore::RenderEmbeddedObject::unavailablePluginIndicatorBounds):
(WebCore::RenderEmbeddedObject::isInUnavailablePluginIndicator):
- rendering/RenderEmbeddedObject.h:
- 12:11 PM Changeset in webkit [218837] by
-
- 31 edits in trunk/Source
Refactor drag start codepaths to plumb a DragItem to client layers
https://bugs.webkit.org/show_bug.cgi?id=173832
Work towards <rdar://problem/32236827>
Reviewed by Ryosuke Niwa and Tim Horton.
Source/WebCore:
Refactor drag start logic in WebCore to set up a DragItem and propagate it to WebDragClient. No change in behavior.
- loader/EmptyClients.cpp:
- page/DragClient.h:
- page/DragController.cpp:
(WebCore::DragController::startDrag):
(WebCore::DragController::doImageDrag):
(WebCore::DragController::doSystemDrag):
Refactor to pass along a DragItem. Also, remove unused drag image anchor computation.
- page/DragController.h:
- platform/DragImage.h:
- platform/DragItem.h:
Add additional information needed to begin a drag on iOS.
(WebCore::DragItem::encode):
(WebCore::DragItem::decode):
Add IPC serialization/deserialization support for DragItem.
- platform/PasteboardWriterData.cpp:
(WebCore::PasteboardWriterData::isEmpty):
- platform/PasteboardWriterData.h:
Source/WebKit/mac:
Refactor code used to start a drag in WebKit1 to plumb a single DragItem from DragController to the client layer
in WebDragClient that contains all the information needed to begin a drag. Also renames _setDataInteractionData:
to _startDrag: to match logic used to start a drag on other platforms.
No behavior change -- the same information should be propagated to AppKit and UIKit when beginning a drag, only
through the DragItem struct.
- WebCoreSupport/WebDragClient.h:
- WebCoreSupport/WebDragClient.mm:
(WebDragClient::startDrag):
(WebDragClient::dragSourceActionMaskForPoint):
(WebDragClient::willPerformDragSourceAction):
(WebDragClient::beginDrag):
- WebView/WebView.mm:
(-[WebView _startDrag:]):
(-[WebView _setDataInteractionData:textIndicator:atClientPosition:anchorPoint:action:]): Deleted.
- WebView/WebViewInternal.h:
Source/WebKit/win:
Adjust for changes to WebDragClient::startDrag by pulling information out of DragItem.
- WebCoreSupport/WebDragClient.cpp:
(WebDragClient::startDrag):
- WebCoreSupport/WebDragClient.h:
Source/WebKit2:
Add support for serializing a DragItem over IPC. This encodes and decodes all members of DragItem except for the
PasteboardWriterData and the DragImage's image. The DragItem is sent over to the UI process in
WebDragClient::startDrag via the WebPageProxy::startDrag IPC message (renamed from WebPageProxy::setDragImage)
which now takes a DragItem.
- UIProcess/Cocoa/WebPageProxyCocoa.mm:
(WebKit::WebPageProxy::startDrag):
(WebKit::WebPageProxy::setDragImage): Deleted.
- UIProcess/PageClient.h:
- UIProcess/WebPageProxy.h:
- UIProcess/WebPageProxy.messages.in:
Rename SetDragImage to StartDrag and refactor to receive only a DragItem and shareable bitmap handle.
- UIProcess/ios/PageClientImplIOS.h:
- UIProcess/ios/PageClientImplIOS.mm:
(WebKit::PageClientImpl::startDrag):
(WebKit::PageClientImpl::startDataInteractionWithImage): Deleted.
- UIProcess/ios/WKContentViewInteraction.h:
- UIProcess/ios/WKContentViewInteraction.mm:
(-[WKContentView _startDrag:item:]):
(-[WKContentView _startDataInteractionWithImage:withIndicatorData:atClientPosition:anchorPoint:action:]): Deleted.
Rename startDataInteractionWithImage to startDrag.
- UIProcess/mac/WebPageProxyMac.mm:
(WebKit::WebPageProxy::startDrag):
(WebKit::WebPageProxy::setDragImage): Deleted.
- WebProcess/WebCoreSupport/WebDragClient.cpp:
(WebKit::WebDragClient::startDrag):
- WebProcess/WebCoreSupport/WebDragClient.h:
- WebProcess/WebCoreSupport/gtk/WebDragClientGtk.cpp:
(WebKit::WebDragClient::startDrag):
- WebProcess/WebCoreSupport/mac/WebDragClientMac.mm:
(WebKit::WebDragClient::startDrag):
- 10:43 AM Changeset in webkit [218836] by
-
- 15 edits3 adds in trunk
Web Inspector: Crash generating object preview for ArrayIterator
https://bugs.webkit.org/show_bug.cgi?id=173754
<rdar://problem/32859012>
Reviewed by Saam Barati.
Source/JavaScriptCore:
When Inspector generates an object preview for an ArrayIterator instance it made
a "clone" of the original ArrayIterator instance by constructing a new object with
the instance's structure. However, user code could have modified that instance's
structure, such as adding / removing properties. Thereturnproperty had special
meaning, and our clone did not fill that slot. This approach is brittle in that
we weren't satisfying the expectations of an object with a particular Structure,
and the original goal of having Web Inspector peek values of built-in Iterators
was to avoid observable behavior.
This tightens Web Inspector's Iterator preview to only peek values if the
Iterators would actually be non-observable. It also builds an ArrayIterator
clone like a regular object construction.
- inspector/JSInjectedScriptHost.cpp:
(Inspector::cloneArrayIteratorObject):
Build up the Object from scratch with a new ArrayIterator prototype.
(Inspector::JSInjectedScriptHost::iteratorEntries):
Only clone and peek iterators if it would not be observable.
Also update iteration to be more in line with IterationOperations, such as when
we call iteratorClose.
- runtime/JSGlobalObject.cpp:
(JSC::JSGlobalObject::JSGlobalObject):
(JSC::JSGlobalObject::init):
- runtime/JSGlobalObject.h:
(JSC::JSGlobalObject::stringIteratorProtocolWatchpoint):
- runtime/JSGlobalObjectInlines.h:
(JSC::JSGlobalObject::isStringPrototypeIteratorProtocolFastAndNonObservable):
Add a StringIterator WatchPoint in line with the Array/Map/Set iterator watchpoints.
- runtime/JSMap.cpp:
(JSC::JSMap::isIteratorProtocolFastAndNonObservable):
(JSC::JSMap::canCloneFastAndNonObservable):
- runtime/JSMap.h:
- runtime/JSSet.cpp:
(JSC::JSSet::isIteratorProtocolFastAndNonObservable):
(JSC::JSSet::canCloneFastAndNonObservable):
- runtime/JSSet.h:
Promote isIteratorProtocolFastAndNonObservable to a method.
- runtime/JSObject.cpp:
(JSC::canDoFastPutDirectIndex):
- runtime/JSTypeInfo.h:
(JSC::TypeInfo::isArgumentsType):
Helper to detect if an Object is an Arguments type.
LayoutTests:
- platform/mac/inspector/model/remote-object-expected.txt:
- inspector/model/remote-object-expected.txt:
- inspector/model/remote-object.html:
Test generating a preview for an ArrayIterator that has had a
returnproperty added to it.
- inspector/model/remote-object-mutated-iterators-expected.txt: Added.
- inspector/model/remote-object-mutated-iterators.html: Added.
Test generating a preview for different iterators after the IteratorPrototypes have been mutated.
- inspector/model/resources/remote-object-utilities.js: Added.
(runInBrowserTest):
(TestPage.registerInitializer):
(TestPage.registerInitializer.checkComplete):
(TestPage.registerInitializer.window.runSteps):
Share code for remote-object dump tests.
- 8:59 AM Changeset in webkit [218835] by
-
- 14 edits in trunk
Some tests to verify forbidden frame navigation time out
https://bugs.webkit.org/show_bug.cgi?id=173657
Patch by Frederic Wang <fwang@igalia.com> on 2017-06-27
Reviewed by Chris Dumez.
LayoutTests/imported/w3c:
- web-platform-tests/html/semantics/embedded-content/the-iframe-element/iframe_sandbox_allow_top_navigation-2-expected.txt: Update the text expectation to PASS.
- web-platform-tests/html/semantics/embedded-content/the-iframe-element/iframe_sandbox_allow_top_navigation_by_user_activation_without_user_gesture-expected.txt: Ditto.
- web-platform-tests/html/semantics/embedded-content/the-iframe-element/iframe_sandbox_navigate_ancestor-1-expected.txt: Ditto.
- web-platform-tests/html/semantics/embedded-content/the-iframe-element/iframe_sandbox_popups_escaping-3-expected.txt: Add the security error until bug 173162 is fixed.
- web-platform-tests/html/semantics/embedded-content/the-iframe-element/iframe_sandbox_popups_nonescaping-3-expected.txt: Ditto.
Source/WebCore:
Currently some tests try and perform a forbidden frame navigation and verify the
corresponding console error. However, WebKit does not raise any exception for such error so
the tests have to wait until the timeout limit to complete, which makes execution slow.
This patch modifies the setters of window.location for which such error may happen in order
to raise an exception so the tests behave as expected.
No new tests, already covered by existing tests.
- page/Location.cpp: Adjust Location::setLocation to return a security exception and pass it
to the callers.
(WebCore::Location::setHref): Adjust function to possibly return an exception.
(WebCore::Location::setProtocol): Ditto.
(WebCore::Location::setHost): Ditto.
(WebCore::Location::setHostname): Ditto.
(WebCore::Location::setPort): Ditto.
(WebCore::Location::setPathname): Ditto.
(WebCore::Location::setSearch): Ditto.
(WebCore::Location::setHash): Ditto.
(WebCore::Location::assign): Ditto.
(WebCore::Location::setLocation): FrameLoader::findFrameForNavigation is really only used
to verify whether navigating m_frame is permitted so it is more simple and clearer to do it
directly. When navigation is not permitted, this function now raises a security exception.
- page/Location.h: Modify some setters to return an ExceptionOr<void>.
- page/Location.idl: Allow some setters to raise an exception.
LayoutTests:
- fast/frames/sandboxed-iframe-navigation-top-denied-expected.txt: Add the security error.
- http/tests/security/frameNavigation/inactive-function-in-popup-navigate-child.html: Adjust
the test to catch and dump the exception and complete immediately.
- http/tests/security/frameNavigation/inactive-function-in-popup-navigate-child-expected.txt:
Add the dumped security error exception.
- 7:49 AM Changeset in webkit [218834] by
-
- 2 edits in trunk/Source/WebKit2
[iOS DnD] [WK2] WKWebView does not handle drag sessions that exit and re-enter
https://bugs.webkit.org/show_bug.cgi?id=173856
<rdar://problem/32957100>
Reviewed by Ryosuke Niwa.
Upon exiting a WKContentView, the dropSession tracked by _dataInteractionState is not cleared out. If the drag
session re-enters the WKWebView, a _different_ UIDropSession is sent to the drop delegate, causing us to bail
from handling the session altogether in -dropInteraction:canHandleSession:. To fix this, we need to clear out
the dropSession when the drag session exits the view.
- UIProcess/ios/WKContentViewInteraction.mm:
(-[WKContentView dropInteraction:sessionDidExit:]):
- 7:47 AM Changeset in webkit [218833] by
-
- 2 edits in trunk/LayoutTests
LayoutTest webrtc/datachannel/multiple-connections.html is a flaky timeout
https://bugs.webkit.org/show_bug.cgi?id=173853
Patch by Youenn Fablet <youenn@apple.com> on 2017-06-27
Reviewed by Eric Carlson.
- webrtc/datachannel/multiple-connections.html:
- 4:13 AM WebKitGTK/2.16.x edited by
- (diff)
Jun 26, 2017:
- 11:41 PM Changeset in webkit [218832] by
-
- 3 edits in trunk/Source/WebCore
[GTK] Layout Test webrtc/video.html issues "stack smashing detected"
https://bugs.webkit.org/show_bug.cgi?id=173862
Patch by Fujii Hironori <Fujii Hironori> on 2017-06-26
Reviewed by Carlos Garcia Campos.
Tests: webrtc/video.html
Passing a bool variable to g_object_get causes out-of-bound write.
gboolean should be used, which is 4 bytes while bool is one byte.
- platform/graphics/gstreamer/MediaPlayerPrivateGStreamerBase.cpp:
(WebCore::MediaPlayerPrivateGStreamerBase::muted): Use gboolean instead of bool.
- platform/graphics/gstreamer/MediaPlayerPrivateGStreamerOwr.cpp:
(WebCore::MediaPlayerPrivateGStreamerOwr::trackEnded): Ditto.
- 11:12 PM Changeset in webkit [218831] by
-
- 1 copy in releases/WebKitGTK/webkit-2.16.5
WebKitGTK+ 2.16.5
- 11:11 PM Changeset in webkit [218830] by
-
- 4 edits in releases/WebKitGTK/webkit-2.16
Unreviewed. Update OptionsGTK.cmake and NEWS for 2.16.5 release.
.:
- Source/cmake/OptionsGTK.cmake:
Source/WebKit2:
- gtk/NEWS: Add release notes for 2.16.5.
- 11:01 PM Changeset in webkit [218829] by
-
- 5 edits in releases/WebKitGTK/webkit-2.16
Merge r218798 - Unreviewed, rolling out r215190.
Broke product select element on GNOME Bugzilla
Reverted changeset:
"[GTK] Misplaced right click menu on web page due to
deprecated gtk_menu_popup()"
https://bugs.webkit.org/show_bug.cgi?id=170553
http://trac.webkit.org/changeset/215190
- 10:49 PM Changeset in webkit [218828] by
-
- 3 edits1 add in trunk
RegExpPrototype.js builtin uses for-of iteration which is almost certainly incorrect
https://bugs.webkit.org/show_bug.cgi?id=173740
Reviewed by Mark Lam.
JSTests:
- stress/regexp-prototype-replace-builtin-should-not-use-for-of.js: Added.
(Array.prototype.Symbol.iterator):
Source/JavaScriptCore:
The builtin was using for-of iteration to iterate over an internal
list in its algorithm. For-of iteration is observable via user code
in the global object, so this approach was wrong as it would break if
a user changed the Array iteration protocol in some way.
- builtins/RegExpPrototype.js:
(replace):
- 8:33 PM Changeset in webkit [218827] by
-
- 10 edits in trunk/Source
WebsiteDataStore::fetchDataForTopPrivatelyControlledDomains() is inefficient
https://bugs.webkit.org/show_bug.cgi?id=173850
Reviewed by Ryosuke Niwa.
Source/WebCore:
- loader/ResourceLoadStatisticsStore.cpp:
(WebCore::ResourceLoadStatisticsStore::updateStatisticsForRemovedDataRecords):
- loader/ResourceLoadStatisticsStore.h:
Source/WebKit2:
Update WebsiteDataRecord::matchesTopPrivatelyControlledDomain() to rely on
SecurityOriginData::host rather than SecurityOriginData::securityOrigin()->host().
SecurityOriginData::securityOrigin() is expensive and it seems unnecessary to call
it here since we already have the host.
Also update WebsiteDataStore::fetchDataForTopPrivatelyControlledDomains() to return
domains as a HashSet rather than a Vector to avoid having duplicate domains.
- UIProcess/WebProcessProxy.cpp:
(WebKit::WebProcessProxy::deleteWebsiteDataForTopPrivatelyControlledDomainsInAllPersistentDataStores):
- UIProcess/WebProcessProxy.h:
- UIProcess/WebResourceLoadStatisticsStore.cpp:
(WebKit::WebResourceLoadStatisticsStore::removeDataRecords):
- UIProcess/WebsiteData/WebsiteDataRecord.cpp:
(WebKit::WebsiteDataRecord::matchesTopPrivatelyControlledDomain):
- UIProcess/WebsiteData/WebsiteDataStore.cpp:
(WebKit::WebsiteDataStore::fetchDataForTopPrivatelyControlledDomains):
(WebKit::WebsiteDataStore::removeDataForTopPrivatelyControlledDomains):
- UIProcess/WebsiteData/WebsiteDataStore.h:
- 8:28 PM Changeset in webkit [218826] by
-
- 9 edits2 adds in trunk
REGRESSION (AsyncImageDecoding): A tab with the WWDC keynote paused is killed for using excessive power (Image thrashing)
https://bugs.webkit.org/show_bug.cgi?id=173804
<rdar://problem/32623745>
Reviewed by Simon Fraser.
Source/WebCore:
When under memory pressure MemoryCache::singleton().pruneLiveResources(true) is called inFrameView::didPaintContents()
after top level paint. We end up decoding and pruning bitmaps repeatedly for each tile, which is not great.
Situation gets worse with async decoding. Painting now doesn’t actually decode the image, it just starts the decoding.
When it completes we trigger another paint to get the bits to the tiles. The paint for the first tile then calls
pruneLiveResources and loses the bitmap and the second tile triggers another round of async decoding. We have code
that prevents pruning of visible images but non-visible images in tiling area can hit this bug easily.
Test: fast/images/low-memory-decode.html
- page/FrameView.cpp:
(WebCore::FrameView::willPaintContents):
(WebCore::FrameView::didPaintContents):
Eliminate synchronous pruning during painting. This is an obsolete mechanism from early iOS times.
- platform/graphics/BitmapImage.cpp:
(WebCore::BitmapImage::imageFrameAvailableAtIndex):
(WebCore::BitmapImage::decodeCountForTesting):
Testing support.
- platform/graphics/BitmapImage.h:
- testing/Internals.cpp:
(WebCore::Internals::imageDecodeCount):
- testing/Internals.h:
- testing/Internals.idl:
LayoutTests:
- fast/images/low-memory-decode-expected.txt: Added.
- fast/images/low-memory-decode.html: Added.
- 5:26 PM Changeset in webkit [218825] by
-
- 2 edits in trunk/LayoutTests
Marked media/media-source/media-source-paint-to-canvas.html as flaky.
https://bugs.webkit.org/show_bug.cgi?id=173565
Unreviewed test gardening.
- platform/mac/TestExpectations:
- 4:47 PM Changeset in webkit [218824] by
-
- 2 edits in trunk/LayoutTests
Marked animations/play-state-suspend.html as flaky.
https://bugs.webkit.org/show_bug.cgi?id=173726
Unreviewed test gardening.
- platform/ios-wk2/TestExpectations:
- 4:39 PM Changeset in webkit [218823] by
-
- 2 edits in trunk/JSTests
Skip a test on 32-bit platforms since we run out of address space.
Rubber stamped by Mark Lam.
- stress/dont-reserve-huge-capacity-lexer.js:
- 3:36 PM Changeset in webkit [218822] by
-
- 4 edits in trunk/Source/WebKit2
Invalidate WebVideoFullscreenManager when WebPage is destroyed.
https://bugs.webkit.org/show_bug.cgi?id=173835
rdar://problem/32969161
Patch by Jeremy Jones <jeremyj@apple.com> on 2017-06-26
Reviewed by Jer Noble.
WebVideoFullscreenManager has a pointer to WebPage, and even null checks it in a few places,
but the only place it is nulled out is in the destructor. This allows a dangling reference.
This changes invalidates that reference when WebPage is destructed and adds nullchecks
or asserts throughout WebVideoFullscreenManager as appropriate.
- WebProcess/WebPage/WebPage.cpp:
(WebKit::WebPage::~WebPage):
- WebProcess/cocoa/WebVideoFullscreenManager.h:
(WebKit::WebVideoFullscreenManager::invalidate):
- WebProcess/cocoa/WebVideoFullscreenManager.mm:
(WebKit::WebVideoFullscreenManager::~WebVideoFullscreenManager):
(WebKit::WebVideoFullscreenManager::enterVideoFullscreenForVideoElement):
(WebKit::WebVideoFullscreenManager::exitVideoFullscreenForVideoElement):
(WebKit::WebVideoFullscreenManager::exitVideoFullscreenToModeWithoutAnimation):
(WebKit::WebVideoFullscreenManager::hasVideoChanged):
(WebKit::WebVideoFullscreenManager::videoDimensionsChanged):
(WebKit::WebVideoFullscreenManager::didSetupFullscreen):
(WebKit::WebVideoFullscreenManager::didEnterFullscreen):
(WebKit::WebVideoFullscreenManager::didCleanupFullscreen):
(WebKit::WebVideoFullscreenManager::fullscreenMayReturnToInline):
- 2:09 PM Changeset in webkit [218821] by
-
- 2 edits in trunk/Source/WebKit2
Disable diagnostic logging in ephemeral sessions
https://bugs.webkit.org/show_bug.cgi?id=173849
<rdar://problem/32987341>
Reviewed by Geoffrey Garen.
- UIProcess/WebPageProxy.cpp:
(WebKit::WebPageProxy::setDiagnosticLoggingClient):
Construct a dummy logging client if the page's session is ephemeral.
(WebKit::WebPageProxy::logDiagnosticMessage):
(WebKit::WebPageProxy::logDiagnosticMessageWithResult):
(WebKit::WebPageProxy::logDiagnosticMessageWithValue):
(WebKit::WebPageProxy::logDiagnosticMessageWithEnhancedPrivacy):
Slight cleanup to make the code look a bit safer.
- 1:17 PM Changeset in webkit [218820] by
-
- 2 edits in trunk/Source/JavaScriptCore
Renamed DumpRegisterFunctor to DumpReturnVirtualPCFunctor.
https://bugs.webkit.org/show_bug.cgi?id=173848
Reviewed by JF Bastien.
This functor only dumps the return VirtualPC.
- interpreter/Interpreter.cpp:
(JSC::DumpReturnVirtualPCFunctor::DumpReturnVirtualPCFunctor):
(JSC::Interpreter::dumpRegisters):
(JSC::DumpRegisterFunctor::DumpRegisterFunctor): Deleted.
(JSC::DumpRegisterFunctor::operator()): Deleted.
- 12:34 PM Changeset in webkit [218819] by
-
- 3 edits1 add in trunk
Crash in JSC::Lexer<unsigned char>::setCode
https://bugs.webkit.org/show_bug.cgi?id=172754
Reviewed by Mark Lam.
JSTests:
- stress/dont-reserve-huge-capacity-lexer.js: Added.
(catch):
Source/JavaScriptCore:
The lexer was asking one of its buffers to reserve initial space that
was O(text size in bytes). For large sources, this would end up causing
the vector to overflow and crash. This patch changes this code be like
the Lexer's other buffers and to only reserve a small starting buffer.
- parser/Lexer.cpp:
(JSC::Lexer<T>::setCode):
- 12:13 PM Changeset in webkit [218818] by
-
- 2 edits in trunk/Source/WebCore
ImageFrameCache::startAsyncDecodingQueue() unsafely passes Strings across threads
https://bugs.webkit.org/show_bug.cgi?id=173842
Reviewed by Simon Fraser.
The URL string was passed across thread without isolated copy.
- platform/graphics/ImageFrameCache.cpp:
(WebCore::ImageFrameCache::startAsyncDecodingQueue):
- 11:51 AM Changeset in webkit [218817] by
-
- 8 edits2 deletes in trunk
Unreviewed, rolling out r218783.
Causing accessibility/mac/setting-attributes-is-
asynchronous.html to crash consistently on mac-wk2 Debug
Reverted changeset:
"AX: Cannot call setValue() on contenteditable or ARIA text
controls"
https://bugs.webkit.org/show_bug.cgi?id=173520
http://trac.webkit.org/changeset/218783
- 11:19 AM Changeset in webkit [218816] by
-
- 39 edits in trunk
[WTF] Drop Thread::create(obsolete things) API since we can use lambda
https://bugs.webkit.org/show_bug.cgi?id=173825
Reviewed by Saam Barati.
Source/JavaScriptCore:
- jsc.cpp:
(startTimeoutThreadIfNeeded):
(timeoutThreadMain): Deleted.
Source/WebCore:
No behavior change.
- Modules/indexeddb/server/IDBServer.cpp:
(WebCore::IDBServer::IDBServer::IDBServer):
(WebCore::IDBServer::IDBServer::databaseThreadEntry): Deleted.
- Modules/indexeddb/server/IDBServer.h:
- Modules/webaudio/AsyncAudioDecoder.cpp:
(WebCore::AsyncAudioDecoder::AsyncAudioDecoder):
(WebCore::AsyncAudioDecoder::threadEntry): Deleted.
- Modules/webaudio/AsyncAudioDecoder.h:
- Modules/webaudio/OfflineAudioDestinationNode.cpp:
(WebCore::OfflineAudioDestinationNode::startRendering):
(WebCore::OfflineAudioDestinationNode::offlineRenderEntry): Deleted.
- Modules/webaudio/OfflineAudioDestinationNode.h:
- Modules/webdatabase/DatabaseThread.cpp:
(WebCore::DatabaseThread::start):
(WebCore::DatabaseThread::databaseThreadStart): Deleted.
- Modules/webdatabase/DatabaseThread.h:
- bindings/js/GCController.cpp:
(WebCore::collect):
(WebCore::GCController::gcTimerFired):
(WebCore::GCController::garbageCollectOnAlternateThreadForDebugging):
- loader/icon/IconDatabase.cpp:
(WebCore::IconDatabase::open):
(WebCore::IconDatabase::iconDatabaseSyncThreadStart): Deleted.
- loader/icon/IconDatabase.h:
- page/ResourceUsageThread.cpp:
(WebCore::ResourceUsageThread::createThreadIfNeeded):
(WebCore::ResourceUsageThread::threadCallback): Deleted.
- page/ResourceUsageThread.h:
- page/scrolling/ScrollingThread.cpp:
(WebCore::ScrollingThread::createThreadIfNeeded):
(WebCore::ScrollingThread::threadCallback): Deleted.
(WebCore::ScrollingThread::threadBody): Deleted.
- page/scrolling/ScrollingThread.h:
- platform/audio/HRTFDatabaseLoader.cpp:
(WebCore::HRTFDatabaseLoader::loadAsynchronously):
(WebCore::databaseLoaderEntry): Deleted.
- platform/audio/HRTFDatabaseLoader.h:
- platform/audio/ReverbConvolver.cpp:
(WebCore::ReverbConvolver::ReverbConvolver):
(WebCore::backgroundThreadEntry): Deleted.
- platform/audio/ReverbConvolver.h:
(WebCore::ReverbConvolver::useBackgroundThreads):
- platform/network/cf/LoaderRunLoopCF.cpp:
(WebCore::loaderRunLoop):
(WebCore::runLoaderThread): Deleted.
- platform/network/curl/CurlManager.cpp:
(WebCore::CurlManager::startThreadIfNeeded):
(WebCore::CurlManager::workerThread):
- platform/network/curl/CurlManager.h:
- workers/WorkerThread.cpp:
(WebCore::WorkerThread::start):
(WebCore::WorkerThread::workerThreadStart): Deleted.
- workers/WorkerThread.h:
Source/WebKit:
- Storage/StorageThread.cpp:
(WebCore::StorageThread::start):
(WebCore::StorageThread::threadEntryPointCallback): Deleted.
- Storage/StorageThread.h:
Source/WTF:
Thread::create(ThreadFunction, void* data, const char* name) is a bit old API.
Since we have C++ lambda, the above API is simply unnecessary. And C++ lambda
based one is better since the above API needs casting data to and from void*.
- wtf/Function.h:
Avoid ambiguity.
- wtf/ParallelJobsGeneric.cpp:
(WTF::ParallelEnvironment::ThreadPrivate::tryLockFor):
(WTF::ParallelEnvironment::ThreadPrivate::workerThread): Deleted.
- wtf/ParallelJobsGeneric.h:
- wtf/Threading.cpp:
- wtf/ThreadingWin.cpp:
(WTF::createThread):
Tools:
- DumpRenderTree/JavaScriptThreading.cpp:
(runJavaScriptThread):
(startJavaScriptThreads):
- DumpRenderTree/mac/DumpRenderTree.mm:
(runThread):
(runPthread):
(testThreadIdentifierMap):
- 11:02 AM Changeset in webkit [218815] by
-
- 3 edits in trunk/Tools
[TestWebKitAPI] Fix false-positive bad release warnings found by clang static analyzer
<https://webkit.org/b/173837>
Reviewed by Geoffrey Garen.
The clang static analyzer can't reason about objects that are
assigned to a delegate as +1 retained, and then released
later in a different method when the delegate is cleared.
Instead, use a common idiom where the object setting the
delegate retains it as an instance variable on initialization.
Then the same object clears the delegate and releases its
instance variable during teardown.
Also add EXPECT_TRUE() tests to make sure the delegate objects
match before teardown.
- TestWebKitAPI/Tests/mac/FullscreenZoomInitialFrame.mm:
(TestWebKitAPI::FullscreenZoomInitialFrame::initializeView):
(TestWebKitAPI::FullscreenZoomInitialFrame::teardownView):
- TestWebKitAPI/Tests/mac/PageVisibilityStateWithWindowChanges.mm:
(TestWebKitAPI::PageVisibilityStateWithWindowChanges::initializeView):
(TestWebKitAPI::PageVisibilityStateWithWindowChanges::teardownView):
- 11:01 AM Changeset in webkit [218814] by
-
- 2 edits in trunk/Source/WebKit2
[GCrypt] Properly initialize libgcrypt before using it
https://bugs.webkit.org/show_bug.cgi?id=173589
Reviewed by Michael Catanzaro.
Initialize libgcrypt in the Unix-specific main() entrypoint. This is early enough
to ensure no other libgcrypt API function is invoked and to also ensure this is
done in a thread-safe manner.
Initialization is initiated through the gcry_check_version() call. 16 kilobytes of
secure memory is pre-allocated before we mark the initialization as complete, as
recommended by the libgcrypt documentation.
- WebProcess/EntryPoint/unix/WebProcessMain.cpp:
(main):
- 11:00 AM Changeset in webkit [218813] by
-
- 2 edits in trunk/Source/WebKit2
When Mission Control closes fullscreen window, allow media element to update its state.
https://bugs.webkit.org/show_bug.cgi?id=173671
rdar://problem/32892671
Patch by Jeremy Jones <jeremyj@apple.com> on 2017-06-26
Reviewed by Eric Carlson.
When Mission Control closes element fullscreen without an animation, we need to notify the
media element that fullscreen has exited.
Without this change, the page still behaves like the element is in fullscreen.
- UIProcess/mac/WKFullScreenWindowController.mm:
(-[WKFullScreenWindowController beganExitFullScreenWithInitialFrame:finalFrame:]):
(-[WKFullScreenWindowController finishedExitFullScreenAnimation:]):
- 10:54 AM Changeset in webkit [218812] by
-
- 2 edits in trunk/Source/WebKit2
[iOS DnD] [WK2] Beginning a drag session should prevent clicking via long press
https://bugs.webkit.org/show_bug.cgi?id=173838
Reviewed by Tim Horton.
After r218579, WebKit no longer attempts to replace out-of-the-box UIKit long-press/drag lift disambiguation by
firing events immediately. However, this means that dragging will defer the highlight long press gesture
recognizer until dragging ends rather than cancel it immediately, which causes WKContentView to dispatch a
synthetic click immediately upon lift. To fix this, we should bail out of the highlight long press gesture when
beginning a drag.
- UIProcess/ios/WKContentViewInteraction.mm:
(-[WKContentView _dragInteraction:prepareForSession:completion:]):
- 10:52 AM Changeset in webkit [218811] by
-
- 2 edits in trunk/LayoutTests
Make webrtc/video-replace-track-to-null.html more robust
https://bugs.webkit.org/show_bug.cgi?id=173834
Patch by Youenn Fablet <youenn@apple.com> on 2017-06-26
Reviewed by Eric Carlson.
- webrtc/video-replace-track-to-null.html:
- 10:48 AM Changeset in webkit [218810] by
-
- 2 edits in trunk/Tools
[TestWebKitAPI] REGRESSION (r218750): Fix leak of NSURLResponse in WKURLSchemeHandler-1.mm
<https://webkit.org/b/173836>
Reviewed by Brady Eidson.
- TestWebKitAPI/Tests/WebKit2Cocoa/WKURLSchemeHandler-1.mm:
(-[RedirectSchemeHandler webView:startURLSchemeTask:]): Use
adoptNS() to fix the leak. Also use auto/adoptNS() for other
objects in this test.
- 9:45 AM Changeset in webkit [218809] by
-
- 14 edits in trunk
[ATK] Add support for aria-details and aria-errormessage
https://bugs.webkit.org/show_bug.cgi?id=172588
Reviewed by Chris Fleizach.
Source/WebCore:
Add methods to retrieve elements referenced by or referencing the new
aria-details and aria-errormessage attributes. Include aria-details and
aria-errormessage in AccessibilityObject::supportsARIAAttributes() to ensure
elements with these attributes will be included in the accessibility tree.
New test cases added to accessibility/gtk/relation-types.html.
- accessibility/AccessibilityObject.cpp:
(WebCore::AccessibilityObject::supportsARIAAttributes):
(WebCore::AccessibilityObject::ariaDetailsElements):
(WebCore::AccessibilityObject::ariaDetailsReferencingElements):
(WebCore::AccessibilityObject::ariaErrorMessageElements):
(WebCore::AccessibilityObject::ariaErrorMessageReferencingElements):
- accessibility/AccessibilityObject.h:
- accessibility/atk/WebKitAccessibleWrapperAtk.cpp:
(setAtkRelationSetFromCoreObject):
- html/HTMLAttributeNames.in:
Tools:
Add additional methods to test ARIA properties which are exposed via
AtkRelation. Also bump minimum versions of at-spi2-core and at-spi2-atk
to versions which support the details and error-message relation types.
- WebKitTestRunner/InjectedBundle/AccessibilityUIElement.h:
(WTR::AccessibilityUIElement::ariaDetailsElementAtIndex):
(WTR::AccessibilityUIElement::ariaDetailsReferencingElementAtIndex):
(WTR::AccessibilityUIElement::ariaErrorMessageElementAtIndex):
(WTR::AccessibilityUIElement::ariaErrorMessageReferencingElementAtIndex):
- WebKitTestRunner/InjectedBundle/Bindings/AccessibilityUIElement.idl:
- WebKitTestRunner/InjectedBundle/atk/AccessibilityUIElementAtk.cpp:
(WTR::AccessibilityUIElement::ariaDetailsElementAtIndex):
(WTR::AccessibilityUIElement::ariaDetailsReferencingElementAtIndex):
(WTR::AccessibilityUIElement::ariaErrorMessageElementAtIndex):
(WTR::AccessibilityUIElement::ariaErrorMessageReferencingElementAtIndex):
- gtk/jhbuild.modules:
LayoutTests:
Add new test cases to relation-types.html.
- accessibility/gtk/relation-types-expected.txt:
- accessibility/gtk/relation-types.html:
- 7:12 AM Changeset in webkit [218808] by
-
- 1 edit5 adds in trunk/LayoutTests
[WPE][GCrypt] Unreviewed gardening
After r218744, land new expectations for AES-CFB tests.
- platform/wpe/crypto/subtle/aes-cbc-cfb-decrypt-malformed-parameters-expected.txt: Added.
- platform/wpe/crypto/subtle/aes-cbc-cfb-encrypt-malformed-parameters-expected.txt: Added.
- platform/wpe/crypto/subtle/aes-export-key-malformed-parameters-expected.txt: Added.
- platform/wpe/crypto/subtle/aes-generate-key-malformed-parameters-expected.txt: Added.
- platform/wpe/crypto/subtle/aes-import-key-malformed-parameters-expected.txt: Added.
- 2:30 AM Changeset in webkit [218807] by
-
- 2 edits in trunk/Tools
Unreviewed. Add libfii to GTK+ install dependencies script.
- gtk/install-dependencies:
- 2:02 AM Changeset in webkit [218806] by
-
- 2 edits in trunk/LayoutTests
[Win] Update expectations for layout tests.
https://bugs.webkit.org/show_bug.cgi?id=172437
Unreviewed test gardening, update test expectations for failing tests.
- platform/win/TestExpectations:
- 1:47 AM Changeset in webkit [218805] by
-
- 2 edits in trunk/Source/JavaScriptCore
Unreviewed, add missing header for CLoop
- runtime/SymbolTable.cpp:
- 1:23 AM Changeset in webkit [218804] by
-
- 4 edits in trunk/Source
Unreviewed, add missing header icncludes
Source/JavaScriptCore:
- parser/Lexer.h:
Source/WebKit2:
- WebProcess/soup/WebKitSoupRequestInputStream.cpp:
- 1:13 AM Changeset in webkit [218803] by
-
- 50 edits in trunk/Source
Remove excessive headers from WebCore/{Modules,animation,crypto,domjit}
https://bugs.webkit.org/show_bug.cgi?id=173824
Reviewed by Darin Adler.
No new tests needed.
- Modules/credentials/CredentialCreationOptions.h:
- Modules/credentials/PasswordCredential.h:
- Modules/fetch/FetchBody.h:
- Modules/fetch/FetchBodyOwner.h:
- Modules/gamepad/GamepadManager.h:
- Modules/geolocation/Coordinates.h:
- Modules/geolocation/Geoposition.h:
- Modules/indexeddb/IDBActiveDOMObject.h:
- Modules/indexeddb/IDBCursor.h:
- Modules/indexeddb/IDBDatabase.h:
- Modules/indexeddb/IDBDatabaseIdentifier.h:
- Modules/indexeddb/IDBObjectStore.cpp:
- Modules/indexeddb/IDBObjectStore.h:
- Modules/indexeddb/server/UniqueIDBDatabase.cpp:
- Modules/indexeddb/server/UniqueIDBDatabase.h:
- Modules/indexeddb/shared/InProcessIDBServer.h:
- Modules/indexeddb/shared/IndexKey.h:
- Modules/mediacontrols/MediaControlsHost.h:
- Modules/mediasession/WebMediaSessionManager.h:
- Modules/mediasource/SourceBufferList.h:
- Modules/mediasource/VideoPlaybackQuality.h:
- Modules/notifications/Notification.h:
- Modules/quota/WorkerNavigatorStorageQuota.h:
- Modules/webaudio/AudioBasicProcessorNode.h:
- Modules/webaudio/AudioContext.h:
- Modules/webaudio/AudioDestinationNode.h:
- Modules/webaudio/AudioParamTimeline.h:
- Modules/webaudio/ConvolverNode.cpp:
- Modules/webaudio/MediaStreamAudioSource.h:
- Modules/webaudio/MediaStreamAudioSourceNode.h:
- Modules/webaudio/PannerNode.h:
- Modules/webaudio/PeriodicWave.h:
- Modules/webaudio/ScriptProcessorNode.h:
- Modules/webdatabase/DatabaseManager.h:
- Modules/webdatabase/DatabaseTask.h:
- Modules/webdatabase/SQLTransaction.h:
- Modules/webdatabase/SQLTransactionBackend.h:
- Modules/webdatabase/SQLTransactionStateMachine.h:
- Modules/websockets/ThreadableWebSocketChannelClientWrapper.h:
- Modules/websockets/WebSocketDeflater.h:
- Modules/websockets/WorkerThreadableWebSocketChannel.h:
- animation/AnimationEffect.h:
- crypto/CryptoKeyPair.h:
- crypto/parameters/CryptoAlgorithmEcdhKeyDeriveParams.h:
- dom/ScriptExecutionContext.h:
- domjit/DOMJITHelpers.h:
- domjit/DOMJITIDLConvert.h:
- domjit/DOMJITIDLType.h:
- 12:57 AM Changeset in webkit [218802] by
-
- 4 edits in trunk/Source
[GTK] Unreviewed, added missing includes to fix debug build
Source/WebCore:
- platform/graphics/gstreamer/MediaPlayerPrivateGStreamerBase.cpp:
Source/WebKit2:
- NetworkProcess/NetworkSession.cpp:
Jun 25, 2017:
- 11:51 PM Changeset in webkit [218801] by
-
- 3 edits1 add in releases/WebKitGTK/webkit-2.16
Merge r215387 - ParseInt intrinsic in DFG backend doesn't properly flush its operands
https://bugs.webkit.org/show_bug.cgi?id=170865
Reviewed by Mark Lam and Geoffrey Garen.
JSTests:
- stress/parse-int-intrinsic-dfg-backend-flush.js: Added.
(assert):
(foo):
Source/JavaScriptCore:
The DFG backend code needed to first call .gpr()/.jsValueRegs()
before calling flushRegisters(), or the input JSValueOperand would
not be flushed.
- dfg/DFGSpeculativeJIT.cpp:
(JSC::DFG::SpeculativeJIT::compileParseInt):
- 2:41 PM Changeset in webkit [218800] by
-
- 2 edits in trunk/Source/WTF
initializeThreading() [first] causes WTFCrash due to maxSingleAllocationSize not being initialized
https://bugs.webkit.org/show_bug.cgi?id=173720
Reviewed by Mark Lam.
When using std::numeric_limits<size_t>::max() for global variable's initialization,
it seems that it invokes static constructor to initialize this in VC++.
We avoid this edge case by introducing a workaround using SIZE_MAX here.
When I perform git-grep, there is only one site (this) using std::numeric_limits<>::xxx()
to initialize global variable.
- wtf/FastMalloc.cpp:
- 1:35 PM Changeset in webkit [218799] by
-
- 90 edits in trunk
Remove excessive headers from WebCore/platform
https://bugs.webkit.org/show_bug.cgi?id=173822
Reviewed by Tim Horton.
Source/WebCore:
No new tests needed.
- html/ColorInputType.h:
- html/MediaElementSession.h:
- page/SecurityOriginData.cpp:
- platform/CalculationValue.cpp:
- platform/CalculationValue.h:
- platform/ColorChooserClient.h:
- platform/ContentType.h:
- platform/LogMacros.h:
- platform/PODRedBlackTree.h:
- platform/PasteboardStrategy.h:
- platform/PlatformPasteboard.h:
- platform/PopupMenuStyle.h:
- platform/ScrollAnimator.h:
- platform/Theme.h:
- platform/ThreadGlobalData.h:
- platform/animation/AnimationList.h:
- platform/audio/AudioArray.h:
- platform/audio/AudioChannel.h:
- platform/audio/DynamicsCompressor.h:
- platform/audio/DynamicsCompressorKernel.h:
- platform/audio/FFTFrame.h:
- platform/audio/HRTFDatabaseLoader.cpp:
- platform/audio/HRTFDatabaseLoader.h:
- platform/audio/HRTFElevation.h:
- platform/audio/MultiChannelResampler.h:
- platform/audio/PlatformMediaSession.h:
- platform/audio/ReverbConvolver.h:
- platform/audio/ReverbConvolverStage.cpp:
- platform/graphics/ANGLEWebKitBridge.h:
- platform/graphics/BitmapImage.h:
- platform/graphics/CrossfadeGeneratedImage.h:
- platform/graphics/FloatPolygon.h:
- platform/graphics/FloatSizeHash.h:
- platform/graphics/Font.h:
- platform/graphics/FontMetrics.h:
- platform/graphics/FontPlatformData.h:
- platform/graphics/FontRanges.h:
- platform/graphics/FontTaggedSettings.h:
- platform/graphics/GraphicsContext3D.h:
- platform/graphics/GraphicsContext3DPrivate.h:
- platform/graphics/GraphicsLayerClient.h:
- platform/graphics/Image.cpp:
- platform/graphics/ImageFrame.h:
- platform/graphics/ImageFrameCache.h:
- platform/graphics/ImageSource.h:
- platform/graphics/IntRectHash.h:
- platform/graphics/IntSizeHash.h:
- platform/graphics/MediaPlaybackTargetClient.h:
- platform/graphics/Pattern.h:
- platform/graphics/PlatformTimeRanges.h:
- platform/graphics/TextRun.h:
- platform/graphics/TiledBacking.h:
- platform/graphics/cairo/FontCustomPlatformData.h:
- platform/graphics/filters/FEConvolveMatrix.h:
- platform/graphics/filters/FELighting.h:
- platform/graphics/filters/SourceAlpha.h:
- platform/graphics/gstreamer/GStreamerUtilities.h:
- platform/graphics/gstreamer/InbandMetadataTextTrackPrivateGStreamer.h:
- platform/graphics/gstreamer/MediaPlayerPrivateGStreamer.h:
- platform/graphics/gstreamer/MediaPlayerPrivateGStreamerBase.h:
- platform/graphics/gstreamer/TrackPrivateBaseGStreamer.h:
- platform/graphics/harfbuzz/HarfBuzzShaper.h:
- platform/graphics/opengl/Extensions3DOpenGL.h:
- platform/graphics/texmap/GraphicsLayerTextureMapper.h:
- platform/graphics/texmap/TextureMapperBackingStore.h:
- platform/graphics/texmap/TextureMapperGL.cpp:
- platform/graphics/texmap/TextureMapperGL.h:
- platform/graphics/texmap/TextureMapperPlatformLayerProxy.h:
- platform/graphics/texmap/TextureMapperTile.h:
- platform/graphics/texmap/coordinated/Tile.h:
- platform/graphics/texmap/coordinated/TiledBackingStore.h:
- platform/graphics/transforms/TransformState.h:
- platform/gtk/PasteboardHelper.h:
- platform/gtk/ScrollbarThemeGtk.h:
- platform/image-decoders/ImageDecoder.h:
- platform/network/BlobData.h:
- platform/network/BlobDataFileReference.h:
- platform/network/ResourceHandle.h:
- platform/network/ResourceHandleInternal.h:
- platform/network/SocketStreamHandleClient.h:
- platform/network/soup/ResourceRequest.h:
- platform/network/soup/SocketStreamHandleImpl.h:
- platform/network/soup/SoupNetworkSession.h:
- platform/sql/SQLiteFileSystem.h:
- platform/text/LocaleICU.h:
Source/WebKit2:
- WebProcess/WebCoreSupport/SessionStateConversion.cpp:
- 12:56 PM Changeset in webkit [218798] by
-
- 5 edits in trunk
Unreviewed, rolling out r215190.
Broke product select element on GNOME Bugzilla
Reverted changeset:
"[GTK] Misplaced right click menu on web page due to
deprecated gtk_menu_popup()"
https://bugs.webkit.org/show_bug.cgi?id=170553
http://trac.webkit.org/changeset/215190
- 11:20 AM Changeset in webkit [218797] by
-
- 2 edits in trunk/Source/WebKit2
[GTK][WPE] Unreviewed, fix typo in translatable string
Thanks to Yuri Chornoivan for reporting this typo.
- UIProcess/API/glib/WebKitWebView.cpp:
(webkit_web_view_class_init):
- 11:18 AM Changeset in webkit [218796] by
-
- 2 edits in trunk/Source/WebCore/platform/gtk/po
[GTK] [l10n] Updated Ukrainian translation of WebKitGTK+
https://bugs.webkit.org/show_bug.cgi?id=173823
Patch by Yuri Chornoivan <yurchor@ukr.net> on 2017-06-25
Rubber-stamped by Michael Catanzaro.
- uk.po:
- 11:06 AM Changeset in webkit [218795] by
-
- 15 edits2 adds in trunk
Remove use of mock webrtc backend factory at injected bundle reset time
https://bugs.webkit.org/show_bug.cgi?id=173817
Patch by Youenn Fablet <youenn@apple.com> on 2017-06-25
Reviewed by Darin Adler.
Source/WebCore:
- testing/Internals.cpp:
(WebCore::Internals::resetToConsistentState): Resetting the peer connection backend.
- testing/MockLibWebRTCPeerConnection.cpp: Doing some clean-up
(WebCore::useRealRTCPeerConnectionFactory):
(WebCore::MockLibWebRTCPeerConnectionFactory::MockLibWebRTCPeerConnectionFactory):
(WebCore::MockLibWebRTCPeerConnectionFactory::CreatePeerConnection):
(WebCore::MockLibWebRTCPeerConnectionFactory::CreateVideoTrack):
(WebCore::MockLibWebRTCPeerConnectionFactory::CreateAudioTrack):
- testing/MockLibWebRTCPeerConnection.h:
LayoutTests:
Making some tests less flaky.
- TestExpectations:
- fast/mediastream/RTCPeerConnection-media-setup-two-dialogs-expected.txt:
- platform/mac/webrtc/captureCanvas-webrtc-software-encoder.html:
- webrtc/audio-peer-connection-webaudio.html:
- webrtc/captureCanvas-webrtc.html:
- webrtc/datachannel/bufferedAmountLowThreshold-default-expected.txt: Added.
- webrtc/datachannel/bufferedAmountLowThreshold-default.html: Added.
- webrtc/datachannel/bufferedAmountLowThreshold-expected.txt:
- webrtc/datachannel/bufferedAmountLowThreshold.html:
- webrtc/routines.js:
- webrtc/video-replace-muted-track.html:
- webrtc/video-unmute.html:
- 10:40 AM Changeset in webkit [218794] by
-
- 305 edits in trunk/Source
Remove excessive headers from JavaScriptCore
https://bugs.webkit.org/show_bug.cgi?id=173812
Reviewed by Darin Adler.
Source/JavaScriptCore:
- API/APIUtils.h:
- assembler/LinkBuffer.cpp:
- assembler/MacroAssemblerCodeRef.cpp:
- b3/air/AirLiveness.h:
- b3/air/AirLowerAfterRegAlloc.cpp:
- bindings/ScriptValue.cpp:
- bindings/ScriptValue.h:
- bytecode/AccessCase.cpp:
- bytecode/AccessCase.h:
- bytecode/ArrayProfile.h:
- bytecode/BytecodeDumper.h:
- bytecode/BytecodeIntrinsicRegistry.cpp:
- bytecode/BytecodeKills.h:
- bytecode/BytecodeLivenessAnalysis.h:
- bytecode/BytecodeUseDef.h:
- bytecode/CallLinkStatus.h:
- bytecode/CodeBlock.h:
- bytecode/CodeOrigin.h:
- bytecode/ComplexGetStatus.h:
- bytecode/GetByIdStatus.h:
- bytecode/GetByIdVariant.h:
- bytecode/InlineCallFrame.h:
- bytecode/InlineCallFrameSet.h:
- bytecode/Instruction.h:
- bytecode/InternalFunctionAllocationProfile.h:
- bytecode/JumpTable.h:
- bytecode/MethodOfGettingAValueProfile.h:
- bytecode/ObjectPropertyConditionSet.h:
- bytecode/Operands.h:
- bytecode/PolymorphicAccess.h:
- bytecode/PutByIdStatus.h:
- bytecode/SpeculatedType.cpp:
- bytecode/StructureSet.h:
- bytecode/StructureStubInfo.h:
- bytecode/UnlinkedCodeBlock.h:
- bytecode/UnlinkedFunctionExecutable.h:
- bytecode/ValueProfile.h:
- bytecompiler/BytecodeGenerator.cpp:
- bytecompiler/BytecodeGenerator.h:
- bytecompiler/Label.h:
- bytecompiler/StaticPropertyAnalysis.h:
- debugger/DebuggerCallFrame.cpp:
- dfg/DFGAbstractInterpreter.h:
- dfg/DFGAdjacencyList.h:
- dfg/DFGArgumentsUtilities.h:
- dfg/DFGArrayMode.h:
- dfg/DFGArrayifySlowPathGenerator.h:
- dfg/DFGBackwardsPropagationPhase.h:
- dfg/DFGBasicBlock.h:
- dfg/DFGCallArrayAllocatorSlowPathGenerator.h:
- dfg/DFGCallCreateDirectArgumentsSlowPathGenerator.h:
- dfg/DFGCapabilities.h:
- dfg/DFGCommon.h:
- dfg/DFGCommonData.h:
- dfg/DFGDesiredIdentifiers.h:
- dfg/DFGDesiredWatchpoints.h:
- dfg/DFGDisassembler.cpp:
- dfg/DFGDominators.h:
- dfg/DFGDriver.cpp:
- dfg/DFGDriver.h:
- dfg/DFGEdgeDominates.h:
- dfg/DFGFinalizer.h:
- dfg/DFGGenerationInfo.h:
- dfg/DFGJITCompiler.cpp:
- dfg/DFGJITCompiler.h:
- dfg/DFGJITFinalizer.h:
- dfg/DFGLivenessAnalysisPhase.h:
- dfg/DFGMinifiedNode.h:
- dfg/DFGMultiGetByOffsetData.h:
- dfg/DFGNaturalLoops.cpp:
- dfg/DFGNaturalLoops.h:
- dfg/DFGNode.h:
- dfg/DFGOSRAvailabilityAnalysisPhase.h:
- dfg/DFGOSRExit.h:
- dfg/DFGOSRExitCompilationInfo.h:
- dfg/DFGOSRExitCompiler.cpp:
- dfg/DFGOSRExitCompiler.h:
- dfg/DFGOSRExitJumpPlaceholder.h:
- dfg/DFGOperations.cpp:
- dfg/DFGOperations.h:
- dfg/DFGPlan.h:
- dfg/DFGPreciseLocalClobberize.h:
- dfg/DFGPromotedHeapLocation.h:
- dfg/DFGRegisteredStructure.h:
- dfg/DFGRegisteredStructureSet.h:
- dfg/DFGSaneStringGetByValSlowPathGenerator.h:
- dfg/DFGSlowPathGenerator.h:
- dfg/DFGSnippetParams.h:
- dfg/DFGSpeculativeJIT.h:
- dfg/DFGToFTLDeferredCompilationCallback.h:
- dfg/DFGToFTLForOSREntryDeferredCompilationCallback.h:
- dfg/DFGValidate.h:
- dfg/DFGValueSource.h:
- dfg/DFGVariableEvent.h:
- dfg/DFGVariableEventStream.h:
- dfg/DFGWorklist.h:
- domjit/DOMJITCallDOMGetterSnippet.h:
- domjit/DOMJITEffect.h:
- ftl/FTLLink.cpp:
- ftl/FTLLowerDFGToB3.cpp:
- ftl/FTLPatchpointExceptionHandle.h:
- heap/AllocatorAttributes.h:
- heap/CodeBlockSet.h:
- heap/DeferGC.h:
- heap/GCSegmentedArray.h:
- heap/Heap.cpp:
- heap/Heap.h:
- heap/IncrementalSweeper.h:
- heap/ListableHandler.h:
- heap/MachineStackMarker.h:
- heap/MarkedAllocator.h:
- heap/MarkedBlock.cpp:
- heap/MarkedBlock.h:
- heap/MarkingConstraint.h:
- heap/SlotVisitor.cpp:
- heap/SlotVisitor.h:
- inspector/ConsoleMessage.cpp:
- inspector/ConsoleMessage.h:
- inspector/InjectedScript.h:
- inspector/InjectedScriptHost.h:
- inspector/InjectedScriptManager.cpp:
- inspector/JSGlobalObjectInspectorController.cpp:
- inspector/JavaScriptCallFrame.h:
- inspector/ScriptCallStack.h:
- inspector/ScriptCallStackFactory.cpp:
- inspector/ScriptDebugServer.h:
- inspector/agents/InspectorConsoleAgent.h:
- inspector/agents/InspectorDebuggerAgent.cpp:
- inspector/agents/InspectorDebuggerAgent.h:
- inspector/agents/InspectorHeapAgent.cpp:
- inspector/agents/InspectorHeapAgent.h:
- inspector/agents/InspectorRuntimeAgent.h:
- inspector/agents/InspectorScriptProfilerAgent.cpp:
- inspector/agents/InspectorScriptProfilerAgent.h:
- inspector/agents/JSGlobalObjectConsoleAgent.h:
- inspector/agents/JSGlobalObjectDebuggerAgent.cpp:
- inspector/agents/JSGlobalObjectDebuggerAgent.h:
- inspector/agents/JSGlobalObjectRuntimeAgent.h:
- inspector/augmentable/AlternateDispatchableAgent.h:
- interpreter/CLoopStack.h:
- interpreter/CachedCall.h:
- interpreter/CallFrame.h:
- interpreter/Interpreter.cpp:
- interpreter/Interpreter.h:
- jit/AssemblyHelpers.cpp:
- jit/AssemblyHelpers.h:
- jit/CCallHelpers.h:
- jit/CallFrameShuffler.h:
- jit/ExecutableAllocator.h:
- jit/GCAwareJITStubRoutine.h:
- jit/HostCallReturnValue.h:
- jit/ICStats.h:
- jit/JIT.cpp:
- jit/JIT.h:
- jit/JITAddGenerator.h:
- jit/JITCall32_64.cpp:
- jit/JITCode.h:
- jit/JITDisassembler.cpp:
- jit/JITExceptions.cpp:
- jit/JITMathIC.h:
- jit/JITOpcodes.cpp:
- jit/JITOperations.cpp:
- jit/JITOperations.h:
- jit/JITThunks.cpp:
- jit/JITThunks.h:
- jit/JSInterfaceJIT.h:
- jit/PCToCodeOriginMap.h:
- jit/PolymorphicCallStubRoutine.h:
- jit/RegisterSet.h:
- jit/Repatch.h:
- jit/SetupVarargsFrame.h:
- jit/Snippet.h:
- jit/SnippetParams.h:
- jit/ThunkGenerators.h:
- jsc.cpp:
- llint/LLIntCLoop.h:
- llint/LLIntEntrypoint.h:
- llint/LLIntExceptions.h:
- llint/LLIntOfflineAsmConfig.h:
- llint/LLIntSlowPaths.cpp:
- parser/NodeConstructors.h:
- parser/Nodes.cpp:
- parser/Nodes.h:
- parser/Parser.cpp:
- parser/Parser.h:
- parser/ParserTokens.h:
- parser/SourceProviderCacheItem.h:
- profiler/ProfilerBytecodeSequence.h:
- profiler/ProfilerDatabase.cpp:
- profiler/ProfilerDatabase.h:
- profiler/ProfilerOrigin.h:
- profiler/ProfilerOriginStack.h:
- profiler/ProfilerProfiledBytecodes.h:
- profiler/ProfilerUID.h:
- runtime/AbstractModuleRecord.h:
- runtime/ArrayConstructor.h:
- runtime/ArrayConventions.h:
- runtime/ArrayIteratorPrototype.h:
- runtime/ArrayPrototype.h:
- runtime/BasicBlockLocation.h:
- runtime/Butterfly.h:
- runtime/CallData.cpp:
- runtime/CodeCache.h:
- runtime/CommonSlowPaths.cpp:
- runtime/CommonSlowPaths.h:
- runtime/CommonSlowPathsExceptions.cpp:
- runtime/Completion.cpp:
- runtime/ControlFlowProfiler.h:
- runtime/DateInstanceCache.h:
- runtime/ErrorConstructor.h:
- runtime/ErrorInstance.h:
- runtime/ExceptionHelpers.cpp:
- runtime/ExceptionHelpers.h:
- runtime/ExecutableBase.h:
- runtime/FunctionExecutable.h:
- runtime/HasOwnPropertyCache.h:
- runtime/Identifier.h:
- runtime/InternalFunction.h:
- runtime/IntlCollator.cpp:
- runtime/IntlCollatorPrototype.h:
- runtime/IntlDateTimeFormatPrototype.h:
- runtime/IntlNumberFormat.cpp:
- runtime/IntlNumberFormatPrototype.h:
- runtime/IteratorOperations.cpp:
- runtime/JSArray.h:
- runtime/JSArrayBufferPrototype.h:
- runtime/JSCJSValue.h:
- runtime/JSCJSValueInlines.h:
- runtime/JSCell.h:
- runtime/JSFunction.cpp:
- runtime/JSFunction.h:
- runtime/JSGenericTypedArrayViewPrototypeFunctions.h:
- runtime/JSGlobalObject.cpp:
- runtime/JSGlobalObject.h:
- runtime/JSGlobalObjectDebuggable.cpp:
- runtime/JSGlobalObjectDebuggable.h:
- runtime/JSGlobalObjectFunctions.cpp:
- runtime/JSGlobalObjectFunctions.h:
- runtime/JSJob.cpp:
- runtime/JSLock.h:
- runtime/JSModuleLoader.cpp:
- runtime/JSModuleNamespaceObject.h:
- runtime/JSModuleRecord.h:
- runtime/JSObject.cpp:
- runtime/JSObject.h:
- runtime/JSRunLoopTimer.h:
- runtime/JSTemplateRegistryKey.h:
- runtime/JSTypedArrayPrototypes.cpp:
- runtime/JSTypedArrayPrototypes.h:
- runtime/JSTypedArrays.h:
- runtime/LiteralParser.h:
- runtime/MatchResult.h:
- runtime/MemoryStatistics.h:
- runtime/PrivateName.h:
- runtime/PromiseDeferredTimer.h:
- runtime/ProxyObject.h:
- runtime/RegExp.h:
- runtime/SamplingProfiler.cpp:
- runtime/SmallStrings.h:
- runtime/StringPrototype.cpp:
- runtime/StringRecursionChecker.h:
- runtime/Structure.h:
- runtime/SymbolConstructor.h:
- runtime/SymbolPrototype.cpp:
- runtime/SymbolPrototype.h:
- runtime/TypeProfiler.h:
- runtime/TypeProfilerLog.h:
- runtime/TypedArrayType.h:
- runtime/VM.cpp:
- runtime/VM.h:
- runtime/VMEntryScope.h:
- runtime/WeakMapData.h:
- runtime/WriteBarrier.h:
- tools/FunctionOverrides.cpp:
- tools/FunctionOverrides.h:
- wasm/WasmBinding.cpp:
- wasm/js/JSWebAssemblyCodeBlock.h:
- wasm/js/WebAssemblyPrototype.cpp:
- yarr/Yarr.h:
- yarr/YarrJIT.cpp:
- yarr/YarrJIT.h:
- yarr/YarrParser.h:
Source/WebCore:
No new tests needed.
- Modules/indexeddb/IDBObjectStore.cpp:
- bindings/js/JSDOMPromiseDeferred.h:
- bindings/js/JSLazyEventListener.cpp:
- bindings/js/JSMainThreadExecState.h:
- bindings/js/ReadableStreamDefaultController.cpp:
- bindings/js/ScriptGlobalObject.cpp:
- bindings/js/SerializedScriptValue.cpp:
- bridge/NP_jsobject.cpp:
- dom/ScriptExecutionContext.cpp:
- html/HTMLPlugInImageElement.cpp:
Source/WebKit/mac:
- Plugins/Hosted/NetscapePluginInstanceProxy.mm:
- WebView/WebView.mm:
Source/WebKit/win:
- Plugins/PluginPackage.cpp:
Source/WebKit2:
- WebProcess/Plugins/Netscape/NPJSObject.cpp:
Source/WTF:
- wtf/Bag.h:
- 3:01 AM Changeset in webkit [218793] by
-
- 96 edits in trunk/Source/WebCore
Remove excessive headers from WebCore/{rendering,style,svg}
https://bugs.webkit.org/show_bug.cgi?id=173773
Reviewed by Darin Adler.
No new tests needed.
- css/StyleBuilderConverter.h:
- editing/TextIterator.cpp:
- rendering/CounterNode.h:
- rendering/FlexibleBoxAlgorithm.h:
- rendering/HitTestLocation.h:
- rendering/HitTestResult.h:
- rendering/HitTestingTransformState.h:
- rendering/OrderIterator.h:
- rendering/RenderButton.h:
- rendering/RenderCombineText.h:
- rendering/RenderFlowThread.h:
- rendering/RenderFullScreen.h:
- rendering/RenderGeometryMap.h:
- rendering/RenderListItem.h:
- rendering/RenderMediaControlElements.cpp:
- rendering/RenderMediaControlElements.h:
- rendering/RenderMediaControls.h:
- rendering/RenderMeter.h:
- rendering/RenderMultiColumnSet.cpp:
- rendering/RenderObject.h:
- rendering/RenderRegion.h:
- rendering/RenderRegionSet.cpp:
- rendering/RenderRegionSet.h:
- rendering/RenderTheme.h:
- rendering/RenderVTTCue.cpp:
- rendering/RenderVTTCue.h:
- rendering/SimpleLineLayoutFunctions.h:
- rendering/SimpleLineLayoutResolver.h:
- rendering/line/BreakingContext.h:
- rendering/line/LineBreaker.h:
- rendering/mathml/MathMLStyle.h:
- rendering/mathml/RenderMathMLOperator.h:
- rendering/mathml/RenderMathMLRoot.h:
- rendering/shapes/RectangleShape.h:
- rendering/style/BasicShapes.h:
- rendering/style/BorderData.h:
- rendering/style/CounterDirectives.h:
- rendering/style/GridArea.h:
- rendering/style/KeyframeList.h:
- rendering/style/RenderStyle.h:
- rendering/style/SVGRenderStyle.h:
- rendering/style/ShapeValue.h:
- rendering/style/StyleCachedImage.h:
- rendering/style/StyleMultiColData.h:
- rendering/style/StyleRareNonInheritedData.h:
- rendering/style/WillChangeData.h:
- rendering/svg/RenderSVGBlock.h:
- rendering/svg/RenderSVGRect.h:
- rendering/svg/RenderSVGResource.h:
- rendering/svg/RenderSVGResourceContainer.h:
- rendering/svg/RenderSVGResourceMarker.h:
- rendering/svg/RenderSVGResourceMasker.h:
- rendering/svg/RenderSVGResourcePattern.h:
- rendering/svg/SVGInlineFlowBox.cpp:
- rendering/svg/SVGInlineTextBox.h:
- rendering/svg/SVGRootInlineBox.h:
- rendering/svg/SVGTextChunk.cpp:
- rendering/svg/SVGTextChunk.h:
- rendering/svg/SVGTextChunkBuilder.cpp:
- rendering/svg/SVGTextLayoutEngine.cpp:
- rendering/svg/SVGTextLayoutEngineSpacing.h:
- style/RenderTreePosition.cpp:
- style/RenderTreePosition.h:
- style/RenderTreeUpdater.h:
- style/StyleFontSizeFunctions.h:
- style/StyleInvalidator.h:
- style/StyleResolveForDocument.h:
- style/StyleScope.h:
- style/StyleTreeResolver.cpp:
- style/StyleTreeResolver.h:
- style/StyleUpdate.h:
- svg/SVGAnimationElement.h:
- svg/SVGFEBlendElement.cpp:
- svg/SVGFEBlendElement.h:
- svg/SVGFEComponentTransferElement.h:
- svg/SVGFEFloodElement.cpp:
- svg/SVGFEFloodElement.h:
- svg/SVGFEImageElement.h:
- svg/SVGFEMergeElement.cpp:
- svg/SVGFEMergeElement.h:
- svg/SVGFEOffsetElement.cpp:
- svg/SVGFEOffsetElement.h:
- svg/SVGFETileElement.cpp:
- svg/SVGFETileElement.h:
- svg/SVGParserUtilities.h:
- svg/SVGPathByteStream.h:
- svg/SVGPathElement.cpp:
- svg/SVGPolyElement.cpp:
- svg/SVGSVGElement.cpp:
- svg/SVGTRefElement.cpp:
- svg/SVGTextPathElement.cpp:
- svg/SVGUseElement.cpp:
- svg/animation/SMILTimeContainer.h:
- svg/graphics/SVGImageCache.h:
- svg/graphics/filters/SVGFilter.h:
- 2:22 AM Changeset in webkit [218792] by
-
- 6 edits in trunk/Source/WebCore
Remove rAf suspension logging
https://bugs.webkit.org/show_bug.cgi?id=173821
Reviewed by Tim Horton.
Now that webkit.org/b/173628 is fixed, we can remove the logging code we added.
- dom/Document.cpp:
(WebCore::Document::requestAnimationFrame):
- dom/ScriptedAnimationController.cpp:
(WebCore::ScriptedAnimationController::suspend):
(WebCore::ScriptedAnimationController::resume):
- page/Page.cpp:
(WebCore::Page::suspendScriptedAnimations):
(WebCore::Page::resumeScriptedAnimations):
(WebCore::Page::setIsVisibleInternal):
- platform/RuntimeApplicationChecks.h:
- platform/cocoa/RuntimeApplicationChecksCocoa.mm:
(WebCore::MacApplication::isDumpRenderTree): Deleted.
Jun 24, 2017:
- 5:07 PM Changeset in webkit [218791] by
-
- 2 edits in trunk/Source/WebKit2
[WK2] Make sure encodeClientTypesAndData() / decodeClientTypesAndData() match exactly
https://bugs.webkit.org/show_bug.cgi?id=173813
Reviewed by Ryosuke Niwa.
Make sure encodeClientTypesAndData() / decodeClientTypesAndData() match exactly. The previous
IPC encoder code would assume types and data vector have the same length. It would first encode
the length of data using |data.size()| but then would encode types.size() values from the
data vector. While there are debug assertions to ensure both vectors have the same size, this
seems unnecessarily fragile in release builds. If both vectors happen to have different sizes,
this will lead to weird IPC bugs.
- Shared/WebCoreArgumentCoders.cpp:
(IPC::encodeClientTypesAndData):
(IPC::decodeClientTypesAndData):
- 3:29 PM Changeset in webkit [218790] by
-
- 7 edits1 add in trunk
[JSC] Clean up Object.entries implementation
https://bugs.webkit.org/show_bug.cgi?id=173759
Reviewed by Sam Weinig.
JSTests:
- microbenchmarks/object-entries.js: Added.
(test):
Source/JavaScriptCore:
This patch cleans up Object.entries implementation.
We drop unused private functions. And we merge the
implementation into Object.entries.
It slightly speeds up Object.entries speed.
baseline patched
object-entries 148.0101+-5.6627 142.1877+-4.8661 might be 1.0409x faster
- builtins/BuiltinNames.h:
- builtins/ObjectConstructor.js:
(entries):
(globalPrivate.enumerableOwnProperties): Deleted.
- runtime/JSGlobalObject.cpp:
(JSC::JSGlobalObject::init):
- runtime/ObjectConstructor.cpp:
(JSC::ownEnumerablePropertyKeys): Deleted.
- runtime/ObjectConstructor.h:
- 2:47 PM Changeset in webkit [218789] by
-
- 7 edits21 adds in trunk/Source/WebCore
[WebIDL] Add complete support for stringifier
https://bugs.webkit.org/show_bug.cgi?id=173724
Patch by Sam Weinig <sam@webkit.org> on 2017-06-24
Reviewed by Darin Adler.
- bindings/scripts/CodeGeneratorJS.pm:
(AddStringifierOperationIfNeeded):
Update AddStringifierOperationIfNeeded to support stringifier on operations and be more
strict about the allowed types. Also copies over all extended attributes to the synthetic
operation.
- bindings/scripts/IDLParser.pm:
(parseInterfaceMember):
(parseOperationOrReadWriteAttributeOrMaplike):
(parseReadOnlyMember):
(parseStringifier):
(parseStaticMember):
(parseAttributeOrOperationForStringifierOrStatic):
(parseReadWriteAttribute):
(parseAttributeRest):
(parseOperation):
(parseSpecialOperation):
(parseMapLikeProperties):
(parseOperationRest):
(parseAttributeOrOperationOrIterator): Deleted.
(parseQualifier): Deleted.
(parseAttributeOrOperationRest): Deleted.
(parseAttribute): Deleted.
(parseOperationOrIterator): Deleted.
- Update parser to more closely resemble the WebIDL grammar, splitting out parseStringifier and parseStaticMember into their own subroutines.
- Move those and parseSerializer, parseStringifier, parseStaticMember, parseIterableRest and a split out parseReadOnlyMembers up into parseInterfaceMember to make it clearer that they are top level members and match the grammar.
- Rename parseAttributeOrOperationOrIterator to parseOperationOrReadWriteAttributeOrMaplike to match the grammar language and make it clear what it does.
- Add parseAttributeOrOperationForStringifierOrStatic which contains most of the logic for parsing stringifiers and static members, which have almost identical grammars.
- Remove creator special which no longer exists in the spec.
- css/DOMMatrixReadOnly.idl:
- css/WebKitCSSMatrix.idl:
- dom/Range.idl:
- html/URLSearchParams.idl:
Fix FIXMEs by switching to use stringifier.
- bindings/scripts/test/JS/JSTestStringifier.cpp: Added.
- bindings/scripts/test/JS/JSTestStringifier.h: Added.
- bindings/scripts/test/JS/JSTestStringifierAnonymousOperation.cpp: Added.
- bindings/scripts/test/JS/JSTestStringifierAnonymousOperation.h: Added.
- bindings/scripts/test/JS/JSTestStringifierNamedOperation.cpp: Added.
- bindings/scripts/test/JS/JSTestStringifierNamedOperation.h: Added.
- bindings/scripts/test/JS/JSTestStringifierOperationImplementedAs.cpp: Added.
- bindings/scripts/test/JS/JSTestStringifierOperationImplementedAs.h: Added.
- bindings/scripts/test/JS/JSTestStringifierOperationNamedToString.cpp: Added.
- bindings/scripts/test/JS/JSTestStringifierOperationNamedToString.h: Added.
- bindings/scripts/test/JS/JSTestStringifierReadOnlyAttribute.cpp: Added.
- bindings/scripts/test/JS/JSTestStringifierReadOnlyAttribute.h: Added.
- bindings/scripts/test/JS/JSTestStringifierReadWriteAttribute.cpp: Added.
- bindings/scripts/test/JS/JSTestStringifierReadWriteAttribute.h: Added.
- bindings/scripts/test/TestStringifier.idl: Added.
- bindings/scripts/test/TestStringifierAnonymousOperation.idl: Added.
- bindings/scripts/test/TestStringifierNamedOperation.idl: Added.
- bindings/scripts/test/TestStringifierOperationImplementedAs.idl: Added.
- bindings/scripts/test/TestStringifierOperationNamedToString.idl: Added.
- bindings/scripts/test/TestStringifierReadOnlyAttribute.idl: Added.
- bindings/scripts/test/TestStringifierReadWriteAttribute.idl: Added.
Add new tests.
- 1:14 PM Changeset in webkit [218788] by
-
- 8 edits in trunk/Source/bmalloc
bmalloc: Add a per-thread line cache
https://bugs.webkit.org/show_bug.cgi?id=173552
Reviewed by Darin Adler.
Previously, any thread could allocate out of any page with free lines.
Now, the first thread to free a line in a page owns that page's free
lines until the whole page becomes free.
This patch is a big speedup on multi-threaded benchmarks.
tree_churn --parallel gets 14% faster on a 2-core (4-hyper-core) MacBook
Air and 2.85X faster on 12-core (24-hyper-core) Mac Pro. Other parallel
benchmarks show significant but smaller speedups.
Thread affinity is a great predictor of object lifetime. The per-thread
line cache avoids the pathology of shuffling pages between threads,
turning predictable lifetimes into unpredictable lifetimes, increasing
fragmentation. On tree_churn --parallel, the per-thread line cache
increases free memory found per page scanned by 2.85X.
Free line scanning in fragmented pages is pretty expensive relative to
other allocate / initialize / free operations. According to Instruments,
on tree_churn --parallel, scanning is about 10X more expensive than
freeing. This explains why a 2.85X improvement in scanning efficiency
translates into a 2.85X overall speedup on tree_churn --parallel.
- bmalloc/Allocator.cpp:
(bmalloc::Allocator::refillAllocatorSlowCase): Pass through our line
cache so the Heap can fill it.
- bmalloc/Deallocator.cpp:
(bmalloc::Deallocator::scavenge): Scavenge our line cache.
(bmalloc::Deallocator::processObjectLog): Deleted.
- bmalloc/Deallocator.h:
(bmalloc::Deallocator::lineCache): Added a line cache.
- bmalloc/Heap.cpp:
(bmalloc::Heap::deallocateLineCache): Deallocation function for thread
destruction.
(bmalloc::Heap::allocateSmallPage):
(bmalloc::Heap::deallocateSmallLine):
(bmalloc::Heap::allocateSmallBumpRangesByMetadata):
(bmalloc::Heap::allocateSmallBumpRangesByObject): Consult the new per-thread line
cache for allocation and deallocation.
- bmalloc/Heap.h:
(bmalloc::Heap::allocateSmallBumpRanges):
(bmalloc::Heap::derefSmallLine):
- bmalloc/List.h:
(bmalloc::List::remove): Remove has always been a logically static
operation. Declare it static now so that the Heap can remove a page from
a thread's line cache without holding a direct pointer to the cache.
- bmalloc/SmallPage.h:
- 8:14 AM Changeset in webkit [218787] by
-
- 3 edits in trunk/Source/WebKit2
[GTK] Introspection: webkit_web_view_new_with_related_view needs to be marked as a constructor
https://bugs.webkit.org/show_bug.cgi?id=173765
Reviewed by Carlos Garcia Campos.
Because the first parameter to this WebKitWebView constructor is itself a WebKitWebView,
the gi-scanner's heuristics decide that it's probably an object method rather than a
constructor, resulting in improper introspection generation. Annotate it with (constructor)
to override this behavior.
- UIProcess/API/gtk/WebKitWebViewGtk.cpp:
- UIProcess/API/wpe/WebKitWebViewWPE.cpp:
- 3:24 AM Changeset in webkit [218786] by
-
- 4 edits in trunk/Source/WebKit2
Unreviewed, rolling out r218785.
https://bugs.webkit.org/show_bug.cgi?id=173808
This change broke the macOS Sierra build (Requested by
annulen|home on #webkit).
Reverted changeset:
"Removed unused lambda captures from WebKit2"
https://bugs.webkit.org/show_bug.cgi?id=173555
http://trac.webkit.org/changeset/218785
Patch by Commit Queue <commit-queue@webkit.org> on 2017-06-24
- 3:16 AM Changeset in webkit [218785] by
-
- 4 edits in trunk/Source/WebKit2
Removed unused lambda captures from WebKit2
https://bugs.webkit.org/show_bug.cgi?id=173555
Reviewed by Alex Christensen.
- UIProcess/Network/NetworkProcessProxy.cpp:
(WebKit::NetworkProcessProxy::fetchWebsiteData):
(WebKit::NetworkProcessProxy::deleteWebsiteData):
(WebKit::NetworkProcessProxy::deleteWebsiteDataForOrigins):
- UIProcess/WebProcessProxy.cpp:
(WebKit::WebProcessProxy::fetchWebsiteData):
(WebKit::WebProcessProxy::deleteWebsiteData):
(WebKit::WebProcessProxy::deleteWebsiteDataForOrigins):
- UIProcess/WebsiteData/WebsiteDataStore.cpp:
(WebKit::WebsiteDataStore::topPrivatelyControlledDomainsWithWebsiteData):
- 1:06 AM Changeset in webkit [218784] by
-
- 19 edits3 deletes in trunk
Remove Reflect.enumerate
https://bugs.webkit.org/show_bug.cgi?id=173806
Patch by Joseph Pecoraro <Joseph Pecoraro> on 2017-06-24
Reviewed by Yusuke Suzuki.
JSTests:
- ChakraCore.yaml:
- es6.yaml:
These tests now fail because they use Reflect.enumerate.
- test262.yaml:
This test now passes, it checked that Reflect.enumerate is undefined!
- stress/property-name-enumerator-should-not-look-into-indexed-values-when-it-is-a-dictionary.js:
Convert to for..in which presented the original issue. See bug <https://webkit.org/b/149811>
- stress/reflect-enumerate.js: Removed.
Remove a test solely for Reflect.enumerate.
Source/JavaScriptCore:
- CMakeLists.txt:
- JavaScriptCore.xcodeproj/project.pbxproj:
- inspector/JSInjectedScriptHost.cpp:
(Inspector::JSInjectedScriptHost::subtype):
(Inspector::JSInjectedScriptHost::getInternalProperties):
(Inspector::JSInjectedScriptHost::iteratorEntries):
- runtime/JSGlobalObject.cpp:
(JSC::JSGlobalObject::init):
(JSC::JSGlobalObject::visitChildren):
- runtime/JSPropertyNameIterator.cpp: Removed.
- runtime/JSPropertyNameIterator.h: Removed.
- runtime/ReflectObject.cpp:
(JSC::reflectObjectEnumerate): Deleted.
Source/WebInspectorUI:
- UserInterface/Models/NativeFunctionParameters.js:
Tools:
- Scripts/run-jsc-stress-tests:
LayoutTests:
- inspector/model/remote-object-expected.txt:
- inspector/model/remote-object.html:
- platform/mac/inspector/model/remote-object-expected.txt:
- 12:14 AM Changeset in webkit [218783] by
-
- 8 edits2 adds in trunk
AX: Cannot call setValue() on contenteditable or ARIA text controls
https://bugs.webkit.org/show_bug.cgi?id=173520
Reviewed by Ryosuke Niwa.
Source/WebCore:
Add support for changing the value of a contenteditable and any other aria text control in setValue().
Test: accessibility/mac/set-value-editable-types.html
- accessibility/AccessibilityRenderObject.cpp:
(WebCore::AccessibilityRenderObject::setValue):
Tools:
Add setValue() method to WKTR (already existed in DRT).
- WebKitTestRunner/InjectedBundle/AccessibilityUIElement.cpp:
(WTR::AccessibilityUIElement::setValue):
- WebKitTestRunner/InjectedBundle/AccessibilityUIElement.h:
- WebKitTestRunner/InjectedBundle/Bindings/AccessibilityUIElement.idl:
- WebKitTestRunner/InjectedBundle/mac/AccessibilityUIElementMac.mm:
(WTR::AccessibilityUIElement::setValue):
LayoutTests:
- accessibility/mac/set-value-editable-types-expected.txt: Added.
- accessibility/mac/set-value-editable-types.html: Added.
Jun 23, 2017:
- 7:54 PM Changeset in webkit [218782] by
-
- 22 edits in trunk
Switch VMTraps to use halt instructions rather than breakpoint instructions
https://bugs.webkit.org/show_bug.cgi?id=173677
Source/JavaScriptCore:
<rdar://problem/32178892>
Reviewed by JF Bastien.
Using the breakpoint instruction for VMTraps caused issues with lldb.
Since we only need some way to stop execution we can, in theory, use
any exceptioning instruction we want. I went with the halt instruction
on X86 since that is the only one byte instruction that does not
breakpoint (in my tests both 0xf1 and 0xd6 produced EXC_BREAKPOINT).
On ARM we use the data cache clearing instruction with the zero register,
which triggers a segmentation fault.
Also, update the platform code to only use signaling VMTraps
on where we have an appropriate instruction (x86 and ARM64).
- API/tests/ExecutionTimeLimitTest.cpp:
(testExecutionTimeLimit):
- assembler/ARM64Assembler.h:
(JSC::ARM64Assembler::replaceWithVMHalt):
(JSC::ARM64Assembler::dataCacheZeroVirtualAddress):
(JSC::ARM64Assembler::replaceWithBkpt): Deleted.
- assembler/ARMAssembler.h:
(JSC::ARMAssembler::replaceWithBkpt): Deleted.
- assembler/ARMv7Assembler.h:
(JSC::ARMv7Assembler::replaceWithBkpt): Deleted.
- assembler/MIPSAssembler.h:
(JSC::MIPSAssembler::replaceWithBkpt): Deleted.
- assembler/MacroAssemblerARM.h:
(JSC::MacroAssemblerARM::replaceWithBreakpoint): Deleted.
- assembler/MacroAssemblerARM64.h:
(JSC::MacroAssemblerARM64::replaceWithVMHalt):
(JSC::MacroAssemblerARM64::replaceWithBreakpoint): Deleted.
- assembler/MacroAssemblerARMv7.h:
(JSC::MacroAssemblerARMv7::storeFence):
(JSC::MacroAssemblerARMv7::replaceWithBreakpoint): Deleted.
- assembler/MacroAssemblerMIPS.h:
(JSC::MacroAssemblerMIPS::replaceWithBreakpoint): Deleted.
- assembler/MacroAssemblerX86Common.h:
(JSC::MacroAssemblerX86Common::replaceWithVMHalt):
(JSC::MacroAssemblerX86Common::replaceWithBreakpoint): Deleted.
- assembler/X86Assembler.h:
(JSC::X86Assembler::replaceWithHlt):
(JSC::X86Assembler::replaceWithInt3): Deleted.
- dfg/DFGJumpReplacement.cpp:
(JSC::DFG::JumpReplacement::installVMTrapBreakpoint):
- runtime/VMTraps.cpp:
(JSC::SignalContext::SignalContext):
(JSC::installSignalHandler):
(JSC::SignalContext::adjustPCToPointToTrappingInstruction): Deleted.
- wasm/WasmFaultSignalHandler.cpp:
(JSC::Wasm::enableFastMemory):
Source/WTF:
<rdar://problem/32178892>
Reviewed by JF Bastien.
Remove the Trap signal handler code since it plays badly with lldb and combine
SIGBUS with SIGSEGV since distiguishing them is generally non-portable.
Also, update the platform code to only use signaling VMTraps
on where we have an appropriate instruction (x86 and ARM64).
- wtf/Platform.h:
- wtf/threads/Signals.cpp:
(WTF::fromMachException):
(WTF::toMachMask):
(WTF::installSignalHandler):
(WTF::jscSignalHandler):
- wtf/threads/Signals.h:
(WTF::toSystemSignal):
(WTF::fromSystemSignal):
Tools:
Reviewed by JF Bastien.
- TestWebKitAPI/Tests/WTF/ThreadMessages.cpp:
(TEST):
- 7:45 PM Changeset in webkit [218781] by
-
- 2 edits in trunk/Source/WebInspectorUI
Web Inspector: Script Timeline bubbles sometimes appear to miss large events
https://bugs.webkit.org/show_bug.cgi?id=173746
<rdar://problem/32950808>
Reviewed by Brian Burg.
- UserInterface/Models/Timeline.js:
(WebInspector.Timeline.prototype.addRecord):
(WebInspector.Timeline.prototype._tryInsertInSortedOrder):
The list of records is assumed to be sorted by the code that draws bubbles
however the order in which we receive them may not be sorted. Make a quick
effort to sort recent records so that as we are drawing the timeline it is
more accurate.
- 7:19 PM Changeset in webkit [218780] by
-
- 2 edits in trunk/Source/WebKit/mac
Remove an unused typedef from WebFrameLoaderClient
https://bugs.webkit.org/show_bug.cgi?id=173795
Reviewed by Simon Fraser.
- WebCoreSupport/WebFrameLoaderClient.h:
This unused typedef in the global namespace conflicts with one in a macOS system header.
Also... it's unused.
Remove it.
- 6:56 PM Changeset in webkit [218779] by
-
- 4 edits in trunk/LayoutTests
fast/events/page-visibility-iframe-delete-test.html is flaky
https://bugs.webkit.org/show_bug.cgi?id=173798
Reviewed by Simon Fraser.
Modernize the test and deflake it by not making it rely on a specific order
for the visibilitychange events. The event queues are per documents and there
is therefore no guarantee those events will fire in frame tree order.
- fast/events/page-visibility-iframe-delete-test-expected.txt:
- fast/events/page-visibility-iframe-delete-test.html:
- 6:25 PM Changeset in webkit [218778] by
-
- 2 edits in trunk/Source/WebCore
Attempt to fix an internal build after r218755.
- bindings/js/JSMainThreadExecStateInstrumentation.h:
- 5:35 PM Changeset in webkit [218777] by
-
- 3 edits in trunk/Source/WebInspectorUI
Web Inspector: RTL: flip all go-to-arrow instances
https://bugs.webkit.org/show_bug.cgi?id=173794
<rdar://problem/32221912>
Reviewed by Joseph Pecoraro.
Move the flipping transform so it applies to all go-to arrows.
- UserInterface/Views/DataGrid.css:
(body[dir=rtl] .data-grid td .go-to-arrow):
- UserInterface/Views/Main.css:
(body[dir=rtl] .go-to-arrow):
- 5:03 PM Changeset in webkit [218776] by
-
- 2 edits in trunk/Source/WTF
Add notifyutil registrations for going in and out of simulated low memory state
https://bugs.webkit.org/show_bug.cgi?id=173797
Reviewed by Andreas Kling.
Add
notifyutil -p org.WebKit.lowMemory.begin
notifyutil -p org.WebKit.lowMemory.end
for debugging.
- wtf/cocoa/MemoryPressureHandlerCocoa.mm:
(WTF::MemoryPressureHandler::install):
- 4:24 PM Changeset in webkit [218775] by
-
- 5 edits in trunk/Source
Add release assertion to make sure callbackIdentifier is not 0 in DocumentLoader::finishedLoadingIcon()
https://bugs.webkit.org/show_bug.cgi?id=173792
Reviewed by Ryosuke Niwa.
Add release assertion to make sure callbackIdentifier is not 0 in DocumentLoader::finishedLoadingIcon()
as this could cause HashTable corruption on WebPageProxy side.
Source/WebCore:
- loader/DocumentLoader.cpp:
(WebCore::DocumentLoader::finishedLoadingIcon):
Source/WebKit2:
- WebProcess/WebCoreSupport/WebFrameLoaderClient.cpp:
(WebKit::WebFrameLoaderClient::finishedLoadingIcon):
- WebProcess/WebCoreSupport/WebFrameLoaderClient.h:
- 4:21 PM Changeset in webkit [218774] by
-
- 1 edit2 adds in trunk/LayoutTests
Add a binary data channel webrtc test
https://bugs.webkit.org/show_bug.cgi?id=173787
Patch by Youenn Fablet <youenn@apple.com> on 2017-06-23
Reviewed by Alex Christensen.
- webrtc/datachannel/binary-expected.txt: Added.
- webrtc/datachannel/binary.html: Added.
- 4:16 PM Changeset in webkit [218773] by
-
- 14 edits in trunk
Set getUserMedia permission to true by default on WTR
https://bugs.webkit.org/show_bug.cgi?id=173610
Patch by Youenn Fablet <youenn@apple.com> on 2017-06-23
Reviewed by Alex Christensen.
LayoutTests/imported/w3c:
- web-platform-tests/webrtc/simplecall-expected.txt:
Tools:
Adding a way to reset the getUserMedia permission to pending.
- WebKitTestRunner/InjectedBundle/Bindings/TestRunner.idl:
- WebKitTestRunner/InjectedBundle/InjectedBundle.cpp:
(WTR::InjectedBundle::setUserMediaPermissionAsUnset):
- WebKitTestRunner/InjectedBundle/InjectedBundle.h:
- WebKitTestRunner/InjectedBundle/TestRunner.cpp:
(WTR::TestRunner::setUserMediaPermissionAsUnset):
- WebKitTestRunner/InjectedBundle/TestRunner.h:
- WebKitTestRunner/TestController.cpp:
(WTR::TestController::resetStateToConsistentValues):
(WTR::TestController::setUserMediaPermissionAsUnset):
- WebKitTestRunner/TestController.h:
- WebKitTestRunner/TestInvocation.cpp:
(WTR::TestInvocation::didReceiveMessageFromInjectedBundle):
LayoutTests:
- fast/mediastream/delayed-permission-allowed.html:
- fast/mediastream/delayed-permission-denied.html:
- 4:08 PM Changeset in webkit [218772] by
-
- 2 edits in trunk/LayoutTests
webrtc/routines.js should call createAnswer once setRemoteDescription promise is resolved
https://bugs.webkit.org/show_bug.cgi?id=173785
Patch by Youenn Fablet <youenn@apple.com> on 2017-06-23
Reviewed by Alex Christensen.
- webrtc/routines.js:
- 4:06 PM Changeset in webkit [218771] by
-
- 3 edits2 adds in trunk
webrtc::WebRtcSession is not handling correctly its state when setLocalDescription fails and is called again
https://bugs.webkit.org/show_bug.cgi?id=173783
Patch by Youenn Fablet <youenn@apple.com> on 2017-06-23
Reviewed by Alex Christensen.
Source/WebCore:
Test: webrtc/libwebrtc/setLocalDescriptionCrash.html
- Modules/mediastream/libwebrtc/LibWebRTCMediaEndpoint.cpp:
(WebCore::LibWebRTCMediaEndpoint::doSetLocalDescription): Fail early if there is no pending remote description and description is for an answer.
LayoutTests:
- webrtc/libwebrtc/setLocalDescriptionCrash-expected.txt: Added.
- webrtc/libwebrtc/setLocalDescriptionCrash.html: Added.
- 3:50 PM Changeset in webkit [218770] by
-
- 2 edits in trunk/LayoutTests
Added additional test expectations for webrtc/video-replace-muted-track.html.
https://bugs.webkit.org/show_bug.cgi?id=173608
Unreviewed test expectations.
- platform/mac-wk2/TestExpectations:
- 3:50 PM Changeset in webkit [218769] by
-
- 4 edits in trunk/Source/WebKit2
Stop passing Vector by value in WebProcessProxy::deleteWebsiteDataForTopPrivatelyControlledDomainsInAllPersistentDataStores()
https://bugs.webkit.org/show_bug.cgi?id=173782
Reviewed by Brent Fulgham.
- UIProcess/WebProcessProxy.cpp:
(WebKit::WebProcessProxy::deleteWebsiteDataForTopPrivatelyControlledDomainsInAllPersistentDataStores):
- UIProcess/WebProcessProxy.h:
- UIProcess/WebResourceLoadStatisticsStore.cpp:
(WebKit::WebResourceLoadStatisticsStore::removeDataRecords):
- 3:10 PM Changeset in webkit [218768] by
-
- 2 edits in trunk/Source/WebCore
[MediaStream macOS] enumerateDevices should only return valid audio capture devices
https://bugs.webkit.org/show_bug.cgi?id=173790
<rdar://problem/32260334>
Reviewed by Youenn Fablet.
- platform/mediastream/mac/CoreAudioCaptureDeviceManager.cpp:
(WebCore::isValidCaptureDevice): A valid device has a name and is not an aggregate device
created by VPIO.
(WebCore::CoreAudioCaptureDeviceManager::refreshAudioCaptureDevices):
- 3:08 PM Changeset in webkit [218767] by
-
- 2 edits in trunk/Source/WebKit2
Avoid copying HashSet to Vector in WebPageProxy::clearLoadDependentCallbacks()
https://bugs.webkit.org/show_bug.cgi?id=173786
Reviewed by Geoffrey Garen.
Avoid copying HashSet to Vector in WebPageProxy::clearLoadDependentCallbacks().
Instead, just move the HashSet and iterate on that. This is OK since we were
clearing the HashSet right after copying it to the vector anyway.
- UIProcess/WebPageProxy.cpp:
(WebKit::WebPageProxy::clearLoadDependentCallbacks):
- 2:34 PM Changeset in webkit [218766] by
-
- 7 edits in tags/Safari-604.1.27.0.1/Source
Versioning.
- 2:33 PM Changeset in webkit [218765] by
-
- 2 edits in trunk/Source/WebKit2
Unreviewed, fix Debug build after r218763
- NetworkProcess/NetworkDataTask.cpp:
- 2:26 PM Changeset in webkit [218764] by
-
- 3 edits in trunk/Source/WebKit2
Add more release logging for process assertions
https://bugs.webkit.org/show_bug.cgi?id=173624
<rdar://problem/32412701>
Reviewed by Geoffrey Garen.
Add more release logging for process assertions to help debug issues such
as <rdar://problem/32412701>.
- UIProcess/ProcessThrottler.cpp:
(WebKit::ProcessThrottler::didConnectToProcess):
- UIProcess/ios/ProcessAssertionIOS.mm:
(WebKit::ProcessAssertion::ProcessAssertion):
(WebKit::ProcessAssertion::~ProcessAssertion):
(WebKit::ProcessAssertion::setState):
- 2:18 PM Changeset in webkit [218763] by
-
- 19 edits in trunk/Source/WebKit2
Consistently use RunLoop::isMain() in WebKit2
https://bugs.webkit.org/show_bug.cgi?id=173745
<rdar://problem/32936527>
Reviewed by Brent Fulgham.
Consistently use RunLoop:isMain() on WebKit2 instead of WTF::isMainThread().
WTF::isMainThread() returns true for the WebThread instead of the UIThread
in WebKit1. Therefore WTF::isMainThread() is unsafe to use in the UIProcess
as the application may be using WebKit1 as well and have a WebThread.
Technically, we only *need* to move from WTF::isMainThread() to
RunLoop::isMain() in the UIProcess code but I ported all of WebKit2 for
consistency. RunLoop::isMain() is always safe in WebKit2.
- NetworkProcess/NetworkDataTask.cpp:
(WebKit::NetworkDataTask::NetworkDataTask):
(WebKit::NetworkDataTask::~NetworkDataTask):
- NetworkProcess/NetworkDataTaskBlob.cpp:
(WebKit::NetworkDataTaskBlob::getSizeForNext):
(WebKit::NetworkDataTaskBlob::didGetSize):
(WebKit::NetworkDataTaskBlob::seek):
(WebKit::NetworkDataTaskBlob::read):
- NetworkProcess/NetworkLoad.cpp:
(WebKit::NetworkLoad::didReceiveResponseNetworkSession):
(WebKit::NetworkLoad::notifyDidReceiveResponse):
- NetworkProcess/NetworkSession.cpp:
(WebKit::NetworkSession::defaultSession):
- NetworkProcess/cache/NetworkCache.cpp:
(WebKit::NetworkCache::resourceType):
(WebKit::NetworkCache::Cache::initialize):
- NetworkProcess/cache/NetworkCacheIOChannelSoup.cpp:
(WebKit::NetworkCache::IOChannel::read):
(WebKit::NetworkCache::IOChannel::readSyncInThread):
- NetworkProcess/cocoa/NetworkSessionCocoa.mm:
(-[WKNetworkSessionDelegate URLSession:dataTask:didReceiveResponse:completionHandler:]):
(WebKit::NetworkSessionCocoa::defaultSession):
(WebKit::NetworkSessionCocoa::dataTaskForIdentifier):
- Shared/CoordinatedGraphics/CoordinatedGraphicsScene.cpp:
(WebKit::CoordinatedGraphicsScene::dispatchOnMainThread):
(WebKit::CoordinatedGraphicsScene::syncRemoteContent):
(WebKit::CoordinatedGraphicsScene::detach):
(WebKit::CoordinatedGraphicsScene::appendUpdate):
- Shared/CoordinatedGraphics/threadedcompositor/CompositingRunLoop.cpp:
(WebKit::WorkQueuePool::singleton):
(WebKit::WorkQueuePool::dispatch):
(WebKit::CompositingRunLoop::~CompositingRunLoop):
(WebKit::CompositingRunLoop::performTask):
(WebKit::CompositingRunLoop::performTaskSync):
- Shared/CoordinatedGraphics/threadedcompositor/ThreadedCompositor.cpp:
(WebKit::ThreadedCompositor::createGLContext):
(WebKit::ThreadedCompositor::renderNextFrame):
(WebKit::ThreadedCompositor::commitScrollOffset):
(WebKit::ThreadedCompositor::updateSceneState):
(WebKit::ThreadedCompositor::releaseUpdateAtlases):
(WebKit::ThreadedCompositor::frameComplete):
- Shared/Plugins/NPObjectProxy.cpp:
(WebKit::NPObjectProxy::NP_Deallocate):
- UIProcess/Storage/LocalStorageDatabaseTracker.cpp:
(WebKit::LocalStorageDatabaseTracker::deleteDatabasesModifiedSince):
- UIProcess/WebProcessProxy.cpp:
(WebKit::WebProcessProxy::deleteWebsiteDataForTopPrivatelyControlledDomainsInAllPersistentDataStores):
(WebKit::WebProcessProxy::topPrivatelyControlledDomainsWithWebiteData):
- UIProcess/WebResourceLoadStatisticsStore.cpp:
(WebKit::WebResourceLoadStatisticsStore::removeDataRecords):
(WebKit::WebResourceLoadStatisticsStore::registerSharedResourceLoadObserver):
(WebKit::WebResourceLoadStatisticsStore::writeStoreToDisk):
(WebKit::WebResourceLoadStatisticsStore::writeEncoderToDisk):
- UIProcess/ios/WKPDFView.mm:
(-[WKPDFView _highlightLinkAnnotation:forDuration:completionHandler:]):
- UIProcess/ios/forms/WKFileUploadPanel.mm:
(-[WKFileUploadPanel _uploadItemForImageData:imageName:successBlock:failureBlock:]):
- WebProcess/WebCoreSupport/soup/WebFrameNetworkingContext.cpp:
(WebKit::WebFrameNetworkingContext::ensurePrivateBrowsingSession):
- WebProcess/soup/WebKitSoupRequestInputStream.cpp:
(webkitSoupRequestInputStreamReadAsync):
- 1:50 PM Changeset in webkit [218762] by
-
- 1 edit2 deletes in trunk/LayoutTests
Remove fast/regions/region-leak-js-information-when-disabled-at-runtime.html
https://bugs.webkit.org/show_bug.cgi?id=173784
Patch by Frederic Wang <fwang@igalia.com> on 2017-06-23
Reviewed by Simon Fraser.
fast/regions/region-leak-js-information-when-disabled-at-runtime.html is supposed to test
that some JS properties are undefined when CSS regions are disabled at runtime. However, this
part is currently commented out and the test is no longer relevant after r200524. Hence we
just remove it.
- fast/regions/region-leak-js-information-when-disabled-at-runtime-expected.txt: Removed.
- fast/regions/region-leak-js-information-when-disabled-at-runtime.html: Removed.
- 1:49 PM Changeset in webkit [218761] by
-
- 1 copy in tags/Safari-604.1.27.0.1
New tag.
- 1:45 PM Changeset in webkit [218760] by
-
- 3 edits in trunk/Source/WebKit2
v3: REGRESSION (r218419): 3 NSMutableDictionary leaks calling -[WKProcessPool _pluginLoadClientPolicies]
<https://webkit.org/b/173689>
Reviewed by Tim Horton.
- UIProcess/API/Cocoa/WKProcessPool.mm:
(policiesHashMapToDictionary): Use adoptNS().get() to avoid
dumping objects into autoreleasepools unnecessarily.
- UIProcess/API/Cocoa/WKProcessPoolPrivate.h:
(-[WKProcessPool _pluginLoadClientPolicies]): Add back 'copy'
attribute to document that we're returning a new object on each
invocation. I shouldn't have removed it in the v2 patch.
- 1:43 PM Changeset in webkit [218759] by
-
- 2 edits in trunk/Source/WebKit2
LibWebRTCSocketClient should not destroy its socket within signalClose callback
https://bugs.webkit.org/show_bug.cgi?id=173739
Patch by Youenn Fablet <youenn@apple.com> on 2017-06-23
Reviewed by Brent Fulgham.
- NetworkProcess/webrtc/LibWebRTCSocketClient.cpp:
(WebKit::LibWebRTCSocketClient::signalClose): Destroy 'this' asynchronously to make the caller of signalClose valid until it is not used.
- 1:12 PM Changeset in webkit [218758] by
-
- 4 edits in trunk/Source/WebKit2
Fix typo in WebProcessProxy::topPrivatelyControlledDomainsWithWebsiteData() naming
https://bugs.webkit.org/show_bug.cgi?id=173781
Reviewed by Brent Fulgham.
- UIProcess/WebProcessProxy.cpp:
(WebKit::WebProcessProxy::topPrivatelyControlledDomainsWithWebsiteData):
(WebKit::WebProcessProxy::topPrivatelyControlledDomainsWithWebiteData): Deleted.
- UIProcess/WebProcessProxy.h:
- UIProcess/WebResourceLoadStatisticsStore.cpp:
(WebKit::WebResourceLoadStatisticsStore::grandfatherExistingWebsiteData):
- 12:32 PM Changeset in webkit [218757] by
-
- 14 edits in trunk/Source
[WK2] Support -[WebAVPlayerController setMuted:]
https://bugs.webkit.org/show_bug.cgi?id=173777
Reviewed by Eric Carlson.
Source/WebCore:
Have -[WebAVPlayerController setMuted:] pass the request to its delegate (the model)
rather than just storing the value.
- platform/cocoa/WebPlaybackSessionModel.h:
- platform/cocoa/WebPlaybackSessionModelMediaElement.h:
- platform/cocoa/WebPlaybackSessionModelMediaElement.mm:
(WebCore::WebPlaybackSessionModelMediaElement::toggleMuted):
(WebCore::WebPlaybackSessionModelMediaElement::setMuted):
- platform/ios/WebAVPlayerController.h:
- platform/ios/WebAVPlayerController.mm:
(-[WebAVPlayerController isMuted]):
(-[WebAVPlayerController setMuted:]):
- platform/ios/WebVideoFullscreenControllerAVKit.mm:
(WebVideoFullscreenControllerContext::setMuted):
Source/WebKit2:
Add a SetMuted message to WebPlaybackSessionManager.
Drive-by refactoring: Because the new setMuted() method conflicts with the existing setMuted()
notification, do a giant rename of the notificiation methods from set(Value) -> (value)Changed.
- UIProcess/Cocoa/WebPlaybackSessionManagerProxy.h:
- UIProcess/Cocoa/WebPlaybackSessionManagerProxy.messages.in:
- UIProcess/Cocoa/WebPlaybackSessionManagerProxy.mm:
(WebKit::WebPlaybackSessionModelContext::setMuted):
(WebKit::WebPlaybackSessionModelContext::playbackStartedTimeChanged):
(WebKit::WebPlaybackSessionModelContext::durationChanged):
(WebKit::WebPlaybackSessionModelContext::currentTimeChanged):
(WebKit::WebPlaybackSessionModelContext::bufferedTimeChanged):
(WebKit::WebPlaybackSessionModelContext::rateChanged):
(WebKit::WebPlaybackSessionModelContext::seekableRangesChanged):
(WebKit::WebPlaybackSessionModelContext::canPlayFastReverseChanged):
(WebKit::WebPlaybackSessionModelContext::audioMediaSelectionOptionsChanged):
(WebKit::WebPlaybackSessionModelContext::legibleMediaSelectionOptionsChanged):
(WebKit::WebPlaybackSessionModelContext::audioMediaSelectionIndexChanged):
(WebKit::WebPlaybackSessionModelContext::legibleMediaSelectionIndexChanged):
(WebKit::WebPlaybackSessionModelContext::externalPlaybackChanged):
(WebKit::WebPlaybackSessionModelContext::wirelessVideoPlaybackDisabledChanged):
(WebKit::WebPlaybackSessionModelContext::mutedChanged):
(WebKit::WebPlaybackSessionManagerProxy::currentTimeChanged):
(WebKit::WebPlaybackSessionManagerProxy::bufferedTimeChanged):
(WebKit::WebPlaybackSessionManagerProxy::seekableRangesVectorChanged):
(WebKit::WebPlaybackSessionManagerProxy::canPlayFastReverseChanged):
(WebKit::WebPlaybackSessionManagerProxy::audioMediaSelectionOptionsChanged):
(WebKit::WebPlaybackSessionManagerProxy::legibleMediaSelectionOptionsChanged):
(WebKit::WebPlaybackSessionManagerProxy::audioMediaSelectionIndexChanged):
(WebKit::WebPlaybackSessionManagerProxy::legibleMediaSelectionIndexChanged):
(WebKit::WebPlaybackSessionManagerProxy::externalPlaybackPropertiesChanged):
(WebKit::WebPlaybackSessionManagerProxy::wirelessVideoPlaybackDisabledChanged):
(WebKit::WebPlaybackSessionManagerProxy::mutedChanged):
(WebKit::WebPlaybackSessionManagerProxy::durationChanged):
(WebKit::WebPlaybackSessionManagerProxy::playbackStartedTimeChanged):
(WebKit::WebPlaybackSessionManagerProxy::rateChanged):
(WebKit::WebPlaybackSessionManagerProxy::setMuted):
(WebKit::WebPlaybackSessionManagerProxy::controlsManagerInterface):
(WebKit::WebPlaybackSessionModelContext::setPlaybackStartedTime): Deleted.
(WebKit::WebPlaybackSessionModelContext::setDuration): Deleted.
(WebKit::WebPlaybackSessionModelContext::setCurrentTime): Deleted.
(WebKit::WebPlaybackSessionModelContext::setBufferedTime): Deleted.
(WebKit::WebPlaybackSessionModelContext::setRate): Deleted.
(WebKit::WebPlaybackSessionModelContext::setSeekableRanges): Deleted.
(WebKit::WebPlaybackSessionModelContext::setCanPlayFastReverse): Deleted.
(WebKit::WebPlaybackSessionModelContext::setAudioMediaSelectionOptions): Deleted.
(WebKit::WebPlaybackSessionModelContext::setLegibleMediaSelectionOptions): Deleted.
(WebKit::WebPlaybackSessionModelContext::setAudioMediaSelectionIndex): Deleted.
(WebKit::WebPlaybackSessionModelContext::setLegibleMediaSelectionIndex): Deleted.
(WebKit::WebPlaybackSessionModelContext::setExternalPlayback): Deleted.
(WebKit::WebPlaybackSessionModelContext::setWirelessVideoPlaybackDisabled): Deleted.
(WebKit::WebPlaybackSessionManagerProxy::setCurrentTime): Deleted.
(WebKit::WebPlaybackSessionManagerProxy::setBufferedTime): Deleted.
(WebKit::WebPlaybackSessionManagerProxy::setSeekableRangesVector): Deleted.
(WebKit::WebPlaybackSessionManagerProxy::setCanPlayFastReverse): Deleted.
(WebKit::WebPlaybackSessionManagerProxy::setAudioMediaSelectionOptions): Deleted.
(WebKit::WebPlaybackSessionManagerProxy::setLegibleMediaSelectionOptions): Deleted.
(WebKit::WebPlaybackSessionManagerProxy::setAudioMediaSelectionIndex): Deleted.
(WebKit::WebPlaybackSessionManagerProxy::setLegibleMediaSelectionIndex): Deleted.
(WebKit::WebPlaybackSessionManagerProxy::setExternalPlaybackProperties): Deleted.
(WebKit::WebPlaybackSessionManagerProxy::setWirelessVideoPlaybackDisabled): Deleted.
(WebKit::WebPlaybackSessionManagerProxy::setDuration): Deleted.
(WebKit::WebPlaybackSessionManagerProxy::setPlaybackStartedTime): Deleted.
(WebKit::WebPlaybackSessionManagerProxy::setRate): Deleted.
- WebProcess/cocoa/WebPlaybackSessionManager.h:
- WebProcess/cocoa/WebPlaybackSessionManager.messages.in:
- WebProcess/cocoa/WebPlaybackSessionManager.mm:
(WebKit::WebPlaybackSessionManager::durationChanged):
(WebKit::WebPlaybackSessionManager::currentTimeChanged):
(WebKit::WebPlaybackSessionManager::bufferedTimeChanged):
(WebKit::WebPlaybackSessionManager::playbackStartedTimeChanged):
(WebKit::WebPlaybackSessionManager::rateChanged):
(WebKit::WebPlaybackSessionManager::seekableRangesChanged):
(WebKit::WebPlaybackSessionManager::canPlayFastReverseChanged):
(WebKit::WebPlaybackSessionManager::audioMediaSelectionOptionsChanged):
(WebKit::WebPlaybackSessionManager::legibleMediaSelectionOptionsChanged):
(WebKit::WebPlaybackSessionManager::externalPlaybackChanged):
(WebKit::WebPlaybackSessionManager::audioMediaSelectionIndexChanged):
(WebKit::WebPlaybackSessionManager::legibleMediaSelectionIndexChanged):
(WebKit::WebPlaybackSessionManager::wirelessVideoPlaybackDisabledChanged):
(WebKit::WebPlaybackSessionManager::mutedChanged):
(WebKit::WebPlaybackSessionManager::setMuted):
- 12:13 PM Changeset in webkit [218756] by
-
- 2 edits in trunk/Source/WebCore
Make RenderLayer::handleTouchEvent use usesAcceleratedScrolling()
https://bugs.webkit.org/show_bug.cgi?id=173763
Patch by Frederic Wang <fwang@igalia.com> on 2017-06-23
Reviewed by Simon Fraser.
No new tests, behavior is unchanged.
- rendering/RenderLayer.cpp:
(WebCore::RenderLayer::handleTouchEvent): Call usesAcceleratedScrolling() instead of
hasTouchScrollableOverflow() for clarity.
- 12:07 PM Changeset in webkit [218755] by
-
- 44 edits1 add in trunk/Source
Remove excessive headers from WebCore/{bindings,css,dom}
https://bugs.webkit.org/show_bug.cgi?id=173766
Reviewed by Simon Fraser.
Source/WebCore:
No new tests needed.
- ForwardingHeaders/runtime/ThrowScope.h: Added.
- Modules/plugins/QuickTimePluginReplacement.mm:
- bindings/js/CachedScriptSourceProvider.h:
- bindings/js/JSCommandLineAPIHostCustom.cpp:
- bindings/js/JSCustomXPathNSResolver.cpp:
- bindings/js/JSDOMConstructorNotConstructable.h:
- bindings/js/JSDOMConvertBase.h:
- bindings/js/JSDOMConvertBufferSource.h:
- bindings/js/JSDOMConvertInterface.h:
- bindings/js/JSDOMConvertStrings.h:
- bindings/js/JSDOMConvertUnion.h:
- bindings/js/JSDOMConvertVariadic.h:
- bindings/js/JSDOMConvertWebGL.h:
- bindings/js/JSDOMExceptionHandling.h:
- bindings/js/JSDOMIterator.h:
- bindings/js/JSDOMMapLike.h:
- bindings/js/JSDOMWindowBase.cpp:
- bindings/js/JSDOMWindowCustom.cpp:
- bindings/js/JSDOMWindowCustom.h:
- bindings/js/JSDOMWindowProperties.h:
- bindings/js/JSDOMWrapperCache.h:
- bindings/js/JSLazyEventListener.cpp:
- bindings/js/JSMainThreadExecStateInstrumentation.h:
- bindings/js/JSMediaListCustom.h:
- bindings/js/JSNodeCustom.h:
- bindings/js/JSNodeListCustom.h:
- bindings/js/JSVideoTrackListCustom.cpp:
- bindings/js/ScheduledAction.h:
- bindings/js/ScriptSourceCode.h:
- bindings/js/SerializedScriptValue.h:
- bindings/js/WebCoreJSClientData.h:
- css/CSSFontFeatureValue.h:
- css/CSSPrimitiveValue.h:
- css/CSSStyleSheet.h:
- css/CSSValue.h:
- css/StyleBuilderCustom.h:
- dom/ContainerNodeAlgorithms.h:
- dom/DataTransfer.h:
- dom/Node.h:
- dom/NodeRareData.h:
- dom/Position.h:
- dom/RenderedDocumentMarker.h:
Source/WebKit/mac:
- DOM/DOMInternal.mm:
- 11:48 AM Changeset in webkit [218754] by
-
- 65 edits1 delete in trunk
Use window.internals instead of overridePreference to set WebCore settings in tests
https://bugs.webkit.org/show_bug.cgi?id=173714
Patch by Frederic Wang <fwang@igalia.com> on 2017-06-23
Reviewed by Simon Fraser.
Source/WebKit2:
This commit remove preference to settings mapping in overrideBoolPreferenceForTestRunner that
are no longer necessary after the changes done in LayoutTests.
- WebProcess/InjectedBundle/InjectedBundle.cpp:
(WebKit::InjectedBundle::overrideBoolPreferenceForTestRunner): Remove mapping for preferences
that no longer require testRunner.overridePreference calls in tests.
LayoutTests:
This commit is a first step to replace the use of testRunner.overridePreference with
internals.settings.set* calls in tests. Such a call will fail immediately and obviously if
its a function that does not exist, and these settings are restored properly between tests.
Some preferences that do not have a WebCore setting or an internals.settings setter are not
handled yet. The temporary failure added in r218480 for fast/frames/flattening/set-preference
is removed and so is a custom expectation caret-mode-document-begin-end for Mac WK1.
- accessibility/gtk/caret-browsing-select-focus.html: Use internals.settings.set* function.
- accessibility/gtk/caret-browsing-text-focus.html: Ditto.
- accessibility/gtk/caret-offsets-and-extraneous-white-spaces.html: Ditto.
- accessibility/gtk/caret-offsets.html: Ditto.
- accessibility/gtk/text-in-span-block-in-a-block.html: Ditto.
- css3/blending/blend-mode-isolation-turn-off-self-painting-layer1.html: Ditto.
- css3/blending/blend-mode-simple.html: Ditto.
- css3/blending/effect-background-blend-mode.html: Ditto.
- css3/filters/blur-filter-page-scroll-parents.html: Ditto.
- css3/filters/blur-filter-page-scroll-self.html: Ditto.
- css3/filters/blur-filter-page-scroll.html: Ditto.
- css3/filters/effect-blur.html: Ditto.
- css3/filters/effect-brightness-clamping.html: Ditto.
- css3/filters/effect-brightness.html: Ditto.
- css3/filters/effect-combined.html: Ditto.
- css3/filters/effect-contrast.html: Ditto.
- css3/filters/effect-drop-shadow.html: Ditto.
- css3/filters/effect-grayscale.html: Ditto.
- css3/filters/effect-hue-rotate.html: Ditto.
- css3/filters/effect-invert.html: Ditto.
- css3/filters/effect-opacity.html: Ditto.
- css3/filters/effect-saturate.html: Ditto.
- css3/filters/effect-sepia.html: Ditto.
- css3/filters/filter-repaint.html: Ditto.
- css3/filters/null-effect-check.html: Ditto.
- editing/selection/caret-mode-document-begin-end.html: Ditto.
- editing/selection/caret-mode-paragraph-keys-navigation.html: Ditto.
- fast/animation/request-animation-frame-disabled.html: Ditto.
- fast/canvas/canvas-blend-image.html: Ditto.
- fast/canvas/canvas-blend-solid.html: Ditto.
- fast/canvas/canvas-imageSmoothingQuality.html: Ditto.
- fast/canvas/drawImage-with-small-values.html: Ditto.
- fast/images/exif-orientation-composited.html: Ditto.
- fast/images/exif-orientation-css.html: Ditto.
- fast/images/exif-orientation.html: Ditto.
- fast/images/image-controls-basic.html: Ditto.
- fast/parser/pre-html5-parser-quirks.html: Ditto.
- fast/repaint/no-caret-repaint-in-non-content-editable-element.html: Ditto.
- http/tests/appcache/disabled.html: Ditto.
- http/tests/contentextensions/block-cookies-in-ping.html: Ditto.
- http/tests/contentextensions/block-ping.html: Ditto.
- http/tests/contentextensions/hide-on-ping-with-ping-that-redirects.html: Ditto.
- http/tests/contentextensions/hide-on-ping.html: Ditto.
- http/tests/misc/favicon-loads-with-icon-loading-override.html: Ditto.
- http/tests/navigation/ping-attribute/anchor-cookie.html: Ditto.
- http/tests/navigation/ping-attribute/anchor-cross-origin-from-https.html: Ditto.
- http/tests/navigation/ping-attribute/anchor-cross-origin.html: Ditto.
- http/tests/navigation/ping-attribute/anchor-ping-and-follow-redirect-when-sending-ping.html: Ditto.
- http/tests/navigation/ping-attribute/anchor-same-origin.html: Ditto.
- http/tests/navigation/ping-attribute/area-cookie.html: Ditto.
- http/tests/navigation/ping-attribute/area-cross-origin-from-https.html: Ditto.
- http/tests/navigation/ping-attribute/area-cross-origin.html: Ditto.
- http/tests/navigation/ping-attribute/area-same-origin.html: Ditto.
- http/tests/security/mixedContent/insecure-script-in-data-iframe-in-main-frame-blocked.html: Ditto.
- imported/blink/fast/css/image-orientation/image-orientation-dynamic-expected.html: Ditto.
- loader/meta-refresh-disabled.html: Ditto.
- media/crash-closing-page-with-media-as-plugin-fallback.html: Ditto.
- platform/mac-wk1/editing/selection/caret-mode-document-begin-end-expected.txt: Removed.
Mac WK1 now produces the same output as other platforms.
- platform/mac/TestExpectations: Remove the failure for FrameFlattening now that the test
uses internals.settings.set*.
- platform/mac/fast/frames/flattening/set-preference.html: Use internals.settings.set*.
- plugins/crash-restoring-plugin-page-from-page-cache.html: Ditto.
- plugins/frameset-with-plugin-frame.html: Ditto.
- plugins/netscape-plugin-page-cache-works.html: Ditto.
- 11:05 AM Changeset in webkit [218753] by
-
- 4 edits in trunk/Source/WebKit2
Avoid moving the same vector multiple times
https://bugs.webkit.org/show_bug.cgi?id=173748
<rdar://problem/32936804>
Reviewed by Chris Dumez.
We discovered that a Vector<String> was being moved inside a loop, causing it to be moved more than once.
We should never do this!
- UIProcess/WebProcessProxy.cpp:
(WebKit::WebProcessProxy::deleteWebsiteDataForTopPrivatelyControlledDomainsInAllPersistentDataStores):
Do not perform a move at each step of the iteration.
- UIProcess/WebsiteData/WebsiteDataStore.cpp:
(WebKit::WebsiteDataStore::fetchDataForTopPrivatelyControlledDomains): Receive vector of top privately
controlled domains as a const reference. Copy this vector into the completion handler. Do not move
origins out of the vector in the inner loop.
(WebKit::WebsiteDataStore::removeDataForTopPrivatelyControlledDomains): Receive vector of top privately
controlled domains as a const reference.
- UIProcess/WebsiteData/WebsiteDataStore.h:
- 10:55 AM WebKitGTK/DependenciesPolicy edited by
- Policy now applies only to runtime dependencies, not to build toolchain (diff)
- 10:54 AM Changeset in webkit [218752] by
-
- 2 edits in trunk/Source/WebCore
Fix broken WinCairo build
[Curl] Extract CurlDownloadManager as shared background task handler
https://bugs.webkit.org/show_bug.cgi?id=173557
Patch by Basuke Suzuki <Basuke Suzuki> on 2017-06-23
Reviewed by Joseph Pecoraro.
- platform/network/curl/CurlManager.h:
- 10:33 AM Changeset in webkit [218751] by
-
- 4 edits in trunk/Source/WebCore
[iOS] Potential crash under WebCore::notifyLowPowerModeChanged(WebCore::LowPowerModeNotifier*, bool)
https://bugs.webkit.org/show_bug.cgi?id=173755
<rdar://problem/32940942>
Reviewed by Mark Lam.
The crash was happening because the WebLowPowerModeObserver would dispatch
a lambda to the main thread but the LowPowerModeNotifier object could be
dead by the time we get to the main thread.
To address the issue, keep a strong ref to the WebLowPowerModeObserver in
the lambda we dispatch to the main thread to make sure it stays alive until
we execute the lambda. In the LowPowerModeNotifier destructor, we now reset
the WebLowPowerModeObserver's notifier pointer to nil and I added a null
check for this notifier in the lambda.
- platform/LowPowerModeNotifier.cpp:
(WebCore::LowPowerModeNotifier::~LowPowerModeNotifier):
- platform/LowPowerModeNotifier.h:
- platform/ios/LowPowerModeNotifierIOS.mm:
(-[WebLowPowerModeObserver initWithNotifier:]):
(-[WebLowPowerModeObserver _didReceiveLowPowerModeChange]):
(WebCore::LowPowerModeNotifier::LowPowerModeNotifier):
(WebCore::LowPowerModeNotifier::~LowPowerModeNotifier):
(WebCore::notifyLowPowerModeChanged):
- 10:25 AM Changeset in webkit [218750] by
-
- 22 edits1 add in trunk
Add SPI to WKURLSchemeTask for redirection
https://bugs.webkit.org/show_bug.cgi?id=173730
Reviewed by Brady Eidson.
Source/WebCore:
- platform/network/CacheValidation.cpp:
(WebCore::computeFreshnessLifetimeForHTTPFamily):
Asserting that redirects are always http/https URLs is no longer valid.
If there's a custom scheme redirect, give it no freshness lifetime in the cache.
Source/WebKit2:
This is needed for testing an upcoming fix for redirect callbacks, and it might be
necessary for loading with custom schemes in general. Right now just responding with
an HTTP 301/302/307/308 response code doesn't work because there is nothing that
synthesizes an NSURLRequest from the Location header like we do in
SynchronousResourceHandleCFURLConnectionDelegate::willSendRequest et al. for HSTS,
and that would require using an NSHTTPURLResponse for non-HTTP responses, which is
conceptually wrong. Instead of waiting for a completion handler in the API, we are
following the pattern of WKNavigationDelegate.didReceiveServerRedirectForProvisionalNavigation
and allowing the SPI to indicate that a redirect has happened to update the state of
WebKit, but not allowing the SPI to wait for a processed request and slow down loading.
This adds WKURLSchemeTaskPrivate._didPerformRedirection which is covered by new API tests.
- UIProcess/API/Cocoa/WKURLSchemeTask.mm:
(raiseExceptionIfNecessary):
(-[WKURLSchemeTaskImpl _didPerformRedirection:newRequest:completionHandler:]):
- UIProcess/API/Cocoa/WKURLSchemeTaskInternal.h:
- UIProcess/API/Cocoa/WKURLSchemeTaskPrivate.h: Added.
- UIProcess/WebPageProxy.cpp:
(WebKit::WebPageProxy::startURLSchemeTask):
(WebKit::WebPageProxy::stopURLSchemeTask):
- UIProcess/WebPageProxy.h:
- UIProcess/WebPageProxy.messages.in:
- UIProcess/WebURLSchemeHandler.cpp:
(WebKit::WebURLSchemeHandler::startTask):
(WebKit::WebURLSchemeHandler::stopTask):
- UIProcess/WebURLSchemeHandler.h:
- UIProcess/WebURLSchemeTask.cpp:
(WebKit::WebURLSchemeTask::didReceiveResponse): Deleted.
(WebKit::WebURLSchemeTask::didReceiveData): Deleted.
(WebKit::WebURLSchemeTask::didComplete): Deleted.
- UIProcess/WebURLSchemeTask.h:
- WebKit2.xcodeproj/project.pbxproj:
- WebProcess/WebPage/WebPage.cpp:
(WebKit::WebPage::urlSchemeTaskDidPerformRedirection):
- WebProcess/WebPage/WebPage.h:
- WebProcess/WebPage/WebPage.messages.in:
- WebProcess/WebPage/WebURLSchemeHandlerProxy.cpp:
(WebKit::WebURLSchemeHandlerProxy::taskDidPerformRedirection):
- WebProcess/WebPage/WebURLSchemeHandlerProxy.h:
- WebProcess/WebPage/WebURLSchemeTaskProxy.cpp:
(WebKit::WebURLSchemeTaskProxy::didPerformRedirection):
- WebProcess/WebPage/WebURLSchemeTaskProxy.h:
Tools:
- TestWebKitAPI/Tests/WebKit2Cocoa/WKURLSchemeHandler-1.mm:
(-[SchemeHandler webView:startURLSchemeTask:]):
(-[SchemeHandler webView:stopURLSchemeTask:]):
(TEST):
(-[RedirectSchemeHandler webView:startURLSchemeTask:]):
(-[RedirectSchemeHandler webView:stopURLSchemeTask:]):
(-[RedirectSchemeHandler webView:didReceiveServerRedirectForProvisionalNavigation:]):
(-[RedirectSchemeHandler webView:decidePolicyForNavigationResponse:decisionHandler:]):
(-[RedirectSchemeHandler userContentController:didReceiveScriptMessage:]):
- 10:21 AM Changeset in webkit [218749] by
-
- 3 edits in trunk/Source/WebKit2
[WK2][macOS] Support Mac Mini Flash Player Features
https://bugs.webkit.org/show_bug.cgi?id=173581
<rdar://problem/32804073>
Reviewed by Dean Jackson.
Revise the sandboxes for additional IOKit properties identified during Flash video player
testing.
- PluginProcess/mac/com.apple.WebKit.plugin-common.sb.in:
- WebProcess/com.apple.WebProcess.sb.in:
- 9:11 AM Changeset in webkit [218748] by
-
- 57 edits in trunk/Source/WebCore
Remove excessive headers from WebCore/{editing,fileapi,history,html,loader,page}
https://bugs.webkit.org/show_bug.cgi?id=173769
Reviewed by Simon Fraser.
No new tests needed.
- css/CSSFontFaceSource.cpp:
- editing/DictationCommand.cpp:
- editing/Editor.h:
- editing/VisiblePosition.h:
- editing/VisibleUnits.h:
- fileapi/BlobBuilder.h:
- history/PageCache.h:
- html/DOMURL.h:
- html/HTMLCollection.h:
- html/HTMLMediaElement.cpp:
- html/HTMLMediaElement.h:
- html/HTMLPlugInImageElement.cpp:
- html/StepRange.h:
- html/canvas/CanvasRenderingContext.h:
- html/canvas/CanvasRenderingContext2D.h:
- html/parser/HTMLDocumentParser.cpp:
- html/parser/HTMLElementStack.h:
- html/parser/HTMLInputStream.h:
- html/shadow/MediaControlElements.cpp:
- html/shadow/MediaControls.cpp:
- html/shadow/MediaControls.h:
- html/track/TrackListBase.h:
- html/track/VTTCue.h:
- html/track/WebVTTParser.cpp:
- html/track/WebVTTParser.h:
- loader/CrossOriginPreflightChecker.cpp:
- loader/DocumentThreadableLoader.cpp:
- loader/FrameLoaderClient.h:
- loader/LinkPreloadResourceClients.h:
- loader/LoadTiming.h:
- loader/ThreadableLoaderClientWrapper.h:
- loader/WorkerThreadableLoader.h:
- loader/cache/CachedResourceLoader.h:
- loader/cache/CachedResourceRequest.h:
- loader/cache/CachedSVGDocument.h:
- loader/cache/CachedTextTrack.h:
- loader/icon/IconLoader.cpp:
- page/AlternativeTextClient.h:
- page/CaptionUserPreferences.cpp:
- page/CaptionUserPreferences.h:
- page/ContextMenuClient.h:
- page/MainFrame.h:
- page/Page.h:
- page/PageGroup.h:
- page/PerformanceEntry.h:
- page/PerformanceResourceTiming.h:
- page/ResourceUsageOverlay.cpp:
- page/ResourceUsageOverlay.h:
- page/UserContentController.h:
- page/UserContentProvider.h:
- page/animation/AnimationBase.h:
- page/animation/CSSPropertyAnimation.h:
- page/animation/CompositeAnimation.h:
- page/scrolling/ScrollingCoordinator.h:
- xml/XSLTProcessorLibxslt.cpp:
- xml/parser/XMLDocumentParserLibxml2.cpp:
- 8:44 AM Changeset in webkit [218747] by
-
- 9 edits in trunk
[mac-wk1] requestAnimationFrame callbacks may not get serviced
https://bugs.webkit.org/show_bug.cgi?id=173628
Reviewed by Simon Fraser.
Source/WebCore:
Page::setIsVisibleInternal() was firing the 'visibilitychange' event
synchronously while in the middle of updating its visibility/activity
state. This allowed the JavaScript to re-enter the method by calling
testRunner.setPageVisibility() / resetPageVisiblity() and we would
end up in an inconsistent state.
No new tests, extended existing test.
- dom/Document.cpp:
(WebCore::Document::visibilityStateChanged):
Do no fire the visibilitychange event synchronously as we are in the
middle of updating the page's activity state. Instead fire the
event asynchronously.
- page/Page.cpp:
(WebCore::Page::setIsVisibleInternal):
Move the calling of Document::visibilityStateChanged() until after we're
done updating the page's visibility state.
- testing/Internals.cpp:
(WebCore::Internals::scriptedAnimationsAreSuspended):
- testing/Internals.h:
- testing/Internals.idl:
Add test infrastructure to check if scripted animations are suspended.
LayoutTests:
Extend layout test coverage.
- fast/events/page-visibility-transition-test-expected.txt:
- fast/events/page-visibility-transition-test.html:
- 8:40 AM Changeset in webkit [218746] by
-
- 5 edits2 adds in trunk/Source/WebCore
[iOS] Respond to AudioSession interruption and resume
https://bugs.webkit.org/show_bug.cgi?id=173718
<rdar://problem/32925263>
Reviewed by Youenn Fablet.
Tested manually.
- WebCore.xcodeproj/project.pbxproj: Add CoreAudioCaptureSourceIOS.mm/.h
- platform/mediastream/RealtimeMediaSource.h: Make createWeakPtr protected so derived classes
can use it.
- platform/mediastream/ios/CoreAudioCaptureSourceIOS.h: Added.
- platform/mediastream/ios/CoreAudioCaptureSourceIOS.mm: Added.
(-[WebCoreAudioCaptureSourceIOSListener initWithCallback:]):
(-[WebCoreAudioCaptureSourceIOSListener invalidate]):
(-[WebCoreAudioCaptureSourceIOSListener handleInterruption:]):
(-[WebCoreAudioCaptureSourceIOSListener sessionMediaServicesWereReset:]):
(WebCore::CoreAudioCaptureSourceIOS::CoreAudioCaptureSourceIOS):
(WebCore::CoreAudioCaptureSourceIOS::~CoreAudioCaptureSourceIOS):
- platform/mediastream/mac/CoreAudioCaptureSource.cpp:
(WebCore::CoreAudioSharedUnit::suspended):
(WebCore::CoreAudioSharedUnit::setupAudioUnit): Clear m_suspended.
(WebCore::CoreAudioSharedUnit::suspend): Don't clear sources or uninitialize the audio unit,
suspend is temporary. Set m_suspended.
(WebCore::CoreAudioSharedUnit::resume): New, restart the audio unit.
(WebCore::CoreAudioCaptureSource::create): Create the correct object.
(WebCore::CoreAudioCaptureSource::scheduleReconfiguration): Dispatch to main thread if necessary.
(WebCore::CoreAudioCaptureSource::beginInterruption): New, suspend from main thread.
(WebCore::CoreAudioCaptureSource::endInterruption): New, resume/reconfigure from main thread.
(WebCore::CoreAudioCaptureSource::interrupted):
- platform/mediastream/mac/CoreAudioCaptureSource.h:
- 5:08 AM Changeset in webkit [218745] by
-
- 3 edits in trunk/Source/WebKit2
[CoordinatedGraphics] Clean up type aliases in CoordinatedGraphicsScene
https://bugs.webkit.org/show_bug.cgi?id=173764
Reviewed by Carlos Garcia Campos.
Drop the various type aliases created in the CoordinatedGraphicsScene
class. We can leave without these by leveragingautowhere necessary.
- Shared/CoordinatedGraphics/CoordinatedGraphicsScene.cpp:
(WebKit::CoordinatedGraphicsScene::updateTilesIfNeeded):
(WebKit::CoordinatedGraphicsScene::updateImageBacking):
(WebKit::CoordinatedGraphicsScene::clearImageBackingContents):
(WebKit::CoordinatedGraphicsScene::assignImageBackingToLayer):
- Shared/CoordinatedGraphics/CoordinatedGraphicsScene.h:
- 5:03 AM Changeset in webkit [218744] by
-
- 3 edits5 adds in trunk
[GCrypt] Drop the AES-CFB support
https://bugs.webkit.org/show_bug.cgi?id=173547
Reviewed by Jiewen Tan.
Source/WebCore:
No new tests -- relevant test baselines are updated.
Stop registering the AES-CFB algorithm as something that's supported by
the libgcrypt implementation. This algorithm was previously included in
the Web Crypto API specification, but has since been dropped from it.
Conveniently, libgcrypt only recently gained support for the CFB8 AES
cipher mode that's required by the specification, meaning we could only
support this algorithm with future releases of the libgcrypt library.
- crypto/gcrypt/CryptoAlgorithmRegistryGCrypt.cpp:
(WebCore::CryptoAlgorithmRegistry::platformRegisterAlgorithms):
LayoutTests:
Add GTK+-specific baselines for AES tests that also cover the CFB variation.
These tests still pass because errors are expected, it's just that errors now
occur because the algorithm isn't supported anymore, rather than because of
operation errors due to malformed parameters.
- platform/gtk/crypto/subtle/aes-cbc-cfb-decrypt-malformed-parameters-expected.txt: Added.
- platform/gtk/crypto/subtle/aes-cbc-cfb-encrypt-malformed-parameters-expected.txt: Added.
- platform/gtk/crypto/subtle/aes-export-key-malformed-parameters-expected.txt: Added.
- platform/gtk/crypto/subtle/aes-generate-key-malformed-parameters-expected.txt: Added.
- platform/gtk/crypto/subtle/aes-import-key-malformed-parameters-expected.txt: Added.
- 5:03 AM Changeset in webkit [218743] by
-
- 4 edits in trunk/Tools
[WPE] Use JSC API to send script messages from web extension in tests
https://bugs.webkit.org/show_bug.cgi?id=173757
Reviewed by Žan Doberšek.
The GTK+ uses DOM bindings to send the messages. We can just build the message string and use JSEvaluateScript
to run postMessage().
Fixes: /wpe/WebKitSecurityManager/file-xhr
/wpe/WebKitConsoleMessage/console-api
/wpe/WebKitConsoleMessage/js-exception
/wpe/WebKitConsoleMessage/network-error
/wpe/WebKitConsoleMessage/security-error
- TestWebKitAPI/Tests/WebKitGLib/TestConsoleMessage.cpp:
(testWebKitConsoleMessageSecurityError): Remove ifdefs.
(beforeAll): Ditto.
- TestWebKitAPI/Tests/WebKitGLib/TestWebKitWebContext.cpp:
(testWebContextSecurityFileXHR): Ditto.
(beforeAll): Ditto.
- TestWebKitAPI/Tests/WebKitGLib/WebExtensionTest.cpp:
(consoleMessageSentCallback): Use JSEvaluateScript in WPE. We need to escape the '"' in the message string.
(pageCreatedCallback): Remove ifdefs.
- 4:48 AM Changeset in webkit [218742] by
-
- 4 edits in trunk/Source/WTF
Remove more unused headers from WTF
https://bugs.webkit.org/show_bug.cgi?id=173761
Reviewed by Yusuke Suzuki.
- wtf/Liveness.h:
- wtf/PageAllocation.h:
- wtf/ParallelJobs.h:
- 4:14 AM Changeset in webkit [218741] by
-
- 4 edits in trunk/Source/WebKit2
[WPE] Crash in wpe_view_backend_get_renderer_host_fd
https://bugs.webkit.org/show_bug.cgi?id=173760
Reviewed by Žan Doberšek.
This can happen if views are created and destroyed quickly. The problem is that CompositingManagerProxy adds a
message receiver but never removes it, so it's possible that the message arrives right after the view has been
destroyed. This is happening most of the times running /wpe/WebKitAutomationSession/request-session that creates
3 web views in a row that are deleted when the test finishes.
Fixes /wpe/WebKitAutomationSession/request-session.
- UIProcess/API/wpe/CompositingManagerProxy.cpp:
(WebKit::CompositingManagerProxy::finalize): Remove the message handler.
- UIProcess/API/wpe/CompositingManagerProxy.h:
- UIProcess/API/wpe/WPEView.cpp:
(WKWPE::View::~View): Call CompositingManagerProxy::finalize().
- 4:11 AM Changeset in webkit [218740] by
-
- 18 edits in trunk
[WPE] User script messages don't work
https://bugs.webkit.org/show_bug.cgi?id=173712
Reviewed by Žan Doberšek.
.:
Enable user script messages.
- Source/cmake/OptionsWPE.cmake:
Source/WebKit2:
When a user script message is received we need to create a WebKitJavascriptResult to deserialize the script
value. WebKitJavascriptResult is created with a WebKitWebView for two reasons: to get its JavaScript global
context and to take a reference that ensures the JavaScript context will be alive while the results is
alive. However, we could create the WebKitJavascriptResult with the JavaScript global context and take a
reference of the context itself. This way we don't need to access the WebKitWebView from the
WebScriptMessageHandler client. But we still need access to the JavaScript context, so I've added a
javascriptGlobalContext() method to WebPageProxy for GTK+ and WPE that is implemented from the page client.
Fixes /wpe/WebKitUserContentManager/script-message-received.
- UIProcess/API/glib/WebKitJavascriptResult.cpp:
(_WebKitJavascriptResult::_WebKitJavascriptResult): Create the WebKitJavascriptResult with a JSGlobalContextRef
instead of a WEbKitWebView.
(webkitJavascriptResultCreate): Pass the js context to the constructor.
(webkit_javascript_result_get_global_context): Return the js context.
- UIProcess/API/glib/WebKitJavascriptResultPrivate.h:
- UIProcess/API/glib/WebKitUserContentManager.cpp:
- UIProcess/API/glib/WebKitWebView.cpp:
(_WebKitWebViewPrivate::~_WebKitWebViewPrivate): Use JSRetainPtr
(webkit_web_view_get_javascript_global_context):
(webkitWebViewRunJavaScriptCallback):
- UIProcess/API/gtk/PageClientImpl.cpp:
(WebKit::PageClientImpl::javascriptGlobalContext):
- UIProcess/API/gtk/PageClientImpl.h:
- UIProcess/API/wpe/APIViewClient.h:
(API::ViewClient::javascriptGlobalContext):
- UIProcess/API/wpe/PageClientImpl.cpp:
(WebKit::PageClientImpl::javascriptGlobalContext):
- UIProcess/API/wpe/PageClientImpl.h:
- UIProcess/API/wpe/WPEView.cpp:
(WKWPE::View::javascriptGlobalContext):
- UIProcess/API/wpe/WPEView.h:
- UIProcess/PageClient.h:
(WebKit::PageClient::javascriptGlobalContext):
- UIProcess/WebPageProxy.h:
- UIProcess/gtk/WebPageProxyGtk.cpp:
(WebKit::WebPageProxy::javascriptGlobalContext):
- UIProcess/wpe/WebPageProxyWPE.cpp:
(WebKit::WebPageProxy::javascriptGlobalContext):
- 4:08 AM Changeset in webkit [218739] by
-
- 4 edits in trunk
[WPE] Enable PUBLIC_SUFFIX_LIST
https://bugs.webkit.org/show_bug.cgi?id=173758
Reviewed by Žan Doberšek.
.:
WebsiteDataStore needs it when fetching data.
Fixes: /wpe/WebKitCookieManager/accept-policy
/wpe/WebKitCookieManager/delete-cookies
/wpe/WebKitCookieManager/cookies-changed
/wpe/WebKitCookieManager/persistent-storage
/wpe/WebKitCookieManager/ephemeral
/wpe/WebKitWebsiteData/ephemeral
/wpe/WebKitWebsiteData/cache
/wpe/WebKitWebsiteData/storage
/wpe/WebKitWebsiteData/databases
/wpe/WebKitWebsiteData/appcache
/wpe/WebKitWebsiteData/cookies
- Source/cmake/OptionsWPE.cmake:
Source/WebCore:
- PlatformWPE.cmake: Add PublicSuffixSoup.cpp to the build.
- 1:21 AM Changeset in webkit [218738] by
-
- 2 edits in trunk/Tools
Unreviewed. Fix WPE /wpe/WebKitWebView/javascript-dialogs and /wpe/WebKitWebView/window-properties
Skip onbeforeunload dialog tests for now in WPE since we don't have a implemention for
simulateUserInteraction(). And do not expect changes in "geometry" property of WebKitWindowProperties that
doesn't exist yet in WPE.
- TestWebKitAPI/Tests/WebKitGLib/TestUIClient.cpp:
(testWebViewJavaScriptDialogs):
(testWebViewWindowProperties):
