Migrate ExtensionPopup from BubbleDelegateView to

Migrate ExtensionPopup from BubbleDelegateView to BubbleDialogDelegateView

In this particular case there's no behavior change and no huge win, except that it will allow
us to one day remove BubbleDelegateView completely (or greatly trim it down).

BUG=591583
Committed: https://crrev.com/7d6d85cc55de671a7585ecb356e2d1121d046a8b
Cr-Commit-Position: refs/heads/master@{#382042}

[Evan Stade, 2016-03-17 02:01:07]

Description was changed from

==========
Migrate ExtensionPopup from BubbleDelegateView to
BubbleDialogDelegateView

In this particular case there's no huge win, except that it will allow
us to one day remove BubbleDelegateView completely (or greatly trim it
down).

BUG=582675
==========

to

==========
Migrate ExtensionPopup from BubbleDelegateView to
BubbleDialogDelegateView

In this particular case there's no behavior change and no huge win, except that
it will allow
us to one day remove BubbleDelegateView completely (or greatly trim it
down).

BUG=582675
==========

[Evan Stade, 2016-03-17 02:01:11]

estade@chromium.org changed reviewers:
+ sky@chromium.org, tapted@chromium.org

[tapted, 2016-03-17 02:29:18]

lgtm

And thanks for updating the not-aura path! I did a spot check - seems to
look/work fine.

[Evan Stade, 2016-03-17 02:51:17]

Description was changed from

==========
Migrate ExtensionPopup from BubbleDelegateView to
BubbleDialogDelegateView

In this particular case there's no behavior change and no huge win, except that
it will allow
us to one day remove BubbleDelegateView completely (or greatly trim it
down).

BUG=582675
==========

to

==========
Migrate ExtensionPopup from BubbleDelegateView to
BubbleDialogDelegateView

In this particular case there's no behavior change and no huge win, except that
it will allow
us to one day remove BubbleDelegateView completely (or greatly trim it down).

BUG=582675
==========

[Evan Stade, 2016-03-17 02:51:33]

Description was changed from

==========
Migrate ExtensionPopup from BubbleDelegateView to
BubbleDialogDelegateView

In this particular case there's no behavior change and no huge win, except that
it will allow
us to one day remove BubbleDelegateView completely (or greatly trim it down).

BUG=582675
==========

to

==========
Migrate ExtensionPopup from BubbleDelegateView to BubbleDialogDelegateView

In this particular case there's no behavior change and no huge win, except that
it will allow
us to one day remove BubbleDelegateView completely (or greatly trim it down).

BUG=582675
==========

[sky, 2016-03-17 13:19:18]

LGTM

[Evan Stade, 2016-03-17 21:00:13]

The CQ bit was checked by estade@chromium.org

[commit-bot: I haz the power, 2016-03-17 21:00:44]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1808083002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1808083002/1

[commit-bot: I haz the power, 2016-03-17 21:28:06]

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

[commit-bot: I haz the power, 2016-03-17 21:28:08]

Try jobs failed on following builders:
  chromeos_amd64-generic_chromium_compile_only_ng on tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_amd64-...)

[Evan Stade, 2016-03-18 18:24:54]

Description was changed from

==========
Migrate ExtensionPopup from BubbleDelegateView to BubbleDialogDelegateView

In this particular case there's no behavior change and no huge win, except that
it will allow
us to one day remove BubbleDelegateView completely (or greatly trim it down).

BUG=582675
==========

to

==========
Migrate ExtensionPopup from BubbleDelegateView to BubbleDialogDelegateView

In this particular case there's no behavior change and no huge win, except that
it will allow
us to one day remove BubbleDelegateView completely (or greatly trim it down).

BUG=591583
==========

[Evan Stade, 2016-03-18 18:24:58]

The CQ bit was checked by estade@chromium.org

[Evan Stade, 2016-03-18 18:24:58]

The patchset sent to the CQ was uploaded after l-g-t-m from tapted@chromium.org,
sky@chromium.org
Link to the patchset: https://codereview.chromium.org/1808083002/#ps20001
(title: "IWYU")

[commit-bot: I haz the power, 2016-03-18 18:25:35]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1808083002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1808083002/20001

[commit-bot: I haz the power, 2016-03-18 19:17:00]

Description was changed from

==========
Migrate ExtensionPopup from BubbleDelegateView to BubbleDialogDelegateView

In this particular case there's no behavior change and no huge win, except that
it will allow
us to one day remove BubbleDelegateView completely (or greatly trim it down).

BUG=591583
==========

to

==========
Migrate ExtensionPopup from BubbleDelegateView to BubbleDialogDelegateView

In this particular case there's no behavior change and no huge win, except that
it will allow
us to one day remove BubbleDelegateView completely (or greatly trim it down).

BUG=591583
==========

[commit-bot: I haz the power, 2016-03-18 19:17:01]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-03-18 19:18:30]

Description was changed from

==========
Migrate ExtensionPopup from BubbleDelegateView to BubbleDialogDelegateView

In this particular case there's no behavior change and no huge win, except that
it will allow
us to one day remove BubbleDelegateView completely (or greatly trim it down).

BUG=591583
==========

to

==========
Migrate ExtensionPopup from BubbleDelegateView to BubbleDialogDelegateView

In this particular case there's no behavior change and no huge win, except that
it will allow
us to one day remove BubbleDelegateView completely (or greatly trim it down).

BUG=591583

Committed: https://crrev.com/7d6d85cc55de671a7585ecb356e2d1121d046a8b
Cr-Commit-Position: refs/heads/master@{#382042}
==========

[commit-bot: I haz the power, 2016-03-18 19:18:32]

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