Oilpan: Enable Oilpan by default in modules/encryptedmedia

Source/platform/ThreadTimers.cpp

Index: Source/platform/ThreadTimers.cpp
diff --git a/Source/platform/ThreadTimers.cpp b/Source/platform/ThreadTimers.cpp
index 43035709c8c1c4d16ea33166efc0f91a2ffb0d33..28c361385fe1759bf7bb5499f76e8b99ba738f47 100644
--- a/Source/platform/ThreadTimers.cpp
+++ b/Source/platform/ThreadTimers.cpp
@@ -140,6 +140,8 @@ void ThreadTimers::sharedTimerFiredInternal()
         // Catch the case where the timer asked timers to fire in a nested event loop, or we are over time limit.
         if (!m_firingTimers || timeToQuit < monotonicallyIncreasingTime())
             break;
+
+        ThreadState::current()->safePoint(ThreadState::NoHeapPointersOnStack);

[haraken, 2014/05/28 11:23:46]

The underlying issue is that a full GC is not invoked _immediately_ after
finishing the timer that called asyncGC(). (A full GC is invoked after finishing
the current event loop, which can contain a bunch of timers that can affect the
result of the GC e.g., MediaKeys::initializeNewSessionTimerFired.)

To address the issue, would it be possible to invoke a full GC _immediately_
after finishing the timer that called asyncGC()? The above change is doing that
and I confirmed that this fixes the test failure.

Note 1: I've not yet verified that it's OK to call safePoint with
NoHeapPointersOnStack from here.

Note 2: Either way, I'm planning to land this CL with a [Pass Failure] mark in
TestExpectation at the moment.

     }
 
     m_firingTimers = false;