Bug 227868 - [GTK][WPE] Support drawing scrollbar corner
Summary: [GTK][WPE] Support drawing scrollbar corner
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKitGTK (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2021-07-12 03:13 PDT by Alice Mikhaylenko
Modified: 2021-07-12 07:36 PDT (History)
11 users (show)

See Also:


Attachments
Patch (10.61 KB, patch)
2021-07-12 03:18 PDT, Alice Mikhaylenko
ews-feeder: commit-queue-
Details | Formatted Diff | Diff
Patch (12.10 KB, patch)
2021-07-12 03:49 PDT, Alice Mikhaylenko
aperez: review+
Details | Formatted Diff | Diff
Patch (12.10 KB, patch)
2021-07-12 04:38 PDT, Alice Mikhaylenko
no flags Details | Formatted Diff | Diff
Patch (12.11 KB, patch)
2021-07-12 04:39 PDT, Alice Mikhaylenko
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 Alice Mikhaylenko 2021-07-12 03:13:57 PDT
See the patch.
Comment 1 Alice Mikhaylenko 2021-07-12 03:18:48 PDT
Created attachment 433308 [details]
Patch
Comment 2 Alice Mikhaylenko 2021-07-12 03:49:20 PDT
Created attachment 433309 [details]
Patch

Missed a use outside WebCore.
Comment 3 Adrian Perez 2021-07-12 04:32:53 PDT
Comment on attachment 433309 [details]
Patch

Patch LGTM, we may want to have someone from Apple confirm that the
additional parameter passed around is okay for them :)
Comment 4 Adrian Perez 2021-07-12 04:34:03 PDT
Ah, and please add the [WPE] in the ChangeLog summary line; the port
uses the same code to paint scrollbars nowadays.
Comment 5 Alice Mikhaylenko 2021-07-12 04:37:37 PDT
Yeah, I noticed the macOS scroll corners were slightly broken with scrollbars on the left since there's no way to check that without the new parameter. :) I have no idea how to request RTL flip there though (it uses native scrollbars) so didn't update it.
Comment 6 Alice Mikhaylenko 2021-07-12 04:38:26 PDT
Created attachment 433310 [details]
Patch

Added [WPE].
Comment 7 Alice Mikhaylenko 2021-07-12 04:39:43 PDT
Created attachment 433311 [details]
Patch

And of course I forgot the second changelog in WebKit/. Updated too.
Comment 8 Michael Catanzaro 2021-07-12 05:48:30 PDT
Comment on attachment 433311 [details]
Patch

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

> Source/WebKit/WebProcess/Plugins/PDF/PDFPlugin.mm:1956
> -        ScrollbarTheme::theme().paintScrollCorner(graphicsContext, scrollCornerRect);
> +        ScrollbarTheme::theme().paintScrollCorner(*this, graphicsContext, scrollCornerRect);

*Technically* this line of the changeset requires owner approval, but it's such a small unproblematic change, only intended to keep this code building, that I think we can skip the formality in this case.
Comment 9 Alice Mikhaylenko 2021-07-12 06:19:16 PDT
Ah yeah, flaky tests again.
Comment 10 EWS 2021-07-12 07:26:51 PDT
Committed r279826 (239587@main): <https://commits.webkit.org/239587@main>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 433311 [details].