Remove base's implicit_cast.

content/browser/cache_storage/cache_storage.cc

Index: content/browser/cache_storage/cache_storage.cc
diff --git a/content/browser/cache_storage/cache_storage.cc b/content/browser/cache_storage/cache_storage.cc
index f511164c6905e416e5ce97e0074a3f437620da68..46716c777f5b2b44af09e68bbb50571632c63c56 100644
--- a/content/browser/cache_storage/cache_storage.cc
+++ b/content/browser/cache_storage/cache_storage.cc
@@ -276,7 +276,7 @@ class CacheStorage::SimpleCacheLoader : public CacheStorage::CacheLoader {
       const BoolCallback& callback,
       const scoped_refptr<base::SingleThreadTaskRunner>& original_task_runner) {
     int bytes_written = base::WriteFile(tmp_path, data.c_str(), data.size());
-    if (bytes_written != implicit_cast<int>(data.size())) {
+    if (bytes_written != static_cast<int>(data.size())) {

[vmpstr, 2015/09/11 21:00:49]

Same here, maybe?

[danakj, 2015/09/11 21:20:28]

Here we pass data.size() to base::WriteFile which takes an int. So that's a
silent truncation right now.

So the assumption here is that data will be within positive integer size. I
think checked_cast makes sense here.

       base::DeleteFile(tmp_path, /* recursive */ false);
       original_task_runner->PostTask(FROM_HERE, base::Bind(callback, false));
     }