Make XML compilation conditional.

tests/SkDOMTest.cpp

 /*
  * Copyright 2016 Google Inc.
  *
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 
 #include "SkDOM.h"
 #include "Test.h"
 
+#if defined(SK_XML)

[dogben, 2016/07/15 14:41:41]

Does this need to be before the includes?

[f(malita), 2016/07/15 14:46:30]

There's a presubmit check I tripped while trying to do that.  But SkDOM.h
doesn't seem to pull in any XML headers, so hopefully not required.

[mtklein, 2016/07/15 16:20:32]

The presubmit is there to guard against

#ifdef SK_BUILD_FOR_PLATFORM_DEFINED_IN_SKTYPES

#include "SkTypes.h"

#endif

If you want to quiet it down spiritually, you can

#include "SkTypes.h"

#if defined(SK_XML)

#include "SkDOM.h"
...

(That'd let SK_XML be defined in SkUserConfig.h, etc.)

[f(malita), 2016/07/15 16:47:55]

Done.

+
 DEF_TEST(SkDOM_test, r) {
     static const char gDoc[] =
         "<root a='1' b='2'>"
             "<elem1 c='3' />"
             "<elem2 d='4' />"
             "<elem3 e='5'>"
                 "<subelem1/>"
                 "<subelem2 f='6' g='7'/>"
             "</elem3>"
             "<elem4 h='8'/>"
         "</root>"
         ;
 
     SkDOM   dom;
     REPORTER_ASSERT(r, !dom.getRootNode());
 
     const SkDOM::Node* root = dom.build(gDoc, sizeof(gDoc) - 1);
     REPORTER_ASSERT(r, root && dom.getRootNode() == root);
 
     const char* v = dom.findAttr(root, "a");
     REPORTER_ASSERT(r, v && !strcmp(v, "1"));
     v = dom.findAttr(root, "b");
     REPORTER_ASSERT(r, v && !strcmp(v, "2"));
     v = dom.findAttr(root, "c");
     REPORTER_ASSERT(r, v == nullptr);
 
     REPORTER_ASSERT(r, dom.getFirstChild(root, "elem1"));
     REPORTER_ASSERT(r, !dom.getFirstChild(root, "subelem1"));
 }
+
+#endif // SK_XML