From a58732f78802a114908ed2ded03d72bbec7dc1fb Mon Sep 17 00:00:00 2001
From: lifegpc <g1710431395@gmail.com>
Date: Thu, 14 Sep 2023 21:00:33 +0800
Subject: [PATCH] Fix gallery's title can not display correct on chrome/firefox

---
 lib/galleries.dart |  3 ++-
 lib/gallery.dart   | 58 ++++++++++++++++++++++++++++++++++------------
 lib/globals.dart   |  4 +++-
 lib/main.dart      | 10 +++++---
 4 files changed, 55 insertions(+), 20 deletions(-)

diff --git a/lib/galleries.dart b/lib/galleries.dart
index b9fb2a2..f845b13 100644
--- a/lib/galleries.dart
+++ b/lib/galleries.dart
@@ -123,7 +123,8 @@ class _GalleriesPage extends State<GalleriesPage> with ThemeModeWidget {
             return ListTile(
               title: Text(item.preferredTitle),
               onTap: () {
-                context.push("/gallery/${item.gid}");
+                context.push("/gallery/${item.gid}",
+                    extra: GalleryPageExtra(title: item.preferredTitle));
               },
             );
           }),
diff --git a/lib/gallery.dart b/lib/gallery.dart
index a2bb011..8ad35b7 100644
--- a/lib/gallery.dart
+++ b/lib/gallery.dart
@@ -1,3 +1,5 @@
+import 'package:dio/dio.dart';
+import 'package:flutter/foundation.dart';
 import 'package:flutter/material.dart';
 import 'package:flutter_gen/gen_l10n/app_localizations.dart';
 import 'package:go_router/go_router.dart';
@@ -9,12 +11,18 @@ import 'globals.dart';
 
 final _log = Logger("GalleryPage");
 
+class GalleryPageExtra {
+  const GalleryPageExtra({this.title});
+  final String? title;
+}
+
 class GalleryPage extends StatefulWidget {
-  const GalleryPage(int gid, {Key? key})
+  const GalleryPage(int gid, {Key? key, this.title})
       : _gid = gid,
         super(key: key);
 
   final int _gid;
+  final String? title;
   static const String routeName = '/gallery/:gid';
 
   @override
@@ -27,26 +35,33 @@ class _GalleryPage extends State<GalleryPage> with ThemeModeWidget {
   GalleryData? _data;
   EhFiles? _files;
   Object? _error;
+  CancelToken? _cancel;
   bool _isLoading = false;
 
   Future<void> _fetchData() async {
     try {
+      _cancel = CancelToken();
       _isLoading = true;
       final data = (await api.getGallery(_gid)).unwrap();
       _data = data;
-      final fileData =
-          (await api.getFiles(data.pages.map((e) => e.token).toList()))
-              .unwrap();
-      setState(() {
-        _files = fileData;
-        _isLoading = false;
-      });
+      final fileData = (await api.getFiles(
+              data.pages.map((e) => e.token).toList(),
+              cancel: _cancel))
+          .unwrap();
+      if (!_cancel!.isCancelled) {
+        setState(() {
+          _files = fileData;
+          _isLoading = false;
+        });
+      }
     } catch (e) {
-      _log.severe("Failed to load gallery $_gid:", e);
-      setState(() {
-        _error = e;
-        _isLoading = false;
-      });
+      if (!_cancel!.isCancelled) {
+        _log.severe("Failed to load gallery $_gid:", e);
+        setState(() {
+          _error = e;
+          _isLoading = false;
+        });
+      }
     }
   }
 
@@ -69,8 +84,15 @@ class _GalleryPage extends State<GalleryPage> with ThemeModeWidget {
         : _data != null
             ? _data!.meta.preferredTitle
             : i18n.gallery;
-    setCurrentTitle(title, Theme.of(context).primaryColor.value,
-        includePrefix: false);
+    if (!kIsWeb || (_data != null && kIsWeb)) {
+      setCurrentTitle(title, Theme.of(context).primaryColor.value,
+          includePrefix: false);
+    } else if (kIsWeb && widget.title != null) {
+      // 设置预加载标题
+      // Chrome 和 Firefox 必须尽快设置标题以确保在历史记录菜单显示正确的标题
+      setCurrentTitle(widget.title!, Theme.of(context).primaryColor.value,
+          includePrefix: false);
+    }
     return Scaffold(
         appBar: _data == null
             ? AppBar(
@@ -95,4 +117,10 @@ class _GalleryPage extends State<GalleryPage> with ThemeModeWidget {
                     child: Text("Error: $_error"),
                   ));
   }
+
+  @override
+  void dispose() {
+    _cancel?.cancel();
+    super.dispose();
+  }
 }
diff --git a/lib/globals.dart b/lib/globals.dart
index da9157a..214079e 100644
--- a/lib/globals.dart
+++ b/lib/globals.dart
@@ -25,6 +25,7 @@ import 'platform/clipboard.dart';
 import 'platform/path.dart';
 import 'tags.dart';
 import 'utils.dart';
+export 'gallery.dart' show GalleryPageExtra;
 
 final dio = Dio()
   ..options.validateStatus = (int? _) {
@@ -307,7 +308,8 @@ void setCurrentTitle(String title, int primaryColor,
     });
   } else {
     SystemChrome.setApplicationSwitcherDescription(
-            ApplicationSwitcherDescription(label: title, primaryColor: primaryColor))
+            ApplicationSwitcherDescription(
+                label: title, primaryColor: primaryColor))
         .then((_) {
       _currentTitle = title;
       if (isPrefix) _prefix = title;
diff --git a/lib/main.dart b/lib/main.dart
index 89137e1..75a0387 100644
--- a/lib/main.dart
+++ b/lib/main.dart
@@ -59,9 +59,13 @@ final _router = GoRouter(
         }),
     GoRoute(
         path: GalleryPage.routeName,
-        builder: (context, state) => GalleryPage(
-              int.parse(state.pathParameters["gid"]!),
-            ),
+        builder: (context, state) {
+          final extra = state.extra as GalleryPageExtra?;
+          return GalleryPage(
+            int.parse(state.pathParameters["gid"]!),
+            title: extra?.title,
+          );
+        },
         redirect: (context, state) {
           try {
             int.parse(state.pathParameters["gid"]!);