[MD settings] remove condition on site list actionable state

[MD settings] remove condition on site list actionable state

This CL removes a condition on whether a site settings row is 'actionable'.

BUG=650557
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Committed: https://crrev.com/a241e7aac6b340112403a9a9f4d2919a41972839
Cr-Commit-Position: refs/heads/master@{#421197}

[dschuyler, 2016-09-23 01:23:28]

Description was changed from

==========
[MD settings] remove condition on site list actionable state

This CL removes a condition on whether a site settings row is 'actionable'.

BUG=626615
==========

to

==========
[MD settings] remove condition on site list actionable state

This CL removes a condition on whether a site settings row is 'actionable'.

BUG=626615
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

[dschuyler, 2016-09-23 01:23:30]

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

[commit-bot: I haz the power, 2016-09-23 01:23:53]

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

[dschuyler, 2016-09-23 01:25:07]

dschuyler@chromium.org changed reviewers:
+ finnur@chromium.org

[dschuyler, 2016-09-23 01:25:07]

This CL is a bit of a touch-up for CL 2354313002 which changes the condition,
but I think it's better removing the condition altogether. WDYT

[commit-bot: I haz the power, 2016-09-23 02:22:49]

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

[commit-bot: I haz the power, 2016-09-23 02:22:50]

Dry run: This issue passed the CQ dry run.

[Finnur, 2016-09-26 13:13:26]

Yes, I agree. This is better. LGTM.

[dschuyler, 2016-09-26 18:19:04]

The CQ bit was checked by dschuyler@chromium.org

[commit-bot: I haz the power, 2016-09-26 18:19:41]

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-09-26 21:13:11]

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

[commit-bot: I haz the power, 2016-09-26 21:13:12]

Try jobs failed on following builders:
  linux_android_rel_ng on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)

[Finnur, 2016-09-27 13:25:36]

Description was changed from

==========
[MD settings] remove condition on site list actionable state

This CL removes a condition on whether a site settings row is 'actionable'.

BUG=626615
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

to

==========
[MD settings] remove condition on site list actionable state

This CL removes a condition on whether a site settings row is 'actionable'.

BUG=650557
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

[Finnur, 2016-09-27 13:27:03]

Updated bug link from 626615 to 650557 since QA filed it as a new bug. 

Also looks like you (Dave) want to check this in but it failed due to env CQ
issue, so I'm retrying on your behalf (to save time).

[Finnur, 2016-09-27 13:27:13]

The CQ bit was checked by finnur@chromium.org

[commit-bot: I haz the power, 2016-09-27 13:27:31]

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-09-27 13:58:25]

Description was changed from

==========
[MD settings] remove condition on site list actionable state

This CL removes a condition on whether a site settings row is 'actionable'.

BUG=650557
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

to

==========
[MD settings] remove condition on site list actionable state

This CL removes a condition on whether a site settings row is 'actionable'.

BUG=650557
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

[commit-bot: I haz the power, 2016-09-27 13:58:26]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-09-27 14:00:33]

Description was changed from

==========
[MD settings] remove condition on site list actionable state

This CL removes a condition on whether a site settings row is 'actionable'.

BUG=650557
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

to

==========
[MD settings] remove condition on site list actionable state

This CL removes a condition on whether a site settings row is 'actionable'.

BUG=650557
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

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

[commit-bot: I haz the power, 2016-09-27 14:00:34]

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