Clear the clipboard closing Incognito window

content/test/clipboard_browser_test.cc

+// Copyright (c) 2013 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 "base/utf_string_conversions.h"
+#include "content/public/browser/render_view_host.h"
+#include "content/public/browser/web_contents.h"
+#include "content/public/test/test_utils.h"
+#include "content/shell/shell.h"
+#include "content/test/content_browser_test.h"
+#include "content/test/content_browser_test_utils.h"
+#include "ui/base/clipboard/clipboard.h"
+#include "ui/base/clipboard/scoped_clipboard_writer.h"
+
+namespace content {
+namespace {
+class ClipboardWriteObserver :
+    private ui::Clipboard::ClipboardObserverForTesting {

[dcheng, 2013/02/06 22:31:40]

Google style is to use public inheritance.

[vasilii, 2013/02/07 17:30:05]

Done.

+ public:
+  explicit ClipboardWriteObserver(ui::Clipboard* clipboard)
+      : seen_(false),
+        running_(false),
+        clipboard_(clipboard) {
+    clipboard_->AddObserver(this);
+  }
+
+  virtual ~ClipboardWriteObserver() {
+    clipboard_->RemoveObserver(this);
+  }
+
+  void Wait() {
+    if (seen_) return;
+
+    running_ = true;
+    message_loop_runner_ = new content::MessageLoopRunner;

[dcheng, 2013/02/06 22:31:40]

Do we need to use a MessageLoopRunner here? Would it be possible to use
base::WaitableEvent instead?

[vasilii, 2013/02/07 17:30:05]

No, copying happens on UI thread and we must dispatch messages to make it
possible. That is, OnWriteObjects() should be called on UI thread. Should I use
ThreadChecker?

[dcheng, 2013/02/08 05:42:19]

I see, I didn't realize that it was important to dispatch messages as well. This
is fine then.

+    message_loop_runner_->Run();
+    EXPECT_TRUE(seen_);
+  }
+
+ private:
+  virtual void OnWriteObjects(ui::Clipboard::Buffer buffer) OVERRIDE {
+    if (buffer != ui::Clipboard::BUFFER_STANDARD) return;
+    seen_ = true;
+    if (!running_) return;
+
+    message_loop_runner_->Quit();
+    running_ = false;
+  }
+
+  bool seen_;
+  bool running_;
+  scoped_refptr<MessageLoopRunner> message_loop_runner_;
+  ui::Clipboard* clipboard_;
+
+  DISALLOW_COPY_AND_ASSIGN(ClipboardWriteObserver);
+};
+
+} // namespace
+
+class OffTheRecordClipboardTest : public ContentBrowserTest {
+ protected:
+  ui::Clipboard* clipboard() const {
+    static ui::Clipboard* clipboard = ui::Clipboard::GetForCurrentThread();
+    return clipboard;
+  }
+  void ExpectStringInClipboard(const string16& pattern) {
+    string16 content;
+    clipboard()->ReadText(ui::Clipboard::BUFFER_STANDARD, &content);
+    EXPECT_EQ(pattern, content);
+  }
+};
+
+// Tests that data copied from content area of OffTheRecord window is destroyed
+// after context destruction.
+IN_PROC_BROWSER_TEST_F(OffTheRecordClipboardTest, PRE_ClearContentData) {
+  Shell* shell_offtherecord = CreateOffTheRecordBrowser();
+  NavigateToURL(shell_offtherecord, GURL("data:text/plain,foo"));
+  // Clear the clipboard. Currently GTK Clipboard::Clear fails to clear
+  // clipboard from external content.
+  {
+    ui::ScopedClipboardWriter clipboard_writer(clipboard(),
+                                               ui::Clipboard::BUFFER_STANDARD);
+    clipboard_writer.WriteText(ASCIIToUTF16("bar"));
+  }
+  ExpectStringInClipboard(ASCIIToUTF16("bar"));
+  // Select and copy web-page content
+  WebContents* web_contents = shell_offtherecord->web_contents();
+  RenderViewHost* view_host = web_contents->GetRenderViewHost();
+  ClipboardWriteObserver clipboard_observer(clipboard());
+  view_host->SelectAll();
+  view_host->Copy();
+  // Run message loop until data have been copied to the clipboard. This happens
+  // on UI thread.
+  clipboard_observer.Wait();
+  ExpectStringInClipboard(ASCIIToUTF16("foo"));
+}
+
+IN_PROC_BROWSER_TEST_F(OffTheRecordClipboardTest, ClearContentData) {
+  EXPECT_FALSE(clipboard()->IsFormatAvailable(
+      ui::Clipboard::GetPlainTextFormatType(), ui::Clipboard::BUFFER_STANDARD));
+}
+} // namespace content