LadybirdBrowser/ladybird
1
1
Fork 0
mirror of https://github.com/LadybirdBrowser/ladybird.git synced 2025-01-07 19:57:45 +03:00
Code Issues Packages Projects Releases Wiki Activity

js: Fix that auto completion of properties failed

Browse Source 
For this we store the global environment in which we can do a lookup
for the references variable. This is probably not entirely as the spec
would specify as we would need a running executing context at all times
you do things with references.

Fixes #10281
This commit is contained in:
davidot 2021-10-03 13:17:33 +02:00 committed by Andreas Kling
parent ac2c3a73b1
commit 0be0e7ea6e
Notes: sideshowbarker 2024-07-18 03:08:36 +09:00 
Author: https://github.com/davidot
Commit: https://github.com/SerenityOS/serenity/commit/0be0e7ea6ef
Pull-request: https://github.com/SerenityOS/serenity/pull/10333
Reviewed-by: https://github.com/linusg ✅
1 changed files with 8 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

19
Userland/Utilities/js.cpp
View File

@ -1174,6 +1174,8 @@ int main(int argc, char** argv)
#endif
interpreter->vm().set_underscore_is_last_value(true);
auto& global_environment = interpreter->realm().global_environment();
s_editor = Line::Editor::construct();
s_editor->load_history(s_history_path);
@ -1246,7 +1248,7 @@ int main(int argc, char** argv)
editor.set_prompt(prompt_for_level(open_indents));
};
auto complete = [&interpreter](Line::Editor const& editor) -> Vector<Line::CompletionSuggestion> {
auto complete = [&interpreter, &global_environment](Line::Editor const& editor) -> Vector<Line::CompletionSuggestion> {
auto line = editor.line(editor.cursor());
JS::Lexer lexer { line };
@ -1337,18 +1339,13 @@ int main(int argc, char** argv)
switch (mode) {
case CompleteProperty: {
Optional<JS::Value> maybe_value;
auto maybe_variable = vm->resolve_binding(variable_name);
auto maybe_variable = vm->resolve_binding(variable_name, &global_environment);
if (vm->exception())
break;
if (!maybe_variable.is_unresolvable()) {
maybe_value = maybe_variable.get_value(interpreter->global_object());
if (vm->exception())
break;
} else {
maybe_value = interpreter->global_object().get(FlyString(variable_name));
if (maybe_value->is_empty())
break;
}
maybe_value = maybe_variable.get_value(interpreter->global_object());
if (vm->exception())
break;
VERIFY(!maybe_value->is_empty());
auto variable = *maybe_value;
if (!variable.is_object())