Global marks; global marks are always recorded for the top frame only.

With global marks, we may later create a new tab when the mark is used.
When doing so, we should always be using the URL and scroll position of
the top frame within the tab.  For example, if a global mark is created
from within the Hangouts frame of Google's Inbox and the mark is later
used, then we should create a new tab for Inbox's URL and scroll
position, not for the Hangout's URL and scroll position.

1 files changed, 15 insertions, 3 deletions

diff --git a/background_scripts/marks.coffee b/background_scripts/marks.coffee
index 3a5af130..6e5f08ba 100644
--- a/background_scripts/marks.coffee
+++ b/background_scripts/marks.coffee
@@ -11,15 +11,27 @@ Marks =
   # tabId can be considered valid.
   create: (req, sender) ->
     chrome.storage.local.get "vimiumSecret", (items) =>
-      item = {}
-      item[@getLocationKey req.markName] =
+      markInfo =
         vimiumSecret: items.vimiumSecret
         markName: req.markName
         url: @getBaseUrl sender.tab.url
         tabId: sender.tab.id
         scrollX: req.scrollX
         scrollY: req.scrollY
-      chrome.storage.sync.set item
+
+      if markInfo.scrollX? and markInfo.scrollY?
+        @saveMark markInfo
+      else
+        # The front-end frame hasn't provided the scroll position (because it's not the top frame within its
+        # tab).  We need to ask the top frame what its scroll position is. (With the frame Id set to 0, below,
+        # the request will only be handled by the top frame within the tab.)
+        chrome.tabs.sendMessage sender.tab.id, name: "getScrollPosition", frameId: 0, (response) =>
+          @saveMark extend markInfo, scrollX: response.scrollX, scrollY: response.scrollY
+
+  saveMark: (markInfo) ->
+    item = {}
+    item[@getLocationKey markInfo.markName] = markInfo
+    chrome.storage.sync.set item
 
   # Goto a global mark.  We try to find the original tab.  If we can't find that, then we try to find another
   # tab with the original URL, and use that.  And if we can't find such an existing tab, then we create a new