Disable WebViewFocusTest on Site Isolation FYI bots.

Disable WebViewFocusTest on Site Isolation FYI bots.

The WebViewFocusTest class is based on WebViewTest, which is currently
disabled on the Site Isolation FYI bots. Therefore there's no real
chance of it working until WebViewTest is fixed for those bots.

This CL adds TODOs to both test classes, as well as disabling
WebViewFocusTest on the Site Isolation FYI bots as well.

BUG=none
TBR=jam@chromium.org

Committed: https://crrev.com/ee5628b34fdc33b4a070fb65f6617e440ed90dfc
Cr-Commit-Position: refs/heads/master@{#335776}

[wjmaclean, 2015-06-23 14:50:00]

wjmaclean@chromium.org changed reviewers:
+ alexmos@chromium.org

[wjmaclean, 2015-06-23 14:50:21]

The CQ bit was checked by wjmaclean@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2015-06-23 14:50:37]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1201213002/1

[wjmaclean, 2015-06-23 14:50:57]

alexmos@ - I've discussed this with nasko@, please take a look.

[commit-bot: I haz the power, 2015-06-23 15:55:32]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-06-23 15:55:33]

Dry run: This issue passed the CQ dry run.

[alexmos, 2015-06-23 16:17:28]

LGTM

[Nico, 2015-06-23 18:16:13]

thakis@chromium.org changed reviewers:
+ maruel@chromium.org, thakis@chromium.org

[Nico, 2015-06-23 18:16:13]

maruel, are these .json files supposed to be updated manually?

https://codereview.chromium.org/1201213002/diff/1/chrome/browser/apps/guest_v...
File chrome/browser/apps/guest_view/web_view_browsertest.cc (right):

https://codereview.chromium.org/1201213002/diff/1/chrome/browser/apps/guest_v...
chrome/browser/apps/guest_view/web_view_browsertest.cc:358: // that happens.
please file a tracking bug for this, and reference it here and below

[M-A Ruel, 2015-06-23 18:37:25]

On 2015/06/23 18:16:13, Nico wrote:
> maruel, are these .json files supposed to be updated manually?

I have exactly zero knowledge about 'Site Isolation Linux'. It is blacklisted by
manage.py. But yes the change James is doing looks fine.

[Nico, 2015-06-23 18:45:08]

On 2015/06/23 18:37:25, M-A Ruel wrote:
> On 2015/06/23 18:16:13, Nico wrote:
> > maruel, are these .json files supposed to be updated manually?
> 
> I have exactly zero knowledge about 'Site Isolation Linux'. It is blacklisted
by
> manage.py. But yes the change James is doing looks fine.

Lgtm with the tracking bug I requested above then

[wjmaclean, 2015-06-23 21:15:59]

The CQ bit was checked by wjmaclean@chromium.org

[wjmaclean, 2015-06-23 21:15:59]

The patchset sent to the CQ was uploaded after l-g-t-m from thakis@chromium.org,
alexmos@chromium.org
Link to the patchset: https://codereview.chromium.org/1201213002/#ps20001
(title: "Add bug tracking info.")

[commit-bot: I haz the power, 2015-06-23 21:17:52]

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

[commit-bot: I haz the power, 2015-06-23 22:26:39]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-06-23 22:27:31]

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