Expose UkmService to content/

content/browser/web_contents/web_contents_impl.cc

Index: content/browser/web_contents/web_contents_impl.cc
diff --git a/content/browser/web_contents/web_contents_impl.cc b/content/browser/web_contents/web_contents_impl.cc
index 41778bf796aa7223f8ca8e384db023cb4509de19..59977b85d2d4e9131bf1dc2f7f318b03726dc946 100644
--- a/content/browser/web_contents/web_contents_impl.cc
+++ b/content/browser/web_contents/web_contents_impl.cc
@@ -35,6 +35,7 @@
 #include "build/build_config.h"
 #include "components/mime_util/mime_util.h"
 #include "components/rappor/public/rappor_utils.h"
+#include "components/ukm/ukm_service.h"
 #include "components/url_formatter/url_formatter.h"
 #include "content/browser/accessibility/browser_accessibility_state_impl.h"
 #include "content/browser/bad_message.h"
@@ -5576,6 +5577,12 @@ bool WebContentsImpl::AddDomainInfoToRapporSample(rappor::Sample* sample) {
   return true;
 }
 
+bool WebContentsImpl::UpdateUrlForUkmSource(ukm::UkmService* service,
+                                            int32_t ukm_source_id) {
+  service->UpdateSourceURL(ukm_source_id, GetLastCommittedURL());

[Charlie Reis, 2017/05/24 16:28:32]

I'm assuming this intentionally only uses the main frame URL, even if the
observed actions may be coming from subframes (possibly from a different
process)?  If so, let's leave a comment to that effect here, so that we don't
worry about this being a bug for subframes or OOPIFs.

[Navid Zolghadr, 2017/05/25 15:45:46]

Done.

+  return true;
+}
+
 void WebContentsImpl::FocusedNodeTouched(bool editable) {
 #if defined(OS_WIN)
   // We use the cursor position to determine where the touch occurred.