[webkit-changes] [246581] trunk/Websites/perf.webkit.org

2019-06-18 Thread dewei_zhu
Title: [246581] trunk/Websites/perf.webkit.org








Revision 246581
Author dewei_...@apple.com
Date 2019-06-18 22:12:35 -0700 (Tue, 18 Jun 2019)


Log Message
Customizable test group form should allow user to supply a revision prefix of a commit and revision starts with 'r'.
https://bugs.webkit.org/show_bug.cgi?id=198940

Reviewed by Ryosuke Niwa.

Customizable test group form should adapt prefix matching when fetching for a commit.

* browser-tests/customizable-test-group-form-tests.js: Updated and added unit tests.
* public/v3/components/customizable-test-group-form.js: Removed loggings those a unintentionally committed.
* public/v3/models/commit-set.js: Adapted prefix matching API when fetching a commit.
(IntermediateCommitSet.prototype._fetchCommitLogAndOwnedCommits):
* unit-tests/commit-set-tests.js: Updated unit tests accordingly.

Modified Paths

trunk/Websites/perf.webkit.org/ChangeLog
trunk/Websites/perf.webkit.org/browser-tests/customizable-test-group-form-tests.js
trunk/Websites/perf.webkit.org/public/v3/components/customizable-test-group-form.js
trunk/Websites/perf.webkit.org/public/v3/models/commit-set.js
trunk/Websites/perf.webkit.org/unit-tests/commit-set-tests.js




Diff

Modified: trunk/Websites/perf.webkit.org/ChangeLog (246580 => 246581)

--- trunk/Websites/perf.webkit.org/ChangeLog	2019-06-19 04:44:11 UTC (rev 246580)
+++ trunk/Websites/perf.webkit.org/ChangeLog	2019-06-19 05:12:35 UTC (rev 246581)
@@ -1,3 +1,18 @@
+2019-06-17  Dewei Zhu  
+
+Customizable test group form should allow user to supply a revision prefix of a commit and revision starts with 'r'.
+https://bugs.webkit.org/show_bug.cgi?id=198940
+
+Reviewed by Ryosuke Niwa.
+
+Customizable test group form should adapt prefix matching when fetching for a commit.
+
+* browser-tests/customizable-test-group-form-tests.js: Updated and added unit tests.
+* public/v3/components/customizable-test-group-form.js: Removed loggings those a unintentionally committed.
+* public/v3/models/commit-set.js: Adapted prefix matching API when fetching a commit.
+(IntermediateCommitSet.prototype._fetchCommitLogAndOwnedCommits):
+* unit-tests/commit-set-tests.js: Updated unit tests accordingly.
+
 2019-06-13  Dewei Zhu  
 
 Custom analysis task configurator should allow supplying commit prefix and revision starts 'r'.


Modified: trunk/Websites/perf.webkit.org/browser-tests/customizable-test-group-form-tests.js (246580 => 246581)

--- trunk/Websites/perf.webkit.org/browser-tests/customizable-test-group-form-tests.js	2019-06-19 04:44:11 UTC (rev 246580)
+++ trunk/Websites/perf.webkit.org/browser-tests/customizable-test-group-form-tests.js	2019-06-19 05:12:35 UTC (rev 246581)
@@ -35,6 +35,18 @@
 "message": "some message",
 };
 
+const commitObjectC = {
+"id": "185336",
+"revision": "210950",
+"repository": 1,
+"previousCommit": null,
+"ownsCommits": false,
+"time": 1541494949682,
+"authorName": "Chris Dumez",
+"authorEmail": "cdu...@apple.com",
+"message": "some message",
+};
+
 function cloneObject(object)
 {
 const clone = {};
@@ -62,8 +74,8 @@
 
 const requests = context.symbols.MockRemoteAPI.requests;
 expect(requests.length).to.be(2);
-expect(requests[0].url).to.be('/api/commits/1/210948');
-expect(requests[1].url).to.be('/api/commits/1/210949');
+expect(requests[0].url).to.be('/api/commits/1/210948?prefix-match=true');
+expect(requests[1].url).to.be('/api/commits/1/210949?prefix-match=true');
 requests[0].resolve({commits: [commitObjectA]});
 requests[1].resolve({commits: [commitObjectB]});
 
@@ -90,6 +102,58 @@
 expect(revisionEditor.value).to.be('210948');
 });
 
+it('should allow user to only provide prefix of a commit as long as the commit is unique in the repository', async () => {
+const context = new BrowsingContext();
+const customizableTestGroupForm = await createCustomizableTestGroupFormWithContext(context);
+const repository = context.symbols.Repository.ensureSingleton(1, {name: 'WebKit'});
+
+const commitA = cloneObject(commitObjectA);
+const commitB = cloneObject(commitObjectB);
+const commitC = cloneObject(commitObjectC);
+commitA.repository = repository;
+commitB.repository = repository;
+commitC.repository = repository;
+const webkitCommitA = context.symbols.CommitLog.ensureSingleton(185326, commitA);
+const webkitCommitB = context.symbols.CommitLog.ensureSingleton(185334, commitB);
+const commitSetA = context.symbols.CommitSet.ensureSingleton(1, {revisionItems: [{commit: webkitCommitA}]});
+const commitSetB = context.symbols.CommitSet.ensureSingleton(2, {revisionItems: [{commit: webkitCommitB}]});
+
+customizableTestGroupForm.setCommitSetMap({A: 

[webkit-changes] [246580] trunk/Source/WebKit

2019-06-18 Thread ryanhaddad
Title: [246580] trunk/Source/WebKit








Revision 246580
Author ryanhad...@apple.com
Date 2019-06-18 21:44:11 -0700 (Tue, 18 Jun 2019)


Log Message
Unreviewed, rolling out r246575.

Breaks internal builds.

Reverted changeset:

"Update WebKit API to separately retrieve actions and use
context menus"
https://bugs.webkit.org/show_bug.cgi?id=198974
https://trac.webkit.org/changeset/246575

Modified Paths

trunk/Source/WebKit/ChangeLog
trunk/Source/WebKit/UIProcess/API/Cocoa/WKUIDelegate.h
trunk/Source/WebKit/UIProcess/API/Cocoa/WKUIDelegatePrivate.h




Diff

Modified: trunk/Source/WebKit/ChangeLog (246579 => 246580)

--- trunk/Source/WebKit/ChangeLog	2019-06-19 02:36:42 UTC (rev 246579)
+++ trunk/Source/WebKit/ChangeLog	2019-06-19 04:44:11 UTC (rev 246580)
@@ -1,3 +1,16 @@
+2019-06-18  Ryan Haddad  
+
+Unreviewed, rolling out r246575.
+
+Breaks internal builds.
+
+Reverted changeset:
+
+"Update WebKit API to separately retrieve actions and use
+context menus"
+https://bugs.webkit.org/show_bug.cgi?id=198974
+https://trac.webkit.org/changeset/246575
+
 2019-06-18  Alex Christensen  
 
 Update WebKit API to separately retrieve actions and use context menus


Modified: trunk/Source/WebKit/UIProcess/API/Cocoa/WKUIDelegate.h (246579 => 246580)

--- trunk/Source/WebKit/UIProcess/API/Cocoa/WKUIDelegate.h	2019-06-19 02:36:42 UTC (rev 246579)
+++ trunk/Source/WebKit/UIProcess/API/Cocoa/WKUIDelegate.h	2019-06-19 04:44:11 UTC (rev 246580)
@@ -39,12 +39,6 @@
 @class WKWebViewConfiguration;
 @class WKWindowFeatures;
 
-#if TARGET_OS_IPHONE
-@class WKContextMenuElementInfo;
-@class UIContextMenuConfiguration;
-@protocol UIContextMenuInteractionCommitAnimating;
-#endif
-
 /*! A class conforming to the WKUIDelegate protocol provides methods for
  presenting native UI on behalf of a webpage.
  */
@@ -135,7 +129,7 @@
  This method will only be invoked for elements that have default preview in WebKit, which is
  limited to links. In the future, it could be invoked for additional elements.
  */
-- (BOOL)webView:(WKWebView *)webView shouldPreviewElement:(WKPreviewElementInfo *)elementInfo WK_API_DEPRECATED_WITH_REPLACEMENT("webView:contextMenuConfigurationForElement:completionHandler:", ios(10.0, WK_IOS_TBA));
+- (BOOL)webView:(WKWebView *)webView shouldPreviewElement:(WKPreviewElementInfo *)elementInfo WK_API_DEPRECATED("This API will be replaced", ios(10.0, WK_IOS_TBA));
 
 /*! @abstract Allows your app to provide a custom view controller to show when the given element is peeked.
  @param webView The web view invoking the delegate method.
@@ -150,54 +144,14 @@
  Returning nil will result in WebKit's default preview behavior. webView:commitPreviewingViewController: will only be invoked
  if a non-nil view controller was returned.
  */
-- (nullable UIViewController *)webView:(WKWebView *)webView previewingViewControllerForElement:(WKPreviewElementInfo *)elementInfo defaultActions:(NSArray> *)previewActions WK_API_DEPRECATED_WITH_REPLACEMENT("webView:contextMenuConfigurationForElement:completionHandler:", ios(10.0, WK_IOS_TBA));
+- (nullable UIViewController *)webView:(WKWebView *)webView previewingViewControllerForElement:(WKPreviewElementInfo *)elementInfo defaultActions:(NSArray> *)previewActions WK_API_DEPRECATED("This API will be replaced", ios(10.0, WK_IOS_TBA));
 
 /*! @abstract Allows your app to pop to the view controller it created.
  @param webView The web view invoking the delegate method.
  @param previewingViewController The view controller that is being popped.
  */
-- (void)webView:(WKWebView *)webView commitPreviewingViewController:(UIViewController *)previewingViewController WK_API_DEPRECATED("webView:contextMenuForElement:willCommitWithAnimator:", ios(10.0, WK_IOS_TBA));
+- (void)webView:(WKWebView *)webView commitPreviewingViewController:(UIViewController *)previewingViewController WK_API_DEPRECATED("This API will be replaced", ios(10.0, WK_IOS_TBA));
 
-/**
- * @abstract Called when a context menu interaction begins.
- *
- * @param webView The web view invoking the delegate method.
- * @param elementInfo The elementInfo for the element the user is touching.
- * @param completionHandler A completion handler to call once a it has been decided whether or not to show a context menu.
- * Pass a valid UIContextMenuConfiguration to show a context menu, or pass nil to not show a context menu.
- */
-
-- (void)webView:(WKWebView *)webView contextMenuConfigurationForElement:(WKContextMenuElementInfo *)elementInfo completionHandler:(void (^)(UIContextMenuConfiguration * _Nullable configuration))completionHandler WK_API_AVAILABLE(ios(WK_IOS_TBA));
-
-/**
- * @abstract Called when the context menu configured by the UIContextMenuConfiguration from
- * webView:contextMenuConfigurationForElement:completionHandler: is committed.
- *
- * @param webView The web view invoking the delegate method.
- * @param elementInfo The elementInfo for the element the user is 

[webkit-changes] [246579] trunk

2019-06-18 Thread sbarati
Title: [246579] trunk








Revision 246579
Author sbar...@apple.com
Date 2019-06-18 19:36:42 -0700 (Tue, 18 Jun 2019)


Log Message
[WHLSL] Support matrices
https://bugs.webkit.org/show_bug.cgi?id=198876


Reviewed by Dean Jackson and Myles Maxfield.

Source/WebCore:

This patch adds in support for matrices to WHLSL. Most matrix related code
is defined by the standard library. This patch just needed to add support
for the native functions operator[] and operator[]= on matrix types. The only
native functions that are named operator[] and operator[]= are for matrix
operations, so we strongly assume when generating code for native operator[] and
operator[]= that we're dealing with matrix types.

operator[]= ignores the write if the index is out of bounds. operator[]
returns a zeroed vector if the index is out of bounds.

This patch also incorporates two bug fixes:
1. This patch takes Robin's patch in https://bugs.webkit.org/show_bug.cgi?id=198313 to ensure
we don't have pointers to values in a hash map. This was needed in this patch
otherwise we'd crash parsing the standard library.

2. This patch fixes how we handle "break" in metal codegen. When I first
implemented break, I strongly assumed we were in a loop. However, break
can be either from a loop or from switch. This patch teaches the metal code
generator to track which context we're in and to emit code accordingly.

Tests: webgpu/whlsl-matrix-2.html
   webgpu/whlsl-matrix.html

* Modules/webgpu/WHLSL/Metal/WHLSLFunctionWriter.cpp:
(WebCore::WHLSL::Metal::FunctionDefinitionWriter::visit):
(WebCore::WHLSL::Metal::FunctionDefinitionWriter::emitLoop):
* Modules/webgpu/WHLSL/Metal/WHLSLMetalCodeGenerator.cpp:
(WebCore::WHLSL::Metal::generateMetalCodeShared):
* Modules/webgpu/WHLSL/Metal/WHLSLNativeFunctionWriter.cpp:
(WebCore::WHLSL::Metal::writeNativeFunction):
* Modules/webgpu/WHLSL/WHLSLChecker.cpp:
(WebCore::WHLSL::Checker::assignTypes):
(WebCore::WHLSL::Checker::getInfo):
(WebCore::WHLSL::Checker::assignType):
(WebCore::WHLSL::Checker::forwardType):
* Modules/webgpu/WHLSL/WHLSLStandardLibrary.txt:

LayoutTests:

* webgpu/whlsl-matrix-2-expected.txt: Added.
* webgpu/whlsl-matrix-2.html: Added.
* webgpu/whlsl-matrix-expected.txt: Added.
* webgpu/whlsl-matrix.html: Added.

Modified Paths

trunk/LayoutTests/ChangeLog
trunk/Source/WebCore/ChangeLog
trunk/Source/WebCore/Modules/webgpu/WHLSL/Metal/WHLSLFunctionWriter.cpp
trunk/Source/WebCore/Modules/webgpu/WHLSL/Metal/WHLSLMetalCodeGenerator.cpp
trunk/Source/WebCore/Modules/webgpu/WHLSL/Metal/WHLSLNativeFunctionWriter.cpp
trunk/Source/WebCore/Modules/webgpu/WHLSL/WHLSLChecker.cpp
trunk/Source/WebCore/Modules/webgpu/WHLSL/WHLSLStandardLibrary.txt


Added Paths

trunk/LayoutTests/webgpu/whlsl-matrix-2-expected.txt
trunk/LayoutTests/webgpu/whlsl-matrix-2.html
trunk/LayoutTests/webgpu/whlsl-matrix-expected.txt
trunk/LayoutTests/webgpu/whlsl-matrix.html




Diff

Modified: trunk/LayoutTests/ChangeLog (246578 => 246579)

--- trunk/LayoutTests/ChangeLog	2019-06-19 01:19:41 UTC (rev 246578)
+++ trunk/LayoutTests/ChangeLog	2019-06-19 02:36:42 UTC (rev 246579)
@@ -1,3 +1,16 @@
+2019-06-18  Saam Barati  
+
+[WHLSL] Support matrices
+https://bugs.webkit.org/show_bug.cgi?id=198876
+
+
+Reviewed by Dean Jackson and Myles Maxfield.
+
+* webgpu/whlsl-matrix-2-expected.txt: Added.
+* webgpu/whlsl-matrix-2.html: Added.
+* webgpu/whlsl-matrix-expected.txt: Added.
+* webgpu/whlsl-matrix.html: Added.
+
 2019-06-18  Russell Epstein  
 
 Layout Test imported/w3c/web-platform-tests/content-security-policy/reporting/report-only-in-meta.sub.html is failing.


Added: trunk/LayoutTests/webgpu/whlsl-matrix-2-expected.txt (0 => 246579)

--- trunk/LayoutTests/webgpu/whlsl-matrix-2-expected.txt	(rev 0)
+++ trunk/LayoutTests/webgpu/whlsl-matrix-2-expected.txt	2019-06-19 02:36:42 UTC (rev 246579)
@@ -0,0 +1,12 @@
+PASS successfullyParsed is true
+
+TEST COMPLETE
+PASS resultsFloat32Array[0] is 2
+PASS resultsFloat32Array[1] is 4
+PASS resultsFloat32Array[2] is 6
+PASS resultsFloat32Array[3] is 8
+PASS resultsFloat32Array[4] is 5
+PASS resultsFloat32Array[5] is 6
+PASS resultsFloat32Array[6] is 7
+PASS resultsFloat32Array[7] is 8
+


Added: trunk/LayoutTests/webgpu/whlsl-matrix-2.html (0 => 246579)

--- trunk/LayoutTests/webgpu/whlsl-matrix-2.html	(rev 0)
+++ trunk/LayoutTests/webgpu/whlsl-matrix-2.html	2019-06-19 02:36:42 UTC (rev 246579)
@@ -0,0 +1,152 @@
+
+
+
+
+
+