In my opinion, extension methods are only a (pretty nice) syntactic sugar for a static helper method. Under the hood, the IL translates your extension method call to a call on the static method, so really it's just an other way to write methods.
The extension method do not have a private access to the type they are extending either, so changing a static method into an extension one will not break any encapsulation.
As for the private keyword, it is indeed the correct accessibility level since your method will not be used outside of your class.
According to MSDN's Guidelines, the major problem of extension methods is the following :
When using an extension method to extend a type whose source code you cannot change, you run the risk that a change in the implementation of the type will cause your extension method to break.
Also, if someday IWebDriver defines a new method called "WaitForTimeout", your extension method won't be called anymore... But the risk is really pretty low.
So if you think readability is increased by using that extension method, use it - you won't violate any principles or break anything, see it as a nice syntactic sugar.