nagai/fune
3
0
Fork 0
forked from mirrors/gecko-dev
Code Pull requests Activity

Bug 1864896: Autofix unused function arguments (layout). r=dshin

Browse source 
Differential Revision: https://phabricator.services.mozilla.com/D202971
This commit is contained in:
Dave Townsend 2024-03-02 15:26:01 +00:00
parent 19b0f6c16b
commit cd02013a7c
3 changed files with 16 additions and 20 deletions
repo.diff.show_diff_stats Download patch file Download diff file Expand all files Collapse all files

2
layout/tools/reftest/api.js
View file

@ -110,7 +110,7 @@ this.reftest = class extends ExtensionAPI {
// find it by its type nor will domwindowcreated be fired. // find it by its type nor will domwindowcreated be fired.
// So we listen to either initial-document-element-inserted which // So we listen to either initial-document-element-inserted which
// indicates when it's okay to search for the main window by type again. // indicates when it's okay to search for the main window by type again.
Services.obs.addObserver(function observer(aSubject, aTopic, aData) { Services.obs.addObserver(function observer(aSubject, aTopic) {
Services.obs.removeObserver(observer, aTopic); Services.obs.removeObserver(observer, aTopic);
startAndroid(GetMainWindow()); startAndroid(GetMainWindow());
}, "initial-document-element-inserted"); }, "initial-document-element-inserted");

20
layout/tools/reftest/reftest-content.js
View file

@ -83,7 +83,7 @@ function IDForEventTarget(event) {
} }
var progressListener = { var progressListener = {
onStateChange(webprogress, request, flags, status) { onStateChange(webprogress, request, flags) {
let uri; let uri;
try { try {
request.QueryInterface(Ci.nsIChannel); request.QueryInterface(Ci.nsIChannel);
@ -286,7 +286,7 @@ function setupViewport(contentRootElement) {
// XXX support viewconfig when needed // XXX support viewconfig when needed
} }
function setupDisplayport(contentRootElement) { function setupDisplayport() {
let promise = content.windowGlobalChild let promise = content.windowGlobalChild
.getActor("ReftestFission") .getActor("ReftestFission")
.SetupDisplayportRoot(); .SetupDisplayportRoot();
@ -896,7 +896,7 @@ function WaitForTestEnd(
LogInfo("MakeProgress: STATE_WAITING_FOR_APZ_FLUSH"); LogInfo("MakeProgress: STATE_WAITING_FOR_APZ_FLUSH");
gFailureReason = "timed out waiting for APZ flush to complete"; gFailureReason = "timed out waiting for APZ flush to complete";
var flushWaiter = function (aSubject, aTopic, aData) { var flushWaiter = function (aSubject, aTopic) {
if (aTopic) { if (aTopic) {
LogInfo("MakeProgress: apz-repaints-flushed fired"); LogInfo("MakeProgress: apz-repaints-flushed fired");
} }
@ -1357,9 +1357,7 @@ function SynchronizeForSnapshot(flags) {
// Setup async scroll offsets now, because any scrollable layers should // Setup async scroll offsets now, because any scrollable layers should
// have had their AsyncPanZoomControllers created. // have had their AsyncPanZoomControllers created.
return setupAsyncScrollOffsets({ allowFailure: false }).then(function ( return setupAsyncScrollOffsets({ allowFailure: false }).then(function () {
result
) {
setupAsyncZoom({ allowFailure: false }); setupAsyncZoom({ allowFailure: false });
}); });
}, },
@ -1370,9 +1368,7 @@ function SynchronizeForSnapshot(flags) {
// Setup async scroll offsets now, because any scrollable layers should // Setup async scroll offsets now, because any scrollable layers should
// have had their AsyncPanZoomControllers created. // have had their AsyncPanZoomControllers created.
return setupAsyncScrollOffsets({ allowFailure: false }).then(function ( return setupAsyncScrollOffsets({ allowFailure: false }).then(function () {
result
) {
setupAsyncZoom({ allowFailure: false }); setupAsyncZoom({ allowFailure: false });
}); });
} }
@ -1380,7 +1376,7 @@ function SynchronizeForSnapshot(flags) {
} }
function RegisterMessageListeners() { function RegisterMessageListeners() {
addMessageListener("reftest:Clear", function (m) { addMessageListener("reftest:Clear", function () {
RecvClear(); RecvClear();
}); });
addMessageListener("reftest:LoadScriptTest", function (m) { addMessageListener("reftest:LoadScriptTest", function (m) {
@ -1392,7 +1388,7 @@ function RegisterMessageListeners() {
addMessageListener("reftest:LoadTest", function (m) { addMessageListener("reftest:LoadTest", function (m) {
RecvLoadTest(m.json.type, m.json.uri, m.json.uriTargetType, m.json.timeout); RecvLoadTest(m.json.type, m.json.uri, m.json.uriTargetType, m.json.timeout);
}); });
addMessageListener("reftest:ResetRenderingState", function (m) { addMessageListener("reftest:ResetRenderingState", function () {
RecvResetRenderingState(); RecvResetRenderingState();
}); });
addMessageListener("reftest:PrintDone", function (m) { addMessageListener("reftest:PrintDone", function (m) {
@ -1552,7 +1548,7 @@ function SendPrintResult(runtimeMs, status, fileName) {
}); });
} }
function SendExpectProcessCrash(runtimeMs) { function SendExpectProcessCrash() {
sendAsyncMessage("reftest:ExpectProcessCrash"); sendAsyncMessage("reftest:ExpectProcessCrash");
} }

14
layout/tools/reftest/reftest.sys.mjs
View file

@ -1770,19 +1770,19 @@ function RegisterMessageListenersAndLoadContentScript(aReload) {
); );
g.browserMessageManager.addMessageListener( g.browserMessageManager.addMessageListener(
"reftest:FailedNoPaint", "reftest:FailedNoPaint",
function (m) { function () {
RecvFailedNoPaint(); RecvFailedNoPaint();
} }
); );
g.browserMessageManager.addMessageListener( g.browserMessageManager.addMessageListener(
"reftest:FailedNoDisplayList", "reftest:FailedNoDisplayList",
function (m) { function () {
RecvFailedNoDisplayList(); RecvFailedNoDisplayList();
} }
); );
g.browserMessageManager.addMessageListener( g.browserMessageManager.addMessageListener(
"reftest:FailedDisplayList", "reftest:FailedDisplayList",
function (m) { function () {
RecvFailedDisplayList(); RecvFailedDisplayList();
} }
); );
@ -1800,7 +1800,7 @@ function RegisterMessageListenersAndLoadContentScript(aReload) {
); );
g.browserMessageManager.addMessageListener( g.browserMessageManager.addMessageListener(
"reftest:InitCanvasWithSnapshot", "reftest:InitCanvasWithSnapshot",
function (m) { function () {
RecvInitCanvasWithSnapshot(); RecvInitCanvasWithSnapshot();
} }
); );
@ -1836,13 +1836,13 @@ function RegisterMessageListenersAndLoadContentScript(aReload) {
); );
g.browserMessageManager.addMessageListener( g.browserMessageManager.addMessageListener(
"reftest:UpdateWholeCanvasForInvalidation", "reftest:UpdateWholeCanvasForInvalidation",
function (m) { function () {
RecvUpdateWholeCanvasForInvalidation(); RecvUpdateWholeCanvasForInvalidation();
} }
); );
g.browserMessageManager.addMessageListener( g.browserMessageManager.addMessageListener(
"reftest:ExpectProcessCrash", "reftest:ExpectProcessCrash",
function (m) { function () {
RecvExpectProcessCrash(); RecvExpectProcessCrash();
} }
); );
@ -2018,7 +2018,7 @@ async function RecvUpdateWholeCanvasForInvalidation() {
SendUpdateCurrentCanvasWithSnapshotDone(true); SendUpdateCurrentCanvasWithSnapshotDone(true);
} }
function OnProcessCrashed(subject, topic, data) { function OnProcessCrashed(subject, topic) {
let id; let id;
let additionalDumps; let additionalDumps;
let propbag = subject.QueryInterface(Ci.nsIPropertyBag2); let propbag = subject.QueryInterface(Ci.nsIPropertyBag2);