Timeline



Jan 1, 2019:

5:37 PM WebKitGTK/AddingNewWebKit2API edited by Michael Catanzaro
Update reminder to add autocleanups when adding new types (diff)
11:29 AM Changeset in webkit [239568] by jeffm@apple.com
  • 15 edits in trunk

Update user-visible copyright strings to include 2019
​https://bugs.webkit.org/show_bug.cgi?id=192811

Reviewed by Mark Lam.

Source/JavaScriptCore:

  • Info.plist:

Source/WebCore:

  • Info.plist:

Source/WebKit:

  • Info.plist:
  • NetworkProcess/EntryPoint/mac/XPCService/NetworkService/Info-OSX.plist:
  • NetworkProcess/EntryPoint/mac/XPCService/NetworkService/Info-iOS.plist:
  • PluginProcess/EntryPoint/mac/XPCService/PluginService.32-64.Info.plist:
  • WebProcess/EntryPoint/mac/XPCService/WebContentService/Info-OSX.plist:
  • WebProcess/EntryPoint/mac/XPCService/WebContentService/Info-iOS.plist:

Source/WebKitLegacy/mac:

  • Info.plist:

WebKitLibraries:

  • win/tools/scripts/COPYRIGHT-END-YEAR:

Dec 31, 2018:

7:31 AM Changeset in webkit [239567] by yusukesuzuki@slowstart.org
  • 2 edits in trunk/Tools

.ycm_extra_conf.py should handle XXXInlines.h
​https://bugs.webkit.org/show_bug.cgi?id=193055

Reviewed by Carlos Garcia Campos.

compile_commands.json database only records flags and options for cpp source files.
So, when we open headers, this database does not have any information for that.
Our .ycm_extra_conf.py alleviates this by using XXX.cpp's configuration for XXX.h.
But this mitigation does not handle XXXInlines.h well since we do not have XXXInlines.cpp.
This patch adds support for XXXInlines.h in .ycm_extra_conf.py. When XXXInlines.h is queried,
we attempt to find XXX.cpp and use the configuration if it exists.

  • gtk/ycm_extra_conf.py:

(getImplementationFilename):
(FlagsForFile):

6:53 AM Changeset in webkit [239566] by Carlos Garcia Campos
  • 3 edits
    1 add in trunk

Unreviewed. Support PHP 7.3 in Debian.

Tools:

Add 7.3 to the list of php versions supported in Debian.

  • Scripts/webkitpy/port/base.py:

(Port._debian_php_version):

LayoutTests:

  • http/conf/debian-httpd-2.4-php7.3.conf: Added.
6:08 AM Changeset in webkit [239565] by Carlos Garcia Campos
  • 15 edits in trunk/LayoutTests

Unreviewed GTK+ gardening. Rebaseline several tests after r239156.

  • platform/gtk/fast/frames/onlyCommentInIFrame-expected.txt:
  • platform/gtk/fast/invalid/junk-data-expected.txt:
  • platform/gtk/fast/invalid/missing-end-tag-expected.txt:
  • platform/gtk/fast/parser/xhtml-alternate-entities-expected.txt:
  • platform/gtk/fast/sub-pixel/selection/selection-rect-in-sub-pixel-table-expected.png:
  • platform/gtk/fast/sub-pixel/selection/selection-rect-in-sub-pixel-table-expected.txt:
  • platform/gtk/fast/xsl/xslt-extra-content-at-end-expected.txt:
  • platform/gtk/fast/xsl/xslt-missing-namespace-in-xslt-expected.txt:
  • platform/gtk/svg/custom/bug45331-expected.txt:
  • platform/gtk/svg/custom/junk-data-expected.txt:
  • platform/gtk/svg/custom/missing-xlink-expected.txt:
  • platform/gtk/svg/custom/path-bad-data-expected.txt:
  • platform/gtk/svg/custom/use-font-face-crash-expected.txt:
  • platform/gtk/svg/hixie/error/012-expected.txt:
5:36 AM Changeset in webkit [239564] by Carlos Garcia Campos
  • 4 edits
    11 adds in trunk/LayoutTests

Unreviewed GTK+ gardening. Rebaseline several tests after r238730.

innerText works differently when using complex text path.
See ​https://bugs.webkit.org/show_bug.cgi?id=192750.

  • platform/gtk/fast/events/nested-window-event-expected.txt: Added.
  • platform/gtk/fast/events/window-events-bubble-expected.txt: Added.
  • platform/gtk/fast/events/window-events-bubble2-expected.txt: Added.
  • platform/gtk/fast/tokenizer/script_extra_close-expected.txt: Added.
  • platform/gtk/imported/w3c/web-platform-tests/css/css-grid/alignment/grid-row-axis-alignment-positioned-items-015-expected.txt: Added.
  • platform/gtk/imported/w3c/web-platform-tests/css/css-grid/alignment/grid-row-axis-alignment-positioned-items-016-expected.txt: Added.
  • platform/gtk/imported/w3c/web-platform-tests/html/semantics/forms/constraints/form-validation-validity-badInput-expected.txt:
  • platform/gtk/imported/w3c/web-platform-tests/html/semantics/forms/constraints/form-validation-willValidate-expected.txt:
  • platform/gtk/imported/w3c/web-platform-tests/html/semantics/forms/the-input-element/selection-expected.txt:
  • platform/gtk/legacy-animation-engine/animations/lineheight-animation-expected.txt: Added.
  • platform/gtk/legacy-animation-engine/animations/simultaneous-start-transform-expected.txt: Added.
  • platform/gtk/legacy-animation-engine/animations/width-using-ems-expected.txt: Added.

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.

Note: See TracTimeline for information about the timeline view.