try parse Uri from request if parsed object not provided

src/app/browser/window/tab/item/page.rs

@@ -919,14 +919,22 @@ fn is_external_uri(subject: &Uri, base: &Uri) -> bool {
     subject.host() != base.host()
 }
 
-/// Make new history record for given `navigation` object
+/// Make new history record in related components
-/// * applies on shared conditions match only
+/// * optional [Uri](https://docs.gtk.org/glib/struct.Uri.html) reference wanted only for performance reasons, to not parse it twice
 fn snap_history(profile: &Profile, navigation: &Navigation, uri: Option<&Uri>) {
     let request = navigation.request.widget.entry.text();
 
     // Add new record into the global memory index (used in global menu)
-    if let Some(uri) = uri {
+    // * if the `Uri` is `None`, try parse it from `request`
-        profile.history.memory.request.set(uri);
+    match uri {
+        Some(uri) => profile.history.memory.request.set(uri.clone()),
+        None => {
+            // this case especially useful for some routes that contain redirects
+            // maybe some parental optimization wanted @TODO
+            if let Some(uri) = navigation.request.uri() {
+                profile.history.memory.request.set(uri);
+            }
+        }
     }
 
     // Add new record into the page navigation history

src/profile/history/memory/request.rs

@@ -32,12 +32,12 @@ impl Request {
 
     /// Add new record with `request` as key and `unix_timestamp` as value
     /// * replace with new value if `request` already exists
-    pub fn set(&self, uri: &Uri) {
+    pub fn set(&self, uri: Uri) {
         self.index.borrow_mut().insert(
             uri.to_str(),
             Value {
                 unix_timestamp: DateTime::now_local().unwrap().to_unix(),
-                uri: uri.clone(),
+                uri,
             },
         );
     }