Noissue - Remove a dead inheritance and some dead code

Noissue - Remove a dead inheritance and some dead code

Remove unused "...Version" member variables in 'CPluginTabBase'.

Remove inheritance of 'CAdblockPlusClient' from 'CPluginClientBase'. The only
inherited member was 's_criticalSectionLocal'. Duplicate that member in
'CAdblockPlusClient'; keep the original member (but with a new name).

Clean up the #include statement in affected headers, removing some unused
headers, swapping out others in exchange for more specific one.

Rename 'CPluginClientBase' to 'LogQueue'. This reflects the only remaining
function of the old base class.

[Eric, 2015-02-18 19:20:25]

http://codereview.adblockplus.org/5994126579335168/diff/5668600916475904/src/...
File src/plugin/PluginTabBase.cpp (left):

http://codereview.adblockplus.org/5994126579335168/diff/5668600916475904/src/...
src/plugin/PluginTabBase.cpp:360: }
I don't know where this code came from originally, but how does one not notice a
constant condition that you've just set on the previous line?

http://codereview.adblockplus.org/5994126579335168/diff/5668600916475904/src/...
File src/plugin/PluginTabBase.cpp (right):

http://codereview.adblockplus.org/5994126579335168/diff/5668600916475904/src/...
src/plugin/PluginTabBase.cpp:313: */
I put this comment into the change set because the present changes (all
worthwhile in their own right) are also a prerequisite for accomplishing this
TODO, which has a number of benefits:

-- Change from one thread per tab to a single thread for the log queue.

-- Use notify/wait within the log queue rather than polling in multiple threads.

-- Eliminate CPluginTabBase member variables m_isActivate and
m_continueThreadRunning, whose only purpose is to manage the loop at the end of
this thread-main function. In turn this allows elimination of two entry points,
OnActivate and OnUpdate, whose only behavior is to refresh m_isActivate.

-- Further localize all the debug/log/trace/reporting capabilities.

I'm willing to replace this TODO comment with an issue in Trac, but if we do
that, we should have at least a bit of discussion beforehand to decide the scope
of the task.

[sergei, 2015-02-19 10:52:45]

LGTM

[Oleksandr, 2015-02-27 10:59:27]

http://codereview.adblockplus.org/5994126579335168/diff/5668600916475904/src/...
File src/plugin/PluginTabBase.cpp (left):

http://codereview.adblockplus.org/5994126579335168/diff/5668600916475904/src/...
src/plugin/PluginTabBase.cpp:360: }
On 2015/02/18 19:20:25, Eric wrote:
> I don't know where this code came from originally, but how does one not notice
a
> constant condition that you've just set on the previous line?

Let's just say there was a lot of optimization happening here. Also, let's not
ever mention this, ok? Ever. :D

http://codereview.adblockplus.org/5994126579335168/diff/5668600916475904/src/...
File src/plugin/PluginTabBase.cpp (right):

http://codereview.adblockplus.org/5994126579335168/diff/5668600916475904/src/...
src/plugin/PluginTabBase.cpp:313: */
On 2015/02/18 19:20:25, Eric wrote:
> I put this comment into the change set because the present changes (all
> worthwhile in their own right) are also a prerequisite for accomplishing this
> TODO, which has a number of benefits:
> 
> -- Change from one thread per tab to a single thread for the log queue.
> 
> -- Use notify/wait within the log queue rather than polling in multiple
threads.
> 
> -- Eliminate CPluginTabBase member variables m_isActivate and
> m_continueThreadRunning, whose only purpose is to manage the loop at the end
of
> this thread-main function. In turn this allows elimination of two entry
points,
> OnActivate and OnUpdate, whose only behavior is to refresh m_isActivate.
> 
> -- Further localize all the debug/log/trace/reporting capabilities.
> 
> I'm willing to replace this TODO comment with an issue in Trac, but if we do
> that, we should have at least a bit of discussion beforehand to decide the
scope
> of the task.

I'd vote for an issue instead of a TODO. Though I don't see any immediate need
in doing this at all. Even if we create an issue I'd vote for it to be P4 or
even P5.

[Eric, 2015-02-27 16:47:47]

New patch set.

http://codereview.adblockplus.org/5994126579335168/diff/5668600916475904/src/...
File src/plugin/PluginTabBase.cpp (left):

http://codereview.adblockplus.org/5994126579335168/diff/5668600916475904/src/...
src/plugin/PluginTabBase.cpp:360: }
On 2015/02/27 10:59:27, Oleksandr wrote:
> Also, let's not
> ever mention this, ok? Ever. :D

I promise I won't bookmark this URL and pull it out whenever I'm particularly
frustrated with code quality.

Really. I promise.

http://codereview.adblockplus.org/5994126579335168/diff/5668600916475904/src/...
File src/plugin/PluginTabBase.cpp (right):

http://codereview.adblockplus.org/5994126579335168/diff/5668600916475904/src/...
src/plugin/PluginTabBase.cpp:313: */
On 2015/02/27 10:59:27, Oleksandr wrote:
> I'd vote for an issue instead of a TODO. 

https://issues.adblockplus.org/ticket/2058

> Though I don't see any immediate need
> in doing this at all. Even if we create an issue I'd vote for it to be P4 or
> even P5.

I don't know of any really pressing need, but it would make finding the right
thread in the debugger easier.

It will also make combining CPluginTabBase and CPluginTab go easier, because
there will be less to combine.

[Oleksandr, 2015-03-03 18:43:59]

LGTM