File upload API in chromedriver

chrome/browser/automation/testing_automation_provider.cc

 // Copyright (c) 2011 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/automation/testing_automation_provider.h"
 
 #include <map>
 #include <set>
 #include <string>
 #include <vector>
@@ skipping 86 matching lines @@
 #include "content/browser/renderer_host/render_process_host.h"
 #include "content/browser/renderer_host/render_view_host.h"
 #include "content/browser/tab_contents/interstitial_page.h"
 #include "content/common/common_param_traits.h"
 #include "content/common/notification_service.h"
 #include "net/base/cookie_store.h"
 #include "third_party/WebKit/Source/WebKit/chromium/public/WebInputEvent.h"
 #include "ui/base/events.h"
 #include "ui/base/keycodes/keyboard_codes.h"
 #include "ui/base/message_box_flags.h"
+#include "webkit/glue/webdropdata.h"
 #include "webkit/plugins/npapi/plugin_list.h"
 
 namespace {
 
 void SendMouseClick(int flags) {
   ui_controls::MouseButton button = ui_controls::LEFT;
   if ((flags & ui::EF_LEFT_BUTTON_DOWN) ==
       ui::EF_LEFT_BUTTON_DOWN) {
     button = ui_controls::LEFT;
   } else if ((flags & ui::EF_RIGHT_BUTTON_DOWN) ==
@@ skipping 956 matching lines @@
   tab_contents->render_view_host()->ForwardMouseEvent(mouse_event);
 
   mouse_event.type = WebKit::WebInputEvent::MouseDown;
   mouse_event.clickCount = 2;
   tab_contents->render_view_host()->ForwardMouseEvent(mouse_event);
 
   mouse_event.type = WebKit::WebInputEvent::MouseUp;
   tab_contents->render_view_host()->ForwardMouseEvent(mouse_event);
 }
 
+void TestingAutomationProvider::SetFilePathsToFileUploadControl(
+    DictionaryValue* args, IPC::Message* reply_message) {
+  TabContents* tab_contents;
+  std::string error;
+  if (!GetTabFromJSONArgs(args, &tab_contents, &error)) {
+    AutomationJSONReply(this, reply_message).SendError(error);
+    return;
+  }
+
+  int x, y;
+  if (!args->GetInteger("x", &x) ||
+      !args->GetInteger("y", &y)) {

[kkania, 2011/05/23 18:43:49]

put this on the line before

[nodchip, 2011/05/26 01:14:53]

Done.

+    AutomationJSONReply(this, reply_message)
+        .SendError("(X,Y) coordinates missing or invalid");
+    return;
+  }
+
+  ListValue* paths = NULL;
+  if (!args->GetList("paths", &paths)) {
+    AutomationJSONReply(this, reply_message)
+        .SendError("paths missing or invalid");
+    return;
+  }
+
+  // Emulate drag and drop to set the file paths to the file upload control.
+  WebDropData drop_data;
+  for (size_t path_index = 0; path_index < paths->GetSize(); ++path_index) {
+    std::string path;
+    if (!paths->GetString(path_index, &path)) {

[kkania, 2011/05/23 18:43:49]

make path a string16 and GetString will do the UTF8->16 conversion for you

[nodchip, 2011/05/26 01:14:53]

Done.

+      AutomationJSONReply(this, reply_message)
+          .SendError("path missing or invalid");
+      return;
+    }
+
+    drop_data.filenames.push_back(UTF8ToUTF16(path));
+  }
+
+  const gfx::Point client(x, y);
+  // Currently DragTarget*** ignore the screen argument.

[kkania, 2011/05/23 18:43:49]

do you think it will always remain like that? do the other clients of this
function provide a real coordinate? If you think eventually we need to add the
screen coordinate, can you do that now instead of a TODO.

[nodchip, 2011/05/26 01:14:53]

I think it will always remain because it does not seem to need a screen
coordinate in drag and drop handling for WebKit. I will change the comment.

On 2011/05/23 18:43:49, kkania wrote:

+  // TODO(hnoda): Calculate the screen coordinate using WindowGetViewBounds().
+  const gfx::Point screen;
+
+  int operations = 0;
+  operations |= WebKit::WebDragOperationCopy;
+  operations |= WebKit::WebDragOperationLink;
+  operations |= WebKit::WebDragOperationMove;
+
+  RenderViewHost* host = tab_contents->render_view_host();
+  host->DragTargetDragEnter(drop_data, client, screen,
+                            (WebKit::WebDragOperationsMask)operations);

[kkania, 2011/05/23 18:43:49]

don't use c-style casting

[nodchip, 2011/05/26 01:14:53]

Done.

+  host->DragTargetDrop(client, screen);
+  AutomationJSONReply(this, reply_message).SendSuccess(NULL);

[kkania, 2011/05/23 18:43:49]

we should wait till the renderer processes the drag operation.

[nodchip, 2011/05/26 01:14:53]

Done. But we have to add additional IPC Message.

+}
+
 void TestingAutomationProvider::GetTabCount(int handle, int* tab_count) {
   *tab_count = -1;  // -1 is the error code
 
   if (browser_tracker_->ContainsHandle(handle)) {
     Browser* browser = browser_tracker_->GetResource(handle);
     *tab_count = browser->tab_count();
   }
 }
 
 void TestingAutomationProvider::GetType(int handle, int* type_as_int) {
@@ skipping 1118 matching lines @@
   handler_map["WebkitMouseClick"] =
       &TestingAutomationProvider::WebkitMouseClick;
   handler_map["WebkitMouseDrag"] =
       &TestingAutomationProvider::WebkitMouseDrag;
   handler_map["WebkitMouseButtonUp"] =
       &TestingAutomationProvider::WebkitMouseButtonUp;
   handler_map["WebkitMouseButtonDown"] =
       &TestingAutomationProvider::WebkitMouseButtonDown;
   handler_map["WebkitMouseDoubleClick"] =
       &TestingAutomationProvider::WebkitMouseDoubleClick;
+  handler_map["SetFilePathToFileUploadControl"] =
+      &TestingAutomationProvider::SetFilePathsToFileUploadControl;
   handler_map["SendWebkitKeyEvent"] =
       &TestingAutomationProvider::SendWebkitKeyEvent;
   handler_map["SendOSLevelKeyEventToTab"] =
       &TestingAutomationProvider::SendOSLevelKeyEventToTab;
   handler_map["ActivateTab"] =
       &TestingAutomationProvider::ActivateTabJSON;
   handler_map["GetAppModalDialogMessage"] =
       &TestingAutomationProvider::GetAppModalDialogMessage;
   handler_map["AcceptOrDismissAppModalDialog"] =
       &TestingAutomationProvider::AcceptOrDismissAppModalDialog;
@@ skipping 3661 matching lines @@
   IPC::ParamTraits<std::vector<GURL> >::Write(reply_message_, redirects_gurl);
 
   Send(reply_message_);
   redirect_query_ = 0;
   reply_message_ = NULL;
 }
 
 void TestingAutomationProvider::OnRemoveProvider() {
   AutomationProviderList::GetInstance()->RemoveProvider(this);
 }