[chromedriver] Reenable ParseCapabilities.MobileEmulationDeviceName, since it no longer depends on …

[chromedriver] Reenable ParseCapabilities.MobileEmulationDeviceName, since it no longer depends on files generated from Blink sources.

BUG=392430
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=284004

[stgao, 2014-07-17 21:43:45]

I remember you wanted to remove the dependency on the device list.
Do you want to do that before enabling this test?

[samuong, 2014-07-17 21:50:01]

On 2014/07/17 21:43:45, Shuotao wrote:
> I remember you wanted to remove the dependency on the device list.
> Do you want to do that before enabling this test?

I wanted to remove the dependency on the Blink sources, which was done here:
https://src.chromium.org/viewvc/chrome?revision=283354&view=revision

We could refactor this so that it is possible to inject a mock device list, but
I don't think we'd gain much from doing that. Let me know what you think.

[stgao, 2014-07-17 21:57:04]

lgtm

But I would still suggest mocking out the list. If the device list is updated
again later, the test will break again.

[samuong, 2014-07-17 22:01:44]

On 2014/07/17 21:57:04, Shuotao wrote:
> lgtm
> 
> But I would still suggest mocking out the list. If the device list is updated
> again later, the test will break again.

OK, let's refactor this when we convert the mobile devices list from a
hard-coded JSON string into a proper data structure.

[samuong, 2014-07-17 22:01:46]

The CQ bit was checked by samuong@chromium.org

[commit-bot: I haz the power, 2014-07-17 22:05:04]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/samuong@chromium.org/399173003/1

[commit-bot: I haz the power, 2014-07-18 02:58:53]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  mac_chromium_compile_dbg on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/mac_chromium_compile_...)

[commit-bot: I haz the power, 2014-07-18 04:17:03]

Change committed as 284004