Title: [183187] trunk/Source/WebCore
Revision
183187
Author
[email protected]
Date
2015-04-23 08:58:16 -0700 (Thu, 23 Apr 2015)

Log 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.

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (183186 => 183187)


--- trunk/Source/WebCore/ChangeLog	2015-04-23 15:52:20 UTC (rev 183186)
+++ trunk/Source/WebCore/ChangeLog	2015-04-23 15:58:16 UTC (rev 183187)
@@ -1,3 +1,16 @@
+2015-04-23  Eric Carlson  <[email protected]>
+
+        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  <[email protected]>
 
         Use std::unique_ptr instead of OwnPtr in ThreadGlobalData

Modified: trunk/Source/WebCore/dom/Document.cpp (183186 => 183187)


--- trunk/Source/WebCore/dom/Document.cpp	2015-04-23 15:52:20 UTC (rev 183186)
+++ trunk/Source/WebCore/dom/Document.cpp	2015-04-23 15:58:16 UTC (rev 183187)
@@ -6550,7 +6550,7 @@
 
 void Document::removePlaybackTargetPickerClient(MediaPlaybackTargetClient& client)
 {
-    const auto& it = m_clientToIDMap.find(&client);
+    auto it = m_clientToIDMap.find(&client);
     if (it == m_clientToIDMap.end())
         return;
 
@@ -6570,7 +6570,7 @@
     if (!page)
         return;
 
-    const auto& it = m_clientToIDMap.find(&client);
+    auto it = m_clientToIDMap.find(&client);
     ASSERT(it != m_clientToIDMap.end());
     if (it == m_clientToIDMap.end())
         return;
@@ -6584,7 +6584,7 @@
     if (!page)
         return;
 
-    const auto& it = m_clientToIDMap.find(&client);
+    auto it = m_clientToIDMap.find(&client);
     ASSERT(it != m_clientToIDMap.end());
     if (it == m_clientToIDMap.end())
         return;
@@ -6594,7 +6594,7 @@
 
 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;
 
@@ -6603,7 +6603,7 @@
 
 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;
 
@@ -6612,7 +6612,7 @@
 
 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;
 
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to