Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(153)

Side by Side Diff: Source/core/frame/WindowTimers.idl

Issue 721033004: Implement WindowTimers.set{Timeout,Interval} without [Custom] (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@idl-overload-with-variadic
Patch Set: rebased Created 6 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « Source/core/frame/DOMWindowTimers.cpp ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 /* 1 /*
2 * Copyright (C) 2006, 2007, 2008, 2009 Apple Inc. All rights reserved. 2 * Copyright (C) 2006, 2007, 2008, 2009 Apple Inc. All rights reserved.
3 * Copyright (C) 2011 Google Inc. All rights reserved. 3 * Copyright (C) 2011 Google Inc. All rights reserved.
4 * Copyright (C) 2013 Samsung Electronics. All rights reserved. 4 * Copyright (C) 2013 Samsung Electronics. All rights reserved.
5 * 5 *
6 * Redistribution and use in source and binary forms, with or without 6 * Redistribution and use in source and binary forms, with or without
7 * modification, are permitted provided that the following conditions 7 * modification, are permitted provided that the following conditions
8 * are met: 8 * are met:
9 * 1. Redistributions of source code must retain the above copyright 9 * 1. Redistributions of source code must retain the above copyright
10 * notice, this list of conditions and the following disclaimer. 10 * notice, this list of conditions and the following disclaimer.
(...skipping 14 matching lines...) Expand all
25 * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. 25 * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
26 */ 26 */
27 27
28 // http://www.whatwg.org/specs/web-apps/current-work/multipage/timers.html#windo wtimers 28 // http://www.whatwg.org/specs/web-apps/current-work/multipage/timers.html#windo wtimers
29 29
30 [ 30 [
31 ImplementedAs=DOMWindowTimers, 31 ImplementedAs=DOMWindowTimers,
32 LegacyTreatAsPartialInterface, 32 LegacyTreatAsPartialInterface,
33 NoInterfaceObject, // Always used on target of 'implements' 33 NoInterfaceObject, // Always used on target of 'implements'
34 ] interface WindowTimers { 34 ] interface WindowTimers {
35 // FIXME: currently using [Custom] and |any| because overload algorithm
36 // can't handle Function/DOMString overload properly
37 // http://crbug.com/293561
38 // FIXME: would be clearer as a union type, like: 35 // FIXME: would be clearer as a union type, like:
39 // typedef (Function or DOMString) Handler 36 // typedef (Function or DOMString) Handler
40 // Needs spec update and better union support: http://crbug.com/240176 37 // Needs spec update and better union support: http://crbug.com/240176
41 [Custom] long setTimeout(any handler, [Default=Undefined] optional long time out); 38 [CallWith=ScriptState] long setTimeout(Function handler, optional long timeo ut = 0, any... arguments);
42 void clearTimeout([Default=Undefined] optional long handle); 39 [CallWith=ScriptState] long setTimeout(DOMString handler, optional long time out = 0, any... arguments);
43 [Custom] long setInterval(any handler, [Default=Undefined] optional long tim eout); 40 void clearTimeout(optional long handle = 0);
44 void clearInterval([Default=Undefined] optional long handle); 41 [CallWith=ScriptState] long setInterval(Function handler, optional long time out = 0, any... arguments);
42 [CallWith=ScriptState] long setInterval(DOMString handler, optional long tim eout = 0, any... arguments);
43 void clearInterval(optional long handle = 0);
45 }; 44 };
OLDNEW
« no previous file with comments | « Source/core/frame/DOMWindowTimers.cpp ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698