Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1019)

Unified Diff: chrome/browser/ui/extensions/shell_window.cc

Issue 11028125: Remove legacy draggable region code and refactor to share common draggable region code (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Patch Created 8 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/ui/extensions/shell_window.cc
diff --git a/chrome/browser/ui/extensions/shell_window.cc b/chrome/browser/ui/extensions/shell_window.cc
index 5851ec146a1233a9b26c2c72328f44bb173a77d1..ebd72f486115efbb5b5de6a17f04fc8f0916c5c2 100644
--- a/chrome/browser/ui/extensions/shell_window.cc
+++ b/chrome/browser/ui/extensions/shell_window.cc
@@ -314,24 +314,7 @@ bool ShellWindow::OnMessageReceived(const IPC::Message& message) {
void ShellWindow::UpdateDraggableRegions(
const std::vector<extensions::DraggableRegion>& regions) {
- // Decide if we want to treat it as old syntax by checking labels.
- // TODO(jianli): to be removed after WebKit patch that changes the draggable
- // region syntax is landed.
- bool new_syntax = true;
- for (std::vector<extensions::DraggableRegion>::const_iterator iter =
- regions.begin();
- iter != regions.end(); ++iter) {
- const extensions::DraggableRegion& region = *iter;
- if (!region.label.empty() || !region.clip.IsEmpty()) {
- new_syntax = false;
- break;
- }
- }
-
- if (new_syntax)
- native_window_->UpdateDraggableRegions(regions);
- else
- native_window_->UpdateLegacyDraggableRegions(regions);
+ native_window_->UpdateDraggableRegions(regions);
}
void ShellWindow::OnImageLoaded(const gfx::Image& image,

Powered by Google App Engine
This is Rietveld 408576698