IDBObserver does not need to be GC finalizable.

IDBObserver does not need to be GC finalizable.

R=haraken
BUG=

Committed: https://crrev.com/50f8cdded77e9d6250181600724bb52d29c86664
Cr-Commit-Position: refs/heads/master@{#431268}

[sof, 2016-11-10 07:46:15]

The CQ bit was checked by sigbjornf@opera.com to run a CQ dry run

[commit-bot: I haz the power, 2016-11-10 07:46:46]

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-11-10 07:53:05]

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

[commit-bot: I haz the power, 2016-11-10 07:53:06]

Dry run: Try jobs failed on following builders:
  chromium_presubmit on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[sof, 2016-11-10 09:19:36]

Description was changed from

==========
IDBObserver does not need to be GC finalizable.

BUG=
==========

to

==========
IDBObserver does not need to be GC finalizable.

R=
BUG=
==========

[sof, 2016-11-10 09:20:28]

sigbjornf@opera.com changed reviewers:
+ haraken@chromium.org, jsbell@chromium.org

[sof, 2016-11-10 09:20:29]

please take a look. :)

[haraken, 2016-11-10 09:21:29]

On 2016/11/10 09:20:29, sof wrote:
> please take a look. :)

Extremely happy to see you here!!

LGTM

[sof, 2016-11-10 09:23:19]

On 2016/11/10 09:21:29, haraken wrote:
> On 2016/11/10 09:20:29, sof wrote:
> > please take a look. :)
> 
> Extremely happy to see you here!!
> 
> LGTM

thanks, very good to "see" you again also :)

[sof, 2016-11-10 11:43:40]

Description was changed from

==========
IDBObserver does not need to be GC finalizable.

R=
BUG=
==========

to

==========
IDBObserver does not need to be GC finalizable.

R=haraken
BUG=
==========

[sof, 2016-11-10 11:43:46]

The CQ bit was checked by sigbjornf@opera.com

[commit-bot: I haz the power, 2016-11-10 11:43:56]

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-11-10 11:50:05]

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

[commit-bot: I haz the power, 2016-11-10 11:50:06]

Try jobs failed on following builders:
  chromium_presubmit on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[sof, 2016-11-10 14:44:17]

The CQ bit was checked by sigbjornf@opera.com

[sof, 2016-11-10 14:44:17]

The patchset sent to the CQ was uploaded after l-g-t-m from haraken@chromium.org
Link to the patchset: https://codereview.chromium.org/2493713002/#ps20001
(title: "formatting only")

[commit-bot: I haz the power, 2016-11-10 14:44:30]

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-11-10 16:16:09]

Description was changed from

==========
IDBObserver does not need to be GC finalizable.

R=haraken
BUG=
==========

to

==========
IDBObserver does not need to be GC finalizable.

R=haraken
BUG=
==========

[commit-bot: I haz the power, 2016-11-10 16:16:11]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-11-10 16:18:58]

Description was changed from

==========
IDBObserver does not need to be GC finalizable.

R=haraken
BUG=
==========

to

==========
IDBObserver does not need to be GC finalizable.

R=haraken
BUG=

Committed: https://crrev.com/50f8cdded77e9d6250181600724bb52d29c86664
Cr-Commit-Position: refs/heads/master@{#431268}
==========

[commit-bot: I haz the power, 2016-11-10 16:18:59]

Patchset 2 (id:??) landed as
https://crrev.com/50f8cdded77e9d6250181600724bb52d29c86664
Cr-Commit-Position: refs/heads/master@{#431268}