Rendering API cleanup

Source/core/rendering/RenderTableSection.h

 /*
  * Copyright (C) 1997 Martin Jones (mjones@kde.org)
  *           (C) 1997 Torben Weis (weis@kde.org)
  *           (C) 1998 Waldo Bastian (bastian@kde.org)
  *           (C) 1999 Lars Knoll (knoll@kde.org)
  *           (C) 1999 Antti Koivisto (koivisto@kde.org)
  * Copyright (C) 2003, 2004, 2005, 2006, 2009, 2013 Apple Inc. All rights reserved.
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Library General Public
@@ skipping 31 matching lines @@
 
 // Helper class for paintObject.
 class CellSpan {
 public:
     CellSpan(unsigned start, unsigned end)
         : m_start(start)
         , m_end(end)
     {
     }
 
-    unsigned start() const { return m_start; }
-    unsigned end() const { return m_end; }
-
     unsigned& start() { return m_start; }
     unsigned& end() { return m_end; }

[Julien - ping for review, 2014/10/02 16:47:49]

This API seems deeply dangerous.

 
 private:
     unsigned m_start;
     unsigned m_end;
 };
 
 class RenderTableCell;
 class RenderTableRow;
 
 class RenderTableSection FINAL : public RenderBox {
@@ skipping 264 matching lines @@
     static const bool needsDestruction = false;
 };
 template<> struct VectorTraits<blink::RenderTableSection::RowStruct> : VectorTraitsBase<blink::RenderTableSection::RowStruct> {
     static const bool needsDestruction = false;
 };
 #endif
 
 }
 
 #endif // RenderTableSection_h