diff options
author | Chris Blume <cblume@google.com> | 2018-11-19 14:25:31 -0800 |
---|---|---|
committer | Chris Blume <cblume@google.com> | 2018-11-19 14:25:31 -0800 |
commit | 0fd02aa33dcbca0296b2fd197cb750e0d11c4925 (patch) | |
tree | bfd3d166320a63939207f376e6ff677c1fbd6881 /native/webview | |
parent | e9f5e860547e7314c4c0e03fa4838c5b06164f5d (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/webview')
-rw-r--r-- | native/webview/plat_support/draw_gl_functor.cpp | 4 |
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"); |