OOPIFs: Deduplicating MHTML parts across frames.

chrome/browser/download/save_page_browsertest.cc

 // Copyright (c) 2012 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.
 
 #include <string>
 #include <vector>
 
 #include "base/bind.h"
 #include "base/bind_helpers.h"
 #include "base/command_line.h"
@@ skipping 886 matching lines @@
   // to twice - from iframes.htm and from b.htm).
   int count = 0;
   size_t pos = 0;
   for (;;) {
     pos = mhtml.find("Content-Type: image/png", pos);
     if (pos == std::string::npos)
       break;
     count++;
     pos++;
   }
-  // TODO(lukasza): Need to dedupe savable resources (i.e. 1.png) across frames.
-  // This will be fixed by crrev.com/1417323006.
-  // EXPECT_EQ(1, count)
-  //     << "Verify number of image/png parts in the mhtml output";
+  EXPECT_EQ(1, count) << "Verify number of image/png parts in the mhtml output";
 }
 
 // Test suite that verifies that the frame tree "looks" the same before
 // and after a save-page-as.
 class SavePageMultiFrameBrowserTest
     : public SavePageSitePerProcessBrowserTest,
       public ::testing::WithParamInterface<content::SavePageType> {
  protected:
   void TestMultiFramePage(content::SavePageType save_page_type,
                           const GURL& url,
@@ skipping 267 matching lines @@
   TestMultiFramePage(save_page_type, url, 6, expected_substrings);
 }
 
 INSTANTIATE_TEST_CASE_P(
     SaveType,
     SavePageMultiFrameBrowserTest,
     ::testing::Values(content::SAVE_PAGE_TYPE_AS_COMPLETE_HTML,
                       content::SAVE_PAGE_TYPE_AS_MHTML));
 
 }  // namespace