Bug 1297306 - part3:make BreakTypeToString() to be a member of nsLineBox. draft
authorJeremy Chen <jeremychen@mozilla.com>
Wed, 07 Sep 2016 10:20:16 +0800
changeset 410886 e1a29a561af449a55c1e6efc4eed791b6e4e9e18
parent 410885 f5bcd380d88f52e3bb2a39686f7d0b3559a39115
child 410887 5ea66f84a8d635679204c20ceba988ea1473bacf
push id28778
push userjichen@mozilla.com
push dateWed, 07 Sep 2016 02:21:00 +0000
bugs1297306
milestone51.0a1
Bug 1297306 - part3:make BreakTypeToString() to be a member of nsLineBox. With this change, we could export BreakTypeToString() to other files that desire to print break type for debugging. MozReview-Commit-ID: 34m1BWAmZTB
layout/generic/nsLineBox.cpp
layout/generic/nsLineBox.h
--- a/layout/generic/nsLineBox.cpp
+++ b/layout/generic/nsLineBox.cpp
@@ -187,27 +187,29 @@ ListFloats(FILE* out, const char* aPrefi
     else {
       str += "\n###!!! NULL out-of-flow frame";
     }
     fprintf_stderr(out, "%s\n", str.get());
     fc = fc->Next();
   }
 }
 
-const char *
-BreakTypeToString(uint8_t aBreakType)
+const char*
+nsLineBox::BreakTypeToString(uint8_t aBreakType) const
 {
   switch (aBreakType) {
-  case NS_STYLE_CLEAR_NONE: return "nobr";
-  case NS_STYLE_CLEAR_LEFT: return "leftbr";
-  case NS_STYLE_CLEAR_RIGHT: return "rightbr";
-  case NS_STYLE_CLEAR_BOTH: return "leftbr+rightbr";
-  case NS_STYLE_CLEAR_LINE: return "linebr";
-  default:
-    break;
+    case NS_STYLE_CLEAR_NONE: return "nobr";
+    case NS_STYLE_CLEAR_LEFT: return "leftbr";
+    case NS_STYLE_CLEAR_RIGHT: return "rightbr";
+    case NS_STYLE_CLEAR_INLINE_START: return "inlinestartbr";
+    case NS_STYLE_CLEAR_INLINE_END: return "inlineendbr";
+    case NS_STYLE_CLEAR_BOTH: return "leftbr+rightbr";
+    case NS_STYLE_CLEAR_LINE: return "linebr";
+    default:
+      break;
   }
   return "unknown";
 }
 
 char*
 nsLineBox::StateToString(char* aBuf, int32_t aBufSize) const
 {
   snprintf(aBuf, aBufSize, "%s,%s,%s,%s,%s,before:%s,after:%s[0x%x]",
--- a/layout/generic/nsLineBox.h
+++ b/layout/generic/nsLineBox.h
@@ -567,16 +567,17 @@ public:
   // list).
   static bool RFindLineContaining(nsIFrame* aFrame,
                                     const nsLineList_iterator& aBegin,
                                     nsLineList_iterator& aEnd,
                                     nsIFrame* aLastFrameBeforeEnd,
                                     int32_t* aFrameIndexInLine);
 
 #ifdef DEBUG_FRAME_DUMP
+  const char* BreakTypeToString(uint8_t aBreakType) const;
   char* StateToString(char* aBuf, int32_t aBufSize) const;
 
   void List(FILE* out, int32_t aIndent, uint32_t aFlags = 0) const;
   void List(FILE* out = stderr, const char* aPrefix = "", uint32_t aFlags = 0) const;
   nsIFrame* LastChild() const;
 #endif
 
 private: