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

Unified Diff: chrome/browser/extensions/activity_log/fullstream_ui_policy_unittest.cc

Issue 23874008: Added memory limit for chrome.activityLogPrivate returned results (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 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
Index: chrome/browser/extensions/activity_log/fullstream_ui_policy_unittest.cc
diff --git a/chrome/browser/extensions/activity_log/fullstream_ui_policy_unittest.cc b/chrome/browser/extensions/activity_log/fullstream_ui_policy_unittest.cc
index 5dc6445b6221576ea2703fe939e84fa65498c0b1..56e5146b6e37ea060a9556edf8c631bbc5da530f 100644
--- a/chrome/browser/extensions/activity_log/fullstream_ui_policy_unittest.cc
+++ b/chrome/browser/extensions/activity_log/fullstream_ui_policy_unittest.cc
@@ -6,6 +6,7 @@
#include "base/command_line.h"
#include "base/memory/scoped_ptr.h"
#include "base/run_loop.h"
+#include "base/strings/stringprintf.h"
#include "base/synchronization/waitable_event.h"
#include "base/test/simple_test_clock.h"
#include "chrome/browser/extensions/activity_log/activity_log.h"
@@ -156,6 +157,11 @@ class FullStreamUIPolicyTest : public testing::Test {
ASSERT_EQ(2, static_cast<int>(i->size()));
}
+ static void RetrieveActions_FetchFilteredActions300(
+ scoped_ptr<std::vector<scoped_refptr<Action> > > i) {
+ ASSERT_EQ(300, static_cast<int>(i->size()));
+ }
+
static void Arguments_Present(scoped_ptr<Action::ActionVector> i) {
scoped_refptr<Action> last = i->front();
std::string args =
@@ -642,4 +648,29 @@ TEST_F(FullStreamUIPolicyTest, RemoveSpecificURLs) {
policy->Close();
}
+TEST_F(FullStreamUIPolicyTest, CapReturns) {
+ FullStreamUIPolicy* policy = new FullStreamUIPolicy(profile_.get());
+
+ for (int i = 0; i < 305; i++) {
+ scoped_refptr<Action> action =
+ new Action("punky",
+ base::Time::Now(),
+ Action::ACTION_API_CALL,
+ base::StringPrintf("apicall_%d", i));
+ policy->ProcessAction(action);
+ }
+ policy->Flush();
+
+ CheckReadFilteredData(
+ policy,
+ "punky",
+ Action::ACTION_ANY,
+ "",
+ "",
+ "",
+ base::Bind(
+ &FullStreamUIPolicyTest::RetrieveActions_FetchFilteredActions300));
+ policy->Close();
+}
+
} // namespace extensions

Powered by Google App Engine
This is Rietveld 408576698