Cherry-picking hangouts.google.com whitelisting change into M51

Cherry-picking hangouts.google.com whitelisting change into M51.

Urgent: whitelist hangouts.google.com further (should've been done in issue 1552383002 from Dec 2015, but...) to allow video effects plugin access; tag on meet.google.com for a similar purpose; tax: refactor whitelist checks into separate thing.

R=sshruthi@chromium.org,govind@chromium.org,sky@chromium.org,tsepez@chromium.org
BUG=614062
NOTRY=true
NOPRESUBMIT=true

Review-Url: https://codereview.chromium.org/1974413003
Cr-Commit-Position: refs/heads/master@{#395172}
(cherry picked from commit 81dcbd6123178d58c5edc53d1453e03f9afa31ca)

[qaz-chromium, 2016-05-23 18:23:35]

Description was changed from

==========
Cherry-picking https://crrev.com/81dcbd6123178d58c5edc53d1453e03f9afa31ca

Urgent: whitelist hangouts.google.com further (should've been done in issue
1552383002 from Dec 2015, but...) to allow video effects plugin access; tag on
meet.google.com for a similar purpose; tax: refactor whitelist checks into
separate thing.

R=sshruthi@chromium.org
BUG=614062

Review-Url: https://codereview.chromium.org/1974413003
Cr-Commit-Position: refs/heads/master@{#395172}
(cherry picked from commit 81dcbd6123178d58c5edc53d1453e03f9afa31ca)
==========

to

==========
Cherry-picking https://crrev.com/81dcbd6123178d58c5edc53d1453e03f9afa31ca

Urgent: whitelist hangouts.google.com further (should've been done in issue
1552383002 from Dec 2015, but...) to allow video effects plugin access; tag on
meet.google.com for a similar purpose; tax: refactor whitelist checks into
separate thing.

R=sshruthi@chromium.org
BUG=614062

Review-Url: https://codereview.chromium.org/1974413003
Cr-Commit-Position: refs/heads/master@{#395172}
(cherry picked from commit 81dcbd6123178d58c5edc53d1453e03f9afa31ca)
==========

[qaz-chromium, 2016-05-23 18:23:35]

qaz@chromium.org changed reviewers:
+ govind@chromium.org

[qaz-chromium, 2016-05-23 18:25:11]

Since I don't have commit permission, git-drover failed due to access error.

Would I be able to submit the cherry-pick from here? What do I need to do
afterwards (that's usually covered by git-drover)?

[qaz-chromium, 2016-05-23 18:52:01]

Description was changed from

==========
Cherry-picking https://crrev.com/81dcbd6123178d58c5edc53d1453e03f9afa31ca

Urgent: whitelist hangouts.google.com further (should've been done in issue
1552383002 from Dec 2015, but...) to allow video effects plugin access; tag on
meet.google.com for a similar purpose; tax: refactor whitelist checks into
separate thing.

R=sshruthi@chromium.org
BUG=614062

Review-Url: https://codereview.chromium.org/1974413003
Cr-Commit-Position: refs/heads/master@{#395172}
(cherry picked from commit 81dcbd6123178d58c5edc53d1453e03f9afa31ca)
==========

to

==========
Cherry-picking https://crrev.com/81dcbd6123178d58c5edc53d1453e03f9afa31ca

Urgent: whitelist hangouts.google.com further (should've been done in issue
1552383002 from Dec 2015, but...) to allow video effects plugin access; tag on
meet.google.com for a similar purpose; tax: refactor whitelist checks into
separate thing.

sshruthi@chromium.org,govind@chromium.org
BUG=614062

Review-Url: https://codereview.chromium.org/1974413003
Cr-Commit-Position: refs/heads/master@{#395172}
(cherry picked from commit 81dcbd6123178d58c5edc53d1453e03f9afa31ca)
==========

[chromium-reviews, 2016-05-23 18:53:03]

Shruthi, could you please answer this?

On Mon, May 23, 2016 at 11:25 AM, <qaz@chromium.org> wrote:

> Since I don't have commit permission, git-drover failed due to access
> error.
>
> Would I be able to submit the cherry-pick from here? What do I need to do
> afterwards (that's usually covered by git-drover)?
>
> https://codereview.chromium.org/2001233002/
>

-- 
You received this message because you are subscribed to the Google Groups
"Chromium-reviews" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to chromium-reviews+unsubscribe@chromium.org.

[qaz-chromium, 2016-05-23 20:17:40]

Description was changed from

==========
Cherry-picking https://crrev.com/81dcbd6123178d58c5edc53d1453e03f9afa31ca

Urgent: whitelist hangouts.google.com further (should've been done in issue
1552383002 from Dec 2015, but...) to allow video effects plugin access; tag on
meet.google.com for a similar purpose; tax: refactor whitelist checks into
separate thing.

sshruthi@chromium.org,govind@chromium.org
BUG=614062

Review-Url: https://codereview.chromium.org/1974413003
Cr-Commit-Position: refs/heads/master@{#395172}
(cherry picked from commit 81dcbd6123178d58c5edc53d1453e03f9afa31ca)
==========

to

==========
Cherry-picking https://crrev.com/81dcbd6123178d58c5edc53d1453e03f9afa31ca

Urgent: whitelist hangouts.google.com further (should've been done in issue
1552383002 from Dec 2015, but...) to allow video effects plugin access; tag on
meet.google.com for a similar purpose; tax: refactor whitelist checks into
separate thing.

sshruthi@chromium.org,govind@chromium.org,sky@chromium.org
BUG=614062

Review-Url: https://codereview.chromium.org/1974413003
Cr-Commit-Position: refs/heads/master@{#395172}
(cherry picked from commit 81dcbd6123178d58c5edc53d1453e03f9afa31ca)
==========

[qaz-chromium, 2016-05-23 20:17:40]

qaz@chromium.org changed reviewers:
+ sky@chromium.org

[qaz-chromium, 2016-05-23 20:19:44]

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

[commit-bot: I haz the power, 2016-05-23 20:20:02]

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

[commit-bot: I haz the power, 2016-05-23 20:20:04]

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

[commit-bot: I haz the power, 2016-05-23 20:20:05]

Dry run: No L-G-T-M from a valid reviewer yet. 
CQ run can only be started by full committers or once the patch has
received an L-G-T-M from a full committer.
Even if an L-G-T-M may have been provided, it was from a non-committer,
_not_ a full super star committer.
See http://www.chromium.org/getting-involved/become-a-committer
Note that this has nothing to do with OWNERS files.

[qaz-chromium, 2016-05-23 20:24:58]

Description was changed from

==========
Cherry-picking https://crrev.com/81dcbd6123178d58c5edc53d1453e03f9afa31ca

Urgent: whitelist hangouts.google.com further (should've been done in issue
1552383002 from Dec 2015, but...) to allow video effects plugin access; tag on
meet.google.com for a similar purpose; tax: refactor whitelist checks into
separate thing.

sshruthi@chromium.org,govind@chromium.org,sky@chromium.org
BUG=614062

Review-Url: https://codereview.chromium.org/1974413003
Cr-Commit-Position: refs/heads/master@{#395172}
(cherry picked from commit 81dcbd6123178d58c5edc53d1453e03f9afa31ca)
==========

to

==========
Cherry-picking hangouts.google.com whitelisting change into M51.

Urgent: whitelist hangouts.google.com further (should've been done in issue
1552383002 from Dec 2015, but...) to allow video effects plugin access; tag on
meet.google.com for a similar purpose; tax: refactor whitelist checks into
separate thing.

sshruthi@chromium.org,govind@chromium.org,sky@chromium.org,tsepez@chromium.org
BUG=614062

Review-Url: https://codereview.chromium.org/1974413003
Cr-Commit-Position: refs/heads/master@{#395172}
(cherry picked from commit 81dcbd6123178d58c5edc53d1453e03f9afa31ca)
==========

[qaz-chromium, 2016-05-23 20:24:58]

qaz@chromium.org changed reviewers:
+ tsepez@chromium.org
- qaz@chromium.org

[qaz-chromium, 2016-05-23 20:26:42]

sky@ and tsepez@, could you approve this cherry-pick? I'd like to get this in by
EoD today to make the M51 stable release cut.

Thanks.

[Tom Sepez, 2016-05-23 20:27:57]

lgtm

[qaz-chromium, 2016-05-23 20:29:29]

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

[commit-bot: I haz the power, 2016-05-23 20:29:45]

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

[commit-bot: I haz the power, 2016-05-23 20:29:46]

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

[commit-bot: I haz the power, 2016-05-23 20:29:47]

Dry run: CLs for remote refs other than refs/pending/heads/master must contain
NOTRY=true and NOPRESUBMIT=true in order for the CQ to process them

[sky, 2016-05-23 20:30:23]

LGTM

[qaz-chromium, 2016-05-23 20:30:42]

The CQ bit was checked by qaz@chromium.org

[commit-bot: I haz the power, 2016-05-23 20:31:25]

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

[commit-bot: I haz the power, 2016-05-23 20:31:26]

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

[commit-bot: I haz the power, 2016-05-23 20:31:27]

CLs for remote refs other than refs/pending/heads/master must contain NOTRY=true
and NOPRESUBMIT=true in order for the CQ to process them

[qaz-chromium, 2016-05-23 20:34:10]

Description was changed from

==========
Cherry-picking hangouts.google.com whitelisting change into M51.

Urgent: whitelist hangouts.google.com further (should've been done in issue
1552383002 from Dec 2015, but...) to allow video effects plugin access; tag on
meet.google.com for a similar purpose; tax: refactor whitelist checks into
separate thing.

sshruthi@chromium.org,govind@chromium.org,sky@chromium.org,tsepez@chromium.org
BUG=614062

Review-Url: https://codereview.chromium.org/1974413003
Cr-Commit-Position: refs/heads/master@{#395172}
(cherry picked from commit 81dcbd6123178d58c5edc53d1453e03f9afa31ca)
==========

to

==========
Cherry-picking hangouts.google.com whitelisting change into M51.

Urgent: whitelist hangouts.google.com further (should've been done in issue
1552383002 from Dec 2015, but...) to allow video effects plugin access; tag on
meet.google.com for a similar purpose; tax: refactor whitelist checks into
separate thing.

R=sshruthi@chromium.org,govind@chromium.org,sky@chromium.org,tsepez@chromium.org
BUG=614062
NOTRY=true
NOPRESUBMIT=true

Review-Url: https://codereview.chromium.org/1974413003
Cr-Commit-Position: refs/heads/master@{#395172}
(cherry picked from commit 81dcbd6123178d58c5edc53d1453e03f9afa31ca)
==========

[qaz-chromium, 2016-05-23 20:34:10]

qaz@chromium.org changed reviewers:
- qaz@chromium.org

[qaz-chromium, 2016-05-23 20:34:34]

The CQ bit was checked by qaz@chromium.org

[commit-bot: I haz the power, 2016-05-23 20:35:08]

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

[commit-bot: I haz the power, 2016-05-23 20:37:45]

Description was changed from

==========
Cherry-picking hangouts.google.com whitelisting change into M51.

Urgent: whitelist hangouts.google.com further (should've been done in issue
1552383002 from Dec 2015, but...) to allow video effects plugin access; tag on
meet.google.com for a similar purpose; tax: refactor whitelist checks into
separate thing.

R=sshruthi@chromium.org,govind@chromium.org,sky@chromium.org,tsepez@chromium.org
BUG=614062
NOTRY=true
NOPRESUBMIT=true

Review-Url: https://codereview.chromium.org/1974413003
Cr-Commit-Position: refs/heads/master@{#395172}
(cherry picked from commit 81dcbd6123178d58c5edc53d1453e03f9afa31ca)
==========

to

==========
Cherry-picking hangouts.google.com whitelisting change into M51.

Urgent: whitelist hangouts.google.com further (should've been done in issue
1552383002 from Dec 2015, but...) to allow video effects plugin access; tag on
meet.google.com for a similar purpose; tax: refactor whitelist checks into
separate thing.

R=sshruthi@chromium.org,govind@chromium.org,sky@chromium.org,tsepez@chromium.org
BUG=614062
NOTRY=true
NOPRESUBMIT=true

Review-Url: https://codereview.chromium.org/1974413003
Cr-Commit-Position: refs/heads/master@{#395172}
(cherry picked from commit 81dcbd6123178d58c5edc53d1453e03f9afa31ca)
==========

[commit-bot: I haz the power, 2016-05-23 20:37:47]

Committed patchset #1 (id:1)