Removed sync error menu item from the wrench menu (material design user menu only)

Removed sync error menu item from the wrench menu (material design user menu only)
This is only for when #show-material-design-user-menu is enabled. For this case, sync error surfacing has been migrated to be in the user menu (https://codereview.chromium.org/2151513002/). Therefore, there shouldn't be a menu item in the wrench menu or any other surfacing UI for sync errors any more.

See project design doc: https://docs.google.com/document/d/17yHrKd_EU6WoIAgljnZmNCSJLRlbM3O2Pxr7aPEnqik/edit?ts=57445a70#heading=h.6xrfx2rucezz

BUG=615893
Committed: https://crrev.com/17508904c91aa277f580b613b36807097a67dea1
Cr-Commit-Position: refs/heads/master@{#406677}

[Jane, 2016-07-20 20:20:16]

janeliulwq@google.com changed reviewers:
+ zea@chromium.org

[Jane, 2016-07-20 20:20:16]

janeliulwq@google.com changed required reviewers:
+ zea@chromium.org

[Jane, 2016-07-20 20:20:17]

Hi Nicolas, PTAL, thanks! 
(I hope I'm doing this the right way!)

[Nicolas Zea, 2016-07-20 20:38:49]

FYI, it's good to have your commit message be limited to 80 character lines as
it makes it easier to read.  It's also common to have the commit title be the
first line, then an empty line, then the description of the commit itself.
Something like:

[material-design] Removed sync error menu item from the wrench menu

<more detailed description here>

LGTM otherwise.

[Jane, 2016-07-20 20:44:51]

Thanks for the quick reply and the FYI - good to learn that!

[Jane, 2016-07-20 20:45:03]

The CQ bit was checked by janeliulwq@google.com to run a CQ dry run

[commit-bot: I haz the power, 2016-07-20 20:45:34]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-07-20 21:33:00]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-07-20 21:33:00]

Dry run: This issue passed the CQ dry run.

[Jane, 2016-07-20 21:33:31]

The CQ bit was checked by janeliulwq@google.com

[commit-bot: I haz the power, 2016-07-20 21:34:13]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-07-20 21:40:11]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-07-20 21:42:43]

Description was changed from

==========
Removed sync error menu item from the wrench menu (material design user menu
only)
This is only for when #show-material-design-user-menu is enabled. For this case,
sync error surfacing has been migrated to be in the user menu
(https://codereview.chromium.org/2151513002/). Therefore, there shouldn't be a
menu item in the wrench menu or any other surfacing UI for sync errors any more.

See project design doc:
https://docs.google.com/document/d/17yHrKd_EU6WoIAgljnZmNCSJLRlbM3O2Pxr7aPEnq...

BUG=615893
==========

to

==========
Removed sync error menu item from the wrench menu (material design user menu
only)
This is only for when #show-material-design-user-menu is enabled. For this case,
sync error surfacing has been migrated to be in the user menu
(https://codereview.chromium.org/2151513002/). Therefore, there shouldn't be a
menu item in the wrench menu or any other surfacing UI for sync errors any more.

See project design doc:
https://docs.google.com/document/d/17yHrKd_EU6WoIAgljnZmNCSJLRlbM3O2Pxr7aPEnq...

BUG=615893

Committed: https://crrev.com/17508904c91aa277f580b613b36807097a67dea1
Cr-Commit-Position: refs/heads/master@{#406677}
==========

[commit-bot: I haz the power, 2016-07-20 21:42:43]

Patchset 1 (id:??) landed as
https://crrev.com/17508904c91aa277f580b613b36807097a67dea1
Cr-Commit-Position: refs/heads/master@{#406677}