Add const accessors to URLRequest and UploadData.

net/base/upload_data.h

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef NET_BASE_UPLOAD_DATA_H_
 #define NET_BASE_UPLOAD_DATA_H_
 
 #include <vector>
 
 #include "base/basictypes.h"
@@ skipping 205 matching lines @@
 
   // Returns true if the upload data is entirely in memory (i.e. the
   // upload data is not chunked, and all elemnts are of TYPE_BYTES).
   bool IsInMemory() const;
 
   // Resets the offset of each upload data element to zero, so that the
   // upload data can be reread. This can happen if the same upload data is
   // reused for a new UploadDataStream.
   void ResetOffset();
 
+  const std::vector<Element>* elements() const {
+    return &elements_;
+  }
+
   std::vector<Element>* elements() {

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

A suggestion: can we rename this one elements_mutable()?  Having both versions
overloaded is acceptable within our coding standard, but distinguishing them
does help readability.

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

Done.

     return &elements_;
   }
 
   void SetElements(const std::vector<Element>& elements);
 
   void swap_elements(std::vector<Element>* elements) {
     elements_.swap(*elements);
   }
 
   // Identifies a particular upload instance, which is used by the cache to
@@ skipping 39 matching lines @@
 
 inline bool operator!=(const UploadData::Element& a,
                        const UploadData::Element& b) {
   return !(a == b);
 }
 #endif  // defined(UNIT_TEST)
 
 }  // namespace net
 
 #endif  // NET_BASE_UPLOAD_DATA_H_