Proper find-mode history for incognito mode.

Approach:
   - We never save the find mode history from an incognito tab in the
     settings, instead they are saved in the tabMapInfo for each
     currently-active incognito tab.
   - When a new incognito tab (or page, after navigation) starts, we
     scan the tabMapInfo looking for the find-mode history of any active
     tab.

In this way, the history is shared between incognito tabs, but discarded
once the last incognito tab closes.

1 files changed, 12 insertions, 0 deletions

diff --git a/background_scripts/main.coffee b/background_scripts/main.coffee
index 54e0491b..b4eac4f1 100644
--- a/background_scripts/main.coffee
+++ b/background_scripts/main.coffee
@@ -193,6 +193,15 @@ handleSettings = (args, port) ->
     # In the case of findModeRawQueryList, this allows each tab to accurately track the find-mode history.
     if args.key in [ "findModeRawQueryList" ]
       sendRequestToAllTabs extend args, name: "updateSettings"
+    # We also track the incognito find-mode history setting in the tabInfoMap for all incognito tabs.
+    if args.incognito and args.key == "findModeRawQueryList"
+      for own _, map of tabInfoMap
+        map.findModeRawQueryList = args.value if map.incognito
+
+# Search through the active tab map for an up-to-date find-mode history for an incognito-mode tab.
+getIncognitoRawQueryList = ->
+  for own id, map of tabInfoMap
+    return map.findModeRawQueryList if map.findModeRawQueryList
 
 refreshCompleter = (request) -> completers[request.name].refresh()
 
@@ -337,6 +346,8 @@ updateOpenTabs = (tab) ->
     scrollX: null
     scrollY: null
     deletor: null
+    incognito: tab.incognito
+    findModeRawQueryList: null
   # Frames are recreated on refresh
   delete frameIdsForTab[tab.id]
 
@@ -661,6 +672,7 @@ sendRequestHandlers =
   createMark: Marks.create.bind(Marks)
   gotoMark: Marks.goto.bind(Marks)
   setBadge: setBadge
+  getIncognitoRawQueryList: getIncognitoRawQueryList
 
 # Convenience function for development use.
 window.runTests = -> open(chrome.runtime.getURL('tests/dom_tests/dom_tests.html'))