Power Save Blocker: Eliminate passing ViewAndroid as WeakPtr

Power Save Blocker: Eliminate passing ViewAndroid as WeakPtr

As of https://codereview.chromium.org/2103243002/, PowerSaveBlockerAndroid
accesses the ViewAndroid pointer it is passed only at a time when that
ViewAndroid pointer is guaranteed to be valid.  This CL moves
PowerSaveBlockerAndroid away from consuming the ViewAndroid pointer as a
WeakPtr. The concrete motivation is to eliminate WakeLockServiceContext needing
to listen for WebContentsObserver::WebContentsDestroyed() as part of decoupling
the wake lock service from //content.

BUG=612346
Committed: https://crrev.com/87cd0848a0d1453e7553a72b0686d42fabf8ff3a
Cr-Commit-Position: refs/heads/master@{#421169}

[blundell, 2016-09-26 11:04:49]

The CQ bit was checked by blundell@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-09-26 11:05:01]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-26 12:18:08]

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

[commit-bot: I haz the power, 2016-09-26 12:18:09]

Dry run: Try jobs failed on following builders:
  linux_android_rel_ng on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)

[boliu, 2016-09-26 17:46:00]

boliu@chromium.org changed reviewers:
+ boliu@chromium.org

[boliu, 2016-09-26 17:46:00]

lgtm

[boliu, 2016-09-26 17:51:18]

On 2016/09/26 17:46:00, boliu wrote:
> lgtm

oh, I didn't look closely, but probably could remove a bunch of weak_ptr
includes

[blundell, 2016-09-27 10:38:55]

Description was changed from

==========
Power Save Blocker: Eliminate passing ViewAndroid as WeakPtr

BUG=612346
==========

to

==========
Power Save Blocker: Eliminate passing ViewAndroid as WeakPtr

As of https://codereview.chromium.org/2103243002/, PowerSaveBlockerAndroid
accesses the ViewAndroid pointer it is passed only at a time when that
ViewAndroid pointer is guaranteed to be valid.  This CL moves
PowerSaveBlockerAndroid away from consuming the ViewAndroid pointer as a
WeakPtr. The concrete motivation is to eliminate WakeLockServiceContext needing
to listen for WebContentsObserver::WebContentsDestroyed() as part of decoupling
the wake lock service from //content.

BUG=612346
==========

[blundell, 2016-09-27 10:45:57]

Thanks! Removed weak_ptr.h from power_save_blocker.h.

[blundell, 2016-09-27 10:46:06]

The CQ bit was checked by blundell@chromium.org

[blundell, 2016-09-27 10:46:06]

The patchset sent to the CQ was uploaded after l-g-t-m from boliu@chromium.org
Link to the patchset: https://codereview.chromium.org/2371723002/#ps40001
(title: "Rebase")

[commit-bot: I haz the power, 2016-09-27 10:46:23]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-27 11:30:50]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-09-27 11:32:31]

Description was changed from

==========
Power Save Blocker: Eliminate passing ViewAndroid as WeakPtr

As of https://codereview.chromium.org/2103243002/, PowerSaveBlockerAndroid
accesses the ViewAndroid pointer it is passed only at a time when that
ViewAndroid pointer is guaranteed to be valid.  This CL moves
PowerSaveBlockerAndroid away from consuming the ViewAndroid pointer as a
WeakPtr. The concrete motivation is to eliminate WakeLockServiceContext needing
to listen for WebContentsObserver::WebContentsDestroyed() as part of decoupling
the wake lock service from //content.

BUG=612346
==========

to

==========
Power Save Blocker: Eliminate passing ViewAndroid as WeakPtr

As of https://codereview.chromium.org/2103243002/, PowerSaveBlockerAndroid
accesses the ViewAndroid pointer it is passed only at a time when that
ViewAndroid pointer is guaranteed to be valid.  This CL moves
PowerSaveBlockerAndroid away from consuming the ViewAndroid pointer as a
WeakPtr. The concrete motivation is to eliminate WakeLockServiceContext needing
to listen for WebContentsObserver::WebContentsDestroyed() as part of decoupling
the wake lock service from //content.

BUG=612346

Committed: https://crrev.com/87cd0848a0d1453e7553a72b0686d42fabf8ff3a
Cr-Commit-Position: refs/heads/master@{#421169}
==========

[commit-bot: I haz the power, 2016-09-27 11:32:32]

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