Title: [220454] trunk/Source/WebCore
- Revision
- 220454
- Author
- commit-qu...@webkit.org
- Date
- 2017-08-09 06:06:11 -0700 (Wed, 09 Aug 2017)
Log Message
Remove unnecessary copies of PlatformMediaSessionManager::m_sessions
https://bugs.webkit.org/show_bug.cgi?id=175375
Patch by Ms2ger <ms2...@igalia.com> on 2017-08-09
Reviewed by Sam Weinig.
Remove unnecessary copies of PlatformMediaSessionManager::m_sessions.
These should have been removed in r207688.
No new tests: no behavior change.
* platform/audio/PlatformMediaSessionManager.cpp:
(WebCore::PlatformMediaSessionManager::applicationWillBecomeInactive const):
(WebCore::PlatformMediaSessionManager::applicationDidBecomeActive const):
(WebCore::PlatformMediaSessionManager::applicationDidEnterBackground const):
(WebCore::PlatformMediaSessionManager::applicationWillEnterForeground const):
Modified Paths
Diff
Modified: trunk/Source/WebCore/ChangeLog (220453 => 220454)
--- trunk/Source/WebCore/ChangeLog 2017-08-09 13:04:39 UTC (rev 220453)
+++ trunk/Source/WebCore/ChangeLog 2017-08-09 13:06:11 UTC (rev 220454)
@@ -1,3 +1,22 @@
+2017-08-09 Ms2ger <ms2...@igalia.com>
+
+ Remove unnecessary copies of PlatformMediaSessionManager::m_sessions
+ https://bugs.webkit.org/show_bug.cgi?id=175375
+
+ Reviewed by Sam Weinig.
+
+ Remove unnecessary copies of PlatformMediaSessionManager::m_sessions.
+
+ These should have been removed in r207688.
+
+ No new tests: no behavior change.
+
+ * platform/audio/PlatformMediaSessionManager.cpp:
+ (WebCore::PlatformMediaSessionManager::applicationWillBecomeInactive const):
+ (WebCore::PlatformMediaSessionManager::applicationDidBecomeActive const):
+ (WebCore::PlatformMediaSessionManager::applicationDidEnterBackground const):
+ (WebCore::PlatformMediaSessionManager::applicationWillEnterForeground const):
+
2017-08-09 Chris Dumez <cdu...@apple.com>
Reinstate active flag for iterators
Modified: trunk/Source/WebCore/platform/audio/PlatformMediaSessionManager.cpp (220453 => 220454)
--- trunk/Source/WebCore/platform/audio/PlatformMediaSessionManager.cpp 2017-08-09 13:04:39 UTC (rev 220453)
+++ trunk/Source/WebCore/platform/audio/PlatformMediaSessionManager.cpp 2017-08-09 13:06:11 UTC (rev 220454)
@@ -298,7 +298,6 @@
{
LOG(Media, "PlatformMediaSessionManager::applicationWillBecomeInactive");
- Vector<PlatformMediaSession*> sessions = m_sessions;
forEachSession([&] (PlatformMediaSession& session, size_t) {
if (m_restrictions[session.mediaType()] & InactiveProcessPlaybackRestricted)
session.beginInterruption(PlatformMediaSession::ProcessInactive);
@@ -309,7 +308,6 @@
{
LOG(Media, "PlatformMediaSessionManager::applicationDidBecomeActive");
- Vector<PlatformMediaSession*> sessions = m_sessions;
forEachSession([&] (PlatformMediaSession& session, size_t) {
if (m_restrictions[session.mediaType()] & InactiveProcessPlaybackRestricted)
session.endInterruption(PlatformMediaSession::MayResumePlaying);
@@ -324,8 +322,7 @@
return;
m_isApplicationInBackground = true;
-
- Vector<PlatformMediaSession*> sessions = m_sessions;
+
forEachSession([&] (PlatformMediaSession& session, size_t) {
if (suspendedUnderLock && m_restrictions[session.mediaType()] & SuspendedUnderLockPlaybackRestricted)
session.beginInterruption(PlatformMediaSession::SuspendedUnderLock);
@@ -343,7 +340,6 @@
m_isApplicationInBackground = false;
- Vector<PlatformMediaSession*> sessions = m_sessions;
forEachSession([&] (PlatformMediaSession& session, size_t) {
if ((suspendedUnderLock && m_restrictions[session.mediaType()] & SuspendedUnderLockPlaybackRestricted) || m_restrictions[session.mediaType()] & BackgroundProcessPlaybackRestricted)
session.endInterruption(PlatformMediaSession::MayResumePlaying);
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes