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

Unified Diff: mojo/edk/system/awakable_list_unittest.cc

Issue 2056763003: EDK: Change Awakable::Awake()'s context, etc., to be a uint64_t. (Closed) Base URL: https://github.com/domokit/mojo.git@master
Patch Set: doh Created 4 years, 6 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « mojo/edk/system/awakable_list.cc ('k') | mojo/edk/system/core.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: mojo/edk/system/awakable_list_unittest.cc
diff --git a/mojo/edk/system/awakable_list_unittest.cc b/mojo/edk/system/awakable_list_unittest.cc
index d698b9bf30a686a2a6f7c6fb091d91583d3b09ae..9772e7af438efb4fdf27790611b193d6f894a4c1 100644
--- a/mojo/edk/system/awakable_list_unittest.cc
+++ b/mojo/edk/system/awakable_list_unittest.cc
@@ -24,7 +24,7 @@ namespace {
TEST(AwakableListTest, BasicCancel) {
MojoResult result;
- uint32_t context;
+ uint64_t context;
// Cancel immediately after thread start.
{
@@ -65,7 +65,7 @@ TEST(AwakableListTest, BasicCancel) {
TEST(AwakableListTest, BasicAwakeSatisfied) {
MojoResult result;
- uint32_t context;
+ uint64_t context;
// Awake immediately after thread start.
{
@@ -115,7 +115,7 @@ TEST(AwakableListTest, BasicAwakeSatisfied) {
TEST(AwakableListTest, BasicAwakeUnsatisfiable) {
MojoResult result;
- uint32_t context;
+ uint64_t context;
// Awake (for unsatisfiability) immediately after thread start.
{
@@ -165,10 +165,10 @@ TEST(AwakableListTest, MultipleAwakables) {
MojoResult result2;
MojoResult result3;
MojoResult result4;
- uint32_t context1;
- uint32_t context2;
- uint32_t context3;
- uint32_t context4;
+ uint64_t context1;
+ uint64_t context2;
+ uint64_t context3;
+ uint64_t context4;
// Cancel two awakables.
{
@@ -289,7 +289,7 @@ class KeepAwakable : public Awakable {
public:
KeepAwakable() : awake_count(0) {}
- bool Awake(MojoResult result, uintptr_t context) override {
+ bool Awake(MojoResult result, uint64_t context) override {
awake_count++;
return true;
}
@@ -303,7 +303,7 @@ class RemoveAwakable : public Awakable {
public:
RemoveAwakable() : awake_count(0) {}
- bool Awake(MojoResult result, uintptr_t context) override {
+ bool Awake(MojoResult result, uint64_t context) override {
awake_count++;
return false;
}
« no previous file with comments | « mojo/edk/system/awakable_list.cc ('k') | mojo/edk/system/core.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698