Bug 74520 - [GTK] Fix PlatformScreen::screenAvailableRect when there's no view widget
Summary: [GTK] Fix PlatformScreen::screenAvailableRect when there's no view widget
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKitGTK (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: James Robinson
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-12-14 10:52 PST by Carlos Garcia Campos
Modified: 2011-12-19 11:26 PST (History)
5 users (show)

See Also:


Attachments
Patch (1.84 KB, patch)
2011-12-14 10:57 PST, Carlos Garcia Campos
no flags Details | Formatted Diff | Diff
Patch (25.26 KB, patch)
2011-12-14 11:36 PST, James Robinson
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Carlos Garcia Campos 2011-12-14 10:52:35 PST
We already fixed the other methods in bug #63256, using the default display/screen, we could do the same for screenAvailableRect().
Comment 1 Carlos Garcia Campos 2011-12-14 10:57:59 PST
Created attachment 119252 [details]
Patch
Comment 2 Martin Robinson 2011-12-14 11:07:19 PST
Comment on attachment 119252 [details]
Patch

Wish there was a way to do this properly for multiscreen setups. I guess eventually we'll need to pipe this information from the UI process.
Comment 3 Carlos Garcia Campos 2011-12-14 11:18:45 PST
Committed r102802: <http://trac.webkit.org/changeset/102802>
Comment 4 James Robinson 2011-12-14 11:35:58 PST
Reopening to attach new patch.
Comment 5 James Robinson 2011-12-14 11:36:00 PST
Created attachment 119265 [details]
Patch
Comment 6 Carlos Garcia Campos 2011-12-14 11:43:02 PST
It seems you reopened the wrong bug . . . :-P
Comment 7 James Robinson 2011-12-14 11:47:44 PST
:(
Comment 8 Eric Seidel (no email) 2011-12-19 11:26:31 PST
Comment on attachment 119265 [details]
Patch

Cleared review? from attachment 119265 [details] so that this bug does not appear in http://webkit.org/pending-review.  If you would like this patch reviewed, please attach it to a new bug (or re-open this bug before marking it for review again).