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

Side by Side Diff: components/scheduler/renderer/web_view_scheduler_impl.h

Issue 2125273002: Fix a problem with duplicate calls to FrameFetchContext::dispatchDidFail (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 5 months 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 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 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 COMPONENTS_SCHEDULER_RENDERER_WEB_VIEW_SCHEDULER_IMPL_H_ 5 #ifndef COMPONENTS_SCHEDULER_RENDERER_WEB_VIEW_SCHEDULER_IMPL_H_
6 #define COMPONENTS_SCHEDULER_RENDERER_WEB_VIEW_SCHEDULER_IMPL_H_ 6 #define COMPONENTS_SCHEDULER_RENDERER_WEB_VIEW_SCHEDULER_IMPL_H_
7 7
8 #include <memory> 8 #include <memory>
9 #include <set> 9 #include <set>
10 #include <string> 10 #include <string>
(...skipping 35 matching lines...) Expand 10 before | Expand all | Expand 10 after
46 void enableVirtualTime() override; 46 void enableVirtualTime() override;
47 bool virtualTimeAllowedToAdvance() const override; 47 bool virtualTimeAllowedToAdvance() const override;
48 void setVirtualTimePolicy(VirtualTimePolicy virtual_time_policy) override; 48 void setVirtualTimePolicy(VirtualTimePolicy virtual_time_policy) override;
49 49
50 // Virtual for testing. 50 // Virtual for testing.
51 virtual void AddConsoleWarning(const std::string& message); 51 virtual void AddConsoleWarning(const std::string& message);
52 52
53 std::unique_ptr<WebFrameSchedulerImpl> createWebFrameSchedulerImpl( 53 std::unique_ptr<WebFrameSchedulerImpl> createWebFrameSchedulerImpl(
54 base::trace_event::BlameContext* blame_context); 54 base::trace_event::BlameContext* blame_context);
55 55
56 void incrementPendingResourceLoadCount(); 56 void DidStartLoading(unsigned long identifier);
57 void decrementPendingResourceLoadCount(); 57 void DidStopLoading(unsigned long identifier);
58 58
59 private: 59 private:
60 friend class WebFrameSchedulerImpl; 60 friend class WebFrameSchedulerImpl;
61 61
62 void Unregister(WebFrameSchedulerImpl* frame_scheduler); 62 void Unregister(WebFrameSchedulerImpl* frame_scheduler);
63 63
64 AutoAdvancingVirtualTimeDomain* virtual_time_domain() const { 64 AutoAdvancingVirtualTimeDomain* virtual_time_domain() const {
65 return virtual_time_domain_.get(); 65 return virtual_time_domain_.get();
66 } 66 }
67 67
68 void setAllowVirtualTimeToAdvance(bool allow_virtual_time_to_advance); 68 void setAllowVirtualTimeToAdvance(bool allow_virtual_time_to_advance);
69 69
70 std::set<WebFrameSchedulerImpl*> frame_schedulers_; 70 std::set<WebFrameSchedulerImpl*> frame_schedulers_;
71 std::set<unsigned long> pending_fetches_;
Sami 2016/07/07 11:21:40 naming nit: how about pending_loads_ to keep the t
alex clarke (OOO till 29th) 2016/07/07 12:10:04 Done.
71 std::unique_ptr<AutoAdvancingVirtualTimeDomain> virtual_time_domain_; 72 std::unique_ptr<AutoAdvancingVirtualTimeDomain> virtual_time_domain_;
72 TaskQueue::PumpPolicy virtual_time_pump_policy_; 73 TaskQueue::PumpPolicy virtual_time_pump_policy_;
73 blink::WebView* web_view_; 74 blink::WebView* web_view_;
74 RendererSchedulerImpl* renderer_scheduler_; 75 RendererSchedulerImpl* renderer_scheduler_;
75 int pending_resource_load_count_;
76 VirtualTimePolicy virtual_time_policy_; 76 VirtualTimePolicy virtual_time_policy_;
77 bool page_visible_; 77 bool page_visible_;
78 bool disable_background_timer_throttling_; 78 bool disable_background_timer_throttling_;
79 bool allow_virtual_time_to_advance_; 79 bool allow_virtual_time_to_advance_;
80 80
81 DISALLOW_COPY_AND_ASSIGN(WebViewSchedulerImpl); 81 DISALLOW_COPY_AND_ASSIGN(WebViewSchedulerImpl);
82 }; 82 };
83 83
84 } // namespace scheduler 84 } // namespace scheduler
85 85
86 #endif // COMPONENTS_SCHEDULER_RENDERER_WEB_VIEW_SCHEDULER_IMPL_H_ 86 #endif // COMPONENTS_SCHEDULER_RENDERER_WEB_VIEW_SCHEDULER_IMPL_H_
OLDNEW
« no previous file with comments | « components/scheduler/renderer/web_frame_scheduler_impl.cc ('k') | components/scheduler/renderer/web_view_scheduler_impl.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698