Bug 216949 - Support EXT_float_blend
Summary: Support EXT_float_blend
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebGL (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: James Darpinian
URL:
Keywords: InRadar
: 219851 (view as bug list)
Depends on:
Blocks: 216014
  Show dependency treegraph
 
Reported: 2020-09-24 16:35 PDT by James Darpinian
Modified: 2020-12-15 12:20 PST (History)
14 users (show)

See Also:


Attachments
Patch (21.93 KB, patch)
2020-09-24 16:44 PDT, James Darpinian
no flags Details | Formatted Diff | Diff
Patch (381.22 KB, patch)
2020-09-24 17:02 PDT, James Darpinian
no flags Details | Formatted Diff | Diff
Patch (179.21 KB, patch)
2020-09-25 14:35 PDT, James Darpinian
no flags Details | Formatted Diff | Diff
Patch (179.21 KB, patch)
2020-09-25 14:53 PDT, James Darpinian
ews-feeder: commit-queue-
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description James Darpinian 2020-09-24 16:35:07 PDT
Support EXT_float_blend
Comment 1 James Darpinian 2020-09-24 16:44:41 PDT
Created attachment 409636 [details]
Patch
Comment 2 EWS Watchlist 2020-09-24 16:45:46 PDT
Note that there are important steps to take when updating ANGLE. See https://trac.webkit.org/wiki/UpdatingANGLE
Comment 3 James Darpinian 2020-09-24 17:02:57 PDT
Created attachment 409637 [details]
Patch
Comment 4 Kenneth Russell 2020-09-24 17:57:20 PDT
Comment on attachment 409637 [details]
Patch

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

Looks good overall! The bulk of this patch is new conformance suite files which it looks like were automatically and maybe accidentally imported; please remove those so that the patch can stay better focused. Thanks.

> Source/ThirdParty/ANGLE/src/libANGLE/renderer/gl/renderergl_utils.cpp:1485
> +                             functions->isAtLeastGLES(gl::Version(3, 2));

Double-checked the ES 3.2 spec, and the reason this is implicitly supported is that floating-point framebuffers were subsumed into the core ES 3.2 spec without the restrictions listed in EXT_color_buffer_float. Not sure whether that's worth a comment.

> Source/WebCore/html/canvas/EXTFloatBlend.cpp:2
> + * Copyright (C) 2014 Apple Inc. All rights reserved.

2020? Here and below.

> LayoutTests/webgl/resources/webgl_test_files/js/desktop-gl-constants.js:1
> +/*

Could you remove the addition of this file? It looks like it's unreferenced.

Actually it looks like many new files were accidentally imported from the top-of-tree conformance suite. Please remove them to keep this patch better focused.
Comment 5 James Darpinian 2020-09-25 14:35:38 PDT
Created attachment 409741 [details]
Patch
Comment 6 James Darpinian 2020-09-25 14:40:08 PDT
Comment on attachment 409637 [details]
Patch

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

>> Source/WebCore/html/canvas/EXTFloatBlend.cpp:2
>> + * Copyright (C) 2014 Apple Inc. All rights reserved.
> 
> 2020? Here and below.

Done.

>> LayoutTests/webgl/resources/webgl_test_files/js/desktop-gl-constants.js:1
>> +/*
> 
> Could you remove the addition of this file? It looks like it's unreferenced.
> 
> Actually it looks like many new files were accidentally imported from the top-of-tree conformance suite. Please remove them to keep this patch better focused.

You're right, this particular file is unnecessary. But I did intentionally import some files from top-of-tree conformance. ext-float-blend.html is not present in the 2.0.0 or 1.0.3 snapshots, so it is not automatically imported. There were already some manually imported tests, so I manually imported ext-float-blend.html alongside them, with some of the supporting files it needs.

I went through and deleted a couple of files that were unused. The rest are used by ext-float-blend.html. Hopefully in the future WebKit will import the 2.0.1 suite and then ext-float-blend.html will be automatically imported.
Comment 7 EWS 2020-09-25 14:47:53 PDT
ChangeLog entry in LayoutTests/ChangeLog contains OOPS!.
Comment 8 James Darpinian 2020-09-25 14:53:37 PDT
Created attachment 409744 [details]
Patch
Comment 9 Kenneth Russell 2020-09-25 14:57:50 PDT
Comment on attachment 409637 [details]
Patch

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

>>> LayoutTests/webgl/resources/webgl_test_files/js/desktop-gl-constants.js:1
>>> +/*
>> 
>> Could you remove the addition of this file? It looks like it's unreferenced.
>> 
>> Actually it looks like many new files were accidentally imported from the top-of-tree conformance suite. Please remove them to keep this patch better focused.
> 
> You're right, this particular file is unnecessary. But I did intentionally import some files from top-of-tree conformance. ext-float-blend.html is not present in the 2.0.0 or 1.0.3 snapshots, so it is not automatically imported. There were already some manually imported tests, so I manually imported ext-float-blend.html alongside them, with some of the supporting files it needs.
> 
> I went through and deleted a couple of files that were unused. The rest are used by ext-float-blend.html. Hopefully in the future WebKit will import the 2.0.1 suite and then ext-float-blend.html will be automatically imported.

Thanks, I understand now. Sounds fine.
Comment 10 EWS 2020-09-25 15:41:32 PDT
Committed r267601: <https://trac.webkit.org/changeset/267601>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 409744 [details].
Comment 11 Radar WebKit Bug Importer 2020-09-25 15:42:21 PDT
<rdar://problem/69598086>
Comment 12 Kenneth Russell 2020-12-15 12:20:35 PST
*** Bug 219851 has been marked as a duplicate of this bug. ***