Adding Date::ResetCache() API so that the cache values in the Date object...

include/v8.h

 // Copyright 2007-2009 the V8 project authors. All rights reserved.
 // Redistribution and use in source and binary forms, with or without
 // modification, are permitted provided that the following conditions are
 // met:
 //
 //     * Redistributions of source code must retain the above copyright
 //       notice, this list of conditions and the following disclaimer.
 //     * Redistributions in binary form must reproduce the above
 //       copyright notice, this list of conditions and the following
 //       disclaimer in the documentation and/or other materials provided
@@ skipping 1337 matching lines @@
  public:
   V8EXPORT static Local<Value> New(double time);
 
   /**
    * A specialization of Value::NumberValue that is more efficient
    * because we know the structure of this object.
    */
   V8EXPORT double NumberValue() const;
 
   static inline Date* Cast(v8::Value* obj);
+
+  /**

[Søren Thygesen Gjesse, 2010/12/17 10:54:55]

Please start the comment with something like this "Notification that the
embedder has changed the time zone or daylight savings." and the follow with "V8
keeps a cache ...".

Also note that this API should not be called more that needed as otherwise the
performance of date operation will suffer.

[marklam, 2010/12/17 11:12:29]

Will do.  I do have one concern with not knowing what the best name for the
notification would be.

+   * Resets the Date cache for the current context so that date/time
+   * changes due to daylight savings, or timezone changes would be
+   * reflected in the Date object.
+   */
+  V8EXPORT static void ResetCache();

[Søren Thygesen Gjesse, 2010/12/17 10:54:55]

I think the name ResetCache here is a bit to low-level. How about
TimeZoneChangeNotification?

[marklam, 2010/12/17 11:12:29]

The implementation is really a reset of all values that is cached in the Date
object.  Would calling it TimeZoneChangeNotification be misleading?  For one,
this notification also covers changes in Daylight Savings time, local time
offset, etc.  Would all those fall under TimeZoneChange?  I'm fine with this
name if you think that the name covers all the types of changes that the Date
object needs to be notified of.

Alternatively, how about calling DateTimeChangeNotification?  That would
describe what this function is trying to do, won't it?

+
  private:
   V8EXPORT static void CheckCast(v8::Value* obj);
 };
 
 
 /**
  * An instance of the built-in RegExp constructor (ECMA-262, 15.10).
  */
 class RegExp : public Value {
  public:
@@ skipping 2399 matching lines @@
 
 
 }  // namespace v8
 
 
 #undef V8EXPORT
 #undef TYPE_CHECK
 
 
 #endif  // V8_H_