Added test case for CopyImageUrl and CopyEmailAddress.

Added test case for CopyImageUrl and CopyEmailAddress.

Test case was missing for copyImageUrl and CopyEmailAddress options
newly added in context menu.
Added test case for same.

BUG=418073
Committed: https://crrev.com/524325fd4502e6394a9ec73fb8d15316df89f04b
Cr-Commit-Position: refs/heads/master@{#298015}

[ankit, 2014-09-30 06:07:30]

ankit2.kumar@samsung.com changed reviewers:
+ bauerb@chromium.org, tedchoc@chromium.org

[ankit, 2014-09-30 06:07:31]

@Bernhard
PTAL

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

https://codereview.chromium.org/614863002/diff/1/chrome/android/javatests/src...
File
chrome/android/javatests/src/org/chromium/chrome/browser/contextmenu/ContextMenuTest.java
(right):

https://codereview.chromium.org/614863002/diff/1/chrome/android/javatests/src...
chrome/android/javatests/src/org/chromium/chrome/browser/contextmenu/ContextMenuTest.java:213:
@MediumTest
Why are these tests medium if e.g. testCopyLinkTextSimple() is large?

https://codereview.chromium.org/614863002/diff/1/chrome/android/javatests/src...
chrome/android/javatests/src/org/chromium/chrome/browser/contextmenu/ContextMenuTest.java:216:
if (Build.VERSION.SDK_INT < Build.VERSION_CODES.JELLY_BEAN) return;
What do we need this check for?

https://codereview.chromium.org/614863002/diff/1/chrome/test/data/android/con...
File chrome/test/data/android/contextmenu/context_menu_test.html (right):

https://codereview.chromium.org/614863002/diff/1/chrome/test/data/android/con...
chrome/test/data/android/contextmenu/context_menu_test.html:16: <a
href="mailto:someone@example.com" id="testEmail">EmailTest</a>
This is in the wrong place. The <a> opened in line 14 needs to be closed first
(the corresponding closing tag is right now in line 26), then we can open a new
one.

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

@Bernhard
PTAL new patch and comments.

https://codereview.chromium.org/614863002/diff/1/chrome/android/javatests/src...
File
chrome/android/javatests/src/org/chromium/chrome/browser/contextmenu/ContextMenuTest.java
(right):

https://codereview.chromium.org/614863002/diff/1/chrome/android/javatests/src...
chrome/android/javatests/src/org/chromium/chrome/browser/contextmenu/ContextMenuTest.java:213:
@MediumTest
On 2014/09/30 08:51:08, Bernhard Bauer wrote:
> Why are these tests medium if e.g. testCopyLinkTextSimple() is large?

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.

https://codereview.chromium.org/614863002/diff/1/chrome/android/javatests/src...
chrome/android/javatests/src/org/chromium/chrome/browser/contextmenu/ContextMenuTest.java:216:
if (Build.VERSION.SDK_INT < Build.VERSION_CODES.JELLY_BEAN) return;
On 2014/09/30 08:51:08, Bernhard Bauer wrote:
> What do we need this check for?

Not required.

https://codereview.chromium.org/614863002/diff/1/chrome/test/data/android/con...
File chrome/test/data/android/contextmenu/context_menu_test.html (right):

https://codereview.chromium.org/614863002/diff/1/chrome/test/data/android/con...
chrome/test/data/android/contextmenu/context_menu_test.html:16: <a
href="mailto:someone@example.com" id="testEmail">EmailTest</a>
On 2014/09/30 08:51:08, Bernhard Bauer wrote:
> This is in the wrong place. The <a> opened in line 14 needs to be closed first
> (the corresponding closing tag is right now in line 26), then we can open a
new
> one.

Done.

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

https://codereview.chromium.org/614863002/diff/1/chrome/android/javatests/src...
File
chrome/android/javatests/src/org/chromium/chrome/browser/contextmenu/ContextMenuTest.java
(right):

https://codereview.chromium.org/614863002/diff/1/chrome/android/javatests/src...
chrome/android/javatests/src/org/chromium/chrome/browser/contextmenu/ContextMenuTest.java:213:
@MediumTest
On 2014/09/30 09:20:46, ankit wrote:
> On 2014/09/30 08:51:08, Bernhard Bauer wrote:
> > Why are these tests medium if e.g. testCopyLinkTextSimple() is large?
> 
> 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.

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.

[ankit, 2014-09-30 11:50:16]

@Bernhard
PTAL
I have replaced @LargeTest with @MediumTest

[Bernhard Bauer, 2014-09-30 12:52:51]

lgtm

[ankit, 2014-09-30 13:09:43]

On 2014/09/30 12:52:51, Bernhard Bauer wrote:
> lgtm
@Bernhard
Thanks.

@Ted
PTAL

[Ted C, 2014-10-01 23:45:51]

lgtm

[ankit, 2014-10-03 07:30:25]

On 2014/10/01 23:45:51, Ted C wrote:
> lgtm

@Ted
Thanks.

[ankit, 2014-10-03 07:30:30]

The CQ bit was checked by ankit2.kumar@samsung.com

[commit-bot: I haz the power, 2014-10-03 07:31:19]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/614863002/40001

[commit-bot: I haz the power, 2014-10-03 08:44:11]

Committed patchset #3 (id:40001) as 7113aaa6bb792bf89b9f910b4002dba44fec5175

[commit-bot: I haz the power, 2014-10-03 08:44:50]

Patchset 3 (id:??) landed as
https://crrev.com/524325fd4502e6394a9ec73fb8d15316df89f04b
Cr-Commit-Position: refs/heads/master@{#298015}