Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(4488)

Unified Diff: Source/web/tests/WebPageNewSerializerTest.cpp

Issue 1177693002: Merge page serializers [6/12] (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Created 5 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: Source/web/tests/WebPageNewSerializerTest.cpp
diff --git a/Source/web/tests/WebPageNewSerializerTest.cpp b/Source/web/tests/WebPageNewSerializerTest.cpp
index 6f7a3a63fc2d5959108084d2a5b5bcdd3e8ca017..a8d7245c1a7712e88f1097a3ae61a7b19873fecd 100644
--- a/Source/web/tests/WebPageNewSerializerTest.cpp
+++ b/Source/web/tests/WebPageNewSerializerTest.cpp
@@ -205,39 +205,6 @@ private:
WebString m_svgMimeType;
};
-// Test that when serializing a page, all CSS resources are reported, including url()'s
-// and imports and links. Note that we don't test the resources contents, we only make sure
-// they are all reported with the right mime type and that they contain some data.
-TEST_F(WebPageNewSerializeTest, FAILS_CSSResources)
-{
- // Register the mocked frame and load it.
- WebURL topFrameURL = setUpCSSTestPage();
- loadURLInTopFrame(topFrameURL);
-
- WebVector<WebPageSerializer::Resource> resources;
- WebPageSerializer::serialize(webView(), &resources);
- ASSERT_FALSE(resources.isEmpty());
-
- // The first resource should be the main-frame.
- const WebPageSerializer::Resource& resource = resources[0];
- EXPECT_TRUE(resource.url == WebURL(toKURL(m_baseURL)));
- EXPECT_EQ(0, resource.mimeType.compare(WebCString("text/html")));
- EXPECT_FALSE(resource.data.isEmpty());
-
- EXPECT_EQ(12U, resources.size()); // There should be no duplicates.
- EXPECT_TRUE(resourceVectorContains(resources, m_baseURL + "link_styles.css", "text/css"));
- EXPECT_TRUE(resourceVectorContains(resources, m_baseURL + "import_styles.css", "text/css"));
- EXPECT_TRUE(resourceVectorContains(resources, m_baseURL + "import_style_from_link.css", "text/css"));
- EXPECT_TRUE(resourceVectorContains(resources, m_baseURL + "red_background.png", "image/png"));
- EXPECT_TRUE(resourceVectorContains(resources, m_baseURL + "orange_background.png", "image/png"));
- EXPECT_TRUE(resourceVectorContains(resources, m_baseURL + "yellow_background.png", "image/png"));
- EXPECT_TRUE(resourceVectorContains(resources, m_baseURL + "green_background.png", "image/png"));
- EXPECT_TRUE(resourceVectorContains(resources, m_baseURL + "blue_background.png", "image/png"));
- EXPECT_TRUE(resourceVectorContains(resources, m_baseURL + "purple_background.png", "image/png"));
- EXPECT_TRUE(resourceVectorContains(resources, m_baseURL + "ul-dot.png", "image/png"));
- EXPECT_TRUE(resourceVectorContains(resources, m_baseURL + "ol-dot.png", "image/png"));
-}
-
TEST_F(WebPageNewSerializeTest, FAILS_TestMHTMLEncoding)
{
// Load a page with some CSS and some images.
« no previous file with comments | « Source/web/tests/PageSerializerTest.cpp ('k') | Source/web/tests/data/pageserializer/css/css_test_page.html » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698