Changeset 220754 in webkit

Timestamp:

Aug 15, 2017 1:21:26 PM (7 years ago)

Author:

commit-queue@webkit.org

Message:

WPT harness errors on leaks bot
​https://bugs.webkit.org/show_bug.cgi?id=175269

Patch by Youenn Fablet <​youenn@apple.com> on 2017-08-15
Reviewed by Alexey Proskuryakov.

To try debugging these errors, let's check whether the WPT server is
sending back resources of the expected size.

• web-platform-tests/tools/wptserve/wptserve/response.py:

(Response.write_content):
(ResponseWriter.write_content):
(ResponseWriter.write):
(ResponseWriter.write_content_file):

Location:

trunk/LayoutTests/imported/w3c

Files:

• ChangeLog (modified) (1 diff)
• web-platform-tests/tools/wptserve/wptserve/response.py (modified) (7 diffs)

trunk/LayoutTests/imported/w3c/ChangeLog

@@ +1 @@
+2017-08-15  Youenn Fablet  <youenn@apple.com>
+
+        WPT harness errors on leaks bot
+        https://bugs.webkit.org/show_bug.cgi?id=175269
+
+        Reviewed by Alexey Proskuryakov.
+
+        To try debugging these errors, let's check whether the WPT server is
+        sending back resources of the expected size.
+
+        * web-platform-tests/tools/wptserve/wptserve/response.py:
+        (Response.write_content):
+        (ResponseWriter.write_content):
+        (ResponseWriter.write):
+        (ResponseWriter.write_content_file):
+
 2017-08-15  Chris Dumez  <cdumez@apple.com>
 

trunk/LayoutTests/imported/w3c/web-platform-tests/tools/wptserve/wptserve/response.py

@@ -3 +3 @@
 import Cookie
 import json
+import traceback
 import types
 import uuid
@@ -206 +207 @@
     def write_content(self):
         """Write out the response content"""
+        response_size = 0
         if self.request.method != "HEAD" or self.send_body_for_head_request:
             for item in self.iter_content():
-                self.writer.write_content(item)
+                response_size += self.writer.write_content(item)
+        self.logger.debug("Wrote content of size %d for %s" % (response_size, self.request.request_path))
 
     def write(self):
@@ -369 +372 @@
         self.request = response.request
         self.file_chunk_size = 32 * 1024
+        self.logger = get_logger()
 
     def write_status(self, code, message=None):
@@ -425 +429 @@
     def write_content(self, data):
         """Write the body of the response."""
+        response_size = 0
         if isinstance(data, types.StringTypes):
-            self.write(data)
+            response_size = self.write(data)
         else:
-            self.write_content_file(data)
+            response_size = self.write_content_file(data)
         if not self._response.explicit_flush:
             self.flush()
+        return response_size
 
     def write(self, data):
@@ -437 +443 @@
         self.content_written = True
         try:
-            self._wfile.write(self.encode(data))
+            buffer = self.encode(data)
+            self._wfile.write(buffer)
+            return len(buffer)
         except socket.error:
             # This can happen if the socket got closed by the remote end
+            self.logger.debug("Got exception when writing response " + traceback.format_exc())
             pass
 
@@ -446 +455 @@
         Does not flush."""
         self.content_written = True
+        written_size = 0
         while True:
             buf = data.read(self.file_chunk_size)
+            written_size += len(buf)
             if not buf:
                 break
@@ -453 +464 @@
                 self._wfile.write(buf)
             except socket.error:
+                self.logger.debug("Got exception when writing response " + traceback.format_exc())
                 break
         data.close()
+        return written_size
 
     def encode(self, data):