Add a new SyncedTabDelegate for Android.

chrome/browser/sessions/session_tab_helper.h

Index: chrome/browser/sessions/session_tab_helper.h
diff --git a/chrome/browser/sessions/session_tab_helper.h b/chrome/browser/sessions/session_tab_helper.h
index 2fe66696dcf4d74fb0f23ae73d95e3bf70818c23..163c90fe7a3787e348c6dd760543c20ef4308830 100644
--- a/chrome/browser/sessions/session_tab_helper.h
+++ b/chrome/browser/sessions/session_tab_helper.h
@@ -19,6 +19,7 @@ class SessionTabHelper : public content::WebContentsObserver,
 
   // Returns the identifier used by session restore for this tab.
   const SessionID& session_id() const { return session_id_; }
+  void SetSessionId(const SessionID& id);

[sky, 2013/05/21 04:53:01]

Why do you need to dynamically change the session id? Doing so could cause all
sorts of problems and I don't want to expose this.

[shashi, 2013/05/21 05:16:35]

Sorry my intent is not to dynamically change but to set it at instantiation
time. Now I realize it can be potentially misused, Perhaps a better way would be
to add a new static method:

static public void CreateForWebContents(WebContents * webcontents, const
SessionId& id) {
  CreateForWebContents(webcontents);
   FromWebContents(webcontents)->SetSessionId(id);
}
 
the new method SetSessionID is then private. This way we never expose this
method.

Background:
On Android we can have tab id assigned to tab even before web contents have been
created and all our tabs do not have web contents, (we do this to reduce memory
footprint of app, otherwise Chrome will be killed by operating system.) This
means if a user switches to a tab which was not in memory the SessionID
associated with it can change, since a lot of places use
SessionTabHelper::FromWebContents()->session_id for getting the tab id.

A lot of places in Chrome use SessionTabHelper to get SessionID.

On 2013/05/21 04:53:01, sky wrote:

[sky, 2013/05/21 15:00:44]

Is it possible to lookup the id you want to assign in SessionTabHelper's
constructor?

[shashi, 2013/05/21 16:01:13]

The id I want to assign is TabAndroid id, I don't think I can look it up in the
constructor, unless I pass the pointer to TabAndroid here.
On 2013/05/21 15:00:44, sky wrote:

[shashi, 2013/05/21 23:11:21]

Done.

 
   // Identifier of the window the tab is in.
   void SetWindowID(const SessionID& id);
@@ -36,7 +37,7 @@ class SessionTabHelper : public content::WebContentsObserver,
   // Unique identifier of the tab for session restore. This id is only unique
   // within the current session, and is not guaranteed to be unique across
   // sessions.
-  const SessionID session_id_;
+  SessionID session_id_;
 
   // Unique identifier of the window the tab is in.
   SessionID window_id_;