2009/9/10  <[email protected]>:
> On 2009/09/10 21:36:07, MikeSamuel wrote:
>>
>> http://codereview.appspot.com/115082/diff/5/6
>> File tests/com/google/caja/plugin/domita_test_untrusted.html (right):
>
>> http://codereview.appspot.com/115082/diff/5/6#newcode30
>> Line 30: <p class="clickme-testcontainer"
>
> id="test-add-event-listener">
>>
>> What is this clickme-testcontainer?
>> I see the code in DomitaTest which checks
>>       clickingList =
>
> driver.findElements(By.xpath("//*[contains(@class,'clickme')]/*"));
>
>> but there should be a space between clickme and testcontainer instead
>
> of a dash.
>
>> Hmm.  But not a problem for this CL.
>
> xpath 'contains' is a substring search, not a word search.
> afaik, xpath doesn't have a word search.

Yes.  I was complaining about the abuse of classes.  There's no need
to glom two classes into one unless there's some IE6 weirdness that
necessitates it.

> http://codereview.appspot.com/115082
>

Reply via email to