Remove base's implicit_cast.

ui/base/page_transition_types.cc

 // Copyright (c) 2010 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.
 
 #include "ui/base/page_transition_types.h"
 
 #include "base/logging.h"
 
 namespace ui {
 
 bool PageTransitionCoreTypeIs(PageTransition lhs,
                               PageTransition rhs) {
   // Expect the rhs to be a compile time constant without qualifiers.
   DCHECK(PageTransitionGetQualifier(rhs) == 0 &&
       PageTransitionIsValidType(rhs));
-  return implicit_cast<int>(PageTransitionStripQualifier(lhs)) ==
-      implicit_cast<int>(PageTransitionStripQualifier(rhs));
+  return static_cast<int>(PageTransitionStripQualifier(lhs)) ==

[vmpstr, 2015/09/11 21:31:16]

This looks like an enum-to-same-enum comparison, so I don't think you need a
cast? 

[danakj, 2015/09/11 21:33:30]

Ya I agree.

+         static_cast<int>(PageTransitionStripQualifier(rhs));
 }
 
 PageTransition PageTransitionStripQualifier(PageTransition type) {
   return static_cast<PageTransition>(type & ~PAGE_TRANSITION_QUALIFIER_MASK);
 }
 
 bool PageTransitionIsValidType(int32 type) {
   PageTransition t = PageTransitionStripQualifier(
       static_cast<PageTransition>(type));
   return (t <= PAGE_TRANSITION_LAST_CORE);
@@ skipping 52 matching lines @@
     case PAGE_TRANSITION_AUTO_TOPLEVEL: return "auto_toplevel";
     case PAGE_TRANSITION_FORM_SUBMIT: return "form_submit";
     case PAGE_TRANSITION_RELOAD: return "reload";
     case PAGE_TRANSITION_KEYWORD: return "keyword";
     case PAGE_TRANSITION_KEYWORD_GENERATED: return "keyword_generated";
   }
   return NULL;
 }
 
 }  // namespace ui