Distinguish actions registered with setTimeout() and setInterval().

Source/core/page/DOMTimer.h

 /*
  * Copyright (C) 2008 Apple 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 22 matching lines @@
 #include "wtf/Compiler.h"
 #include "wtf/OwnPtr.h"
 #include "wtf/PassOwnPtr.h"
 
 namespace WebCore {
 
 class ScriptExecutionContext;
 
 class DOMTimer : public SuspendableTimer {
 public:
+    enum TimerType {
+        TimerTypeTimeout,
+        TimerTypeInterval

[Ken Russell (switch to Gerrit), 2013/07/18 19:05:13]

This seems verbose; since it's scoped in DOMTimer, can it not just be enum Type,
with TimeoutType and IntervalType members?

[Yuta Kitamura, 2013/07/19 05:14:08]

Done.

+    };
+
     // Creates a new timer owned by the ScriptExecutionContext, starts it and returns its ID.
-    static int install(ScriptExecutionContext*, PassOwnPtr<ScheduledAction>, int timeout, bool singleShot);
-    static void removeByID(ScriptExecutionContext*, int timeoutID);
+    static int install(ScriptExecutionContext*, TimerType, PassOwnPtr<ScheduledAction>, int timeout);
+    static void removeByIDIfTypeMatches(ScriptExecutionContext*, TimerType, int timeoutID);
 
     virtual ~DOMTimer();
 
     int timeoutID() const;
+    TimerType timerType() const;
 
     // ActiveDOMObject
     virtual void contextDestroyed() OVERRIDE;
     virtual void stop() OVERRIDE;
 
     virtual void reportMemoryUsage(MemoryObjectInfo*) const OVERRIDE;
 
     // The following are essentially constants. All intervals are in seconds.
     static double hiddenPageAlignmentInterval();
     static double visiblePageAlignmentInterval();
 
 private:
     friend class ScriptExecutionContext; // For create().
 
     // Should only be used by ScriptExecutionContext.
-    static PassOwnPtr<DOMTimer> create(ScriptExecutionContext* context, PassOwnPtr<ScheduledAction> action, int timeout, bool singleShot, int timeoutID)
+    static PassOwnPtr<DOMTimer> create(ScriptExecutionContext* context, TimerType timerType, PassOwnPtr<ScheduledAction> action, int timeout, int timeoutID)
     {
-        return adoptPtr(new DOMTimer(context, action, timeout, singleShot, timeoutID));
+        return adoptPtr(new DOMTimer(context, timerType, action, timeout, timeoutID));
     }
 
-    DOMTimer(ScriptExecutionContext*, PassOwnPtr<ScheduledAction>, int interval, bool singleShot, int timeoutID);
-    virtual void fired();
+    DOMTimer(ScriptExecutionContext*, TimerType, PassOwnPtr<ScheduledAction>, int interval, int timeoutID);
+    virtual void fired() OVERRIDE;
 
-    // Retuns timer fire time rounded to the next multiple of timer alignment interval.
-    virtual double alignedFireTime(double) const;
+    // Returns timer fire time rounded to the next multiple of timer alignment interval.
+    virtual double alignedFireTime(double) const OVERRIDE;
 
+    TimerType m_timerType;
     int m_timeoutID;
     int m_nestingLevel;
     OwnPtr<ScheduledAction> m_action;
     RefPtr<UserGestureToken> m_userGestureToken;
 };
 
 } // namespace WebCore
 
 #endif // DOMTimer_h