Clear the clipboard closing Incognito window

content/public/browser/browser_context.h

 // 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.
 
 #ifndef CONTENT_PUBLIC_BROWSER_BROWSER_CONTEXT_H_
 #define CONTENT_PUBLIC_BROWSER_BROWSER_CONTEXT_H_
 
 #include "base/callback_forward.h"
 #include "base/hash_tables.h"
 #include "base/memory/scoped_ptr.h"
@@ skipping 76 matching lines @@
   // that the ResourceContext is ready.
   static void EnsureResourceContextInitialized(BrowserContext* browser_context);
 
   // Tells the HTML5 objects on this context to persist their session state
   // across the next restart.
   static void SaveSessionState(BrowserContext* browser_context);
 
   // Tells the HTML5 objects on this context to purge any uneeded memory.
   static void PurgeMemory(BrowserContext* browser_context);
 
+  // We want to add a special flag stored in OS clipboard in order to delete
+  // this entry upon OffTheRecord context destruction.
+  // Returns a Clipboard::SourceTag (pointer) if |context| is OffTheRecord
+  // context. Otherwise, NULL.
+  // If the clipboard contains that SourceTag at the time of |context|
+  // destruction it will be flushed.

[jam, 2013/02/07 21:52:17]

nit: this comment is a bit verbose. can you shrink it down a bit?

also, the return type should be a "ui::Clipboard::SourceTag" (then you wouldn't
need to say that in the comment) so that it matches the cc file.

i.e. Returns a tag, for OffTheRecord contexts only, that identifies the contents
of the clipboard as such.

[vasilii, 2013/02/08 09:27:36]

If I include clipboard.h, apps/app_restore_service.cc compilation breaks. This
is because a command doesn't include /usr/include/gtk-2.0 directory and gtk.h
can't be found. How should I circumvent that?

[jam, 2013/02/08 22:26:21]

i'm not familiar with gtk. you can ask erg

[Elliot Glaysher, 2013/02/09 00:22:21]

The only thing I can think of is that you're doing a linux_aura build. We don't
put gtk in the build path in most of linux_aura except for the very small
coordined off library that can access it. (We don't use ClipboardGtk there,
either.)

[vasilii, 2013/02/13 16:57:25]

Done.

+  static void* GetMarkerForOffTheRecordContext(BrowserContext* context);
+
   virtual ~BrowserContext();
 
   // Returns the path of the directory where this context's data is stored.
   virtual base::FilePath GetPath() = 0;
 
   // Return whether this context is incognito. Default is false.
   // This doesn't belong here; http://crbug.com/89628
   virtual bool IsOffTheRecord() const = 0;
 
   // Returns the request context information associated with this context.  Call
@@ skipping 57 matching lines @@
 struct hash<content::BrowserContext*> {
   std::size_t operator()(content::BrowserContext* const& p) const {
     return reinterpret_cast<std::size_t>(p);
   }
 };
 
 }  // namespace BASE_HASH_NAMESPACE
 #endif
 
 #endif  // CONTENT_PUBLIC_BROWSER_BROWSER_CONTEXT_H_