webkitpy: use the filesystem object properly in fileuploader
authordpranke@chromium.org <dpranke@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 9 Apr 2012 09:22:44 +0000 (09:22 +0000)
committerdpranke@chromium.org <dpranke@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 9 Apr 2012 09:22:44 +0000 (09:22 +0000)
https://bugs.webkit.org/show_bug.cgi?id=83326

Reviewed by Adam Barth.

We should be using the filesystem object rather than the codecs
object.

* Scripts/webkitpy/common/net/file_uploader.py:
(FileUploader.upload_as_multipart_form_data):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@113562 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Tools/ChangeLog
Tools/Scripts/webkitpy/common/net/file_uploader.py

index 7035879..8d358dc 100644 (file)
@@ -1,3 +1,16 @@
+2012-04-09  Dirk Pranke  <dpranke@chromium.org>
+
+        webkitpy: use the filesystem object properly in fileuploader
+        https://bugs.webkit.org/show_bug.cgi?id=83326
+
+        Reviewed by Adam Barth.
+
+        We should be using the filesystem object rather than the codecs
+        object.
+
+        * Scripts/webkitpy/common/net/file_uploader.py:
+        (FileUploader.upload_as_multipart_form_data):
+
 2012-04-09  Eric Seidel  <eric@webkit.org>
 
         Enable webkit_unit_tests for commit queue and EWS while tracking failures
index c783356..fa8345e 100644 (file)
@@ -27,7 +27,6 @@
 # (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
 # OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
 
-import codecs
 import mimetypes
 import time
 import urllib2
@@ -93,9 +92,7 @@ class FileUploader(object):
     def upload_as_multipart_form_data(self, filesystem, files, attrs):
         file_objs = []
         for filename, path in files:
-            # FIXME: We should talk to the filesytem via a Host object.
-            with codecs.open(path, "rb") as file:
-                file_objs.append(('file', filename, file.read()))
+            file_objs.append(('file', filename, filesystem.read_binary_file(path)))
 
         # FIXME: We should use the same variable names for the formal and actual parameters.
         content_type, data = _encode_multipart_form_data(attrs, file_objs)