Call sqlite3Preload() only when not using system sqlite. Modify the...

Call sqlite3Preload() only when not using system sqlite.  Modify the
sqlite3.h header to not include a stub definition of sqlite3Preload().

BUG=None
TEST=None

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=72637

[rohitrao (ping after 24h), 2011-01-25 22:25:33]

CL as promised.  I tried building this on my linux box with use_system_sqlite=1
and it compiled ok.  The resulting binary seems to run ok, although I'm not
entirely sure how to test it. 

If I try to build with use_system_sqlite=1 but without these changes, I end up
with an error.  I'm not sure why gentoo users don't hit this:
./third_party/sqlite/sqlite3.h:17: error: ‘int sqlite3Preload(sqlite3*)’ defined
but not used

It looks like the trybots are compiling as well, so this CL should be good when
use_system_sqlite=0.

[Paweł Hajdan Jr., 2011-01-26 07:41:19]

LGTM

[rohitrao (ping after 24h), 2011-01-26 14:54:21]

Submitted as r72637.  Thanks for the review.

I don't expect this to cause any problems, but I've probably just gone
and jinxed myself.  Please let me know if you hear of anyone having
problems and I'll roll this back.

- Rohit

On Wed, Jan 26, 2011 at 2:41 AM,  <phajdan.jr@chromium.org> wrote:
> LGTM
>
> http://codereview.chromium.org/6356012/
>

[Paweł Hajdan Jr., 2011-01-26 14:57:43]

No worries, use_system_sqlite is busted anyway, due to
http://code.google.com/p/chromium/issues/detail?id=70589

On Wed, Jan 26, 2011 at 15:45, Rohit Rao <rohitrao@chromium.org> wrote:

> Submitted as r72637.  Thanks for the review.
>
> I don't expect this to cause any problems, but I've probably just gone
> and jinxed myself.  Please let me know if you hear of anyone having
> problems and I'll roll this back.
>
> - Rohit
>
> On Wed, Jan 26, 2011 at 2:41 AM,  <phajdan.jr@chromium.org> wrote:
> > LGTM
> >
> > http://codereview.chromium.org/6356012/
> >
>