DevTools: fix inspector-protocol css tests

DevTools: fix inspector-protocol css tests

Currently, there are helper methods that constantly re-query document
node id. This turns out to be harmful as 'getDocument' command is
supposed to be invoked only once in the beginning of the session.

R=vsevik, apavlov

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

[lushnikov, 2014-08-07 17:06:39]

ptal

[apavlov, 2014-08-07 19:52:51]

lgtm

[dgozman, 2014-08-08 05:24:49]

> Currently, there are helper methods that constantly re-query document
> node id. This turns out to be harmful as 'getDocument' command is
> supposed to be invoked only once in the beginning of the session.

I'd say, this is a protocol implementation problem rather then client. I don't
see this restriction mentioned in protocol.json and it doesn't make sense at
all.

[vsevik, 2014-08-08 08:53:15]

On 2014/08/08 05:24:49, dgozman wrote:
> > Currently, there are helper methods that constantly re-query document
> > node id. This turns out to be harmful as 'getDocument' command is
> > supposed to be invoked only once in the beginning of the session.
> 
> I'd say, this is a protocol implementation problem rather then client. I don't
> see this restriction mentioned in protocol.json and it doesn't make sense at
> all.

This is definitely out of scope of this bug and I don't think we plan to change
any time soon.

[vsevik, 2014-08-08 08:53:19]

The CQ bit was checked by vsevik@chromium.org

[commit-bot: I haz the power, 2014-08-08 08:54:01]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/lushnikov@chromium.org/450533008/1

[dgozman, 2014-08-08 08:55:43]

> This is definitely out of scope of this bug and I don't think we plan to
change
> any time soon.

Looks like this patch doesn't have BUG at all.

[commit-bot: I haz the power, 2014-08-08 10:58:31]

Change committed as 179817