Bug 1667475 - Remove no-longer-used 'aOutPainted' parameters. r=kats

It looks like these are unused as of bug 1629466.

Differential Revision: https://phabricator.services.mozilla.com/D92013
This commit is contained in:
Botond Ballo 2020-10-03 21:07:41 +00:00
Родитель 5d6a075ae9
Коммит 3665360cf1
2 изменённых файлов: 13 добавлений и 25 удалений

Просмотреть файл

@ -368,8 +368,7 @@ enum class MaxSizeExceededBehaviour {
static bool GetDisplayPortImpl(
nsIContent* aContent, nsRect* aResult, float aMultiplier,
MaxSizeExceededBehaviour aBehaviour = MaxSizeExceededBehaviour::Assert,
bool* aOutPainted = nullptr) {
MaxSizeExceededBehaviour aBehaviour = MaxSizeExceededBehaviour::Assert) {
DisplayPortPropertyData* rectData = nullptr;
DisplayPortMarginsPropertyData* marginsData = nullptr;
@ -377,10 +376,6 @@ static bool GetDisplayPortImpl(
return false;
}
if (aOutPainted) {
*aOutPainted = rectData ? rectData->mPainted : marginsData->mPainted;
}
nsIFrame* frame = aContent->GetPrimaryFrame();
if (frame && !frame->PresShell()->AsyncPanZoomEnabled()) {
return false;
@ -444,14 +439,13 @@ static void TranslateFromScrollPortToScrollFrame(nsIContent* aContent,
bool DisplayPortUtils::GetDisplayPort(
nsIContent* aContent, nsRect* aResult,
DisplayportRelativeTo aRelativeTo /* = DisplayportRelativeTo::ScrollPort */,
bool* aOutPainted /* = nullptr */) {
DisplayportRelativeTo
aRelativeTo /* = DisplayportRelativeTo::ScrollPort */) {
float multiplier = StaticPrefs::layers_low_precision_buffer()
? 1.0f / StaticPrefs::layers_low_precision_resolution()
: 1.0f;
bool usingDisplayPort =
GetDisplayPortImpl(aContent, aResult, multiplier,
MaxSizeExceededBehaviour::Assert, aOutPainted);
bool usingDisplayPort = GetDisplayPortImpl(aContent, aResult, multiplier,
MaxSizeExceededBehaviour::Assert);
if (aResult && usingDisplayPort &&
aRelativeTo == DisplayportRelativeTo::ScrollFrame) {
TranslateFromScrollPortToScrollFrame(aContent, aResult);
@ -694,11 +688,10 @@ void DisplayPortUtils::SetDisplayPortBaseIfNotSet(nsIContent* aContent,
}
bool DisplayPortUtils::GetCriticalDisplayPort(nsIContent* aContent,
nsRect* aResult,
bool* aOutPainted) {
nsRect* aResult) {
if (StaticPrefs::layers_low_precision_buffer()) {
return GetDisplayPortImpl(aContent, aResult, 1.0f,
MaxSizeExceededBehaviour::Assert, aOutPainted);
MaxSizeExceededBehaviour::Assert);
}
return false;
}
@ -708,13 +701,11 @@ bool DisplayPortUtils::HasCriticalDisplayPort(nsIContent* aContent) {
}
bool DisplayPortUtils::GetHighResolutionDisplayPort(nsIContent* aContent,
nsRect* aResult,
bool* aOutPainted) {
nsRect* aResult) {
if (StaticPrefs::layers_low_precision_buffer()) {
return GetCriticalDisplayPort(aContent, aResult, aOutPainted);
return GetCriticalDisplayPort(aContent, aResult);
}
return GetDisplayPort(aContent, aResult, DisplayportRelativeTo::ScrollPort,
aOutPainted);
return GetDisplayPort(aContent, aResult, DisplayportRelativeTo::ScrollPort);
}
void DisplayPortUtils::RemoveDisplayPort(nsIContent* aContent) {

Просмотреть файл

@ -51,8 +51,7 @@ class DisplayPortUtils {
*/
static bool GetDisplayPort(
nsIContent* aContent, nsRect* aResult,
DisplayportRelativeTo aRelativeTo = DisplayportRelativeTo::ScrollPort,
bool* aOutPainted = nullptr);
DisplayportRelativeTo aRelativeTo = DisplayportRelativeTo::ScrollPort);
/**
* Check whether the given element has a displayport.
@ -148,8 +147,7 @@ class DisplayPortUtils {
/**
* Get the critical display port for the given element.
*/
static bool GetCriticalDisplayPort(nsIContent* aContent, nsRect* aResult,
bool* aOutPainted = nullptr);
static bool GetCriticalDisplayPort(nsIContent* aContent, nsRect* aResult);
/**
* Check whether the given element has a critical display port.
@ -161,8 +159,7 @@ class DisplayPortUtils {
* GetCriticalDisplayPort. Otherwise, delegates to GetDisplayPort.
*/
static bool GetHighResolutionDisplayPort(nsIContent* aContent,
nsRect* aResult,
bool* aOutPainted = nullptr);
nsRect* aResult);
/**
* Remove the displayport for the given element.