Refactoring: Remove StyledMarkupAccumulator::shouldAnnotate* methods

Source/core/editing/StyledMarkupAccumulator.h

 /*
  * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009 Apple Inc. All rights reserved.
  * Copyright (C) 2008, 2009, 2010, 2011 Google Inc. All rights reserved.
  * Copyright (C) 2011 Igalia S.L.
  * Copyright (C) 2011 Motorola Mobility. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1. Redistributions of source code must retain the above copyright
@@ skipping 62 matching lines @@
 
     void appendElement(StringBuilder&, Element&, Namespaces*);
     void appendElement(StringBuilder&, Element&, bool, RangeFullySelectsNode);
     void appendStyleNodeOpenTag(StringBuilder&, StylePropertySet*, bool isBlock = false);
 
     // TODO(hajimehoshi): These functions are called from the serializer, but
     // should not.
     Node* highestNodeToBeSerialized() { return m_highestNodeToBeSerialized.get(); }
     void setHighestNodeToBeSerialized(Node* highestNodeToBeSerialized) { m_highestNodeToBeSerialized = highestNodeToBeSerialized; }
     void setWrappingStyle(PassRefPtrWillBeRawPtr<EditingStyle> wrappingStyle) { m_wrappingStyle = wrappingStyle; }
-    bool shouldAnnotate() const { return m_shouldAnnotate == AnnotateForInterchange || m_shouldAnnotate == AnnotateForNavigationTransition; }
-    bool shouldAnnotateForNavigationTransition() const { return m_shouldAnnotate == AnnotateForNavigationTransition; }
-    bool shouldAnnotateForInterchange() const { return m_shouldAnnotate == AnnotateForInterchange; }
 
     size_t length() const { return m_accumulator.length(); }
     void concatenateMarkup(StringBuilder&) const;
 
 private:
     void appendText(StringBuilder&, Text&);
 
     String renderedText(Text&);
     String stringValueForRange(const Text&);
 
     bool shouldApplyWrappingStyle(const Node&) const;
 
+    bool shouldAnnotate() const { return m_shouldAnnotate == AnnotateForInterchange || m_shouldAnnotate == AnnotateForNavigationTransition; }

[yosin_UTC9, 2015/05/18 08:49:35]

Let's implement this function in .cpp file:
http://dev.chromium.org/developers/coding-style/cpp-dos-and-donts#TOC-Stop-in...

[hajimehoshi, 2015/05/18 09:46:25]

Done.

+
     MarkupAccumulator m_accumulator;
     const TextOffset m_start;
     const TextOffset m_end;
     const RefPtrWillBeMember<Document> m_document;
     const EAnnotateForInterchange m_shouldAnnotate;
     RawPtrWillBeMember<Node> m_highestNodeToBeSerialized;
     RefPtrWillBeMember<EditingStyle> m_wrappingStyle;
 };
 
 } // namespace blink
 
 #endif // StyledMarkupAccumulator_h