Changeset 183187 in webkit

Timestamp:

Apr 23, 2015, 8:58:16 AM (11 years ago)

Author:

eric.carlson@apple.com

Message:

Unreviewed, make a suggested change I overlooked in Darin's review of
​https://bugs.webkit.org/show_bug.cgi?id=144098

• dom/Document.cpp:

(WebCore::Document::removePlaybackTargetPickerClient): const auto& -> auto
(WebCore::Document::showPlaybackTargetPicker): Ditto.
(WebCore::Document::playbackTargetPickerClientStateDidChange): Ditto.
(WebCore::Document::playbackTargetAvailabilityDidChange): Ditto.
(WebCore::Document::setPlaybackTarget): Ditto.
(WebCore::Document::setShouldPlayToPlaybackTarget): Ditto.

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• dom/Document.cpp (modified) (6 diffs)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2015-04-23  Eric Carlson  <eric.carlson@apple.com>
+
+        Unreviewed, make a suggested change I overlooked in Darin's review of
+        https://bugs.webkit.org/show_bug.cgi?id=144098
+
+        * dom/Document.cpp:
+        (WebCore::Document::removePlaybackTargetPickerClient): const auto& -> auto
+        (WebCore::Document::showPlaybackTargetPicker): Ditto.
+        (WebCore::Document::playbackTargetPickerClientStateDidChange): Ditto.
+        (WebCore::Document::playbackTargetAvailabilityDidChange): Ditto.
+        (WebCore::Document::setPlaybackTarget): Ditto.
+        (WebCore::Document::setShouldPlayToPlaybackTarget): Ditto.
+
 2015-04-23  Joonghun Park  <jh718.park@samsung.com>
 

trunk/Source/WebCore/dom/Document.cpp

@@ -6551 +6551 @@
 void Document::removePlaybackTargetPickerClient(MediaPlaybackTargetClient& client)
 {
-    const auto& it = m_clientToIDMap.find(&client);
+    auto it = m_clientToIDMap.find(&client);
     if (it == m_clientToIDMap.end())
         return;
@@ -6571 +6571 @@
         return;
 
-    const auto& it = m_clientToIDMap.find(&client);
+    auto it = m_clientToIDMap.find(&client);
     ASSERT(it != m_clientToIDMap.end());
     if (it == m_clientToIDMap.end())
@@ -6585 +6585 @@
         return;
 
-    const auto& it = m_clientToIDMap.find(&client);
+    auto it = m_clientToIDMap.find(&client);
     ASSERT(it != m_clientToIDMap.end());
     if (it == m_clientToIDMap.end())
@@ -6595 +6595 @@
 void Document::playbackTargetAvailabilityDidChange(uint64_t clientId, bool available)
 {
-    const auto& it = m_idToClientMap.find(clientId);
+    auto it = m_idToClientMap.find(clientId);
     if (it == m_idToClientMap.end())
         return;
@@ -6604 +6604 @@
 void Document::setPlaybackTarget(uint64_t clientId, Ref<MediaPlaybackTarget>&& target)
 {
-    const auto& it = m_idToClientMap.find(clientId);
+    auto it = m_idToClientMap.find(clientId);
     if (it == m_idToClientMap.end())
         return;
@@ -6613 +6613 @@
 void Document::setShouldPlayToPlaybackTarget(uint64_t clientId, bool shouldPlay)
 {
-    const auto& it = m_idToClientMap.find(clientId);
+    auto it = m_idToClientMap.find(clientId);
     if (it == m_idToClientMap.end())
         return;