This is to allow extensions to determine the total length of Javascript blobs.

This cannot AFAIK be done via passing the length in a custom binding, as base::Value does not support int64 a.k.a. 'long long' types.

The total blob length is going to be used in media_galleries_api.cc to construct the SafeMediaMetadataParser in a followup CL. It appears unused in this patch.

BUG=

Review URL: https://codereview.chromium.org/129823003

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@244860 0039d316-1c4b-4281-b951-d872f2087c98

4 files changed, 12 insertions, 6 deletions

diff --git a/chrome/browser/extensions/api/feedback_private/feedback_service.cc b/chrome/browser/extensions/api/feedback_private/feedback_service.cc
index b66831f..f32840cc 100644
--- a/chrome/browser/extensions/api/feedback_private/feedback_service.cc
+++ b/chrome/browser/extensions/api/feedback_private/feedback_service.cc
@@ -66,7 +66,8 @@ void FeedbackService::SendFeedback(
   CompleteSendFeedback();
 }
 
-void FeedbackService::AttachedFileCallback(scoped_ptr<std::string> data) {
+void FeedbackService::AttachedFileCallback(scoped_ptr<std::string> data,
+                                           int64 /* total_blob_length */) {
   if (!data.get())
     feedback_data_->set_attached_file_uuid(std::string());
   else
@@ -75,7 +76,8 @@ void FeedbackService::AttachedFileCallback(scoped_ptr<std::string> data) {
   CompleteSendFeedback();
 }
 
-void FeedbackService::ScreenshotCallback(scoped_ptr<std::string> data) {
+void FeedbackService::ScreenshotCallback(scoped_ptr<std::string> data,
+                                         int64 /* total_blob_length */) {
   if (!data.get())
     feedback_data_->set_screenshot_uuid(std::string());
   else
diff --git a/chrome/browser/extensions/api/feedback_private/feedback_service.h b/chrome/browser/extensions/api/feedback_private/feedback_service.h
index 9886149..3daff71 100644
--- a/chrome/browser/extensions/api/feedback_private/feedback_service.h
+++ b/chrome/browser/extensions/api/feedback_private/feedback_service.h
@@ -57,8 +57,10 @@ class FeedbackService {
   virtual base::WeakPtr<FeedbackService> GetWeakPtr() = 0;
 
   // Callbacks to receive blob data.
-  void AttachedFileCallback(scoped_ptr<std::string> data);
-  void ScreenshotCallback(scoped_ptr<std::string> data);
+  void AttachedFileCallback(scoped_ptr<std::string> data,
+                            int64 total_blob_length);
+  void ScreenshotCallback(scoped_ptr<std::string> data,
+                          int64 total_blob_length);
 
   // Checks if we have read all the blobs we need to; signals the feedback
   // data object once all the requisite data has been populated.
diff --git a/chrome/browser/extensions/api/media_galleries/media_galleries_api.cc b/chrome/browser/extensions/api/media_galleries/media_galleries_api.cc
index 7d8a98f..0f12528 100644
--- a/chrome/browser/extensions/api/media_galleries/media_galleries_api.cc
+++ b/chrome/browser/extensions/api/media_galleries/media_galleries_api.cc
@@ -558,7 +558,8 @@ void MediaGalleriesGetMetadataFunction::OnPreferencesInit(
 }
 
 void MediaGalleriesGetMetadataFunction::SniffMimeType(
-    bool mime_type_only, scoped_ptr<std::string> blob_header) {
+    bool mime_type_only, scoped_ptr<std::string> blob_header,
+    int64 /* total_blob_length */) {
   DCHECK(content::BrowserThread::CurrentlyOn(content::BrowserThread::UI));
 
   MediaGalleries::MediaMetadata metadata;
diff --git a/chrome/browser/extensions/api/media_galleries/media_galleries_api.h b/chrome/browser/extensions/api/media_galleries/media_galleries_api.h
index 9918704..121eba1 100644
--- a/chrome/browser/extensions/api/media_galleries/media_galleries_api.h
+++ b/chrome/browser/extensions/api/media_galleries/media_galleries_api.h
@@ -125,7 +125,8 @@ class MediaGalleriesGetMetadataFunction : public ChromeAsyncExtensionFunction {
   // Bottom half for RunImpl, invoked after the preferences is initialized.
   void OnPreferencesInit(bool mime_type_only, const std::string& blob_uuid);
 
-  void SniffMimeType(bool mime_type_only, scoped_ptr<std::string> blob_header);
+  void SniffMimeType(bool mime_type_only, scoped_ptr<std::string> blob_header,
+                     int64 total_blob_length);
 };
 
 }  // namespace extensions