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

Unified Diff: sync/internal_api/public/attachments/attachment_uploader_impl.h

Issue 710073003: Store attachment crc in AttachmentStore (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebase Created 6 years, 1 month 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: sync/internal_api/public/attachments/attachment_uploader_impl.h
diff --git a/sync/internal_api/public/attachments/attachment_uploader_impl.h b/sync/internal_api/public/attachments/attachment_uploader_impl.h
index fde544c59c1e39da44bfa80b912161b6372ddab3..31944261e4fc2a8d77948666672fd4278529a9ae 100644
--- a/sync/internal_api/public/attachments/attachment_uploader_impl.h
+++ b/sync/internal_api/public/attachments/attachment_uploader_impl.h
@@ -51,14 +51,12 @@ class SYNC_EXPORT AttachmentUploaderImpl : public AttachmentUploader,
static GURL GetURLForAttachmentId(const GURL& sync_service_url,
const AttachmentId& attachment_id);
- // Return the crc32c of the memory described by |data| and |size|.
+ // Format crc32c to pass into X-Goog-Hash header
maniscalco 2014/11/11 00:44:54 nit: needs a period at the end.
pavely 2014/11/11 22:27:15 Done.
//
// The value is base64 encoded, big-endian format. Suitable for use in the
// X-Goog-Hash header
// (https://cloud.google.com/storage/docs/reference-headers#xgooghash).
- //
- // Potentially expensive.
- static std::string ComputeCrc32cHash(const char* data, size_t size);
+ static std::string FormatCrc32cHash(uint32_t crc);
private:
class UploadState;

Powered by Google App Engine
This is Rietveld 408576698