Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(648)

Unified Diff: chrome/browser/chromeos/printer_detector/printer_detector.cc

Issue 2483573002: [CUPS] USB printer setup notification: direct the user to settings page if failed. (Closed)
Patch Set: Rebase. Created 4 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/chromeos/printer_detector/printer_detector.cc
diff --git a/chrome/browser/chromeos/printer_detector/printer_detector.cc b/chrome/browser/chromeos/printer_detector/printer_detector.cc
index 2330743260d03208db5e9ebb2ffbd52090d0768e..b1d7b86b1d2f00b2bd2a4f5e82a58579460d8abd 100644
--- a/chrome/browser/chromeos/printer_detector/printer_detector.cc
+++ b/chrome/browser/chromeos/printer_detector/printer_detector.cc
@@ -23,7 +23,6 @@
#include "chrome/browser/notifications/notification_ui_manager.h"
#include "chrome/browser/ui/browser_navigator.h"
#include "chrome/browser/ui/browser_navigator_params.h"
-#include "chrome/common/chrome_features.h"
#include "chrome/common/chrome_switches.h"
#include "chrome/common/extensions/api/webstore_widget_private.h"
#include "chrome/common/extensions/extension_constants.h"
@@ -386,14 +385,12 @@ void PrinterDetector::ClickOnNotificationButton(int button_index) {
} else if (command_ == ButtonCommand::CANCEL_SETUP) {
// TODO(skau/xdai): call the CUPS backend to cancel the printer setup.
} else if (command_ == ButtonCommand::GET_HELP) {
- if (base::FeatureList::IsEnabled(features::kMaterialDesignSettings)) {
- chrome::NavigateParams params(profile_,
- GURL(chrome::kChromeUIMdCupsSettingsURL),
- ui::PAGE_TRANSITION_LINK);
- params.disposition = WindowOpenDisposition::NEW_FOREGROUND_TAB;
- params.window_action = chrome::NavigateParams::SHOW_WINDOW;
- chrome::Navigate(&params);
- }
+ chrome::NavigateParams params(profile_,
+ GURL(chrome::kChromeUIMdCupsSettingsURL),
+ ui::PAGE_TRANSITION_LINK);
+ params.disposition = WindowOpenDisposition::NEW_FOREGROUND_TAB;
+ params.window_action = chrome::NavigateParams::SHOW_WINDOW;
+ chrome::Navigate(&params);
}
}
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698