Avoid using a cast for a overload selection

content/browser/indexed_db/indexed_db_unittest.cc

Index: content/browser/indexed_db/indexed_db_unittest.cc
diff --git a/content/browser/indexed_db/indexed_db_unittest.cc b/content/browser/indexed_db/indexed_db_unittest.cc
index ea958de168bacf12b03c5ebea89aca98b59a6350..b3736a50ce0fa3dc39cd37e4e92bdee9f0a9b764 100644
--- a/content/browser/indexed_db/indexed_db_unittest.cc
+++ b/content/browser/indexed_db/indexed_db_unittest.cc
@@ -247,11 +247,11 @@ TEST_F(IndexedDBTest, DeleteFailsIfDirectoryLocked) {
   ASSERT_TRUE(lock);
 
   // TODO(jsbell): Remove static_cast<> when overloads are eliminated.

[jsbell, 2016/07/07 16:49:05]

Would have been nice to eliminate the comment too. :) Follow-up?

+  void (IndexedDBContextImpl::* delete_for_origin)(const Origin&) =
+      &IndexedDBContextImpl::DeleteForOrigin;
   idb_context->TaskRunner()->PostTask(
       FROM_HERE,
-      base::Bind(static_cast<void (IndexedDBContextImpl::*)(const Origin&)>(
-                     &IndexedDBContextImpl::DeleteForOrigin),
-                 idb_context, kTestOrigin));
+      base::Bind(delete_for_origin, idb_context, kTestOrigin));
   FlushIndexedDBTaskRunner();
 
   EXPECT_TRUE(base::DirectoryExists(test_path));