SearchEngineTabHelper: added delegate_ member initializer to constructor.

SearchEngineTabHelper: added delegate_ member initializer to constructor.

BUG=none

Committed: https://crrev.com/a45e34acb629d58dcc3e8f9a61b2974934d4fb31
Cr-Commit-Position: refs/heads/master@{#304789}

[Lukasz Jagielski, 2014-11-18 11:36:47]

ljagielski@opera.com changed reviewers:
+ pkasting@chromium.org

[Lukasz Jagielski, 2014-11-18 11:37:20]

Hi,
PTAL at my CL proposition.
Regards,
Łukasz

[Peter Kasting, 2014-11-18 20:38:24]

LGTM; is there a bug this was causing?  Is so add it to the BUG= line, otherwise
write "BUG=none".

[Lukasz Jagielski, 2014-11-19 08:12:38]

The CQ bit was checked by ljagielski@opera.com

[Lukasz Jagielski, 2014-11-19 08:13:40]

On 2014/11/18 20:38:24, Peter Kasting wrote:
> LGTM; is there a bug this was causing?  Is so add it to the BUG= line,
otherwise
> write "BUG=none".

Well I just saw use of uninitialized value in memcheck. Should I report a bug
for this in the future?

[commit-bot: I haz the power, 2014-11-19 08:13:52]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/735683002/1

[commit-bot: I haz the power, 2014-11-19 09:05:21]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2014-11-19 09:05:58]

Patchset 1 (id:??) landed as
https://crrev.com/a45e34acb629d58dcc3e8f9a61b2974934d4fb31
Cr-Commit-Position: refs/heads/master@{#304789}

[Peter Kasting, 2014-11-19 09:13:26]

On 2014/11/19 08:13:40, Lukasz Jagielski wrote:
> On 2014/11/18 20:38:24, Peter Kasting wrote:
> > LGTM; is there a bug this was causing?  Is so add it to the BUG= line,
> otherwise
> > write "BUG=none".
> 
> Well I just saw use of uninitialized value in memcheck. Should I report a bug
> for this in the future?

No, no need, but do add something to the CL description in the future in such
cases about how the fix came about (e.g. "saw this uninitialized value in
memcheck").