Uploaded image for project: 'JDK'
  1. JDK
  2. JDK-8244531

Tests: add support to identify recurring issues with controls et al

    XMLWordPrintable

Details

    Description

      Currently, we identified several recurring issues with skin implementations, all (at least loosely) related to replacing skins. Amongst them:

      - memory leaks: JDK-8241364
      - NPE, IOOBE or other harmful side-effects, f.i. JDK-8244110
      - contract violation in dispose JDK-8244112
      - maybe others?

      The very first task is to _find_ all these issues across all skins (not just by chance). The solution are (hopefully :) tests that are parameterized in control classes, focusing onto just a single (or several related) common misbehavior. That's already done for the contract violation in dispose.

      The task addressed here is use a similar pattern for memory leaks/side-effects.

      Attachments

        Issue Links

          blocks

          Bug - A problem which impairs or prevents the functions of the product. JDK-8245282 Button/Combo Behavior: memory leak on dispose

          • P4 - Minor loss of function, or other problem where easy workaround is present.
          • Resolved
          relates to

          Bug - A problem which impairs or prevents the functions of the product. JDK-8256649 Parameterized tests must not use instances as parameters

          • P3 - Major loss of function.
          • Resolved

          Bug - A problem which impairs or prevents the functions of the product. JDK-8244112 Skin implementations: must not violate contract of dispose

          • P4 - Minor loss of function, or other problem where easy workaround is present.
          • Resolved

          Task - A task that needs to be done. JDK-8241364 ☂ Cleanup skin implementations to allow switching

          • P4 - Minor loss of function, or other problem where easy workaround is present.
          • Open
          links to

          Web Link Commit openjdk/jfx/f3190db7

          Web Link Review openjdk/jfx/223

          Activity

            People

              fastegal Jeanette Winzenburg
              fastegal Jeanette Winzenburg
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: