SelectionOwner: add support for TIMESTAMP target

components/bookmarks/browser/bookmark_utils_unittest.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 "components/bookmarks/browser/bookmark_utils.h"
 
 #include <stddef.h>
 #include <utility>
 #include <vector>
 
 #include "base/macros.h"
 #include "base/message_loop/message_loop.h"
 #include "base/strings/utf_string_conversions.h"
 #include "build/build_config.h"
 #include "components/bookmarks/browser/base_bookmark_model_observer.h"
 #include "components/bookmarks/browser/bookmark_model.h"
 #include "components/bookmarks/browser/bookmark_node_data.h"
 #include "components/bookmarks/test/test_bookmark_client.h"
 #include "testing/gtest/include/gtest/gtest.h"
 #include "ui/base/clipboard/clipboard.h"
 #include "ui/base/clipboard/scoped_clipboard_writer.h"
 
+#if defined(USE_X11)
+
+#include "ui/events/platform/x11/x11_event_source_glib.h"  // nogncheck
+#include "ui/gfx/x/x11_types.h"                            // nogncheck
+
+#endif  // defined(USE_X11)
+
 using base::ASCIIToUTF16;
 using std::string;
 
 namespace bookmarks {
 namespace {
 
 class BookmarkUtilsTest : public testing::Test,
                           public BaseBookmarkModelObserver {
  public:
   BookmarkUtilsTest()
       : grouped_changes_beginning_count_(0),
-        grouped_changes_ended_count_(0) {}
+#if defined(USE_X11)
+        grouped_changes_ended_count_(0),
+        event_source_(gfx::GetXDisplay()) {

[Elliot Glaysher, 2016/04/20 17:37:22]

I believe this is supposed to be:

     other_thing()
  #if defined(USE_X11)
     , event_source...()
  #endif
  {}

Also, this part is really unfortunate. :-/

[dcheng, 2016/04/20 18:55:42]

clang-format mangles this pretty horrifically.

I've updated this to just use a unique_ptr, so I can set it in the body of the
constructor instead.

+  }
+#else
+        grouped_changes_ended_count_(0) {
+  }
+#endif  // defined(USE_x11)
   ~BookmarkUtilsTest() override {}
 
 // Copy and paste is not yet supported on iOS. http://crbug.com/228147
 #if !defined(OS_IOS)
   void TearDown() override {
     ui::Clipboard::DestroyClipboardForCurrentThread();
   }
 #endif  // !defined(OS_IOS)
 
   // Certain user actions require multiple changes to the bookmark model,
@@ skipping 21 matching lines @@
 
   void GroupedBookmarkChangesEnded(BookmarkModel* model) override {
     ++grouped_changes_ended_count_;
   }
 
   int grouped_changes_beginning_count_;
   int grouped_changes_ended_count_;
 
   // Clipboard requires a message loop.
   base::MessageLoopForUI loop_;
+#if defined(USE_X11)
+  // On Linux, Clipboard also depends on an event source being present, to get
+  // the timestamp.
+  ui::X11EventSourceGlib event_source_;
+#endif  // defined(USE_X11)
 
   DISALLOW_COPY_AND_ASSIGN(BookmarkUtilsTest);
 };
 
 TEST_F(BookmarkUtilsTest, GetBookmarksMatchingPropertiesWordPhraseQuery) {
   scoped_ptr<BookmarkModel> model(TestBookmarkClient::CreateModel());
   const BookmarkNode* node1 = model->AddURL(model->other_node(),
                                             0,
                                             ASCIIToUTF16("foo bar"),
                                             GURL("http://www.google.com"));
@@ skipping 511 matching lines @@
   model->GetNodesByURL(url, &nodes);
   ASSERT_EQ(1u, nodes.size());
   EXPECT_TRUE(model->bookmark_bar_node()->empty());
   EXPECT_TRUE(model->other_node()->empty());
   EXPECT_TRUE(model->mobile_node()->empty());
   EXPECT_EQ(1, extra_node->child_count());
 }
 
 }  // namespace
 }  // namespace bookmarks