Harmony - Update anchors for the rest of the location bar icons that

Harmony - Update anchors for the rest of the location bar icons that
show bubbles.

BUG=635172
Committed: https://crrev.com/a5b678c030986c136a5696b7287d399d85032dab
Cr-Commit-Position: refs/heads/master@{#424615}

[Evan Stade, 2016-10-06 21:01:03]

estade@chromium.org changed reviewers:
+ msw@chromium.org

[Evan Stade, 2016-10-06 21:01:19]

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

[commit-bot: I haz the power, 2016-10-06 21:02:08]

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-10-06 22:29:45]

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

[commit-bot: I haz the power, 2016-10-06 22:29:45]

Dry run: This issue passed the CQ dry run.

[msw, 2016-10-11 17:04:54]

lgtm with a q; sorry this fell off my plate.

https://codereview.chromium.org/2394143004/diff/1/chrome/browser/ui/views/loc...
File chrome/browser/ui/views/location_bar/zoom_bubble_view.cc (right):

https://codereview.chromium.org/2394143004/diff/1/chrome/browser/ui/views/loc...
chrome/browser/ui/views/location_bar/zoom_bubble_view.cc:50: views::View*
anchor_view =
Should this still anchor to the zoom_view if not
ui::MaterialDesignController::IsSecondaryUiMaterial()?

[Evan Stade, 2016-10-11 23:46:14]

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

[commit-bot: I haz the power, 2016-10-11 23:47:05]

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

[Evan Stade, 2016-10-11 23:49:50]

https://codereview.chromium.org/2394143004/diff/1/chrome/browser/ui/views/loc...
File chrome/browser/ui/views/location_bar/zoom_bubble_view.cc (right):

https://codereview.chromium.org/2394143004/diff/1/chrome/browser/ui/views/loc...
chrome/browser/ui/views/location_bar/zoom_bubble_view.cc:50: views::View*
anchor_view =
On 2016/10/11 17:04:54, msw wrote:
> Should this still anchor to the zoom_view if not
> ui::MaterialDesignController::IsSecondaryUiMaterial()?

yes, thanks for catching.

[Evan Stade, 2016-10-11 23:49:52]

The CQ bit was checked by estade@chromium.org

[Evan Stade, 2016-10-11 23:49:53]

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

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

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-10-12 00:41:21]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-10-12 00:45:38]

Description was changed from

==========
Harmony - Update anchors for the rest of the location bar icons that
show bubbles.

BUG=635172
==========

to

==========
Harmony - Update anchors for the rest of the location bar icons that
show bubbles.

BUG=635172

Committed: https://crrev.com/a5b678c030986c136a5696b7287d399d85032dab
Cr-Commit-Position: refs/heads/master@{#424615}
==========

[commit-bot: I haz the power, 2016-10-12 00:45:39]

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