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

Unified Diff: Source/core/html/DOMFormDataTest.cpp

Issue 1312333005: Remove FormDataList::Entry. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 5 years, 3 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
« no previous file with comments | « Source/core/html/DOMFormData.cpp ('k') | Source/core/html/FormDataList.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/core/html/DOMFormDataTest.cpp
diff --git a/Source/core/html/DOMFormDataTest.cpp b/Source/core/html/DOMFormDataTest.cpp
index 7c361aebdccafd8a30e108c2f683b5214aec0751..09d2107e65a606015d9b9b7e6740b309a76e2947 100644
--- a/Source/core/html/DOMFormDataTest.cpp
+++ b/Source/core/html/DOMFormDataTest.cpp
@@ -20,8 +20,9 @@ TEST(DOMFormDataTest, opacityGet)
EXPECT_TRUE(result.isUSVString());
EXPECT_EQ("value1", result.getAsUSVString());
- FormDataList::Entry entry = fd->getEntry("name1");
- EXPECT_EQ("value1", entry.string());
+ const FormDataList::Item& entry = fd->items()[0];
+ EXPECT_STREQ("name1", entry.key().data());
+ EXPECT_STREQ("value1", entry.data().data());
fd->makeOpaque();
@@ -31,8 +32,9 @@ TEST(DOMFormDataTest, opacityGet)
EXPECT_TRUE(opaqueResult.isNull());
// Internal interface should be uneffected.
- FormDataList::Entry opaqueEntry = fd->getEntry("name1");
- EXPECT_EQ("value1", opaqueEntry.string());
+ const FormDataList::Item& entry2 = fd->items()[0];
+ EXPECT_STREQ("name1", entry2.key().data());
+ EXPECT_STREQ("value1", entry2.data().data());
}
TEST(DOMFormDataTest, opacityGetAll)
« no previous file with comments | « Source/core/html/DOMFormData.cpp ('k') | Source/core/html/FormDataList.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698