Add Startup.BrowserView.FirstPaint / .CompositingEnded histograms.

chrome/browser/ui/cocoa/tabs/tab_window_controller.mm

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #import "chrome/browser/ui/cocoa/tabs/tab_window_controller.h"
 
 #include "base/logging.h"
 #import "base/mac/sdk_forward_declarations.h"
+#include "base/memory/ptr_util.h"
+#include "chrome/browser/ui/browser_view_histogram_helper.h"
 #import "chrome/browser/ui/cocoa/browser_window_layout.h"
 #import "chrome/browser/ui/cocoa/fast_resize_view.h"
 #import "chrome/browser/ui/cocoa/framed_browser_window.h"
 #import "chrome/browser/ui/cocoa/tabs/tab_strip_background_view.h"
 #import "chrome/browser/ui/cocoa/tabs/tab_strip_view.h"
 #import "chrome/browser/ui/cocoa/themed_window.h"
 #import "ui/base/cocoa/focus_tracker.h"
 #include "ui/base/material_design/material_design_controller.h"
 #include "ui/base/theme_provider.h"
 
@@ skipping 38 matching lines @@
   return [[self parentWindow] hasDarkTheme];
 }
 
 - (BOOL)inIncognitoModeWithSystemTheme {
   return [[self parentWindow] inIncognitoModeWithSystemTheme];
 }
 
 @end
 
 // Subview of the window's contentView, contains everything but the tab strip.
-@interface ChromeContentView : NSView
+@interface ChromeContentView : NSView {
+ @private
+  std::unique_ptr<BrowserViewHistogramHelper> histogram_helper_;

[tapted, 2017/04/07 00:49:34]

histogramHelper_  (since this is declared between @foo/@end). Also does this
need to be unique_ptr? (i.e. why not just `BrowserViewHistogramHelper
histogramHelper_;`). Then you don't need an initializer.

[themblsha, 2017/04/10 17:28:18]

I'm using unique_ptr in browser_view.h so I can forward-declare the class
without adding it as a dependency for all the units. Here however it's a good
concern :)

Decided to use it without an instance variable in BrowserWindowCocoa::Show().

+}
 @end
 
 @implementation ChromeContentView
 
 // NSView overrides.
 
+- (instancetype)initWithFrame:(NSRect)frameRect {
+  if ((self = [super initWithFrame:frameRect])) {
+    histogram_helper_ = base::MakeUnique<BrowserViewHistogramHelper>();
+  }
+  return self;
+}
+
 // Since Auto Layout and frame-based layout behave differently in small but
 // important ways (e.g. Auto Layout can restrict window resizing, frame-based
 // layout doesn't log a warning when a view's autoresizing mask can't be
 // maintained), ensure that it's on instead of letting it depend on content.
 + (BOOL)requiresConstraintBasedLayout {
   return YES;
 }
 
+- (void)drawRect:(NSRect)dirtyRect {
+  // -drawRect: is called from -drawLayer:inContext:, which is called before all
+  // ChromeContentView's got a chance to paint. But there seems to be no
+  // non-hacky way of getting that notification.

[tapted, 2017/04/07 00:49:34]

Does void BrowserWindowCocoa::Show() already record the value you want for
Cocoa?

Cocoa drawing is typically quite happy just to block the UI thread until
essential drawing is done. In BrowserWindowCocoa::Show() we make the assumption
that all drawing occurs under that `[window()
makeKeyAndOrderFront:controller_];` call. When it returns, drawing is "done".

(in MacViews we can probably cheat a little and fire off a compositing task
before calling makeKeyAndOrderFront: so drawing is parallelised with the
WindowServer IPC)

I'd need to do some more research to see if there's an alternative hook that
captures when drawing is finished versus when the window is mapped to the
screen, but it's probably not worth collecting UMA separately since the user
needs both to happen. (we may still want to capture it for analysis though).

[themblsha, 2017/04/10 17:28:18]

Thanks for the tip! Moved the code there.

The BrowserView solution with ui::Compositor already compiles and works on
MacViews. I've traced the BrowserView::Show() (around the frame_->Show(), which
calls makeKeyAndOrderFront: inside), and frame_->Show() call by itself took 56ms
on Debug build on my i7, Startup.BrowserView.FirstPaint was reported 420ms
later, and compositing took 164ms according to my new histograms. 

+  histogram_helper_->OnDidPaintChildren(nullptr);
+}
+
 @end
 
 @implementation TabWindowController
 
 - (id)initTabWindowControllerWithTabStrip:(BOOL)hasTabStrip
                                  titleBar:(BOOL)hasTitleBar {
   const CGFloat kDefaultWidth = 750;
   const CGFloat kDefaultHeight = 600;
 
   NSRect contentRect = NSMakeRect(60, 229, kDefaultWidth, kDefaultHeight);
@@ skipping 380 matching lines @@
 
 - (void)windowWillEnterFullScreenNotification:(NSNotification*)notification {
   [[visualEffectView_ animator] setAlphaValue:0.0];
 }
 
 - (void)windowWillExitFullScreenNotification:(NSNotification*)notification {
   [[visualEffectView_ animator] setAlphaValue:1.0];
 }
 
 @end