[WIP] Fix history.replaceState() not propagating back favicons

components/history/core/browser/history_types.h

Index: components/history/core/browser/history_types.h
diff --git a/components/history/core/browser/history_types.h b/components/history/core/browser/history_types.h
index 43468238e323c911a71ba4e25507b96a84a2799a..a057031ae6ffc591a18e5315dec8d69f06c32396 100644
--- a/components/history/core/browser/history_types.h
+++ b/components/history/core/browser/history_types.h
@@ -18,6 +18,7 @@
 #include "base/containers/stack_container.h"
 #include "base/macros.h"
 #include "base/memory/ref_counted_memory.h"
+#include "base/optional.h"
 #include "base/strings/string16.h"
 #include "base/time/time.h"
 #include "components/favicon_base/favicon_types.h"
@@ -365,7 +366,7 @@ struct HistoryAddPageArgs {
                      const RedirectList& redirects,
                      ui::PageTransition transition,
                      VisitSource source,
-                     bool did_replace_entry,
+                     base::Optional<GURL> replaced_entry_url,
                      bool consider_for_ntp_most_visited);
   HistoryAddPageArgs(const HistoryAddPageArgs& other);
   ~HistoryAddPageArgs();
@@ -378,7 +379,7 @@ struct HistoryAddPageArgs {
   RedirectList redirects;
   ui::PageTransition transition;
   VisitSource visit_source;
-  bool did_replace_entry;
+  base::Optional<GURL> replaced_entry_url;
   // Specifies whether a page visit should contribute to the Most Visited tiles
   // in the New Tab Page. Note that setting this to true (most common case)
   // doesn't guarantee it's relevant for Most Visited, since other requirements