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

Side by Side Diff: content/renderer/render_widget.cc

Issue 1897123002: Remove current implementation of frame timing events. (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
« no previous file with comments | « content/renderer/render_widget.h ('k') | third_party/WebKit/Source/core/dom/Document.cpp » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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 "content/renderer/render_widget.h" 5 #include "content/renderer/render_widget.h"
6 6
7 #include <memory> 7 #include <memory>
8 #include <utility> 8 #include <utility>
9 9
10 #include "base/auto_reset.h" 10 #include "base/auto_reset.h"
(...skipping 938 matching lines...) Expand 10 before | Expand all | Expand 10 after
949 TRACE_EVENT0("renderer", "RenderWidget::OnSwapBuffersComplete"); 949 TRACE_EVENT0("renderer", "RenderWidget::OnSwapBuffersComplete");
950 950
951 // Notify subclasses that composited rendering was flushed to the screen. 951 // Notify subclasses that composited rendering was flushed to the screen.
952 DidFlushPaint(); 952 DidFlushPaint();
953 } 953 }
954 954
955 void RenderWidget::OnSwapBuffersPosted() { 955 void RenderWidget::OnSwapBuffersPosted() {
956 TRACE_EVENT0("renderer", "RenderWidget::OnSwapBuffersPosted"); 956 TRACE_EVENT0("renderer", "RenderWidget::OnSwapBuffersPosted");
957 } 957 }
958 958
959 void RenderWidget::RecordFrameTimingEvents(
960 std::unique_ptr<cc::FrameTimingTracker::CompositeTimingSet>
961 composite_events,
962 std::unique_ptr<cc::FrameTimingTracker::MainFrameTimingSet>
963 main_frame_events) {
964 for (const auto& composite_event : *composite_events) {
965 int64_t frameId = composite_event.first;
966 const std::vector<cc::FrameTimingTracker::CompositeTimingEvent>& events =
967 composite_event.second;
968 std::vector<blink::WebFrameTimingEvent> webEvents;
969 for (size_t i = 0; i < events.size(); ++i) {
970 webEvents.push_back(blink::WebFrameTimingEvent(
971 events[i].frame_id,
972 (events[i].timestamp - base::TimeTicks()).InSecondsF()));
973 }
974 webwidget_->recordFrameTimingEvent(blink::WebWidget::CompositeEvent,
975 frameId, webEvents);
976 }
977 for (const auto& main_frame_event : *main_frame_events) {
978 int64_t frameId = main_frame_event.first;
979 const std::vector<cc::FrameTimingTracker::MainFrameTimingEvent>& events =
980 main_frame_event.second;
981 std::vector<blink::WebFrameTimingEvent> webEvents;
982 for (size_t i = 0; i < events.size(); ++i) {
983 webEvents.push_back(blink::WebFrameTimingEvent(
984 events[i].frame_id,
985 (events[i].timestamp - base::TimeTicks()).InSecondsF(),
986 (events[i].end_time - base::TimeTicks()).InSecondsF()));
987 }
988 webwidget_->recordFrameTimingEvent(blink::WebWidget::RenderEvent, frameId,
989 webEvents);
990 }
991 }
992
993 void RenderWidget::RequestScheduleAnimation() { 959 void RenderWidget::RequestScheduleAnimation() {
994 scheduleAnimation(); 960 scheduleAnimation();
995 } 961 }
996 962
997 void RenderWidget::UpdateVisualState() { 963 void RenderWidget::UpdateVisualState() {
998 webwidget_->updateAllLifecyclePhases(); 964 webwidget_->updateAllLifecyclePhases();
999 } 965 }
1000 966
1001 void RenderWidget::WillBeginCompositorFrame() { 967 void RenderWidget::WillBeginCompositorFrame() {
1002 TRACE_EVENT0("gpu", "RenderWidget::willBeginCompositorFrame"); 968 TRACE_EVENT0("gpu", "RenderWidget::willBeginCompositorFrame");
(...skipping 1183 matching lines...) Expand 10 before | Expand all | Expand 10 after
2186 } 2152 }
2187 2153
2188 float RenderWidget::GetOriginalDeviceScaleFactor() const { 2154 float RenderWidget::GetOriginalDeviceScaleFactor() const {
2189 return 2155 return
2190 screen_metrics_emulator_ ? 2156 screen_metrics_emulator_ ?
2191 screen_metrics_emulator_->original_screen_info().deviceScaleFactor : 2157 screen_metrics_emulator_->original_screen_info().deviceScaleFactor :
2192 device_scale_factor_; 2158 device_scale_factor_;
2193 } 2159 }
2194 2160
2195 } // namespace content 2161 } // namespace content
OLDNEW
« no previous file with comments | « content/renderer/render_widget.h ('k') | third_party/WebKit/Source/core/dom/Document.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698