Need to set the MICROSOFT_TABLETPENSERVICE_PROPERTY window property even for child windows

base/win/win_util.h

Index: base/win/win_util.h
diff --git a/base/win/win_util.h b/base/win/win_util.h
index b261ce7b5eae91dd12f826e124b552be92a18b44..39d302eb53f42dd4f4785886b44e37f21a7fc91e 100644
--- a/base/win/win_util.h
+++ b/base/win/win_util.h
@@ -174,6 +174,12 @@ BASE_EXPORT bool IsUser32AndGdi32Available();
 BASE_EXPORT bool GetLoadedModulesSnapshot(HANDLE process,
                                           std::vector<HMODULE>* snapshot);
 
+// Adds or removes the MICROSOFT_TABLETPENSERVICE_PROPERTY property with the
+// TABLET_DISABLE_FLICKS & TABLET_DISABLE_FLICKFALLBACKKEYS flags in order to
+// disable pen flick gestures for the given HWND. If allow == true, then the
+// property is removed from the HWND else the property is added.
+BASE_EXPORT void AllowFlicks(HWND hwnd, bool allow);

[sky, 2016/04/07 17:57:54]

AllowFlicks with a boolean parameter named allow is rather confusing. I vote for
two functions: EnableFlicks and DisableFlicks

[kylix_rd, 2016/04/07 18:46:25]

Done.

[kylix_rd, 2016/04/07 18:46:25]

I'd rather this have been a method on a common base between the
HWNDMessageHandler & LegacyRenderWidgetHostHWND. At that point it would have
been a "property" of the class and the method would have been something like
HWNDBase::setAllowFlicks(bool allow){}.

Alas, that's not the case so the consolidation into a single function is causing
some confusion.

I'll be happy to break it into separate functions.

+
 }  // namespace win
 }  // namespace base