[Downloads] Add context parameter to help center article URL.

chrome/browser/download/download_commands_unittest.cc

Index: chrome/browser/download/download_commands_unittest.cc
diff --git a/chrome/browser/download/download_commands_unittest.cc b/chrome/browser/download/download_commands_unittest.cc
index 46717c455f1ead71d29ef6224654d506514691db..e404d93b645b6e38037871f16b13fa7e4e8fdf7b 100644
--- a/chrome/browser/download/download_commands_unittest.cc
+++ b/chrome/browser/download/download_commands_unittest.cc
@@ -26,6 +26,8 @@ const base::FilePath::CharType kDefaultTargetFilePath[] =
 // Default URL for a mock download item in DownloadCommandsTest.
 const char kDefaultURL[] = "http://example.com/foo.bar";
 
+}  // namespace
+
 class DownloadCommandsTest : public testing::Test {
  public:
   DownloadCommandsTest() : commands_(&item_) {}
@@ -80,8 +82,6 @@ class DownloadCommandsTest : public testing::Test {
   DownloadCommands commands_;
 };
 
-}  // namespace
-
 TEST_F(DownloadCommandsTest, InProgress) {
   EXPECT_TRUE(IsCommandEnabled(DownloadCommands::SHOW_IN_FOLDER));
   EXPECT_TRUE(IsCommandEnabled(DownloadCommands::CANCEL));
@@ -154,3 +154,11 @@ TEST_F(DownloadCommandsTest, DoResume) {
   EXPECT_CALL(item(), Resume()).Times(1);
   commands().ExecuteCommand(DownloadCommands::RESUME);
 }
+
+TEST_F(DownloadCommandsTest, LearnMoreInterruptedURL) {
+  EXPECT_CALL(item(), GetLastReason())
+      .WillOnce(
+          Return(content::DOWNLOAD_INTERRUPT_REASON_NETWORK_DISCONNECTED));
+  GURL learn_more_url = commands().GetLearnMoreURLForInterruptedDownload();
+  EXPECT_EQ("p=ui_download_errors&ctx=22", learn_more_url.query());

[Randy Smith (Not in Mondays), 2015/04/21 17:28:37]

I'd rather compare against the download interrupt reason casted to an integer. 
We don't want this test to be dependent on the reason -> integer mappings, even
if we need to keep those fixed so that we don't mess up the help articles.

[asanka, 2015/04/22 16:58:47]

Done.

+}