Replace FIXMEs with correct comments.
As discussed in https://crbug.com/753130, there is no additional work
needed for OOPIFs to be notified of localStorage/sessionStorage updates.
This CL removes the FIXMEs and replaces them with a more accurate comment.
Bug: 753130
Change-Id: I7b9a0c3869168c716281837dae2572adb01a09c5
Reviewed-on: https://chromium-review.googlesource.com/1080286
Reviewed-by: Marijn Kruisselbrink <mek@chromium.org>
Reviewed-by: Victor Costan <pwnall@chromium.org>
Commit-Queue: Victor Costan <pwnall@chromium.org>
Cr-Commit-Position: refs/heads/master@{#563441}diff --git a/third_party/blink/renderer/modules/storage/storage_area.cc b/third_party/blink/renderer/modules/storage/storage_area.cc
index c856132..b562e26 100644
--- a/third_party/blink/renderer/modules/storage/storage_area.cc
+++ b/third_party/blink/renderer/modules/storage/storage_area.cc
@@ -167,8 +167,8 @@
for (Page* page : Page::OrdinaryPages()) {
for (Frame* frame = page->MainFrame(); frame;
frame = frame->Tree().TraverseNext()) {
- // FIXME: We do not yet have a way to dispatch events to out-of-process
- // frames.
+ // Remote frames are cross-origin and do not need to be notified of
+ // events.
if (!frame->IsLocalFrame())
continue;
LocalFrame* local_frame = ToLocalFrame(frame);
@@ -219,8 +219,7 @@
for (Frame* frame = page->MainFrame(); frame;
frame = frame->Tree().TraverseNext()) {
- // FIXME: We do not yet have a way to dispatch events to out-of-process
- // frames.
+ // Remote frames are cross-origin and do not need to be notified of events.
if (!frame->IsLocalFrame())
continue;
LocalFrame* local_frame = ToLocalFrame(frame);
