commit 983c175aa134c7f6794a4b3407d7e99536b1f7f1
author Andreas Gampe <agampe@google.com> Mon Jan 23 19:46:56 2017 -0800
committer Andreas Gampe <agampe@google.com> Tue Jan 24 15:21:18 2017 -0800
tree d3f8f79f85398138af49b1bb1e43a8d9ae94972d
parent 461bbbf6f2e8dddb33a124c598e806b75929c9bb

ART: Refactor TI events

Refactor for type safety of event dispatch. Use template specialization
to ensure that only correct calls will work.

Bug: 31684920
Test: m test-art-host
Change-Id: Ifbb47447fb46dd6e799bc824a60df51b15b73618

runtime/openjdkjvmti/ti_phase.cc

diff --git a/runtime/openjdkjvmti/ti_phase.cc b/runtime/openjdkjvmti/ti_phase.cc
index 154406a..4970288 100644
--- a/runtime/openjdkjvmti/ti_phase.cc
+++ b/runtime/openjdkjvmti/ti_phase.cc
@@ -64,7 +64,7 @@
       case RuntimePhase::kStart:
         {
           art::ScopedThreadSuspension sts(art::Thread::Current(), art::ThreadState::kNative);
-          event_handler->DispatchEvent(nullptr, ArtJvmtiEvent::kVmStart, GetJniEnv());
+          event_handler->DispatchEvent<ArtJvmtiEvent::kVmStart>(nullptr, GetJniEnv());
           PhaseUtil::current_phase_ = JVMTI_PHASE_START;
         }
         break;
@@ -72,17 +72,14 @@
         {
           ScopedLocalRef<jthread> thread(GetJniEnv(), GetCurrentJThread());
           art::ScopedThreadSuspension sts(art::Thread::Current(), art::ThreadState::kNative);
-          event_handler->DispatchEvent(nullptr,
-                                       ArtJvmtiEvent::kVmInit,
-                                       GetJniEnv(),
-                                       thread.get());
+          event_handler->DispatchEvent<ArtJvmtiEvent::kVmInit>(nullptr, GetJniEnv(), thread.get());
           PhaseUtil::current_phase_ = JVMTI_PHASE_LIVE;
         }
         break;
       case RuntimePhase::kDeath:
         {
           art::ScopedThreadSuspension sts(art::Thread::Current(), art::ThreadState::kNative);
-          event_handler->DispatchEvent(nullptr, ArtJvmtiEvent::kVmDeath, GetJniEnv());
+          event_handler->DispatchEvent<ArtJvmtiEvent::kVmDeath>(nullptr, GetJniEnv());
           PhaseUtil::current_phase_ = JVMTI_PHASE_DEAD;
         }
         // TODO: Block events now.