Re-enable reading list tests

ios/web/public/test/http_server/http_server.h

 // Copyright 2014 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 IOS_WEB_PUBLIC_TEST_HTTP_SERVER_HTTP_SERVER_H_
 #define IOS_WEB_PUBLIC_TEST_HTTP_SERVER_HTTP_SERVER_H_
 
 #include <memory>
 #include <vector>
 
@@ skipping 62 matching lines @@
   // Starts the server on the default port 8080. CHECKs if the server can not be
   // started.
   // Must be called from the main thread.
   void StartOrDie();
   // Stops the server and prevents it from accepting new requests.
   // Must be called from the main thread.
   void Stop();
   // Returns true if the server is running.
   // Must be called from the main thread.
   bool IsRunning() const;
+  // Sets the server to hang and return no response.
+  void setSuspend(BOOL suspended);

[baxley, 2017/06/06 18:14:17]

would it be more consistent if this is "bool"? what do you think?

[huangml1, 2017/06/07 18:10:54]

I feel that setSuspend is more like an action than returning a state.  Do you
think we should make it consistent with the bool IsRunning Style?  

[baxley, 2017/06/07 18:21:06]

I meant more that the rest of this is C++. If the callers are Objective-C then I
guess it's okay, and maybe better? Both should be fine.

 
   // Returns the port that the server is running on. Thread Safe
   NSUInteger GetPort() const;
 
   // Adds a ResponseProvider. Takes ownership of the ResponseProvider.
   // Note for using URLs inside of the |response_provider|:
   // The HttpServer cannot run on default HTTP port 80, so URLs used in
   // ResponseProviders must be converted at runtime after the HttpServer's port
   // is determined. Please use |MakeUrl| to handle converting URLs.
   // Must be called from the main thread.
@@ skipping 36 matching lines @@
   std::vector<scoped_refptr<RefCountedResponseProviderWrapper>> providers_;
   // Returns the response providers for a request.
   ResponseProvider* GetResponseProviderForProviderRequest(
       const web::ResponseProvider::Request& request);
   // Provides a shim between EmbeddedTestServer and ResponseProvider. This
   // handler converts an EmbeddedTestServer request to ResponseProvider
   // request, look up the corresponding providers, and then converts the
   // ResponseProvider response back to EmbeddedTestServer response.
   std::unique_ptr<net::test_server::HttpResponse> GetResponse(
       const net::test_server::HttpRequest& request);
+  // The status that if the server hangs.

[baxley, 2017/06/06 18:14:17]

"Status tracking if the server is hung."

[huangml1, 2017/06/07 18:10:54]

Done.

+  BOOL isSuspended;

[baxley, 2017/06/06 18:14:17]

bool?

[huangml1, 2017/06/07 18:10:54]

Done.

   DISALLOW_COPY_AND_ASSIGN(HttpServer);
 };
 
 }  // namespace test
 }  // namspace web
 
 #endif  // IOS_WEB_PUBLIC_TEST_HTTP_SERVER_HTTP_SERVER_H_