Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: test/Notification.cpp

Issue 5708936187478016: Issue 1906 - Fix broken Android build (Closed)
Patch Set: Fixed some formatting issues, shared_ptr.reset() Created Jan. 29, 2015, 5:07 p.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « libadblockplus.gyp ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: test/Notification.cpp
diff --git a/test/Notification.cpp b/test/Notification.cpp
index 06c8656ee7a3ddf1f4ed0615ff380d853ea5980d..6faf2cc39d344ddd6a877cb8ba1d1a342a417525 100644
--- a/test/Notification.cpp
+++ b/test/Notification.cpp
@@ -37,7 +37,7 @@ namespace
jsEngine->SetFileSystem(FileSystemPtr(new LazyFileSystem()));
jsEngine->SetWebRequest(WebRequestPtr(new LazyWebRequest()));
jsEngine->SetLogSystem(LogSystemPtr(new DefaultLogSystem()));
- filterEngine = std::tr1::make_shared<FilterEngine>(jsEngine);
+ filterEngine.reset(new FilterEngine(jsEngine));
}
void AddNotification(const std::string& notification)
@@ -80,7 +80,8 @@ namespace
void SetUp() override
{
BaseJsTest::SetUp();
- jsEngine->SetFileSystem(std::tr1::make_shared<LazyFileSystem>());
+ jsEngine->SetFileSystem(
+ std::tr1::shared_ptr<LazyFileSystem>(new LazyFileSystem()));
const char* responseJsonText = "{"
"\"notifications\": [{"
"\"id\": \"some id\","
@@ -91,9 +92,10 @@ namespace
"\"title\": \"Title\""
"}]"
"}";
- jsEngine->SetWebRequest(std::tr1::make_shared<MockWebRequest>(responseJsonText));
+ jsEngine->SetWebRequest(std::tr1::shared_ptr<MockWebRequest>(
+ new MockWebRequest(responseJsonText)));
jsEngine->SetLogSystem(LogSystemPtr(new DefaultLogSystem()));
- filterEngine = std::tr1::make_shared<FilterEngine>(jsEngine);
+ filterEngine.reset(new FilterEngine(jsEngine));
}
};
#endif
@@ -166,4 +168,4 @@ TEST_F(NotificationTest, MarkAsShown)
ASSERT_TRUE(notification);
notification->MarkAsShown();
EXPECT_EQ(NULL, filterEngine->GetNextNotificationToShow().get());
-}
+}
« no previous file with comments | « libadblockplus.gyp ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld