Implement comparison operators for base::Value

chrome/browser/ui/webui/options/preferences_browsertest.cc

Index: chrome/browser/ui/webui/options/preferences_browsertest.cc
diff --git a/chrome/browser/ui/webui/options/preferences_browsertest.cc b/chrome/browser/ui/webui/options/preferences_browsertest.cc
index 17bd68dfb98a0c45ee2301899445fb9bf203c2f3..e33a963b2c526aa6914709f173723f61f92b2ad1 100644
--- a/chrome/browser/ui/webui/options/preferences_browsertest.cc
+++ b/chrome/browser/ui/webui/options/preferences_browsertest.cc
@@ -67,11 +67,6 @@ using testing::_;
 
 namespace base {
 
-// Helper for using EXPECT_EQ() with base::Value.
-bool operator==(const base::Value& first, const base::Value& second) {
-  return first.Equals(&second);
-}
-
 // Helper for pretty-printing the contents of base::Value in case of failures.
 void PrintTo(const base::Value& value, std::ostream* stream) {

[jdoerrie, 2017/03/24 16:33:16]

Given the presence of |std::ostream& operator<<(std::ostream& out, const Value&
value)| that pretty much does the same thing I feel like this can be removed as
well. However, it is not really related to this CL. WDYT?

   std::string json;