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

Scene@MouseHandler.pick() is not efficient

    Details

      Description

      Implementation relies on Node.contains() method that might be very time consuming (see RT-5278 and RT-5279 for example).
      And pick() is actually called on each pulse.

      We should not be doing more than needed.

      In particular it has no sense to check whether point is in the node if the following is true:
         a) Node has no childs (e.g. text nodes never have any childs)
         b) Node itself is not interested in any mouse updates

      This may significantly improve performance.
      E.g. solving this in addition to fix for Rt-5278 improves fps for transformed text testcase from RT-5278 from 2 to 250.

        Attachments

          Issue Links

            Activity

              People

              • Assignee:
                ckyang Chien Yang (Inactive)
                Reporter:
                igor Igor Nekrestyanov (Inactive)
              • Votes:
                0 Vote for this issue
                Watchers:
                4 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Imported:

                  Time Tracking

                  Estimated:
                  Original Estimate - 1 week
                  1w
                  Remaining:
                  Remaining Estimate - 1 week
                  1w
                  Logged:
                  Time Spent - Not Specified
                  Not Specified