De-duplicate steps between chromium and chromium_trybot recipes

scripts/slave/recipe_modules/chromium/chromium_chromiumos.py

Index: scripts/slave/recipe_modules/chromium/chromium_chromiumos.py
diff --git a/scripts/slave/recipe_modules/chromium/chromium_chromiumos.py b/scripts/slave/recipe_modules/chromium/chromium_chromiumos.py
index 2771dccbea268894a5ff427b6872032daec38d0f..1ec0d769e9ef1c14f23a641da87ee4e7d4ef6abe 100644
--- a/scripts/slave/recipe_modules/chromium/chromium_chromiumos.py
+++ b/scripts/slave/recipe_modules/chromium/chromium_chromiumos.py
@@ -57,7 +57,6 @@ SPEC = {
             gs_acl='public-read',
         ),
         steps.Deps2GitTest(),
-        steps.Deps2SubmodulesTest(),

[agable, 2014/06/27 18:34:42]

Why is this removed in this CL? I mean, it should be removed, because submodules
are no longer supported, but maybe it should be a separate CL. Also, I don't see
any corresponding expectations updates.

[Paweł Hajdan Jr., 2014/06/27 18:58:22]

That's right, because nothing changes. Deps2Git from the tryserver runs
deps2submodules under the hood.

I'm happy to add a TODO or something to sort this out as a follow-up.

[agable, 2014/06/27 21:04:22]

That's terrifying that deps2git runs it, and that having this step then
*doesn't* run it because it's already been run. Or something like that.
Aaaaaaahh.

         steps.CheckpermsTest(),
       ],
       'testing': {