On Thu, 22 Jun 2023 21:31:52 GMT, Sergey Bylokhov <[email protected]> wrote:
> I guess we should call "s->VerifyState();" after the "s->SetInsets(env);" > similar to what we do in the "AwtScrollPane::_SetInsets" @mrserb An interesting question… You're right, I've introduced an inconsistency. Yet I don't think it matters because `SetInsets` doesn't change any members of `AwtScrollPane` object which are asserted in `VerifyState`. In addition to it, `VerifyState` is a no-op in release builds. I'll take this into account in [JDK-8310056](https://bugs.openjdk.org/browse/JDK-8310056). ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/14478#discussion_r1254275679
