Files.app: Suppress error about removing a non-existent menu item.

Files.app: Suppress error about removing a non-existent menu item.

As there's no API to know existence of certain menu item, we have to try
removing non-existent menu item.
Outputting error on console for such case is confusing, so this CL suppress the
error by reading runtime.lastError.

BUG=none
TEST=none

Committed: https://crrev.com/6ef076cdc5a9f66ce27e8c529f9b166b747344f9
Cr-Commit-Position: refs/heads/master@{#321288}

[fukino, 2015-03-18 11:47:46]

fukino@chromium.org changed reviewers:
+ hirono@chromium.org

[fukino, 2015-03-18 11:47:47]

Could you take a look?
This suppresses the usual error on background script.

[hirono, 2015-03-18 12:25:37]

lgtm!

https://codereview.chromium.org/1003303003/diff/1/ui/file_manager/file_manage...
File ui/file_manager/file_manager/background/js/background.js (right):

https://codereview.chromium.org/1003303003/diff/1/ui/file_manager/file_manage...
ui/file_manager/file_manager/background/js/background.js:622:
console.log(chrome.runtime.lastError.message);
nit: I think we even don't need to output the message.

[fukino, 2015-03-19 03:35:35]

Thank you!

https://codereview.chromium.org/1003303003/diff/1/ui/file_manager/file_manage...
File ui/file_manager/file_manager/background/js/background.js (right):

https://codereview.chromium.org/1003303003/diff/1/ui/file_manager/file_manage...
ui/file_manager/file_manager/background/js/background.js:622:
console.log(chrome.runtime.lastError.message);
On 2015/03/18 12:25:37, hirono wrote:
> nit: I think we even don't need to output the message.

Done.
Modified the code to just read and ignore it.

[fukino, 2015-03-19 03:35:48]

The CQ bit was checked by fukino@chromium.org

[fukino, 2015-03-19 03:35:49]

The patchset sent to the CQ was uploaded after l-g-t-m from hirono@chromium.org
Link to the patchset: https://codereview.chromium.org/1003303003/#ps20001
(title: "Do not output the log.")

[commit-bot: I haz the power, 2015-03-19 03:36:07]

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

[commit-bot: I haz the power, 2015-03-19 04:06:52]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-03-19 04:07:53]

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