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

Side by Side Diff: third_party/WebKit/Source/core/dom/TaskRunnerHelper.h

Issue 2556993005: [blink] Split TaskType::Internal into InternalTimer and InternalLoading. (Closed)
Patch Set: addressed skyostil@'s comments Created 4 years 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
OLDNEW
1 // Copyright 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef TaskRunnerHelper_h 5 #ifndef TaskRunnerHelper_h
6 #define TaskRunnerHelper_h 6 #define TaskRunnerHelper_h
7 7
8 #include "core/CoreExport.h" 8 #include "core/CoreExport.h"
9 #include "wtf/Allocator.h" 9 #include "wtf/Allocator.h"
10 #include "wtf/HashTraits.h" 10 #include "wtf/HashTraits.h"
(...skipping 19 matching lines...) Expand all
30 Microtask, 30 Microtask,
31 Timer, 31 Timer,
32 RemoteEvent, 32 RemoteEvent,
33 WebSocket, 33 WebSocket,
34 PostedMessage, 34 PostedMessage,
35 UnshippedPortMessage, 35 UnshippedPortMessage,
36 36
37 // Other internal tasks that cannot fit any of the above task runners 37 // Other internal tasks that cannot fit any of the above task runners
38 // can be posted here, but the usage is not encouraged. The task runner 38 // can be posted here, but the usage is not encouraged. The task runner
39 // may be throttled. 39 // may be throttled.
40 Internal, 40 //
41 // InternalLoading type should be used for all tasks associated with
42 // loading page content, InternalTimer should be used for all other purposes.
43 InternalTimer,
44 InternalLoading,
haraken 2016/12/09 00:04:27 Shall we rename these to UnspecedTimerTask and Uns
41 45
42 // Tasks that must not be throttled should be posted here, but the usage 46 // Tasks that must not be throttled should be posted here, but the usage
43 // should be very limited. 47 // should be very limited.
44 Unthrottled, 48 Unthrottled,
45 49
46 // Tasks that any other TaskType is not assigned to. This should be 50 // Tasks that any other TaskType is not assigned to. This should be
47 // transitional and should be removed. 51 // transitional and should be removed.
48 Unspecified, 52 Unspecified,
49 }; 53 };
50 54
(...skipping 15 matching lines...) Expand all
66 public: 70 public:
67 static WebTaskRunner* get(TaskType, LocalFrame*); 71 static WebTaskRunner* get(TaskType, LocalFrame*);
68 static WebTaskRunner* get(TaskType, Document*); 72 static WebTaskRunner* get(TaskType, Document*);
69 static WebTaskRunner* get(TaskType, ExecutionContext*); 73 static WebTaskRunner* get(TaskType, ExecutionContext*);
70 static WebTaskRunner* get(TaskType, ScriptState*); 74 static WebTaskRunner* get(TaskType, ScriptState*);
71 }; 75 };
72 76
73 } // namespace blink 77 } // namespace blink
74 78
75 #endif 79 #endif
OLDNEW
« no previous file with comments | « third_party/WebKit/Source/core/dom/Document.cpp ('k') | third_party/WebKit/Source/core/dom/TaskRunnerHelper.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698