refactor(runtime/{bridge,webview}): more safety checks

Author: jwerle

src/runtime/bridge/bridge.cc

@@ -482,8 +482,8 @@ export * from '{{url}}'
 
           const auto app = App::sharedApplication();
           const auto options = serviceworker::Fetch::Options { request->client };
-          const auto fetched = serviceWorker->fetch(fetch, options, [=, this, &request] (auto res) mutable {
-            if (!request->isActive()) {
+          const auto fetched = serviceWorker->fetch(fetch, options, [=, this] (auto res) mutable {
+            if (!request ||  !request->isActive()) {
               return;
             }
 
@@ -592,7 +592,7 @@ export * from '{{url}}'
           });
 
           if (fetched) {
-             this->getRuntime()->services.timers.setTimeout(32000, [&request] () mutable {
+             this->getRuntime()->services.timers.setTimeout(32000, [request] () mutable {
                if (request->isActive()) {
                  auto response = SchemeHandlers::Response(request, 408);
                  response.fail("ServiceWorker request timed out.");

src/runtime/webview/scheme_handlers.cc

@@ -936,6 +936,9 @@ namespace ssc::runtime::webview {
 
   bool SchemeHandlers::Request::isActive () const {
     auto app = app::App::sharedApplication();
+    if (this->handlers == nullptr) {
+      return false;
+    }
     auto window = app->runtime.windowManager.getWindowForBridge(&this->handlers->bridge);
 
     // only a scheme handler owned by this bridge and attached to a