Reland of Mark removable Drive caches for cryptohome.

Reland of Mark removable Drive caches for cryptohome. (patchset #2 id:80001 of https://codereview.chromium.org/1989863005/ )

Reason for revert:
Reland for M53.

BUG=533750
Original issue's description:
> Revert of Mark removable Drive caches for cryptohome. (patchset #17 id:320001 of https://codereview.chromium.org/1918243004/ )
>
> Reason for revert:
> Corresponding changes for Chrome OS [1] haven't been submitted. Let's punt this change to M53.
> [1] https://chromium-review.googlesource.com/#/c/343538/
>
> Original issue's description:
> > Mark removable Drive caches for cryptohome.
> >
> > Design doc: https://docs.google.com/document/d/1Kdq-b5kpudbotQ1q2GRNd52f-oZUrf7mfuZb1XUqH4U/edit#heading=h.sa3sh9r88p48
> >
> > - Mark the Drive cache dir with +d (chattr) and user.GCacheFiles (setxattr), and mark removable files in it with +d.
> > - Toggle the status of metadata if cache files have been removed by
> >   cryptohome.
> > - Remove a file if it exsits but metadata says not (inconsistency possibly happens on abrupt shutdown).
> > - Toggle +d as necessary when pinned/dirty status are changed.
> >
> > Committed: https://crrev.com/1d5bdf653f3859d29ae3b6ad7535cefc3702f45d
> > Cr-Commit-Position: refs/heads/master@{#393229}
>
> TBR=hashimoto@chromium.org,yawano@chromium.org
> # Not skipping CQ checks because original CL landed more than 1 days ago.
>
> Committed: https://crrev.com/29e3ff432556adf39449824074e1032afb4de27f
> Cr-Commit-Position: refs/heads/master@{#394720}

TBR=hashimoto@chromium.org,yawano@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.

Committed: https://crrev.com/c6aac5025193119560369c1f3ca099a35f8ea86d
Cr-Commit-Position: refs/heads/master@{#395304}

[oka, 2016-05-22 22:43:48]

Created Reland of Mark removable Drive caches for cryptohome.

[commit-bot: I haz the power, 2016-05-22 22:43:51]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2006503002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/2006503002/1

[commit-bot: I haz the power, 2016-05-22 22:43:52]

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

[commit-bot: I haz the power, 2016-05-22 22:43:53]

No L-G-T-M from a valid reviewer yet. 
CQ run can only be started by full committers or once the patch has
received an L-G-T-M from a full committer.
Even if an L-G-T-M may have been provided, it was from a non-committer,
_not_ a full super star committer.
See http://www.chromium.org/getting-involved/become-a-committer
Note that this has nothing to do with OWNERS files.

[oka, 2016-05-23 04:40:45]

Description was changed from

==========
Reland of Mark removable Drive caches for cryptohome. (patchset #2 id:80001 of
https://codereview.chromium.org/1989863005/ )

Reason for revert:
Reland for M53.

Original issue's description:
> Revert of Mark removable Drive caches for cryptohome. (patchset #17 id:320001
of https://codereview.chromium.org/1918243004/ )
> 
> Reason for revert:
> Corresponding changes for Chrome OS [1] haven't been submitted. Let's punt
this change to M53.
> [1] https://chromium-review.googlesource.com/#/c/343538/
> 
> Original issue's description:
> > Mark removable Drive caches for cryptohome.
> >
> > Design doc:
https://docs.google.com/document/d/1Kdq-b5kpudbotQ1q2GRNd52f-oZUrf7mfuZb1XUqH...
> >
> > - Mark the Drive cache dir with +d (chattr) and user.GCacheFiles (setxattr),
and mark removable files in it with +d.
> > - Toggle the status of metadata if cache files have been removed by
> >   cryptohome.
> > - Remove a file if it exsits but metadata says not (inconsistency possibly
happens on abrupt shutdown).
> > - Toggle +d as necessary when pinned/dirty status are changed.
> >
> > Committed: https://crrev.com/1d5bdf653f3859d29ae3b6ad7535cefc3702f45d
> > Cr-Commit-Position: refs/heads/master@{#393229}
> 
> TBR=hashimoto@chromium.org,yawano@chromium.org
> # Not skipping CQ checks because original CL landed more than 1 days ago.
> 
> Committed: https://crrev.com/29e3ff432556adf39449824074e1032afb4de27f
> Cr-Commit-Position: refs/heads/master@{#394720}

TBR=hashimoto@chromium.org,yawano@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
==========

to

==========
Reland of Mark removable Drive caches for cryptohome. (patchset #2 id:80001 of
https://codereview.chromium.org/1989863005/ )

Reason for revert:
Reland for M53.

BUG=533750

Original issue's description:
> Revert of Mark removable Drive caches for cryptohome. (patchset #17 id:320001
of https://codereview.chromium.org/1918243004/ )
> 
> Reason for revert:
> Corresponding changes for Chrome OS [1] haven't been submitted. Let's punt
this change to M53.
> [1] https://chromium-review.googlesource.com/#/c/343538/
> 
> Original issue's description:
> > Mark removable Drive caches for cryptohome.
> >
> > Design doc:
https://docs.google.com/document/d/1Kdq-b5kpudbotQ1q2GRNd52f-oZUrf7mfuZb1XUqH...
> >
> > - Mark the Drive cache dir with +d (chattr) and user.GCacheFiles (setxattr),
and mark removable files in it with +d.
> > - Toggle the status of metadata if cache files have been removed by
> >   cryptohome.
> > - Remove a file if it exsits but metadata says not (inconsistency possibly
happens on abrupt shutdown).
> > - Toggle +d as necessary when pinned/dirty status are changed.
> >
> > Committed: https://crrev.com/1d5bdf653f3859d29ae3b6ad7535cefc3702f45d
> > Cr-Commit-Position: refs/heads/master@{#393229}
> 
> TBR=hashimoto@chromium.org,yawano@chromium.org
> # Not skipping CQ checks because original CL landed more than 1 days ago.
> 
> Committed: https://crrev.com/29e3ff432556adf39449824074e1032afb4de27f
> Cr-Commit-Position: refs/heads/master@{#394720}

TBR=hashimoto@chromium.org,yawano@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
==========

[oka, 2016-05-23 04:41:31]

PTAL. It requires LGTM since I'm not a full committer.

[hashimoto, 2016-05-23 05:45:38]

lgtm

[oka, 2016-05-23 08:07:22]

The CQ bit was checked by oka@chromium.org

[commit-bot: I haz the power, 2016-05-23 08:07:33]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2006503002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/2006503002/1

[commit-bot: I haz the power, 2016-05-23 10:46:26]

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

[commit-bot: I haz the power, 2016-05-23 10:46:27]

Try jobs failed on following builders:
  linux_chromium_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[oka, 2016-05-23 11:39:49]

The CQ bit was checked by oka@chromium.org

[commit-bot: I haz the power, 2016-05-23 11:40:06]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2006503002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/2006503002/1

[commit-bot: I haz the power, 2016-05-23 12:17:10]

Description was changed from

==========
Reland of Mark removable Drive caches for cryptohome. (patchset #2 id:80001 of
https://codereview.chromium.org/1989863005/ )

Reason for revert:
Reland for M53.

BUG=533750

Original issue's description:
> Revert of Mark removable Drive caches for cryptohome. (patchset #17 id:320001
of https://codereview.chromium.org/1918243004/ )
> 
> Reason for revert:
> Corresponding changes for Chrome OS [1] haven't been submitted. Let's punt
this change to M53.
> [1] https://chromium-review.googlesource.com/#/c/343538/
> 
> Original issue's description:
> > Mark removable Drive caches for cryptohome.
> >
> > Design doc:
https://docs.google.com/document/d/1Kdq-b5kpudbotQ1q2GRNd52f-oZUrf7mfuZb1XUqH...
> >
> > - Mark the Drive cache dir with +d (chattr) and user.GCacheFiles (setxattr),
and mark removable files in it with +d.
> > - Toggle the status of metadata if cache files have been removed by
> >   cryptohome.
> > - Remove a file if it exsits but metadata says not (inconsistency possibly
happens on abrupt shutdown).
> > - Toggle +d as necessary when pinned/dirty status are changed.
> >
> > Committed: https://crrev.com/1d5bdf653f3859d29ae3b6ad7535cefc3702f45d
> > Cr-Commit-Position: refs/heads/master@{#393229}
> 
> TBR=hashimoto@chromium.org,yawano@chromium.org
> # Not skipping CQ checks because original CL landed more than 1 days ago.
> 
> Committed: https://crrev.com/29e3ff432556adf39449824074e1032afb4de27f
> Cr-Commit-Position: refs/heads/master@{#394720}

TBR=hashimoto@chromium.org,yawano@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
==========

to

==========
Reland of Mark removable Drive caches for cryptohome. (patchset #2 id:80001 of
https://codereview.chromium.org/1989863005/ )

Reason for revert:
Reland for M53.

BUG=533750

Original issue's description:
> Revert of Mark removable Drive caches for cryptohome. (patchset #17 id:320001
of https://codereview.chromium.org/1918243004/ )
> 
> Reason for revert:
> Corresponding changes for Chrome OS [1] haven't been submitted. Let's punt
this change to M53.
> [1] https://chromium-review.googlesource.com/#/c/343538/
> 
> Original issue's description:
> > Mark removable Drive caches for cryptohome.
> >
> > Design doc:
https://docs.google.com/document/d/1Kdq-b5kpudbotQ1q2GRNd52f-oZUrf7mfuZb1XUqH...
> >
> > - Mark the Drive cache dir with +d (chattr) and user.GCacheFiles (setxattr),
and mark removable files in it with +d.
> > - Toggle the status of metadata if cache files have been removed by
> >   cryptohome.
> > - Remove a file if it exsits but metadata says not (inconsistency possibly
happens on abrupt shutdown).
> > - Toggle +d as necessary when pinned/dirty status are changed.
> >
> > Committed: https://crrev.com/1d5bdf653f3859d29ae3b6ad7535cefc3702f45d
> > Cr-Commit-Position: refs/heads/master@{#393229}
> 
> TBR=hashimoto@chromium.org,yawano@chromium.org
> # Not skipping CQ checks because original CL landed more than 1 days ago.
> 
> Committed: https://crrev.com/29e3ff432556adf39449824074e1032afb4de27f
> Cr-Commit-Position: refs/heads/master@{#394720}

TBR=hashimoto@chromium.org,yawano@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
==========

[commit-bot: I haz the power, 2016-05-23 12:17:11]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-05-23 12:18:05]

Description was changed from

==========
Reland of Mark removable Drive caches for cryptohome. (patchset #2 id:80001 of
https://codereview.chromium.org/1989863005/ )

Reason for revert:
Reland for M53.

BUG=533750

Original issue's description:
> Revert of Mark removable Drive caches for cryptohome. (patchset #17 id:320001
of https://codereview.chromium.org/1918243004/ )
> 
> Reason for revert:
> Corresponding changes for Chrome OS [1] haven't been submitted. Let's punt
this change to M53.
> [1] https://chromium-review.googlesource.com/#/c/343538/
> 
> Original issue's description:
> > Mark removable Drive caches for cryptohome.
> >
> > Design doc:
https://docs.google.com/document/d/1Kdq-b5kpudbotQ1q2GRNd52f-oZUrf7mfuZb1XUqH...
> >
> > - Mark the Drive cache dir with +d (chattr) and user.GCacheFiles (setxattr),
and mark removable files in it with +d.
> > - Toggle the status of metadata if cache files have been removed by
> >   cryptohome.
> > - Remove a file if it exsits but metadata says not (inconsistency possibly
happens on abrupt shutdown).
> > - Toggle +d as necessary when pinned/dirty status are changed.
> >
> > Committed: https://crrev.com/1d5bdf653f3859d29ae3b6ad7535cefc3702f45d
> > Cr-Commit-Position: refs/heads/master@{#393229}
> 
> TBR=hashimoto@chromium.org,yawano@chromium.org
> # Not skipping CQ checks because original CL landed more than 1 days ago.
> 
> Committed: https://crrev.com/29e3ff432556adf39449824074e1032afb4de27f
> Cr-Commit-Position: refs/heads/master@{#394720}

TBR=hashimoto@chromium.org,yawano@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
==========

to

==========
Reland of Mark removable Drive caches for cryptohome. (patchset #2 id:80001 of
https://codereview.chromium.org/1989863005/ )

Reason for revert:
Reland for M53.

BUG=533750

Original issue's description:
> Revert of Mark removable Drive caches for cryptohome. (patchset #17 id:320001
of https://codereview.chromium.org/1918243004/ )
>
> Reason for revert:
> Corresponding changes for Chrome OS [1] haven't been submitted. Let's punt
this change to M53.
> [1] https://chromium-review.googlesource.com/#/c/343538/
>
> Original issue's description:
> > Mark removable Drive caches for cryptohome.
> >
> > Design doc:
https://docs.google.com/document/d/1Kdq-b5kpudbotQ1q2GRNd52f-oZUrf7mfuZb1XUqH...
> >
> > - Mark the Drive cache dir with +d (chattr) and user.GCacheFiles (setxattr),
and mark removable files in it with +d.
> > - Toggle the status of metadata if cache files have been removed by
> >   cryptohome.
> > - Remove a file if it exsits but metadata says not (inconsistency possibly
happens on abrupt shutdown).
> > - Toggle +d as necessary when pinned/dirty status are changed.
> >
> > Committed: https://crrev.com/1d5bdf653f3859d29ae3b6ad7535cefc3702f45d
> > Cr-Commit-Position: refs/heads/master@{#393229}
>
> TBR=hashimoto@chromium.org,yawano@chromium.org
> # Not skipping CQ checks because original CL landed more than 1 days ago.
>
> Committed: https://crrev.com/29e3ff432556adf39449824074e1032afb4de27f
> Cr-Commit-Position: refs/heads/master@{#394720}

TBR=hashimoto@chromium.org,yawano@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.

Committed: https://crrev.com/c6aac5025193119560369c1f3ca099a35f8ea86d
Cr-Commit-Position: refs/heads/master@{#395304}
==========

[commit-bot: I haz the power, 2016-05-23 12:18:06]

Patchset 1 (id:??) landed as
https://crrev.com/c6aac5025193119560369c1f3ca099a35f8ea86d
Cr-Commit-Position: refs/heads/master@{#395304}