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

Unified Diff: sky/engine/core/painting/TransferMode.cpp

Issue 1144233004: Sky: Convert TransferMode to an enum. (Closed) Base URL: git@github.com:/domokit/mojo.git@master
Patch Set: review 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | sky/engine/core/painting/TransferMode.dart » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: sky/engine/core/painting/TransferMode.cpp
diff --git a/sky/engine/core/painting/TransferMode.cpp b/sky/engine/core/painting/TransferMode.cpp
index 8141b5947e9314652d637245326125faa5a627f1..afd168e3d1425408ee6fa6d4d4af299c0cfec1fd 100644
--- a/sky/engine/core/painting/TransferMode.cpp
+++ b/sky/engine/core/painting/TransferMode.cpp
@@ -12,6 +12,11 @@
namespace blink {
+// If this fails, it's because SkXfermode has changed. We need to change
+// TransferMode.dart to ensure the TransferMode enum is in sync with the C++
+// values.
+COMPILE_ASSERT(SkXfermode::kLastMode == 28, Need_to_update_TransferMode_dart);
Hixie 2015/06/03 20:10:47 that's great.
+
// Convert dart_mode => SkXfermode::Mode.
SkXfermode::Mode DartConverter<TransferMode>::FromArgumentsWithNullCheck(
Dart_NativeArguments args,
@@ -23,7 +28,7 @@ SkXfermode::Mode DartConverter<TransferMode>::FromArgumentsWithNullCheck(
DCHECK(!LogIfError(dart_mode));
Dart_Handle value =
- Dart_GetField(dart_mode, DOMDartState::Current()->value_handle());
+ Dart_GetField(dart_mode, DOMDartState::Current()->index_handle());
uint64_t mode = 0;
Dart_Handle rv = Dart_IntegerToUint64(value, &mode);
« no previous file with comments | « no previous file | sky/engine/core/painting/TransferMode.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698