Remove legacy draggable region code and refactor to share common draggable region code

chrome/renderer/extensions/extension_helper.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/renderer/extensions/extension_helper.h"
 
 #include "base/bind.h"
 #include "base/bind_helpers.h"
 #include "base/command_line.h"
 #include "base/json/json_string_value_serializer.h"
@@ skipping 247 matching lines @@
 }
 
 void ExtensionHelper::DraggableRegionsChanged(WebKit::WebFrame* frame) {
   WebKit::WebVector<WebKit::WebDraggableRegion> webregions =
       frame->document().draggableRegions();
   std::vector<extensions::DraggableRegion> regions;
   for (size_t i = 0; i < webregions.size(); ++i) {
     extensions::DraggableRegion region;
     region.bounds = webregions[i].bounds;
     region.draggable = webregions[i].draggable;
-
-    // TODO(jianli): to be removed after WebKit patch that changes the draggable
-    // region syntax is landed.
-    region.label = UTF16ToASCII(webregions[i].label);
-    region.clip = webregions[i].clip;
-
     regions.push_back(region);
   }
   Send(new ExtensionHostMsg_UpdateDraggableRegions(routing_id(), regions));
 }
 
 void ExtensionHelper::FrameDetached(WebFrame* frame) {
   // This could be called before DidCreateDataSource, in which case the frame
   // won't be in the map.
   SchedulerMap::iterator i = g_schedulers.Get().find(frame);
   if (i == g_schedulers.Get().end())
@@ skipping 246 matching lines @@
       case content::CONSOLE_MESSAGE_LEVEL_ERROR:
         target_level = WebConsoleMessage::LevelError;
         break;
     }
     render_view()->GetWebView()->mainFrame()->addMessageToConsole(
         WebConsoleMessage(target_level, message));
   }
 }
 
 }  // namespace extensions