PlzNavigate: make MHTML iframe load working.

content/common/navigation_params_unittest.cc

 // 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.
 
 #include "content/common/navigation_params.h"
 
 #include "content/public/common/browser_side_navigation_policy.h"
 #include "testing/gtest/include/gtest/gtest.h"
 #include "url/gurl.h"
 
 namespace content {
 
 TEST(NavigationParamsTest, ShouldMakeNetworkRequestForURL) {
   if (!IsBrowserSideNavigationEnabled())
     return;
 
   EXPECT_TRUE(ShouldMakeNetworkRequestForURL(GURL("http://foo/bar.html")));
   EXPECT_TRUE(ShouldMakeNetworkRequestForURL(GURL("https://foo/bar.html")));
+  // Even though a "data" url doesn't generate actual network requests. It is

[nasko, 2017/05/04 15:48:18]

nit: "data:", since it is a scheme. "requests, it is" (one sentence).
Let's also include some context if we are adding a comment giving an example why
- some data: URLs result in downloads, which are handled in the network stack.

[arthursonzogni, 2017/05/05 09:07:46]

I will list some reasons why some data-urls require using the network stack.

+  // handled by the network stack and so must return true for this check.
   EXPECT_TRUE(ShouldMakeNetworkRequestForURL(GURL("data://foo")));
 
   EXPECT_FALSE(ShouldMakeNetworkRequestForURL(GURL("about:blank")));
   EXPECT_FALSE(ShouldMakeNetworkRequestForURL(GURL("about:srcdoc")));
   EXPECT_FALSE(ShouldMakeNetworkRequestForURL(GURL("javascript://foo.js")));
   EXPECT_FALSE(ShouldMakeNetworkRequestForURL(GURL("cid:foo@bar")));
   EXPECT_FALSE(ShouldMakeNetworkRequestForURL(GURL()));
 }
 
 }  // namespace content