[Search] Store and recall search terms using NavigationEntry to improve search term extraction

content/browser/web_contents/navigation_entry_impl.h

Index: content/browser/web_contents/navigation_entry_impl.h
diff --git a/content/browser/web_contents/navigation_entry_impl.h b/content/browser/web_contents/navigation_entry_impl.h
index c65bdb32b75a9d9dea2f212e344bb4000ae06810..cff4289e87cdf9e701bc6fed8deed7d8913e8d2c 100644
--- a/content/browser/web_contents/navigation_entry_impl.h
+++ b/content/browser/web_contents/navigation_entry_impl.h
@@ -75,6 +75,10 @@ class CONTENT_EXPORT NavigationEntryImpl
   virtual bool GetCanLoadLocalResources() const OVERRIDE;
   virtual void SetFrameToNavigate(const std::string& frame_name) OVERRIDE;
   virtual const std::string& GetFrameToNavigate() const OVERRIDE;
+  virtual void SetExtraData(const std::string& key,
+                            const string16& data) OVERRIDE;
+  virtual const bool GetExtraData(const std::string& key,
+                                  string16* data) const OVERRIDE;

[sreeram, 2013/01/30 22:04:05]

Drop the const in "virtual const bool" (here and everywhere else). It adds no
value.

[Mathieu, 2013/01/30 22:19:34]

Done.

 
   void set_unique_id(int unique_id) {
     unique_id_ = unique_id;
@@ -265,6 +269,9 @@ class CONTENT_EXPORT NavigationEntryImpl
   // persisted, because it is currently only used in tests.
   std::string frame_to_navigate_;
 
+  // Used to store extra data to support browser features.
+  std::map<std::string, string16> extra_data_;
+
   // Copy and assignment is explicitly allowed for this class.
 };