Timeline



Dec 30, 2018:

1:56 PM Changeset in webkit [239563] by ddkilzer@apple.com
  • 3 edits in trunk/Source/WebCore

clang-tidy: Save 8 padding bytes on WebCore::BorderEdge
<​https://webkit.org/b/193058>
<rdar://problem/46979275>

Reviewed by Yusuke Suzuki.

  • rendering/BorderEdge.cpp:

(WebCore::BorderEdge::BorderEdge): Reorder initialization to
match new instance variable order.

  • rendering/BorderEdge.h: Reorder instance variables to save

8 padding bytes.

1:23 PM Changeset in webkit [239562] by yusukesuzuki@slowstart.org
  • 2 edits in trunk

gperf is only used in WebCore
​https://bugs.webkit.org/show_bug.cgi?id=193061

Reviewed by Don Olmstead.

gperf is only used in WebCore. So if the port does not have WebCore,
it is not necessary. For example, JSCOnly port does not require it.
This patch removes gperf dependency in these ports by checking
ENABLE_WEBCORE.

  • Source/cmake/WebKitCommon.cmake:
1:19 PM Changeset in webkit [239561] by yusukesuzuki@slowstart.org
  • 5 edits in trunk

Add ENABLE_UNIFIED_BUILDS option to cmake ports
​https://bugs.webkit.org/show_bug.cgi?id=193045

Reviewed by Don Olmstead.

.:

This patch adds ENABLE_UNIFIED_BUILDS option to cmake ports, which allows us to disable Unified Builds in WebKit.
This makes compile_commands.json sane when the unified builds is disabled, and various tools can use this
compilation database. For example, YouCompleteMe completion server can use it.

  • Source/cmake/WebKitFeatures.cmake:
  • Source/cmake/WebKitMacros.cmake:

Source/WTF:

  • Scripts/generate-unified-source-bundles.rb:
12:53 PM Changeset in webkit [239560] by yusukesuzuki@slowstart.org
  • 2 edits in trunk/JSTests

Unreviewed, JSTests gardening with memoryLimited

  • stress/string-overflow-createError.js:
9:38 AM Changeset in webkit [239559] by Ross Kirsling
  • 33 edits in trunk

[JSC] Identifier validity should be based on ID_Start / ID_Continue properties
​https://bugs.webkit.org/show_bug.cgi?id=193050

Reviewed by Yusuke Suzuki.

JSTests:

  • test262.yaml:
  • test262/expectations.yaml:

Mark 16 tests as passing.

Source/JavaScriptCore:

From ​https://tc39.github.io/ecma262/#sec-names-and-keywords:

UnicodeIDStart
any Unicode code point with the Unicode property "ID_Start"
UnicodeIDContinue
any Unicode code point with the Unicode property "ID_Continue"
  • parser/Lexer.cpp:

(JSC::Lexer<T>::Lexer):
(JSC::isNonLatin1IdentStart):
(JSC::isNonLatin1IdentPart):
(JSC::isIdentPart):
(JSC::Lexer<T>::lex):
Ensure identifier start / part is based on ID_Start / ID_Continue.
(Implies a special case for U+00B7, which is Latin-1 but Other_ID_Continue.)

LayoutTests:

  • sputnik/Unicode/Unicode_218/S7.6_A1.1_T2-expected.txt:
  • sputnik/Unicode/Unicode_218/S7.6_A1.1_T6-expected.txt:
  • sputnik/Unicode/Unicode_218/S7.6_A5.2_T2-expected.txt:
  • sputnik/Unicode/Unicode_218/S7.6_A5.2_T6-expected.txt:
  • sputnik/Unicode/Unicode_218/S7.6_A5.3_T1-expected.txt:
  • sputnik/Unicode/Unicode_218/S7.6_A5.3_T2-expected.txt:
  • sputnik/Unicode/Unicode_320/S7.6_A1.1_T6-expected.txt:
  • sputnik/Unicode/Unicode_320/S7.6_A2.3-expected.txt:
  • sputnik/Unicode/Unicode_320/S7.6_A3.1-expected.txt:
  • sputnik/Unicode/Unicode_320/S7.6_A5.2_T6-expected.txt:
  • sputnik/Unicode/Unicode_320/S7.6_A5.2_T9-expected.txt:
  • sputnik/Unicode/Unicode_320/S7.6_A5.3_T1-expected.txt:
  • sputnik/Unicode/Unicode_320/S7.6_A5.3_T2-expected.txt:
  • sputnik/Unicode/Unicode_410/S7.6_A1.1_T6-expected.txt:
  • sputnik/Unicode/Unicode_410/S7.6_A3.1-expected.txt:
  • sputnik/Unicode/Unicode_410/S7.6_A5.2_T6-expected.txt:
  • sputnik/Unicode/Unicode_410/S7.6_A5.3_T1-expected.txt:
  • sputnik/Unicode/Unicode_410/S7.6_A5.3_T2-expected.txt:
  • sputnik/Unicode/Unicode_500/S7.6_A1.1_T6-expected.txt:
  • sputnik/Unicode/Unicode_500/S7.6_A3.1-expected.txt:
  • sputnik/Unicode/Unicode_500/S7.6_A5.2_T6-expected.txt:
  • sputnik/Unicode/Unicode_500/S7.6_A5.3_T1-expected.txt:
  • sputnik/Unicode/Unicode_500/S7.6_A5.3_T2-expected.txt:
  • sputnik/Unicode/Unicode_510/S7.6_A1.1_T4-expected.txt:
  • sputnik/Unicode/Unicode_510/S7.6_A1.1_T6-expected.txt:
  • sputnik/Unicode/Unicode_510/S7.6_A5.2_T4-expected.txt:
  • sputnik/Unicode/Unicode_510/S7.6_A5.2_T6-expected.txt:

Update expectations for outdated tests.

Dec 28, 2018:

9:50 PM Changeset in webkit [239558] by Wenson Hsieh
  • 8 edits
    2 adds in trunk/Source/WebKit

Move WKEditCommandObjC and WKEditorUndoTargetObjC into a separate file
​https://bugs.webkit.org/show_bug.cgi?id=193049

Reviewed by Sam Weinig.

Pull common code (WKEditCommandObjC and WKEditorUndoTargetObjC) on iOS and macOS out into a separate file. No
change in behavior.

  • SourcesCocoa.txt:
  • UIProcess/Cocoa/WKEditCommand.h: Added.
  • UIProcess/Cocoa/WKEditCommand.mm: Added.

Rename WKEditCommandObjC to WKEditCommand, and WKEditorUndoTargetObjC to WKEditorUndoTarget. The ObjC suffix in
the name seems to diverge from the common naming scheme in other parts of WebKit, where most WK- and _WK-
prefixed names already refer to Objective-C objects. Additionally, mark -[WKEditCommand init] as unavailable.

(-[WKEditCommand initWithWebEditCommandProxy:]):

Make the return type instancetype instead of id, and also make this take a Ref<WebEditCommandProxy>&& instead
of a RefPtr.

(-[WKEditCommand command]):

Make this return a reference to the WebEditCommandProxy, rather than a pointer, since the WebEditCommandProxy
should always be non-null.

(-[WKEditorUndoTarget undoEditing:]):
(-[WKEditorUndoTarget redoEditing:]):

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

(WebKit::WebViewImpl::WebViewImpl):
(WebKit::WebViewImpl::registerEditCommand):

Use WTFMove instead of copying the Ref when creating a WKEditCommand.

(-[WKEditCommandObjC initWithWebEditCommandProxy:]): Deleted.
(-[WKEditCommandObjC command]): Deleted.
(-[WKEditorUndoTargetObjC undoEditing:]): Deleted.
(-[WKEditorUndoTargetObjC redoEditing:]): Deleted.

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

(WebKit::PageClientImpl::PageClientImpl):
(WebKit::PageClientImpl::registerEditCommand):

Use WTFMove instead of creating a new RefPtr when creating a WKEditCommand.

(-[WKEditCommandObjC initWithWebEditCommandProxy:]): Deleted.
(-[WKEditCommandObjC command]): Deleted.
(-[WKEditorUndoTargetObjC undoEditing:]): Deleted.
(-[WKEditorUndoTargetObjC redoEditing:]): Deleted.

  • UIProcess/mac/PageClientImplMac.h:
  • WebKit.xcodeproj/project.pbxproj:
8:24 PM Changeset in webkit [239557] by yusukesuzuki@slowstart.org
  • 9 edits in trunk/Source

[JSC] Remove one indirection in JSObject::toStringName
​https://bugs.webkit.org/show_bug.cgi?id=193037

Reviewed by Keith Miller.

Source/JavaScriptCore:

We should not have additional one-level indirection in JSObject::toStringName.
JSObject::toStringName is dispatched through methodTable. Even after that, we
need to call JSObject::className function through methodTable again. But className
function is rarely defined. So instead of introducing this indirection here,
classes having className functions should have toStringName function too. This can
remove one-level indirection in toStringName in major cases.

  • API/JSCallbackObject.h:
  • API/JSCallbackObjectFunctions.h:

(JSC::JSCallbackObject<Parent>::toStringName):

  • debugger/DebuggerScope.cpp:

(JSC::DebuggerScope::toStringName):

  • debugger/DebuggerScope.h:
  • runtime/JSObject.cpp:

(JSC::JSObject::toStringName):

Source/WebCore:

Use old JSObject::toStringName function here.

  • bindings/js/JSDOMConstructorBase.cpp:

(WebCore::JSDOMConstructorBase::className):
(WebCore::JSDOMConstructorBase::toStringName):

  • bindings/js/JSDOMConstructorBase.h:

(WebCore::JSDOMConstructorBase::className): Deleted.

Dec 27, 2018:

8:13 AM Changeset in webkit [239556] by achristensen@apple.com
  • 30 edits
    1 delete in trunk

Resurrect Mac CMake build
​https://bugs.webkit.org/show_bug.cgi?id=192658

Reviewed by Yusuke Suzuki.

.:

  • Source/cmake/OptionsMac.cmake:

Source/JavaScriptCore:

  • PlatformMac.cmake:

Source/WebCore:

This makes it so JSC and most of WebCore builds.
WebCore's use of ARC seems inconsistent, so I'll fix that later.

  • PlatformMac.cmake:
  • config.h:

Source/WebCore/PAL:

  • pal/PlatformMac.cmake:

Source/WebKit:

  • PlatformMac.cmake:

Source/WebKitLegacy:

  • PlatformMac.cmake:

Source/WTF:

  • wtf/PlatformMac.cmake:
  • wtf/cf/CFURLExtras.cpp:
  • wtf/cf/CFURLExtras.h:
  • wtf/cf/URLCF.cpp:
  • wtf/cocoa/NSURLExtras.h:
  • wtf/cocoa/NSURLExtras.mm:
  • wtf/cocoa/URLCocoa.mm:

Tools:

  • DumpRenderTree/PlatformMac.cmake:
  • MiniBrowser/mac/CMakeLists.txt:
  • TestWebKitAPI/PlatformMac.cmake:

Dec 26, 2018:

8:01 PM Changeset in webkit [239555] by Fujii Hironori
  • 3 edits in trunk/Source/WebKit

[MSVC] error C2381: 'WebKit::NetworkHTTPSUpgradeChecker::~NetworkHTTPSUpgradeChecker': redefinition; 'declspec(noreturn)' or 'noreturn' differs
​https://bugs.webkit.org/show_bug.cgi?id=193030

Reviewed by Ross Kirsling.

Windows Debug builds are failing to compile due to
NO_RETURN_DUE_TO_ASSERT inconsistency of
NetworkHTTPSUpgradeChecker destructor's definition and
declaration.

  • NetworkProcess/NetworkHTTPSUpgradeChecker.h: Added

NO_RETURN_DUE_TO_ASSERT to NetworkHTTPSUpgradeChecker destructor
declaration.

  • NetworkProcess/NetworkHTTPSUpgradeChecker.cpp: Removed

NO_RETURN_DUE_TO_ASSERT from the definition.

12:42 PM Changeset in webkit [239554] by commit-queue@webkit.org
  • 3 edits in trunk/Source/WebCore

[FreeType] Restore conditional compilation logic for recent HarfBuzz refactoring
​https://bugs.webkit.org/show_bug.cgi?id=193036

Patch by Jim Mason <​jmason@ibinx.com> on 2018-12-26
Reviewed by Michael Catanzaro.

  • platform/graphics/FontPlatformData.h:
  • platform/graphics/freetype/FontPlatformDataFreeType.cpp:
10:30 AM Changeset in webkit [239553] by Adrian Perez de Castro
  • 6 edits in trunk/Tools

[Tools] Remove references to Bison and Flex
​https://bugs.webkit.org/show_bug.cgi?id=193033

Reviewed by Michael Catanzaro.

The Bison and Flex tools are not needed anymore since r209883.

  • CygwinDownloader/cygwin-downloader.py:
  • EWSTools/ubuntu-ews-packages:
  • Scripts/filter-build-webkit:

(shouldIgnoreLine):

  • gtk/install-dependencies:
  • wpe/install-dependencies:
Note: See TracTimeline for information about the timeline view.