Add more tests to arc_navigation_throttle.cc

components/arc/intent_helper/arc_intent_helper_bridge.h

Index: components/arc/intent_helper/arc_intent_helper_bridge.h
diff --git a/components/arc/intent_helper/arc_intent_helper_bridge.h b/components/arc/intent_helper/arc_intent_helper_bridge.h
index 92d9a70f5f24d53bde57198ab30194f0fc3a850a..39369f8cd69244d25e731f8a52ddb80e0c9a6f2d 100644
--- a/components/arc/intent_helper/arc_intent_helper_bridge.h
+++ b/components/arc/intent_helper/arc_intent_helper_bridge.h
@@ -88,6 +88,9 @@ class ArcIntentHelperBridge
       const std::string& method_name_for_logging,
       uint32_t min_instance_version);
 
+  static constexpr char kArcIntentHelperPackageName[] =

[Luis Héctor Chávez, 2016/10/24 20:24:34]

would this cause an ODR violation?

[Yusuke Sato, 2016/10/24 22:27:16]

does it? I thought if the variable is odr-used, a definition in .cc was required
(but I may be wrong.)

anyway, I've changed it to const. No class in Chromium seems to have static
constexpr char[], and introducing it here seems like a bad idea. Done.

+      "org.chromium.arc.intent_helper";
+
  private:
   mojo::Binding<mojom::IntentHelperHost> binding_;
   scoped_refptr<ActivityIconLoader> icon_loader_;