fit-webview-bridge 0.2.1a1__tar.gz → 0.2.1a4__tar.gz

{fit_webview_bridge-0.2.1a1 → fit_webview_bridge-0.2.1a4}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.2
 Name: fit-webview-bridge
-Version: 0.2.1a1
+Version: 0.2.1a4
 Summary: Qt native WebView bridge with PySide6 bindings
 Author: FIT Project
 License: LGPL-3.0-or-later

{fit_webview_bridge-0.2.1a1 → fit_webview_bridge-0.2.1a4}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "fit-webview-bridge"
-version = "0.2.1a1"
+version = "0.2.1a4"
 description = "Qt native WebView bridge with PySide6 bindings"
 requires-python = ">=3.11,<3.14"
 dependencies = ["PySide6==6.9.0", "shiboken6==6.9.0", "shiboken6-generator==6.9.0"]

{fit_webview_bridge-0.2.1a1 → fit_webview_bridge-0.2.1a4}/src/macos/WKWebViewWidget.mm

@@ -45,13 +45,13 @@ static NSURL* toNSURL(QUrl u);
 }
 @end
 
-// =======================
-// Navigation + Download delegate
-// =======================
+// ===== WKNavDelegate =====
 @interface WKNavDelegate : NSObject <WKNavigationDelegate, WKDownloadDelegate>
 @property(nonatomic, assign) WKWebViewWidget* owner;
-// stato download (nel delegate, non toccare i privati del widget)
-@property(nonatomic, strong) NSMapTable<WKDownload*, NSString*>* downloadPaths;
+// mappe per download
+@property(nonatomic, strong) NSMapTable<WKDownload*, NSString*>* downloadPaths;   // weak key -> strong value
+@property(nonatomic, strong) NSMapTable<NSProgress*, WKDownload*>* progressToDownload; // weak->weak
+@property(nonatomic, strong) NSHashTable<NSProgress*>* completedProgresses;      // weak set
 @end
 
 @implementation WKNavDelegate
@@ -59,6 +59,8 @@ static NSURL* toNSURL(QUrl u);
 - (instancetype)init {
     if ((self = [super init])) {
         _downloadPaths = [NSMapTable weakToStrongObjectsMapTable];
+        _progressToDownload = [NSMapTable weakToWeakObjectsMapTable];
+        _completedProgresses = [NSHashTable weakObjectsHashTable];
     }
     return self;
 }
@@ -118,8 +120,7 @@ decisionHandler:(void (^)(WKNavigationResponsePolicy))decisionHandler
     if (navigationResponse.canShowMIMEType) {
         decisionHandler(WKNavigationResponsePolicyAllow);
     } else {
-        // API moderna: "Download" (non BecomeDownload)
-        decisionHandler(WKNavigationResponsePolicyDownload);
+        decisionHandler(WKNavigationResponsePolicyDownload); // API moderna
     }
 }
 
@@ -130,14 +131,20 @@ navigationAction:(WKNavigationAction *)navigationAction
 didBecomeDownload:(WKDownload *)download
 {
     download.delegate = self;
-    if (self.owner) {
-        emit self.owner->downloadStarted(QString(), QString());
-    }
-    // Progress via NSProgress (best effort): alcuni siti non lo popolano
-    [download.progress addObserver:self
-                        forKeyPath:@"fractionCompleted"
-                           options:NSKeyValueObservingOptionNew
+    if (self.owner) emit self.owner->downloadStarted(QString(), QString());
+
+    // KVO su NSProgress (3 keyPath, con INITIAL)
+    [download.progress addObserver:self forKeyPath:@"fractionCompleted"
+                           options:(NSKeyValueObservingOptionNew | NSKeyValueObservingOptionInitial)
+                           context:NULL];
+    [download.progress addObserver:self forKeyPath:@"completedUnitCount"
+                           options:(NSKeyValueObservingOptionNew | NSKeyValueObservingOptionInitial)
+                           context:NULL];
+    [download.progress addObserver:self forKeyPath:@"totalUnitCount"
+                           options:(NSKeyValueObservingOptionNew | NSKeyValueObservingOptionInitial)
                            context:NULL];
+
+    [self.progressToDownload setObject:download forKey:download.progress];
 }
 
 - (void)webView:(WKWebView *)webView
@@ -148,15 +155,22 @@ didBecomeDownload:(WKDownload *)download
 
     NSString* suggested = navigationResponse.response.suggestedFilename ?: @"download";
     if (self.owner) {
-        QString destDir = self.owner->downloadDirectory();
-        QString destPath = destDir + "/" + QString::fromUtf8(suggested.UTF8String);
-        emit self.owner->downloadStarted(QString::fromUtf8(suggested.UTF8String),
-                                         destPath);
+        QString dir = self.owner->downloadDirectory();
+        QString path = dir + "/" + QString::fromUtf8(suggested.UTF8String);
+        emit self.owner->downloadStarted(QString::fromUtf8(suggested.UTF8String), path);
     }
-    [download.progress addObserver:self
-                        forKeyPath:@"fractionCompleted"
-                           options:NSKeyValueObservingOptionNew
+
+    [download.progress addObserver:self forKeyPath:@"fractionCompleted"
+                           options:(NSKeyValueObservingOptionNew | NSKeyValueObservingOptionInitial)
+                           context:NULL];
+    [download.progress addObserver:self forKeyPath:@"completedUnitCount"
+                           options:(NSKeyValueObservingOptionNew | NSKeyValueObservingOptionInitial)
                            context:NULL];
+    [download.progress addObserver:self forKeyPath:@"totalUnitCount"
+                           options:(NSKeyValueObservingOptionNew | NSKeyValueObservingOptionInitial)
+                           context:NULL];
+
+    [self.progressToDownload setObject:download forKey:download.progress];
 }
 
 #pragma mark - Scegli destinazione
@@ -188,9 +202,7 @@ completionHandler:(void (^)(NSURL * _Nullable destination))completionHandler
 
     QString qdir = self.owner->downloadDirectory();
     NSString* dir = [NSString stringWithUTF8String:qdir.toUtf8().constData()];
-    if (!dir.length) {
-        dir = [NSHomeDirectory() stringByAppendingPathComponent:@"Downloads"];
-    }
+    if (!dir.length) dir = [NSHomeDirectory() stringByAppendingPathComponent:@"Downloads"];
 
     [[NSFileManager defaultManager] createDirectoryAtPath:dir
                               withIntermediateDirectories:YES
@@ -212,41 +224,58 @@ completionHandler:(void (^)(NSURL * _Nullable destination))completionHandler
 - (void)observeValueForKeyPath:(NSString *)keyPath ofObject:(id)obj
                         change:(NSDictionary *)change context:(void *)ctx
 {
-    if (![keyPath isEqualToString:@"fractionCompleted"]) {
+    if (![obj isKindOfClass:[NSProgress class]] || !self.owner) {
         [super observeValueForKeyPath:keyPath ofObject:obj change:change context:ctx];
         return;
     }
-    if (!self.owner) return;
-
     NSProgress* prog = (NSProgress*)obj;
-    int64_t total = prog.totalUnitCount;     // può essere -1 (sconosciuto)
+
+    // ignora aggiornamenti dopo il completamento
+    if ([self.completedProgresses containsObject:prog]) return;
+
+    int64_t total = prog.totalUnitCount;     // -1 se sconosciuto
     int64_t done  = prog.completedUnitCount;
-    emit self.owner->downloadProgress(done, total >= 0 ? total : -1);
+
+    dispatch_async(dispatch_get_main_queue(), ^{
+        emit self.owner->downloadProgress(done, (total >= 0 ? total : -1));
+    });
 }
 
 - (void)downloadDidFinish:(WKDownload *)download {
     if (!self.owner) return;
 
-    @try { [download.progress removeObserver:self forKeyPath:@"fractionCompleted"]; } @catch (...) {}
+    @try {
+        [download.progress removeObserver:self forKeyPath:@"fractionCompleted"];
+        [download.progress removeObserver:self forKeyPath:@"completedUnitCount"];
+        [download.progress removeObserver:self forKeyPath:@"totalUnitCount"];
+    } @catch (...) {}
+
+    [self.completedProgresses addObject:download.progress];
 
     NSString* finalPath = [self.downloadPaths objectForKey:download];
-    if (finalPath) {
-        emit self.owner->downloadFinished(QString::fromUtf8(finalPath.UTF8String));
-        [self.downloadPaths removeObjectForKey:download];
-    } else {
-        emit self.owner->downloadFinished(QString());
-    }
+    emit self.owner->downloadFinished(finalPath ? QString::fromUtf8(finalPath.UTF8String) : QString());
+    if (finalPath) [self.downloadPaths removeObjectForKey:download];
+    [self.progressToDownload removeObjectForKey:download.progress];
 }
 
 - (void)download:(WKDownload *)download didFailWithError:(NSError *)error resumeData:(NSData *)resumeData {
     if (!self.owner) return;
 
-    @try { [download.progress removeObserver:self forKeyPath:@"fractionCompleted"]; } @catch (...) {}
+    @try {
+        [download.progress removeObserver:self forKeyPath:@"fractionCompleted"];
+        [download.progress removeObserver:self forKeyPath:@"completedUnitCount"];
+        [download.progress removeObserver:self forKeyPath:@"totalUnitCount"];
+    } @catch (...) {}
 
-   NSString* finalPath = [self.downloadPaths objectForKey:download];
-    QString qpath = finalPath ? QString::fromUtf8(finalPath.UTF8String) : QString();
-    emit self.owner->downloadFailed(qpath, QString::fromUtf8(error.localizedDescription.UTF8String));
+    [self.completedProgresses addObject:download.progress];
+
+    NSString* finalPath = [self.downloadPaths objectForKey:download];
+    emit self.owner->downloadFailed(
+        finalPath ? QString::fromUtf8(finalPath.UTF8String) : QString(),
+        QString::fromUtf8(error.localizedDescription.UTF8String)
+    );
     if (finalPath) [self.downloadPaths removeObjectForKey:download];
+    [self.progressToDownload removeObjectForKey:download.progress];
 }
 
 @end