Oilpan: more lazy sweep ASan exemptions.

Source/platform/Timer.h

 /*
  * Copyright (C) 2006 Apple Computer, Inc.  All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
  *    notice, this list of conditions and the following disclaimer in the
@@ skipping 43 matching lines @@
     {
         start(interval, 0, caller);
     }
 
     void stop();
     bool isActive() const;
     const WebTraceLocation& location() const { return m_location; }
 
     double nextFireInterval() const;
     double nextUnalignedFireInterval() const;
+    NO_LAZY_SWEEP_SANITIZE_ADDRESS
     double repeatInterval() const { return m_repeatInterval; }

[haraken, 2015/05/20 22:27:26]

I'm a bit afraid that this function is inlined in Release builds, meaning that
we might need to add NO_LAZY_SWEEP_SANITIZE_ADDRESS to the caller sites of this
function.

[sof, 2015/05/21 05:31:36]

Maybe so. I don't know what it does, but if ASan doesn't restrict inlining, I
would expect/hope that annotations like this carry over across to the inline
site.

 
     void augmentRepeatInterval(double delta) {
         setNextFireTime(m_nextFireTime + delta);
         m_repeatInterval += delta;
     }
 
     void didChangeAlignmentInterval();
 
 private:
     virtual void fired() = 0;
@@ skipping 86 matching lines @@
 NO_LAZY_SWEEP_SANITIZE_ADDRESS
 inline bool TimerBase::isActive() const
 {
     ASSERT(m_thread == currentThread());
     return m_nextFireTime;
 }
 
 }
 
 #endif