Remove ContentClient::CanHandleWhileSwappedOut().

Remove ContentClient::CanHandleWhileSwappedOut().

According to the issue 265021, ContentClient::CanHandleWhileSwappedOut() was
introduced to support synchronous browser->renderer messages for WebView.
But this is now obsolete, so we can remove it.

BUG=265021
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=268226

[Sungmann Cho, 2014-04-23 03:43:20]

Please take a look. Thanks.

[joth, 2014-04-23 04:02:28]

Nice. patch looks good to me. thanks!

You need to sign the CLA as per
http://dev.chromium.org/developers/contributing-code/external-contributor-che...


(+boliu@chromium.org could you take the review from here ?)

[Sungmann Cho, 2014-04-23 04:17:22]

Thank you Joth! And I've already signed the CLA.

[joth, 2014-04-23 04:50:54]

On 22 April 2014 21:17, <sungmann.cho@navercorp.com> wrote:

> Thank you Joth! And I've already signed the CLA.
>
> Looks like you might have signed it 3 times, but never with the @
navercorp.com email address?



> https://codereview.chromium.org/248703002/
>

To unsubscribe from this group and stop receiving emails from it, send an email
to chromium-reviews+unsubscribe@chromium.org.

[Sungmann Cho, 2014-04-23 05:15:56]

I don’t think so. I’ve signed the CLA with sungmann.cho@navercorp.com and it
exists in AUTHORS.
I had no problem to submit patches until now. Could you please check again?

[joth, 2014-04-23 05:23:02]

OK. I see a gmail address (twice) and @nhn.com -- but looks like nhn.com is
an alias for navercorp.



On 22 April 2014 22:15, <sungmann.cho@navercorp.com> wrote:

> I don't think so. I've signed the CLA with sungmann.cho@navercorp.com and
> it
> exists in AUTHORS.
> I had no problem to submit patches until now. Could you please check again?
>
> https://codereview.chromium.org/248703002/
>

To unsubscribe from this group and stop receiving emails from it, send an email
to chromium-reviews+unsubscribe@chromium.org.

[Sungmann Cho, 2014-04-23 05:28:44]

On 2014/04/23 05:23:02, joth (inactive) wrote:
> OK. I see a gmail address (twice) and @nhn.com -- but looks like
http://nhn.com is
> an alias for navercorp.
> 
> 
> 
> On 22 April 2014 22:15, <mailto:sungmann.cho@navercorp.com> wrote:
> 
> > I don't think so. I've signed the CLA with mailto:sungmann.cho@navercorp.com
and
> > it
> > exists in AUTHORS.
> > I had no problem to submit patches until now. Could you please check again?
> >
> > https://codereview.chromium.org/248703002/
> >
> 
> To unsubscribe from this group and stop receiving emails from it, send an
email
> to mailto:chromium-reviews+unsubscribe@chromium.org.

That's right but I've not used @nhn.com. :(
So what should I do?

[joth, 2014-04-23 16:42:18]

From
http://dev.chromium.org/developers/contributing-code/external-contributor-che...
says:
"If there's an entry for the right name, but a different email address, ask
the person to either change the AUTHORS CL or re-execute the CLA with the
proper email address"

i.e. the email addresses in AUTHORS and CLA should match. And to avoid
confusion, be the same address as you upload to
https://codereview.chromium.org/ with .

+tedchoc & dtrainer who reviewed your previous patches and may know more
(and may offer to take over the review from here ;-)
background: AUTHORS contains sungmann.cho@navercorp.com but the CLA only
has @gmail and @nhn.com addresses for Sugmann Cho




On 22 April 2014 22:28, <sungmann.cho@navercorp.com> wrote:

> On 2014/04/23 05:23:02, joth (inactive) wrote:
>
>> OK. I see a gmail address (twice) and @nhn.com -- but looks like
>>
> http://nhn.com is
>
>> an alias for navercorp.
>>
>
>
>
>  On 22 April 2014 22:15, <mailto:sungmann.cho@navercorp.com> wrote:
>>
>
>  > I don't think so. I've signed the CLA with mailto:
>> sungmann.cho@navercorp.com
>>
> and
>
>> > it
>> > exists in AUTHORS.
>> > I had no problem to submit patches until now. Could you please check
>> again?
>> >
>> > https://codereview.chromium.org/248703002/
>> >
>>
>
>  To unsubscribe from this group and stop receiving emails from it, send an
>>
> email
>
>> to mailto:chromium-reviews+unsubscribe@chromium.org.
>>
>
> That's right but I've not used @nhn.com. :(
> So what should I do?
>
> https://codereview.chromium.org/248703002/
>

To unsubscribe from this group and stop receiving emails from it, send an email
to chromium-reviews+unsubscribe@chromium.org.

[Sungmann Cho, 2014-04-23 17:45:48]

On 2014/04/23 16:42:18, joth (inactive) wrote:
> From
>
http://dev.chromium.org/developers/contributing-code/external-contributor-che...
> says:
> "If there's an entry for the right name, but a different email address, ask
> the person to either change the AUTHORS CL or re-execute the CLA with the
> proper email address"
> 
> i.e. the email addresses in AUTHORS and CLA should match. And to avoid
> confusion, be the same address as you upload to
> https://codereview.chromium.org/ with .
> 
> +tedchoc & dtrainer who reviewed your previous patches and may know more
> (and may offer to take over the review from here ;-)
> background: AUTHORS contains mailto:sungmann.cho@navercorp.com but the CLA
only
> has @gmail and @nhn.com addresses for Sugmann Cho
> 
> 
> 
> 
> On 22 April 2014 22:28, <mailto:sungmann.cho@navercorp.com> wrote:
> 
> > On 2014/04/23 05:23:02, joth (inactive) wrote:
> >
> >> OK. I see a gmail address (twice) and @nhn.com -- but looks like
> >>
> > http://nhn.com is
> >
> >> an alias for navercorp.
> >>
> >
> >
> >
> >  On 22 April 2014 22:15, <mailto:sungmann.cho@navercorp.com> wrote:
> >>
> >
> >  > I don't think so. I've signed the CLA with mailto:
> >> mailto:sungmann.cho@navercorp.com
> >>
> > and
> >
> >> > it
> >> > exists in AUTHORS.
> >> > I had no problem to submit patches until now. Could you please check
> >> again?
> >> >
> >> > https://codereview.chromium.org/248703002/
> >> >
> >>
> >
> >  To unsubscribe from this group and stop receiving emails from it, send an
> >>
> > email
> >
> >> to mailto:chromium-reviews+unsubscribe@chromium.org.
> >>
> >
> > That's right but I've not used @nhn.com. :(
> > So what should I do?
> >
> > https://codereview.chromium.org/248703002/
> >
> 
> To unsubscribe from this group and stop receiving emails from it, send an
email
> to mailto:chromium-reviews+unsubscribe@chromium.org.

I signed the CLA with sungmann.cho@navercorp.com now.
Thank you for pointing this out.

[joth, 2014-04-23 17:59:22]

lgtm

[Sungmann Cho, 2014-04-23 20:45:48]

The CQ bit was checked by sungmann.cho@navercorp.com

[commit-bot: I haz the power, 2014-04-23 20:46:22]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/sungmann.cho@navercorp.com/248703002/1

[boliu, 2014-04-23 20:47:53]

I was dropped from this thread (or never added?)

Looks like joth took care of everything already. But you still need to wait for
jam@ to approve this.

[Sungmann Cho, 2014-04-23 20:53:29]

The CQ bit was unchecked by sungmann.cho@navercorp.com

[Sungmann Cho, 2014-04-24 00:39:31]

On 2014/04/23 20:47:53, boliu wrote:
> I was dropped from this thread (or never added?)
> 
> Looks like joth took care of everything already. But you still need to wait
for
> jam@ to approve this.

Thank you for your comment. I'm waiting for jam@ to approve this now.

[Sungmann Cho, 2014-04-24 00:43:37]

jam@ please take a look.

[Sungmann Cho, 2014-04-26 01:41:12]

ping...

[Sungmann Cho, 2014-04-29 05:01:53]

ping...

[Sungmann Cho, 2014-05-01 20:53:46]

ping...

[boliu, 2014-05-01 20:55:20]

Maybe ask jam on irc?

[Sungmann Cho, 2014-05-01 21:39:33]

On 2014/05/01 20:55:20, boliu wrote:
> Maybe ask jam on irc?

OK. Thanks! :)

[jabdelmalek, 2014-05-02 23:38:46]

On 2014/05/01 21:39:33, Sungmann Cho wrote:
> On 2014/05/01 20:55:20, boliu wrote:
> > Maybe ask jam on irc?
> 
> OK. Thanks! :)

lgtm, apologies for missing the pings

[Sungmann Cho, 2014-05-03 00:45:04]

The CQ bit was checked by sungmann.cho@navercorp.com

[commit-bot: I haz the power, 2014-05-03 00:46:16]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/sungmann.cho@navercorp.com/248703002/1

[commit-bot: I haz the power, 2014-05-03 00:59:44]

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

[commit-bot: I haz the power, 2014-05-03 00:59:44]

Try jobs failed on following builders:
  chromium_presubmit on tryserver.chromium

[boliu, 2014-05-03 01:32:21]

jam@ You used your google.com account...maybe it's ok to tbr now

[jam, 2014-05-05 15:34:00]

sorry, lgtm from this account

[Sungmann Cho, 2014-05-05 18:07:42]

The CQ bit was checked by sungmann.cho@navercorp.com

[commit-bot: I haz the power, 2014-05-05 18:08:40]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/sungmann.cho@navercorp.com/248703002/1

[commit-bot: I haz the power, 2014-05-05 18:31:52]

Change committed as 268226