content/test/data/accessibility/ins-expected-android.txt - Issue 543953003: Adding DumpAccessibilityTree Tests (8 of 20)

Side by Side Diff

Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.

Keyboard Shortcuts

	File
u :	up to issue
j / k :	jump to file after / before current file
J / K :	jump to next file with a comment after / before current file
	Side-by-side diff
i :	toggle intra-line diffs
e :	expand all comments
c :	collapse all comments
s :	toggle showing all comments
n / p :	next / previous diff chunk or comment
N / P :	next / previous comment
<Up> / <Down> :	next / previous line

	Issue
u :	up to list of issues
j / k :	jump to patch after / before current patch
o / <Enter> :	open current patch in side-by-side view
i :	open current patch in unified diff view

	Issue List
j / k :	jump to issue after / before current issue
o / <Enter> :	open current issue

Side by Side Diff: content/test/data/accessibility/ins-expected-android.txt

Issue 543953003: Adding DumpAccessibilityTree Tests (8 of 20) (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master

Patch Set: Updating aria-row to get correct results Created 6 years, 3 months ago

Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.

Jump to:

OLD	NEW
(Empty)
	1 android.webkit.WebView focusable focused scrollable

	2 android.view.View clickable name='My favorite browser isABCChrome!'
	dmazzoni 2014/09/11 17:02:38 This is a bug that the Android text runs together This is a bug that the Android text runs together like this. The name should be 'My favorite browser is ABC Chrome!', right? What you should do in this case is make the expected-android.txt file look the way it SHOULD be, not the way it actually is - then add a SKIP line to the top of the file, and file a new bug for it. This saves a lot of work for whoever fixes the bug - they have a test that shows them exactly what happens now, what should happen, and they just have to delete the "skip" line when they fix it.
OLD	NEW