corysanin/tubestation
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Backed out 2 changesets (bug 1588791) for causing test_bug369370.html and test_transformed_scrolling_repaints_3.html to permafail

Browse Source 
Backed out changeset 52e661ff161d (bug 1588791)
Backed out changeset d59e691bda9e (bug 1588791)
This commit is contained in:
Ciure Andrei
2020-01-13 04:49:16 +02:00
parent 9ea8c6570f
commit dcf9890bea
19 changed files with 54 additions and 80 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
dom/html/nsGenericHTMLFrameElement.cpp
View File

@@ -8,7 +8,6 @@
#include "mozilla/dom/HTMLIFrameElement.h"
#include "mozilla/dom/XULFrameElement.h"
#include "mozilla/dom/BrowserBridgeChild.h"
#include "mozilla/dom/WindowProxyHolder.h"
#include "mozilla/Preferences.h"
#include "mozilla/PresShell.h"
@@ -292,15 +291,10 @@ nsresult nsGenericHTMLFrameElement::AfterSetAttr(
if (aNameSpaceID == kNameSpaceID_None) {
if (aName == nsGkAtoms::scrolling) {
if (mFrameLoader) {
ScrollbarPreference pref = MapScrollingAttribute(aValue);
// FIXME(bug 1588791): This should work for fission iframes.
if (nsIDocShell* docshell = mFrameLoader->GetExistingDocShell()) {
nsDocShell::Cast(docshell)->SetScrollbarPreference(pref);
} else if (auto* child = mFrameLoader->GetBrowserBridgeChild()) {
// NOTE(emilio): We intentionally don't deal with the
// GetBrowserParent() case, and only deal with the fission iframe
// case. There's no reason not to make it work, but it's a bit of
// boilerplate for something that we don't use...
child->SendScrollbarPreferenceChanged(pref);
nsDocShell::Cast(docshell)->SetScrollbarPreference(
MapScrollingAttribute(aValue));
}
}
} else if (aName == nsGkAtoms::mozbrowser) {