TabControl::DeactivatePage should return bool

Change-Id: I9fb774a235db2b9bc759976d2d3f84363ed944ec

chart2/source/controller/dialogs/dlg_DataSource.cxx

@@ -86,7 +86,7 @@ class DataSourceTabControl : public TabControl
 public:
     DataSourceTabControl(Window* pParent);
 
-    virtual long DeactivatePage();
+    virtual bool DeactivatePage() SAL_OVERRIDE;
 
     void DisableTabToggling();
     void EnableTabToggling();
@@ -101,14 +101,9 @@ DataSourceTabControl::DataSourceTabControl(Window* pParent)
 {
 }
 
-// Note that the result is long, but is intended to be a bool
-long DataSourceTabControl::DeactivatePage()
+bool DataSourceTabControl::DeactivatePage()
 {
-    bool bCanDeactivate( TabControl::DeactivatePage() != 0 );
-
-    bCanDeactivate = (bCanDeactivate && m_bTogglingEnabled);
-
-    return (bCanDeactivate ? 1 : 0 );
+    return TabControl::DeactivatePage() && m_bTogglingEnabled;
 }
 
 void DataSourceTabControl::DisableTabToggling()

include/vcl/tabctrl.hxx

@@ -100,7 +100,7 @@ public:
     virtual bool        PreNotify( NotifyEvent& rNEvt );
 
     virtual void        ActivatePage();
-    virtual long        DeactivatePage();
+    virtual bool        DeactivatePage();
 
     virtual Size GetOptimalSize() const;
 

vcl/source/control/tabctrl.cxx

@@ -1698,12 +1698,9 @@ void TabControl::ActivatePage()
 
 // -----------------------------------------------------------------------
 
-long TabControl::DeactivatePage()
+bool TabControl::DeactivatePage()
 {
-    if ( maDeactivateHdl.IsSet() )
-        return maDeactivateHdl.Call( this );
-    else
-        return sal_True;
+    return !maDeactivateHdl.IsSet() || maDeactivateHdl.Call( this );
 }
 
 // -----------------------------------------------------------------------