Fix implicit scoped_refptr<T> to T* conversion in TestMetroViewerProcessHost

ash/test/test_metro_viewer_process_host.cc

Index: ash/test/test_metro_viewer_process_host.cc
diff --git a/ash/test/test_metro_viewer_process_host.cc b/ash/test/test_metro_viewer_process_host.cc
index 156cf99770c84fe96e732ff9bba86068f489614b..00c00d257c18bb4af126c45ec7618befe7143228 100644
--- a/ash/test/test_metro_viewer_process_host.cc
+++ b/ash/test/test_metro_viewer_process_host.cc
@@ -14,9 +14,8 @@ namespace ash {
 namespace test {
 
 TestMetroViewerProcessHost::TestMetroViewerProcessHost(
-    base::SingleThreadTaskRunner* ipc_task_runner)
-    : MetroViewerProcessHost(ipc_task_runner),
-      closed_unexpectedly_(false) {
+    const scoped_refptr<base::SingleThreadTaskRunner>& ipc_task_runner)
+    : MetroViewerProcessHost(ipc_task_runner), closed_unexpectedly_(false) {

[Daniel Erat, 2014/11/24 18:53:55]

nit: back to one per line since they don't all fit on the same line as the
signature

(does clang-format still do this? :-( )

[dcheng, 2014/11/24 19:02:36]

Yes. I'd prefer not to fight clang-format, because that means I'll have to keep
fighting clang-format in every patch that ever touches this code.

I don't think this is actually against the style guide:
http://google-styleguide.googlecode.com/svn/trunk/cppguide.html#Constructor_I...
states that "[c]onstructor initializer lists can be all on one line". It does
not mention anything about the initializer list having to be on the same line as
all the parameters. I see no Chromium-specific style guidelines that would apply
in this situation.

[Nico, 2014/11/24 19:05:22]

I think clang-format used to do one-per-line-or-all-on-one-line, and then
pkasting decreed that that isn't correct and clang-format changed in response to
that, see: https://code.google.com/p/chromium/issues/detail?id=430111

So I think this is working as intended.

 }
 
 TestMetroViewerProcessHost::~TestMetroViewerProcessHost() {