fix for builds with webrtc disabled

content/renderer/media/media_stream_video_source.h

Index: content/renderer/media/media_stream_video_source.h
diff --git a/content/renderer/media/media_stream_video_source.h b/content/renderer/media/media_stream_video_source.h
index ba66e439cdf9f1b42db9a96de2d8e4a76cef4b93..207c1e80748bc1b72c628f8179bee4797057ede4 100644
--- a/content/renderer/media/media_stream_video_source.h
+++ b/content/renderer/media/media_stream_video_source.h
@@ -78,10 +78,10 @@ class CONTENT_EXPORT MediaStreamVideoSource
 
   // Default resolution. If no constraints are specified and the delegate
   // support it, this is the resolution that will be used.
-  static const int kDefaultWidth;
-  static const int kDefaultHeight;
-  static const int kDefaultFrameRate;
-  static const int kUnknownFrameRate;
+  static const int kDefaultWidth = 640;

[DaleCurtis, 2015/03/04 21:26:04]

Because you don't want static values in a header file, each includer gets a new
copy:

http://stackoverflow.com/questions/92546/variable-declarations-in-header-file...

Which is why I suggested you make them enum values :)

[Mostyn Bramley-Moore, 2015/03/04 21:49:39]

I figured since it was only a few ints (as opposed to eg command line arg
strings) it wasn't so bad.  It builds for me locally but not on GOMA/the try
bots so the point is moot.  
Hmm, but don't you then need to cast to int every time you want to use these
enum values?  That would be annoying if the values are used often.

[DaleCurtis, 2015/03/04 21:54:23]

It depends on how they're used; in the majority of cases it'll be an implicit
conversion without issue.

[Mostyn Bramley-Moore, 2015/03/04 21:57:39]

OK.  I will try to eliminate the need for this later in the week as part of
crbug.com/463829.

+  static const int kDefaultHeight = 480;
+  static const int kDefaultFrameRate = 30;
+  static const int kUnknownFrameRate = 0;
 
  protected:
   void DoStopSource() override;