Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(475)

Side by Side Diff: Source/web/WebMediaPlayerClientImpl.cpp

Issue 1186853004: Move paint invalidation up into HTMLMediaElement::clearMediaPlayer(). (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Created 5 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « Source/core/html/HTMLMediaElement.cpp ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2009 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2009 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "config.h" 5 #include "config.h"
6 #include "web/WebMediaPlayerClientImpl.h" 6 #include "web/WebMediaPlayerClientImpl.h"
7 7
8 #include "core/frame/LocalFrame.h" 8 #include "core/frame/LocalFrame.h"
9 #include "core/html/HTMLMediaElement.h" 9 #include "core/html/HTMLMediaElement.h"
10 #include "core/html/TimeRanges.h" 10 #include "core/html/TimeRanges.h"
(...skipping 41 matching lines...) Expand 10 before | Expand all | Expand 10 after
52 { 52 {
53 return m_webMediaPlayer.get(); 53 return m_webMediaPlayer.get();
54 } 54 }
55 55
56 // WebMediaPlayerClient -------------------------------------------------------- 56 // WebMediaPlayerClient --------------------------------------------------------
57 57
58 WebMediaPlayerClientImpl::~WebMediaPlayerClientImpl() 58 WebMediaPlayerClientImpl::~WebMediaPlayerClientImpl()
59 { 59 {
60 // Explicitly destroy the WebMediaPlayer to allow verification of tear down. 60 // Explicitly destroy the WebMediaPlayer to allow verification of tear down.
61 m_webMediaPlayer.clear(); 61 m_webMediaPlayer.clear();
62
63 if (mediaElement().layoutObject())
64 mediaElement().layoutObject()->setShouldDoFullPaintInvalidation();
65 } 62 }
66 63
67 void WebMediaPlayerClientImpl::networkStateChanged() 64 void WebMediaPlayerClientImpl::networkStateChanged()
68 { 65 {
69 m_client->mediaPlayerNetworkStateChanged(); 66 m_client->mediaPlayerNetworkStateChanged();
70 } 67 }
71 68
72 void WebMediaPlayerClientImpl::readyStateChanged() 69 void WebMediaPlayerClientImpl::readyStateChanged()
73 { 70 {
74 m_client->mediaPlayerReadyStateChanged(); 71 m_client->mediaPlayerReadyStateChanged();
(...skipping 233 matching lines...) Expand 10 before | Expand all | Expand 10 after
308 } 305 }
309 306
310 DEFINE_TRACE(WebMediaPlayerClientImpl::AudioClientImpl) 307 DEFINE_TRACE(WebMediaPlayerClientImpl::AudioClientImpl)
311 { 308 {
312 visitor->trace(m_client); 309 visitor->trace(m_client);
313 } 310 }
314 311
315 #endif 312 #endif
316 313
317 } // namespace blink 314 } // namespace blink
OLDNEW
« no previous file with comments | « Source/core/html/HTMLMediaElement.cpp ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698