{"push_id":1062963545,"size":1,"distinct_size":1,"ref":"refs/heads/master","head":"2dcc34c1e14d6847a6340da3d03fbe5e04801bfb","before":"94794f2da297bae6cbd5b45b1be9f912faac421d","commits":[{"sha":"2dcc34c1e14d6847a6340da3d03fbe5e04801bfb","author":{"email":"b544291d7f48f6f75eb9193791590ff82c680a4e@chromium.org","name":"chongz"},"message":"Add enum class |WebEditingCommandType| for EditorCommand\n\nMotivation: Using strings for actions is error prone (see\nhttp://crbug.com/251275) and ideally we want to replace\nstring names with enum (e.g. replace strings in\nhttps://goo.gl/VeUZFX). Also with enum we could eliminate\nthe huge |EditorCommand::CommandMap| with switch statements\nto improve readability.\n\nThis CL is the first step and adds |WebEditingCommandType| enum class in public/platform/ and\nprovides conversion between string command name and enum type in\nEditorCommand.cpp.\n\nAlso |EditorInternalCommand.idForUserMetrics| is replaced by\n|EditorInternalCommand.commandType| but remains the same enum value, so\nthe user matrics won't be affected.\n\nBUG=596064\n\nReview URL: https://codereview.chromium.org/1841143002\n\nCr-Commit-Position: refs/heads/master@{#386702}","distinct":true,"url":"https://api.github.com/repos/dstockwell/chromium/commits/2dcc34c1e14d6847a6340da3d03fbe5e04801bfb"}]}