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

Unified Diff: components/nacl/renderer/manifest_downloader.h

Issue 288773002: Pepper: Simplify manifest download logic. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 7 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 | « no previous file | components/nacl/renderer/manifest_downloader.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/nacl/renderer/manifest_downloader.h
diff --git a/components/nacl/renderer/manifest_downloader.h b/components/nacl/renderer/manifest_downloader.h
index dbdbf8453c6d1a3b200a9fa1bdbc84e9f62a9e4e..7488cab0c2dfb0e40df5092deb4bafd586784ae0 100644
--- a/components/nacl/renderer/manifest_downloader.h
+++ b/components/nacl/renderer/manifest_downloader.h
@@ -26,6 +26,12 @@ class ManifestDownloader : public blink::WebURLLoaderClient {
ManifestDownloader(bool is_installed, ManifestDownloaderCallback cb);
virtual ~ManifestDownloader();
+ // This is a pretty arbitrary limit on the byte size of the NaCl manifest
+ // file.
+ // Note that the resulting string object has to have at least one byte extra
+ // for the null termination character.
+ static const size_t kNaClManifestMaxFileBytes = 1024 * 1024;
+
private:
// WebURLLoaderClient implementation.
virtual void didReceiveResponse(blink::WebURLLoader* loader,
« no previous file with comments | « no previous file | components/nacl/renderer/manifest_downloader.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698