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

Unified Diff: third_party/android_crazy_linker/src/src/crazy_linker_system_unittest.cpp

Issue 1100763002: Inject CanAddURLToHistory into TopSitesImpl (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@prefs
Patch Set: Fix error introduced during rebase Created 5 years, 8 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: third_party/android_crazy_linker/src/src/crazy_linker_system_unittest.cpp
diff --git a/third_party/android_crazy_linker/src/src/crazy_linker_system_unittest.cpp b/third_party/android_crazy_linker/src/src/crazy_linker_system_unittest.cpp
index 93ee1af409f17197d3bbee839609422112b80901..8bf082f64e6e9670fa0270df922f5989486e86d9 100644
--- a/third_party/android_crazy_linker/src/src/crazy_linker_system_unittest.cpp
+++ b/third_party/android_crazy_linker/src/src/crazy_linker_system_unittest.cpp
@@ -39,33 +39,4 @@ TEST(System, PathExistsWithBadPath) {
EXPECT_FALSE(PathExists("/tmp/foo"));
}
-TEST(System, IsSystemLibrary) {
- SystemMock sys;
- static const struct {
- const char* name;
- bool success;
- } kData[] = {
- {"libEGL.so", true},
- {"libGLESv1_CM.so", true},
- {"libGLESv1.so", false},
- {"libGLESv2.so", true},
- {"libOpenMAXAL.so", true},
- {"libOpenSLES.so", true},
- {"libandroid.so", true},
- {"libc.so", true},
- {"libdl.so", true},
- {"libjnigraphics.so", true},
- {"libm.so", true},
- {"libstdc++.so", true},
- {"libstlport.so", false},
- {"libz.so", true},
- {"/system/lib/libc.so", true},
- {"/system/libc.so/libfoo.so", false},
- };
- for (size_t n = 0; n < ARRAY_LEN(kData); ++n) {
- TEST_TEXT << "Checking " << kData[n].name;
- EXPECT_EQ(kData[n].success, IsSystemLibrary(kData[n].name));
- }
-}
-
} // namespace crazy

Powered by Google App Engine
This is Rietveld 408576698