Remove popup overlay from MHTML when requested

components/offline_pages/core/client_policy_controller.h

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef COMPONENTS_OFFLINE_PAGES_CORE_CLIENT_POLICY_CONTROLLER_H_
 #define COMPONENTS_OFFLINE_PAGES_CORE_CLIENT_POLICY_CONTROLLER_H_
 
 #include <stdint.h>
 
 #include <map>
@@ skipping 38 matching lines @@
   // currently only available on NTP.
   bool IsShownAsRecentlyVisitedSite(const std::string& name_space) const;
   const std::vector<std::string>& GetNamespacesShownAsRecentlyVisitedSite()
       const;
 
   // Returns whether pages for |name_space| should never be shown outside the
   // tab they were generated in.
   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);
 
  private:
   // The map from name_space to a client policy. Will be generated
   // as pre-defined values for now.
   std::map<std::string, OfflinePageClientPolicy> policies_;
 
   // Memoizing results.
   mutable std::unique_ptr<std::vector<std::string>> download_namespace_cache_;
   mutable std::unique_ptr<std::vector<std::string>> recent_tab_namespace_cache_;
   mutable std::unique_ptr<std::vector<std::string>> show_in_original_tab_cache_;
 
   DISALLOW_COPY_AND_ASSIGN(ClientPolicyController);
 };
 
 }  // namespace offline_pages
 
 #endif  // COMPONENTS_OFFLINE_PAGES_CORE_CLIENT_POLICY_CONTROLLER_H_