On Tue, 8 Aug 2023 16:39:30 GMT, Kevin Rushforth <[email protected]> wrote:
>> Jay Bhaskar has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> clean up according to review comments
>
> modules/javafx.web/src/main/native/Source/WebCore/platform/java/ModernMediaControlResource.cpp
> line 28:
>
>> 26: #include "ModernMediaControlResource.h"
>> 27:
>> 28: ModernMediaControlResource::ModernMediaControlResource() {
>
> Can you add a comment indicating that the Base64 images in this file were
> generated from the resources in
> `WebCore/Modules/modern-media-controls/images/adwaita/` using the
> `Tools/Scripts/generate_icon_resource.py` script?
done
-------------
PR Review Comment: https://git.openjdk.org/jfx/pull/1201#discussion_r1287448895
