New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
8258788: incorrect response to change in window insets [lanai] #3308
Conversation
👋 Welcome back avu! A progress list of the required criteria for merging this PR into |
To avoid this situation, create a new branch for your changes and reset the
Then proceed to create a new pull request with |
Webrevs
|
Is it possible to automatically test it? |
Yes, just added the test. |
test/jdk/java/awt/Window/FullWindowContentTest/FullWindowContentRenderTest.java
Outdated
Show resolved
Hide resolved
@avu Test passes without fix also. |
Verified test case attached in JBS : https://bugs.openjdk.java.net/browse/JDK-8258788 . I see that fix resolves identified issue in JBS. Also jtreg and JCK test run is green with and without Metal API validation flags. |
|
@avu I am running test in 13 inch Macbook Early 2015 with integrated Intel Iris Graphics 6100. |
|
Perform replaceSurfaceData on insets change
I've created a simpler solution within another pull request. Please, have a look https://git.openjdk.java.net/jdk/pull/3390 |
Dynamically change MTLLayer insets depending on FULL_WINDOW_CONTENT property. MTLLayer.h header cleanup.
Progress
Issue
Reviewing
Using
git
Checkout this PR locally:
$ git fetch https://git.openjdk.java.net/jdk pull/3308/head:pull/3308
$ git checkout pull/3308
Update a local copy of the PR:
$ git checkout pull/3308
$ git pull https://git.openjdk.java.net/jdk pull/3308/head
Using Skara CLI tools
Checkout this PR locally:
$ git pr checkout 3308
View PR using the GUI difftool:
$ git pr show -t 3308
Using diff file
Download this PR as a diff file:
https://git.openjdk.java.net/jdk/pull/3308.diff