Adding support for opening URL in new tab from inline Content

Adding support for opening URL in new tab from Content

Currently after enabling Tab manager in Chrome, links which
supposed to be opened on new tab is not working correctly.

This change takes care of opening the link in new tab whenever
applicable.

Committed: https://crrev.com/336159cf3f6740237f304e01bb6a393f03b8b135
Cr-Commit-Position: refs/heads/master@{#304012}

[AKVT, 2014-11-13 09:10:56]

ajith.v@samsung.com changed reviewers:
+ aruslan@chromium.org, bauerb@chromium.org

[AKVT, 2014-11-13 09:10:56]

@bauerb PTAL this change.

[Bernhard Bauer, 2014-11-13 09:38:03]

https://codereview.chromium.org/723973002/diff/1/chrome/android/java/src/org/...
File chrome/android/java/src/org/chromium/chrome/browser/Tab.java (right):

https://codereview.chromium.org/723973002/diff/1/chrome/android/java/src/org/...
chrome/android/java/src/org/chromium/chrome/browser/Tab.java:254: }
Empty line between methods.

https://codereview.chromium.org/723973002/diff/1/chrome/android/java/src/org/...
chrome/android/java/src/org/chromium/chrome/browser/Tab.java:258: for
(TabObserver observer : mObservers) observer.webContentsCreated(Tab.this,
You can only leave out braces if the whole things fits onto a single line.

[AKVT, 2014-11-13 09:48:55]

@Bernhard PTAL.

https://codereview.chromium.org/723973002/diff/1/chrome/android/java/src/org/...
File chrome/android/java/src/org/chromium/chrome/browser/Tab.java (right):

https://codereview.chromium.org/723973002/diff/1/chrome/android/java/src/org/...
chrome/android/java/src/org/chromium/chrome/browser/Tab.java:254: }
On 2014/11/13 09:38:02, Bernhard Bauer wrote:
> Empty line between methods.

Done.

https://codereview.chromium.org/723973002/diff/1/chrome/android/java/src/org/...
chrome/android/java/src/org/chromium/chrome/browser/Tab.java:258: for
(TabObserver observer : mObservers) observer.webContentsCreated(Tab.this,
On 2014/11/13 09:38:03, Bernhard Bauer wrote:
> You can only leave out braces if the whole things fits onto a single line.

Done.

[Bernhard Bauer, 2014-11-13 10:00:14]

lgtm

[AKVT, 2014-11-13 10:01:09]

The CQ bit was checked by ajith.v@samsung.com

[commit-bot: I haz the power, 2014-11-13 10:01:49]

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

[commit-bot: I haz the power, 2014-11-13 10:46:04]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2014-11-13 10:46:52]

Patchset 2 (id:??) landed as
https://crrev.com/336159cf3f6740237f304e01bb6a393f03b8b135
Cr-Commit-Position: refs/heads/master@{#304012}