Right clicking outside of a selected area should deselect text and bring up the correct context men…

pdf/pdfium/pdfium_engine.h

Index: pdf/pdfium/pdfium_engine.h
diff --git a/pdf/pdfium/pdfium_engine.h b/pdf/pdfium/pdfium_engine.h
index 11d304a8379bcdd9ed2fa89cb9216b053ac1d4f8..fe7269a6c119ca8409b04f07d7221513a25523f4 100644
--- a/pdf/pdfium/pdfium_engine.h
+++ b/pdf/pdfium/pdfium_engine.h
@@ -246,6 +246,10 @@ class PDFiumEngine : public PDFEngine,
   // PDFiumPage because we might not have that structure when we need this.
   pp::Size GetPageSize(int index);
 
+  void GetAllScreenRectsUnion(std::vector<PDFiumRange>& rect_range,

[gene, 2015/01/07 23:16:01]

Could you please make it pointer then (since GetScreenRects() is not const)?
non-const references are against Chrome style guide:
http://google-styleguide.googlecode.com/svn/trunk/cppguide.html#Reference_Arg...

[Deepak, 2015/01/08 08:44:57]

Done.

+                              const pp::Point offset_point,

[gene, 2015/01/07 23:16:01]

nit: const reference please

[Deepak, 2015/01/08 08:44:57]

Done.

+                              std::vector<pp::Rect>* rect_vector);
+
   void UpdateTickMarks();
 
   // Called to continue searching so we don't block the main thread.