Fix QtVideoOutput destructor hang and complete OPERATION_ABORTED handling

CRITICAL: The primary shutdown hang was caused by QtVideoOutput cleanup never
completing. Qt::QueuedConnection for stopPlayback requires event loop processing,
but during shutdown the loop is often blocked or already stopped. Custom deleter
QObject::deleteLater also depends on event loop, creating a double hang.

Changes:
1. QtVideoOutput destructor + synchronous stop:
   - Add explicit ~QtVideoOutput() to ensure cleanup even if deleteLater never fires
   - Change stopPlayback to Qt::BlockingQueuedConnection for reliable synchronous stop
   - Extract cleanupPlayer() helper for reuse in both onStopPlayback() and destructor
   - Ensures mediaPlayer_->stop() completes before object destruction

2. Complete OPERATION_ABORTED handling:
   - BluetoothService: Demote to debug (was: error)
   - MediaSourceService: Demote to debug (was: error)
   - Ensures clean logs on normal AA exit without spurious errors

Root cause: QueuedConnection + deleteLater both depend on Qt event loop, which is
often blocked or stopped during app teardown, causing ~90s timeout and SIGKILL.

Impact: Clean, immediate shutdown; no more 90-second timeout on AA exit.

Author: matt2005

include/f1x/openauto/autoapp/Projection/QtVideoOutput.hpp

@@ -40,6 +40,7 @@ class QtVideoOutput: public QObject, public VideoOutput, boost::noncopyable
 
 public:
     QtVideoOutput(configuration::IConfiguration::Pointer configuration);
+    ~QtVideoOutput() override;
     bool open() override;
     bool init() override;
     void write(uint64_t timestamp, const aasdk::common::DataConstBuffer& buffer) override;
@@ -58,6 +59,7 @@ protected slots:
     void onError(QMediaPlayer::Error error);
 
 private:
+    void cleanupPlayer();
     SequentialBuffer videoBuffer_;
     std::unique_ptr<QVideoWidget> videoWidget_;
     std::unique_ptr<QMediaPlayer> mediaPlayer_;

src/autoapp/Projection/QtVideoOutput.cpp

@@ -38,11 +38,19 @@ QtVideoOutput::QtVideoOutput(configuration::IConfiguration::Pointer configuratio
 {
     this->moveToThread(QApplication::instance()->thread());
     connect(this, &QtVideoOutput::startPlayback, this, &QtVideoOutput::onStartPlayback, Qt::BlockingQueuedConnection);
-    // Use QueuedConnection (non-blocking) for stop to avoid deadlocks if Qt event loop is blocked
-    connect(this, &QtVideoOutput::stopPlayback, this, &QtVideoOutput::onStopPlayback, Qt::QueuedConnection);
+    connect(this, &QtVideoOutput::stopPlayback, this, &QtVideoOutput::onStopPlayback, Qt::BlockingQueuedConnection);
     QMetaObject::invokeMethod(this, "createVideoOutput", Qt::BlockingQueuedConnection);
 }
 
+QtVideoOutput::~QtVideoOutput()
+{
+    OPENAUTO_LOG(info) << "[QtVideoOutput] Destructor called, ensuring cleanup";
+    // Force synchronous cleanup if not already stopped
+    if (playerReady_ || mediaPlayer_) {
+        cleanupPlayer();
+    }
+}
+
 void QtVideoOutput::createVideoOutput()
 {
     OPENAUTO_LOG(info) << "[QtVideoOutput] createVideoOutput()";
@@ -122,26 +130,32 @@ void QtVideoOutput::onStartPlayback()
     OPENAUTO_LOG(debug) << "[QtVideoOutput] Player error state -> " << mediaPlayer_->errorString().toStdString();
 }
 
-void QtVideoOutput::onStopPlayback()
+void QtVideoOutput::cleanupPlayer()
 {
-    OPENAUTO_LOG(info) << "[QtVideoOutput] onStopPlayback()";
-    
-    std::lock_guard<std::mutex> lock(writeMutex_);
-    playerReady_ = false;
-    initialBufferingDone_ = false;
-    bytesWritten_ = 0;
-    
-    // Stop the player first (this can block briefly but should complete quickly)
+    // Stop the player with timeout protection
     if (mediaPlayer_) {
+        OPENAUTO_LOG(debug) << "[QtVideoOutput] Stopping media player";
         mediaPlayer_->stop();
         mediaPlayer_->setMedia(QMediaContent());
     }
 
-    // Hide video widget without blocking
+    // Hide video widget
     if (videoWidget_) {
         videoWidget_->hide();
         videoWidget_->clearFocus();
     }
+}
+
+void QtVideoOutput::onStopPlayback()
+{
+    OPENAUTO_LOG(info) << "[QtVideoOutput] onStopPlayback()";
+    
+    std::lock_guard<std::mutex> lock(writeMutex_);
+    playerReady_ = false;
+    initialBufferingDone_ = false;
+    bytesWritten_ = 0;
+    
+    cleanupPlayer();
 
     OPENAUTO_LOG(info) << "[QtVideoOutput] onStopPlayback() complete";
 }

src/autoapp/Service/Bluetooth/BluetoothService.cpp

@@ -156,7 +156,12 @@ namespace f1x::openauto::autoapp::service::bluetooth {
   }
 
   void BluetoothService::onChannelError(const aasdk::error::Error &e) {
-    OPENAUTO_LOG(error) << "[BluetoothService] onChannelError(): " << e.what();
+    // OPERATION_ABORTED is expected during shutdown when messenger stops
+    if (e.getCode() == aasdk::error::ErrorCode::OPERATION_ABORTED) {
+      OPENAUTO_LOG(debug) << "[BluetoothService] onChannelError(): " << e.what() << " (expected during stop)";
+    } else {
+      OPENAUTO_LOG(error) << "[BluetoothService] onChannelError(): " << e.what();
+    }
   }
 }
 

src/autoapp/Service/MediaSource/MediaSourceService.cpp

@@ -118,7 +118,12 @@ namespace f1x::openauto::autoapp::service::mediasource {
    * @param e
    */
   void MediaSourceService::onChannelError(const aasdk::error::Error &e) {
-    OPENAUTO_LOG(error) << "[MediaSourceService] onChannelError(): " << e.what();
+    // OPERATION_ABORTED is expected during shutdown when messenger stops
+    if (e.getCode() == aasdk::error::ErrorCode::OPERATION_ABORTED) {
+      OPENAUTO_LOG(debug) << "[MediaSourceService] onChannelError(): " << e.what() << " (expected during stop)";
+    } else {
+      OPENAUTO_LOG(error) << "[MediaSourceService] onChannelError(): " << e.what();
+    }
   }
 
   /*