Commit da55f564 authored by Julian Watson's avatar Julian Watson Committed by Commit Bot

crostini: handle export/import failure notification clicks

Bug: 998319
Change-Id: I46594add1dbec54e29ba2b2d6bd515ce180a6cd9
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1773003Reviewed-by: default avatarJoel Hockey <joelhockey@chromium.org>
Reviewed-by: default avatarScott Violet <sky@chromium.org>
Commit-Queue: Julian Watson <juwa@google.com>
Cr-Commit-Position: refs/heads/master@{#691705}
parent afa24bdc
......@@ -11,6 +11,11 @@
#include "chrome/browser/notifications/notification_display_service.h"
#include "chrome/browser/platform_util.h"
#include "chrome/browser/profiles/profile.h"
#include "chrome/browser/ui/browser_navigator.h"
#include "chrome/browser/ui/browser_navigator_params.h"
#include "chrome/browser/ui/settings_window_manager_chromeos.h"
#include "chrome/common/url_constants.h"
#include "chrome/common/webui_url_constants.h"
#include "chrome/grit/generated_resources.h"
#include "ui/base/l10n/l10n_util.h"
#include "ui/base/text/bytes_formatting.h"
......@@ -252,6 +257,25 @@ void CrostiniExportImportNotification::Click(
platform_util::ShowItemInFolder(profile_, path_);
}
return;
case Status::FAILED_UNKNOWN_REASON:
DCHECK(!button_index);
chrome::SettingsWindowManager::GetInstance()->ShowOSSettings(
profile_, chrome::kCrostiniExportImportSubPage);
return;
case Status::FAILED_ARCHITECTURE_MISMATCH: {
DCHECK(!button_index);
NavigateParams params(profile_, GURL(chrome::kLinuxExportImportHelpURL),
ui::PAGE_TRANSITION_LINK);
params.disposition = WindowOpenDisposition::NEW_FOREGROUND_TAB;
params.window_action = NavigateParams::SHOW_WINDOW;
Navigate(&params);
}
return;
case Status::FAILED_INSUFFICIENT_SPACE:
DCHECK(!button_index);
chrome::SettingsWindowManager::GetInstance()->ShowOSSettings(
profile_, chrome::kStorageSubPage);
return;
default:
DCHECK(!button_index);
}
......
......@@ -339,6 +339,9 @@ const char kLearnMoreEnterpriseURL[] =
const char kLinuxAppsLearnMoreURL[] =
"https://support.google.com/chromebook?p=chromebook_linuxapps";
const char kLinuxExportImportHelpURL[] =
"https://support.google.com/chromebook?p=linux_backup_restore";
const char kLinuxCreditsPath[] =
"/opt/google/chrome/resources/linux_credits.html";
......
......@@ -285,6 +285,9 @@ extern const char kLearnMoreEnterpriseURL[];
// The URL for the Learn More page about Linux for Chromebooks.
extern const char kLinuxAppsLearnMoreURL[];
// The URL for additional help that is given when Linux export/import fails.
extern const char kLinuxExportImportHelpURL[];
// Credits for Linux for Chromebooks.
extern const char kLinuxCreditsPath[];
......
......@@ -383,6 +383,7 @@ const char kChangePictureSubPage[] = "changePicture";
// 'multidevice/features' is a child of the 'multidevice' route
const char kConnectedDevicesSubPage[] = "multidevice/features";
const char kCrostiniSharedUsbDevicesSubPage[] = "crostini/sharedUsbDevices";
const char kCrostiniExportImportSubPage[] = "crostini/exportImport";
const char kDateTimeSubPage[] = "dateTime";
const char kDisplaySubPage[] = "display";
const char kHelpSubPage[] = "help";
......
......@@ -339,6 +339,7 @@ extern const char kBluetoothSubPage[];
extern const char kChangePictureSubPage[];
extern const char kConnectedDevicesSubPage[];
extern const char kCrostiniSharedUsbDevicesSubPage[];
extern const char kCrostiniExportImportSubPage[];
extern const char kDateTimeSubPage[];
extern const char kDisplaySubPage[];
extern const char kHelpSubPage[];
......
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