OLD | NEW |
---|---|
(Empty) | |
1 // Copyright 2014 The Chromium Authors. All rights reserved. | |
2 // Use of this source code is governed by a BSD-style license that can be | |
3 // found in the LICENSE file. | |
4 | |
5 #ifndef CONTENT_COMMON_READBACK_TYPES_H_ | |
piman
2014/10/02 18:08:26
This file should go into content/public/browser, s
sivag
2014/11/06 13:34:16
Done.
| |
6 #define CONTENT_COMMON_READBACK_TYPES_H_ | |
7 | |
8 #include "base/callback.h" | |
9 | |
10 class SkBitmap; | |
11 | |
12 namespace content { | |
13 | |
14 enum ReadbackResponse { | |
15 READBACK_SUCCESS, | |
16 READBACK_FAILED, | |
17 READBACK_FORMAT_NOT_SUPPORTED, | |
18 READBACK_NOTSUPPORTED, | |
piman
2014/10/02 18:08:26
nit: NOT_SUPPORTED ?
sivag
2014/11/06 13:34:16
Done.
| |
19 READBACK_SURFACE_UNAVAILABLE, | |
20 READBACK_RESULT_EMPTY, | |
piman
2014/10/02 18:08:26
I don't think we need both RESULT_EMPTY and FAILED
sivag
2014/11/06 13:34:16
Done.
| |
21 READBACK_MEMORY_ALLOCATION_FAILURE, | |
22 }; | |
23 | |
24 typedef const base::Callback<void(bool, const SkBitmap&, const int&)> | |
piman
2014/10/02 18:08:26
Why int instead of ReadbackResponse?
piman
2014/10/02 18:08:26
We shouldn't need both the bool and the ReadbackRe
sivag
2014/11/06 13:34:16
Done.
sivag
2014/11/06 13:34:16
Done.
| |
25 CopyFromCompositingSurfaceCallback; | |
piman
2014/10/02 18:08:26
We should be consistent in the naming, "Copy" vs "
sivag
2014/11/06 13:34:16
Done.
| |
26 | |
27 } // namespace | |
28 | |
29 #endif // CONTENT_COMMON_READBACK_TYPES_H_ | |
OLD | NEW |