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

Unified Diff: chrome/renderer/extensions/extension_localization_peer.cc

Issue 2105713002: Render process changes for ResourceTiming sizes. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@resource_timing_sizes_browser_process
Patch Set: Fix lint error: ASSERT -> DCHECK Created 4 years, 6 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/renderer/extensions/extension_localization_peer.cc
diff --git a/chrome/renderer/extensions/extension_localization_peer.cc b/chrome/renderer/extensions/extension_localization_peer.cc
index 87b623ca4c0588b7a9750485555843b4763ea399..0c8bd432535a145fa4c5ce8780a13ac144939004 100644
--- a/chrome/renderer/extensions/extension_localization_peer.cc
+++ b/chrome/renderer/extensions/extension_localization_peer.cc
@@ -28,6 +28,9 @@ class StringData final : public content::RequestPeer::ReceivedData {
const char* payload() const override { return data_.data(); }
int length() const override { return data_.size(); }
int encoded_length() const override { return -1; }
Kunihiko Sakamoto 2016/07/01 10:12:35 encoded_data_length?
Adam Rice 2016/07/04 02:44:18 Done. Sorry for making you act as a human compiler
+ // The original data has substitutions applied, so the original
+ // encoded_body_length no longer applies.
+ int encoded_body_length() const override { return data.size(); }
Kunihiko Sakamoto 2016/07/01 10:12:35 s/data/data_/
Adam Rice 2016/07/04 02:44:18 Done.
private:
std::string data_;

Powered by Google App Engine
This is Rietveld 408576698