Make test runner library compiled with the chromium-style clang plugin on

Sprinkle the code with OVERRIDE and move method bodies from headers to
cc files.

BUG=331342
R=battre@chromium.org

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

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@242885 0039d316-1c4b-4281-b951-d872f2087c98
Revert 164972 "Add WebTestRunner namespace to the list of "bad n..."

> Add WebTestRunner namespace to the list of "bad namespaces" so we don't warn about not putting OVERRIDE on virtual methods.
> 
> The WebTestRunner namespace is similar to the WebKit namespace in that its classes change often and are upstream in WebKit
> 
> Review URL: https://codereview.chromium.org/11339049

TBR=erg@chromium.org
BUG=331342

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

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@242886 0039d316-1c4b-4281-b951-d872f2087c98
SVN changes up to revision 242886
Only close chrome://settings/clearBrowserData if it is visible

The chrome://settings/clearBrowserData overlay is closed when the clear
operation has finished. This CL ensures that we only close the overlay if it is
actually visible. If the user navigated to a different overlay, that overlay
should not be closed when the browsing data has been cleared.

BUG=330706
R=dbeam@chromium.org

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

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