Chromium Code Reviews| Index: third_party/WebKit/Source/core/layout/LayoutTextTrackContainer.cpp |
| diff --git a/third_party/WebKit/Source/core/layout/LayoutTextTrackContainer.cpp b/third_party/WebKit/Source/core/layout/LayoutTextTrackContainer.cpp |
| index fd075d22f90cae58c0aae7ffcdafaa6007a50768..d4cce8ae60e6f5c30b9c7603cf3ac3a1b84925a5 100644 |
| --- a/third_party/WebKit/Source/core/layout/LayoutTextTrackContainer.cpp |
| +++ b/third_party/WebKit/Source/core/layout/LayoutTextTrackContainer.cpp |
| @@ -69,10 +69,10 @@ bool LayoutTextTrackContainer::updateSizes( |
| // the behavior is currently not portable. fontSize may have precision higher |
| // than m_fontSize thus straight comparison can fail despite they cast to the |
| // same float value. |
| - volatile float& m_fontSize = this->m_fontSize; |
| - float oldFontSize = m_fontSize; |
| - m_fontSize = fontSize; |
| - return m_fontSize != oldFontSize; |
| + volatile float& currentFontSize = this->m_fontSize; |
|
cbiesinger
2017/01/04 22:30:42
I don't think you need the "this->" part anymore
Łukasz Anforowicz
2017/01/04 22:35:25
Good point. Done.
|
| + float oldFontSize = currentFontSize; |
| + currentFontSize = fontSize; |
| + return currentFontSize != oldFontSize; |
| } |
| } // namespace blink |