Timeline



Dec 23, 2014:

9:34 PM Changeset in webkit [177721] by ChangSeok Oh
  • 19 edits
    2 moves in trunk/Source/WebKit2

[GTK] Build fix after r177692
https://bugs.webkit.org/show_bug.cgi?id=139919

Reviewed by Gyuyoung Kim.

This change addresses build failures introduced after r177692.

  • PlatformEfl.cmake: Renamed WebContextSoup to WebProcessPoolSoup.cpp
  • PlatformGTK.cmake: ditto.
  • UIProcess/API/gtk/PageClientImpl.cpp:
  • UIProcess/API/gtk/WebKitDownloadClient.cpp:
  • UIProcess/API/gtk/WebKitFormSubmissionRequest.cpp:

(webkitFormSubmissionRequestCreate):
(webkit_form_submission_request_get_text_fields):

  • UIProcess/API/gtk/WebKitFormSubmissionRequestPrivate.h:
  • UIProcess/API/gtk/WebKitInjectedBundleClient.cpp:

(didReceiveWebViewMessageFromInjectedBundle):
(didReceiveMessageFromInjectedBundle):

  • UIProcess/API/gtk/WebKitSecurityManager.cpp:

(registerSecurityPolicyForURIScheme):

  • UIProcess/API/gtk/WebKitWebContext.cpp:

(webkitWebContextConstructed):
(webkit_web_context_prefetch_dns):
(webkitWebContextGetContext):

  • UIProcess/API/gtk/WebKitWebContextPrivate.h:
  • UIProcess/API/gtk/WebKitWebView.cpp:

(webkit_web_view_get_snapshot):

  • UIProcess/API/gtk/WebKitWebViewBase.cpp:

(webkitWebViewBaseCreate):
(webkitWebViewBaseCreateWebPage):

  • UIProcess/API/gtk/WebKitWebViewBasePrivate.h:
  • UIProcess/API/gtk/WebKitWindowProperties.cpp:
  • UIProcess/gtk/WebInspectorProxyGtk.cpp:

(WebKit::WebInspectorProxy::platformCreateInspectorPage):

  • UIProcess/gtk/WebProcessPoolGtk.cpp: Renamed from Source/WebKit2/UIProcess/gtk/WebContextGtk.cpp.

(WebKit::initInspectorServer):
(WebKit::WebProcessPool::platformDefaultApplicationCacheDirectory):
(WebKit::WebProcessPool::platformInitializeWebProcess):
(WebKit::WebProcessPool::platformInvalidateContext):
(WebKit::WebProcessPool::platformDefaultWebSQLDatabaseDirectory):
(WebKit::WebProcessPool::platformDefaultIndexedDBDatabaseDirectory):
(WebKit::WebProcessPool::platformDefaultIconDatabasePath):
(WebKit::WebProcessPool::platformDefaultLocalStorageDirectory):
(WebKit::WebProcessPool::platformDefaultMediaKeysStorageDirectory):
(WebKit::WebProcessPool::platformDefaultDiskCacheDirectory):
(WebKit::WebProcessPool::platformDefaultCookieStorageDirectory):
(WebKit::WebProcessPool::setIgnoreTLSErrors):

  • UIProcess/soup/WebProcessPoolSoup.cpp: Renamed from Source/WebKit2/UIProcess/soup/WebContextSoup.cpp.

(WebKit::WebProcessPool::platformInitializeNetworkProcess):

  • WebProcess/InjectedBundle/API/gtk/WebKitWebExtension.cpp:

(webkitWebExtensionDidReceiveMessage):

  • WebProcess/InjectedBundle/API/gtk/WebKitWebPage.cpp:

(didInitiateLoadForResource):
(willSendRequestForFrame):
(didReceiveResponseForResource):
(didReceiveContentLengthForResource):
(didFinishLoadForResource):
(didFailLoadForResource):
(webkitWebPageDidReceiveMessage):

  • WebProcess/InjectedBundle/API/gtk/WebKitWebPagePrivate.h:
8:58 PM Changeset in webkit [177720] by Chris Dumez
  • 2 edits in trunk/Source/WebCore

Simplify RenderStyle::invalidColor() function
https://bugs.webkit.org/show_bug.cgi?id=139898

Reviewed by Andreas Kling.

Simplify RenderStyle::invalidColor() function by getting rid of the
static variable. The default constructor for Color is very cheap
anyway.

  • rendering/style/RenderStyle.h:
8:50 PM Changeset in webkit [177719] by mitz@apple.com
  • 2 edits in trunk/Source/WebKit2

iOS build fix

  • UIProcess/API/Cocoa/WKProcessGroup.mm:

(-[WKProcessGroup _geolocationProvider]):

8:43 PM Changeset in webkit [177718] by ap@apple.com
  • 2 edits in trunk/Tools

Don't force -O1 for debug ASan builds
https://bugs.webkit.org/show_bug.cgi?id=139926

Reviewed by Darin Adler.

  • asan/asan.xcconfig:
8:33 PM Changeset in webkit [177717] by ljaehun.lim@samsung.com
  • 2 edits in trunk/Source/WebCore

Unreviewed, fix build warning after r177712

../../Source/WebCore/history/PageCache.cpp: In function ‘const char* WebCore::pruningReasonToFeatureCounterKey(WebCore::PruningReason)’:
../../Source/WebCore/history/PageCache.cpp:449:1: error: control reaches end of non-void function [-Werror=return-type]

No new tests, no behavior changes.

  • history/PageCache.cpp:

(WebCore::pruningReasonToFeatureCounterKey):

8:09 PM Changeset in webkit [177716] by gyuyoung.kim@samsung.com
  • 22 edits in trunk/Source/WebKit2

[EFL][WK2] Fix a lot of build break since r177692
https://bugs.webkit.org/show_bug.cgi?id=139932

Unreviewed EFL build fix.

Rename WebContext with WebProcessPool in EFL port files.

  • CMakeLists.txt:
  • UIProcess/API/efl/EwkView.cpp:
  • UIProcess/API/efl/ewk_context.cpp:
  • UIProcess/API/efl/tests/test_ewk2_context.cpp:

(TEST_F):

  • UIProcess/CoordinatedGraphics/WebView.cpp:

(WebKit::WebView::WebView):

  • UIProcess/CoordinatedGraphics/WebView.h:
  • UIProcess/Network/CustomProtocols/soup/CustomProtocolManagerProxySoup.cpp:

(WebKit::CustomProtocolManagerProxy::CustomProtocolManagerProxy):
(WebKit::CustomProtocolManagerProxy::startLoading):
(WebKit::CustomProtocolManagerProxy::stopLoading):

  • UIProcess/Network/CustomProtocols/soup/WebSoupCustomProtocolRequestManager.cpp:

(WebKit::WebSoupCustomProtocolRequestManager::create):
(WebKit::WebSoupCustomProtocolRequestManager::WebSoupCustomProtocolRequestManager):
(WebKit::WebSoupCustomProtocolRequestManager::registerSchemeForCustomProtocol):
(WebKit::WebSoupCustomProtocolRequestManager::unregisterSchemeForCustomProtocol):
(WebKit::WebSoupCustomProtocolRequestManager::didReceiveResponse):
(WebKit::WebSoupCustomProtocolRequestManager::didLoadData):
(WebKit::WebSoupCustomProtocolRequestManager::didFailWithError):
(WebKit::WebSoupCustomProtocolRequestManager::didFinishLoading):

  • UIProcess/Network/CustomProtocols/soup/WebSoupCustomProtocolRequestManager.h:
  • UIProcess/WebBatteryManagerProxy.cpp:

(WebKit::WebBatteryManagerProxy::create):
(WebKit::WebBatteryManagerProxy::WebBatteryManagerProxy):
(WebKit::WebBatteryManagerProxy::providerDidChangeBatteryStatus):
(WebKit::WebBatteryManagerProxy::providerUpdateBatteryStatus):

  • UIProcess/WebBatteryManagerProxy.h:
  • UIProcess/WebTextChecker.cpp:

(WebKit::updateStateForAllContexts):

  • UIProcess/WebVibrationProxy.cpp:

(WebKit::WebVibrationProxy::WebVibrationProxy):
(WebKit::WebVibrationProxy::~WebVibrationProxy):

  • UIProcess/efl/RequestManagerClientEfl.h:
  • UIProcess/efl/WebContextEfl.cpp:

(WebKit::WebProcessPool::platformDefaultApplicationCacheDirectory):
(WebKit::WebProcessPool::platformInitializeWebProcess):
(WebKit::WebProcessPool::platformInvalidateContext):
(WebKit::WebProcessPool::platformDefaultWebSQLDatabaseDirectory):
(WebKit::WebProcessPool::platformDefaultIndexedDBDatabaseDirectory):
(WebKit::WebProcessPool::platformDefaultIconDatabasePath):
(WebKit::WebProcessPool::platformDefaultLocalStorageDirectory):
(WebKit::WebProcessPool::platformDefaultMediaKeysStorageDirectory):
(WebKit::WebProcessPool::platformDefaultDiskCacheDirectory):
(WebKit::WebProcessPool::platformDefaultCookieStorageDirectory):
(WebKit::WebProcessPool::setIgnoreTLSErrors):
(WebKit::WebContext::platformDefaultApplicationCacheDirectory): Deleted.
(WebKit::WebContext::platformInitializeWebProcess): Deleted.
(WebKit::WebContext::platformInvalidateContext): Deleted.
(WebKit::WebContext::platformDefaultWebSQLDatabaseDirectory): Deleted.
(WebKit::WebContext::platformDefaultIndexedDBDatabaseDirectory): Deleted.
(WebKit::WebContext::platformDefaultIconDatabasePath): Deleted.
(WebKit::WebContext::platformDefaultLocalStorageDirectory): Deleted.
(WebKit::WebContext::platformDefaultMediaKeysStorageDirectory): Deleted.
(WebKit::WebContext::platformDefaultDiskCacheDirectory): Deleted.
(WebKit::WebContext::platformDefaultCookieStorageDirectory): Deleted.
(WebKit::WebContext::setIgnoreTLSErrors): Deleted.

  • UIProcess/efl/WebInspectorProxyEfl.cpp:

(WebKit::WebInspectorProxy::platformCreateInspectorPage):

  • UIProcess/efl/WebViewEfl.cpp:

(WebKit::WebView::create):
(WebKit::WebViewEfl::WebViewEfl):

  • UIProcess/efl/WebViewEfl.h:
  • UIProcess/soup/WebContextSoup.cpp:

(WebKit::WebProcessPool::platformInitializeNetworkProcess):
(WebKit::WebContext::platformInitializeNetworkProcess): Deleted.

  • UIProcess/soup/WebCookieManagerProxySoup.cpp:

(WebKit::WebCookieManagerProxy::setCookiePersistentStorage):

  • UIProcess/soup/WebSoupRequestManagerProxy.h:
7:09 PM Changeset in webkit [177715] by weinig@apple.com
  • 4 edits in trunk/Source/WebKit2

Remove telephone number context support from ContextMenuContextData, it is unused
https://bugs.webkit.org/show_bug.cgi?id=139931

Reviewed by Alexey Proskuryakov.

m_isTelephoneNumberContext was only ever set to true in the ContextMenuContextData
constructor that took a TelephoneNumberContextTag, which never got called. Remove it
all.

  • Shared/ContextMenuContextData.cpp:

(WebKit::ContextMenuContextData::ContextMenuContextData):
(WebKit::ContextMenuContextData::encode):
(WebKit::ContextMenuContextData::decode):

  • Shared/ContextMenuContextData.h:

(WebKit::ContextMenuContextData::ContextMenuContextData):
(WebKit::ContextMenuContextData::isTelephoneNumberContext): Deleted.

  • UIProcess/mac/WebContextMenuProxyMac.mm:

(WebKit::WebContextMenuProxyMac::showContextMenu):

6:51 PM Changeset in webkit [177714] by weinig@apple.com
  • 2 edits in trunk/Source/WebKit2

Attempt to fix the Mountain Lion build.

  • Shared/ContextMenuContextData.h:

(WebKit::ContextMenuContextData::isTelephoneNumberContext):

5:20 PM Changeset in webkit [177713] by andersca@apple.com
  • 3 edits in trunk/Source/WebKit2

Handle WKTypeRefWrapper when converting object graphs
https://bugs.webkit.org/show_bug.cgi?id=139927

Reviewed by Tim Horton.

  • UIProcess/Cocoa/WebProcessProxyCocoa.mm:

(WebKit::WebProcessProxy::transformHandlesToObjects):
(WebKit::WebProcessProxy::transformObjectsToHandles):

  • WebProcess/cocoa/WebProcessCocoa.mm:

(WebKit::WebProcess::transformHandlesToObjects):
(WebKit::WebProcess::transformObjectsToHandles):

5:14 PM Changeset in webkit [177712] by Chris Dumez
  • 11 edits in trunk/Source

[iOS] Log better using FeatureCounter why PageCache is failing due to pruned resources
https://bugs.webkit.org/show_bug.cgi?id=139921
<rdar://problem/19255690>

Reviewed by Gavin Barraclough.

Source/WebCore:

Log better using FeatureCounter why PageCache is failing due to pruned
resources. In particular, we now distinguish if the resource was pruned
due to:

  • Memory pressure
  • Page cache capacity reached
  • WebProcess suspended (WK2)
  • WebCore.exp.in:
  • history/HistoryItem.cpp:

(WebCore::HistoryItem::HistoryItem):

  • history/HistoryItem.h:
  • history/PageCache.cpp:

(WebCore::PageCache::pruneToCapacityNow):
(WebCore::PageCache::setCapacity):
(WebCore::pruningReasonToFeatureCounterKey):
(WebCore::PageCache::add):
(WebCore::PageCache::take):
(WebCore::PageCache::get):
(WebCore::PageCache::prune):

  • history/PageCache.h:
  • loader/FrameLoader.cpp:

(WebCore::FrameLoader::commitProvisionalLoad):

  • platform/FeatureCounterKeys.h:
  • platform/MemoryPressureHandler.cpp:

(WebCore::MemoryPressureHandler::releaseCriticalMemory):

Source/WebKit2:

Call PageCache::pruneToCapacityNow() instead of multiple calls to
setCapacity() as it does exactly what we want. Also pass the new
PrunedReason argument so help us do better logging.

  • WebProcess/WebProcess.cpp:

(WebKit::WebProcess::releasePageCache):

5:11 PM Changeset in webkit [177711] by akling@apple.com
  • 2 edits in trunk/Source/WebCore

[Mac] Fix nullptr dereference in EventHandler::platformPrepareForWheelEvents().
<https://webkit.org/b/139923>
<rdar://problem/19182531>

Reviewed by Chris Dumez.

Don't dereference 'wheelEventTarget' until we know it's non-null.
This manifested as a crash in Node::renderBox() with some versions of LLVM.

The compiler was free to assume that the ContainerNode* 'candidate' in
findEnclosingScrollableContainer() would always be non-null on the first
iteration of the loop.

I suspect that we can get here with a null 'wheelEventTarget' during the
rubberbanding phase of a wheel scroll; if the cursor is sitting over the
exposed "gutter" area.

  • page/mac/EventHandlerMac.mm:

(WebCore::findEnclosingScrollableContainer):
(WebCore::EventHandler::platformPrepareForWheelEvents):

5:09 PM Changeset in webkit [177710] by weinig@apple.com
  • 3 edits in trunk/Source/WebKit2

Attempt to fix the iOS build.

  • UIProcess/API/Cocoa/WKProcessGroup.mm:

(-[WKProcessGroup _geolocationProvider]):

  • UIProcess/API/ios/WKViewIOS.mm:

(-[WKView _commonInitializationWithContextRef:pageGroupRef:relatedToPage:]):

5:01 PM Changeset in webkit [177709] by andersca@apple.com
  • 6 edits
    1 add in trunk/Source/WebKit2

Handle coding object graphs in the UI process
https://bugs.webkit.org/show_bug.cgi?id=139925

Reviewed by Sam Weinig.

  • UIProcess/Cocoa/WebProcessProxyCocoa.mm: Added.

(WebKit::WebProcessProxy::transformHandlesToObjects):
(WebKit::WebProcessProxy::transformObjectsToHandles):

  • UIProcess/WebProcessProxy.cpp:

(WebKit::WebProcessProxy::transformHandlesToObjects):
(WebKit::WebProcessProxy::transformObjectsToHandles):

  • UIProcess/WebProcessProxy.h:
  • WebKit2.xcodeproj/project.pbxproj:
  • WebProcess/WebProcess.cpp:

(WebKit::WebProcess::transformObjectsToHandles):

  • WebProcess/WebProcess.h:
5:00 PM Changeset in webkit [177708] by weinig@apple.com
  • 5 edits in trunk/Source/WebKit2

ContextMenuContextData should store a ShareableBitmap not a ShareableBitmap::Handle
https://bugs.webkit.org/show_bug.cgi?id=139924

Reviewed by Anders Carlsson.

Replaces the ShareableBitmap::Handle in ContextMenuContextData with a RefPtr<ShareableBitmap>.
Removes the now unnecessary override copy-constructor and assignment-operator.

  • Shared/ContextMenuContextData.cpp:

(WebKit::ContextMenuContextData::ContextMenuContextData):
(WebKit::ContextMenuContextData::encode):
(WebKit::ContextMenuContextData::decode):
(WebKit::ContextMenuContextData::controlledDataIsEditable):
(WebKit::ContextMenuContextData::operator=): Deleted.

  • Shared/ContextMenuContextData.h:

(WebKit::ContextMenuContextData::ContextMenuContextData):
(WebKit::ContextMenuContextData::controlledImage):
(WebKit::ContextMenuContextData::needsServicesMenu):
(WebKit::ContextMenuContextData::controlledImageHandle): Deleted.

  • UIProcess/WebPageProxy.cpp:

(WebKit::WebPageProxy::internalShowContextMenu):

  • UIProcess/mac/WebContextMenuProxyMac.mm:

(WebKit::WebContextMenuProxyMac::setupServicesMenu):

4:46 PM Changeset in webkit [177707] by Chris Dumez
  • 2 edits in trunk/Source/WebKit2

Unreviewed iOS build fix after 177692.

Missing WebKit:: namespace.

  • UIProcess/ios/WKContentView.h:
4:37 PM Changeset in webkit [177706] by andersca@apple.com
  • 2 edits in trunk/Source/WebKit2

Fix 32-build.

  • WebProcess/cocoa/WebProcessCocoa.mm:

(WebKit::WebProcess::transformHandlesToObjects):
(WebKit::WebProcess::transformObjectsToHandles):

4:14 PM Changeset in webkit [177705] by timothy_horton@apple.com
  • 7 edits in trunk/Source

Install a TextIndicator for link immediate actions
https://bugs.webkit.org/show_bug.cgi?id=139897
<rdar://problem/19333076>

Reviewed by Anders Carlsson.

  • UIProcess/mac/WKImmediateActionController.mm:

(-[WKImmediateActionController _defaultAnimationController]):
Install the link's TextIndicator.

(-[WKImmediateActionController menuItemDidClose:]):
Dismiss the TextIndicator and clean up when the item closes.

  • WebView/WebImmediateActionController.h:
  • WebView/WebImmediateActionController.mm:

(-[WebImmediateActionController _clearImmediateActionState]):
Tear down the TextIndicator, just like WebKit2.

(-[WebImmediateActionController _defaultAnimationController]):
Install a TextIndicator for links.

(-[WebImmediateActionController menuItemDidClose:]):
Tear down the TextIndicator when the QL preview closes.

(-[WebImmediateActionController _menuItemForDataDetectedText]):
Directly install and uninstall the TextIndicator for DataDetectors.

(-[WebImmediateActionController _showTextIndicator]): Deleted.
(-[WebImmediateActionController _hideTextIndicator]): Deleted.
Remove the isShowingTextIndicator mechanism, just like it was removed from WebKit2.

  • platform/spi/mac/QuickLookMacSPI.h:

Add a new piece of SPI.

4:13 PM Changeset in webkit [177704] by timothy_horton@apple.com
  • 3 edits in trunk/Source/WebKit2

Expose ElementBoundingBox on WKHitTestResult
https://bugs.webkit.org/show_bug.cgi?id=139895
<rdar://problem/19333075>

Reviewed by Dan Bernstein.

  • UIProcess/API/C/WKHitTestResult.cpp:

(WKHitTestResultGetElementBoundingBox):

  • UIProcess/API/C/WKHitTestResult.h:

Expose the rect.

4:13 PM Changeset in webkit [177703] by ap@apple.com
  • 6 edits in trunk

Simplify building with ASan
https://bugs.webkit.org/show_bug.cgi?id=139916

Reviewed by Mark Rowe.

.:

  • Makefile.shared: Invoke set-webkit-configuration to store ASan choice as appropriate.

Tools:

  • Scripts/set-webkit-configuration: Store ASan state into a new configuration file.

We could also update Configuration file format, but that's a little scary because
of how many places in code read it.

  • Scripts/webkitdirs.pm:

(determineASanIsEnabled): Read it from ASan configuration file.
(argumentsForConfiguration): Added a FIXME.
(asanIsEnabled): A caching wrapper similar to what we have for other configuration options.
(XcodeOptions): Pass the options needed for ASan.

  • asan/asan.xcconfig: Use the right toolchains. Made warnings fatal again, as there

no warnings to avoid. Removed explicit linking options, as -fsanitize=address does
that automatically.

4:13 PM Changeset in webkit [177702] by timothy_horton@apple.com
  • 6 edits
    4 deletes in trunk/Source/WebKit2

Remove the custom web page preview code
https://bugs.webkit.org/show_bug.cgi?id=139894
<rdar://problem/19333074>

Reviewed by Dan Bernstein.

  • UIProcess/API/Cocoa/WKViewPrivate.h:
  • UIProcess/API/mac/WKView.mm:

(-[WKView _dismissContentRelativeChildWindows]):
(-[WKView _viewForPreviewingURL:initialFrameSize:]): Deleted.
(-[WKView _titleForPreviewOfURL:]): Deleted.
(-[WKView _setPreviewTitle:]): Deleted.
(-[WKView _setPreviewLoading:]): Deleted.
(-[WKView _setPreviewOverrideImage:]): Deleted.
(-[WKView _finishPreviewingURL:withPreviewView:]): Deleted.
(-[WKView _handleClickInPreviewView:URL:]): Deleted.
(-[WKView _shouldUseStandardQuickLookPreview]): Deleted.

  • UIProcess/mac/WKImmediateActionController.h:
  • UIProcess/mac/WKImmediateActionController.mm:

(-[WKImmediateActionController willDestroyView:]):
(-[WKImmediateActionController _clearImmediateActionState]):
(-[WKImmediateActionController immediateActionRecognizerWillPrepare:]):
(-[WKImmediateActionController _defaultAnimationController]):
(-[WKImmediateActionController hidePreview]): Deleted.
(-[WKImmediateActionController setPreviewTitle:]): Deleted.
(-[WKImmediateActionController setPreviewLoading:]): Deleted.
(-[WKImmediateActionController setPreviewOverrideImage:]): Deleted.
(-[WKImmediateActionController _animationControllerForCustomPreview]): Deleted.

  • UIProcess/mac/WKPagePreviewViewController.h: Removed.
  • UIProcess/mac/WKPagePreviewViewController.mm: Removed.
  • UIProcess/mac/WKPreviewPopoverAnimationController.h: Removed.
  • UIProcess/mac/WKPreviewPopoverAnimationController.mm: Removed.
  • WebKit2.xcodeproj/project.pbxproj:

Remove all of the custom web page preview code. Clients that want the custom
preview can implement it by overriding the immediate action.

3:58 PM Changeset in webkit [177701] by andersca@apple.com
  • 3 edits in trunk/Source/WebKit2

Add WebProcessProxy::transformHandlesToObjects
https://bugs.webkit.org/show_bug.cgi?id=139922

Reviewed by Tim Horton.

This will be used to convert between the handles and their respective UI side objects.

  • UIProcess/WebProcessProxy.cpp:

(WebKit::WebProcessProxy::transformHandlesToObjects):
(WebKit::WebProcessProxy::transformObjectsToHandles):
(WebKit::WebProcessProxy::apiObjectByConvertingToHandles): Deleted.

  • UIProcess/WebProcessProxy.h:
3:35 PM Changeset in webkit [177700] by andersca@apple.com
  • 6 edits in trunk/Source/WebKit2

Add handle transform functions to WebConnection
https://bugs.webkit.org/show_bug.cgi?id=139920

Reviewed by Sam Weinig.

This will be used by the web connection code to convert between objects and their respective handles.

  • Shared/WebConnection.h:
  • UIProcess/WebConnectionToWebProcess.cpp:

(WebKit::WebConnectionToWebProcess::transformHandlesToObjects):
(WebKit::WebConnectionToWebProcess::transformObjectsToHandles):

  • UIProcess/WebConnectionToWebProcess.h:
  • WebProcess/WebConnectionToUIProcess.cpp:

(WebKit::WebConnectionToUIProcess::transformHandlesToObjects):
(WebKit::WebConnectionToUIProcess::transformObjectsToHandles):

  • WebProcess/WebConnectionToUIProcess.h:
3:07 PM Changeset in webkit [177699] by weinig@apple.com
  • 35 edits
    2 moves
    2 deletes in trunk/Source/WebKit2

Rename ImmutableDictionary to API::Dictionary and merge MutableDictionary into it
https://bugs.webkit.org/show_bug.cgi?id=139917

Reviewed by Anders Carlsson.

  • Renames ImmutableDictionary -> API::Dictionary
  • Merges MutableDictionary into the new API::Dictionary

(as a result WKDictionaryIsMutable() now always returns true)

  • File list elided *
3:06 PM Changeset in webkit [177698] by andersca@apple.com
  • 4 edits in trunk/Source/WebKit2

Add WebProcess::transformObjectsToHandles
https://bugs.webkit.org/show_bug.cgi?id=139918

Reviewed by Tim Horton.

This function will be used for transforming bundle objects to their respective handle types
when sending messages to the UI process.

  • WebProcess/WebProcess.cpp:

(WebKit::WebProcess::transformHandlesToObjects):
(WebKit::WebProcess::transformObjectsToHandles):

  • WebProcess/WebProcess.h:
  • WebProcess/cocoa/WebProcessCocoa.mm:

(WebKit::WebProcess::transformHandlesToObjects):
(WebKit::WebProcess::transformObjectsToHandles):

2:27 PM Changeset in webkit [177697] by andersca@apple.com
  • 4 edits in trunk/Source

Move dynamic_objc_cast to RetainPtr.h
https://bugs.webkit.org/show_bug.cgi?id=139915

Reviewed by Sam Weinig.

Source/WebKit2:

  • Shared/mac/ObjCObjectGraph.mm:

(WebKit::dynamic_objc_cast): Deleted.

Source/WTF:

  • wtf/RetainPtr.h:

(WTF::dynamic_objc_cast):

2:16 PM Changeset in webkit [177696] by andersca@apple.com
  • 3 edits in trunk/Source/WTF

Re-indent RefPtr.h and RetainPtr.h.

Rubber-stamped by Sam Weinig.

  • wtf/RefPtr.h:
  • wtf/RetainPtr.h:
2:14 PM Changeset in webkit [177695] by andersca@apple.com
  • 3 edits in trunk/Source/WebKit2

Add WebProcess::transformHandlesToObjects
https://bugs.webkit.org/show_bug.cgi?id=139913

Reviewed by Tim Horton.

Rename objectGraphByTransformingObjectsToHandles to transformObjectsToHandles and add
transformHandlesToObjects that will try to convert handles to the actual bundle type objects.

  • WebProcess/WebProcess.h:
  • WebProcess/cocoa/WebProcessCocoa.mm:

(WebKit::WebProcess::transformHandlesToObjects):
(WebKit::WebProcess::transformObjectsToHandles):
(WebKit::WebProcess::objectGraphByTransformingObjectsToHandles): Deleted.

1:53 PM Changeset in webkit [177694] by andersca@apple.com
  • 4 edits in trunk/Source/WebKit2

Add WebProcess::objectGraphByTransformingObjectsToHandles
https://bugs.webkit.org/show_bug.cgi?id=139912

Reviewed by Tim Horton.

This will be used to convert Objective-C object graphs containing bundle types to their handle counterparts.

  • WebProcess/WebProcess.cpp:

(WebKit::WebProcess::transformHandlesToObjects):
(WebKit::WebProcess::apiObjectByConvertingFromHandles): Deleted.
Rename apiObjectByConvertingFromHandles to transformHandlesToObjects.

  • WebProcess/WebProcess.h:
  • WebProcess/cocoa/WebProcessCocoa.mm:

(WebKit::WebProcess::objectGraphByTransformingObjectsToHandles):

12:44 PM Changeset in webkit [177693] by andersca@apple.com
  • 4 edits in trunk/Source/WebKit2

Fix UserMessage::transform to only call shouldTransform once
https://bugs.webkit.org/show_bug.cgi?id=139911

Reviewed by Sam Weinig.

Also, make it possible for transformers to return null objects and change the transformers
to return the passed in object when there's no need to do a transform.

  • Shared/UserData.cpp:

(WebKit::transformGraph):
(WebKit::UserData::transform):
(WebKit::UserData::encode): Deleted.
(WebKit::UserData::decode): Deleted.

  • UIProcess/WebProcessProxy.cpp:

(WebKit::WebProcessProxy::apiObjectByConvertingToHandles):

  • WebProcess/WebProcess.cpp:

(WebKit::WebProcess::apiObjectByConvertingFromHandles):

12:29 PM Changeset in webkit [177692] by weinig@apple.com
  • 107 edits
    6 moves
    1 delete in trunk/Source

Rename WebContext to WebProcessPool
https://bugs.webkit.org/show_bug.cgi?id=139909

Rubber-stamped by Anders Carlsson.

Source/WebCore:

Merge SessionID and SessionIDHash into one file to make auto-generation of
HashMaps with SessionIDs easier.

  • WebCore.xcodeproj/project.pbxproj:
  • loader/cache/MemoryCache.h:
  • page/SessionID.h:

(WTF::SessionIDHash::hash):
(WTF::SessionIDHash::equal):
(WTF::HashTraits<WebCore::SessionID>::emptyValue):
(WTF::HashTraits<WebCore::SessionID>::constructDeletedValue):
(WTF::HashTraits<WebCore::SessionID>::isDeletedValue):

  • page/SessionIDHash.h: Removed.

Source/WebKit2:

  • Major renames:

WebContext -> WebProcessPool
WebContextConfiguration -> WebProcessPoolConfiguration
API::ContextConfiguration -> API::ProcessPoolConfiguration

  • File list elided *
12:15 PM Changeset in webkit [177691] by andersca@apple.com
  • 3 edits in trunk/Source/WebKit2

Only call shouldTransformGraph once per graph
https://bugs.webkit.org/show_bug.cgi?id=139910

Reviewed by Sam Weinig.

Don't call shouldTransformGraph for each node in the object graph. Instead, call it once
and then transform the entire graph if needed.

  • Shared/mac/ObjCObjectGraph.h:
  • Shared/mac/ObjCObjectGraph.mm:

(WebKit::shouldTransformGraph):
(WebKit::transformGraph):
(WebKit::ObjCObjectGraph::transform):
(WebKit::shouldTransform): Deleted.

12:01 PM Changeset in webkit [177690] by ap@apple.com
  • 3 edits in trunk/Tools

TestWebKitAPI build is broken when ASan is enabled
https://bugs.webkit.org/show_bug.cgi?id=139902

Reviewed by Darin Adler.

  • TestWebKitAPI/Configurations/InjectedBundle.xcconfig:
  • TestWebKitAPI/Configurations/TestWebKitAPILibrary.xcconfig:

Inherit linker flags from DebugRelease, which respects ASAN.

11:23 AM Changeset in webkit [177689] by mmaxfield@apple.com
  • 7 edits in trunk/Source/WebCore

platform/mac/editing/input/devanagari-ligature.html is flaky on Yosemite, ligature fails to form
https://bugs.webkit.org/show_bug.cgi?id=138683

Reviewed by Darin Adler.

This patch changes how we check fonts for equality. In particular, this patch adds a
objectForEqualityCheck() to Cocoa's FontPlatformData, and callers should pass this object
to CFEqual() to determine if two platform fonts are equal. This patch also migrates all
call sites to using this function.

I don't want to implement operator==() because there are many cases where the same font
is compared against many others, and this solution is cleaner than caching a comparison
object inside the font object itself.

No new tests because this is covered by platform/mac/editing/input/devanagari-ligature.html.

  • platform/graphics/FontPlatformData.h:
  • platform/graphics/cocoa/FontPlatformDataCocoa.mm:

(WebCore::FontPlatformData::objectForEqualityCheck):

  • platform/graphics/mac/ComplexTextControllerCoreText.mm:

(WebCore::ComplexTextController::collectComplexTextRunsForCharacters):

  • platform/graphics/mac/GlyphPageTreeNodeMac.cpp:

(WebCore::GlyphPage::fill):

  • platform/graphics/mac/SimpleFontDataMac.mm:

(WebCore::SimpleFontData::canRenderCombiningCharacterSequence):

  • platform/spi/cocoa/CoreTextSPI.h:
10:45 AM Changeset in webkit [177688] by mmaxfield@apple.com
  • 2 edits in trunk/Source/WebCore

[SVG -> OTF Converter] Make Placeholder a move-only type
https://bugs.webkit.org/show_bug.cgi?id=139870

Reviewed by Anders Carlsson.

No new tests because there is no behavior change.

  • svg/SVGToOTFFontConversion.cpp:

(WebCore::SVGToOTFFontConverter::Placeholder::Placeholder):
(WebCore::SVGToOTFFontConverter::Placeholder::populate):
(WebCore::SVGToOTFFontConverter::Placeholder::~Placeholder):

10:37 AM Changeset in webkit [177687] by Chris Dumez
  • 5 edits in trunk/Source/WebCore

Move color CSS properties to the new StyleBuilder
https://bugs.webkit.org/show_bug.cgi?id=139898

Reviewed by Antti Koivisto.

Move color CSS properties to the new StyleBuilder by teaching the
StyleBuilder generator how to handle them.

  • css/CSSPropertyNames.in:
  • Move color properties to the new StyleBuilder.
  • css/DeprecatedStyleBuilder.cpp:

(WebCore::DeprecatedStyleBuilder::DeprecatedStyleBuilder):
(WebCore::defaultInitialColor): Deleted.
(WebCore::ApplyPropertyColor::applyInheritValue): Deleted.
(WebCore::ApplyPropertyColor::applyInitialValue): Deleted.
(WebCore::ApplyPropertyColor::applyValue): Deleted.
(WebCore::ApplyPropertyColor::applyColorValue): Deleted.
(WebCore::ApplyPropertyColor::createHandler): Deleted.

  • Drop color properties support from the DeprecatedStyleBuilder.
  • css/makeprop.pl:
  • Teach the StyleBuilder generator how to handle color properties.
  • Use a class for StyleBuilderFunctions instead of a namespace so that it can be made a friend of RenderStyle class.
  • rendering/style/RenderStyle.h:
  • Mark StyleBuilderFunctions class as friend as its functions can set RenderStyle members directly. This is similar to what is already done for StyleBuilderCustom class. The new color properties handling calls private methods on RenderStyle.
  • Mark invalidColor() method as static as it does not require an instance.
10:04 AM Changeset in webkit [177686] by Alan Bujtas
  • 4 edits
    6 adds in trunk

Incorrect dashed and dotted border painting.
https://bugs.webkit.org/show_bug.cgi?id=139872
rdar://problem/18024205

Reviewed by Simon Fraser.

This patch makes dashed/dotted border painting symmetric and consistent.
It also works with subpixel positioning.

Source/WebCore:

Tests: fast/borders/border-painting-correctness-dashed.html

fast/borders/border-painting-correctness-dotted.html

  • platform/graphics/cg/GraphicsContextCG.cpp:

(WebCore::GraphicsContext::drawLine):

  • rendering/RenderObject.cpp:

(WebCore::RenderObject::drawLineForBoxSide):

LayoutTests:

  • fast/borders/border-painting-correctness-dashed-expected.html: Added.
  • fast/borders/border-painting-correctness-dashed.html: Added.
  • fast/borders/border-painting-correctness-dotted-expected.html: Added.
  • fast/borders/border-painting-correctness-dotted.html: Added.
  • fast/borders/resources/border-painting-correctness-dashed-expected.png: Added.
  • fast/borders/resources/border-painting-correctness-dotted-expected.png: Added.
8:52 AM Changeset in webkit [177685] by andersca@apple.com
  • 5 edits in trunk/Source/WebKit2

Use a Transformer for UserData as well, to avoid always copying arrays and dictionaries
https://bugs.webkit.org/show_bug.cgi?id=139896

Reviewed by Darin Adler.

  • Shared/UserData.cpp:

(WebKit::shouldTransform):
(WebKit::UserData::transform):

  • Shared/UserData.h:

(WebKit::UserData::Transformer::~Transformer):

  • UIProcess/WebProcessProxy.cpp:

(WebKit::WebProcessProxy::apiObjectByConvertingToHandles):

  • WebProcess/WebProcess.cpp:

(WebKit::WebProcess::apiObjectByConvertingFromHandles):

2:29 AM Changeset in webkit [177684] by g.czajkowski@samsung.com
  • 2 edits in trunk/LayoutTests

editing/spelling/grammar-paste.html is flaky in debug after r177682
https://bugs.webkit.org/show_bug.cgi?id=139903

Unreviewed gardening.

Mark it as flaky.

1:36 AM Changeset in webkit [177683] by g.czajkowski@samsung.com
  • 2 edits in trunk/LayoutTests

Add proper expectations for two spelling/ tests after r177682

Unreviewed gardening.

12:26 AM Changeset in webkit [177682] by g.czajkowski@samsung.com
  • 4 edits in trunk/LayoutTests

shouldBecomeEqual() behaves as shouldBe() if the testing expression returns the expected value
https://bugs.webkit.org/show_bug.cgi?id=133939

Reviewed by Darin Adler.

First call of eval(expression) inside shouldBecomeEqual was always
made synchronously. If the testing expression returns the same value
as expected one then shouldBecomeEqual() will immediately report PASS,
for example,

shouldBecomeEqual(internals.hasSpellingMarker("wellcome"), false);

in consequence, assuming asynchronous path of spellchecking, spelling markers
may appear after a while.

The bug was caused by checking a condition at the beginning of shouldBecomeEqual(),
before calling a timer. As a result, queued asynchronous events doesn't effect
this checking.

This fix reveals new bugs in execDeleteCommand() behaviour, two tests need
to marked as failure now.

  • resources/js-test-pre.js:

(.condition):
(shouldBecomeEqual):
(shouldBecomeDifferent):

  • resources/js-test.js:

(.condition):
(shouldBecomeEqual):
(shouldBecomeDifferent):
Always check a condition on timer.

Dec 22, 2014:

11:28 PM Changeset in webkit [177681] by Chris Dumez
  • 3 edits in trunk/Source/WebCore

[iOS] Log which type of resources are commonly loaded using FeatureCounter
https://bugs.webkit.org/show_bug.cgi?id=139890

Reviewed by Darin Adler.

Log which type of resources are commonly loaded using FeatureCounter
(e.g. stylesheets, scripts, fonts, images, ...).

  • loader/SubresourceLoader.cpp:

(WebCore::logResourceLoadedUsingFeatureCounter):
(WebCore::SubresourceLoader::didFinishLoading):

  • platform/FeatureCounterKeys.h:
11:23 PM Changeset in webkit [177680] by Chris Dumez
  • 5 edits in trunk/Source/WebCore

[iOS] Log using FeatureCounter when a PacheCache fails due to memory pressure
https://bugs.webkit.org/show_bug.cgi?id=139874
<rdar://problem/19255690>

Reviewed by Darin Adler.

Log using FeatureCounter when a PacheCache fails due to memory
pressure. To detect this, a flag is added to HistoryItem to mark
items that are no longer in the page becaused they were pruned
(either because of a low memory handling or because the page cache
reached its maximum capacity).

  • history/HistoryItem.cpp:

(WebCore::HistoryItem::HistoryItem):

  • history/HistoryItem.h:
  • history/PageCache.cpp:

(WebCore::PageCache::add):
(WebCore::PageCache::take):
(WebCore::PageCache::get):
(WebCore::PageCache::prune):

  • platform/FeatureCounterKeys.h:
11:17 PM Changeset in webkit [177679] by ap@apple.com
  • 2 edits in trunk/LayoutTests

platform/mac-wk2/tiled-drawing/scrolling/frames/frameset-frame-scrollability.html is flaky
https://bugs.webkit.org/show_bug.cgi?id=139901

  • platform/mac-wk2/TestExpectations: Mark it as such.
11:15 PM Changeset in webkit [177678] by commit-queue@webkit.org
  • 4 edits
    6 deletes in trunk

Unreviewed, rolling out r177658.
https://bugs.webkit.org/show_bug.cgi?id=139900

Caused many assertion failures (Requested by ap on #webkit).

Reverted changeset:

"Incorrect dashed and dotted border painting."
https://bugs.webkit.org/show_bug.cgi?id=139872
http://trac.webkit.org/changeset/177658

7:24 PM Changeset in webkit [177677] by andersca@apple.com
  • 3 edits in trunk/Source/WebKit2

Make it possible to transform Objective-C object graphs, replacing certain elements with others
https://bugs.webkit.org/show_bug.cgi?id=139893

Reviewed by Sam Weinig.

  • Shared/mac/ObjCObjectGraph.h:

(WebKit::ObjCObjectGraph::Transformer::~Transformer):

  • Shared/mac/ObjCObjectGraph.mm:

(WebKit::dynamic_objc_cast):
(WebKit::shouldTransform):
(WebKit::ObjCObjectGraph::transform):

7:21 PM Changeset in webkit [177676] by ljaehun.lim@samsung.com
  • 4 edits in trunk/Source

Unreviewed build fix after r177661 and r177662

Source/WebCore:

  • dom/SecurityOriginPolicy.cpp: Modify #include path.

Source/WebKit2:

  • CMakeLists.txt: Add APIPageGroupHandle.cpp to the source list.
6:48 PM Changeset in webkit [177675] by mark.lam@apple.com
  • 2 edits in trunk/Source/JavaScriptCore

Use ctiPatchCallByReturnAddress() in JITOperations.cpp.
<https://webkit.org/b/139892>

Reviewed by Michael Saboff.

The code in JITOperations.cpp sometimes calls RepatchBuffer::relinkCallerToFunction()
directly, and sometimes uses a helper function, ctiPatchCallByReturnAddress().
This patch changes it to use the helper function consistently.

  • jit/JITOperations.cpp:
6:24 PM Changeset in webkit [177674] by mrowe@apple.com
  • 2 edits in trunk/Source/WebCore

[Mac] Engineering builds of WebCore on OS X 10.8 and 10.9 shouldn't build with -gline-tables-only
<https://webkit.org/b/139883> / <rdar://problem/19297261>

Reviewed by Alexey Proskuryakov.

  • Configurations/DebugRelease.xcconfig: Override the setting using conditional settings

so that they take precedence over the conditional settings in Base.xcconfig.

5:59 PM Changeset in webkit [177673] by andersca@apple.com
  • 3 edits in trunk/Source/WebKit2

Convert pages and page groups to their respective handle objects
https://bugs.webkit.org/show_bug.cgi?id=139891

Reviewed by Tim Horton.

  • UIProcess/WebProcessProxy.cpp:

(WebKit::WebProcessProxy::apiObjectByConvertingToHandles):

  • WebProcess/WebProcess.cpp:

(WebKit::WebProcess::apiObjectByConvertingFromHandles):

5:54 PM Changeset in webkit [177672] by Brent Fulgham
  • 2 edits in trunk/Source/WebKit

[Win] Build fix after r177668.

  • WebKit.vcxproj/WebKitExportGenerator/WebKitExports.def.in: Add missing export definition

for new symbols.

5:32 PM Changeset in webkit [177671] by andersca@apple.com
  • 12 edits in trunk/Source/WebKit2

Add a bunch of missing header includes in preparation for making toImpl require the complete type
https://bugs.webkit.org/show_bug.cgi?id=139887

Reviewed by Andreas Kling.

  • WebProcess/InjectedBundle/API/c/WKBundle.cpp:
  • WebProcess/InjectedBundle/API/c/WKBundleFrame.cpp:
  • WebProcess/InjectedBundle/API/c/WKBundlePage.cpp:
  • WebProcess/InjectedBundle/API/c/WKBundlePageOverlay.cpp:
  • WebProcess/InjectedBundle/InjectedBundlePageContextMenuClient.cpp:
  • WebProcess/InjectedBundle/InjectedBundlePageDiagnosticLoggingClient.cpp:
  • WebProcess/InjectedBundle/InjectedBundlePageEditorClient.cpp:
  • WebProcess/InjectedBundle/InjectedBundlePageFormClient.cpp:
  • WebProcess/InjectedBundle/InjectedBundlePageLoaderClient.cpp:
  • WebProcess/InjectedBundle/InjectedBundlePageResourceLoadClient.cpp:
  • WebProcess/InjectedBundle/InjectedBundlePageUIClient.cpp:
5:23 PM Changeset in webkit [177670] by ap@apple.com
  • 2 edits in trunk/Source/WebCore

Unreviewed build fix.

  • WebCore.exp.in: Move SQLiteDatabaseTracker::setClient export out of PLATFORM(IOS) section.
5:22 PM Changeset in webkit [177669] by bshafiei@apple.com
  • 1 copy in tags/Safari-600.1.4.13.3

New tag.

4:54 PM Changeset in webkit [177668] by andersca@apple.com
  • 2 edits in trunk/Source/WebCore

Remove two functions from SecurityOrigin
https://bugs.webkit.org/show_bug.cgi?id=139885

Reviewed by Andreas Kling.

  • page/SecurityOrigin.h:

(WebCore::SecurityOrigin::canAccessPasswordManager): Deleted.
(WebCore::SecurityOrigin::canAccessFileSystem): Deleted.

4:50 PM Changeset in webkit [177667] by mark.lam@apple.com
  • 2 edits in trunk/Source/JavaScriptCore

Fix some typos in a comment.
<https://webkit.org/b/139882>

Reviewed by Michael Saboff.

  • jit/JITPropertyAccess.cpp:

(JSC::JIT::emit_op_get_by_val):

4:48 PM Changeset in webkit [177666] by Chris Dumez
  • 7 edits in trunk/Source

[iOS] Log using FeatureCounter user-triggered zooming
https://bugs.webkit.org/show_bug.cgi?id=139879
<rdar://problem/19329130>

Reviewed by Benjamin Poulain.

Log using FeatureCounter user-triggered zooming on iOS.

Source/WebCore:

  • WebCore.exp.in:
  • WebCore.xcodeproj/project.pbxproj:
  • platform/FeatureCounter.h:
  • platform/FeatureCounterKeys.h:

Source/WebKit2:

  • WebProcess/WebPage/ios/WebPageIOS.mm:

(WebKit::WebPage::willStartUserTriggeredZooming):

4:45 PM Changeset in webkit [177665] by matthew_hanson@apple.com
  • 5 edits in branches/safari-600.3-branch/Source

Versioning.

4:38 PM Changeset in webkit [177664] by benjamin@webkit.org
  • 8 edits
    28 adds in trunk

Form elements should match :valid and :invalid based on their associated elements
https://bugs.webkit.org/show_bug.cgi?id=139850

Patch by Benjamin Poulain <bpoulain@apple.com> on 2014-12-22
Reviewed by Darin Adler.

Source/WebCore:

In the latest HTML spec, form elements can match :valid or :invalid based
on their associated element.

The tricky part is that object lifetime is a mess. When elements are associated
with forms by the parser, the form is set by the constructor of HTMLFormControlElement.
At that point, the real object has not been initialized yet which
makes it impossible to find its validity.

To work around the lifetime problem, the code of HTMLFormControlElement::didChangeForm()
uses m_willValidateInitialized and m_willValidate direclty instead
of invoking willValidate(). That way we don't try to validate an incomplete object.

When the object really validates, HTMLFormControlElement::setNeedsWillValidateCheck()
takes care of updating the form.

Tests: fast/css/pseudo-invalid-form-and-fieldset-basics.html

fast/css/pseudo-invalid-form-basics.html
fast/css/pseudo-invalid-form-dynamically-created-basics.html
fast/css/pseudo-invalid-form-invalidation-optimization.html
fast/css/pseudo-valid-form-and-fieldset-basics.html
fast/css/pseudo-valid-form-basics.html
fast/css/pseudo-valid-form-dynamically-created-basics.html
fast/css/pseudo-valid-form-invalidation-optimization.html
fast/selectors/invalid-form-style-update-1.html
fast/selectors/invalid-form-style-update-2.html
fast/selectors/invalid-form-style-update-3.html
fast/selectors/valid-form-style-update-1.html
fast/selectors/valid-form-style-update-2.html
fast/selectors/valid-form-style-update-3.html

  • css/StyleResolver.cpp:

(WebCore::StyleResolver::canShareStyleWithControl):
(WebCore::StyleResolver::canShareStyleWithElement):
HTMLFormElement is not a FormControl, we have to generalize the test
for :valid/:invalid with style sharing.

  • html/HTMLFormControlElement.cpp:

(WebCore::HTMLFormControlElement::~HTMLFormControlElement):
Since we now have willChangeForm(), didChangeForm(), we have to null
the form ourself, as documented by FormAssociatedElement.

(WebCore::HTMLFormControlElement::setNeedsWillValidateCheck):
(WebCore::HTMLFormControlElement::willChangeForm):
(WebCore::HTMLFormControlElement::didChangeForm):
(WebCore::HTMLFormControlElement::updateValidity):
Update the owner form when any of the associated form element changes
to invalid.

  • html/HTMLFormControlElement.h:
  • html/HTMLFormElement.cpp:

(WebCore::HTMLFormElement::registerInvalidAssociatedFormControl):
(WebCore::HTMLFormElement::removeInvalidAssociatedFormControlIfNeeded):
(WebCore::HTMLFormElement::matchesValidPseudoClass):
(WebCore::HTMLFormElement::matchesInvalidPseudoClass):
This is very similar to the code of fieldset, but we have much weaker
invariants due to the insane way FormAssociatedElement initializes.

  • html/HTMLFormElement.h:
  • html/HTMLInputElement.cpp:

(WebCore::HTMLInputElement::initializeInputType):
For completeness, we should always validate when changing the type.
The code was only doing that when the element was inserted into the tree,
that was too fragile.

LayoutTests:

The style update tests are largely based on the fieldset test suite.
The other tests are for various aspects of the patch.

  • fast/css/pseudo-invalid-form-and-fieldset-basics-expected.html: Added.
  • fast/css/pseudo-invalid-form-and-fieldset-basics.html: Added.
  • fast/css/pseudo-invalid-form-basics-expected.html: Added.
  • fast/css/pseudo-invalid-form-basics.html: Added.
  • fast/css/pseudo-invalid-form-dynamically-created-basics-expected.html: Added.
  • fast/css/pseudo-invalid-form-dynamically-created-basics.html: Added.
  • fast/css/pseudo-invalid-form-invalidation-optimization-expected.txt: Added.
  • fast/css/pseudo-invalid-form-invalidation-optimization.html: Added.
  • fast/css/pseudo-valid-form-and-fieldset-basics-expected.html: Added.
  • fast/css/pseudo-valid-form-and-fieldset-basics.html: Added.
  • fast/css/pseudo-valid-form-basics-expected.html: Added.
  • fast/css/pseudo-valid-form-basics.html: Added.
  • fast/css/pseudo-valid-form-dynamically-created-basics-expected.html: Added.
  • fast/css/pseudo-valid-form-dynamically-created-basics.html: Added.
  • fast/css/pseudo-valid-form-invalidation-optimization-expected.txt: Added.
  • fast/css/pseudo-valid-form-invalidation-optimization.html: Added.
  • fast/selectors/invalid-form-style-update-1-expected.txt: Added.
  • fast/selectors/invalid-form-style-update-1.html: Added.
  • fast/selectors/invalid-form-style-update-2-expected.txt: Added.
  • fast/selectors/invalid-form-style-update-2.html: Added.
  • fast/selectors/invalid-form-style-update-3-expected.txt: Added.
  • fast/selectors/invalid-form-style-update-3.html: Added.
  • fast/selectors/valid-form-style-update-1-expected.txt: Added.
  • fast/selectors/valid-form-style-update-1.html: Added.
  • fast/selectors/valid-form-style-update-2-expected.txt: Added.
  • fast/selectors/valid-form-style-update-2.html: Added.
  • fast/selectors/valid-form-style-update-3-expected.txt: Added.
  • fast/selectors/valid-form-style-update-3.html: Added.
4:02 PM Changeset in webkit [177663] by andersca@apple.com
  • 2 edits in trunk/Source/WebCore

Try to fix the Windows build.

  • dom/DOMAllInOne.cpp:
3:56 PM Changeset in webkit [177662] by andersca@apple.com
  • 7 edits
    2 copies in trunk/Source/WebKit2

Add an API::PageGroupHandle object
https://bugs.webkit.org/show_bug.cgi?id=139881

Reviewed by Sam Weinig.

API::PageGroupHandle won't be exposed to any API, it will only be used to facilitate passing page groups
between the UI and web process inside UserData objects.

  • Shared/API/APIObject.h:
  • Shared/API/APIPageGroupHandle.cpp: Added.

(API::PageGroupHandle::create):
(API::PageGroupHandle::~PageGroupHandle):
(API::PageGroupHandle::PageGroupHandle):

  • Shared/API/APIPageGroupHandle.h: Added.

(API::PageGroupHandle::webPageGroupData):

  • Shared/UserData.cpp:

(WebKit::UserData::UserData):
(WebKit::UserData::encode):
(WebKit::UserData::decode):

  • Shared/UserData.h:
  • UIProcess/WebPageGroup.h:

(WebKit::WebPageGroup::data):

  • UIProcess/WebProcessProxy.cpp:

(WebKit::WebProcessProxy::apiObjectByConvertingToHandles):

  • WebKit2.xcodeproj/project.pbxproj:
3:42 PM Changeset in webkit [177661] by andersca@apple.com
  • 16 edits
    2 adds in trunk/Source/WebCore

Add a SecurityOriginPolicy class
https://bugs.webkit.org/show_bug.cgi?id=139875

Reviewed by Sam Weinig.

This is the first step towards making SecurityOrigin immutable. SecurityOriginPolicy will hold a
SecurityOrigin and will handle things that would otherwise mutate SecurityOrigin.

  • CMakeLists.txt:
  • WebCore.exp.in:
  • WebCore.vcxproj/WebCore.vcxproj:
  • WebCore.vcxproj/WebCore.vcxproj.filters:
  • WebCore.xcodeproj/project.pbxproj:
  • dom/DOMImplementation.cpp:

(WebCore::DOMImplementation::createDocument):
(WebCore::DOMImplementation::createHTMLDocument):

  • dom/Document.cpp:

(WebCore::Document::create):
(WebCore::Document::open):
(WebCore::Document::cloneDataFromDocument):
(WebCore::Document::initSecurityContext):

  • dom/ScriptExecutionContext.h:
  • dom/SecurityContext.cpp:

(WebCore::SecurityContext::setSecurityOriginPolicy):
(WebCore::SecurityContext::securityOrigin):
(WebCore::SecurityContext::isSecureTransitionTo):
(WebCore::SecurityContext::enforceSandboxFlags):
(WebCore::SecurityContext::setSecurityOrigin): Deleted.

  • dom/SecurityContext.h:

(WebCore::SecurityContext::securityOriginPolicy):
(WebCore::SecurityContext::securityOrigin): Deleted.

  • dom/SecurityOriginPolicy.cpp: Added.

(WebCore::SecurityOriginPolicy::create):
(WebCore::SecurityOriginPolicy::SecurityOriginPolicy):
(WebCore::SecurityOriginPolicy::~SecurityOriginPolicy):

  • dom/SecurityOriginPolicy.h: Added.

(WebCore::SecurityOriginPolicy::origin):

  • loader/DocumentWriter.cpp:

(WebCore::DocumentWriter::begin):

  • workers/WorkerGlobalScope.cpp:

(WebCore::WorkerGlobalScope::WorkerGlobalScope):

  • xml/XMLHttpRequest.cpp:

(WebCore::XMLHttpRequest::responseXML):

  • xml/XMLTreeViewer.cpp:

(WebCore::XMLTreeViewer::transformDocumentToTreeView):

  • xml/XSLTProcessor.cpp:

(WebCore::XSLTProcessor::createDocumentFromSource):

3:30 PM Changeset in webkit [177660] by ap@apple.com
  • 2 edits in trunk/LayoutTests

compositing/rtl/rtl-fixed-overflow-scrolled.html had a 1600x1200 expected PNG
accidentally checked in, let's land 800x600 again.

  • platform/mac/compositing/rtl/rtl-fixed-overflow-scrolled-expected.png:
3:27 PM Changeset in webkit [177659] by Simon Fraser
  • 2 edits in trunk/Tools

Sort the Xcode project file.

  • TestWebKitAPI/TestWebKitAPI.xcodeproj/project.pbxproj:
3:18 PM Changeset in webkit [177658] by Alan Bujtas
  • 4 edits
    6 adds in trunk

Incorrect dashed and dotted border painting.
https://bugs.webkit.org/show_bug.cgi?id=139872
rdar://problem/18024205

Reviewed by Simon Fraser.

This patch makes dashed/dotted border painting symmetric and consistent.
It also works with subpixel positioning.

Source/WebCore:

Tests: fast/borders/border-painting-correctness-dashed.html

fast/borders/border-painting-correctness-dotted.html

  • platform/graphics/cg/GraphicsContextCG.cpp:

(WebCore::GraphicsContext::drawLine):

  • rendering/RenderObject.cpp:

(WebCore::RenderObject::drawLineForBoxSide):

LayoutTests:

  • fast/borders/border-painting-correctness-dashed-expected.html: Added.
  • fast/borders/border-painting-correctness-dashed.html: Added.
  • fast/borders/border-painting-correctness-dotted-expected.html: Added.
  • fast/borders/border-painting-correctness-dotted.html: Added.
  • fast/borders/resources/border-painting-correctness-dashed-expected.png: Added.
  • fast/borders/resources/border-painting-correctness-dotted-expected.png: Added.
3:09 PM Changeset in webkit [177657] by mark.lam@apple.com
  • 2 edits in trunk/Source/JavaScriptCore

Assert that Array elements not copied when changing shape to ArrayStorage type are indeed holes.
<https://webkit.org/b/138118>

Reviewed by Michael Saboff.

  • runtime/JSObject.cpp:

(JSC::JSObject::convertInt32ToArrayStorage):
(JSC::JSObject::convertDoubleToArrayStorage):
(JSC::JSObject::convertContiguousToArrayStorage):

2:17 PM Changeset in webkit [177656] by timothy_horton@apple.com
  • 4 edits in trunk/Source/WebCore

TextIndicatorWindow is larger than it needs to be, especially when not bouncing
https://bugs.webkit.org/show_bug.cgi?id=139876
<rdar://problem/19311017>

Reviewed by Sam Weinig.

  • page/TextIndicator.cpp:

(WebCore::TextIndicator::wantsBounce):
(WebCore::TextIndicator::wantsContentCrossfade):
(WebCore::TextIndicator::wantsFadeIn):

  • page/TextIndicator.h:
  • page/mac/TextIndicatorWindow.mm:

(-[WebTextIndicatorView initWithFrame:textIndicator:margin:]):
(-[WebTextIndicatorView _animationDuration]):
(-[WebTextIndicatorView present]):
(WebCore::TextIndicatorWindow::setTextIndicator):
(-[WebTextIndicatorView _textIndicatorWantsBounce]): Deleted.
(-[WebTextIndicatorView _textIndicatorWantsContentCrossfade]): Deleted.
(-[WebTextIndicatorView _textIndicatorWantsFadeIn]): Deleted.
Move the various switches over TextIndicatorPresentationTransition into TextIndicator.
Adjust the margin; only inflate by the shadow if we're not bouncing.
If we are bouncing, fix the math to more tightly fit the bounce.
Previously we were inflating by way too much, and even when we didn't need to.

1:45 PM Changeset in webkit [177655] by Lucas Forschler
  • 2 edits in trunk/Tools

Update apple-mac-114 to bot629.

Unreviewed.

  • BuildSlaveSupport/build.webkit.org-config/config.json:
1:37 PM Changeset in webkit [177654] by Lucas Forschler
  • 2 edits in trunk/Tools

Replace Apple bot104 with bot124.

Unreviewed.

  • BuildSlaveSupport/build.webkit.org-config/config.json:
1:20 PM Changeset in webkit [177653] by ap@apple.com
  • 2 edits in trunk/LayoutTests

REGRESSION: fast/xmlhttprequest/xmlhttprequest-recursive-sync-event.html is flaky
https://bugs.webkit.org/show_bug.cgi?id=139840

1:12 PM Changeset in webkit [177652] by dbates@webkit.org
  • 2 edits in trunk/Source/WebKit2

Fix the iOS build after <https://trac.webkit.org/changeset/177639>
(https://bugs.webkit.org/show_bug.cgi?id=139866)

Substitute APISecurityOrigin.h for WebSecurityOrigin.h as the latter was renamed to the former.

  • UIProcess/ios/WKGeolocationProviderIOS.mm:
12:55 PM Changeset in webkit [177651] by Csaba Osztrogonác
  • 2 edits in trunk/Source/WebKit2

URTBF after r177649.

  • CMakeLists.txt:
12:48 PM Changeset in webkit [177650] by Csaba Osztrogonác
  • 7 edits in trunk/Source/WebKit2

Last URTBF after r177639, GTK builds now.

  • UIProcess/API/gtk/WebKitJavascriptResult.cpp:
  • UIProcess/API/gtk/WebKitUIClient.cpp:
  • UIProcess/API/gtk/WebKitUserContentManager.cpp:
  • UIProcess/API/gtk/WebKitUserMediaPermissionRequest.cpp:

(webkitUserMediaPermissionRequestCreate):

  • UIProcess/API/gtk/WebKitUserMediaPermissionRequestPrivate.h:
  • UIProcess/API/gtk/WebKitWebView.cpp:

(webkitWebViewRunJavaScriptCallback):
(webkit_web_view_run_javascript):
(resourcesStreamReadCallback):

12:39 PM Changeset in webkit [177649] by andersca@apple.com
  • 2 edits
    22 moves in trunk/Source/WebKit2

Move more API classes from Shared to Shared/API.

Rubber-stamped by Sam Weinig.

  • Shared/API/APIArray.cpp: Renamed from Source/WebKit2/Shared/APIArray.cpp.
  • Shared/API/APIArray.h: Renamed from Source/WebKit2/Shared/APIArray.h.
  • Shared/API/APIClient.h: Renamed from Source/WebKit2/Shared/APIClient.h.
  • Shared/API/APIData.cpp: Renamed from Source/WebKit2/Shared/APIData.cpp.
  • Shared/API/APIData.h: Renamed from Source/WebKit2/Shared/APIData.h.
  • Shared/API/APIError.cpp: Renamed from Source/WebKit2/Shared/APIError.cpp.
  • Shared/API/APIError.h: Renamed from Source/WebKit2/Shared/APIError.h.
  • Shared/API/APIFrameHandle.cpp: Renamed from Source/WebKit2/Shared/APIFrameHandle.cpp.
  • Shared/API/APIFrameHandle.h: Renamed from Source/WebKit2/Shared/APIFrameHandle.h.
  • Shared/API/APIGeometry.cpp: Renamed from Source/WebKit2/Shared/APIGeometry.cpp.
  • Shared/API/APIGeometry.h: Renamed from Source/WebKit2/Shared/APIGeometry.h.
  • Shared/API/APINumber.h: Renamed from Source/WebKit2/Shared/APINumber.h.
  • Shared/API/APIObject.cpp: Renamed from Source/WebKit2/Shared/APIObject.cpp.
  • Shared/API/APIObject.h: Renamed from Source/WebKit2/Shared/APIObject.h.
  • Shared/API/APIPageHandle.cpp: Renamed from Source/WebKit2/Shared/APIPageHandle.cpp.
  • Shared/API/APIPageHandle.h: Renamed from Source/WebKit2/Shared/APIPageHandle.h.
  • Shared/API/APIString.h: Renamed from Source/WebKit2/Shared/APIString.h.
  • Shared/API/APIURL.h: Renamed from Source/WebKit2/Shared/APIURL.h.
  • Shared/API/APIURLRequest.cpp: Renamed from Source/WebKit2/Shared/APIURLRequest.cpp.
  • Shared/API/APIURLRequest.h: Renamed from Source/WebKit2/Shared/APIURLRequest.h.
  • Shared/API/APIURLResponse.cpp: Renamed from Source/WebKit2/Shared/APIURLResponse.cpp.
  • Shared/API/APIURLResponse.h: Renamed from Source/WebKit2/Shared/APIURLResponse.h.
12:13 PM Changeset in webkit [177648] by ap@apple.com
  • 2 edits in trunk/LayoutTests

Update some inspector test expectations for the new timeout detection.
Also, unmarked some tests that appear stable on bots now.

  • platform/mac/TestExpectations:
12:13 PM Changeset in webkit [177647] by Chris Dumez
  • 6 edits in trunk/Source/WebCore

[iOS] Log using FeatureCounter when a PacheCache entry is not reused because it expired
https://bugs.webkit.org/show_bug.cgi?id=139869
<rdar://problem/19255690>

Reviewed by Darin Adler.

Log using FeatureCounter when a PacheCache entry is not reused because
it expired.

  • history/PageCache.cpp:

(WebCore::PageCache::take):
(WebCore::PageCache::get):

  • history/PageCache.h:
  • loader/FrameLoader.cpp:

(WebCore::FrameLoader::commitProvisionalLoad):
(WebCore::FrameLoader::loadDifferentDocumentItem):

  • loader/HistoryController.cpp:

(WebCore::HistoryController::invalidateCurrentItemCachedPage):

  • platform/FeatureCounterKeys.h:
12:13 PM Changeset in webkit [177646] by Csaba Osztrogonác
  • 2 edits in trunk/Source/WebKit2

One more URTBF after r177639 to try to make GTK happy.

  • UIProcess/API/gtk/WebKitUserContentManager.cpp:
12:10 PM Changeset in webkit [177645] by bshafiei@apple.com
  • 2 edits in branches/safari-600.1.4.13-branch/Source/WebKit2

Merged r177463. rdar://problem/19327421

12:08 PM Changeset in webkit [177644] by bshafiei@apple.com
  • 2 edits in branches/safari-600.1.4.13-branch/Source/WebKit2

Merged r173973. rdar://problem/19327404

12:08 PM Changeset in webkit [177643] by Csaba Osztrogonác
  • 2 edits in trunk/Source/WebKit2

URTBF after r177639 to try to make GTK happy.

  • UIProcess/API/gtk/WebKitJavascriptResult.cpp:
12:05 PM Changeset in webkit [177642] by Antti Koivisto
  • 2 edits in trunk/Source/WebCore

Try to fix the windows build.

  • platform/graphics/win/SimpleFontDataCGWin.cpp:
11:56 AM Changeset in webkit [177641] by timothy_horton@apple.com
  • 2 edits in trunk/Source/WebCore

Adjust TextIndicator shadow parameters
https://bugs.webkit.org/show_bug.cgi?id=139871
<rdar://problem/19283300>

Reviewed by Sam Weinig.

  • page/mac/TextIndicatorWindow.mm:

(-[WebTextIndicatorView initWithFrame:textIndicator:margin:]):
Slightly adjust the shadow and bounce size.

11:47 AM Changeset in webkit [177640] by Csaba Osztrogonác
  • 2 edits in trunk/Tools

URTBF after r177639.

  • TestWebKitAPI/CMakeLists.txt:
11:27 AM Changeset in webkit [177639] by andersca@apple.com
  • 47 edits
    3 moves in trunk/Source/WebKit2

Rename a bunch of API classes and move them to Shared/API
https://bugs.webkit.org/show_bug.cgi?id=139866

Reviewed by Sam Weinig.

  • Shared/API/APISecurityOrigin.h: Renamed from Source/WebKit2/Shared/WebSecurityOrigin.h.

(API::SecurityOrigin::createFromString):
(API::SecurityOrigin::create):
(API::SecurityOrigin::securityOrigin):
(API::SecurityOrigin::SecurityOrigin):

  • Shared/API/APISerializedScriptValue.h: Renamed from Source/WebKit2/Shared/WebSerializedScriptValue.h.

(API::SerializedScriptValue::create):
(API::SerializedScriptValue::adopt):
(API::SerializedScriptValue::deserialize):
(API::SerializedScriptValue::dataReference):
(API::SerializedScriptValue::internalRepresentation):
(API::SerializedScriptValue::SerializedScriptValue):

  • Shared/API/APIUserContentURLPattern.h: Renamed from Source/WebKit2/Shared/WebUserContentURLPattern.h.

(API::UserContentURLPattern::create):
(API::UserContentURLPattern::host):
(API::UserContentURLPattern::scheme):
(API::UserContentURLPattern::isValid):
(API::UserContentURLPattern::matchesURL):
(API::UserContentURLPattern::matchesSubdomains):
(API::UserContentURLPattern::patternString):
(API::UserContentURLPattern::UserContentURLPattern):

  • Shared/API/c/WKSecurityOrigin.cpp:

(WKSecurityOriginGetTypeID):
(WKSecurityOriginCreateFromString):
(WKSecurityOriginCreateFromDatabaseIdentifier):
(WKSecurityOriginCreate):

  • Shared/API/c/WKSerializedScriptValue.cpp:

(WKSerializedScriptValueGetTypeID):
(WKSerializedScriptValueCreate):
(WKSerializedScriptValueCreateWithInternalRepresentation):

  • Shared/API/c/WKSharedAPICast.h:

(WebKit::toCopiedAPI):

  • Shared/API/c/WKUserContentURLPattern.cpp:

(WKUserContentURLPatternGetTypeID):
(WKUserContentURLPatternCreate):

  • Shared/SecurityOriginData.cpp:

(WebKit::performAPICallbackWithSecurityOriginDataVector):

  • Shared/UserData.cpp:

(WebKit::UserData::encode):
(WebKit::UserData::decode):

  • Shared/UserMessageCoders.h:

(WebKit::UserMessageEncoder::baseEncode):
(WebKit::UserMessageDecoder::baseDecode):

  • UIProcess/API/APIUIClient.h:

(API::UIClient::exceededDatabaseQuota):
(API::UIClient::decidePolicyForGeolocationPermissionRequest):
(API::UIClient::decidePolicyForUserMediaPermissionRequest):
(API::UIClient::decidePolicyForNotificationPermissionRequest):

  • UIProcess/API/C/WKNotification.cpp:
  • UIProcess/API/C/WKPage.cpp:

(WKPageSetPageUIClient):

  • UIProcess/API/Cocoa/WKWebView.mm:

(-[WKWebView evaluateJavaScript:completionHandler:]):

  • UIProcess/Cocoa/UIDelegate.h:
  • UIProcess/Cocoa/UIDelegate.mm:

(WebKit::UIDelegate::UIClient::exceededDatabaseQuota):

  • UIProcess/GenericCallback.h:
  • UIProcess/Notifications/WebNotification.cpp:

(WebKit::WebNotification::WebNotification):

  • UIProcess/Notifications/WebNotification.h:

(WebKit::WebNotification::origin):

  • UIProcess/Notifications/WebNotificationManagerProxy.cpp:

(WebKit::WebNotificationManagerProxy::providerDidUpdateNotificationPolicy):
(WebKit::WebNotificationManagerProxy::providerDidRemoveNotificationPolicies):

  • UIProcess/Notifications/WebNotificationManagerProxy.h:
  • UIProcess/Notifications/WebNotificationProvider.cpp:
  • UIProcess/Notifications/WebNotificationProvider.h:
  • UIProcess/PageClient.h:

(WebKit::PageClient::decidePolicyForGeolocationPermissionRequest):

  • UIProcess/WebApplicationCacheManagerProxy.cpp:

(WebKit::WebApplicationCacheManagerProxy::deleteEntriesForOrigin):

  • UIProcess/WebApplicationCacheManagerProxy.h:
  • UIProcess/WebCookieManagerProxy.cpp:
  • UIProcess/WebDatabaseManagerProxy.cpp:

(WebKit::WebDatabaseManagerProxy::didGetDatabasesByOrigin):
(WebKit::WebDatabaseManagerProxy::didGetDatabaseOrigins):
(WebKit::WebDatabaseManagerProxy::deleteDatabaseWithNameForOrigin):
(WebKit::WebDatabaseManagerProxy::deleteDatabasesForOrigin):
(WebKit::WebDatabaseManagerProxy::setQuotaForOrigin):
(WebKit::WebDatabaseManagerProxy::didModifyOrigin):
(WebKit::WebDatabaseManagerProxy::didModifyDatabase):

  • UIProcess/WebDatabaseManagerProxy.h:
  • UIProcess/WebDatabaseManagerProxyClient.cpp:

(WebKit::WebDatabaseManagerProxyClient::didModifyOrigin):
(WebKit::WebDatabaseManagerProxyClient::didModifyDatabase):

  • UIProcess/WebDatabaseManagerProxyClient.h:
  • UIProcess/WebKeyValueStorageManager.cpp:

(WebKit::WebKeyValueStorageManager::getKeyValueStorageOrigins):
(WebKit::WebKeyValueStorageManager::getStorageDetailsByOrigin):
(WebKit::WebKeyValueStorageManager::deleteEntriesForOrigin):

  • UIProcess/WebKeyValueStorageManager.h:
  • UIProcess/WebMediaCacheManagerProxy.cpp:
  • UIProcess/WebOriginDataManagerProxy.cpp:

(WebKit::WebOriginDataManagerProxy::didGetOrigins):
(WebKit::WebOriginDataManagerProxy::deleteEntriesForOrigin):

  • UIProcess/WebOriginDataManagerProxy.h:
  • UIProcess/WebPageProxy.cpp:

(WebKit::WebPageProxy::runJavaScriptInMainFrame):
(WebKit::WebPageProxy::scriptValueCallback):
(WebKit::WebPageProxy::exceededDatabaseQuota):
(WebKit::WebPageProxy::requestGeolocationPermissionForFrame):
(WebKit::WebPageProxy::requestUserMediaPermissionForFrame):
(WebKit::WebPageProxy::requestNotificationPermission):

  • UIProcess/WebPageProxy.h:
  • UIProcess/WebResourceCacheManagerProxy.cpp:

(WebKit::WebResourceCacheManagerProxy::clearCacheForOrigin):

  • UIProcess/WebResourceCacheManagerProxy.h:
  • UIProcess/ios/PageClientImplIOS.h:
  • UIProcess/ios/PageClientImplIOS.mm:

(WebKit::PageClientImpl::decidePolicyForGeolocationPermissionRequest):

  • UIProcess/ios/WKContentView.h:
  • WebKit2.xcodeproj/project.pbxproj:
  • WebProcess/InjectedBundle/API/APIInjectedBundlePageUIClient.h:

(API::InjectedBundle::PageUIClient::didReachApplicationCacheOriginQuota):
(API::InjectedBundle::PageUIClient::didExceedDatabaseQuota):

  • WebProcess/InjectedBundle/API/c/WKBundleFrame.cpp:
  • WebProcess/InjectedBundle/InjectedBundlePageUIClient.cpp:

(WebKit::InjectedBundlePageUIClient::didReachApplicationCacheOriginQuota):
(WebKit::InjectedBundlePageUIClient::didExceedDatabaseQuota):

  • WebProcess/InjectedBundle/InjectedBundlePageUIClient.h:
  • WebProcess/WebCoreSupport/WebChromeClient.cpp:

(WebKit::WebChromeClient::exceededDatabaseQuota):
(WebKit::WebChromeClient::reachedApplicationCacheOriginQuota):

11:21 AM Changeset in webkit [177638] by Chris Dumez
  • 3 edits in trunk/Source/WebCore

Consistently use "FeatureCounter*Key" naming for FeatureCounter keys
https://bugs.webkit.org/show_bug.cgi?id=139867

Reviewed by Darin Adler.

Consistently use "FeatureCounter*Key" naming for FeatureCounter keys.

  • loader/FrameLoader.cpp:

(WebCore::logNavigationWithFeatureCounter):

  • platform/FeatureCounterKeys.h:
11:14 AM Changeset in webkit [177637] by Antti Koivisto
  • 10 edits in trunk/Source/WebCore

Generic font code should not know about SVG font missing glyph
https://bugs.webkit.org/show_bug.cgi?id=139864

Reviewed by Andreas Kling and Myles Maxfield.

The defined missing glyph is an SVG font concept and should be handled in SVG code.

  • platform/graphics/FontGlyphs.cpp:

(WebCore::FontGlyphs::glyphDataForSystemFallback):
(WebCore::FontGlyphs::glyphDataForVariant):

Return null glyph instead of the missing glyph (the missing glyph was already a null glyph in all non-svg-font cases).
Use early return style.

  • platform/graphics/FontGlyphs.h:
  • platform/graphics/SegmentedFontData.cpp:
  • platform/graphics/SimpleFontData.cpp:

(WebCore::SimpleFontData::platformGlyphInit):

  • platform/graphics/SimpleFontData.h:

Remove the missingGlyph member.

  • platform/graphics/WidthIterator.cpp:

(WebCore::WidthIterator::advanceInternal):

Explicitly skip over null glyphs. Before they had non-null fontData and would get skipped implicitly.

  • platform/graphics/mac/SimpleFontDataMac.mm:
  • rendering/svg/SVGTextRunRenderingContext.cpp:

(WebCore::missingGlyphForFont):

Get the missing glyph from the SVG font element.

(WebCore::SVGTextRunRenderingContext::glyphDataForCharacter):

Return the missing glyph if the normal lookup didn't produce results.

  • svg/SVGFontData.cpp:

(WebCore::SVGFontData::initializeFontData):

11:04 AM Changeset in webkit [177636] by weinig@apple.com
  • 17 edits
    1 move
    2 deletes in trunk/Source/WebKit2

Allow the WebContext's history client to coexist with WKWebView's history delegate
https://bugs.webkit.org/show_bug.cgi?id=139857

Reviewed by Anders Carlsson.

  • Renames API::HistoryClient to API::LegacyContextHistoryClient to make it clear what it is for.
  • Re-routes the history client functions from the WebProcessProxy to the WebPageProxy.
  • Add history client functions to API::LoaderClient and implement them for NavigationState's LoaderClient subclass. Removes the need for WKProcessPool to have to setup a history client and forward to the NavigationState.
  • UIProcess/API/APIHistoryClient.h: Removed.
  • UIProcess/API/APILegacyContextHistoryClient.h: Copied from Source/WebKit2/UIProcess/API/APIHistoryClient.h.

(API::LegacyContextHistoryClient::~LegacyContextHistoryClient):
(API::LegacyContextHistoryClient::didNavigateWithNavigationData):
(API::LegacyContextHistoryClient::didPerformClientRedirect):
(API::LegacyContextHistoryClient::didPerformServerRedirect):
(API::LegacyContextHistoryClient::didUpdateHistoryTitle):
(API::LegacyContextHistoryClient::populateVisitedLinks):
(API::HistoryClient::~HistoryClient): Deleted.
(API::HistoryClient::didNavigateWithNavigationData): Deleted.
(API::HistoryClient::didPerformClientRedirect): Deleted.
(API::HistoryClient::didPerformServerRedirect): Deleted.
(API::HistoryClient::didUpdateHistoryTitle): Deleted.
(API::HistoryClient::populateVisitedLinks): Deleted.

  • UIProcess/API/APILoaderClient.h:

(API::LoaderClient::didNavigateWithNavigationData):
(API::LoaderClient::didPerformClientRedirect):
(API::LoaderClient::didPerformServerRedirect):
(API::LoaderClient::didUpdateHistoryTitle):

  • UIProcess/API/C/WKContext.cpp:

(WKContextSetHistoryClient):

  • UIProcess/API/Cocoa/WKProcessPool.mm:

(-[WKProcessPool _initWithConfiguration:]):

  • UIProcess/API/mac/WKView.mm:
  • UIProcess/Cocoa/HistoryClient.h: Removed.
  • UIProcess/Cocoa/HistoryClient.mm: Removed.
  • UIProcess/Cocoa/NavigationState.h:
  • UIProcess/Cocoa/NavigationState.mm:

(WebKit::NavigationState::LoaderClient::didNavigateWithNavigationData):
(WebKit::NavigationState::LoaderClient::didPerformClientRedirect):
(WebKit::NavigationState::LoaderClient::didPerformServerRedirect):
(WebKit::NavigationState::LoaderClient::didUpdateHistoryTitle):
(WebKit::NavigationState::didNavigateWithNavigationData): Deleted.
(WebKit::NavigationState::didPerformClientRedirect): Deleted.
(WebKit::NavigationState::didPerformServerRedirect): Deleted.
(WebKit::NavigationState::didUpdateHistoryTitle): Deleted.

  • UIProcess/WebContext.cpp:

(WebKit::WebContext::WebContext):
(WebKit::WebContext::setHistoryClient):
(WebKit::WebContext::populateVisitedLinks):

  • UIProcess/WebContext.h:

(WebKit::WebContext::historyClient):

  • UIProcess/WebPageProxy.cpp:

(WebKit::WebPageProxy::didNavigateWithNavigationData):
(WebKit::WebPageProxy::didPerformClientRedirect):
(WebKit::WebPageProxy::didPerformServerRedirect):
(WebKit::WebPageProxy::didUpdateHistoryTitle):

  • UIProcess/WebPageProxy.h:
  • UIProcess/WebPageProxy.messages.in:
  • UIProcess/WebProcessProxy.cpp:

(WebKit::WebProcessProxy::didNavigateWithNavigationData): Deleted.
(WebKit::WebProcessProxy::didPerformClientRedirect): Deleted.
(WebKit::WebProcessProxy::didPerformServerRedirect): Deleted.
(WebKit::WebProcessProxy::didUpdateHistoryTitle): Deleted.

  • UIProcess/WebProcessProxy.h:
  • UIProcess/WebProcessProxy.messages.in:
  • WebKit2.xcodeproj/project.pbxproj:
  • WebProcess/WebCoreSupport/WebFrameLoaderClient.cpp:

(WebKit::WebFrameLoaderClient::updateGlobalHistory):
(WebKit::WebFrameLoaderClient::updateGlobalHistoryRedirectLinks):
(WebKit::WebFrameLoaderClient::setTitle):

10:40 AM Changeset in webkit [177635] by ap@apple.com
  • 2 edits in trunk/LayoutTests

Remove a Slow expectation for js/promises-tests/promises-tests-2-3-3.html,
as we now skip the test.

10:15 AM Changeset in webkit [177634] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WebKit2

[GTK] another outdated comment
https://bugs.webkit.org/show_bug.cgi?id=139863

Patch by Michael Catanzaro <Michael Catanzaro> on 2014-12-22
Reviewed by Darin Adler.

Remove outdated comment. We nowadays block untrusted connections by
default.

  • WebProcess/gtk/WebProcessMainGtk.cpp:
9:39 AM Changeset in webkit [177633] by ap@apple.com
  • 2 edits in trunk/LayoutTests

js/promises-tests/promises-tests-2-3-3.html sometimes crashes under DFG::SpeculativeJIT::compile
https://bugs.webkit.org/show_bug.cgi?id=139865
rdar://problem/19326058

7:30 AM Changeset in webkit [177632] by g.czajkowski@samsung.com
  • 2 edits in trunk/Tools

[EFL] MiniBrowser steals web view's focus
https://bugs.webkit.org/show_bug.cgi?id=139482

Reviewed by Gyuyoung Kim.

Although we set focus on ewk_view while MiniBrowser initialization
Ecore steals it in a favor of main window. As a result
focused element on web page is unable to receive events.

  • MiniBrowser/efl/main.c:

(window_create):
Prevent window from stealing web view's focus on start up.

3:31 AM Changeset in webkit [177631] by Manuel Rego Casasnovas
  • 6 edits in trunk/LayoutTests

[CSS Grid Layout] Get rid of legacy WebKitCSSGridLayoutEnabled
https://bugs.webkit.org/show_bug.cgi?id=139860

Reviewed by Sergio Villar Senin.

Remove the missing bits regarding WebKitCSSGridLayoutEnabled.
Most of it was already removed in r168416.

  • fast/css-grid-layout/grid-item-margin-auto-columns-rows-horizontal-ltr.html:
  • fast/css-grid-layout/grid-item-margin-auto-columns-rows-horizontal-rtl.html:
  • fast/css-grid-layout/grid-item-margin-auto-columns-rows-vertical-ltr.html:
  • fast/css-grid-layout/grid-item-margin-auto-columns-rows-vertical-rtl.html:
  • fast/css-grid-layout/grid-shorthand-get-set.html:
2:38 AM Changeset in webkit [177630] by Chris Dumez
  • 5 edits in trunk/Source/WebCore

Move "Auto" CSS properties to the new StyleBuilder
https://bugs.webkit.org/show_bug.cgi?id=139858

Reviewed by Antti Koivisto.

Move "Auto" CSS properties from DeprecatedStyleBuilder to the new
StyleBuilder, by adding support in the generator for using hasAutoXXX()
/ setHasAutoXXX() methods on RenderStyle.

No new tests, no web-exposed behavior change.

  • css/CSSPropertyNames.in:
  • css/DeprecatedStyleBuilder.cpp:

(WebCore::DeprecatedStyleBuilder::DeprecatedStyleBuilder):
(WebCore::ApplyPropertyAuto::setValue): Deleted.
(WebCore::ApplyPropertyAuto::value): Deleted.
(WebCore::ApplyPropertyAuto::hasAuto): Deleted.
(WebCore::ApplyPropertyAuto::setAuto): Deleted.
(WebCore::ApplyPropertyAuto::applyInheritValue): Deleted.
(WebCore::ApplyPropertyAuto::applyInitialValue): Deleted.
(WebCore::ApplyPropertyAuto::applyValue): Deleted.
(WebCore::ApplyPropertyAuto::createHandler): Deleted.

  • css/StyleBuilderCustom.h:

(WebCore::StyleBuilderCustom::applyInitialColumnGap):
(WebCore::StyleBuilderCustom::applyInheritColumnGap):
(WebCore::StyleBuilderCustom::applyValueColumnGap):

  • css/makeprop.pl:

(getNameForMethods):
(getAutoGetter):
(getAutoSetter):
(generateInitialValueSetter):
(generateInheritValueSetter):
(generateValueSetter):

Dec 21, 2014:

6:00 PM Changeset in webkit [177629] by commit-queue@webkit.org
  • 3 edits
    2 adds in trunk

HTMLSelectElement add() should support adding group of options element (HTMLOptGroupElement).
https://bugs.webkit.org/show_bug.cgi?id=139806

Patch by Shivakumar JM <shiva.jm@samsung.com> on 2014-12-21
Reviewed by Darin Adler.

Source/WebCore:

HTMLSelectElement add() should support adding group of options (optgroup) element.

Test: fast/dom/HTMLSelectElement/select-add-optgroup.html

  • html/HTMLSelectElement.cpp:

(WebCore::HTMLSelectElement::add):

LayoutTests:

  • fast/dom/HTMLSelectElement/select-add-optgroup-expected.txt: Added.
  • fast/dom/HTMLSelectElement/select-add-optgroup.html: Added.
5:52 PM Changeset in webkit [177628] by Chris Dumez
  • 2 edits in trunk/Source/WebCore

Fix missing assertion in r177623.
https://bugs.webkit.org/show_bug.cgi?id=139334

Reviewed by Darin Adler.

ValueWithCalculation::setCalculation() was meant to include an
"isCalculation()" assertion but I inadvertently omitted the
ASSERT() around the statement.

  • css/CSSParser.h:

(WebCore::CSSParser::ValueWithCalculation::setCalculation):

3:56 PM Changeset in webkit [177627] by rgabor@webkit.org
  • 3 edits
    2 adds in trunk

AX: Hidden aria table crash
https://bugs.webkit.org/show_bug.cgi?id=139856

Reviewed by Chris Fleizach.

Source/WebCore:

Change axCaption to pointer and check it's value because
AXObjectCache::getOrCreate() can return with nullptr.

Test: accessibility/aria-hidden-crash.html

  • accessibility/AccessibilityTable.cpp:

(WebCore::AccessibilityTable::addChildren):

LayoutTests:

Add layout test to cover this crash.

  • accessibility/aria-hidden-crash-expected.txt: Added.
  • accessibility/aria-hidden-crash.html: Added.
2:11 PM Changeset in webkit [177626] by ap@apple.com
  • 2 edits in trunk/LayoutTests

Update expectations for two WebGL tests to match what bots see.

  • platform/mac/TestExpectations:
9:01 AM Changeset in webkit [177625] by eric.carlson@apple.com
  • 2 edits in trunk/Source/WebKit2

[iOS] avoid WebVideoFullscreenManager crash
https://bugs.webkit.org/show_bug.cgi?id=139852
<rdar://problem/19319302>

Reviewed by Darin Adler.

  • WebProcess/ios/WebVideoFullscreenManager.mm:

(WebKit::WebVideoFullscreenManager::didExitFullscreen): NULL-check m_layerHostingContext and m_page.
(WebKit::WebVideoFullscreenManager::setVideoLayerFrameFenced): NULL-check m_layerHostingContext.

2:49 AM Changeset in webkit [177624] by ddkilzer@apple.com
  • 3 edits in trunk/Source/WebKit2

Fix WebKit2 build with newer clang compiler

Fixes the following build error:

In file included from DerivedSources/WebKit2/WebVideoFullscreenManagerProxyMessageReceiver.cpp:32:
Source/WebKit2/Platform/IPC/HandleMessage.h:16:25: error: non-const lvalue reference to type 'Vector<[3 * ...]>' cannot bind to a temporary of type 'Vector<[3 * ...]>'

(object->*function)(std::get<ArgsIndex>(std::forward<ArgsTuple>(args))...);

~

Source/WebKit2/Platform/IPC/HandleMessage.h:22:5: note: in instantiation of function template specialization 'IPC::callMemberFunctionImpl<WebKit::WebVideoFullscreenManagerProxy, void (WebKit::WebVideoFullscreenManagerProxy::*)(WTF::Vector<std::__1::pair<double, double>, 0, WTF::CrashOnOverflow> &), std::1::tuple<WTF::Vector<std::__1::pair<double, double>, 0, WTF::CrashOnOverflow> >, 0>' requested here

callMemberFunctionImpl(object, function, std::forward<ArgsTuple>(args), ArgsIndicies());

Source/WebKit2/Platform/IPC/HandleMessage.h:120:5: note: in instantiation of function template specialization 'IPC::callMemberFunction<WebKit::WebVideoFullscreenManagerProxy, void (WebKit::WebVideoFullscreenManagerProxy::*)(WTF::Vector<std::__1::pair<double, double>, 0, WTF::CrashOnOverflow> &), std::1::tuple<WTF::Vector<std::__1::pair<double, double>, 0, WTF::CrashOnOverflow> >, std::make_index_sequence<1> >' requested here

callMemberFunction(WTF::move(arguments), object, function);

DerivedSources/WebKit2/WebVideoFullscreenManagerProxyMessageReceiver.cpp:55:14: note: in instantiation of function template specialization 'IPC::handleMessage<Messages::WebVideoFullscreenManagerProxy::SetSeekableRangesVector, WebKit::WebVideoFullscreenManagerProxy, void (WebKit::WebVideoFullscreenManagerProxy::*)(WTF::Vector<std::__1::pair<double, double>, 0, WTF::CrashOnOverflow> &)>' requested here

IPC::handleMessage<Messages::WebVideoFullscreenManagerProxy::SetSeekableRangesVector>(decoder, this, &WebVideoFullscreenManagerProxy::setSeekableRangesVector);


  • UIProcess/ios/WebVideoFullscreenManagerProxy.h:
  • UIProcess/ios/WebVideoFullscreenManagerProxy.mm:

(WebKit::WebVideoFullscreenManagerProxy::setSeekableRangesVector):
Make Vector<std::pair<double, double>>& argument const because a
temporary is not allowed to bind to a non-const reference.

Dec 20, 2014:

6:56 PM Changeset in webkit [177623] by Chris Dumez
  • 4 edits in trunk/Source/WebCore

Get rid of error-prone ReleaseParsedCalcValueCondition argument in CSSParser
https://bugs.webkit.org/show_bug.cgi?id=139334

Reviewed by Darin Adler.

Get rid of error-prone ReleaseParsedCalcValueCondition argument in
CSSParser that determines the the m_parsedCalculation member should
be released.

Instead, introduce a new ValueWithCalculation type which wraps a
CSSParserValue and an optional parsed calculation value. This way,
the parsed calculation value is now associated with its CSSParserValue.
This makes it very difficult to use a parsed calculation value for the
wrong CSSParserValue. The API is also easier to use as developers no
longer need to explicitly indicate if the calculation value should be
released or not.

No new tests, no web-behavior change.

10:28 AM Changeset in webkit [177622] by eric.carlson@apple.com
  • 22 edits in trunk/Source

[iOS] add optimized fullscreen API
https://bugs.webkit.org/show_bug.cgi?id=139833
<rdar://problem/18844486>

Reviewed by Simon Fraser.

Source/JavaScriptCore:

  • Configurations/FeatureDefines.xcconfig: Add ENABLE_VIDEO_PRESENTATION_MODE.

Source/WebCore:

  • Configurations/FeatureDefines.xcconfig: Add ENABLE_VIDEO_PRESENTATION_MODE.
  • Modules/mediacontrols/MediaControlsHost.cpp:

(WebCore::MediaControlsHost::optimizedFullscreenSupported): Deleted.
(WebCore::MediaControlsHost::fullscreenMode): Deleted.

  • Modules/mediacontrols/MediaControlsHost.h:
  • Modules/mediacontrols/MediaControlsHost.idl:
  • Modules/mediacontrols/mediaControlsiOS.js:

(ControllerIOS.prototype.addVideoListeners): Listen for 'webkitpresentationmodechanged'.
(ControllerIOS.prototype.removeVideoListeners): Stop listening for 'webkitpresentationmodechanged'.
(ControllerIOS.prototype.configureInlineControls): Call video element instead of using the now

removed host property.

(ControllerIOS.prototype.updateControls): Ditto.
(ControllerIOS.prototype.presentationMode): New, return webkitPresentationMode when possible.
(ControllerIOS.prototype.handleFullscreenButtonClicked): Use webkitSetPresentationMode when possible.
(ControllerIOS.prototype.handleOptimizedFullscreenButtonClicked): Use video element instead of

using the now removed host method.

(ControllerIOS.prototype.handlePresentationModeChange): New, react to presentation mode changes.
(ControllerIOS.prototype.handleFullscreenChange): Call handlePresentationModeChange.

  • dom/EventNames.h: Add webkitpresentationmodechanged.
  • html/HTMLAttributeNames.in: Add onwebkitpresentationmodechanged.
  • html/HTMLMediaElement.cpp:

(WebCore::HTMLMediaElement::enterFullscreen): Early return when asked to change to the current

mode. Call fullscreenModeChanged() instead of whacking the instance variable directly.

(WebCore::HTMLMediaElement::exitFullscreen): Ditto.

  • html/HTMLMediaElement.h:

(WebCore::HTMLMediaElement::fullscreenModeChanged): Make virtual.

  • html/HTMLVideoElement.cpp:

(WebCore::HTMLVideoElement::parseAttribute): Deal with onwebkitpresentationmodechange.
(WebCore::presentationModeFullscreen): New.
(WebCore::presentationModeOptimized): Ditto.
(WebCore::presentationModeInline): Ditto.
(WebCore::HTMLVideoElement::webkitSupportsPresentationMode): Ditto.
(WebCore::HTMLVideoElement::webkitSetPresentationMode): Ditto.
(WebCore::HTMLVideoElement::webkitPresentationMode): Ditto.
(WebCore::HTMLVideoElement::fullscreenModeChanged): Ditto.

  • html/HTMLVideoElement.h:
  • html/HTMLVideoElement.idl:
  • platform/ios/WebVideoFullscreenInterfaceAVKit.mm:

(WebVideoFullscreenInterfaceAVKit::setIsOptimized): Simplify the logic a bit. Don't report

a mode change when

Source/WebKit/mac:

  • Configurations/FeatureDefines.xcconfig: Add ENABLE_VIDEO_PRESENTATION_MODE.

Source/WebKit2:

  • Configurations/FeatureDefines.xcconfig: Add ENABLE_VIDEO_PRESENTATION_MODE.

Source/WTF:

  • wtf/Platform.h: Define ENABLE_VIDEO_PRESENTATION_MODE.
9:26 AM Changeset in webkit [177621] by ddkilzer@apple.com
  • 42 edits in trunk/Source

Switch from using PLATFORM_NAME to SDK selectors in WebCore, WebInspectorUI, WebKit, WebKit2
<http://webkit.org/b/139463>

Reviewed by Mark Rowe.

Source/JavaScriptCore:

  • Configurations/JavaScriptCore.xcconfig:
  • Simplify SECTORDER_FLAGS.

Source/WebCore:

Also remove all uses of SQLITE3_HEADER_SEARCH_PATHS and
WEBCORE_SQLITE3_HEADER_SEARCH_PATHS which were phased out in
r132859 and needed for Leopard.

  • Configurations/Base.xcconfig:
  • Only set GCC_ENABLE_OBJC_GC, GCC_MODEL_TUNING and TOOLCHAINS on OS X.
  • Set GCC_OPTIMIZATION_LEVEL_normal based on SDK.
  • Simplify SQLITE3_HEADER_SEARCH_PATHS.
  • Configurations/DebugRelease.xcconfig:
  • Only set MACOSX_DEPLOYMENT_TARGET and SDKROOT on OS X.
  • Configurations/Version.xcconfig:
  • Set SYSTEM_VERSION_PREFIX separately for iOS and OS X.
  • Configurations/WebCore.xcconfig:
  • Set EXPORTED_SYMBOLS_FILE_i386, EXPORTED_SYMBOLS_FILE_x86_64, FRAMEWORK_SEARCH_PATHS, INSTALL_PATH, DYLIB_INSTALL_NAME_BASE, OTHER_LDFLAGS, SECTORDER_FLAGS, NORMAL_WEBCORE_FRAMEWORKS_DIR, WEBCORE_FRAMEWORKS_DIR, NORMAL_PRODUCTION_FRAMEWORKS_DIR, PRODUCTION_FRAMEWORKS_DIR, JAVASCRIPTCORE_PRIVATE_HEADERS_DIR_Production and EXCLUDED_SOURCE_FILE_NAMES based on SDK.
  • Make sure DYLIB_INSTALL_NAME_BASE and OTHER_LDFLAGS are overrideable by WebCoreTestSupport.xcconfig.
  • Configurations/WebCoreTestSupport.xcconfig:
  • Set PRIVATE_HEADERS_FOLDER_PATH_Production and INSTALL_PATH_Production based on SDK.
  • Override SECT_ORDER_FLAGS from WebCore.xcconfig.

Source/WebInspectorUI:

  • Configurations/Base.xcconfig:
  • Only set TOOLCHAINS on OS X.
  • Configurations/DebugRelease.xcconfig:
  • Only set MACOSX_DEPLOYMENT_TARGET and SDKROOT on OS X.
  • Configurations/Version.xcconfig:
  • Set SYSTEM_VERSION_PREFIX separately for iOS and OS X.
  • Configurations/WebInspectorUIFramework.xcconfig:
  • Set NORMAL_PRODUCTION_FRAMEWORKS_DIR, PRODUCTION_FRAMEWORKS_DIR and JAVASCRIPTCORE_PRIVATE_HEADERS_DIR_Production by SDK.

Source/WebKit/mac:

  • Configurations/Base.xcconfig:
  • Only set GCC_ENABLE_OBJC_GC, GCC_MODEL_TUNING and TOOLCHAINS on OS X.
  • Set GCC_OPTIMIZATION_LEVEL_normal based on SDK.
  • Configurations/DebugRelease.xcconfig:
  • Only set MACOSX_DEPLOYMENT_TARGET and SDKROOT on OS X.
  • Set WEBKIT_SYSTEM_INTERFACE_LIBRARY based on SDK.
  • Configurations/Version.xcconfig:
  • Set SYSTEM_VERSION_PREFIX separately for iOS and OS X.
  • Configurations/WebKitLegacy.xcconfig:
  • Set EXCLUDED_SOURCE_FILE_NAMES, EXPORTED_SYMBOLS_FILE_x86_64, FRAMEWORK_SEARCH_PATHS, OTHER_CFLAGS, INSTALL_PATH, DYLIB_INSTALL_NAME_BASE, OTHER_LDFLAGS, SECTORDER_FLAGS, NORMAL_WEBKIT_LEGACY_FRAMEWORKS_DIR, WEBKIT_LEGACY_FRAMEWORKS_DIR, NORMAL_PRODUCTION_FRAMEWORKS_DIR, PRODUCTION_FRAMEWORKS_DIR, PRODUCTION_ROOT, WEBCORE_PRIVATE_HEADERS_DIR_Production and WEBKITSYSTEMINTERFACE_STATIC_LIBRARY_HEADERS_FOLDER_PATH_engineering based on SDK.
  • Fix PRODUCTION_ROOT to actually be set properly.
  • Simplify EXPORTED_SYMBOLS_FILE_i386.
  • migrate-headers.sh:
  • Get rid of unused WEBKIT2_FRAMEWORKS_DIR variable.
  • Change WEBCORE_PRIVATE_HEADERS_DIR_macosx_Production to WEBCORE_PRIVATE_HEADERS_DIR_Production based on changes to WebKitLegacy.xcconfig.

Source/WebKit2:

  • Configurations/All.xcconfig:
  • Only set EXCLUDED_SOURCE_FILE_NAMES on iOS.
  • Configurations/Base.xcconfig:
  • Only set FRAMEWORK_CONTENT_DIRECTORY, GCC_ENABLE_OBJC_GC and TOOLCHAINS on OS X.
  • Configurations/BaseLegacyProcess.xcconfig:
  • Only set SKIP_INSTALL on iOS.
  • Configurations/BaseTarget.xcconfig:
  • Set WEBKIT_FRAMEWORKS_DIR and UMBRELLA_FRAMEWORKS_DIR based on SDK, and simply them.
  • Configurations/BaseXPCService.xcconfig:
  • Set INSTALL_PATH_ACTUAL_Normal based on SDK.
  • Configurations/DebugRelease.xcconfig:
  • Only set MACOSX_DEPLOYMENT_TARGET and SDKROOT on OS X.
  • Set WEBKIT_SYSTEM_INTERFACE_LIBRARY based on SDK.
  • Configurations/NetworkProcess.xcconfig:
  • Only set EXCLUDED_SHIM_FILE_NAME and CODE_SIGN_ENTITLEMENTS on iOS.
  • Configurations/NetworkService.Development.xcconfig:
  • Set INFOPLIST_FILE based on SDK.
  • Simplify CODE_SIGN_ENTITLEMENTS for iOS.
  • Configurations/NetworkService.xcconfig:
  • Set INFOPLIST_FILE based on SDK.
  • Simplify CODE_SIGN_ENTITLEMENTS for iOS.
  • Configurations/PluginProcess.xcconfig:
  • Only set EXCLUDED_SHIM_FILE_NAME and SKIP_INSTALL on iOS.
  • Configurations/PluginProcessShim.xcconfig:
  • Set OTHER_LDFLAGS based on SDK.
  • Configurations/PluginService.32.xcconfig:
  • Only set VALID_ARCHS on OS X.
  • Set OTHER_LDFLAGS based on platform-specific values.
  • Only set SKIP_INSTALL on iOS.
  • Configurations/PluginService.64.xcconfig:
  • Set ARCHS based on SDK.
  • Set OTHER_LDFLAGS based on platform-specific values.
  • Only set SKIP_INSTALL on iOS.
  • Configurations/PluginService.Development.xcconfig:
  • Only set SKIP_INSTALL on iOS.
  • Set OTHER_LDFLAGS based on platform-specific values.
  • Configurations/SecItemShim.xcconfig:
  • Set OTHER_LDFLAGS based on platform-specific values.
  • Configurations/Version.xcconfig:
  • Set SYSTEM_VERSION_PREFIX separately for iOS and OS X.
  • Configurations/WebContentProcess.xcconfig:
  • Only set EXCLUDED_SHIM_FILE_NAME on iOS.
  • Only set CODE_SIGN_ENTITLEMENTS on iOS Simulator.
  • Configurations/WebContentService.Development.xcconfig:
  • Set INFOPLIST_FILE and BUNDLE_LOCALIZATION_KEY based on SDK.
  • Only set CODE_SIGN_ENTITLEMENTS on iOS hardware.
  • Set OTHER_LDFLAGS based on platform-specific values.
  • Configurations/WebContentService.xcconfig: Ditto.
  • Configurations/WebKit.xcconfig:
  • Set FRAMEWORK_AND_LIBRARY_LDFLAGS, SECTORDER_FLAGS, and WK_API_AVAILABILITY_ENABLED based on SDK.
  • Set OTHER_LDFLAGS based on platform-specific values.
  • Only set EXCLUDED_SOURCE_FILE_NAMES on iOS.
  • Configurations/WebKit2.xcconfig:
  • Set INSTALL_PATH, INSTALL_PATH_ACTUAL, DYLIB_INSTALL_NAME_BASE, NORMAL_WEBKIT2_FRAMEWORKS_DIR, WEBKIT2_FRAMEWORKS_DIR and NORMAL_PRODUCTION_FRAMEWORKS_DIR based on SDK.
  • Set SKIP_INSTALL on iOS.
  • Configurations/WebProcessShim.xcconfig:
  • Set OTHER_LDFLAGS based on platform-specific values.

Dec 19, 2014:

8:41 PM Changeset in webkit [177620] by mmaxfield@apple.com
  • 2 edits in trunk/Source/WebCore

[SVG -> OTF Converter] Make placeholders more robust
https://bugs.webkit.org/show_bug.cgi?id=139836

Reviewed by Dan Bernstein.

This patch creates a Placeholder object which hides the complexity of
populating offsets to subtables.

No new tests because there is no behavior change.

  • svg/SVGToOTFFontConversion.cpp:

(WebCore::SVGToOTFFontConverter::Placeholder::Placeholder):
(WebCore::SVGToOTFFontConverter::Placeholder::populate):
(WebCore::SVGToOTFFontConverter::Placeholder::~Placeholder):
(WebCore::SVGToOTFFontConverter::appendArabicReplacementSubtable):
(WebCore::SVGToOTFFontConverter::appendGSUBTable):

8:39 PM Changeset in webkit [177619] by jberlin@webkit.org
  • 2 edits in trunk/Source/WebKit2

Build fix.

  • UIProcess/mac/WebInspectorProxyMac.mm:
8:30 PM Changeset in webkit [177618] by jberlin@webkit.org
  • 2 edits in trunk/Source/WebKit2

Simplify the build fixes from r177615 and r177617.

Rubber-stamped by Dan Bernstein.

  • UIProcess/mac/WebInspectorProxyMac.mm:
7:55 PM Changeset in webkit [177617] by jberlin@webkit.org
  • 2 edits in trunk/Source/WebKit2

Another attempt at a speculative build fix.

  • UIProcess/mac/WebInspectorProxyMac.mm:
7:41 PM Changeset in webkit [177616] by Chris Dumez
  • 7 edits
    2 adds in trunk

Fix initial / inherit support for '-webkit-perspective-origin' CSS property
https://bugs.webkit.org/show_bug.cgi?id=139843

Reviewed by Simon Fraser.

Source/WebCore:

Fix initial / inherit support for '-webkit-perspective-origin' CSS
property. These previously had no effect.

This patch updates the code to be consistent with
'-webkit-transform-origin' and gets rid of the code in
DeprecatedStyleBuilder as it isn't used.

Test: fast/css/perspective-origin-initial-inherit.html

  • css/DeprecatedStyleBuilder.cpp:

(WebCore::DeprecatedStyleBuilder::DeprecatedStyleBuilder):
(WebCore::ApplyPropertyPerspectiveOrigin::applyInheritValue): Deleted.
(WebCore::ApplyPropertyPerspectiveOrigin::applyInitialValue): Deleted.
(WebCore::ApplyPropertyPerspectiveOrigin::applyValue): Deleted.
(WebCore::ApplyPropertyPerspectiveOrigin::createHandler): Deleted.

  • css/StyleProperties.cpp:

(WebCore::StyleProperties::getPropertyValue):
(WebCore::StyleProperties::asText):

  • css/StylePropertyShorthand.cpp:

(WebCore::webkitPerspectiveOriginShorthand):
(WebCore::shorthandForProperty):
(WebCore::matchingShorthandsForLonghand):

  • css/StylePropertyShorthand.h:
  • css/StyleResolver.cpp:

(WebCore::StyleResolver::applyProperty):

LayoutTests:

Add layout test to cover 'initial' / 'inherit' support for
'-webkit-perspective-origin' CSS property.

  • fast/css/perspective-origin-initial-inherit-expected.txt: Added.
  • fast/css/perspective-origin-initial-inherit.html: Added.
6:39 PM Changeset in webkit [177615] by jberlin@webkit.org
  • 2 edits in trunk/Source/WebKit2

Speculative build fix after r177610.

  • UIProcess/mac/WebInspectorProxyMac.mm:
6:38 PM Changeset in webkit [177614] by rniwa@webkit.org
  • 6 edits
    1 add in trunk/Websites/perf.webkit.org

Perf dashboard should support authentication via a slave password
https://bugs.webkit.org/show_bug.cgi?id=139837

Reviewed by Andreas Kling.

For historical reasons, perf dashboard conflated builders and build slaves. As a result we ended up
having to add multiple builders with the same password when a single build slave is shared among them.

This patch introduces the concept of build_slave into the perf dashboard to end this madness.

  • init-database.sql: Added build_slave table as well as references to it in builds and reports.
  • public/admin/build-slaves.php: Added.
  • public/admin/builders.php: Added the support for updating passwords.
  • public/include/admin-header.php:

(update_field): Takes an extra argument when a new value needs to be supplied by the caller instead of
being retrieved from $_POST.
(AdministrativePage::render_table): Use array_get to retrieve a value out of the database row since
the raw may not exist (e.g. new_password).
(AdministrativePage::render_form_to_add): Added the support for post_insertion. Don't render the form
control here when this flag evaluates to TRUE.

  • public/include/report-processor.php:

(ReportProcessor::process): Added the logic to authenticate with slaveName and slavePassword if those
values are present in the report. In addition, try authenticating builderName with slavePassword if
builderPassword is not specified. When neither password is specified, exit with BuilderNotFound.
Also insert the slave or the builder whichever is missing after we've successfully authenticated.
(ReportProcessor::construct_build_data): Takes a builder ID and an optional slave ID instead of
a builder row.
(ReportProcessor::store_report): Store the slave ID with the report.
(ReportProcessor::resolve_build_id): Exit with MismatchingBuildSlave when the slave associated with
the matching build is different from what's being reported.

  • tests/api-report.js: Added a bunch of tests to test the new features of /api/report.

(.addSlave): Added.

5:21 PM Changeset in webkit [177613] by Brent Fulgham
  • 2 edits in trunk/Tools

[Win] Revise DRT logic to match Mac.

Rubberstamped by Anders Carlsson.

Update the Windows logic to perform initialization of preferences and WebKit
settings to match Mac. Also make sure that we only ever emit #EOF tokens under
the same circumstances as the Mac version.

  • DumpRenderTree/win/DumpRenderTree.cpp:

(dump):
(resetWebPreferencesToConsistentValues):
(setDefaultsToConsistentValuesForTesting):
(resetWebViewToConsistentStateBeforeTesting):
(sizeWebViewForCurrentTest):
(runTest):
(createWebViewAndOffscreenWindow):
(allocateGlobalControllers):
(prepareConsistentTestingEnvironment):
(main):
(resetDefaultsToConsistentValues): Deleted.

5:11 PM Changeset in webkit [177612] by Chris Dumez
  • 8 edits in trunk/Source/WebKit2

[iOS] Add didEnterFullscreen / didExitFullscreen callbacks to WKUIDelegatePrivate
https://bugs.webkit.org/show_bug.cgi?id=139835
<rdar://problem/19312042>

Reviewed by Anders Carlsson.

Add didEnterFullscreen / didExitFullscreen callbacks to WKUIDelegatePrivate
so that they can be implemented in MobileSafari.

  • UIProcess/API/APIUIClient.h:

(API::UIClient::didEnterFullscreen):
(API::UIClient::didExitFullscreen):

  • UIProcess/API/Cocoa/WKUIDelegatePrivate.h:
  • UIProcess/Cocoa/UIDelegate.h:
  • UIProcess/Cocoa/UIDelegate.mm:

(WebKit::UIDelegate::setDelegate):
(WebKit::UIDelegate::UIClient::didEnterFullscreen):
(WebKit::UIDelegate::UIClient::didExitFullscreen):

  • UIProcess/WebPageProxy.cpp:

(WebKit::WebPageProxy::didEnterFullscreen):
(WebKit::WebPageProxy::didExitFullscreen):

  • UIProcess/WebPageProxy.h:
  • UIProcess/ios/WebVideoFullscreenManagerProxy.mm:

(WebKit::WebVideoFullscreenManagerProxy::didExitFullscreen):
(WebKit::WebVideoFullscreenManagerProxy::didEnterFullscreen):

5:02 PM Changeset in webkit [177611] by Simon Fraser
  • 2 edits in trunk/Source/WebKit/mac

Fix pink background on CoreAnimation plug-ins in WebKit1
https://bugs.webkit.org/show_bug.cgi?id=139845
rdar://problem/19313075

Reviewed by Tim Horton.

r154078 left in some debugging code that set a pink background on plug-in hosting
layers. Remove it.

  • Plugins/Hosted/WebHostedNetscapePluginView.mm:

(-[WebHostedNetscapePluginView createPluginLayer]):

4:57 PM Changeset in webkit [177610] by commit-queue@webkit.org
  • 4 edits in trunk/Source/WebKit2

Web Inspector: Inspector does not reopen correctly after Inspector Process crashes
https://bugs.webkit.org/show_bug.cgi?id=139838

Patch by Joseph Pecoraro <Joseph Pecoraro> on 2014-12-19
Reviewed by Timothy Hatcher.

When the WebContentProcess holding the Inspector Frontend crashes, we
would properly tear down the inspector objects. However, the next time
the inspector is opened, when creating the new page in the Inspector
Page Group, a WebPageProxy is created with a Terminated WebProcessProxy.

That Terminated WebProcessProxy is automatically replaced with the next
load request is given. The newly created process was missing the
message listeners and assumed URL access settings that the
WebInspectorProxy had set on the old process.

So, WebInspectorProxy now listens for and resets the process properties
when the inspector process is recreated.

  • UIProcess/WebInspectorProxy.h:
  • UIProcess/WebInspectorProxy.cpp:

(WebKit::WebInspectorProxy::didRelaunchInspectorPageProcess):
Reset process properties when the process is relaunched.

(WebKit::WebInspectorProxy::createInspectorPage):
(WebKit::WebInspectorProxy::didClose):
Save connection identifier if we need to establish connections later.

  • UIProcess/mac/WebInspectorProxyMac.mm:

(-[WKWebInspectorProxyObjCAdapter didRelaunchProcess]):
(-[WKWebInspectorWKView _didRelaunchProcess]):
(WebKit::WebInspectorProxy::closeTimerFired):
(WebKit::WebInspectorProxy::platformCreateInspectorPage):
Send WebInspectorProxy a message when the process underlying the
WKView is relaunched.

4:52 PM Changeset in webkit [177609] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WebCore

Unreviewed, rolling out r177599.
https://bugs.webkit.org/show_bug.cgi?id=139846

this breaks so many things (Requested by thorton_ on #webkit).

Reverted changeset:

"TextIndicator's window can be pushed down if it intersects
the menu bar"
https://bugs.webkit.org/show_bug.cgi?id=139841
http://trac.webkit.org/changeset/177599

4:46 PM Changeset in webkit [177608] by andersca@apple.com
  • 20 edits in trunk/Source/WebKit2

Get rid of the hardcoded set of structs in the message generation script
https://bugs.webkit.org/show_bug.cgi?id=139844

Reviewed by Tim Horton.

Instead, make it possible to use 'struct' in front of the type name to declare that the type is a struct.

  • DatabaseProcess/DatabaseProcess.messages.in:
  • DatabaseProcess/IndexedDB/DatabaseProcessIDBConnection.messages.in:
  • NetworkProcess/NetworkProcess.messages.in:
  • PluginProcess/PluginProcess.messages.in:
  • PluginProcess/WebProcessConnection.messages.in:
  • Scripts/webkit/messages.py:

(struct_or_class):
(forward_declarations_for_namespace):
(forward_declarations_and_headers):

  • Scripts/webkit/model.py:

(Parameter.init):

  • Scripts/webkit/parser.py:

(parse_parameters_string):

  • UIProcess/Storage/StorageManager.messages.in:
  • UIProcess/WebContext.messages.in:
  • UIProcess/WebPageProxy.messages.in:
  • UIProcess/WebProcessProxy.messages.in:
  • WebProcess/ApplicationCache/WebApplicationCacheManager.messages.in:
  • WebProcess/Databases/IndexedDB/WebIDBServerConnection.messages.in:
  • WebProcess/OriginData/WebOriginDataManager.messages.in:
  • WebProcess/ResourceCache/WebResourceCacheManager.messages.in:
  • WebProcess/WebPage/DrawingArea.messages.in:
  • WebProcess/WebPage/WebPage.messages.in:
  • WebProcess/WebProcess.messages.in:
4:38 PM Changeset in webkit [177607] by roger_fong@apple.com
  • 2 edits in trunk/Tools

Unreviewed. Rollout r177605.

  • EWSTools/start-queue-win.sh:
4:26 PM Changeset in webkit [177606] by matthew_hanson@apple.com
  • 1 copy in tags/Safari-600.3.15

New Tag.

4:26 PM Changeset in webkit [177605] by roger_fong@apple.com
  • 2 edits in trunk/Tools

Unreviewed. Try running EWS scripts without —no-confirm, since it defaults to true.

  • EWSTools/start-queue-win.sh:
4:22 PM Changeset in webkit [177604] by timothy_horton@apple.com
  • 2 edits in trunk/Source/WebKit2

Fix the build.

  • UIProcess/API/mac/WKView.mm:

(-[WKView _setIgnoresNonWheelEvents:]):

4:17 PM Changeset in webkit [177603] by dbates@webkit.org
  • 4 edits
    3 adds in trunk

[iOS] Add WebKitSystemInterface for iOS 8.1
https://bugs.webkit.org/show_bug.cgi?id=139831

Reviewed by Alexey Proskuryakov.

Tools:

Implement support in copy-webkitlibraries-to-product-directory to copy
WebKitSystemInterface for iOS to the product directory and have build-webkit
pass --wksi to copy-webkitlibraries-to-product-directory when building for iOS.

  • Scripts/build-webkit:
  • Scripts/copy-webkitlibraries-to-product-directory:

WebKitLibraries:

Towards building the iOS WebKit port using the public SDK, add the
WebKitSystemInterface for iOS version 8.1.

  • WebKitSystemInterfaceIOS.h: Added.
  • libWebKitSystemInterfaceIOSDevice8.1.a: Added.
  • libWebKitSystemInterfaceIOSSimulator8.1.a: Added.
4:15 PM Changeset in webkit [177602] by dbates@webkit.org
  • 2 edits in trunk/Tools

[iOS] Add WebKitSystemInterface for iOS 8.1
https://bugs.webkit.org/show_bug.cgi?id=139831

Reviewed by Alexey Proskuryakov.

Implement support in copy-webkitlibraries-to-product-directory to copy
WebKitSystemInterface for iOS to the product directory and have build-webkit
pass --wksi to copy-webkitlibraries-to-product-directory when building for iOS.

  • Scripts/build-webkit:
  • Scripts/copy-webkitlibraries-to-product-directory:
4:13 PM Changeset in webkit [177601] by Beth Dakin
  • 2 edits in trunk/Source/WebKit2

Preview popovers don't show an arrow for large links
https://bugs.webkit.org/show_bug.cgi?id=139842
-and corresponding-
<rdar://problem/19301470>

Reviewed by Tim Horton.

This patch addresses the case where the link is so large that there is no
available space on any side once you factor in screen padding. In that case, we
should give up early on not obscuring the link.

  • UIProcess/mac/WKPreviewPopoverAnimationController.mm:

(-[WKPreviewPopoverAnimationController _targetSizeForPagePreview]):
(-[WKPreviewPopoverAnimationController _effectivePaddingWithPreviewPadding:]):
(-[WKPreviewPopoverAnimationController _maxSpaceAvailableOnYEdgeForOriginRect:withScreenRect:previewPadding:]):
(-[WKPreviewPopoverAnimationController _maxSpaceAvailableOnXEdgeForOriginRect:withScreenRect:previewPadding:]):
(-[WKPreviewPopoverAnimationController _preferredPopoverSizeWithPreviewPadding:forTargetSize:]):

3:54 PM Changeset in webkit [177600] by timothy_horton@apple.com
  • 2 edits in trunk/Source/WebKit2

WKView's ignoresNonWheelEvents should ignore immediate actions
https://bugs.webkit.org/show_bug.cgi?id=139839
<rdar://problem/19312111>

Reviewed by Anders Carlsson.

  • UIProcess/API/mac/WKView.mm:

(-[WKView initWithFrame:context:configuration:webView:]):
(-[WKView _setIgnoresNonWheelEvents:]):
Install/uninstall the immediate action gesture recognizer when
toggling ignoresNonWheelEvents.

3:53 PM Changeset in webkit [177599] by timothy_horton@apple.com
  • 2 edits in trunk/Source/WebCore

TextIndicator's window can be pushed down if it intersects the menu bar
https://bugs.webkit.org/show_bug.cgi?id=139841
<rdar://problem/19311017>

Reviewed by Anders Carlsson.

  • page/mac/TextIndicatorWindow.mm:

(WebCore::TextIndicatorWindow::setTextIndicator):
Ensure that the textIndicatorWindow sits above the menubar, and re-set
its frame so that it moves under the menubar if required.

3:25 PM Changeset in webkit [177598] by ap@apple.com
  • 6 edits in trunk

REGRESSION (177368): Some tests started to immediately time out
https://bugs.webkit.org/show_bug.cgi?id=139826

Reviewed by Daniel Bates.

Tools:

  • WebKitTestRunner/InjectedBundle/InjectedBundle.cpp: (WTR::InjectedBundle::done):
  • WebKitTestRunner/InjectedBundle/InjectedBundlePage.cpp: (WTR::InjectedBundlePage::dump):

Move invalidateWaitToDumpWatchdogTimer() call from dump() to done(). When a test
ends in a load failure, we don't dump, so we used to leak the timer.

  • WebKitTestRunner/InjectedBundle/TestRunner.cpp: (WTR::TestRunner::notifyDone):

Added a comment.

LayoutTests:

  • platform/wk2/TestExpectations: The culprit was this test that I unskipped to see

whether it passes. It in fact does pass, so let's remove the expectation entirely.

2:14 PM Changeset in webkit [177597] by akling@apple.com
  • 2 edits in trunk/Source/JavaScriptCore

Plug leak below LLVMCopyStringRepOfTargetData().
<https://webkit.org/b/139832>

Reviewed by Michael Saboff.

LLVMCopyStringRepOfTargetData() returns a strdup()'ed string, so make sure
to free() it after we're done using it.

  • ftl/FTLCompile.cpp:

(JSC::FTL::mmAllocateDataSection):

2:09 PM Changeset in webkit [177596] by Csaba Osztrogonác
  • 7 edits in trunk/Source/WebKit2

URTBF after r177593.

  • PluginProcess/PluginProcess.h:
  • PluginProcess/unix/PluginProcessUnix.cpp:

(WebKit::PluginProcess::platformInitializePluginProcess):

  • Shared/WebProcessCreationParameters.h:
  • WebProcess/Plugins/Netscape/NetscapeBrowserFuncs.cpp:
  • WebProcess/WebProcess.h:
  • WebProcess/soup/WebProcessSoup.cpp:

(WebKit::WebProcess::platformInitializeWebProcess):

2:06 PM Changeset in webkit [177595] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WebKit/win

[WinCairo] Crash after failing to create GL context in accelerated compositing mode.
https://bugs.webkit.org/show_bug.cgi?id=139830

Patch by peavo@outlook.com <peavo@outlook.com> on 2014-12-19
Reviewed by Alex Christensen.

There is missing a null pointer check.

  • WebCoreSupport/AcceleratedCompositingContext.cpp:

(AcceleratedCompositingContext::initialize):

1:51 PM Changeset in webkit [177594] by Brent Fulgham
  • 2 edits in trunk/Source/WebKit

[Win] Build fix after r177588.

  • WebKit.vcxproj/WebKitExportGenerator/WebKitExports.def.in: Fix

symbol declaration.

1:48 PM Changeset in webkit [177593] by andersca@apple.com
  • 27 edits in trunk/Source

Use WebCore::MachSendRights for the compositing render server port
https://bugs.webkit.org/show_bug.cgi?id=139834

Reviewed by Tim Horton.

Source/WebCore:

  • WebCore.exp.in:

Source/WebKit2:

  • Platform/IPC/HandleMessage.h:

(IPC::callMemberFunctionImpl):

  • Platform/mac/LayerHostingContext.h:
  • Platform/mac/LayerHostingContext.mm:

(WebKit::LayerHostingContext::createForPort):

  • PluginProcess/PluginControllerProxy.h:
  • PluginProcess/PluginProcess.cpp:

(WebKit::PluginProcess::PluginProcess):
(WebKit::PluginProcess::initializePluginProcess):

  • PluginProcess/PluginProcess.h:

(WebKit::PluginProcess::compositingRenderServerPort):

  • PluginProcess/mac/PluginControllerProxyMac.mm:

(WebKit::PluginControllerProxy::compositingRenderServerPort):

  • PluginProcess/mac/PluginProcessMac.mm:

(WebKit::PluginProcess::platformInitializePluginProcess):

  • Shared/Plugins/PluginProcessCreationParameters.cpp:
  • Shared/Plugins/PluginProcessCreationParameters.h:
  • Shared/WebCoreArgumentCoders.h:
  • Shared/WebProcessCreationParameters.h:
  • Shared/mac/WebCoreArgumentCodersMac.mm:

(IPC::ArgumentCoder<MachSendRight>::encode):

  • UIProcess/Plugins/mac/PluginProcessProxyMac.mm:

(WebKit::PluginProcessProxy::platformInitializePluginProcess):

  • UIProcess/mac/WebContextMac.mm:

(WebKit::WebContext::platformInitializeWebProcess):

  • WebProcess/Plugins/Netscape/NetscapeBrowserFuncs.cpp:

(WebKit::NPN_GetValue):

  • WebProcess/Plugins/Netscape/NetscapePlugin.h:
  • WebProcess/Plugins/Netscape/mac/NetscapePluginMac.mm:

(WebKit::NetscapePlugin::compositingRenderServerPort):
(WebKit::NetscapePlugin::setLayerHostingMode):

  • WebProcess/Plugins/PluginController.h:
  • WebProcess/Plugins/PluginView.cpp:

(WebKit::PluginView::compositingRenderServerPort):

  • WebProcess/Plugins/PluginView.h:
  • WebProcess/WebProcess.cpp:

(WebKit::WebProcess::WebProcess):
(WebKit::WebProcess::initializeWebProcess):

  • WebProcess/WebProcess.h:

(WebKit::WebProcess::compositingRenderServerPort):

  • WebProcess/cocoa/WebProcessCocoa.mm:

(WebKit::WebProcess::platformInitializeWebProcess):

1:26 PM Changeset in webkit [177592] by Brent Fulgham
  • 2 edits in trunk/LayoutTests

[Win] Unreviewed gardening.

Unskip a test that started working after r177542. Also clear up some
lint warnings about the TestExpectations file.

  • platform/win/TestExpectations:
1:07 PM Changeset in webkit [177591] by Chris Dumez
  • 4 edits in trunk/Source/WebCore

[iOS] Log how often media element playback happens using FeatureCounter
https://bugs.webkit.org/show_bug.cgi?id=139819
<rdar://problem/19309988>

Reviewed by Eric Carlson.

Log using FeatureCounter how often we start loading for audio / video
elements, and how often they end up being played.

  • html/HTMLMediaElement.cpp:

(WebCore::HTMLMediaElement::HTMLMediaElement):
Add flag to identify when a media element starts playing for the first
time.

(WebCore::HTMLMediaElement::loadResource):
Log when a media element starts loading.

(WebCore::HTMLMediaElement::updatePlayState):
Log when a media element starts playing for the first time.

  • html/HTMLMediaElement.h:

Add flag to identify when a media element starts playing for the first
time.

  • platform/FeatureCounterKeys.h:

Add FeatureCounter keys for HTMLMediaElement loading / playback.

12:53 PM Changeset in webkit [177590] by matthew_hanson@apple.com
  • 2 edits in branches/safari-600.3-branch/Source/WebCore

Merged r176725. rdar://problems/19300244

11:50 AM Changeset in webkit [177589] by ap@apple.com
  • 2 edits in trunk/LayoutTests

media/video-seek-past-end-paused.html frequently times out on Mavericks
https://bugs.webkit.org/show_bug.cgi?id=139825

  • platform/mac/TestExpectations: Added an expectation.
11:48 AM Changeset in webkit [177588] by akling@apple.com
  • 7 edits in trunk/Source

Ref-ify TextIterator API.
<https://webkit.org/b/139823>

Reviewed by Antti Koivisto.

Source/WebCore:

Change TextIterator functions that always return Range objects to return
Ref<Range> instead of PassRefPtr<Range>.

One API (rangeFromLocationAndLength) may return nullptr, so its return type
is now RefPtr<Range>.

  • editing/TextIterator.cpp:

(WebCore::TextIterator::range):
(WebCore::TextIterator::node):
(WebCore::SimplifiedBackwardsTextIterator::range):
(WebCore::CharacterIterator::range):
(WebCore::characterSubrange):
(WebCore::BackwardsCharacterIterator::range):
(WebCore::TextIterator::subrange):
(WebCore::TextIterator::rangeFromLocationAndLength):
(WebCore::TextIterator::getLocationAndLengthFromRange):
(WebCore::collapsedToBoundary):
(WebCore::findPlainText):

  • editing/TextIterator.h:

Source/WebKit/mac:

  • WebView/WebTextIterator.mm:

(-[WebTextIterator currentRange]):

Source/WebKit2:

  • WebProcess/InjectedBundle/API/mac/WKDOMTextIterator.mm:

(-[WKDOMTextIterator currentRange]):

11:42 AM Changeset in webkit [177587] by commit-queue@webkit.org
  • 3 edits in trunk/Source/WebCore

Web Inspector: CRASH in inspector-protocol/debugger/terminate-dedicated-worker-while-paused.html
https://bugs.webkit.org/show_bug.cgi?id=139792

Patch by Joseph Pecoraro <Joseph Pecoraro> on 2014-12-19
Reviewed by Timothy Hatcher.

  • bindings/js/WorkerScriptDebugServer.cpp:

(WebCore::WorkerScriptDebugServer::removeListener):
When removing listeners, we could be during WorkerGlobalObject shutdown
and WorkerGlobalObject::script may be null.

  • workers/WorkerMessagingProxy.cpp:

(WebCore::WorkerMessagingProxy::WorkerMessagingProxy):
(WebCore::WorkerMessagingProxy::postMessageToPageInspector):
This postTask may happen after m_pageInspector has been cleared, so
only execute it if m_pageInspector is still valid.

11:37 AM Changeset in webkit [177586] by Chris Dumez
  • 2 edits in trunk/Source/WebCore

Drop useless HTMLMediaElement::m_previousProgress member
https://bugs.webkit.org/show_bug.cgi?id=139822

Reviewed by Eric Carlson.

Drop HTMLMediaElement::m_previousProgress member and it is not used or
even initialized.

  • html/HTMLMediaElement.h:
11:37 AM Changeset in webkit [177585] by commit-queue@webkit.org
  • 8 edits in trunk/Source

Web Inspector: CRASH inspector-protocol/debugger/breakpoint-action-detach.html
https://bugs.webkit.org/show_bug.cgi?id=139797

Patch by Joseph Pecoraro <Joseph Pecoraro> on 2014-12-19
Reviewed by Mark Lam.

Source/JavaScriptCore:

  • debugger/Debugger.h:
  • debugger/Debugger.cpp:

(JSC::Debugger::isAttached):
Check if we are the debugger for a particular global object.
(JSC::Debugger::pauseIfNeeded):
Pass the global object on when hitting a brekapoint.

  • inspector/ScriptDebugServer.h:
  • inspector/ScriptDebugServer.cpp:

(Inspector::ScriptDebugServer::handleBreakpointHit):
Stop evaluting breakpoint actions if a previous action caused the
debugger to detach from this global object.
(Inspector::ScriptDebugServer::handlePause):
Standardize on passing JSGlobalObject parameter first.

Source/WebKit/mac:

  • WebView/WebScriptDebugger.h:
  • WebView/WebScriptDebugger.mm:

(WebScriptDebugger::handlePause):
Standardize on passing JSGlobalObject parameter first.

11:34 AM Changeset in webkit [177584] by ap@apple.com
  • 2 edits in trunk/LayoutTests

fast/speechsynthesis/speech-synthesis-cancel.html times out very frequently
https://bugs.webkit.org/show_bug.cgi?id=139824

  • platform/mac-wk2/TestExpectations: Added an expectation.
11:26 AM Changeset in webkit [177583] by ap@apple.com
  • 6 edits in trunk/LayoutTests

Move some entries for tests that are slow in debug to root TestExpectations file.

  • TestExpectations:
  • platform/efl/TestExpectations:
  • platform/gtk/TestExpectations:
  • platform/mac-wk2/TestExpectations:
  • platform/mac/TestExpectations:
11:26 AM Changeset in webkit [177582] by mark.lam@apple.com
  • 2 edits in trunk/Source/JavaScriptCore

[Win] Endless compiler warnings created by DFGEdge.h.
<https://webkit.org/b/139801>

Reviewed by Brent Fulgham.

Add a cast to fix the type just the way the 64-bit version does.

  • dfg/DFGEdge.h:

(JSC::DFG::Edge::makeWord):

11:15 AM Changeset in webkit [177581] by Chris Dumez
  • 3 edits in trunk/Source/WebCore

[iOS] Log how successful the memory cache is using FeatureCounter
https://bugs.webkit.org/show_bug.cgi?id=139802

Reviewed by Andreas Kling.

Log how successful the memory cache is using FeatureCounter and why we
choose not to use the resource in the memory cache when it is present.

  • loader/cache/CachedResourceLoader.cpp:

(WebCore::CachedResourceLoader::requestResource):
(WebCore::CachedResourceLoader::determineRevalidationPolicy):

  • platform/FeatureCounterKeys.h:
11:07 AM Changeset in webkit [177580] by commit-queue@webkit.org
  • 42 edits in trunk/Source

Unreviewed, rolling out r177574.
https://bugs.webkit.org/show_bug.cgi?id=139821

"Broke Production builds by installing
libWebCoreTestSupport.dylib in the wrong directory" (Requested
by ddkilzer on #webkit).

Reverted changeset:

"Switch from using PLATFORM_NAME to SDK selectors in WebCore,
WebInspectorUI, WebKit, WebKit2"
https://bugs.webkit.org/show_bug.cgi?id=139463
http://trac.webkit.org/changeset/177574

10:58 AM Changeset in webkit [177579] by Brent Fulgham
  • 8 edits in trunk

Source/WebKit/win:
[Win] Add accessor for serializing HTTP loads.
https://bugs.webkit.org/show_bug.cgi?id=139817

Reviewed by Mark Lam.

  • Interfaces/IWebViewPrivate.idl: Add new method.
  • WebView.cpp:

(WebView::setLoadResourcesSerially): Provide implementation.

  • WebView.h:

Tools:
[Win] Add accessor for serializing HTTP loads
https://bugs.webkit.org/show_bug.cgi?id=139817

Reviewed by Mark Lam.

  • DumpRenderTree/win/DumpRenderTree.cpp: Reset serialized HTTP loads in our reset method.
  • DumpRenderTree/win/TestRunnerWin.cpp: Provide implementation for serializing HTTP. Also

remove some redundant code from the destructor.

10:48 AM Changeset in webkit [177578] by msaboff@apple.com
  • 5 edits
    3 adds in trunk

REGRESSION(174226): Captured arguments in a using function compiled by the DFG have the initial value when the closure was invoked
https://bugs.webkit.org/show_bug.cgi?id=139808

Reviewed by Oliver Hunt.

Source/JavaScriptCore:

There are three changes here.
1) Create a VariableWatchpointSet for captured arguments variables.
2) Properly use the VariableWatchpointSet* found in op_put_to_scope in the 64 bit LLInt code.
3) Add the same putLocalClosureVar path to the 32 bit LLInt code that exists in the 64 bit version.

  • bytecompiler/BytecodeGenerator.cpp:

(JSC::BytecodeGenerator::BytecodeGenerator):

  • llint/LowLevelInterpreter32_64.asm:
  • llint/LowLevelInterpreter64.asm:

LayoutTests:

New regression test.

  • js/regress-139808-expected.txt: Added.
  • js/regress-139808.html: Added.
  • js/script-tests/regress-139808.js: Added.

(theClosureFunction.rot13):
(theClosureFunction):

10:43 AM Changeset in webkit [177577] by ap@apple.com
  • 5 edits in trunk/LayoutTests

Updte WebKit2 test expectations based on what bots see now.

  • TestExpectations:
  • platform/gtk/TestExpectations:
  • platform/mac-wk2/TestExpectations:
  • platform/mac/TestExpectations:
10:36 AM Changeset in webkit [177576] by commit-queue@webkit.org
  • 3 edits
    2 adds in trunk

Only when the SVG is inline and only when a shape is referenced before it is defined, this shape will not be drawn.
https://bugs.webkit.org/show_bug.cgi?id=139451.

Patch by Said Abou-Hallawa <sabouhallawa@apple.com> on 2014-12-19
Reviewed by Antti Koivisto.

Source/WebCore:

Tests: svg/in-html/defs-after-use.html.

When parsing the children of an SVG element is finished, we need to notify the
other SVG elements which have incomplete shadow trees because of early referencing
to this element. The referencing elements need to rebuild their shadow trees and
make new copies of the this element and its sub-tree.

This is the case where a <use> tag references target elements before these target
elements are defined. Updating the shadow DOM tree of a <use> element should update
the corresponding shadow render tree as well.

  • svg/SVGElement.cpp:

(WebCore::SVGElement::finishParsingChildren):
Invalidate all the referencing elements of a target element whose sub-tree has
just finished parsing.

LayoutTests:

Ensure that when a <use> element inside an inline SVG references another
target SVG element, which has not been defined yet, gets its shadow DOM
tree updated once the target element is created. Updating the shadow DOM
tree of the <use> element should update the corresponding shadow render
tree as well.

  • svg/in-html/defs-after-use-expected.html: Added.
  • svg/in-html/defs-after-use.html: Added.
9:57 AM Changeset in webkit [177575] by ap@apple.com
  • 2 edits in trunk/Tools

Clean up a fix for performance test bot failing to build
https://bugs.webkit.org/show_bug.cgi?id=139818

Reviewed by Csaba Osztrogonác.

The "_1" in step name was annoying.

  • BuildSlaveSupport/build.webkit.org-config/master.cfg:

(CleanBuildIfScheduled.start):
(RunAndUploadPerfTests):
(DownloadAndPerfTestFactory.init):
(DownloadAndPerfTestWebKit2Factory.init):
(CleanBuildUnconditionally): Deleted.
(CleanBuildUnconditionally.start): Deleted.

9:23 AM Changeset in webkit [177574] by ddkilzer@apple.com
  • 42 edits in trunk/Source

Switch from using PLATFORM_NAME to SDK selectors in WebCore, WebInspectorUI, WebKit, WebKit2
<http://webkit.org/b/139463>

Reviewed by Mark Rowe.

Source/JavaScriptCore:

  • Configurations/JavaScriptCore.xcconfig:
  • Simplify SECTORDER_FLAGS.

Source/WebCore:

Also remove all uses of SQLITE3_HEADER_SEARCH_PATHS and
WEBCORE_SQLITE3_HEADER_SEARCH_PATHS which were phased out in
r132859 and needed for Leopard.

  • Configurations/Base.xcconfig:
  • Only set GCC_ENABLE_OBJC_GC, GCC_MODEL_TUNING and TOOLCHAINS on OS X.
  • Set GCC_OPTIMIZATION_LEVEL_normal based on SDK.
  • Simplify SQLITE3_HEADER_SEARCH_PATHS.
  • Configurations/DebugRelease.xcconfig:
  • Only set MACOSX_DEPLOYMENT_TARGET and SDKROOT on OS X.
  • Configurations/Version.xcconfig:
  • Set SYSTEM_VERSION_PREFIX separately for iOS and OS X.
  • Configurations/WebCore.xcconfig:
  • Set EXPORTED_SYMBOLS_FILE_i386, EXPORTED_SYMBOLS_FILE_x86_64, FRAMEWORK_SEARCH_PATHS, INSTALL_PATH, DYLIB_INSTALL_NAME_BASE, OTHER_LDFLAGS, SECTORDER_FLAGS, NORMAL_WEBCORE_FRAMEWORKS_DIR, WEBCORE_FRAMEWORKS_DIR, NORMAL_PRODUCTION_FRAMEWORKS_DIR, PRODUCTION_FRAMEWORKS_DIR, JAVASCRIPTCORE_PRIVATE_HEADERS_DIR_Production and EXCLUDED_SOURCE_FILE_NAMES based on SDK.
  • Make sure DYLIB_INSTALL_NAME_BASE and OTHER_LDFLAGS are overrideable by WebCoreTestSupport.xcconfig.
  • Configurations/WebCoreTestSupport.xcconfig:
  • Set PRIVATE_HEADERS_FOLDER_PATH_Production and INSTALL_PATH_Production based on SDK.
  • Override SECT_ORDER_FLAGS from WebCore.xcconfig.

Source/WebInspectorUI:

  • Configurations/Base.xcconfig:
  • Only set TOOLCHAINS on OS X.
  • Configurations/DebugRelease.xcconfig:
  • Only set MACOSX_DEPLOYMENT_TARGET and SDKROOT on OS X.
  • Configurations/Version.xcconfig:
  • Set SYSTEM_VERSION_PREFIX separately for iOS and OS X.
  • Configurations/WebInspectorUIFramework.xcconfig:
  • Set NORMAL_PRODUCTION_FRAMEWORKS_DIR, PRODUCTION_FRAMEWORKS_DIR and JAVASCRIPTCORE_PRIVATE_HEADERS_DIR_Production by SDK.

Source/WebKit/mac:

  • Configurations/Base.xcconfig:
  • Only set GCC_ENABLE_OBJC_GC, GCC_MODEL_TUNING and TOOLCHAINS on OS X.
  • Set GCC_OPTIMIZATION_LEVEL_normal based on SDK.
  • Configurations/DebugRelease.xcconfig:
  • Only set MACOSX_DEPLOYMENT_TARGET and SDKROOT on OS X.
  • Set WEBKIT_SYSTEM_INTERFACE_LIBRARY based on SDK.
  • Configurations/Version.xcconfig:
  • Set SYSTEM_VERSION_PREFIX separately for iOS and OS X.
  • Configurations/WebKitLegacy.xcconfig:
  • Set EXCLUDED_SOURCE_FILE_NAMES, EXPORTED_SYMBOLS_FILE_x86_64, FRAMEWORK_SEARCH_PATHS, OTHER_CFLAGS, INSTALL_PATH, DYLIB_INSTALL_NAME_BASE, OTHER_LDFLAGS, SECTORDER_FLAGS, NORMAL_WEBKIT_LEGACY_FRAMEWORKS_DIR, WEBKIT_LEGACY_FRAMEWORKS_DIR, NORMAL_PRODUCTION_FRAMEWORKS_DIR, PRODUCTION_FRAMEWORKS_DIR, PRODUCTION_ROOT, WEBCORE_PRIVATE_HEADERS_DIR_Production and WEBKITSYSTEMINTERFACE_STATIC_LIBRARY_HEADERS_FOLDER_PATH_engineering based on SDK.
  • Fix PRODUCTION_ROOT to actually be set properly.
  • Simplify EXPORTED_SYMBOLS_FILE_i386.
  • migrate-headers.sh:
  • Get rid of unused WEBKIT2_FRAMEWORKS_DIR variable.
  • Change WEBCORE_PRIVATE_HEADERS_DIR_macosx_Production to WEBCORE_PRIVATE_HEADERS_DIR_Production based on changes to WebKitLegacy.xcconfig.

Source/WebKit2:

  • Configurations/All.xcconfig:
  • Only set EXCLUDED_SOURCE_FILE_NAMES on iOS.
  • Configurations/Base.xcconfig:
  • Only set FRAMEWORK_CONTENT_DIRECTORY, GCC_ENABLE_OBJC_GC and TOOLCHAINS on OS X.
  • Configurations/BaseLegacyProcess.xcconfig:
  • Only set SKIP_INSTALL on iOS.
  • Configurations/BaseTarget.xcconfig:
  • Set WEBKIT_FRAMEWORKS_DIR and UMBRELLA_FRAMEWORKS_DIR based on SDK, and simply them.
  • Configurations/BaseXPCService.xcconfig:
  • Set INSTALL_PATH_ACTUAL_Normal based on SDK.
  • Configurations/DebugRelease.xcconfig:
  • Only set MACOSX_DEPLOYMENT_TARGET and SDKROOT on OS X.
  • Set WEBKIT_SYSTEM_INTERFACE_LIBRARY based on SDK.
  • Configurations/NetworkProcess.xcconfig:
  • Only set EXCLUDED_SHIM_FILE_NAME and CODE_SIGN_ENTITLEMENTS on iOS.
  • Configurations/NetworkService.Development.xcconfig:
  • Set INFOPLIST_FILE based on SDK.
  • Simplify CODE_SIGN_ENTITLEMENTS for iOS.
  • Configurations/NetworkService.xcconfig:
  • Set INFOPLIST_FILE based on SDK.
  • Simplify CODE_SIGN_ENTITLEMENTS for iOS.
  • Configurations/PluginProcess.xcconfig:
  • Only set EXCLUDED_SHIM_FILE_NAME and SKIP_INSTALL on iOS.
  • Configurations/PluginProcessShim.xcconfig:
  • Set OTHER_LDFLAGS based on SDK.
  • Configurations/PluginService.32.xcconfig:
  • Only set VALID_ARCHS on OS X.
  • Set OTHER_LDFLAGS based on platform-specific values.
  • Only set SKIP_INSTALL on iOS.
  • Configurations/PluginService.64.xcconfig:
  • Set ARCHS based on SDK.
  • Set OTHER_LDFLAGS based on platform-specific values.
  • Only set SKIP_INSTALL on iOS.
  • Configurations/PluginService.Development.xcconfig:
  • Only set SKIP_INSTALL on iOS.
  • Set OTHER_LDFLAGS based on platform-specific values.
  • Configurations/SecItemShim.xcconfig:
  • Set OTHER_LDFLAGS based on platform-specific values.
  • Configurations/Version.xcconfig:
  • Set SYSTEM_VERSION_PREFIX separately for iOS and OS X.
  • Configurations/WebContentProcess.xcconfig:
  • Only set EXCLUDED_SHIM_FILE_NAME on iOS.
  • Only set CODE_SIGN_ENTITLEMENTS on iOS Simulator.
  • Configurations/WebContentService.Development.xcconfig:
  • Set INFOPLIST_FILE and BUNDLE_LOCALIZATION_KEY based on SDK.
  • Only set CODE_SIGN_ENTITLEMENTS on iOS hardware.
  • Set OTHER_LDFLAGS based on platform-specific values.
  • Configurations/WebContentService.xcconfig: Ditto.
  • Configurations/WebKit.xcconfig:
  • Set FRAMEWORK_AND_LIBRARY_LDFLAGS, SECTORDER_FLAGS, and WK_API_AVAILABILITY_ENABLED based on SDK.
  • Set OTHER_LDFLAGS based on platform-specific values.
  • Only set EXCLUDED_SOURCE_FILE_NAMES on iOS.
  • Configurations/WebKit2.xcconfig:
  • Set INSTALL_PATH, INSTALL_PATH_ACTUAL, DYLIB_INSTALL_NAME_BASE, NORMAL_WEBKIT2_FRAMEWORKS_DIR, WEBKIT2_FRAMEWORKS_DIR and NORMAL_PRODUCTION_FRAMEWORKS_DIR based on SDK.
  • Set SKIP_INSTALL on iOS.
  • Configurations/WebProcessShim.xcconfig:
  • Set OTHER_LDFLAGS based on platform-specific values.
2:02 AM Changeset in webkit [177573] by alex.christensen@flexsim.com
  • 2 edits in trunk/Source/WebKit

[Win64] Unreviewed build fix.

  • WebKit.vcxproj/WebKitExportGenerator/WebKitExports.def.in:

Fixed some 64-bit linker symbols.

12:57 AM Changeset in webkit [177572] by Chris Dumez
  • 3 edits in trunk/Source/WebCore

Clean up StyleBuilderCustom and DeprecatedStyleBuilder
https://bugs.webkit.org/show_bug.cgi?id=139807

Reviewed by Antti Koivisto.

Clean up StyleBuilderCustom and DeprecatedStyleBuilder.

  • css/DeprecatedStyleBuilder.cpp:
  • Remove unnecessary header includes.

(WebCore::ApplyPropertyPerspectiveOrigin::applyInheritValue):
(WebCore::ApplyPropertyPerspectiveOrigin::applyInitialValue):
(WebCore::ApplyPropertyPerspectiveOrigin::applyValue):
(WebCore::DeprecatedStyleBuilder::DeprecatedStyleBuilder):
(WebCore::ApplyPropertyExpanding::applyInheritValue): Deleted.
(WebCore::ApplyPropertyExpanding::applyInitialValue): Deleted.
(WebCore::ApplyPropertyExpanding::applyValue): Deleted.
(WebCore::ApplyPropertyExpanding::createHandler): Deleted.

  • Rename ApplyPropertyExpanding to ApplyPropertyPerspectiveOrigin as PerspectiveOrigin is the only remaining user of this class. This also allows us to simplify the implementation a lot.

(WebCore::ApplyPropertyDefaultBase::setValue): Deleted.
(WebCore::ApplyPropertyDefaultBase::value): Deleted.
(WebCore::ApplyPropertyDefaultBase::initial): Deleted.
(WebCore::ApplyPropertyDefaultBase::applyInheritValue): Deleted.
(WebCore::ApplyPropertyDefaultBase::applyInitialValue): Deleted.
(WebCore::ApplyPropertyDefaultBase::applyValue): Deleted.

  • There is no longer any user of this templated class.
  • css/StyleBuilderCustom.h:
  • Introduce a macro that declares all 3 handlers (inherit, initial and value) in the StyleBuilderCustom class. This greatly reduces the number of lines in this class.
  • Also alphabetize the handlers.
12:37 AM Changeset in webkit [177571] by bshafiei@apple.com
  • 2 edits in branches/safari-600.1.4.13-branch/Source/WebCore

Merged r177566. rdar://problem/19299453

12:29 AM Changeset in webkit [177570] by bshafiei@apple.com
  • 2 edits in tags/Safari-600.1.4.13.2/Source/WebCore

Merged r177566. rdar://problem/19299453

12:24 AM Changeset in webkit [177569] by Csaba Osztrogonác
  • 2 edits in trunk/Tools

REGRESSION(r177535): It broke the GTK performance bot
https://bugs.webkit.org/show_bug.cgi?id=139811

Reviewed by Carlos Garcia Campos.

Trigger clean build only on mac platform.

  • BuildSlaveSupport/build.webkit.org-config/master.cfg:

(DownloadAndPerfTestFactory.init):
(DownloadAndPerfTestWebKit2Factory.init):

Note: See TracTimeline for information about the timeline view.