Ignore import errors in cr recursive module loading

Ignore import errors in cr recursive module loading

Without this change, cr throws an ImportError whenever it tries to
import a folder with Python containing other folders (without Python
code).

BUG=

Committed: https://crrev.com/9bf5fea4e21eac6a98d4ae6fca5ee84ec352f416
Cr-Commit-Position: refs/heads/master@{#332650}

[petrcermak, 2015-06-03 11:01:25]

petrcermak@chromium.org changed reviewers:
+ skyostil@chromium.org

[petrcermak, 2015-06-03 11:01:26]

Hi, please review this patch.

Thanks,
Petr

[Sami, 2015-06-03 12:51:24]

The CL description could explain a bit why we need to do this.

https://codereview.chromium.org/1155193003/diff/1/tools/cr/cr/loader.py
File tools/cr/cr/loader.py (right):

https://codereview.chromium.org/1155193003/diff/1/tools/cr/cr/loader.py#newco...
tools/cr/cr/loader.py:81: module = _Import(name)
Could you add a _TryImport helper function to reduce the nesting here a bit?

[petrcermak, 2015-06-03 13:50:35]

Thanks for your comments. PTAL.

Petr

https://codereview.chromium.org/1155193003/diff/1/tools/cr/cr/loader.py
File tools/cr/cr/loader.py (right):

https://codereview.chromium.org/1155193003/diff/1/tools/cr/cr/loader.py#newco...
tools/cr/cr/loader.py:81: module = _Import(name)
On 2015/06/03 12:51:24, Sami wrote:
> Could you add a _TryImport helper function to reduce the nesting here a bit?

Done.

[Sami, 2015-06-03 14:30:56]

Thanks, lgtm.

[petrcermak, 2015-06-03 18:13:42]

The CQ bit was checked by petrcermak@chromium.org

[commit-bot: I haz the power, 2015-06-03 18:16:33]

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

[commit-bot: I haz the power, 2015-06-03 18:29:03]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-06-03 18:30:57]

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