summaryrefslogtreecommitdiff
path: root/native
diff options
context:
space:
mode:
authorChris Blume <cblume@google.com>2018-11-19 14:25:31 -0800
committerChris Blume <cblume@google.com>2018-11-19 14:25:31 -0800
commit0fd02aa33dcbca0296b2fd197cb750e0d11c4925 (patch)
treebfd3d166320a63939207f376e6ff677c1fbd6881 /native
parente9f5e860547e7314c4c0e03fa4838c5b06164f5d (diff)
Use "override" keyword instead of "virtual".
"override" provides a compile-time gaurantee that the function is indeed overriding a virtual function. This prevents the potential mistake of creating a new virtual function rather than overriding the original. I happened to notice we could use "override" here instead of "virtual". Might as well tidy up a bit. Test: Built locally BUG=115613038 Change-Id: I7f43f4a466d8ceaa1b863d6a2af054e69618d0c8
Diffstat (limited to 'native')
-rw-r--r--native/webview/plat_support/draw_gl_functor.cpp4
1 files changed, 2 insertions, 2 deletions
diff --git a/native/webview/plat_support/draw_gl_functor.cpp b/native/webview/plat_support/draw_gl_functor.cpp
index 7cb49da483de..e3e52b1ea1f1 100644
--- a/native/webview/plat_support/draw_gl_functor.cpp
+++ b/native/webview/plat_support/draw_gl_functor.cpp
@@ -42,10 +42,10 @@ AwDrawGLFunction* g_aw_drawgl_function = NULL;
class DrawGLFunctor : public Functor {
public:
explicit DrawGLFunctor(jlong view_context) : view_context_(view_context) {}
- virtual ~DrawGLFunctor() {}
+ ~DrawGLFunctor() override {}
// Functor
- virtual status_t operator ()(int what, void* data) {
+ status_t operator ()(int what, void* data) override {
using uirenderer::DrawGlInfo;
if (!g_aw_drawgl_function) {
ALOGE("Cannot draw: no DrawGL Function installed");