Record first run startup metrics.

chrome/browser/first_run/first_run_dialog.h

Index: chrome/browser/first_run/first_run_dialog.h
diff --git a/chrome/browser/first_run/first_run_dialog.h b/chrome/browser/first_run/first_run_dialog.h
index 76e507361f63efc237075aab06d6be407e4e540d..ad51259a61c62ea713a20940cede6a044795357f 100644
--- a/chrome/browser/first_run/first_run_dialog.h
+++ b/chrome/browser/first_run/first_run_dialog.h
@@ -10,7 +10,8 @@ class Profile;
 namespace first_run {
 
 // Shows the first run dialog. Only called if IsOrganicFirstRun() is true.
-void ShowFirstRunDialog(Profile* profile);
+// Returns true if the dialog was shown.
+bool ShowFirstRunDialog(Profile* profile);

[jeremy, 2013/05/05 12:35:56]

Rather than changing the prototype, can you #ifdef the calling code
appropriately and remove the dummy function body for this function?

[gab, 2013/05/06 16:19:54]

Removed the views implementation and moved the call to first_run_posix.cc. This
also makes it more explicit that it's broken for Linux+Aura
(http://crbug.com/234637) -- and also easier to fix since adding a views
implementation will no longer trigger this to show on Windows where it shouldn't
show (I think the EULA replaces this on Windows).

Still modified the interface to return a bool to specify whether the dialog was
shown though as there are many cases where it is not shown and this will help us
get first run stats on Linux/Mac which we wouldn't get if we just blindly assume
that calling this method resulted in a dialog being shown.

 
 }  // namespace first_run