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

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

Issue 1865913005: Nuke WebPassOwnPtr<T> and replace it with std::unique_ptr<T>. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: rebase Created 4 years, 8 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 #include "components/scheduler/renderer/web_view_scheduler_impl.h" 5 #include "components/scheduler/renderer/web_view_scheduler_impl.h"
6 6
7 #include "base/logging.h" 7 #include "base/logging.h"
8 #include "components/scheduler/base/virtual_time_domain.h" 8 #include "components/scheduler/base/virtual_time_domain.h"
9 #include "components/scheduler/child/scheduler_tqm_delegate.h" 9 #include "components/scheduler/child/scheduler_tqm_delegate.h"
10 #include "components/scheduler/renderer/auto_advancing_virtual_time_domain.h" 10 #include "components/scheduler/renderer/auto_advancing_virtual_time_domain.h"
(...skipping 44 matching lines...) Expand 10 before | Expand all | Expand 10 after
55 scoped_ptr<WebFrameSchedulerImpl> 55 scoped_ptr<WebFrameSchedulerImpl>
56 WebViewSchedulerImpl::createWebFrameSchedulerImpl( 56 WebViewSchedulerImpl::createWebFrameSchedulerImpl(
57 base::trace_event::BlameContext* blame_context) { 57 base::trace_event::BlameContext* blame_context) {
58 scoped_ptr<WebFrameSchedulerImpl> frame_scheduler( 58 scoped_ptr<WebFrameSchedulerImpl> frame_scheduler(
59 new WebFrameSchedulerImpl(renderer_scheduler_, this, blame_context)); 59 new WebFrameSchedulerImpl(renderer_scheduler_, this, blame_context));
60 frame_scheduler->setPageVisible(page_visible_); 60 frame_scheduler->setPageVisible(page_visible_);
61 frame_schedulers_.insert(frame_scheduler.get()); 61 frame_schedulers_.insert(frame_scheduler.get());
62 return frame_scheduler; 62 return frame_scheduler;
63 } 63 }
64 64
65 blink::WebPassOwnPtr<blink::WebFrameScheduler> 65 std::unique_ptr<blink::WebFrameScheduler>
66 WebViewSchedulerImpl::createFrameScheduler(blink::BlameContext* blame_context) { 66 WebViewSchedulerImpl::createFrameScheduler(blink::BlameContext* blame_context) {
67 return blink::adoptWebPtr( 67 return createWebFrameSchedulerImpl(blame_context);
68 createWebFrameSchedulerImpl(blame_context).release());
69 } 68 }
70 69
71 void WebViewSchedulerImpl::Unregister(WebFrameSchedulerImpl* frame_scheduler) { 70 void WebViewSchedulerImpl::Unregister(WebFrameSchedulerImpl* frame_scheduler) {
72 DCHECK(frame_schedulers_.find(frame_scheduler) != frame_schedulers_.end()); 71 DCHECK(frame_schedulers_.find(frame_scheduler) != frame_schedulers_.end());
73 frame_schedulers_.erase(frame_scheduler); 72 frame_schedulers_.erase(frame_scheduler);
74 } 73 }
75 74
76 void WebViewSchedulerImpl::AddConsoleWarning(const std::string& message) { 75 void WebViewSchedulerImpl::AddConsoleWarning(const std::string& message) {
77 if (!web_view_ || !web_view_->mainFrame()) 76 if (!web_view_ || !web_view_->mainFrame())
78 return; 77 return;
(...skipping 28 matching lines...) Expand all
107 106
108 allow_virtual_time_to_advance_ = allow_virtual_time_to_advance; 107 allow_virtual_time_to_advance_ = allow_virtual_time_to_advance;
109 108
110 if (virtual_time_domain_) { 109 if (virtual_time_domain_) {
111 virtual_time_domain_->SetCanAdvanceVirtualTime( 110 virtual_time_domain_->SetCanAdvanceVirtualTime(
112 allow_virtual_time_to_advance); 111 allow_virtual_time_to_advance);
113 } 112 }
114 } 113 }
115 114
116 } // namespace scheduler 115 } // namespace scheduler
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698