Changeset 273797 in webkit

Timestamp:

Mar 2, 2021, 7:08:58 PM (6 years ago)

Author:

Alan Coon

Message:

Cherry-pick r273499. rdar://problem/74952752

Avoid setting page muted state if unchanged
​https://bugs.webkit.org/show_bug.cgi?id=222395

Reviewed by Brent Fulgham.

Return early in Page::setMuted if muted state did not change.

No new tests, covered by existing tests.

• page/Page.cpp: (WebCore::Page::setMuted):

git-svn-id: ​https://svn.webkit.org/repository/webkit/trunk@273499 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Location:

branches/safari-612.1.5-branch/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• page/Page.cpp (modified) (1 diff)

branches/safari-612.1.5-branch/Source/WebCore/ChangeLog

@@ +1 @@
+2021-03-02  Alan Coon  <alancoon@apple.com>
+
+        Cherry-pick r273499. rdar://problem/74952752
+
+    Avoid setting page muted state if unchanged
+    https://bugs.webkit.org/show_bug.cgi?id=222395
+    
+    Reviewed by Brent Fulgham.
+    
+    Return early in Page::setMuted if muted state did not change.
+    
+    No new tests, covered by existing tests.
+    
+    * page/Page.cpp:
+    (WebCore::Page::setMuted):
+    
+    
+    git-svn-id: https://svn.webkit.org/repository/webkit/trunk@273499 268f45cc-cd09-0410-ab3c-d52691b4dbfc
+
+    2021-02-25  Per Arne Vollan  <pvollan@apple.com>
+
+            Avoid setting page muted state if unchanged
+            https://bugs.webkit.org/show_bug.cgi?id=222395
+
+            Reviewed by Brent Fulgham.
+
+            Return early in Page::setMuted if muted state did not change.
+
+            No new tests, covered by existing tests.
+
+            * page/Page.cpp:
+            (WebCore::Page::setMuted):
+
 2021-03-02  Alan Coon  <alancoon@apple.com>
 

branches/safari-612.1.5-branch/Source/WebCore/page/Page.cpp

@@ -2108 +2108 @@
 void Page::setMuted(MediaProducer::MutedStateFlags muted)
 {
+    if (m_mutedState == muted)
+        return;
+
     m_mutedState = muted;