[MD settings] right align cookies filter and remove all buttons

[MD settings] right align cookies filter and remove all buttons

This CL moves the remove cookies button and the cookies site filter to
the end of the row. This was specifically approved by Alan since it
differs from the current mocks (i.e. the mocks should be updated to
reflect this CL).

This CL will also clear the site search filter after all the items
in the search are deleted.

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

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

[dschuyler, 2016-11-02 18:55:47]

Description was changed from

==========
[MD settings] right align cookies filter and remove all buttons

This CL moves the remove cookies button and the cookies site filter to
the end of the row. This was specifically approved by Alan since it
differs from the current mocks (i.e. the mocks should be updated to
reflect this CL).

BUG=660192
==========

to

==========
[MD settings] right align cookies filter and remove all buttons

This CL moves the remove cookies button and the cookies site filter to
the end of the row. This was specifically approved by Alan since it
differs from the current mocks (i.e. the mocks should be updated to
reflect this CL).

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

[dschuyler, 2016-11-02 18:55:50]

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

[commit-bot: I haz the power, 2016-11-02 18:56:34]

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

[dschuyler, 2016-11-02 18:56:44]

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

[dschuyler, 2016-11-02 19:06:00]

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

[Dan Beam, 2016-11-02 19:06:07]

can i have before and after screenshots or demo?

[commit-bot: I haz the power, 2016-11-02 19:06:09]

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

[dschuyler, 2016-11-02 19:31:15]

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

[dschuyler, 2016-11-02 19:32:31]

added margin; clearing filter after delete

[commit-bot: I haz the power, 2016-11-02 19:32:50]

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

[dschuyler, 2016-11-02 19:33:10]

Description was changed from

==========
[MD settings] right align cookies filter and remove all buttons

This CL moves the remove cookies button and the cookies site filter to
the end of the row. This was specifically approved by Alan since it
differs from the current mocks (i.e. the mocks should be updated to
reflect this CL).

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

to

==========
[MD settings] right align cookies filter and remove all buttons

This CL moves the remove cookies button and the cookies site filter to
the end of the row. This was specifically approved by Alan since it
differs from the current mocks (i.e. the mocks should be updated to
reflect this CL).

This CL will also clear the site search filter after all the items
in the search are deleted.

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

[commit-bot: I haz the power, 2016-11-02 19:47:43]

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

[commit-bot: I haz the power, 2016-11-02 19:47:43]

Dry run: Try jobs failed on following builders:
  closure_compilation on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/closure_compila...)

[Dan Beam, 2016-11-03 06:31:34]

lgtm

https://codereview.chromium.org/2471223002/diff/40001/chrome/browser/resource...
File chrome/browser/resources/settings/site_settings/site_data.js (right):

https://codereview.chromium.org/2471223002/diff/40001/chrome/browser/resource...
chrome/browser/resources/settings/site_settings/site_data.js:121:
this.$.filter.setValue('');
you need to cast this.$.filter

[dschuyler, 2016-11-03 18:23:58]

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

[dschuyler, 2016-11-03 18:24:36]

https://codereview.chromium.org/2471223002/diff/40001/chrome/browser/resource...
File chrome/browser/resources/settings/site_settings/site_data.js (right):

https://codereview.chromium.org/2471223002/diff/40001/chrome/browser/resource...
chrome/browser/resources/settings/site_settings/site_data.js:121:
this.$.filter.setValue('');
On 2016/11/03 06:31:34, Dan Beam wrote:
> you need to cast this.$.filter

Done.

[commit-bot: I haz the power, 2016-11-03 18:24:39]

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-11-03 20:36:45]

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

[commit-bot: I haz the power, 2016-11-03 20:36:45]

Dry run: This issue passed the CQ dry run.

[dschuyler, 2016-11-03 20:53:24]

The CQ bit was checked by dschuyler@chromium.org

[dschuyler, 2016-11-03 20:53:25]

The patchset sent to the CQ was uploaded after l-g-t-m from dbeam@chromium.org
Link to the patchset: https://codereview.chromium.org/2471223002/#ps60001
(title: "closure fix")

[commit-bot: I haz the power, 2016-11-03 20:54:20]

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-11-03 21:01:13]

Description was changed from

==========
[MD settings] right align cookies filter and remove all buttons

This CL moves the remove cookies button and the cookies site filter to
the end of the row. This was specifically approved by Alan since it
differs from the current mocks (i.e. the mocks should be updated to
reflect this CL).

This CL will also clear the site search filter after all the items
in the search are deleted.

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

to

==========
[MD settings] right align cookies filter and remove all buttons

This CL moves the remove cookies button and the cookies site filter to
the end of the row. This was specifically approved by Alan since it
differs from the current mocks (i.e. the mocks should be updated to
reflect this CL).

This CL will also clear the site search filter after all the items
in the search are deleted.

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

[commit-bot: I haz the power, 2016-11-03 21:01:17]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2016-11-03 21:03:38]

Description was changed from

==========
[MD settings] right align cookies filter and remove all buttons

This CL moves the remove cookies button and the cookies site filter to
the end of the row. This was specifically approved by Alan since it
differs from the current mocks (i.e. the mocks should be updated to
reflect this CL).

This CL will also clear the site search filter after all the items
in the search are deleted.

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

to

==========
[MD settings] right align cookies filter and remove all buttons

This CL moves the remove cookies button and the cookies site filter to
the end of the row. This was specifically approved by Alan since it
differs from the current mocks (i.e. the mocks should be updated to
reflect this CL).

This CL will also clear the site search filter after all the items
in the search are deleted.

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

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

[commit-bot: I haz the power, 2016-11-03 21:03:39]

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