[Translate] Integrate TranslateEventProto UMA logging into TranslateManager.

components/translate/core/browser/BUILD.gn

Index: components/translate/core/browser/BUILD.gn
diff --git a/components/translate/core/browser/BUILD.gn b/components/translate/core/browser/BUILD.gn
index dfdc5a5320a4899fd82511ee76225cf55dcecb20..13c651779afc452aa82191316e0649c0b8ad8d59 100644
--- a/components/translate/core/browser/BUILD.gn
+++ b/components/translate/core/browser/BUILD.gn
@@ -67,6 +67,10 @@ static_library("browser") {
     "//url",
   ]
 
+  public_deps = [
+    "//components/metrics/proto",

[hamelphi, 2016/10/13 15:13:56]

This dependency is also in deps. I suppose we only need it once. If I understand
public_deps correctly, you can remove the deps here and in unit_test.

[Roger McFarlane (Chromium), 2016/10/13 19:30:16]

fixed

the public_deps is needed: public_deps is transitive, deps is not.

public_deps is needed because the translate event proto header is included in
translate_manager.h, which is included elsewhere.

I could remove the public_deps by holding the proto via a unique_ptr, but the
lifecycle of the proto and the translate manager are exactly the same, so it
makes more sense for it to be a direct member.

[hamelphi, 2016/10/13 20:47:02]

My point was that you probably can remove "//components/metrics/proto" in the
'deps' section since you add it to public_deps. But I might be wrong. In any
case, I don't feel strongly.

[Roger McFarlane (Chromium), 2016/10/14 19:59:05]

ah... yeah, that's what I did. I thought you were suggesting the opposite. sorry
for the confusion.

+  ]
+
   if (!use_aura) {
     sources += [
       "translate_infobar_delegate.cc",