Add const accessors to URLRequest and UploadData.

net/url_request/url_request.h

Index: net/url_request/url_request.h
diff --git a/net/url_request/url_request.h b/net/url_request/url_request.h
index f405de209cc9b836317ab7f58f5a457d5a763d5f..2ecd2b0e095d1804279249c2f3ba4b6329011ea5 100644
--- a/net/url_request/url_request.h
+++ b/net/url_request/url_request.h
@@ -413,6 +413,7 @@ class NET_EXPORT URLRequest : NON_EXPORTED_BASE(public base::NonThreadSafe),
   void set_upload(UploadData* upload);
 
   // Get the upload data directly.
+  const UploadData* get_upload() const;

[gavinp, 2012/07/27 20:28:56]

Hrm, I wonder why this method name doesn't meet our coding standard: normally
hacker_style methods should only be inline simple getters.

[vabr (Chromium), 2012/07/28 20:37:09]

Well, I don't know, just copied the style from the non-const getter :). As you
suggest the _mutable name again in hacker_style I understand you don't want me
to change it. If you do, let me know, I can set-up a separate CL for that.
Alternatively to renaming, we could also inline these functions, as there indeed
are just simple getters.

   UploadData* get_upload();

[gavinp, 2012/07/27 20:28:56]

get_upload_mutable() here, too?

[vabr (Chromium), 2012/07/28 20:37:09]

Done.

 
   // Returns true if the request has a non-empty message body to upload.