Expose UkmService to content/

content/browser/web_contents/web_contents_impl.cc

 // Copyright (c) 2012 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.
 
 #include "content/browser/web_contents/web_contents_impl.h"
 
 #include <stddef.h>
 
 #include <cmath>
 #include <utility>
@@ skipping 17 matching lines @@
 #include "base/strings/string16.h"
 #include "base/strings/string_number_conversions.h"
 #include "base/strings/string_util.h"
 #include "base/strings/utf_string_conversions.h"
 #include "base/threading/thread_task_runner_handle.h"
 #include "base/time/time.h"
 #include "base/trace_event/trace_event.h"
 #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"
 #include "content/browser/browser_plugin/browser_plugin_embedder.h"
 #include "content/browser/browser_plugin/browser_plugin_guest.h"
 #include "content/browser/child_process_security_policy_impl.h"
 #include "content/browser/devtools/render_frame_devtools_agent_host.h"
 #include "content/browser/dom_storage/dom_storage_context_wrapper.h"
 #include "content/browser/dom_storage/session_storage_namespace_impl.h"
 #include "content/browser/download/download_stats.h"
@@ skipping 5521 matching lines @@
     binding_sets_.erase(it);
 }
 
 bool WebContentsImpl::AddDomainInfoToRapporSample(rappor::Sample* sample) {
   sample->SetStringField("Domain", ::rappor::GetDomainAndRegistrySampleFromGURL(
                                        GetLastCommittedURL()));
 
   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.
   RenderWidgetHostView* view = GetRenderWidgetHostView();
   if (!view)
     return;
   POINT cursor_pos = {};
   ::GetCursorPos(&cursor_pos);
   float scale = GetScaleFactorForView(view);
   gfx::Point location_dips_screen =
@@ skipping 42 matching lines @@
     RenderViewHost* render_view_host = RenderViewHost::From(render_widget_host);
     if (!render_view_host)
       continue;
     render_view_host_set.insert(render_view_host);
   }
   for (RenderViewHost* render_view_host : render_view_host_set)
     render_view_host->OnWebkitPreferencesChanged();
 }
 
 }  // namespace content