Bug 1613431 - Part 3: Handle synced setters return value. r=nika

Depends on D83646 Differential Revision: https://phabricator.services.mozilla.com/D83647
2020-07-31 13:37:13 +00:00
parent a531ef1d94
commit b447c24f6c
23 changed files with 136 additions and 108 deletions
--- a/dom/html/HTMLIFrameElement.cpp
+++ b/dom/html/HTMLIFrameElement.cpp
@@ -187,8 +187,8 @@ nsresult HTMLIFrameElement::AfterSetAttr(int32_t aNameSpaceID, nsAtom* aName,
               aName == nsGkAtoms::mozallowfullscreen) {
      if (mFrameLoader) {
        if (auto* bc = mFrameLoader->GetExtantBrowsingContext()) {
-          // This can go away once we remove the featurePolicy pref.
-          bc->SetFullscreenAllowedByOwner(AllowFullscreen());
+          MOZ_ALWAYS_SUCCEEDS(
+              bc->SetFullscreenAllowedByOwner(AllowFullscreen()));
        }
      }
    }