Remove done() call on SubMonitor in Refactoring

[vogella]

done() call not necessary here

See https://www.eclipse.org/articles/Article-Progress-Monitors/article.html

[github-actions]

Test Results

 1 821 files   1 821 suites   1h 31m 6s ⏱️
 7 719 tests  7 489 ✅ 228 💤 2 ❌
24 318 runs  23 567 ✅ 749 💤 2 ❌

For more details on these failures, see this check.

Results for commit f4a3c41.

♻️ This comment has been updated with latest results.

[jukzi]

The caller Will not expect that the callee invokes done() on the monitor. The caller must either use an SubMonitor (or a similar implementation which does not require done() to be invoked), or it must take responsibility for calling done() on the monitor after the callee has finished.
Reads like this done() is valid

[vogella]

The caller Will not expect that the callee invokes done() on the monitor. The caller must either use an SubMonitor (or a similar implementation which does not require done() to be invoked), or it must take responsibility for calling done() on the monitor after the callee has finished. Reads like this done() is valid

See Changes in Eclipse 4.7

See 5.3 Changes in Eclipse 4.7