Yura Zenevich [:yzen] (please use "needinfo") Assignee
	Description • 9 years ago

We should get rid of an option for switching between 2 types of order and just use one where ancestry is announced first.

	Yura Zenevich [:yzen] (please use "needinfo") Assignee
	Comment 1 • 9 years ago

What we actually want is to give context (document/dialog) right away, before presenting the actual virtual cursor position. The result of this improvement, the user will first hear the document utterance and then virtual cursor pivot context.

Example:

Before:
* Current page heading level one, Calendar

After:
* Calendar, Current page heading level one

	Yura Zenevich [:yzen] (please use "needinfo") Assignee
	Comment 2 • 9 years ago

Will open a separate bug for ignoring context when closing dialog.

	Yura Zenevich [:yzen] (please use "needinfo") Assignee
	Comment 3 • 9 years ago

https://treeherder.mozilla.org/#/jobs?repo=try&revision=c5b90213c5c7

	Yura Zenevich [:yzen] (please use "needinfo") Assignee
	Comment 4 • 9 years ago

https://hg.mozilla.org/integration/mozilla-inbound/rev/8fbe5948f1fa

	Carsten Book [:Tomcat]
	Comment 5 • 9 years ago

https://hg.mozilla.org/mozilla-central/rev/8fbe5948f1fa

	Yura Zenevich [:yzen] (please use "needinfo") Assignee
	Comment 6 • 9 years ago

Comment on attachment 8673706 [details] [diff] [review]
1212528 patch

Review of attachment 8673706 [details] [diff] [review]:
-----------------------------------------------------------------

This is a dependent for bug 1209062 which is scheduled for 2.2r.

	Wesley Huang [:wesley_huang] (EPM) (NI me)
	Comment 7 • 9 years ago

According to the release management planning on 2.2R[1], approval should be auto granted because 2.2R isn't FL yet.

https://wiki.mozilla.org/Release_Management/FirefoxOS/2_2_R_Schedule#Patch_Uplifts

	Carsten Book [:Tomcat]
	Comment 8 • 9 years ago

failed to apply to 2.2r 

applying 1212528.patch
patching file accessible/tests/mochitest/jsat/test_content_integration.html
Hunk #2 FAILED at 73
Hunk #3 FAILED at 100
Hunk #4 FAILED at 137
3 out of 4 hunks FAILED -- saving rejects to file accessible/tests/mochitest/jsat/test_content_integration.html.rej

	Carsten Book [:Tomcat]
	Comment 10 • 9 years ago

(In reply to Yura Zenevich [:yzen] from comment #9)
> Created attachment 8675721 [details] [diff] [review]
> 1212528 patch for b2g37_v2_2r

Hi, this failed to apply with :

patching file accessible/jsat/OutputGenerator.jsm
Hunk #1 FAILED at 67
1 out of 1 hunks FAILED -- saving rejects to file accessible/jsat/OutputGenerator.jsm.rej
patching file accessible/tests/mochitest/jsat/test_content_integration.html
Hunk #1 FAILED at 26
1 out of 5 hunks FAILED -- saving rejects to file accessible/tests/mochitest/jsat/test_content_integration.html.rej
patching file accessible/tests/mochitest/jsat/test_content_text.html
Hunk #1 FAILED at 25
1 out of 1 hunks FAILED -- saving rejects to file accessible/tests/mochitest/jsat/test_content_text.html.rej
patch failed, unable to continue (try -v)
patch failed, rejects left in working directory
errors during apply, please fix and refresh 1212528.b2g37_v2_2r.patch

	Yura Zenevich [:yzen] (please use "needinfo") Assignee
	Comment 11 • 9 years ago

(In reply to Carsten Book [:Tomcat] from comment #10)
> (In reply to Yura Zenevich [:yzen] from comment #9)
> > Created attachment 8675721 [details] [diff] [review]
> > 1212528 patch for b2g37_v2_2r
> 
> Hi, this failed to apply with :
> 
> patching file accessible/jsat/OutputGenerator.jsm
> Hunk #1 FAILED at 67
> 1 out of 1 hunks FAILED -- saving rejects to file
> accessible/jsat/OutputGenerator.jsm.rej
> patching file accessible/tests/mochitest/jsat/test_content_integration.html
> Hunk #1 FAILED at 26
> 1 out of 5 hunks FAILED -- saving rejects to file
> accessible/tests/mochitest/jsat/test_content_integration.html.rej
> patching file accessible/tests/mochitest/jsat/test_content_text.html
> Hunk #1 FAILED at 25
> 1 out of 1 hunks FAILED -- saving rejects to file
> accessible/tests/mochitest/jsat/test_content_text.html.rej
> patch failed, unable to continue (try -v)
> patch failed, rejects left in working directory
> errors during apply, please fix and refresh 1212528.b2g37_v2_2r.patch

Im not sure how this is possible. The patch part at least for OutputGenerator.jsm is exactly the same as the one that was attempted to be applied previously. But in the initial attempt it looked like it applied. Same goes for test_content_text.html ...

	Yura Zenevich [:yzen] (please use "needinfo") Assignee
	Comment 12 • 9 years ago

:Tomcat, would you know how i can check out b2g37_v2_2r locally from hg to try to apply the patch. I've been only able to do that with git mirror (https://github.com/mozilla/gecko-dev/tree/b2g37_v2_2r)

	Carsten Book [:Tomcat]
	Comment 13 • 9 years ago

Hey Yura, v2.2r is http://hg.mozilla.org/releases/mozilla-b2g37_v2_2r/

so hg clone http://hg.mozilla.org/releases/mozilla-b2g37_v2_2r/ should work :)

	Yura Zenevich [:yzen] (please use "needinfo") Assignee
	Comment 14 • 9 years ago

(In reply to Carsten Book [:Tomcat] from comment #13)
> Hey Yura, v2.2r is http://hg.mozilla.org/releases/mozilla-b2g37_v2_2r/
> 
> so hg clone http://hg.mozilla.org/releases/mozilla-b2g37_v2_2r/ should work
> :)

OK, though I don't think it's any different from my previous patch. But it definitely applies for me locally to an up to date clone of http://hg.mozilla.org/releases/mozilla-b2g37_v2_2r/

	Carsten Book [:Tomcat]
	Comment 15 • 9 years ago

interesting seems there was really a problem with my local repo. thanks yura, pushed this now as https://hg.mozilla.org/releases/mozilla-b2g37_v2_2r/rev/b26683793614

	Yura Zenevich [:yzen] (please use "needinfo") Assignee
	Comment 16 • 9 years ago

Comment on attachment 8673706 [details] [diff] [review]
1212528 patch

Review of attachment 8673706 [details] [diff] [review]:
-----------------------------------------------------------------

Removing since it's landed in 2.2r