Commit 6463b63e authored by Roman Aleksandrov's avatar Roman Aleksandrov Committed by Commit Bot

oobe: Check view in destructor before screen unbinding.

Fixed: 1136386
Change-Id: I9f6bf45c94fb8b26e05ecff5a5304e9eef857e81
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2552396
Commit-Queue: Roman Aleksandrov <raleksandrov@google.com>
Reviewed-by: default avatarRoman Sorokin [CET] <rsorokin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#830244}
parent af130f1b
...@@ -28,7 +28,8 @@ AppDownloadingScreen::AppDownloadingScreen( ...@@ -28,7 +28,8 @@ AppDownloadingScreen::AppDownloadingScreen(
} }
AppDownloadingScreen::~AppDownloadingScreen() { AppDownloadingScreen::~AppDownloadingScreen() {
view_->Bind(nullptr); if (view_)
view_->Bind(nullptr);
} }
void AppDownloadingScreen::ShowImpl() { void AppDownloadingScreen::ShowImpl() {
......
...@@ -112,7 +112,8 @@ FingerprintSetupScreen::FingerprintSetupScreen( ...@@ -112,7 +112,8 @@ FingerprintSetupScreen::FingerprintSetupScreen(
} }
FingerprintSetupScreen::~FingerprintSetupScreen() { FingerprintSetupScreen::~FingerprintSetupScreen() {
view_->Bind(nullptr); if (view_)
view_->Bind(nullptr);
} }
bool FingerprintSetupScreen::MaybeSkip(WizardContext* context) { bool FingerprintSetupScreen::MaybeSkip(WizardContext* context) {
......
...@@ -97,7 +97,8 @@ MarketingOptInScreen::MarketingOptInScreen( ...@@ -97,7 +97,8 @@ MarketingOptInScreen::MarketingOptInScreen(
} }
MarketingOptInScreen::~MarketingOptInScreen() { MarketingOptInScreen::~MarketingOptInScreen() {
view_->Bind(nullptr); if (view_)
view_->Bind(nullptr);
} }
bool MarketingOptInScreen::MaybeSkip(WizardContext* context) { bool MarketingOptInScreen::MaybeSkip(WizardContext* context) {
......
...@@ -57,7 +57,8 @@ PinSetupScreen::PinSetupScreen(PinSetupScreenView* view, ...@@ -57,7 +57,8 @@ PinSetupScreen::PinSetupScreen(PinSetupScreenView* view,
} }
PinSetupScreen::~PinSetupScreen() { PinSetupScreen::~PinSetupScreen() {
view_->Bind(nullptr); if (view_)
view_->Bind(nullptr);
} }
bool PinSetupScreen::MaybeSkip(WizardContext* context) { bool PinSetupScreen::MaybeSkip(WizardContext* context) {
......
...@@ -101,7 +101,8 @@ SyncConsentScreen::SyncConsentScreen(SyncConsentScreenView* view, ...@@ -101,7 +101,8 @@ SyncConsentScreen::SyncConsentScreen(SyncConsentScreenView* view,
} }
SyncConsentScreen::~SyncConsentScreen() { SyncConsentScreen::~SyncConsentScreen() {
view_->Bind(NULL); if (view_)
view_->Bind(nullptr);
} }
void SyncConsentScreen::Init() { void SyncConsentScreen::Init() {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment