Worked around a trybot change by replacing spaces with "+" for the indentation in accessibility bro…

Worked around a trybot change by replacing spaces with "+" for the indentation in accessibility browser tests.

R=dmazzoni

Committed: https://crrev.com/1608a41fc82eb51414f6412fabbd3adebdfe51c2
Cr-Commit-Position: refs/heads/master@{#315635}

[dmazzoni, 2015-02-09 17:00:13]

lgtm++

If it isn't too much trouble, I might prefer two plus signs, or a plus and a
space, to make it more visually readable. A wider indent is easier to see
visually than a single-character indent.

Maybe a plus and a space per indent would work well, would that be okay?

If it'd be a huge pain, this is okay. Note that you may have to rebase one or
two tests depending on what lands between when you wrote this change and when
this lands.

+je_julie.kim as FYI

There are a couple of other changes in the commit queue right now that add or
modify DumpAccessibilityTree tests. My suggestion is to run try jobs to see that
this change is basically valid, then rebase your change so it's up-to-the-minute
and land it manually using "git cl dcommit":

http://www.chromium.org/developers/contributing-code/direct-commit

[chromium-reviews, 2015-02-09 17:09:24]

Sure. I'll change +s to a + and a space and wait for any patches to land 
before committing.

To unsubscribe from this group and stop receiving emails from it, send an email
to chromium-reviews+unsubscribe@chromium.org.

[je_julie(Not used), 2015-02-10 05:47:30]

On 2015/02/09 17:09:24, chromium-reviews wrote:
> Sure. I'll change +s to a + and a space and wait for any patches to land 
> before committing.
> 
> To unsubscribe from this group and stop receiving emails from it, send an
email
> to mailto:chromium-reviews+unsubscribe@chromium.org.

Hi nektarios,
I just landed aria/aria-dropeffect.html at crrev.com/874893003/.
Please include that TC to your rebase.
Thanks,

[nektarios, 2015-02-10 17:39:09]

New patchsets have been uploaded after l-g-t-m from dmazzoni@chromium.org

[chromium-reviews, 2015-02-10 17:44:53]

Sure. I rebased to take care of latest submits.
@dmazzoni
I am also using two +s instead of one for each level of indentation.
I considered using a plus and a space, but developers who use screen 
readers might find it harder to verify the presence of such a 
combinations. Multiple +s are announced by the screen reader whilst +s 
with spaces might not be announced straight away.

To unsubscribe from this group and stop receiving emails from it, send an email
to chromium-reviews+unsubscribe@chromium.org.

[nektarios, 2015-02-10 17:55:37]

The CQ bit was checked by nektar@chromium.org

[commit-bot: I haz the power, 2015-02-10 17:56:47]

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

[commit-bot: I haz the power, 2015-02-10 19:13:56]

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

[commit-bot: I haz the power, 2015-02-10 19:13:58]

Try jobs failed on following builders:
  mac_chromium_rel_ng on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[nektarios, 2015-02-10 19:18:12]

The CQ bit was checked by nektar@chromium.org

[commit-bot: I haz the power, 2015-02-10 19:19:26]

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

[commit-bot: I haz the power, 2015-02-10 21:06:05]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2015-02-10 21:07:12]

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