(no longer active) Reporter
	Description • 14 years ago

waitForFocus is really a misnomer, because the function actually sets focus in addition to waiting for it.  One possible better name suggested by dbaron was focusAndWait.

Once bug 554873 is fixed, waitForFocus also waits for pages to be loaded all the time as well, so maybe focusAndWaitForLoad would be an even better name.

	neil@parkwaycc.co.uk
	Comment 1 • 14 years ago

How about EnsureFocus?

	Joel Maher ( :jmaher ) (UTC -8)
	Comment 3 • 11 years ago

Comment on attachment 8389117 [details] [diff] [review]
ensureFocus.patch

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

this is a great start for the simpletest file, but we have a lot of files that reference it:
http://dxr.mozilla.org/mozilla-central/search?q=waitForFocus&redirect=true

Make sure you can run some tests locally after making these changes, then I can push them to our try server to see how it works on all the tests.

	Jaspreet Singh
	Comment 4 • 11 years ago

I will make further changes in the next iteration.
Please confirm that ensureFocus() will be correct name for the function

	Joel Maher ( :jmaher ) (UTC -8)
	Comment 5 • 11 years ago

I like ensureFocus, :ehsan, any issues with it?

	(no longer active) Reporter
	Comment 6 • 11 years ago

What was wrong with focusAndWaitForLoad suggested in comment 0?  That name seems to reflect exactly what this function does.  ensureFocus doesn't have that property.

	Joel Maher ( :jmaher ) (UTC -8)
	Comment 7 • 11 years ago

I understand waitForFocus isn't the most representative name for the function, either focusAndWaitForLoad or ensureFocus.  As Ehsan filed this bug, and looking at the feature, his suggestion of focusAndWaitForLoad seems the right way to go.