Bug 178874 - Demonstrate a possible structure of the WebGPU API
Summary: Demonstrate a possible structure of the WebGPU API
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Myles C. Maxfield
URL:
Keywords: InRadar
Depends on:
Blocks: 178912
  Show dependency treegraph
 
Reported: 2017-10-26 10:01 PDT by Myles C. Maxfield
Modified: 2017-11-15 12:52 PST (History)
5 users (show)

See Also:


Attachments
Patch (248.62 KB, patch)
2017-10-26 10:15 PDT, Myles C. Maxfield
no flags Details | Formatted Diff | Diff
Patch (248.66 KB, patch)
2017-10-26 11:57 PDT, Myles C. Maxfield
dino: review+
commit-queue: commit-queue-
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Myles C. Maxfield 2017-10-26 10:01:55 PDT
Demonstrate a possible structure of the WebGPU API
Comment 1 Myles C. Maxfield 2017-10-26 10:15:39 PDT
Created attachment 325024 [details]
Patch
Comment 2 Build Bot 2017-10-26 10:19:01 PDT
Attachment 325024 [details] did not pass style-queue:


ERROR: Unexpected diff format when parsing a chunk: 'The file will have its original line endings in your working directory.'
ERROR: Unexpected diff format when parsing a chunk: 'nformation>true</GenerateDebugInformation>'
ERROR: Tools/ChangeLog:77:  Line contains tab character.  [whitespace/tab] [5]
ERROR: Tools/ChangeLog:78:  Line contains tab character.  [whitespace/tab] [5]
ERROR: Tools/ChangeLog:79:  Line contains tab character.  [whitespace/tab] [5]
ERROR: Tools/ChangeLog:80:  Line contains tab character.  [whitespace/tab] [5]
ERROR: Tools/ChangeLog:81:  Line contains tab character.  [whitespace/tab] [5]
ERROR: Tools/ChangeLog:82:  Line contains tab character.  [whitespace/tab] [5]
ERROR: Tools/ChangeLog:83:  Line contains tab character.  [whitespace/tab] [5]
ERROR: Tools/ChangeLog:84:  Line contains tab character.  [whitespace/tab] [5]
ERROR: Tools/ChangeLog:179:  Line contains tab character.  [whitespace/tab] [5]
ERROR: Tools/ChangeLog:180:  Line contains tab character.  [whitespace/tab] [5]
ERROR: Tools/ChangeLog:181:  Line contains tab character.  [whitespace/tab] [5]
Total errors found: 11 in 42 files


If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 3 Myles C. Maxfield 2017-10-26 11:57:49 PDT
Created attachment 325035 [details]
Patch
Comment 4 Dean Jackson 2017-10-26 12:19:13 PDT
Comment on attachment 325035 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=325035&action=review

> Tools/ChangeLog:11
> +        Over the past few weeks, we've been putting together an example showing that a WebGPU API
> +        which has implicit barriers can work on all three low-level graphics APIs. We've implemented
> +        it on top of Vulkan first, because this is the API which has the strictest requirements and
> +        is most difficult to use.

I think most of this ChangeLog should go in a README.txt file, because it is both useful and something you'd want to refer to directly within the project.
Comment 5 Myles C. Maxfield 2017-10-26 12:20:35 PDT
Comment on attachment 325035 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=325035&action=review

>> Tools/ChangeLog:11
>> +        is most difficult to use.
> 
> I think most of this ChangeLog should go in a README.txt file, because it is both useful and something you'd want to refer to directly within the project.

Yep! I will make it a .md file.
Comment 6 WebKit Commit Bot 2017-10-26 12:42:21 PDT
Comment on attachment 325035 [details]
Patch

Rejecting attachment 325035 [details] from commit-queue.

Failed to run "['/Volumes/Data/EWS/WebKit/Tools/Scripts/webkit-patch', '--status-host=webkit-queues.webkit.org', '--bot-id=webkit-cq-02', 'land-attachment', '--force-clean', '--non-interactive', '--parent-command=commit-queue', 325035, '--port=mac']" exit_code: 2 cwd: /Volumes/Data/EWS/WebKit

Last 500 characters of output:
sing rebase:
:040000 040000 883c0b3ecf28c0344a8fb3de466ce9bec4b4de14 b307e97f51fcd059a31d836192577582eebdf63a M	Tools
Current branch master is up to date.
ERROR: Not all changes have been committed into SVN, however the committed
ones (if any) seem to be successfully integrated into the working tree.
Please see the above messages for details.


Failed to run "['git', 'svn', 'dcommit', '--rmdir']" exit_code: 1 cwd: /Volumes/Data/EWS/WebKit
Updating OpenSource
Current branch master is up to date.

Full output: http://webkit-queues.webkit.org/results/5001765
Comment 7 Myles C. Maxfield 2017-10-26 16:32:24 PDT
Committed r224065: <https://trac.webkit.org/changeset/224065>
Comment 8 Radar WebKit Bug Importer 2017-11-15 12:52:05 PST
<rdar://problem/35568310>