More OStreamsUse OStreams more often.

src/property.cc

 // Copyright 2014 the V8 project 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 "src/property.h"
 
 #include "src/handles-inl.h"
 
 namespace v8 {
 namespace internal {
 
 void LookupResult::Iterate(ObjectVisitor* visitor) {
   LookupResult* current = this;  // Could be NULL.
   while (current != NULL) {
     visitor->VisitPointer(BitCast<Object**>(&current->holder_));
     visitor->VisitPointer(BitCast<Object**>(&current->transition_));
     current = current->next_;
   }
 }
 
 
-#ifdef OBJECT_PRINT
-void LookupResult::Print(FILE* out) {
-  OFStream os(out);
-  if (!IsFound()) {
-    os << "Not Found\n";
-    return;
-  }
+OStream& operator<<(OStream& os, const LookupResult& r) {
+  if (!r.IsFound()) return os << "Not Found\n";
 
   os << "LookupResult:\n";
-  os << " -cacheable = " << (IsCacheable() ? "true" : "false") << "\n";
-  os << " -attributes = " << hex << GetAttributes() << dec << "\n";
-  if (IsTransition()) {
+  os << " -cacheable = " << (r.IsCacheable() ? "true" : "false") << "\n";
+  os << " -attributes = " << hex << r.GetAttributes() << dec << "\n";
+  if (r.IsTransition()) {
     os << " -transition target:\n";
-    GetTransitionTarget()->Print(out);
+    r.GetTransitionTarget()->Print(os);
     os << "\n";
   }
-  switch (type()) {
+  switch (r.type()) {
     case NORMAL:
-      os << " -type = normal\n"
-         << " -entry = " << GetDictionaryEntry() << "\n";
-      break;
+      return os << " -type = normal\n"
+                << " -entry = " << r.GetDictionaryEntry() << "\n";
     case CONSTANT:
       os << " -type = constant\n"
          << " -value:\n";
-      GetConstant()->Print(out);
-      os << "\n";
-      break;
+      r.GetConstant()->Print(os);
+      return os << "\n";
     case FIELD:
       os << " -type = field\n"
-         << " -index = " << GetFieldIndex().property_index() << "\n"
+         << " -index = " << r.GetFieldIndex().property_index() << "\n"
          << " -field type:";
-      GetFieldType()->PrintTo(os);
-      os << "\n";
-      break;
+      r.GetFieldType()->PrintTo(os);
+      return os << "\n";
     case CALLBACKS:
       os << " -type = call backs\n"
          << " -callback object:\n";
-      GetCallbackObject()->Print(out);
-      break;
+      r.GetCallbackObject()->Print(os);
+      return os;
     case HANDLER:
-      os << " -type = lookup proxy\n";
-      break;
+      return os << " -type = lookup proxy\n";
     case INTERCEPTOR:
-      os << " -type = lookup interceptor\n";
-      break;
+      return os << " -type = lookup interceptor\n";
     case NONEXISTENT:
       UNREACHABLE();
       break;
   }
+  return os;
 }
 
 
-void Descriptor::Print(FILE* out) {
-  PrintF(out, "Descriptor ");
-  GetKey()->ShortPrint(out);
-  PrintF(out, " @ ");
-  GetValue()->ShortPrint(out);
+OStream& operator<<(OStream& os, const Descriptor& d) {
+  return os << "Descriptor " << Brief(*d.GetKey()) << " @ "
+            << Brief(*d.GetValue());
 }
-#endif
 
 } }  // namespace v8::internal