fix(protractor): Shows locator used when a timeout occur.

[luishvz]

Fixes #2996

[googlebot]

Thanks for your pull request. It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

📝 Please visit https://cla.developers.google.com/ to sign.

Once you've signed, please reply here (e.g. I signed it!) and we'll verify. Thanks.

---

• If you've already signed a CLA, it's possible we don't have your GitHub username or you're using a different email address. Check your existing CLA data and verify that your email is set on your git commits.
• If you signed the CLA as a corporation, please let us know the company's name.

[googlebot]

Thanks for your pull request. It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

📝 Please visit https://cla.developers.google.com/ to sign.

Once you've signed, please reply here (e.g. I signed it!) and we'll verify. Thanks.

---

• If you've already signed a CLA, it's possible we don't have your GitHub username or you're using a different email address. Check your existing CLA data and verify that your email is set on your git commits.
• If you signed the CLA as a corporation, please let us know the company's name.

[luishvz]

Signed it!

[googlebot]

CLAs look good, thanks!

[googlebot]

CLAs look good, thanks!

[sjelin]

Maybe some newlines here? Also, the description won't always be a locator

[luishvz]

@sjelin modified the parameter sent to waitForAngular() and now the description is only added to error message when is a locator.

Now error message looks like this.
Failed: Timed out waiting for Protractor to synchronize with the page after 11 seconds. Please see https://github.com/angular/protractor/blob/master/docs/faq.md - Locator: by.binding("slowAngularTimeoutPromiseStatus"). The following tasks were pending: - $timeout: function () { // intentionally empty }

[cnishina]

closed with b6f1a63

[juliemr]

For the future - changes like this are not a 'fix', since it's not fixing a bug. I would call it a 'feat' or 'chore'.

[luishvz]

got it @juliemr , thanks.