Updating MakeMenuItemStringsFor() to include custom items if provided by blink.

content/shell/renderer/test_runner/event_sender.cc

 // Copyright 2014 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 "content/shell/renderer/test_runner/event_sender.h"
 
 #include "base/basictypes.h"
 #include "base/logging.h"
 #include "base/strings/stringprintf.h"
 #include "content/public/common/page_zoom.h"
@@ skipping 14 matching lines @@
 #include "v8/include/v8.h"
 
 using blink::WebContextMenuData;
 using blink::WebDragData;
 using blink::WebDragOperationsMask;
 using blink::WebFloatPoint;
 using blink::WebFrame;
 using blink::WebGestureEvent;
 using blink::WebInputEvent;
 using blink::WebKeyboardEvent;
+using blink::WebMenuItemInfo;
 using blink::WebMouseEvent;
 using blink::WebMouseWheelEvent;
 using blink::WebPoint;
 using blink::WebString;
 using blink::WebTouchEvent;
 using blink::WebTouchPoint;
 using blink::WebVector;
 using blink::WebView;
 
 namespace content {
@@ skipping 69 matching lines @@
     gin::Converter<std::vector<std::string> >::FromV8(
         NULL, value, &modifier_names);
   }
   return GetKeyModifiers(modifier_names);
 }
 
 // Maximum distance (in space and time) for a mouse click to register as a
 // double or triple click.
 const double kMultipleClickTimeSec = 1;
 const int kMultipleClickRadiusPixels = 5;
+const char kSubMenuDepthIdentifier[] = "_";
+const char kSubMenuIdentifier[] = " >";
 
 bool OutsideMultiClickRadius(const WebPoint& a, const WebPoint& b) {
   return ((a.x - b.x) * (a.x - b.x) + (a.y - b.y) * (a.y - b.y)) >
          kMultipleClickRadiusPixels * kMultipleClickRadiusPixels;
 }
 
+void PopulateCustomItems(const WebVector<WebMenuItemInfo>& customItems,

[tkent, 2014/08/05 09:08:45]

This doesn't follow Chromium coding style.
 - 'out' argument must be a pointer, not a reference
 - 'out' argument must be the last argument.

So, this function should be:
   PopulateCustomItems(const WebVector<WebMenuItemInfo>& customItems, const
std::string& prefix, std::vector<std::string>* strings)

[pals, 2014/08/05 09:17:29]

Done.

+    std::vector<std::string>& strings, const std::string& prefix) {
+  for (size_t i = 0; i < customItems.size(); ++i) {
+    if (customItems[i].type == blink::WebMenuItemInfo::SubMenu) {
+      strings.push_back(prefix + customItems[i].label.utf8() + kSubMenuIdentifier);

[tkent, 2014/08/05 09:08:45]

wrap in 80 columns

[pals, 2014/08/05 09:17:29]

Done.

+      PopulateCustomItems(customItems[i].subMenuItems, strings, prefix + kSubMenuDepthIdentifier);

[tkent, 2014/08/05 09:08:45]

Ditto.

[pals, 2014/08/05 09:17:29]

Done.

+    } else {
+      strings.push_back(prefix + customItems[i].label.utf8());
+    }
+  }
+}
+
 // Because actual context menu is implemented by the browser side,
 // this function does only what LayoutTests are expecting:
 // - Many test checks the count of items. So returning non-zero value makes
 // sense.
 // - Some test compares the count before and after some action. So changing the
 // count based on flags also makes sense. This function is doing such for some
 // flags.
 // - Some test even checks actual string content. So providing it would be also
 // helpful.
 std::vector<std::string> MakeMenuItemStringsFor(
@@ skipping 25 matching lines @@
     "<separator>",
     0
   };
 
   // This is possible because mouse events are cancelleable.
   if (!context_menu)
     return std::vector<std::string>();
 
   std::vector<std::string> strings;
 
+  // Populate custom menu items if provided by the blink internals.

[tkent, 2014/08/05 09:08:45]

nit: |by the blink internals.| -> |by Blink.|

[pals, 2014/08/05 09:17:29]

Done.

+  PopulateCustomItems(context_menu->customItems, strings, "");
+
   if (context_menu->isEditable) {
     for (const char** item = kEditableMenuStrings; *item; ++item) {
       strings.push_back(*item);
     }
     WebVector<WebString> suggestions;
     MockSpellCheck::FillSuggestionList(context_menu->misspelledWord,
                                        &suggestions);
     for (size_t i = 0; i < suggestions.size(); ++i) {
       strings.push_back(suggestions[i].utf8());
     }
@@ skipping 2073 matching lines @@
       }
       default:
         NOTREACHED();
     }
   }
 
   replaying_saved_events_ = false;
 }
 
 }  // namespace content