Chromium Code Reviews| Index: chrome/renderer/page_load_histograms.h |
| diff --git a/chrome/renderer/page_load_histograms.h b/chrome/renderer/page_load_histograms.h |
| index 39f8cc5a9cb8c75d3d70b7dbaa4f5d0c9c38cfc2..dec1d82b759180fb338b08e2d41825b4359f65ba 100644 |
| --- a/chrome/renderer/page_load_histograms.h |
| +++ b/chrome/renderer/page_load_histograms.h |
| @@ -24,8 +24,8 @@ class PageLoadHistograms : public content::RenderViewObserver { |
| private: |
| // RenderViewObserver implementation. |
| - void FrameWillClose(blink::WebFrame* frame) override; |
| - void ClosePage() override; |
| + void FrameWillClose(blink::WebFrame* frame, bool lofi_used) override; |
| + void ClosePage(bool lofi_used) override; |
| void DidUpdateLayout() override; |
| // Dump all page load histograms appropriate for the given frame. |
| @@ -47,7 +47,7 @@ class PageLoadHistograms : public content::RenderViewObserver { |
| // redirect had been done (the user never requested the page) |
| // Also, it's possible to load a page without ever laying it out |
| // so first_paint and first_paint_after_load can be 0. |
| - void Dump(blink::WebFrame* frame); |
| + void Dump(blink::WebFrame* frame, bool lofi_used); |
|
bengr
2015/08/25 00:00:01
Add a description of this new parameter to the com
megjablon
2015/08/25 20:29:45
Done.
|
| bool ShouldDump(blink::WebFrame* frame); |
| void MaybeDumpFirstLayoutHistograms(); |