Catch exception when creating device log entries

Catch exception when creating device log entries

Committed: https://crrev.com/2fe78b441548411fa343e5215fa54cf63212d739
Cr-Commit-Position: refs/heads/master@{#372060}

[fbeaufortchromium, 2016-01-27 11:56:59]

fbeaufort@chromium.org changed reviewers:
+ fbeaufort@chromium.org, stevenjb@chromium.org

[fbeaufortchromium, 2016-01-27 11:57:00]

If there are no log entries, this fails to parse JSON object as it returns "No
log entries" string.

This CL simply adds a try/catch to avoid JS error popping up in DevTools
Console.

[stevenjb, 2016-01-27 18:13:08]

lgtm

https://codereview.chromium.org/1643513002/diff/1/chrome/browser/resources/de...
File chrome/browser/resources/device_log_ui/device_log_ui.js (right):

https://codereview.chromium.org/1643513002/diff/1/chrome/browser/resources/de...
chrome/browser/resources/device_log_ui/device_log_ui.js:100: } catch(e) {}
This is fine as-is, but we might want to put something like "No log entries" in
log-container if you're up for it.

[fbeaufortchromium, 2016-01-28 09:06:21]

On 2016/01/27 18:13:08, stevenjb wrote:
> lgtm
> 
>
https://codereview.chromium.org/1643513002/diff/1/chrome/browser/resources/de...
> File chrome/browser/resources/device_log_ui/device_log_ui.js (right):
> 
>
https://codereview.chromium.org/1643513002/diff/1/chrome/browser/resources/de...
> chrome/browser/resources/device_log_ui/device_log_ui.js:100: } catch(e) {}
> This is fine as-is, but we might want to put something like "No log entries"
in
> log-container if you're up for it.

I've added "No log entries" in log-container.

[fbeaufortchromium, 2016-01-28 09:06:27]

The CQ bit was checked by fbeaufort@chromium.org

[fbeaufortchromium, 2016-01-28 09:06:28]

The patchset sent to the CQ was uploaded after l-g-t-m from
stevenjb@chromium.org
Link to the patchset: https://codereview.chromium.org/1643513002/#ps20001
(title: "Added No log entries to container")

[commit-bot: I haz the power, 2016-01-28 09:07:03]

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

[commit-bot: I haz the power, 2016-01-28 09:52:12]

Committed patchset #2 (id:20001)

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

Description was changed from

==========
Catch exception when creating device log entries
==========

to

==========
Catch exception when creating device log entries

Committed: https://crrev.com/2fe78b441548411fa343e5215fa54cf63212d739
Cr-Commit-Position: refs/heads/master@{#372060}
==========

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

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