[NOT COMMITTED] Inline TaskTraits' destructor per it only having POD type members

base/task_scheduler/task_traits.h

Index: base/task_scheduler/task_traits.h
diff --git a/base/task_scheduler/task_traits.h b/base/task_scheduler/task_traits.h
index 523fd137b66557e7146f1774b661b8b8e3cf6d5e..644ffea95702828c99adb4ba43ad7ca2c0fe130e 100644
--- a/base/task_scheduler/task_traits.h
+++ b/base/task_scheduler/task_traits.h
@@ -87,7 +87,7 @@ class BASE_EXPORT TaskTraits {
   TaskTraits();
   TaskTraits(const TaskTraits& other) = default;
   TaskTraits& operator=(const TaskTraits& other) = default;
-  ~TaskTraits();
+  ~TaskTraits() = default;

[robliao, 2016/04/07 17:43:18]

The Chromium guidance is

"Define them in the implementation file unless you really need them to be
inlined."

Also noted, this also apparently breaks forward declared variables (presumably
since the compiler now needs to know everything about setting the object up). 

https://www.chromium.org/developers/coding-style/cpp-dos-and-donts#TOC-Stop-i...

Having said that, do we gain something from inlining this destructor?

[gab, 2016/04/08 21:00:04]

Hmm, interesting, do we expect TaskTraits to be fwd-declared much?
If it's not inlined then SchedulerParallelTaskRunner also has to have an
explicit out-of-line destructor:
https://codereview.chromium.org/1708773002/diff/550001/base/task_scheduler/sc...

(that review is why I wondered whether TaskTraits had a trivial inline
destructor with POD types only)

But I guess the point of that style rule is that having that dependency on
SchedulerParallelTaskRunner is no good anyways (probably doesn't matter much in
that case since all callers won't inline but rather use virtual dispatch to get
to the destructor but wtv).

So I guess not inlining is the right thing here after all.

Closing, thanks.

[gab, 2016/04/08 21:01:48]

Actually that destructor is private so virtual dispatch is really the only way
to get to it and inlining isn't a concern, so nothing needs to change, ignore me
:-)!

 
   // Allows tasks with these traits to do file I/O.
   TaskTraits& WithFileIO();