Remove popup overlay from MHTML when requested

components/offline_pages/core/client_policy_controller.h

Index: components/offline_pages/core/client_policy_controller.h
diff --git a/components/offline_pages/core/client_policy_controller.h b/components/offline_pages/core/client_policy_controller.h
index 5e5cd21467a6901c41f4bdebd7e6949207cadeb6..32a4484b55136147e098f8e3a0d86bb6b5e2726c 100644
--- a/components/offline_pages/core/client_policy_controller.h
+++ b/components/offline_pages/core/client_policy_controller.h
@@ -56,6 +56,10 @@ class ClientPolicyController {
   bool IsRestrictedToOriginalTab(const std::string& name_space) const;
   const std::vector<std::string>& GetNamespacesRestrictedToOriginalTab() const;
 
+  // Returns whether to remove popup overlay that obstructs viewing normal
+  // content.
+  bool ShouldRemovePopupOverlay(const std::string& name_space) const;

[fgorski, 2017/01/19 17:59:08]

I've just discussed this with Dmitry. 2 comments:
1. It should only be affecting background work, not snapshotting the live page
(which should be reflected by name of the method and documentation)
2. You have more than just async loading to worry about:
* "download" can be handled in the background
* "async_loading"
* "ntp_suggestions"
(This second part applies to initializing policies in cc file)

[jianli, 2017/01/19 23:56:10]

Changed to pass the background save info in SavePageParams.

+
   void AddPolicyForTest(const std::string& name_space,
                         const OfflinePageClientPolicyBuilder& builder);