Added test case for CopyImageUrl and CopyEmailAddress.

chrome/android/javatests/src/org/chromium/chrome/browser/contextmenu/ContextMenuTest.java

 // Copyright 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 package org.chromium.chrome.browser.contextmenu;
 
 import android.content.ClipData;
 import android.content.ClipboardManager;
 import android.content.Context;
 import android.os.Build;
 import android.test.FlakyTest;
+import android.test.suitebuilder.annotation.MediumTest;
 import android.view.ContextMenu;
 
 import junit.framework.Assert;
 
 import org.chromium.base.test.util.Feature;
 import org.chromium.chrome.R;
 import org.chromium.chrome.browser.Tab;
 import org.chromium.chrome.shell.ChromeShellTestBase;
 import org.chromium.chrome.test.util.TestHttpServerClient;
 import org.chromium.chrome.test.util.browser.contextmenu.ContextMenuUtils;
@@ skipping 180 matching lines @@
 
         Assert.assertTrue("Activity did not regain focus.",
                 CriteriaHelper.pollForCriteria(new Criteria() {
                     @Override
                     public boolean isSatisfied() {
                         return getActivity().hasWindowFocus();
                     }
                 }));
     }
 
+    @MediumTest

[Bernhard Bauer, 2014/09/30 08:51:08]

Why are these tests medium if e.g. testCopyLinkTextSimple() is large?

[ankit, 2014/09/30 09:20:46]

I tried running these test cases with @MediumTest and it was passing without any
time out.
So I have used @MediumTest. Please suggest if we should use @LargeTest for some
reasons.

[Bernhard Bauer, 2014/09/30 10:05:25]

Okay, let's try @MediumTest, but can you also change the commented-out
@LargeTest for the others to that? If they get reenabled, they should have the
same size.

+    @Feature({"Browser"})
+    public void testCopyImageURL() throws InterruptedException, TimeoutException {
+        if (Build.VERSION.SDK_INT < Build.VERSION_CODES.JELLY_BEAN) return;

[Bernhard Bauer, 2014/09/30 08:51:08]

What do we need this check for?

[ankit, 2014/09/30 09:20:46]

Not required. 

+
+        Tab tab = getActivity().getActiveTab();
+        ContextMenuUtils.selectContextMenuItem(this, tab, "testImage",
+                R.id.contextmenu_copy_image_url);
+
+        String expectedUrl = TestHttpServerClient.getUrl(
+                "chrome/test/data/android/contextmenu/test_image.png");
+
+        assertEquals("Clipboard text is not correct", expectedUrl, getClipboardText());
+    }
+
+    @MediumTest
+    @Feature({"Browser"})
+    public void testCopyEmailAddress() throws InterruptedException, TimeoutException {
+        if (Build.VERSION.SDK_INT < Build.VERSION_CODES.JELLY_BEAN) return;
+
+        Tab tab = getActivity().getActiveTab();
+        ContextMenuUtils.selectContextMenuItem(this, tab, "testEmail",
+                R.id.contextmenu_copy_email_address);
+
+        assertEquals("Clipboard text is not correct", "someone@example.com", getClipboardText());
+    }
+
     private String getClipboardText() {
         ClipboardManager clipMgr =
                 (ClipboardManager) getActivity().getSystemService(Context.CLIPBOARD_SERVICE);
         ClipData clipData = clipMgr.getPrimaryClip();
         assertNotNull("Primary clip is null", clipData);
         assertTrue("Primary clip contains no items.", clipData.getItemCount() > 0);
         return clipData.getItemAt(0).getText().toString();
     }
 
     private void assertStringContains(String subString, String superString) {
         assertTrue("String '" + superString + "' does not contain '" + subString + "'",
                 superString.contains(subString));
     }
 }