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

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

Issue 23093020: Set the WM_CLASS property of X11 windows in Linux Aura build. (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: Created 7 years, 4 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 67525970be722580b60b834b2b797037fc6435aa..dbd2a21b12b8b5d7f9d90fd554b6b42b42c73047 100644
--- a/chrome/browser/web_applications/web_app.h
+++ b/chrome/browser/web_applications/web_app.h
@@ -97,12 +97,10 @@ void GetIconsInfo(const WebApplicationInfo& app_info,
IconInfoList* icons);
#endif
-#if defined(TOOLKIT_GTK)
-// GTK+ windows that correspond to web apps need to have a deterministic (and
-// different) WMClass than normal chrome windows so the window manager groups
-// them as a separate application.
+// On Linux, windows that correspond to web apps need to have a deterministic
benwells 2013/08/26 00:10:29 Should the ifdef become #if defined(OS_LINUX) inst
Matt Giuca 2013/08/26 04:46:09 Done.
+// (and different) WMClass than normal chrome windows so the window manager
+// groups them as a separate application.
std::string GetWMClassFromAppName(std::string app_name);
-#endif
// Gets the name of the Chrome Apps menu folder in which to place app shortcuts.
string16 GetAppShortcutsSubdirName();

Powered by Google App Engine
This is Rietveld 408576698