RFR: 8260528: Clean glass-gtk sizing and positioning code [v3]

Thiago Milczarek Sayao tsayao at openjdk.org
Mon Oct 17 00:10:29 UTC 2022


> This cleans size and positioning code, reducing special cases, code complexity and size.
> 
> Changes:
> 
> - cached extents: 28, 1, 1, 1 are old defaults - modern gnome uses different sizes. It does not assume any size because it varies - it does cache because it's unlikely to vary on the same system - but if it does occur, it will only waste a resize event.
> - frame extents are received in process_property_notify;
> - window geometry, min/max size are centralized in update_window_constraints and uses  `XSetWMNormalHints` because it supports initial position;
> - window_configure: Changes can be requested directly to the window manager with `XReconfigureWMWindow` - it's compatible with `windowChangesMask`, `windowChanges`;
> - request_focus uses `_NET_ACTIVE_WINDOW` directly as it does not seem to have focus stealing problems.
> 
> I have left some debug print statements which will be removed when we finish testing.

Thiago Milczarek Sayao has refreshed the contents of this pull request, and previous commits have been removed. The incremental views will show differences compared to the previous content of the PR. The pull request contains one new commit since the last revision:

  8260528: Clean glass-gtk sizing and positioning code

-------------

Changes:
  - all: https://git.openjdk.org/jfx/pull/915/files
  - new: https://git.openjdk.org/jfx/pull/915/files/621ea2dc..9fb69ece

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jfx&pr=915&range=02
 - incr: https://webrevs.openjdk.org/?repo=jfx&pr=915&range=01-02

  Stats: 0 lines in 0 files changed: 0 ins; 0 del; 0 mod
  Patch: https://git.openjdk.org/jfx/pull/915.diff
  Fetch: git fetch https://git.openjdk.org/jfx pull/915/head:pull/915

PR: https://git.openjdk.org/jfx/pull/915


More information about the openjfx-dev mailing list