Rename isDone() to hasFinishedLoading() for HTMLImport

Rename isDone() to haveFinishedLoading() for HTMLImport

This CL renames isDone() to haveFinishedLoading() for HTMLImport.

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=192902

[hyunjunekim2, 2015-03-28 13:38:35]

hyunjune.kim@samsung.com changed reviewers:
+ fs@opera.com, mkwst@chromium.org

[hyunjunekim2, 2015-03-28 13:39:21]

Hi, I'm hyunjunekim (hyunjune.kim@samsung.com, hykim0777@gamil.com). I fixed up.
Could you check this one? Thank you.

[hyunjunekim2, 2015-03-29 02:16:56]

hyunjune.kim@samsung.com changed reviewers:
+ kbr@chromium.org

[Mike West, 2015-03-30 04:55:33]

mkwst@chromium.org changed reviewers:
+ morrita@chromium.org

[Mike West, 2015-03-30 04:55:34]

I'm fine with renaming the method (though `haveFinishedLoading` sounds awkward;
I'd suggest something like `hasFinishedLoading`, though it's not clear to me
what the method does).

+morrita, who knows things about Imports and will have an opinion about the
renaming.

[hyunjunekim2, 2015-03-30 07:09:49]

On 2015/03/30 04:55:34, Mike West wrote:
> I'm fine with renaming the method (though `haveFinishedLoading` sounds
awkward;
> I'd suggest something like `hasFinishedLoading`, though it's not clear to me
> what the method does).
> 
> +morrita, who knows things about Imports and will have an opinion about the
> renaming.

I am going to change a name like `hasFinishedLoading`.

[hyunjunekim2, 2015-03-30 08:49:18]

> > I'd suggest something like `hasFinishedLoading`, though it's not clear to me
> > what the method does).

Mike West, I changed it to `hasFinishedLoading`. Thank you.

[Mike West, 2015-03-30 08:59:29]

mkwst@chromium.org changed required reviewers:
+ morrita@chromium.org

[Mike West, 2015-03-30 08:59:29]

LGTM, but I'll defer to morrita@, who has a deeper relationship with this code
than I do.

[hyunjunekim2, 2015-03-31 01:30:28]

Morrita, Hello, I'm hyunjunekim (hyunjune.kim@samsung.com, hykim0777@gamil.com).
Could you give me a review? Thank you.

[Hajime Morrita, 2015-03-31 17:10:13]

lgtm

[hyunjunekim2, 2015-04-01 01:58:28]

The CQ bit was checked by hyunjune.kim@samsung.com

[commit-bot: I haz the power, 2015-04-01 01:59:01]

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

[commit-bot: I haz the power, 2015-04-01 04:04:58]

Committed patchset #2 (id:20001) as
https://src.chromium.org/viewvc/blink?view=rev&revision=192902