events: Remove KeyEvent::Copy().

It is possible to copy a key-event using the copy-constructor, without needing
a separate method for it. So remove the redundant code for copying a KeyEvent.

BUG=none
R=estade@chromium.org, sky@chromium.org

Review URL: https://codereview.chromium.org/97963002

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@238205 0039d316-1c4b-4281-b951-d872f2087c98
SVN changes up to revision 238205
[chromedriver] Fix typo in test expectations.

BUG=None
TBR=craigdh@chromium.org

Review URL: https://codereview.chromium.org/100323002

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@238206 0039d316-1c4b-4281-b951-d872f2087c98
SVN changes up to revision 238206
Fix GN deadlock on error handling.

This would deadlock in some error cases when the failing thread tried to acquire the lock to report an error, but the main thread was in the process of shutting down (due to a previous error) and blocking on the worker threads finishing while holding thc lock.

This moves the worker pool shutdown to outside the lock, and adds some extra shutdown assertions.

It also fixes a syntax error in the base Linux build file.

BUG=
R=scottmg@chromium.org

Review URL: https://codereview.chromium.org/100193002

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@238207 0039d316-1c4b-4281-b951-d872f2087c98
SVN changes up to revision 238207