Drop special idle handling

Idles are just instant timers, which should most time have a low
priority, By dropping most special idle handling we'll just
schedule by priority.

This also reverts SalYieldResult back to a bool, which just
indicates if any event was processed.

Change-Id: Ia0b91b06dffb77af066f01838d8f9483523bf67d

4 files changed, 7 insertions, 6 deletions

diff --git a/include/vcl/idle.hxx b/include/vcl/idle.hxx
index 2112fc62464c..157d14d96816 100644
--- a/include/vcl/idle.hxx
+++ b/include/vcl/idle.hxx
@@ -35,7 +35,7 @@ private:
     sal_uInt64    GetTimeout() const = delete;
 
 protected:
-    virtual bool ReadyForSchedule( bool bIdle, sal_uInt64 nTimeNow ) const override;
+    virtual bool ReadyForSchedule( sal_uInt64 nTimeNow ) const override;
     virtual sal_uInt64 UpdateMinPeriod( sal_uInt64 nMinPeriod, sal_uInt64 nTimeNow ) const override;
 
     Idle( bool bAuto, const sal_Char *pDebugName = nullptr );
diff --git a/include/vcl/scheduler.hxx b/include/vcl/scheduler.hxx
index f68b4a3b1a60..1dd127c96f9a 100644
--- a/include/vcl/scheduler.hxx
+++ b/include/vcl/scheduler.hxx
@@ -47,16 +47,17 @@ public:
     static void       ImplDeInitScheduler();
 
     /// Process one pending Timer with highhest priority
-    static void       CallbackTaskScheduling( bool ignore );
+    static void       CallbackTaskScheduling();
     /// Are there any pending tasks to process?
     static bool       HasPendingTasks();
     /// Process one pending task ahead of time with highest priority.
-    static bool       ProcessTaskScheduling( bool bIdle );
+    static bool       ProcessTaskScheduling();
+    /// Process all events until we are idle
+    static void       ProcessEventsToIdle();
     /**
      * Process events until the parameter turns true,
      * allows processing until a specific event has been processed
      */
-    static void       ProcessEventsToIdle();
     static void       ProcessEventsToSignal(bool& bSignal);
 
     /// Control the deterministic mode.  In this mode, two subsequent runs of
diff --git a/include/vcl/task.hxx b/include/vcl/task.hxx
index f2cee319a64f..21eb934fee6d 100644
--- a/include/vcl/task.hxx
+++ b/include/vcl/task.hxx
@@ -57,7 +57,7 @@ protected:
 
     virtual void SetDeletionFlags();
     /// Is this item ready to be dispatched at nTimeNow
-    virtual bool ReadyForSchedule( bool bIdle, sal_uInt64 nTimeNow ) const = 0;
+    virtual bool ReadyForSchedule( sal_uInt64 nTimeNow ) const = 0;
     /**
      * Adjust nMinPeriod downwards if we want to be notified before
      * then, nTimeNow is the current time.
diff --git a/include/vcl/timer.hxx b/include/vcl/timer.hxx
index c4cb297022d1..5be766cb6aa7 100644
--- a/include/vcl/timer.hxx
+++ b/include/vcl/timer.hxx
@@ -31,7 +31,7 @@ class VCL_DLLPUBLIC Timer : public Task
 
 protected:
     virtual void SetDeletionFlags() override;
-    virtual bool ReadyForSchedule( bool bIdle, sal_uInt64 nTimeNow ) const override;
+    virtual bool ReadyForSchedule( sal_uInt64 nTimeNow ) const override;
     virtual sal_uInt64 UpdateMinPeriod( sal_uInt64 nMinPeriod, sal_uInt64 nTimeNow ) const override;
 
     Timer( bool bAuto, const sal_Char *pDebugName = nullptr );