remove support for InputMethodOptions_Open_* which hasn't been used for a while and is now removed.

remove support for InputMethodOptions_Open_* which hasn't been used for a while and is now removed.

R=asvitkine@chromium.org
BUG=287730
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=255885

[Junichi Uekawa, 2014-03-05 23:59:03]

Hi Alexei,

I've removed the logging code path that wasn't doing anything; and now I'd like
to remove the part from the python script. I believe it's a noop cleanup.

[Alexei Svitkine (slow), 2014-03-06 16:23:23]

Sorry for not getting back to this sooner.

We've recently expanded the infrastructure for actions (see
http://crrev.com/255357) and now there's an ability to mark actions as obsolete.
Could you also update the actions.xml file to mark these actions as obsolete?

[Junichi Uekawa, 2014-03-07 02:20:01]

On 2014/03/06 16:23:23, Alexei Svitkine wrote:
> Sorry for not getting back to this sooner.
> 
> We've recently expanded the infrastructure for actions (see
> http://crrev.com/255357) and now there's an ability to mark actions as
obsolete.
> Could you also update the actions.xml file to mark these actions as obsolete?

Hi,
Seems to me like the change has been rolled back and in the process of being
relanded.
https://codereview.chromium.org/188793003
https://codereview.chromium.org/180743027

[Junichi Uekawa, 2014-03-07 04:09:52]

On 2014/03/07 02:20:01, uekawa wrote:
> On 2014/03/06 16:23:23, Alexei Svitkine wrote:
> > Sorry for not getting back to this sooner.
> > 
> > We've recently expanded the infrastructure for actions (see
> > http://crrev.com/255357) and now there's an ability to mark actions as
> obsolete.
> > Could you also update the actions.xml file to mark these actions as
obsolete?
> 
> Hi,
> Seems to me like the change has been rolled back and in the process of being
> relanded.
> https://codereview.chromium.org/188793003
> https://codereview.chromium.org/180743027

I've created https://codereview.chromium.org/189873002/ that depends on the
change landing as a separate CL.

[Alexei Svitkine (slow), 2014-03-07 22:36:47]

On 2014/03/07 04:09:52, uekawa wrote:
> On 2014/03/07 02:20:01, uekawa wrote:
> > On 2014/03/06 16:23:23, Alexei Svitkine wrote:
> > > Sorry for not getting back to this sooner.
> > > 
> > > We've recently expanded the infrastructure for actions (see
> > > http://crrev.com/255357) and now there's an ability to mark actions as
> > obsolete.
> > > Could you also update the actions.xml file to mark these actions as
> obsolete?
> > 
> > Hi,
> > Seems to me like the change has been rolled back and in the process of being
> > relanded.
> > https://codereview.chromium.org/188793003
> > https://codereview.chromium.org/180743027
> 
> I've created https://codereview.chromium.org/189873002/ that depends on the
> change landing as a separate CL.

The other CL re-landed. I think it would be better to land the two CLs together
(i.e. combine them).

Sorry to keep you waiting, but wanted to make sure the entry can be marked as
obsolete with the new infrastructure.

[Junichi Uekawa, 2014-03-08 02:03:07]

On 2014/03/07 22:36:47, Alexei Svitkine wrote:
> On 2014/03/07 04:09:52, uekawa wrote:
> > On 2014/03/07 02:20:01, uekawa wrote:
> > > On 2014/03/06 16:23:23, Alexei Svitkine wrote:
> > > > Sorry for not getting back to this sooner.
> > > > 
> > > > We've recently expanded the infrastructure for actions (see
> > > > http://crrev.com/255357) and now there's an ability to mark actions as
> > > obsolete.
> > > > Could you also update the actions.xml file to mark these actions as
> > obsolete?
> > > 
> > > Hi,
> > > Seems to me like the change has been rolled back and in the process of
being
> > > relanded.
> > > https://codereview.chromium.org/188793003
> > > https://codereview.chromium.org/180743027
> > 
> > I've created https://codereview.chromium.org/189873002/ that depends on the
> > change landing as a separate CL.
> 
> The other CL re-landed. I think it would be better to land the two CLs
together
> (i.e. combine them).
> 
> Sorry to keep you waiting, but wanted to make sure the entry can be marked as
> obsolete with the new infrastructure.

ptal

[Alexei Svitkine (slow), 2014-03-08 03:05:47]

LGTM, thanks for your patience!

[Junichi Uekawa, 2014-03-08 05:50:59]

The CQ bit was checked by uekawa@chromium.org

[commit-bot: I haz the power, 2014-03-08 11:14:25]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/uekawa@chromium.org/179873004/20001

[commit-bot: I haz the power, 2014-03-08 12:41:04]

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

[commit-bot: I haz the power, 2014-03-08 12:41:05]

Retried try job too often on win_rel for step(s) base_unittests, browser_tests,
interactive_ui_tests, net_unittests, unit_tests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win_rel&nu...

[Junichi Uekawa, 2014-03-09 22:31:01]

The CQ bit was checked by uekawa@chromium.org

[commit-bot: I haz the power, 2014-03-09 22:31:20]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/uekawa@chromium.org/179873004/20001

[commit-bot: I haz the power, 2014-03-10 06:30:54]

Change committed as 255885