Context Menu "Open image" doesn't make sense in a Image URL Tab

Context Menu "Open image" doesn't make sense in a Image URL Tab

When only image is opened then on long press context menu appears.
From context menu on selection of "open image" page reloads which
does not make sense.Added condition to hide option "open image"
if page url and image url for which context menu appears is same.

BUG=270682
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=283534

[ankit, 2014-07-16 07:11:07]

PTAL

[Bernhard Bauer, 2014-07-16 08:05:59]

LGTM, thanks!

[Bernhard Bauer, 2014-07-16 08:06:27]

https://codereview.chromium.org/396063009/diff/1/chrome/android/java/src/org/...
File
chrome/android/java/src/org/chromium/chrome/browser/contextmenu/ChromeContextMenuPopulator.java
(right):

https://codereview.chromium.org/396063009/diff/1/chrome/android/java/src/org/...
chrome/android/java/src/org/chromium/chrome/browser/contextmenu/ChromeContextMenuPopulator.java:78:
if (mDelegate.getPageUrl().equals(params.getSrcUrl())) {
You could add a comment here though to explain why we do this.

[ankit, 2014-07-16 08:57:11]

@Bernhard
PTAL new patch set.

[Bernhard Bauer, 2014-07-16 09:01:12]

LGTM!

[ankit, 2014-07-16 09:01:52]

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

[commit-bot: I haz the power, 2014-07-16 09:03:36]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/ankit2.kumar@samsung.com/396063009/20001

[commit-bot: I haz the power, 2014-07-16 21:58:18]

Change committed as 283534