Clarify full and prefix variables.

Clarify full and prefix variables.

BUG=640625
Committed: https://crrev.com/0b8739a23d597e4602e4de5a794908d9820e0bcc
Cr-Commit-Position: refs/heads/master@{#414539}

[Peter Wen, 2016-08-25 12:45:26]

The CQ bit was checked by wnwen@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-08-25 12:45:41]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Peter Wen, 2016-08-25 13:39:28]

wnwen@chromium.org changed reviewers:
+ twellington@chromium.org

[commit-bot: I haz the power, 2016-08-25 13:52:59]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-08-25 13:53:00]

Dry run: This issue passed the CQ dry run.

[Theresa, 2016-08-25 15:13:57]

lgtm - this should probably land after Ted's change to move some tabbed mode
specific logic out of TabPersistentStore:
https://codereview.chromium.org/2277603002/

https://codereview.chromium.org/2275113004/diff/1/chrome/android/java/src/org...
File
chrome/android/java/src/org/chromium/chrome/browser/tabmodel/TabPersistentStore.java
(right):

https://codereview.chromium.org/2275113004/diff/1/chrome/android/java/src/org...
chrome/android/java/src/org/chromium/chrome/browser/tabmodel/TabPersistentStore.java:1633:
return  SAVED_STATE_PREFIX + Integer.toString(selectorIndex);
nit: remove double space here

[Peter Wen, 2016-08-25 15:49:53]

wnwen@chromium.org changed reviewers:
+ dfalcantara@chromium.org

[Peter Wen, 2016-08-25 15:49:54]

Thanks Theresa. Will land after Ted's changes.

+dfalcantara@ for OWNERS.

https://codereview.chromium.org/2275113004/diff/1/chrome/android/java/src/org...
File
chrome/android/java/src/org/chromium/chrome/browser/tabmodel/TabPersistentStore.java
(right):

https://codereview.chromium.org/2275113004/diff/1/chrome/android/java/src/org...
chrome/android/java/src/org/chromium/chrome/browser/tabmodel/TabPersistentStore.java:1633:
return  SAVED_STATE_PREFIX + Integer.toString(selectorIndex);
On 2016/08/25 15:13:56, Theresa Wellington wrote:
> nit: remove double space here

Done.

[gone, 2016-08-25 16:10:47]

Lgtm, thanks!

[Peter Wen, 2016-08-25 19:58:25]

The CQ bit was checked by wnwen@chromium.org

[Peter Wen, 2016-08-25 19:58:25]

The patchset sent to the CQ was uploaded after l-g-t-m from
twellington@chromium.org, dfalcantara@chromium.org
Link to the patchset: https://codereview.chromium.org/2275113004/#ps40001
(title: "Rebase.")

[commit-bot: I haz the power, 2016-08-25 19:59:11]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-08-25 21:14:26]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-08-25 21:16:22]

Description was changed from

==========
Clarify full and prefix variables.

BUG=640625
==========

to

==========
Clarify full and prefix variables.

BUG=640625

Committed: https://crrev.com/0b8739a23d597e4602e4de5a794908d9820e0bcc
Cr-Commit-Position: refs/heads/master@{#414539}
==========

[commit-bot: I haz the power, 2016-08-25 21:16:23]

Patchset 3 (id:??) landed as
https://crrev.com/0b8739a23d597e4602e4de5a794908d9820e0bcc
Cr-Commit-Position: refs/heads/master@{#414539}