Revert of Use a new Supplement constructor in Navigator supplements (part 2)

Revert of Use a new Supplement constructor in Navigator supplements (part 2) (patchset #3 id:40001 of https://codereview.chromium.org/2649073002/ )

Reason for revert:
Manual bisection shows that the following regressions occur starting at 560d55d7d:

Regressions: Unexpected leaks (6)
  fast/dom/Window/property-access-on-cached-properties-after-frame-navigated.html [ Leak ]
  fast/dom/Window/property-access-on-cached-properties-after-frame-removed-and-gced.html [ Leak ]
  fast/dom/Window/property-access-on-cached-properties-after-frame-removed.html [ Leak ]
  fast/dom/Window/property-access-on-cached-window-after-frame-navigated.html [ Leak ]
  fast/dom/Window/property-access-on-cached-window-after-frame-removed-and-gced.html [ Leak ]
  fast/dom/Window/property-access-on-cached-window-after-frame-removed.html [ Leak ]

The leaked object is {"numberOfLiveSuspendableObjects":[2,3]}

Original issue's description:
> Use a new Supplement constructor in Navigator supplements (part 2)
> 
> This is a step to deprecate the default constructor of Supplement<>.
> 
> BUG=610176
> 
> Review-Url: https://codereview.chromium.org/2649073002
> Cr-Commit-Position: refs/heads/master@{#445357}
> Committed: https://chromium.googlesource.com/chromium/src/+/560d55d7d2122116b456e157f761eaa252246e4f

TBR=sigbjornf@opera.com,haraken@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=610176
Review-Url: https://codereview.chromium.org/2655463002
Cr-Commit-Position: refs/heads/master@{#445415}
Committed: https://chromium.googlesource.com/chromium/src/+/cf7b58737b355ece16d0511aa43c95d6d11d0445

[waffles, 2017-01-23 18:31:28]

The CQ bit was checked by waffles@chromium.org

[waffles, 2017-01-23 18:31:28]

Created Revert of Use a new Supplement constructor in Navigator supplements
(part 2)

[commit-bot: I haz the power, 2017-01-23 18:32:54]

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, 2017-01-23 18:33:51]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1485196288340580, "parent_rev":
"8faf57e342f36ad4edb0a99b260f81889a2b6f77", "commit_rev":
"cf7b58737b355ece16d0511aa43c95d6d11d0445"}

[commit-bot: I haz the power, 2017-01-23 18:34:20]

Description was changed from

==========
Revert of Use a new Supplement constructor in Navigator supplements (part 2)
(patchset #3 id:40001 of https://codereview.chromium.org/2649073002/ )

Reason for revert:
Manual bisection shows that the following regressions occur starting at
560d55d7d:

Regressions: Unexpected leaks (6)
 
fast/dom/Window/property-access-on-cached-properties-after-frame-navigated.html
[ Leak ]
 
fast/dom/Window/property-access-on-cached-properties-after-frame-removed-and-gced.html
[ Leak ]
  fast/dom/Window/property-access-on-cached-properties-after-frame-removed.html
[ Leak ]
  fast/dom/Window/property-access-on-cached-window-after-frame-navigated.html [
Leak ]
 
fast/dom/Window/property-access-on-cached-window-after-frame-removed-and-gced.html
[ Leak ]
  fast/dom/Window/property-access-on-cached-window-after-frame-removed.html [
Leak ]

The leaked object is {"numberOfLiveSuspendableObjects":[2,3]}

Original issue's description:
> Use a new Supplement constructor in Navigator supplements (part 2)
> 
> This is a step to deprecate the default constructor of Supplement<>.
> 
> BUG=610176
> 
> Review-Url: https://codereview.chromium.org/2649073002
> Cr-Commit-Position: refs/heads/master@{#445357}
> Committed:
https://chromium.googlesource.com/chromium/src/+/560d55d7d2122116b456e157f761...

TBR=sigbjornf@opera.com,haraken@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=610176
==========

to

==========
Revert of Use a new Supplement constructor in Navigator supplements (part 2)
(patchset #3 id:40001 of https://codereview.chromium.org/2649073002/ )

Reason for revert:
Manual bisection shows that the following regressions occur starting at
560d55d7d:

Regressions: Unexpected leaks (6)
 
fast/dom/Window/property-access-on-cached-properties-after-frame-navigated.html
[ Leak ]
 
fast/dom/Window/property-access-on-cached-properties-after-frame-removed-and-gced.html
[ Leak ]
  fast/dom/Window/property-access-on-cached-properties-after-frame-removed.html
[ Leak ]
  fast/dom/Window/property-access-on-cached-window-after-frame-navigated.html [
Leak ]
 
fast/dom/Window/property-access-on-cached-window-after-frame-removed-and-gced.html
[ Leak ]
  fast/dom/Window/property-access-on-cached-window-after-frame-removed.html [
Leak ]

The leaked object is {"numberOfLiveSuspendableObjects":[2,3]}

Original issue's description:
> Use a new Supplement constructor in Navigator supplements (part 2)
> 
> This is a step to deprecate the default constructor of Supplement<>.
> 
> BUG=610176
> 
> Review-Url: https://codereview.chromium.org/2649073002
> Cr-Commit-Position: refs/heads/master@{#445357}
> Committed:
https://chromium.googlesource.com/chromium/src/+/560d55d7d2122116b456e157f761...

TBR=sigbjornf@opera.com,haraken@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=610176

Review-Url: https://codereview.chromium.org/2655463002
Cr-Commit-Position: refs/heads/master@{#445415}
Committed:
https://chromium.googlesource.com/chromium/src/+/cf7b58737b355ece16d0511aa43c...
==========

[commit-bot: I haz the power, 2017-01-23 18:34:21]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/cf7b58737b355ece16d0511aa43c...

[haraken, 2017-01-23 23:47:24]

Thanks, LGTM