Remove some naked |new| statements in favor of MakeUnique.

net/http/http_network_transaction.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_HTTP_HTTP_NETWORK_TRANSACTION_H_
 #define NET_HTTP_HTTP_NETWORK_TRANSACTION_H_
 
 #include <stdint.h>
 
 #include <memory>
-#include <string>

[mgersh, 2017/05/24 15:26:40]

Not sure why this was deleted, it looks like it's still used.

[Bence, 2017/05/24 15:44:56]

Reverted.  I am sorry for the oversight.  Since string is only used by
reference, I thought a forward declaration would be enough (which I forgot to
add anyway), but common wisdom seems to be that whenever std::string is
referenced, the best thing is just to include the header.  (It's a typedef so
it's not possible to forward declare, consequently there's no forward declare
header for it.)

 
 #include "base/gtest_prod_util.h"
 #include "base/macros.h"
 #include "base/memory/ref_counted.h"
 #include "base/time/time.h"
 #include "crypto/ec_private_key.h"
 #include "net/base/net_error_details.h"
 #include "net/base/net_export.h"
 #include "net/base/network_throttle_manager.h"
 #include "net/base/request_priority.h"
@@ skipping 375 matching lines @@
   // Communicate lifetime of transaction to the throttler, and
   // throttled state to the transaction.
   std::unique_ptr<NetworkThrottleManager::Throttle> throttle_;
 
   DISALLOW_COPY_AND_ASSIGN(HttpNetworkTransaction);
 };
 
 }  // namespace net
 
 #endif  // NET_HTTP_HTTP_NETWORK_TRANSACTION_H_