Commit 9e6fe08e authored by tonyg@chromium.org's avatar tonyg@chromium.org

Fix switch statement in DataFetcherSharedMemory::SetBufferAvailableState.

I noticed this because a Telemetry unittest was triggering the NOTREACHED
during session restore on Windows. I don't know much about this code, but the
fallthroughs certainly look unintentional.

BUG=

Review URL: https://codereview.chromium.org/317163006

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@275425 0039d316-1c4b-4281-b951-d872f2087c98
parent 8d869ac7
...@@ -384,12 +384,14 @@ void DataFetcherSharedMemory::SetBufferAvailableState( ...@@ -384,12 +384,14 @@ void DataFetcherSharedMemory::SetBufferAvailableState(
orientation_buffer_->data.allAvailableSensorsAreActive = enabled; orientation_buffer_->data.allAvailableSensorsAreActive = enabled;
orientation_buffer_->seqlock.WriteEnd(); orientation_buffer_->seqlock.WriteEnd();
} }
break;
case CONSUMER_TYPE_MOTION: case CONSUMER_TYPE_MOTION:
if (motion_buffer_) { if (motion_buffer_) {
motion_buffer_->seqlock.WriteBegin(); motion_buffer_->seqlock.WriteBegin();
motion_buffer_->data.allAvailableSensorsAreActive = enabled; motion_buffer_->data.allAvailableSensorsAreActive = enabled;
motion_buffer_->seqlock.WriteEnd(); motion_buffer_->seqlock.WriteEnd();
} }
break;
default: default:
NOTREACHED(); NOTREACHED();
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment