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

Unified Diff: chrome/browser/ui/ash/chrome_screenshot_grabber.cc

Issue 2526893003: chromeos: Add NoteTakingHelper class. (Closed)
Patch Set: drop is-note-taking-app-window stuff Created 4 years 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
Index: chrome/browser/ui/ash/chrome_screenshot_grabber.cc
diff --git a/chrome/browser/ui/ash/chrome_screenshot_grabber.cc b/chrome/browser/ui/ash/chrome_screenshot_grabber.cc
index 37a9bd431b011dc669c0c2ff22f1f1f3c6c2dc1c..9ba9b8ac1398e1acb89f5783096d1adef5867bd1 100644
--- a/chrome/browser/ui/ash/chrome_screenshot_grabber.cc
+++ b/chrome/browser/ui/ash/chrome_screenshot_grabber.cc
@@ -21,7 +21,7 @@
#include "chrome/browser/browser_process.h"
#include "chrome/browser/chromeos/drive/file_system_util.h"
#include "chrome/browser/chromeos/file_manager/open_util.h"
-#include "chrome/browser/chromeos/note_taking_app_utils.h"
+#include "chrome/browser/chromeos/note_taking_helper.h"
#include "chrome/browser/download/download_prefs.h"
#include "chrome/browser/notifications/notification_ui_manager.h"
#include "chrome/browser/notifications/notifier_state_tracker.h"
@@ -136,8 +136,9 @@ class ScreenshotGrabberNotificationDelegate : public NotificationDelegate {
break;
}
case BUTTON_ANNOTATE: {
- if (chromeos::IsNoteTakingAppAvailable(profile_)) {
- chromeos::LaunchNoteTakingAppForNewNote(profile_, screenshot_path_);
+ chromeos::NoteTakingHelper* helper = chromeos::NoteTakingHelper::Get();
+ if (helper->IsAppAvailable(profile_)) {
+ helper->LaunchAppForNewNote(profile_, screenshot_path_);
content::RecordAction(base::UserMetricsAction("Screenshot_Annotate"));
}
break;
@@ -437,7 +438,7 @@ Notification* ChromeScreenshotGrabber::CreateNotification(
IDR_NOTIFICATION_SCREENSHOT_COPY_TO_CLIPBOARD);
optional_field.buttons.push_back(copy_button);
- if (chromeos::IsNoteTakingAppAvailable(GetProfile())) {
+ if (chromeos::NoteTakingHelper::Get()->IsAppAvailable(GetProfile())) {
message_center::ButtonInfo annotate_button(l10n_util::GetStringUTF16(
IDS_MESSAGE_CENTER_NOTIFICATION_BUTTON_ANNOTATE_SCREENSHOT));
annotate_button.icon =

Powered by Google App Engine
This is Rietveld 408576698