Fixed StyledLabel size caching

ui/views/controls/styled_label.h

 // Copyright 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef UI_VIEWS_CONTROLS_STYLED_LABEL_H_
 #define UI_VIEWS_CONTROLS_STYLED_LABEL_H_
 
 #include <list>
 #include <map>
 
@@ skipping 129 matching lines @@
   // The ranges that should be linkified, sorted by start position.
   StyleRanges style_ranges_;
 
   // A mapping from a view to the range it corresponds to in |text_|. Only views
   // that correspond to ranges with is_link style set will be added to the map.
   std::map<View*, gfx::Range> link_targets_;
 
   // This variable saves the result of the last GetHeightForWidth call in order
   // to avoid repeated calculation.
   mutable gfx::Size calculated_size_;
+  int layout_built_for_width_;

[sky, 2014/12/12 16:01:10]

This needs a description (and a newline). Also, the name is rather confusing.
How about width_at_last_layout_?

[sky, 2014/12/15 17:03:07]

You seemed to have missed this comment.

[edjomin, 2014/12/16 08:51:52]

Yes, I've missed it, sorry

[edjomin, 2014/12/16 08:51:52]

Done.

 
   // Background color on which the label is drawn, for auto color readability.
   SkColor displayed_on_background_color_;
   bool displayed_on_background_color_set_;
 
   // Controls whether the text is automatically re-colored to be readable on the
   // background.
   bool auto_color_readability_enabled_;
 
   DISALLOW_COPY_AND_ASSIGN(StyledLabel);
 };
 
 }  // namespace views
 
 #endif  // UI_VIEWS_CONTROLS_STYLED_LABEL_H_