LibJS+LibWeb: Move script parse time logging from JS::Script to LibWeb

Let's only log HTML::ClassicScript parse times for now. Otherwise things
will get excessively noisy in test-js and the test262 runner.
This commit is contained in:
Andreas Kling 2021-09-14 21:00:41 +02:00
parent 0a09eaf3a1
commit 20fb900937
Notes: sideshowbarker 2024-07-18 03:57:55 +09:00
2 changed files with 2 additions and 6 deletions

View File

@ -4,7 +4,6 @@
* SPDX-License-Identifier: BSD-2-Clause
*/
#include <LibCore/ElapsedTimer.h>
#include <LibJS/AST.h>
#include <LibJS/Lexer.h>
#include <LibJS/Parser.h>
@ -15,11 +14,6 @@ namespace JS {
// 16.1.5 ParseScript ( sourceText, realm, hostDefined ), https://tc39.es/ecma262/#sec-parse-script
Result<NonnullRefPtr<Script>, Vector<Parser::Error>> Script::parse(StringView source_text, Realm& realm, StringView filename)
{
auto timer = Core::ElapsedTimer::start_new();
ScopeGuard timer_guard([&] {
dbgln("JS::Script: Parsed {} in {}ms", filename, timer.elapsed());
});
// 1. Let body be ParseText(sourceText, Script).
auto parser = Parser(Lexer(source_text, filename));
auto body = parser.parse_program();

View File

@ -36,7 +36,9 @@ NonnullRefPtr<ClassicScript> ClassicScript::create(String filename, StringView s
// FIXME: 9. Set script's parse error and error to rethrow to null.
// 10. Let result be ParseScript(source, settings's Realm, script).
auto parse_timer = Core::ElapsedTimer::start_new();
auto result = JS::Script::parse(source, realm, script->filename());
dbgln("ClassicScript: Parsed {} in {}ms", script->filename(), parse_timer.elapsed());
// 11. If result is a list of errors, then: