Switch cases that returns same value should use fall through.

Using Fall Through for switch case help in making code cleaner and easy to understandable.

BUG=543098
Committed: https://crrev.com/c40535a11b581c7cb6776aa06a51a4d9f03c7060
Cr-Commit-Position: refs/heads/master@{#354244}

[Deepak, 2015-10-14 12:41:53]

deepak.m1@samsung.com changed reviewers:
+ lazyboy@chromium.org

[Deepak, 2015-10-14 12:41:54]

PTAL

[lazyboy, 2015-10-14 15:54:11]

https://codereview.chromium.org/1406743002/diff/1/components/renderer_context...
File components/renderer_context_menu/context_menu_content_type.cc (right):

https://codereview.chromium.org/1406743002/diff/1/components/renderer_context...
components/renderer_context_menu/context_menu_content_type.cc:146: case
ITEM_GROUP_DEVELOPER:
I'd revert changes in this file as the ordering actually makes things readable.
The ordering in the switch is the same in the enum.

[Deepak, 2015-10-15 03:55:24]

PTAL
Thanks

https://codereview.chromium.org/1406743002/diff/1/components/renderer_context...
File components/renderer_context_menu/context_menu_content_type.cc (right):

https://codereview.chromium.org/1406743002/diff/1/components/renderer_context...
components/renderer_context_menu/context_menu_content_type.cc:146: case
ITEM_GROUP_DEVELOPER:
On 2015/10/14 15:54:11, lazyboy wrote:
> I'd revert changes in this file as the ordering actually makes things
readable.
> The ordering in the switch is the same in the enum.

Done.

[lazyboy, 2015-10-15 05:19:30]

renderer_context_menu/* LGTM

[Deepak, 2015-10-15 08:01:09]

deepak.m1@samsung.com changed reviewers:
+ bauerb@chromium.org

[Deepak, 2015-10-15 08:01:10]

@Bernhard

PTAL for
chrome/browser/android/preferences/pref_service_bridge.cc 

Thanks

[Bernhard Bauer, 2015-10-15 08:11:08]

LGTM, thanks!

[Deepak, 2015-10-15 08:36:29]

The CQ bit was checked by deepak.m1@samsung.com

[commit-bot: I haz the power, 2015-10-15 08:36:41]

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

[commit-bot: I haz the power, 2015-10-15 09:55:57]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-10-15 09:56:34]

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