RenderText: Allow strike-through line thickness to be customized.

chrome/browser/android/vr_shell/textures/render_text_wrapper.cc

 // Copyright 2017 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 "chrome/browser/android/vr_shell/textures/render_text_wrapper.h"
 
 namespace vr_shell {
 
 RenderTextWrapper::RenderTextWrapper(gfx::RenderText* render_text)
     : render_text_(render_text) {}
@@ skipping 11 matching lines @@
 void RenderTextWrapper::SetStyle(gfx::TextStyle style, bool value) {
   render_text_->SetStyle(style, value);
 }
 
 void RenderTextWrapper::ApplyStyle(gfx::TextStyle style,
                                    bool value,
                                    const gfx::Range& range) {
   render_text_->ApplyStyle(style, value, range);
 }
 
+void RenderTextWrapper::set_strike_thickness_factor(SkScalar f) {

[msw, 2017/07/05 19:55:00]

Ah, as this is defined out-of-line with a 'non-trivial' body, this function
should have a CamelCase name, SetStrikeThicknessFactor, and the |f| param can be
renamed |factor| for clarity.

[cjgrant, 2017/07/05 20:44:43]

Done. Heh, back to how it was. ;)

[msw, 2017/07/05 20:51:55]

Yeah, thanks for addressing my erratic comments, and sorry for churn.

+  render_text_->set_strike_thickness_factor(f);
+}
+
 }  // namespace vr_shell