Fix C4334 (32-bit shift converted to 64-bit) warnings

net/base/mime_sniffer_perftest.cc

Index: net/base/mime_sniffer_perftest.cc
diff --git a/net/base/mime_sniffer_perftest.cc b/net/base/mime_sniffer_perftest.cc
index 72ff0dc1ad8ff69467d661c9efe5417ca1c0fba5..be81ef7b653655fb59067d45d32ee65284f9fb6b 100644
--- a/net/base/mime_sniffer_perftest.cc
+++ b/net/base/mime_sniffer_perftest.cc
@@ -88,9 +88,8 @@ TEST(MimeSnifferTest, PlainTextPerfTest) {
   // The purpose of the static_cast<size_t>() here is to prevent MSVC from
   // complaining about an implicit promotion to 64 bits when compiling 64-bit.

[asanka, 2016/04/06 02:05:56]

Nit: obsolete?

[brucedawson, 2016/04/06 17:53:42]

Good catch. Done.

   size_t expected_size =
-      plaintext.size() *
-      static_cast<size_t>(
-          1u << base::bits::Log2Ceiling(kTargetSize / plaintext.size()));
+      plaintext.size() * static_cast<size_t>(1)

[asanka, 2016/04/06 02:05:57]

Nit: Can we get rid of the "* static_cast<size_t>(1)" altogether?

[brucedawson, 2016/04/06 17:53:42]

Hey, good point. That's quite weird, and unnecessary. Removed.

+      << base::bits::Log2Ceiling(kTargetSize / plaintext.size());
   plaintext.reserve(expected_size);
   while (plaintext.size() < kTargetSize)
     plaintext += plaintext;