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

Unified Diff: chrome/common/extensions/extension_localization_peer_unittest.cc

Issue 6621006: Take out the is_content_filtered bool that gets passed around betwen renderer... (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: '' Created 9 years, 10 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 | « chrome/common/extensions/extension_localization_peer.cc ('k') | chrome/common/render_messages_params.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/common/extensions/extension_localization_peer_unittest.cc
===================================================================
--- chrome/common/extensions/extension_localization_peer_unittest.cc (revision 76828)
+++ chrome/common/extensions/extension_localization_peer_unittest.cc (working copy)
@@ -63,9 +63,8 @@
const webkit_glue::ResourceResponseInfo& info,
bool* has_new_first_party_for_cookies,
GURL* new_first_party_for_cookies));
- MOCK_METHOD2(OnReceivedResponse, void(
- const webkit_glue::ResourceResponseInfo& info,
- bool content_filtered));
+ MOCK_METHOD1(OnReceivedResponse, void(
+ const webkit_glue::ResourceResponseInfo& info));
MOCK_METHOD1(OnDownloadedData, void(int len));
MOCK_METHOD2(OnReceivedData, void(const char* data, int len));
MOCK_METHOD3(OnCompletedRequest, void(
@@ -139,7 +138,7 @@
// It will self-delete once it exits OnCompletedRequest.
ExtensionLocalizationPeer* filter_peer = filter_peer_.release();
- EXPECT_CALL(*original_peer_, OnReceivedResponse(_, true));
+ EXPECT_CALL(*original_peer_, OnReceivedResponse(_));
EXPECT_CALL(*original_peer_, OnCompletedRequest(
IsURLRequestEqual(net::URLRequestStatus::CANCELED), "", base::Time()));
@@ -155,7 +154,7 @@
EXPECT_CALL(*original_peer_, OnReceivedData(_, _)).Times(0);
EXPECT_CALL(*sender_, Send(_)).Times(0);
- EXPECT_CALL(*original_peer_, OnReceivedResponse(_, true));
+ EXPECT_CALL(*original_peer_, OnReceivedResponse(_));
EXPECT_CALL(*original_peer_, OnCompletedRequest(
IsURLRequestEqual(net::URLRequestStatus::SUCCESS), "", base::Time()));
@@ -176,7 +175,7 @@
EXPECT_CALL(*original_peer_,
OnReceivedData(StrEq(data.data()), data.length())).Times(2);
- EXPECT_CALL(*original_peer_, OnReceivedResponse(_, true)).Times(2);
+ EXPECT_CALL(*original_peer_, OnReceivedResponse(_)).Times(2);
EXPECT_CALL(*original_peer_, OnCompletedRequest(
IsURLRequestEqual(
net::URLRequestStatus::SUCCESS), "", base::Time())).Times(2);
@@ -214,7 +213,7 @@
EXPECT_CALL(*original_peer_,
OnReceivedData(StrEq(data.data()), data.length()));
- EXPECT_CALL(*original_peer_, OnReceivedResponse(_, true));
+ EXPECT_CALL(*original_peer_, OnReceivedResponse(_));
EXPECT_CALL(*original_peer_, OnCompletedRequest(
IsURLRequestEqual(net::URLRequestStatus::SUCCESS), "", base::Time()));
@@ -244,7 +243,7 @@
EXPECT_CALL(*original_peer_,
OnReceivedData(StrEq(message.data()), message.length()));
- EXPECT_CALL(*original_peer_, OnReceivedResponse(_, true));
+ EXPECT_CALL(*original_peer_, OnReceivedResponse(_));
EXPECT_CALL(*original_peer_, OnCompletedRequest(
IsURLRequestEqual(net::URLRequestStatus::SUCCESS), "", base::Time()));
« no previous file with comments | « chrome/common/extensions/extension_localization_peer.cc ('k') | chrome/common/render_messages_params.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698