Histogram the scheme of an origin on the 1st navigation to it.

chrome/browser/tab_contents/origins_seen_service_factory.h

+// Copyright 2016 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#ifndef CHROME_BROWSER_TAB_CONTENTS_ORIGINS_SEEN_SERVICE_FACTORY_H_
+#define CHROME_BROWSER_TAB_CONTENTS_ORIGINS_SEEN_SERVICE_FACTORY_H_
+
+#include "base/memory/singleton.h"
+#include "chrome/browser/tab_contents/origins_seen_service.h"
+#include "components/keyed_service/content/browser_context_keyed_service_factory.h"
+
+class Profile;

[Avi (use Gerrit), 2016/04/14 02:43:26]

remove this because...

[palmer, 2016/04/14 18:47:33]

Done.

+class OriginsSeenService;
+
+class OriginsSeenServiceFactory : public BrowserContextKeyedServiceFactory {
+ public:
+  static OriginsSeenService* GetForProfile(Profile* profile);

[Avi (use Gerrit), 2016/04/14 02:43:26]

You should make this be:

static OriginsSeenService* GetForBrowserContext(content::BrowserContext*
context);

[palmer, 2016/04/14 18:47:34]

Done.

+
+  static OriginsSeenServiceFactory* GetInstance();
+
+  // Used to create instances for testing.
+  static KeyedService* BuildInstanceFor(Profile* profile);

[Avi (use Gerrit), 2016/04/14 02:43:26]

and

static KeyedService* BuildInstanceFor(content::BrowserContext* context);

[palmer, 2016/04/14 18:47:33]

Done.

+
+ private:
+  friend struct base::DefaultSingletonTraits<OriginsSeenServiceFactory>;
+
+  OriginsSeenServiceFactory();
+  ~OriginsSeenServiceFactory() override;
+
+  // BrowserContextKeyedServiceFactory:
+  KeyedService* BuildServiceInstanceFor(
+      content::BrowserContext* profile) const override;
+};
+
+#endif  // CHROME_BROWSER_TAB_CONTENTS_ORIGINS_SEEN_SERVICE_FACTORY_H_