Add a flag to change when android's progress bar completes

content/common/content_switches_internal.cc

Index: content/common/content_switches_internal.cc
diff --git a/content/common/content_switches_internal.cc b/content/common/content_switches_internal.cc
index 17cacedc66173b8aa6a8a512c0035bf603c986e0..e55101d5c0797451147768b99fbb19238934a35e 100644
--- a/content/common/content_switches_internal.cc
+++ b/content/common/content_switches_internal.cc
@@ -77,4 +77,20 @@ bool IsUseZoomForDSFEnabled() {
   return enabled;
 }
 
+ProgressBarCompletionOptions GetProgressBarCompletionOptions() {
+#if defined(OS_ANDROID)
+  const base::CommandLine& command_line =
+      *base::CommandLine::ForCurrentProcess();
+  std::string progress_bar_completion =
+      command_line.GetSwitchValueASCII(switches::kProgressBarCompletion);
+  if (progress_bar_completion == "loadEvent")
+    return PROGRESS_BAR_COMPLETION_LOAD_EVENT;
+  if (progress_bar_completion == "dclAndImages")

[dcheng, 2016/05/10 06:28:50]

My main concern here is that it's not immediately obvious that dcl ==
"DOMContentLoaded"

[Nate Chapin, 2016/05/11 00:06:58]

I expanded the flag to domContentLoadedAndImages, but left
IDS_FLAGS_PROGRESS_BAR_COMPLETION_DCL_AND_IMAGES, is that OK?

+    return PROGRESS_BAR_COMPLETION_DOM_CONTENT_LOADED_AND_IMAGES;
+  if (progress_bar_completion == "domContentLoaded")
+    return PROGRESS_BAR_COMPLETION_DOM_CONTENT_LOADED;
+#endif
+  return PROGRESS_BAR_COMPLETION_LOAD_EVENT;
+}
+
 } // namespace content