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

Unified Diff: chrome/browser/web_applications/web_app.h

Issue 2703283005: Destroy web_app::ShortcutInfo on UI thread (Closed)
Patch Set: . Created 3 years, 10 months 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/web_applications/web_app.h
diff --git a/chrome/browser/web_applications/web_app.h b/chrome/browser/web_applications/web_app.h
index 1e4234920bfc7d50a934120ed5b8d6a01a645c23..40246e269f48d3ad3fa562cb732ae798734fd822 100644
--- a/chrome/browser/web_applications/web_app.h
+++ b/chrome/browser/web_applications/web_app.h
@@ -229,24 +229,23 @@ std::vector<base::FilePath> GetShortcutPaths(
// shortcut, and is also used as the UserDataDir for platform app shortcuts.
// |shortcut_info| contains info about the shortcut to create, and
// |creation_locations| contains information about where to create them.
-bool CreatePlatformShortcuts(
- const base::FilePath& shortcut_data_path,
- std::unique_ptr<ShortcutInfo> shortcut_info,
- const ShortcutLocations& creation_locations,
- ShortcutCreationReason creation_reason);
+bool CreatePlatformShortcuts(const base::FilePath& shortcut_data_path,
+ ShortcutInfo* shortcut_info,
+ const ShortcutLocations& creation_locations,
+ ShortcutCreationReason creation_reason);
// Delete all the shortcuts we have added for this extension. This is the
// platform specific implementation of the DeleteAllShortcuts function, and
// is executed on the FILE thread.
void DeletePlatformShortcuts(const base::FilePath& shortcut_data_path,
- std::unique_ptr<ShortcutInfo> shortcut_info);
+ ShortcutInfo* shortcut_info);
// Updates all the shortcuts we have added for this extension. This is the
// platform specific implementation of the UpdateAllShortcuts function, and
// is executed on the FILE thread.
void UpdatePlatformShortcuts(const base::FilePath& shortcut_data_path,
const base::string16& old_app_title,
- std::unique_ptr<ShortcutInfo> shortcut_info);
+ ShortcutInfo* shortcut_info);
// Delete all the shortcuts for an entire profile.
// This is executed on the FILE thread.
@@ -256,6 +255,10 @@ void DeleteAllShortcutsForProfile(const base::FilePath& profile_path);
// on-disk file name .
base::FilePath GetSanitizedFileName(const base::string16& name);
+// Clears |shortcut_info| and invokes |callback| unless it's null.
+void DeleteShortcutInfoOnUIThread(std::unique_ptr<ShortcutInfo> shortcut_info,
+ const base::Closure& callback);
+
} // namespace internals
} // namespace web_app
« no previous file with comments | « no previous file | chrome/browser/web_applications/web_app.cc » ('j') | chrome/browser/web_applications/web_app.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698