remove transitive curl include

Author: scotthart

google/cloud/storage/testing/mock_http_request.cc

@@ -13,6 +13,7 @@
 // limitations under the License.
 
 #include "google/cloud/storage/testing/mock_http_request.h"
+#include "google/cloud/internal/curl_handle_factory.h"
 #include <memory>
 
 namespace google {
@@ -22,6 +23,13 @@ namespace testing {
 
 std::shared_ptr<MockHttpRequestBuilder::Impl> MockHttpRequestBuilder::mock_;
 
+MockHttpRequestBuilder::MockHttpRequestBuilder(
+    std::string url,
+    // NOLINTNEXTLINE(performance-unnecessary-value-param)
+    std::shared_ptr<google::cloud::rest_internal::CurlHandleFactory> h) {
+  mock_->Constructor(std::move(url), h->cainfo(), h->capath());
+}
+
 }  // namespace testing
 }  // namespace storage
 }  // namespace cloud

google/cloud/storage/testing/mock_http_request.h

@@ -17,7 +17,6 @@
 
 #include "google/cloud/storage/internal/http_response.h"
 #include "google/cloud/storage/well_known_parameters.h"
-#include "google/cloud/internal/curl_handle_factory.h"
 #include "google/cloud/status_or.h"
 #include <gmock/gmock.h>
 #include <nlohmann/json.hpp>
@@ -27,6 +26,11 @@
 
 namespace google {
 namespace cloud {
+namespace rest_internal {
+GOOGLE_CLOUD_CPP_INLINE_NAMESPACE_BEGIN
+class CurlHandleFactory;
+GOOGLE_CLOUD_CPP_INLINE_NAMESPACE_END
+}  // namespace rest_internal
 namespace storage {
 namespace testing {
 /**
@@ -68,9 +72,8 @@ class MockHttpRequestBuilder {
  public:
   explicit MockHttpRequestBuilder(
       // NOLINTNEXTLINE(performance-unnecessary-value-param)
-      std::string url, std::shared_ptr<rest_internal::CurlHandleFactory> h) {
-    mock_->Constructor(std::move(url), h->cainfo(), h->capath());
-  }
+      std::string url,
+      std::shared_ptr<google::cloud::rest_internal::CurlHandleFactory> h);
 
   using RequestType = MockHttpRequest;