Dragging panels near screen edge

chrome/browser/extensions/api/tabs/tabs_api.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 "chrome/browser/extensions/api/tabs/tabs_api.h"
 
 #include <algorithm>
 #include <limits>
 #include <vector>
 
@@ skipping 734 matching lines @@
       break;
     default:
       break;
   }
 
   gfx::Rect bounds;
   if (controller->window()->IsMinimized())
     bounds = controller->window()->GetRestoredBounds();
   else
     bounds = controller->window()->GetBounds();
+  // TODO: Updating bounds during a drag can cause problems and a more general
+  // solution is needed. See http://crbug.com/251813 .

[flackr, 2013/06/20 23:00:57]

nit: I'd put this on the call to SetBounds.

[varkha, 2013/06/20 23:10:04]

Done.

   bool set_bounds = false;
 
   // Any part of the bounds can optionally be set by the caller.
   int bounds_val;
   if (update_props->HasKey(keys::kLeftKey)) {
     EXTENSION_FUNCTION_VALIDATE(update_props->GetInteger(
         keys::kLeftKey,
         &bounds_val));
     bounds.set_x(bounds_val);
     set_bounds = true;
@@ skipping 1302 matching lines @@
   execute_tab_id_ = tab_id;
   details_ = details.Pass();
   return true;
 }
 
 bool TabsInsertCSSFunction::ShouldInsertCSS() const {
   return true;
 }
 
 }  // namespace extensions