Timeline
Aug 1, 2015:
- 5:48 PM Changeset in webkit [187729] by
-
- 35 edits2 adds in trunk
[OS X] Font fallback is not language-sensitive
https://bugs.webkit.org/show_bug.cgi?id=147390
Reviewed by Dean Jackson.
Source/WebCore:
We need to make our font fallback code sensitive to locale.
Test: fast/text/fallback-language-han.html
- platform/graphics/mac/FontCacheMac.mm:
(WebCore::lookupCTFont):
(WebCore::FontCache::systemFallbackForCharacters):
LayoutTests:
Updating tests.
- css3/line-break/line-break-auto-centered-expected.html:
- css3/line-break/line-break-auto-centered.html:
- css3/line-break/line-break-auto-postfixes-expected.html:
- css3/line-break/line-break-auto-postfixes.html:
- css3/line-break/line-break-loose-centered-expected.html:
- css3/line-break/line-break-loose-centered.html:
- css3/line-break/line-break-loose-hyphens-expected.html:
- css3/line-break/line-break-loose-hyphens.html:
- css3/line-break/line-break-loose-postfixes-expected.html:
- css3/line-break/line-break-loose-postfixes.html:
- css3/line-break/line-break-loose-sound-marks-expected.html:
- css3/line-break/line-break-loose-sound-marks.html:
- css3/line-break/line-break-normal-centered-expected.html:
- css3/line-break/line-break-normal-centered.html:
- css3/line-break/line-break-normal-hyphens-expected.html:
- css3/line-break/line-break-normal-hyphens.html:
- css3/line-break/line-break-normal-postfixes-expected.html:
- css3/line-break/line-break-normal-postfixes.html:
- css3/line-break/line-break-normal-sound-marks-expected.html:
- css3/line-break/line-break-normal-sound-marks.html:
- css3/line-break/line-break-strict-centered-expected.html:
- css3/line-break/line-break-strict-centered.html:
- css3/line-break/line-break-strict-hyphens-expected.html:
- css3/line-break/line-break-strict-hyphens.html:
- css3/line-break/line-break-strict-postfixes-expected.html:
- css3/line-break/line-break-strict-postfixes.html:
- css3/line-break/line-break-strict-sound-marks-expected.html:
- css3/line-break/line-break-strict-sound-marks.html:
- fast/text/fallback-language-han-expected.html: Added.
- fast/text/fallback-language-han.html: Added.
- platform/efl/TestExpectations:
- platform/gtk/TestExpectations:
- platform/win/TestExpectations:
- 4:49 PM Changeset in webkit [187728] by
-
- 2 edits in trunk/Source/WebCore
CharacterFallbackMapKey should be locale-specific
https://bugs.webkit.org/show_bug.cgi?id=147532
Reviewed by Dean Jackson.
This is in preparation for locale-specific font fallback.
No new tests because there is no behavior change.
- platform/graphics/Font.cpp:
(WebCore::CharacterFallbackMapKey::CharacterFallbackMapKey):
(WebCore::CharacterFallbackMapKey::operator==):
(WebCore::CharacterFallbackMapKeyHash::hash):
(WebCore::Font::systemFallbackFontForCharacter):
- 4:39 PM Changeset in webkit [187727] by
-
- 5 edits in branches/safari-601.1-branch/Source
Versioning.
- 2:53 PM Changeset in webkit [187726] by
-
- 2 edits in trunk/Source/WebKit
Fix 64-bit CMake build on Windows.
- PlatformWin.cmake:
Win64 needs PaintHooks.asm.
- 1:43 PM Changeset in webkit [187725] by
-
- 2 edits in trunk/LayoutTests
[Win] Test gardening
Unreviewed
Patch by Myles C. Maxfield <mmaxfield@apple.com> on 2015-08-01
- platform/win/TestExpectations:
- 1:15 PM Changeset in webkit [187724] by
-
- 2 edits in trunk/Source/WebCore
Expand CharacterFallbackMapKey to a struct
https://bugs.webkit.org/show_bug.cgi?id=147530
Reviewed by Dean Jackson.
This is in prepraration for making this struct locale-specific.
No new tests because there is no behavior change.
- platform/graphics/Font.cpp:
(WebCore::CharacterFallbackMapKey::CharacterFallbackMapKey):
(WebCore::CharacterFallbackMapKey::isHashTableDeletedValue):
(WebCore::CharacterFallbackMapKey::operator==):
(WebCore::CharacterFallbackMapKeyHash::hash):
(WebCore::CharacterFallbackMapKeyHash::equal):
(WebCore::Font::systemFallbackFontForCharacter):
- 1:14 PM Changeset in webkit [187723] by
-
- 2 edits in trunk/Source/WTF
HashTraits<AtomicString> can use SimpleClassHashTraits
https://bugs.webkit.org/show_bug.cgi?id=147528
Reviewed by Andreas Kling.
No point in duplicating the code inside SimpleClassHashTraits.
- wtf/text/AtomicStringHash.h:
(WTF::HashTraits<WTF::AtomicString>::constructDeletedValue): Deleted.
(WTF::HashTraits<WTF::AtomicString>::isDeletedValue): Deleted.
- 1:08 PM Changeset in webkit [187722] by
-
- 2 edits in trunk/Source/WebCore
Fix the build after r187721
Unreviewed.
Patch by Myles C. Maxfield <mmaxfield@apple.com> on 2015-08-01
- bindings/js/JSDOMBinding.h:
(WebCore::createWrapper):
- 12:55 PM Changeset in webkit [187721] by
-
- 6 edits in trunk/Source
Fix compile errors with VS2015
https://bugs.webkit.org/show_bug.cgi?id=147526
Reviewed by Myles Maxfield.
Source/WebCore:
- bindings/js/JSDOMBinding.h:
(WebCore::createWrapper):
Explicitly call the Ref constructor to avoid unresolved symbols with VS2015.
Otherwise, VS2015 tries to use CSSPrimitiveValue::operator<Ref<CSSPrimitiveValue> > which doesn't exist.
Source/WebKit/win:
- WebKitDLL.cpp:
(shutDownWebKit):
(loadResourceIntoBuffer): Needs to be exported from WebKit.dll to link with CMake using VS2015.
Source/WTF:
- wtf/StringExtras.h:
(snprintf):
(wtf_vsnprintf):
VS2015 has an implementation of snprintf.
- 11:16 AM Changeset in webkit [187720] by
-
- 1 copy in tags/Safari-601.1.46
New tag.
- 10:11 AM WebKitGTK/2.8.x edited by
- Propose some merges (diff)
- 10:06 AM Changeset in webkit [187719] by
-
- 2 edits in branches/safari-601.1-branch/Source/WebCore
Merged r187686. rdar://problem/22083354
- 8:34 AM Changeset in webkit [187718] by
-
- 2 edits in branches/safari-601.1-branch/Source/WebCore
Merged r187687. rdar://problem/21939014
- 8:27 AM Changeset in webkit [187717] by
-
- 2 edits in trunk/Source/JavaScriptCore
Unreviewed. Remove unintentional "print" statement in test case.
https://bugs.webkit.org/show_bug.cgi?id=142567
- tests/stress/class-syntax-definition-semantics.js:
(shouldBeSyntaxError):
- 8:23 AM Changeset in webkit [187716] by
-
- 5 edits in branches/safari-601.1-branch/Source
Versioning.
- 3:55 AM Changeset in webkit [187715] by
-
- 3 edits in trunk/Source/WebInspectorUI
Web Inspector: Replace "unknown-mac" CSS class with "el-capitan"
https://bugs.webkit.org/show_bug.cgi?id=147524
Reviewed by Timothy Hatcher.
- UserInterface/Base/Platform.js:
No need to repeat WebInspector.Platform.version.release for every possible case.
- UserInterface/Views/Toolbar.css:
(body:not(.el-capitan) .toolbar .dashboard-container):
(body:not(.el-capitan) .toolbar .search-bar > input[type="search"]):
(body:not(.el-capitan) .toolbar .search-bar > input[type="search"]:focus):
(body:not(.el-capitan) .toolbar .item.button:active):
(body.window-inactive:not(.el-capitan) .toolbar .dashboard-container):
(body.el-capitan .toolbar .dashboard-container):
(body.el-capitan .toolbar .search-bar > input[type="search"]):
(body.el-capitan .toolbar .search-bar > input[type="search"]:focus):
(@media (-webkit-min-device-pixel-ratio: 2)):
(body.el-capitan .toolbar .item.button:active):
(body.el-capitan.window-inactive .toolbar .dashboard-container):
(body:not(.unknown-mac) .toolbar .dashboard-container): Deleted.
(body:not(.unknown-mac) .toolbar .search-bar > input[type="search"]): Deleted.
(body:not(.unknown-mac) .toolbar .search-bar > input[type="search"]:focus): Deleted.
(body:not(.unknown-mac) .toolbar .item.button:active): Deleted.
(body.window-inactive:not(.unknown-mac) .toolbar .dashboard-container): Deleted.
(body.unknown-mac .toolbar .dashboard-container): Deleted.
(body.unknown-mac .toolbar .search-bar > input[type="search"]): Deleted.
(body.unknown-mac .toolbar .search-bar > input[type="search"]:focus): Deleted.
(body.unknown-mac .toolbar .item.button:active): Deleted.
(body.unknown-mac.window-inactive .toolbar .dashboard-container): Deleted.
Jul 31, 2015:
- 11:48 PM Changeset in webkit [187714] by
-
- 5 edits in trunk/Source/WebInspectorUI
Web Inspector: inherited CSS rules disappear from Styles sidebar while editing
https://bugs.webkit.org/show_bug.cgi?id=147441
Reviewed by Timothy Hatcher.
If the user edits a style declaration such that it would dissapear mid-edit, prevent
the rules panel from refreshing until that editor is blurred.
- UserInterface/Views/CSSStyleDeclarationSection.js:
(WebInspector.CSSStyleDeclarationSection):
(WebInspector.CSSStyleDeclarationSection.prototype.get editorActive):
(WebInspector.CSSStyleDeclarationSection.prototype.get _hasInvalidSelector):
(WebInspector.CSSStyleDeclarationSection.prototype._editorContentChanged):
(WebInspector.CSSStyleDeclarationSection.prototype._editorBlurred):
- UserInterface/Views/CSSStyleDeclarationTextEditor.js:
(WebInspector.CSSStyleDeclarationTextEditor.prototype._editorBlured):
(WebInspector.CSSStyleDeclarationTextEditor.prototype._contentChanged):
- UserInterface/Views/RulesStyleDetailsPanel.js:
(WebInspector.RulesStyleDetailsPanel):
(WebInspector.RulesStyleDetailsPanel.prototype._removeSectionWithActiveEditor):
- UserInterface/Views/StyleDetailsPanel.js:
(WebInspector.StyleDetailsPanel.prototype._refreshPreservingScrollPosition):
- 11:47 PM Changeset in webkit [187713] by
-
- 9 edits in trunk/Source
Media Session: inform the UI process when the focused Content media element changes
https://bugs.webkit.org/show_bug.cgi?id=147323
Reviewed by Tim Horton.
WebCore:
- html/HTMLMediaElement.cpp:
(WebCore::HTMLMediaElement::playInternal): Eventually tell the UI process that the focused Content media element changed.
- page/ChromeClient.h:
WebKit2:
- UIProcess/WebPageProxy.cpp:
(WebKit::WebPageProxy::isPlayingMediaDidChange):
- UIProcess/WebPageProxy.h:
- UIProcess/WebPageProxy.messages.in:
- WebProcess/WebCoreSupport/WebChromeClient.cpp:
(WebKit::WebChromeClient::focusedContentMediaElementDidChange):
- WebProcess/WebCoreSupport/WebChromeClient.h:
- 11:39 PM Changeset in webkit [187712] by
-
- 3 edits in trunk/Source/WebCore
Media Session: give media elements unique IDs https://bugs.webkit.org/show_bug.cgi?id=147322
Reviewed by Eric Carlson.
- html/HTMLMediaElement.cpp:
(WebCore::elementIDsToElements): Create a global map of element IDs to elements.
(WebCore::HTMLMediaElement::elementWithID): Retrieve the element with the given ID.
(WebCore::HTMLMediaElement::HTMLMediaElement): Give the element a unique ID.
(WebCore::HTMLMediaElement::playInternal): Removed whitespace.
(WebCore::HTMLMediaElement::pauseInternal): Ditto.
- html/HTMLMediaElement.h:
(WebCore::HTMLMediaElement::elementID):
- 11:27 PM Changeset in webkit [187711] by
-
- 1 edit1 add in trunk/LayoutTests
Adding -expected.png for fast/text/international/arabic-justify.html
- platform/mac-yosemite/fast/text/international/arabic-justify-expected.png: Added
- 10:33 PM Changeset in webkit [187710] by
-
- 1 edit2 adds in trunk/LayoutTests
Media Session: add test to ensure that switching media sessions pauses elements
https://bugs.webkit.org/show_bug.cgi?id=147493
Reviewed by Eric Carlson.
- media/session/switching-sessions-expected.txt: Added.
- media/session/switching-sessions.html: Added.
- platform/mac/TestExpectations: Media Session support is disabled by default.
- 9:27 PM Changeset in webkit [187709] by
-
- 2 edits in trunk/Source/WebCore
Fix the build
Unreviewed.
- platform/graphics/mac/FontCacheMac.mm:
(WebCore::lookupCTFont):
- 9:15 PM Changeset in webkit [187708] by
-
- 2 edits in trunk/Source/WebInspectorUI
Web Inspector: Autocomplete: Undo (Cmd+Z) doesn't work as expected
https://bugs.webkit.org/show_bug.cgi?id=147316
Reviewed by Timothy Hatcher.
Instead of replacing the text for a completion, which messes up the undo history, add
a unique marker that contains the remaining text for the current completion.
- UserInterface/Controllers/CodeMirrorCompletionController.js:
(WebInspector.CodeMirrorCompletionController.prototype._createCompletionHintMarker):
(WebInspector.CodeMirrorCompletionController.prototype._applyCompletionHint.update):
(WebInspector.CodeMirrorCompletionController.prototype._applyCompletionHint):
(WebInspector.CodeMirrorCompletionController.prototype._removeCompletionHint.update):
(WebInspector.CodeMirrorCompletionController.prototype._removeCompletionHint):
- 9:06 PM Changeset in webkit [187707] by
-
- 6 edits in trunk/Source/WebCore
[OS X] Migrate to CTFontCreateForCharactersWithLanguage from [NSFont findFontLike:forString:withRange:inLanguage]
https://bugs.webkit.org/show_bug.cgi?id=147483
Reviewed by Dean Jackson.
[NSFont findFontLike:forString:withRange:inLanguage] doesn't properly handle its last argument. In
addition, we want to be moving away from NSFont in the first place and on to Core Text. This new
CoreText function correctly handles its language argument, which is required for language-specific
font fallback.
No new tests because there is no behavior change.
- platform/graphics/FontCache.cpp:
(WebCore::FontCache::purgeInactiveFontData):
- platform/graphics/FontCache.h:
(WebCore::FontCache::platformPurgeInactiveFontData):
- platform/graphics/mac/FontCacheMac.mm:
(WebCore::fallbackDedupSet):
(WebCore::FontCache::platformPurgeInactiveFontData):
(WebCore::lookupCTFont):
(WebCore::FontCache::systemFallbackForCharacters):
- platform/spi/cocoa/CoreTextSPI.h:
- platform/spi/mac/NSFontSPI.h:
- 8:41 PM Changeset in webkit [187706] by
-
- 31 edits in trunk/Source
Drop dummy Timer callbacks
https://bugs.webkit.org/show_bug.cgi?id=147516
Reviewed by Alexey Proskuryakov.
Drop dummy Timer callbacks in the code base. A lot of Timer callbacks
do nothing else beside calling a particular method. In such case, we
can use the called method as Timer callback instead.
Source/WebCore:
- css/FontLoader.cpp:
(WebCore::FontLoader::FontLoader):
- css/FontLoader.h:
- dom/Document.cpp:
(WebCore::Document::Document):
(WebCore::Document::styleRecalcTimerFired): Deleted.
(WebCore::Document::domCookieCacheExpiryTimerFired): Deleted.
- dom/Document.h:
- html/MediaController.cpp:
(MediaController::MediaController):
(MediaController::timeupdateTimerFired): Deleted.
- html/MediaController.h:
- loader/cache/CachedResource.cpp:
(WebCore::CachedResource::CachedResource):
- loader/cache/CachedResourceLoader.cpp:
(WebCore::CachedResourceLoader::CachedResourceLoader):
(WebCore::CachedResourceLoader::garbageCollectDocumentResourcesTimerFired): Deleted.
- loader/cache/CachedResourceLoader.h:
- loader/cache/MemoryCache.cpp:
(WebCore::MemoryCache::MemoryCache):
(WebCore::MemoryCache::pruneTimerFired): Deleted.
- loader/cache/MemoryCache.h:
- page/EventSource.cpp:
(WebCore::EventSource::EventSource):
(WebCore::EventSource::connectTimerFired): Deleted.
- page/EventSource.h:
- page/FrameView.cpp:
(WebCore::FrameView::FrameView):
(WebCore::FrameView::delayedScrollEventTimerFired): Deleted.
(WebCore::FrameView::postLayoutTimerFired): Deleted.
- page/FrameView.h:
- page/scrolling/ios/ScrollingCoordinatorIOS.h:
- page/scrolling/ios/ScrollingCoordinatorIOS.mm:
(WebCore::ScrollingCoordinatorIOS::ScrollingCoordinatorIOS):
(WebCore::ScrollingCoordinatorIOS::scrollingStateTreeCommitterTimerFired): Deleted.
- page/scrolling/mac/ScrollingCoordinatorMac.h:
- page/scrolling/mac/ScrollingCoordinatorMac.mm:
(WebCore::ScrollingCoordinatorMac::ScrollingCoordinatorMac):
(WebCore::ScrollingCoordinatorMac::scrollingStateTreeCommitterTimerFired): Deleted.
- platform/graphics/FontCache.cpp:
(WebCore::FontCache::FontCache):
(WebCore::FontCache::purgeTimerFired): Deleted.
- platform/graphics/FontCache.h:
- platform/graphics/ShadowBlur.cpp:
(WebCore::ScratchBuffer::ScratchBuffer):
(WebCore::ScratchBuffer::timerFired): Deleted.
- platform/graphics/avfoundation/objc/MediaPlayerPrivateMediaSourceAVFObjC.h:
- platform/graphics/avfoundation/objc/MediaPlayerPrivateMediaSourceAVFObjC.mm:
(WebCore::MediaPlayerPrivateMediaSourceAVFObjC::MediaPlayerPrivateMediaSourceAVFObjC):
(WebCore::MediaPlayerPrivateMediaSourceAVFObjC::seekTimerFired): Deleted.
Source/WebKit/mac:
- Plugins/WebNetscapePluginStream.h:
- Plugins/WebNetscapePluginStream.mm:
(WebNetscapePluginStream::WebNetscapePluginStream):
(WebNetscapePluginStream::deliverDataTimerFired): Deleted.
Source/WebKit2:
- WebProcess/WebPage/mac/RemoteLayerTreeDrawingArea.h:
- WebProcess/WebPage/mac/RemoteLayerTreeDrawingArea.mm:
(WebKit::RemoteLayerTreeDrawingArea::RemoteLayerTreeDrawingArea):
(WebKit::RemoteLayerTreeDrawingArea::layerFlushTimerFired): Deleted.
- 8:03 PM Changeset in webkit [187705] by
-
- 1 copy in tags/Safari-601.1.45
New tag.
- 7:39 PM Changeset in webkit [187704] by
-
- 2 edits in trunk/Source/WebCore
Removed pointSize from _addMarkersToList
https://bugs.webkit.org/show_bug.cgi?id=147515
Reviewed by Eric Carlson.
- editing/cocoa/HTMLConverter.mm:
(HTMLConverter::_addMarkersToList): Deleted.
- 7:39 PM Changeset in webkit [187703] by
-
- 4 edits in branches/safari-601.1-branch/Source/WebCore
Roll out r187466. rdar://problem/21960398
- 6:56 PM Changeset in webkit [187702] by
-
- 8 edits in trunk
Prepare for VS2015
https://bugs.webkit.org/show_bug.cgi?id=146579
Reviewed by Jon Honeycutt.
Source/JavaScriptCore:
- heap/Heap.h:
Fix compiler error by explicitly casting zombifiedBits to the size of a pointer.
Source/WebCore:
- WebCore.vcxproj/build-generated-files.pl:
Find a correct preprocessor when using VS2015.
Source/WTF:
- wtf/StdLibExtras.h:
(std::exchange):
Added preprocessor macros around functions implemented in VS2015's standard library.
Tools:
- WinLauncher/stdafx.h:
Define STDC_LIMIT_MACROS to have access to INTMAX_MAX.
- 6:36 PM Changeset in webkit [187701] by
-
- 4 edits in trunk/LayoutTests
Unreviewed gardening. Skip Media Sessions tests, the feature is not yet enabled.
- TestExpectations:
- platform/efl/TestExpectations:
- platform/mac/TestExpectations:
- 6:32 PM Changeset in webkit [187700] by
-
- 2 edits in trunk/Source/WebInspectorUI
Web Inspector: Scrolling "jumps" in console
https://bugs.webkit.org/show_bug.cgi?id=147482
Reviewed by Joseph Pecoraro.
Removed focus call on messageElement.
- UserInterface/Views/LogContentView.js:
(WebInspector.LogContentView):
(WebInspector.LogContentView.prototype._mousedown): Deleted.
(WebInspector.LogContentView.prototype._didFocus): Deleted.
(WebInspector.LogContentView.prototype._didBlur): Deleted.
- 6:22 PM Changeset in webkit [187699] by
-
- 4 edits in trunk/LayoutTests
Unreviewed gardening. Update inspector test skips based on bots.
- TestExpectations:
- platform/gtk/TestExpectations:
- platform/win/TestExpectations:
- 5:43 PM Changeset in webkit [187698] by
-
- 4 edits in trunk/Source/WebKit2
[WK2][NetworkCache] Drop HTTP method from NetworkCache::Key
https://bugs.webkit.org/show_bug.cgi?id=143348
Reviewed by Antti Koivisto.
Drop HTTP method from NetworkCache::Key as we only cache GET responses
for now. Even when we start caching HEAD responses, we likely will not
want the method to be part of the key because:
- A HEAD response can be used to update the headers of a previously cached response to GET
- A cached GET response may be used to satisfy subsequent HEAD requests
- NetworkProcess/cache/NetworkCache.cpp:
(WebKit::NetworkCache::makeCacheKey):
- NetworkProcess/cache/NetworkCacheKey.cpp:
(WebKit::NetworkCache::Key::Key):
(WebKit::NetworkCache::Key::operator==):
(WebKit::NetworkCache::Key::decode):
(WebKit::NetworkCache::Key::operator=): Deleted.
(WebKit::NetworkCache::Key::computeHash): Deleted.
(WebKit::NetworkCache::Key::encode): Deleted.
- NetworkProcess/cache/NetworkCacheKey.h:
(WebKit::NetworkCache::Key::method): Deleted.
- 5:29 PM Changeset in webkit [187697] by
-
- 3 edits in branches/safari-601.1-branch/Source/WebCore
Merged r187694. rdar://problem/22015509
- 5:27 PM Changeset in webkit [187696] by
-
- 6 edits in branches/safari-601.1-branch/Source/WebCore
Merged r187688. rdar://problem/22015509
- 5:19 PM Changeset in webkit [187695] by
-
- 3 edits in branches/safari-601.1-branch/Source/WebCore
Merged r187684. rdar://problem/22000865
- 5:09 PM Changeset in webkit [187694] by
-
- 3 edits in trunk/Source/WebCore
[iOS] use a media-specific user gesture check
https://bugs.webkit.org/show_bug.cgi?id=147509
Reviewed by Tim Horton.
Change an enum name added in r187688.
- dom/UserGestureIndicator.cpp:
(WebCore::isDefinite): DefinitelyProcessingMediaUserGesture -> DefinitelyProcessingPotentialUserGesture
(WebCore::UserGestureIndicator::processingUserGestureForMedia): Ditto.
- dom/UserGestureIndicator.h: Ditto.
- 5:00 PM Changeset in webkit [187693] by
-
- 2 edits in trunk/Source/WebCore
[Cocoa] Latin quotes are used with the system font on Chinese devices
https://bugs.webkit.org/show_bug.cgi?id=147504
Reviewed by Dean Jackson.
The system font has some fancy logic regarding character selection which requires
using Core Text for glyph selection.
No new tests because tests can't change the system language of the device.
- platform/graphics/mac/GlyphPageMac.cpp:
(WebCore::shouldUseCoreText):
- 4:57 PM Changeset in webkit [187692] by
-
- 10 edits1 move1 add in trunk
Web Inspector: Unskip / Unflake inspector tests after r187627
https://bugs.webkit.org/show_bug.cgi?id=147503
Reviewed by Brian Burg.
Source/WebInspectorUI:
- UserInterface/Base/Test.js:
(InspectorTest.completeTest):
(InspectorTest.completeTest.signalCompletionToTestPage): Deleted.
(InspectorTest.testPageDidLoad):
Instead of immediately resending results if completeTest was called
during a reload before the load has completed, we just set a flag
to complete the test when the load completes. This consistently
resends the output to the new page for test output.
(InspectorTest._resendResults):
Delete clear results. If the page had logged anything directly on
the page side (using InspectorTestProxy) this clearMessages would
have deleted those messages and not been able to recreate them.
Clearing messages wasn't really serving a purpose.
LayoutTests:
- TestExpectations:
- platform/mac-wk2/TestExpectations:
- platform/mac/TestExpectations:
Unskip and unflake inspector tests.
- http/tests/inspector/inspector-test.js:
(InspectorTestProxy.addResult):
Simplify with the new append API.
(InspectorTestProxy.clearResults): Deleted.
Delete, as it is no longer used.
- inspector/debugger/probe-manager-add-remove-actions-expected.txt:
Include output that was getting erronously cleared by clearResults
which we removed.
- inspector/dom/highlightSelector-expected.txt:
- inspector/dom/highlightSelector.html:
Make this test more reliable when run with other tests.
Never assume the frame id for the main frame or sub frame,
instead compute them and mask them in the output.
- inspector/dom/resources/highlightSelector-iframe.html: Renamed from LayoutTests/inspector/dom/highlightSelector-iframe.html.
Move the non-test iframe html resource into a sub-folder
so it won't be treated like an actual test.
- 4:56 PM Changeset in webkit [187691] by
-
- 5 edits in trunk/Source
Source/WebCore:
Coalesce authentication credential requests
https://bugs.webkit.org/show_bug.cgi?id=128006
<rdar://problem/16839069>
Reviewed by Alexey Proskuryakov.
Export symbol for ProtectionSpace::compare() so it can be called from
WebKit2.
- platform/network/ProtectionSpaceBase.h:
Source/WebKit2:
Coalesce concurrent authentication challenges for the same ProtectionSpace
https://bugs.webkit.org/show_bug.cgi?id=147496
<rdar://problem/16839069>
Reviewed by Alexey Proskuryakov.
Coalesce concurrent authentication challenges for the same ProtectionSpace.
Ideally, this would be done in the network layer but short term, this will
make sure the user no longer gets repeating authentication dialogs for the
same protection space.
This gets rid of a long-standing bug in WebKit / Safari where visiting a
site using HTTP authentication would pop up one authentication dialog for
each subresource being loaded from the network (especially when the main
resource is loaded from the cache).
- Shared/Authentication/AuthenticationManager.cpp:
(WebKit::canCoalesceChallenge):
(WebKit::AuthenticationManager::addChallengeToChallengeMap):
(WebKit::AuthenticationManager::shouldCoalesceChallenge):
(WebKit::AuthenticationManager::coalesceChallengesMatching):
(WebKit::AuthenticationManager::didReceiveAuthenticationChallenge):
(WebKit::AuthenticationManager::useCredentialForChallenge):
(WebKit::AuthenticationManager::useCredentialForSingleChallenge):
(WebKit::AuthenticationManager::continueWithoutCredentialForChallenge):
(WebKit::AuthenticationManager::continueWithoutCredentialForSingleChallenge):
(WebKit::AuthenticationManager::cancelChallenge):
(WebKit::AuthenticationManager::cancelSingleChallenge):
(WebKit::AuthenticationManager::performDefaultHandling):
(WebKit::AuthenticationManager::performDefaultHandlingForSingleChallenge):
(WebKit::AuthenticationManager::rejectProtectionSpaceAndContinue):
(WebKit::AuthenticationManager::rejectProtectionSpaceAndContinueForSingleChallenge):
(WebKit::AuthenticationManager::AuthenticationManager): Deleted.
- Shared/Authentication/AuthenticationManager.h:
- 4:50 PM Changeset in webkit [187690] by
-
- 2 edits in trunk/LayoutTests
Unreviewed gardening. Skip more inspector tests on Windows.
It may be that no inspector tests are working on Windows,
if that is the case we may move to skipping all of them
and investigate unskipping in chunks.
- platform/win/TestExpectations:
- 4:27 PM Changeset in webkit [187689] by
-
- 20 edits in trunk/Source/WebInspectorUI
Web Inspector: Convert timeline view classes to ES6 classes
https://bugs.webkit.org/show_bug.cgi?id=147381
Reviewed by Joseph Pecoraro.
Along the way, inline a few style class names.
Elided mechanical changes from the Changelog.
- UserInterface/Views/LayoutTimelineOverviewGraph.js:
- UserInterface/Views/LayoutTimelineView.js:
- UserInterface/Views/LinearTimelineOverview.js:
- UserInterface/Views/NetworkTimelineOverviewGraph.js:
- UserInterface/Views/NetworkTimelineView.js:
- UserInterface/Views/OverviewTimelineView.js:
- UserInterface/Views/RenderingFrameTimelineOverview.js:
- UserInterface/Views/RenderingFrameTimelineOverviewGraph.js:
- UserInterface/Views/RenderingFrameTimelineView.js:
- UserInterface/Views/ScriptTimelineOverviewGraph.js:
- UserInterface/Views/ScriptTimelineView.js:
- UserInterface/Views/TimelineOverview.js:
- UserInterface/Views/TimelineOverviewGraph.js:
- UserInterface/Views/TimelineRecordBar.js:
- UserInterface/Views/TimelineRecordFrame.js:
- UserInterface/Views/TimelineRecordingContentView.js:
- UserInterface/Views/TimelineRuler.js:
- UserInterface/Views/TimelineTabContentView.js:
- UserInterface/Views/TimelineView.js:
- 3:59 PM Changeset in webkit [187688] by
-
- 6 edits in trunk/Source/WebCore
[iOS] use a media-specific user gesture check
https://bugs.webkit.org/show_bug.cgi?id=147509
Reviewed by Jer Noble.
- bindings/js/ScriptController.cpp:
(WebCore::ScriptController::processingUserGestureForMedia): New.
- bindings/js/ScriptController.h:
- dom/UserGestureIndicator.cpp:
(WebCore::isDefinite): Allow DefinitelyProcessingMediaUserGesture.
(WebCore::UserGestureIndicator::processingUserGestureForMedia): New.
- dom/UserGestureIndicator.h:
- html/HTMLMediaElement.cpp:
(WebCore::HTMLMediaElement::load): Use ScriptController::processingUserGestureForMedia instead of
ScriptController::processingUserGesture
(WebCore::HTMLMediaElement::play): Ditto.
- 3:43 PM Changeset in webkit [187687] by
-
- 2 edits in trunk/Source/WebCore
Crashes under HTMLMediaElement::updateActiveTextTrackCues() when destroying CachedPage.
<https://webkit.org/b/147506>
<rdar://problem/21939014>
Reviewed by Chris Dumez.
Don't mess with the media element's text tracks below its ActiveDOMObject::stop()
implementation, since that may cause DOM mutations.
I don't have a repro or a test for this, but plenty of crash logs to indicate that
we're getting ourselves into trouble by modifying the DOM during CachedPage teardown.
- html/HTMLMediaElement.cpp:
(WebCore::HTMLMediaElement::configureTextTrackDisplay):
- 3:34 PM Changeset in webkit [187686] by
-
- 2 edits in trunk/Source/WebCore
[iOS] DOMNode preview snapshot rects are wrong for user-select: none links
https://bugs.webkit.org/show_bug.cgi?id=147513
<rdar://problem/22083354>
Reviewed by Simon Fraser.
- bindings/objc/DOM.mm:
(-[DOMNode getPreviewSnapshotImage:andRects:]):
Use the same code as WebKit2 to compute the fallback rect (if TextIndicator fails),
asking the RenderObject (or RenderImage) for its bounding box instead of using the
(often wrong) Range bounding rect.
Make sure to use the fallback rect *any* time TextIndicator fails (before
we would return no rects at all if TextIndicator::createWithRange returned null,
and the fallback rect if it returned with an empty image).
Inverse-page-scale the margin, to match the appearance in WebKit2.
- 3:24 PM Changeset in webkit [187685] by
-
- 3 edits3 adds in trunk
[SVG -> OTF Converter] Crash when converting Arabic fonts
https://bugs.webkit.org/show_bug.cgi?id=147510
Reviewed by Anders Carlsson.
Source/WebCore:
SVGToOTFFontConverter::compareCodepointsLexicographically() wasn't transitive.
Test: fast/text/arabic-duplicate-glyph-font.html
- svg/SVGToOTFFontConversion.cpp:
(WebCore::SVGToOTFFontConverter::compareCodepointsLexicographically):
LayoutTests:
- fast/text/arabic-duplicate-glyph-font-expected.txt: Added.
- fast/text/arabic-duplicate-glyph-font.html: Added.
- fast/text/resources/arabic-duplicate-glyph-font.svg: Added.
- 3:20 PM Changeset in webkit [187684] by
-
- 3 edits in trunk/Source/WebCore
Rename AVPlayerLayerView to _AVPlayerLayerView.
https://bugs.webkit.org/show_bug.cgi?id=147399
Patch by Jeremy Jones <jeremyj@apple.com> on 2015-07-31
Reviewed by Eric Carlson.
Change class name AVPlayerLayerView to match change in AVKit SPI.
This prevents conflicts with 3rd party apps.
- platform/ios/WebVideoFullscreenInterfaceAVKit.mm:
(WebAVPlayerLayerView_dealloc):
(getWebAVPlayerLayerViewClass):
- platform/spi/cocoa/AVKitSPI.h:
- 2:40 PM Changeset in webkit [187683] by
-
- 2 edits in branches/jsc-tailcall/Source/JavaScriptCore
jsc-tailcall: REGRESSION(r187676): Broke stress/tail-call-no-stack-overflow.js
https://bugs.webkit.org/show_bug.cgi?id=147507
Reviewed by Basile Clement.
Changed strictLoopArityFixup1 and strictLoopArityFixup2 to recursively call
themselves instead of nonexistent strictLoopArityFixup.
- tests/stress/tail-call-no-stack-overflow.js:
(strictLoopArityFixup1):
(strictLoopArityFixup2):
- 2:26 PM Changeset in webkit [187682] by
-
- 2 edits in trunk/Source/WebKit2
Unreviewed parameter name typo fix.
- UIProcess/API/Cocoa/WKPreferences.mm:
(-[WKPreferences _setHiddenPageDOMTimerThrottlingEnabled:]):
- 2:16 PM Changeset in webkit [187681] by
-
- 3 edits in trunk/Source/WebCore
ImageBufferDataCG should not add IOSurfaces to the pool
https://bugs.webkit.org/show_bug.cgi?id=147494
Reviewed by Tim Horton.
ImageBufferDataCG should not add IOSurfaces to the pool as it is not
safe. IOSurfaceIsInUse() does not return accurate result for those
surfaces.
This is a partial revert of r181301.
- platform/graphics/cg/ImageBufferDataCG.cpp:
(WebCore::ImageBufferData::~ImageBufferData): Deleted.
- platform/graphics/cg/ImageBufferDataCG.h:
- 2:05 PM Changeset in webkit [187680] by
-
- 25 edits2 adds in trunk
ES6 class syntax should use block scoping
https://bugs.webkit.org/show_bug.cgi?id=142567
Reviewed by Geoffrey Garen.
Source/JavaScriptCore:
We treat class declarations like we do "let" declarations.
The class name is under TDZ until the class declaration
statement is evaluated. Class declarations also follow
the same rules as "let": No duplicate definitions inside
a lexical environment.
- parser/ASTBuilder.h:
(JSC::ASTBuilder::createClassDeclStatement):
- parser/Parser.cpp:
(JSC::Parser<LexerType>::parseClassDeclaration):
- tests/stress/class-syntax-block-scoping.js: Added.
(assert):
(truth):
(.):
- tests/stress/class-syntax-definition-semantics.js: Added.
(shouldBeSyntaxError):
(shouldNotBeSyntaxError):
(truth):
- tests/stress/class-syntax-tdz.js:
(assert):
(shouldThrowTDZ):
(truth):
(.):
LayoutTests:
- js/class-constructor-return-expected.txt:
- js/class-syntax-call-expected.txt:
- js/class-syntax-declaration-expected.txt:
- js/class-syntax-default-constructor-expected.txt:
- js/class-syntax-extends-expected.txt:
- js/class-syntax-name-expected.txt:
- js/class-syntax-super-expected.txt:
- js/script-tests/class-constructor-return.js:
(shouldThrow):
(shouldNotThrow):
(shouldBeTrue):
(shouldBeFalse):
- js/script-tests/class-syntax-call.js:
(A):
(B):
(shouldThrow):
(shouldNotThrow):
- js/script-tests/class-syntax-declaration.js:
(shouldThrow):
(shouldNotThrow):
(shouldBe):
- js/script-tests/class-syntax-default-constructor.js:
(shouldThrow):
(shouldBe):
(shouldBeTrue):
(assert):
(A):
(B):
- js/script-tests/class-syntax-extends.js:
(shouldThrow):
(shouldNotThrow):
(shouldBe):
(shouldBeTrue):
(Base):
(Base.prototype.baseMethod):
- js/script-tests/class-syntax-name.js:
(shouldThrow):
(shouldNotThrow):
(shouldBe):
(shouldBeTrue):
(runTestShouldBe):
- js/script-tests/class-syntax-super.js:
(shouldThrow):
(shouldNotThrow):
(shouldBe):
(shouldBeTrue):
(shouldBeFalse):
- 1:38 PM Changeset in webkit [187679] by
-
- 2 edits in branches/safari-601.1-branch/Source/WebCore
Merged r187675. rdar://problem/21632211
- 1:36 PM Changeset in webkit [187678] by
-
- 3 edits in branches/jsc-tailcall/Source/JavaScriptCore
jsc-tailcall: Emit a single callee-save restoration for fast and slow tail call paths
https://bugs.webkit.org/show_bug.cgi?id=147501
Reviewed by Michael Saboff.
We are currently emitting this twice for no reason.
- jit/JITCall.cpp:
(JSC::JIT::compileOpCall):
- jit/JITCall32_64.cpp:
(JSC::JIT::compileOpCall):
- 1:34 PM Changeset in webkit [187677] by
-
- 9 edits5 adds in trunk/Source/JavaScriptCore
Implement WebAssembly module parser
https://bugs.webkit.org/show_bug.cgi?id=147293
Patch by Sukolsak Sakshuwong <Sukolsak Sakshuwong> on 2015-07-31
Reviewed by Mark Lam.
Re-landing after fix for the "..\..\jsc.cpp(46): fatal error C1083: Cannot open
include file: 'JSWASMModule.h'" issue on Windows.
Implement WebAssembly module parser for WebAssembly files produced by pack-asmjs
<https://github.com/WebAssembly/polyfill-prototype-1>. This patch only checks
the magic number at the beginning of the files. Parsing of the rest will be
implemented in a subsequent patch.
- CMakeLists.txt:
- JavaScriptCore.vcxproj/JavaScriptCore.vcxproj:
- JavaScriptCore.vcxproj/JavaScriptCore.vcxproj.filters:
- JavaScriptCore.xcodeproj/project.pbxproj:
- jsc.cpp:
(GlobalObject::finishCreation):
(functionLoadWebAssembly):
- parser/SourceProvider.h:
(JSC::WebAssemblySourceProvider::create):
(JSC::WebAssemblySourceProvider::data):
(JSC::WebAssemblySourceProvider::WebAssemblySourceProvider):
- runtime/JSGlobalObject.cpp:
(JSC::JSGlobalObject::init):
(JSC::JSGlobalObject::visitChildren):
- runtime/JSGlobalObject.h:
(JSC::JSGlobalObject::wasmModuleStructure):
- wasm/WASMMagicNumber.h: Added.
- wasm/WASMModuleParser.cpp: Added.
(JSC::WASMModuleParser::WASMModuleParser):
(JSC::WASMModuleParser::parse):
(JSC::WASMModuleParser::parseModule):
(JSC::parseWebAssembly):
- wasm/WASMModuleParser.h: Added.
- wasm/WASMReader.cpp: Added.
(JSC::WASMReader::readUnsignedInt32):
(JSC::WASMReader::readFloat):
(JSC::WASMReader::readDouble):
- wasm/WASMReader.h: Added.
(JSC::WASMReader::WASMReader):
- 1:31 PM Changeset in webkit [187676] by
-
- 6 edits in branches/jsc-tailcall/Source/JavaScriptCore
jsc-tailcall: Don't waste stack space when arity fixup was performed
https://bugs.webkit.org/show_bug.cgi?id=147447
Reviewed by Michael Saboff.
r187618 tried to handle this, but did not include the LLInt and did it
wrong anyway. When making a call, the caller first aligns the callee's
frame, then makes the call. If the callee later performs arity fixup,
it does not try to reuse the potential slot left empty by the caller's
frame alignment: instead, it aligns the parameters - arguments delta.
So when trying to find out where our frame starts, we should replicate
this behavior, which we were previously not doing and was the cause of
https://bugs.webkit.org/show_bug.cgi?id=147491.
While here, I also removed the extra slot that used to save the
returnPC for the arity fixup return thunk (that we got rid of in
https://bugs.webkit.org/show_bug.cgi?id=146847) and was now just
wasting stack space.
- jit/CCallHelpers.h:
- jit/ThunkGenerators.cpp:
(JSC::arityFixupGenerator):
- llint/LowLevelInterpreter.asm:
- runtime/CommonSlowPaths.h:
(JSC::CommonSlowPaths::arityCheckFor):
- tests/stress/tail-call-no-stack-overflow.js:
(strictLoopArityFixup1):
(strictLoopArityFixup2):
- 1:21 PM Changeset in webkit [187675] by
-
- 2 edits in trunk/Source/WebCore
CRASH at WebCore::TaskDispatcher<WebCore::ScriptExecutionContext>::postTask + 38
https://bugs.webkit.org/show_bug.cgi?id=147485
Rubber-stamped by Joseph Pecoraro.
Follow-up test crash fix: call superclass's contextDestroyed() which allows the
m_scriptExecutionContext variable to be cleared.
- html/HTMLMediaElement.cpp: (WebCore::HTMLMediaElement::contextDestroyed):
- 1:17 PM Changeset in webkit [187674] by
-
- 2 edits in branches/jsc-tailcall/Source/JavaScriptCore
jsc-tailcall: REGRESSION(r187639): Need to restore callee saves in LLInt before tail call
https://bugs.webkit.org/show_bug.cgi?id=147500
Reviewed by Basile Clement.
Restore callee saves at the head of the prepareForTailCall() macro.
- llint/LowLevelInterpreter.asm:
- 1:15 PM Changeset in webkit [187673] by
-
- 4 edits in branches/safari-601.1-branch/Source/WebCore
Merge r187655. rdar://problem/21632211
- 1:15 PM Changeset in webkit [187672] by
-
- 2 edits in branches/safari-601.1-branch/Source/WebCore
Merge r187630. rdar://problem/18835799
- 1:15 PM Changeset in webkit [187671] by
-
- 2 edits in branches/safari-601.1-branch/Source/WebKit2
Merge r187614. rdar://problem/22064327
- 1:01 PM Changeset in webkit [187670] by
-
- 2 edits in branches/jsc-tailcall/Source/JavaScriptCore
jsc-tailcall:Eliminate the unsafe use of regT5 in arityFixupGenerator() for X86-64 Windows
https://bugs.webkit.org/show_bug.cgi?id=147499
Reviewed by Basile Clement.
For X86-64, Windows, regT0 is free so use that register instead of regT5.
- jit/ThunkGenerators.cpp:
(JSC::arityFixupGenerator):
- 12:43 PM Changeset in webkit [187669] by
-
- 2 edits in trunk/Source/WebKit/mac
REGRESSION (r174791): Returning a zombie NSImage from sharingService:transitionImageForShareItem:contentRect:
<rdar://problem/22060183> and https://bugs.webkit.org/show_bug.cgi?id=147497
Reviewed by Tim Horton.
- WebCoreSupport/WebContextMenuClient.mm:
(WebContextMenuClient::imageForCurrentSharingServicePickerItem): Don't return what is basically a temporary NSImage.
Instead do a [[retain] autorelease] on it while still valid.
- 12:25 PM Changeset in webkit [187668] by
-
- 2 edits in trunk/Source/WebCore
Only generate "themeWin*" UserAgentStyleSheet constants for Windows builds
https://bugs.webkit.org/show_bug.cgi?id=147378
Patch by Joseph Pecoraro <Joseph Pecoraro> on 2015-07-31
Reviewed by Alex Christensen.
- DerivedSources.make:
Only include windows user agent style sheets for Windows OSes.
- 11:41 AM Changeset in webkit [187667] by
-
- 2 edits1 copy in branches/safari-601.1-branch/Source/JavaScriptCore
Merged r187579. rdar://problem/21668986
- 11:38 AM Changeset in webkit [187666] by
-
- 3 edits in branches/safari-601.1-branch/Source/WebCore
Merged r187557. rdar://problem/21412186
- 11:33 AM Changeset in webkit [187665] by
-
- 8 edits in branches/safari-601.1-branch
Merged r187556. rdar://problem/21771620
- 11:30 AM Changeset in webkit [187664] by
-
- 5 edits2 copies in branches/safari-601.1-branch
Merged r187535. rdar://problem/21852603
- 11:22 AM Changeset in webkit [187663] by
-
- 2 edits in branches/safari-601.1-branch/Source/WebKit2
Merged r187530. rdar://problem/21833987
- 11:19 AM Changeset in webkit [187662] by
-
- 3 edits2 copies in branches/safari-601.1-branch
Merged r187525. rdar://problem/22022011
- 11:10 AM Changeset in webkit [187661] by
-
- 26 edits in branches/safari-601.1-branch/LayoutTests
Merged r187523. rdar://problem/22010303
- 11:08 AM Changeset in webkit [187660] by
-
- 13 edits in branches/safari-601.1-branch/Source
Merged r187522. rdar://problem/19892014
- 11:07 AM Changeset in webkit [187659] by
-
- 18 edits in trunk
Prepare for debug CMake builds on Windows.
https://bugs.webkit.org/show_bug.cgi?id=147484
Reviewed by Tim Horton.
.:
- Source/cmake/OptionsWindows.cmake:
Don't use debug runtimes in debug builds because the dependencies are only built with multithreaded runtimes.
Source/WebCore:
In order to get debug builds to link on Windows with CMake, we will need to use AllInOnes.
This patch includes more files in the AllInOnes and removes some duplicate code.
- WebCore.vcxproj/WebCore.vcxproj:
- bindings/js/JSBindingsAllInOne.cpp:
- css/CSSAllInOne.cpp:
Many files were added to the AllInOne files. They should not be included in the Visual Studio solutions.
- css/CSSCalculationValue.cpp:
(WebCore::CSSCalcValue::create):
Use CSSParserString::equalIgnoringCase to avoid ambiguous calls to overloaded functions.
- css/StyleResolver.cpp:
(WebCore::StyleResolver::pseudoStyleRulesForElement):
(WebCore::shouldApplyPropertyInParseOrder):
(WebCore::convertToFloatLength): Deleted.
- css/TransformFunctions.cpp:
(WebCore::transformOperationType):
(WebCore::convertToFloatLength):
- css/TransformFunctions.h:
Removed duplicate convertToFloatLength function.
- html/HTMLElementsAllInOne.cpp:
- rendering/FlowThreadController.cpp:
(WebCore::FlowThreadController::updateNamedFlowsLayerListsIfNeeded):
(WebCore::compareZIndexFlowThreadController): Renamed from compareZIndex to prevent naming conflicts.
(WebCore::FlowThreadController::collectFixedPositionedLayers):
(WebCore::compareZIndex): Deleted.
- rendering/RenderLayer.cpp:
(WebCore::RenderLayer::shouldRepaintAfterLayout):
(WebCore::compositedWithOwnBackingStore):
- rendering/RenderLayer.h:
- rendering/RenderLayerBacking.cpp:
(WebCore::RenderLayerBacking::isSimpleContainerCompositingLayer):
(WebCore::descendantLayerPaintsIntoAncestor):
(WebCore::RenderLayerBacking::paintIntoLayer):
(WebCore::RenderLayerBacking::paintContents):
(WebCore::compositedWithOwnBackingStore): Deleted.
Removed duplicate compositedWithOwnBackingStore.
(WebCore::paintScrollbar): Deleted duplicate function.
- rendering/RenderLayerCompositor.cpp:
(WebCore::RenderLayerCompositor::requiresScrollLayer):
(WebCore::paintScrollbar):
- rendering/RenderLayerCompositor.h:
- rendering/RenderingAllInOne.cpp:
- 11:01 AM Changeset in webkit [187658] by
-
- 4 edits in branches/safari-601.1-branch/Source/WebCore
Merged r187521. rdar://problem/22038421
- 10:56 AM Changeset in webkit [187657] by
-
- 4 edits in branches/safari-601.1-branch/Source/WebCore
Merged r187516. rdar://problem/20847009
- 10:49 AM Changeset in webkit [187656] by
-
- 2 edits in branches/safari-601.1-branch/LayoutTests
Merged r187512. rdar://problem/22010303
- 10:49 AM Changeset in webkit [187655] by
-
- 4 edits in trunk/Source/WebCore
CRASH at WebCore::TaskDispatcher<WebCore::ScriptExecutionContext>::postTask + 38
https://bugs.webkit.org/show_bug.cgi?id=147485
Reviewed by Eric Carlson.
CrashLogs indicate a use-after-free of the ScriptExecutionContext (i.e., Document) used by
the GenericTaskQueue objects owned by HTMLMediaElement. When the ScriptExecutionContext
notifies its ActiveDOMObjects that it is about to be destroyed, close() the
GenericTaskQueues so that they can no longer accept new tasks.
Previously, enqueueing a task on a closed GenericTaskQueue ASSERTed in debug builds, but
silently succeeded in release builds. Calling enqueueTask() on a closed GenericTaskQueue is
now a no-op.
- html/HTMLMediaElement.cpp:
(WebCore::HTMLMediaElement::contextDestroyed):
- html/HTMLMediaElement.h:
- platform/GenericTaskQueue.h:
(WebCore::GenericTaskQueue::enqueueTask):
- 10:08 AM Changeset in webkit [187654] by
-
- 2 edits2 adds in trunk/LayoutTests
Media Session: add test for MediaSession.releaseSession()
https://bugs.webkit.org/show_bug.cgi?id=147478
Reviewed by Eric Carlson.
- media/session/releasing-sessions-expected.txt: Added.
- media/session/releasing-sessions.html: Added.
- platform/mac/TestExpectations: Media Session support is disabled by default.
- 9:59 AM Changeset in webkit [187653] by
-
- 2 edits2 adds in trunk/LayoutTests
Media Session: add test to ensure next/previous-track event handlers are not fired when they are disabled
https://bugs.webkit.org/show_bug.cgi?id=147477
Reviewed by Eric Carlson.
- media/session/track-media-events-disabled-expected.txt: Added.
- media/session/track-media-events-disabled.html: Added.
- platform/mac/TestExpectations: Media Session support is disabled by default.
- 9:51 AM Changeset in webkit [187652] by
-
- 2 edits2 adds in trunk/LayoutTests
Media Session: add test to ensure media sessions are in an Idle state after creation
https://bugs.webkit.org/show_bug.cgi?id=147471
Reviewed by Eric Carlson.
- media/session/session-creation-expected.txt: Added.
- media/session/session-creation.html: Added.
- platform/mac/TestExpectations: Media Session support is disabled by default.
- 9:44 AM Changeset in webkit [187651] by
-
- 12 edits in branches/safari-601.1-branch/Source
Merged r187491. rdar://problem/21718407
- 9:42 AM Changeset in webkit [187650] by
-
- 18 edits34 copies2 deletes in branches/safari-601.1-branch
Merged r187489. rdar://problem/22010303
- 9:35 AM Changeset in webkit [187649] by
-
- 2 edits in branches/safari-601.1-branch/Source/WebKit2
Merged r187471. rdar://problem/22021772
- 9:33 AM Changeset in webkit [187648] by
-
- 4 edits in branches/safari-601.1-branch/Source/WebCore
Merged r187466. rdar://problem/21960398
- 9:25 AM Changeset in webkit [187647] by
-
- 2 edits in branches/safari-601.1-branch/Source/WebKit2
Merged r187462. rdar://problem/22020770
- 9:05 AM Changeset in webkit [187646] by
-
- 2 edits in branches/safari-601.1-branch/Source/WebKit2
Merged r187459. rdar://problem/21701858
- 9:01 AM Changeset in webkit [187645] by
-
- 2 edits in branches/safari-601.1-branch/Source/WebCore
Merged r187448. rdar://problem/21948230
- 9:00 AM Changeset in webkit [187644] by
-
- 2 edits in branches/safari-601.1-branch/Source/WebCore
Merged r187379. rdar://problem/21948230
- 8:58 AM Changeset in webkit [187643] by
-
- 9 edits in branches/safari-601.1-branch/Source
Merged r187375. rdar://problem/21948230
- 7:28 AM Changeset in webkit [187642] by
-
- 2 edits in trunk/Tools
[EFL] Fix pixman build on ARM with GCC 4.9.3
https://bugs.webkit.org/show_bug.cgi?id=147224
Reviewed by Gyuyoung Kim.
- efl/jhbuild.modules: Disable ARM IWMMXT fast paths as workarounding a GCC bug.
- 5:24 AM Changeset in webkit [187641] by
-
- 3 edits in trunk/Source/WebKit2
[GTK] NPAPI code crashes on Wayland
https://bugs.webkit.org/show_bug.cgi?id=147297
Reviewed by Žan Doberšek.
Do not crete NPAPI plugins when runing on Wayland, since they are
not supported. ENABLE_NETSCAPE_PLUGIN_API option used to be
incompatible with building with the Wayland target, but now that
we allow to build X11 and Wayland targets at the same time, we
need runtime checks to avoid creating NPAPI plugins in Wayland.
- UIProcess/API/gtk/WebKitSettings.cpp:
(webkit_settings_set_enable_plugins): Do not allow to change the
setting when running in Wayland.
- UIProcess/gtk/WebPreferencesGtk.cpp:
(WebKit::WebPreferences::platformInitializeStore): Initialize
EnablePlugins preference to false when running in Wayland.
- 1:06 AM Changeset in webkit [187640] by
-
- 5 edits in trunk/Source/WebCore
[GTK] Have DataObjectGtk::unknownTypes() return a reference to the HashMap object
https://bugs.webkit.org/show_bug.cgi?id=147401
Reviewed by Carlos Garcia Campos.
Don't copy the DataObjectGtk::m_unknownTypes HashMap on every retrieval through
DataObjectGtk::unknownTypes(). The range-based for-loops that iterate over the
map in PasteboardGtk.cpp and PasteboardHelper.cpp are also cleaned up.
- platform/gtk/DataObjectGtk.cpp:
(WebCore::DataObjectGtk::unknownTypes):
- platform/gtk/DataObjectGtk.h:
- platform/gtk/PasteboardGtk.cpp:
(WebCore::Pasteboard::writePasteboard):
(WebCore::Pasteboard::types):
- platform/gtk/PasteboardHelper.cpp:
(WebCore::PasteboardHelper::fillSelectionData):