Remove custom getters for File.lastModified{Date}

Remove custom getters for File.lastModified{Date}

No particular need for these to be custom, so turned into File accessors
instead.

Also drop a runtime feature check for FileConstructor; it is now stable.

R=haraken
BUG=

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=179944

[sof, 2014-08-11 10:23:00]

Please take a look.

[haraken, 2014-08-11 10:39:43]

LGTM

https://codereview.chromium.org/458953002/diff/20001/Source/bindings/core/v8/...
File Source/bindings/core/v8/custom/V8FileCustom.cpp (left):

https://codereview.chromium.org/458953002/diff/20001/Source/bindings/core/v8/...
Source/bindings/core/v8/custom/V8FileCustom.cpp:96: // current time when the
last modification time is unknown.

Would you keep this comment somewhere? There was a lot of discussion about this
behavior once before.

[sof, 2014-08-11 10:58:08]

https://codereview.chromium.org/458953002/diff/20001/Source/bindings/core/v8/...
File Source/bindings/core/v8/custom/V8FileCustom.cpp (left):

https://codereview.chromium.org/458953002/diff/20001/Source/bindings/core/v8/...
Source/bindings/core/v8/custom/V8FileCustom.cpp:96: // current time when the
last modification time is unknown.
On 2014/08/11 10:39:42, haraken wrote:
> 
> Would you keep this comment somewhere? There was a lot of discussion about
this
> behavior once before.

Yes, there was. Moved (shortened) comment into the getters.

[sof, 2014-08-11 10:58:11]

The CQ bit was checked by sigbjornf@opera.com

[commit-bot: I haz the power, 2014-08-11 10:59:51]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/sigbjornf@opera.com/458953002/40001

[commit-bot: I haz the power, 2014-08-11 11:56:55]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  win_blink_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/win_blink_rel/builds/22028)

[sof, 2014-08-11 12:29:59]

The CQ bit was unchecked by sigbjornf@opera.com

[sof, 2014-08-11 12:30:30]

The CQ bit was checked by sigbjornf@opera.com

[commit-bot: I haz the power, 2014-08-11 12:31:23]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/sigbjornf@opera.com/458953002/40001

[kinuko, 2014-08-11 12:58:28]

lgtm
 CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/sigbjornf@opera.com/458953002/40001


https://codereview.chromium.org/458953002/

To unsubscribe from this group and stop receiving emails from it, send an email
to blink-reviews+unsubscribe@chromium.org.

[commit-bot: I haz the power, 2014-08-11 13:04:00]

Change committed as 179944