Bug 1862692 - Part 17: Use 1-origin column number type in JS::ReadOnlyCompileOptions. r=iain

Differential Revision: https://phabricator.services.mozilla.com/D193030
This commit is contained in:
Tooru Fujisawa
2023-11-09 11:41:11 +00:00
parent 8b51569f3f
commit 438dbbd9d1
6 changed files with 18 additions and 18 deletions

View File

@@ -20,8 +20,7 @@
#include "jsapi.h"
#include "jsfriendapi.h"
#include "js/Array.h" // JS::GetArrayLength
#include "js/ColumnNumber.h" // #include "js/CompilationAndEvaluation.h"
#include "js/ColumnNumber.h" // JS::ColumnNumberZeroOrigin, JS::ColumnNumberOneOrigin
#include "js/CompilationAndEvaluation.h"
#include "js/CompileOptions.h" // JS::CompileOptions, JS::OwningCompileOptions, JS::DecodeOptions, JS::OwningDecodeOptions, JS::DelazificationOption
#include "js/ContextOptions.h" // JS::ContextOptionsRef
@@ -2332,8 +2331,8 @@ nsresult ScriptLoader::FillCompileOptionsForRequest(
if (aRequest->GetScriptLoadContext()->mIsInline &&
aRequest->GetScriptLoadContext()->GetParserCreated() ==
FROM_PARSER_NETWORK) {
aOptions->setColumn(JS::ColumnNumberZeroOrigin(
aRequest->GetScriptLoadContext()->mColumnNo));
aOptions->setColumn(JS::ColumnNumberOneOrigin(JS::ColumnNumberZeroOrigin(
aRequest->GetScriptLoadContext()->mColumnNo)));
}
aOptions->setIsRunOnce(true);
aOptions->setNoScriptRval(true);