bluetooth: Replace buttons in device table with action links.

bluetooth: Replace buttons in device table with action links.

When a large number of rows are present in the device table, scrolling
performance drops considerably. This is due to the table recalculating its size
due to button elements being in each row of the table.

Replaces buttons in device table with action links to speed up rendering of the
device table.

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

Committed: https://crrev.com/7af445f71d1af6b6be39a5f59c1def3ac113b3d0
Cr-Commit-Position: refs/heads/master@{#437620}

[mbrunson, 2016-12-07 01:23:13]

Description was changed from

==========
bluetooth: Replace buttons in device table with action links.

When a large number of rows are present in the device table, scrolling
performance drops considerably. This is due to the table recalculating its size
due to buttom elements being in each row of the table.

Replaces buttons in device table with action links to speed up rendering of the
device table.

BUG=651282
==========

to

==========
bluetooth: Replace buttons in device table with action links.

When a large number of rows are present in the device table, scrolling
performance drops considerably. This is due to the table recalculating its size
due to buttom elements being in each row of the table.

Replaces buttons in device table with action links to speed up rendering of the
device table.

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

[mbrunson, 2016-12-07 19:42:20]

mbrunson@chromium.org changed reviewers:
+ dbeam@chromium.org, ortuno@chromium.org, scheib@chromium.org

[mbrunson, 2016-12-07 19:42:21]

This is a very small patch.

[Dan Beam, 2016-12-07 21:57:49]

lgtm

https://codereview.chromium.org/2558493004/diff/1/chrome/browser/resources/bl...
File chrome/browser/resources/bluetooth_internals/device_table.js (right):

https://codereview.chromium.org/2558493004/diff/1/chrome/browser/resources/bl...
chrome/browser/resources/bluetooth_internals/device_table.js:124: var
inspectButton = document.createElement('a', 'action-link');
nit: Button -> Link?

[mbrunson, 2016-12-07 22:02:04]

Description was changed from

==========
bluetooth: Replace buttons in device table with action links.

When a large number of rows are present in the device table, scrolling
performance drops considerably. This is due to the table recalculating its size
due to buttom elements being in each row of the table.

Replaces buttons in device table with action links to speed up rendering of the
device table.

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

to

==========
bluetooth: Replace buttons in device table with action links.

When a large number of rows are present in the device table, scrolling
performance drops considerably. This is due to the table recalculating its size
due to button elements being in each row of the table.

Replaces buttons in device table with action links to speed up rendering of the
device table.

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

[scheib, 2016-12-07 23:21:22]

LGTM

[ortuno, 2016-12-08 00:00:45]

lgtm

[mbrunson, 2016-12-08 01:20:32]

The CQ bit was checked by mbrunson@chromium.org

[mbrunson, 2016-12-08 01:20:32]

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/2558493004/#ps20001
(title: "Rename button to link, merge upstream")

[commit-bot: I haz the power, 2016-12-08 01:21:36]

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

[commit-bot: I haz the power, 2016-12-08 01:21:36]

This CL has an open dependency (Issue 2538653002 Patch 320001). Please resolve
the dependency and try again.
If you are sure that there is no real dependency, please use one of the options
listed in https://goo.gl/9Es4OR to land the CL.

[mbrunson, 2016-12-09 18:10:31]

The CQ bit was checked by mbrunson@chromium.org

[mbrunson, 2016-12-09 18:11:00]

The CQ bit was unchecked by mbrunson@chromium.org

[commit-bot: I haz the power, 2016-12-09 18:11:02]

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

[mbrunson, 2016-12-09 18:15:35]

The CQ bit was checked by mbrunson@chromium.org

[mbrunson, 2016-12-09 18:15:35]

The patchset sent to the CQ was uploaded after l-g-t-m from scheib@chromium.org,
ortuno@chromium.org, dbeam@chromium.org
Link to the patchset: https://codereview.chromium.org/2558493004/#ps40001
(title: "Change handleInspectBtn_ to handleInspectClick_")

[commit-bot: I haz the power, 2016-12-09 18:16:22]

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-12-09 20:24:25]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1481307335103760,
"parent_rev": "fe2273aaca10d22c335f976779db35627640f6e1", "commit_rev":
"7e63d80d1d7a3f35d4d0efe520dedf65e7dccd8b"}

[commit-bot: I haz the power, 2016-12-09 20:24:58]

Description was changed from

==========
bluetooth: Replace buttons in device table with action links.

When a large number of rows are present in the device table, scrolling
performance drops considerably. This is due to the table recalculating its size
due to button elements being in each row of the table.

Replaces buttons in device table with action links to speed up rendering of the
device table.

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

to

==========
bluetooth: Replace buttons in device table with action links.

When a large number of rows are present in the device table, scrolling
performance drops considerably. This is due to the table recalculating its size
due to button elements being in each row of the table.

Replaces buttons in device table with action links to speed up rendering of the
device table.

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

Review-Url: https://codereview.chromium.org/2558493004
==========

[commit-bot: I haz the power, 2016-12-09 20:24:59]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-12-12 14:37:16]

Description was changed from

==========
bluetooth: Replace buttons in device table with action links.

When a large number of rows are present in the device table, scrolling
performance drops considerably. This is due to the table recalculating its size
due to button elements being in each row of the table.

Replaces buttons in device table with action links to speed up rendering of the
device table.

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

Review-Url: https://codereview.chromium.org/2558493004
==========

to

==========
bluetooth: Replace buttons in device table with action links.

When a large number of rows are present in the device table, scrolling
performance drops considerably. This is due to the table recalculating its size
due to button elements being in each row of the table.

Replaces buttons in device table with action links to speed up rendering of the
device table.

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

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

[commit-bot: I haz the power, 2016-12-12 14:37:17]

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