Remove usage of MainEntryPointTime() inside the impl which has direct access to the variable.

Remove usage of MainEntryPointTime() inside the impl which has direct access to the variable.

BUG=544131
(makes that cleanup slightly easier)

Committed: https://crrev.com/693129d89e57ff0d5fbfdbad6f6501a806467720
Cr-Commit-Position: refs/heads/master@{#358687}

[gab, 2015-11-09 19:14:34]

gab@chromium.org changed reviewers:
+ fdoray@chromium.org

[gab, 2015-11-09 19:14:35]

Francois PTAL :-).

Thanks,
Gab

[fdoray, 2015-11-09 19:19:54]

lgtm, but needs a rebase because some bots complain about a "Patch failure".
Will also conflict with https://codereview.chromium.org/1413153010/ , which is
in the CQ.

[gab, 2015-11-09 20:17:08]

On 2015/11/09 19:19:54, fdoray wrote:
> lgtm, but needs a rebase because some bots complain about a "Patch failure".
> Will also conflict with https://codereview.chromium.org/1413153010/ , which is
> in the CQ.

Ah indeed, this is because it's in fact based on top of
https://codereview.chromium.org/1413153010/ locally :-), so I'll just wait for
that to land and it should be fine!

Also, I just tweaked this CL to remove a bit more code, PTAL.

Thanks,
Gab

[fdoray, 2015-11-09 20:20:32]

lgtm
My CL landed so you can rebase now.

[fdoray, 2015-11-09 20:20:32]

lgtm
My CL landed so you can rebase now.

[gab, 2015-11-09 20:22:01]

On 2015/11/09 20:20:32, fdoray wrote:
> lgtm
> My CL landed so you can rebase now.

Don't think I have to rebase since I was based on it locally :-) -- crossing
fingers + CQ!

[gab, 2015-11-09 20:22:06]

The CQ bit was checked by gab@chromium.org

[commit-bot: I haz the power, 2015-11-09 20:24:21]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1413193011/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1413193011/20001

[commit-bot: I haz the power, 2015-11-09 20:40:02]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-11-09 20:40:03]

Try jobs failed on following builders:
  mac_chromium_gn_rel on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_gn_r...)

[fdoray, 2015-11-09 20:46:09]

oops... MainEntryPointTime() is used here
https://code.google.com/p/chromium/codesearch#chromium/src/chrome/browser/mac...

[gab, 2015-11-09 21:23:39]

Description was changed from

==========
Remove unused MainEntryPointTime() public interface.

BUG=544131 (makes that cleanup slightly easier)
==========

to

==========
Remove usage of MainEntryPointTime() inside the impl which has direct access to
the variable.

BUG=544131 (makes that cleanup slightly easier)
==========

[gab, 2015-11-09 21:24:27]

On 2015/11/09 20:46:09, fdoray wrote:
> oops... MainEntryPointTime() is used here
>
https://code.google.com/p/chromium/codesearch#chromium/src/chrome/browser/mac...

Oops indeed, not sure why my git grep missed that (or rather why my eyes missed
it in the output which it ought to have been in..).

I'd still like to do the rest of this change though (changed description
accordingly), WDYT?

[gab, 2015-11-09 21:24:42]

On 2015/11/09 21:24:27, gab wrote:
> On 2015/11/09 20:46:09, fdoray wrote:
> > oops... MainEntryPointTime() is used here
> >
>
https://code.google.com/p/chromium/codesearch#chromium/src/chrome/browser/mac...
> 
> Oops indeed, not sure why my git grep missed that (or rather why my eyes
missed
> it in the output which it ought to have been in..).
> 
> I'd still like to do the rest of this change though (changed description
> accordingly), WDYT?

(please CQ if it still LGTYs, thanks!)

[fdoray, 2015-11-09 21:28:08]

still lgtm. committing.

[fdoray, 2015-11-09 21:28:11]

The CQ bit was checked by fdoray@chromium.org

[commit-bot: I haz the power, 2015-11-09 21:29:55]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1413193011/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1413193011/40001

[commit-bot: I haz the power, 2015-11-09 22:37:14]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2015-11-09 22:38:59]

Patchset 3 (id:??) landed as
https://crrev.com/693129d89e57ff0d5fbfdbad6f6501a806467720
Cr-Commit-Position: refs/heads/master@{#358687}