Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(221)

Side by Side Diff: third_party/sudden_motion_sensor/sudden_motion_sensor_mac.cc

Issue 1134753004: Update Sudden Motion Sensor notes for early 2015 Apple notebook computers. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 // 4 //
5 // This file is based on the SMSLib library. 5 // This file is based on the SMSLib library.
6 // 6 //
7 // SMSLib Sudden Motion Sensor Access Library 7 // SMSLib Sudden Motion Sensor Access Library
8 // Copyright (c) 2010 Suitable Systems 8 // Copyright (c) 2010 Suitable Systems
9 // All rights reserved. 9 // All rights reserved.
10 // 10 //
(...skipping 116 matching lines...) Expand 10 before | Expand all | Expand 10 after
127 127
128 // Tested by Adam Gerson (via avi) on a 13" MacBook. 128 // Tested by Adam Gerson (via avi) on a 13" MacBook.
129 { "MacBook5,2", NULL, { { 0, false }, { 2, true }, { 4, true } } }, 129 { "MacBook5,2", NULL, { { 0, false }, { 2, true }, { 4, true } } },
130 130
131 // Tested by tommyw on a 13" MacBook. 131 // Tested by tommyw on a 13" MacBook.
132 { "MacBook6,1", NULL, { { 0, true }, { 2, true }, { 4, false } } }, 132 { "MacBook6,1", NULL, { { 0, true }, { 2, true }, { 4, false } } },
133 133
134 // Tested by avi on a 13" MacBook. 134 // Tested by avi on a 13" MacBook.
135 { "MacBook7,1", NULL, { { 0, true }, { 2, true }, { 4, false } } }, 135 { "MacBook7,1", NULL, { { 0, true }, { 2, true }, { 4, false } } },
136 136
137 // Note:
138 // - MacBook8,1 (12" MacBook, early 2015)
139 // has no accelerometer sensors.
Robert Sesek 2015/05/18 15:55:48 Interesting.
140
137 // Tested by crc on a 13" MacBook Air. 141 // Tested by crc on a 13" MacBook Air.
138 { "MacBookAir1,1", NULL, { { 0, true }, { 2, true }, { 4, false } } }, 142 { "MacBookAir1,1", NULL, { { 0, true }, { 2, true }, { 4, false } } },
139 143
140 // Tested by sfiera, pjw on a 13" MacBook Air. 144 // Tested by sfiera, pjw on a 13" MacBook Air.
141 { "MacBookAir2,1", NULL, { { 0, true }, { 2, true }, { 4, false } } }, 145 { "MacBookAir2,1", NULL, { { 0, true }, { 2, true }, { 4, false } } },
142 146
143 // Note: 147 // Note:
144 // - MacBookAir3,1 (11" MacBook Air, late 2010) 148 // - MacBookAir3,1 (11" MacBook Air, late 2010)
145 // - MacBookAir3,2 (13" MacBook Air, late 2010) 149 // - MacBookAir3,2 (13" MacBook Air, late 2010)
146 // - MacBookAir4,1 (11" MacBook Air, mid 2011) 150 // - MacBookAir4,1 (11" MacBook Air, mid 2011)
147 // - MacBookAir4,2 (13" MacBook Air, mid 2011) 151 // - MacBookAir4,2 (13" MacBook Air, mid 2011)
148 // - MacBookAir5,1 (11" MacBook Air, mid 2012) 152 // - MacBookAir5,1 (11" MacBook Air, mid 2012)
149 // - MacBookAir5,2 (13" MacBook Air, mid 2012) 153 // - MacBookAir5,2 (13" MacBook Air, mid 2012)
150 // - MacBookAir6,1 (11" MacBook Air, mid 2013) 154 // - MacBookAir6,1 (11" MacBook Air, mid 2013)
151 // - MacBookAir6,2 (13" MacBook Air, mid 2013) 155 // - MacBookAir6,2 (13" MacBook Air, mid 2013)
156 // - MacBookAir7,1 (11" MacBook Air, early 2015)
157 // - MacBookAir7,2 (13" MacBook Air, early 2015)
152 // have no accelerometer sensors. 158 // have no accelerometer sensors.
153 159
154 // Tested by crc on a 15" MacBook Pro. 160 // Tested by crc on a 15" MacBook Pro.
155 { "MacBookPro1,1", NULL, { { 0, true }, { 2, true }, { 4, false } } }, 161 { "MacBookPro1,1", NULL, { { 0, true }, { 2, true }, { 4, false } } },
156 162
157 // Tested by Raul Cuza (via avi) on a 17" MacBook Pro. 163 // Tested by Raul Cuza (via avi) on a 17" MacBook Pro.
158 { "MacBookPro1,2", NULL, { { 0, true }, { 2, true }, { 4, false } } }, 164 { "MacBookPro1,2", NULL, { { 0, true }, { 2, true }, { 4, false } } },
159 165
160 // Tested by L.V. (via avi) on a 17" MacBook Pro. 166 // Tested by L.V. (via avi) on a 17" MacBook Pro.
161 { "MacBookPro2,1", NULL, { { 0, true }, { 2, false }, { 4, true } } }, 167 { "MacBookPro2,1", NULL, { { 0, true }, { 2, false }, { 4, true } } },
(...skipping 55 matching lines...) Expand 10 before | Expand all | Expand 10 after
217 // - MacBookPro10,1 (15" MacBook Pro with Retina display) 223 // - MacBookPro10,1 (15" MacBook Pro with Retina display)
218 // has no accelerometer sensors. 224 // has no accelerometer sensors.
219 225
220 // MacBookPro10,2 handled by the generic case below. 226 // MacBookPro10,2 handled by the generic case below.
221 // Tested by avi on a 13" MacBook Pro with Retina display. 227 // Tested by avi on a 13" MacBook Pro with Retina display.
222 228
223 // Note: 229 // Note:
224 // - MacBookPro11,1 (13" MacBook Pro with Retina display) 230 // - MacBookPro11,1 (13" MacBook Pro with Retina display)
225 // - MacBookPro11,2 (15" MacBook Pro with Retina display) 231 // - MacBookPro11,2 (15" MacBook Pro with Retina display)
226 // - MacBookPro11,3 (15" MacBook Pro with Retina display) 232 // - MacBookPro11,3 (15" MacBook Pro with Retina display)
233 // - MacBookPro12,1 (13" MacBook Pro with Retina display)
227 // have no accelerometer sensors. 234 // have no accelerometer sensors.
228 235
229 // Generic MacBook accelerometer sensor data, to be used for future models 236 // Generic MacBook accelerometer sensor data, to be used for future models
230 // as well as models for which it is verified to be correct. Note that this 237 // as well as models for which it is verified to be correct. Note that this
231 // configuration may have problems with inverted axes when used generically 238 // configuration may have problems with inverted axes when used generically
232 // for untested models. 239 // for untested models.
233 { "", NULL, { { 0, false }, { 2, false }, { 4, false } } } 240 { "", NULL, { { 0, false }, { 2, false }, { 4, false } } }
234 }; 241 };
235 242
236 // Create a SuddenMotionSensor object and return NULL if no valid sensor found. 243 // Create a SuddenMotionSensor object and return NULL if no valid sensor found.
(...skipping 178 matching lines...) Expand 10 before | Expand all | Expand 10 after
415 if (value & 0x00800000) 422 if (value & 0x00800000)
416 return value | 0xff000000; 423 return value | 0xff000000;
417 break; 424 break;
418 425
419 default: 426 default:
420 LOG(FATAL) << "Invalid integer size for sign extension: " << size; 427 LOG(FATAL) << "Invalid integer size for sign extension: " << size;
421 } 428 }
422 429
423 return value; 430 return value;
424 } 431 }
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698