| Index: chrome/renderer/extensions/extension_localization_peer_unittest.cc
|
| diff --git a/chrome/renderer/extensions/extension_localization_peer_unittest.cc b/chrome/renderer/extensions/extension_localization_peer_unittest.cc
|
| index d7f59b5f05acc49ed451689aad5816ef477567ee..94a9b5647347a8043d01b899aa2f29783695ce17 100644
|
| --- a/chrome/renderer/extensions/extension_localization_peer_unittest.cc
|
| +++ b/chrome/renderer/extensions/extension_localization_peer_unittest.cc
|
| @@ -72,13 +72,10 @@ class MockRequestPeer : public content::RequestPeer {
|
| MOCK_METHOD2(OnDownloadedData, void(int len, int encoded_data_length));
|
| void OnReceivedData(
|
| std::unique_ptr<RequestPeer::ReceivedData> data) override {
|
| - OnReceivedDataInternal(data->payload(), data->length(),
|
| - data->encoded_data_length());
|
| + OnReceivedDataInternal(data->payload(), data->length());
|
| }
|
| - MOCK_METHOD3(OnReceivedDataInternal,
|
| - void(const char* data,
|
| - int data_length,
|
| - int encoded_data_length));
|
| + MOCK_METHOD2(OnReceivedDataInternal, void(const char* data, int data_length));
|
| + MOCK_METHOD1(OnTransferSizeUpdated, void(int transfer_size_diff));
|
| MOCK_METHOD6(OnCompletedRequest,
|
| void(int error_code,
|
| bool was_ignored_by_handler,
|
| @@ -139,12 +136,12 @@ TEST_F(ExtensionLocalizationPeerTest, OnReceivedData) {
|
|
|
| const std::string data_chunk("12345");
|
| filter_peer_->OnReceivedData(base::MakeUnique<content::FixedReceivedData>(
|
| - data_chunk.data(), data_chunk.length(), -1));
|
| + data_chunk.data(), data_chunk.length()));
|
|
|
| EXPECT_EQ(data_chunk, GetData());
|
|
|
| filter_peer_->OnReceivedData(base::MakeUnique<content::FixedReceivedData>(
|
| - data_chunk.data(), data_chunk.length(), -1));
|
| + data_chunk.data(), data_chunk.length()));
|
| EXPECT_EQ(data_chunk + data_chunk, GetData());
|
| }
|
|
|
| @@ -165,7 +162,7 @@ TEST_F(ExtensionLocalizationPeerTest, OnCompletedRequestBadURLRequestStatus) {
|
| TEST_F(ExtensionLocalizationPeerTest, OnCompletedRequestEmptyData) {
|
| SetUpExtensionLocalizationPeer("text/css", GURL(kExtensionUrl_1));
|
|
|
| - EXPECT_CALL(*original_peer_, OnReceivedDataInternal(_, _, _)).Times(0);
|
| + EXPECT_CALL(*original_peer_, OnReceivedDataInternal(_, _)).Times(0);
|
| EXPECT_CALL(*sender_, Send(_)).Times(0);
|
|
|
| EXPECT_CALL(*original_peer_, OnReceivedResponse(_));
|
| @@ -186,7 +183,7 @@ TEST_F(ExtensionLocalizationPeerTest, OnCompletedRequestNoCatalogs) {
|
| std::string data = GetData();
|
| EXPECT_CALL(*original_peer_, OnReceivedResponse(_)).Times(1);
|
| EXPECT_CALL(*original_peer_,
|
| - OnReceivedDataInternal(StrEq(data.c_str()), data.length(), -1))
|
| + OnReceivedDataInternal(StrEq(data.c_str()), data.length()))
|
| .Times(1);
|
| EXPECT_CALL(*original_peer_, OnCompletedRequest(net::OK, false, false,
|
| base::TimeTicks(), -1, -1))
|
| @@ -200,7 +197,7 @@ TEST_F(ExtensionLocalizationPeerTest, OnCompletedRequestNoCatalogs) {
|
| SetUpExtensionLocalizationPeer("text/css", GURL(kExtensionUrl_1));
|
| EXPECT_CALL(*original_peer_, OnReceivedResponse(_)).Times(1);
|
| EXPECT_CALL(*original_peer_,
|
| - OnReceivedDataInternal(StrEq(data.c_str()), data.length(), -1))
|
| + OnReceivedDataInternal(StrEq(data.c_str()), data.length()))
|
| .Times(1);
|
| EXPECT_CALL(*original_peer_, OnCompletedRequest(net::OK, false, false,
|
| base::TimeTicks(), -1, -1))
|
| @@ -228,7 +225,7 @@ TEST_F(ExtensionLocalizationPeerTest, OnCompletedRequestWithCatalogs) {
|
| std::string data("some new text");
|
| EXPECT_CALL(*original_peer_, OnReceivedResponse(_));
|
| EXPECT_CALL(*original_peer_,
|
| - OnReceivedDataInternal(StrEq(data.c_str()), data.length(), -1));
|
| + OnReceivedDataInternal(StrEq(data.c_str()), data.length()));
|
|
|
| EXPECT_CALL(*original_peer_, OnCompletedRequest(net::OK, false, false,
|
| base::TimeTicks(), -1, -1));
|
| @@ -254,8 +251,8 @@ TEST_F(ExtensionLocalizationPeerTest, OnCompletedRequestReplaceMessagesFails) {
|
|
|
| // __MSG_missing_message__ is missing, so message stays the same.
|
| EXPECT_CALL(*original_peer_, OnReceivedResponse(_));
|
| - EXPECT_CALL(*original_peer_, OnReceivedDataInternal(StrEq(message.c_str()),
|
| - message.length(), -1));
|
| + EXPECT_CALL(*original_peer_,
|
| + OnReceivedDataInternal(StrEq(message.c_str()), message.length()));
|
|
|
| EXPECT_CALL(*original_peer_, OnCompletedRequest(net::OK, false, false,
|
| base::TimeTicks(), -1, 0));
|
|
|