Spreadsheet: Wrap raw JS::Cell*/& fields in GCPtr/NonnullGCPtr

Author: https://github.com/mattco98
Commit: https://github.com/SerenityOS/serenity/commit/b5f01bd212
Pull-request: https://github.com/SerenityOS/serenity/pull/17743
Reviewed-by: https://github.com/linusg ✅

Userland/Applications/Spreadsheet/Spreadsheet.h

@@ -155,7 +155,7 @@ private:
     HashTable<Position> m_selected_cells;
 
     Workbook& m_workbook;
-    mutable SheetGlobalObject* m_global_object;
+    mutable JS::GCPtr<SheetGlobalObject> m_global_object;
 
     NonnullOwnPtr<JS::Interpreter> m_interpreter;
 

Userland/Applications/Spreadsheet/Workbook.h

@@ -46,7 +46,7 @@ private:
     NonnullRefPtr<JS::VM> m_vm;
     NonnullOwnPtr<JS::Interpreter> m_interpreter;
     JS::VM::InterpreterExecutionScope m_interpreter_scope;
-    WorkbookObject* m_workbook_object { nullptr };
+    JS::GCPtr<WorkbookObject> m_workbook_object;
     JS::ExecutionContext m_main_execution_context;
     GUI::Window& m_parent_window;