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

Side by Side Diff: third_party/WebKit/Source/core/frame/UseCounter.cpp

Issue 2877893002: Make UseCounter take a LocaFrame instead of any Frame (Closed)
Patch Set: Rebase Created 3 years, 7 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 /* 1 /*
2 * Copyright (C) 2012 Google, Inc. All rights reserved. 2 * Copyright (C) 2012 Google, Inc. All rights reserved.
3 * 3 *
4 * Redistribution and use in source and binary forms, with or without 4 * Redistribution and use in source and binary forms, with or without
5 * modification, are permitted provided that the following conditions 5 * modification, are permitted provided that the following conditions
6 * are met: 6 * are met:
7 * 1. Redistributions of source code must retain the above copyright 7 * 1. Redistributions of source code must retain the above copyright
8 * notice, this list of conditions and the following disclaimer. 8 * notice, this list of conditions and the following disclaimer.
9 * 2. Redistributions in binary form must reproduce the above copyright 9 * 2. Redistributions in binary form must reproduce the above copyright
10 * notice, this list of conditions and the following disclaimer in the 10 * notice, this list of conditions and the following disclaimer in the
(...skipping 1160 matching lines...) Expand 10 before | Expand all | Expand 10 after
1171 features_recorded_.ClearAll(); 1171 features_recorded_.ClearAll();
1172 css_recorded_.ClearAll(); 1172 css_recorded_.ClearAll();
1173 animated_css_recorded_.ClearAll(); 1173 animated_css_recorded_.ClearAll();
1174 if (!disable_reporting_ && !mute_count_) { 1174 if (!disable_reporting_ && !mute_count_) {
1175 FeaturesHistogram().Count(kPageVisits); 1175 FeaturesHistogram().Count(kPageVisits);
1176 CssHistogram().Count(totalPagesMeasuredCSSSampleId()); 1176 CssHistogram().Count(totalPagesMeasuredCSSSampleId());
1177 AnimatedCSSHistogram().Count(totalPagesMeasuredCSSSampleId()); 1177 AnimatedCSSHistogram().Count(totalPagesMeasuredCSSSampleId());
1178 } 1178 }
1179 } 1179 }
1180 1180
1181 void UseCounter::Count(const Frame* frame, Feature feature) { 1181 void UseCounter::Count(const LocalFrame* frame, Feature feature) {
1182 if (!frame) 1182 if (!frame)
1183 return; 1183 return;
1184 Page* page = frame->GetPage(); 1184 Page* page = frame->GetPage();
1185 if (!page) 1185 if (!page)
1186 return; 1186 return;
1187 1187
1188 page->GetUseCounter().Count(feature); 1188 page->GetUseCounter().Count(feature);
1189 } 1189 }
1190 1190
1191 void UseCounter::Count(const Document& document, Feature feature) { 1191 void UseCounter::Count(const Document& document, Feature feature) {
(...skipping 218 matching lines...) Expand 10 before | Expand all | Expand 10 after
1410 } 1410 }
1411 } 1411 }
1412 1412
1413 if (needs_pages_measured_update) 1413 if (needs_pages_measured_update)
1414 css_properties_histogram.Count(totalPagesMeasuredCSSSampleId()); 1414 css_properties_histogram.Count(totalPagesMeasuredCSSSampleId());
1415 1415
1416 css_bits_.ClearAll(); 1416 css_bits_.ClearAll();
1417 } 1417 }
1418 1418
1419 } // namespace blink 1419 } // namespace blink
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698