Implements core logic for Pixel Canvas

ui/views/bubble/bubble_frame_view.cc

 // 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.
 
 #include "ui/views/bubble/bubble_frame_view.h"
 
 #include <algorithm>
 #include <utility>
 
 #include "build/build_config.h"
@@ skipping 366 matching lines @@
 
 void BubbleFrameView::OnPaint(gfx::Canvas* canvas) {
   OnPaintBackground(canvas);
   // Border comes after children.
 }
 
 void BubbleFrameView::PaintChildren(const ui::PaintContext& context) {
   NonClientFrameView::PaintChildren(context);
 
   ui::PaintCache paint_cache;
-  ui::PaintRecorder recorder(context, size(), &paint_cache);
+  ui::PaintRecorder recorder(context, context.ScaleToEffectivePixelSize(size()),

[oshima, 2017/06/16 00:15:03]

shouldn't this be context.pixel_size()? and if so, can we eliminate the
parameter?

[malaykeshav, 2017/06/16 20:41:58]

I think its not always the case. But in this case it may be true.

+                             &paint_cache);
   OnPaintBorder(recorder.canvas());
 }
 
 void BubbleFrameView::ButtonPressed(Button* sender, const ui::Event& event) {
   if (sender == close_) {
     close_button_clicked_ = true;
     GetWidget()->Close();
   }
 }
 
@@ skipping 148 matching lines @@
 
   DialogDelegate* dialog_delegate =
       GetWidget()->widget_delegate()->AsDialogDelegate();
   if (dialog_delegate && dialog_delegate->ShouldSnapFrameWidth())
     size.set_width(LayoutProvider::Get()->GetSnappedDialogWidth(size.width()));
 
   return size;
 }
 
 }  // namespace views