| commit | 54c405e5ef445eb423ff7a992b0752afe6fc4222 | |
| author | jyasskin <jyasskin@chromium.org> | |
| Tue, 14 Jul 2015 23:59:48 +0000 (14 16:59 -0700) | ||
| committer | Commit bot <commit-bot@chromium.org> | |
| Wed, 15 Jul 2015 00:00:18 +0000 (15 00:00 +0000) | ||
| tree | e1d87751ca51f6141332ea24fbc4c257e242b9e6 | treesnapshot (tar.gz zip) |
| parent | 185ebee1f24d5bc94d4670bece9291ae9abd48eb | commitdiff |
Use the WebBluetoothErrorMessage enum to avoid sending message strings over IPC.
By using the Blink enum directly in the IPC, we can drop the content::BluetoothError enum and the function translating it to WebBluetoothError::ErrorType.
This is the second of a 5-patch series:
1. Add a Blink enum under a temporary name, WebBluetoothErrorMessage, that represents errors and their message strings. WebBluetoothError has a temporary conversion from this enum. (https://codereview.chromium.org/1232063007)
2. Replace message strings in IPCs with the Blink enum, converting to WebBluetoothError on entry to Blink. (This patch)
3. Replace the old message+error class with the new enum. Content's conversion to WebBluetoothError is now a no-op. (https://codereview.chromium.org/1230193003)
4. Replace Content's uses of the temporary enum name with the final name. (https://codereview.chromium.org/1228323005)
5. Remove the temporary enum name from Blink. (https://codereview.chromium.org/1236723002)
BUG=508299
Review URL: https://codereview.chromium.org/1224273009
Cr-Commit-Position: refs/heads/master@{#338777}
By using the Blink enum directly in the IPC, we can drop the content::BluetoothError enum and the function translating it to WebBluetoothError::ErrorType.
This is the second of a 5-patch series:
1. Add a Blink enum under a temporary name, WebBluetoothErrorMessage, that represents errors and their message strings. WebBluetoothError has a temporary conversion from this enum. (https://codereview.chromium.org/1232063007)
2. Replace message strings in IPCs with the Blink enum, converting to WebBluetoothError on entry to Blink. (This patch)
3. Replace the old message+error class with the new enum. Content's conversion to WebBluetoothError is now a no-op. (https://codereview.chromium.org/1230193003)
4. Replace Content's uses of the temporary enum name with the final name. (https://codereview.chromium.org/1228323005)
5. Remove the temporary enum name from Blink. (https://codereview.chromium.org/1236723002)
BUG=508299
Review URL: https://codereview.chromium.org/1224273009
Cr-Commit-Position: refs/heads/master@{#338777}