Base: Don't overwrite the thread id when ThreadMain exits. There's

Base: Don't overwrite the thread id when ThreadMain exits. There's
no need to do that, and it makes debugging easier to have the value.

BUG=127931
TEST=none
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=138014

[willchan no longer on Chromium, 2012-05-18 21:49:20]

Lgtm
On May 18, 2012 2:43 PM, <rvargas@chromium.org> wrote:

> Reviewers: willchan,
>
> Description:
> Base: Don't overwrite the thread id when ThreadMain exits. There's
> no need to do that, and it makes debugging easier to have the value.
>
> BUG=127931
> TEST=none
>
> Please review this at
https://chromiumcodereview.**appspot.com/10388202/<https://chromiumcodereview...
>
> SVN Base:
svn://svn.chromium.org/chrome/**trunk/src/<http://svn.chromium.org/chrome/trunk/src/>
>
> Affected files:
>  M     base/threading/thread.cc
>
>
> Index: base/threading/thread.cc
> ==============================**==============================**=======
> --- base/threading/thread.cc    (revision 137792)
> +++ base/threading/thread.cc    (working copy)
> @@ -171,7 +171,6 @@
>     // We can't receive messages anymore.
>     message_loop_ = NULL;
>   }
> -  thread_id_ = kInvalidThreadId;
>  }
>
>  }  // namespace base
>
>
>

[commit-bot: I haz the power, 2012-05-18 22:58:43]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/rvargas@chromium.org/10388202/1

[commit-bot: I haz the power, 2012-05-18 23:24:51]

Try job failure for 10388202-1 on win for step "update".
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win&number...

Step "update" is always a major failure.
Look at the try server FAQ for more details.

[commit-bot: I haz the power, 2012-05-19 01:39:21]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/rvargas@chromium.org/10388202/1

[commit-bot: I haz the power, 2012-05-19 02:36:52]

Try job failure for 10388202-1 (retry) on mac_rel for steps "base_unittests,
browser_tests, unit_tests, content_unittests, media_unittests".
It's a second try, previously, steps "base_unittests, browser_tests, unit_tests,
content_unittests, media_unittests" failed.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=mac_rel&nu...