Propagate inert state to OOPIFs when a modal dialog is active

third_party/WebKit/Source/core/frame/LocalFrame.h

 /*
  * Copyright (C) 1998, 1999 Torben Weis <weis@kde.org>
  *                     1999-2001 Lars Knoll <knoll@kde.org>
  *                     1999-2001 Antti Koivisto <koivisto@kde.org>
  *                     2000-2001 Simon Hausmann <hausmann@kde.org>
  *                     2000-2001 Dirk Mueller <mueller@kde.org>
  *                     2000 Stefan Schimanski <1Stein@gmx.de>
  * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009, 2010 Apple Inc. All rights
  * reserved.
  * Copyright (C) 2008 Nokia Corporation and/or its subsidiary(-ies)
@@ skipping 107 matching lines @@
                 UserGestureStatus) override;
   void Navigate(const FrameLoadRequest&) override;
   void Reload(FrameLoadType, ClientRedirectPolicy) override;
   void Detach(FrameDetachType) override;
   bool ShouldClose() override;
   SecurityContext* GetSecurityContext() const override;
   void PrintNavigationErrorMessage(const Frame&, const char* reason) override;
   void PrintNavigationWarning(const String&) override;
   bool PrepareForCommit() override;
   void DidChangeVisibilityState() override;
+  // This sets the is_inert_ flag and also recurses through this frame's
+  // subtree, updating the inert bit on all descendant frames.
+  void SetIsInert(bool) override;
 
   void DetachChildren();
   void DocumentAttached();
 
   Frame* FindFrameForNavigation(const AtomicString& name,
                                 LocalFrame& active_frame);
 
   // Note: these two functions are not virtual but intentionally shadow the
   // corresponding method in the Frame base class to return the
   // LocalFrame-specific subclass.
@@ skipping 107 matching lines @@
 
   PerformanceMonitor* GetPerformanceMonitor() { return performance_monitor_; }
 
   // Convenience function to allow loading image placeholders for the request if
   // either the flag in Settings() for using image placeholders is set, or if
   // the embedder decides that Client Lo-Fi should be used for this request.
   void MaybeAllowImagePlaceholder(FetchParameters&) const;
 
   std::unique_ptr<WebURLLoader> CreateURLLoader();
 
+  bool IsInert() const { return is_inert_; }

[alexmos, 2017/06/16 02:17:58]

Is this used anywhere?  I might've missed it, but I only see IsInert called on
HTMLFrameOwnerElements.

[kenrb, 2017/06/19 19:26:23]

Yes, in Node::IsInert().

[alexmos, 2017/06/20 18:46:10]

Acknowledged.

+
   using FrameInitCallback = void (*)(LocalFrame*);
   // Allows for the registration of a callback that is invoked whenever a new
   // LocalFrame is initialized. Callbacks are executed in the order that they
   // were added using registerInitializationCallback, and there are no checks
   // for adding a callback multiple times.
   static void RegisterInitializationCallback(FrameInitCallback);
 
   // If the frame hosts a PluginDocument, this method returns the
   // WebPluginContainerBase that hosts the plugin. If the provided node is a
   // plugin, then it returns its WebPluginContainerBase. Otherwise, uses the
@@ skipping 20 matching lines @@
   // Intentionally private to prevent redundant checks when the type is
   // already LocalFrame.
   bool IsLocalFrame() const override { return true; }
   bool IsRemoteFrame() const override { return false; }
 
   void EnableNavigation() { --navigation_disable_count_; }
   void DisableNavigation() { ++navigation_disable_count_; }
 
   bool CanNavigateWithoutFramebusting(const Frame&, String& error_reason);
 
+  void PropagateInertToChildFrames();
+
   std::unique_ptr<WebFrameScheduler> frame_scheduler_;
 
   mutable FrameLoader loader_;
   Member<NavigationScheduler> navigation_scheduler_;
 
   // Cleared by LocalFrame::detach(), so as to keep the observable lifespan
   // of LocalFrame::view().
   Member<LocalFrameView> view_;
   // Usually 0. Non-null if this is the top frame of PagePopup.
   Member<Element> page_popup_owner_;
 
   const Member<ScriptController> script_controller_;
   const Member<Editor> editor_;
   const Member<SpellChecker> spell_checker_;
   const Member<FrameSelection> selection_;
   const Member<EventHandler> event_handler_;
   const Member<FrameConsole> console_;
   const Member<InputMethodController> input_method_controller_;
 
   int navigation_disable_count_;
 
   float page_zoom_factor_;
   float text_zoom_factor_;
 
   bool in_view_source_mode_;
 
+  bool is_inert_;
+
   Member<CoreProbeSink> probe_sink_;
   Member<PerformanceMonitor> performance_monitor_;
 
   InterfaceProvider* const interface_provider_;
   InterfaceRegistry* const interface_registry_;
 
   IntRect remote_viewport_intersection_;
   Member<FrameResourceCoordinator> frame_resource_coordinator_;
 };
 
@@ skipping 89 matching lines @@
   explicit ScopedFrameBlamer(LocalFrame*);
   ~ScopedFrameBlamer();
 
  private:
   Member<LocalFrame> frame_;
 };
 
 }  // namespace blink
 
 #endif  // LocalFrame_h