Coalesce more hardcoded schemes to using predefined constants.

chrome/browser/back_forward_menu_model_unittest.cc

 // Copyright (c) 2006-2008 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 "chrome/browser/back_forward_menu_model.h"
 
 #include "base/file_path.h"
 #include "base/file_util.h"
 #include "base/path_service.h"
 #include "chrome/browser/profile_manager.h"
 #include "chrome/browser/tab_contents/navigation_controller.h"
 #include "chrome/browser/tab_contents/navigation_entry.h"
 #include "chrome/browser/tab_contents/tab_contents_factory.h"
+#include "chrome/common/url_constants.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
 #if defined(OS_WIN)
 // TODO(port): port this header.
 #include "chrome/browser/tab_contents/tab_contents.h"
 #elif defined(OS_POSIX)
 #include "chrome/common/temp_scaffolding_stubs.h"
 #endif
 
 const TabContentsType kHTTPTabContentsType =
@@ skipping 24 matching lines @@
 };
 
 // This constructs our fake TabContents.
 class BackFwdMenuModelTestTabContentsFactory : public TabContentsFactory {
  public:
   virtual TabContents* CreateInstance() {
     return new BackFwdMenuModelTestTabContents;
   }
 
   virtual bool CanHandleURL(const GURL& url) {
-    return url.scheme() == "http";
+    return url.SchemeIs(chrome::kHttpScheme);
   }
 };
 
 BackFwdMenuModelTestTabContentsFactory factory;
 
 class BackFwdMenuModelTest : public testing::Test {
  public:
 
   // Overridden from testing::Test
   virtual void SetUp() {
@@ skipping 417 matching lines @@
     EXPECT_EQ(4, back_model->GetIndexOfNextChapterStop(3, true));
     // And try backwards as well.
     EXPECT_EQ(3, back_model->GetIndexOfNextChapterStop(4, false));
     EXPECT_EQ(1, back_model->GetIndexOfNextChapterStop(3, false));
     EXPECT_EQ(1, back_model->GetIndexOfNextChapterStop(2, false));
     EXPECT_EQ(-1, back_model->GetIndexOfNextChapterStop(1, false));
   }
   contents->CloseContents();
 }