Chromium Code Reviews| OLD | NEW |
|---|---|
| 1 /* | 1 /* |
| 2 * Copyright (C) 2011, 2012 Apple Inc. All rights reserved. | 2 * Copyright (C) 2011, 2012 Apple Inc. All rights reserved. |
| 3 * Copyright (C) 2011, 2012 Google Inc. All rights reserved. | 3 * Copyright (C) 2011, 2012 Google Inc. All rights reserved. |
| 4 * | 4 * |
| 5 * Redistribution and use in source and binary forms, with or without | 5 * Redistribution and use in source and binary forms, with or without |
| 6 * modification, are permitted provided that the following conditions | 6 * modification, are permitted provided that the following conditions |
| 7 * are met: | 7 * are met: |
| 8 * 1. Redistributions of source code must retain the above copyright | 8 * 1. Redistributions of source code must retain the above copyright |
| 9 * notice, this list of conditions and the following disclaimer. | 9 * notice, this list of conditions and the following disclaimer. |
| 10 * 2. Redistributions in binary form must reproduce the above copyright | 10 * 2. Redistributions in binary form must reproduce the above copyright |
| (...skipping 166 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
| 177 m_timeline->setPosition(mediaElement().currentTime()); | 177 m_timeline->setPosition(mediaElement().currentTime()); |
| 178 | 178 |
| 179 if (!mediaElement().hasAudio()) | 179 if (!mediaElement().hasAudio()) |
| 180 m_volumeSlider->hide(); | 180 m_volumeSlider->hide(); |
| 181 else | 181 else |
| 182 m_volumeSlider->show(); | 182 m_volumeSlider->show(); |
| 183 updateVolume(); | 183 updateVolume(); |
| 184 | 184 |
| 185 refreshClosedCaptionsButtonVisibility(); | 185 refreshClosedCaptionsButtonVisibility(); |
| 186 | 186 |
| 187 if (mediaElement().hasVideo() && fullscreenIsSupported(document())) | 187 if ((mediaElement().hasVideo() && fullscreenIsSupported(document())) |
| 188 || mediaElement().isFullscreen()) | |
|
xhwang
2015/06/23 20:22:01
Maybe add some comment here. Otherwise it makes pe
eae
2015/06/23 20:30:45
Seconded.
liberato (no reviews please)
2015/06/23 20:37:41
third-ed. i had additional logic that i removed,
| |
| 188 m_fullScreenButton->show(); | 189 m_fullScreenButton->show(); |
| 189 else | 190 else |
| 190 m_fullScreenButton->hide(); | 191 m_fullScreenButton->hide(); |
| 191 | 192 |
| 192 refreshCastButtonVisibility(); | 193 refreshCastButtonVisibility(); |
| 193 makeOpaque(); | 194 makeOpaque(); |
| 194 } | 195 } |
| 195 | 196 |
| 196 LayoutObject* MediaControls::layoutObjectForTextTrackLayout() | 197 LayoutObject* MediaControls::layoutObjectForTextTrackLayout() |
| 197 { | 198 { |
| (...skipping 335 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
| 533 visitor->trace(m_toggleClosedCaptionsButton); | 534 visitor->trace(m_toggleClosedCaptionsButton); |
| 534 visitor->trace(m_fullScreenButton); | 535 visitor->trace(m_fullScreenButton); |
| 535 visitor->trace(m_durationDisplay); | 536 visitor->trace(m_durationDisplay); |
| 536 visitor->trace(m_enclosure); | 537 visitor->trace(m_enclosure); |
| 537 visitor->trace(m_castButton); | 538 visitor->trace(m_castButton); |
| 538 visitor->trace(m_overlayCastButton); | 539 visitor->trace(m_overlayCastButton); |
| 539 HTMLDivElement::trace(visitor); | 540 HTMLDivElement::trace(visitor); |
| 540 } | 541 } |
| 541 | 542 |
| 542 } | 543 } |
| OLD | NEW |