Unflake AppViewTest.TestAppViewGoodDataShouldSucceed under UBSan Vptr.

extensions/browser/api/web_request/web_request_api.cc

Index: extensions/browser/api/web_request/web_request_api.cc
diff --git a/extensions/browser/api/web_request/web_request_api.cc b/extensions/browser/api/web_request/web_request_api.cc
index 12f38e1e7588732455c16edd6b14b03d24d4b813..055a3d076910a48cb63afc2e69f1c29978c2761a 100644
--- a/extensions/browser/api/web_request/web_request_api.cc
+++ b/extensions/browser/api/web_request/web_request_api.cc
@@ -967,8 +967,12 @@ void ExtensionWebRequestEventRouter::OnErrorOccurred(
     net::URLRequest* request,
     bool started,
     int net_error) {
+  ExtensionsBrowserClient* client = ExtensionsBrowserClient::Get();
+  if (!client) {

[lazyboy, 2016/11/09 00:10:11]

I'd add a comment here saying |client| could be null during shutdown.

[krasin1, 2016/11/09 00:36:05]

Done.

+    return;
+  }
   ExtensionNavigationUIData* navigation_ui_data =
-      ExtensionsBrowserClient::Get()->GetExtensionNavigationUIData(request);
+      client->GetExtensionNavigationUIData(request);
   // We hide events from the system context as well as sensitive requests.
   // However, if the request first became sensitive after redirecting we have
   // already signaled it and thus we have to signal the end of it. This is