Prepare to enable RenderTextHarfBuzz by default

ui/gfx/render_text_harfbuzz.cc

 // Copyright 2014 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.
 
 #include "ui/gfx/render_text_harfbuzz.h"
 
 #include <map>
 
 #include "base/debug/leak_annotations.h"
 #include "base/i18n/bidi_line_iterator.h"
@@ skipping 613 matching lines @@
           pos = begin;
           break;
         }
         pos = iter.pos() - iter.GetString().length();
       }
     }
   }
   return SelectionModel(pos, CURSOR_FORWARD);
 }
 
 Range RenderTextHarfBuzz::GetGlyphBounds(size_t index) {

[msw, 2014/07/26 02:54:14]

nit: reorder this definition to match its declaration order.

[ckocagil, 2014/07/26 03:55:59]

Done.

+  EnsureLayout();
   const size_t run_index =
       GetRunContainingCaret(SelectionModel(index, CURSOR_FORWARD));
   // Return edge bounds if the index is invalid or beyond the layout text size.
   if (run_index >= runs_.size())
     return Range(GetStringSize().width());
   const size_t layout_index = TextIndexToLayoutIndex(index);
   return Range(runs_[run_index]->GetGlyphXBoundary(layout_index, false),
                runs_[run_index]->GetGlyphXBoundary(layout_index, true));
 }
 
@@ skipping 392 matching lines @@
     run->positions[i].set(run->width + x_offset, -y_offset);
     run->width +=
         SkScalarRoundToInt(SkFixedToScalar(hb_positions[i].x_advance));
   }
 
   hb_buffer_destroy(buffer);
   hb_font_destroy(harfbuzz_font);
 }
 
 }  // namespace gfx