Rename m_fontSize *variable* to currentFontSize.

Rename m_fontSize *variable* to currentFontSize.

This is needed to avoid a conflict after automatic renaming done by
rewrite_to_chrome_style clang tool that would rename 2 local variables
to the same new name.  Specifically, the tool would:
- Rename |fontSize| variable on line 65 to |font_size|
- Rename |m_fontSize| variable on line 72 to |font_size|

This CL preemptively avoids the problem described above, by manually
renaming |m_fontSize| variable to |currentFontSize|.

BUG=641004
Committed: https://crrev.com/0fa434091907b99a0fa6fb96b5b85ca1a2519d25
Cr-Commit-Position: refs/heads/master@{#441520}

[Łukasz Anforowicz, 2017-01-04 22:20:44]

The CQ bit was checked by lukasza@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-01-04 22:21:27]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Łukasz Anforowicz, 2017-01-04 22:21:38]

lukasza@chromium.org changed reviewers:
+ cbiesinger@chromium.org

[Łukasz Anforowicz, 2017-01-04 22:21:39]

cbiesinger@, could you PTAL?

[cbiesinger, 2017-01-04 22:30:42]

lgtm

https://codereview.chromium.org/2612093003/diff/1/third_party/WebKit/Source/c...
File third_party/WebKit/Source/core/layout/LayoutTextTrackContainer.cpp (right):

https://codereview.chromium.org/2612093003/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/layout/LayoutTextTrackContainer.cpp:72: volatile
float& currentFontSize = this->m_fontSize;
I don't think you need the "this->" part anymore

[Łukasz Anforowicz, 2017-01-04 22:35:25]

Thanks.

https://codereview.chromium.org/2612093003/diff/1/third_party/WebKit/Source/c...
File third_party/WebKit/Source/core/layout/LayoutTextTrackContainer.cpp (right):

https://codereview.chromium.org/2612093003/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/layout/LayoutTextTrackContainer.cpp:72: volatile
float& currentFontSize = this->m_fontSize;
On 2017/01/04 22:30:42, cbiesinger wrote:
> I don't think you need the "this->" part anymore

Good point.  Done.

[Łukasz Anforowicz, 2017-01-04 22:35:28]

The CQ bit was checked by lukasza@chromium.org

[Łukasz Anforowicz, 2017-01-04 22:35:29]

The patchset sent to the CQ was uploaded after l-g-t-m from
cbiesinger@chromium.org
Link to the patchset: https://codereview.chromium.org/2612093003/#ps20001
(title: "Remove the no longer needed |this->| part.")

[commit-bot: I haz the power, 2017-01-04 22:35:57]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-01-04 23:56:31]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1483569328787370,
"parent_rev": "67f118e6cf08cf5edd07ea458b8b4988313c08bf", "commit_rev":
"4f19c7aa4c9eb98002bd5d538057a4d24868025d"}

[commit-bot: I haz the power, 2017-01-04 23:56:56]

Description was changed from

==========
Rename m_fontSize *variable* to currentFontSize.

This is needed to avoid a conflict after automatic renaming done by
rewrite_to_chrome_style clang tool that would rename 2 local variables
to the same new name.  Specifically, the tool would:
- Rename |fontSize| variable on line 65 to |font_size|
- Rename |m_fontSize| variable on line 72 to |font_size|

This CL preemptively avoids the problem described above, by manually
renaming |m_fontSize| variable to |currentFontSize|.

BUG=641004
==========

to

==========
Rename m_fontSize *variable* to currentFontSize.

This is needed to avoid a conflict after automatic renaming done by
rewrite_to_chrome_style clang tool that would rename 2 local variables
to the same new name.  Specifically, the tool would:
- Rename |fontSize| variable on line 65 to |font_size|
- Rename |m_fontSize| variable on line 72 to |font_size|

This CL preemptively avoids the problem described above, by manually
renaming |m_fontSize| variable to |currentFontSize|.

BUG=641004

Review-Url: https://codereview.chromium.org/2612093003
==========

[commit-bot: I haz the power, 2017-01-04 23:56:57]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2017-01-04 23:59:20]

Description was changed from

==========
Rename m_fontSize *variable* to currentFontSize.

This is needed to avoid a conflict after automatic renaming done by
rewrite_to_chrome_style clang tool that would rename 2 local variables
to the same new name.  Specifically, the tool would:
- Rename |fontSize| variable on line 65 to |font_size|
- Rename |m_fontSize| variable on line 72 to |font_size|

This CL preemptively avoids the problem described above, by manually
renaming |m_fontSize| variable to |currentFontSize|.

BUG=641004

Review-Url: https://codereview.chromium.org/2612093003
==========

to

==========
Rename m_fontSize *variable* to currentFontSize.

This is needed to avoid a conflict after automatic renaming done by
rewrite_to_chrome_style clang tool that would rename 2 local variables
to the same new name.  Specifically, the tool would:
- Rename |fontSize| variable on line 65 to |font_size|
- Rename |m_fontSize| variable on line 72 to |font_size|

This CL preemptively avoids the problem described above, by manually
renaming |m_fontSize| variable to |currentFontSize|.

BUG=641004

Committed: https://crrev.com/0fa434091907b99a0fa6fb96b5b85ca1a2519d25
Cr-Commit-Position: refs/heads/master@{#441520}
==========

[commit-bot: I haz the power, 2017-01-04 23:59:21]

Patchset 2 (id:??) landed as
https://crrev.com/0fa434091907b99a0fa6fb96b5b85ca1a2519d25
Cr-Commit-Position: refs/heads/master@{#441520}