Changing focus ring color by node. Focus ring can now change color without changing bounds

chrome/browser/chromeos/ui/focus_ring_layer.cc

 // Copyright 2013 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 "chrome/browser/chromeos/ui/focus_ring_layer.h"
 
 #include "ui/aura/window.h"
 #include "ui/compositor/compositor_animation_observer.h"
 #include "ui/compositor/layer.h"
 #include "ui/compositor/paint_recorder.h"
@@ skipping 60 matching lines @@
     layer_->set_delegate(this);
     layer_->SetFillsBoundsOpaquely(false);
     root_layer->Add(layer_.get());
   }
 
   // Keep moving it to the top in case new layers have been added
   // since we created this layer.
   layer_->parent()->StackAtTop(layer_.get());
 
   layer_->SetBounds(bounds);
+  gfx::Rect* layer_bounds =

[dmazzoni, 2017/05/09 17:05:09]

If you create an object with "new" you also have to delete it. Simpler is to
just create it on the stack like this:

gfx::Rect layer_bounds(0, 0, bounds.width(), bounds.height());

Then you don't need the * when passing it to SchedulePaint.

[elichtenberg, 2017/05/09 17:26:55]

Done.

+      new gfx::Rect(0, 0, bounds.width(), bounds.height());
+  layer_->SchedulePaint(*layer_bounds);
 
   // Update the animation observer.
   display::Display display =
       display::Screen::GetScreen()->GetDisplayMatching(bounds);
   ui::Compositor* compositor = root_window->layer()->GetCompositor();
   if (compositor != compositor_) {
     if (compositor_ && compositor_->HasAnimationObserver(this))
       compositor_->RemoveAnimationObserver(this);
     compositor_ = compositor;
     if (compositor_ && !compositor_->HasAnimationObserver(this))
@@ skipping 38 matching lines @@
 }
 
 void FocusRingLayer::OnCompositingShuttingDown(ui::Compositor* compositor) {
   if (compositor == compositor_) {
     compositor->RemoveAnimationObserver(this);
     compositor_ = nullptr;
   }
 }
 
 }  // namespace chromeos