Fix a bug where read-only volumes were recognized as read-write.

Fix a bug where read-only volumes were recognized as read-write.

This problem happened because the read-only attribute of the disk volume was
unconditionally overwritten by the values in |access_modes_|. The patch fixes
this problem by applying the values in |access_modes_| only when it's needed.

The bug was introduced in crrev.com/2230713003 .

BUG=644077
NOTRY=true
NOPRESUBMIT=true

Review-Url: https://codereview.chromium.org/2311863002
Cr-Commit-Position: refs/heads/master@{#416575}
(cherry picked from commit 21448d5b5c6aaa880981393d0c55ef865b754596)

[yamaguchi, 2016-09-07 13:34:05]

yamaguchi@chromium.org changed reviewers:
+ satorux@chromium.org

[yamaguchi, 2016-09-07 13:34:06]

I would like to cherrypick this patch to fix the known issue with current M54
release branch. No manual work was needed for applying the patch. PTAL.

[satorux1, 2016-09-08 05:30:34]

satorux@chromium.org changed reviewers:
+ fukino@chromium.org

[satorux1, 2016-09-08 05:30:35]

lgtm, but do you need a code review for merging?

[fukino, 2016-09-08 05:31:54]

On 2016/09/08 05:30:35, satorux1 wrote:
> lgtm, but do you need a code review for merging?

I assume only committers can use TBR.

[yamaguchi, 2016-09-08 05:32:10]

The CQ bit was checked by yamaguchi@chromium.org

[commit-bot: I haz the power, 2016-09-08 05:32:31]

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-09-08 05:32:32]

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

[commit-bot: I haz the power, 2016-09-08 05:32:33]

CLs for remote refs other than refs/pending/heads/master must contain NOTRY=true
and NOPRESUBMIT=true in order for the CQ to process them

[yamaguchi, 2016-09-08 05:35:33]

Description was changed from

==========
Fix a bug where read-only volumes were recognized as read-write.

This problem happened because the read-only attribute of the disk volume was
unconditionally overwritten by the values in |access_modes_|. The patch fixes
this problem by applying the values in |access_modes_| only when it's needed.

The bug was introduced in crrev.com/2230713003 .

BUG=644077

Review-Url: https://codereview.chromium.org/2311863002
Cr-Commit-Position: refs/heads/master@{#416575}
(cherry picked from commit 21448d5b5c6aaa880981393d0c55ef865b754596)
==========

to

==========
Fix a bug where read-only volumes were recognized as read-write.

This problem happened because the read-only attribute of the disk volume was
unconditionally overwritten by the values in |access_modes_|. The patch fixes
this problem by applying the values in |access_modes_| only when it's needed.

The bug was introduced in crrev.com/2230713003 .

BUG=644077
NOTRY=true
NOPRESUBMIT=true

Review-Url: https://codereview.chromium.org/2311863002
Cr-Commit-Position: refs/heads/master@{#416575}
(cherry picked from commit 21448d5b5c6aaa880981393d0c55ef865b754596)
==========

[yamaguchi, 2016-09-08 05:35:39]

The CQ bit was checked by yamaguchi@chromium.org

[commit-bot: I haz the power, 2016-09-08 05:35:55]

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-09-08 05:47:38]

Description was changed from

==========
Fix a bug where read-only volumes were recognized as read-write.

This problem happened because the read-only attribute of the disk volume was
unconditionally overwritten by the values in |access_modes_|. The patch fixes
this problem by applying the values in |access_modes_| only when it's needed.

The bug was introduced in crrev.com/2230713003 .

BUG=644077
NOTRY=true
NOPRESUBMIT=true

Review-Url: https://codereview.chromium.org/2311863002
Cr-Commit-Position: refs/heads/master@{#416575}
(cherry picked from commit 21448d5b5c6aaa880981393d0c55ef865b754596)
==========

to

==========
Fix a bug where read-only volumes were recognized as read-write.

This problem happened because the read-only attribute of the disk volume was
unconditionally overwritten by the values in |access_modes_|. The patch fixes
this problem by applying the values in |access_modes_| only when it's needed.

The bug was introduced in crrev.com/2230713003 .

BUG=644077
NOTRY=true
NOPRESUBMIT=true

Review-Url: https://codereview.chromium.org/2311863002
Cr-Commit-Position: refs/heads/master@{#416575}
(cherry picked from commit 21448d5b5c6aaa880981393d0c55ef865b754596)
==========

[commit-bot: I haz the power, 2016-09-08 05:47:40]

Committed patchset #1 (id:1)