Fire controllerchange event on ServiceWorkerContainers

Comment on attachment 8539216 [details] [diff] [review] Fire controllerchange event on controlled documents ServiceWorkerContainer Review of attachment 8539216 [details] [diff] [review]: ----------------------------------------------------------------- ::: dom/workers/ServiceWorkerManager.cpp @@ +2093,5 @@ > + if (aValue != contextReg) { > + return PL_DHASH_NEXT; > + } > + > + nsCOMPtr<nsIDocument> aDoc = do_QueryInterface(aKey); doc @@ +2094,5 @@ > + return PL_DHASH_NEXT; > + } > + > + nsCOMPtr<nsIDocument> aDoc = do_QueryInterface(aKey); > + if (!aDoc) { if (NS_WARN_IF(!doc)) { .. ? @@ +2100,5 @@ > + } > + > + nsCOMPtr<nsPIDOMWindow> w = aDoc->GetWindow(); > + MOZ_ASSERT(w); > + nsGlobalWindow* window = static_cast<nsGlobalWindow*>(w.get()); auto* window ? @@ +2102,5 @@ > + nsCOMPtr<nsPIDOMWindow> w = aDoc->GetWindow(); > + MOZ_ASSERT(w); > + nsGlobalWindow* window = static_cast<nsGlobalWindow*>(w.get()); > + if (!window) { > + NS_WARNING("No valid nsGlobalWindow"); if (NS_WARN_IF(!window)) { .. ?