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

Unified Diff: chrome/common/url_constants.cc

Issue 10546072: Use different help URLs for menus, accelerators, and WebUI. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: ugh core file Created 8 years, 6 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/common/url_constants.cc
diff --git a/chrome/common/url_constants.cc b/chrome/common/url_constants.cc
index da738ecbae3e0b6fe51f04ef363c0bd4065c2be2..9917d246bdab05f5d632ace2c833ecec3a92b1e4 100644
--- a/chrome/common/url_constants.cc
+++ b/chrome/common/url_constants.cc
@@ -284,16 +284,38 @@ const char kPasswordManagerLearnMoreURL[] =
"https://support.google.com/chrome/?p=settings_password";
#endif
-const char kChromeHelpURL[] =
+const char kChromeHelpViaKeyboardURL[] =
#if defined(OS_CHROMEOS)
#if defined(OFFICIAL_BUILD)
"chrome-extension://honijodknafkokifofgiaalefdiedpko/main.html";
#else
- "https://support.google.com/chromeos/?p=help";
+ "https://support.google.com/chromeos/?p=help&ctx=keyboard";
#endif // defined(OFFICIAL_BUILD
#else
- "https://support.google.com/chrome/?p=help";
-#endif
+ "https://support.google.com/chrome/?p=help&ctx=keyboard";
+#endif // defined(OS_CHROMEOS)
+
+const char kChromeHelpViaMenuURL[] =
+#if defined(OS_CHROMEOS)
+#if defined(OFFICIAL_BUILD)
+ "chrome-extension://honijodknafkokifofgiaalefdiedpko/main.html";
+#else
+ "https://support.google.com/chromeos/?p=help&ctx=menu";
+#endif // defined(OFFICIAL_BUILD
+#else
+ "https://support.google.com/chrome/?p=help&ctx=menu";
+#endif // defined(OS_CHROMEOS)
+
+const char kChromeHelpViaWebUIURL[] =
+#if defined(OS_CHROMEOS)
+#if defined(OFFICIAL_BUILD)
+ "chrome-extension://honijodknafkokifofgiaalefdiedpko/main.html";
+#else
+ "https://support.google.com/chromeos/?p=help&ctx=settings";
+#endif // defined(OFFICIAL_BUILD
+#else
+ "https://support.google.com/chrome/?p=help&ctx=settings";
+#endif // defined(OS_CHROMEOS)
const char kChromeSyncLearnMoreURL[] =
#if defined(OS_CHROMEOS)

Powered by Google App Engine
This is Rietveld 408576698