Switch MetroPinTabHelper to use WebContentsUserData.

Switch MetroPinTabHelper to use WebContentsUserData.

BUG=107201
TEST=no visible change

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=155420

[Avi (use Gerrit), 2012-08-29 18:36:23]

benwells@ - review
ben@, jam@ - FYI

[Ben Goodger (Google), 2012-08-29 19:41:07]

sweet!

[benwells, 2012-08-30 01:37:23]

lgtm!

[commit-bot: I haz the power, 2012-08-30 03:53:47]

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

[commit-bot: I haz the power, 2012-08-30 03:53:50]

Presubmit check for 10891033-1 failed and returned exit status 1.


Running presubmit commit checks ...

** Presubmit ERRORS **
Missing LGTM from an OWNER for files in these directories:
    chrome/browser/ui

[Avi (use Gerrit), 2012-08-30 03:53:58]

On 2012/08/30 01:37:23, benwells wrote:
> lgtm!

Congrats, you are the first TabHelper to move over!

[Avi (use Gerrit), 2012-08-30 03:55:13]

On 2012/08/29 19:41:07, Ben Goodger (Google) wrote:
> sweet!

Ben, can you OK this as an OWNER?

[Ben Goodger (Google), 2012-08-30 18:07:22]

yes lgtm

[commit-bot: I haz the power, 2012-08-30 18:11:34]

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

[commit-bot: I haz the power, 2012-08-30 21:01:18]

Try job failure for 10891033-1 (previous was lost) (previous was lost) (retry)
on linux_rel for steps "nacl_integration, interactive_ui_tests, browser_tests,
sync_integration_tests".
It's a second try, previously, steps "nacl_integration, interactive_ui_tests,
browser_tests, sync_integration_tests" failed.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=linux_rel&...

[commit-bot: I haz the power, 2012-09-05 14:16:43]

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

[commit-bot: I haz the power, 2012-09-05 15:28:15]

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

[commit-bot: I haz the power, 2012-09-05 15:42:21]

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

[commit-bot: I haz the power, 2012-09-05 17:01:34]

The commit queue went berserk retrying too often for a
seemingly flaky test. Builder is mac_rel, revision is 154973, job name
was 10891033-1 (retry) (retry) (retry) (retry).

[commit-bot: I haz the power, 2012-09-07 00:55:24]

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

[commit-bot: I haz the power, 2012-09-07 02:19:03]

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

[commit-bot: I haz the power, 2012-09-07 12:51:42]

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

[commit-bot: I haz the power, 2012-09-07 13:53:38]

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

[commit-bot: I haz the power, 2012-09-07 14:31:42]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/avi@chromium.org/10891033/14008

[commit-bot: I haz the power, 2012-09-07 17:07:01]

Try job failure for 10891033-14008 (retry) (retry) on mac_rel for step
"sync_integration_tests".
It's a second try, previously, step "sync_integration_tests" failed.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=mac_rel&nu...