Commit 53ee0994 authored by Daniel Murphy's avatar Daniel Murphy Committed by Commit Bot

[BMO] Actually check the weakptr in chrome://apps uninstall

R=calamity@chromium.org, loyso@chromium

Fixed: 1070185
Change-Id: I84aafb1203586e272bb0d68d8f0aed49f0d7f827
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2147805
Auto-Submit: Daniel Murphy <dmurph@chromium.org>
Commit-Queue: calamity <calamity@chromium.org>
Reviewed-by: default avatarcalamity <calamity@chromium.org>
Cr-Commit-Position: refs/heads/master@{#759127}
parent f5249052
...@@ -865,7 +865,8 @@ void AppLauncherHandler::HandleUninstallApp(const base::ListValue* args) { ...@@ -865,7 +865,8 @@ void AppLauncherHandler::HandleUninstallApp(const base::ListValue* args) {
[](base::WeakPtr<AppLauncherHandler> app_launcher_handler, [](base::WeakPtr<AppLauncherHandler> app_launcher_handler,
bool success) { bool success) {
LOCAL_HISTOGRAM_BOOLEAN("Apps.Launcher.UninstallSuccess", success); LOCAL_HISTOGRAM_BOOLEAN("Apps.Launcher.UninstallSuccess", success);
app_launcher_handler->CleanupAfterUninstall(); if (app_launcher_handler)
app_launcher_handler->CleanupAfterUninstall();
}, },
weak_ptr_factory_.GetWeakPtr()); weak_ptr_factory_.GetWeakPtr());
......
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